* [PATCH v5 0/2] introduce UNSUPPORTED
@ 2021-01-26 19:03 Stefano Stabellini
2021-01-26 19:03 ` [PATCH v5 1/2] xen: EXPERT clean-up and " Stefano Stabellini
2021-01-26 19:03 ` [PATCH v5 2/2] xen: add (EXPERT) to one-line description of XEN_SHSTK Stefano Stabellini
0 siblings, 2 replies; 4+ messages in thread
From: Stefano Stabellini @ 2021-01-26 19:03 UTC (permalink / raw)
To: xen-devel
Cc: sstabellini, andrew.cooper3, george.dunlap, iwj, jbeulich, julien, wl
Hi all,
A recent thread [1] has exposed a couple of issues with our current way
of handling EXPERT.
1) It is not obvious that "Configure standard Xen features (expert
users)" is actually the famous EXPERT we keep talking about on xen-devel
2) It is not obvious when we need to enable EXPERT to get a specific
feature
In particular if you want to enable ACPI support so that you can boot
Xen on an ACPI platform, you have to enable EXPERT first. But searching
through the kconfig menu it is really not clear (type '/' and "ACPI"):
nothing in the description tells you that you need to enable EXPERT to
get the option.
This series makes things easier by doing the following:
- introduce a new kconfig option UNSUPPORTED which is clearly to enable
UNSUPPORTED features as defined by SUPPORT.md
- change EXPERT options to UNSUPPORTED where it makes sense: keep
depending on EXPERT for features made for experts
- tag unsupported features by adding (UNSUPPORTED) to the one-line
description
- clarify the EXPERT one-line description
[1] https://marc.info/?l=xen-devel&m=160333101228981
Cheers,
Stefano
Stefano Stabellini (2):
xen: EXPERT clean-up and introduce UNSUPPORTED
xen: add (EXPERT) to one-line description of XEN_SHSTK
xen/Kconfig | 11 ++++++++++-
xen/arch/arm/Kconfig | 6 +++---
xen/arch/x86/Kconfig | 8 ++++----
xen/common/Kconfig | 2 +-
xen/common/sched/Kconfig | 6 +++---
5 files changed, 21 insertions(+), 12 deletions(-)
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH v5 1/2] xen: EXPERT clean-up and introduce UNSUPPORTED
2021-01-26 19:03 [PATCH v5 0/2] introduce UNSUPPORTED Stefano Stabellini
@ 2021-01-26 19:03 ` Stefano Stabellini
2021-01-26 19:03 ` [PATCH v5 2/2] xen: add (EXPERT) to one-line description of XEN_SHSTK Stefano Stabellini
1 sibling, 0 replies; 4+ messages in thread
From: Stefano Stabellini @ 2021-01-26 19:03 UTC (permalink / raw)
To: xen-devel
Cc: sstabellini, andrew.cooper3, george.dunlap, iwj, jbeulich,
julien, wl, Stefano Stabellini
From: Stefano Stabellini <stefano.stabellini@xilinx.com>
A recent thread [1] has exposed a couple of issues with our current way
of handling EXPERT.
1) It is not obvious that "Configure standard Xen features (expert
users)" is actually the famous EXPERT we keep talking about on xen-devel
2) It is not obvious when we need to enable EXPERT to get a specific
feature
In particular if you want to enable ACPI support so that you can boot
Xen on an ACPI platform, you have to enable EXPERT first. But searching
through the kconfig menu it is really not clear (type '/' and "ACPI"):
nothing in the description tells you that you need to enable EXPERT to
get the option.
So this patch makes things easier by doing two things:
- introduce a new kconfig option UNSUPPORTED which is clearly to enable
UNSUPPORTED features as defined by SUPPORT.md
- change EXPERT options to UNSUPPORTED where it makes sense: keep
depending on EXPERT for features made for experts
- tag unsupported features by adding (UNSUPPORTED) to the one-line
description
- clarify the EXPERT one-line description
[1] https://marc.info/?l=xen-devel&m=160333101228981
Signed-off-by: Stefano Stabellini <stefano.stabellini@xilinx.com>
Reviewed-by: Jan Beulich <jbeulich@suse.com> [x86,common]
Reviewed-by: Bertrand Marquis <bertrand.marquis@arm.com>
CC: andrew.cooper3@citrix.com
CC: george.dunlap@citrix.com
CC: iwj@xenproject.org
CC: jbeulich@suse.com
CC: julien@xen.org
CC: wl@xen.org
---
Changes in v5:
- add reviwed-by
- remove changes to ARM_SSBD and HARDEN_BRANCH_PREDICTOR
Changes in v4:
- clarify support statement of UNSUPPORTED
- move UNSUPPORTED past EXPERT
- add default EXPERT to UNSUPPORTED
Changes in v3:
- improve UNSUPPORTED text description
- avoid changing XEN_SHSTK and EFI_SET_VIRTUAL_ADDRESS_MAP
- update HVM_FEP to be UNSUPPORTED
Changes in v2:
- introduce UNSUPPORTED
- don't switch all EXPERT options to UNSUPPORTED
See as reference the v2 thread here:
https://marc.info/?l=xen-devel&m=160566066013723
---
xen/Kconfig | 11 ++++++++++-
xen/arch/arm/Kconfig | 6 +++---
xen/arch/x86/Kconfig | 6 +++---
xen/common/Kconfig | 2 +-
xen/common/sched/Kconfig | 6 +++---
5 files changed, 20 insertions(+), 11 deletions(-)
diff --git a/xen/Kconfig b/xen/Kconfig
index 34c318bfa2..bcbd2758e5 100644
--- a/xen/Kconfig
+++ b/xen/Kconfig
@@ -35,7 +35,7 @@ config DEFCONFIG_LIST
default ARCH_DEFCONFIG
config EXPERT
- bool "Configure standard Xen features (expert users)"
+ bool "Configure EXPERT features"
help
This option allows certain base Xen options and settings
to be disabled or tweaked. This is for specialized environments
@@ -45,6 +45,15 @@ config EXPERT
supported.
default n
+config UNSUPPORTED
+ bool "Configure UNSUPPORTED features"
+ default EXPERT
+ help
+ This option allows certain unsupported Xen options to be changed,
+ which includes non-security-supported, experimental, and tech
+ preview features as defined by SUPPORT.md. (Note that if an option
+ doesn't depend on UNSUPPORTED it doesn't imply that is supported.)
+
config LTO
bool "Link Time Optimisation"
depends on BROKEN
diff --git a/xen/arch/arm/Kconfig b/xen/arch/arm/Kconfig
index c3eb13ea73..330bbf6232 100644
--- a/xen/arch/arm/Kconfig
+++ b/xen/arch/arm/Kconfig
@@ -32,7 +32,7 @@ menu "Architecture Features"
source "arch/Kconfig"
config ACPI
- bool "ACPI (Advanced Configuration and Power Interface) Support" if EXPERT
+ bool "ACPI (Advanced Configuration and Power Interface) Support (UNSUPPORTED)" if UNSUPPORTED
depends on ARM_64
---help---
@@ -49,7 +49,7 @@ config GICV3
If unsure, say Y
config HAS_ITS
- bool "GICv3 ITS MSI controller support" if EXPERT
+ bool "GICv3 ITS MSI controller support (UNSUPPORTED)" if UNSUPPORTED
depends on GICV3 && !NEW_VGIC
config HVM
@@ -104,7 +104,7 @@ config HARDEN_BRANCH_PREDICTOR
If unsure, say Y.
config TEE
- bool "Enable TEE mediators support" if EXPERT
+ bool "Enable TEE mediators support (UNSUPPORTED)" if UNSUPPORTED
default n
help
This option enables generic TEE mediators support. It allows guests
diff --git a/xen/arch/x86/Kconfig b/xen/arch/x86/Kconfig
index 78f351f94b..302334d3e4 100644
--- a/xen/arch/x86/Kconfig
+++ b/xen/arch/x86/Kconfig
@@ -147,7 +147,7 @@ config BIGMEM
If unsure, say N.
config HVM_FEP
- bool "HVM Forced Emulation Prefix support" if EXPERT
+ bool "HVM Forced Emulation Prefix support (UNSUPPORTED)" if UNSUPPORTED
default DEBUG
depends on HVM
---help---
@@ -166,7 +166,7 @@ config HVM_FEP
If unsure, say N.
config TBOOT
- bool "Xen tboot support" if EXPERT
+ bool "Xen tboot support (UNSUPPORTED)" if UNSUPPORTED
default y if !PV_SHIM_EXCLUSIVE
select CRYPTO
---help---
@@ -252,7 +252,7 @@ config HYPERV_GUEST
endif
config MEM_SHARING
- bool "Xen memory sharing support" if EXPERT
+ bool "Xen memory sharing support (UNSUPPORTED)" if UNSUPPORTED
depends on HVM
endmenu
diff --git a/xen/common/Kconfig b/xen/common/Kconfig
index b5c91a1664..39451e8350 100644
--- a/xen/common/Kconfig
+++ b/xen/common/Kconfig
@@ -272,7 +272,7 @@ config LATE_HWDOM
If unsure, say N.
config ARGO
- bool "Argo: hypervisor-mediated interdomain communication" if EXPERT
+ bool "Argo: hypervisor-mediated interdomain communication (UNSUPPORTED)" if UNSUPPORTED
---help---
Enables a hypercall for domains to ask the hypervisor to perform
data transfer of messages between domains.
diff --git a/xen/common/sched/Kconfig b/xen/common/sched/Kconfig
index 61231aacaa..94c9e20139 100644
--- a/xen/common/sched/Kconfig
+++ b/xen/common/sched/Kconfig
@@ -15,7 +15,7 @@ config SCHED_CREDIT2
optimized for lower latency and higher VM density.
config SCHED_RTDS
- bool "RTDS scheduler support (EXPERIMENTAL)"
+ bool "RTDS scheduler support (UNSUPPORTED)" if UNSUPPORTED
default y
---help---
The RTDS scheduler is a soft and firm real-time scheduler for
@@ -23,14 +23,14 @@ config SCHED_RTDS
in the cloud, and general low-latency workloads.
config SCHED_ARINC653
- bool "ARINC653 scheduler support (EXPERIMENTAL)"
+ bool "ARINC653 scheduler support (UNSUPPORTED)" if UNSUPPORTED
default DEBUG
---help---
The ARINC653 scheduler is a hard real-time scheduler for single
cores, targeted for avionics, drones, and medical devices.
config SCHED_NULL
- bool "Null scheduler support (EXPERIMENTAL)"
+ bool "Null scheduler support (UNSUPPORTED)" if UNSUPPORTED
default y
---help---
The null scheduler is a static, zero overhead scheduler,
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH v5 2/2] xen: add (EXPERT) to one-line description of XEN_SHSTK
2021-01-26 19:03 [PATCH v5 0/2] introduce UNSUPPORTED Stefano Stabellini
2021-01-26 19:03 ` [PATCH v5 1/2] xen: EXPERT clean-up and " Stefano Stabellini
@ 2021-01-26 19:03 ` Stefano Stabellini
2021-01-27 8:23 ` Jan Beulich
1 sibling, 1 reply; 4+ messages in thread
From: Stefano Stabellini @ 2021-01-26 19:03 UTC (permalink / raw)
To: xen-devel
Cc: sstabellini, andrew.cooper3, george.dunlap, iwj, jbeulich,
julien, wl, Stefano Stabellini
From: Stefano Stabellini <stefano.stabellini@xilinx.com>
Add an "(EXPERT)" tag to the one-line description of Kconfig options
that depend on EXPERT. (Not where just the prompt depends on EXPERT.)
Today we only have one such option: XEN_SHSTK.
Signed-off-by: Stefano Stabellini <stefano.stabellini@xilinx.com>
CC: andrew.cooper3@citrix.com
CC: george.dunlap@citrix.com
CC: iwj@xenproject.org
CC: jbeulich@suse.com
CC: julien@xen.org
CC: wl@xen.org
---
Changes in v5:
- actually, only change one-line description of options that depends on
EXPERT (and not just the prompt)
Changes in v4:
- new patch
---
xen/arch/x86/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/xen/arch/x86/Kconfig b/xen/arch/x86/Kconfig
index 302334d3e4..3f630b89e8 100644
--- a/xen/arch/x86/Kconfig
+++ b/xen/arch/x86/Kconfig
@@ -103,7 +103,7 @@ config HVM
If unsure, say Y.
config XEN_SHSTK
- bool "Supervisor Shadow Stacks"
+ bool "Supervisor Shadow Stacks (EXPERT)"
depends on HAS_AS_CET_SS && EXPERT
default y
---help---
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v5 2/2] xen: add (EXPERT) to one-line description of XEN_SHSTK
2021-01-26 19:03 ` [PATCH v5 2/2] xen: add (EXPERT) to one-line description of XEN_SHSTK Stefano Stabellini
@ 2021-01-27 8:23 ` Jan Beulich
0 siblings, 0 replies; 4+ messages in thread
From: Jan Beulich @ 2021-01-27 8:23 UTC (permalink / raw)
To: Stefano Stabellini
Cc: andrew.cooper3, george.dunlap, iwj, julien, wl,
Stefano Stabellini, xen-devel
On 26.01.2021 20:03, Stefano Stabellini wrote:
> From: Stefano Stabellini <stefano.stabellini@xilinx.com>
>
> Add an "(EXPERT)" tag to the one-line description of Kconfig options
> that depend on EXPERT. (Not where just the prompt depends on EXPERT.)
>
> Today we only have one such option: XEN_SHSTK.
>
> Signed-off-by: Stefano Stabellini <stefano.stabellini@xilinx.com>
Acked-by: Jan Beulich <jbeulich@suse.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-01-27 8:23 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-26 19:03 [PATCH v5 0/2] introduce UNSUPPORTED Stefano Stabellini
2021-01-26 19:03 ` [PATCH v5 1/2] xen: EXPERT clean-up and " Stefano Stabellini
2021-01-26 19:03 ` [PATCH v5 2/2] xen: add (EXPERT) to one-line description of XEN_SHSTK Stefano Stabellini
2021-01-27 8:23 ` Jan Beulich
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).