00:05 karolherbst[d]: okay.. so interesting think about the fp32 CL perf is that it's only slow on zink on my intel GPU
00:05 karolherbst[d]: like 10x slower
00:12 mhenning[d]: so it's zink's fault? 😛
00:15 karolherbst[d]: I don't know yet...
00:15 karolherbst[d]: the kernel is.. uhm..
00:16 karolherbst[d]: this beauty: https://gist.github.com/karolherbst/9ad293a8fb28addffe25df5e16c49d7d
00:17 karolherbst[d]: it looks trivial enough.. I wonder where codegen goes wrong or if it's something else going wrong
00:18 karolherbst[d]: ohhh shit...
00:18 karolherbst[d]: it's using the software fma emulation 🙃
00:18 karolherbst[d]: https://gist.githubusercontent.com/karolherbst/f8c7faaca6c2f3e81bcba658525c575c/raw/4a284ae99d8aeb7008f48e254d6eb9ba8482c702/gistfile1.txt
00:18 karolherbst[d]: so it's getting compiled to this 😄
00:18 mhenning[d]: well that would do it
00:18 karolherbst[d]: yeah...
00:19 karolherbst[d]: that means that zink is setting `lower_ffma32` mhh?
00:19 karolherbst[d]: man,.....
00:19 mhenning[d]: yeah weird
00:19 karolherbst[d]: what's the situation with fma in vulkan?
00:20 karolherbst[d]: like CL requires fma to be like fma
00:21 karolherbst[d]: there is `SPV_KHR_fma` mhh...
00:22 karolherbst[d]: which... seems to be fused
00:22 karolherbst[d]: `VK_KHR_shader_fma`
00:23 karolherbst[d]: I don't think we support this one in mesa?
00:23 mhenning[d]: There's a recent extension that requires fma to be fused
00:23 karolherbst[d]: right... I guess thats why people pinged on the fma MR I've opened a while ago 🙃
00:24 karolherbst[d]: anyway, I'd need it for rusticl
00:24 mhenning[d]: There's work on it but not landed yet. Also, nobody's worked on it for nvk yet afaik
00:24 karolherbst[d]: well
00:24 karolherbst[d]: it's mostly a "make fma sane in nir" thing
00:24 karolherbst[d]: because atm it's not
00:25 mhenning[d]: Right, it's mostly common code changes and flipping it on
00:25 mhenning[d]: so I figured I'd let radv or anv land it first
00:25 karolherbst[d]: mhhh
00:27 mhenning[d]: https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/37807 has some discussion
00:27 mhenning[d]: anyway I wasn't aware it's important for rusticl on zink. That might make it higher priority
00:32 karolherbst[d]: yeah...
00:32 karolherbst[d]: CL's fma is like.. always fused and it's not optional 🙃
00:36 karolherbst[d]: I suspect the same goes on for fp16...
00:37 karolherbst[d]: ohh no.. that's doing +*
00:39 karolherbst[d]: anyway.. one mystery solved 😄
22:40 phomes_[d]: the phoronix article mentions stability problems on DiRT Rally 2.0. I have been testing it today but the only thing I have been able to crash is the car in-game
23:03 karolherbst[d]: could be related to the graphics settings
23:03 karolherbst[d]: you can set up the phoronix test suite locally and try to run the same benchmark
23:03 karolherbst[d]: the last time I did it was like 3 years ago or so 🙃
23:03 karolherbst[d]: but it is doable