Re: [External] Re: [PATCH v5 03/21] mm/hugetlb: Introduce a new config HUGETLB_PAGE_FREE_VMEMMAP
From: Michal Hocko
Date: Fri Nov 20 2020 - 03:48:14 EST
On Fri 20-11-20 16:35:16, Muchun Song wrote:
[...]
> > That being said, unless there are huge advantages to introduce a
> > config option I would rather not add it because our config space is huge
> > already and the more we add the more future code maintainance that will
> > add. If you want the config just for dependency checks then fine by me.
>
> Yeah, it is only for dependency checks :)
OK, I must have misread the definition to think that it requires user to
enable explicitly.
Anyway this feature cannot be really on by default due to overhead. So
the command line option default has to be flipped.
--
Michal Hocko
SUSE Labs