Re: [Libhugetlbfs-devel] [PATCH] libhugetlbfs: consolidate to one constructor

2008-03-03 Thread Adam Litke
On Wed, 2008-02-27 at 13:29 -0800, Nishanth Aravamudan wrote: > On 22.02.2008 [14:20:13 -0600], Andrew Hastings wrote: > > Nish, > > > > Thanks for the review! > > > > > Glad you suggested this! It turns out there's a dependency on the order in > > which the three libhugetlbfs constructors ru

Re: [Libhugetlbfs-devel] [PATCH] libhugetlbfs: consolidate to one constructor

2008-02-27 Thread Andrew Hastings
Nish, Thanks! You beat me to it! Nishanth Aravamudan wrote: > --- /dev/null > +++ b/init.c > @@ -0,0 +1,8 @@ > +#include "libhugetlbfs_internal.h" > + > +static void __attribute__ ((constructor)) setup_libhugetlbfs(void) > +{ > + setup_morecore(); > + setup_elflink(); > + setup_debug

[Libhugetlbfs-devel] [PATCH] libhugetlbfs: consolidate to one constructor

2008-02-27 Thread Nishanth Aravamudan
On 22.02.2008 [14:20:13 -0600], Andrew Hastings wrote: > Nish, > > Thanks for the review! > Glad you suggested this! It turns out there's a dependency on the order in > which the three libhugetlbfs constructors run, which in turns depends on > the order the objects appear in the link line. I