Re: [Qemu-devel] [Qemu-arm] [PATCH] target/arm: Fix broken VQADD and VQSUB causing abort

2019-05-21 Thread Torbjorn SVENSSON
Hi Peter,

Thanks for the fast response!
Great to hear that a patch for the issue is on its way into master. 

Kind regards,
Torbjörn

-Original Message-
From: Peter Maydell  
Sent: den 17 maj 2019 18:24
To: Torbjorn SVENSSON 
Cc: qemu-...@nongnu.org; Samuel HULTGREN ; 
qemu-devel@nongnu.org
Subject: Re: [Qemu-arm] [PATCH] target/arm: Fix broken VQADD and VQSUB causing 
abort

On Fri, 17 May 2019 at 17:04, Torbjorn SVENSSON  
wrote:
>
> In commit 89e68b575e138d0af1435f11a8ffcd8779c237bd, the handling of 
> VQADD and VQSUB was changed for Cortex-A and the new handling does not 
> return properly after calling tcg_gen_gvec_4(), thus the code after is 
> executed and that does not know about the VQADD or VQSUB instructions 
> and calls abort.
>
> Detected running GCC testsuite for Cortex-A7 and executing the tests 
> in QEMU using Cortex-A15 since Cortex-A7 and Cortex-A15 have similar 
> instruction set.
>
> Signed-off-by: Torbjörn SVENSSON 

Hi Torbjorn; thanks for this patch. As it happens there's already a fix for 
this on list:
 https://patchwork.ozlabs.org/patch/1100645/
so it should get into git master some time next week.

thanks
-- PMM


Re: [Qemu-devel] [Qemu-arm] [PATCH] target/arm: Fix broken VQADD and VQSUB causing abort

2019-05-17 Thread Peter Maydell
On Fri, 17 May 2019 at 17:04, Torbjorn SVENSSON
 wrote:
>
> In commit 89e68b575e138d0af1435f11a8ffcd8779c237bd, the handling of
> VQADD and VQSUB was changed for Cortex-A and the new handling does
> not return properly after calling tcg_gen_gvec_4(), thus the code
> after is executed and that does not know about the VQADD or VQSUB
> instructions and calls abort.
>
> Detected running GCC testsuite for Cortex-A7 and executing the tests
> in QEMU using Cortex-A15 since Cortex-A7 and Cortex-A15 have similar
> instruction set.
>
> Signed-off-by: Torbjörn SVENSSON 

Hi Torbjorn; thanks for this patch. As it happens there's already
a fix for this on list:
 https://patchwork.ozlabs.org/patch/1100645/
so it should get into git master some time next week.

thanks
-- PMM