Re: [PATCH v2] xfstests 299-305: remove fio config files after finished test

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

 



On Wed, Mar 27, 2013 at 01:05:33PM +1100, Dave Chinner wrote:
> On Sun, Mar 24, 2013 at 08:14:26PM +0800, Zheng Liu wrote:
> > From: Zheng Liu <wenqing.lz@xxxxxxxxxx>
> > 
> > After finished test, temporarily fio config file should be removed.
> > Meanwhile in #305 poweron_scratch_dev is removed because no one uses
> > it.
> > 
> > Signed-off-by: Zheng Liu <wenqing.lz@xxxxxxxxxx>
> > Cc: Eric Sandeen <sandeen@xxxxxxxxxxx>
> > Cc: Dave Chinner <david@xxxxxxxxxxxxx>
> > ---
> >  299 |  9 +++++----
> >  300 |  9 +++++----
> >  301 |  9 +++++----
> >  302 |  9 +++++----
> >  303 |  9 +++++----
> >  304 |  9 +++++----
> >  305 | 11 ++++++-----
> >  7 files changed, 36 insertions(+), 29 deletions(-)
> >  mode change 100644 => 100755 299
> >  mode change 100644 => 100755 300
> >  mode change 100644 => 100755 301
> >  mode change 100644 => 100755 302
> >  mode change 100644 => 100755 303
> >  mode change 100644 => 100755 304
> >  mode change 100644 => 100755 305
> 
> Mode changes should be in a separate patch. And if you are going to
> do this, please change all the tests that need this to be done in
> one sweep.

Hi Dave,

Thanks for pointing it out.  I will send a latest version.  Meanwhile I
notice that now test cases have been moved into tests dir and splitted
into different parts.  So I will rebase my patch aginst latest xfstests.

Thanks,
                                                - Zheng
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Reiser Filesystem Development]     [Ceph FS]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite National Park]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Device Mapper]     [Linux Media]

  Powered by Linux