On 06/12/2018 02:44 PM, Richard Henderson wrote:
> On 06/12/2018 06:37 AM, Philippe Mathieu-Daudé wrote:
>>> /* Both preadv and pwritev merge args 4/5 into a 64-bit offset.
>>> * Moreover, the parts are *always* in little-endian order.
>>> */
>>> #if TARGET_ABI_BITS == 32
>>> SYSCALL_ARGS(preadv_pwritev)
>>> {
>>> - /* We have already assigned out[0-3]. */
>>> - abi_ulong lo = in[4], hi = in[5];
>>> - out[4] = ((hi << (TARGET_ABI_BITS - 1)) << 1) | lo;
>>> + /* We have already assigned out[0-2]. */
>>> + abi_ulong lo = in[3], hi = in[4];
>>> + out[3] = ((hi << (TARGET_ABI_BITS - 1)) << 1) | lo;
>> Why this change? If correct, can you update the comment?
>> Also this seems related to your previous patch #14: "Split out preadv,
>> pwritev, readv, writev", not this one.
>>
>
> You're right, this hunk should be folded back into the previous patch.
So for this patch without this hunk:
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
