2021-03-05 01:01:52

by Mike Travis

[permalink] [raw]
Subject: [PATCH] x86/platform/uv: Add set of kernel block size for hubless arches

The commit below added a call to set the block size value that is needed
to set the value used by the kernel. This was done for UV Hubbed systems.
This commit adds that same set call to hubless systems which supports the
same NVRAMS and Intel BIOS thus the same problem occurs.

Fixes: bbbd2b51a2aa ...
Add a call to the new function to "adjust" the current fixed
UV memory block size of 2GB so it can be changed to a different
physical boundary. This accommodates changes in the Intel BIOS,
and therefore UV BIOS, which now can align boundaries different
than the previous UV standard of 2GB.

Signed-off-by: Mike Travis <[email protected]>
Reviewed-by: Steve Wahl <[email protected]>
Reviewed-by: Russ Anderson <[email protected]>
---
arch/x86/kernel/apic/x2apic_uv_x.c | 3 +++
1 file changed, 3 insertions(+)

diff --git a/arch/x86/kernel/apic/x2apic_uv_x.c b/arch/x86/kernel/apic/x2apic_uv_x.c
index 52bc217ca8c3..c9ddd233e32f 100644
--- a/arch/x86/kernel/apic/x2apic_uv_x.c
+++ b/arch/x86/kernel/apic/x2apic_uv_x.c
@@ -1671,6 +1671,9 @@ static __init int uv_system_init_hubless(void)
if (rc < 0)
return rc;

+ /* Set section block size for current node memory */
+ set_block_size();
+
/* Create user access node */
if (rc >= 0)
uv_setup_proc_files(1);
--
2.21.0


2021-03-05 13:19:55

by Borislav Petkov

[permalink] [raw]
Subject: Re: [PATCH] x86/platform/uv: Add set of kernel block size for hubless arches

On Thu, Mar 04, 2021 at 04:39:55PM -0600, Mike Travis wrote:
> The commit below added a call to set the block size value that is needed
> to set the value used by the kernel. This was done for UV Hubbed systems.
> This commit adds that same set call to hubless systems which supports the
> same NVRAMS and Intel BIOS thus the same problem occurs.
>
> Fixes: bbbd2b51a2aa ...

The format is:

Fixes: bbbd2b51a2aa ("x86/platform/UV: Use new set memory block size function")

and you don't need to paste the commit text.

And if you add a Fixes tag, then this is supposed to fix something but
I'm unclear what exactly this fixes?

IOW, should this patch be backported to stable because UV hubless
systems would somehow fail without it?

Thx.

--
Regards/Gruss,
Boris.

https://people.kernel.org/tglx/notes-about-netiquette