From: Pierre Tardy [mailto:[email protected]]
Sent: Tuesday, May 17, 2011 9:55 PM
To: Arjan van de Ven
Cc: Mathew, JohnX; [email protected]
Subject: Re: [PATCH] intel_cpu: Add Medfield processor as Intel cpu

>
> In the 2.6.35 kernel, there is no /dev/cpu/%d/msr. Instead msr dev is 
> registered as msr%d.
actually there is...

has been there since a really really long time.
Actually, issue is not from 2.6.35, it is rather android's "udev" like system, 
that does not put msr in its own directory. Hence the change in order for 
powertop to support android.

I agree it has to come in another patch.

John,
What I dont understand is how you disable MSR_CORE_RESIDENCY and 
MSR_CPU_RESIDENCY in this patch.

It is done by not adding the Medfield model id to the checks to create "new 
class nhm_cpu" and  "new class nhm_core" subsequently. It is in these two class 
implementations that msr registers related to core and cpu will be checked.

Regards,
Pierre
---------------------------------------------------------------------
Intel Corporation SAS (French simplified joint stock company)
Registered headquarters: "Les Montalets"- 2, rue de Paris, 
92196 Meudon Cedex, France
Registration Number:  302 456 199 R.C.S. NANTERRE
Capital: 4,572,000 Euros

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.
_______________________________________________
Power mailing list
[email protected]
https://bughost.org/mailman/listinfo/power

Reply via email to