linux-kernel - Re: [tip:x86/cpu] x86, intel, power: Correct the MSR_IA32_ENERGY_PERF

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <1313688187.3202.26.camel@lovely>
Date:	Thu, 18 Aug 2011 19:23:07 +0200
From:	Christoph Fritz <chf.fritz@...glemail.com>
To:	"H. Peter Anvin" <hpa@...or.com>, Len Brown <lenb@...nel.org>
Cc:	stable@...nel.org, Ingo Molnar <mingo@...hat.com>,
	Thomas Gleixner <tglx@...utronix.de>,
	linux-kernel@...r.kernel.org
Subject: Re: [tip:x86/cpu] x86, intel, power: Correct the
 MSR_IA32_ENERGY_PERF_BIAS message

On Sat, 2011-07-16 at 18:52 +0000, tip-bot for Len Brown wrote:
> Commit-ID:  17edf2d79f1ea6dfdb4c444801d928953b9f98d6
> Gitweb:     http://git.kernel.org/tip/17edf2d79f1ea6dfdb4c444801d928953b9f98d6
> Author:     Len Brown <len.brown@...el.com>
> AuthorDate: Fri, 15 Jul 2011 17:37:15 -0400
> Committer:  H. Peter Anvin <hpa@...ux.intel.com>
> CommitDate: Fri, 15 Jul 2011 15:13:55 -0700
> 
> x86, intel, power: Correct the MSR_IA32_ENERGY_PERF_BIAS message
> 
> Fix the printk_once() so that it actually prints (didn't print before
> due to a stray comma.)
> 
> [ hpa: changed to an incremental patch and adjusted the description
>   accordingly. ]
> 
> Signed-off-by: Len Brown <len.brown@...el.com>
> Link: http://lkml.kernel.org/r/alpine.LFD.2.02.1107151732480.18606@x980
> Cc: <table@...nel.org>

This "table" should be stable :)

Cc: <stable@...nel.org>
Reviewed-by: Christoph Fritz <chf.fritz@...glemail.com>

Thanks,
 Christoph

> Signed-off-by: H. Peter Anvin <hpa@...ux.intel.com>
> ---
>  arch/x86/kernel/cpu/intel.c |   10 +++++-----
>  1 files changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/arch/x86/kernel/cpu/intel.c b/arch/x86/kernel/cpu/intel.c
> index da0d779..ed6086e 100644
> --- a/arch/x86/kernel/cpu/intel.c
> +++ b/arch/x86/kernel/cpu/intel.c
> @@ -465,11 +465,11 @@ static void __cpuinit init_intel(struct cpuinfo_x86 *c)
>  		u64 epb;
>  
>  		rdmsrl(MSR_IA32_ENERGY_PERF_BIAS, epb);
> -		if ((epb & 0xF) == 0) {
> -			printk_once(KERN_WARNING, "x86: updated energy_perf_bias"
> -				" to 'normal' from 'performance'\n"
> -				"You can view and update epb via utility,"
> -				" such as x86_energy_perf_policy(8)\n");
> +		if ((epb & 0xF) == ENERGY_PERF_BIAS_PERFORMANCE) {
> +			printk_once(KERN_WARNING "ENERGY_PERF_BIAS:"
> +				" Set to 'normal', was 'performance'\n"
> +				"ENERGY_PERF_BIAS: View and update with"
> +				" x86_energy_perf_policy(8)\n");
>  			epb = (epb & ~0xF) | ENERGY_PERF_BIAS_NORMAL;
>  			wrmsrl(MSR_IA32_ENERGY_PERF_BIAS, epb);
>  		}
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/