On 08/21/2013 04:34 AM, Dave Kleikamp wrote:
> gather_node_cpus() fails if the /sys/devices/system/node/nodeX/cpuX/online
> file is missing for any cpu other than cpu0. The absence of the online file
> should not be treated as a failure, but as an indication that the cpu is
> not hot-pluggable and cannot be taken offline.
> 
> Signed-off-by: Dave Kleikamp <dave.kleik...@oracle.com>

Reviewed-by: Wanlong Gao <gaowanl...@cn.fujitsu.com>


> 
> diff --git a/testcases/kernel/mem/lib/mem.c b/testcases/kernel/mem/lib/mem.c
> index 9bc926e..99243a6 100644
> --- a/testcases/kernel/mem/lib/mem.c
> +++ b/testcases/kernel/mem/lib/mem.c
> @@ -786,16 +786,14 @@ static void gather_node_cpus(char *cpus, long nd)
>               if (path_exist(path, nd, i)) {
>                       snprintf(path1, BUFSIZ, "%s/online", path);
>                       /*
> -                      * No cpu0/online knob, as it can't support to
> -                      * on/offline cpu0, so if the 'nd' node contains
> -                      * cpu0, it should skip to check cpu0/online's value.
> +                      * if there is no online knob, then the cpu cannot
> +                      * be taken offline
>                        */
> -                     if (i == 0)
> -                             goto next;
> -                     SAFE_FILE_SCANF(cleanup, path1, "%ld", &online);
> -                     if (online == 0)
> -                             continue;
> -next:
> +                     if (path_exist(path1)) {
> +                             SAFE_FILE_SCANF(cleanup, path1, "%ld", &online);
> +                             if (online == 0)
> +                                     continue;
> +                     }
>                       sprintf(buf, "%d,", i);
>                       strcat(cpus, buf);
>               }
> 
> ------------------------------------------------------------------------------
> Introducing Performance Central, a new site from SourceForge and 
> AppDynamics. Performance Central is your source for news, insights, 
> analysis and resources for efficient Application Performance Management. 
> Visit us today!
> http://pubads.g.doubleclick.net/gampad/clk?id=48897511&iu=/4140/ostg.clktrk
> _______________________________________________
> Ltp-list mailing list
> Ltp-list@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/ltp-list
> 


------------------------------------------------------------------------------
Introducing Performance Central, a new site from SourceForge and 
AppDynamics. Performance Central is your source for news, insights, 
analysis and resources for efficient Application Performance Management. 
Visit us today!
http://pubads.g.doubleclick.net/gampad/clk?id=48897511&iu=/4140/ostg.clktrk
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to