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

login
register
mail settings
Submitter Ivan Hu
Date March 11, 2013, 7:35 a.m.
Message ID <1362987328-23156-1-git-send-email-ivan.hu@canonical.com>
Download mbox | patch
Permalink /patch/226486/
State Rejected
Headers show

Comments

Ivan Hu - March 11, 2013, 7:35 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, 8:41 a.m.
On 11/03/13 07:35, 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..a0288eb 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": "LVTOffsetAssignConflict"
> +  },  {
> +   "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": "LVTOffsetAssignConflict"
> +  },
> +  {
> +   "compare_mode": "regex",
>      "log_level": "LOG_LEVEL_HIGH",
>      "tag": "FWTS_TAG_ACPI",
>      "pattern": "\\[Firmware Bug\\]:.*",
>

Just one thing,  klog labels have "Klog" as a prefix, so we should have:

"label": "KlogLVTOffsetAssignConflict"

..it is one of those historical design issues in the fwts log scanning..

Colin

Patch

diff --git a/data/klog.json b/data/klog.json
index 2847ce0..a0288eb 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": "LVTOffsetAssignConflict"
+  },  {
+   "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": "LVTOffsetAssignConflict"
+  },
+  {
+   "compare_mode": "regex",
    "log_level": "LOG_LEVEL_HIGH",
    "tag": "FWTS_TAG_ACPI",
    "pattern": "\\[Firmware Bug\\]:.*",