02:05 fdobridge: <g​fxstrand> Here's a codegen run:
02:05 fdobridge: <g​fxstrand> `Pass: 312031, Fail: 1296, Crash: 137, Skip: 1673845, Flake: 66, Duration: 28:48`
04:09 fdobridge: <g​fxstrand> NAK: `Pass: 311397, Fail: 920, Crash: 1098, Skip: 1673427, Flake: 33, Duration: 1:00:1`
07:31 fdobridge: <g​fxstrand> NAK: `Pass: 311523, Fail: 917, Crash: 1042, Skip: 1673845, Flake: 48, Duration: 59:58`
16:43 fdobridge: <g​fxstrand> NAK: `Pass: 311510, Fail: 912, Crash: 1042, Skip: 1673845, Flake: 66, Duration: 59:51`
16:43 fdobridge: <g​fxstrand> Image atomics were totally busted and fixing them fixed... 5 tests?!?!?
16:50 fdobridge: <g​fxstrand> Aparently, all the image atomic tests require SPIR-V 1.3 🙃
17:05 fdobridge: <g​fxstrand> VK_KHR_spirv_1_4 here we go!
17:16 fdobridge: <!​[NVK Whacker] Echo (she) 🇱🇹> Have you tested vulkanMamoryModel yet?
17:16 fdobridge: <!​[NVK Whacker] Echo (she) 🇱🇹> Have you tested vulkanMemoryModel yet? (edited)
17:21 fdobridge: <g​fxstrand> No but it should mostly pass with NAK. It's mostly about not optimizing too much which means codegen will probably fail.
17:26 fdobridge: <g​fxstrand> Codegen with VK_KHR_spirv_1_4:
17:26 fdobridge: <g​fxstrand> `Pass: 313672, Fail: 1355, Crash: 161, Skip: 1672133, Flake: 54, Duration: 29:33`
17:26 fdobridge: <g​fxstrand> Apparently image atomics are broken in codegen. 🙄
17:29 fdobridge: <g​fxstrand> It's probably getting signed vs. unsigned wrong. You have to look at the atomic op itself, not the image type for that.
18:28 fdobridge: <m​henning> oop, yep we do that wrong
18:39 fdobridge: <m​henning> @gfxstrand Untested but https://gitlab.freedesktop.org/mhenning/mesa/-/commit/99810829b3b60765d7f4202d66cfb39551892e1c might fix codegen
19:06 fdobridge: <k​arolherbst🐧🦀> ahh yeah... do we have some nir functions to query that information these days?
19:10 fdobridge: <m​henning> I don't see signedness in nir_intrinsic_info
19:17 fdobridge: <m​henning> I also don't immediately see anything in the gl cts that would test that patch
19:17 fdobridge: <k​arolherbst🐧🦀> isn't there a function to retrieve the type directly or something now?
19:21 fdobridge: <m​henning> looks like Faith has a pr https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/22724
19:21 fdobridge: <m​henning> but not yet
20:21 fdobridge: <g​fxstrand> Looks like it should work. I'm running the CTS on it now. If it works, I'll push it to the nvk/main branch and let you know.
23:12 fdobridge: <g​fxstrand> Fixes the bug. Feel free to slap a `Reviewed-by: Faith Ekstrand <faith.ekstrand@collabora.com>` on it and merge upstream.
23:13 fdobridge: <g​fxstrand> Or, yaknow, maybe make an MR and give others a chance to review too if they want. 😅 Go ahead and put my tag on it, though. I'm gonna push it to nvk/main right now.
23:46 fdobridge: <S​amantas5855> is codegen like the default compiler for all drivers on mesa?
23:47 fdobridge: <!​[NVK Whacker] Echo (she) 🇱🇹> (it's not)
23:48 fdobridge: <S​amantas5855> what is it then
23:48 fdobridge: <!​[NVK Whacker] Echo (she) 🇱🇹> I think it's src/compiler 🐸
23:51 fdobridge: <Z​iemas> codegen is the old nouevau compiler
23:51 fdobridge: <Z​iemas> codegen is the old nouveau compiler (edited)
23:52 fdobridge: <S​amantas5855> ohhh
23:53 fdobridge: <S​amantas5855> how's perf btw
23:53 fdobridge: <g​fxstrand> F1 2015 runs fine. I didn't have an overlay up so I don't know how many FPS but it was smooth.
23:54 fdobridge: <S​amantas5855> hype
23:54 fdobridge: <S​amantas5855> so your 980m laptop is maxwell2
23:54 fdobridge: <S​amantas5855> but my 940m is maxwell 1?
23:55 fdobridge: <M​ohamexiety> NVIDIA does not have the best naming scheme for mobile to say the least. there's even a cursed 9x0m SKU that's based on _fermi_
23:55 fdobridge: <M​ohamexiety> (the 920m iirc?)
23:56 fdobridge: <k​arolherbst🐧🦀> don't think too much about it
23:57 fdobridge: <k​arolherbst🐧🦀> they got a lot better tho
23:57 fdobridge: <M​ohamexiety> yeah. these days are better
23:58 fdobridge: <M​ohamexiety> there's still a weird SKU in this lineup, but it's ultra low end so meh..
23:58 fdobridge: <M​ohamexiety> (RTX 2050... it's actually Ampere, not Turing)
23:58 fdobridge: <S​amantas5855> there's a 940m with nvenc
23:58 fdobridge: <S​amantas5855> and there's my 940m without nvenc
23:58 fdobridge: <S​amantas5855> terrible naming
23:58 fdobridge: <k​arolherbst🐧🦀> there are like cursed models like the GT 640 (GF116, GK107 or GK208) or the GT 630 (GF108, GK107, GK208
23:59 fdobridge: <M​ohamexiety> yeah....
23:59 fdobridge: <S​amantas5855> so karol technically can we reclock pascal
23:59 fdobridge: <S​amantas5855> without PMU firmware?
23:59 fdobridge: <k​arolherbst🐧🦀> no
23:59 fdobridge: <k​arolherbst🐧🦀> I mean...
23:59 fdobridge: <k​arolherbst🐧🦀> technically yes, but we can't change the voltage