e_hwc: reduce log length
authorChangyeon Lee <cyeon.lee@samsung.com>
Mon, 26 Jun 2023 06:42:18 +0000 (15:42 +0900)
committerJunseok Kim <juns.kim@samsung.com>
Thu, 29 Jun 2023 02:29:04 +0000 (11:29 +0900)
Change-Id: Ie7cffb0e5f5cdaefffbd63131e2f061ddac97f7d

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

index ac4f378..541e780 100644 (file)
@@ -165,7 +165,7 @@ _get_composition_type(E_Hwc_Window_State state)
         break;
       default:
         composition_type = TDM_HWC_WIN_COMPOSITION_NONE;
-        EHWERR("unknown state of hwc_window.", NULL, NULL, NULL);
+        EHWERR("unknown state of hwc_window", NULL, NULL, NULL);
      }
 
    return composition_type;
@@ -295,7 +295,7 @@ _e_hwc_window_buffer_queue_unset(E_Hwc_Window *hwc_window)
         hwc_window->queue = NULL;
      }
 
-    EHWTRACE("Unset constranints BUFFER_QUEUE -- {%s}",
+    EHWTRACE("Unset constraints BUFFER_QUEUE - {%s}",
               hwc_window->ec, hwc_window->hwc, hwc_window, e_client_util_name_get(hwc_window->ec));
 }
 
@@ -306,7 +306,7 @@ e_hwc_window_constraints_reset(E_Hwc_Window *hwc_window)
 
    hwc_window->constraints = TDM_HWC_WIN_CONSTRAINT_NONE;
 
-   EHWTRACE("Reset constranints -- {%s}",
+   EHWTRACE("Reset constraints - {%s}",
             hwc_window->ec, hwc_window->hwc, hwc_window, e_client_util_name_get(hwc_window->ec));
 }
 
@@ -860,7 +860,7 @@ _e_hwc_window_client_cb_zone_set(void *data, int type, void *event)
 
    if (is_video) e_hwc_window_video_set(hwc_window, is_video);
 
-   EHWINF("set on eout:%p, zone_id:%d.",
+   EHWINF("set on eout:%p, zone_id:%d",
           ec, hwc_window->hwc, hwc_window, output, zone->id);
 
 end:
@@ -994,7 +994,7 @@ e_hwc_window_composition_update(E_Hwc_Window *hwc_window)
 
    if (e_hwc_window_is_target(hwc_window))
      {
-        EHWERR("target window cannot update at e_hwc_window_composition_update.", hwc_window->ec, hwc_window->hwc, hwc_window);
+        EHWERR("target window cannot update at e_hwc_window_composition_update", hwc_window->ec, hwc_window->hwc, hwc_window);
         return EINA_FALSE;
      }
 
@@ -1291,7 +1291,7 @@ _e_hwc_window_video_info_get(E_Hwc_Window *hwc_window, tdm_hwc_window_info *hwc_
 
    if (!e_client_video_info_get(ec, &vinfo))
      {
-        EHWERR("Video window does not get the video info.", hwc_window->ec, hwc_window->hwc, hwc_window);
+        EHWERR("Video window does not get the video info", hwc_window->ec, hwc_window->hwc, hwc_window);
         return EINA_FALSE;
      }
 
@@ -1470,7 +1470,7 @@ _e_hwc_window_buffer_set(E_Hwc_Window *hwc_window, tbm_surface_h tsurface,
           }
      }
 
-   EHWTRACE("FET ts:%10p ------- {%25s}, state:%s, zpos:%d, deleted:%s cursor:%d video:%d",
+   EHWTRACE("FET ts:%10p - {%s} state:%s zpos:%d deleted:%s cursor:%d video:%d",
             hwc_window->ec, hwc_window->hwc, hwc_window,
             tsurface, e_hwc_window_name_get(hwc_window),
             e_hwc_window_state_string_get(hwc_window->state),
@@ -1518,7 +1518,7 @@ e_hwc_window_buffer_fetch(E_Hwc_Window *hwc_window)
         if (!_e_hwc_window_cursor_image_update(hwc_window))
           return EINA_FALSE;
 
-        EHWTRACE("FET img_ptr:%p ------- {%25s}, state:%s, zpos:%d, deleted:%s",
+        EHWTRACE("FET img_ptr:%p - {%s} state:%s zpos:%d deleted:%s",
                  hwc_window->ec, hwc_window->hwc, hwc_window,
                  hwc_window->cursor.img_ptr, e_hwc_window_name_get(hwc_window),
                  e_hwc_window_state_string_get(hwc_window->state),
@@ -1649,7 +1649,7 @@ e_hwc_window_commit_data_acquire(E_Hwc_Window *hwc_window)
         e_hwc_window_buffer_set(&commit_data->buffer, NULL, NULL, 0);
      }
 
-   EHWTRACE("COM data:%p ts:%p ------- {%25s}, state:%s, zpos:%d",
+   EHWTRACE("COM data:%p ts:%p - {%s} state:%s zpos:%d",
             hwc_window->ec, hwc_window->hwc, hwc_window,
             commit_data, commit_data->buffer.tsurface,
             e_hwc_window_name_get(hwc_window),
@@ -1750,7 +1750,7 @@ e_hwc_window_commit_data_release(E_Hwc_Window *hwc_window, E_Hwc_Window_Commit_D
    queue = commit_data->buffer.queue;
    queue_buffer = commit_data->queue_buffer_ref.buffer;
 
-   EHWTRACE("DON data:%p ts:%p ------- {%25s}, state:%s, zpos:%d",
+   EHWTRACE("DON data:%p ts:%p - {%s} state:%s zpos:%d",
             hwc_window->ec, hwc_window->hwc, hwc_window,
             commit_data, tsurface, e_hwc_window_name_get(hwc_window),
             e_hwc_window_state_string_get(hwc_window->state),
@@ -1897,7 +1897,7 @@ e_hwc_window_deactivate(E_Hwc_Window *hwc_window)
         hwc_window->cqueue = NULL;
      }
 
-   EHWINF("Deactivate -- {%s}",
+   EHWINF("Deactivate - {%s}",
           hwc_window->ec, hwc_window->hwc, hwc_window,
           e_hwc_window_name_get(hwc_window));
 
@@ -1948,7 +1948,7 @@ e_hwc_window_accepted_state_set(E_Hwc_Window *hwc_window, E_Hwc_Window_State sta
         if (hwc_window->accepted_state == E_HWC_WINDOW_STATE_NONE)
           hwc_window->zpos = E_HWC_WINDOW_ZPOS_NONE;
 
-        EHWINF("Set Accepted state:%s -- {%s}",
+        EHWINF("Set Accepted state:%s - {%s}",
                 hwc_window->ec, hwc_window->hwc, hwc_window, e_hwc_window_state_string_get(state),
                 e_hwc_window_name_get(hwc_window));
 
@@ -2262,7 +2262,7 @@ e_hwc_window_transition_set(E_Hwc_Window *hwc_window, E_Hwc_Window_Transition tr
      e_hwc_window_constraints_reset(hwc_window);
 
    if (transition)
-     EHWINF(" [%25s] is on TRANSITION [%s -> %s].",
+     EHWINF(" [%s] is on TRANSITION [%s -> %s]",
             hwc_window->ec, hwc_window->hwc, hwc_window, e_hwc_window_name_get(hwc_window),
             e_hwc_window_state_string_get(hwc_window->accepted_state),
             e_hwc_window_state_string_get(hwc_window->state));
@@ -2357,7 +2357,7 @@ _e_hwc_window_client_recover(E_Hwc_Window *hwc_window, Eina_Bool *clear_attach)
    else
      recover_buffer = comp_wl_buffer;
 
-   EHWTRACE("Recover ts:%p -- {%s}",
+   EHWTRACE("Recover ts:%p - {%s}",
             hwc_window->ec, hwc_window->hwc, hwc_window, recover_buffer->tbm_surface,
             e_hwc_window_name_get(hwc_window));
 
@@ -2416,7 +2416,7 @@ _e_hwc_window_rendered_window_set(E_Hwc_Window *hwc_window, Eina_Bool set)
              if (!ret)
                e_comp_object_render_update_del(ec->frame);
 
-             EHWTRACE("Redirect -- {%s}",
+             EHWTRACE("Redirect - {%s}",
                       hwc_window->ec, hwc_window->hwc, hwc_window, e_hwc_window_name_get(hwc_window));
           }
 
@@ -2429,7 +2429,7 @@ _e_hwc_window_rendered_window_set(E_Hwc_Window *hwc_window, Eina_Bool set)
           {
              e_comp_object_redirected_set(ec->frame, EINA_FALSE);
 
-             EHWTRACE("Unredirect -- {%s}",
+             EHWTRACE("Unredirect - {%s}",
                       hwc_window->ec, hwc_window->hwc, hwc_window, e_hwc_window_name_get(hwc_window));
           }
      }
@@ -2683,7 +2683,7 @@ e_hwc_window_set_property(E_Hwc_Window *hwc_window, unsigned int id, const char
         ret = tdm_hwc_window_set_property(hwc_window->thwc_window, id, value);
         EINA_SAFETY_ON_TRUE_RETURN_VAL(ret != TDM_ERROR_NONE, ret);
 
-        EHWTRACE("Set Property: property(%s) value(%d)) -- {%s}",
+        EHWTRACE("Set Property: property(%s) value(%d) - {%s}",
                  hwc_window->ec, hwc_window->hwc, hwc_window,
                  name, (unsigned int)value.u32,
                  e_hwc_window_name_get(hwc_window));
@@ -2695,7 +2695,7 @@ e_hwc_window_set_property(E_Hwc_Window *hwc_window, unsigned int id, const char
           {
              if (!strncmp(name, prop->name, TDM_NAME_LEN))
                {
-                  EHWTRACE("Change Property: property(%s) update value(%d -> %d) -- {%s}",
+                  EHWTRACE("Change Property: property(%s) update value(%d -> %d) - {%s}",
                            hwc_window->ec, hwc_window->hwc, hwc_window,
                            prop->name, (unsigned int)prop->value.u32, (unsigned int)value.u32,
                            e_hwc_window_name_get(hwc_window));
@@ -2712,7 +2712,7 @@ e_hwc_window_set_property(E_Hwc_Window *hwc_window, unsigned int id, const char
         memcpy(prop->name, name, sizeof(TDM_NAME_LEN));
         hwc_window->prop_list = eina_list_append(hwc_window->prop_list, prop);
 
-        EHWTRACE("Set Property: property(%s) value(%d)) -- {%s}",
+        EHWTRACE("Set Property: property(%s) value(%d) - {%s}",
                  hwc_window->ec, hwc_window->hwc, hwc_window,
                  prop->name, (unsigned int)value.u32,
                  e_hwc_window_name_get(hwc_window));
index b7d83fc..27e0183 100644 (file)
@@ -147,7 +147,7 @@ _backup_tsurface_create(tbm_surface_h tsurface)
    ret = tbm_surface_map(tsurface, TBM_SURF_OPTION_READ, &src_info);
    if (ret != TBM_SURFACE_ERROR_NONE)
      {
-        EHWQERR("fail to map the tsurface.", NULL, NULL, NULL);
+        EHWQERR("fail to map the tsurface", NULL, NULL, NULL);
         return NULL;
      }
 
@@ -161,7 +161,7 @@ _backup_tsurface_create(tbm_surface_h tsurface)
         new_tsurface = tbm_surface_create(src_info.width, src_info.height, src_info.format);
         if (!new_tsurface)
           {
-             EHWQERR("fail to allocate the new_tsurface.", NULL, NULL, NULL);
+             EHWQERR("fail to allocate the new_tsurface", NULL, NULL, NULL);
              tbm_surface_unmap(tsurface);
              return NULL;
           }
@@ -172,7 +172,7 @@ _backup_tsurface_create(tbm_surface_h tsurface)
    ret = tbm_surface_map(new_tsurface, TBM_SURF_OPTION_WRITE, &dst_info);
    if (ret != TBM_SURFACE_ERROR_NONE)
      {
-        EHWQERR("fail to map the new_tsurface.", NULL, NULL, NULL);
+        EHWQERR("fail to map the new_tsurface", NULL, NULL, NULL);
         tbm_surface_destroy(new_tsurface);
         tbm_surface_unmap(tsurface);
         return NULL;
@@ -573,7 +573,7 @@ _e_hwc_window_queue_buffers_hand_over(E_Hwc_Window_Queue *queue, E_Hwc_Window *h
 
         queue->state = E_HWC_WINDOW_QUEUE_STATE_SET_WAITING_BUFFER;
 
-        EHWQINF("Set Waiting buffer user ehw:%p -- {%s}",
+        EHWQINF("Set Waiting buffer user ehw:%p - {%s}",
                 hwc_window->ec, queue->hwc, queue, hwc_window,
                 (hwc_window->ec ? hwc_window->ec->icccm.title : "UNKNOWN"));
      }
@@ -581,7 +581,7 @@ _e_hwc_window_queue_buffers_hand_over(E_Hwc_Window_Queue *queue, E_Hwc_Window *h
      {
         queue->state = E_HWC_WINDOW_QUEUE_STATE_SET_WAITING_DEQUEUEABLE;
 
-        EHWQINF("Set Waiting dequeueable user ehw:%p -- {%s}",
+        EHWQINF("Set Waiting dequeueable user ehw:%p - {%s}",
                 hwc_window->ec, queue->hwc, queue, hwc_window,
                 (hwc_window->ec ? hwc_window->ec->icccm.title : "UNKNOWN"));
      }
@@ -598,7 +598,7 @@ _e_hwc_window_queue_wait_usable_cb(struct wayland_tbm_client_queue *cqueue, void
 
    if (queue->state != E_HWC_WINDOW_QUEUE_STATE_SET_WAITING_WAIT_USABLE) return;
 
-   EHWQINF("Get wait usable event ehw:%p -- {%s}",
+   EHWQINF("Get wait usable event ehw:%p - {%s}",
            queue->user->ec, queue->hwc, queue, queue->user,
            (queue->user->ec ? queue->user->ec->icccm.title : "UNKNOWN"));
 
@@ -920,7 +920,7 @@ _e_hwc_window_queue_prepare_set(E_Hwc_Window_Queue *queue, E_Hwc_Window *hwc_win
    if (e_comp->hwc_send_redraw_request)
      e_comp_wl_renderer_client_redraw_request_send(hwc_window->ec);
 
-   EHWQINF("Set Waiting wait usable user ehw:%p -- {%s}",
+   EHWQINF("Set Waiting wait usable user ehw:%p - {%s}",
            hwc_window->ec, queue->hwc, queue, hwc_window,
            (hwc_window->ec ? hwc_window->ec->icccm.title : "UNKNOWN"));
 
@@ -946,7 +946,7 @@ _e_hwc_window_queue_set(E_Hwc_Window_Queue *queue)
    /* set the queue_state_set */
    queue->state = E_HWC_WINDOW_QUEUE_STATE_SET;
 
-   EHWQINF("Set user ehw:%p -- {%s}",
+   EHWQINF("Set user ehw:%p - {%s}",
            queue->user->ec, queue->hwc, queue, queue->user,
            (queue->user->ec ? queue->user->ec->icccm.title : "UNKNOWN"));
 }
@@ -1070,7 +1070,7 @@ _e_hwc_window_queue_unset(E_Hwc_Window_Queue *queue)
    /* release the tqueue */
    _e_hwc_window_queue_tqueue_release(queue->tqueue, queue->user);
 
-   EHWQINF("Unset user ehw:%p -- {%s}",
+   EHWQINF("Unset user ehw:%p - {%s}",
            queue->user->ec, queue->hwc, queue, queue->user,
            (queue->user->ec ? queue->user->ec->icccm.title : "UNKNOWN"));
 
@@ -1509,7 +1509,7 @@ e_hwc_window_queue_user_unset(E_Hwc_Window_Queue *queue, E_Hwc_Window *hwc_windo
         /* _e_hwc_window_queue_tqueue_release(tqueue, hwc_window) */
         _e_hwc_window_queue_tqueue_release(queue->tqueue, hwc_window);
 
-        EHWQINF("Remove user_pending_set ehw:%p -- {%s}",
+        EHWQINF("Remove user_pending_set ehw:%p - {%s}",
                 hwc_window->ec, queue->hwc, queue, hwc_window,
                 (hwc_window->ec ? hwc_window->ec->icccm.title : "UNKNOWN"));
 
@@ -1529,7 +1529,7 @@ e_hwc_window_queue_user_unset(E_Hwc_Window_Queue *queue, E_Hwc_Window *hwc_windo
         queue->user_waiting_unset = queue->user;
         queue->user = NULL;
 
-        EHWQINF("Unset Waiting user ehw:%p -- {%s}",
+        EHWQINF("Unset Waiting user ehw:%p - {%s}",
                 hwc_window->ec, queue->hwc, queue, hwc_window,
                 (hwc_window->ec ? hwc_window->ec->icccm.title : "UNKNOWN"));
      }
index bafce13..d857ae9 100644 (file)
@@ -772,21 +772,21 @@ _e_hwc_windows_root_target_buffer_fetch(E_Hwc *hwc)
              n_thw = eina_list_count(rendered_windows);
              if (n_thw)
                {
-                  EHWSTRACE("FET {%s} ts:%p state:%s has hwc_windows to render below.",
+                  EHWSTRACE("FET {%s} ts:%p state:%s has hwc_windows to render below",
                            NULL, hwc, e_hwc_window_name_get(hwc_window), hwc_window->current.buffer.tsurface,
                            e_hwc_window_state_string_get(hwc_window->state));
 
                   i = 0;
                   EINA_LIST_FOREACH(rendered_windows, l, hw)
                     {
-                       EHWSTRACE("  (%d) ehw:%p ts:%p -- {%25s}, state:%s, zpos:%d, deleted:%s",
+                       EHWSTRACE("  (%d) ehw:%p ts:%p - {%s} state:%s zpos:%d deleted:%s",
                                 hwc_window->ec, hwc, i++, hw, hw->current.buffer.tsurface, e_hwc_window_name_get(hw),
                                 e_hwc_window_state_string_get(hw->state), hwc_window->zpos,
                                 (hwc_window->is_deleted ? "yes" : "no"));
                     }
                 }
               else
-                EHWSTRACE("FET {%s} ts:%p state:%s has no hwc_windows to render.",
+                EHWSTRACE("FET {%s} ts:%p state:%s has no hwc_windows to render",
                          NULL, hwc, e_hwc_window_name_get(hwc_window), hwc_window->current.buffer.tsurface,
                          e_hwc_window_state_string_get(hwc_window->state));
           }
@@ -1719,7 +1719,7 @@ _e_hwc_windows_target_window_render_flush_post_cb(void *data, Evas *e EINA_UNUSE
    E_Client *ec;
    E_Comp_Wl_Client_Data *cdata;
 
-   EHWSTRACE("{%s} gets render_flush_post noti.", NULL, target_hwc_window->hwc, e_hwc_window_name_get((E_Hwc_Window *)target_hwc_window));
+   EHWSTRACE("{%s} gets render_flush_post noti", NULL, target_hwc_window->hwc, e_hwc_window_name_get((E_Hwc_Window *)target_hwc_window));
 
    if (!target_hwc_window->dequeued_tsurface)
      {
@@ -2035,7 +2035,7 @@ _e_hwc_windows_status_print(E_Hwc *hwc, Eina_Bool with_target)
 
    EINA_LIST_FOREACH(visible_windows, l, hwc_window)
      {
-        EHWSTRACE("  ehw:%p ts:%p -- {%25s}, state:%s, zpos:%d, deleted:%s restrict:%s",
+        EHWSTRACE("  ehw:%p ts:%p - {%s} state:%s zpos:%d deleted:%s restrict:%s",
                   hwc_window->ec, hwc, hwc_window,
                   hwc_window->current.buffer.tsurface, e_hwc_window_name_get(hwc_window),
                   e_hwc_window_state_string_get(hwc_window->state),
@@ -2085,7 +2085,7 @@ _e_hwc_windows_window_state_get(tdm_hwc_window_composition composition_type)
         break;
       default:
         state = E_HWC_WINDOW_STATE_NONE;
-        EHWSERR("unknown state of hwc_window.", NULL);
+        EHWSERR("unknown state of hwc_window", NULL);
      }
 
    return state;
@@ -2364,7 +2364,7 @@ _e_hwc_windows_accept(E_Hwc *hwc)
    terror = tdm_hwc_accept_validation(hwc->thwc);
    if (terror != TDM_ERROR_NONE)
      {
-        EHWSERR("failed to accept validation.", hwc);
+        EHWSERR("failed to accept validation", hwc);
         return EINA_FALSE;
      }
 
@@ -2478,7 +2478,7 @@ _e_hwc_windows_visible_windows_states_update(E_Hwc *hwc)
                   else
                     {
                        e_hwc_window_state_set(hwc_window, E_HWC_WINDOW_STATE_DEVICE, EINA_TRUE);
-                       EHWSTRACE("   ehw:%p -- {%25s} is Force hwc_acceptable(Redirected FALSE).",
+                       EHWSTRACE("   ehw:%p - {%s} is Force hwc_acceptable(Redirected FALSE)",
                                hwc_window->ec, hwc, hwc_window, e_hwc_window_name_get(hwc_window));
                     }
                }
@@ -2501,14 +2501,14 @@ _e_hwc_windows_visible_windows_states_update(E_Hwc *hwc)
              if (ec->redirected)
                {
                   e_hwc_window_state_set(hwc_window, E_HWC_WINDOW_STATE_CLIENT, EINA_TRUE);
-                  EHWSTRACE("   ehw:%p -- {%25s} is Force hwc_unacceptable.%s",
+                  EHWSTRACE("   ehw:%p - {%s} is Force hwc_unacceptable%s",
                           hwc_window->ec, hwc, hwc_window, e_hwc_window_name_get(hwc_window),
                           hwc->pp_set ? "(PP applied)" : "");
                }
              else
                {
                   e_hwc_window_state_set(hwc_window, E_HWC_WINDOW_STATE_DEVICE, EINA_TRUE);
-                  EHWSTRACE("   ehw:%p -- {%25s} is Force hwc_acceptable(Redirected FALSE).",
+                  EHWSTRACE("   ehw:%p - {%s} is Force hwc_acceptable(Redirected FALSE)",
                           hwc_window->ec, hwc, hwc_window, e_hwc_window_name_get(hwc_window));
                }
           }
@@ -2848,14 +2848,14 @@ _e_hwc_windows_pixman_copy(E_Hwc *hwc, tbm_surface_h src_tsurface, tbm_surface_h
    ret = tbm_surface_map(src_tsurface, TBM_SURF_OPTION_READ | TBM_SURF_OPTION_WRITE, &src_tinfo);
    if (ret != TBM_SURFACE_ERROR_NONE)
      {
-        EHWSERR("tbm_surface_map fails on src_tsurface.", hwc);
+        EHWSERR("tbm_surface_map fails on src_tsurface", hwc);
         goto end;
      }
 
    ret = tbm_surface_map(dst_tsurface, TBM_SURF_OPTION_READ | TBM_SURF_OPTION_WRITE, &dst_tinfo);
    if (ret != TBM_SURFACE_ERROR_NONE)
      {
-        EHWSERR("tbm_surface_map fails on dst_tsurface.", hwc);
+        EHWSERR("tbm_surface_map fails on dst_tsurface", hwc);
         goto src_unmap;
      }
 
@@ -2908,7 +2908,7 @@ _e_hwc_windows_buffer_clear(E_Hwc *hwc, tbm_surface_h tsurface)
    ret = tbm_surface_map(tsurface, TBM_SURF_OPTION_READ | TBM_SURF_OPTION_WRITE, &tinfo);
    if (ret != TBM_SURFACE_ERROR_NONE)
      {
-        EHWSERR("tbm_surface_map fail on tsurface.", hwc);
+        EHWSERR("tbm_surface_map fail on tsurface", hwc);
         return;
      }
 
@@ -2975,7 +2975,7 @@ _e_hwc_windows_evaluate(E_Hwc *hwc, E_Output_Display_Mode display_mode)
    /* validate the visible hwc_windows' states*/
    if (!_e_hwc_windows_validate(hwc, &num_changes))
      {
-        EHWSERR("_e_hwc_windows_validate failed.", hwc);
+        EHWSERR("_e_hwc_windows_validate failed", hwc);
         goto re_evaluate;
      }
 
@@ -2984,7 +2984,7 @@ _e_hwc_windows_evaluate(E_Hwc *hwc, E_Output_Display_Mode display_mode)
      {
         if (!_e_hwc_windows_validated_changes_update(hwc, num_changes))
           {
-             EHWSERR("_e_hwc_windows_validated_changes_update failed.", hwc);
+             EHWSERR("_e_hwc_windows_validated_changes_update failed", hwc);
              goto re_evaluate;
           }
      }
@@ -3009,17 +3009,17 @@ _e_hwc_windows_evaluate(E_Hwc *hwc, E_Output_Display_Mode display_mode)
    /* set the state of the target_window */
    if (hwc_mode == E_HWC_MODE_NONE)
      {
-        EHWSTRACE(" HWC_MODE is NONE composition.", NULL, hwc);
+        EHWSTRACE(" HWC_MODE is NONE composition", NULL, hwc);
         _e_hwc_windows_target_state_set(hwc->target_hwc_window, E_HWC_WINDOW_STATE_DEVICE);
      }
    else if (hwc_mode == E_HWC_MODE_HYBRID)
      {
-        EHWSTRACE(" HWC_MODE is HYBRID composition.", NULL, hwc);
+        EHWSTRACE(" HWC_MODE is HYBRID composition", NULL, hwc);
         _e_hwc_windows_target_state_set(hwc->target_hwc_window, E_HWC_WINDOW_STATE_DEVICE);
      }
    else
      {
-        EHWSTRACE(" HWC_MODE is FULL HW composition.", NULL, hwc);
+        EHWSTRACE(" HWC_MODE is FULL HW composition", NULL, hwc);
         _e_hwc_windows_target_state_set(hwc->target_hwc_window, E_HWC_WINDOW_STATE_NONE);
      }
 
@@ -3041,7 +3041,7 @@ _e_hwc_windows_evaluate(E_Hwc *hwc, E_Output_Display_Mode display_mode)
 
    if (_e_hwc_windows_present_sync_check(hwc))
      {
-        EHWSTRACE(" Not accpet display sync.", NULL, hwc);
+        EHWSTRACE(" Not accpet display sync", NULL, hwc);
         goto re_evaluate;
      }
 
@@ -3054,7 +3054,7 @@ _e_hwc_windows_evaluate(E_Hwc *hwc, E_Output_Display_Mode display_mode)
    /* accept the result of the validation */
    if (!_e_hwc_windows_accept(hwc))
      {
-        EHWSERR("_e_hwc_windows_validated_changes_update failed.", hwc);
+        EHWSERR("_e_hwc_windows_validated_changes_update failed", hwc);
         goto re_evaluate;
      }
 
@@ -3206,7 +3206,7 @@ e_hwc_windows_render(E_Hwc *hwc)
 
    if (e_hwc_norender_get(hwc) > 0)
      {
-        EHWSTRACE("NoRender get. Do not ecore_evas_manual_render.", NULL, hwc);
+        EHWSTRACE("NoRender get. Do not ecore_evas_manual_render", NULL, hwc);
         return EINA_TRUE;
      }
 
@@ -3476,7 +3476,7 @@ e_hwc_windows_commit(E_Hwc *hwc, E_Output_Display_Mode display_mode)
 
    if (e_hwc_norender_get(hwc) > 0)
      {
-        EHWSTRACE("No Render get.", NULL, hwc);
+        EHWSTRACE("No Render get", NULL, hwc);
         return EINA_TRUE;
      }
 
@@ -3520,7 +3520,7 @@ e_hwc_windows_commit(E_Hwc *hwc, E_Output_Display_Mode display_mode)
    error = tdm_hwc_commit(hwc->thwc, 0, _e_hwc_windows_commit_handler, wins_commit_data);
    if (error != TDM_ERROR_NONE)
      {
-        EHWSERR("tdm_hwc_commit failed.", hwc);
+        EHWSERR("tdm_hwc_commit failed", hwc);
         _e_hwc_windows_commit_handler(hwc->thwc, 0, 0, 0, wins_commit_data);
         goto fail;
      }
@@ -3539,7 +3539,7 @@ e_hwc_windows_commit(E_Hwc *hwc, E_Output_Display_Mode display_mode)
              _e_hwc_windows_release_fence_fd_update(hwc);
           }
         else
-          EHWSERR("tdm_hwc_get_commit_sync_fence_fd failed.", hwc);
+          EHWSERR("tdm_hwc_get_commit_sync_fence_fd failed", hwc);
      }
 
    /* send tizen_hwc_commit feedback committed */
@@ -3927,7 +3927,7 @@ e_hwc_windows_rendered_window_add(E_Hwc_Window *hwc_window)
 
    e_hwc_window_ref(hwc_window);
 
-   EHWSTRACE(" add ehw:%p ts:%p to the render_list -- {%25s}.", ec, hwc, hwc_window,
+   EHWSTRACE(" add ehw:%p ts:%p to the render_list - {%s}", ec, hwc, hwc_window,
             hwc_window->current.buffer.tsurface, e_hwc_window_name_get(hwc_window));
 }
 
@@ -4318,7 +4318,7 @@ e_hwc_windows_mirror_set(E_Hwc *hwc, E_Hwc *src_hwc)
 
    if (!e_hwc_windows_pp_set(hwc, &src_rect, &dst_rect))
      {
-        EHWSERR("e_hwc_windows_pp_set failed.", hwc);
+        EHWSERR("e_hwc_windows_pp_set failed", hwc);
         return EINA_FALSE;
      }