From 222f1272ba8fe2f9a504ff9cd655d6f0e6b2e056 Mon Sep 17 00:00:00 2001 From: Martin Roth Date: Thu, 20 Oct 2022 16:58:31 -0600 Subject: soc/amd/common: Initialize STB Spill-to-DRAM Signed-off-by: Martin Roth Change-Id: I547671d2bcfe011566466665b14e151b8ec05430 Reviewed-on: https://review.coreboot.org/c/coreboot/+/68927 Tested-by: build bot (Jenkins) Reviewed-by: Fred Reitberger Reviewed-by: Jason Glenesk --- src/soc/amd/common/block/include/amdblocks/stb.h | 1 - src/soc/amd/common/block/smu/Makefile.inc | 3 +++ src/soc/amd/common/block/stb/Kconfig | 18 +++++++++++++ src/soc/amd/common/block/stb/stb.c | 33 ++++++++++++++++++++++++ 4 files changed, 54 insertions(+), 1 deletion(-) (limited to 'src/soc') diff --git a/src/soc/amd/common/block/include/amdblocks/stb.h b/src/soc/amd/common/block/include/amdblocks/stb.h index c8851f89a3..db032b9dde 100644 --- a/src/soc/amd/common/block/include/amdblocks/stb.h +++ b/src/soc/amd/common/block/include/amdblocks/stb.h @@ -16,7 +16,6 @@ struct stb_entry_struct { }; void write_stb_to_console(void); -void init_spill_buffer(void); void add_stb_to_timestamp_buffer(void); #endif /* AMD_BLOCK_STB_H */ diff --git a/src/soc/amd/common/block/smu/Makefile.inc b/src/soc/amd/common/block/smu/Makefile.inc index 2afd81abde..13720a3ccf 100644 --- a/src/soc/amd/common/block/smu/Makefile.inc +++ b/src/soc/amd/common/block/smu/Makefile.inc @@ -1 +1,4 @@ smm-$(CONFIG_SOC_AMD_COMMON_BLOCK_SMU) += smu.c +bootblock-$(CONFIG_SOC_AMD_COMMON_BLOCK_SMU) += smu.c +romstage-$(CONFIG_SOC_AMD_COMMON_BLOCK_SMU) += smu.c +ramstage-$(CONFIG_SOC_AMD_COMMON_BLOCK_SMU) += smu.c diff --git a/src/soc/amd/common/block/stb/Kconfig b/src/soc/amd/common/block/stb/Kconfig index fe2b5b7a1a..28c78fd4df 100644 --- a/src/soc/amd/common/block/stb/Kconfig +++ b/src/soc/amd/common/block/stb/Kconfig @@ -1,6 +1,7 @@ config SOC_AMD_COMMON_BLOCK_STB bool select SOC_AMD_COMMON_BLOCK_SMN + select SOC_AMD_COMMON_BLOCK_SMU help Select in the SOC if it supports the Smart Trace Buffer @@ -9,11 +10,28 @@ if SOC_AMD_COMMON_BLOCK_STB config WRITE_STB_BUFFER_TO_CONSOLE bool "Write STB entries to the console log" default n + depends on !ENABLE_STB_SPILL_TO_DRAM help This option will tell coreboot to print the STB buffer at various points through the boot process. Note that this will prevent the entries from being stored if the Spill-to-DRAM feature is enabled. +config ENABLE_STB_SPILL_TO_DRAM + bool "Enable Smart Trace Buffer Spill-to-DRAM" + default n + help + Spill-to-DRAM is an STB feature that extends the buffer from using + just the small SRAM buffer to a much larger area reserved in main + memory. + +config AMD_STB_SIZE_IN_MB + int "Smart Trace Buffer Spill-to-DRAM buffer size in MB" + default 3 + range 3 16 + depends on ENABLE_STB_SPILL_TO_DRAM + help + Size of the STB Spill-to-DRAM buffer in MB. + config ADD_POSTCODES_TO_STB bool "Add coreboot postcodes to STB" default y diff --git a/src/soc/amd/common/block/stb/stb.c b/src/soc/amd/common/block/stb/stb.c index 0cea5c3d67..934a49b344 100644 --- a/src/soc/amd/common/block/stb/stb.c +++ b/src/soc/amd/common/block/stb/stb.c @@ -1,9 +1,13 @@ /* SPDX-License-Identifier: GPL-2.0-only */ +#include "commonlib/bsd/cb_err.h" #include +#include #include #include +#include #include +#include #include #define STB_ENTRIES_PER_ROW 4 @@ -59,9 +63,37 @@ void write_stb_to_console(void) if ((i % STB_ENTRIES_PER_ROW) == STB_ENTRIES_PER_ROW - 1) printk(BIOS_DEBUG, "\n"); printed_data = 1; + } +} + +static void init_spill_buffer(void *unused) +{ + struct smu_payload smu_payload = {0}; + uintptr_t stb; + uint32_t size = CONFIG_AMD_STB_SIZE_IN_MB * MiB; + int i; + if (!CONFIG(ENABLE_STB_SPILL_TO_DRAM)) + return; + + stb = (uintptr_t)cbmem_add(CBMEM_ID_AMD_STB, size); + if (!stb) { + printk(BIOS_ERR, "Could not allocate cbmem buffer for STB\n"); + return; } + smu_payload.msg[0] = (uint32_t)stb; + smu_payload.msg[1] = 0; + smu_payload.msg[2] = size; + + printk(BIOS_DEBUG, "STB spill buffer: allocated %d MiB at %lx\n", + CONFIG_AMD_STB_SIZE_IN_MB, stb); + + if (send_smu_message(SMC_MSG_SET_S2D_ADDR, &smu_payload) == CB_ERR) + printk(BIOS_ERR, "Could not enable STB Spill-to-dram\n"); + + for (i = 0; i < SMU_NUM_ARGS; i++) + printk(BIOS_DEBUG, "smu_payload.msg[%d]: 0x%x\n", i, smu_payload.msg[i]); } static void final_stb_console(void *unused) @@ -70,4 +102,5 @@ static void final_stb_console(void *unused) write_stb_to_console(); } +BOOT_STATE_INIT_ENTRY(BS_PRE_DEVICE, BS_ON_ENTRY, init_spill_buffer, NULL); BOOT_STATE_INIT_ENTRY(BS_PAYLOAD_BOOT, BS_ON_ENTRY, final_stb_console, NULL); -- cgit v1.2.3