Re: [Nouveau] Regression in 5.15 in nouveau

[Stefan Fritsch <sf@sfritsch.de>]

There is a pretty obvious typo in there:

--- a/drivers/gpu/drm/nouveau/nouveau_fence.c
+++ b/drivers/gpu/drm/nouveau/nouveau_fence.c
@@ -359,7 +359,7 @@ nouveau_fence_sync(struct nouveau_bo *nvbo, struct 
nouveau_channel *chan, bool e
                 fobj = dma_resv_shared_list(resv);
         }

-       for (i = 0; (i < fobj ? fobj->shared_count : 0) && !ret; ++i) {
+       for (i = 0; i < (fobj ? fobj->shared_count : 0) && !ret; ++i) {
                 struct nouveau_channel *prev = NULL;
                 bool must_wait = true;


With that it works and I don't see the flickering in a short test. I 
will do more testing, but maybe Dan can test, too.

Cheers,
Stefan

On 07.12.21 20:01, Dan Moulding wrote:
>> Please test if that patch changes anything.
> 
> Looks like the driver is not functional after applying that patch. As
> soon as the display manager is supposed to start I get a black screen
> with just a (working) mouse pointer. VT switching doesn't work after
> that point.
> 
> I got the following warning when compiling with that patch applied:
> 
> drivers/gpu/drm/nouveau/nouveau_fence.c: In function ‘nouveau_fence_sync’:
> drivers/gpu/drm/nouveau/nouveau_fence.c:362:24: warning: comparison between pointer and integer
>    362 |         for (i = 0; (i < fobj ? fobj->shared_count : 0) && !ret; ++i) {
>        |                        ^
> 
> Below are the relevant portions from dmesg after attempting to run
> with the patch applied.
> 
> Cheers,
> 
> -- Dan
> 
> dmesg:
> =====
> 
> <snip>
> 
> [    0.269958] nouveau 0000:01:00.0: NVIDIA GP104 (134000a1)
> [    0.377100] nouveau 0000:01:00.0: bios: version 86.04.50.80.13
> [    0.377210] nouveau 0000:01:00.0: pmu: firmware unavailable
> [    0.377711] nouveau 0000:01:00.0: fb: 8192 MiB GDDR5
> [    0.391160] nouveau 0000:01:00.0: DRM: VRAM: 8192 MiB
> [    0.391164] nouveau 0000:01:00.0: DRM: GART: 536870912 MiB
> [    0.391166] nouveau 0000:01:00.0: DRM: BIT table 'A' not found
> [    0.391168] nouveau 0000:01:00.0: DRM: BIT table 'L' not found
> [    0.391170] nouveau 0000:01:00.0: DRM: TMDS table version 2.0
> [    0.391172] nouveau 0000:01:00.0: DRM: DCB version 4.1
> [    0.391174] nouveau 0000:01:00.0: DRM: DCB outp 00: 01000f42 00020030
> [    0.391176] nouveau 0000:01:00.0: DRM: DCB outp 01: 04811f96 04600020
> [    0.391178] nouveau 0000:01:00.0: DRM: DCB outp 02: 04011f92 00020020
> [    0.391180] nouveau 0000:01:00.0: DRM: DCB outp 03: 04822f86 04600010
> [    0.391182] nouveau 0000:01:00.0: DRM: DCB outp 04: 04022f82 00020010
> [    0.391184] nouveau 0000:01:00.0: DRM: DCB outp 06: 02033f62 00020010
> [    0.391186] nouveau 0000:01:00.0: DRM: DCB outp 07: 02844f76 04600020
> [    0.391188] nouveau 0000:01:00.0: DRM: DCB outp 08: 02044f72 00020020
> [    0.391190] nouveau 0000:01:00.0: DRM: DCB conn 00: 00001031
> [    0.391191] nouveau 0000:01:00.0: DRM: DCB conn 01: 02000146
> [    0.391193] nouveau 0000:01:00.0: DRM: DCB conn 02: 01000246
> [    0.391194] nouveau 0000:01:00.0: DRM: DCB conn 03: 00010361
> [    0.391196] nouveau 0000:01:00.0: DRM: DCB conn 04: 00020446
> [    0.391489] nouveau 0000:01:00.0: DRM: MM: using COPY for buffer copies
> [    0.891103] nouveau 0000:01:00.0: DRM: allocated 1920x1080 fb: 0x200000, bo 00000000bba11dd4
> [    0.892559] fbcon: nouveau (fb0) is primary device
> [    1.298487] tsc: Refined TSC clocksource calibration: 2999.999 MHz
> [    1.298492] clocksource: tsc: mask: 0xffffffffffffffff max_cycles: 0x2b3e44b2357, max_idle_ns: 440795324996 ns
> [    1.298555] clocksource: Switched to clocksource tsc
> [    1.340790] Console: switching to colour frame buffer device 240x67
> [    1.341249] nouveau 0000:01:00.0: [drm] fb0: nouveau frame buffer device
> [    1.341412] [drm] Initialized nouveau 1.3.1 20120801 for 0000:01:00.0 on minor 0
> [    1.341420] nouveau 0000:01:00.0: DRM: Disabling PCI power management to avoid bug
> 
> <snip>
> 
> [   19.742986] general protection fault, probably for non-canonical address 0x3e40c25cd2e657bd: 0000 [#1] SMP
> [   19.742989] CPU: 0 PID: 3588 Comm: X Tainted: G                T 5.15.6p2+ #1
> [   19.742991] Hardware name: Dell Inc. XPS 8930/0T2HR0, BIOS 1.1.17 06/22/2021
> [   19.742992] RIP: 0010:nouveau_fence_sync+0x6f/0x240
> [   19.742995] Code: 00 8b 7b 10 85 ff 0f 84 a7 00 00 00 41 89 cd 31 ed 31 d2 49 be ff ff ff ff ff ff ff 7f 4c 8b 7c d3 18 49 8b 94 24 90 00 00 00 <49> 8b 47 08 48 3d 00 37 0f a0 74 0c 48 3d 60 37 0f a0 0f 85 16 01
> [   19.742996] RSP: 0018:ffffb3bc413e7c10 EFLAGS: 00010202
> [   19.742998] RAX: 0000000000001001 RBX: ffff897383c07980 RCX: 0000000000000003
> [   19.742999] RDX: ffff897380cce000 RSI: 0000000000000001 RDI: 0000000000000001
> [   19.743000] RBP: 0000000000000001 R08: ffff897382af6c00 R09: ffff8973868641b8
> [   19.743001] R10: 0000000000000002 R11: ffff8973868641dc R12: ffff897386862400
> [   19.743001] R13: 0000000000000001 R14: 7fffffffffffffff R15: 3e40c25cd2e657b5
> [   19.743002] FS:  00007f5e661be8c0(0000) GS:ffff897ae0c00000(0000) knlGS:0000000000000000
> [   19.743003] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> [   19.743004] CR2: 000055c9e8e33d50 CR3: 00000001152b7001 CR4: 00000000003706f0
> [   19.743005] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
> [   19.743006] DR3: 0000000000000000 DR6: 00000000fffe0ff0 DR7: 0000000000000400
> [   19.743006] Call Trace:
> [   19.743008]  <TASK>
> [   19.743009]  nouveau_gem_ioctl_pushbuf+0x6ba/0x11a0
> [   19.743011]  ? nouveau_gem_ioctl_new+0x100/0x100
> [   19.743012]  drm_ioctl_kernel+0x9f/0xe0
> [   19.743015]  drm_ioctl+0x214/0x3f0
> [   19.743016]  ? nouveau_gem_ioctl_new+0x100/0x100
> [   19.743017]  ? syscall_exit_to_user_mode+0x1d/0x40
> [   19.743019]  ? do_syscall_64+0x63/0x80
> [   19.743021]  __x64_sys_ioctl+0x80/0xa0
> [   19.743023]  do_syscall_64+0x56/0x80
> [   19.743025]  ? exc_page_fault+0x18c/0x4e0
> [   19.743026]  ? __do_sys_getpid+0x15/0x20
> [   19.743029]  entry_SYSCALL_64_after_hwframe+0x44/0xae
> [   19.743030] RIP: 0033:0x7f5e666c52d7
> [   19.743031] Code: 3c 1c e8 2c ff ff ff 85 c0 79 97 5b 49 c7 c4 ff ff ff ff 5d 4c 89 e0 41 5c c3 66 0f 1f 84 00 00 00 00 00 b8 10 00 00 00 0f 05 <48> 3d 01 f0 ff ff 73 01 c3 48 8b 0d 69 4b 0c 00 f7 d8 64 89 01 48
> [   19.743032] RSP: 002b:00007ffe8a5c2588 EFLAGS: 00000246 ORIG_RAX: 0000000000000010
> [   19.743034] RAX: ffffffffffffffda RBX: 00007ffe8a5c25e0 RCX: 00007f5e666c52d7
> [   19.743034] RDX: 00007ffe8a5c25e0 RSI: 00000000c0406481 RDI: 000000000000000e
> [   19.743035] RBP: 00000000c0406481 R08: 0000000000000000 R09: 00007f5e6678ac00
> [   19.743036] R10: 000055c9e96d3f10 R11: 0000000000000246 R12: 000055c9e8e17eb0
> [   19.743036] R13: 000000000000000e R14: 000055c9e8e14340 R15: 000055c9e9791120
> [   19.743037]  </TASK>
> [   19.743038] ---[ end trace e5a33326c4208bdb ]---
> [   19.743039] RIP: 0010:nouveau_fence_sync+0x6f/0x240
> [   19.743040] Code: 00 8b 7b 10 85 ff 0f 84 a7 00 00 00 41 89 cd 31 ed 31 d2 49 be ff ff ff ff ff ff ff 7f 4c 8b 7c d3 18 49 8b 94 24 90 00 00 00 <49> 8b 47 08 48 3d 00 37 0f a0 74 0c 48 3d 60 37 0f a0 0f 85 16 01
> [   19.743041] RSP: 0018:ffffb3bc413e7c10 EFLAGS: 00010202
> [   19.743042] RAX: 0000000000001001 RBX: ffff897383c07980 RCX: 0000000000000003
> [   19.743043] RDX: ffff897380cce000 RSI: 0000000000000001 RDI: 0000000000000001
> [   19.743043] RBP: 0000000000000001 R08: ffff897382af6c00 R09: ffff8973868641b8
> [   19.743044] R10: 0000000000000002 R11: ffff8973868641dc R12: ffff897386862400
> [   19.743045] R13: 0000000000000001 R14: 7fffffffffffffff R15: 3e40c25cd2e657b5
> [   19.743045] FS:  00007f5e661be8c0(0000) GS:ffff897ae0c00000(0000) knlGS:0000000000000000
> [   19.743046] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> [   19.743047] CR2: 000055c9e8e33d50 CR3: 00000001152b7001 CR4: 00000000003706f0
> [   19.743048] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
> [   19.743048] DR3: 0000000000000000 DR6: 00000000fffe0ff0 DR7: 0000000000000400
>