svn commit: r326383 - head/sys/x86/cpufreq

[Conrad Meyer]

On Thu, Nov 30, 2017 at 12:08 PM, Jung-uk Kim <jkim at freebsd.org> wrote:
> On 11/30/2017 14:32, Conrad Meyer wrote:
>> Hi,
>>
>> I don't think this answers the second question about the conditional.
>> It seems like PCPU_GET() for the initial CPU should be pulled out of
>> the loop, which binds the thread to a different CPU every iteration.
>
> Ah, good catch.  I'll fix it soon.  Sorry.

Thanks! :-)

>> Also, as a side effect of disabling verification, you have fixed PR
>> 221621, 219213, and probably 218262.
>
> Probably.  However, I am just trying to fix my FX-8350 and A10-6800 and
> I don't have Zen processors to verify the MSRs are actually working on
> those CPUs.

I have one, I can verify if needed (although the change looks good to
me).  On some Zen systems (including mine) it seems that the hardware
can successfully set a P-state, but will fail to read it back.  For me
it is P1 but other users have reported P0.  That's the root issue of
all of those PRs.  If reading back isn't required, maybe that's a
solution to the issue.

Best,
Conrad