[PATCH v2 0/8] Multiple ppc instructions fixes

Víctor Colombo posted 8 patches 1 year, 7 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20220906125523.38765-1-victor.colombo@eldorado.org.br
Maintainers: "Cédric Le Goater" <clg@kaod.org>, Daniel Henrique Barboza <danielhb413@gmail.com>, David Gibson <david@gibson.dropbear.id.au>, Greg Kurz <groug@kaod.org>
target/ppc/cpu.h        |  6 +-----
target/ppc/dfp_helper.c | 31 ++++++++++++++++++++++++++++---
target/ppc/fpu_helper.c | 39 +++++++++++++++++++++++++++------------
target/ppc/int_helper.c |  4 ++--
target/ppc/translate.c  |  8 ++++++++
5 files changed, 66 insertions(+), 22 deletions(-)
[PATCH v2 0/8] Multiple ppc instructions fixes
Posted by Víctor Colombo 1 year, 7 months ago
This patch set fixes multiple instructions for PPC targets that were
producing incorrect results, or setting the wrong bits in FPSCR.

Patch 1 is just a style fix, trivial.
Patch 8 adds helper_reset_fpstatus() calls to instructions
    that have an issue where the exception flags are being kept from
    the previous instruction, causing incorrect bits to be set,
    specially the non-sticky FI bit.
Other patches fixes other specific situations.

v1->v2:
- Squash patches 8 through 19 and write a better commit message to it.
- Dropped Daniel's R-b in the squashed patches, as the squash merged
    both reviewed and non-reviewed patches. Now require a new, single
    R-b.

Víctor Colombo (8):
  target/ppc: Remove extra space from s128 field in ppc_vsr_t
  target/ppc: Remove unused xer_* macros
  target/ppc: Zero second doubleword in DFP instructions
  target/ppc: Set result to QNaN for DENBCD when VXCVI occurs
  target/ppc: Zero second doubleword for VSX madd instructions
  target/ppc: Set OV32 when OV is set
  target/ppc: Zero second doubleword of VSR registers for FPR insns
  target/ppc: Clear fpstatus flags on helpers missing it

 target/ppc/cpu.h        |  6 +-----
 target/ppc/dfp_helper.c | 31 ++++++++++++++++++++++++++++---
 target/ppc/fpu_helper.c | 39 +++++++++++++++++++++++++++------------
 target/ppc/int_helper.c |  4 ++--
 target/ppc/translate.c  |  8 ++++++++
 5 files changed, 66 insertions(+), 22 deletions(-)

-- 
2.25.1


Re: [PATCH v2 0/8] Multiple ppc instructions fixes
Posted by Daniel Henrique Barboza 1 year, 7 months ago
Queued in gitlab.com/danielhb/qemu/tree/ppc-next. Thanks,


Daniel

On 9/6/22 09:55, Víctor Colombo wrote:
> This patch set fixes multiple instructions for PPC targets that were
> producing incorrect results, or setting the wrong bits in FPSCR.
> 
> Patch 1 is just a style fix, trivial.
> Patch 8 adds helper_reset_fpstatus() calls to instructions
>      that have an issue where the exception flags are being kept from
>      the previous instruction, causing incorrect bits to be set,
>      specially the non-sticky FI bit.
> Other patches fixes other specific situations.
> 
> v1->v2:
> - Squash patches 8 through 19 and write a better commit message to it.
> - Dropped Daniel's R-b in the squashed patches, as the squash merged
>      both reviewed and non-reviewed patches. Now require a new, single
>      R-b.
> 
> Víctor Colombo (8):
>    target/ppc: Remove extra space from s128 field in ppc_vsr_t
>    target/ppc: Remove unused xer_* macros
>    target/ppc: Zero second doubleword in DFP instructions
>    target/ppc: Set result to QNaN for DENBCD when VXCVI occurs
>    target/ppc: Zero second doubleword for VSX madd instructions
>    target/ppc: Set OV32 when OV is set
>    target/ppc: Zero second doubleword of VSR registers for FPR insns
>    target/ppc: Clear fpstatus flags on helpers missing it
> 
>   target/ppc/cpu.h        |  6 +-----
>   target/ppc/dfp_helper.c | 31 ++++++++++++++++++++++++++++---
>   target/ppc/fpu_helper.c | 39 +++++++++++++++++++++++++++------------
>   target/ppc/int_helper.c |  4 ++--
>   target/ppc/translate.c  |  8 ++++++++
>   5 files changed, 66 insertions(+), 22 deletions(-)
>