Patchwork [Resend] data: klog.json: add message for the LVT offset conflict issue

login
register
mail settings
Submitter Ivan Hu
Date March 11, 2013, 8:54 a.m.
Message ID <1362992056-23984-1-git-send-email-ivan.hu@canonical.com>
Download mbox | patch
Permalink /patch/226499/
State Accepted
Headers show

Comments

Ivan Hu - March 11, 2013, 8:54 a.m.
Some AMD platform that firmware local vector table(LVT) offsets setting
conflict and cause the APIC setup failed.
such as,
https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1116659
Add the advice for these issues.
see: arch/x86/kernel/apic/apic.c, setup_APIC_eilvt()

Signed-off-by: Ivan Hu <ivan.hu@canonical.com>
---
 data/klog.json |   15 +++++++++++++++
 1 file changed, 15 insertions(+)
Colin King - March 11, 2013, 9:09 a.m.
On 11/03/13 08:54, Ivan Hu wrote:
> Some AMD platform that firmware local vector table(LVT) offsets setting
> conflict and cause the APIC setup failed.
> such as,
> https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1116659
> Add the advice for these issues.
> see: arch/x86/kernel/apic/apic.c, setup_APIC_eilvt()
>
> Signed-off-by: Ivan Hu <ivan.hu@canonical.com>
> ---
>   data/klog.json |   15 +++++++++++++++
>   1 file changed, 15 insertions(+)
>
> diff --git a/data/klog.json b/data/klog.json
> index 2847ce0..0f21d41 100644
> --- a/data/klog.json
> +++ b/data/klog.json
> @@ -3396,6 +3396,21 @@
>     },
>     {
>      "compare_mode": "regex",
> +   "log_level": "LOG_LEVEL_MEDIUM",
> +   "tag": "FWTS_TAG_BIOS",
> +   "pattern": "cpu .* try to use APIC.* for vector .*, but the register is already in use for vector .* on another cpu",
> +   "advice": "The local vector table (LVT) offsets that provided by firmware have conflicts, the assignment to the same offset causes the second APIC setup failed.",
> +   "label": "KlogLVTOffsetAssignConflict"
> +  },  {
> +   "compare_mode": "regex",
> +   "log_level": "LOG_LEVEL_MEDIUM",
> +   "tag": "FWTS_TAG_BIOS",
> +   "pattern": "cpu .* try to use APIC.* for vector .*, but the register is already in use for vector .* on this cpu" ,
> +   "advice": "The local vector table (LVT) offsets that provided by firmware have conflicts, the assignment to the same offset causes the second APIC setup failed.",
> +   "label": "KlogLVTOffsetAssignConflict"
> +  },
> +  {
> +   "compare_mode": "regex",
>      "log_level": "LOG_LEVEL_HIGH",
>      "tag": "FWTS_TAG_ACPI",
>      "pattern": "\\[Firmware Bug\\]:.*",
>

Thanks Ivan.

Acked-by: Colin Ian King <colin.king@canonical.com>
Keng-Yu Lin - March 11, 2013, 9:44 a.m.
On Mon, Mar 11, 2013 at 4:54 PM, Ivan Hu <ivan.hu@canonical.com> wrote:
> Some AMD platform that firmware local vector table(LVT) offsets setting
> conflict and cause the APIC setup failed.
> such as,
> https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1116659
> Add the advice for these issues.
> see: arch/x86/kernel/apic/apic.c, setup_APIC_eilvt()
>
> Signed-off-by: Ivan Hu <ivan.hu@canonical.com>
> ---
>  data/klog.json |   15 +++++++++++++++
>  1 file changed, 15 insertions(+)
>
> diff --git a/data/klog.json b/data/klog.json
> index 2847ce0..0f21d41 100644
> --- a/data/klog.json
> +++ b/data/klog.json
> @@ -3396,6 +3396,21 @@
>    },
>    {
>     "compare_mode": "regex",
> +   "log_level": "LOG_LEVEL_MEDIUM",
> +   "tag": "FWTS_TAG_BIOS",
> +   "pattern": "cpu .* try to use APIC.* for vector .*, but the register is already in use for vector .* on another cpu",
> +   "advice": "The local vector table (LVT) offsets that provided by firmware have conflicts, the assignment to the same offset causes the second APIC setup failed.",
> +   "label": "KlogLVTOffsetAssignConflict"
> +  },  {
> +   "compare_mode": "regex",
> +   "log_level": "LOG_LEVEL_MEDIUM",
> +   "tag": "FWTS_TAG_BIOS",
> +   "pattern": "cpu .* try to use APIC.* for vector .*, but the register is already in use for vector .* on this cpu" ,
> +   "advice": "The local vector table (LVT) offsets that provided by firmware have conflicts, the assignment to the same offset causes the second APIC setup failed.",
> +   "label": "KlogLVTOffsetAssignConflict"
> +  },
> +  {
> +   "compare_mode": "regex",
>     "log_level": "LOG_LEVEL_HIGH",
>     "tag": "FWTS_TAG_ACPI",
>     "pattern": "\\[Firmware Bug\\]:.*",
> --
> 1.7.10.4
>

Thanks Ivan. I saw the error a few times too.

Acked-by: Keng-Yu Lin <kengyu@canonical.com>

Patch

diff --git a/data/klog.json b/data/klog.json
index 2847ce0..0f21d41 100644
--- a/data/klog.json
+++ b/data/klog.json
@@ -3396,6 +3396,21 @@ 
   },
   {
    "compare_mode": "regex",
+   "log_level": "LOG_LEVEL_MEDIUM",
+   "tag": "FWTS_TAG_BIOS",
+   "pattern": "cpu .* try to use APIC.* for vector .*, but the register is already in use for vector .* on another cpu",
+   "advice": "The local vector table (LVT) offsets that provided by firmware have conflicts, the assignment to the same offset causes the second APIC setup failed.",
+   "label": "KlogLVTOffsetAssignConflict"
+  },  {
+   "compare_mode": "regex",
+   "log_level": "LOG_LEVEL_MEDIUM",
+   "tag": "FWTS_TAG_BIOS",
+   "pattern": "cpu .* try to use APIC.* for vector .*, but the register is already in use for vector .* on this cpu" ,
+   "advice": "The local vector table (LVT) offsets that provided by firmware have conflicts, the assignment to the same offset causes the second APIC setup failed.",
+   "label": "KlogLVTOffsetAssignConflict"
+  },
+  {
+   "compare_mode": "regex",
    "log_level": "LOG_LEVEL_HIGH",
    "tag": "FWTS_TAG_ACPI",
    "pattern": "\\[Firmware Bug\\]:.*",