GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

soc/intel/xeon_sp: Fix very small total memory when CXL is enabled 

Processor attached memory should not use reserved_ram_from_to and
treat the calculation of gi_mem_size size as 64MB.

By default SOC_INTEL_HAS_CXL is enabled for Sapphire Rapids platforms,
this should fix small total memory issue. Before the fix running
command 'free -g -h' under Linux shows the total memory is only 1.4Gi,
after the fix it's showing the expected total memory size 15Gi.

Tested=On AC without attaching CXL memory, the total memory size is
the same as de-selecting SOC_INTEL_HAS_CXL.
On OCP Crater Lake with CXL memory attached, CXL memory can be recognized
in NUMA node 1:
numactl -H
available: 2 nodes (0-1)
node 0 cpus: 0 1 2 3 4 5 6 .. 59
node 0 size: 95854 MB
node 0 free: 93860 MB
node 1 cpus:
node 1 size: 63488 MB
node 1 free: 63488 MB
node distances:
node   0   1
  0:  10  14
  1:  14  10

Change-Id: I38e9d138fd284620ac616a65f444e943f1774869
Signed-off-by: Johnny Lin <johnny_lin@wiwynn.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/74296
Reviewed-by: Patrick Rudolph <patrick.rudolph@9elements.com>
Reviewed-by: Shuo Liu <shuo.liu@intel.com>
Reviewed-by: Johnny Lin <Johnny_Lin@wiwynn.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Lean Sheng Tan <sheng.tan@9elements.com>
This commit is contained in:
Johnny Lin 2023-04-11 15:30:02 +08:00 committed by Lean Sheng Tan
parent 31f502a6be
commit 514930c2af
1 changed files with 13 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
src/soc/intel/xeon_sp/uncore.c
View File

@ -249,10 +249,16 @@ static void mc_add_dram_resources(struct device *dev, int *res_count)
if (CONFIG(SOC_INTEL_HAS_CXL)) { if (CONFIG(SOC_INTEL_HAS_CXL)) {
/* 4GiB -> CXL Memory */ /* 4GiB -> CXL Memory */
uint32_t gi_mem_size; uint32_t gi_mem_size;
gi_mem_size = get_generic_initiator_mem_size(); gi_mem_size = get_generic_initiator_mem_size(); /* unit: 64MB */
/*
res = reserved_ram_from_to(dev, index++, 0x100000000, * Memory layout when there is CXL HDM (Host-managed Device Memory):
mc_values[TOHM_REG] - (uint64_t)gi_mem_size + 1); * -------------- <- TOHM
* CXL memory regions (pds global variable records the base/size of them)
* Processor attached high memory
* -------------- <- 0x100000000 (4GB)
*/
res = upper_ram_end(dev, index++,
mc_values[TOHM_REG] - ((uint64_t)gi_mem_size << 26) + 1);
LOG_RESOURCE("high_ram", dev, res); LOG_RESOURCE("high_ram", dev, res);
/* CXL Memory */ /* CXL Memory */
@ -269,8 +275,9 @@ static void mc_add_dram_resources(struct device *dev, int *res_count)
else else
flags |= IORESOURCE_STORED; flags |= IORESOURCE_STORED;
res = fixed_mem_range_flags(dev, index++, (uint64_t)pds.pds[i].base, res = fixed_mem_range_flags(dev, index++,
(uint64_t)pds.pds[i].size, flags); (uint64_t)pds.pds[i].base << 26,
(uint64_t)pds.pds[i].size << 26, flags);
if (cxl_mode == CXL_SPM) if (cxl_mode == CXL_SPM)
LOG_RESOURCE("specific_purpose_memory", dev, res); LOG_RESOURCE("specific_purpose_memory", dev, res);
else else