Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1391684
| From | Jeff Layton <jlayton@poochiereds.net> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [RFC][PATCH 0/6] Enhanced file stat system call |
| Date | 2016-04-30 23:10 +0200 |
| Message-ID | <rtKjg-69D-5@gated-at.bofh.it> (permalink) |
| References | <rtgbv-69i-7@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On Fri, 2016-04-29 at 13:57 +0100, David Howells wrote: > Implement a new system call to provide enhanced file stats. The patches can > be found here: > > http://git.kernel.org/cgit/linux/kernel/git/dhowells/linux-fs.git/log/?h=xstat > > > =========== > DESCRIPTION > =========== > > The first patch provides this new system call: > > long ret = statx(int dfd, > const char *filename, > unsigned atflag, > unsigned mask, > struct statx *buffer); > > This is an enhanced file stat function that provides a number of useful > features, in summary: > > (1) More information: creation time, data version number, inode generation > number and flags. A subset of these is available through a number of > filesystems (such as CIFS, NFS, AFS, Ext4 and BTRFS). > > (2) Lightweight stat (AT_NO_ATTR_SYNC): Ask for just those details of > interest, and allow a network fs to approximate anything not of > interest, without going to the server. > > (3) Heavyweight stat (AT_FORCE_ATTR_SYNC): Force a network fs to flush > buffers and go to the server, even if it thinks its cached attributes > are up to date. > > (4) Allow the filesystem to indicate what it can/cannot provide: A > filesystem can now say it doesn't support a standard stat feature if > that isn't available. > > (5) Make the fields a consistent size on all arches, and make them large. > > (6) Can be extended by using more request flags and using up the padding > space in the statx struct. > > Note that no lstat() equivalent is required as that can be implemented > through statx() with atflag == 0. There is also no fstat() equivalent as > that can be implemented through statx() with filename == NULL and the > relevant fd passed as dfd. > > > ======= > TESTING > ======= > > A test program is added into samples/statx/ by the first patch. > > David > --- > David Howells (6): > statx: Add a system call to make enhanced file info available > statx: AFS: Return enhanced file attributes > statx: Ext4: Return enhanced file attributes > statx: NFS: Return enhanced file attributes > statx: Make windows attributes available for CIFS, NTFS and FAT to use > statx: CIFS: Return enhanced attributes > > > arch/x86/entry/syscalls/syscall_32.tbl | 1 > arch/x86/entry/syscalls/syscall_64.tbl | 1 > fs/afs/inode.c | 24 ++- > fs/cifs/cifsfs.h | 4 > fs/cifs/cifsglob.h | 8 + > fs/cifs/dir.c | 2 > fs/cifs/inode.c | 117 +++++++++--- > fs/exportfs/expfs.c | 4 > fs/ext4/ext4.h | 2 > fs/ext4/file.c | 2 > fs/ext4/inode.c | 30 +++ > fs/ext4/namei.c | 2 > fs/ext4/symlink.c | 2 > fs/nfs/inode.c | 41 ++++ > fs/stat.c | 306 +++++++++++++++++++++++++++++--- > include/linux/fs.h | 5 - > include/linux/stat.h | 15 +- > include/linux/syscalls.h | 4 > include/uapi/linux/fcntl.h | 2 > include/uapi/linux/stat.h | 135 ++++++++++++++ > samples/Makefile | 2 > samples/statx/Makefile | 10 + > samples/statx/test-statx.c | 274 +++++++++++++++++++++++++++++ > 23 files changed, 910 insertions(+), 83 deletions(-) > create mode 100644 samples/statx/Makefile > create mode 100644 samples/statx/test-statx.c > I looked over the patchset and it looks fairly straightforward to me. Are there things that I'd have probably done differently? Yes, but nothing really sticks out to me as show-stopping and this has the potential to be helpful in all sorts of ways. I think we really ought to resist excessive bikeshedding this time around. With the way the interface is designed, we should be able to expand on it later if we want to add new attributes after the interface itself is merged. Reviewed-by: Jeff Layton <jlayton@poochiereds.net>
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[RFC][PATCH 0/6] Enhanced file stat system call David Howells <dhowells@redhat.com> - 2016-04-29 15:00 +0200
[PATCH 4/6] statx: NFS: Return enhanced file attributes David Howells <dhowells@redhat.com> - 2016-04-29 15:00 +0200
Re: [PATCH 4/6] statx: NFS: Return enhanced file attributes Andreas Dilger <adilger@dilger.ca> - 2016-05-03 00:50 +0200
[PATCH 6/6] statx: CIFS: Return enhanced attributes David Howells <dhowells@redhat.com> - 2016-04-29 15:00 +0200
[PATCH 3/6] statx: Ext4: Return enhanced file attributes David Howells <dhowells@redhat.com> - 2016-04-29 15:00 +0200
Re: [PATCH 3/6] statx: Ext4: Return enhanced file attributes Andreas Dilger <adilger@dilger.ca> - 2016-05-03 00:50 +0200
Re: [PATCH 3/6] statx: Ext4: Return enhanced file attributes David Howells <dhowells@redhat.com> - 2016-05-03 22:30 +0200
Re: [PATCH 3/6] statx: Ext4: Return enhanced file attributes Christoph Hellwig <hch@infradead.org> - 2016-05-08 10:40 +0200
[PATCH 5/6] statx: Make windows attributes available for CIFS, NTFS and FAT to use David Howells <dhowells@redhat.com> - 2016-04-29 15:10 +0200
Re: [PATCH 5/6] statx: Make windows attributes available for CIFS, NTFS and FAT to use Andreas Dilger <adilger@dilger.ca> - 2016-05-03 01:00 +0200
Re: [PATCH 5/6] statx: Make windows attributes available for CIFS, NTFS and FAT to use David Howells <dhowells@redhat.com> - 2016-05-03 22:30 +0200
Re: [PATCH 5/6] statx: Make windows attributes available for CIFS, NTFS and FAT to use Christoph Hellwig <hch@infradead.org> - 2016-05-08 10:40 +0200
[PATCH 2/6] statx: AFS: Return enhanced file attributes David Howells <dhowells@redhat.com> - 2016-04-29 15:20 +0200
Re: [RFC][PATCH 0/6] Enhanced file stat system call Jeff Layton <jlayton@poochiereds.net> - 2016-04-30 23:10 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available David Howells <dhowells@redhat.com> - 2016-05-03 18:00 +0200
Re: [RFC][PATCH 0/6] Enhanced file stat system call Arnd Bergmann <arnd@arndb.de> - 2016-05-04 15:50 +0200
Re: [RFC][PATCH 0/6] Enhanced file stat system call Steve French <smfrench@gmail.com> - 2016-05-06 04:10 +0200
Re: [RFC][PATCH 0/6] Enhanced file stat system call Arnd Bergmann <arnd@arndb.de> - 2016-05-09 15:10 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available Dave Chinner <david@fromorbit.com> - 2016-05-05 01:00 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available NeilBrown <nfbrown@novell.com> - 2016-05-05 02:20 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available Jeff Layton <jlayton@poochiereds.net> - 2016-05-05 21:50 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available David Howells <dhowells@redhat.com> - 2016-05-05 22:10 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available Dave Chinner <david@fromorbit.com> - 2016-05-06 03:50 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available bfields@fieldses.org (J. Bruce Fields) - 2016-05-06 20:10 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available bfields@fieldses.org (J. Bruce Fields) - 2016-05-06 20:30 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available Dave Chinner <david@fromorbit.com> - 2016-05-09 03:50 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available "J. Bruce Fields" <bfields@fieldses.org> - 2016-05-09 04:50 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available NeilBrown <nfbrown@novell.com> - 2016-05-05 02:00 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available Christoph Hellwig <hch@infradead.org> - 2016-05-08 10:40 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available Jeff Layton <jlayton@poochiereds.net> - 2016-05-09 14:10 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available Christoph Hellwig <hch@infradead.org> - 2016-05-10 09:10 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available Jeff Layton <jlayton@poochiereds.net> - 2016-05-10 15:30 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available David Howells <dhowells@redhat.com> - 2016-05-09 15:00 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available David Howells <dhowells@redhat.com> - 2016-05-09 15:10 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available Trond Myklebust <trondmy@primarydata.com> - 2016-05-09 15:40 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available Christoph Hellwig <hch@infradead.org> - 2016-05-10 09:10 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available David Howells <dhowells@redhat.com> - 2016-05-10 10:30 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available Christoph Hellwig <hch@infradead.org> - 2016-05-12 11:20 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available David Howells <dhowells@redhat.com> - 2016-05-09 15:40 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available Christoph Hellwig <hch@infradead.org> - 2016-05-10 09:10 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available David Howells <dhowells@redhat.com> - 2016-05-10 10:50 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available Christoph Hellwig <hch@infradead.org> - 2016-05-12 11:20 +0200
Re: [PATCH 1/6] statx: Add a system call to make enhanced file info available David Howells <dhowells@redhat.com> - 2016-05-09 15:50 +0200
csiph-web