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

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

[Markos Chandras]

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

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

[Carmelo AMOROSO]

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