Thanks a lot for the review, Laurent, I am going to add TARGET_SH4 and 
TARGET_ALPHA to this change.

I test LTP syscall test set on mips, arm and intel platforms as my regression 
tests, but also some number of individual LTP tests (the choice of tests 
depends on the change) on more platforms, and minimal test examples 
illustrating the bug and the fix as well.

Aleksandar
________________________________________
From: Laurent Vivier [laur...@vivier.eu]
Sent: Friday, September 16, 2016 11:22 AM
To: Aleksandar Markovic; qemu-devel@nongnu.org; riku.voi...@iki.fi; 
peter.mayd...@linaro.org; aurel...@aurel32.net; Leon Alrae; Petar Jovanovic; 
Miodrag Dinic; Aleksandar Rikalo; Aleksandar Markovic
Subject: Re: [Qemu-devel] [PATCH v6 1/7] linux-user: Fix TARGET_SIOCATMARK 
definition for Mips

Le 16/09/2016 à 13:14, Aleksandar Markovic a écrit :
> From: Aleksandar Markovic <aleksandar.marko...@imgtec.com>
>
> This patch fixes wrong definition of TARGET_SIOCATMARK for Mips.
>
> The current definition is:
>
>   #define SIOCATMARK      0x8905
>
> while the correct definition is:
>
>   #define SIOCATMARK      TARGET_IOR('s', 7, int)

According to linux sources, it seems true also for alpha and sh4.

Perhaps you can add them to the patch?
Do you have something to test it doesn't break anything?

Thanks,
Laurent

Reply via email to