Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753908AbdLMVuT (ORCPT ); Wed, 13 Dec 2017 16:50:19 -0500 Received: from ipmail06.adl6.internode.on.net ([150.101.137.145]:60150 "EHLO ipmail06.adl6.internode.on.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753144AbdLMVuQ (ORCPT ); Wed, 13 Dec 2017 16:50:16 -0500 Date: Thu, 14 Dec 2017 08:50:13 +1100 From: Dave Chinner To: "Luis R. Rodriguez" Cc: fstests@vger.kernel.org, linux-xfs@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 5/9] tests/xfs/group: add group for tests which require a logdev Message-ID: <20171213215013.GW4094@dastard> References: <20171213004519.29340-1-mcgrof@kernel.org> <20171213004519.29340-6-mcgrof@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20171213004519.29340-6-mcgrof@kernel.org> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1390 Lines: 46 On Tue, Dec 12, 2017 at 04:45:15PM -0800, Luis R. Rodriguez wrote: > This should make it easy to run these separately or exclude them. These should notrun automatically if you don't have an external log device configured. Every test should either work with an external logdev or explicitly notrun them, so I'm not sure what you're trying to acheive here.... > > Signed-off-by: Luis R. Rodriguez > --- > tests/xfs/group | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/tests/xfs/group b/tests/xfs/group > index d23006041ea2..cce98847de53 100644 > --- a/tests/xfs/group > +++ b/tests/xfs/group > @@ -42,7 +42,7 @@ > 042 fsr ioctl auto > 043 dump ioctl tape > 044 other auto > -045 other auto quick > +045 other auto quick logdev This change also looks wrong because: xfs/044 [not run] This test requires a valid $SCRATCH_LOGDEV xfs/045 1s ... 1s xfs/044 is the external logdev test, and xfs/045 is a duplicate uuid mount test that has nothign to do with external log devices. And, FWIW, we already have a "log" group to indicate tests that exercise the log, and that mostly includes all the tests that use external logs. It would be better to tag all the tests that exercise the log with "log" rather than create some new group that doesn't really provide any added benefit.... Cheers, Dave. -- Dave Chinner david@fromorbit.com