ACK: [PATCH] klog: change the MSR_IA32_ENERGY_PERF_BIAS test from fauilure to info

[Alex Hung]

On 10/25/2012 01:44 PM, Ivan Hu wrote:
> Intel IA32 program guild doesn't mention that this MSR should be set by
> firmare, instead, it mentions "Software can use whatever criteria it sees
> fit to program the MSR with the appropriate value.". Discuss with IBV (AMI),
> they'd like to keep this MSR as default, and let OS to change value if desired.
>
> Signed-off-by: Ivan Hu <ivan.hu at canonical.com>
> ---
>   data/klog.json |    4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/data/klog.json b/data/klog.json
> index 653b592..65d4545 100644
> --- a/data/klog.json
> +++ b/data/klog.json
> @@ -76,10 +76,10 @@
>     },
>     {
>      "compare_mode": "regex",
> -   "log_level": "LOG_LEVEL_HIGH",
> +   "log_level": "LOG_LEVEL_INFO",
>      "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."
> +   "advice": "This is not exactly a failure but a warning from the kernel. The MSR_IA32_ENERGY_PERF_BIAS was initialized and defaulted to a high performance bias setting. The kernel has detected this and changed it down to a 'normal' bias setting."
>     },
>     {
>      "compare_mode": "regex",
>
Acked-by: Alex Hung <alex.hung at canonical.com>