Patchwork data: klog.json: Add support for E820 handler in 3.2.x Kernels

login
register
mail settings
Submitter Colin King
Date Nov. 6, 2012, 11:49 a.m.
Message ID <1352202578-21293-1-git-send-email-colin.king@canonical.com>
Download mbox | patch
Permalink /patch/197470/
State Accepted
Headers show

Comments

Colin King - Nov. 6, 2012, 11:49 a.m.
From: Colin Ian King <colin.king@canonical.com>

Add support to check for messages in E820 driver

Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
 data/klog.json |    8 ++++++++
 1 file changed, 8 insertions(+)
Keng-Yu Lin - Nov. 8, 2012, 3:04 a.m.
On Tue, Nov 6, 2012 at 12:49 PM, Colin King <colin.king@canonical.com> wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> Add support to check for messages in E820 driver
>
> 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 766d115..1b5a402 100644
> --- a/data/klog.json
> +++ b/data/klog.json
> @@ -165,6 +165,14 @@
>     "label": "KlogE820TooManyEntries"
>    },
>    {
> +   "compare_mode": "string",
> +   "log_level": "LOG_LEVEL_CRITICAL",
> +   "tag": "FWTS_TAG_ACPI",
> +   "pattern": "Ooops! Too many entries in the memory map!",
> +   "advice": "Legacy E820 BIOS has a limit of 128 entries in the E820 memory map but the firmware seems to have more than this. For EFI Numa hardare, the limit is 128 + 3 * maximum number of nodes.  The kernel has ignore the extra entries.  This could lead to regions of memory being used by the kernel when in fact is should be reserved and this can lead to unexpected behaviour.",
> +   "label": "KlogE820TooManyEntries"
> +  },
> +  {
>     "compare_mode": "regex",
>     "log_level": "LOG_LEVEL_LOW",
>     "tag": "FWTS_TAG_ACPI",
> --
> 1.7.10.4
>
Acked-by: Keng-Yu Lin <kengyu@canonical.com>
Alex Hung - Nov. 21, 2012, 3:20 a.m.
On 11/06/2012 07:49 PM, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> Add support to check for messages in E820 driver
>
> 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 766d115..1b5a402 100644
> --- a/data/klog.json
> +++ b/data/klog.json
> @@ -165,6 +165,14 @@
>      "label": "KlogE820TooManyEntries"
>     },
>     {
> +   "compare_mode": "string",
> +   "log_level": "LOG_LEVEL_CRITICAL",
> +   "tag": "FWTS_TAG_ACPI",
> +   "pattern": "Ooops! Too many entries in the memory map!",
> +   "advice": "Legacy E820 BIOS has a limit of 128 entries in the E820 memory map but the firmware seems to have more than this. For EFI Numa hardare, the limit is 128 + 3 * maximum number of nodes.  The kernel has ignore the extra entries.  This could lead to regions of memory being used by the kernel when in fact is should be reserved and this can lead to unexpected behaviour.",
> +   "label": "KlogE820TooManyEntries"
> +  },
> +  {
>      "compare_mode": "regex",
>      "log_level": "LOG_LEVEL_LOW",
>      "tag": "FWTS_TAG_ACPI",
>
Acked-by: Alex Hung <alex.hung@canonical.com>

Patch

diff --git a/data/klog.json b/data/klog.json
index 766d115..1b5a402 100644
--- a/data/klog.json
+++ b/data/klog.json
@@ -165,6 +165,14 @@ 
    "label": "KlogE820TooManyEntries"
   },
   {
+   "compare_mode": "string",
+   "log_level": "LOG_LEVEL_CRITICAL",
+   "tag": "FWTS_TAG_ACPI",
+   "pattern": "Ooops! Too many entries in the memory map!",
+   "advice": "Legacy E820 BIOS has a limit of 128 entries in the E820 memory map but the firmware seems to have more than this. For EFI Numa hardare, the limit is 128 + 3 * maximum number of nodes.  The kernel has ignore the extra entries.  This could lead to regions of memory being used by the kernel when in fact is should be reserved and this can lead to unexpected behaviour.",
+   "label": "KlogE820TooManyEntries"
+  },
+  {
    "compare_mode": "regex",
    "log_level": "LOG_LEVEL_LOW",
    "tag": "FWTS_TAG_ACPI",