Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-15 Thread Takashi Iwai
On Mon, 14 Aug 2023 17:06:02 +0200, Takashi Iwai wrote: > > On Mon, 14 Aug 2023 16:51:08 +0200, > Karol Herbst wrote: > > > > I've sent a patch out to address this memory corruption > > https://patchwork.freedesktop.org/patch/552642/ > > > > It might or might not fix regressions from the

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-14 Thread Takashi Iwai
On Mon, 14 Aug 2023 16:51:08 +0200, Karol Herbst wrote: > > I've sent a patch out to address this memory corruption > https://patchwork.freedesktop.org/patch/552642/ > > It might or might not fix regressions from the original I2C fix, so > please test and report if there are remaining issues.

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-14 Thread Karol Herbst
On Mon, Aug 14, 2023 at 4:19 PM Karol Herbst wrote: > > On Mon, Aug 14, 2023 at 3:35 PM Takashi Iwai wrote: > > > > On Mon, 14 Aug 2023 15:19:11 +0200, > > Karol Herbst wrote: > > > > > > On Mon, Aug 14, 2023 at 2:56 PM Karol Herbst wrote: > > > > > > > > On Mon, Aug 14, 2023 at 2:48 PM Takashi

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-14 Thread Karol Herbst
On Mon, Aug 14, 2023 at 3:35 PM Takashi Iwai wrote: > > On Mon, 14 Aug 2023 15:19:11 +0200, > Karol Herbst wrote: > > > > On Mon, Aug 14, 2023 at 2:56 PM Karol Herbst wrote: > > > > > > On Mon, Aug 14, 2023 at 2:48 PM Takashi Iwai wrote: > > > > > > > > On Mon, 14 Aug 2023 14:38:18 +0200, > > >

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-14 Thread Takashi Iwai
On Mon, 14 Aug 2023 15:19:11 +0200, Karol Herbst wrote: > > On Mon, Aug 14, 2023 at 2:56 PM Karol Herbst wrote: > > > > On Mon, Aug 14, 2023 at 2:48 PM Takashi Iwai wrote: > > > > > > On Mon, 14 Aug 2023 14:38:18 +0200, > > > Karol Herbst wrote: > > > > > > > > On Wed, Aug 9, 2023 at 6:16 PM

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-14 Thread Karol Herbst
On Tue, Aug 8, 2023 at 3:47 PM Borislav Petkov wrote: > > On Tue, Aug 08, 2023 at 12:39:32PM +0200, Karol Herbst wrote: > > ahh, that would have been good to know :) > > Yeah, I didn't see it before - it would only freeze. Only after I added > the printk you requested. > > > Mind figuring out

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-14 Thread Karol Herbst
On Mon, Aug 14, 2023 at 2:56 PM Karol Herbst wrote: > > On Mon, Aug 14, 2023 at 2:48 PM Takashi Iwai wrote: > > > > On Mon, 14 Aug 2023 14:38:18 +0200, > > Karol Herbst wrote: > > > > > > On Wed, Aug 9, 2023 at 6:16 PM Takashi Iwai wrote: > > > > > > > > On Wed, 09 Aug 2023 16:46:38 +0200, > >

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-14 Thread Karol Herbst
On Mon, Aug 14, 2023 at 2:48 PM Takashi Iwai wrote: > > On Mon, 14 Aug 2023 14:38:18 +0200, > Karol Herbst wrote: > > > > On Wed, Aug 9, 2023 at 6:16 PM Takashi Iwai wrote: > > > > > > On Wed, 09 Aug 2023 16:46:38 +0200, > > > Takashi Iwai wrote: > > > > > > > > On Wed, 09 Aug 2023 15:13:23

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-14 Thread Takashi Iwai
On Mon, 14 Aug 2023 14:38:18 +0200, Karol Herbst wrote: > > On Wed, Aug 9, 2023 at 6:16 PM Takashi Iwai wrote: > > > > On Wed, 09 Aug 2023 16:46:38 +0200, > > Takashi Iwai wrote: > > > > > > On Wed, 09 Aug 2023 15:13:23 +0200, > > > Takashi Iwai wrote: > > > > > > > > On Wed, 09 Aug 2023

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-14 Thread Karol Herbst
On Wed, Aug 9, 2023 at 6:16 PM Takashi Iwai wrote: > > On Wed, 09 Aug 2023 16:46:38 +0200, > Takashi Iwai wrote: > > > > On Wed, 09 Aug 2023 15:13:23 +0200, > > Takashi Iwai wrote: > > > > > > On Wed, 09 Aug 2023 14:19:23 +0200, > > > Karol Herbst wrote: > > > > > > > > On Wed, Aug 9, 2023 at

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-09 Thread Karol Herbst
On Wed, Aug 9, 2023 at 8:28 PM Karol Herbst wrote: > > On Wed, Aug 9, 2023 at 4:04 PM Thorsten Leemhuis > wrote: > > > > On 09.08.23 15:13, Takashi Iwai wrote: > > > > > > If this can't be fixed quickly, I suppose it's safer to revert it from > > > 6.4.y for now. 6.5 is still being cooked, but

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-09 Thread Karol Herbst
On Wed, Aug 9, 2023 at 4:04 PM Thorsten Leemhuis wrote: > > On 09.08.23 15:13, Takashi Iwai wrote: > > > > If this can't be fixed quickly, I suppose it's safer to revert it from > > 6.4.y for now. 6.5 is still being cooked, but 6.4.x is already in > > wide deployment, hence the regression has to

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-09 Thread Takashi Iwai
On Wed, 09 Aug 2023 16:46:38 +0200, Takashi Iwai wrote: > > On Wed, 09 Aug 2023 15:13:23 +0200, > Takashi Iwai wrote: > > > > On Wed, 09 Aug 2023 14:19:23 +0200, > > Karol Herbst wrote: > > > > > > On Wed, Aug 9, 2023 at 1:46 PM Takashi Iwai wrote: > > > > > > > > On Wed, 09 Aug 2023 13:42:09

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-09 Thread Takashi Iwai
On Wed, 09 Aug 2023 15:13:23 +0200, Takashi Iwai wrote: > > On Wed, 09 Aug 2023 14:19:23 +0200, > Karol Herbst wrote: > > > > On Wed, Aug 9, 2023 at 1:46 PM Takashi Iwai wrote: > > > > > > On Wed, 09 Aug 2023 13:42:09 +0200, > > > Karol Herbst wrote: > > > > > > > > On Wed, Aug 9, 2023 at 11:22 

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-09 Thread Thorsten Leemhuis
On 09.08.23 15:13, Takashi Iwai wrote: > > If this can't be fixed quickly, I suppose it's safer to revert it from > 6.4.y for now. 6.5 is still being cooked, but 6.4.x is already in > wide deployment, hence the regression has to be addressed quickly. Good luck with that. To quote

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-09 Thread Takashi Iwai
On Wed, 09 Aug 2023 14:19:23 +0200, Karol Herbst wrote: > > On Wed, Aug 9, 2023 at 1:46 PM Takashi Iwai wrote: > > > > On Wed, 09 Aug 2023 13:42:09 +0200, > > Karol Herbst wrote: > > > > > > On Wed, Aug 9, 2023 at 11:22 AM Takashi Iwai wrote: > > > > > > > > On Tue, 08 Aug 2023 12:39:32 +0200,

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-09 Thread Karol Herbst
On Wed, Aug 9, 2023 at 1:46 PM Takashi Iwai wrote: > > On Wed, 09 Aug 2023 13:42:09 +0200, > Karol Herbst wrote: > > > > On Wed, Aug 9, 2023 at 11:22 AM Takashi Iwai wrote: > > > > > > On Tue, 08 Aug 2023 12:39:32 +0200, > > > Karol Herbst wrote: > > > > > > > > On Mon, Aug 7, 2023 at 5:05 PM

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-09 Thread Takashi Iwai
On Wed, 09 Aug 2023 13:42:09 +0200, Karol Herbst wrote: > > On Wed, Aug 9, 2023 at 11:22 AM Takashi Iwai wrote: > > > > On Tue, 08 Aug 2023 12:39:32 +0200, > > Karol Herbst wrote: > > > > > > On Mon, Aug 7, 2023 at 5:05 PM Borislav Petkov wrote: > > > > > > > > On Mon, Aug 07, 2023 at

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-09 Thread Karol Herbst
On Wed, Aug 9, 2023 at 11:22 AM Takashi Iwai wrote: > > On Tue, 08 Aug 2023 12:39:32 +0200, > Karol Herbst wrote: > > > > On Mon, Aug 7, 2023 at 5:05 PM Borislav Petkov wrote: > > > > > > On Mon, Aug 07, 2023 at 01:49:42PM +0200, Karol Herbst wrote: > > > > in what way does it stop? Just not

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-09 Thread Takashi Iwai
On Tue, 08 Aug 2023 12:39:32 +0200, Karol Herbst wrote: > > On Mon, Aug 7, 2023 at 5:05 PM Borislav Petkov wrote: > > > > On Mon, Aug 07, 2023 at 01:49:42PM +0200, Karol Herbst wrote: > > > in what way does it stop? Just not progressing? That would be kinda > > > concerning. Mind tracing with

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-08 Thread Borislav Petkov
On Tue, Aug 08, 2023 at 12:39:32PM +0200, Karol Herbst wrote: > ahh, that would have been good to know :) Yeah, I didn't see it before - it would only freeze. Only after I added the printk you requested. > Mind figuring out what's exactly NULL inside nvif_object_mthd? Or > rather what line

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-08 Thread Karol Herbst
On Mon, Aug 7, 2023 at 5:05 PM Borislav Petkov wrote: > > On Mon, Aug 07, 2023 at 01:49:42PM +0200, Karol Herbst wrote: > > in what way does it stop? Just not progressing? That would be kinda > > concerning. Mind tracing with what arguments `nvkm_uevent_add` is > > called with and without that

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-07 Thread Borislav Petkov
On Mon, Aug 07, 2023 at 01:49:42PM +0200, Karol Herbst wrote: > in what way does it stop? Just not progressing? That would be kinda > concerning. Mind tracing with what arguments `nvkm_uevent_add` is > called with and without that patch? Well, me dumping those args I guess made the box not freeze

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-07 Thread Karol Herbst
On Sun, Aug 6, 2023 at 11:40 PM Borislav Petkov wrote: > > Hi folks, > > the patch in $Subject breaks booting here on one of my test boxes, see > below. > > Reverting it ontop of -rc4 fixes the issue. > > Thx. > > [3.580535] ACPI: \_PR_.CP04: Found 4 idle states > [3.585694] ACPI:

Re: 2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-07 Thread Thorsten Leemhuis
[CCing the regression list, as it should be in the loop for regressions: https://docs.kernel.org/admin-guide/reporting-regressions.html] [TLDR: I'm adding this report to the list of tracked Linux kernel regressions; the text you find below is based on a few templates paragraphs you might have

2b5d1c29f6c4 ("drm/nouveau/disp: PIOR DP uses GPIO for HPD, not PMGR AUX interrupts")

2023-08-06 Thread Borislav Petkov
Hi folks, the patch in $Subject breaks booting here on one of my test boxes, see below. Reverting it ontop of -rc4 fixes the issue. Thx. [3.580535] ACPI: \_PR_.CP04: Found 4 idle states [3.585694] ACPI: \_PR_.CP05: Found 4 idle states [3.590852] ACPI: \_PR_.CP06: Found 4 idle