linux-kernel - Re: [-v2 PATCH 2/6] powernow-k8: Add core performance boost support

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 for Android: free password hash cracker in your pocket

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

Message-Id: <20100330154229.dc8ebd24.akpm@linux-foundation.org>
Date:	Tue, 30 Mar 2010 15:42:29 -0700
From:	Andrew Morton <akpm@...ux-foundation.org>
To:	Borislav Petkov <bp@...64.org>
Cc:	davej@...hat.com, trenn@...e.de, linux@...inikbrodowski.net,
	<mingo@...e.hu>, <hpa@...or.com>, <tglx@...utronix.de>,
	cpufreq@...r.kernel.org, <x86@...nel.org>,
	<linux-kernel@...r.kernel.org>
Subject: Re: [-v2 PATCH 2/6] powernow-k8: Add core performance boost support

On Fri, 26 Mar 2010 14:39:56 +0100
Borislav Petkov <bp@...64.org> wrote:

> From: Borislav Petkov <borislav.petkov@....com>
> 
> Starting with F10h, revE, AMD processors add support for a dynamic
> core boosting feature called Core Performance Boost. When a specific
> condition is present, a subset of the cores on a system are boosted
> beyond their P0 operating frequency to speed up the performance of
> single-threaded applications.
> 
> In the normal case, the system comes out of reset with core boosting
> enabled. This patch adds a sysfs knob with which core boosting can be
> switched on or off for benchmarking purposes.
> 
> While at it, cleanup the driver init codepath and update copyrights.
> 
> ...
>
> +static void _cpb_toggle_msrs(bool t)
> +{
> +	int cpu;
> +
> +	rdmsr_on_cpus(cpu_online_mask, MSR_K7_HWCR, msrs);
> +
> +	for_each_cpu(cpu, cpu_online_mask) {
> +		struct msr *reg = per_cpu_ptr(msrs, cpu);
> +		if (t)
> +			reg->l &= ~BIT(25);
> +		else
> +			reg->l |= BIT(25);
> +	}
> +	wrmsr_on_cpus(cpu_online_mask, MSR_K7_HWCR, msrs);
> +}

What prevents cpu_online_mask from changing while this is happening?

> @@ -1422,15 +1490,34 @@ static int __cpuinit powernowk8_init(void)
>  			supported_cpus++;
>  	}
>  
> -	if (supported_cpus == num_online_cpus()) {
> -		printk(KERN_INFO PFX "Found %d %s "
> -			"processors (%d cpu cores) (" VERSION ")\n",
> -			num_online_nodes(),
> -			boot_cpu_data.x86_model_id, supported_cpus);
> -		return cpufreq_register_driver(&cpufreq_amd64_driver);
> +	if (supported_cpus != num_online_cpus())
> +		return -ENODEV;
> +
> +	printk(KERN_INFO PFX "Found %d %s (%d cpu cores) (" VERSION ")\n",
> +		num_online_nodes(), boot_cpu_data.x86_model_id, supported_cpus);
> +
> +	if (boot_cpu_has(X86_FEATURE_CPB)) {
> +
> +		cpb_capable = true;
> +
> +		msrs = msrs_alloc();
> +		if (!msrs) {
> +			printk(KERN_ERR "%s: Error allocating msrs!\n", __func__);
> +			return -ENOMEM;
> +		}
> +
> +		rdmsr_on_cpus(cpu_online_mask, MSR_K7_HWCR, msrs);
> +
> +		for_each_cpu(cpu, cpu_online_mask) {
> +			struct msr *reg = per_cpu_ptr(msrs, cpu);
> +			cpb_enabled |= !(!!(reg->l & BIT(25)));
> +		}

Ditto.

I assume these changes get applied to new CPUs as they come online?

> ...
--
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/