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