From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from relay.sgi.com (relay3.corp.sgi.com [198.149.34.15]) by oss.sgi.com (Postfix) with ESMTP id DCE3B29DF8 for ; Sun, 12 Jan 2014 19:52:43 -0600 (CST) Received: from cuda.sgi.com (cuda2.sgi.com [192.48.176.25]) by relay3.corp.sgi.com (Postfix) with ESMTP id 5ACE0AC001 for ; Sun, 12 Jan 2014 17:52:43 -0800 (PST) Received: from ipmail05.adl6.internode.on.net (ipmail05.adl6.internode.on.net [150.101.137.143]) by cuda.sgi.com with ESMTP id iHSvzWANHNFELJUM for ; Sun, 12 Jan 2014 17:52:41 -0800 (PST) Date: Mon, 13 Jan 2014 12:52:39 +1100 From: Dave Chinner Subject: Re: [PATCH] xfstests: Add pairing mount options test Message-ID: <20140113015239.GD3469@dastard> References: <1389162648-19309-1-git-send-email-quwenruo@cn.fujitsu.com> <52D01CA9.4040107@sandeen.net> <52D33F92.2000609@cn.fujitsu.com> <52D342F0.5020402@sandeen.net> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <52D342F0.5020402@sandeen.net> List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: xfs-bounces@oss.sgi.com Sender: xfs-bounces@oss.sgi.com To: Eric Sandeen Cc: Qu Wenruo , Eric Sandeen , linux-btrfs , xfs@oss.sgi.com On Sun, Jan 12, 2014 at 07:35:44PM -0600, Eric Sandeen wrote: > On 1/12/14, 7:21 PM, Qu Wenruo wrote: > > On fri, 10 Jan 2014 10:15:37 -0600, Eric Sandeen wrote: > >> On 1/8/14, 12:30 AM, Qu Wenruo wrote: > >>> Test remount btrfs with different pairing options like barrier and no barrier. > >> It seems that while this tests that the remount succeeds, and that > >> the option string is present in /proc/mounts, it does not test that > >> the mount option is actually in effect. > > > > Yes, this is what the new test case is intended to do. > > This case was just a test case tests the mount options themselves > > to ensure all the pairing mount options works during remounting, > > since most pairing options are missing before. > >> > >> I suppose for many of these options that would be hard to test; for > >> i.e. acl though it should be trivial. > >> > >> What do you think, is this enough to ensure that remount handling > >> is working as expected for all of these options? > > In my opinion, this test should just focuses on the remount handling and > > the pairing options. > > For the detailed function should be examineed in other test cases. > > Except those won't test that a remount with those options actually *worked*; > in fact they don't do remount at all. > > In other words, all this does is test that an option flag was set or unset in > the superblock, but it doesn't really test whether the option has been > properly set up (or torn down) as a result. > > I won't say no to this, but it seems to be of somewhat limited use. What happens to the test when mount options are deprecated/removed? How are we going to handle the matrix of testable/untestable mount options across kernels with different mount option support? Cheers, Dave. -- Dave Chinner david@fromorbit.com _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs