diff mbox

data/klog.json: Scan for all the ACPI_ERROR_METHOD kernel errors

Message ID 1354550423-13524-1-git-send-email-colin.king@canonical.com
State Accepted
Headers show

Commit Message

Colin Ian King Dec. 3, 2012, 4 p.m. UTC
From: Colin Ian King <colin.king@canonical.com>

Scan the kernel log for all the ACPI_ERROR_METHOD kernel errors.  We
relax some old existing rules to make this easier for newer kernels
add add in some new messages too.  Also update some of the advice to
make it a little more meaningful.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
 data/klog.json | 30 +++++++++++++++++++++++-------
 1 file changed, 23 insertions(+), 7 deletions(-)

Comments

Keng-Yu Lin Dec. 6, 2012, 7:27 a.m. UTC | #1
On Tue, Dec 4, 2012 at 12:00 AM, Colin King <colin.king@canonical.com> wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> Scan the kernel log for all the ACPI_ERROR_METHOD kernel errors.  We
> relax some old existing rules to make this easier for newer kernels
> add add in some new messages too.  Also update some of the advice to
> make it a little more meaningful.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
>  data/klog.json | 30 +++++++++++++++++++++++-------
>  1 file changed, 23 insertions(+), 7 deletions(-)
>
> diff --git a/data/klog.json b/data/klog.json
> index ed6d559..9084fac 100644
> --- a/data/klog.json
> +++ b/data/klog.json
> @@ -2625,11 +2625,19 @@
>     "compare_mode": "regex",
>     "log_level": "LOG_LEVEL_HIGH",
>     "tag": "FWTS_TAG_ACPI_PARSE_EXEC_FAIL",
> -   "pattern": "ACPI Error.*Method parse\/execution failed.*AE_AML_NO_RETURN_VALUE",
> +   "pattern": "Method parse\/execution failed.*AE_AML_NO_RETURN_VALUE",
>     "advice": "The ACPI Method was expected to return a value and did not.",
>     "label": "KlogAcpiMethodNoReturnValue"
>    },
>    {
> +   "compare_mode": "string",
> +   "log_level": "LOG_LEVEL_HIGH",
> +   "tag": "FWTS_TAG_ACPI_PARSE_EXEC_FAIL",
> +   "pattern": "Method parse/execution failed",
> +   "advice": "The ACPI driver encounter a parsing error on some ACPI AML code. This is unexepcted and normally indicates that there is a bug in the compiled AML code.  It could be a programming fault or an AML compiler issue.  See acpi_ps_parse_aml().",
> +   "label": "KlogAcpiParseExecAmlFailed"
> +  },
> +  {
>     "compare_mode": "regex",
>     "log_level": "LOG_LEVEL_HIGH",
>     "tag": "FWTS_TAG_ACPI_PARSE_EXEC_FAIL",
> @@ -3286,11 +3294,19 @@
>     "label": "KlogAcpiExecFailure"
>    },
>    {
> -   "compare_mode": "regex",
> +   "compare_mode": "string",
>     "log_level": "LOG_LEVEL_HIGH",
>     "tag": "FWTS_TAG_ACPI_PARSE_EXEC_FAIL",
> -   "pattern": "\\*\\*\\* Error.*Method execution failed",
> -   "advice": "Execution of an ACPI AML method failed.",
> +   "pattern": "No object was returned from",
> +   "advice": "The ACPI interpreter was requested to evaluate an ACPI namespace object and was expecting an ACPI object to be returned, but nothing was returned.  This appears to be an ACPI AML bug.",
> +   "label": "KlogAcpiEvalObjNoObjReturned"
> +  },
> +  {
> +   "compare_mode": "string",
> +   "log_level": "LOG_LEVEL_HIGH",
> +   "tag": "FWTS_TAG_ACPI_PARSE_EXEC_FAIL",
> +   "pattern": "Method execution failed",
> +   "advice": "Execution of an ACPI AML method failed. This is normally a bug in the ACPI AML code",
>     "label": "KlogAcpiExecFailure"
>    },
>    {
> @@ -3310,11 +3326,11 @@
>     "label": "KlogAcpiRecursionTooDeep"
>    },
>    {
> -   "compare_mode": "regex",
> +   "compare_mode": "string",
>     "log_level": "LOG_LEVEL_HIGH",
>     "tag": "FWTS_TAG_ACPI_METHOD_RETURN",
> -   "pattern": "\\*\\*\\* Error.*Return object type is incorrect",
> -   "advice": "Return object type is not the correct type, this is an AML error in the DSDT or SSDT",
> +   "pattern": "Return object type is incorrect",
> +   "advice": "Return object type is not the correct type, this is an AML error in the DSDT or SSDT. See also acpi_ut_evaluate_object().",
>     "label": "KlogAcpiReturnObjectTypeIncorrect"
>    }
>   ],
> --
> 1.8.0
>
Acked-by: Keng-Yu Lin <kengyu@canonical.com>
Alex Hung Dec. 12, 2012, 8:05 a.m. UTC | #2
On 12/04/2012 12:00 AM, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> Scan the kernel log for all the ACPI_ERROR_METHOD kernel errors.  We
> relax some old existing rules to make this easier for newer kernels
> add add in some new messages too.  Also update some of the advice to
> make it a little more meaningful.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
>   data/klog.json | 30 +++++++++++++++++++++++-------
>   1 file changed, 23 insertions(+), 7 deletions(-)
>
> diff --git a/data/klog.json b/data/klog.json
> index ed6d559..9084fac 100644
> --- a/data/klog.json
> +++ b/data/klog.json
> @@ -2625,11 +2625,19 @@
>      "compare_mode": "regex",
>      "log_level": "LOG_LEVEL_HIGH",
>      "tag": "FWTS_TAG_ACPI_PARSE_EXEC_FAIL",
> -   "pattern": "ACPI Error.*Method parse\/execution failed.*AE_AML_NO_RETURN_VALUE",
> +   "pattern": "Method parse\/execution failed.*AE_AML_NO_RETURN_VALUE",
>      "advice": "The ACPI Method was expected to return a value and did not.",
>      "label": "KlogAcpiMethodNoReturnValue"
>     },
>     {
> +   "compare_mode": "string",
> +   "log_level": "LOG_LEVEL_HIGH",
> +   "tag": "FWTS_TAG_ACPI_PARSE_EXEC_FAIL",
> +   "pattern": "Method parse/execution failed",
> +   "advice": "The ACPI driver encounter a parsing error on some ACPI AML code. This is unexepcted and normally indicates that there is a bug in the compiled AML code.  It could be a programming fault or an AML compiler issue.  See acpi_ps_parse_aml().",
> +   "label": "KlogAcpiParseExecAmlFailed"
> +  },
> +  {
>      "compare_mode": "regex",
>      "log_level": "LOG_LEVEL_HIGH",
>      "tag": "FWTS_TAG_ACPI_PARSE_EXEC_FAIL",
> @@ -3286,11 +3294,19 @@
>      "label": "KlogAcpiExecFailure"
>     },
>     {
> -   "compare_mode": "regex",
> +   "compare_mode": "string",
>      "log_level": "LOG_LEVEL_HIGH",
>      "tag": "FWTS_TAG_ACPI_PARSE_EXEC_FAIL",
> -   "pattern": "\\*\\*\\* Error.*Method execution failed",
> -   "advice": "Execution of an ACPI AML method failed.",
> +   "pattern": "No object was returned from",
> +   "advice": "The ACPI interpreter was requested to evaluate an ACPI namespace object and was expecting an ACPI object to be returned, but nothing was returned.  This appears to be an ACPI AML bug.",
> +   "label": "KlogAcpiEvalObjNoObjReturned"
> +  },
> +  {
> +   "compare_mode": "string",
> +   "log_level": "LOG_LEVEL_HIGH",
> +   "tag": "FWTS_TAG_ACPI_PARSE_EXEC_FAIL",
> +   "pattern": "Method execution failed",
> +   "advice": "Execution of an ACPI AML method failed. This is normally a bug in the ACPI AML code",
>      "label": "KlogAcpiExecFailure"
>     },
>     {
> @@ -3310,11 +3326,11 @@
>      "label": "KlogAcpiRecursionTooDeep"
>     },
>     {
> -   "compare_mode": "regex",
> +   "compare_mode": "string",
>      "log_level": "LOG_LEVEL_HIGH",
>      "tag": "FWTS_TAG_ACPI_METHOD_RETURN",
> -   "pattern": "\\*\\*\\* Error.*Return object type is incorrect",
> -   "advice": "Return object type is not the correct type, this is an AML error in the DSDT or SSDT",
> +   "pattern": "Return object type is incorrect",
> +   "advice": "Return object type is not the correct type, this is an AML error in the DSDT or SSDT. See also acpi_ut_evaluate_object().",
>      "label": "KlogAcpiReturnObjectTypeIncorrect"
>     }
>    ],
>
Acked-by: Alex Hung <alex.hung@canonical.com>
diff mbox

Patch

diff --git a/data/klog.json b/data/klog.json
index ed6d559..9084fac 100644
--- a/data/klog.json
+++ b/data/klog.json
@@ -2625,11 +2625,19 @@ 
    "compare_mode": "regex",
    "log_level": "LOG_LEVEL_HIGH",
    "tag": "FWTS_TAG_ACPI_PARSE_EXEC_FAIL",
-   "pattern": "ACPI Error.*Method parse\/execution failed.*AE_AML_NO_RETURN_VALUE",
+   "pattern": "Method parse\/execution failed.*AE_AML_NO_RETURN_VALUE",
    "advice": "The ACPI Method was expected to return a value and did not.",
    "label": "KlogAcpiMethodNoReturnValue"
   },
   {
+   "compare_mode": "string",
+   "log_level": "LOG_LEVEL_HIGH",
+   "tag": "FWTS_TAG_ACPI_PARSE_EXEC_FAIL",
+   "pattern": "Method parse/execution failed",
+   "advice": "The ACPI driver encounter a parsing error on some ACPI AML code. This is unexepcted and normally indicates that there is a bug in the compiled AML code.  It could be a programming fault or an AML compiler issue.  See acpi_ps_parse_aml().",
+   "label": "KlogAcpiParseExecAmlFailed"
+  },
+  {
    "compare_mode": "regex",
    "log_level": "LOG_LEVEL_HIGH",
    "tag": "FWTS_TAG_ACPI_PARSE_EXEC_FAIL",
@@ -3286,11 +3294,19 @@ 
    "label": "KlogAcpiExecFailure"
   },
   {
-   "compare_mode": "regex",
+   "compare_mode": "string",
    "log_level": "LOG_LEVEL_HIGH",
    "tag": "FWTS_TAG_ACPI_PARSE_EXEC_FAIL",
-   "pattern": "\\*\\*\\* Error.*Method execution failed",
-   "advice": "Execution of an ACPI AML method failed.",
+   "pattern": "No object was returned from",
+   "advice": "The ACPI interpreter was requested to evaluate an ACPI namespace object and was expecting an ACPI object to be returned, but nothing was returned.  This appears to be an ACPI AML bug.",
+   "label": "KlogAcpiEvalObjNoObjReturned"
+  },
+  {
+   "compare_mode": "string",
+   "log_level": "LOG_LEVEL_HIGH",
+   "tag": "FWTS_TAG_ACPI_PARSE_EXEC_FAIL",
+   "pattern": "Method execution failed",
+   "advice": "Execution of an ACPI AML method failed. This is normally a bug in the ACPI AML code",
    "label": "KlogAcpiExecFailure"
   },
   {
@@ -3310,11 +3326,11 @@ 
    "label": "KlogAcpiRecursionTooDeep"
   },
   {
-   "compare_mode": "regex",
+   "compare_mode": "string",
    "log_level": "LOG_LEVEL_HIGH",
    "tag": "FWTS_TAG_ACPI_METHOD_RETURN",
-   "pattern": "\\*\\*\\* Error.*Return object type is incorrect",
-   "advice": "Return object type is not the correct type, this is an AML error in the DSDT or SSDT",
+   "pattern": "Return object type is incorrect",
+   "advice": "Return object type is not the correct type, this is an AML error in the DSDT or SSDT. See also acpi_ut_evaluate_object().",
    "label": "KlogAcpiReturnObjectTypeIncorrect"
   } 
  ],