linux-kernel - Re: [PATCH] xfs: fail dax mount if reflink is enabled on a partition

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date:   Thu, 3 Nov 2022 13:32:27 +1100
From:   Dave Chinner <david@...morbit.com>
To:     "ruansy.fnst@...itsu.com" <ruansy.fnst@...itsu.com>
Cc:     "Darrick J. Wong" <djwong@...nel.org>,
        Dan Williams <dan.j.williams@...el.com>,
        "yangx.jy@...itsu.com" <yangx.jy@...itsu.com>,
        "Yasunori Gotou (Fujitsu)" <y-goto@...itsu.com>,
        Brian Foster <bfoster@...hat.com>,
        "hch@...radead.org" <hch@...radead.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "linux-xfs@...r.kernel.org" <linux-xfs@...r.kernel.org>,
        "nvdimm@...ts.linux.dev" <nvdimm@...ts.linux.dev>,
        "linux-fsdevel@...r.kernel.org" <linux-fsdevel@...r.kernel.org>,
        "zwisler@...nel.org" <zwisler@...nel.org>,
        Jeff Moyer <jmoyer@...hat.com>,
        "dm-devel@...hat.com" <dm-devel@...hat.com>,
        "toshi.kani@....com" <toshi.kani@....com>,
        Theodore Ts'o <tytso@....edu>
Subject: Re: [PATCH] xfs: fail dax mount if reflink is enabled on a partition

On Wed, Nov 02, 2022 at 05:17:18AM +0000, ruansy.fnst@...itsu.com wrote:
> 
> 在 2022/11/2 8:45, Darrick J. Wong 写道:
> > On Sun, Oct 30, 2022 at 05:31:43PM +0800, Shiyang Ruan wrote:
> > FWIW things have calmed down as of 6.1-rc3 -- if I disable reflink,
> > fstests runs without complaint.  Now it only seems to be affecting
> > reflink=1 filesystems. >
> >> And how is your recent test?  Still failed with those dmesg warnings? If so,
> >> could you zip the test result and send it to me?
> > 
> > https://djwong.org/docs/kernel/daxbad.zip
> 
> Thanks for your info!
> 
> (To Dave) I need your recent test result too.  If cases won't fail when 
> reflink disabled, I'll focusing on solving the warning when reflink enabled.

My first run on 6.1-rc3 with reflink disabled was clean. Then I ran
a few tests with reflink enabled, and they all failed as expected.
Then I ran the no-reflink tests again, and then they all failed,
too. Nothing changed between test configs, I didn't even reboot the
test machine:

$ history |tail -5
500  sudo ./run_check.sh --mkfs-opts "-m rmapbt=1,reflink=0" --run-opts "-s xfs_dax xfs/55[12]"
501  sudo ./run_check.sh --mkfs-opts "-m rmapbt=1,reflink=0"
  --run-opts "-s xfs_dax_noreflink generic/051 generic/068
  generic/074 generic/075 generic/083 generic/112 generic/231
  generic/232 generic/269 generic/270 generic/340 generic/388
  generic/461 generic/471 generic/476 generic/519 generic/560
  generic/561 generic/617 generic/650 generic/656 xfs/011 xfs/013
  xfs/017 xfs/073 xfs/297 xfs/305 xfs/517 xfs/538"
502  sudo ./run_check.sh --mkfs-opts "-m rmapbt=1" --run-opts
    "-s xfs_dax generic/051 generic/068 generic/074 generic/075
    generic/083 generic/112 generic/231 generic/232 generic/269
    generic/270 generic/340 generic/388 generic/461 generic/471
    generic/476 generic/519 generic/560 generic/561 generic/617
    generic/650 generic/656 xfs/011 xfs/013 xfs/017 xfs/073 xfs/297
    xfs/305 xfs/517 xfs/538"
503  sudo ./run_check.sh --mkfs-opts "-m rmapbt=1,reflink=0"
      --run-opts "-s xfs_dax_noreflink generic/051 generic/068
      generic/074 generic/075 generic/083 generic/112 generic/231
      generic/232 generic/269 generic/270 generic/340 generic/388
      generic/461 generic/471 generic/476 generic/519 generic/560
      generic/561 generic/617 generic/650 generic/656 xfs/011
      xfs/013 xfs/017 xfs/073 xfs/297 xfs/305 xfs/517 xfs/538"
504  history |tail -5
$

The first noreflink run:

SECTION       -- xfs_dax_noreflink
=========================
Failures: generic/471 generic/519 xfs/073
Failed 3 of 29 tests

Which are typical failures for this config.

The first reflink enabled run I killed almost immediately as it
threw multiple warnings in the first couple of tests:

Running: MOUNT_OPTIONS= ./check -R xunit -b -s xfs_dax generic/051 generic/068 generic/074 generic/075 generic/083 generic/112 generic/231 generic/232 generic/269 generic/270 generic/340 generic/388 generic/461 generic/471 generic/476 generic/519 generic/560 generic/561 generic/617 generic/650 generic/656 xfs/011 xfs/013 xfs/017 xfs/073 xfs/297 xfs/305 xfs/517 xfs/538
SECTION       -- xfs_dax
FSTYP         -- xfs (debug)
PLATFORM      -- Linux/x86_64 test3 6.1.0-rc3-dgc+ #1649 SMP PREEMPT_DYNAMIC Wed Nov  2 07:58:17 AEDT 2022
MKFS_OPTIONS  -- -f -m rmapbt=1 /dev/pmem1
MOUNT_OPTIONS -- -o dax=always -o context=system_u:object_r:root_t:s0 /dev/pmem1 /mnt/scratch

generic/051 79s ... _check_dmesg: something found in dmesg (see /home/dave/src/xfstests-dev/results//xfs_dax/generic/051.dmesg)

generic/068 43s ... ^C
real    1m46.278s
user    0m17.465s
sys     2m9.981s
$

And then I ran the noreflink tests again to make sure the first run
wasn't a fluke:

SECTION       -- xfs_dax_noreflink
=========================
Failures: generic/051 generic/068 generic/231 generic/269
generic/270 generic/340 generic/388 generic/461 generic/471
generic/476 generic/519 generic/560 generic/561 xfs/011 xfs/013
xfs/073 xfs/297 xfs/305 xfs/517 xfs/538
Failed 20 of 29 tests

It was a fluke - most of the tests failed this time with dax
mapping warnings. dmesg from the entire set of test runs is
attached.

-Dave.
-- 
Dave Chinner
david@...morbit.com

Download attachment "dmesg.log.gz" of type "application/gzip" (50151 bytes)