From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alex Elder Subject: Re: [PATCH v3 1/2] xfstests: make more tests generic Date: Wed, 10 Aug 2011 11:36:09 -0500 Message-ID: <1312994169.2865.41.camel@doink> References: <20110810155214.GE20461@twin.jikos.cz> Reply-To: Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: , , Stefan Behrens , To: Return-path: Received: from relay1.sgi.com ([192.48.179.29]:59527 "EHLO relay.sgi.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753840Ab1HJQgO (ORCPT ); Wed, 10 Aug 2011 12:36:14 -0400 In-Reply-To: <20110810155214.GE20461@twin.jikos.cz> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Wed, 2011-08-10 at 17:52 +0200, David Sterba wrote: > Hi, > > On Mon, Aug 01, 2011 at 12:31:19PM +0200, Stefan Behrens wrote: > > Use _scratch_mkfs / _scratch_mkfs_sized instead of _scratch_mkfs_xfs > > where possible. > > Execute 015, 062, 083, 117, 120 and 192 for all filesystems, these > > tests used to be XFS specific. > > this patch is now in xfstests-dev and somehow breaks btrfs testing. . . . > there's a hardcoded path for mkfs.btrfs in common.rc:_scratch_mkfs_sized() > > 335 btrfs) > 336 /sbin/mkfs.$FSTYP $MKFS_OPTIONS $SCRATCH_DEV -b $fssize > 337 ;; > > I have a /usr/local/ installation of btrfsprogs from git, > _scratch_mkfs_sized() was not called for btrfs before. > > Possible ways to fix this: > 1) use /sbin/mkfs -t $FSTYP like _scratch_mkfs, or > 2) set $MKFS_BTRFS_PROG like for some other filesystems > > > What is preferred? Whatever works. My personal preference would be option (1) if it works, since it's does the generic thing directly rather than suggesting something special is needed for btrfs. But if that doesn't work, consistency rules so MKFS_BTRFS_PROG "like for some other filesystems" is good. -Alex