<ajax>
no wonder the screen argument is null, we're not even trying
<zmike>
hm yea that looks like the only difference
<zmike>
I guess a victim of taking out renderdoc
* zmike
was planning to do this later on once the branch was ready to be landed
<zmike>
also looks like you added back some zink_screen struct members I deleted
<zmike>
bool needs_mesa_wsi;
<zmike>
bool needs_mesa_flush_wsi;
<ajax>
guh, thought i got those right
<zmike>
otherwise everything looks like I'd expect
<ajax>
yeah must have been fallout from renderdoc, the initial commit just moved the uqi so the later commit to fix that hunk would have maybe gotten confused
<ajax>
ah well
<ajax>
have a meeting now, will fix up shortly
<ajax>
maybe gotten confused meaning i f'd up the resolution obviously
<zmike>
meh it happens
<ajax>
:|
<ajax>
on main (or, where director's cut diverges from main anyway) i reliably crash in glmark2
<kusma>
But you can run it separately by doing "deqp/external/openglcts/modules/glcts -n dEQP-EGL.functional.color_clears.single_context.gles2.rgb888_window", for instance
<kusma>
ajax: Yeah, looks like what I found also...
<kusma>
batch usage is uninitialized, seems we're trying to reclaim garbage
<kusma>
That is, free'd memory
<ajax>
zink_batch_state_destroy not waiting for all outstanding batches?
<zmike>
looks more like a context was destroyed and somehow its usage remained on the resource
<zmike>
which shouldn't be possible :thinking:
<kusma>
Does the zink_bo maybe need to get the zink_batch_usage-pointers nulled out somehow?
<kusma>
I mean,the zink_bo objects live longer in this case, and reference the zink_batch_usage-structs
<kusma>
But I can't say I understand all this cache-magic here
<kusma>
Anyway, I need to hit the sack
<zmike>
that happens during batch reset
<zmike>
I guess I'll look tomorrow now that I have repro cases if it's not solved by the time I get up
* ajax
walking back in history to find what broke glmark2
<ajax>
the answer is it's been broken since before meson devenv support was added, and i can't be bothered to rewrite the wrapper script to still do the old thing
<ajax>
ngh
<zmike>
yeah I'd guess it's been broken in one form or another for a very long time