From: Steve French Subject: Re: [PATCH v18 00/22] Richacls (Core and Ext4) Date: Tue, 15 Mar 2016 22:40:00 -0500 Message-ID: References: <1456733847-17982-1-git-send-email-agruenba@redhat.com> <20160311140134.GA14808@infradead.org> <20160315071439.GE19747@infradead.org> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Cc: Andreas Gruenbacher , Alexander Viro , "J. Bruce Fields" , Linux NFS Mailing List , "Theodore Ts'o" , "linux-cifs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Linux API , Trond Myklebust , LKML , XFS Developers , Andreas Dilger , linux-fsdevel , Jeff Layton , linux-ext4 , Anna Schumaker To: Christoph Hellwig Return-path: In-Reply-To: <20160315071439.GE19747-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org> Sender: linux-api-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-ext4.vger.kernel.org On Tue, Mar 15, 2016 at 2:14 AM, Christoph Hellwig wrote: > On Fri, Mar 11, 2016 at 02:05:16PM -0600, Steve French wrote: >> A loosely related question is what can be done for tools around existing >> interfaces for ACLs. I recently found out NTFS-3g has this xattr: >> >> static const char nf_ns_xattr_ntfs_acl[] = "system.ntfs_acl"; >> >> which allows you to query system.ntfs_acl xattr to get their full ACL > > Bah. Filesystems really have no business exposing random system xattrs, > and we really need to add a filter to fs/xattr.c to not expose > arbitrary attrs ouside the user.* prefix. Hopefully we don't consider them random system xattrs, it is plausible that ntfs uses these for user space tools that I don't have. At least for cifs.ko a similar subset (querying ACLs, streams and reparse info e.g.) to the ntfs set would be very helpful. For example, Being able to query the actual ACL over the wire, is important for backup and for debug, the only question is whether to do it via an xattr (possibly being able to have some synergy with existing ntfs-3g tools) or an ioctl (since adding an NTFS specific syscall for a couple fs doesn't make sense). For the specific example of the odd ntfs.streams.list xattr, I can see why they have it. I would have mixed feelings about having no way to tell streams and EAs from each other since NTFS-3g displaying streams as xattrs and also Extended Attributes (EAs) as xattrs (and if they didn't have an additional xattr to list streams) without a way to tell the difference (at least a system xattr to list the alternate data streams is useful). There is useful information in alternate data streams that backup (and debugging) programs need for some workloads, for example the origin (where internet explorer downloaded a file from) and file classification information. -- Thanks, Steve