Thread (59 messages) flat view 59 messages, 6 authors, 6d ago

Re: [PATCH v7 02/23] KVM: Make device name configurable

From: Sean Christopherson <seanjc@google.com>
Date: 2026-09-03 20:58:22
Also in: kvm, kvmarm, linux-s390, lkml

On Thu, Sep 03, 2026, Paolo Bonzini wrote:
On Thu, Sep 3, 2026 at 5:54 PM Sean Christopherson [off-list ref] wrote:
quoted
quoted
I don't want KVM to be the. one that does things differently *once more*.
Too late :-D

arch/arm64/kvm/hyp/nvhe/Makefile:ccflags-y := -D__KVM_NVHE_HYPERVISOR__ -D__DISABLE_EXPORTS -D__DISABLE_TRACE_MMIO__
arch/arm64/kvm/hyp/vhe/Makefile:ccflags-y := -D__KVM_VHE_HYPERVISOR__
Sure, VHE/nVHE is fair game because the nVHE case is in the same boat
as vDSO, arch/*/boot, and so on.

Looking at a grep for (cflags|CFLAGS).*\-D most are -DDEBUG,
-DDISABLE_BRANCH_PROFILING, -DDEFAULT_SYMBOL_NAMESPACE, etc. and in
most cases they seem like a bad idea. Don't do it for KVM please;
requiring -D for all architectures and for random pieces of config,
when you can put it in asm/kvm_host.h, is just as bad if not worse.
Boo, it's never fun when a parent shows up and takes away your toys :-D
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help