Thread (138 messages) 138 messages, 7 authors, 2016-04-14
STALE3714d
Revisions (7)
  1. rfc [diff vs current]
  2. rfc [diff vs current]
  3. rfc [diff vs current]
  4. rfc [diff vs current]
  5. rfc current
  6. rfc [diff vs current]
  7. v5 [diff vs current]

[RFC PATCH 00/45] KVM: arm/arm64: Rework virtual GIC emulation

From: Christoffer Dall <hidden>
Date: 2016-03-31 18:30:19
Also in: kvm, kvmarm

On Fri, Mar 25, 2016 at 02:04:23AM +0000, Andre Przywara wrote:
This series is a joint effort to re-implement KVM's GIC emulation.

While the current implementation is centered around providing
efficient MMIO emulation, the hot path for most guests is actually
the guest entry and exit, which currently is rather costly.
Also the existing emulation has a global distributor lock, which
quickly becomes a bottleneck once the number of VCPUs increases.
Additionally the emulation was originally designed for GICv2, adding
GICv3 ITS emulation support to this proved to be rather painful.
Last, but not least the existing code became less and less
maintainable, with many special cases handled explicitly.

The new implementation is build around a struct vgic_irq data data
structure, which holds all information about a virtual interrupt.
Interruts which should be injected are hold in a per-VCPU list, this
make the entry/exit path much more efficient. Also the new structure
allows to have more fine grained locking - per IRQ and per VCPU -
getting rid of the global distributor lock.
As a result of the new design ITS emulation fits in more nicely, the
respective code will be provided as a follow-up series.

This series implements the same feature set as the existing emulation,
as a goodie we now implement priorities correctly.
To allow an easy transition with good test coverage, but still maintain
stability, both implementations live side by side, selectable via a
Kconfig option. The default is the new implementation.
If this code proves to be reliable, we will later remove the current
implementation with an extra patch set.

Please have a look at the series, review it and give the code some
serious testing (and possibly debugging). All feedback is appreciated.
Hmph, starting a guest a couple of times and running hackbench inside
the guest actually gave me (twice) the following error:

NMI watchdog: BUG: soft lockup - CPU#0 stuck for 23s! [kworker/0:0:4]

(that is using your branch on Mustang).

-Christoffer
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help