correct email address
[platform/core/uifw/libtdm.git] / src / tdm_capture.c
index 2737805..4b58fbd 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 CAPTURE_FUNC_ENTRY() \
        tdm_func_capture *func_capture; \
        tdm_private_display *private_display; \
+       tdm_private_module *private_module; \
        tdm_private_capture *private_capture; \
        tdm_error ret = TDM_ERROR_NONE; \
        TDM_RETURN_VAL_IF_FAIL(capture != NULL, TDM_ERROR_INVALID_PARAMETER); \
        private_capture = (tdm_private_capture*)capture; \
        private_display = private_capture->private_display; \
-       func_capture = &private_display->func_capture
+       private_module = private_capture->private_module; \
+       func_capture = &private_module->func_capture
 
-static tdm_error
-_tdm_capture_check_if_exist(tdm_private_capture *private_capture,
-                                                       tbm_surface_h buffer)
+static void
+_tdm_capture_print_list(struct list_head *list)
 {
-       tdm_buffer_info *buf_info = NULL;
-
-       LIST_FOR_EACH_ENTRY(buf_info, &private_capture->buffer_list, link) {
-               if (buf_info->buffer == buffer) {
-                       TDM_ERR("%p attached twice", buffer);
-                       return TDM_ERROR_BAD_REQUEST;
-               }
+       tdm_capture_private_buffer *b = NULL;
+       char str[512], *p;
+       int len = sizeof(str);
+
+       TDM_RETURN_IF_FAIL(list != NULL);
+
+       str[0] = '\0';
+       p = str;
+       LIST_FOR_EACH_ENTRY(b, list, link) {
+               if (len > 0) {
+                       tbm_bo bo = tbm_surface_internal_get_bo(b->buffer, 0);
+                       int flags = tbm_bo_get_flags(bo);
+                       int l = snprintf(p, len, " (%p[bo_flags:%x])", b->buffer, flags);
+                       p += l;
+                       len -= l;
+               } else
+                       break;
        }
 
-       LIST_FOR_EACH_ENTRY(buf_info, &private_capture->pending_buffer_list, link) {
-               if (buf_info->buffer == buffer) {
-                       TDM_ERR("%p attached twice", buffer);
-                       return TDM_ERROR_BAD_REQUEST;
-               }
+       TDM_INFO("\t %s", str);
+}
+
+static tdm_capture_private_buffer *
+_tdm_capture_find_tbm_buffer(struct list_head *list, tbm_surface_h buffer)
+{
+       tdm_capture_private_buffer *b = NULL, *bb = NULL;
+
+       LIST_FOR_EACH_ENTRY_SAFE(b, bb, list, link) {
+               if (b->buffer == buffer)
+                       return b;
        }
 
-       return TDM_ERROR_NONE;
+       return NULL;
 }
 
-INTERN void
-tdm_capture_cb_done(tdm_capture *capture_backend, tbm_surface_h buffer,
-                                       void *user_data)
+static tdm_capture_private_buffer *
+_tdm_capture_find_buffer(struct list_head *list, tdm_capture_private_buffer *capture_buffer)
 {
-       tdm_private_capture *private_capture = user_data;
-       tdm_private_display *private_display = private_capture->private_display;
-       tdm_buffer_info *buf_info;
-       tbm_surface_h first_entry;
+       tdm_capture_private_buffer *b = NULL, *bb = NULL;
 
-       TDM_RETURN_IF_FAIL(TDM_MUTEX_IS_LOCKED());
+       LIST_FOR_EACH_ENTRY_SAFE(b, bb, list, link) {
+               if (b == capture_buffer)
+                       return b;
+       }
 
-       if (private_capture->owner_tid != syscall(SYS_gettid)) {
-               tdm_thread_cb_capture_done capture_done;
-               tdm_error ret;
+       return NULL;
+}
 
-               capture_done.base.type = TDM_THREAD_CB_PP_DONE;
-               capture_done.base.length = sizeof capture_done;
-               capture_done.capture_stamp = private_capture->stamp;
-               capture_done.buffer = buffer;
-               capture_done.user_data = user_data;
+static void
+_tdm_capture_thread_cb_done(tdm_private_display *private_display, void *object, tdm_thread_cb_base *cb_base, void *user_data)
+{
+       tdm_thread_cb_capture_done *capture_done = (tdm_thread_cb_capture_done *)cb_base;
+       tdm_private_capture *private_capture = object;
+       tdm_capture_private_buffer *capture_buffer = NULL, *first_entry = NULL;
+       tbm_surface_h buffer;
 
-               ret = tdm_thread_send_cb(private_display->private_loop, &capture_done.base);
-               TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
+       TDM_RETURN_IF_FAIL(TDM_MUTEX_IS_LOCKED());
 
-               return;
-       }
+       assert(private_capture->owner_tid == syscall(SYS_gettid));
 
-       if (private_capture->owner_tid != syscall(SYS_gettid))
-               TDM_NEVER_GET_HERE();
+       buffer = capture_done->buffer;
 
        if (tdm_debug_dump & TDM_DUMP_FLAG_CAPTURE) {
+               /* LCOV_EXCL_START */
                char str[TDM_PATH_LEN];
                static int i;
                snprintf(str, TDM_PATH_LEN, "capture_%03d", i++);
-               tdm_helper_dump_buffer_str(buffer, str);
+               tdm_helper_dump_buffer_str(buffer, tdm_debug_dump_dir, str);
+               /* LCOV_EXCL_STOP */
        }
 
        if (tdm_debug_module & TDM_DEBUG_BUFFER)
                TDM_INFO("capture(%p) done: %p", private_capture, buffer);
 
-       first_entry = tdm_buffer_list_get_first_entry(&private_capture->buffer_list);
-       if (first_entry != buffer)
-               TDM_ERR("%p is skipped", first_entry);
+       if (tdm_ttrace_module & TDM_TTRACE_CAPTURE) {
+               tbm_bo bo = tbm_surface_internal_get_bo(buffer, 0);
+               TDM_TRACE_ASYNC_END((intptr_t)private_capture, "[CAPTURE] %d", tbm_bo_export(bo));
+       }
 
-       if ((buf_info = tdm_buffer_get_info(buffer)))
-               LIST_DEL(&buf_info->link);
+       if (!LIST_IS_EMPTY(&private_capture->buffer_list)) {
+               first_entry = container_of((&private_capture->buffer_list)->next, capture_buffer, link);
+               if (first_entry->buffer != buffer)
+                       TDM_ERR("buffer(%p) is skipped", first_entry->buffer);
+       }
 
-       _pthread_mutex_unlock(&private_display->lock);
-       tdm_buffer_unref_backend(buffer);
-       _pthread_mutex_lock(&private_display->lock);
+       if ((capture_buffer = _tdm_capture_find_tbm_buffer(&private_capture->buffer_list, buffer))) {
+               LIST_DEL(&capture_buffer->link);
+               LIST_DELINIT(&capture_buffer->commit_link);
+
+               _pthread_mutex_unlock(&private_display->lock);
+               if (private_capture->done_func)
+                       private_capture->done_func(capture_buffer, buffer, private_capture->done_user_data);
+               tdm_buffer_unref_backend(buffer);
+               _pthread_mutex_lock(&private_display->lock);
+
+               free(capture_buffer);
+       }
 }
 
-INTERN tdm_private_capture *
-tdm_capture_find_stamp(tdm_private_display *private_display, unsigned long stamp)
+static void
+_tdm_capture_cb_done(tdm_capture *capture_module, tbm_surface_h buffer, void *user_data)
 {
+       tdm_private_capture *private_capture = user_data;
+       tdm_thread_cb_capture_done capture_done;
+       tdm_error ret;
+
+       TDM_RETURN_IF_FAIL(TDM_MUTEX_IS_LOCKED());
+
+       memset(&capture_done, 0, sizeof capture_done);
+       capture_done.base.type = TDM_THREAD_CB_CAPTURE_DONE;
+       capture_done.base.length = sizeof capture_done;
+       capture_done.base.object_stamp = private_capture->stamp;
+       capture_done.base.data = NULL;
+       capture_done.base.sync = 0;
+       capture_done.buffer = buffer;
+
+       ret = tdm_thread_cb_call(private_capture, &capture_done.base, 1);
+       TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
+}
+
+static void *
+_tdm_capture_find_object(tdm_private_display *private_display, double stamp)
+{
+       tdm_private_module *private_module = NULL;
        tdm_private_capture *private_capture = NULL;
 
        TDM_RETURN_VAL_IF_FAIL(TDM_MUTEX_IS_LOCKED(), NULL);
 
-       LIST_FOR_EACH_ENTRY(private_capture, &private_display->capture_list, link) {
-               if (private_capture->stamp == stamp)
-                       return private_capture;
+       LIST_FOR_EACH_ENTRY(private_module, &private_display->module_list, link) {
+               LIST_FOR_EACH_ENTRY(private_capture, &private_module->capture_list, display_link) {
+                       if (private_capture->stamp == stamp)
+                               return private_capture;
+               }
        }
 
        return NULL;
 }
 
+INTERN tdm_error
+tdm_capture_init(tdm_private_display *private_display)
+{
+       tdm_thread_cb_set_find_func(TDM_THREAD_CB_CAPTURE_DONE, _tdm_capture_find_object);
+
+       return TDM_ERROR_NONE;
+}
+
 INTERN tdm_private_capture *
 tdm_capture_create_output_internal(tdm_private_output *private_output,
                                                                   tdm_error *error)
 {
        tdm_private_display *private_display = private_output->private_display;
-       tdm_func_output *func_output = &private_display->func_output;
-       tdm_func_capture *func_capture = &private_display->func_capture;
+       tdm_private_module *private_module = private_output->private_module;
+       tdm_func_output *func_output = &private_module->func_output;
+       tdm_func_capture *func_capture = &private_module->func_capture;
        tdm_private_capture *private_capture = NULL;
-       tdm_capture *capture_backend = NULL;
+       tdm_capture *capture_module = NULL;
        tdm_error ret = TDM_ERROR_NONE;
 
        TDM_RETURN_VAL_IF_FAIL(TDM_MUTEX_IS_LOCKED(), NULL);
 
-       if (!(private_display->capabilities & TDM_DISPLAY_CAPABILITY_CAPTURE)) {
-               TDM_ERR("no capture capability");
+       if (!(private_module->capabilities & TDM_DISPLAY_CAPABILITY_CAPTURE)) {
+               /* LCOV_EXCL_START */
+               TDM_ERR("backend(%s) no capture capability", private_module->module_data->name);
+               if (error)
+                       *error = TDM_ERROR_NO_CAPABILITY;
+               return NULL;
+               /* LCOV_EXCL_STOP */
+       }
+
+       if (!(private_module->caps_capture.capabilities & TDM_CAPTURE_CAPABILITY_OUTPUT)) {
+               /* LCOV_EXCL_START */
+               TDM_ERR("backend(%s) no output capture capability", private_module->module_data->name);
                if (error)
                        *error = TDM_ERROR_NO_CAPABILITY;
                return NULL;
+               /* LCOV_EXCL_STOP */
        }
 
-       capture_backend = func_output->output_create_capture(
+       capture_module = func_output->output_create_capture(
                                                  private_output->output_backend, &ret);
        if (ret != TDM_ERROR_NONE) {
+               /* LCOV_EXCL_START */
                if (error)
                        *error = ret;
                return NULL;
+               /* LCOV_EXCL_STOP */
        }
 
        private_capture = calloc(1, sizeof(tdm_private_capture));
        if (!private_capture) {
+               /* LCOV_EXCL_START */
                TDM_ERR("failed: alloc memory");
-               func_capture->capture_destroy(capture_backend);
+               func_capture->capture_destroy(capture_module);
                if (error)
                        *error = TDM_ERROR_OUT_OF_MEMORY;
                return NULL;
+               /* LCOV_EXCL_STOP */
        }
 
-       ret = func_capture->capture_set_done_handler(capture_backend,
-                       tdm_capture_cb_done, private_capture);
+       ret = tdm_thread_cb_add(private_capture, TDM_THREAD_CB_CAPTURE_DONE, NULL, _tdm_capture_thread_cb_done, NULL);
        if (ret != TDM_ERROR_NONE) {
+               TDM_ERR("capture tdm_thread_cb_add failed");
+               func_capture->capture_destroy(capture_module);
+               free(private_capture);
+               if (error)
+                       *error = ret;
+               return NULL;
+       }
+
+       ret = func_capture->capture_set_done_handler(capture_module, _tdm_capture_cb_done, private_capture);
+       if (ret != TDM_ERROR_NONE) {
+               /* LCOV_EXCL_START */
                TDM_ERR("capture(%p) set capture_done_handler failed", private_capture);
-               func_capture->capture_destroy(capture_backend);
+               func_capture->capture_destroy(capture_module);
+               free(private_capture);
                if (error)
                        *error = ret;
                return NULL;
+               /* LCOV_EXCL_STOP */
        }
 
-       private_capture->stamp = tdm_helper_get_time_in_millis();
-       while (tdm_capture_find_stamp(private_display, private_capture->stamp))
+       private_capture->stamp = tdm_helper_get_time();
+       while (_tdm_capture_find_object(private_display, private_capture->stamp))
                private_capture->stamp++;
 
        LIST_ADD(&private_capture->link, &private_output->capture_list);
-       LIST_ADD(&private_capture->display_link, &private_display->capture_list);
+       LIST_ADD(&private_capture->display_link, &private_module->capture_list);
+
+       private_capture->private_module = private_module;
 
        private_capture->target = TDM_CAPTURE_TARGET_OUTPUT;
        private_capture->private_display = private_display;
        private_capture->private_output = private_output;
        private_capture->private_layer = NULL;
-       private_capture->capture_backend = capture_backend;
+       private_capture->capture_module = capture_module;
        private_capture->owner_tid = syscall(SYS_gettid);
 
        LIST_INITHEAD(&private_capture->pending_buffer_list);
        LIST_INITHEAD(&private_capture->buffer_list);
 
+       TDM_DBG("capture(%p) create", private_capture);
+
        if (error)
                *error = TDM_ERROR_NONE;
 
        return private_capture;
 }
 
+/* LCOV_EXCL_START */
 INTERN tdm_private_capture *
 tdm_capture_create_layer_internal(tdm_private_layer *private_layer,
                                                                  tdm_error *error)
 {
        tdm_private_output *private_output = private_layer->private_output;
+       tdm_private_module *private_module = private_layer->private_module;
        tdm_private_display *private_display = private_output->private_display;
-       tdm_func_layer *func_layer = &private_display->func_layer;
-       tdm_func_capture *func_capture = &private_display->func_capture;
+       tdm_func_layer *func_layer = &private_module->func_layer;
+       tdm_func_capture *func_capture = &private_module->func_capture;
        tdm_private_capture *private_capture = NULL;
-       tdm_capture *capture_backend = NULL;
+       tdm_capture *capture_module = NULL;
        tdm_error ret = TDM_ERROR_NONE;
 
        TDM_RETURN_VAL_IF_FAIL(TDM_MUTEX_IS_LOCKED(), NULL);
 
-       if (!(private_display->capabilities & TDM_DISPLAY_CAPABILITY_CAPTURE)) {
-               TDM_ERR("no capture capability");
+       if (!(private_module->capabilities & TDM_DISPLAY_CAPABILITY_CAPTURE)) {
+               TDM_ERR("backend(%s) no capture capability", private_module->module_data->name);
                if (error)
                        *error = TDM_ERROR_NO_CAPABILITY;
                return NULL;
        }
 
-       capture_backend = func_layer->layer_create_capture(private_layer->layer_backend,
+       if (!(private_module->caps_capture.capabilities & TDM_CAPTURE_CAPABILITY_LAYER)) {
+               TDM_ERR("backend(%s) no layer capture capability", private_module->module_data->name);
+               if (error)
+                       *error = TDM_ERROR_NO_CAPABILITY;
+               return NULL;
+       }
+
+       capture_module = func_layer->layer_create_capture(private_layer->layer_backend,
                                          &ret);
        if (ret != TDM_ERROR_NONE)
                return NULL;
@@ -241,41 +340,48 @@ tdm_capture_create_layer_internal(tdm_private_layer *private_layer,
        private_capture = calloc(1, sizeof(tdm_private_capture));
        if (!private_capture) {
                TDM_ERR("failed: alloc memory");
-               func_capture->capture_destroy(capture_backend);
+               func_capture->capture_destroy(capture_module);
                if (error)
                        *error = TDM_ERROR_OUT_OF_MEMORY;
                return NULL;
        }
 
-       private_capture->stamp = tdm_helper_get_time_in_millis();
-       while (tdm_capture_find_stamp(private_display, private_capture->stamp))
+       private_capture->stamp = tdm_helper_get_time();
+       while (_tdm_capture_find_object(private_display, private_capture->stamp))
                private_capture->stamp++;
 
        LIST_ADD(&private_capture->link, &private_layer->capture_list);
-       LIST_ADD(&private_capture->display_link, &private_display->capture_list);
+       LIST_ADD(&private_capture->display_link, &private_module->capture_list);
+
+       private_capture->private_module = private_module;
 
        private_capture->target = TDM_CAPTURE_TARGET_LAYER;
        private_capture->private_display = private_display;
        private_capture->private_output = private_output;
        private_capture->private_layer = private_layer;
-       private_capture->capture_backend = capture_backend;
+       private_capture->capture_module = capture_module;
        private_capture->owner_tid = syscall(SYS_gettid);
 
        LIST_INITHEAD(&private_capture->pending_buffer_list);
        LIST_INITHEAD(&private_capture->buffer_list);
 
+       TDM_DBG("capture(%p) create", private_capture);
+
        if (error)
                *error = TDM_ERROR_NONE;
 
        return private_capture;
 }
+/* LCOV_EXCL_STOP */
 
 INTERN void
 tdm_capture_destroy_internal(tdm_private_capture *private_capture)
 {
        tdm_private_display *private_display;
+       tdm_private_module *private_module;
        tdm_func_capture *func_capture;
-       tdm_buffer_info *b = NULL, *bb = NULL;
+       tdm_capture_private_buffer *b = NULL, *bb = NULL;
+       struct list_head clone_list;
 
        TDM_RETURN_IF_FAIL(TDM_MUTEX_IS_LOCKED());
 
@@ -284,34 +390,63 @@ tdm_capture_destroy_internal(tdm_private_capture *private_capture)
 
        private_display = private_capture->private_display;
 
+       tdm_thread_cb_remove(private_capture, TDM_THREAD_CB_CAPTURE_DONE, NULL, _tdm_capture_thread_cb_done, NULL);
+
        LIST_DEL(&private_capture->link);
        LIST_DEL(&private_capture->display_link);
 
-       func_capture = &private_capture->private_display->func_capture;
-       func_capture->capture_destroy(private_capture->capture_backend);
+       private_module = private_capture->private_module;
+       func_capture = &private_module->func_capture;
+       func_capture->capture_destroy(private_capture->capture_module);
 
        if (!LIST_IS_EMPTY(&private_capture->pending_buffer_list)) {
                TDM_WRN("capture(%p) not finished:", private_capture);
-               tdm_buffer_list_dump(&private_capture->pending_buffer_list);
+               _tdm_capture_print_list(&private_capture->pending_buffer_list);
 
+               LIST_INITHEAD(&clone_list);
                LIST_FOR_EACH_ENTRY_SAFE(b, bb, &private_capture->pending_buffer_list, link) {
                        LIST_DEL(&b->link);
-                       _pthread_mutex_unlock(&private_display->lock);
+                       LIST_ADDTAIL(&b->link, &clone_list);
+               }
+
+               _pthread_mutex_unlock(&private_display->lock);
+               LIST_FOR_EACH_ENTRY_SAFE(b, bb, &clone_list, link) {
+                       LIST_DEL(&b->link);
+
+                       if (tdm_ttrace_module & TDM_TTRACE_CAPTURE) {
+                               tbm_bo bo = tbm_surface_internal_get_bo(b->buffer, 0);
+                               TDM_TRACE_ASYNC_END((intptr_t)private_capture, "[CAPTURE] %d", tbm_bo_export(bo));
+                       }
+
                        tdm_buffer_unref_backend(b->buffer);
-                       _pthread_mutex_lock(&private_display->lock);
+                       free(b);
                }
+               _pthread_mutex_lock(&private_display->lock);
        }
 
        if (!LIST_IS_EMPTY(&private_capture->buffer_list)) {
                TDM_WRN("capture(%p) not finished:", private_capture);
-               tdm_buffer_list_dump(&private_capture->buffer_list);
+               _tdm_capture_print_list(&private_capture->buffer_list);
 
+               LIST_INITHEAD(&clone_list);
                LIST_FOR_EACH_ENTRY_SAFE(b, bb, &private_capture->buffer_list, link) {
                        LIST_DEL(&b->link);
-                       _pthread_mutex_unlock(&private_display->lock);
+                       LIST_ADDTAIL(&b->link, &clone_list);
+               }
+
+               _pthread_mutex_unlock(&private_display->lock);
+               LIST_FOR_EACH_ENTRY_SAFE(b, bb, &clone_list, link) {
+                       LIST_DEL(&b->link);
+
+                       if (tdm_ttrace_module & TDM_TTRACE_CAPTURE) {
+                               tbm_bo bo = tbm_surface_internal_get_bo(b->buffer, 0);
+                               TDM_TRACE_ASYNC_END((intptr_t)private_capture, "[CAPTURE] %d", tbm_bo_export(bo));
+                       }
+
                        tdm_buffer_unref_backend(b->buffer);
-                       _pthread_mutex_lock(&private_display->lock);
+                       free(b);
                }
+               _pthread_mutex_lock(&private_display->lock);
        }
 
        private_capture->stamp = 0;
@@ -327,6 +462,8 @@ tdm_capture_destroy(tdm_capture *capture)
        if (!private_capture)
                return;
 
+       TDM_DBG("capture(%p) destroy", private_capture);
+
        private_display = private_capture->private_display;
 
        _pthread_mutex_lock(&private_display->lock);
@@ -340,16 +477,32 @@ tdm_capture_set_info(tdm_capture *capture, tdm_info_capture *info)
        CAPTURE_FUNC_ENTRY();
 
        TDM_RETURN_VAL_IF_FAIL(info != NULL, TDM_ERROR_INVALID_PARAMETER);
+       TDM_RETURN_VAL_IF_FAIL(info->type != 0, TDM_ERROR_INVALID_PARAMETER);
 
        _pthread_mutex_lock(&private_display->lock);
 
        if (!func_capture->capture_set_info) {
+               /* LCOV_EXCL_START */
                _pthread_mutex_unlock(&private_display->lock);
                TDM_DBG("failed: not implemented!!");
                return TDM_ERROR_NOT_IMPLEMENTED;
+               /* LCOV_EXCL_STOP */
        }
 
-       ret = func_capture->capture_set_info(private_capture->capture_backend, info);
+       if (info->type == TDM_CAPTURE_TYPE_STREAM && info->frequency == 0) {
+               tdm_private_output *private_output = private_capture->private_output;
+               info->frequency = private_output->current_mode->vrefresh;
+       }
+
+       TDM_INFO("capture(%p) info: dst(%ux%u %u,%u %ux%u %c%c%c%c) trans(%d) type(%d) freq(%d) flags(%x)",
+                        private_capture,
+                        info->dst_config.size.h, info->dst_config.size.v,
+                        info->dst_config.pos.x, info->dst_config.pos.y,
+                        info->dst_config.pos.w, info->dst_config.pos.h,
+                        FOURCC_STR(info->dst_config.format),
+                        info->transform, info->type, info->frequency, info->flags);
+
+       ret = func_capture->capture_set_info(private_capture->capture_module, info);
        TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
 
        private_capture->info = *info;
@@ -360,8 +513,34 @@ tdm_capture_set_info(tdm_capture *capture, tdm_info_capture *info)
 }
 
 EXTERN tdm_error
+tdm_capture_set_done_handler(tdm_capture *capture, tdm_capture_done_handler func, void *user_data)
+{
+       tdm_private_display *private_display;
+       tdm_private_capture *private_capture;
+       tdm_error ret = TDM_ERROR_NONE;
+
+       TDM_RETURN_VAL_IF_FAIL(capture != NULL, TDM_ERROR_INVALID_PARAMETER);
+
+       private_capture = (tdm_private_capture*)capture;
+       private_display = private_capture->private_display;
+
+       TDM_RETURN_VAL_IF_FAIL(func != NULL, TDM_ERROR_INVALID_PARAMETER);
+
+       _pthread_mutex_lock(&private_display->lock);
+
+       private_capture->done_func = func;
+       private_capture->done_user_data = user_data;
+
+       _pthread_mutex_unlock(&private_display->lock);
+
+       return ret;
+}
+
+EXTERN tdm_error
 tdm_capture_attach(tdm_capture *capture, tbm_surface_h buffer)
 {
+       tdm_capture_private_buffer *capture_buffer;
+
        CAPTURE_FUNC_ENTRY();
 
        TDM_RETURN_VAL_IF_FAIL(buffer != NULL, TDM_ERROR_INVALID_PARAMETER);
@@ -369,30 +548,60 @@ tdm_capture_attach(tdm_capture *capture, tbm_surface_h buffer)
        _pthread_mutex_lock(&private_display->lock);
 
        if (!func_capture->capture_attach) {
+               /* LCOV_EXCL_START */
                _pthread_mutex_unlock(&private_display->lock);
                TDM_DBG("failed: not implemented!!");
                return TDM_ERROR_NOT_IMPLEMENTED;
+               /* LCOV_EXCL_STOP */
        }
 
-       ret = _tdm_capture_check_if_exist(private_capture, buffer);
-       if (ret != TDM_ERROR_NONE) {
+       if (tdm_module_check_abi(private_module, 1, 2) &&
+               private_module->caps_capture.max_attach_count > 0) {
+               /* LCOV_EXCL_START */
+               int length = LIST_LENGTH(&private_capture->pending_buffer_list) +
+                                        LIST_LENGTH(&private_capture->buffer_list);
+               if (length >= private_module->caps_capture.max_attach_count) {
+                       _pthread_mutex_unlock(&private_display->lock);
+                       TDM_DBG("failed: backend(%s) too many attached!! max_attach_count(%d)",
+                                       private_module->module_data->name, private_module->caps_capture.max_attach_count);
+                       return TDM_ERROR_BAD_REQUEST;
+               }
+               /* LCOV_EXCL_STOP */
+       }
+
+       capture_buffer = calloc(1, sizeof * capture_buffer);
+       if (!capture_buffer) {
+               /* LCOV_EXCL_START */
                _pthread_mutex_unlock(&private_display->lock);
-               return ret;
+               TDM_ERR("alloc failed");
+               return TDM_ERROR_OUT_OF_MEMORY;
+               /* LCOV_EXCL_STOP */
        }
 
-       ret = func_capture->capture_attach(private_capture->capture_backend, buffer);
+       ret = func_capture->capture_attach(private_capture->capture_module, buffer);
        TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
 
-       if (ret == TDM_ERROR_NONE) {
-               tdm_buffer_info *buf_info;
+       if (ret != TDM_ERROR_NONE) {
+               /* LCOV_EXCL_START */
+               free(capture_buffer);
+               _pthread_mutex_unlock(&private_display->lock);
+               TDM_ERR("attach failed");
+               return ret;
+               /* LCOV_EXCL_STOP */
+       }
 
-               if ((buf_info = tdm_buffer_get_info(buffer)))
-                       LIST_ADDTAIL(&buf_info->link, &private_capture->pending_buffer_list);
+       LIST_ADDTAIL(&capture_buffer->link, &private_capture->pending_buffer_list);
+       capture_buffer->buffer = tdm_buffer_ref_backend(buffer);
 
-               if (tdm_debug_module & TDM_DEBUG_BUFFER) {
-                       TDM_INFO("capture(%p) attached:", private_capture);
-                       tdm_buffer_list_dump(&private_capture->buffer_list);
-               }
+       if (tdm_debug_module & TDM_DEBUG_BUFFER) {
+               TDM_INFO("capture(%p) attached:", private_capture);
+               _tdm_capture_print_list(&private_capture->pending_buffer_list);
+               _tdm_capture_print_list(&private_capture->buffer_list);
+       }
+
+       if (tdm_ttrace_module & TDM_TTRACE_CAPTURE) {
+               tbm_bo bo = tbm_surface_internal_get_bo(buffer, 0);
+               TDM_TRACE_ASYNC_BEGIN((intptr_t)capture, "[CAPTURE] %d", tbm_bo_export(bo));
        }
 
        _pthread_mutex_unlock(&private_display->lock);
@@ -403,15 +612,18 @@ tdm_capture_attach(tdm_capture *capture, tbm_surface_h buffer)
 EXTERN tdm_error
 tdm_capture_commit(tdm_capture *capture)
 {
-       tdm_buffer_info *b = NULL, *bb = NULL;
+       tdm_capture_private_buffer *b = NULL, *bb = NULL;
        tdm_private_output *private_output;
+       struct list_head commit_buffer_list;
 
        CAPTURE_FUNC_ENTRY();
 
        _pthread_mutex_lock(&private_display->lock);
 
        private_output = private_capture->private_output;
-       if (private_output->current_dpms_value > TDM_OUTPUT_DPMS_ON) {
+
+       /* TODO: possible when standby mode? can't decide it yet. no scenario. */
+       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);
@@ -419,23 +631,53 @@ tdm_capture_commit(tdm_capture *capture)
        }
 
        if (!func_capture->capture_commit) {
+               /* LCOV_EXCL_START */
                _pthread_mutex_unlock(&private_display->lock);
                TDM_DBG("failed: not implemented!!");
                return TDM_ERROR_NOT_IMPLEMENTED;
+               /* LCOV_EXCL_STOP */
+       }
+
+       LIST_INITHEAD(&commit_buffer_list);
+
+       LIST_FOR_EACH_ENTRY_SAFE(b, bb, &private_capture->pending_buffer_list, link) {
+               LIST_DEL(&b->link);
+               LIST_ADDTAIL(&b->link, &private_capture->buffer_list);
+               LIST_ADDTAIL(&b->commit_link, &commit_buffer_list);
+       }
+
+       if (tdm_debug_module & TDM_DEBUG_BUFFER) {
+               TDM_INFO("capture(%p) committed:", private_capture);
+               _tdm_capture_print_list(&private_capture->pending_buffer_list);
+               _tdm_capture_print_list(&private_capture->buffer_list);
        }
 
-       ret = func_capture->capture_commit(private_capture->capture_backend);
+       ret = func_capture->capture_commit(private_capture->capture_module);
        TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
 
-       if (ret == TDM_ERROR_NONE) {
-               LIST_FOR_EACH_ENTRY_SAFE(b, bb, &private_capture->pending_buffer_list, link) {
+       LIST_FOR_EACH_ENTRY_SAFE(b, bb, &commit_buffer_list, commit_link) {
+               LIST_DELINIT(&b->commit_link);
+
+               if (!_tdm_capture_find_buffer(&private_capture->buffer_list, b))
+                       continue;
+
+               if (ret != TDM_ERROR_NONE) {
+                       /* LCOV_EXCL_START */
+                       /* Not to call the user release handler when failed.
+                        * Do we have to call this function here really?
+                        * User better use set_done_handler to know when pp is done. Using
+                        * buffer_release_handler is not good.
+                        */
+                       tdm_buffer_remove_release_handler_internal(b->buffer);
+
+                       _pthread_mutex_unlock(&private_display->lock);
+                       tdm_buffer_unref_backend(b->buffer);
+                       _pthread_mutex_lock(&private_display->lock);
                        LIST_DEL(&b->link);
-                       tdm_buffer_ref_backend(b->buffer);
-                       LIST_ADDTAIL(&b->link, &private_capture->buffer_list);
+
+                       free(b);
+                       /* LCOV_EXCL_STOP */
                }
-       } else {
-               LIST_FOR_EACH_ENTRY_SAFE(b, bb, &private_capture->pending_buffer_list, link)
-                       LIST_DEL(&b->link);
        }
 
        _pthread_mutex_unlock(&private_display->lock);