Re: [Xen-devel] [PATCH] ACPI/ERST: Name ERST table in otherwise opaque error messages

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 19.03.13 at 23:55, Andrew Cooper <andrew.cooper3@xxxxxxxxxx> wrote:
> Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> 
> diff -r a6b81234b189 -r 7fdef201c0b4 xen/drivers/acpi/apei/erst.c
> --- a/xen/drivers/acpi/apei/erst.c
> +++ b/xen/drivers/acpi/apei/erst.c
> @@ -781,11 +781,11 @@ int __init erst_init(void)
>       status = acpi_get_table(ACPI_SIG_ERST, 0,
>                               (struct acpi_table_header **)&erst_tab);
>       if (status == AE_NOT_FOUND) {
> -             printk(KERN_ERR "Table is not found!\n");
> +             printk(KERN_ERR "ERST table is not found!\n");
>               return -ENODEV;
>       } else if (ACPI_FAILURE(status)) {
>               const char *msg = acpi_format_exception(status);
> -             printk(KERN_ERR "Failed to get table, %s\n", msg);
> +             printk(KERN_ERR "Failed to get ESRT table, %s\n", msg);

I'll fix the spelling, and once at it also lower the severities (these
really aren't errors - the former is informational, while the latter
is no more than a warning imo).

Jan

>               return -EINVAL;
>       }
>  




_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel