Re: [PATCH v3 4/6] KVM: SVM: Don't advise the user to do force_avic=y (when x2AVIC is detected)

[Naveen N Rao]

On Thu, Sep 18, 2025 at 05:21:34PM -0700, Sean Christopherson wrote:
> Don't advise the end user to try to force enable AVIC when x2AVIC is
> reported as supported in CPUID,
			in CPUID but AVIC isn't,

> as forcefully enabling AVIC isn't something
> that should be done lightly.  E.g. some Zen4 client systems hide AVIC but
> leave x2AVIC behind, and while such a configuration is indeed due to buggy
> firmware in the sense the reporting x2AVIC without AVIC is nonsensical,
			^^ that

> KVM has no idea _why_ firmware disabled AVIC in the first place.
> 
> Suggesting that the user try to run with force_avic=y is sketchy even if
> the user explicitly tries to enable AVIC, and will be downright
> irresponsible once KVM starts enabling AVIC by default.  Alternatively,
> KVM could print the message only when the user explicitly asks for AVIC,
> but running with force_avic=y isn't something that should be encouraged
> for random users.  force_avic is a useful knob for developers and perhaps
> even advanced users, but isn't something that KVM should advertise broadly.
> 
> Opportunistically append a newline to the pr_warn() so that it prints out
> immediately, and tweak the message to say that AVIC is unsupported instead
> of disabled (disabled suggests that the kernel/KVM is somehow responsible).
> 
> Suggested-by: Naveen N Rao <naveen@xxxxxxxxxx>

Suggested-by: Naveen N Rao (AMD) <naveen@xxxxxxxxxx>

> Signed-off-by: Sean Christopherson <seanjc@xxxxxxxxxx>
> ---
>  arch/x86/kvm/svm/avic.c | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)

Reviewed-by: Naveen N Rao (AMD) <naveen@xxxxxxxxxx>


- Naveen