[PATCH v6] Implement decoding of statx syscall

Dmitry V. Levin ldv at altlinux.org
Sun Mar 19 00:02:59 UTC 2017


On Sat, Mar 18, 2017 at 05:36:27PM +0300, Victor Krapivensky wrote:
> On Sat, Mar 18, 2017 at 10:44:09AM +0100, Eugene Syromyatnikov wrote:
[...]
> > Only one branch in regards of of stx.field.nsec value is covered in all
> > instances of this macro.
> 
> Yes. This seems to be a property of a filesystem. Anything I can do
> about that?

If the filesystem doesn't support non-zero nsec, I wouldn't bother.

> > Only branch with abbrev(tcp) == 0 is covered.
> 
> > ... and this one, but this is a common problem of all current *stat*
> > checks.
> 
> Should I add a test that checks that it abbreviates correctly?

If you ever in doubt whether to add a test or not, add a test. :)


-- 
ldv
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.strace.io/pipermail/strace-devel/attachments/20170319/be4200d3/attachment.bin>


More information about the Strace-devel mailing list