ACK: [PATCH] klog.json: Add some more kernel messages to klog data base

[ivanhu]

On 2017年02月06日 18:43, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>  data/klog.json | 70 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 70 insertions(+)
>
> diff --git a/data/klog.json b/data/klog.json
> index 1f81444..0e3a3d1 100644
> --- a/data/klog.json
> +++ b/data/klog.json
> @@ -2,6 +2,76 @@
>   "firmware_error_warning_patterns":
>   [
>    {
> +   "compare_mode": "regex",
> +   "log_level": "LOG_LEVEL_MEDIUM",
> +   "pattern": "ACPI MWAIT C-state .* not supported by HW",
> +   "advice": "The ACPI _CST (C-State) object is reporting a CPU C-State that the CPU does not actually support (as determined by CPUID_MWAIT_LEAF)",
> +   "label": "KlogAcpiCstateNotSupported",
> +  },
> +  {
> +   "compare_mode": "regex",
> +   "log_level": "LOG_LEVEL_MEDIUM",
> +   "pattern": "Invalid character.* in name .*, repaired:",
> +   "advice": "An ACPI object name contained an invalid character and the ACPI core fixed this up. This incorrectly named object could cause incorrect execution behaviour of ACPI AMI code.",
> +   "label": "KlogAcpiInvalidObjectName",
> +  },
> +  {
> +   "compare_mode": "regex",
> +   "log_level": "LOG_LEVEL_LOW",
> +   "pattern": "Platform took .* usec to enter ACPI mode",
> +   "advice": "An ACPI GPE took longer than expected to enter ACPI mode. This could lead to unexpected behaviour",
> +   "label": "KlogAcpiGpeEventEnterAcpiTooSlow",
> +  },
> +  {
> +   "compare_mode": "string",
> +   "log_level": "LOG_LEVEL_LOW",
> +   "pattern": "Cannot release the ACPI Global Lock, it has not been acquired",
> +   "advice": "An attempt to release ACPI Global Lock failed, it had not been acquired previously.",
> +   "label": "KlogAcpiGlobalLockReleaseFailed",
> +  },
> +  {
> +   "compare_mode": "regex",
> +   "log_level": "LOG_LEVEL_HIGH",
> +   "pattern": "UtTrackAllocation: Allocation .* already present in global list",
> +   "advice": "The ACPI memory tracking debug has detected an allocation that already exists in the global list of, this indicates a bug in the ACPICA core.",
> +   "label": "KlogAcpiUtTrackDuplicateAllocation",
> +  },
> +  {
> +   "compare_mode": "string",
> +   "log_level": "LOG_LEVEL_HIGH",
> +   "pattern": "Attempt to delete a NULL address",
> +   "advice": "The ACPI memory tracking debug has detected an attempt to delete a NULL address, this indicates a bug in the ACPICA core.",
> +   "label": "KlogAcpiUtTrackNullAddrDelete",
> +  },
> +  {
> +   "compare_mode": "string",
> +   "log_level": "LOG_LEVEL_HIGH",
> +   "pattern": "The lid device is not compliant to SW_LID",
> +   "advice": "The ACPI button driver has detected a non compliant LID device switch. This can occur because the platform reports the lid as closed immediately after booting or resuming, or because the device never reports open events.  The ACPI button driver has inverted the LID switch event to try to ensure expected behaviour",
> +   "label": "KlogAcpiLidInvertedSwitchEvent",
> +  },
> +  {
> +   "compare_mode": "regex",
> +   "log_level": "LOG_LEVEL_LOW",
> +   "pattern": "DMI detected: .*force ACPI _REV to 5",
> +   "advice": "The machine is a DELL that has switches sound between HDA and I2S depending on what the ACPI _REV object returns. The machine has been quirked to force _REV to return 5 to indicated sound should use HDA",
> +   "label": "KlogAcpiDellRev5Quirked",
> +  },
> +  {
> +   "compare_mode": "string",
> +   "log_level": "LOG_LEVEL_LOW",
> +   "pattern": "sysfs_get_dirent 'nfit' failed",
> +   "advice": "The ACPI NFIT driver failed to get the NFIT sysfs directory entry",
> +   "label": "KlogNfitSysfsDirentFailed",
> +  },
> +  {
> +   "compare_mode": "string",
> +   "log_level": "LOG_LEVEL_LOW",
> +   "pattern": "sysfs_get_dirent 'scrub' failed",
> +   "advice": "The ACPI NFIT driver failed to get the NFIT scrub sysfs directory entry",
> +   "label": "KlogNfitScrubSysfsDirentFailed",
> +  },
> +  {
>     "compare_mode": "string",
>     "log_level": "LOG_LEVEL_MEDIUM",
>     "pattern": "PCI: unsupported BIOS32 revision",
>

Acked-by: Ivan Hu <ivan.hu at canonical.com>