This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
Other format: | [Raw text] |
On 18 Nov 2015 12:02, Adhemerval Zanella wrote: > This patch add two new macros to use along with off_t and off64_t argument > syscalls. The rationale for this change is: > > 1. Remove multiple implementations for the same syscall for different > architectures (for instance, pread have 6 different implementations). > > 2. Also remove the requirement to use syscall wrappers for cancellable > entrypoints. > > The macro usage should be used along __ALIGNMENT_ARG to follow ABI constrains > for architecture where it applies. For instance, pread can be rewritten as: > > return SYSCALL_CANCEL (pread, fd, buf, count, > __ALIGNMENT_ARG SYSCALL_LL (offset)); > > Another macro, SYSCALL_LL64, is provided for off64_t. > > The changes itself are not currently used in any implementation, so no > code change is expected. a quick scan looks like you're doing this purely along 32-bit/64-bit lines. why not add the 32-bit definition to sysdeps/unix/sysv/linux/sysdep.h and the 64-bit definition to sysdeps/unix/sysv/linux/wordsize-64/sysdep.h ? sysdeps/unix/sysv/linux/sysdep.h: #ifndef SYSCALL_LL # define SYSCALL_LL ... #endif sysdeps/unix/sysv/linux/wordsize-64/sysdep.h: #define SYSCALL_LL ... #include <sysdeps/unix/sysv/linux/sysdep.h> you would have to update all the 64-bit arches to pull in the wordsize-64 header since it looks like sysdep.h has a lot of direct includes rather than stacking & using include_next. -mike
Attachment:
signature.asc
Description: Digital signature
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |