ACK: [PATCH] data: klog.json: Add a circular list ACPI warning checks

[IvanHu]

On 04/22/2014 09:14 PM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> ACPICA added a couple of circular list sanity checks in commit
> 0e88479c988c0e8e76a63338bfc09bf8a7c06a58, so add checks for these
> appearing in the kernel log.
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>   data/klog.json | 14 ++++++++++++++
>   1 file changed, 14 insertions(+)
>
> diff --git a/data/klog.json b/data/klog.json
> index 0d6abad..95177bb 100644
> --- a/data/klog.json
> +++ b/data/klog.json
> @@ -2,6 +2,20 @@
>    "firmware_error_warning_patterns":
>    [
>     {
> +   "compare_mode": "string",
> +   "log_level": "LOG_LEVEL_HIGH",
> +   "pattern": "Circular handler list in region object",
> +   "advice": "Internal corruption to the ACPI region handler list has occurred and ACPI has detected this issue while detaching a region and aborted an infinite loop while attempting to traverse the list. This has been known to occur with some firmware tripping a possible internal ACPI bug.",
> +   "label": "KlogAcpiCircularList"
> +  },
> +  {
> +   "compare_mode": "string",
> +   "log_level": "LOG_LEVEL_HIGH",
> +   "pattern": "Circular region list in address handler object",
> +   "advice": "Internal corruption to the ACPI region handler list has occurred and ACPI has detected this issue during deletion of a region object and aborted an infinite loop while attempting to traverse the list. This has been known to occur with some firmware tripping a possible internal ACPI bug.",
> +   "label": "KlogAcpiCircularList"
> +  },
> +  {
>      "compare_mode": "regex",
>      "log_level": "LOG_LEVEL_MEDIUM",
>      "pattern": "Path has too many parent prefixes.*reached beyond root node",
>

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