ERST isn't a mandatory table, and also isn't very common to find. The message is unnecessary noise during boot. Furthermore, it is redundant with the list of found ACPI tables printed just ahead.
Signed-off-by: Andrew Cooper <[email protected]> --- CC: Jan Beulich <[email protected]> --- xen/drivers/acpi/apei/erst.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/xen/drivers/acpi/apei/erst.c b/xen/drivers/acpi/apei/erst.c index 23a4e822e7..c5df512b98 100644 --- a/xen/drivers/acpi/apei/erst.c +++ b/xen/drivers/acpi/apei/erst.c @@ -791,10 +791,9 @@ int __init erst_init(void) return -ENODEV; status = acpi_get_table_phys(ACPI_SIG_ERST, 0, &erst_addr, &erst_len); - if (status == AE_NOT_FOUND) { - printk(KERN_INFO "ERST table was not found\n"); + if (status == AE_NOT_FOUND) return -ENODEV; - } + if (ACPI_FAILURE(status)) { const char *msg = acpi_format_exception(status); printk(KERN_WARNING "Failed to get ERST table: %s\n", msg); -- 2.11.0 _______________________________________________ Xen-devel mailing list [email protected] https://lists.xenproject.org/mailman/listinfo/xen-devel
