Nouveau - Aug 2018 - [Bug 107447] New: regression in thermal management of Quadro FX 1500

https://bugs.freedesktop.org/show_bug.cgi?id=107447

            Bug ID: 107447
           Summary: regression in thermal management of  Quadro FX 1500
           Product: Mesa
           Version: unspecified
          Hardware: x86-64 (AMD64)
                OS: Linux (All)
            Status: NEW
          Severity: normal
          Priority: medium
         Component: Drivers/DRI/nouveau
          Assignee: nouveau at lists.freedesktop.org
          Reporter: Thomas.Blume at suse.com
        QA Contact: nouveau at lists.freedesktop.org

Created attachment 140926
  --> https://bugs.freedesktop.org/attachment.cgi?id=140926&action=edit
dmesg kernel 4.4

On my machine with an:

  Model: "nVidia Quadro FX 1500"
  Vendor: pci 0x10de "nVidia Corporation"
  Device: pci 0x029e "Quadro FX 1500"
  SubVendor: pci 0x10de "nVidia Corporation"
  SubDevice: pci 0x032c 
  Revision: 0xa1
  Driver: "nouveau"
  Driver Modules: "drm"

graphics card, the gpu fan start running at high load.
The issue started after installing openSUSE Leap15 with kernel version 4.12.
The same issue is visible using kernel version 4.17.
It was working fine on older openSUSE versions, up to kernel version 4.4.
Installing kernel version 4.4 on Leap 15 makes the issue go away.
The nouveau driver debug logs show when booting with kernel 4.12:

-->

 grep therm /mnt/dmesg-4_12.txt
[    5.985658] nouveau 0000:01:00.0: therm: FAN control: PWM
[    5.985667] nouveau 0000:01:00.0: therm: parsing the fan table failed
[    5.985680] nouveau 0000:01:00.0: therm: fan management: automatic
[    5.985687] nouveau 0000:01:00.0: therm: FAN target request: 90%
[    5.985692] nouveau 0000:01:00.0: therm: FAN target: 90
[    5.985697] nouveau 0000:01:00.0: therm: FAN update: 23
[    5.985707] nouveau 0000:01:00.0: therm: internal sensor: yes
[    6.005559] nouveau 0000:01:00.0: therm: programmed thresholds [ 90(3),
95(3), 130(2), 135(5) ]
[    6.485750] nouveau 0000:01:00.0: therm: FAN update: 26
[    6.985841] nouveau 0000:01:00.0: therm: FAN update: 29
[    7.485929] nouveau 0000:01:00.0: therm: FAN update: 32
[    7.986009] nouveau 0000:01:00.0: therm: FAN update: 35
[    8.486100] nouveau 0000:01:00.0: therm: FAN update: 38
[    8.986193] nouveau 0000:01:00.0: therm: FAN update: 41
[    9.486270] nouveau 0000:01:00.0: therm: FAN update: 44
[    9.987905] nouveau 0000:01:00.0: therm: FAN update: 47
[   10.489188] nouveau 0000:01:00.0: therm: FAN update: 50
[   10.990485] nouveau 0000:01:00.0: therm: FAN update: 53
[   11.491779] nouveau 0000:01:00.0: therm: FAN update: 56
[   11.993032] nouveau 0000:01:00.0: therm: FAN update: 59
[   12.494276] nouveau 0000:01:00.0: therm: FAN update: 62
[   12.995621] nouveau 0000:01:00.0: therm: FAN update: 65
[   13.497070] nouveau 0000:01:00.0: therm: FAN update: 68
[   13.998516] nouveau 0000:01:00.0: therm: FAN update: 71
[   14.500055] nouveau 0000:01:00.0: therm: FAN update: 74
[   15.001850] nouveau 0000:01:00.0: therm: FAN update: 77
[   15.505237] nouveau 0000:01:00.0: therm: FAN update: 80
[   16.006924] nouveau 0000:01:00.0: therm: FAN update: 83
[   16.508429] nouveau 0000:01:00.0: therm: FAN update: 86
[   17.009946] nouveau 0000:01:00.0: therm: FAN update: 89
[   17.511480] nouveau 0000:01:00.0: therm: FAN update: 90
--<

with kernel 4.4 it only shows:

-->
 grep therm /mnt/dmesg-4_4.txt 
[    7.547113] nouveau 0000:01:00.0: therm: FAN control: PWM
[    7.547117] nouveau 0000:01:00.0: therm: parsing the fan table failed
[    7.547121] nouveau 0000:01:00.0: therm: fan management: automatic
[    7.547124] nouveau 0000:01:00.0: therm: internal sensor: yes
[    7.566971] nouveau 0000:01:00.0: therm: programmed thresholds [ 90(3),
95(3), 130(2), 135(5) ]
--<

Attaching dmesg with noveau debug logs for kernel 4.4. and kernel 4.12.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/nouveau/attachments/20180801/fb635262/attachment.html>

https://bugs.freedesktop.org/show_bug.cgi?id=107447

--- Comment #1 from Thomas Blume <Thomas.Blume at suse.com> ---
Created attachment 140927
  --> https://bugs.freedesktop.org/attachment.cgi?id=140927&action=edit
dmesg kernel 4.12

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/nouveau/attachments/20180801/850bde28/attachment.html>

https://bugs.freedesktop.org/show_bug.cgi?id=107447

--- Comment #2 from Takashi Iwai <tiwai at suse.de> ---
We confirmed that the commit below brought the regression:
commit 800efb4c2857ec543fdc33585bbcb1fd5ef28337
    drm/nouveau/drm/therm/fan: add a fallback if no fan control is specified in
the vbios

Reverting the commit fixes the problem on Thomas' machine.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/nouveau/attachments/20180803/adfdb3b3/attachment.html>

https://bugs.freedesktop.org/show_bug.cgi?id=107447

--- Comment #3 from caguduzexi at spam4.me ---
Takashi Iwai, thanks a lot for finding a solution for that!
I am having this issue since over a year and nothing happens. I am willing to
switch over to suse when this would finally get fixed and the GPU card wont
spin at 100% the whole time.
Here one of mine bugreports:

https://bugs.freedesktop.org/show_bug.cgi?id=102352

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/nouveau/attachments/20180812/e4ad18f2/attachment-0001.html>

https://bugs.freedesktop.org/show_bug.cgi?id=107447

--- Comment #4 from Takashi Iwai <tiwai at suse.de> ---
I *guess* that a more better fix would be to check the error from the fan-table
parsing and avoid the linear fallback when it failed, instead of the whole
revert.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/nouveau/attachments/20180813/51a00bcb/attachment.html>

https://bugs.freedesktop.org/show_bug.cgi?id=107447

--- Comment #5 from Takashi Iwai <tiwai at suse.de> ---
One problem seems to be that the thermal target is set only once by the linear
fallback, and forgotten.  The patch below "fixes" that, at least, to
keep
updating the fan with the linear fallback.

But, according to Thomas, this still seems too high.  I'm not sure whether
it's
the correct behavior or it was just because the fan never worked in the past.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/nouveau/attachments/20180903/50fef0e2/attachment.html>

https://bugs.freedesktop.org/show_bug.cgi?id=107447

--- Comment #6 from Takashi Iwai <tiwai at suse.de> ---
Created attachment 141427
  --> https://bugs.freedesktop.org/attachment.cgi?id=141427&action=edit
Test fix patch

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/nouveau/attachments/20180903/d6571b45/attachment.html>

https://bugs.freedesktop.org/show_bug.cgi?id=107447

GitLab Migration User <gitlab-migration at fdo.invalid> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |MOVED
             Status|NEW                         |RESOLVED

--- Comment #7 from GitLab Migration User <gitlab-migration at
fdo.invalid> ---
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been
closed from further activity.

You can subscribe and participate further through the new bug through this link
to our GitLab instance: https://gitlab.freedesktop.org/mesa/mesa/issues/1161.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<https://lists.freedesktop.org/archives/nouveau/attachments/20190918/8282a251/attachment-0001.html>