xfs
[Top] [All Lists]

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

To: Dave Chinner <david@xxxxxxxxxxxxx>
Subject: Re: [PATCH v2] xfstests 299-305: remove fio config files after finished test
From: Zheng Liu <gnehzuil.liu@xxxxxxxxx>
Date: Wed, 27 Mar 2013 19:55:02 +0800
Cc: xfs@xxxxxxxxxxx, linux-ext4@xxxxxxxxxxxxxxx, Zheng Liu <wenqing.lz@xxxxxxxxxx>, Eric Sandeen <sandeen@xxxxxxxxxxx>
Delivered-to: xfs@xxxxxxxxxxx
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=x-received:date:from:to:cc:subject:message-id:mail-followup-to :references:mime-version:content-type:content-disposition :in-reply-to:user-agent; bh=HDWBYhTCDV5N4SgNxBPBT4zNA0N/vav1EgdCndYjrjM=; b=V02fM3EyNFeNU8BZzTzRDHqYRW5lm77HN57TFQJAG2bN6zIFZYTMYepYguAxiSIVv4 FCRuXr7b9XBDOnJ7npSQIuTUrDn/xTmxBrpkBdm4mMVcqnurngfvASG38ELf5aGrzPRU q0xWL26JcKUIP3VR2HKtT4M2xSM1CN/6gNDuXFwYygzIRwOWp3Tcazi7zwImtCo0iKA2 LkSwZuWSWiqIuo0eh0JiUHTbw1nxDPSXAgGRpMeRuQQb9YGVlBdFu2Rwo5F1jVv3EUsP v4mtlty+5nXRom2iI5LHP83XBXr0XfZKZ1EojtT+yil+Js2UCdmIgcllyO5XG3cNjvgX nlFg==
In-reply-to: <20130327020532.GP6369@dastard>
Mail-followup-to: Dave Chinner <david@xxxxxxxxxxxxx>, xfs@xxxxxxxxxxx, linux-ext4@xxxxxxxxxxxxxxx, Zheng Liu <wenqing.lz@xxxxxxxxxx>, Eric Sandeen <sandeen@xxxxxxxxxxx>
References: <1364127266-13325-1-git-send-email-wenqing.lz@xxxxxxxxxx> <20130327020532.GP6369@dastard>
User-agent: Mutt/1.5.21 (2010-09-15)
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

<Prev in Thread] Current Thread [Next in Thread>