virtual: add voutput disconnect and destroy test to tdm_test
[platform/core/uifw/libtdm.git] / src / tdm_output.c
index 6b83f36..902543e 100644 (file)
@@ -9,7 +9,7 @@
  *          Taeheon Kim <th908.kim@samsung.com>,
  *          YoungJun Cho <yj44.cho@samsung.com>,
  *          SooChan Lim <sc1.lim@samsung.com>,
- *          Boram Park <sc1.lim@samsung.com>
+ *          Boram Park <boram1288.park@samsung.com>
  *
  * Permission is hereby granted, free of charge, to any person obtaining a
  * copy of this software and associated documentation files (the
 #include "config.h"
 #endif
 
-#include "tdm.h"
-#include "tdm_backend.h"
 #include "tdm_private.h"
-#include "tdm_helper.h"
 
 #define COUNT_MAX   10
 
        private_output = (tdm_private_output*)output; \
        private_display = private_output->private_display
 
+#define OUTPUT_FUNC_ENTRY_VOID() \
+       tdm_private_display *private_display; \
+       tdm_private_output *private_output; \
+       TDM_RETURN_IF_FAIL(tdm_output_is_valid(output)); \
+       private_output = (tdm_private_output*)output; \
+       private_display = private_output->private_display
+
+static void
+_tdm_output_vblank_timeout_update(tdm_private_output *private_output, int ms_delay);
+
+static tdm_error
+_tdm_output_vblank_timeout_cb(void *user_data)
+{
+       tdm_private_output *private_output = user_data;
+       tdm_private_output_vblank_handler *v = NULL;
+
+       TDM_RETURN_VAL_IF_FAIL(private_output != NULL, TDM_ERROR_OPERATION_FAILED);
+
+       private_output->vblank_timeout_timer_expired++;
+
+       TDM_ERR("TDM output(%d) vblank TIMEOUT!! (%d time%s)",
+                       private_output->pipe,
+                       private_output->vblank_timeout_timer_expired,
+                       (private_output->vblank_timeout_timer_expired > 1) ? "s" : "");
+
+       LIST_FOR_EACH_ENTRY(v, &private_output->vblank_handler_list, link) {
+               TDM_ERR("vblank_handler(%p) interval(%d) sync(%d) sent_to_frontend(%u) owner_tid(%d)",
+                               v, v->interval, v->sync, v->sent_to_frontend, v->owner_tid);
+       }
+
+       return TDM_ERROR_NONE;
+}
+
+INTERN void
+tdm_output_vblank_print_wait_information(tdm_private_output *private_output, void *user_data)
+{
+       tdm_private_output_vblank_handler *v = NULL;
+
+       TDM_RETURN_IF_FAIL(private_output != NULL);
+       TDM_RETURN_IF_FAIL(user_data != NULL);
+
+       TDM_ERR("TDM output(%d) vblank user_data(%p) info!!", private_output->pipe, user_data);
+
+       LIST_FOR_EACH_ENTRY(v, &private_output->vblank_handler_list, link) {
+               if (v->user_data != user_data)
+                       continue;
+               TDM_ERR("vblank_handler(%p) interval(%d) sync(%d) sent_to_frontend(%u) owner_tid(%d)",
+                               v, v->interval, v->sync, v->sent_to_frontend, v->owner_tid);
+       }
+}
+
+static void
+_tdm_output_vblank_timeout_update(tdm_private_output *private_output, int ms_delay)
+{
+       tdm_error ret;
+
+       if (!private_output->vblank_timeout_timer) {
+               private_output->vblank_timeout_timer =
+                       tdm_event_loop_add_timer_handler(private_output->private_display,
+                                                                                        _tdm_output_vblank_timeout_cb,
+                                                                                        private_output,
+                                                                                        &ret);
+               if (!private_output->vblank_timeout_timer) {
+                       TDM_ERR("output(%d) couldn't add timer", private_output->pipe);
+                       return;
+               }
+               TDM_INFO("output(%d) create vblank timeout timer", private_output->pipe);
+               private_output->vblank_timeout_timer_expired = 0;
+       }
+
+       ret = tdm_event_loop_source_timer_update(private_output->vblank_timeout_timer, ms_delay);
+       if (ret != TDM_ERROR_NONE) {
+               TDM_ERR("output(%d) couldn't update timer", private_output->pipe);
+               return;
+       }
+}
+
+INTERN tdm_error
+tdm_output_init(tdm_private_display *private_display)
+{
+       tdm_thread_cb_set_find_func(TDM_THREAD_CB_OUTPUT_DESTROY, tdm_display_find_output_stamp);
+       tdm_thread_cb_set_find_func(TDM_THREAD_CB_OUTPUT_COMMIT, tdm_display_find_output_stamp);
+       tdm_thread_cb_set_find_func(TDM_THREAD_CB_OUTPUT_VBLANK, tdm_display_find_output_stamp);
+       tdm_thread_cb_set_find_func(TDM_THREAD_CB_OUTPUT_STATUS, tdm_display_find_output_stamp);
+       tdm_thread_cb_set_find_func(TDM_THREAD_CB_OUTPUT_DPMS, tdm_display_find_output_stamp);
+
+       return TDM_ERROR_NONE;
+}
+
+INTERN tdm_error
+tdm_output_call_thread_cb_destroy(tdm_private_output *private_output)
+{
+       tdm_thread_cb_output_destroy output_destroy;
+       tdm_error ret;
+
+       memset(&output_destroy, 0, sizeof output_destroy);
+       output_destroy.base.type = TDM_THREAD_CB_OUTPUT_DESTROY;
+       output_destroy.base.length = sizeof output_destroy;
+       output_destroy.base.object_stamp = private_output->stamp;
+       output_destroy.base.data = NULL;
+       output_destroy.base.sync = 1;
+
+       ret = tdm_thread_cb_call(private_output, &output_destroy.base, 1);
+       TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
+
+       return TDM_ERROR_NONE;
+}
+
+INTERN void
+tdm_output_thread_cb_destroy(tdm_private_display *private_display, void *object, tdm_thread_cb_base *cb_base, void *user_data)
+{
+       tdm_private_output *private_output = object;
+       tdm_private_output_destroy_handler *destroy_handler = user_data;
+
+       TDM_RETURN_IF_FAIL(TDM_MUTEX_IS_LOCKED());
+
+       assert(destroy_handler->owner_tid == syscall(SYS_gettid));
+
+       _pthread_mutex_unlock(&private_display->lock);
+       destroy_handler->func(private_output, destroy_handler->user_data);
+       _pthread_mutex_lock(&private_display->lock);
+}
+
+EXTERN tdm_error
+tdm_output_add_destroy_handler(tdm_output *output,
+                                                          tdm_output_destroy_handler func,
+                                                          void *user_data)
+{
+       tdm_private_output_destroy_handler *destroy_handler = NULL;
+       OUTPUT_FUNC_ENTRY();
+
+       TDM_RETURN_VAL_IF_FAIL(func != NULL, TDM_ERROR_INVALID_PARAMETER);
+
+       _pthread_mutex_lock(&private_display->lock);
+
+       LIST_FOR_EACH_ENTRY(destroy_handler, &private_output->destroy_handler_list, link) {
+               if (destroy_handler->func == func && destroy_handler->user_data == user_data) {
+                       TDM_ERR("can't add twice");
+                       _pthread_mutex_unlock(&private_display->lock);
+                       return TDM_ERROR_BAD_REQUEST;
+               }
+       }
+
+       destroy_handler = calloc(1, sizeof(tdm_private_output_destroy_handler));
+       if (!destroy_handler) {
+               TDM_ERR("failed: alloc memory");
+               _pthread_mutex_unlock(&private_display->lock);
+               return TDM_ERROR_OUT_OF_MEMORY;
+       }
+
+       ret = tdm_thread_cb_add(private_output, TDM_THREAD_CB_OUTPUT_DESTROY, NULL, tdm_output_thread_cb_destroy, destroy_handler);
+       if (ret != TDM_ERROR_NONE) {
+               TDM_ERR("tdm_thread_cb_add failed");
+               free(destroy_handler);
+               _pthread_mutex_unlock(&private_display->lock);
+               return TDM_ERROR_OPERATION_FAILED;
+       }
+
+       destroy_handler->private_output = private_output;
+       destroy_handler->func = func;
+       destroy_handler->user_data = user_data;
+       destroy_handler->owner_tid = syscall(SYS_gettid);
+
+       LIST_ADDTAIL(&destroy_handler->link, &private_output->destroy_handler_list);
+
+       _pthread_mutex_unlock(&private_display->lock);
+
+       return ret;
+}
+
+
+EXTERN void
+tdm_output_remove_destroy_handler(tdm_output *output,
+                                                                 tdm_output_destroy_handler func,
+                                                                 void *user_data)
+{
+       tdm_private_display *private_display;
+       tdm_private_output *private_output;
+       tdm_private_output_destroy_handler *destroy_handler = NULL, *hh = NULL;
+
+       TDM_RETURN_IF_FAIL(output != NULL);
+       TDM_RETURN_IF_FAIL(func != NULL);
+
+       private_output = (tdm_private_output*)output;
+       private_display = private_output->private_display;
+
+       _pthread_mutex_lock(&private_display->lock);
+
+       LIST_FOR_EACH_ENTRY_SAFE(destroy_handler, hh, &private_output->destroy_handler_list, link) {
+               if (destroy_handler->func != func || destroy_handler->user_data != user_data)
+                       continue;
+
+               tdm_thread_cb_remove(private_output, TDM_THREAD_CB_OUTPUT_DESTROY, NULL, tdm_output_thread_cb_destroy, destroy_handler);
+
+               LIST_DEL(&destroy_handler->link);
+               free(destroy_handler);
+
+               _pthread_mutex_unlock(&private_display->lock);
+
+               return;
+       }
+
+       _pthread_mutex_unlock(&private_display->lock);
+}
+
+EXTERN tdm_module *
+tdm_output_get_backend_module(tdm_output *output, tdm_error *error)
+{
+       tdm_private_module *private_module;
+
+       OUTPUT_FUNC_ENTRY_ERROR();
+
+       _pthread_mutex_lock(&private_display->lock);
+
+       private_module = private_output->private_module;
+
+       if (error)
+               *error = TDM_ERROR_NONE;
+
+       _pthread_mutex_unlock(&private_display->lock);
+
+       return private_module;
+}
+
 EXTERN tdm_error
 tdm_output_get_model_info(tdm_output *output, const char **maker,
                                                  const char **model, const char **name)
@@ -112,66 +333,117 @@ tdm_output_get_conn_status(tdm_output *output, tdm_output_conn_status *status)
        return ret;
 }
 
-static void
-_tdm_output_update(tdm_output *output_backend, void *user_data)
+INTERN void
+tdm_output_thread_cb_change(tdm_private_display *private_display, void *object, tdm_thread_cb_base *cb_base, void *user_data)
 {
-       tdm_private_display *private_display;
-       tdm_private_output *private_output = user_data;
+       tdm_private_output *private_output = object;
+       tdm_private_output_change_handler *change_handler = user_data;
+       tdm_output_change_type type = TDM_OUTPUT_CHANGE_CONNECTION;
+       tdm_value value = {.u32 = 0 };
+
+       TDM_RETURN_IF_FAIL(TDM_MUTEX_IS_LOCKED());
+
+       assert(change_handler->owner_tid == syscall(SYS_gettid));
+
+       if (cb_base->type == TDM_THREAD_CB_OUTPUT_STATUS) {
+               tdm_thread_cb_output_status *output_status = (tdm_thread_cb_output_status *)cb_base;
+               type = TDM_OUTPUT_CHANGE_CONNECTION;
+               value.u32 = output_status->status;
+       } else if (cb_base->type == TDM_THREAD_CB_OUTPUT_DPMS) {
+               tdm_thread_cb_output_dpms *output_dpms = (tdm_thread_cb_output_dpms *)cb_base;
+               type = TDM_OUTPUT_CHANGE_DPMS;
+               value.u32 = output_dpms->dpms;
+       } else {
+               TDM_NEVER_GET_HERE();
+               return;
+       }
+
+       _pthread_mutex_unlock(&private_display->lock);
+       change_handler->func(private_output, type, value, change_handler->user_data);
+       _pthread_mutex_lock(&private_display->lock);
+
+}
+
+static tdm_error
+_tdm_output_call_thread_cb_status(tdm_private_output *private_output, tdm_output_conn_status status)
+{
+       tdm_thread_cb_output_status output_status;
+       pid_t caller_tid = syscall(SYS_gettid);
        tdm_error ret;
 
-       TDM_RETURN_IF_FAIL(private_output);
+       memset(&output_status, 0, sizeof output_status);
+       output_status.base.type = TDM_THREAD_CB_OUTPUT_STATUS;
+       output_status.base.length = sizeof output_status;
+       output_status.base.object_stamp = private_output->stamp;
+       output_status.base.data = NULL;
+       if (tdm_thread_in_display_thread(caller_tid))
+               output_status.base.sync = 0;
+       else
+               output_status.base.sync = 1;
+       output_status.status = status;
 
-       private_display = private_output->private_display;
+       ret = tdm_thread_cb_call(private_output, &output_status.base, 1);
+       TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
+
+       return TDM_ERROR_NONE;
+}
 
-       ret = tdm_display_update_output(private_display, output_backend, private_output->pipe);
-       TDM_RETURN_IF_FAIL(ret == TDM_ERROR_NONE);
+static tdm_error
+_tdm_output_call_thread_cb_dpms(tdm_private_output *private_output, tdm_output_dpms dpms)
+{
+       tdm_thread_cb_output_dpms output_dpms;
+       tdm_error ret;
+
+       memset(&output_dpms, 0, sizeof output_dpms);
+       output_dpms.base.type = TDM_THREAD_CB_OUTPUT_DPMS;
+       output_dpms.base.length = sizeof output_dpms;
+       output_dpms.base.object_stamp = private_output->stamp;
+       output_dpms.base.data = NULL;
+       output_dpms.base.sync = 0;
+       output_dpms.dpms = dpms;
+
+       ret = tdm_thread_cb_call(private_output, &output_dpms.base, 1);
+       TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
+
+       return TDM_ERROR_NONE;
 }
 
 INTERN void
-tdm_output_cb_status(tdm_output *output_backend, tdm_output_conn_status status,
-                                        void *user_data)
+tdm_output_cb_status(tdm_output *output_backend, tdm_output_conn_status status, void *user_data)
 {
-       tdm_private_display *private_display;
        tdm_private_output *private_output = user_data;
-       tdm_value value;
+       tdm_error ret;
 
-       TDM_RETURN_IF_FAIL(TDM_MUTEX_IS_LOCKED());
        TDM_RETURN_IF_FAIL(private_output);
 
-       private_display = private_output->private_display;
-
-       if (!tdm_thread_in_display_thread(syscall(SYS_gettid))) {
-               tdm_thread_cb_output_status output_status;
-               tdm_error ret;
+       TDM_INFO("output(%d) %s", private_output->pipe, tdm_status_str(status));
 
-               _tdm_output_update(output_backend, user_data);
-
-               output_status.base.type = TDM_THREAD_CB_OUTPUT_STATUS;
-               output_status.base.length = sizeof output_status;
-               output_status.output_stamp = private_output->stamp;
-               output_status.status = status;
-               output_status.user_data = user_data;
+       if ((private_output->caps.status == TDM_OUTPUT_CONN_STATUS_DISCONNECTED && status != TDM_OUTPUT_CONN_STATUS_DISCONNECTED) ||
+               (private_output->caps.status != TDM_OUTPUT_CONN_STATUS_DISCONNECTED && status == TDM_OUTPUT_CONN_STATUS_DISCONNECTED)) {
+               ret = tdm_display_update_output(private_output->private_module, output_backend);
+               TDM_RETURN_IF_FAIL(ret == TDM_ERROR_NONE);
+       } else {
+               private_output->caps.status = status;
+       }
 
-               value.u32 = status;
-               tdm_output_call_change_handler_internal(private_output,
-                                                                                               &private_output->change_handler_list_sub,
-                                                                                               TDM_OUTPUT_CHANGE_CONNECTION,
-                                                                                               value, 0);
+       ret = _tdm_output_call_thread_cb_status(private_output, status);
+       TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
+}
 
-               ret = tdm_thread_send_cb(private_display->private_loop, &output_status.base);
-               TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
+INTERN void
+tdm_output_cb_dpms(tdm_output *output_backend, tdm_output_dpms dpms, void *user_data)
+{
+       tdm_private_output *private_output = user_data;
+       tdm_error ret;
 
-               return;
-       }
+       TDM_INFO("output(%d) %s", private_output->pipe, tdm_status_str(dpms));
 
-       if (!tdm_thread_is_running())
-               _tdm_output_update(output_backend, user_data);
+       private_output->current_dpms_value = dpms;
+       private_output->waiting_dpms_change = 0;
+       TDM_INFO("output(%d) dpms async '%s' done", private_output->pipe, tdm_dpms_str(dpms));
 
-       value.u32 = status;
-       tdm_output_call_change_handler_internal(private_output,
-                                                                                       &private_output->change_handler_list_main,
-                                                                                       TDM_OUTPUT_CHANGE_CONNECTION,
-                                                                                       value, 0);
+       ret = _tdm_output_call_thread_cb_dpms(private_output, dpms);
+       TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
 }
 
 EXTERN tdm_error
@@ -179,18 +451,49 @@ tdm_output_add_change_handler(tdm_output *output,
                                                          tdm_output_change_handler func,
                                                          void *user_data)
 {
-       tdm_private_change_handler *change_handler;
+       tdm_private_output_change_handler *change_handler = NULL;
        OUTPUT_FUNC_ENTRY();
 
        TDM_RETURN_VAL_IF_FAIL(func != NULL, TDM_ERROR_INVALID_PARAMETER);
 
-       pthread_mutex_lock(&private_display->lock);
+       _pthread_mutex_lock(&private_display->lock);
+
+       LIST_FOR_EACH_ENTRY(change_handler, &private_output->change_handler_list, link) {
+               if (change_handler->func == func && change_handler->user_data == user_data) {
+                       TDM_ERR("can't add twice");
+                       _pthread_mutex_unlock(&private_display->lock);
+                       return TDM_ERROR_BAD_REQUEST;
+               }
+       }
 
-       change_handler = calloc(1, sizeof(tdm_private_change_handler));
+       change_handler = calloc(1, sizeof(tdm_private_output_change_handler));
        if (!change_handler) {
+               /* LCOV_EXCL_START */
                TDM_ERR("failed: alloc memory");
                _pthread_mutex_unlock(&private_display->lock);
                return TDM_ERROR_OUT_OF_MEMORY;
+               /* LCOV_EXCL_STOP */
+       }
+
+       ret = tdm_thread_cb_add(private_output, TDM_THREAD_CB_OUTPUT_DPMS, NULL, tdm_output_thread_cb_change, change_handler);
+       if (ret != TDM_ERROR_NONE) {
+               /* LCOV_EXCL_START */
+               TDM_ERR("tdm_thread_cb_add failed");
+               free(change_handler);
+               _pthread_mutex_unlock(&private_display->lock);
+               return TDM_ERROR_OPERATION_FAILED;
+               /* LCOV_EXCL_STOP */
+       }
+
+       ret = tdm_thread_cb_add(private_output, TDM_THREAD_CB_OUTPUT_STATUS, NULL, tdm_output_thread_cb_change, change_handler);
+       if (ret != TDM_ERROR_NONE) {
+               /* LCOV_EXCL_START */
+               TDM_ERR("tdm_thread_cb_add failed");
+               tdm_thread_cb_remove(private_output, TDM_THREAD_CB_OUTPUT_DPMS, NULL, tdm_output_thread_cb_change, change_handler);
+               free(change_handler);
+               _pthread_mutex_unlock(&private_display->lock);
+               return TDM_ERROR_OPERATION_FAILED;
+               /* LCOV_EXCL_STOP */
        }
 
        change_handler->private_output = private_output;
@@ -198,10 +501,7 @@ tdm_output_add_change_handler(tdm_output *output,
        change_handler->user_data = user_data;
        change_handler->owner_tid = syscall(SYS_gettid);
 
-       if (!tdm_thread_in_display_thread(change_handler->owner_tid))
-               LIST_ADDTAIL(&change_handler->link, &private_output->change_handler_list_sub);
-       else
-               LIST_ADDTAIL(&change_handler->link, &private_output->change_handler_list_main);
+       LIST_ADDTAIL(&change_handler->link, &private_output->change_handler_list);
 
        _pthread_mutex_unlock(&private_display->lock);
 
@@ -215,7 +515,7 @@ tdm_output_remove_change_handler(tdm_output *output,
 {
        tdm_private_display *private_display;
        tdm_private_output *private_output;
-       tdm_private_change_handler *h = NULL, *hh = NULL;
+       tdm_private_output_change_handler *change_handler = NULL, *hh = NULL;
 
        TDM_RETURN_IF_FAIL(output != NULL);
        TDM_RETURN_IF_FAIL(func != NULL);
@@ -225,24 +525,15 @@ tdm_output_remove_change_handler(tdm_output *output,
 
        _pthread_mutex_lock(&private_display->lock);
 
-       LIST_FOR_EACH_ENTRY_SAFE(h, hh, &private_output->change_handler_list_main, link) {
-               if (h->func != func || h->user_data != user_data)
+       LIST_FOR_EACH_ENTRY_SAFE(change_handler, hh, &private_output->change_handler_list, link) {
+               if (change_handler->func != func || change_handler->user_data != user_data)
                        continue;
 
-               LIST_DEL(&h->link);
-               free(h);
-
-               _pthread_mutex_unlock(&private_display->lock);
-
-               return;
-       }
-
-       LIST_FOR_EACH_ENTRY_SAFE(h, hh, &private_output->change_handler_list_sub, link) {
-               if (h->func != func || h->user_data != user_data)
-                       continue;
+               tdm_thread_cb_remove(private_output, TDM_THREAD_CB_OUTPUT_DPMS, NULL, tdm_output_thread_cb_change, change_handler);
+               tdm_thread_cb_remove(private_output, TDM_THREAD_CB_OUTPUT_STATUS, NULL, tdm_output_thread_cb_change, change_handler);
 
-               LIST_DEL(&h->link);
-               free(h);
+               LIST_DEL(&change_handler->link);
+               free(change_handler);
 
                _pthread_mutex_unlock(&private_display->lock);
 
@@ -279,6 +570,13 @@ tdm_output_get_layer_count(tdm_output *output, int *count)
 
        _pthread_mutex_lock(&private_display->lock);
 
+       if (private_output->caps.capabilities & TDM_OUTPUT_CAPABILITY_HWC) {
+               TDM_ERR("output(%p) support HWC. Use HWC functions", private_output);
+               *count = 0;
+               _pthread_mutex_unlock(&private_display->lock);
+               return TDM_ERROR_BAD_REQUEST;
+       }
+
        *count = 0;
        LIST_FOR_EACH_ENTRY(private_layer, &private_output->layer_list, link)
        (*count)++;
@@ -305,6 +603,14 @@ tdm_output_get_layer(tdm_output *output, int index, tdm_error *error)
        if (error)
                *error = TDM_ERROR_NONE;
 
+       if (private_output->caps.capabilities & TDM_OUTPUT_CAPABILITY_HWC) {
+               _pthread_mutex_unlock(&private_display->lock);
+               TDM_ERR("output(%p) support HWC. Use HWC functions", private_output);
+               if (error)
+                       *error = TDM_ERROR_BAD_REQUEST;
+               return NULL;
+       }
+
        LIST_FOR_EACH_ENTRY(private_layer, &private_output->layer_list, link) {
                if (private_layer->index == index) {
                        _pthread_mutex_unlock(&private_display->lock);
@@ -387,22 +693,9 @@ tdm_output_get_cursor_available_size(tdm_output *output, int *min_w, int *min_h,
 
        _pthread_mutex_lock(&private_display->lock);
 
-       if (!tdm_display_check_module_abi(private_display, 1, 5)) {
-
-               if (min_w)
-                       *min_w = -1;
-               if (min_h)
-                       *min_h = -1;
-               if (max_w)
-                       *max_w = -1;
-               if (max_h)
-                       *max_h = -1;
-               if (preferred_align)
-                       *preferred_align = -1;
-
+       if (!tdm_module_check_abi(private_output->private_module, 1, 5)) {
                _pthread_mutex_unlock(&private_display->lock);
-
-               return TDM_ERROR_BAD_MODULE;
+               return TDM_ERROR_BAD_REQUEST;
        }
 
        if (min_w)
@@ -494,17 +787,21 @@ tdm_output_get_primary_index(tdm_output *output, int *index)
 EXTERN tdm_error
 tdm_output_set_property(tdm_output *output, unsigned int id, tdm_value value)
 {
+       tdm_private_module *private_module;
        tdm_func_output *func_output;
        OUTPUT_FUNC_ENTRY();
 
        _pthread_mutex_lock(&private_display->lock);
 
-       func_output = &private_display->func_output;
+       private_module = private_output->private_module;
+       func_output = &private_module->func_output;
 
        if (!func_output->output_set_property) {
+               /* LCOV_EXCL_START */
                _pthread_mutex_unlock(&private_display->lock);
-               TDM_ERR("not implemented!!");
+               TDM_WRN("not implemented!!");
                return TDM_ERROR_NOT_IMPLEMENTED;
+               /* LCOV_EXCL_STOP */
        }
 
        ret = func_output->output_set_property(private_output->output_backend, id,
@@ -518,6 +815,7 @@ tdm_output_set_property(tdm_output *output, unsigned int id, tdm_value value)
 EXTERN tdm_error
 tdm_output_get_property(tdm_output *output, unsigned int id, tdm_value *value)
 {
+       tdm_private_module *private_module;
        tdm_func_output *func_output;
        OUTPUT_FUNC_ENTRY();
 
@@ -525,12 +823,15 @@ tdm_output_get_property(tdm_output *output, unsigned int id, tdm_value *value)
 
        _pthread_mutex_lock(&private_display->lock);
 
-       func_output = &private_display->func_output;
+       private_module = private_output->private_module;
+       func_output = &private_module->func_output;
 
        if (!func_output->output_get_property) {
+               /* LCOV_EXCL_START */
                _pthread_mutex_unlock(&private_display->lock);
-               TDM_ERR("not implemented!!");
+               TDM_WRN("not implemented!!");
                return TDM_ERROR_NOT_IMPLEMENTED;
+               /* LCOV_EXCL_STOP */
        }
 
        ret = func_output->output_get_property(private_output->output_backend, id,
@@ -541,44 +842,26 @@ tdm_output_get_property(tdm_output *output, unsigned int id, tdm_value *value)
        return ret;
 }
 
-INTERN void
-tdm_output_cb_vblank(tdm_output *output_backend, unsigned int sequence,
-                                        unsigned int tv_sec, unsigned int tv_usec, void *user_data)
+static void
+_tdm_output_thread_cb_vblank(tdm_private_display *private_display, void *object, tdm_thread_cb_base *cb_base, void *user_data)
 {
-       tdm_private_vblank_handler *vblank_handler = user_data;
-       tdm_private_vblank_handler *v = NULL, *vv = NULL;
-       tdm_private_output *private_output;
-       tdm_private_display *private_display;
+       tdm_thread_cb_output_vblank *output_vblank = (tdm_thread_cb_output_vblank *)cb_base;
+       tdm_private_output_vblank_handler *vblank_handler = output_vblank->base.data;
+       tdm_private_output_vblank_handler *v = NULL, *vv = NULL;
+       tdm_private_output *private_output = object;
        struct list_head clone_list;
        int interval, sync;
        pid_t tid = syscall(SYS_gettid);
 
        TDM_RETURN_IF_FAIL(TDM_MUTEX_IS_LOCKED());
-       TDM_RETURN_IF_FAIL(vblank_handler);
-
-       private_output = vblank_handler->private_output;
-       private_display = private_output->private_display;
-
-       if (vblank_handler->owner_tid != tid) {
-               tdm_thread_cb_output_vblank output_vblank;
-               tdm_error ret;
 
-               output_vblank.base.type = TDM_THREAD_CB_OUTPUT_VBLANK;
-               output_vblank.base.length = sizeof output_vblank;
-               output_vblank.output_stamp = vblank_handler->private_output->stamp;
-               output_vblank.sequence = sequence;
-               output_vblank.tv_sec = tv_sec;
-               output_vblank.tv_usec = tv_usec;
-               output_vblank.user_data = user_data;
+       assert(vblank_handler->owner_tid == tid);
 
-               ret = tdm_thread_send_cb(private_display->private_loop, &output_vblank.base);
-               TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
+       vblank_handler->sent_to_frontend = 0;
 
-               return;
-       }
+       _tdm_output_vblank_timeout_update(private_output, 0);
 
-       if (vblank_handler->owner_tid != tid)
-               TDM_NEVER_GET_HERE();
+       tdm_thread_cb_remove(private_output, TDM_THREAD_CB_OUTPUT_VBLANK, vblank_handler, _tdm_output_thread_cb_vblank, NULL);
 
        interval = vblank_handler->interval;
        sync = vblank_handler->sync;
@@ -601,9 +884,15 @@ tdm_output_cb_vblank(tdm_output *output_backend, unsigned int sequence,
                if (tdm_debug_module & TDM_DEBUG_COMMIT)
                        TDM_INFO("handler(%p)", v);
 
-               if (v->func)
-                       v->func(v->private_output, sequence, tv_sec, tv_usec, v->user_data);
                LIST_DEL(&v->link);
+
+               if (v->func)
+                       v->func(v->private_output,
+                                       output_vblank->sequence,
+                                       output_vblank->tv_sec,
+                                       output_vblank->tv_usec,
+                                       v->user_data);
+
                free(v);
        }
        _pthread_mutex_lock(&private_display->lock);
@@ -612,13 +901,39 @@ tdm_output_cb_vblank(tdm_output *output_backend, unsigned int sequence,
                TDM_INFO("-----------------------------------------...");
 }
 
-INTERN void
-tdm_output_cb_commit(tdm_output *output_backend, unsigned int sequence,
-                                        unsigned int tv_sec, unsigned int tv_usec, void *user_data)
+static void
+_tdm_output_cb_vblank(tdm_output *output_backend, unsigned int sequence,
+                                         unsigned int tv_sec, unsigned int tv_usec, void *user_data)
 {
-       tdm_private_output_commit_handler *output_commit_handler = user_data;
-       tdm_private_display *private_display;
-       tdm_private_output *private_output;
+       tdm_private_output_vblank_handler *vblank_handler = user_data;
+       tdm_thread_cb_output_vblank output_vblank;
+       tdm_error ret;
+
+       memset(&output_vblank, 0, sizeof output_vblank);
+       output_vblank.base.type = TDM_THREAD_CB_OUTPUT_VBLANK;
+       output_vblank.base.length = sizeof output_vblank;
+       output_vblank.base.object_stamp = vblank_handler->private_output->stamp;
+       output_vblank.base.data = vblank_handler;
+       output_vblank.base.sync = 0;
+       output_vblank.sequence = sequence;
+       output_vblank.tv_sec = tv_sec;
+       output_vblank.tv_usec = tv_usec;
+
+       vblank_handler->sent_to_frontend = 1;
+
+       if (tdm_debug_module & TDM_DEBUG_COMMIT)
+               TDM_INFO("output(%d) wait_vblank: handler(%p)", vblank_handler->private_output->pipe, vblank_handler);
+
+       ret = tdm_thread_cb_call(vblank_handler->private_output, &output_vblank.base, 1);
+       TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
+}
+
+static void
+_tdm_output_thread_cb_commit(tdm_private_display *private_display, void *object, tdm_thread_cb_base *cb_base, void *user_data)
+{
+       tdm_thread_cb_output_commit *output_commit = (tdm_thread_cb_output_commit *)cb_base;
+       tdm_private_output_commit_handler *output_commit_handler = output_commit->base.data;
+       tdm_private_output *private_output = object;
        tdm_private_layer *private_layer = NULL;
 
        TDM_RETURN_IF_FAIL(TDM_MUTEX_IS_LOCKED());
@@ -626,33 +941,18 @@ tdm_output_cb_commit(tdm_output *output_backend, unsigned int sequence,
        if (!output_commit_handler)
                return;
 
-       private_output = output_commit_handler->private_output;
-       private_display = private_output->private_display;
-
-       if (output_commit_handler->owner_tid != syscall(SYS_gettid)) {
-               tdm_thread_cb_output_commit output_commit;
-               tdm_error ret;
+       assert(output_commit_handler->owner_tid == syscall(SYS_gettid));
 
-               output_commit.base.type = TDM_THREAD_CB_OUTPUT_COMMIT;
-               output_commit.base.length = sizeof output_commit;
-               output_commit.output_stamp = private_output->stamp;
-               output_commit.sequence = sequence;
-               output_commit.tv_sec = tv_sec;
-               output_commit.tv_usec = tv_usec;
-               output_commit.user_data = user_data;
+       tdm_thread_cb_remove(private_output, TDM_THREAD_CB_OUTPUT_COMMIT, output_commit_handler, _tdm_output_thread_cb_commit, NULL);
 
-               ret = tdm_thread_send_cb(private_display->private_loop, &output_commit.base);
-               TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
-
-               return;
-       }
+       LIST_DEL(&output_commit_handler->link);
 
        if (tdm_debug_module & TDM_DEBUG_COMMIT) {
                TDM_INFO("----------------------------------------- output(%d) committed", private_output->pipe);
                TDM_INFO("handler(%p)", output_commit_handler);
        }
 
-       if (private_display->commit_type == TDM_COMMIT_TYPE_OUTPUT) {
+       if (private_output->commit_type == TDM_COMMIT_TYPE_OUTPUT) {
                /* In case of layer commit, the below will be handled in the layer commit callback */
                LIST_FOR_EACH_ENTRY(private_layer, &private_output->layer_list, link) {
                        if (private_layer->committed_buffer)
@@ -662,55 +962,90 @@ tdm_output_cb_commit(tdm_output *output_backend, unsigned int sequence,
 
        if (output_commit_handler->func) {
                _pthread_mutex_unlock(&private_display->lock);
-               output_commit_handler->func(private_output, sequence,
-                                                                       tv_sec, tv_usec, output_commit_handler->user_data);
+               output_commit_handler->func(private_output,
+                                                                       output_commit->sequence,
+                                                                       output_commit->tv_sec,
+                                                                       output_commit->tv_usec,
+                                                                       output_commit_handler->user_data);
                _pthread_mutex_lock(&private_display->lock);
        }
 
-       LIST_DEL(&output_commit_handler->link);
        free(output_commit_handler);
 
        if (tdm_debug_module & TDM_DEBUG_COMMIT)
                TDM_INFO("-----------------------------------------...");
 }
 
-/* add_front: To distinguish between the user vblank handlers and the layer
- *            commit vblank handlers. The layer commit handlers will be called
- *            before calling the user vblank handlers.
+static void
+_tdm_output_cb_commit(tdm_output *output_backend, unsigned int sequence,
+                                         unsigned int tv_sec, unsigned int tv_usec, void *user_data)
+{
+       tdm_private_output_commit_handler *output_commit_handler = user_data;
+       tdm_private_output *private_output;
+       tdm_thread_cb_output_commit output_commit;
+       tdm_error ret;
+
+       if (output_commit_handler)
+               private_output = output_commit_handler->private_output;
+       else
+               private_output = tdm_display_find_private_output(tdm_display_get(), output_backend);
+
+       memset(&output_commit, 0, sizeof output_commit);
+       output_commit.base.type = TDM_THREAD_CB_OUTPUT_COMMIT;
+       output_commit.base.length = sizeof output_commit;
+       output_commit.base.object_stamp = private_output->stamp;
+       output_commit.base.data = output_commit_handler;
+       output_commit.base.sync = 0;
+       output_commit.sequence = sequence;
+       output_commit.tv_sec = tv_sec;
+       output_commit.tv_usec = tv_usec;
+
+       ret = tdm_thread_cb_call(private_output, &output_commit.base, 1);
+       TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
+}
+
+/* add_front: To distinguish between the user vblank handlers and the layer
+ *            commit vblank handlers. The layer commit handlers will be called
+ *            before calling the user vblank handlers.
  */
 static tdm_error
-_tdm_output_wait_vblank(tdm_output *output, int interval, int sync,
+_tdm_output_wait_vblank(tdm_private_output *private_output, int interval, int sync,
                                                tdm_output_vblank_handler func, void *user_data,
                                                unsigned int add_front)
 {
+       tdm_private_module *private_module;
        tdm_func_output *func_output;
-       tdm_private_vblank_handler *vblank_handler = NULL, *v = NULL;
+       tdm_private_output_vblank_handler *vblank_handler = NULL, *v = NULL;
        unsigned int skip_request = 0;
        pid_t tid = syscall(SYS_gettid);
+       tdm_error ret = TDM_ERROR_NONE;
 
-       OUTPUT_FUNC_ENTRY();
-
-       func_output = &private_display->func_output;
+       private_module = private_output->private_module;
+       func_output = &private_module->func_output;
 
        /* interval SHOULD be at least 1 */
        if (interval <= 0)
                interval = 1;
 
        if (!func_output->output_wait_vblank) {
-               TDM_ERR("not implemented!!");
+               /* LCOV_EXCL_START */
+               TDM_WRN("not implemented!!");
                return TDM_ERROR_NOT_IMPLEMENTED;
+               /* LCOV_EXCL_STOP */
        }
 
        if (!private_output->regist_vblank_cb) {
                private_output->regist_vblank_cb = 1;
                ret = func_output->output_set_vblank_handler(private_output->output_backend,
-                               tdm_output_cb_vblank);
+                               _tdm_output_cb_vblank);
        }
 
-       vblank_handler = calloc(1, sizeof(tdm_private_vblank_handler));
+       vblank_handler = calloc(1, sizeof(tdm_private_output_vblank_handler));
        if (!vblank_handler) {
+               /* LCOV_EXCL_START */
                TDM_ERR("failed: alloc memory");
                return TDM_ERROR_OUT_OF_MEMORY;
+               /* LCOV_EXCL_STOP */
        }
 
        if (tdm_debug_module & TDM_DEBUG_COMMIT)
@@ -737,10 +1072,18 @@ _tdm_output_wait_vblank(tdm_output *output, int interval, int sync,
 
        /* If there is the previous request, we can skip to call output_wait_vblank() */
        if (!skip_request) {
+               ret = tdm_thread_cb_add(private_output, TDM_THREAD_CB_OUTPUT_VBLANK, vblank_handler, _tdm_output_thread_cb_vblank, NULL);
+               if (ret != TDM_ERROR_NONE) {
+                       TDM_ERR("tdm_thread_cb_add failed");
+                       goto wait_failed;
+               }
+
                ret = func_output->output_wait_vblank(private_output->output_backend, interval,
                                                                                          sync, vblank_handler);
                TDM_GOTO_IF_FAIL(ret == TDM_ERROR_NONE, wait_failed);
 
+               _tdm_output_vblank_timeout_update(private_output, 1000);
+
                if (tdm_debug_module & TDM_DEBUG_COMMIT)
                        TDM_INFO("output(%d) backend wait_vblank", private_output->pipe);
        }
@@ -748,11 +1091,14 @@ _tdm_output_wait_vblank(tdm_output *output, int interval, int sync,
        return ret;
 
 wait_failed:
+       /* LCOV_EXCL_START */
        if (vblank_handler) {
+               tdm_thread_cb_remove(private_output, TDM_THREAD_CB_OUTPUT_VBLANK, vblank_handler, _tdm_output_thread_cb_vblank, NULL);
                LIST_DEL(&vblank_handler->link);
                free(vblank_handler);
        }
        return ret;
+       /* LCOV_EXCL_STOP */
 }
 
 EXTERN tdm_error
@@ -760,23 +1106,25 @@ tdm_output_wait_vblank(tdm_output *output, int interval, int sync,
                                           tdm_output_vblank_handler func, void *user_data)
 {
        OUTPUT_FUNC_ENTRY();
+       TDM_RETURN_VAL_IF_FAIL(interval > 0, TDM_ERROR_INVALID_PARAMETER);
 
        _pthread_mutex_lock(&private_display->lock);
 
-       if (private_output->current_dpms_value > TDM_OUTPUT_DPMS_ON) {
+       if (TDM_OUTPUT_DPMS_VSYNC_IS_OFF(private_output->current_dpms_value)) {
                TDM_WRN("output(%d) dpms: %s", private_output->pipe,
                                tdm_dpms_str(private_output->current_dpms_value));
                _pthread_mutex_unlock(&private_display->lock);
                return TDM_ERROR_DPMS_OFF;
        }
 
-       ret = _tdm_output_wait_vblank(output, interval, sync, func, user_data, 0);
+       ret = _tdm_output_wait_vblank(private_output, interval, sync, func, user_data, 0);
 
        _pthread_mutex_unlock(&private_display->lock);
 
        return ret;
 }
 
+/* LCOV_EXCL_START */
 EXTERN tdm_error
 tdm_output_wait_vblank_add_front(tdm_output *output, int interval, int sync,
                                                                 tdm_output_vblank_handler func, void *user_data)
@@ -785,61 +1133,210 @@ tdm_output_wait_vblank_add_front(tdm_output *output, int interval, int sync,
 
        _pthread_mutex_lock(&private_display->lock);
 
-       if (private_output->current_dpms_value > TDM_OUTPUT_DPMS_ON) {
+       if (TDM_OUTPUT_DPMS_VSYNC_IS_OFF(private_output->current_dpms_value)) {
                TDM_WRN("output(%d) dpms: %s", private_output->pipe,
                                tdm_dpms_str(private_output->current_dpms_value));
                _pthread_mutex_unlock(&private_display->lock);
                return TDM_ERROR_DPMS_OFF;
        }
 
-       ret = _tdm_output_wait_vblank(output, interval, sync, func, user_data, 1);
+       ret = _tdm_output_wait_vblank(private_output, interval, sync, func, user_data, 1);
+
+       _pthread_mutex_unlock(&private_display->lock);
+
+       return ret;
+}
+/* LCOV_EXCL_STOP */
+
+INTERN void
+tdm_output_remove_vblank_handler_internal(tdm_output *output, tdm_output_vblank_handler func, void *user_data)
+{
+       tdm_private_output *private_output = (tdm_private_output*)output;
+       tdm_private_output_vblank_handler *v = NULL;
+
+       TDM_RETURN_IF_FAIL(private_output != NULL);
+       TDM_RETURN_IF_FAIL(TDM_MUTEX_IS_LOCKED());
+
+       LIST_FOR_EACH_ENTRY(v, &private_output->vblank_handler_list, link) {
+               if (v->func == func && v->user_data == user_data) {
+                       /* only set func & user_data to NULL. It will be freed when an event occurs */
+                       v->func = NULL;
+                       v->user_data = NULL;
+                       break;
+               }
+       }
+}
+
+INTERN void
+tdm_output_remove_commit_handler_internal(tdm_output *output, tdm_output_commit_handler func, void *user_data)
+{
+       tdm_private_output *private_output = (tdm_private_output*)output;
+       tdm_private_output_commit_handler *c = NULL;
+
+       TDM_RETURN_IF_FAIL(private_output != NULL);
+       TDM_RETURN_IF_FAIL(TDM_MUTEX_IS_LOCKED());
+
+       LIST_FOR_EACH_ENTRY(c, &private_output->output_commit_handler_list, link) {
+               if (c->func == func && c->user_data == user_data) {
+                       /* only set func & user_data to NULL. It will be freed when an event occurs */
+                       c->func = NULL;
+                       c->user_data = NULL;
+                       break;
+               }
+       }
+}
+
+EXTERN tdm_error
+tdm_output_remove_vblank_handler(tdm_output *output, tdm_output_vblank_handler func, void *user_data)
+{
+       OUTPUT_FUNC_ENTRY();
+
+       _pthread_mutex_lock(&private_display->lock);
+
+       tdm_output_remove_vblank_handler_internal(output, func, user_data);
+
+       _pthread_mutex_unlock(&private_display->lock);
+
+       return ret;
+}
+
+EXTERN tdm_error
+tdm_output_remove_commit_handler(tdm_output *output, tdm_output_commit_handler func, void *user_data)
+{
+       OUTPUT_FUNC_ENTRY();
+
+       _pthread_mutex_lock(&private_display->lock);
+
+       tdm_output_remove_commit_handler_internal(output, func, user_data);
+
+       _pthread_mutex_unlock(&private_display->lock);
+
+       return ret;
+}
+
+tdm_error
+_tdm_output_commit_virtual(tdm_output *output, int sync, tdm_output_commit_handler func, void *user_data)
+{
+       tdm_private_output *private_output;
+       tdm_private_output_commit_handler *output_commit_handler = NULL;
+       tdm_private_layer *private_layer = NULL;
+       tdm_private_display *private_display;
+       tbm_surface_h buffer;
+       tdm_error ret = TDM_ERROR_NONE;
+
+       TDM_RETURN_VAL_IF_FAIL(output != NULL, TDM_ERROR_INVALID_PARAMETER);
+
+       private_output = (tdm_private_output*)output;
+       private_display = private_output->private_display;
+
+       if (!LIST_IS_EMPTY(&private_output->output_commit_handler_list)) {
+               /* support only 1 commit */
+               TDM_ERR("voutput canot commit while commiting");
+               return TDM_ERROR_OPERATION_FAILED;
+       }
+
+       output_commit_handler = calloc(1, sizeof(tdm_private_output_commit_handler));
+       if (!output_commit_handler) {
+               /* LCOV_EXCL_START */
+               TDM_ERR("failed: alloc memory");
+               return TDM_ERROR_OUT_OF_MEMORY;
+               /* LCOV_EXCL_STOP */
+       }
+
+       LIST_ADDTAIL(&output_commit_handler->link, &private_output->output_commit_handler_list);
+       output_commit_handler->private_output = private_output;
+       output_commit_handler->func = func;
+       output_commit_handler->user_data = user_data;
+       output_commit_handler->owner_tid = syscall(SYS_gettid);
+
+       /* voutput use only 1 layer */
+       LIST_FOR_EACH_ENTRY(private_layer, &private_output->layer_list, link) {
+               if (private_layer) break;
+       }
+       TDM_GOTO_IF_FAIL(private_layer, commit_failed);
+
+       buffer = private_layer->commiting_buffer;
+       TDM_GOTO_IF_FAIL(buffer != NULL, commit_failed);
 
        _pthread_mutex_unlock(&private_display->lock);
+       ret = tdm_output_attach_buffer(private_output, buffer);
+       TDM_GOTO_IF_FAIL(ret == TDM_ERROR_NONE, commit_failed);
+       ret = tdm_output_commit_buffer(private_output);
+       TDM_GOTO_IF_FAIL(ret == TDM_ERROR_NONE, commit_failed);
+       _pthread_mutex_lock(&private_display->lock);
 
        return ret;
+
+commit_failed:
+       /* LCOV_EXCL_START */
+       if (output_commit_handler) {
+               LIST_DEL(&output_commit_handler->link);
+               free(output_commit_handler);
+       }
+       return ret;
+/* LCOV_EXCL_STOP */
 }
 
 INTERN tdm_error
 tdm_output_commit_internal(tdm_output *output, int sync, tdm_output_commit_handler func, void *user_data)
 {
+       tdm_private_display *private_display;
+       tdm_private_output *private_output;
+       tdm_private_module *private_module;
        tdm_func_output *func_output;
        tdm_private_output_commit_handler *output_commit_handler = NULL;
        tdm_private_layer *private_layer = NULL;
        tdm_output_dpms dpms_value = TDM_OUTPUT_DPMS_ON;
+       tdm_error ret = TDM_ERROR_NONE;
 
-       OUTPUT_FUNC_ENTRY();
+       TDM_RETURN_VAL_IF_FAIL(output != NULL, TDM_ERROR_INVALID_PARAMETER);
 
-       func_output = &private_display->func_output;
+       private_output = (tdm_private_output*)output;
+       private_module = private_output->private_module;
+       private_display = private_module->private_display;
+       func_output = &private_module->func_output;
+
+       if (private_module == private_display->virtual_module)
+               return _tdm_output_commit_virtual(output, sync, func, user_data);
 
        if (!func_output->output_commit) {
-               TDM_ERR("not implemented!!");
+               /* LCOV_EXCL_START */
+               TDM_WRN("not implemented!!");
                return TDM_ERROR_NOT_IMPLEMENTED;
+               /* LCOV_EXCL_STOP */
        }
 
-       tdm_output_get_dpms_internal(output, &dpms_value);
-
-       if (dpms_value == TDM_OUTPUT_DPMS_ON) {
-               if (func) {
-                       if (!private_output->regist_commit_cb) {
-                               private_output->regist_commit_cb = 1;
-                               ret = func_output->output_set_commit_handler(private_output->output_backend,
-                                                       tdm_output_cb_commit);
-                               TDM_GOTO_IF_FAIL(ret == TDM_ERROR_NONE, commit_failed);
-                       }
-
-                       output_commit_handler = calloc(1, sizeof(tdm_private_output_commit_handler));
-                       if (!output_commit_handler) {
-                               TDM_ERR("failed: alloc memory");
-                               return TDM_ERROR_OUT_OF_MEMORY;
-                       }
-
-                       LIST_ADDTAIL(&output_commit_handler->link, &private_output->output_commit_handler_list);
-                       output_commit_handler->private_output = private_output;
-                       output_commit_handler->func = func;
-                       output_commit_handler->user_data = user_data;
-                       output_commit_handler->owner_tid = syscall(SYS_gettid);
+       ret = tdm_output_get_dpms_internal(output, &dpms_value);
+       TDM_RETURN_VAL_IF_FAIL(ret == TDM_ERROR_NONE, ret);
+
+       if (!TDM_OUTPUT_DPMS_VSYNC_IS_OFF(dpms_value)) {
+               if (!private_output->regist_commit_cb) {
+                       private_output->regist_commit_cb = 1;
+                       ret = func_output->output_set_commit_handler(private_output->output_backend, _tdm_output_cb_commit);
+                       TDM_GOTO_IF_FAIL(ret == TDM_ERROR_NONE, commit_failed);
                }
 
+               output_commit_handler = calloc(1, sizeof(tdm_private_output_commit_handler));
+               if (!output_commit_handler) {
+                       /* LCOV_EXCL_START */
+                       TDM_ERR("failed: alloc memory");
+                       return TDM_ERROR_OUT_OF_MEMORY;
+                       /* LCOV_EXCL_STOP */
+               }
+
+               ret = tdm_thread_cb_add(private_output, TDM_THREAD_CB_OUTPUT_COMMIT, output_commit_handler, _tdm_output_thread_cb_commit, NULL);
+               if (ret != TDM_ERROR_NONE) {
+                       TDM_ERR("tdm_thread_cb_add failed");
+                       free(output_commit_handler);
+                       return ret;
+               }
+
+               LIST_ADDTAIL(&output_commit_handler->link, &private_output->output_commit_handler_list);
+               output_commit_handler->private_output = private_output;
+               output_commit_handler->func = func;
+               output_commit_handler->user_data = user_data;
+               output_commit_handler->owner_tid = syscall(SYS_gettid);
+
                ret = func_output->output_commit(private_output->output_backend, sync,
                                output_commit_handler);
                TDM_GOTO_IF_FAIL(ret == TDM_ERROR_NONE, commit_failed);
@@ -849,10 +1346,22 @@ tdm_output_commit_internal(tdm_output *output, int sync, tdm_output_commit_handl
                                        private_output->pipe, output_commit_handler, func, user_data);
        }
 
+       /* Even if DPMS is off, committed_buffer should be changed because it will be referred
+        * for tdm_layer_committed() function.
+        */
        LIST_FOR_EACH_ENTRY(private_layer, &private_output->layer_list, link) {
                if (!private_layer->waiting_buffer)
                        continue;
 
+               if (private_layer->committed_buffer) {
+                       tdm_layer_free_buffer(private_layer, private_layer->committed_buffer);
+                       private_layer->committed_buffer = NULL;
+                       if (tdm_debug_module & TDM_DEBUG_BUFFER)
+                               TDM_INFO("layer(%p) waiting_buffer(%p) committed_buffer(%p)",
+                                                private_layer, private_layer->waiting_buffer->buffer,
+                                                private_layer->committed_buffer);
+               }
+
                private_layer->committed_buffer = private_layer->waiting_buffer;
                private_layer->waiting_buffer = NULL;
                if (tdm_debug_module & TDM_DEBUG_BUFFER)
@@ -861,8 +1370,8 @@ tdm_output_commit_internal(tdm_output *output, int sync, tdm_output_commit_handl
                                         private_layer->committed_buffer->buffer);
        }
 
-       if (dpms_value != TDM_OUTPUT_DPMS_ON) {
-               TDM_WRN("TDM_OUTPUT_DPMS_OFF. Directly call commit handler instead of commit.");
+       if (TDM_OUTPUT_DPMS_VSYNC_IS_OFF(dpms_value)) {
+               TDM_WRN("dpms %s. Directly call commit handler instead of commit.", tdm_dpms_str(dpms_value));
                if (func)
                        func(output, 0, 0, 0, user_data);
        }
@@ -870,36 +1379,41 @@ tdm_output_commit_internal(tdm_output *output, int sync, tdm_output_commit_handl
        return ret;
 
 commit_failed:
+       /* LCOV_EXCL_START */
        if (output_commit_handler) {
+               tdm_thread_cb_remove(private_output, TDM_THREAD_CB_OUTPUT_COMMIT, output_commit_handler, _tdm_output_thread_cb_commit, NULL);
                LIST_DEL(&output_commit_handler->link);
                free(output_commit_handler);
        }
        return ret;
+       /* LCOV_EXCL_STOP */
 }
 
 EXTERN tdm_error
 tdm_output_commit(tdm_output *output, int sync, tdm_output_commit_handler func,
                                  void *user_data)
 {
+       tdm_private_layer *private_layer = NULL;
+
        OUTPUT_FUNC_ENTRY();
 
        _pthread_mutex_lock(&private_display->lock);
 
-       if (private_display->commit_type == TDM_COMMIT_TYPE_NONE)
-               private_display->commit_type = TDM_COMMIT_TYPE_OUTPUT;
-       else if (private_display->commit_type == TDM_COMMIT_TYPE_LAYER) {
+       if (private_output->commit_type == TDM_COMMIT_TYPE_NONE)
+               private_output->commit_type = TDM_COMMIT_TYPE_OUTPUT;
+       else if (private_output->commit_type == TDM_COMMIT_TYPE_LAYER) {
                TDM_ERR("Can't supported. Use tdm_layer_commit");
                _pthread_mutex_unlock(&private_display->lock);
                return TDM_ERROR_BAD_REQUEST;
        }
 
-       if (private_display->commit_per_vblank) {
+       if (private_output->commit_per_vblank) {
                TDM_ERR("Use tdm_layer_commit");
                _pthread_mutex_unlock(&private_display->lock);
                return TDM_ERROR_BAD_REQUEST;
        }
 
-       if (private_output->current_dpms_value > TDM_OUTPUT_DPMS_ON) {
+       if (TDM_OUTPUT_DPMS_VSYNC_IS_OFF(private_output->current_dpms_value)) {
                TDM_ERR("output(%d) dpms: %s", private_output->pipe,
                                tdm_dpms_str(private_output->current_dpms_value));
                _pthread_mutex_unlock(&private_display->lock);
@@ -909,6 +1423,11 @@ tdm_output_commit(tdm_output *output, int sync, tdm_output_commit_handler func,
        if (tdm_debug_module & TDM_DEBUG_COMMIT)
                TDM_INFO("output(%d) commit", private_output->pipe);
 
+       /* apply the pending data of all layers */
+       LIST_FOR_EACH_ENTRY(private_layer, &private_output->layer_list, link) {
+               tdm_layer_commit_pending_data(private_layer);
+       }
+
        ret = tdm_output_commit_internal(output, sync, func, user_data);
 
        _pthread_mutex_unlock(&private_display->lock);
@@ -919,6 +1438,7 @@ tdm_output_commit(tdm_output *output, int sync, tdm_output_commit_handler func,
 EXTERN tdm_error
 tdm_output_set_mode(tdm_output *output, const tdm_output_mode *mode)
 {
+       tdm_private_module *private_module;
        tdm_func_output *func_output;
        OUTPUT_FUNC_ENTRY();
 
@@ -926,17 +1446,24 @@ tdm_output_set_mode(tdm_output *output, const tdm_output_mode *mode)
 
        _pthread_mutex_lock(&private_display->lock);
 
-       func_output = &private_display->func_output;
+       private_module = private_output->private_module;
+       func_output = &private_module->func_output;
 
        if (!func_output->output_set_mode) {
+               /* LCOV_EXCL_START */
                _pthread_mutex_unlock(&private_display->lock);
-               TDM_ERR("not implemented!!");
+               TDM_WRN("not implemented!!");
                return TDM_ERROR_NOT_IMPLEMENTED;
+               /* LCOV_EXCL_STOP */
        }
 
        ret = func_output->output_set_mode(private_output->output_backend, mode);
-       if (ret == TDM_ERROR_NONE)
+       if (ret == TDM_ERROR_NONE) {
                private_output->current_mode = mode;
+               private_output->need_set_target_info = 1;
+               TDM_INFO("mode: %dx%d %dhz", mode->hdisplay, mode->vdisplay, mode->vrefresh);
+       }
+
        _pthread_mutex_unlock(&private_display->lock);
 
        return ret;
@@ -958,171 +1485,199 @@ tdm_output_get_mode(tdm_output *output, const tdm_output_mode **mode)
        return ret;
 }
 
-static tdm_error
-_tdm_output_dpms_changed_timeout(void *user_data)
+EXTERN tdm_error
+tdm_output_set_dpms(tdm_output *output, tdm_output_dpms dpms_value)
 {
-       tdm_private_output *private_output = user_data;
-       tdm_value value;
-
-       value.u32 = private_output->current_dpms_value;
-       tdm_output_call_change_handler_internal(private_output,
-                                                                                       &private_output->change_handler_list_sub,
-                                                                                       TDM_OUTPUT_CHANGE_DPMS,
-                                                                                       value, 0);
+       tdm_private_module *private_module;
+       tdm_func_output *func_output;
+       OUTPUT_FUNC_ENTRY();
 
-       return TDM_ERROR_NONE;
-}
+       if (dpms_value > TDM_OUTPUT_DPMS_OFF) {
+               if (dpms_value & TDM_OUTPUT_DPMS_DEFAULT_MASK) {
+                       TDM_ERR("Don't use the low-4bit for an extended DPMS mode: dpms_value(%x)", dpms_value);
+                       return TDM_ERROR_BAD_REQUEST;
+               }
 
-INTERN void
-tdm_output_cb_dpms(tdm_output *output_backend, tdm_output_dpms dpms, void *user_data)
-{
-       tdm_private_display *private_display;
-       tdm_private_output *private_output = user_data;
-       tdm_value value;
+               if (!(private_output->caps.capabilities & TDM_OUTPUT_CAPABILITY_EXTENDED_DPMS)) {
+                       TDM_ERR("output(%d) doesn't support the extended DPMS control: '%s'",
+                                       private_output->pipe, tdm_dpms_str(dpms_value));
+                       return TDM_ERROR_BAD_REQUEST;
+               }
+       }
 
-       TDM_RETURN_IF_FAIL(TDM_MUTEX_IS_LOCKED());
-       TDM_RETURN_IF_FAIL(private_output);
+       _pthread_mutex_lock(&private_display->lock);
 
-       private_display = private_output->private_display;
+       if (private_output->waiting_dpms_change) {
+               TDM_ERR("DPMS is not changed yet. Can't be changed twice");
+               _pthread_mutex_unlock(&private_display->lock);
+               return TDM_ERROR_BAD_REQUEST;
+       }
 
-       if (!tdm_thread_in_display_thread(syscall(SYS_gettid))) {
-               tdm_thread_cb_output_dpms output_dpms;
-               tdm_error ret;
+       private_module = private_output->private_module;
+       func_output = &private_module->func_output;
 
-               _tdm_output_update(output_backend, user_data);
+       TDM_INFO("output(%d) dpms '%s'", private_output->pipe, tdm_dpms_str(dpms_value));
 
-               output_dpms.base.type = TDM_THREAD_CB_OUTPUT_DPMS;
-               output_dpms.base.length = sizeof output_dpms;
-               output_dpms.output_stamp = private_output->stamp;
-               output_dpms.dpms = dpms;
-               output_dpms.user_data = user_data;
+       if (func_output->output_set_dpms)
+               ret = func_output->output_set_dpms(private_output->output_backend, dpms_value);
+       else {
+               /* LCOV_EXCL_START */
+               ret = TDM_ERROR_NONE;
+               TDM_WRN("not implemented!!");
+               goto done;
+               /* LCOV_EXCL_STOP */
+       }
 
-               value.u32 = dpms;
-               tdm_output_call_change_handler_internal(private_output,
-                                                                                               &private_output->change_handler_list_sub,
-                                                                                               TDM_OUTPUT_CHANGE_DPMS,
-                                                                                               value, 0);
+done:
+       if (ret == TDM_ERROR_NONE) {
+               if (private_output->current_dpms_value != dpms_value) {
+                       private_output->current_dpms_value = dpms_value;
+                       _tdm_output_call_thread_cb_dpms(private_output, dpms_value);
+               }
+               TDM_INFO("output(%d) dpms '%s' done", private_output->pipe, tdm_dpms_str(dpms_value));
+       } else {
+               tdm_output_dpms temp = TDM_OUTPUT_DPMS_OFF;
 
-               ret = tdm_thread_send_cb(private_display->private_loop, &output_dpms.base);
-               TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
+               /* update current_dpms_value forcely */
+               tdm_output_get_dpms_internal(output, &temp);
 
-               return;
+               TDM_ERR("output(%d) set_dpms failed: dpms '%s'", private_output->pipe, tdm_dpms_str(temp));
        }
 
-       private_output->current_dpms_value = dpms;
+       _pthread_mutex_unlock(&private_display->lock);
 
-       value.u32 = dpms;
-       tdm_output_call_change_handler_internal(private_output,
-                                                                                       &private_output->change_handler_list_main,
-                                                                                       TDM_OUTPUT_CHANGE_DPMS,
-                                                                                       value, 0);
+       return ret;
 }
 
+/* LCOV_EXCL_START */
 EXTERN tdm_error
-tdm_output_set_dpms(tdm_output *output, tdm_output_dpms dpms_value)
+tdm_output_set_dpms_async(tdm_output *output, tdm_output_dpms dpms_value)
 {
+       tdm_private_module *private_module;
        tdm_func_output *func_output;
        OUTPUT_FUNC_ENTRY();
 
-       if (dpms_value > TDM_OUTPUT_DPMS_OFF)
-               dpms_value = TDM_OUTPUT_DPMS_OFF;
+       if (!(private_output->caps.capabilities & TDM_OUTPUT_CAPABILITY_ASYNC_DPMS)) {
+               TDM_ERR("output doesn't support the asynchronous DPMS control!");
+               return TDM_ERROR_BAD_REQUEST;
+       }
+
+       if (dpms_value > TDM_OUTPUT_DPMS_OFF) {
+               if (dpms_value & TDM_OUTPUT_DPMS_DEFAULT_MASK) {
+                       TDM_ERR("Don't use the low-4bit for an extended DPMS mode: dpms_value(%x)", dpms_value);
+                       return TDM_ERROR_BAD_REQUEST;
+               }
+
+               if (!(private_output->caps.capabilities & TDM_OUTPUT_CAPABILITY_EXTENDED_DPMS)) {
+                       TDM_ERR("output(%d) doesn't support the extended DPMS control: '%s'",
+                                       private_output->pipe, tdm_dpms_str(dpms_value));
+                       return TDM_ERROR_BAD_REQUEST;
+               }
+       }
 
        _pthread_mutex_lock(&private_display->lock);
 
-       if (private_output->current_dpms_value == dpms_value) {
+       if (private_output->waiting_dpms_change) {
+               TDM_ERR("DPMS is not changed yet. Can't be changed twice");
                _pthread_mutex_unlock(&private_display->lock);
-               return TDM_ERROR_NONE;
+               return TDM_ERROR_BAD_REQUEST;
        }
 
-       /** Use timer to call the output change callback of the sub-thread.
-        * The output change callback of tdm_server and tdm_vblank was called
-        * in the main thread. And it made the multi thread issue. If we use
-        * the timer, we can call the sub-thread's output change callback in
-        * sub-thread.
-        */
-       if (!private_output->dpms_changed_timer) {
-               private_output->dpms_changed_timer =
-                       tdm_event_loop_add_timer_handler(private_output->private_display,
-                                                                                        _tdm_output_dpms_changed_timeout, private_output, NULL);
-               if (!private_output->dpms_changed_timer) {
-                       TDM_ERR("can't create dpms timer!!");
-                       _pthread_mutex_unlock(&private_display->lock);
-                       return TDM_ERROR_OUT_OF_MEMORY;
-               }
+       private_module = private_output->private_module;
+       func_output = &private_module->func_output;
+       if (!func_output->output_set_dpms_handler) {
+               TDM_WRN("not implemented: output_set_dpms_handler");
+               _pthread_mutex_unlock(&private_display->lock);
+               return TDM_ERROR_NOT_IMPLEMENTED;
        }
 
-       func_output = &private_display->func_output;
-
-       if (!func_output->output_set_dpms) {
+       if (!func_output->output_set_dpms_async) {
+               TDM_WRN("not implemented: output_set_dpms_async");
                _pthread_mutex_unlock(&private_display->lock);
-               private_output->current_dpms_value = dpms_value;
-               TDM_WRN("not implemented!!");
-               return TDM_ERROR_NONE;
+               return TDM_ERROR_NOT_IMPLEMENTED;
        }
 
-       if (func_output->output_set_dpms_handler) {
-               if (!private_output->regist_dpms_cb) {
-                       private_output->regist_dpms_cb = 1;
-                       ret = func_output->output_set_dpms_handler(private_output->output_backend,
-                                       tdm_output_cb_dpms, private_output);
-                       if (ret != TDM_ERROR_NONE) {
-                               _pthread_mutex_unlock(&private_display->lock);
-                               TDM_ERR("Can't set the dpms handler!!");
-                               return ret;
-                       }
+       if (!private_output->regist_dpms_cb) {
+               private_output->regist_dpms_cb = 1;
+               ret = func_output->output_set_dpms_handler(private_output->output_backend,
+                               tdm_output_cb_dpms, private_output);
+               if (ret != TDM_ERROR_NONE) {
+                       _pthread_mutex_unlock(&private_display->lock);
+                       TDM_ERR("Can't set the dpms handler!!");
+                       return ret;
                }
        }
 
-       ret = func_output->output_set_dpms(private_output->output_backend, dpms_value);
+       TDM_INFO("output(%d) dpms async '%s'", private_output->pipe, tdm_dpms_str(dpms_value));
 
-       if (ret == TDM_ERROR_NONE && !func_output->output_set_dpms_handler) {
-               tdm_value value;
+       ret = func_output->output_set_dpms_async(private_output->output_backend, dpms_value);
 
-               private_output->current_dpms_value = dpms_value;
+       if (ret == TDM_ERROR_NONE) {
+               private_output->waiting_dpms_change = 1;
+               TDM_INFO("output(%d) dpms async '%s' waiting", private_output->pipe, tdm_dpms_str(dpms_value));
+       } else {
+               tdm_output_dpms temp = TDM_OUTPUT_DPMS_OFF;
 
-               value.u32 = dpms_value;
-               tdm_output_call_change_handler_internal(private_output,
-                                                                                               &private_output->change_handler_list_main,
-                                                                                               TDM_OUTPUT_CHANGE_DPMS,
-                                                                                               value, 0);
+               /* update current_dpms_value forcely */
+               tdm_output_get_dpms_internal(output, &temp);
 
-               if (!LIST_IS_EMPTY(&private_output->change_handler_list_sub)) {
-                       ret = tdm_event_loop_source_timer_update(private_output->dpms_changed_timer, 1);
-                       if (ret != TDM_ERROR_NONE)
-                               TDM_NEVER_GET_HERE();
-               }
+               TDM_ERR("output(%d) set_dpms_async failed: dpms '%s'", private_output->pipe, tdm_dpms_str(temp));
        }
 
        _pthread_mutex_unlock(&private_display->lock);
 
        return ret;
 }
+/* LCOV_EXCL_STOP */
 
 INTERN tdm_error
 tdm_output_get_dpms_internal(tdm_output *output, tdm_output_dpms *dpms_value)
 {
+       tdm_private_output *private_output;
+       tdm_private_module *private_module;
        tdm_func_output *func_output;
-       OUTPUT_FUNC_ENTRY();
+       tdm_error ret = TDM_ERROR_NONE;
 
-       TDM_RETURN_VAL_IF_FAIL(private_output != NULL, TDM_ERROR_INVALID_PARAMETER);
+       TDM_RETURN_VAL_IF_FAIL(output != NULL, TDM_ERROR_INVALID_PARAMETER);
 
-       func_output = &private_display->func_output;
+       private_output = (tdm_private_output*)output;
+
+       /* TODO: this is ugly. But before calling backend's output_get_dpms(), we have
+        * to check if all backends's DPMS operation has no problem. In future, we'd
+        * better use new env instead of using commit_per_vblank variable to distinguish
+        * whether we use the stored value or backend's output_get_dpms.
+        */
+       if (!private_output->commit_per_vblank) {
+               *dpms_value = private_output->current_dpms_value;
+               return TDM_ERROR_NONE;
+       }
+
+       private_module = private_output->private_module;
+       func_output = &private_module->func_output;
 
        if (!func_output->output_get_dpms) {
+               /* LCOV_EXCL_START */
                *dpms_value = private_output->current_dpms_value;
                TDM_WRN("not implemented!!");
                return TDM_ERROR_NONE;
+               /* LCOV_EXCL_STOP */
        }
 
        ret = func_output->output_get_dpms(private_output->output_backend, dpms_value);
        if (ret != TDM_ERROR_NONE) {
+               /* LCOV_EXCL_START */
                TDM_ERR("output_get_dpms failed");
                *dpms_value = TDM_OUTPUT_DPMS_OFF;
+               /* LCOV_EXCL_STOP */
        }
 
+       /* checking with backend's value */
        if (*dpms_value != private_output->current_dpms_value) {
+               TDM_ERR("output(%d) dpms changed suddenly: %s -> %s",
+                               private_output->pipe, tdm_dpms_str(private_output->current_dpms_value),
+                               tdm_dpms_str(*dpms_value));
                private_output->current_dpms_value = *dpms_value;
-               TDM_ERR("output(%d) dpms changed suddenly: %s", private_output->pipe, tdm_dpms_str(*dpms_value));
+               _tdm_output_call_thread_cb_dpms(private_output, *dpms_value);
        }
 
        return ret;
@@ -1144,6 +1699,31 @@ tdm_output_get_dpms(tdm_output *output, tdm_output_dpms *dpms_value)
        return ret;
 }
 
+EXTERN tdm_error
+tdm_output_has_capture_capability(tdm_output *output, unsigned int *has_capability)
+{
+       tdm_private_module *private_module;
+
+       OUTPUT_FUNC_ENTRY();
+
+       TDM_RETURN_VAL_IF_FAIL(has_capability != NULL, TDM_ERROR_INVALID_PARAMETER);
+
+       _pthread_mutex_lock(&private_display->lock);
+
+       private_module = private_output->private_module;
+
+       if (!(private_module->capabilities & TDM_DISPLAY_CAPABILITY_CAPTURE))
+               *has_capability = 0;
+       else if (!(private_module->caps_capture.capabilities & TDM_CAPTURE_CAPABILITY_OUTPUT))
+               *has_capability = 0;
+       else
+               *has_capability = 1;
+
+       _pthread_mutex_unlock(&private_display->lock);
+
+       return ret;
+}
+
 EXTERN tdm_capture *
 tdm_output_create_capture(tdm_output *output, tdm_error *error)
 {
@@ -1160,39 +1740,175 @@ tdm_output_create_capture(tdm_output *output, tdm_error *error)
        return capture;
 }
 
-INTERN void
-tdm_output_call_change_handler_internal(tdm_private_output *private_output,
-                                                                               struct list_head *change_handler_list,
-                                                                               tdm_output_change_type type,
-                                                                               tdm_value value,
-                                                                               int no_check_thread_id)
+EXTERN tdm_hwc *
+tdm_output_get_hwc(tdm_output *output, tdm_error *error)
 {
-       tdm_private_display *private_display;
-       tdm_private_change_handler *change_handler = NULL;
+       tdm_private_hwc *private_hwc = NULL;
 
-       TDM_RETURN_IF_FAIL(TDM_MUTEX_IS_LOCKED());
-       TDM_RETURN_IF_FAIL(private_output);
+       OUTPUT_FUNC_ENTRY_ERROR();
 
-       private_display = private_output->private_display;
-       if (!tdm_thread_in_display_thread(syscall(SYS_gettid))) {
-               if (type & TDM_OUTPUT_CHANGE_CONNECTION)
-                       TDM_INFO("output(%d) changed: %s (%d)",
-                                        private_output->pipe, tdm_status_str(value.u32), value.u32);
-               if (type & TDM_OUTPUT_CHANGE_DPMS)
-                       TDM_INFO("output(%d) changed: dpms %s (%d)",
-                                        private_output->pipe, tdm_dpms_str(value.u32), value.u32);
+       _pthread_mutex_lock(&private_display->lock);
+
+       private_hwc = private_output->private_hwc;
+       if (!private_hwc) {
+               if (error)
+                       *error = TDM_ERROR_BAD_REQUEST;
+               _pthread_mutex_unlock(&private_display->lock);
+               return NULL;
        }
 
-       if (LIST_IS_EMPTY(change_handler_list))
-               return;
+       if (error)
+               *error = TDM_ERROR_NONE;
 
-       LIST_FOR_EACH_ENTRY(change_handler, change_handler_list, link) {
-               if (!no_check_thread_id && change_handler->owner_tid != syscall(SYS_gettid))
-                       TDM_NEVER_GET_HERE();
+       _pthread_mutex_unlock(&private_display->lock);
 
-               _pthread_mutex_unlock(&private_display->lock);
-               change_handler->func(private_output, type,
-                                                        value, change_handler->user_data);
-               _pthread_mutex_lock(&private_display->lock);
+       return private_hwc;
+}
+
+INTERN tdm_error
+tdm_output_choose_commit_per_vblank_mode(tdm_private_output *private_output, int mode)
+{
+       if (!private_output)
+               return TDM_ERROR_INVALID_PARAMETER;
+
+       if (mode < 0 || mode > 2)
+               return TDM_ERROR_INVALID_PARAMETER;
+
+       private_output->commit_per_vblank = mode;
+
+       if (private_output->commit_per_vblank == 0)
+               TDM_INFO("commit per vblank: disable");
+       else if (private_output->commit_per_vblank == 1)
+               TDM_INFO("commit per vblank: enable (1 layer)");
+       else if (private_output->commit_per_vblank == 2)
+               TDM_INFO("commit per vblank: enable (previous commit)");
+
+       return TDM_ERROR_NONE;
+}
+
+INTERN tdm_error
+tdm_output_set_available_mode(tdm_output *output, const tdm_output_mode *modes, int count)
+{
+       tdm_private_module *private_module = NULL;
+       tdm_func_output *func_output = NULL;
+
+       OUTPUT_FUNC_ENTRY();
+
+       TDM_RETURN_VAL_IF_FAIL(modes != NULL, TDM_ERROR_INVALID_PARAMETER);
+       TDM_RETURN_VAL_IF_FAIL(count != 0, TDM_ERROR_INVALID_PARAMETER);
+
+       private_module = private_output->private_module;
+       TDM_RETURN_VAL_IF_FAIL(private_module == private_display->virtual_module, TDM_ERROR_BAD_MODULE);
+
+       _pthread_mutex_lock(&private_display->lock);
+
+       func_output = &private_module->func_output;
+       if (func_output->output_set_available_mode)
+               ret = func_output->output_set_available_mode(private_output->output_backend, modes, count);
+
+       _pthread_mutex_unlock(&private_display->lock);
+
+       return ret;
+}
+
+INTERN tdm_error
+tdm_output_set_physical_size(tdm_output *output,  unsigned int mmwidth, unsigned int mmheight)
+{
+       tdm_private_module *private_module = NULL;
+       tdm_func_output *func_output = NULL;
+
+       OUTPUT_FUNC_ENTRY();
+
+       TDM_RETURN_VAL_IF_FAIL(mmwidth != 0, TDM_ERROR_INVALID_PARAMETER);
+       TDM_RETURN_VAL_IF_FAIL(mmheight != 0, TDM_ERROR_INVALID_PARAMETER);
+
+       private_module = private_output->private_module;
+       TDM_RETURN_VAL_IF_FAIL(private_module == private_display->virtual_module, TDM_ERROR_BAD_MODULE);
+
+       _pthread_mutex_lock(&private_display->lock);
+
+       func_output = &private_module->func_output;
+       if (func_output->output_set_physical_size)
+               ret = func_output->output_set_physical_size(private_output->output_backend, mmwidth, mmheight);
+
+       _pthread_mutex_unlock(&private_display->lock);
+
+       return ret;
+}
+
+INTERN tdm_error
+tdm_output_set_connect(tdm_output *output)
+{
+       tdm_private_module *private_module = NULL;
+       tdm_func_output *func_output = NULL;
+
+       OUTPUT_FUNC_ENTRY();
+
+       private_module = private_output->private_module;
+       TDM_RETURN_VAL_IF_FAIL(private_module == private_display->virtual_module, TDM_ERROR_BAD_MODULE);
+
+       _pthread_mutex_lock(&private_display->lock);
+
+       func_output = &private_module->func_output;
+       if (func_output->output_set_connect)
+               ret = func_output->output_set_connect(private_output->output_backend);
+
+       _pthread_mutex_unlock(&private_display->lock);
+
+       return ret;
+}
+
+INTERN tdm_error
+tdm_output_set_disconnect(tdm_output *output)
+{
+       tdm_private_module *private_module = NULL;
+       tdm_func_output *func_output = NULL;
+
+       OUTPUT_FUNC_ENTRY();
+
+       private_module = private_output->private_module;
+       TDM_RETURN_VAL_IF_FAIL(private_module == private_display->virtual_module, TDM_ERROR_BAD_MODULE);
+
+       _pthread_mutex_lock(&private_display->lock);
+
+       func_output = &private_module->func_output;
+       if (func_output->output_set_disconnect)
+               ret = func_output->output_set_disconnect(private_output->output_backend);
+
+       _pthread_mutex_unlock(&private_display->lock);
+
+       return ret;
+}
+
+INTERN tdm_error
+tdm_output_commit_done(tdm_output *output, tbm_surface_h buffer)
+{
+       tdm_private_layer *private_layer = NULL;
+       tdm_private_output_commit_handler *output_commit_handler = NULL;
+       static int sequence = 0;
+       tdm_private_output *private_output;
+       TDM_RETURN_VAL_IF_FAIL(output != NULL, TDM_ERROR_INVALID_PARAMETER); \
+       private_output = (tdm_private_output*)output;
+
+       LIST_FOR_EACH_ENTRY(private_layer, &private_output->layer_list, link) {
+               if (private_layer) break;
        }
+
+       private_layer->commiting_buffer = NULL;
+
+       LIST_FOR_EACH_ENTRY(output_commit_handler, &private_output->output_commit_handler_list, link) {
+               if (output_commit_handler) break;
+       }
+       TDM_RETURN_VAL_IF_FAIL(output_commit_handler, TDM_ERROR_INVALID_PARAMETER);
+
+       LIST_DEL(&output_commit_handler->link);
+
+       output_commit_handler->func(private_output,
+                                                               sequence++,
+                                                               0, 0,
+                                                               output_commit_handler->user_data);
+       free(output_commit_handler);
+
+       return TDM_ERROR_NONE;
 }
+/* LCOV_EXCL_STOP */