linux-kernel - Re: [PATCH] powercap/rapl: add support for cpu model 0x3f

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] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 13 Aug 2014 15:50:35 -0700
From:	Jacob Pan <jacob.jun.pan@...ux.intel.com>
To:	Jason Baron <jbaron@...mai.com>
Cc:	rafael.j.wysocki@...el.com, durgadoss.r@...el.com,
	srivatsa.bhat@...ux.vnet.ibm.com,
	srinivas.pandruvada@...ux.intel.com, linux-kernel@...r.kernel.org,
	"Rafael J. Wysocki" <rafael.j.wysocki@...el.com>
Subject: Re: [PATCH] powercap/rapl: add support for cpu model 0x3f

On Wed, 13 Aug 2014 20:33:26 +0000 (GMT)
Jason Baron <jbaron@...mai.com> wrote:

> I've confirmed that monitoring the package power usage as well
> as setting power limits appear to be working as expected. However,
> I do see in the logs:
> 
> [    5.082632] intel_rapl: RAPL domain core detection failed
> [    5.088370] intel_rapl: RAPL domain uncore detection failed
> 
> So I'm not sure if the h/w simply doesn't support that, or if the
> driver could be improved here. In any case, adding the 0x3f makes
> the driver useful for me. Tested aginst cpu:
> 
> Intel(R) Xeon(R) CPU E5-2650 v3 @ 2.30GHz
> 
I don't have this CPU model to test. Most server processors don't
have uncore domain so detection failed is normal. Looks like you just
have package and dram domain?
Thanks for testing it out, looks good to me. can you send it to
linux-pm list and cc rafael?

> Signed-off-by: Jason Baron <jbaron@...mai.com>
> ---
>  drivers/powercap/intel_rapl.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/powercap/intel_rapl.c
> b/drivers/powercap/intel_rapl.c index b1cda6f..a362dcc 100644
> --- a/drivers/powercap/intel_rapl.c
> +++ b/drivers/powercap/intel_rapl.c
> @@ -953,6 +953,7 @@ static const struct x86_cpu_id rapl_ids[] = {
>  	{ X86_VENDOR_INTEL, 6, 0x3a},/* Ivy Bridge */
>  	{ X86_VENDOR_INTEL, 6, 0x3c},/* Haswell */
>  	{ X86_VENDOR_INTEL, 6, 0x3d},/* Broadwell */
> +	{ X86_VENDOR_INTEL, 6, 0x3f},/* Haswell */
>  	{ X86_VENDOR_INTEL, 6, 0x45},/* Haswell ULT */
>  	/* TODO: Add more CPU IDs after testing */
>  	{}

[Jacob Pan]
--
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/