On Fri, Jan 13, 2017 at 05:33:07AM +0300, Dmitry V. Levin wrote:
> On Wed, Jan 11, 2017 at 02:25:56PM +0000, James Cowgill wrote:
> > The newfstatat testcase on mips64 currently fails because:
> > - The BOGUS_STRUCT_STAT test segfaults inside glibc.
> > - The result of the fstatat call gives incorrect dates because the
> >   kernel struct stat uses unsigned int timestamps.
> > 
> > Fix by using avoiding the glibc wrapper and using the relevant syscall
> > directly. This obviously avoids the first problem, and avoids the second
> > problem because print_stat always sign extends dates (unlike glibc which
> > will zero extend them).
> 
> Unfortunately, this makes fstatat64.test fail on x86
> because of struct stat mismatch.
> 
> If tests/fstatat.c is changed to define USE_ASM_STAT, then
> tests/fstatat64.c would need more stat64 related definitions
> (like in tests/fstat64.c), and I cannot tell off-hand whether
> fstatat64 takes struct stat64 on all platforms like it's declared
> in kernel's include/linux/syscalls.h file.
> 
> btw, tests/fstatat.c is the last user of tests/xstatx.c that
> does not define USE_ASM_STAT yet.

OK, I've changed both newfstatat and fstatat64 tests to invoke syscalls
directly.


-- 
ldv

Attachment: pgpQ_oR4dEjhi.pgp
Description: PGP signature

------------------------------------------------------------------------------
Developer Access Program for Intel Xeon Phi Processors
Access to Intel Xeon Phi processor-based developer platforms.
With one year of Intel Parallel Studio XE.
Training and support from Colfax.
Order your platform today. http://sdm.link/xeonphi
_______________________________________________
Strace-devel mailing list
Strace-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/strace-devel

Reply via email to