From: Tony Luck <tony.luck@xxxxxxxxx> Although the ACPI specification says this is a "one time polled" error source, it makes no mention that the OS should stomp on fields in the error status block to enforce this. There are user mode BERT harvesting tools that dig into /sys/firmware/acpi/tables/BERT which might like to see this summary value. Drop the: estatus->block_status = 0; Cc: Len Brown <lenb@xxxxxxxxxx> Cc: Huang Ying <ying.huang@xxxxxxxxx> Cc: Borislav Petkov <bp@xxxxxxx> Cc: Tomasz Nowicki <tomasz.nowicki@xxxxxxxxxx> Cc: Jonathan (Zhixiong) Zhang <zjzhang@xxxxxxxxxxxxxx> Cc: Tyler Baicar <tbaicar@xxxxxxxxxxxxxx> Cc: linux-acpi@xxxxxxxxxxxxxxx Signed-off-by: Tony Luck <tony.luck@xxxxxxxxx> --- drivers/acpi/apei/bert.c | 7 ------- 1 file changed, 7 deletions(-) diff --git a/drivers/acpi/apei/bert.c b/drivers/acpi/apei/bert.c index 12771fcf0417..9197555dc3c7 100644 --- a/drivers/acpi/apei/bert.c +++ b/drivers/acpi/apei/bert.c @@ -62,13 +62,6 @@ static void __init bert_print_all(struct acpi_bert_region *region, cper_estatus_print(KERN_INFO HW_ERR, estatus); - /* - * Because the boot error source is "one-time polled" type, - * clear Block Status of current Generic Error Status Block, - * once it's printed. - */ - estatus->block_status = 0; - estatus = (void *)estatus + estatus_len; /* No more error records. */ if (!estatus->block_status) -- 2.11.0 -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
