Enhanced the error logs. 90/234690/1
authorJoonbum Ko <joonbum.ko@samsung.com>
Thu, 28 May 2020 03:52:09 +0000 (12:52 +0900)
committerJoonbum Ko <joonbum.ko@samsung.com>
Thu, 28 May 2020 03:52:09 +0000 (12:52 +0900)
Change-Id: I0ec3e186f1318a050c75fe1596b60803191f8100
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
src/tpl_wl_egl_thread.c

index 5ca869171aa03119b53b7ed1daa5ca0b228cc629..15697a47a99106bd56f56875774a0e620dae9174 100755 (executable)
@@ -448,7 +448,7 @@ __tpl_wl_egl_surface_fini(tpl_surface_t *surface)
 
 static tpl_result_t
 __tpl_wl_egl_surface_set_rotation_capability(tpl_surface_t *surface,
-                                                                                                 tpl_bool_t set)
+                                                                                        tpl_bool_t set)
 {
        tpl_wayland_egl_surface_t *wayland_egl_surface = NULL;
 
@@ -459,8 +459,8 @@ __tpl_wl_egl_surface_set_rotation_capability(tpl_surface_t *surface,
 
        wayland_egl_surface = (tpl_wayland_egl_surface_t *)surface->backend.data;
        if (!wayland_egl_surface) {
-               TPL_ERR("Invalid parameter. wayland_egl_surface(%p)",
-                               wayland_egl_surface);
+               TPL_ERR("Invalid parameter. surface(%p) wayland_egl_surface(%p)",
+                               surface, wayland_egl_surface);
                return TPL_ERROR_INVALID_PARAMETER;
        }
 
@@ -478,7 +478,7 @@ __tpl_wl_egl_surface_set_rotation_capability(tpl_surface_t *surface,
 
 static tpl_result_t
 __tpl_wl_egl_surface_set_post_interval(tpl_surface_t *surface,
-               int post_interval)
+                                                                          int post_interval)
 {
        tpl_wayland_egl_surface_t *wayland_egl_surface = NULL;
 
@@ -489,8 +489,8 @@ __tpl_wl_egl_surface_set_post_interval(tpl_surface_t *surface,
 
        wayland_egl_surface = (tpl_wayland_egl_surface_t *)surface->backend.data;
        if (!wayland_egl_surface) {
-               TPL_ERR("Invalid parameter. wayland_egl_surface(%p)",
-                               wayland_egl_surface);
+               TPL_ERR("Invalid parameter. surface(%p) wayland_egl_surface(%p)",
+                               surface, wayland_egl_surface);
                return TPL_ERROR_INVALID_PARAMETER;
        }
 
@@ -527,8 +527,8 @@ __tpl_wl_egl_surface_enqueue_buffer(tpl_surface_t *surface,
        bo_name = tbm_bo_export(tbm_surface_internal_get_bo(tbm_surface, 0));
 
        if (!wayland_egl_surface) {
-               TPL_ERR("Invalid parameter. wayland_egl_surface(%p)",
-                               wayland_egl_surface);
+               TPL_ERR("Invalid parameter. surface(%p) wayland_egl_surface(%p)",
+                               surface, wayland_egl_surface);
                TRACE_ASYNC_END((int)tbm_surface, "[DEQ]~[ENQ] BO_NAME:%d", bo_name);
                TPL_OBJECT_UNLOCK(wayland_egl_surface);
                return TPL_ERROR_INVALID_PARAMETER;
@@ -600,8 +600,8 @@ __tpl_wl_egl_surface_enqueue_buffer(tpl_surface_t *surface,
                                                                                tbm_surface);
        if (tsq_err != TBM_SURFACE_QUEUE_ERROR_NONE) {
                tbm_surface_internal_unref(tbm_surface);
-               TPL_ERR("Failed to enqueue tbm_surface(%p). tsq_err=%d",
-                               tbm_surface, tsq_err);
+               TPL_ERR("Failed to enqueue tbm_surface(%p). tpl_surface(%p) tsq_err=%d",
+                               tbm_surface, surface, tsq_err);
                TRACE_ASYNC_END((int)tbm_surface, "[DEQ]~[ENQ] BO_NAME:%d", bo_name);
                TPL_OBJECT_UNLOCK(wayland_egl_surface);
                return TPL_ERROR_INVALID_OPERATION;
@@ -655,7 +655,8 @@ __tpl_wl_egl_surface_cancel_dequeued_buffer(tpl_surface_t *surface,
        tsq_err = tbm_surface_queue_cancel_dequeue(wayland_egl_surface->tbm_queue,
                                                                                           tbm_surface);
        if (tsq_err != TBM_SURFACE_QUEUE_ERROR_NONE) {
-               TPL_ERR("Failed to release tbm_surface(%p)", tbm_surface);
+               TPL_ERR("Failed to release tbm_surface(%p) surface(%p)",
+                               tbm_surface, surface);
                return TPL_ERROR_INVALID_OPERATION;
        }
 
@@ -701,11 +702,12 @@ __tpl_wl_egl_surface_dequeue_buffer(tpl_surface_t *surface, uint64_t timeout_ns,
        lock_ret = twe_display_lock(wayland_egl_display->twe_display);
 
        if (tsq_err == TBM_SURFACE_QUEUE_ERROR_TIMEOUT) {
-               TPL_ERR("[CAN_DEQUEUE_TIMEOUT] queue(%p) will be reset",
-                               wayland_egl_surface->tbm_queue);
+               TPL_ERR("[CAN_DEQUEUE_TIMEOUT] queue(%p) will be reset. surface(%p)",
+                               wayland_egl_surface->tbm_queue, surface);
                if (twe_surface_queue_force_flush(wayland_egl_surface->twe_surface)
                        != TPL_ERROR_NONE) {
-                       TPL_ERR("Failed to timeout reset. tbm_queue(%p)", wayland_egl_surface->tbm_queue);
+                       TPL_ERR("Failed to timeout reset. tbm_queue(%p) surface(%p)",
+                                       wayland_egl_surface->tbm_queue, surface);
                        if (lock_ret == TPL_ERROR_NONE)
                                twe_display_unlock(wayland_egl_display->twe_display);
                        return NULL;
@@ -715,7 +717,8 @@ __tpl_wl_egl_surface_dequeue_buffer(tpl_surface_t *surface, uint64_t timeout_ns,
        }
 
        if (tsq_err != TBM_SURFACE_QUEUE_ERROR_NONE) {
-               TPL_ERR("Failed to query can_dequeue. tbm_queue(%p)", wayland_egl_surface->tbm_queue);
+               TPL_ERR("Failed to query can_dequeue. tbm_queue(%p) surface(%p)",
+                               wayland_egl_surface->tbm_queue, surface);
                if (lock_ret == TPL_ERROR_NONE)
                        twe_display_unlock(wayland_egl_display->twe_display);
                return NULL;
@@ -766,8 +769,8 @@ __tpl_wl_egl_surface_dequeue_buffer(tpl_surface_t *surface, uint64_t timeout_ns,
        tsq_err = tbm_surface_queue_dequeue(wayland_egl_surface->tbm_queue,
                                                                                &tbm_surface);
        if (!tbm_surface) {
-               TPL_ERR("Failed to get tbm_surface from tbm_surface_queue | tsq_err = %d",
-                               tsq_err);
+               TPL_ERR("Failed to dequeue from tbm_queue(%p) surface(%p)| tsq_err = %d",
+                               wayland_egl_surface->tbm_queue, surface, tsq_err);
                if (lock_ret == TPL_ERROR_NONE)
                        twe_display_unlock(wayland_egl_display->twe_display);
                return NULL;