[PATCH] Add the statx function

[Florian Weimer]

On 07/05/2018 08:36 PM, Paul Eggert wrote:
> Florian Weimer wrote:
> 
>> +Â  *buf = (struct statx)
>> +Â Â Â  {
>> +Â Â Â Â Â  /* We copy everything from fstat64, which corresponds the basic
>> +         fstat64.  */
>> +Â Â Â Â Â  .stx_mask = STATX_BASIC_STATS,
> 
> That assignment to *BUF unnecessarily clears all *BUF fields not 
> mentioned. It should be a bit more efficient to have a separate 
> assignment for each mentioned field, e.g., 'buf->stx_mask = 
> STATX_BASIC_STATS;'.

Clearing all padding fields is part of the userspace interface.  It's 
described in the UAPI header file, and the kernel implements that.

I've updated the comment.

I also fixed a C&P mistake in the major/minor extraction.

>> +int statx (int __dirfd, const char *__path, int __flags,
>> +Â Â Â Â Â Â Â Â Â Â  unsigned int __mask, struct statx *__buf)
>> +Â  __THROW __nonnull ((2, 5));
> 
> The two pointer parameters should both be declared with __restrict.

Okay, I've added __restrict qualifiers.

As Andreas suggested, I've added STATX_ALL and STATX__RESERVED.

Thanks,
Florian
-------------- next part --------------
A non-text attachment was scrubbed...
Name: statx.patch
Type: text/x-patch
Size: 33049 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/libc-alpha/attachments/20180705/55070c96/attachment.bin>