[PATCH] Sanity check for MSR_IA32_ENERGY_PERF_BIAS messages.

[Keng-Yü Lin]

On Tue, Jan 3, 2012 at 7:55 PM, Colin King <colin.king at canonical.com> wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> Since commit abe48b108247e9b90b4c6739662a2e5c765ed114 the kernel has
> been able to detect if the MSR_IA32_ENERGY_PERF_BIAS was set incorrectly
> by the firmware.  We may as well use this information to catch any
> bad MSR_IA32_ENERGY_PERF_BIAS defaults.
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>  data/klog.json |    7 +++++++
>  1 files changed, 7 insertions(+), 0 deletions(-)
>
> diff --git a/data/klog.json b/data/klog.json
> index 4d66a61..4598754 100644
> --- a/data/klog.json
> +++ b/data/klog.json
> @@ -69,6 +69,13 @@
>  [
>   {
>    "compare_mode": "regex",
> +   "log_level": "LOG_LEVEL_HIGH",
> +   "tag": "FWTS_TAG_BIOS",
> +   "pattern": "ENERGY_PERF_BIAS.*Set to 'normal'.* was 'performance'",
> +   "advice": "The MSR_IA32_ENERGY_PERF_BIAS was initialized incorrectly and was defaulted to a high performance bias setting.  The kernel has detected this and changed it down to a 'normal' bias setting.  The BIOS vendor should be informed that the MSR is not defaulted to the normal bias setting."
> +  },
> +  {
> +   "compare_mode": "regex",
>    "log_level": "LOG_LEVEL_CRITICAL",
>    "tag": "FWTS_TAG_ACPI",
>    "pattern": "pnp.*can't evaluate _CRS: 12311",
> --
> 1.7.5.4
>

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