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