From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: linux-nfs-owner@vger.kernel.org Received: from verein.lst.de ([213.95.11.211]:40771 "EHLO newverein.lst.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751334AbaB0U4Q (ORCPT ); Thu, 27 Feb 2014 15:56:16 -0500 Date: Thu, 27 Feb 2014 21:50:29 +0100 From: Christoph Hellwig To: Chuck Lever Cc: Linux NFS Mailing List , xfs@oss.sgi.com Subject: Re: xfstests with NFS Message-ID: <20140227205029.GA29741@lst.de> References: <27AB99CF-CEF3-4AC5-AFBD-6648F3B3BED2@oracle.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 In-Reply-To: <27AB99CF-CEF3-4AC5-AFBD-6648F3B3BED2@oracle.com> Sender: linux-nfs-owner@vger.kernel.org List-ID: On Thu, Feb 27, 2014 at 10:32:51AM -0800, Chuck Lever wrote: > > I’m running xfstests to validate functionality of a particular > NFS client/transport/server set up. > > There are a couple of tests that may not always be appropriate for > FSTYP=nfs. We weren’t able to identify a bugzilla where we can report > these issues. Just send the to the xfs list. (and cap lines after 75 chars like for all Linux lists..) > generic/192 tests atime changes. The Linux NFS client (and probably > others) typically optimizes away atime changes. Thus this test fails > for NFS mounts on Linux. I’m wondering if we can treat NFS as always > a noatime filesystem. I guess there's two components to this if we want to do it properly: - make sure nfs always reports noatime in /proc/self/mounts instead of incorrectly claiming it supports relatime by default - _notrun 192 if it detects the filesystems reports noatime. > generic/237 tries setfacl. On NFS mounts, the operation always returns > “Operation not supported,” but the test is looking for “Operation not > permitted.” The former is an acceptable pass, IMO. I considered > disabling this test for NFS, but I would bet some NFS client > implementations actually do support setfacl(). The NFSv3 client support ACLs perfectly, it's just NFSv4 that's messed up. 237 does a _require_acls before ding thæ test, which checks for ENOSYS or EOPNOTSUPP, which in theory should catch NFSv4 in theory, but if this doesn't happen try to debug it by looking at the _require_acls shell function, the chacl implementation, the NFSv4 ACL stubs and anything else related. From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from relay.sgi.com (relay1.corp.sgi.com [137.38.102.111]) by oss.sgi.com (Postfix) with ESMTP id CCA087F3F for ; Thu, 27 Feb 2014 14:50:33 -0600 (CST) Received: from cuda.sgi.com (cuda3.sgi.com [192.48.176.15]) by relay1.corp.sgi.com (Postfix) with ESMTP id AA8718F8059 for ; Thu, 27 Feb 2014 12:50:33 -0800 (PST) Received: from newverein.lst.de (verein.lst.de [213.95.11.211]) by cuda.sgi.com with ESMTP id VoB4hxGMGPciXPeB (version=TLSv1 cipher=AES256-SHA bits=256 verify=NO) for ; Thu, 27 Feb 2014 12:50:32 -0800 (PST) Date: Thu, 27 Feb 2014 21:50:29 +0100 From: Christoph Hellwig Subject: Re: xfstests with NFS Message-ID: <20140227205029.GA29741@lst.de> References: <27AB99CF-CEF3-4AC5-AFBD-6648F3B3BED2@oracle.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <27AB99CF-CEF3-4AC5-AFBD-6648F3B3BED2@oracle.com> List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Errors-To: xfs-bounces@oss.sgi.com Sender: xfs-bounces@oss.sgi.com To: Chuck Lever Cc: Linux NFS Mailing List , xfs@oss.sgi.com T24gVGh1LCBGZWIgMjcsIDIwMTQgYXQgMTA6MzI6NTFBTSAtMDgwMCwgQ2h1Y2sgTGV2ZXIgd3Jv dGU6Cj4gCj4gSeKAmW0gcnVubmluZyB4ZnN0ZXN0cyB0byB2YWxpZGF0ZSBmdW5jdGlvbmFsaXR5 IG9mIGEgcGFydGljdWxhcgo+IE5GUyBjbGllbnQvdHJhbnNwb3J0L3NlcnZlciBzZXQgdXAuCj4g Cj4gVGhlcmUgYXJlIGEgY291cGxlIG9mIHRlc3RzIHRoYXQgbWF5IG5vdCBhbHdheXMgYmUgYXBw cm9wcmlhdGUgZm9yCj4gRlNUWVA9bmZzLiAgV2Ugd2VyZW7igJl0IGFibGUgdG8gaWRlbnRpZnkg YSBidWd6aWxsYSB3aGVyZSB3ZSBjYW4gcmVwb3J0Cj4gdGhlc2UgaXNzdWVzLgoKSnVzdCBzZW5k IHRoZSB0byB0aGUgeGZzIGxpc3QuICAoYW5kIGNhcCBsaW5lcyBhZnRlciA3NSBjaGFycyBsaWtl IGZvciBhbGwKTGludXggbGlzdHMuLikKCj4gZ2VuZXJpYy8xOTIgdGVzdHMgYXRpbWUgY2hhbmdl cy4gIFRoZSBMaW51eCBORlMgY2xpZW50IChhbmQgcHJvYmFibHkKPiBvdGhlcnMpIHR5cGljYWxs eSBvcHRpbWl6ZXMgYXdheSBhdGltZSBjaGFuZ2VzLiAgVGh1cyB0aGlzIHRlc3QgZmFpbHMKPiBm b3IgTkZTIG1vdW50cyBvbiBMaW51eC4gIEnigJltIHdvbmRlcmluZyBpZiB3ZSBjYW4gdHJlYXQg TkZTIGFzIGFsd2F5cwo+IGEgbm9hdGltZSBmaWxlc3lzdGVtLgoKSSBndWVzcyB0aGVyZSdzIHR3 byBjb21wb25lbnRzIHRvIHRoaXMgaWYgd2Ugd2FudCB0byBkbyBpdCBwcm9wZXJseToKCiAtIG1h a2Ugc3VyZSBuZnMgYWx3YXlzIHJlcG9ydHMgbm9hdGltZSBpbiAvcHJvYy9zZWxmL21vdW50cyBp bnN0ZWFkCiAgIG9mIGluY29ycmVjdGx5IGNsYWltaW5nIGl0IHN1cHBvcnRzIHJlbGF0aW1lIGJ5 IGRlZmF1bHQKIC0gX25vdHJ1biAxOTIgaWYgaXQgZGV0ZWN0cyB0aGUgZmlsZXN5c3RlbXMgcmVw b3J0cyBub2F0aW1lLgoKPiBnZW5lcmljLzIzNyB0cmllcyBzZXRmYWNsLiAgT24gTkZTIG1vdW50 cywgdGhlIG9wZXJhdGlvbiBhbHdheXMgcmV0dXJucwo+IOKAnE9wZXJhdGlvbiBub3Qgc3VwcG9y dGVkLOKAnSBidXQgdGhlIHRlc3QgaXMgbG9va2luZyBmb3Ig4oCcT3BlcmF0aW9uIG5vdAo+IHBl cm1pdHRlZC7igJ0gIFRoZSBmb3JtZXIgaXMgYW4gYWNjZXB0YWJsZSBwYXNzLCBJTU8uICBJIGNv bnNpZGVyZWQKPiBkaXNhYmxpbmcgdGhpcyB0ZXN0IGZvciBORlMsIGJ1dCBJIHdvdWxkIGJldCBz b21lIE5GUyBjbGllbnQKPiBpbXBsZW1lbnRhdGlvbnMgYWN0dWFsbHkgZG8gc3VwcG9ydCBzZXRm YWNsKCkuCgpUaGUgTkZTdjMgY2xpZW50IHN1cHBvcnQgQUNMcyBwZXJmZWN0bHksIGl0J3MganVz dCBORlN2NCB0aGF0J3MgbWVzc2VkIHVwLgoKMjM3IGRvZXMgYSBfcmVxdWlyZV9hY2xzIGJlZm9y ZSBkaW5nIHRow6YgdGVzdCwgd2hpY2ggY2hlY2tzIGZvcgpFTk9TWVMgb3IgRU9QTk9UU1VQUCwg d2hpY2ggaW4gdGhlb3J5IHNob3VsZCBjYXRjaCBORlN2NCBpbiB0aGVvcnksIGJ1dAppZiB0aGlz IGRvZXNuJ3QgaGFwcGVuIHRyeSB0byBkZWJ1ZyBpdCBieSBsb29raW5nIGF0IHRoZSBfcmVxdWly ZV9hY2xzIHNoZWxsCmZ1bmN0aW9uLCB0aGUgY2hhY2wgaW1wbGVtZW50YXRpb24sIHRoZSBORlN2 NCBBQ0wgc3R1YnMgYW5kIGFueXRoaW5nIGVsc2UKcmVsYXRlZC4KCl9fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCnhmcyBtYWlsaW5nIGxpc3QKeGZzQG9zcy5z Z2kuY29tCmh0dHA6Ly9vc3Muc2dpLmNvbS9tYWlsbWFuL2xpc3RpbmZvL3hmcwo=