<kchibisov>
Since I think while debugging my issue with native egl resizing I was trying to do something you've done, but for all the egl code.
* kchibisov
is talking about the lines you've added for wayland egl platform.
<zmike>
what client were you triggering this with
<kchibisov>
alacritty.
<kchibisov>
But it happens only on window creation and when asking for multiple egl surfaces...
<kchibisov>
Well, or delition. But you should trigger resize of all the windows near at the same time.
<kchibisov>
I've checked the client, libwayland-egl, and only mesa was doing something I don't understand.
<kchibisov>
I got kind of lost when debugging it, since it happens randomly when closing one egl surface and others get resized at the same time.
<kchibisov>
And in those rare cases they all mark the surface for resize, but continue to use old buffer for the swap_buffers.
<kchibisov>
Note, you'd need something to trigger resize at the same time for all the surfaces for one particular instance, so some tiling manager should be used. Not sure such things could happen on e.g. gnome given that other surfaces don't resize...
<kchibisov>
zmike: I just don't quite get the line you've added after swapBuffers, since mesa is doing exactly the same before swapping and pointing to the back that will be used for the next frame, but then you manually advance it?
<zmike>
no, mesa doesn't handle this
<kchibisov>
Ah, zink is using swrast?
<zmike>
no, zink is using kopper
<kchibisov>
No, I mean the dri2_wl_swrast_display_vtbl?
<zmike>
ah
<zmike>
yes
<kchibisov>
Yeah, it's doing so on the other path for native opengl.
<kchibisov>
So probably I'd be able to verify whether my original issue with resizing is present with zink(which is why I've decided to use zink in the first place).
<kchibisov>
Since if it's not, it's other platform path issue.
<ajax>
zmike: i have to run to an appointment in a few, will take a look when i get back but that might be much later tonight
<zmike>
yeah no rush
<Soroush>
Hi, what is the coding style for zink?
<zmike>
same as general mesa
<zmike>
3 spaces
<Soroush>
Thanks, I am looking into implementing PIPE_QUERY_PRIMITIVES_GENERATED with VK_QUERY_TYPE_PRIMITIVES_GENERATED_EXT instead of the current situation where it uses 3 different queries. The platform I am working on doesn't have pipelineStatisticsQuery
<zmike>
that's already been done though 🤔
<Soroush>
Oh is it? I am looking at the 22.0 branch. I'll check the main
<zmike>
might be new in 22.1
<Soroush>
yup implemented in a9451f25
<Soroush>
I'll see if I can backport it easily to 22.0
eukara has quit [Remote host closed the connection]
<zmike>
ajax: got more dmabuf/wsi-related MRs up now
<zmike>
they're multiplying
<kchibisov>
zmike: the patch you've linked doesn't compile for me when applying on mesa main. I'd assume it's based on some other work, given that there's no `drm_fd` on `zink_screen`?