Re: [Nouveau] NOUVEAU_LEGACY_CTX_SUPPORT Fan Speed
Sure: $ lspci -nn -d 10de: 01:00.0 VGA compatible controller [0300]: NVIDIA Corporation GT200 [GeForce GTX 260] [10de:05e2] (rev a1) On Saturday, June 22, 2019, 2:19:31 PM EDT, Ilia Mirkin wrote: Mar - can you provide the output of lspci -nn -d 10de: On Sat, Jun 22, 2019 at 2:17 PM Lyude Paul wrote: Hi, is this actually an nv50 GPU, or some other model? I can try to take a closer look at this On Sun, 2019-06-16 at 10:28 -0400, Ilia Mirkin wrote: I don't really see anything between v5.0..v5.1 which would account for this. Could have been a subtle change to the i2c logic somewhere. The fastest way to identify the problem would be to do a bisect on the kernel to identify the commit that caused this. There are many guides for this online. On Sat, Jun 15, 2019 at 12:17 PM Mar Mel wrote: Unfortunately, even with this change now reverted in kernel 5.1.10, the fan speed issue persists. If someone could point me in the direction of a relevant commit(s) I'll happily file a bug report. On Thursday, June 13, 2019, 11:19:25 AM EDT, Mar Mel wrote: As of kernel 5.1.9, on resume from suspend, my NV50 fan runs at full speed. Not sure if it has to do with this new config option (NOUVEAU_LEGACY_CTX_SUPPORT)? This issue is not present using kernel 5.0.21. Years ago I filed a similar issue: 60704 – [nouveau, git regression] - I2C PWM fan control broken on nv50 adt7475 on kernels 3.3.x+ Thanks. | | | | 60704 – [nouveau, git regression] - I2C PWM fan control broken on nv50 a... | | | ___ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau ___Nouveau mailing listnouv...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau -- Cheers, Lyude Paul ___ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau
Re: [Nouveau] NOUVEAU_LEGACY_CTX_SUPPORT Fan Speed
Hi, is this actually an nv50 GPU, or some other model? I can try to take a closer look at this On Sun, 2019-06-16 at 10:28 -0400, Ilia Mirkin wrote: > I don't really see anything between v5.0..v5.1 which would account for this. > Could have been a subtle change to the i2c logic somewhere. The fastest way > to identify the problem would be to do a bisect on the kernel to identify > the commit that caused this. There are many guides for this online. > > On Sat, Jun 15, 2019 at 12:17 PM Mar Mel wrote: > > Unfortunately, even with this change now reverted in kernel > > 5.1.10, the fan speed issue persists. > > If someone could point me in the direction of a relevant commit(s) I'll > > happily file a bug report. > > > > > > > > > > > > > > > > On Thursday, June 13, 2019, 11:19:25 AM EDT, Mar Mel < > > marmel6...@yahoo.com> wrote: > > > > > > > > > > > > As of kernel 5.1.9, on resume from suspend, my NV50 fan > > runs at full speed. > > > > Not sure if it has to do with this new config option > > (NOUVEAU_LEGACY_CTX_SUPPORT)? > > > > This issue is not present using kernel 5.0.21. > > > > Years ago I filed a similar issue: > > > > 60704 – [nouveau, git regression] - I2C PWM fan control broken on nv50 > > adt7475 on kernels 3.3.x+ > > > > Thanks. > > > > > > 60704 – [nouveau, git regression] - I2C PWM fan control broken on nv50 > > a... > > > > > > > > > > > > ___ > > > > Nouveau mailing list > > > > Nouveau@lists.freedesktop.org > > > > https://lists.freedesktop.org/mailman/listinfo/nouveau > > ___Nouveau mailing > listnouv...@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/nouveau -- Cheers, Lyude Paul ___ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau
Re: [Nouveau] [PATCH] drm/nouveau: fix bogus GPL-2 license header
Acked-by: Karol Herbst On Fri, Jun 21, 2019 at 8:12 PM Emil Velikov wrote: > > On 2019/06/19, Ilia Mirkin wrote: > > The bulk SPDX addition made all these files into GPL-2.0 licensed files. > > However the remainder of the project is MIT-licensed, these files > > (primarily header files) were simply missing the boiler plate and got > > caught up in the global update. > > > > Fixes: b24413180f5 (License cleanup: add SPDX GPL-2.0 license identifier to > > files with no license) > > Signed-off-by: Ilia Mirkin > > --- > > > > Ben, you did like 99.7% of this work, so ultimately your call. Pretty > > much all of these were split out from other MIT-licensed files, and most > > are header files anyways. > > > All of my glorious 23 patches to nouveau are meant to be under MIT. > Acked-by: Emil Velikov > > > Another change might be to add the SPDX identifier to files *with* > > the MIT boilerplate, but I didn't want to do too much here. > > > No objections on my end :-) > > -Emil > > ___ > Nouveau mailing list > Nouveau@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/nouveau ___ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau
Re: [Nouveau] NOUVEAU_LEGACY_CTX_SUPPORT Fan Speed
I can confirm that reverting just this commit fixes the fan speed regression using kernel 5.1.12. How do we go about getting this reverted from stable kernels? CCing the author. On Friday, June 21, 2019, 4:28:19 PM EDT, Mar Mel wrote: Thanks for finding this Emil! The change you point out also landed on 4.19.49 and I am able to reproduce the fan speed issue with that kernel as well, whereas previous 4.19.x series kernels were fine. I'll attempt to build both the latest 4.19.x and 5.1.x stable kernels with that patch reverted and see if it's the sole culprit. On Friday, June 21, 2019, 1:32:19 PM EDT, Emil Velikov wrote: Some weird interaction with disabling the i2c access, in the following patch perhaps? commit cd68344b283174a9b38e9488d5a929464e1f417c Author: Lyude Paul Date: Tue Apr 9 16:23:30 2019 -0400 drm/nouveau/i2c: Disable i2c bus access after ->fini() On Sun, 16 Jun 2019 at 15:28, Ilia Mirkin wrote: I don't really see anything between v5.0..v5.1 which would account for this. Could have been a subtle change to the i2c logic somewhere. The fastest way to identify the problem would be to do a bisect on the kernel to identify the commit that caused this. There are many guides for this online. On Sat, Jun 15, 2019 at 12:17 PM Mar Mel wrote: Unfortunately, even with this change now reverted in kernel 5.1.10, the fan speed issue persists. If someone could point me in the direction of a relevant commit(s) I'll happily file a bug report. On Thursday, June 13, 2019, 11:19:25 AM EDT, Mar Mel wrote: As of kernel 5.1.9, on resume from suspend, my NV50 fan runs at full speed. Not sure if it has to do with this new config option (NOUVEAU_LEGACY_CTX_SUPPORT)? This issue is not present using kernel 5.0.21. Years ago I filed a similar issue: 60704 – [nouveau, git regression] - I2C PWM fan control broken on nv50 adt7475 on kernels 3.3.x+ Thanks. | | | | 60704 – [nouveau, git regression] - I2C PWM fan control broken on nv50 a... | | | ___ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau ___ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau ___ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau