Skip to contentSkip to conversation

kvm: locking and API rework for iosignalfd

PATCH Superseded Michael S. Tsirkin 2 participants 2 messages Jun 29, 2009 KVM

v3 of 5 Superseded Latest is v5 →

lore
Share & tools
lore
#0 lore raw

v3 cover kvm: locking and API rework for iosignalfd

This series switches kvm_io_device to slots_lock from kvm->lock mutex, and uses that to rework io bus API. This takes less locks on data path, and uses less lines of code. These changes will also be useful to serve as basis for Greg's iosignalfd work.

---

I have split the patch and reworked along the lines suggested by Avi. This works for me, but please review carefully.

Michael S. Tsirkin (4):
kvm: document locking for kvm_io_device_ops
kvm: switch coalesced mmio changes to slots_lock
kvm: convert bus to slots_lock
kvm: remove in_range from io devices

 arch/ia64/kvm/kvm-ia64.c  |   28 +++--------
 arch/x86/kvm/i8254.c      |   53 +++++++++++----------
 arch/x86/kvm/i8259.c      |   22 +++++----
 arch/x86/kvm/lapic.c      |   44 ++++++++----------
 arch/x86/kvm/x86.c        |  110 +++++++++++++--------------------------------
 include/linux/kvm_host.h  |    9 +++-
 virt/kvm/coalesced_mmio.c |   28 +++++------
 virt/kvm/ioapic.c         |   24 +++++-----
 virt/kvm/iodev.h          |   42 +++++++----------
 virt/kvm/kvm_main.c       |   31 ++++++++-----
 10 files changed, 171 insertions(+), 220 deletions(-)
#1 lore raw
Quoted 5 lines · Michael S. Tsirkin

This series switches kvm_io_device to slots_lock from kvm->lock mutex, and uses that to rework io bus API. This takes less locks on data path, and uses less lines of code. These changes will also be useful to serve as basis for Greg's iosignalfd work.

Looks good to me. Marcelo?

Signature · 1 line
error compiling committee.c: too many arguments to function