On Mon, 2010-01-25 at 11:30 -0800, Jiaying Zhang wrote:
> Use 'dir' instead of 'testdir' prameter in fsstress test so we can run it on 
> any partition.
> 
> TESTED:
> verified that it works through autotest cli.
> 
> Signed-off-by: Jiaying Zhang <[email protected]>

Hi Jiaying, I read your patch but didn't understand how the name of the
test dir variable was affecting one's ability to run fsstress on any
partition. Could you explain it to me?

> --- autotest/client/tests/fsstress/fsstress.py        2010-01-25 
> 11:11:50.000000000 -0800
> +++ autotest/client/tests/fsstress/fsstress.py        2010-01-25 
> 11:11:50.000000000 -0800
> @@ -19,10 +19,10 @@
>          utils.system('make fsstress')
>  
> 
> -    def run_once(self, testdir = None, extra_args = '', nproc = '1000', nops 
> = '1000'):
> -        if not testdir:
> -            testdir = self.tmpdir
> +    def run_once(self, dir = None, extra_args = '', nproc = '1000', nops = 
> '1000'):
> +        if not dir:
> +            dir = self.tmpdir
>  
> -        args = '-d %s -p %s -n %s %s' % (testdir, nproc, nops, extra_args)
> +        args = '-d %s -p %s -n %s %s' % (dir, nproc, nops, extra_args)
>          cmd = self.srcdir + '/fsstress ' + args
>          utils.system(cmd)
> _______________________________________________
> Autotest mailing list
> [email protected]
> http://test.kernel.org/cgi-bin/mailman/listinfo/autotest


_______________________________________________
Autotest mailing list
[email protected]
http://test.kernel.org/cgi-bin/mailman/listinfo/autotest

Reply via email to