On Wed, Jun 08, 2011 at 11:16:42AM -0500, Andrew Hastings wrote: > On 06/07/11 11:08, Eric B Munson wrote: > > On Fri, 03 Jun 2011, Andrew Hastings wrote: [snip] > >> Might it make sense for HUGETLB_MORECORE=thp to automatically turn > >> on the madvise(MADV_HUGEPAGE)? If necessary, a separate env. var. > >> could disable the automatic madvise. > > > > It is possible to use THP without being madvise'd, infact if I read the > > transhuge documentation correctly, the madvise is only useful when > > system-wide > > transparent huge pages are not enabled (the sysctl is set to madvise). > > Sure, but if the user has gone to all the bother of setting > HUGETLB_MORECORE=thp, the principal of least surprise suggests that that > setting should work even if the sysctl is set to madvise. It doesn't > hurt to do the madvise if the sysctl has a different setting, does it? > > But if the current semantics are preferred, the documentation should at > least provide a explanation of why HUGETLB_MORECORE=thp alone might not > work as expected (i.e., the sysctl is set to madvise).
And indeed, if you're doing thp without madvise, what is the hugetlbfs morecore actually doing that default morecore wouldn't. -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson ------------------------------------------------------------------------------ EditLive Enterprise is the world's most technically advanced content authoring tool. Experience the power of Track Changes, Inline Image Editing and ensure content is compliant with Accessibility Checking. http://p.sf.net/sfu/ephox-dev2dev _______________________________________________ Libhugetlbfs-devel mailing list Libhugetlbfs-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/libhugetlbfs-devel