From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from userp1040.oracle.com ([156.151.31.81]:41561 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752079AbdJ0SWK (ORCPT ); Fri, 27 Oct 2017 14:22:10 -0400 Date: Fri, 27 Oct 2017 11:22:05 -0700 From: "Darrick J. Wong" Subject: Re: [PATCH 9/6] generic/459: fix test running errors Message-ID: <20171027182205.GQ5483@magnolia> References: <150899709935.18389.17266737014565285073.stgit@magnolia> <20171027004446.GG5486@magnolia> <20171027044201.GU3235@eguan.usersys.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20171027044201.GU3235@eguan.usersys.redhat.com> Sender: linux-xfs-owner@vger.kernel.org List-ID: List-Id: xfs To: Eryu Guan Cc: linux-xfs@vger.kernel.org, fstests@vger.kernel.org On Fri, Oct 27, 2017 at 12:42:01PM +0800, Eryu Guan wrote: > On Thu, Oct 26, 2017 at 05:44:46PM -0700, Darrick J. Wong wrote: > > If the DISCARD of the thin device somehow fails with this message: > > > > device-mapper: thin: Data device (dm-1) discard unsupported: Disabling discard passdown. > > > > Then we can end up with arbitrary gunk in the thin device. This causes > > mkfs to fail because it's afraid to format the device. Don't be afraid, > > just zap it. FWIW mkfs.xfs thinks that the thinp device has an xfs > > external log because sometimes the thinp device just happen to be backed > > by the log of the previous test's scratch fs. > > > > Signed-off-by: Darrick J. Wong > > --- > > tests/generic/459 | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/tests/generic/459 b/tests/generic/459 > > index 7c10c2a..3dd3681 100755 > > --- a/tests/generic/459 > > +++ b/tests/generic/459 > > @@ -92,7 +92,7 @@ $LVM_PROG lvcreate --virtualsize $virtsize \ > > -T $vgname/$poolname \ > > -n $lvname >>$seqres.full 2>&1 > > > > -_mkfs_dev /dev/mapper/$vgname-$lvname >>$seqres.full 2>&1 > > +_mkfs_dev -f /dev/mapper/$vgname-$lvname >>$seqres.full 2>&1 > > I think we should fix it in _mkfs_dev, add a new switch entry for xfs. Will do. --D > Thanks, > Eryu > > > > > > > # Running the test over the original volume doesn't reproduce the problem > -- > To unsubscribe from this list: send the line "unsubscribe linux-xfs" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html