qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Richard Henderson <richard.henderson@linaro.org>
To: deller@kernel.org, qemu-devel@nongnu.org
Cc: Michael Tokarev <mjt@tls.msk.ru>, Bruno Haible <bruno@clisp.org>,
	"Nelson H . F . Beebe" <beebe@math.utah.edu>,
	Helge Deller <deller@gmx.de>
Subject: Re: [PATCH v2 6/9] target/hppa: Strip upper 32-bits of IOR on unaligned access error
Date: Tue, 9 Jan 2024 21:05:31 +1100	[thread overview]
Message-ID: <679c70ac-b25f-45e3-a74f-64e322ce886c@linaro.org> (raw)
In-Reply-To: <20240107132237.50553-7-deller@kernel.org>

On 1/8/24 00:22, deller@kernel.org wrote:
> From: Helge Deller <deller@gmx.de>
> 
> Limit IOR to the lower 32-bits on failure.
> Keep patch short for easier backporting.
> 
> Signed-off-by: Helge Deller <deller@gmx.de>
> ---
>   target/hppa/cpu.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/target/hppa/cpu.c b/target/hppa/cpu.c
> index 04de1689d7..3c384d5855 100644
> --- a/target/hppa/cpu.c
> +++ b/target/hppa/cpu.c
> @@ -112,7 +112,7 @@ void hppa_cpu_do_unaligned_access(CPUState *cs, vaddr addr,
>       cs->exception_index = EXCP_UNALIGN;
>       if (env->psw & PSW_Q) {
>           /* ??? Needs tweaking for hppa64.  */
> -        env->cr[CR_IOR] = addr;
> +        env->cr[CR_IOR] = (uint32_t)addr;
>           env->cr[CR_ISR] = addr >> 32;
>       }
>   

Likewise, might need raise_exception_with_ior.

r~


  reply	other threads:[~2024-01-09 10:05 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-01-07 13:22 [PATCH v2 0/9] target/hppa qemu v8.2 regression fixes deller
2024-01-07 13:22 ` [PATCH v2 1/9] hw/hppa/machine: Allow up to 3840 MB total memory deller
2024-01-09  9:21   ` Richard Henderson
2024-01-07 13:22 ` [PATCH v2 2/9] hw/hppa/machine: Disable default devices with --nodefaults option deller
2024-01-09  9:57   ` Richard Henderson
2024-01-09 11:16     ` Helge Deller
2024-01-09 16:01       ` Richard Henderson
2024-01-11 22:28         ` Helge Deller
2024-01-12  5:09           ` Thomas Huth
2024-01-12  7:52             ` Helge Deller
2024-01-07 13:22 ` [PATCH v2 3/9] hw/pci-host/astro: Add missing astro & elroy registers for NetBSD deller
2024-01-07 13:22 ` [PATCH v2 4/9] target/hppa: Fix PDC address translation on PA2.0 with PSW.W=0 deller
2024-01-09  9:14   ` Richard Henderson
2024-01-09 11:22     ` Helge Deller
2024-01-09 16:18       ` Richard Henderson
2024-01-09 21:06         ` Helge Deller
2024-01-09 21:54           ` Richard Henderson
2024-01-07 13:22 ` [PATCH v2 5/9] target/hppa: Strip upper 32-bits of IOR on error in probe deller
2024-01-09 10:04   ` Richard Henderson
2024-01-07 13:22 ` [PATCH v2 6/9] target/hppa: Strip upper 32-bits of IOR on unaligned access error deller
2024-01-09 10:05   ` Richard Henderson [this message]
2024-01-07 13:22 ` [PATCH v2 7/9] hw/hppa: Move software power button address back into PDC deller
2024-01-07 13:22 ` [PATCH v2 8/9] target/hppa: Avoid accessing %gr0 when raising exception deller
2024-01-09 10:06   ` Richard Henderson
2024-01-07 13:22 ` [PATCH v2 9/9] target/hppa: Update SeaBIOS-hppa to version 15 deller
2024-01-07 13:49 ` [PATCH v2 0/9] target/hppa qemu v8.2 regression fixes Bruno Haible

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=679c70ac-b25f-45e3-a74f-64e322ce886c@linaro.org \
    --to=richard.henderson@linaro.org \
    --cc=beebe@math.utah.edu \
    --cc=bruno@clisp.org \
    --cc=deller@gmx.de \
    --cc=deller@kernel.org \
    --cc=mjt@tls.msk.ru \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).