Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1423237

Re: [RFC PATCH] x86: Move away from /dev/cpu/*/msr

Show all headers | View raw

On Wed, Jun 15, 2016 at 12:41:21PM -0400, Len Brown wrote:
> Recent hardware has an additional MSR field
> 
> MSR_IA32_HWP_REQUEST.ENERGY_PERFORMANCE_PREFERENCE
> that replaces
> 
> MSR_IA32_ENERGY_PERF_BIAS
> for the purpose of P-state control.
>
> 
> Both MSRs/fields exist and have effect at the same time.
> 
> so the API
> energy_policy_pref_hint
> 
> will not work -- as it isn't clear which MSR it refers to.

Surely we can make the new interface work too - perhaps add a new sysfs
file for the new thing. The old MSR_IA32_ENERGY_PERF_BIAS would be
needed on those older boxes.

Or we can have a sysfs file which is called something like
"perf_preference" or whatnot and that thing either maps input to the old
MSR_IA32_ENERGY_PERF_BIAS or to the new thing.

> I've updated x86_energy_perf_policy to talk to this MSR
> and a number of others for the benefit of HWP.  The
> patch is over 1000 lines.  I'll post it shortly.

So we should *not* give ourselves the example that using msr.ko for
other things *besides* debugging is ok. It is very wrong to talk to
naked MSRs and we have done it by now because this thing was there and
well, sure, why not use it.

But poking at MSRs is dangerous and we need proper abstraction. And we
should work towards that instead perpetuating wrong use.

-- 
Regards/Gruss,
    Boris.

ECO tip #101: Trim your mails when you reply.

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[RFC PATCH] x86: Move away from /dev/cpu/*/msr Borislav Petkov <bp@alien8.de> - 2016-06-15 12:10 +0200
  Re: [RFC PATCH] x86: Move away from /dev/cpu/*/msr chenyu <yu.chen.surf@gmail.com> - 2016-06-15 12:30 +0200
    Re: [RFC PATCH] x86: Move away from /dev/cpu/*/msr Borislav Petkov <bp@alien8.de> - 2016-06-15 12:40 +0200
      Re: [RFC PATCH] x86: Move away from /dev/cpu/*/msr Borislav Petkov <bp@alien8.de> - 2016-06-15 16:10 +0200
  Re: [RFC PATCH] x86: Move away from /dev/cpu/*/msr Len Brown <lenb@kernel.org> - 2016-06-15 18:50 +0200
    Re: [RFC PATCH] x86: Move away from /dev/cpu/*/msr Borislav Petkov <bp@alien8.de> - 2016-06-15 19:00 +0200
      Re: [RFC PATCH] x86: Move away from /dev/cpu/*/msr Len Brown <lenb@kernel.org> - 2016-06-15 19:30 +0200
        Re: [RFC PATCH] x86: Move away from /dev/cpu/*/msr Borislav Petkov <bp@alien8.de> - 2016-06-15 19:40 +0200
          Re: [RFC PATCH] x86: Move away from /dev/cpu/*/msr Len Brown <lenb@kernel.org> - 2016-06-15 19:50 +0200
            Re: [RFC PATCH] x86: Move away from /dev/cpu/*/msr Borislav Petkov <bp@alien8.de> - 2016-06-15 20:00 +0200
  Re: [RFC PATCH] x86: Move away from /dev/cpu/*/msr Andi Kleen <andi@firstfloor.org> - 2016-06-20 23:20 +0200

csiph-web