[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[PATCH 10/11] xen/x86: remove "depends on !PV_SHIM_EXCLUSIVE"


  • To: <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Penny Zheng <Penny.Zheng@xxxxxxx>
  • Date: Mon, 7 Jul 2025 11:13:45 +0800
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=lists.xenproject.org smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0)
  • 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=qJ0yWQQ3+ezfc84j0CJ2nrRzdn7Cpfu7Pu0S/BuaQIE=; b=FFa/DUGkluWVgzSOt8NvVeKgwaGbIUCDQ1JS77cTNYn8BRoCfhK2or69FQPaPniFOp/4+VGZjeUpreKJFyGyTX3kaUw5UUZBaenyBK1GfN35HDq0jE4Pkl48epC1zOzt67VoXMAVtFBKKoT0MKT56z/2V9TSsc6l1siai5olNXc3ItSXBCh8E9ZAdxLGx+eBkj/BniveQPs6vz9AXqc8I79a9KxZvXrExYKLzS6pYn+3igwwJf5HpCZIYGJ50e0LvONEl9npQwl0Q19cAlWJmeYMOzm1q0udj471l9tY/3LC0v9h++ZIY1xzz7GbHIlmu9aevFt25gqYijtW02ZXrw==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=G9sh/t4tziq4DB/Ko8P2Dbht7DdhXZJ+0qNmmhO1UGd9e4C9W4bu4i7uFRnk6X9djh4fNufcKvBozEIsr3b3yi5NgeRs7qskp2ECr24puKtFVyKHXeB61IzENkm7KjU/3NRxtD0/tqo1eZvvYlxQa5JgYYOdgKssrRtyIDD+y6DPn2tcxaaPpqIa6rBmqDSfMx+VPX/yiLxi4EEatNUqC3GaCnkRokMhc2XBpmcvtiJi5e6pLG3T64+EEKbqy3WG9xraKpjbO1lvvbn5lcyg8ue8mVfQZ6nw37XDPoTk5zNlB4U0OipjrytFBzuSSnptMcqT1UQnckJhdzwjidXJnA==
  • Cc: <ray.huang@xxxxxxx>, Penny Zheng <Penny.Zheng@xxxxxxx>, Jan Beulich <jbeulich@xxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>, "Julien Grall" <julien@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>
  • Delivery-date: Mon, 07 Jul 2025 03:14:43 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

Remove all "depends on !PV_SHIM_EXCLUSIVE" (also the functionally
equivalent "if !...") in Kconfig file, since negative dependancy will badly
affect allyesconfig. Although "if !PV_SHIM_EXCLUSIVE" for CONFIG_VGA is not
truly a dependency, setting PV_SHIM_EXCLUSIVE y still makes it unconfigurable.
So we remove it here too

Add "#CONFIG_xxx is not set" for above options in presets for x86 PV shim, as
the explicit declaration is to ephasize setting for the shim is different
from the general default.

Signed-off-by: Penny Zheng <Penny.Zheng@xxxxxxx>
Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
---
This commit shall be commited together with the next one, which is also the
last commit.
---
v2 -> v3:
- remove comment for PV_SHIM_EXCLUSIVE
---
v3 -> v4:
- explicitly state "CONFIG_xxx is not set" in "pvshim_defconfig"
- Add "default y" for SHADOW_PAGING and TBOOT
- refactor commit message
---
v4 -> v5:
- For not breaking allyesconfig, changes to defaults are actually not needed.
So remove them all
- Leave one blank lines
---
v5 -> v6
- add description for change: removing "if !PV_SHIM_EXCLUSIVE" for CONFIG_VGA
---
 xen/arch/x86/Kconfig      | 4 ----
 xen/arch/x86/hvm/Kconfig  | 1 -
 xen/drivers/video/Kconfig | 2 +-
 3 files changed, 1 insertion(+), 6 deletions(-)

diff --git a/xen/arch/x86/Kconfig b/xen/arch/x86/Kconfig
index 752d5141bb..b45f17a16e 100644
--- a/xen/arch/x86/Kconfig
+++ b/xen/arch/x86/Kconfig
@@ -289,8 +289,6 @@ config PV_SHIM_EXCLUSIVE
 
          If unsure, say N.
 
-if !PV_SHIM_EXCLUSIVE
-
 config HYPERV_GUEST
        bool "Hyper-V Guest"
        select GUEST
@@ -299,8 +297,6 @@ config HYPERV_GUEST
 
          If unsure, say N.
 
-endif
-
 config REQUIRE_NX
        bool "Require NX (No eXecute) support"
        help
diff --git a/xen/arch/x86/hvm/Kconfig b/xen/arch/x86/hvm/Kconfig
index 2def0f98e2..b903764bda 100644
--- a/xen/arch/x86/hvm/Kconfig
+++ b/xen/arch/x86/hvm/Kconfig
@@ -1,6 +1,5 @@
 menuconfig HVM
        bool "HVM support"
-       depends on !PV_SHIM_EXCLUSIVE
        default !PV_SHIM
        select COMPAT
        select IOREQ_SERVER
diff --git a/xen/drivers/video/Kconfig b/xen/drivers/video/Kconfig
index 245030beea..0a51e87eb2 100644
--- a/xen/drivers/video/Kconfig
+++ b/xen/drivers/video/Kconfig
@@ -3,7 +3,7 @@ config VIDEO
        bool
 
 config VGA
-       bool "VGA support" if !PV_SHIM_EXCLUSIVE
+       bool "VGA support"
        select VIDEO
        depends on X86
        default y if !PV_SHIM_EXCLUSIVE
-- 
2.34.1




 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.