[Nouveau] [PATCH] bios/fan: hardcode the fan mode to linear

[Martin Peres]

On 05/01/16 10:38, Ben Skeggs wrote:
> On 01/05/2016 06:35 PM, Martin Peres wrote:
>> On 04/01/16 18:42, Emil Velikov wrote:
>>> On 4 January 2016 at 14:56, Martin Peres <martin.peres at free.fr> wrote:
>>>> On 17/12/15 19:18, Martin Peres wrote:
>>>>> On 29/11/15 16:10, Martin Peres wrote:
>>>>>> This is an oversight that made use of the trip-point-based fan
>>>>>> managenent
>>>>>> on
>>>>>> cards that never expose those. This led the fan to stay at fan_min.
>>>>>>
>>>>>> Fortunately, the emergency code would kick when the temperature would
>>>>>> reach
>>>>>> 90°C.
>>>>>>
>>>>>> Reported-by: Tom Englund <tomenglund26 at gmail.com>
>>>>>> Tested-by: Tom Englund <tomenglund26 at gmail.com>
>>>>> Tested-by: Daemon32 <lnf.purple at gmail.com>
>>>>> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=92126
>>>>>
>>>>>> Signed-off-by: Martin Peres <martin.peres at free.fr>
>>>>>> ---
>>>>>>     drm/nouveau/nvkm/subdev/bios/fan.c | 1 +
>>>>>>     1 file changed, 1 insertion(+)
>>>>>>
>>>>>> diff --git a/drm/nouveau/nvkm/subdev/bios/fan.c
>>>>>> b/drm/nouveau/nvkm/subdev/bios/fan.c
>>>>>> index 43006db..80fed7e 100644
>>>>>> --- a/drm/nouveau/nvkm/subdev/bios/fan.c
>>>>>> +++ b/drm/nouveau/nvkm/subdev/bios/fan.c
>>>>>> @@ -83,6 +83,7 @@ nvbios_fan_parse(struct nvkm_bios *bios, struct
>>>>>> nvbios_therm_fan *fan)
>>>>>>                 fan->type = NVBIOS_THERM_FAN_UNK;
>>>>>>             }
>>>>>>
>>>>>> +        fan->fan_mode = NVBIOS_THERM_FAN_LINEAR;
>>>>>>             fan->min_duty = nvbios_rd08(bios, data + 0x02);
>>>>>>             fan->max_duty = nvbios_rd08(bios, data + 0x03);
>>>>>>
>>>>>>
>>>>> Ben, can you merge this patch? It is kind of critical :s And it
>>>>> should be
>>>>> CCed to stable too, without it, some kepler/maxwell get 0% fan power
>>>>> which
>>>>> is kind of dangerous, even without reclocking!
>>>> Ping, this is really important :s
>>>>
>>> Not sure who picked it up, but it seems to be in the latest fixes
>>> batch [1].
>>>
>>> [1]
>>> http://lists.freedesktop.org/archives/dri-devel/2015-December/097589.html
>> Thanks Emil, Karol pointed it out to me on IRC too. Thanks to whoever
>> picked it up :)
> I did, my apologies for not communicating that better!
No probs :) Didn't know you moved to github too, it may be worth sending 
an email on the ML and we need to update the wiki. Are you planning on 
using the merge request feature to keep track of the review you need to 
make?

We can also ask Damien Lespiau to set up a Nouveau instance of 
patchwork: https://patchwork.freedesktop.org/