On Fri, Aug 8, 2014 at 7:02 PM, Sanjay Singh Rawat <sanjay.ra...@linaro.org> wrote: > > On Friday 08 August 2014 10:43 AM, Amit Kucheria wrote: >> >> On Fri, Aug 8, 2014 at 7:05 AM, Lisa Nguyen <lisa.ngu...@linaro.org> >> wrote: >>> >>> Create a global variable that will look for number of cpus >>> for a platform and use it in multiple functions to make the >>> code simplier to read. >> >> Agree with the patch. >> >>> Signed-off-by: Lisa Nguyen <lisa.ngu...@linaro.org> >>> --- >>> diff --git a/cpuhotplug/cpuhotplug_sanity.sh >>> b/cpuhotplug/cpuhotplug_sanity.sh >>> index d9b6af8..2ad24ff 100755 >>> --- a/cpuhotplug/cpuhotplug_sanity.sh >>> +++ b/cpuhotplug/cpuhotplug_sanity.sh >>> @@ -33,7 +33,6 @@ fi >>> >>> check_cpuhotplug_sysfs_entry() { >>> >>> - cpus=$(ls $CPU_PATH | grep "cpu[0-9].*") >>> cpunum=$(ls $CPU_PATH | grep "cpu[0-9].*" -c) >>> >>> if [ $cpunum -eq 1 ]; then >>> diff --git a/include/functions.sh b/include/functions.sh >>> index e0f08fb..74b2294 100644 >>> --- a/include/functions.sh >>> +++ b/include/functions.sh >>> @@ -30,6 +30,7 @@ TEST_NAME=$(basename ${0%.sh}) >>> PREFIX=$TEST_NAME >>> INC=0 >>> CPU= >>> +cpus=$(ls $CPU_PATH | grep "cpu[0-9].*") >> >> Let's fix this properly while we're at it. We already have boards with >> more than 10 cpus now - D01. > > > - Its fine, it will work. The sqare brackets filters the pattern, e.g. with > this filter
Right. I misread the regex. What's new. > cpu1 and also all cpu1*** will be filtered IN. > > - Issue with this filter here is it will return cpu is improper order, as > > cpu0 > cpu1 > cpu10 > cpu11 > cpu12 > cpu13 > cpu14 > cpu15 > cpu2 > cpu3 > cpu4 > > so here after cpu1, cpu10 will be processed instead of logically cpu2. We > can sort using *-v* > option of ls here to get cpus sequentially > > cpus=$(ls -1v $CPU_PATH | grep "cpu[0-9].*") > > >> >> Regards, >> Amit > > > -- > sanjay > _______________________________________________ linaro-dev mailing list linaro-dev@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-dev