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

soc/intel/common/crashlog: Fix checking PMC record size

Browse source 
Check pmc_record_size variable for collecting PMC records,
instead of cpu_record_size variable.

Bug=None
TEST=Able to build. With Meteor Lake SOC related patch, able to
capture and decode crashlog.

Change-Id: I4c35ba2bcf757231aa2872802eb82d4d50742cd9
Signed-off-by: Pratikkumar Prajapati <pratikkumar.v.prajapati@intel.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/75526
Reviewed-by: Subrata Banik <subratabanik@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Pratikkumar Prajapati 2023-05-30 10:38:23 -07:00 committed by Felix Held
parent d9ece3daa9
commit d4330e7dfb
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/soc/intel/common/block/acpi/acpi_bert.c
View file

@ -68,8 +68,8 @@ static enum cb_err record_crashlog_into_bert(void **region, size_t *length)
pmc_record_size = cl_get_pmc_record_size(); pmc_record_size = cl_get_pmc_record_size();
if (pmc_record_size) { if (pmc_record_size) {
/* Allocate new FW ERR structure in case CPU crashlog is present */ /* Allocate new FW ERR structure in case PMC crashlog is present */
if (cpu_record_size && !bert_append_fw_err(status)) { if (pmc_record_size && !bert_append_fw_err(status)) {
printk(BIOS_ERR, "Crashlog PMC entry would " printk(BIOS_ERR, "Crashlog PMC entry would "
"exceed available region\n"); "exceed available region\n");
return CB_ERR; return CB_ERR;