All the mail mirrored from lore.kernel.org
 help / color / mirror / Atom feed
* [LTP] [PATCH 1/2] testscripts: make redirects happen in correct order
@ 2015-09-21  9:33 Wei Jiangang
  2015-09-21  9:33 ` [LTP] [PATCH 2/2] cpuhotplug_hotplug.sh: do/done should be left-aligned Wei Jiangang
  0 siblings, 1 reply; 4+ messages in thread
From: Wei Jiangang @ 2015-09-21  9:33 UTC (permalink / raw
  To: ltp

It should redirect stdout to /dev/null firstly,
then redirect stderr to whatever stdout currently points at.

Signed-off-by: Wei Jiangang <weijg.fnst@cn.fujitsu.com>
---
 testscripts/ltpstress.sh   |  4 ++--
 testscripts/runEALtests.sh | 10 +++++-----
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/testscripts/ltpstress.sh b/testscripts/ltpstress.sh
index 39a5caf..1a16779 100755
--- a/testscripts/ltpstress.sh
+++ b/testscripts/ltpstress.sh
@@ -265,10 +265,10 @@ fi
 ulimit -u unlimited
 
 if [ $PROC_NUM -gt 0 ];then
-  genload --vm $PROC_NUM --vm-bytes 1073741824 2>&1 1>/dev/null &
+  genload --vm $PROC_NUM --vm-bytes 1073741824 1>/dev/null 2>&1 &
 fi
 if [ $leftover_memsize -gt 0 ];then
-  genload --vm 1 --vm-bytes $(($leftover_memsize * 1024)) 2>&1 1>/dev/null &
+  genload --vm 1 --vm-bytes $(($leftover_memsize * 1024)) 1>/dev/null 2>&1 &
 fi
 
 if [ $NO_NETWORK -eq 0 ];then
diff --git a/testscripts/runEALtests.sh b/testscripts/runEALtests.sh
index bcd3f29..577be28 100755
--- a/testscripts/runEALtests.sh
+++ b/testscripts/runEALtests.sh
@@ -84,7 +84,7 @@ fi
 while getopts cd:f:hi:l:m:Nnpqr:t:x arg
 do  case $arg in
     c)
-            $LTPROOT/../testcases/bin/genload --cpu 1 2>&1 1>/dev/null &
+            $LTPROOT/../testcases/bin/genload --cpu 1 1>/dev/null 2>&1 &
 	    GenLoad=1 ;;
 
     d)      # append $$ to TMP, as it is recursively
@@ -97,9 +97,9 @@ do  case $arg in
 
     i)
             bytesize=$(($OPTARG * 1024 * 1024))
-            $LTPROOT/../testcases/bin/genload --io 1 2>&1 1>/dev/null &
+            $LTPROOT/../testcases/bin/genload --io 1 1>/dev/null 2>&1 &
             $LTPROOT/../testcases/bin/genload --hdd 0 --hdd-bytes $bytesize \
-            2>&1 1>/dev/null &
+            1>/dev/null 2>&1 &
 	    GenLoad=1 ;;
 
     l)
@@ -125,7 +125,7 @@ do  case $arg in
     m)
             memsize=$(($OPTARG * 1024 * 1024))
 	    $LTPROOT/../testcases/bin/genload  --vm 0 --vm-bytes $memsize\
-            2>&1 1>/dev/null &
+            1>/dev/null 2>&1 &
 	    GenLoad=1;;
 
     N)	    run_netest=1;;
@@ -217,7 +217,7 @@ fi
 
 if [ $GenLoad -eq 1 ]
 then
-	killall -9 genload
+	killall -9 genload >/dev/null 2>&1
 fi
 
 if [ $NetPipe -eq 1 ]
-- 
1.9.3


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

* [LTP] [PATCH 2/2] cpuhotplug_hotplug.sh: do/done should be left-aligned
  2015-09-21  9:33 [LTP] [PATCH 1/2] testscripts: make redirects happen in correct order Wei Jiangang
@ 2015-09-21  9:33 ` Wei Jiangang
  0 siblings, 0 replies; 4+ messages in thread
From: Wei Jiangang @ 2015-09-21  9:33 UTC (permalink / raw
  To: ltp

Signed-off-by: Wei Jiangang <weijg.fnst@cn.fujitsu.com>
---
 testcases/kernel/hotplug/cpu_hotplug/include/cpuhotplug_hotplug.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/testcases/kernel/hotplug/cpu_hotplug/include/cpuhotplug_hotplug.sh b/testcases/kernel/hotplug/cpu_hotplug/include/cpuhotplug_hotplug.sh
index d9cf965..ad8675a 100644
--- a/testcases/kernel/hotplug/cpu_hotplug/include/cpuhotplug_hotplug.sh
+++ b/testcases/kernel/hotplug/cpu_hotplug/include/cpuhotplug_hotplug.sh
@@ -28,7 +28,7 @@ migrate_irq()
     MASK=$((1<<${CPU}))
     IRQS=$2
     for irq in ${IRQS}
-      do
+    do
       echo $MASK > /proc/irq/${irq}/smp_affinity || \
         tst_resm TINFO "It is NOT permitted to change the IRQ $irq smp_affinity"
     done
-- 
1.9.3


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

* [LTP] [PATCH 2/2] cpuhotplug_hotplug.sh: do/done should be left-aligned
  2015-09-30  3:25 [LTP] [PATCH 1/2] testscripts: make redirects happen in correct order Wei Jiangang
@ 2015-09-30  3:25 ` Wei Jiangang
  2015-09-30 12:18   ` Cyril Hrubis
  0 siblings, 1 reply; 4+ messages in thread
From: Wei Jiangang @ 2015-09-30  3:25 UTC (permalink / raw
  To: ltp

Signed-off-by: Wei Jiangang <weijg.fnst@cn.fujitsu.com>
---
 testcases/kernel/hotplug/cpu_hotplug/include/cpuhotplug_hotplug.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/testcases/kernel/hotplug/cpu_hotplug/include/cpuhotplug_hotplug.sh b/testcases/kernel/hotplug/cpu_hotplug/include/cpuhotplug_hotplug.sh
index d9cf965..ad8675a 100644
--- a/testcases/kernel/hotplug/cpu_hotplug/include/cpuhotplug_hotplug.sh
+++ b/testcases/kernel/hotplug/cpu_hotplug/include/cpuhotplug_hotplug.sh
@@ -28,7 +28,7 @@ migrate_irq()
     MASK=$((1<<${CPU}))
     IRQS=$2
     for irq in ${IRQS}
-      do
+    do
       echo $MASK > /proc/irq/${irq}/smp_affinity || \
         tst_resm TINFO "It is NOT permitted to change the IRQ $irq smp_affinity"
     done
-- 
1.9.3


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

* [LTP] [PATCH 2/2] cpuhotplug_hotplug.sh: do/done should be left-aligned
  2015-09-30  3:25 ` [LTP] [PATCH 2/2] cpuhotplug_hotplug.sh: do/done should be left-aligned Wei Jiangang
@ 2015-09-30 12:18   ` Cyril Hrubis
  0 siblings, 0 replies; 4+ messages in thread
From: Cyril Hrubis @ 2015-09-30 12:18 UTC (permalink / raw
  To: ltp

Hi!
> diff --git a/testcases/kernel/hotplug/cpu_hotplug/include/cpuhotplug_hotplug.sh b/testcases/kernel/hotplug/cpu_hotplug/include/cpuhotplug_hotplug.sh
> index d9cf965..ad8675a 100644
> --- a/testcases/kernel/hotplug/cpu_hotplug/include/cpuhotplug_hotplug.sh
> +++ b/testcases/kernel/hotplug/cpu_hotplug/include/cpuhotplug_hotplug.sh
> @@ -28,7 +28,7 @@ migrate_irq()
>      MASK=$((1<<${CPU}))
>      IRQS=$2
>      for irq in ${IRQS}
> -      do
> +    do

The rest of the file has for cycles with semicolon and do on the same
line. Let's be consistent and change this one to the same style as well.

-- 
Cyril Hrubis
chrubis@suse.cz

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

end of thread, other threads:[~2015-09-30 12:18 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-09-21  9:33 [LTP] [PATCH 1/2] testscripts: make redirects happen in correct order Wei Jiangang
2015-09-21  9:33 ` [LTP] [PATCH 2/2] cpuhotplug_hotplug.sh: do/done should be left-aligned Wei Jiangang
  -- strict thread matches above, loose matches on Subject: below --
2015-09-30  3:25 [LTP] [PATCH 1/2] testscripts: make redirects happen in correct order Wei Jiangang
2015-09-30  3:25 ` [LTP] [PATCH 2/2] cpuhotplug_hotplug.sh: do/done should be left-aligned Wei Jiangang
2015-09-30 12:18   ` Cyril Hrubis

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.