You are right. Please ignore this change. Sorry about the mistake. Jiaying
On Tue, Jan 26, 2010 at 9:16 AM, Lucas Meneghel Rodrigues <[email protected]>wrote: > 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
