Re: [RFC2 PATCH 00/23] ARM64: support ILP32

From: Adhemerval Zanella
Date: Tue Jun 28 2016 - 13:32:17 EST


Hi Yury,

Please address all previous reviews comment and concerns before send
a new RFC/patch set. As Joseph pointed out, he raised different question
in different patch iterations (some even back from 2014) and there is
no point is just resent the same patch without first review such
comments.

On 28/06/2016 13:39, Yury Norov wrote:
> This series enables aarch64 port with ilp32 mode.
>
> ABI details:
> - types are taken from AARCH32, next types turned to 64-bit,
> as modern requirement for new APIs tells:
> ino_t is u64 type
> off_t is s64 type
> blkcnt_t is s64 type
> fsblkcnt_t is u64 type
> fsfilcnt_t is u64 type
> - 64-bit arguments are passed in syscall as register pair,
> as kernel internally clears top halves for all input regs;
> - standard syscall table is used;
> - 32-bit time_t is used. AARCH64/ILP32 is waiting for general
> fix of Y2038 problem just like other 32-bit arches;
> - stat{64}, statfs{64} structures are of the identical layout
> with LP64. Corresponding syscalls are taken from 64-bit code
> - struct utmp, utmpx layouts are the same.
>
> v1: https://sourceware.org/ml/libc-alpha/2016-06/msg00730.html
> v2:
> - rebased on master;
> - dropped unneeded/unrelated patches;
> - pread family platform implementation removed;
> - addressed v1 comments (I'm really sorry if I missed something,
> there are a lot of them, and I am really thankfull for detailed review);
>
> Tested with LTP. Regressions are like this:
> abort01 FAIL 2
> kill11 FAIL 2
> mmap16 FAIL 6
> open12 FAIL 2
> rename11 FAIL 2
> rmdir02 FAIL 2
> umount2_01 FAIL 2
> umount2_02 FAIL 2
> umount2_03 FAIL 2
> utime06 FAIL 2
>
> It's better than v1, and there are ~5 additional regressions comparing to
> aarch64, all are related to filesystem.
>
> Andrew Pinski (17):
> [AARCH64] define word size for lp64 and ilp32
> [AARCH64] Add header guards to sysdep.h headers.
> Add dynamic ILP32 AARCH64 relocations to elf.h
> [AARCH64] Add PTR_REG, PTR_LOG_SIZE, and PTR_SIZE. Use it in
> LDST_PCREL and LDST_GLOBAL.
> [AARCH64] Use PTR_REG in crti.S.
> [AARCH64] Use PTR_REG/PTR_SIZE/PTR_SIZE_LOG in dl-tlsesc.S
> [AARCH64] Use PTR_* macros in dl-trampoline.S
> [AARCH64] Use PTR_* in start.S
> [AARCH64] Use PTR_REG in getcontext.S.
> [AARCH64] Detect ILP32 in configure scripts.
> [AARCH64] Add ILP32 support to elf_machine_load_address.
> [AARCH64] Add ILP32 to makefiles
> [AARCH64] Add support to ldconfig for ILP32 and libilp32
> [AARCH64] Add ILP32 ld.so to the known interpreter names.
> [AARCH64] Add ldd-rewrite.sed so that ilp32 ld.so can be found
> [AARCH64] Make lp64 and ilp32 directories.
> [AARCH64] Fix ILP32 warning
>
> Yury Norov (6):
> [AARCH64] ILP32: introduce syscalls that pass off_t
> [AARCH64] ILP32: support stat syscall family
> [AARCH64] delouse input arguments in system functions
> [AARCH64] Make __SIZEOF_SEM_T 16 for ILP32
> off_t: fix register pair calculation for 64-bit case
> [AARCH64] Take utmp{,x}.h from s390 port
>
> elf/cache.c | 3 +
> sysdeps/aarch64/Implies | 6 -
> sysdeps/aarch64/__longjmp.S | 6 +-
> sysdeps/aarch64/bits/wordsize.h | 25 +++
> sysdeps/aarch64/configure | 15 +-
> sysdeps/aarch64/configure.ac | 11 +-
> sysdeps/aarch64/crti.S | 3 +-
> sysdeps/aarch64/dl-irel.h | 3 +-
> sysdeps/aarch64/dl-machine.h | 199 ++++++++++++---------
> sysdeps/aarch64/dl-tlsdesc.S | 56 +++---
> sysdeps/aarch64/dl-trampoline.S | 18 +-
> sysdeps/aarch64/ilp32/Implies | 6 +
> sysdeps/aarch64/jmpbuf-unwind.h | 2 +-
> sysdeps/aarch64/lp64/Implies | 7 +
> sysdeps/aarch64/memcmp.S | 3 +
> sysdeps/aarch64/memcpy.S | 8 +
> sysdeps/aarch64/memset.S | 3 +
> sysdeps/aarch64/nptl/bits/semaphore.h | 4 +
> sysdeps/aarch64/preconfigure | 11 +-
> sysdeps/aarch64/setjmp.S | 5 +-
> sysdeps/aarch64/start.S | 20 ++-
> sysdeps/aarch64/strchr.S | 1 +
> sysdeps/aarch64/strchrnul.S | 1 +
> sysdeps/aarch64/strcmp.S | 2 +
> sysdeps/aarch64/strcpy.S | 2 +
> sysdeps/aarch64/strlen.S | 2 +
> sysdeps/aarch64/strncmp.S | 3 +
> sysdeps/aarch64/strnlen.S | 3 +
> sysdeps/aarch64/strrchr.S | 1 +
> sysdeps/aarch64/sysdep.h | 39 +++-
> sysdeps/generic/ldconfig.h | 1 +
> sysdeps/generic/stdint.h | 9 +-
> sysdeps/unix/sysv/linux/aarch64/Implies | 2 -
> sysdeps/unix/sysv/linux/aarch64/Makefile | 16 +-
> sysdeps/unix/sysv/linux/aarch64/bits/fcntl.h | 6 +-
> sysdeps/unix/sysv/linux/aarch64/bits/stat.h | 195 ++++++++++++++++++++
> sysdeps/unix/sysv/linux/aarch64/bits/statfs.h | 64 +++++++
> sysdeps/unix/sysv/linux/aarch64/bits/typesizes.h | 86 +++++++++
> sysdeps/unix/sysv/linux/aarch64/bits/utmp.h | 1 +
> sysdeps/unix/sysv/linux/aarch64/bits/utmpx.h | 1 +
> sysdeps/unix/sysv/linux/aarch64/clone.S | 7 +
> sysdeps/unix/sysv/linux/aarch64/configure | 24 ++-
> sysdeps/unix/sysv/linux/aarch64/configure.ac | 11 +-
> sysdeps/unix/sysv/linux/aarch64/dl-cache.h | 13 +-
> sysdeps/unix/sysv/linux/aarch64/getcontext.S | 3 +-
> sysdeps/unix/sysv/linux/aarch64/ilp32/Implies | 4 +
> .../unix/sysv/linux/aarch64/ilp32/dl-fxstatat64.c | 7 +
> sysdeps/unix/sysv/linux/aarch64/ilp32/dl-xstat64.c | 6 +
> sysdeps/unix/sysv/linux/aarch64/ilp32/fstatfs.c | 29 +++
> sysdeps/unix/sysv/linux/aarch64/ilp32/fstatfs64.c | 29 +++
> sysdeps/unix/sysv/linux/aarch64/ilp32/ftruncate.c | 1 +
> .../unix/sysv/linux/aarch64/ilp32/ftruncate64.c | 4 +
> sysdeps/unix/sysv/linux/aarch64/ilp32/fxstat.c | 51 ++++++
> sysdeps/unix/sysv/linux/aarch64/ilp32/fxstat64.c | 44 +++++
> sysdeps/unix/sysv/linux/aarch64/ilp32/fxstatat.c | 48 +++++
> sysdeps/unix/sysv/linux/aarch64/ilp32/fxstatat64.c | 52 ++++++
> sysdeps/unix/sysv/linux/aarch64/ilp32/getdents.c | 77 ++++++++
> sysdeps/unix/sysv/linux/aarch64/ilp32/getdents64.c | 1 +
> sysdeps/unix/sysv/linux/aarch64/ilp32/lseek.c | 36 ++++
> sysdeps/unix/sysv/linux/aarch64/ilp32/lxstat.c | 47 +++++
> sysdeps/unix/sysv/linux/aarch64/ilp32/lxstat64.c | 47 +++++
> sysdeps/unix/sysv/linux/aarch64/ilp32/mmap.c | 1 +
> sysdeps/unix/sysv/linux/aarch64/ilp32/mmap64.c | 1 +
> sysdeps/unix/sysv/linux/aarch64/ilp32/msgctl.c | 32 ++++
> .../unix/sysv/linux/aarch64/ilp32/posix_fadvise.c | 1 +
> .../sysv/linux/aarch64/ilp32/posix_fadvise64.c | 2 +
> sysdeps/unix/sysv/linux/aarch64/ilp32/readahead.c | 1 +
> sysdeps/unix/sysv/linux/aarch64/ilp32/readdir64.c | 1 +
> sysdeps/unix/sysv/linux/aarch64/ilp32/semctl.c | 53 ++++++
> .../unix/sysv/linux/aarch64/ilp32/shlib-versions | 7 +
> sysdeps/unix/sysv/linux/aarch64/ilp32/shmctl.c | 32 ++++
> sysdeps/unix/sysv/linux/aarch64/ilp32/statfs.c | 30 ++++
> sysdeps/unix/sysv/linux/aarch64/ilp32/statfs64.c | 29 +++
> .../unix/sysv/linux/aarch64/ilp32/syscalls.list | 0
> sysdeps/unix/sysv/linux/aarch64/ilp32/truncate.c | 1 +
> sysdeps/unix/sysv/linux/aarch64/ilp32/truncate64.c | 4 +
> sysdeps/unix/sysv/linux/aarch64/ilp32/xstat.c | 47 +++++
> sysdeps/unix/sysv/linux/aarch64/ilp32/xstat64.c | 47 +++++
> sysdeps/unix/sysv/linux/aarch64/ioctl.S | 31 ----
> sysdeps/unix/sysv/linux/aarch64/kernel-features.h | 27 +++
> sysdeps/unix/sysv/linux/aarch64/ldconfig.h | 2 +
> sysdeps/unix/sysv/linux/aarch64/ldd-rewrite.sed | 1 +
> sysdeps/unix/sysv/linux/aarch64/lp64/Implies | 4 +
> sysdeps/unix/sysv/linux/aarch64/lp64/ioctl.S | 31 ++++
> sysdeps/unix/sysv/linux/aarch64/lp64/mmap.c | 34 ++++
> .../unix/sysv/linux/aarch64/lp64/shlib-versions | 7 +
> sysdeps/unix/sysv/linux/aarch64/mmap.c | 34 ----
> sysdeps/unix/sysv/linux/aarch64/setcontext.S | 1 +
> sysdeps/unix/sysv/linux/aarch64/shlib-versions | 7 -
> sysdeps/unix/sysv/linux/aarch64/sigcontextinfo.h | 2 +-
> sysdeps/unix/sysv/linux/aarch64/swapcontext.S | 1 +
> sysdeps/unix/sysv/linux/aarch64/sysdep.h | 8 +-
> sysdeps/unix/sysv/linux/arm/readelflib.c | 4 +-
> sysdeps/unix/sysv/linux/fallocate.c | 4 +
> sysdeps/unix/sysv/linux/fallocate64.c | 4 +
> sysdeps/unix/sysv/linux/generic/brk.c | 2 +-
> sysdeps/unix/sysv/linux/posix_fadvise.c | 4 +
> sysdeps/unix/sysv/linux/posix_fadvise64.c | 4 +
> sysdeps/unix/sysv/linux/posix_fallocate.c | 4 +
> sysdeps/unix/sysv/linux/posix_fallocate64.c | 4 +
> 100 files changed, 1649 insertions(+), 250 deletions(-)
> delete mode 100644 sysdeps/aarch64/Implies
> create mode 100644 sysdeps/aarch64/bits/wordsize.h
> mode change 100644 => 100755 sysdeps/aarch64/configure
> create mode 100644 sysdeps/aarch64/ilp32/Implies
> create mode 100644 sysdeps/aarch64/lp64/Implies
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/bits/stat.h
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/bits/statfs.h
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/bits/typesizes.h
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/bits/utmp.h
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/bits/utmpx.h
> mode change 100644 => 100755 sysdeps/unix/sysv/linux/aarch64/configure
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/Implies
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/dl-fxstatat64.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/dl-xstat64.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/fstatfs.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/fstatfs64.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/ftruncate.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/ftruncate64.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/fxstat.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/fxstat64.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/fxstatat.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/fxstatat64.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/getdents.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/getdents64.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/lseek.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/lxstat.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/lxstat64.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/mmap.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/mmap64.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/msgctl.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/posix_fadvise.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/posix_fadvise64.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/readahead.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/readdir64.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/semctl.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/shlib-versions
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/shmctl.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/statfs.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/statfs64.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/syscalls.list
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/truncate.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/truncate64.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/xstat.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ilp32/xstat64.c
> delete mode 100644 sysdeps/unix/sysv/linux/aarch64/ioctl.S
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/kernel-features.h
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/ldd-rewrite.sed
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/lp64/Implies
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/lp64/ioctl.S
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/lp64/mmap.c
> create mode 100644 sysdeps/unix/sysv/linux/aarch64/lp64/shlib-versions
> delete mode 100644 sysdeps/unix/sysv/linux/aarch64/mmap.c
> delete mode 100644 sysdeps/unix/sysv/linux/aarch64/shlib-versions
>