From: Steve French <smfrench@gmail.com> To: Andreas Dilger <adilger@dilger.ca> Cc: David Howells <dhowells@redhat.com>, linux-fsdevel <linux-fsdevel@vger.kernel.org>, linux-afs@vger.kernel.org, "linux-nfs@vger.kernel.org" <linux-nfs@vger.kernel.org>, samba-technical <samba-technical@lists.samba.org>, LKML <linux-kernel@vger.kernel.org>, "linux-ext4@vger.kernel.org" <linux-ext4@vger.kernel.org> Subject: Re: [PATCH 5/6] statx: Make windows attributes available for CIFS, NTFS and FAT to use Date: Mon, 3 Oct 2016 16:03:49 -0500 [thread overview] Message-ID: <CAH2r5mv4WSM==ACMO2OZxF-GJgMqKE2Y0T08f3JoWCX5SM0VmQ@mail.gmail.com> (raw) In-Reply-To: <D21EA8F8-0A7A-4275-8D4C-E9CFCC2A7293@dilger.ca> On Mon, May 2, 2016 at 5:52 PM, Andreas Dilger <adilger@dilger.ca> wrote: > On Apr 29, 2016, at 6:58 AM, David Howells <dhowells@redhat.com> wrote: >> >> Make windows attributes available for CIFS, NTFS and FAT to use in the >> statx struct. The attribute flags map directly by value to those in the >> CIFS PDU flags. Some of these bits can also be used by JFS, UFS and HPFS. >> >> The statx struct acquires: >> >> __u32 st_win_attrs; > > It seems some of these flags are duplicated with the st_information field, > and some are duplicate with FS_IOC_GETFLAGS values, and returning the same > information in multiple ways is confusing. > > If these flags are part of the CIFS protocol, and are directly usable by > Samba then I can understand we wouldn't want to change them once in the > kernel and then convert them back in userspace, but I'm a bit reluctant > to have flags only for CIFS/NTFS/FAT that might also be useful for other filesystems. Would we want to be able to get translated st_win_attrs > flags in ext4 attrs when it is being exported by Samba? It could be useful to have them in statx in an fs neutral way (and could be implemented by CIFS/NTFS/FAT and probably NFS), but I am ok with doing this via file system specific xattrs as we do now: NTFS has an ntfs specific pseudo-xattr for these and for birth time (and now cifs with recent patches can return the SMB3 inode attributes, including the older DOS attributes noted below). Samba server also stores them in an xattr (although they ndr encode them so it is not useful to us in the kernel) in local file systems. NFS v4 (and later) can also return (in theory) some of these flags but they are optional. >> The value in this is present if STATX_WIN_ATTRS is set. >> >> The defined flags in this are: >> >> STATX_WIN_ATTR_READONLY >> STATX_WIN_ATTR_HIDDEN >> STATX_WIN_ATTR_SYSTEM >> STATX_WIN_ATTR_DIRECTORY > > How does this differ from (st_mode & S_IFMT) == S_IFDIR)? > >> STATX_WIN_ATTR_ARCHIVE >> STATX_WIN_ATTR_NORMAL > > How does this differ from (st_mode & S_IFMT) == S_IFREG)? > >> STATX_WIN_ATTR_TEMPORARY > > How does this differ from STATX_INFO_TEMPORARY? > >> STATX_WIN_ATTR_SPARSE_FILE >> STATX_WIN_ATTR_REPARSE_POINT >> STATX_WIN_ATTR_COMPRESSED >> STATX_WIN_ATTR_OFFLINE >> STATX_WIN_ATTR_NOT_CONTENT_INDEXED >> STATX_WIN_ATTR_ENCRYPTED > > How does this differ from STATX_INFO_ENCRYPTED? As you noted, directory, compressed, encrypted overlap with existing posix file type (directory vs. file) and FS_IOC_FLAGS (although cifs.ko does not allow you to set encrypted yet as we do with compressed through the IOC flags, I could add a patch for setting the encrypted flag since it should be trivial as it does not affect reads/writes/open/close over the network fs since the file would be only encrypted at rest on the server fs) -- Thanks, Steve
WARNING: multiple messages have this Message-ID (diff)
From: Steve French <smfrench-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> To: Andreas Dilger <adilger-m1MBpc4rdrD3fQ9qLvQP4Q@public.gmane.org> Cc: David Howells <dhowells-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>, linux-fsdevel <linux-fsdevel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, linux-afs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, "linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" <linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, samba-technical <samba-technical-w/Ol4Ecudpl8XjKLYN78aQ@public.gmane.org>, LKML <linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, "linux-ext4-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" <linux-ext4-u79uwXL29TY76Z2rM5mHXA@public.gmane.org> Subject: Re: [PATCH 5/6] statx: Make windows attributes available for CIFS, NTFS and FAT to use Date: Mon, 3 Oct 2016 16:03:49 -0500 [thread overview] Message-ID: <CAH2r5mv4WSM==ACMO2OZxF-GJgMqKE2Y0T08f3JoWCX5SM0VmQ@mail.gmail.com> (raw) In-Reply-To: <D21EA8F8-0A7A-4275-8D4C-E9CFCC2A7293-m1MBpc4rdrD3fQ9qLvQP4Q@public.gmane.org> On Mon, May 2, 2016 at 5:52 PM, Andreas Dilger <adilger-m1MBpc4rdrD3fQ9qLvQP4Q@public.gmane.org> wrote: > On Apr 29, 2016, at 6:58 AM, David Howells <dhowells-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> wrote: >> >> Make windows attributes available for CIFS, NTFS and FAT to use in the >> statx struct. The attribute flags map directly by value to those in the >> CIFS PDU flags. Some of these bits can also be used by JFS, UFS and HPFS. >> >> The statx struct acquires: >> >> __u32 st_win_attrs; > > It seems some of these flags are duplicated with the st_information field, > and some are duplicate with FS_IOC_GETFLAGS values, and returning the same > information in multiple ways is confusing. > > If these flags are part of the CIFS protocol, and are directly usable by > Samba then I can understand we wouldn't want to change them once in the > kernel and then convert them back in userspace, but I'm a bit reluctant > to have flags only for CIFS/NTFS/FAT that might also be useful for other filesystems. Would we want to be able to get translated st_win_attrs > flags in ext4 attrs when it is being exported by Samba? It could be useful to have them in statx in an fs neutral way (and could be implemented by CIFS/NTFS/FAT and probably NFS), but I am ok with doing this via file system specific xattrs as we do now: NTFS has an ntfs specific pseudo-xattr for these and for birth time (and now cifs with recent patches can return the SMB3 inode attributes, including the older DOS attributes noted below). Samba server also stores them in an xattr (although they ndr encode them so it is not useful to us in the kernel) in local file systems. NFS v4 (and later) can also return (in theory) some of these flags but they are optional. >> The value in this is present if STATX_WIN_ATTRS is set. >> >> The defined flags in this are: >> >> STATX_WIN_ATTR_READONLY >> STATX_WIN_ATTR_HIDDEN >> STATX_WIN_ATTR_SYSTEM >> STATX_WIN_ATTR_DIRECTORY > > How does this differ from (st_mode & S_IFMT) == S_IFDIR)? > >> STATX_WIN_ATTR_ARCHIVE >> STATX_WIN_ATTR_NORMAL > > How does this differ from (st_mode & S_IFMT) == S_IFREG)? > >> STATX_WIN_ATTR_TEMPORARY > > How does this differ from STATX_INFO_TEMPORARY? > >> STATX_WIN_ATTR_SPARSE_FILE >> STATX_WIN_ATTR_REPARSE_POINT >> STATX_WIN_ATTR_COMPRESSED >> STATX_WIN_ATTR_OFFLINE >> STATX_WIN_ATTR_NOT_CONTENT_INDEXED >> STATX_WIN_ATTR_ENCRYPTED > > How does this differ from STATX_INFO_ENCRYPTED? As you noted, directory, compressed, encrypted overlap with existing posix file type (directory vs. file) and FS_IOC_FLAGS (although cifs.ko does not allow you to set encrypted yet as we do with compressed through the IOC flags, I could add a patch for setting the encrypted flag since it should be trivial as it does not affect reads/writes/open/close over the network fs since the file would be only encrypted at rest on the server fs) -- Thanks, Steve -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2016-10-03 21:04 UTC|newest] Thread overview: 76+ messages / expand[flat|nested] mbox.gz Atom feed top 2016-04-29 12:57 [RFC][PATCH 0/6] Enhanced file stat system call David Howells 2016-04-29 12:57 ` [PATCH 1/6] statx: Add a system call to make enhanced file info available David Howells 2016-05-02 22:46 ` Andreas Dilger 2016-05-02 22:46 ` Andreas Dilger 2016-05-03 15:53 ` David Howells 2016-05-04 22:56 ` Dave Chinner 2016-05-05 0:09 ` NeilBrown 2016-05-05 0:09 ` NeilBrown 2016-05-05 19:48 ` Jeff Layton 2016-05-06 18:07 ` J. Bruce Fields 2016-05-06 18:07 ` J. Bruce Fields 2016-05-05 20:04 ` David Howells 2016-05-05 20:04 ` David Howells 2016-05-06 1:39 ` Dave Chinner 2016-05-06 1:39 ` Dave Chinner 2016-05-06 1:39 ` Dave Chinner 2016-05-06 18:29 ` J. Bruce Fields 2016-05-09 1:45 ` Dave Chinner 2016-05-09 2:46 ` J. Bruce Fields 2016-05-04 23:56 ` NeilBrown 2016-05-08 8:35 ` Christoph Hellwig 2016-05-08 8:35 ` Christoph Hellwig 2016-05-09 12:02 ` Jeff Layton 2016-05-09 12:02 ` Jeff Layton 2016-05-10 7:00 ` Christoph Hellwig 2016-05-10 7:00 ` Christoph Hellwig 2016-05-10 13:21 ` Jeff Layton 2016-05-10 13:21 ` Jeff Layton 2016-05-09 12:57 ` David Howells 2016-05-09 12:57 ` David Howells 2016-05-09 13:23 ` Trond Myklebust 2016-05-09 13:23 ` Trond Myklebust 2016-05-09 13:23 ` Trond Myklebust 2016-05-10 7:04 ` Christoph Hellwig 2016-05-10 8:25 ` David Howells 2016-05-12 9:11 ` Christoph Hellwig 2016-05-13 15:28 ` Arnd Bergmann 2016-05-13 15:28 ` Arnd Bergmann 2016-05-23 8:22 ` Christoph Hellwig 2016-05-23 9:33 ` David Howells 2016-05-18 10:55 ` David Howells 2016-05-09 13:00 ` David Howells 2016-05-09 13:00 ` David Howells 2016-05-09 13:38 ` David Howells 2016-05-10 7:08 ` Christoph Hellwig 2016-05-10 8:43 ` David Howells 2016-05-12 9:12 ` Christoph Hellwig 2016-05-09 13:40 ` David Howells 2016-04-29 12:57 ` [PATCH 2/6] statx: AFS: Return enhanced file attributes David Howells 2016-04-29 12:57 ` [PATCH 3/6] statx: Ext4: " David Howells 2016-05-02 22:48 ` Andreas Dilger 2016-05-03 20:24 ` David Howells 2016-05-03 20:24 ` David Howells 2016-05-08 8:38 ` Christoph Hellwig 2016-05-08 8:38 ` Christoph Hellwig 2016-04-29 12:58 ` [PATCH 4/6] statx: NFS: " David Howells 2016-05-02 22:48 ` Andreas Dilger 2016-04-29 12:58 ` [PATCH 5/6] statx: Make windows attributes available for CIFS, NTFS and FAT to use David Howells 2016-05-02 22:52 ` Andreas Dilger 2016-10-03 21:03 ` Steve French [this message] 2016-10-03 21:03 ` Steve French 2016-05-03 20:23 ` David Howells 2016-05-08 8:39 ` Christoph Hellwig 2016-05-08 8:39 ` Christoph Hellwig 2016-04-29 12:58 ` [PATCH 6/6] statx: CIFS: Return enhanced attributes David Howells 2016-04-30 21:05 ` [RFC][PATCH 0/6] Enhanced file stat system call Jeff Layton 2016-04-30 21:05 ` Jeff Layton 2016-05-04 13:46 ` Arnd Bergmann 2016-05-04 13:46 ` Arnd Bergmann 2016-05-05 22:54 ` Steve French 2016-05-06 2:00 ` Steve French 2016-05-09 13:09 ` Arnd Bergmann 2016-05-09 13:09 ` Arnd Bergmann 2016-05-13 14:28 ` Richard Sharpe 2016-05-13 14:28 ` Richard Sharpe 2016-05-13 15:08 ` Arnd Bergmann
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to='CAH2r5mv4WSM==ACMO2OZxF-GJgMqKE2Y0T08f3JoWCX5SM0VmQ@mail.gmail.com' \ --to=smfrench@gmail.com \ --cc=adilger@dilger.ca \ --cc=dhowells@redhat.com \ --cc=linux-afs@vger.kernel.org \ --cc=linux-ext4@vger.kernel.org \ --cc=linux-fsdevel@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-nfs@vger.kernel.org \ --cc=samba-technical@lists.samba.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.