mbox series

[v2,0/7] target/sparc: vis fixes

Message ID 20240502165528.244004-1-richard.henderson@linaro.org
Headers show
Series target/sparc: vis fixes | expand

Message

Richard Henderson May 2, 2024, 4:55 p.m. UTC
Split out from my vis4 patch set, with just the bug fixes.
I've fixed the issue in patch 6, as noticed by Mark, but
include the follow-up that cleans up all of the macros by
removing them.


r~


Richard Henderson (7):
  linux-user/sparc: Add more hwcap bits for sparc64
  target/sparc: Fix FEXPAND
  target/sparc: Fix FMUL8x16
  target/sparc: Fix FMUL8x16A{U,L}
  target/sparc: Fix FMULD8*X16
  target/sparc: Fix FPMERGE
  target/sparc: Split out do_ms16b

 target/sparc/helper.h     |  11 +--
 target/sparc/insns.decode |   2 +-
 linux-user/elfload.c      |  48 +++++++---
 target/sparc/translate.c  | 129 ++++++++++++++++++++++---
 target/sparc/vis_helper.c | 195 ++++++++++----------------------------
 5 files changed, 207 insertions(+), 178 deletions(-)

Comments

Philippe Mathieu-Daudé May 3, 2024, 6:18 p.m. UTC | #1
On 2/5/24 18:55, Richard Henderson wrote:
> Split out from my vis4 patch set, with just the bug fixes.
> I've fixed the issue in patch 6, as noticed by Mark, but
> include the follow-up that cleans up all of the macros by
> removing them.
> 
> 
> r~
> 
> 
> Richard Henderson (7):
>    linux-user/sparc: Add more hwcap bits for sparc64
>    target/sparc: Fix FEXPAND
>    target/sparc: Fix FMUL8x16
>    target/sparc: Fix FMUL8x16A{U,L}
>    target/sparc: Fix FMULD8*X16
>    target/sparc: Fix FPMERGE
>    target/sparc: Split out do_ms16b

I'm wondering about the coverage here, since various patches
fix bugs since VIS intro in commit e9ebed4d41 from 2007, so
being broken for 17 years.
Mark Cave-Ayland May 5, 2024, 8:05 p.m. UTC | #2
On 02/05/2024 17:55, Richard Henderson wrote:

> Split out from my vis4 patch set, with just the bug fixes.
> I've fixed the issue in patch 6, as noticed by Mark, but
> include the follow-up that cleans up all of the macros by
> removing them.
>  
> r~
> 
> 
> Richard Henderson (7):
>    linux-user/sparc: Add more hwcap bits for sparc64
>    target/sparc: Fix FEXPAND
>    target/sparc: Fix FMUL8x16
>    target/sparc: Fix FMUL8x16A{U,L}
>    target/sparc: Fix FMULD8*X16
>    target/sparc: Fix FPMERGE
>    target/sparc: Split out do_ms16b
> 
>   target/sparc/helper.h     |  11 +--
>   target/sparc/insns.decode |   2 +-
>   linux-user/elfload.c      |  48 +++++++---
>   target/sparc/translate.c  | 129 ++++++++++++++++++++++---
>   target/sparc/vis_helper.c | 195 ++++++++++----------------------------
>   5 files changed, 207 insertions(+), 178 deletions(-)

Thanks Richard. I've applied v2 to my qmeu-sparc branch, and will send a PR assuming 
everything passes in GitLab.


ATB,

Mark.
Mark Cave-Ayland May 5, 2024, 8:13 p.m. UTC | #3
On 03/05/2024 19:18, Philippe Mathieu-Daudé wrote:

> On 2/5/24 18:55, Richard Henderson wrote:
>> Split out from my vis4 patch set, with just the bug fixes.
>> I've fixed the issue in patch 6, as noticed by Mark, but
>> include the follow-up that cleans up all of the macros by
>> removing them.
>>
>>
>> r~
>>
>>
>> Richard Henderson (7):
>>    linux-user/sparc: Add more hwcap bits for sparc64
>>    target/sparc: Fix FEXPAND
>>    target/sparc: Fix FMUL8x16
>>    target/sparc: Fix FMUL8x16A{U,L}
>>    target/sparc: Fix FMULD8*X16
>>    target/sparc: Fix FPMERGE
>>    target/sparc: Split out do_ms16b
> 
> I'm wondering about the coverage here, since various patches
> fix bugs since VIS intro in commit e9ebed4d41 from 2007, so
> being broken for 17 years.

I've definitely seen the VIS instructions in use in my test images, however I can't 
recall exactly whether it was the particular ones fixed in this series. I'm certainly 
keen for some more VIS instruction coverage though, especially for VIS2 and later 
which I'm unlikely to come across in my day-to-day testing.


ATB,

Mark.