Re: [PATCH] mm: thp: remove use_zero_page sysfs knob
From: David Rientjes
Date: Mon Jul 23 2018 - 19:14:07 EST
On Mon, 23 Jul 2018, Yang Shi wrote:
> > > I agree to keep it for a while to let that security bug cool down,
> > > however, if
> > > there is no user anymore, it sounds pointless to still keep a dead knob.
> > >
> > It's not a dead knob. We use it, and for reasons other than
> > CVE-2017-1000405. To mitigate the cost of constantly compacting memory to
> > allocate it after it has been freed due to memry pressure, we can either
> > continue to disable it, allow it to be persistently available, or use a
> > new value for use_zero_page to specify it should be persistently
> > available.
>
> My understanding is the cost of memory compaction is *not* unique for huge
> zero page, right? It is expected when memory pressure is met, even though huge
> zero page is disabled.
>
It's caused by fragmentation, not necessarily memory pressure. We've
disabled it because compacting for tens of thousands of huge zero pages in
the background has a noticeable impact on cpu. Additionally, if the hzp
cannot be allocated at runtime it increases the rss of applications that
map it, making it unpredictable. Making it persistent, as I've been
suggesting, fixes these issues.