public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
* [LTP] [PATCH ltp] cpuhotplug_hotplug.sh: 'part' here is a variable
@ 2016-12-01  6:57 Han, NingX
  2016-12-01 12:33 ` Cyril Hrubis
  0 siblings, 1 reply; 2+ messages in thread
From: Han, NingX @ 2016-12-01  6:57 UTC (permalink / raw)
  To: ltp

Here 'part' is not a command or a function, just a variable, should not  add ()

Signed-off-by: Ning Han <ningx.han@intel.com<mailto:ningx.han@intel.com>>
---
testcases/kernel/hotplug/cpu_hotplug/include/cpuhotplug_hotplug.sh | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/testcases/kernel/hotplug/cpu_hotplug/include/cpuhotplug_hotplug.sh b/testcases/kernel/hotplug/cpu_hotplug/include/cpuhotplug_hotplug.sh
index 91d2a19..9f856e5 100644
--- a/testcases/kernel/hotplug/cpu_hotplug/include/cpuhotplug_hotplug.sh
+++ b/testcases/kernel/hotplug/cpu_hotplug/include/cpuhotplug_hotplug.sh
@@ -150,8 +150,8 @@ get_present_cpus()
             range_low=$(echo $part | cut -d - -f 1)
             range_high=$(echo $part | cut -d - -f 2)
         else
-            range_low=$(part)
-            range_high=$(part)
+            range_low=$part
+            range_high=$part
         fi
         for cpu in $(seq $range_low $range_high); do
             if [ -e /sys/devices/system/cpu/cpu$cpu ]; then
--
2.7.4

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.linux.it/pipermail/ltp/attachments/20161201/177382be/attachment-0001.html>

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [LTP] [PATCH ltp] cpuhotplug_hotplug.sh: 'part' here is a variable
  2016-12-01  6:57 [LTP] [PATCH ltp] cpuhotplug_hotplug.sh: 'part' here is a variable Han, NingX
@ 2016-12-01 12:33 ` Cyril Hrubis
  0 siblings, 0 replies; 2+ messages in thread
From: Cyril Hrubis @ 2016-12-01 12:33 UTC (permalink / raw)
  To: ltp

Hi!
> Signed-off-by: Ning Han <ningx.han@intel.com<mailto:ningx.han@intel.com>>
                                                  ^
						  I've removed the
						  mailto: part here

And pushed, thanks.

-- 
Cyril Hrubis
chrubis@suse.cz

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-12-01 12:33 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-12-01  6:57 [LTP] [PATCH ltp] cpuhotplug_hotplug.sh: 'part' here is a variable Han, NingX
2016-12-01 12:33 ` Cyril Hrubis

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox