From: Dave Airlie <airl...@redhat.com>

Signed-off-by: Dave Airlie <airl...@redhat.com>
---
 drivers/gpu/drm/amd/display/dc/core/dc_surface.c | 20 ++++++++++----------
 drivers/gpu/drm/amd/display/dc/dc.h              |  2 +-
 2 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_surface.c 
b/drivers/gpu/drm/amd/display/dc/core/dc_surface.c
index 0950075..d43783a 100644
--- a/drivers/gpu/drm/amd/display/dc/core/dc_surface.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc_surface.c
@@ -71,8 +71,8 @@ struct dc_plane_state *dc_create_plane_state(struct dc *dc)
        if (NULL == plane_state)
                return NULL;
 
+       kref_init(&plane_state->refcount);
        construct(core_dc->ctx, plane_state);
-       atomic_inc(&plane_state->ref_count);
 
        return plane_state;
 }
@@ -112,19 +112,19 @@ const struct dc_plane_status *dc_plane_get_status(
 
 void dc_plane_state_retain(struct dc_plane_state *plane_state)
 {
-       ASSERT(atomic_read(&plane_state->ref_count) > 0);
-       atomic_inc(&plane_state->ref_count);
+       kref_get(&plane_state->refcount);
 }
 
-void dc_plane_state_release(struct dc_plane_state *plane_state)
+static void dc_plane_state_free(struct kref *kref)
 {
-       ASSERT(atomic_read(&plane_state->ref_count) > 0);
-       atomic_dec(&plane_state->ref_count);
+       struct dc_plane_state *plane_state = container_of(kref, struct 
dc_plane_state, refcount);
+       destruct(plane_state);
+       kfree(plane_state);
+}
 
-       if (atomic_read(&plane_state->ref_count) == 0) {
-               destruct(plane_state);
-               kfree(plane_state);
-       }
+void dc_plane_state_release(struct dc_plane_state *plane_state)
+{
+       kref_put(&plane_state->refcount, dc_plane_state_free);
 }
 
 void dc_gamma_retain(struct dc_gamma *gamma)
diff --git a/drivers/gpu/drm/amd/display/dc/dc.h 
b/drivers/gpu/drm/amd/display/dc/dc.h
index 17f0c73..c4d89d2 100644
--- a/drivers/gpu/drm/amd/display/dc/dc.h
+++ b/drivers/gpu/drm/amd/display/dc/dc.h
@@ -385,7 +385,7 @@ struct dc_plane_state {
 
        /* private to dc_surface.c */
        enum dc_irq_source irq_source;
-       atomic_t ref_count;
+       struct kref refcount;
 };
 
 struct dc_plane_info {
-- 
2.9.5

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to