[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] fix disabling interrupts in sun4u
From: |
tsnsaito |
Subject: |
Re: [Qemu-devel] [PATCH] fix disabling interrupts in sun4u |
Date: |
Thu, 28 Jul 2011 19:31:16 +0900 |
User-agent: |
Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (Goj$(D+W(B) APEL/10.8 Emacs/23.3 (i386-portbld-freebsd8.2) MULE/6.0 (HANACHIRUSATO) |
Hi,
At Mon, 25 Jul 2011 19:22:38 +0200,
Artyom Tarasenko wrote:
> clear interrupt request if the interrupt priority < CPU pil
> clear hardware interrupt request if interrupts are disabled
Not directly related to the fix, but I'd like to note a problem
of hw/sun4u.c interrupt code:
The interrupt code probably mixes hardware interrupts and
software interrupts.
%pil is for software interrupts (interrupt_level_n traps).
%pil can not mask hardware interrupts (interrupt_vector traps);
the CPU raises interrupt_vector traps even on %pil=15.
But in cpu_check_irqs() and cpu_set_irq(), hardware interrupts
seem to be masked by %pil.
> Signed-off-by: Artyom Tarasenko <address@hidden>
> ---
> hw/sun4u.c | 6 ++++--
> 1 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/hw/sun4u.c b/hw/sun4u.c
> index d7dcaf0..7f95aeb 100644
> --- a/hw/sun4u.c
> +++ b/hw/sun4u.c
> @@ -255,7 +255,7 @@ void cpu_check_irqs(CPUState *env)
> pil |= 1 << 14;
> }
>
> - if (!pil) {
> + if (pil < (2 << env->psrpil)){
> if (env->interrupt_request & CPU_INTERRUPT_HARD) {
> CPUIRQ_DPRINTF("Reset CPU IRQ (current interrupt %x)\n",
> env->interrupt_index);
> @@ -287,10 +287,12 @@ void cpu_check_irqs(CPUState *env)
> break;
> }
> }
> - } else {
> + } else if (env->interrupt_request & CPU_INTERRUPT_HARD) {
> CPUIRQ_DPRINTF("Interrupts disabled, pil=%08x pil_in=%08x
> softint=%08x "
> "current interrupt %x\n",
> pil, env->pil_in, env->softint, env->interrupt_index);
> + env->interrupt_index = 0;
> + cpu_reset_interrupt(env, CPU_INTERRUPT_HARD);
> }
> }
>
> --
> 1.7.3.4
----
Tsuneo Saito <address@hidden>
- [Qemu-devel] [PATCH] fix disabling interrupts in sun4u, Artyom Tarasenko, 2011/07/25
- Re: [Qemu-devel] [PATCH] fix disabling interrupts in sun4u,
tsnsaito <=
- Re: [Qemu-devel] [PATCH] fix disabling interrupts in sun4u, Artyom Tarasenko, 2011/07/28
- Re: [Qemu-devel] [PATCH] fix disabling interrupts in sun4u, Artyom Tarasenko, 2011/07/28
- Re: [Qemu-devel] [PATCH] fix disabling interrupts in sun4u, tsnsaito, 2011/07/28
- Re: [Qemu-devel] [PATCH] fix disabling interrupts in sun4u, Artyom Tarasenko, 2011/07/28
- Re: [Qemu-devel] [PATCH] fix disabling interrupts in sun4u, tsnsaito, 2011/07/28
- Re: [Qemu-devel] [PATCH] fix disabling interrupts in sun4u, Blue Swirl, 2011/07/30
Re: [Qemu-devel] [PATCH] fix disabling interrupts in sun4u, Blue Swirl, 2011/07/30