On 12 October 2017 at 17:49, Laurent Vivier <laur...@vivier.eu> wrote:
> Le 12/10/2017 à 17:30, Peter Maydell a écrit :
>> +#if defined(TARGET_SPARC) || defined(TARGET_MIPS)
>> +typedef abi_long target_kernel_daddr_t;
>> +#else
>> +typedef abi_int target_kernel_daddr_t;
>> +#endif
>
> Perhaps you can add these ones into include/exec/user/abitypes.h ?
I don't think they belong there -- that file is for basic
CPU ABI dependent types, not things which are just part of
the kernel interface.
>> +struct target_mtget {
>> + abi_long mt_type;
>> + abi_long mt_resid;
>> + abi_long mt_dsreg;
>> + abi_long mt_gstat;
>> + abi_long mt_erreg;
>> + target_kernel_daddr_t mt_fileno;
>> + target_kernel_daddr_t mt_blkno;
>> +};
>
> I think you need to update STRUCT(mtget, ...) in
> linux-user/syscall_types.h to reflect the size difference for MIPS and
> SPARC.
I thought about that but I wasn't feeling too enthusiastic
due to not having a test case... I suppose it's better to
change them both though.
thanks
-- PMM
