On 9/20/16 6:39 AM, Zorro Lang wrote: ... > Then I got problem when I ran xfs/073 with MKFS_OPTIONS="-m crc=1": > [root@dhcp-13-149 xfstests-dev]# diff -Nurp tests/xfs/073.out results/xfs/073.out.bad > --- tests/xfs/073.out 2016-04-13 11:44:17.809000000 +0800 > +++ results/xfs/073.out.bad 2016-09-20 19:12:40.194000000 +0800 > @@ -59,8 +59,5 @@ comparing new image directories to old > comparing new image geometry to old > unmounting and removing new image > checking new image > -mounting new image on loopback > -comparing new image files to old > -comparing new image directories to old > -comparing new image geometry to old > -unmounting and removing new image > +_check_xfs_filesystem: filesystem on /mnt/test/9672.image2 is inconsistent (c) (see /root/git/xfstests-dev/results//xfs/073.full) > +_check_xfs_filesystem: filesystem on /mnt/test/9672.image2 is inconsistent (r) (see /root/git/xfstests-dev/results//xfs/073.full) > > > The last test stes of xfs/073 failed: > echo === copying scratch device to multiple targets > $XFS_COPY_PROG -d -L$imgs.log -b $SCRATCH_DEV $imgs.image1 $imgs.image2 \ > | _filter_copy '#' $imgs.image1 '#' $imgs.image2 > _verify_copy $imgs.image1 $SCRATCH_DEV $SCRATCH_MNT > _verify_copy $imgs.image2 $SCRATCH_DEV $SCRATCH_MNT > > > The full output is too long, so I paste it to below link (this link will > keep alive in one month): > http://paste.fedoraproject.org/431361/74370897/ > > I can't find this problem on V4 XFS(crc=0). I tested this bug with upstream > linux v4.8-rc7(d2ffb01 Merge branch 'akpm' (patches from Andrew)) and xfsprogs > v4.8-rc1(130093a xfsprogs: Release 4.8.0-rc1). > > This problem can be simply reproduced by below steps: > 1. mkfs.xfs -d name=fsfile,size=512m,file=1 -m crc=1 > 2. xfs_copy fsfile target1 target2 > 3. xfs_repair -n target2 > 4. xfs_repair -n target1 > > But these steps can't make target1 corrupted(in my test). Thanks for finding this Zorro - looks like it is confusing which uuid it should be using across the multiple copies. $ xfs_db -c "sb 0" -c "p uuid" -c "p meta_uuid" fsfile uuid = a439b9f6-a660-46ab-ad10-841485de27a4 meta_uuid = 00000000-0000-0000-0000-000000000000 $ xfs_db -c "sb 0" -c "p uuid" -c "p meta_uuid" copy1 uuid = e0d30f68-533c-48c2-bb96-d719526c5d64 meta_uuid = a439b9f6-a660-46ab-ad10-841485de27a4 (ok, meta_uuid matches original) $ xfs_db -c "sb 0" -c "p uuid" -c "p meta_uuid" copy2 Metadata corruption detected at xfs_agf block 0x1/0x200 xfs_db: cannot init perag data (-117). Continuing anyway. uuid = 50bead00-e076-476e-9b7f-db3349253bbf meta_uuid = e0d30f68-533c-48c2-bb96-d719526c5d64 (not OK, meta_uuid matches superblock UUID of first copy, not original) I'll get it fixed up today. -Eric > Thanks, > Zorro > -- > To unsubscribe from this list: send the line "unsubscribe linux-xfs" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html