2022-08-10 05:21:44

by Lukas Bulwahn

[permalink] [raw]
Subject: [PATCH 1/4] xen: x86: set the config XEN_512GB after config change

Commit c70727a5bc18 ("xen: allow more than 512 GB of RAM for 64 bit
pv-domains") from July 2015 replaces the config XEN_MAX_DOMAIN_MEMORY with
a new config XEN_512GB, but misses to adjust arch/x86/configs/xen.config.

Adjust this config fragment now.

Signed-off-by: Lukas Bulwahn <[email protected]>
---
arch/x86/configs/xen.config | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/x86/configs/xen.config b/arch/x86/configs/xen.config
index d9fc7139fd46..8a6c88f48e75 100644
--- a/arch/x86/configs/xen.config
+++ b/arch/x86/configs/xen.config
@@ -14,7 +14,7 @@ CONFIG_CPU_FREQ=y

# x86 xen specific config options
CONFIG_XEN_PVH=y
-CONFIG_XEN_MAX_DOMAIN_MEMORY=500
+CONFIG_XEN_512GB=y
CONFIG_XEN_SAVE_RESTORE=y
# CONFIG_XEN_DEBUG_FS is not set
CONFIG_XEN_MCE_LOG=y
--
2.17.1


2022-08-10 07:17:08

by Jan Beulich

[permalink] [raw]
Subject: Re: [PATCH 1/4] xen: x86: set the config XEN_512GB after config change

On 10.08.2022 07:07, Lukas Bulwahn wrote:
> --- a/arch/x86/configs/xen.config
> +++ b/arch/x86/configs/xen.config
> @@ -14,7 +14,7 @@ CONFIG_CPU_FREQ=y
>
> # x86 xen specific config options
> CONFIG_XEN_PVH=y
> -CONFIG_XEN_MAX_DOMAIN_MEMORY=500
> +CONFIG_XEN_512GB=y

Does this actually need setting here? The option's default is y, so
it ought to be turned on "automatically". Hence I think it's only
the stale line which wants deleting.

Jan

2022-08-17 04:44:38

by Lukas Bulwahn

[permalink] [raw]
Subject: Re: [PATCH 1/4] xen: x86: set the config XEN_512GB after config change

On Wed, Aug 10, 2022 at 9:07 AM Jan Beulich <[email protected]> wrote:
>
> On 10.08.2022 07:07, Lukas Bulwahn wrote:
> > --- a/arch/x86/configs/xen.config
> > +++ b/arch/x86/configs/xen.config
> > @@ -14,7 +14,7 @@ CONFIG_CPU_FREQ=y
> >
> > # x86 xen specific config options
> > CONFIG_XEN_PVH=y
> > -CONFIG_XEN_MAX_DOMAIN_MEMORY=500
> > +CONFIG_XEN_512GB=y
>
> Does this actually need setting here? The option's default is y, so
> it ought to be turned on "automatically". Hence I think it's only
> the stale line which wants deleting.
>

Jan, that makes sense. I will send a patch v2 to do so.

Lukas