Re: [LTP] [PATCH] lib/mem.c: add HAVE_NUMA_H to avoid compiling error

[Markos Chandras <Markos.Chandras@imgtec.com>]

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