[PATCH 3/4] Consolidate pread/pread64 implementations
Mike Frysinger
vapier@gentoo.org
Wed Feb 24 22:12:00 GMT 2016
On 23 Feb 2016 15:42, Adhemerval Zanella wrote:
> On 23-02-2016 14:57, Joseph Myers wrote:
> > Also, you're including sysdeps/unix/sysv/linux/generic/sysdep.h for
> > architectures that don't use the generic ABI. If that's safe, that
> > suggests it should just be merged into sysdeps/unix/sysv/linux/sysdep.h
> > (in a separate patch with a careful argument for why the merge is safe)
> > rather than keeping the headers separate.
>
> I decided to add it on generic sysdep.h because it is where port will get
> the definition of __ALIGNMENT_ARG. However I can split the second path
> in two, one to include generic sysdep.h and another to define SYSCALL_LL{64}.
i don't think splitting the path addresses the contention. Joseph's
point (which i agree with) is that linux/generic/ is for the "generic
ABI" which the kernel is trying to push for new arches. if you have
stuff which works across all linux ABI's, then instead of being in
linux/generic/xxx, it should be in linux/xxx.
if that means some things get hoisted out of linux/generic/xxx and
into linux/xxx, then that's OK.
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://sourceware.org/pipermail/libc-alpha/attachments/20160224/221aa485/attachment.sig>
More information about the Libc-alpha
mailing list