Skip to content

mmio/pio cleanup

Gregory Haskins kvm lkml 3 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v3 Latest3 patches 4 repliesIn review
  2. v23 patches 3 repliesSuperseded
  3. v11 patch 3 repliesSuperseded

Changes in v3 from the cover letter

*) Addressed feedback from Chris Wright w.r.t. patch 2/3 *) Rebased to kvm.git/master:7ff90748

Patches in v3 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3kvm: fix potential coalesced_mmio leak on shutdownvirt/kvm/coalesced_mmio.c+4 -11 reply
2/3kvm: cleanup io_device codearch/x86/kvm/i8254.c arch/x86/kvm/i8259.c arch/x86/kvm/lapic.c +5 more+109 -531 reply
3/3kvm: do not register i8254 PIO regions until we are initializedarch/x86/kvm/i8254.c+8 -91 reply

Reviews trailers per patch

All 3 patches have a Reviewed-by or Acked-by.

PatchAvi KivityChris Wright
1/3A-b
2/3A-b
3/3Found-by A-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v3 thread

Cover letter [KVM PATCH v3 0/3] mmio/pio cleanup

(This is v3 of the series, applies to kvm.git/master:7ff90748)

This is in prep for some more substantial mmio/pio work for iosignalfd, coming later.

[ Changelog:

v3:
*) Addressed feedback from Chris Wright w.r.t. patch 2/3 *) Rebased to kvm.git/master:7ff90748

]

---

Gregory Haskins (3):
kvm: do not register i8254 PIO regions until we are initialized
kvm: cleanup io_device code
kvm: fix potential coalesced_mmio leak on shutdown

 arch/x86/kvm/i8254.c      |   53 +++++++++++++++++++++++++++++----------------
 arch/x86/kvm/i8259.c      |   20 ++++++++++++-----
 arch/x86/kvm/lapic.c      |   22 +++++++++++++------
 arch/x86/kvm/x86.c        |    2 +-
 virt/kvm/coalesced_mmio.c |   26 ++++++++++++++--------
 virt/kvm/ioapic.c         |   22 +++++++++++++------
 virt/kvm/iodev.h          |   29 +++++++++++++++++--------
 virt/kvm/kvm_main.c       |    2 +-
 8 files changed, 117 insertions(+), 59 deletions(-)
Signature · 1 line
Signature