From 5f4359ea0ed54b06d443e0ba040eb73406fc3e34 Mon Sep 17 00:00:00 2001 From: Axel Davy Date: Wed, 28 Dec 2016 18:58:48 +0100 Subject: [PATCH] st/nine: Flush the queue at device dtor Flush the queue to get refcounts right, and properly release the items, instead of throwing away all pending commands. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/device9.c b/src/gallium/state_trackers/nine/device9.c index b150faaf49d..92bc72ce28d 100644 --- a/src/gallium/state_trackers/nine/device9.c +++ b/src/gallium/state_trackers/nine/device9.c @@ -535,8 +535,13 @@ NineDevice9_dtor( struct NineDevice9 *This ) DBG("This=%p\n", This); - /* Do not call nine_csmt_process here. The device is dead! */ + /* Flush all pending commands to get refcount right, + * and properly release bound objects. It is ok to still + * execute commands while we are in device dtor, because + * we haven't released anything yet. Note that no pending + * command can increase the device refcount. */ if (This->csmt_active && This->csmt_ctx) { + nine_csmt_process(This); nine_csmt_destroy(This, This->csmt_ctx); This->csmt_active = FALSE; This->csmt_ctx = NULL; -- 2.30.2