e_hwc_windows: enhance the logs 38/193338/1
authorSooChan Lim <sc1.lim@samsung.com>
Mon, 19 Nov 2018 10:31:29 +0000 (19:31 +0900)
committerSooChan Lim <sc1.lim@samsung.com>
Mon, 19 Nov 2018 10:31:29 +0000 (19:31 +0900)
Change-Id: I79d5799f6446cb5fceb66a04527f890e1f475bc5

src/bin/e_hwc_window_queue.c
src/bin/e_hwc_windows.c

index ec9718506f81546ff0d23de96b3b1389169e1fa9..6e6cad8d2bf333f61bd2483f4a7071245a226d0b 100644 (file)
@@ -956,9 +956,11 @@ e_hwc_window_queue_user_set(E_Hwc_Window *hwc_window)
      {
         _e_hwc_window_queue_user_pending_set_add(queue, hwc_window);
 
-        EHWQINF("Add user_pending_set ehw:%p -- {%s}",
+        EHWQINF("Add user_pending_set ehw:%p {%s} user:{%s} unset_waiting:%d",
                 hwc_window->ec, queue, hwc_window,
-                (hwc_window->ec ? hwc_window->ec->icccm.title : "UNKNOWN"));
+                (hwc_window->ec ? hwc_window->ec->icccm.title : "UNKNOWN"),
+                (queue->user&&queue->user->ec?queue->user->ec->icccm.title : "UNKNOWN"),
+                (queue->state == E_HWC_WINDOW_QUEUE_STATE_UNSET_WAITING?1:0));
 
         return queue;
      }
index 8dedb79a3520c8fd4c674221814571a3d106a0e1..b6acf1df18ffa0b7a6927f84432c38007a4afbdd 100644 (file)
@@ -161,6 +161,8 @@ _e_hwc_windows_commit_handler(tdm_hwc *thwc, unsigned int sequence,
    E_Hwc *hwc = (E_Hwc *)user_data;
    EINA_SAFETY_ON_NULL_RETURN(hwc);
 
+   EHWSTRACE("!!!!!!!! HWC Commit Handler !!!!!!!!", NULL);
+
    if (hwc->pp_tsurface && !hwc->output->zoom_set)
      {
         tbm_surface_internal_unref(hwc->pp_tsurface);
@@ -169,8 +171,6 @@ _e_hwc_windows_commit_handler(tdm_hwc *thwc, unsigned int sequence,
 
    _e_hwc_windows_commit_data_release(hwc, sequence, tv_sec, tv_usec);
 
-   EHWSTRACE("!!!!!!!! Output Commit Handler !!!!!!!!", NULL);
-
    /* 'wait_commit' is mechanism to make 'fetch and commit' no more than one time per a frame;
     * a 'page flip' happened so it's time to allow to make 'fetch and commit' for the e_output */
    hwc->wait_commit = EINA_FALSE;
@@ -186,6 +186,9 @@ _e_hwc_windows_offscreen_commit(E_Hwc *hwc)
      {
         if (!e_hwc_window_commit_data_acquire(hwc_window)) continue;
 
+        EHWSTRACE("!!!!!!!! HWC OffScreen Commit !!!!!!!!", NULL);
+
+
         /* send frame event enlightenment doesn't send frame event in nocomp */
         if (hwc_window->ec)
           e_pixmap_image_clear(hwc_window->ec->pixmap, 1);
@@ -872,7 +875,13 @@ _e_hwc_windows_transition_check(E_Hwc *hwc)
                }
           }
      }
-
+#if 0
+    if (transition)
+      EHWSTRACE(" [%25s(ehw:%p)] is on TRANSITION [%s -> %s].",
+                hwc_window->ec, e_hwc_window_name_get(hwc_window), hwc_window,
+                e_hwc_window_state_string_get(hwc_window->accepted_state),
+                e_hwc_window_state_string_get(hwc_window->state));
+#endif
     return transition;
 }
 
@@ -905,7 +914,7 @@ _e_hwc_windows_validated_changes_update(E_Hwc *hwc, uint32_t num_changes)
         goto fail;
      }
 
-   EHWSTRACE("Changes NUM : %d", NULL, num_changes);
+   EHWSTRACE(" Changes NUM : %d", NULL, num_changes);
 
    for (i = 0; i < num_changes; ++i)
      {
@@ -946,14 +955,11 @@ _e_hwc_windows_accept(E_Hwc *hwc)
    tdm_error terror;
    const Eina_List *l;
 
-   /* _e_hwc_windows_accept */
-   EHWSTRACE("HWC Accept", NULL);
-
    /* accept changes */
-   terror = tdm_hwc_accept_validation(hwc->thwc);
+   terror = tdm_hwc_accept_changes(hwc->thwc);
    if (terror != TDM_ERROR_NONE)
      {
-        ERR("HWC-WINS: failed to accept changes.");
+        ERR("HWC-WINS: failed to accept validation.");
         return EINA_FALSE;
      }
 
@@ -973,6 +979,9 @@ _e_hwc_windows_accept(E_Hwc *hwc)
           e_hwc_window_activate(hwc_window, NULL);
      }
 
+   /* _e_hwc_windows_accept */
+   EHWSTRACE("======= HWC Accept Validation =======", NULL);
+
    return EINA_TRUE;
 }
 
@@ -989,7 +998,7 @@ _e_hwc_windows_validate(E_Hwc *hwc, uint32_t *num_changes)
    Eina_List *visible_windows = hwc->visible_windows;
 
 #if DBG_EVALUATE
-   EHWSTRACE(" Request HWC Validation to TDM HWC:", NULL);
+   EHWSTRACE("======= HWC Request Validation to TDM HWC =====", NULL);
    _e_hwc_windows_status_print(hwc, EINA_FALSE);
 #endif
 
@@ -1407,12 +1416,10 @@ _e_hwc_windows_evaluate(E_Hwc *hwc)
         goto re_evaluate;
      }
 
-   EHWSTRACE(" Succeed the compsition_evaulation.", NULL);
-
    return EINA_TRUE;
 
 re_evaluate:
-   EHWSTRACE(" Need the comopsition re-evaulation.", NULL);
+   EHWSTRACE("======= HWC NOT Accept Validation Yet !! =======", NULL);
 
    return EINA_FALSE;
 }