On 29.11.2006 [09:38:28 +1100], David Gibson wrote: > On Tue, Nov 28, 2006 at 02:18:05PM -0800, Nishanth Aravamudan wrote: > > On 28.11.2006 [13:51:28 -0800], Nishanth Aravamudan wrote: > > > On 28.11.2006 [10:41:13 +1100], David Gibson wrote: > > > > On Mon, Nov 27, 2006 at 12:21:05PM -0800, Nishanth Aravamudan wrote: > > <snip> > > > > Huh? Setting HUGETLB_PATH should work fine. It even won't have > > > > problems with spaces, as long as the double quotes are added around > > > > the call to hugetlbfs_path. > > > > > > Yes, I meant without any changes. And I think, in general, if > > > HUGETLB_PATH is set and it's something crazy (and yes, I consider "/ > > > hugetlb_dir/" to be crazy), then it's operator error. I think we > > > should do our best to avoid badness as a result, but it's still > > > operator error. > > > > Let me be clear here to make Dave feel better. I don't know that it's > > wrong to have spaces in paths on a box. (I think it's silly, yes). But > > if you're administering a box, and you want hugepage support, and you > > make your hugetlbfs mountpoint "/ hugetlb_dir/", I don't want to use > > your box. That's what I meant, perhaps it should have been phrased as > > "operator oddness", not error. > > Right. But 'rm -rf /' is a... rather disproportionate punishment for > operator oddness. > > Actually, it occurs to me that just this sort of thing is why I think > it's a big shame the testsuite now has to be run as root. I think it > would be a good idea to change it so it can run as an unprivileged > user again, maybe with some small run-as-root sections for configuring > the shm settings. It means the potential damage from screwups > (operator or library) is much smaller. > > I might have a go at this if I get a chance.
I am working on your idea of having a small test app right now, and will add this in to that sequence of changes and post what I come up with. Thanks, Nish -- Nishanth Aravamudan <[EMAIL PROTECTED]> IBM Linux Technology Center ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ Libhugetlbfs-devel mailing list Libhugetlbfs-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/libhugetlbfs-devel
