[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [XEN][PATCH v5] x86: make Viridian support optional
- To: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
- From: Grygorii Strashko <grygorii_strashko@xxxxxxxx>
- Date: Fri, 31 Oct 2025 17:37:16 +0200
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=epam.com; dmarc=pass action=none header.from=epam.com; dkim=pass header.d=epam.com; arc=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=jYpG/V3hyL1rFtK/XM294qm1s0OtrnGX6CFA97tMHoU=; b=i/AWUXfJpsjc/DmYGTtPVwVkartaQt2Nb85++d1+H94ZyFm1oXb1tW/ARBfAJ7FKXNen/u3LIGbujZnUxlWRvQmj4mveBuWKb0inRbFu7i7ImqULrHkmagnzJpgr6kHQrn19x/YIr395X0OnNhm8UmWm0fOtCxg9UKOGZHeCYb2aK4uvGd1wiGcqqSMgMVpNhA23BSxVGl54iSW1ZYdTi9Bn5/7xbBIh8euoNor0Y0lGXyr90CNq+uF8yPvLNLIk7HMUXWOMoPQCN/aM6l61lTwb2sIPdIG++8REGxjhgPyVgQcuejamEoX3D2b8qcYt7D4+bGaYvKqmwNN7oldxMQ==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=dB34uLa0wylYsSXnzwuEea7oiw0ibB0lftPRzZ5kl0MSk/3J9l6mX9aFceGRXf3MzxFrOCnGz2LUGBhYrwwOSC4vdi168DcHUhcTPaAh8LcuseY7O27sMQBaqDd3lolsbZVf/9wisQsYlGP+NTonGm8LUTXvAnT7SplrPKtvu/jtyaYZCyt4VGyqo09qI/56Ul9ryX5/VUUeT8fXzS80D+T6JxO4BboX7Yq2eQavjTMM8Em8CI/eHq18jP048KP3RgniAkm0jKKhx7TTv2ZsFdigiPOjxLzk1meVrl+jiURkNkKPCQkUIK9oGQ/A2erRDoNpZMMydaccyuhcEFwGbw==
- Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=epam.com;
- Cc: Sergiy Kibrik <Sergiy_Kibrik@xxxxxxxx>, Jan Beulich <jbeulich@xxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Paul Durrant <paul@xxxxxxx>
- Delivery-date: Fri, 31 Oct 2025 15:37:35 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
Hi
It's actually v6, sorry.
On 23.10.25 18:18, Grygorii Strashko wrote:
From: Sergiy Kibrik <Sergiy_Kibrik@xxxxxxxx>
Add config option VIRIDIAN that covers viridian code within HVM.
Calls to viridian functions guarded by is_viridian_domain() and related macros.
Having this option may be beneficial by reducing code footprint for systems
that are not using Hyper-V.
[grygorii_strashko@xxxxxxxx: fixed NULL pointer deref in
viridian_save_domain_ctxt(); stub viridian_vcpu/domain_init/deinit()]
Signed-off-by: Sergiy Kibrik <Sergiy_Kibrik@xxxxxxxx>
Signed-off-by: Grygorii Strashko <grygorii_strashko@xxxxxxxx>
---
changes in v6:
- add stubs for viridian_vcpu/domain_init/deinit()
- update Kconfig description
- make set(HVM_PARAM_VIRIDIAN) return -ENODEV
if (!IS_ENABLED(CONFIG_VIRIDIAN) && value)
changes in v5:
- drop "depends on AMD_SVM || INTEL_VMX"
- return -EILSEQ from viridian_load_x() if !VIRIDIAN
changes in v4:
- s/HVM_VIRIDIAN/VIRIDIAN
- add "depends on AMD_SVM || INTEL_VMX"
- add guard !is_viridian_vcpu() checks in
viridian_load_vcpu_ctxt/viridian_load_domain_ctxt
changes in v3:
- fixed NULL pointer deref in viridian_save_domain_ctxt() reported for v2,
which caused v2 revert by commit 1fffcf10cd71 ("Revert "x86: make Viridian
support optional"")
[...]
Best regards,
-grygorii
|
