Skip to content

KVM: X86: Cleanups on dest_mode and headers

Peter Xu kvm lkml 4 versions – Applied

Status from this message: Queued, thanks.

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v5 Latest6 patches 1 replyApplied
  2. v46 patches 6 repliesSuperseded
  3. v35 patches 9 repliesSuperseded
Earlier versions · 1
VersionPostedPatchesRepliesStatus
v23 patches6 repliesSuperseded

Changes in v5 from the cover letter

- rename param of ioapic_to_lapic_dest_mode to dest_mode_logical [Sean]
- in patch 5, also do s/short_hand/shorthand/ for kvm_apic_match_dest [Vitaly]
- one more r-b picked

Patches in v5 Applied

#Subject and filesSizeRepliesApplied
1/6KVM: X86: Fix kvm_bitmap_or_dest_vcpus() to use irq shorthandarch/x86/kvm/lapic.c+1 -10 replies
2/6KVM: X86: Move irrelevant declarations out of ioapic.harch/x86/kvm/hyperv.c arch/x86/kvm/ioapic.h arch/x86/kvm/irq.h +1 more+5 -70 replies
3/6KVM: X86: Use APIC_DEST_* macros properly in kvm_lapic_irq.dest_modearch/x86/include/asm/kvm_host.h arch/x86/kvm/ioapic.c arch/x86/kvm/irq_comm.c +1 more+16 -70 replies
4/6KVM: X86: Drop KVM_APIC_SHORT_MASK and KVM_APIC_DEST_MASKarch/x86/kvm/lapic.c arch/x86/kvm/lapic.h arch/x86/kvm/svm.c+5 -70 replies
5/6KVM: X86: Fix callers of kvm_apic_match_dest() to use correct macrosarch/x86/kvm/ioapic.c arch/x86/kvm/irq_comm.c arch/x86/kvm/lapic.c +1 more+12 -80 replies
6/6KVM: X86: Conert the last users of "shorthand = 0" to use macrosarch/x86/kvm/ioapic.c arch/x86/kvm/irq_comm.c arch/x86/kvm/x86.c+4 -40 replies

Reviews trailers per patch

5 of 6 patches have a Reviewed-by or Acked-by.

PatchVitaly Kuznetsov
1/6R-b
2/6R-b
3/6
4/6R-b
5/6R-b
6/6R-b

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

Read the v5 thread

Cover letter [PATCH v5 0/6] KVM: X86: Cleanups on dest_mode and headers

v5:
- rename param of ioapic_to_lapic_dest_mode to dest_mode_logical [Sean]
- in patch 5, also do s/short_hand/shorthand/ for kvm_apic_match_dest [Vitaly]
- one more r-b picked

v4:
- address all comments from Vitaly, adding r-bs properly
- added one more trivial patch:
"KVM: X86: Conert the last users of "shorthand = 0" to use macros"

v3:
- address all the comments from both Vitaly and Sean
- since at it, added patches:
"KVM: X86: Fix kvm_bitmap_or_dest_vcpus() to use irq shorthand" "KVM: X86: Drop KVM_APIC_SHORT_MASK and KVM_APIC_DEST_MASK"

Each patch explains itself.

Please have a look, thanks.

Peter Xu (6):
KVM: X86: Fix kvm_bitmap_or_dest_vcpus() to use irq shorthand
KVM: X86: Move irrelevant declarations out of ioapic.h
KVM: X86: Use APIC_DEST_* macros properly in kvm_lapic_irq.dest_mode
KVM: X86: Drop KVM_APIC_SHORT_MASK and KVM_APIC_DEST_MASK
KVM: X86: Fix callers of kvm_apic_match_dest() to use correct macros
KVM: X86: Conert the last users of "shorthand = 0" to use macros

 arch/x86/include/asm/kvm_host.h |  5 +++++
 arch/x86/kvm/hyperv.c           |  1 +
 arch/x86/kvm/ioapic.c           | 24 +++++++++++++++---------
 arch/x86/kvm/ioapic.h           |  6 ------
 arch/x86/kvm/irq.h              |  3 +++
 arch/x86/kvm/irq_comm.c         | 12 +++++++-----
 arch/x86/kvm/lapic.c            |  9 +++------
 arch/x86/kvm/lapic.h            |  9 +++++----
 arch/x86/kvm/svm.c              |  4 ++--
 arch/x86/kvm/x86.c              |  4 ++--
 10 files changed, 43 insertions(+), 34 deletions(-)
Signature · 1 line
2.21.0