This is the mail archive of the 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]

Re: [PATCH 2/4] Consolidate off_t/off64_t syscall argument passing

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 ?

#ifndef SYSCALL_LL
# define SYSCALL_LL ...

#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.

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]