From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f200.google.com (mail-pl1-f200.google.com [209.85.214.200]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id DC9FF353A82 for ; Wed, 30 Sep 2026 01:13:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.200 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790730819; cv=none; b=XZwJutxPGRPKgSWzYHhKQaTVkGd32eunnoPh0nrrypSyx07K+RfQd24WokxDjA46S5DlIwvriQrxMxEPS7lDAB3bEMxCMQ0+sUysMWZPiY6J3OoOlpbttqujp2EaAmmXJK9j1iFEEgJKLYZ48Fkg+a7/36L3HrzLZX7hUmDxyBk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790730819; c=relaxed/simple; bh=9/5VYKp+FpTlacyCJ0B7NN+WKgSUpEps0uP+5xXDmOw=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Cc:Content-Type; b=NiCdZ1Thqy56+a3z7rVtNt1ifXwbVS9XuZ1gBanqsmkGSQpiGi4GJ6aGV9lZF7K8hCQq3N3eTflFQhWQR408ucCZahLqh7kvWjnbf/XElsJgiVyfU6Lxbw5jFXFw+8TzAyLgw/gYNoot4TTv2TaJulr4QI8yh3pZTVrByC9/LJg= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--seanjc.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=hlpWwgg+; arc=none smtp.client-ip=209.85.214.200 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--seanjc.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="hlpWwgg+" Received: by mail-pl1-f200.google.com with SMTP id d9443c01a7336-2d6f80c76e6so39963855ad.3 for ; Tue, 29 Sep 2026 18:13:29 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20251104; t=1790730809; x=1791335609; darn=vger.kernel.org; h=content-type:cc:to:from:subject:message-id:references:mime-version :in-reply-to:date:reply-to:from:to:cc:subject:date:message-id :reply-to:content-type; bh=wtnEMbDhnNePMM9FVDMKnptaMxqEpAp14r/XB4S9VOI=; b=hlpWwgg+dReTzch/dblaKPDJh+i3buCOoGBD0kBLKEReYvRTGdCeyoIHUpx4VCM0jh ce8OvxQFee71WLP7lAiMhgD6gjBij3lmWKl67HOs2ABY8nwqRWeLrfd6p7djWc0J/mIM JakusBqTDpMplgZyXdRS8x4L5sMVV6LbMwa7WbRCeccxe8i2rT+nE/UWaCXJ+fP5HxQJ 0dmKOjrudaEwm1lHYO7VWfz/PMeQAa3mf5onbqkD6HwlbvRT8vilXjl/6xxcDzk/XesQ rMlbmeUFnf9hTriKId7ijqx94rt9Dv2VtywKPm6EY4Iu/F1vdoUhTah9s1kxnbp/rLBC qfEQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1790730809; x=1791335609; h=content-type:cc:to:from:subject:message-id:references:mime-version :in-reply-to:date:reply-to:x-gm-message-state:from:to:cc:subject :date:message-id:reply-to:content-type; bh=wtnEMbDhnNePMM9FVDMKnptaMxqEpAp14r/XB4S9VOI=; b=KyY/Ou+x1+1NzMBFKn2j8Y0qAqetPkQ2/6xjAvjpPry+jj7TdVydzzZpQQRMWvv+Jj 449Pflg2/7MxpolBjQlNLjDS7VblFnBZ9SgkSKMsvK017yg40cHDIUDnhmnqjhsdPSmj 0OalSSBfIgXxDOZjZLd/+Rt6kaAfwBFGuHbWEMNWpcLvXJ8S0hAammAOpd/psZXjMc3u rpOggyRZ31+TD2Dme8SlV9AjVBfdzuxuk2AvRmq/NG4xVvtxqWnVeI1lrEgXFtWln3os 6WoRq+c3ZQZZ+5zqiBh/POn/oUTUchRkTrBfnGVhaJR7YfyknlF6+xVSf1PbqthvxUvM 5yGA== X-Forwarded-Encrypted: i=1; AKwUvBzgRKAI/y31G8JQSZ4CQK8RQNcBsb9NdYquq14PmYuS/eQ0xDuNXg+5xz+SdN+vKBULuUlVoNSt94iSlsc=@vger.kernel.org X-Gm-Message-State: AFq9FYKv69IOsxPjleJ1YUJmh5xRsnBQFxjjeTjQRv+dOaGgDWd9vp6e WVHdxgoOtI1+BIMu4FEoUipHPoycYFLd6IwCsVLbo1x6QclOn4T743gay8fvnmE5hCajGijzBYP 15gWOhg== X-Received: from plme15.prod.google.com ([2002:a17:903:490f:b0:2df:b0be:1e4f]) (user=seanjc job=prod-delivery.src-stubby-dispatcher) by 2002:a17:902:f710:b0:2d3:716e:e2ae with SMTP id d9443c01a7336-2e2de421dacmr7280435ad.6.1790730808564; Tue, 29 Sep 2026 18:13:28 -0700 (PDT) Reply-To: Sean Christopherson Date: Tue, 29 Sep 2026 18:13:12 -0700 In-Reply-To: <20260930011313.3197688-1-seanjc@google.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20260930011313.3197688-1-seanjc@google.com> X-Mailer: git-send-email 2.56.0.rc1.315.gc6ed9934b7-goog Message-ID: <20260930011313.3197688-12-seanjc@google.com> Subject: [PATCH v5 11/12] KVM: SVM: Add a vCPU-aware helper to get supported DEBUGCTL bits From: Sean Christopherson To: Sean Christopherson , Paolo Bonzini Cc: kvm@vger.kernel.org, linux-kernel@vger.kernel.org, Yosry Ahmed , Shivansh Dhiman , Nikunj A Dadhania , Ravi Bangoria , Tom Lendacky Content-Type: text/plain; charset="UTF-8" From: Shivansh Dhiman Replace DEBUGCTL_RESERVED_BITS with a vCPU-aware helper that provides the set of supported bits so that KVM can handle bits that are conditionally supported based on the vCPU model, e.g. for features like Bus Lock Detect. No functional change intended. Signed-off-by: Shivansh Dhiman Reviewed-by: Nikunj A Dadhania [sean: invert from reserved => supported, massage changelog] Signed-off-by: Sean Christopherson --- arch/x86/kvm/svm/nested.c | 8 ++++---- arch/x86/kvm/svm/svm.c | 6 +++--- arch/x86/kvm/svm/svm.h | 7 +++++-- 3 files changed, 12 insertions(+), 9 deletions(-) diff --git a/arch/x86/kvm/svm/nested.c b/arch/x86/kvm/svm/nested.c index c7a69e309a36..af0ef678433d 100644 --- a/arch/x86/kvm/svm/nested.c +++ b/arch/x86/kvm/svm/nested.c @@ -818,7 +818,7 @@ static void nested_vmcb02_prepare_save(struct vcpu_svm *svm) * svm_set_msr's definition of reserved bits. */ svm_copy_lbrs(&vmcb02->save, save); - vmcb02->save.dbgctl &= ~DEBUGCTL_RESERVED_BITS; + vmcb02->save.dbgctl &= svm_get_supported_debugctl(vcpu); } else { svm_copy_lbrs(&vmcb02->save, &vmcb01->save); } @@ -1207,7 +1207,7 @@ int nested_svm_vmrun(struct kvm_vcpu *vcpu) } /* Copy state save area fields which are handled by VMRUN */ -void svm_copy_vmrun_state(struct vmcb_save_area *to_save, +void svm_copy_vmrun_state(struct kvm_vcpu *vcpu, struct vmcb_save_area *to_save, struct vmcb_save_area *from_save) { to_save->es = from_save->es; @@ -1234,7 +1234,7 @@ void svm_copy_vmrun_state(struct vmcb_save_area *to_save, if (kvm_cpu_cap_has(X86_FEATURE_LBRV)) { svm_copy_lbrs(to_save, from_save); - to_save->dbgctl &= ~DEBUGCTL_RESERVED_BITS; + to_save->dbgctl &= svm_get_supported_debugctl(vcpu); } } @@ -2081,7 +2081,7 @@ static int svm_set_nested_state(struct kvm_vcpu *vcpu, svm->nested.vmcb12_gpa = kvm_state->hdr.svm.vmcb_pa; - svm_copy_vmrun_state(&svm->vmcb01.ptr->save, save); + svm_copy_vmrun_state(vcpu, &svm->vmcb01.ptr->save, save); nested_copy_vmcb_control_to_cache(svm, ctl); svm_switch_vmcb(svm, &svm->nested.vmcb02); diff --git a/arch/x86/kvm/svm/svm.c b/arch/x86/kvm/svm/svm.c index 5253c0143666..1e58ad8691e9 100644 --- a/arch/x86/kvm/svm/svm.c +++ b/arch/x86/kvm/svm/svm.c @@ -3210,7 +3210,7 @@ static int svm_set_msr(struct kvm_vcpu *vcpu, struct msr_data *msr) data &= ~DEBUGCTLMSR_BTF; } - if (data & DEBUGCTL_RESERVED_BITS) + if (data & ~svm_get_supported_debugctl(vcpu)) return 1; if (svm->vmcb->save.dbgctl == data) @@ -5164,7 +5164,7 @@ static int svm_enter_smm(struct kvm_vcpu *vcpu, union kvm_smram *smram) BUILD_BUG_ON(offsetof(struct vmcb, save) != 0x400); - svm_copy_vmrun_state(m_save.map.hva + 0x400, &svm->vmcb01.ptr->save); + svm_copy_vmrun_state(vcpu, m_save.map.hva + 0x400, &svm->vmcb01.ptr->save); return 0; } @@ -5204,7 +5204,7 @@ static int svm_leave_smm(struct kvm_vcpu *vcpu, const union kvm_smram *smram) * used during SMM (see svm_enter_smm()) */ - svm_copy_vmrun_state(&svm->vmcb01.ptr->save, m_save.map.hva + 0x400); + svm_copy_vmrun_state(vcpu, &svm->vmcb01.ptr->save, m_save.map.hva + 0x400); /* * Enter the nested guest now diff --git a/arch/x86/kvm/svm/svm.h b/arch/x86/kvm/svm/svm.h index ac6160d78e15..c3380d865b75 100644 --- a/arch/x86/kvm/svm/svm.h +++ b/arch/x86/kvm/svm/svm.h @@ -787,7 +787,10 @@ BUILD_SVM_MSR_BITMAP_HELPERS(bool, test, test) BUILD_SVM_MSR_BITMAP_HELPERS(void, clear, __clear) BUILD_SVM_MSR_BITMAP_HELPERS(void, set, __set) -#define DEBUGCTL_RESERVED_BITS (~DEBUGCTLMSR_LBR) +static inline u64 svm_get_supported_debugctl(struct kvm_vcpu *vcpu) +{ + return DEBUGCTLMSR_LBR; +} /* svm.c */ extern bool dump_invalid_vmcb; @@ -877,7 +880,7 @@ void svm_leave_nested(struct kvm_vcpu *vcpu); void svm_free_nested(struct vcpu_svm *svm); int svm_allocate_nested(struct vcpu_svm *svm); int nested_svm_vmrun(struct kvm_vcpu *vcpu); -void svm_copy_vmrun_state(struct vmcb_save_area *to_save, +void svm_copy_vmrun_state(struct kvm_vcpu *vcpu, struct vmcb_save_area *to_save, struct vmcb_save_area *from_save); void svm_copy_vmloadsave_state(struct vmcb *to_vmcb, struct vmcb *from_vmcb); void nested_svm_vmexit(struct vcpu_svm *svm); -- 2.56.0.rc1.315.gc6ed9934b7-goog