diff mbox

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

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

Commit Message

Colin Ian King Nov. 6, 2012, 11:49 a.m. UTC
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(+)

Comments

Keng-Yu Lin Nov. 8, 2012, 3:04 a.m. UTC | #1
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. UTC | #2
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>
diff mbox

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",