Re: [PATCH 1/3] Skip _check_test_fs unless a test is actually run

xfs

Re: [PATCH 1/3] Skip _check_test_fs unless a test is actually run

To:	"Theodore Ts'o" <tytso@xxxxxxx>
Subject:	Re: [PATCH 1/3] Skip _check_test_fs unless a test is actually run
From:	Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date:	Sat, 20 Jun 2009 13:02:22 -0400
Cc:	xfs@xxxxxxxxxxx
In-reply-to:	<1245434013-9433-1-git-send-email-tytso@xxxxxxx>
References:	<1245434013-9433-1-git-send-email-tytso@xxxxxxx>
User-agent:	Mutt/1.5.18 (2008-05-17)

On Fri, Jun 19, 2009 at 01:53:31PM -0400, Theodore Ts'o wrote:
> Running _check_test_fs can take a non-trivial amount of time, and if a
> test has been skipped because it doesn't work on Linux, or it doesn't
> work of for ext4, it's pointless to re-run _check_test_fs.  So move
> the call to _check_test_fs so it is only run if a test is actually run.

Thanks, I've put this in.

[More with this subject...]

<Prev in Thread]	Current Thread	[Next in Thread>
[PATCH 1/3] Skip _check_test_fs unless a test is actually run, Theodore Ts'o [PATCH 3/3] Fix _require_scratch test for extN, resierfs, gfs2, and btrfs, Theodore Ts'o Re: [PATCH 3/3] Fix _require_scratch test for extN, resierfs, gfs2, and btrfs, Felix Blyakher [PATCH] Fix _require_scratch test for extN, resierfs, gfs2, and btrfs, Theodore Ts'o Re: [PATCH] Fix _require_scratch test for extN, resierfs, gfs2, and btrfs, Christoph Hellwig [PATCH 2/3] Fix common.config to allow SCRATCH_DEV and SCRATCH_MNT to be optional, Theodore Ts'o Re: [PATCH 2/3] Fix common.config to allow SCRATCH_DEV and SCRATCH_MNT to be optional, Christoph Hellwig Re: [PATCH 1/3] Skip _check_test_fs unless a test is actually run, Felix Blyakher Re: [PATCH 1/3] Skip _check_test_fs unless a test is actually run, Christoph Hellwig <=

Previous by Date:	Re: 2.6.30 panic - xfs_fs_destroy_inode, Patrick Schreurs
Next by Date:	Re: [PATCH] Fix _require_scratch test for extN, resierfs, gfs2, and btrfs, Christoph Hellwig
Previous by Thread:	Re: [PATCH 1/3] Skip _check_test_fs unless a test is actually run, Felix Blyakher
Next by Thread:	Re: [PATCH] fs: Add new pre-allocation ioctls to vfs for compatibility with legacy xfs ioctls, Christoph Hellwig
Indexes:	[Date] [Thread] [Top] [All Lists]