On Thu, Apr 02, 2015 at 09:41:01AM +1100, Dave Chinner wrote: > On Wed, Apr 01, 2015 at 09:54:15PM +0800, Eryu Guan wrote: > > xfs used to panic in this test, this xfs commit fix the bug > > > > 8d6c121 xfs: fix buffer use after free on IO error > > > > ext4 and btrfs trigger WARNING on current 4.0-rc3 kernel > > > > Signed-off-by: Eryu Guan <eguan@xxxxxxxxxx> > .... > > +# lvm uses some space for metadata > > +_scratch_mkfs_sized $((300 * 1024 * 1024)) >>$seqres.full 2>&1 > > +$LVM_PROG vgcreate -f $vgname $SCRATCH_DEV >>$seqres.full 2>&1 > > +$LVM_PROG lvcreate --yes -L 256M -n $lvname $vgname >>$seqres.full 2>&1 > > + > > +# _mkfs_dev exits the test on failure, this can make sure lv is created in > > +# above vgcreate/lvcreate steps > > +_mkfs_dev /dev/mapper/$vgname-$lvname > > So on my 1p test VM, this fails with > > +mkfs.xfs: cannot open /dev/mapper/vg_081-base_081: Device or resource busy > > The problem is that udev has not finished setting up the device > before mkfs is run. Hence we need a "udevadm settle" call after the > lvcreate call. This results in mkfs succeeding on this machine. > > Eryu, I'm going to commit the test as it stands as it works on all > my other test systems - can you write a followup patch that does the > udev settle call in a portable manner? i.e. older systems used to > have a 'udev-settle' command, do we still care about that? Sure, I'll do that, along with the fix of the cleanup error, as Brian suggested. As a distribution tester of RHEL, I don't care much about older systems now like RHEL5, but I'll try to make it portable if possible. Thanks, Eryu -- To unsubscribe from this list: send the line "unsubscribe fstests" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
