ACK: [PATCH] klog: data.json: Add klog checking for kernel EC errors from drivers/acpi/ec.c

[ivanhu]

On 2016年07月06日 16:53, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> Add more klog error message checking for ACPI EC kernel messages
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>   data/klog.json | 21 +++++++++++++++++++++
>   1 file changed, 21 insertions(+)
>
> diff --git a/data/klog.json b/data/klog.json
> index 6926b72..2767031 100644
> --- a/data/klog.json
> +++ b/data/klog.json
> @@ -3,6 +3,27 @@
>    [
>     {
>      "compare_mode": "string",
> +   "log_level": "LOG_LEVEL_LOW",
> +   "pattern": "stale EC events cleared",
> +   "advice": "The ACPI Embedded Controller (EC) driver has cleared the maximum number of pending ACPI _Q events that have accumulated in the EC before giving up. There may be more events pending that have not been cleared.",
> +   "label": "KlogAcpiEcStaleEvents"
> +  },
> +  {
> +   "compare_mode": "string",
> +   "log_level": "LOG_LEVEL_MEDIUM",
> +   "pattern": "failed to remove space handler",
> +   "advice": "The ACPI Embedded Controller (EC) driver failed to remove the EC address space handler.",
> +   "label": "KlogAcpiEcHandlerRemoveFailed"
> +  },
> +  {
> +   "compare_mode": "string",
> +   "log_level": "LOG_LEVEL_MEDIUM",
> +   "pattern": "failed to remove gpe handler",
> +   "advice": "The ACPI Embedded Controller (EC) driver failed to remove the EC General Purpose Events (GPE) handler.",
> +   "label": "KlogAcpiEcHandlerRemoveFailed"
> +  },
> +  {
> +   "compare_mode": "string",
>      "log_level": "LOG_LEVEL_MEDIUM",
>      "pattern": "While parsing/handling command line",
>      "advice": "The ACPI debugger failed to parse the debug command",
>

Acked-by: Ivan Hu <ivan.hu at canonical.com>