* [PATCH 1/2] RISC-V: KVM: Fix APLIC setipnum_le/be write emulation
2024-03-21 8:50 [PATCH 0/2] KVM RISC-V APLIC fixes Anup Patel
@ 2024-03-21 8:50 ` Anup Patel
2024-03-21 8:50 ` [PATCH 2/2] RISC-V: KVM: Fix APLIC in_clrip[x] read emulation Anup Patel
` (2 subsequent siblings)
3 siblings, 0 replies; 6+ messages in thread
From: Anup Patel @ 2024-03-21 8:50 UTC (permalink / raw)
To: Paolo Bonzini, Atish Patra
Cc: Palmer Dabbelt, Paul Walmsley, Andrew Jones, Anup Patel, kvm,
kvm-riscv, linux-riscv, linux-kernel, Anup Patel, stable
The writes to setipnum_le/be register for APLIC in MSI-mode have special
consideration for level-triggered interrupts as-per the section "4.9.2
Special consideration for level-sensitive interrupt sources" of the RISC-V
AIA specification.
Particularly, the below text from the RISC-V AIA specification defines
the behaviour of writes to setipnum_le/be register for level-triggered
interrupts:
"A second option is for the interrupt service routine to write the
APLIC’s source identity number for the interrupt to the domain’s
setipnum register just before exiting. This will cause the interrupt’s
pending bit to be set to one again if the source is still asserting
an interrupt, but not if the source is not asserting an interrupt."
Fix setipnum_le/be write emulation for in-kernel APLIC by implementing
the above behaviour in aplic_write_pending() function.
Cc: stable@vger.kernel.org
Fixes: 74967aa208e2 ("RISC-V: KVM: Add in-kernel emulation of AIA APLIC")
Signed-off-by: Anup Patel <apatel@ventanamicro.com>
---
arch/riscv/kvm/aia_aplic.c | 16 +++++++++++++---
1 file changed, 13 insertions(+), 3 deletions(-)
diff --git a/arch/riscv/kvm/aia_aplic.c b/arch/riscv/kvm/aia_aplic.c
index 39e72aa016a4..5e842b92dc46 100644
--- a/arch/riscv/kvm/aia_aplic.c
+++ b/arch/riscv/kvm/aia_aplic.c
@@ -137,11 +137,21 @@ static void aplic_write_pending(struct aplic *aplic, u32 irq, bool pending)
raw_spin_lock_irqsave(&irqd->lock, flags);
sm = irqd->sourcecfg & APLIC_SOURCECFG_SM_MASK;
- if (!pending &&
- ((sm == APLIC_SOURCECFG_SM_LEVEL_HIGH) ||
- (sm == APLIC_SOURCECFG_SM_LEVEL_LOW)))
+ if (sm == APLIC_SOURCECFG_SM_INACTIVE)
goto skip_write_pending;
+ if (sm == APLIC_SOURCECFG_SM_LEVEL_HIGH ||
+ sm == APLIC_SOURCECFG_SM_LEVEL_LOW) {
+ if (!pending)
+ goto skip_write_pending;
+ if ((irqd->state & APLIC_IRQ_STATE_INPUT) &&
+ sm == APLIC_SOURCECFG_SM_LEVEL_LOW)
+ goto skip_write_pending;
+ if (!(irqd->state & APLIC_IRQ_STATE_INPUT) &&
+ sm == APLIC_SOURCECFG_SM_LEVEL_HIGH)
+ goto skip_write_pending;
+ }
+
if (pending)
irqd->state |= APLIC_IRQ_STATE_PENDING;
else
--
2.34.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH 2/2] RISC-V: KVM: Fix APLIC in_clrip[x] read emulation
2024-03-21 8:50 [PATCH 0/2] KVM RISC-V APLIC fixes Anup Patel
2024-03-21 8:50 ` [PATCH 1/2] RISC-V: KVM: Fix APLIC setipnum_le/be write emulation Anup Patel
@ 2024-03-21 8:50 ` Anup Patel
2024-03-22 11:35 ` [PATCH 0/2] KVM RISC-V APLIC fixes Bing Fan
2024-03-26 4:07 ` Anup Patel
3 siblings, 0 replies; 6+ messages in thread
From: Anup Patel @ 2024-03-21 8:50 UTC (permalink / raw)
To: Paolo Bonzini, Atish Patra
Cc: Palmer Dabbelt, Paul Walmsley, Andrew Jones, Anup Patel, kvm,
kvm-riscv, linux-riscv, linux-kernel, Anup Patel
The reads to APLIC in_clrip[x] registers returns rectified input values
of the interrupt sources.
A rectified input value of an interrupt source is defined by the section
"4.5.2 Source configurations (sourcecfg[1]–sourcecfg[1023])" of the
RISC-V AIA specification as:
rectified input value = (incoming wire value) XOR (source is inverted)
Update the riscv_aplic_input() implementation to match the above.
Fixes: 74967aa208e2 ("RISC-V: KVM: Add in-kernel emulation of AIA APLIC")
Signed-off-by: Anup Patel <apatel@ventanamicro.com>
---
arch/riscv/kvm/aia_aplic.c | 21 ++++++++++++++++++---
1 file changed, 18 insertions(+), 3 deletions(-)
diff --git a/arch/riscv/kvm/aia_aplic.c b/arch/riscv/kvm/aia_aplic.c
index 5e842b92dc46..b467ba5ed910 100644
--- a/arch/riscv/kvm/aia_aplic.c
+++ b/arch/riscv/kvm/aia_aplic.c
@@ -197,16 +197,31 @@ static void aplic_write_enabled(struct aplic *aplic, u32 irq, bool enabled)
static bool aplic_read_input(struct aplic *aplic, u32 irq)
{
- bool ret;
- unsigned long flags;
+ u32 sourcecfg, sm, raw_input, irq_inverted;
struct aplic_irq *irqd;
+ unsigned long flags;
+ bool ret = false;
if (!irq || aplic->nr_irqs <= irq)
return false;
irqd = &aplic->irqs[irq];
raw_spin_lock_irqsave(&irqd->lock, flags);
- ret = (irqd->state & APLIC_IRQ_STATE_INPUT) ? true : false;
+
+ sourcecfg = irqd->sourcecfg;
+ if (sourcecfg & APLIC_SOURCECFG_D)
+ goto skip;
+
+ sm = sourcecfg & APLIC_SOURCECFG_SM_MASK;
+ if (sm == APLIC_SOURCECFG_SM_INACTIVE)
+ goto skip;
+
+ raw_input = (irqd->state & APLIC_IRQ_STATE_INPUT) ? 1 : 0;
+ irq_inverted = (sm == APLIC_SOURCECFG_SM_LEVEL_LOW ||
+ sm == APLIC_SOURCECFG_SM_EDGE_FALL) ? 1 : 0;
+ ret = !!(raw_input ^ irq_inverted);
+
+skip:
raw_spin_unlock_irqrestore(&irqd->lock, flags);
return ret;
--
2.34.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH 0/2] KVM RISC-V APLIC fixes
2024-03-21 8:50 [PATCH 0/2] KVM RISC-V APLIC fixes Anup Patel
2024-03-21 8:50 ` [PATCH 1/2] RISC-V: KVM: Fix APLIC setipnum_le/be write emulation Anup Patel
2024-03-21 8:50 ` [PATCH 2/2] RISC-V: KVM: Fix APLIC in_clrip[x] read emulation Anup Patel
@ 2024-03-22 11:35 ` Bing Fan
2024-03-22 11:50 ` Anup Patel
2024-03-26 4:07 ` Anup Patel
3 siblings, 1 reply; 6+ messages in thread
From: Bing Fan @ 2024-03-22 11:35 UTC (permalink / raw)
To: Anup Patel, Paolo Bonzini, Atish Patra
Cc: Palmer Dabbelt, Paul Walmsley, Andrew Jones, Anup Patel, kvm,
kvm-riscv, linux-riscv, linux-kernel
Hi,
As you mentioned as below, riscv's aia patch in
https://github.com/avpatel/linux.git
Why is this series of patches not merged into upstream?
在 2024/3/21 16:50, Anup Patel 写道:
> Few fixes for KVM RISC-V in-kernel APLIC emulation which were discovered
> during Linux AIA driver patch reviews.
>
> These patches can also be found in the riscv_kvm_aplic_fixes_v1
> branch at: https://github.com/avpatel/linux.git
>
> Anup Patel (2):
> RISC-V: KVM: Fix APLIC setipnum_le/be write emulation
> RISC-V: KVM: Fix APLIC in_clrip[x] read emulation
>
> arch/riscv/kvm/aia_aplic.c | 37 +++++++++++++++++++++++++++++++------
> 1 file changed, 31 insertions(+), 6 deletions(-)
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 0/2] KVM RISC-V APLIC fixes
2024-03-22 11:35 ` [PATCH 0/2] KVM RISC-V APLIC fixes Bing Fan
@ 2024-03-22 11:50 ` Anup Patel
0 siblings, 0 replies; 6+ messages in thread
From: Anup Patel @ 2024-03-22 11:50 UTC (permalink / raw)
To: Bing Fan
Cc: Anup Patel, Paolo Bonzini, Atish Patra, Palmer Dabbelt,
Paul Walmsley, Andrew Jones, kvm, kvm-riscv, linux-riscv,
linux-kernel
On Fri, Mar 22, 2024 at 5:05 PM Bing Fan <hptsfb@gmail.com> wrote:
>
>
> Hi,
>
>
> As you mentioned as below, riscv's aia patch in
> https://github.com/avpatel/linux.git
>
> Why is this series of patches not merged into upstream?
This will be sent as part of Linux-6.9-rcX fixes (after 1-2 weeks).
Regards,
Anup
>
>
> 在 2024/3/21 16:50, Anup Patel 写道:
> > Few fixes for KVM RISC-V in-kernel APLIC emulation which were discovered
> > during Linux AIA driver patch reviews.
> >
> > These patches can also be found in the riscv_kvm_aplic_fixes_v1
> > branch at: https://github.com/avpatel/linux.git
> >
> > Anup Patel (2):
> > RISC-V: KVM: Fix APLIC setipnum_le/be write emulation
> > RISC-V: KVM: Fix APLIC in_clrip[x] read emulation
> >
> > arch/riscv/kvm/aia_aplic.c | 37 +++++++++++++++++++++++++++++++------
> > 1 file changed, 31 insertions(+), 6 deletions(-)
> >
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 0/2] KVM RISC-V APLIC fixes
2024-03-21 8:50 [PATCH 0/2] KVM RISC-V APLIC fixes Anup Patel
` (2 preceding siblings ...)
2024-03-22 11:35 ` [PATCH 0/2] KVM RISC-V APLIC fixes Bing Fan
@ 2024-03-26 4:07 ` Anup Patel
3 siblings, 0 replies; 6+ messages in thread
From: Anup Patel @ 2024-03-26 4:07 UTC (permalink / raw)
To: Anup Patel
Cc: Paolo Bonzini, Atish Patra, Palmer Dabbelt, Paul Walmsley,
Andrew Jones, kvm, kvm-riscv, linux-riscv, linux-kernel
On Thu, Mar 21, 2024 at 2:20 PM Anup Patel <apatel@ventanamicro.com> wrote:
>
> Few fixes for KVM RISC-V in-kernel APLIC emulation which were discovered
> during Linux AIA driver patch reviews.
>
> These patches can also be found in the riscv_kvm_aplic_fixes_v1
> branch at: https://github.com/avpatel/linux.git
>
> Anup Patel (2):
> RISC-V: KVM: Fix APLIC setipnum_le/be write emulation
> RISC-V: KVM: Fix APLIC in_clrip[x] read emulation
Queued this series for Linux-6.9 fixes.
Thanks,
Anup
>
> arch/riscv/kvm/aia_aplic.c | 37 +++++++++++++++++++++++++++++++------
> 1 file changed, 31 insertions(+), 6 deletions(-)
>
> --
> 2.34.1
>
^ permalink raw reply [flat|nested] 6+ messages in thread