Hi Thomas，
At 2025-04-14 21:48:12, "Thomas Zimmermann" <[email protected]> wrote: >Instead of testing import_attach for imported GEM buffers, invoke >drm_gem_is_imported() to do the test. The helper tests the dma_buf >itself while import_attach is just an artifact of the import. Prepares >to make import_attach optional. > >Signed-off-by: Thomas Zimmermann <[email protected]> >Cc: Sandy Huang <[email protected]> >Cc: "Heiko Stübner" <[email protected]> >Cc: Andy Yan <[email protected]> >--- > drivers/gpu/drm/rockchip/rockchip_drm_gem.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c >b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c >index 6330b883efc3..e44396d46dc1 100644 >--- a/drivers/gpu/drm/rockchip/rockchip_drm_gem.c >+++ b/drivers/gpu/drm/rockchip/rockchip_drm_gem.c >@@ -332,7 +332,7 @@ void rockchip_gem_free_object(struct drm_gem_object *obj) > struct rockchip_drm_private *private = drm->dev_private; > struct rockchip_gem_object *rk_obj = to_rockchip_obj(obj); > >- if (obj->import_attach) { >+ if (drm_gem_is_imported(obj)) { After applying this patch, when I tested glmark2-es2-wayland under Weston, the weston would freeze. It seems it goto the else path. I'm still conducting further analysis to figure out the exact cause. > if (private->domain) { > rockchip_gem_iommu_unmap(rk_obj); > } else { >-- >2.49.0
