A couple of questions about the mlock testcase added in commit
ff2f5343e4a4e2d2e62173d557a7e5952cfa7733.

First, is there a rationale for mlock()ing all the available hugepages
(as passed in on the command line).  I don't see an obvious reason
that that's a useful thing to test (as opposed to just mlock()ing a
sample page), and it makes the testcase run quite slowly.  I'd like to
keep the "functional" (rather than "stress") portion of the testsuite
nice and fast, making for quicker cycle time when tracking down
regressions.

Second, the testcase maps the file both normally and with MAP_LOCKED,
which ought to lock the file as with mlock() at mmap() time.  It also
explicitly mlock()s the file, but only when mapped with MAP_LOCKED
which seems backwards.  Surely we should be checking mlock() when the
file was not originally locked.  Is this really the intended
behaviour.

-- 
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

-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Libhugetlbfs-devel mailing list
Libhugetlbfs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/libhugetlbfs-devel

Reply via email to