The function ttm_bo_get acquires a reference on a TTM buffer object. The
function's name is more aligned to the Linux kernel convention of naming
ref-counting function _get and _put.

Signed-off-by: Thomas Zimmermann <[email protected]>
---
 drivers/gpu/drm/vmwgfx/vmwgfx_bo.c       | 5 +++--
 drivers/gpu/drm/vmwgfx/vmwgfx_drv.h      | 5 ++---
 drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c  | 9 ++++++---
 drivers/gpu/drm/vmwgfx/vmwgfx_resource.c | 3 ++-
 4 files changed, 13 insertions(+), 9 deletions(-)

diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_bo.c 
b/drivers/gpu/drm/vmwgfx/vmwgfx_bo.c
index 2dda03345761..f6cb695e3b72 100644
--- a/drivers/gpu/drm/vmwgfx/vmwgfx_bo.c
+++ b/drivers/gpu/drm/vmwgfx/vmwgfx_bo.c
@@ -613,7 +613,8 @@ int vmw_user_bo_alloc(struct vmw_private *dev_priv,
        if (unlikely(ret != 0))
                return ret;
 
-       tmp = ttm_bo_reference(&user_bo->vbo.base);
+       ttm_bo_get(&user_bo->vbo.base);
+       tmp = &user_bo->vbo.base;
        ret = ttm_prime_object_init(tfile,
                                    size,
                                    &user_bo->prime,
@@ -910,7 +911,7 @@ int vmw_user_bo_lookup(struct ttm_object_file *tfile,
 
        vmw_user_bo = container_of(base, struct vmw_user_buffer_object,
                                   prime.base);
-       (void)ttm_bo_reference(&vmw_user_bo->vbo.base);
+       ttm_bo_get(&vmw_user_bo->vbo.base);
        if (p_base)
                *p_base = base;
        else
diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_drv.h 
b/drivers/gpu/drm/vmwgfx/vmwgfx_drv.h
index 1abe21758b0d..7ae942840213 100644
--- a/drivers/gpu/drm/vmwgfx/vmwgfx_drv.h
+++ b/drivers/gpu/drm/vmwgfx/vmwgfx_drv.h
@@ -1281,9 +1281,8 @@ static inline void vmw_bo_unreference(struct 
vmw_buffer_object **buf)
 static inline struct vmw_buffer_object *
 vmw_bo_reference(struct vmw_buffer_object *buf)
 {
-       if (ttm_bo_reference(&buf->base))
-               return buf;
-       return NULL;
+       ttm_bo_get(&buf->base);
+       return buf;
 }
 
 static inline struct ttm_mem_global *vmw_mem_glob(struct vmw_private *dev_priv)
diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c 
b/drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c
index 1f134570b759..e8e30ff36d3b 100644
--- a/drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c
+++ b/drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c
@@ -580,8 +580,9 @@ static int vmw_bo_to_validate_list(struct vmw_sw_context 
*sw_context,
                        return ret;
                }
                ++sw_context->cur_val_buf;
+               ttm_bo_get(&vbo->base);
                val_buf = &vval_buf->base;
-               val_buf->bo = ttm_bo_reference(&vbo->base);
+               val_buf->bo = &vbo->base;
                val_buf->shared = false;
                list_add_tail(&val_buf->head, &sw_context->validate_nodes);
                vval_buf->validate_as_mob = validate_as_mob;
@@ -4408,11 +4409,13 @@ void __vmw_execbuf_release_pinned_bo(struct vmw_private 
*dev_priv,
 
        INIT_LIST_HEAD(&validate_list);
 
-       pinned_val.bo = ttm_bo_reference(&dev_priv->pinned_bo->base);
+       ttm_bo_get(&dev_priv->pinned_bo->base);
+       pinned_val.bo = &dev_priv->pinned_bo->base;
        pinned_val.shared = false;
        list_add_tail(&pinned_val.head, &validate_list);
 
-       query_val.bo = ttm_bo_reference(&dev_priv->dummy_query_bo->base);
+       ttm_bo_get(&dev_priv->dummy_query_bo->base);
+       query_val.bo = &dev_priv->dummy_query_bo->base;
        query_val.shared = false;
        list_add_tail(&query_val.head, &validate_list);
 
diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_resource.c 
b/drivers/gpu/drm/vmwgfx/vmwgfx_resource.c
index 92003ea5a219..485a3e8d82db 100644
--- a/drivers/gpu/drm/vmwgfx/vmwgfx_resource.c
+++ b/drivers/gpu/drm/vmwgfx/vmwgfx_resource.c
@@ -458,7 +458,8 @@ vmw_resource_check_buffer(struct ww_acquire_ctx *ticket,
        }
 
        INIT_LIST_HEAD(&val_list);
-       val_buf->bo = ttm_bo_reference(&res->backup->base);
+       ttm_bo_get(&res->backup->base);
+       val_buf->bo = &res->backup->base;
        val_buf->shared = false;
        list_add_tail(&val_buf->head, &val_list);
        ret = ttm_eu_reserve_buffers(ticket, &val_list, interruptible, NULL);
-- 
2.18.0

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to