00:36 karolherbst: marex: I actually have a GT218...
00:37 karolherbst: to my big surprise
00:37 marex: karolherbst: is that similar to that quadro ?
00:38 karolherbst: maybe
00:38 marex: if you need PCI IDs or whatnot, the machine is still on
00:40 karolherbst: well.. would'nt help if I don't have such a GPU.. I was just wondering if I should try it out myself and see if I can reproduce it on any of my cards
00:41 marex: karolherbst: this MTHD thing happens during suspend/resume cycle reliably
00:42 marex: karolherbst: the sway/wlroots problem happens with default sway config, just start sway and suspend ... it will take a while
00:42 karolherbst: sure.. but it doesn't help if I can't reproduce it :D
00:42 marex: that;s probably the reproducer ^
00:42 marex: in case you want to attempt it
00:43 karolherbst: I know, but sometimes issues are annoying and they depend on weirdo timing
00:44 marex: that's a valid point
00:48 karolherbst: anyway.. just updated this machine to 5.14 :D
00:49 karolherbst: mhh.. saw weird artefacts on the console
00:49 karolherbst: "bios: OOB 1 d7500086 d7500086" that's not good at least... oh well
00:49 karolherbst: for later
00:49 marex: heh
00:50 karolherbst: yeah so.. well
00:50 karolherbst: the machine suspended
00:50 karolherbst: resuming on the other hand...
00:50 karolherbst: "[ 74.018029] nouveau 0000:01:00.0: fb: trapped read at 0000026000 on channel 1 [1fbbe000 DRM] engine 0d [PCE0] client 13 [] subclient 01 [] reason 00000006 [NULL_DMAOBJ]"
00:50 karolherbst: something seems busted :)
00:51 marex: I might;ve seen that in the log too
00:54 karolherbst: okay.. at least I am hitting one bug here :D
01:31 karolherbst: marex: yeah.. seems like the error is always different
10:45 marex: karolherbst: in my case the error is always the same, but I dont exercise the GPU too much, just kms console emulation and sway/wlroots
16:12 karolherbst: marex: seems like it broke with 5.12
17:04 marex: karolherbst: it seems it got a lot of testing since nobody noticed until now :)
17:16 karolherbst: most people use distributions like debian or so and stay on old kernels for a long time
17:16 karolherbst: but I think we have a few bug reports, but never got ti either reproduce it or whatever
17:17 karolherbst: could also be there is a patch but it never got applied
18:23 marex: karolherbst: I am using distributions like debian or so :)
18:24 marex: karolherbst: unstable and experimental are fine, unreleased is a bit flaky
18:26 marex: karolherbst: but I get the idea, dont worry
18:50 karolherbst: uhhg.. bisecting is annoying if you hit compilation errors in between
19:27 karolherbst: marex: uhhh.... I think I know what this is...
19:27 karolherbst: skeggsb_: did you had time to look into this engine_id bug on nv50?
19:30 karolherbst: marex: mind verifying that 64f7c698bea9 is broken and the commit before that works?
19:31 KitsuWhooa: Nice :p
19:35 karolherbst: Lyude: ping on that MR
19:37 karolherbst: even have gitlab issue for that https://gitlab.freedesktop.org/drm/nouveau/-/issues/120
20:03 marex: karolherbst: do you think ->engine_id() is called with unsupported engine ?
20:03 marex: if so, then I would hit the WARN_ON(1) and get a splat
20:22 Lyude: karolherbst: can you link to it? was just about to try to review it but I don't see it on my gitlab fdo todo list
20:22 karolherbst: Lyude: https://gitlab.freedesktop.org/drm/nouveau/-/merge_requests/8
20:22 Lyude: thanks
20:23 karolherbst: marex: the patch is broken so we end up using the wrong id
20:23 karolherbst: G84_FIFO_ENGN_GR is 1 but g84_fifo_chan_engine returned 0 for NVKM_ENGINE_GR before
21:10 marex: ah
21:45 marex: karolherbst: I'll try https://paste.debian.net/1214537/ since its easier to test right now
21:45 marex: it should have the same effect
21:51 karolherbst: maybe
22:15 marex: karolherbst: so far so good
23:18 marex: karolherbst: still good :)
23:28 skeggsb: that patch will be fine, you don't need the SW check though, it'll return already above for that case
23:29 skeggsb: re-send from earlier, because i don't believe my reply went through...
23:38 karolherbst: skeggsb: ah okay, so this will be enough to fix it? cool
23:38 karolherbst: I think I mentioned an idea like that a few weeks back
23:39 karolherbst: marex: if you send the patch to the ML tomorrow with a "Fixes:...." that would be great