Commit b1a4286b authored by Paul Mackerras's avatar Paul Mackerras Committed by Paul Mackerras

KVM: PPC: Book3S HV: Re-enable XICS fast path for irqfd-generated interrupts

Commit c9a5ecca ("kvm/eventfd: add arch-specific set_irq",
2015-10-16) added the possibility for architecture-specific code
to handle the generation of virtual interrupts in atomic context
where possible, without having to schedule a work function.

Since we can easily generate virtual interrupts on XICS without
having to do anything worse than take a spinlock, we define a
kvm_arch_set_irq_inatomic() for XICS.  We also remove kvm_set_msi()
since it is not used any more.

The one slightly tricky thing is that with the new interface, we
don't get told whether the interrupt is an MSI (or other edge
sensitive interrupt) vs. level-sensitive.  The difference as far
as interrupt generation is concerned is that for LSIs we have to
set the asserted flag so it will continue to fire until it is
explicitly cleared.

In fact the XICS code gets told which interrupts are LSIs by userspace
when it configures the interrupt via the KVM_DEV_XICS_GRP_SOURCES
attribute group on the XICS device.  To store this information, we add
a new "lsi" field to struct ics_irq_state.  With that we can also do a
better job of returning accurate values when reading the attribute
group.
Signed-off-by: default avatarPaul Mackerras <paulus@samba.org>
parent 07f8ab25
...@@ -92,7 +92,7 @@ static int ics_deliver_irq(struct kvmppc_xics *xics, u32 irq, u32 level) ...@@ -92,7 +92,7 @@ static int ics_deliver_irq(struct kvmppc_xics *xics, u32 irq, u32 level)
* we are the only setter, thus concurrent access is undefined * we are the only setter, thus concurrent access is undefined
* to begin with. * to begin with.
*/ */
if (level == 1 || level == KVM_INTERRUPT_SET_LEVEL) if ((level == 1 && state->lsi) || level == KVM_INTERRUPT_SET_LEVEL)
state->asserted = 1; state->asserted = 1;
else if (level == 0 || level == KVM_INTERRUPT_UNSET) { else if (level == 0 || level == KVM_INTERRUPT_UNSET) {
state->asserted = 0; state->asserted = 0;
...@@ -1174,9 +1174,11 @@ static int xics_get_source(struct kvmppc_xics *xics, long irq, u64 addr) ...@@ -1174,9 +1174,11 @@ static int xics_get_source(struct kvmppc_xics *xics, long irq, u64 addr)
prio = irqp->saved_priority; prio = irqp->saved_priority;
} }
val |= prio << KVM_XICS_PRIORITY_SHIFT; val |= prio << KVM_XICS_PRIORITY_SHIFT;
if (irqp->asserted) if (irqp->lsi) {
val |= KVM_XICS_LEVEL_SENSITIVE | KVM_XICS_PENDING; val |= KVM_XICS_LEVEL_SENSITIVE;
else if (irqp->masked_pending || irqp->resend) if (irqp->asserted)
val |= KVM_XICS_PENDING;
} else if (irqp->masked_pending || irqp->resend)
val |= KVM_XICS_PENDING; val |= KVM_XICS_PENDING;
ret = 0; ret = 0;
} }
...@@ -1228,9 +1230,13 @@ static int xics_set_source(struct kvmppc_xics *xics, long irq, u64 addr) ...@@ -1228,9 +1230,13 @@ static int xics_set_source(struct kvmppc_xics *xics, long irq, u64 addr)
irqp->priority = prio; irqp->priority = prio;
irqp->resend = 0; irqp->resend = 0;
irqp->masked_pending = 0; irqp->masked_pending = 0;
irqp->lsi = 0;
irqp->asserted = 0; irqp->asserted = 0;
if ((val & KVM_XICS_PENDING) && (val & KVM_XICS_LEVEL_SENSITIVE)) if (val & KVM_XICS_LEVEL_SENSITIVE) {
irqp->asserted = 1; irqp->lsi = 1;
if (val & KVM_XICS_PENDING)
irqp->asserted = 1;
}
irqp->exists = 1; irqp->exists = 1;
arch_spin_unlock(&ics->lock); arch_spin_unlock(&ics->lock);
local_irq_restore(flags); local_irq_restore(flags);
...@@ -1249,11 +1255,10 @@ int kvm_set_irq(struct kvm *kvm, int irq_source_id, u32 irq, int level, ...@@ -1249,11 +1255,10 @@ int kvm_set_irq(struct kvm *kvm, int irq_source_id, u32 irq, int level,
return ics_deliver_irq(xics, irq, level); return ics_deliver_irq(xics, irq, level);
} }
int kvm_set_msi(struct kvm_kernel_irq_routing_entry *irq_entry, struct kvm *kvm, int kvm_arch_set_irq_inatomic(struct kvm_kernel_irq_routing_entry *irq_entry,
int irq_source_id, int level, bool line_status) struct kvm *kvm, int irq_source_id,
int level, bool line_status)
{ {
if (!level)
return -1;
return kvm_set_irq(kvm, irq_source_id, irq_entry->gsi, return kvm_set_irq(kvm, irq_source_id, irq_entry->gsi,
level, line_status); level, line_status);
} }
......
...@@ -39,6 +39,7 @@ struct ics_irq_state { ...@@ -39,6 +39,7 @@ struct ics_irq_state {
u8 saved_priority; u8 saved_priority;
u8 resend; u8 resend;
u8 masked_pending; u8 masked_pending;
u8 lsi; /* level-sensitive interrupt */
u8 asserted; /* Only for LSI */ u8 asserted; /* Only for LSI */
u8 exists; u8 exists;
}; };
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment