erebion/coreboot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

arch/x86/acpi_bert_storage.c: rename check -> proc_err_info

Browse source 
The name check is a bit confusing, since it is not a check structure.
The check structure is below it.

Change-Id: I000e9e5f2ce8210fce76ef81b4242150d02fceed
Signed-off-by: Maximilian Brune <maximilian.brune@9elements.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/90478
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
This commit is contained in:
Maximilian Brune 2025-08-27 20:32:19 +02:00 • committed by Matt DeVillier
commit 2b9653cf34
1 changed files with 6 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

12
src/arch/x86/acpi_bert_storage.c
View file

@ -386,7 +386,7 @@ cper_ia32x64_proc_error_info_t *new_cper_ia32x64_check(
cper_ia32x64_proc_error_section_t *x86err,
enum cper_x86_check_type type)
{
cper_ia32x64_proc_error_info_t *check;
cper_ia32x64_proc_error_info_t *proc_err_info;
static const char * const check_names[] = {
"cache",
"TLB",
@ -409,19 +409,19 @@ cper_ia32x64_proc_error_info_t *new_cper_ia32x64_check(
return NULL;
}
check = bert_allocate_storage(sizeof(*check));
if (!check) {
proc_err_info = bert_allocate_storage(sizeof(*proc_err_info));
if (!proc_err_info) {
printk(BIOS_ERR, "New IA32X64 %s check entry would exceed available region\n",
check_names[type]);
return NULL;
}
revise_error_sizes(status, sizeof(*check));
revise_error_sizes(status, sizeof(*proc_err_info));
guidcpy(&check->type, &check_guids[type]);
guidcpy(&proc_err_info->type, &check_guids[type]);
cper_bump_ia32x64_chk_count(x86err);
return check;
return proc_err_info;
}
/* Helper to append an ACPI Generic Error Data Entry plus a CPER IA32/X64