From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pf1-f196.google.com ([209.85.210.196]:44625 "EHLO mail-pf1-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726451AbfFYCpw (ORCPT ); Mon, 24 Jun 2019 22:45:52 -0400 Date: Tue, 25 Jun 2019 10:45:46 +0800 From: Eryu Guan Subject: Re: [PATCH v2] xfs: test xfs_info on block device and mountpoint Message-ID: <20190625024546.GO15846@desktop> References: <20190622153827.4448-1-zlang@redhat.com> <20190623214919.GD5387@magnolia> <20190624012103.GF30864@dhcp-12-102.nay.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190624012103.GF30864@dhcp-12-102.nay.redhat.com> Sender: linux-xfs-owner@vger.kernel.org List-ID: List-Id: xfs To: Zorro Lang Cc: "Darrick J. Wong" , fstests@vger.kernel.org, linux-xfs@vger.kernel.org On Mon, Jun 24, 2019 at 09:21:03AM +0800, Zorro Lang wrote: > On Sun, Jun 23, 2019 at 02:49:19PM -0700, Darrick J. Wong wrote: > > On Sat, Jun 22, 2019 at 11:38:27PM +0800, Zorro Lang wrote: > > > There was a bug, xfs_info fails on a mounted block device: > > > > > > # xfs_info /dev/mapper/testdev > > > xfs_info: /dev/mapper/testdev contains a mounted filesystem > > > > > > fatal error -- couldn't initialize XFS library > > > > > > xfsprogs has fixed it by: > > > > > > bbb43745 xfs_info: use findmnt to handle mounted block devices > > > > > > Signed-off-by: Zorro Lang > > > > Aha! I remembered something -- xfs/449 already checks for consistency > > in the various xfs geometry reports that each command provides, so why > > not just add the $XFS_INFO_PROG $SCRATCH_DEV case at the end? > > Wow, there're so many cases, can't sure what we've covered now:) > > Sure, I can do this change on xfs/449, if Eryu thinks it's fine to increase > the test coverage of a known case. Given that we're having more and more tests and the test time grows quickly, I'm fine now with adding such small & similar test to existing test case to reuse the test setups, especially when XFS maintainer agrees to do so :) Thanks, Eryu