[rteval PATCH] rteval: stressng.py: Fix arguments for Popen

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Make each element of self.args contains only one argument.
This makes '--stressng-timeout' option work as expected,
and makes '--stressng-arg' usable for multiple options:
i.e. '--stressng-arg="8 --memcpy-method=libc"'

Signed-off-by: Atsushi Nemoto <atsushi.nemoto@xxxxxxxxxx>
---
change from previous version: split out fix for --taskset, update commit log.
---
 rteval/modules/loads/stressng.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/rteval/modules/loads/stressng.py b/rteval/modules/loads/stressng.py
index f677956..3edfed7 100644
--- a/rteval/modules/loads/stressng.py
+++ b/rteval/modules/loads/stressng.py
@@ -50,9 +50,9 @@ class Stressng(CommandLineLoad):
         self.args = ['stress-ng']
         self.args.append('--%s' % str(self.cfg.option))
         if self.cfg.arg is not None:
-            self.args.append(self.cfg.arg)
+            self.args.extend(self.cfg.arg.split())
         if self.cfg.timeout is not None:
-            self.args.append('--timeout %s' % str(self.cfg.timeout))
+            self.args.extend(['--timeout', str(self.cfg.timeout)])
 
         systop = SysTopology()
         # get the number of nodes
-- 
2.11.0




[Index of Archives]     [RT Stable]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]

  Powered by Linux