Re: [PATCH] hwmon: (fam15h_power) Disable preemption when reading registers

From: Huang, Ray
Date: Wed Jun 01 2016 - 12:10:00 EST



Sent from my iPad

> On Jun 1, 2016, at 11:27 PM, Huang, Ray <Ray.Huang@xxxxxxx> wrote:
>
>
>
>>> On Jun 1, 2016, at 9:23 PM, Guenter Roeck <linux@xxxxxxxxxxxx> wrote:
>>>
>>> On 06/01/2016 03:04 AM, Borislav Petkov wrote:
>>> From: Borislav Petkov <bp@xxxxxxx>
>>>
>>> We need to read a bunch of registers on each compute unit and possibly
>>> on the current CPU too. Disable preemption around it.
>>
>> An explanation would be helpful. Is this a bug fix ? I would like to get
>> a confirmation from someone at AMD that this is really necessary.
>>
>
> This change looks good for me. But I am in office this week, I will test it on CZ platform next week. :-)

Sorry, fix typo. I am *not* in office this week.

Rui