Skip to content

anv: uninitialize queues before utrace

Lionel Landwerlin requested to merge llandwerlin/mesa:review/anv-timeline-semaphore-tests-fix into main

What does this MR do and why?

anv: uninitialize queues before utrace

We need to shut down the runtime queue threads before tearing down anything else.

Gets rid of helgrind errors like this :

==212772== Possible data race during write of size 4 at 0xADCBFB0 by thread #1
==212772== Locks held: 1, at address 0x6B8F260
==212772==    at 0x8AC3EFF: simple_mtx_destroy (simple_mtx.h:97)
==212772==    by 0x8ACB24D: intel_ds_device_fini (intel_driver_ds.cc:603)
==212772==    by 0x6CBD4D4: anv_device_utrace_finish (anv_utrace.c:471)
==212772==    by 0x6C71577: anv_DestroyDevice (anv_device.c:3679)
==212772==    by 0x6B2F1E2: loader_layer_destroy_device (loader.c:4358)
==212772==    by 0x6B3F10B: vkDestroyDevice (trampoline.c:983)

Signed-off-by: Lionel Landwerlin lionel.g.landwerlin@intel.com Fixes: cc5843a5 ("anv: implement u_trace support") Closes: #10010 (closed)

Merge request reports