* Re: [LTP] [PATCH] lib/mem.c: add HAVE_NUMA_H to avoid compiling error [not found] <cbed67486755eab955f21fcaf82f5351f14e6ead.1329458810.git.zliu@redhat.com> @ 2012-02-17 16:35 ` Markos Chandras 2012-02-17 16:45 ` Carmelo AMOROSO 0 siblings, 1 reply; 2+ messages in thread From: Markos Chandras @ 2012-02-17 16:35 UTC (permalink / raw) To: ltp-list On 02/17/2012 06:09 AM, Zhouping Liu wrote: > In the patch<9b70f0970>, I used numa_max_node(), which's included in <numa.h>, > but I forgot to add precompiled header: HAVE_NUMA_H, the patch fixed it. > > Signed-off-by: Zhouping Liu <zliu@redhat.com> > --- > testcases/kernel/mem/lib/mem.c | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/testcases/kernel/mem/lib/mem.c b/testcases/kernel/mem/lib/mem.c > index 25cdfd4..3843785 100644 > --- a/testcases/kernel/mem/lib/mem.c > +++ b/testcases/kernel/mem/lib/mem.c > @@ -148,6 +148,7 @@ void testoom(int mempolicy, int lite, int numa) > oom(KSM, mempolicy, lite); > } > > +#ifdef HAVE_NUMA_H > long count_numa(void) > { > int nnodes = 0; > @@ -161,6 +162,7 @@ long count_numa(void) > > return nnodes; > } > +#endif > > int path_exist(const char *path, ...) > { Yes, that works for me Tested-by: Markos Chandras <markos.chandras@imgtec.com> -- markos ------------------------------------------------------------------------------ Virtualization & Cloud Management Using Capacity Planning Cloud computing makes use of virtualization - but cloud computing also focuses on allowing computing to be delivered as a service. http://www.accelacomm.com/jaw/sfnl/114/51521223/ _______________________________________________ Ltp-list mailing list Ltp-list@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/ltp-list ^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [LTP] [PATCH] lib/mem.c: add HAVE_NUMA_H to avoid compiling error 2012-02-17 16:35 ` [LTP] [PATCH] lib/mem.c: add HAVE_NUMA_H to avoid compiling error Markos Chandras @ 2012-02-17 16:45 ` Carmelo AMOROSO 0 siblings, 0 replies; 2+ messages in thread From: Carmelo AMOROSO @ 2012-02-17 16:45 UTC (permalink / raw) To: ltp-list On 17/02/2012 17.35, Markos Chandras wrote: > On 02/17/2012 06:09 AM, Zhouping Liu wrote: > >> In the patch<9b70f0970>, I used numa_max_node(), which's included in <numa.h>, >> but I forgot to add precompiled header: HAVE_NUMA_H, the patch fixed it. >> >> Signed-off-by: Zhouping Liu <zliu@redhat.com> >> --- >> testcases/kernel/mem/lib/mem.c | 2 ++ >> 1 files changed, 2 insertions(+), 0 deletions(-) >> >> diff --git a/testcases/kernel/mem/lib/mem.c b/testcases/kernel/mem/lib/mem.c >> index 25cdfd4..3843785 100644 >> --- a/testcases/kernel/mem/lib/mem.c >> +++ b/testcases/kernel/mem/lib/mem.c >> @@ -148,6 +148,7 @@ void testoom(int mempolicy, int lite, int numa) >> oom(KSM, mempolicy, lite); >> } >> >> +#ifdef HAVE_NUMA_H >> long count_numa(void) >> { >> int nnodes = 0; >> @@ -161,6 +162,7 @@ long count_numa(void) >> >> return nnodes; >> } >> +#endif >> >> int path_exist(const char *path, ...) >> { > > > Yes, that works for me > > Tested-by: Markos Chandras <markos.chandras@imgtec.com> > confirmed, works for me too carmelo ------------------------------------------------------------------------------ Virtualization & Cloud Management Using Capacity Planning Cloud computing makes use of virtualization - but cloud computing also focuses on allowing computing to be delivered as a service. http://www.accelacomm.com/jaw/sfnl/114/51521223/ _______________________________________________ Ltp-list mailing list Ltp-list@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/ltp-list ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2012-02-17 16:45 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <cbed67486755eab955f21fcaf82f5351f14e6ead.1329458810.git.zliu@redhat.com>
2012-02-17 16:35 ` [LTP] [PATCH] lib/mem.c: add HAVE_NUMA_H to avoid compiling error Markos Chandras
2012-02-17 16:45 ` Carmelo AMOROSO
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.