| Message ID | 20240502165528.244004-1-richard.henderson@linaro.org |
|---|---|
| Headers | show |
| Series | target/sparc: vis fixes | expand |
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.
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.
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.
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(-)