On 03/02/2012 03:31 PM, Zhouping Liu wrote:
> HAS_NUMA_H should be HAVE_NUMA_H, and HAVE_NUMA_H is defined
> in config.h, so added the header.

Nice catch!

Just one tip:

config.h is suggested to be included at the top of file.

Thanks,
Caspar

> 
> Signed-off-by: Zhouping Liu <z...@redhat.com>
> ---
>  testcases/kernel/numa/numa_node_size.c |   10 ++++++----
>  1 files changed, 6 insertions(+), 4 deletions(-)
> 
> diff --git a/testcases/kernel/numa/numa_node_size.c 
> b/testcases/kernel/numa/numa_node_size.c
> index 3d71ff2..45bc2ac 100644
> --- a/testcases/kernel/numa/numa_node_size.c
> +++ b/testcases/kernel/numa/numa_node_size.c
> @@ -33,7 +33,9 @@
>  
>  #include <stdio.h>
>  #include <stdlib.h>
> -#if HAS_NUMA_H
> +#include "config.h"


> +
> +#if HAVE_NUMA_H
>  #include <numa.h>
>  #endif
>  
> @@ -48,7 +50,7 @@ char *fmt_mem(unsigned long long mem, char *buf)
>  }
>  void hardware(void)
>  {
> -#if HAS_NUMA_H
> +#if HAVE_NUMA_H
>          int i;
>          int maxnode = numa_max_node();
>          printf("available: %d nodes (0-%d)\n", 1+maxnode, maxnode);
> @@ -64,7 +66,7 @@ void hardware(void)
>  }
>  int main()
>  {
> -#if HAS_NUMA_H
> +#if HAVE_NUMA_H
>          nodemask_t nodemask;
>          void hardware();
>          if (numa_available() < 0)
> @@ -83,4 +85,4 @@ int main()
>          printf("NUMA is not available\n");
>          return 1;
>  #endif
> -}
> \ No newline at end of file
> +}


------------------------------------------------------------------------------
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

Reply via email to