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.

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

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

Reply via email to