All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eryu Guan <guaneryu@gmail.com>
To: "Darrick J. Wong" <darrick.wong@oracle.com>
Cc: Eryu Guan <guaneryu@aliyun.com>,
	linux-xfs@vger.kernel.org, fstests@vger.kernel.org
Subject: Re: [PATCH 1/1] xfs: test xfs_scrub phase 6 media error reporting
Date: Wed, 4 Mar 2020 14:13:20 +0800	[thread overview]
Message-ID: <20200304050737.GC3128153@desktop> (raw)
In-Reply-To: <20200304035057.GE1752567@magnolia>

[-- Attachment #1: Type: text/plain, Size: 4588 bytes --]

On Tue, Mar 03, 2020 at 07:50:57PM -0800, Darrick J. Wong wrote:
> On Wed, Mar 04, 2020 at 10:51:32AM +0800, Eryu Guan wrote:
> > On Tue, Mar 03, 2020 at 10:06:26AM -0800, Darrick J. Wong wrote:
> > > On Mon, Mar 02, 2020 at 12:00:52AM +0800, Eryu Guan wrote:
> > > > On Tue, Feb 04, 2020 at 04:02:23PM -0800, Darrick J. Wong wrote:
> > > > > From: Darrick J. Wong <darrick.wong@oracle.com>
> > > > > 
> > > > > Add new helpers to dmerror to provide for marking selected ranges
> > > > > totally bad -- both reads and writes will fail.  Create a new test for
> > > > > xfs_scrub to check that it reports media errors correctly.
> > > > > 
> > > > > Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
> > > > 
> > > > So is this expected to fail with latest xfsprogs for-next branch? I got
> > > > failures like:
> > > > 
> > > >  QA output created by 515
> > > >   Scrub for injected media error
> > > >  -Corruption: disk offset NNN: media error in inodes. (!)
> > > >  -SCRATCH_MNT: Unmount and run xfs_repair.
> > > 
> > > The test should pass ... and I can't reproduce it all here.  What are
> > > you MKFS_OPTIONS and MOUNT_OPTIONS and kernel?  Here's mine:
> > 
> > FSTYP         -- xfs (debug)
> > PLATFORM      -- Linux/x86_64 fedoravm 5.6.0-rc2 #46 SMP Mon Feb 17 11:37:03 CST 2020
> > MKFS_OPTIONS  -- -f -f -b size=4k -m reflink=1,rmapbt=1 /dev/mapper/testvg-lv2
> > MOUNT_OPTIONS -- /dev/mapper/testvg-lv2 /mnt/scratch
> > 
> > xfs/515 - output mismatch (see /root/workspace/xfstests/results//xfs_4k_reflink/xfs/515.out.bad)
> >     --- tests/xfs/515.out       2020-03-01 22:42:19.569613781 +0800
> >     +++ /root/workspace/xfstests/results//xfs_4k_reflink/xfs/515.out.bad        2020-03-01 23:06:33.546230712 +0800
> >     @@ -1,5 +1,3 @@
> >      QA output created by 515
> >      Scrub for injected media error
> >     -Corruption: disk offset NNN: media error in inodes. (!)
> >     -SCRATCH_MNT: Unmount and run xfs_repair.
> >      Scrub after removing injected media error
> > 
> > And I'm using xfsprogs for-next branch, HEAD is
> > 
> > commit fbbb184b189c62beed2a694d14e83bd316fd4140
> > Author: Eric Sandeen <sandeen@redhat.com>
> > Date:   Thu Feb 27 23:20:42 2020 -0500
> > 
> >     xfs_repair: join realtime inodes to transaction only once
> 
> Hmm, that's really odd.  Can you please send me a metadump of the
> scratch fs after the test runs?  I tried your exact mkfs/mount options

Sure, please see attachment.

Eryu

> and it ran just fine here:
> 
> FSTYP         -- xfs (debug)
> PLATFORM      -- Linux/x86_64 magnolia-mtr00 5.6.0-rc4-xfsx #rc4 SMP PREEMPT Mon Mar 2 21:02:17 PST 2020
> MKFS_OPTIONS  -- -f -f -b size=4k -m reflink=1,rmapbt=1 /dev/sdf
> MOUNT_OPTIONS -- /dev/sdf /opt
> 
> xfs/747  3s
> Ran: xfs/747
> Passed all 1 tests
> 
> --D
> 
> > > --D
> > > 
> > > FSTYP         -- xfs (debug)
> > > PLATFORM      -- Linux/x86_64 magnolia-mtr00 5.6.0-rc4-xfsx #rc4 SMP PREEMPT Mon Mar 2 21:02:17 PST 2020
> > > MKFS_OPTIONS  -- -f -m reflink=1,rmapbt=1, -i sparse=1, /dev/sdf
> > > MOUNT_OPTIONS -- -o usrquota,grpquota,prjquota, /dev/sdf /opt
> > 
> > I think the problem is the mount option, adding the quota related
> > options to my config then test passed as well.
> > 
> > Thanks,
> > Eryu
> > 
> > > 
> > > xfs/747  3s
> > > Ran: xfs/747
> > > Passed all 1 tests
> > > 
> > > -------------------
> > > 
> > > FSTYP         -- xfs (debug)
> > > PLATFORM      -- Linux/x86_64 magnolia-mtr00 5.6.0-rc4-xfsx #rc4 SMP PREEMPT Mon Mar 2 21:02:17 PST 2020
> > > MKFS_OPTIONS  -- -f -m crc=0,reflink=0,rmapbt=0, -i sparse=0, /dev/sdf
> > > MOUNT_OPTIONS -- -o usrquota,grpquota, /dev/sdf /opt
> > > 
> > > xfs/747 [not run] crc feature not supported by this filesystem
> > > Ran: xfs/747
> > > Not run: xfs/747
> > > Passed all 1 tests
> > > 
> > > -------------------
> > > 
> > > FSTYP         -- xfs (debug)
> > > PLATFORM      -- Linux/x86_64 magnolia-mtr00 5.6.0-rc4-xfsx #rc4 SMP PREEMPT Mon Mar 2 21:02:17 PST 2020
> > > MKFS_OPTIONS  -- -f -m reflink=1,rmapbt=0, -i sparse=1, /dev/sdf
> > > MOUNT_OPTIONS -- -o usrquota,grpquota,prjquota, /dev/sdf /opt
> > > 
> > > xfs/747  2s
> > > Ran: xfs/747
> > > Passed all 1 tests
> > > 
> > > --------------------
> > > 
> > > FSTYP         -- xfs (debug)
> > > PLATFORM      -- Linux/x86_64 magnolia-mtr00 5.6.0-rc4-xfsx #rc4 SMP PREEMPT Mon Mar 2 21:02:17 PST 2020
> > > MKFS_OPTIONS  -- -f -m reflink=0,rmapbt=0, -i sparse=1, /dev/sdf
> > > MOUNT_OPTIONS -- -o usrquota,grpquota,prjquota, /dev/sdf /opt
> > > 
> > > xfs/747  3s
> > > Ran: xfs/747
> > > Passed all 1 tests
> > > 
> > > --D

[-- Attachment #2: xfs.metadump.bz2 --]
[-- Type: application/x-bzip2, Size: 80865 bytes --]

  reply	other threads:[~2020-03-04  6:13 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-02-05  0:02 [PATCH 0/1] xfs: test xfs_scrub media scan Darrick J. Wong
2020-02-05  0:02 ` [PATCH 1/1] xfs: test xfs_scrub phase 6 media error reporting Darrick J. Wong
2020-03-01 16:00   ` Eryu Guan
2020-03-03 18:06     ` Darrick J. Wong
     [not found]       ` <20200304025132.GB3128153@desktop>
2020-03-04  3:50         ` Darrick J. Wong
2020-03-04  6:13           ` Eryu Guan [this message]
  -- strict thread matches above, loose matches on Subject: below --
2022-11-02 22:36 [PATCHSET v23.1 0/1] fstests: test xfs_scrub media scan Darrick J. Wong
2022-11-02 22:36 ` [PATCH 1/1] xfs: test xfs_scrub phase 6 media error reporting Darrick J. Wong
2022-11-03  3:11   ` Zorro Lang
2022-11-03  4:13     ` Darrick J. Wong
2022-10-18 22:45 [PATCHSET v23.1 0/1] fstests: test xfs_scrub media scan Darrick J. Wong
2022-10-18 22:45 ` [PATCH 1/1] xfs: test xfs_scrub phase 6 media error reporting Darrick J. Wong
2022-11-01 16:43   ` Zorro Lang
2022-11-02  0:59     ` Darrick J. Wong
2022-11-02  3:45       ` Zorro Lang
2022-11-02 17:38         ` Darrick J. Wong
2020-01-16  5:11 [PATCH 0/1] xfs: test xfs_scrub media scan Darrick J. Wong
2020-01-16  5:11 ` [PATCH 1/1] xfs: test xfs_scrub phase 6 media error reporting Darrick J. Wong

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200304050737.GC3128153@desktop \
    --to=guaneryu@gmail.com \
    --cc=darrick.wong@oracle.com \
    --cc=fstests@vger.kernel.org \
    --cc=guaneryu@aliyun.com \
    --cc=linux-xfs@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.