[PW_SID:970599] KVM: Remove include/kvm, standardize includes#502
[PW_SID:970599] KVM: Remove include/kvm, standardize includes#502linux-riscv-bot wants to merge 8 commits into
Conversation
Move arm_hypercalls.h and arm_psci.h into arch/arm64/kvm now that KVM no longer supports 32-bit ARM, i.e. now that there's no reason to make the hypercall and PSCI APIs "public". Signed-off-by: Anish Ghulati <aghulati@google.com> [sean: squash into one patch, write changelog] Signed-off-by: Sean Christopherson <seanjc@google.com> Signed-off-by: Linux RISC-V bot <linux.riscv.bot@gmail.com>
Include include/uapi/linux/kvm.h and include/linux/kvm_types.h in ARM's public arm_arch_timer.h and arm_pmu.h headers to get forward declarations of things like "struct kvm_vcpu" and "struct kvm_device_attr", which are referenced but never declared (neither file includes *any* KVM headers). The missing includes don't currently cause problems because of the order of includes in parent files, but that order is largely arbitrary and is subject to change, e.g. a future commit will move the ARM specific headers to arch/arm64/include/asm and reorder parent includes to maintain alphabetic ordering. Reported-by: kernel test robot <lkp@intel.com> Signed-off-by: Sean Christopherson <seanjc@google.com> Signed-off-by: Linux RISC-V bot <linux.riscv.bot@gmail.com>
Move kvm/arm_{arch_timer,pmu,vgic}.h to arch/arm64/include/asm and drop
the "arm" prefix from all file names. Now that KVM no longer supports
32-bit ARM, there is no reason to expose ARM specific headers to other
architectures beyond arm64.
Cc: Colton Lewis <coltonlewis@google.com>
Signed-off-by: Sean Christopherson <seanjc@google.com>
Signed-off-by: Linux RISC-V bot <linux.riscv.bot@gmail.com>
Move iodev.h, the last remaining holdout in include/kvm, to the standard include/linux directory as kvm_iodev.h and delete include/kvm. Acked-by: Anup Patel <anup@brainfault.org> Signed-off-by: Sean Christopherson <seanjc@google.com> Signed-off-by: Linux RISC-V bot <linux.riscv.bot@gmail.com>
Don't add virt/kvm to KVM MIPS' include path, the headers in virt/kvm are intended to be used only by other code in virt/kvm, i.e. are "private" to the core KVM code. It's not clear that MIPS *ever* included a header from virt/kvm, i.e. odds are good the "-Ivirt/kvm" was copied from a different architecture's Makefile when MIPS support was first added. Signed-off-by: Sean Christopherson <seanjc@google.com> Signed-off-by: Linux RISC-V bot <linux.riscv.bot@gmail.com>
Don't add virt/kvm to KVM PPC's include path, the headers in virt/kvm are intended to be used only by other code in virt/kvm, i.e. are "private" to the core KVM code. It's not clear that PPC *ever* included a header from virt/kvm, i.e. odds are good the "-Ivirt/kvm" was copied from a different architecture's Makefile when PPC support was first added. Signed-off-by: Sean Christopherson <seanjc@google.com> Signed-off-by: Linux RISC-V bot <linux.riscv.bot@gmail.com>
Don't add virt/kvm to KVM s390's include path, the headers in virt/kvm are intended to be used only by other code in virt/kvm, i.e. are "private" to the core KVM code. It's not clear that s390 *ever* included a header from virt/kvm, i.e. odds are good the "-Ivirt/kvm" was copied from a x86's Makefile when s390 support was first added. The only headers in virt/kvm at the time were the x86 specific ioapic.h, and iodev.h, neither of which shows up as an #include in the diff for the commit range 37817f2..e976a2b. Acked-by: Claudio Imbrenda <imbrenda@linux.ibm.com> Reviewed-by: Thomas Huth <thuth@redhat.com> Signed-off-by: Sean Christopherson <seanjc@google.com> Signed-off-by: Linux RISC-V bot <linux.riscv.bot@gmail.com>
Standardize KVM's include paths across all architectures by declaring the KVM-specific includes in the common Makefile.kvm. Having common KVM "own" the included paths reduces the temptation to unnecessarily add virt/kvm to arch include paths, and conversely if allowing arch code to grab headers from virt/kvm becomes desirable, virt/kvm can be added to all architecture's include path with a single line update. Having the common KVM makefile append to ccflags also provides a convenient location to append other things, e.g. KVM-specific #defines. Note, this changes the behavior of s390 and PPC, as s390 and PPC previously overwrote ccflags-y instead of adding on. There is no evidence that overwriting ccflags-y was necessary or even deliberate, as both s390 and PPC switched to the overwrite behavior without so much as a passing mention when EXTRA_CFLAGS was replaced with ccflags-y (commit c73028a ("s390: change to new flag variable") and commit 4108d9b ("powerpc/Makefiles: Change to new flag variables")). Acked-by: Anup Patel <anup@brainfault.org> Signed-off-by: Sean Christopherson <seanjc@google.com> Signed-off-by: Linux RISC-V bot <linux.riscv.bot@gmail.com>
|
Patch 1: "[1/8] KVM: arm64: Move arm_{psci,hypercalls}.h to an internal KVM path" |
|
Patch 1: "[1/8] KVM: arm64: Move arm_{psci,hypercalls}.h to an internal KVM path" |
|
Patch 1: "[1/8] KVM: arm64: Move arm_{psci,hypercalls}.h to an internal KVM path" |
|
Patch 1: "[1/8] KVM: arm64: Move arm_{psci,hypercalls}.h to an internal KVM path" |
|
Patch 1: "[1/8] KVM: arm64: Move arm_{psci,hypercalls}.h to an internal KVM path" |
|
Patch 1: "[1/8] KVM: arm64: Move arm_{psci,hypercalls}.h to an internal KVM path" |
|
Patch 1: "[1/8] KVM: arm64: Move arm_{psci,hypercalls}.h to an internal KVM path" |
|
Patch 1: "[1/8] KVM: arm64: Move arm_{psci,hypercalls}.h to an internal KVM path" |
|
Patch 1: "[1/8] KVM: arm64: Move arm_{psci,hypercalls}.h to an internal KVM path" |
|
Patch 1: "[1/8] KVM: arm64: Move arm_{psci,hypercalls}.h to an internal KVM path" |
|
Patch 1: "[1/8] KVM: arm64: Move arm_{psci,hypercalls}.h to an internal KVM path" |
|
Patch 1: "[1/8] KVM: arm64: Move arm_{psci,hypercalls}.h to an internal KVM path" |
|
Patch 2: "[2/8] KVM: arm64: Include KVM headers to get forward declarations" |
|
Patch 2: "[2/8] KVM: arm64: Include KVM headers to get forward declarations" |
|
Patch 2: "[2/8] KVM: arm64: Include KVM headers to get forward declarations" |
|
Patch 2: "[2/8] KVM: arm64: Include KVM headers to get forward declarations" |
|
Patch 2: "[2/8] KVM: arm64: Include KVM headers to get forward declarations" |
|
Patch 2: "[2/8] KVM: arm64: Include KVM headers to get forward declarations" |
|
Patch 2: "[2/8] KVM: arm64: Include KVM headers to get forward declarations" |
|
Patch 2: "[2/8] KVM: arm64: Include KVM headers to get forward declarations" |
|
Patch 2: "[2/8] KVM: arm64: Include KVM headers to get forward declarations" |
|
Patch 2: "[2/8] KVM: arm64: Include KVM headers to get forward declarations" |
|
Patch 6: "[6/8] KVM: PPC: Stop adding virt/kvm to the arch include path" |
|
Patch 6: "[6/8] KVM: PPC: Stop adding virt/kvm to the arch include path" |
|
Patch 6: "[6/8] KVM: PPC: Stop adding virt/kvm to the arch include path" |
|
Patch 6: "[6/8] KVM: PPC: Stop adding virt/kvm to the arch include path" |
|
Patch 7: "[7/8] KVM: s390: Stop adding virt/kvm to the arch include path" |
|
Patch 7: "[7/8] KVM: s390: Stop adding virt/kvm to the arch include path" |
|
Patch 7: "[7/8] KVM: s390: Stop adding virt/kvm to the arch include path" |
|
Patch 7: "[7/8] KVM: s390: Stop adding virt/kvm to the arch include path" |
|
Patch 7: "[7/8] KVM: s390: Stop adding virt/kvm to the arch include path" |
|
Patch 7: "[7/8] KVM: s390: Stop adding virt/kvm to the arch include path" |
|
Patch 7: "[7/8] KVM: s390: Stop adding virt/kvm to the arch include path" |
|
Patch 7: "[7/8] KVM: s390: Stop adding virt/kvm to the arch include path" |
|
Patch 7: "[7/8] KVM: s390: Stop adding virt/kvm to the arch include path" |
|
Patch 7: "[7/8] KVM: s390: Stop adding virt/kvm to the arch include path" |
|
Patch 7: "[7/8] KVM: s390: Stop adding virt/kvm to the arch include path" |
|
Patch 7: "[7/8] KVM: s390: Stop adding virt/kvm to the arch include path" |
|
Patch 8: "[8/8] KVM: Standardize include paths across all architectures" |
|
Patch 8: "[8/8] KVM: Standardize include paths across all architectures" |
|
Patch 8: "[8/8] KVM: Standardize include paths across all architectures" |
|
Patch 8: "[8/8] KVM: Standardize include paths across all architectures" |
|
Patch 8: "[8/8] KVM: Standardize include paths across all architectures" |
|
Patch 8: "[8/8] KVM: Standardize include paths across all architectures" |
|
Patch 8: "[8/8] KVM: Standardize include paths across all architectures" |
|
Patch 8: "[8/8] KVM: Standardize include paths across all architectures" |
|
Patch 8: "[8/8] KVM: Standardize include paths across all architectures" |
|
Patch 8: "[8/8] KVM: Standardize include paths across all architectures" |
|
Patch 8: "[8/8] KVM: Standardize include paths across all architectures" |
|
Patch 8: "[8/8] KVM: Standardize include paths across all architectures" |
3 participants
PR for series 970599 applied to workflow__riscv__fixes
Name: KVM: Remove include/kvm, standardize includes
URL: https://patchwork.kernel.org/project/linux-riscv/list/?series=970599
Version: 1