Re: [External] Re: [PATCH v13 02/12] mm: hugetlb: introduce a new config HUGETLB_PAGE_FREE_VMEMMAP

2021-01-26 Thread David Rientjes
On Tue, 26 Jan 2021, Muchun Song wrote: > > I'm not sure that Kconfig is the right place to document functional > > behavior of the kernel, especially for non-configurable options. Seems > > like this is already served by existing comments added by this patch > > series in the files where the des

Re: [External] Re: [PATCH v13 02/12] mm: hugetlb: introduce a new config HUGETLB_PAGE_FREE_VMEMMAP

2021-01-26 Thread Muchun Song
On Tue, Jan 26, 2021 at 2:47 AM David Rientjes wrote: > > On Mon, 25 Jan 2021, Muchun Song wrote: > > > > >> I'm not sure I understand the rationale for providing this help text if > > > >> this is def_bool depending on CONFIG_HUGETLB_PAGE. Are you intending > > > >> that > > > >> this is actual

Re: [PATCH v13 02/12] mm: hugetlb: introduce a new config HUGETLB_PAGE_FREE_VMEMMAP

2021-01-26 Thread Miaohe Lin
Hi: On 2021/1/17 23:10, Muchun Song wrote: > The HUGETLB_PAGE_FREE_VMEMMAP option is used to enable the freeing > of unnecessary vmemmap associated with HugeTLB pages. The config > option is introduced early so that supporting code can be written > to depend on the option. The initial version of th

Re: [External] Re: [PATCH v13 02/12] mm: hugetlb: introduce a new config HUGETLB_PAGE_FREE_VMEMMAP

2021-01-26 Thread David Rientjes
On Mon, 25 Jan 2021, Muchun Song wrote: > > >> I'm not sure I understand the rationale for providing this help text if > > >> this is def_bool depending on CONFIG_HUGETLB_PAGE. Are you intending > > >> that > > >> this is actually configurable and we want to provide guidance to the > > >> admin

Re: [External] Re: [PATCH v13 02/12] mm: hugetlb: introduce a new config HUGETLB_PAGE_FREE_VMEMMAP

2021-01-24 Thread Muchun Song
On Mon, Jan 25, 2021 at 12:09 PM Randy Dunlap wrote: > > On 1/24/21 8:06 PM, Muchun Song wrote: > > On Mon, Jan 25, 2021 at 7:58 AM David Rientjes wrote: > >> > >> > >> On Sun, 17 Jan 2021, Muchun Song wrote: > >> > >>> The HUGETLB_PAGE_FREE_VMEMMAP option is used to enable the freeing > >>> of u

Re: [External] Re: [PATCH v13 02/12] mm: hugetlb: introduce a new config HUGETLB_PAGE_FREE_VMEMMAP

2021-01-24 Thread Randy Dunlap
On 1/24/21 8:06 PM, Muchun Song wrote: > On Mon, Jan 25, 2021 at 7:58 AM David Rientjes wrote: >> >> >> On Sun, 17 Jan 2021, Muchun Song wrote: >> >>> The HUGETLB_PAGE_FREE_VMEMMAP option is used to enable the freeing >>> of unnecessary vmemmap associated with HugeTLB pages. The config >>> option

Re: [External] Re: [PATCH v13 02/12] mm: hugetlb: introduce a new config HUGETLB_PAGE_FREE_VMEMMAP

2021-01-24 Thread Muchun Song
On Mon, Jan 25, 2021 at 7:58 AM David Rientjes wrote: > > > On Sun, 17 Jan 2021, Muchun Song wrote: > > > The HUGETLB_PAGE_FREE_VMEMMAP option is used to enable the freeing > > of unnecessary vmemmap associated with HugeTLB pages. The config > > option is introduced early so that supporting code c

Re: [PATCH v13 02/12] mm: hugetlb: introduce a new config HUGETLB_PAGE_FREE_VMEMMAP

2021-01-24 Thread Randy Dunlap
On 1/24/21 3:58 PM, David Rientjes wrote: > On Sun, 17 Jan 2021, Muchun Song wrote: > >> The HUGETLB_PAGE_FREE_VMEMMAP option is used to enable the freeing >> of unnecessary vmemmap associated with HugeTLB pages. The config >> option is introduced early so that supporting code can be written >> to

Re: [PATCH v13 02/12] mm: hugetlb: introduce a new config HUGETLB_PAGE_FREE_VMEMMAP

2021-01-24 Thread David Rientjes
On Sun, 17 Jan 2021, Muchun Song wrote: > The HUGETLB_PAGE_FREE_VMEMMAP option is used to enable the freeing > of unnecessary vmemmap associated with HugeTLB pages. The config > option is introduced early so that supporting code can be written > to depend on the option. The initial version of the

[PATCH v13 02/12] mm: hugetlb: introduce a new config HUGETLB_PAGE_FREE_VMEMMAP

2021-01-17 Thread Muchun Song
The HUGETLB_PAGE_FREE_VMEMMAP option is used to enable the freeing of unnecessary vmemmap associated with HugeTLB pages. The config option is introduced early so that supporting code can be written to depend on the option. The initial version of the code only provides support for x86-64. Like othe