Re: [PATCH v7] Implement decoding of ustat syscall

2017-01-13 Thread Dmitry V. Levin
On Fri, Jan 13, 2017 at 10:41:06PM +0800, JingPiao Chen wrote: > >> +# ifdef HAVE_USTAT_H > >> +# include > >> +# else > >> +# include > >> +struct ustat { > >> + int32_t f_tfree; > >> + uint64_t f_tinode; > > > >This is not correct: > >f_tfree is long on mips and sparc, and int on othe

Re: [PATCH v7] Implement decoding of ustat syscall

2017-01-13 Thread JingPiao Chen
>> +# ifdef HAVE_USTAT_H >> +# include >> +# else >> +# include >> +struct ustat { >> + int32_t f_tfree; >> + uint64_t f_tinode; > >This is not correct: >f_tfree is long on mips and sparc, and int on other architectures; >f_tinode is unsigned int on alpha and s390, and kernel_ulong_t on

Re: [PATCH v7] Implement decoding of ustat syscall

2017-01-13 Thread Dmitry V. Levin
On Fri, Jan 13, 2017 at 10:01:41PM +0800, JingPiao Chen wrote: [...] > +# ifdef HAVE_USTAT_H > +# include > +# else > +# include > +struct ustat { > + int32_t f_tfree; > + uint64_t f_tinode; This is not correct: f_tfree is long on mips and sparc, and int on other architectures; f_tinod

[PATCH v7] Implement decoding of ustat syscall

2017-01-13 Thread JingPiao Chen
* ustat.c: New file. * configure.ac (AC_CHECK_HEADERS): Add ustat.h. * Makefile.am (strace_SOURCES): Add it. * linux/dummy.h: Remove. * tests/ustat.c: New file. * tests/ustat.test: New test. * tests/.gitignore: Add ustat. * tests/Makefile.am (check_PROGRAMS): Likewise. (DECODER_TESTS): Add ustat.te