[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] target/ppc: Fix inequality check in do_lstxv_X
From: |
Fabiano Rosas |
Subject: |
Re: [PATCH] target/ppc: Fix inequality check in do_lstxv_X |
Date: |
Fri, 13 Sep 2024 08:32:36 -0300 |
Harsh Prateek Bora <harshpb@linux.ibm.com> writes:
> This fix was earlier introduced for do_lstxv_D form with 2cc0e449d173
> however got missed for _X form. This patch fixes the same.
>
> Cc: qemu-stable@nongnu.org
> Suggested-by: Fabiano Rosas <farosas@suse.de>
> Fixes: 70426b5bb738 ("target/ppc: moved stxvx and lxvx from legacy to
> decodtree")
> Signed-off-by: Harsh Prateek Bora <harshpb@linux.ibm.com>
> ---
> target/ppc/translate/vsx-impl.c.inc | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/target/ppc/translate/vsx-impl.c.inc
> b/target/ppc/translate/vsx-impl.c.inc
> index 40a87ddc4a..a869f30e86 100644
> --- a/target/ppc/translate/vsx-impl.c.inc
> +++ b/target/ppc/translate/vsx-impl.c.inc
> @@ -2244,7 +2244,7 @@ static bool do_lstxv_PLS_D(DisasContext *ctx, arg_PLS_D
> *a,
>
> static bool do_lstxv_X(DisasContext *ctx, arg_X *a, bool store, bool paired)
> {
> - if (paired || a->rt >= 32) {
> + if (paired || a->rt < 32) {
> REQUIRE_VSX(ctx);
> } else {
> REQUIRE_VECTOR(ctx);
Hi Harsh,
Seems I was quicker than you =)
20240911141651.6914-1-farosas@suse.de">https://lore.kernel.org/r/20240911141651.6914-1-farosas@suse.de
I'll give my RB and leave up to the maintainers which patch to take:
Reviewed-by: Fabiano Rosas <farosas@suse.de>