01:23mareko: I think ChatGPT would be a decent CEO
03:03jenatali: No comment
03:20HdkR: :D
04:45robclark: jenatali: so MS/openai is trying to break into the CEO-as-a-service market? :-P
05:14mattst88:buys chatgpt.ceo domain
12:49Hazematman: Hey all, based on the discussion yesterday on DMABUFs I created this MR to change the default path for `PIPE_CAP_DMABUF` in gallium to call `drmGetCap`. That way drivers that want to support importing DMABUFs without drm can simply override the default behavior in their driver. I would appreciate some feedback on the change :)
12:49Hazematman: https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/21654
13:09Lynne: err, why in unholy tarnation is vkGetMemoryHostPointerPropertiesEXT on AMD returning VK_MEMORY_PROPERTY_PROTECTED_BIT, which specifically forbids host-visible memory?
13:10zmike: define "AMD"
13:10Lynne: radv, of course
13:10zmike: bnieuwenhuizen hakzsam ^
13:12hakzsam: Lynne: can you fill a bug report?
13:14bnieuwenhuizen: Lynne: what do you mean VK_MEMORY_PROPERTY_PROTECTED_BIT? It only returns memory types, no?
13:15bnieuwenhuizen: and I don't think we have a type returning VK_MEMORY_PROPERTY_PROTECTED_BIT so I'm confused
13:15Lynne: yeah, me too, it's only returning 0x20 in memoryTypeBits
13:16bnieuwenhuizen: right, which memory type is that on your system?
13:16bnieuwenhuizen: (should be memory type 5?)
13:16Lynne: ah... I mistook VkMemoryPropertyFlagBits for memoryTypeBits
14:23gawin: wanna merge this hasvk patch https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/21444 someone has something against?
15:46karolherbst: any quick review on this llvmpipe patch? Or does somebody wants to trade review? I kind of want to land it https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/21604/diffs?commit_id=b5566bb33da40b0318ff15bb230eaf8bd47e2c3d
15:47karolherbst: and for our current EVoC student I'd need this MR looked at from llvmpipe folks as well: https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/20378/diffs?commit_id=fd73b7eaf241485e0e238900ca2a463c251a9f65
15:53zmike: probably tag airlied/sroland/brianp
19:16eric_engestrom: `weston --xwayland` prints `xserver listening on display :1`, but is there a way to programmatically get that from outside of weston?
19:16eric_engestrom: (for scripting)
19:20emersion: setup a pipe before starting the compositor, write DISPLAY to that pipe with a script auto-started by the compositor
19:20emersion: would be the most reliable way
19:22alyssa: pendingchaos: I see you're on board for cmarcelo's plan to delete all non-scoped_barriers :D
19:23alyssa: once all the infrastructure needed has landed I guess we should make a checklist of drivers to convert
19:24alyssa: admittedly these cross-tree reworks can be... slow to proliferate
19:25pendingchaos: deleting all non-scoped_barriers sounds like a good idea
19:25pendingchaos: but honestly, removing ACO's implementation of memory_barrier_buffer is more because it's an ugly hack that doesn't behave exactly like the scoped equivalent
19:26alyssa: nv50, zink, r600/sfn, v3d
19:26alyssa: ok I guess there are only 4 drivers that need to be converted to scoped barriers
19:26alyssa: I was expecting scarier
19:27alyssa: admitedly r600 and nv50 both kinda scare me :p
19:30alyssa: zmike: can I interest you in deleting zink code
19:31zmike:grimaces
19:31zmike: my license for utilizing the patented Delete The Code methodology ran out at the start of the year
19:31zmike: I haven
19:31zmike: t renewed
19:33eric_engestrom: emersion: thanks, that's a good idea
19:33eric_engestrom:looking up how to setup a start script in weston
19:34daniels: eric_engestrom: man weston.ini, search for 'autolaunch'
19:34eric_engestrom: thanks
19:34jenatali: alyssa: I think we have some code to delete if they go away entirely
19:36alyssa: zmike: :R
19:36alyssa: wiring up nir_intrinsic_scoped_barrier in zink should be Easy* since it's supposed to just be SPIR-V
19:36alyssa: *terms and conditions may apply
19:37alyssa: jenatali: yeah I didn't get around to deleting the dxil code but !21634 lands you can delete all your memory_barrier* and control_barrier code
19:37zmike: I imagine you could figure it out then if it's easy :P
19:37alyssa: zmike: man i've got 3 compilers to worry about and none of em are zink's :p
19:37jenatali: Sweet
19:37alyssa: dxil already handles scoped_barrier so there's nothing to do but delete
19:37alyssa: hence why it wasn't on my driver list of shame
19:38alyssa: zmike: Oooh ooh I know, I don't have any Vulkan drivers to test Zink with
19:39airlied: i wonder does virgl need some work
19:39zmike: lavapipe
19:39alyssa: That's my excuse and I'm sticking with it 😎
19:39alyssa: Oh drat
19:39alyssa: foiled
19:39airlied: not sure what tgsi barriers are
19:39zmike: mega foiled
19:39alyssa: airlied: I think virgl is ok
19:40alyssa: at least, virgl shouldn't be any more broken by this than it already is
19:40alyssa: venus otoh..
19:41alyssa: er wait venus doesn't use tgsi does it?
19:41cmarcelo: alyssa: airlied: tgsi has the idea of memory barrier atomic counter, which we currently don't have in scoped barrier. at one point we thought about havign a nir_var_atomic_counter and for all but TGSI users lower it down into ssbo, but not sure if it is worth it.
19:41alyssa: TGSI is on its last breath so I don't care
19:41alyssa: any new TGSI backends are auto-NAK from me
19:41alyssa: if gfxstrand doesn't beat me to it ;p
19:42zmike: I still need to implement ttv to optimize zink for nine
19:42alyssa: auto NAK
19:42zmike: it's happening
19:42alyssa: er
19:42alyssa:auto naks
19:43alyssa: If you really want to cut out ttn for zink9 then teach nine to produce NIR directly instead
19:45cmarcelo: alyssa: I thought TGSI was a key part of virgl, is it moving to use sth else?
19:46alyssa: cmarcelo: erm
19:46alyssa: I guess I should leave that to a virgl person
19:46airlied: it cant move
19:46airlied: its in the protocol
19:47alyssa: right, i think the question then is "is it currently broken?" because if not I think we're ok
19:48alyssa: ooh
19:48cmarcelo: alyssa: it shouldn't be broken now, because drivers are asking for the non-scoped version
19:48alyssa: OK, I see how this works
19:48alyssa: cmarcelo: Still ok :-)
19:49alyssa: Er wait
19:49alyssa: misread
19:49alyssa: hum right yes ok I see the problem now :|
19:50alyssa: Would it be acceptable to translate memory_barrier_buffer to "memoryBarrierBuffer(); memoryBufferAtomic();"?
19:50alyssa: If so, then we can just have ntt set the extra TGSI_MEMBAR_ATOMIC_BUFFER flag for buffer barriers
19:50alyssa: and everything works out conservatively
19:50cmarcelo: I had a local patch to let drivers ask for "do scoped but still feed me the atomic_counter intrinsic", that could also be sufficient
19:51cmarcelo: as long as TGSI drivers explicit ask for it (may need a PIPECAP)
19:51alyssa: worst case I guess we could do that
19:51alyssa: but the above solution is a local patch to tgsi-to-nir
19:51alyssa: and won't change perf on hardware that lowers atomics to SSBOs
19:51alyssa: i.e. everyone?
19:52alyssa: This is my preference anyway
19:54cmarcelo: need to see what users of TGSI prefer. I think it is fine either have a atomic_counter intrinsic for their sake (would now be properly documented as such) or even the nir_var_atomic_counter (although it seems more noisy).
19:56alyssa: Other than virgl/svga, I don't think any of the TGSI users care
19:56alyssa: virgl/svga just passes the buck up to the host
19:56alyssa: and on top of SSBO hw there's no perf difference
19:57alyssa: actually, it's r600/sfn that I'm worried about
19:57alyssa: since it seems to have "real" atomic counters (uniquely)
19:57alyssa: and it's a NIR backend
19:58alyssa: though I suspect it's fine to eat an extra "wait_ack" instruction in the rare case where memoryBarrierAtomic() is used but memoryBarrierBuffer() is not
21:00mareko: lavapipe flake: https://gitlab.freedesktop.org/mesa/mesa/-/jobs/37357938
21:00marex: jannau: hey, I dug this out New USB device found, idVendor=174c, idProduct=55aa, bcdDevice= 1.00
21:00marex: jannau: is that similar to what you have on the m1 ?
21:01marex: Product: ICY BOX IB-366StU3+B
21:01daniels: mareko: please put it in an MR
22:30jannau: marex: asmedia seems to reuse product ids for different products. probably a different chip since it supports only 5 gbps instead of 10 gpbs but worth a try
22:54marex: jannau: I'll see what I can do, unless someone really wants to start debugging it