Patchwork [4/6] data: klog.json: ACPICA ACPI_ERROR messages

login
register
mail settings
Submitter Colin King
Date Nov. 15, 2012, 10:56 a.m.
Message ID <1352977013-19927-5-git-send-email-colin.king@canonical.com>
Download mbox | patch
Permalink /patch/199247/
State Accepted
Headers show

Comments

Colin King - Nov. 15, 2012, 10:56 a.m.
From: Colin Ian King <colin.king@canonical.com>

ACPICA ACPI_ERROR messages from exsystem.c

Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
 data/klog.json |    8 ++++++++
 1 file changed, 8 insertions(+)
Alex Hung - Nov. 22, 2012, 2:14 a.m.
On 11/15/2012 06:56 PM, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> ACPICA ACPI_ERROR messages from exsystem.c
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
>   data/klog.json |    8 ++++++++
>   1 file changed, 8 insertions(+)
>
> diff --git a/data/klog.json b/data/klog.json
> index 98f67c4..1420105 100644
> --- a/data/klog.json
> +++ b/data/klog.json
> @@ -79,6 +79,14 @@
>     {
>      "compare_mode": "string",
>      "log_level": "LOG_LEVEL_HIGH",
> +   "tag": "FWTS_TAG_ACPI",
> +   "pattern": "Time parameter is too large",
> +   "advice": "ACPI Stall() op was requested to wait for more than 255 usec.  The ACPI specification states that the maximum should be 100 usec but to accommodate some firmware a maximum of 255 is supported by the kernel.",
> +   "label": "KlogAcpiStallTooLarge"
> +  },
> +  {
> +   "compare_mode": "string",
> +   "log_level": "LOG_LEVEL_HIGH",
>      "tag": "FWTS_TAG_ACPI_BRIGHTNESS",
>      "pattern": "Evaluating _BCM failed",
>      "advice": "Could not set the brightness level, the ACPI control _BCM failed to execute correctly.",
>
Acked-by: Alex Hung <alex.hung@canonical.com>
Keng-Yu Lin - Nov. 22, 2012, 5:46 a.m.
On Thu, Nov 15, 2012 at 6:56 PM, Colin King <colin.king@canonical.com> wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> ACPICA ACPI_ERROR messages from exsystem.c
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
>  data/klog.json |    8 ++++++++
>  1 file changed, 8 insertions(+)
>
> diff --git a/data/klog.json b/data/klog.json
> index 98f67c4..1420105 100644
> --- a/data/klog.json
> +++ b/data/klog.json
> @@ -79,6 +79,14 @@
>    {
>     "compare_mode": "string",
>     "log_level": "LOG_LEVEL_HIGH",
> +   "tag": "FWTS_TAG_ACPI",
> +   "pattern": "Time parameter is too large",
> +   "advice": "ACPI Stall() op was requested to wait for more than 255 usec.  The ACPI specification states that the maximum should be 100 usec but to accommodate some firmware a maximum of 255 is supported by the kernel.",
> +   "label": "KlogAcpiStallTooLarge"
> +  },
> +  {
> +   "compare_mode": "string",
> +   "log_level": "LOG_LEVEL_HIGH",
>     "tag": "FWTS_TAG_ACPI_BRIGHTNESS",
>     "pattern": "Evaluating _BCM failed",
>     "advice": "Could not set the brightness level, the ACPI control _BCM failed to execute correctly.",
> --
> 1.7.10.4
>
Acked-by: Keng-Yu Lin <kengyu@canonical.com>

Patch

diff --git a/data/klog.json b/data/klog.json
index 98f67c4..1420105 100644
--- a/data/klog.json
+++ b/data/klog.json
@@ -79,6 +79,14 @@ 
   {
    "compare_mode": "string",
    "log_level": "LOG_LEVEL_HIGH",
+   "tag": "FWTS_TAG_ACPI",
+   "pattern": "Time parameter is too large",
+   "advice": "ACPI Stall() op was requested to wait for more than 255 usec.  The ACPI specification states that the maximum should be 100 usec but to accommodate some firmware a maximum of 255 is supported by the kernel.",
+   "label": "KlogAcpiStallTooLarge"
+  },
+  {
+   "compare_mode": "string",
+   "log_level": "LOG_LEVEL_HIGH",
    "tag": "FWTS_TAG_ACPI_BRIGHTNESS",
    "pattern": "Evaluating _BCM failed",
    "advice": "Could not set the brightness level, the ACPI control _BCM failed to execute correctly.",