From 6d27c16f13e7f5a1b91f9aaad4cb55fd74773e26 Mon Sep 17 00:00:00 2001 From: Yiwei Zhang Date: Mon, 16 Jun 2025 01:44:06 -0700 Subject: [PATCH] v3dv: free up hwvulkan_device_t upon driver unloading Per , the hw_device_t::close() function is called upon driver unloading. The behavior has been like this since Android 10. Acked-by: Rob Clark Part-of: --- src/broadcom/vulkan/v3dv_android.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/src/broadcom/vulkan/v3dv_android.c b/src/broadcom/vulkan/v3dv_android.c index 8574823c7d5..e722db9b00d 100644 --- a/src/broadcom/vulkan/v3dv_android.c +++ b/src/broadcom/vulkan/v3dv_android.c @@ -97,6 +97,11 @@ v3dv_hal_open(const struct hw_module_t *mod, static int v3dv_hal_close(struct hw_device_t *dev) { - /* hwvulkan.h claims that hw_device_t::close() is never called. */ - return -1; + /* the hw_device_t::close() function is called upon driver unloading */ + assert(dev->version == HWVULKAN_DEVICE_API_VERSION_0_1); + assert(dev->module == &HAL_MODULE_INFO_SYM.common); + + hwvulkan_device_t *hal_dev = container_of(dev, hwvulkan_device_t, common); + free(hal_dev); + return 0; }