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

Reply via email to