00:36fdobridge: <gfxstrand> @airlied Eric wants modifiers in my Wednesday if we want it in 24.1. in order for that to happen, you or Karol needs to review the Mesa MR or at least enough to be sure the kernel API is good and we need to merge the kernel patch.
00:36fdobridge: <airlied> yeah I'm going through it at the moment
00:45fdobridge: <gfxstrand> Cool, Thanks!
00:56fdobridge: <airlied> just fixing up the conditional for using zink to check the new getparam seems to make things fine in most use cases where I add the use zink by default change on top
00:56fdobridge: <airlied> the copr seems to work my desktop now, with an old kernel and a new kernel with the mod patch
01:00fdobridge: <airlied> though steam is throwing up a kopper error about missing interfaces which suggests I might need more work in that area
01:05fdobridge: <airlied> @zmike. seeing some flickering white on steam app while moving mouse across menus
01:09fdobridge: <zmike.> 🤔
01:09fdobridge: <zmike.> just on nvk?
01:10fdobridge: <airlied> not sure I've got an alternate test box to confirm
01:10fdobridge: <airlied> seems like a sync issue, since it renderers the menu box in white, then the content
01:10fdobridge: <zmike.> 🤔
01:10fdobridge: <zmike.> I know steam was working fine very recently
01:10fdobridge: <zmike.> something to look at tomorrow I guess
01:16fdobridge: <airlied> https://photos.app.goo.gl/R9noKEUwd75GsS64A
01:56fdobridge: <Misyl with Max-Q Design> Your video will be ready soon
01:58fdobridge: <airlied> discord must be doing something
03:07fdobridge: <airlied> bleh still some race in the kernel now, seeing an mmu fault after runpm
03:13fdobridge: <mhenning> @airlied might be https://gitlab.freedesktop.org/drm/nouveau/-/issues/346
03:15fdobridge: <airlied> ah yes that's the one, just thought fixing the evict stuff fixed that as well
04:28fdobridge: <airlied> @skeggsb9778 cleaned up the dma code and got it working I think
04:29fdobridge: <airlied> but I've got a bunch of further cleanups I think I can do, so I might play around a bit more
05:00fdobridge: <airlied> bleh dma_alloc_contiguous doesn't work on x86 if you want single pages yet
06:34fdobridge: <airlied> bleh dma_alloc_noncontiguous doesn't work on x86 if you want single pages yet (edited)
06:48airlied: Lyude: sent out a newer fix to the BUG_ON , not sure it's fully correct either, but seems to work here
08:48fdobridge: <redsheep> That video still is not viewable, but I have also seen flicker in steam and it's similar to the flicker I get in various of kde apps.
08:49fdobridge: <redsheep> Given the feedback about correct usage of gamescope with mangohud I am getting a new screenshot so the blig post won't show off using it wrong
08:49fdobridge: <redsheep> Given the feedback about correct usage of gamescope with mangohud I am getting a new screenshot so the blog post won't show off using it wrong (edited)
09:01fdobridge: <redsheep> For whatever reason `gamescope --mangoapp -- %command%` did not result in mangohud rendering so here's `gamescope --mangoapp -- %command%`
09:01fdobridge: <redsheep> https://cdn.discordapp.com/attachments/1034184951790305330/1239502838783017100/image.png?ex=66432879&is=6641d6f9&hm=0b7ec501c5bce62f64dd48fe34530490407fd5d2ea0e3b51e82ad61d30ae80f9&
09:03fdobridge: <!DodoNVK (she) 🇱🇹> These commands look the same to me
09:04fdobridge: <redsheep> *sigh* Yeah the actual command used is the one in the screenshot, `gamescope -- mangohud %command%` I just pasted wrong
09:05fdobridge: <redsheep> Just working from Joshie's feedback here: https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/24795
09:11fdobridge: <karolherbst🐧🦀> does anybody know what games/applications heavily rely on glMultiDraw* being low CPU overhead? @zmike. maybe?
09:13fdobridge: <redsheep> Hmm I am trying to remember if there are any notable RTS games that are native GL. Those would be a great candidate, I think
09:15fdobridge: <djdeath3483> like indirects? gravitymark uses a bunch of 10k+ indirect at 200k asteroids
09:15fdobridge: <djdeath3483> it's got gl/vk backends
09:15fdobridge: <karolherbst🐧🦀> Civilization doesn't use it :ferrisUpsideDown:
09:15fdobridge: <karolherbst🐧🦀> at least 5 doesn't
09:16fdobridge: <karolherbst🐧🦀> which.. I think explains the bad performance 😄
09:16fdobridge: <karolherbst🐧🦀> not indirects, glMultiDraw*, it's just batched draws
09:17fdobridge: <karolherbst🐧🦀> like.. instead of calling draw 100 times with the same state, you call glMultiDraw* with 100 draws
09:17fdobridge: <djdeath3483> hmmm okay, I don't know then
09:32fdobridge: <Sid> I wonder if Stellaris does
09:32fdobridge: <Sid> I should test that save file I got from my friend
09:33fdobridge: <magic_rb.> I think unity can do it, whether the devs could be bothered to use multidraw, ive no idea
09:34fdobridge: <magic_rb.> But stellaris is mostly cpu bound afaik so they might have
09:35fdobridge: <karolherbst🐧🦀> well.. so is civilization 5
09:35fdobridge: <karolherbst🐧🦀> and they haven't
09:36fdobridge: <karolherbst🐧🦀> but I don't know what engine they are using
09:36fdobridge: <Sid> afaik paradox uses their own engine
09:36fdobridge: <Sid> yup
09:36fdobridge: <Sid> https://en.m.wikipedia.org/wiki/Paradox_Development_Studio#Game_engines
09:37fdobridge: <Sid> Clausewitz Engine since 2007
09:37fdobridge: <karolherbst🐧🦀> uhh.. I have no idea if I have any games from them
09:38fdobridge: <Sid> Europa Universalis
09:38fdobridge: <karolherbst🐧🦀> EU3 and CK2 apparently
09:38fdobridge: <Sid> Hearts of Iron
09:38fdobridge: <Sid> yeah there you go
09:39fdobridge: <karolherbst🐧🦀> but like.. if thye don't have a benchmark mode I'm kinda scrwed 😄
09:44fdobridge: <Sid> they don't
09:44fdobridge: <Sid> there's a reason I asked my friend for that late game save file :p
12:13fdobridge: <zmike.> I think some source engine stuff does
12:13fdobridge: <zmike.> unigine benchmarks do
12:13fdobridge: <zmike.> oh
12:13fdobridge: <zmike.> actual glMultiDraw?
12:13fdobridge: <zmike.> no idea
12:14fdobridge: <zmike.> DISREGARD
12:14fdobridge: <karolherbst🐧🦀> like whatever ends up calling `draw_vbo` with `num_draws > 1`
12:14fdobridge: <zmike.> not necessarily the same
12:14fdobridge: <zmike.> that could be multiple draw calls batched together
12:14fdobridge: <zmike.> why not just use drawoverhead
12:14fdobridge: <karolherbst🐧🦀> does that happen in a non threaded context?
12:14fdobridge: <zmike.> no
12:15fdobridge: <karolherbst🐧🦀> soo nouveau doesn't use threaded context 🙂
12:15fdobridge: <karolherbst🐧🦀> yeah, and the numbers are x60 improvement and all that there, just want to test games being more complex and stuff
12:17fdobridge: <zmike.> I don't usually inspect api command streams
12:17fdobridge: <zmike.> I just notice sometimes when I get multidraws
12:17fdobridge: <zmike.> doom2016 has some
12:19fdobridge: <karolherbst🐧🦀> I see
12:52fdobridge: <zmike.> what do I need in my cross file to do a 32bit nvk build
12:52fdobridge: <zmike.> for rust
12:58fdobridge: <karolherbst🐧🦀> `rust = '/home/kherbst/.rustup/toolchains/1.59-i686-unknown-linux-gnu/bin/rustc'` or something
12:59fdobridge: <karolherbst🐧🦀> just need to point to the 32 bit toolchain rustc thing
13:01fdobridge: <zmike.> fedora doesn't seem to provided i686 packages for rust?
13:01fdobridge: <zmike.> fedora doesn't seem to provide i686 packages for rust? (edited)
13:02fdobridge: <karolherbst🐧🦀> yeah dunno
13:02fdobridge: <karolherbst🐧🦀> I don't use distribution provided toolchains
13:03HdkR: I currently have rusticl and asahi disabled for 32-bit due to the lack of i686 packages and cross compiling problems
13:03HdkR: In Fedora
13:03fdobridge: <zmike.> :grimace:
13:04fdobridge: <zmike.> that's not ideal since I apparently have to build this shit to test airlied's issue
13:04fdobridge: <zmike.> maybe I can get away with swrast
13:15fdobridge: <zmike.> the answer is no
13:24fdobridge: <redsheep> Are you talking about testing the flicker on steam when running with a zink session on the modifiers branch?
13:27fdobridge: <redsheep> He linked to builds he did for fedora that specifically include the modifiers patches and I think they enable zink by default as well https://discord.com/channels/1033216351990456371/1034184951790305330/1238010389271154708
13:31fdobridge: <zmike.> hmmm
13:31fdobridge: <gfxstrand> Here's my cross file.
13:31fdobridge: <gfxstrand> https://cdn.discordapp.com/attachments/1034184951790305330/1239570681868324894/x86.cross?ex=664367a8&is=66421628&hm=e3f812b5fb67bd5ca122948c03b910598821b0288825396dcd5d3a99f674b103&
13:32fdobridge: <clangcat> Yes they do and iirc it's limited to one CPU core.
13:33fdobridge: <clangcat> Like for paradox games it seems much more likely that your CPU will be the bottleneck long before your GPU is.
13:33fdobridge: <zmike.> thx, though that doesn't help with fedora not shipping i686 :/
13:33fdobridge: <redsheep> It would still be better to get a newer build since I don't see one from him that is new enough to have the latest patches
13:35fdobridge: <gfxstrand> It's one executable. Are you sure Fedora doesn't package the i686 target?
13:36fdobridge: <zmike.> hmmm
13:36fdobridge: <redsheep> Also, do you have a kernel with the vm bind patch needed for NVK modifiers? https://lists.freedesktop.org/archives/nouveau/2024-May/044705.html
13:37fdobridge: <zmike.> apparently all I needed was the rust lines
13:42fdobridge: <zmike.> ...and a manual rustup
13:42fdobridge: <zmike.> https://gitlab.freedesktop.org/mesa/mesa/-/issues/11150 as a tracker
13:45fdobridge: <zmike.> ughhhhhhhhhhhh steam can't be apitraced anymore
14:00fdobridge: <zmike.> great news, it's not a zink bug
14:04fdobridge: <zmike.> conferring with people internally, this is expected behavior for the underlying CEF context
14:04fdobridge: <zmike.> so
14:04fdobridge: <zmike.> live with it
14:04HdkR: Quality CEF moment
14:05fdobridge: <zmike.> yeah apparently if you're in a composited environment we make the window transparent so that (hopefully) you don't see this
14:05fdobridge: <zmike.> but also sometimes we can't enable transparency because CEF does what it wants
14:07fdobridge: <redsheep> Sounds like a different bug from the flicker I had been seeing then, I had hoped they were the same. I don't think Konsole or Kate are CEF based
14:09fdobridge: <zmike.> I'm only talking about the steam white rect loading flicker
14:09fdobridge: <gfxstrand> What's CEF?
14:09fdobridge: <redsheep> Right
14:09fdobridge: <redsheep> Chromium embedded framework
14:10fdobridge: <redsheep> Browser based app ui
14:10fdobridge: <gfxstrand> ah
14:11fdobridge: <zmike.> it's teh thing that makes steam work
14:11fdobridge: <Misyl with Max-Q Design> or not work depending on who you ask
14:12fdobridge: <zmike.> or both at the same time!
14:12fdobridge: <zmike.> anyway we're apparently attempting to flatten them down onto the main interface, so at some point this problem should go away on its own
14:12fdobridge: <Misyl with Max-Q Design> My imgui UI does that
14:12fdobridge: <Misyl with Max-Q Design> Ehe
14:13fdobridge: <Misyl with Max-Q Design> Im excited for modifier stuff to be merged
14:23fdobridge: <redsheep> @gfxstrand You will probably want to swap out your screenshot on the MR for this one
14:24fdobridge: <redsheep> So as to not misinform the people with my unknowingly incorrect usage
15:06OftenTimeConsuming: Hmm, speed-dreams-2 is really the ultimate driver tester - any level that almost reaches 60fps causes a kref push error.
15:24fdobridge: <clangcat> Damn if want to test nvk I need to compile Mesa.
15:24fdobridge: <clangcat> This is slowly turning into my system just being custom compiled.
15:26OftenTimeConsuming: I can compile mesa just fine, the main issue with NVK is that it's rusted - you can't compile it without rusting up your toolchain.
15:32fdobridge: <clangcat> Ahhh well good to know I need rust installed
15:53fdobridge: <gfxstrand> Done. I'd already updated the blog post but not the MR.
15:57fdobridge: <redsheep> Sounds good, I am excited for the read when that goes up
16:16fdobridge: <!DodoNVK (she) 🇱🇹> ~~Rewrite Steam in VGUI2~~
16:38fdobridge: <gfxstrand> NVK stickers are now up: https://my-store-f6eea7.creator-spring.com/listing/nvk-stickers?product=663
16:50fdobridge: <zmike.> smh not even product 666
17:11fdobridge: <gfxstrand> And, black stickers if that's more your jam: https://my-store-f6eea7.creator-spring.com/listing/nvk-stickers-black
17:58fdobridge: <gfxstrand> @airlied What tree are we planning to land the tiled BO kernel patch through?
18:00fdobridge: <airlied> It'll probably go in via misc-next-fixes so I get it into 6.10rc1 in 2 werks
18:00airlied: dakr: fyi ^
18:11fdobridge: <gfxstrand> Please let me know when it lands.
18:38dakr: gfxstrand, airlied: Will have a look. I wasn't aware of the patch..
18:38dakr: I think it did not hit the Nouveau list, probably need to relax the spam filter a bit.
18:39airlied: not sure if author was subscribed, no idea who does nouveau list moderation if anyone
18:40airlied: https://patchwork.freedesktop.org/patch/593396/
18:41airlied: just land that one into drm-misc-next-fixes I think
20:19fdobridge: <skeggsb9778> Did you figure out why Lyude's patch broke?
20:45fdobridge: <airlied> Just appears we were getting a kmalloc allocation that wasn't suitable to DMA from which confuses me, using the non coherent API seems to avoid the assert and that problem
20:45fdobridge: <airlied> Not sure if it was just some kmalloc alignment problem, though I did try using a kcache to no avail
22:32Lyude: gfxstrand: didn't realize you were making stickers!
22:33Lyude: i should advertise the sticker business my roommate runs more lol
23:07fdobridge: <gfxstrand> dakr: Thanks! I just kicked the NVK bits off to marge. 😁