diff options
author | Adhemerval Zanella <adhemerval.zanella@linaro.org> | 2016-07-04 11:34:35 -0300 |
---|---|---|
committer | Adhemerval Zanella <adhemerval.zanella@linaro.org> | 2016-07-08 15:41:21 -0300 |
commit | 468700675f7f36d55758c36b1ed902250470e8ec (patch) | |
tree | 64a677daa16b8653843e48be613a07892533788c /misc/tst-preadvwritev64.c | |
parent | 6320de95367a6d5a7849d2a2ccdd9189277fc181 (diff) | |
download | glibc-468700675f7f36d55758c36b1ed902250470e8ec.tar.gz glibc-468700675f7f36d55758c36b1ed902250470e8ec.tar.xz glibc-468700675f7f36d55758c36b1ed902250470e8ec.zip |
Fix LO_HI_LONG definition
The p{read,write}v{64} consolidation patch [1] added a wrong guard for LO_HI_LONG definition. It currently uses both '__WORDSIZE == 64' and 'defined __ASSUME_WORDSIZE64_ILP32' to set the value to be passed in one argument, otherwise it will be split in two. However it fails on MIPS64n32 where syscalls n32 uses the compat implementation in the kernel meaning the off_t arguments are passed in two separate registers. GLIBC already defines a macro for such cases (__OFF_T_MATCHES_OFF64_T), so this patch uses it instead. Checked on x86_64, i686, x32, aarch64, armhf, and s390. * sysdeps/unix/sysv/linux/sysdep.h [__WORDSIZE == 64 || __ASSUME_WORDSIZE64_ILP32] (LO_HI_LONG): Remove guards. * misc/tst-preadvwritev-common.c: New file. * misc/tst-preadvwritev.c: Use tst-preadvwritev-common.c. * misc/tst-preadvwritev64.c: Use tst-preadwritev-common.c and add a check for files larger than 2GB. [1] 4751bbe2ad4d1bfa05774e29376d553ecfe563b0
Diffstat (limited to 'misc/tst-preadvwritev64.c')
-rw-r--r-- | misc/tst-preadvwritev64.c | 35 |
1 files changed, 32 insertions, 3 deletions
diff --git a/misc/tst-preadvwritev64.c b/misc/tst-preadvwritev64.c index ff6e134eab..53e153ed90 100644 --- a/misc/tst-preadvwritev64.c +++ b/misc/tst-preadvwritev64.c @@ -16,7 +16,36 @@ License along with the GNU C Library; if not, see <http://www.gnu.org/licenses/>. */ -#define PREADV preadv64 -#define PWRITEV pwritev64 +#define _FILE_OFFSET_BITS 64 +#include "tst-preadvwritev-common.c" -#include "tst-preadvwritev.c" +static int +do_test (void) +{ + int ret; + + ret = do_test_with_offset (0); + + /* Create a sparse file larger than 4GB to check if offset is handled + correctly in p{write,read}v64. */ + off_t base_offset = UINT32_MAX + 2048LL; + ret += do_test_with_offset (base_offset); + + struct stat st; + if (fstat (temp_fd, &st) == -1) + { + printf ("error: fstat on temporary file failed: %m"); + return 1; + } + + /* The total size should base_offset plus 2 * 96. */ + off_t expected_value = base_offset + (2 * (96LL)); + if (st.st_size != expected_value) + { + printf ("error: file size different than expected (%jd != %jd)\n", + (intmax_t) expected_value, (intmax_t) st.st_size); + return 1; + } + + return ret; +} |