Re: [RFC][PATCH 1/2] hugetlb: introduce nr_overcommit_hugepages sysctl

[Nishanth Aravamudan <nacc@us.ibm.com>]

On 13.12.2007 [08:44:53 -0800], Nishanth Aravamudan wrote:
> On 13.12.2007 [08:17:08 -0800], Dave Hansen wrote:
> > On Wed, 2007-12-12 at 23:41 -0800, Nishanth Aravamudan wrote:
> > > While examining the code to support /proc/sys/vm/hugetlb_dynamic_pool, I
> > > became convinced that having a boolean sysctl was insufficient:
> > > 
> > > 1) To support per-node control of hugepages, I have previously submitted
> > > patches to add a sysfs attribute related to nr_hugepages. However, with
> > > a boolean global value and per-mount quota enforcement constraining the
> > > dynamic pool, adding corresponding control of the dynamic pool on a
> > > per-node basis seems inconsistent to me.
> > 
> > Documentation/sysctl, please :)
> 
> Err, yes, will need to updated that. I note that the old sysctl is not
> there...nor is nr_hugepages, for that matter. So maybe I'll just add a
> 3rd patch to fix the Documentation? I really just wanted to get the
> patches out there as soon as I got them tested...

Hrm, nr_hugepages is documented in vm/hugetlbpage.txt and not
sysctl/vm.txt Should I document this sysctl there too?

Thanks,
Nish

-- 
Nishanth Aravamudan <nacc@us.ibm.com>
IBM Linux Technology Center

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>