Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1525526

Re: [PATCH 1/4] statx: Add a system call to make enhanced file info available [ver #2]

Show all headers | View raw

On Friday, November 18, 2016 4:20:38 PM CET David Howells wrote:
> Add a system call to make extended file information available, including
> file creation and some attribute flags where available through the
> underlying filesystem.

Looks great!

Is there still hope of scheduling this for v4.10, or are you
planning it for v4.11 at this point?

> ========
> OVERVIEW
> ========
> 
> The idea was initially proposed as a set of xattrs that could be retrieved
> with getxattr(), but the general preferance proved to be for a new syscall
> with an extended stat structure.
> 
> This has a number of uses:
...
> 
> Note that not all of the above are implemented here.

It's good to have the whole list of possible features here, but
the list of features that are actually implemented is probably
more important for the changelog. You seem to have that list
in the introductory mail but not in the actual patch.


> -       stat->atime = inode->i_atime;
> 
>         stat->mtime = inode->i_mtime;
>         stat->ctime = inode->i_ctime;
> 
> -       stat->blksize = (1 << inode->i_blkbits);
> +       stat->size = i_size_read(inode);
> 
>         stat->blocks = inode->i_blocks;
> 
> -}
> +       stat->blksize = 1 << inode->i_blkbits;
> 
> +       stat->result_mask |= STATX_BASIC_STATS;
> +       if (IS_NOATIME(inode))
> +               stat->result_mask &= ~STATX_ATIME;
> +       else
> +               stat->atime = inode->i_atime;

This looks like you are changing the behavior of the existing
stat call to no longer return the atime from the on-disk
inode if NOATIME is set.

Is my interpretation correct, and is this an intentional
change?

	Arnd

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[RFC][PATCH 0/4] Enhanced file stat system call [ver #2] David Howells <dhowells@redhat.com> - 2016-11-18 17:30 +0100
  [PATCH 3/4] statx: NFS: Return enhanced file attributes [ver #2] David Howells <dhowells@redhat.com> - 2016-11-18 17:30 +0100
  Re: [PATCH 1/4] statx: Add a system call to make enhanced file info available [ver #2] Arnd Bergmann <arnd@arndb.de> - 2016-11-18 18:00 +0100

csiph-web