diff options
| author | Pratikkumar Prajapati <pratikkumar.v.prajapati@intel.com> | 2023-05-30 10:38:23 -0700 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2023-06-01 13:13:51 +0000 |
| commit | d4330e7dfb121db43399dd18855b2b9af0f02c4a (patch) | |
| tree | b7d1e6bb5b4614f77da281a791b20a23e4f886ca /src/soc/intel/common | |
| parent | d9ece3daa9c9e09a2d4a3e0cdb64eee39eefb014 (diff) | |
| download | coreboot-d4330e7dfb121db43399dd18855b2b9af0f02c4a.tar.gz coreboot-d4330e7dfb121db43399dd18855b2b9af0f02c4a.tar.bz2 coreboot-d4330e7dfb121db43399dd18855b2b9af0f02c4a.zip | |
soc/intel/common/crashlog: Fix checking PMC record size
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>
Diffstat (limited to 'src/soc/intel/common')
| -rw-r--r-- | src/soc/intel/common/block/acpi/acpi_bert.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/soc/intel/common/block/acpi/acpi_bert.c b/src/soc/intel/common/block/acpi/acpi_bert.c index 7743ccc72fa0..16df60a9d247 100644 --- a/src/soc/intel/common/block/acpi/acpi_bert.c +++ b/src/soc/intel/common/block/acpi/acpi_bert.c @@ -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(); if (pmc_record_size) { - /* Allocate new FW ERR structure in case CPU crashlog is present */ - if (cpu_record_size && !bert_append_fw_err(status)) { + /* Allocate new FW ERR structure in case PMC crashlog is present */ + if (pmc_record_size && !bert_append_fw_err(status)) { printk(BIOS_ERR, "Crashlog PMC entry would " "exceed available region\n"); return CB_ERR; |