Hi Thomas，
At 2025-04-14 21:48:12, "Thomas Zimmermann" <tzimmerm...@suse.de> 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 <tzimmerm...@suse.de>
>Cc: Sandy Huang <h...@rock-chips.com>
>Cc: "Heiko Stübner" <he...@sntech.de>
>Cc: Andy Yan <andy....@rock-chips.com>
>---
> 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
