Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753179AbcLFS6K (ORCPT ); Tue, 6 Dec 2016 13:58:10 -0500 Received: from fieldses.org ([173.255.197.46]:44708 "EHLO fieldses.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751593AbcLFS6I (ORCPT ); Tue, 6 Dec 2016 13:58:08 -0500 Date: Tue, 6 Dec 2016 13:58:06 -0500 From: "J. Bruce Fields" To: Andreas Gruenbacher Cc: Miklos Szeredi , Andreas =?utf-8?Q?Gr=C3=BCnbacher?= , Patrick Plagwitz , "linux-unionfs@vger.kernel.org" , Linux NFS list , Linux FS-devel Mailing List , Linux Kernel Mailing List Subject: Re: [PATCH] overlayfs: ignore empty NFSv4 ACLs in ext4 upperdir Message-ID: <20161206185806.GC31197@fieldses.org> References: <20161205151933.GA17517@fieldses.org> <20161205162559.GB17517@fieldses.org> <266c571f-e4e2-7c61-5ee2-8ece0c2d06e9@web.de> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: 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: 1483 Lines: 37 On Tue, Dec 06, 2016 at 02:18:31PM +0100, Andreas Gruenbacher wrote: > On Tue, Dec 6, 2016 at 11:08 AM, Miklos Szeredi wrote: > > On Tue, Dec 6, 2016 at 12:24 AM, Andreas Grünbacher > > wrote: > >> 2016-12-06 0:19 GMT+01:00 Andreas Grünbacher : > > > >>> It's not hard to come up with a heuristic that determines if a > >>> system.nfs4_acl value is equivalent to a file mode, and to ignore the > >>> attribute in that case. (The file mode is transmitted in its own > >>> attribute already, so actually converting .) That way, overlayfs could > >>> still fail copying up files that have an actual ACL. It's still an > >>> ugly hack ... > >> > >> Actually, that kind of heuristic would make sense in the NFS client > >> which could then hide the "system.nfs4_acl" attribute. > > > > Even simpler would be if knfsd didn't send the attribute if not > > necessary. Looks like there's code actively creating the nfs4_acl on > > the wire even if the filesystem had none: > > > > pacl = get_acl(inode, ACL_TYPE_ACCESS); > > if (!pacl) > > pacl = posix_acl_from_mode(inode->i_mode, GFP_KERNEL); > > > > What's the point? > > That's how the protocol is specified. Yep, even if we could make that change to nfsd it wouldn't help the client with the large number of other servers that are out there (including older knfsd's). --b. > (I'm not saying that that's very helpful.) > > Andreas