Am Dienstag, 4. Jänner 2022, 18:52:11 CET schrieb Zebediah Figura (she/her):
FWIW, patch 2/5 is broken where it is; wined3d_device_context_upload_bo() does acquire the resource. Ultimately that just means that the patch needs to be moved later in the series, though.
Oh indeed, I missed that WINED3D_CS_OP_UPDATE_SUB_RESOURCE is used from two places. Still, doesn't make much of a difference in the whole scheme.
I can't move it behind patch 3 as-is because as of patch 3 we can't fence map ops any longer, so we'd set a nonsense access time. Merging it into patch 3 would be an option, but I am sure we can do this more elegantly.