From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ipmail06.adl2.internode.on.net ([150.101.137.129]:33723 "EHLO ipmail06.adl2.internode.on.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753559AbaKMEoI (ORCPT ); Wed, 12 Nov 2014 23:44:08 -0500 Date: Thu, 13 Nov 2014 15:44:05 +1100 From: Dave Chinner Subject: Re: xfstest status on current kernels Message-ID: <20141113044405.GF28565@dastard> References: <20141113032626.GA28565@dastard> <20141113042231.GD28565@dastard> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: fstests-owner@vger.kernel.org To: Steve French Cc: fstests@vger.kernel.org List-ID: On Wed, Nov 12, 2014 at 10:33:37PM -0600, Steve French wrote: > On Wed, Nov 12, 2014 at 10:22 PM, Dave Chinner wrote: > > On Wed, Nov 12, 2014 at 09:38:45PM -0600, Steve French wrote: > >> On Wed, Nov 12, 2014 at 9:26 PM, Dave Chinner wrote: > >> > On Wed, Nov 12, 2014 at 05:31:20PM -0600, Steve French wrote: > >> > >> >> I want to make sure that I don't accidentally ignore a test (e.g. test > >> >> generic/003 doesn't run with a message " [not run] relatime not > >> >> supported by the current kernel" and want to make sure I am not > >> >> missing something). > >> I have been going through these one at a time as I have time (to > >> see if they are workable on cifs/smb3 etc) - but it can get > >> tricky (e.g. in this case relatime may be default behavior for fs so > >> lack of a mount option with this exact name may be unnecessarily > >> disabling this one on some fs) > > > > I don't think that's the case. Any kernel that supports relatime > > supports the relatime mount option. It doesn't matter if the > > filesystem defaults to it or not, MS_RELATIME is a valid mount > > flag on all supported kernels. > > relatime is rejected by the kernel cifs.ko driver (I will need to fix > that) but should be ignored. relatime should never get to the kernel as a string fs specific string. It's supposed to be parsed by the mount binary and passed as the MS_RELATIME flag to the kernel, not as a string. > Looking at the xfstest logs looks like > noatime is going to have the same problem as well. > I will fix these in the cifs mount parsing. Again: MS_NOATIME. This sounds like a bug in the mount.cifs binary, not the kernel. Cheers, Dave. -- Dave Chinner david@fromorbit.com