On Fri, Oct 17, 2008 at 03:11:01PM -0500, Adam Litke wrote:
> On Fri, 2008-10-17 at 17:14 +0100, Andy Whitcroft wrote:
> > diff --git a/tests/Makefile b/tests/Makefile
> > index e4e1ce2..f03ec68 100644
> > --- a/tests/Makefile
> > +++ b/tests/Makefile
> > @@ -22,7 +22,7 @@ BADTOOLCHAIN = bad-toolchain.sh
> > 
> >  CFLAGS = -O2 -Wall -g
> >  CPPFLAGS = -I..
> > -LDLIBS = -ldl -lpthread
> > +LDLIBS = -ldl -lpthread -lhugetlbfs_privutils
> >  LDFLAGS32 = -L../obj32
> >  LDFLAGS64 = -L../obj64
> >  INSTALL = install
> > diff --git a/tests/counters.c b/tests/counters.c
> > index 522a00d..0284809 100644
> > --- a/tests/counters.c
> > +++ b/tests/counters.c
> > @@ -181,6 +181,7 @@ void _set_nr_hugepages(unsigned long count, int line)
> >  out:
> >     verify_counters(line, et, ef, er, es);
> >  }
> > +#undef set_nr_hugepages
> 
> Why is this needed?
> 
> >  #define set_nr_hugepages(c) _set_nr_hugepages(c, __LINE__)

Because this test is overriding set_nr_hugepages in itself to also
track the count and verify the overall expected counter changes.
set_nr_hugepages is already a define in this new scheme so we need to
undef it to replace it.

-apw

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Libhugetlbfs-devel mailing list
Libhugetlbfs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/libhugetlbfs-devel

Reply via email to