Re: + preadv-pwritev-add-preadv-and-pwritev-system-calls.patch added to -mm tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Gerd Hoffmann wrote:
> H. Peter Anvin wrote:
>> akpm@xxxxxxxxxxxxxxxxxxxx wrote:
>>> This prototype has one problem though: On 32bit archs is the (64bit)
>>> offset argument unaligned, which the syscall ABI of several archs doesn't
>>> allow to do.  At least s390 needs a wrapper in glibc to handle this.  As
>>> we'll need a wrappers in glibc anyway I've decided to push problem to
>>> glibc entriely and use a syscall prototype which works without
>>> arch-specific wrappers inside the kernel: The offset argument is
>>> explicitly splitted into two 32bit values.
>> That rather sucks.  It'd be cleaner to just shuffle the argument order.
> 
> That was discussed too.  Doesn't solve the problem that you need some
> wrap-o-magic in glibc (this time to swap arguments instead of splitting
> the offset).  And is has the drawback that it is confusing to have
> different argument ordering at application and syscall level (think
> strace).  Check the archives for the details.
> 

A pointer would help.

I would say that different argument order in strace is a lot better than
having an argument completely mangled, which is what one gets with this
as proposed.

All in all, I think it is WRONG to make sane architectures suffer for
what broken architectures have to do, and we should implement this the
sane way without any shuffling, splitting, or other braindamage.

	-hpa

-- 
H. Peter Anvin, Intel Open Source Technology Center
I work for Intel.  I don't speak on their behalf.

--
To unsubscribe from this list: send the line "unsubscribe linux-api" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux