tbm_bufmgr: lock/unlock tbm_bufmgr_mutex at tbm_bufmgr function
[platform/core/uifw/libtbm.git] / src / tbm_bo.c
index cb18292..1987327 100644 (file)
@@ -34,70 +34,26 @@ SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 #include "tbm_bufmgr_int.h"
 #include "list.h"
 
-static pthread_mutex_t tbm_bo_lock = PTHREAD_MUTEX_INITIALIZER;
-static __thread tbm_error_e tbm_last_error = TBM_ERROR_NONE;
-static void _tbm_bo_mutex_unlock(void);
-
 /* check condition */
 #define TBM_BO_RETURN_IF_FAIL(cond) {\
        if (!(cond)) {\
-               TBM_LOG_E("'%s' failed.\n", #cond);\
-               _tbm_bo_mutex_unlock();\
+               TBM_ERR("'%s' failed.\n", #cond);\
+               _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER);\
+               _tbm_bufmgr_mutex_unlock();\
                return;\
        } \
 }
 
 #define TBM_BO_RETURN_VAL_IF_FAIL(cond, val) {\
        if (!(cond)) {\
-               TBM_LOG_E("'%s' failed.\n", #cond);\
-               _tbm_bo_mutex_unlock();\
+               TBM_ERR("'%s' failed.\n", #cond);\
+               _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER);\
+               _tbm_bufmgr_mutex_unlock();\
                return val;\
        } \
 }
 
-static void
-_tbm_set_last_result(tbm_error_e err)
-{
-       tbm_last_error = err;
-}
-
-/* LCOV_EXCL_START */
-static bool
-_tbm_bo_mutex_init(void)
-{
-       static bool tbm_bo_mutex_init = false;
-
-       if (tbm_bo_mutex_init)
-               return true;
-
-       if (pthread_mutex_init(&tbm_bo_lock, NULL)) {
-               TBM_LOG_E("fail: Cannot pthread_mutex_init for tbm_bo_lock.\n");
-               return false;
-       }
-
-       tbm_bo_mutex_init = true;
-
-       return true;
-}
-
-static void
-_tbm_bo_mutex_lock(void)
-{
-       if (!_tbm_bo_mutex_init()) {
-               TBM_LOG_E("fail: _tbm_bo_mutex_init()\n");
-               return;
-       }
-
-       pthread_mutex_lock(&tbm_bo_lock);
-}
-
-static void
-_tbm_bo_mutex_unlock(void)
-{
-       pthread_mutex_unlock(&tbm_bo_lock);
-}
-
-static char *
+char *
 _tbm_flag_to_str(int f)
 {
        static char str[255];
@@ -130,22 +86,10 @@ _tbm_flag_to_str(int f)
        return str;
 }
 
-static void
-_tbm_util_check_bo_cnt(tbm_bufmgr bufmgr)
-{
-       static int last_chk_bo_cnt = 0;
-
-       if ((bufmgr->bo_cnt >= 500) && ((bufmgr->bo_cnt % 20) == 0) &&
-               (bufmgr->bo_cnt > last_chk_bo_cnt)) {
-               TBM_DEBUG("============TBM BO CNT DEBUG: bo_cnt=%d\n", bufmgr->bo_cnt);
-               tbm_bufmgr_debug_show(bufmgr);
-               last_chk_bo_cnt = bufmgr->bo_cnt;
-       }
-}
 /* LCOV_EXCL_STOP */
 
-tbm_user_data
-*user_data_lookup(struct list_head *user_data_list, unsigned long key)
+tbm_user_data *
+user_data_lookup(struct list_head *user_data_list, unsigned long key)
 {
        tbm_user_data *old_data = NULL;
 
@@ -160,15 +104,16 @@ tbm_user_data
        return NULL;
 }
 
-tbm_user_data
-*user_data_create(unsigned long key, tbm_data_free data_free_func)
+tbm_user_data *
+user_data_create(unsigned long key, tbm_data_free data_free_func)
 {
        tbm_user_data *user_data;
 
        user_data = calloc(1, sizeof(tbm_user_data));
        if (!user_data) {
                /* LCOV_EXCL_START */
-               TBM_LOG_E("fail to allocate an user_date\n");
+               TBM_ERR("fail to allocate an user_date\n");
+               _tbm_set_last_result(TBM_ERROR_OUT_OF_MEMORY);
                return NULL;
                /* LCOV_EXCL_STOP */
        }
@@ -193,10 +138,24 @@ user_data_delete(tbm_user_data *user_data)
 static int
 _bo_lock(tbm_bo bo, int device, int opt)
 {
-       int ret = 1;
+       tbm_error_e error;
+       int ret;
 
-       if (bo->bufmgr->backend->bo_lock)
-               ret = bo->bufmgr->backend->bo_lock(bo, device, opt);
+       error = tbm_bo_data_lock(bo->bo_data, device, opt);
+       _tbm_set_last_result(error);
+       switch(error) {
+               case TBM_ERROR_NONE:
+                       ret = 1;
+                       break;
+               case TBM_ERROR_NOT_SUPPORTED:
+                       ret = 1;
+                       _tbm_set_last_result(TBM_ERROR_NONE);
+                       break;
+               default:
+                       ret = 0;
+                       TBM_WRN("fail to lock. error:%d", error);
+                       break;
+       }
 
        return ret;
 }
@@ -204,8 +163,20 @@ _bo_lock(tbm_bo bo, int device, int opt)
 static void
 _bo_unlock(tbm_bo bo)
 {
-       if (bo->bufmgr->backend->bo_unlock)
-               bo->bufmgr->backend->bo_unlock(bo);
+       tbm_error_e error;
+
+       error = tbm_bo_data_unlock(bo->bo_data);
+       _tbm_set_last_result(error);
+       switch(error) {
+               case TBM_ERROR_NONE:
+                       break;
+               case TBM_ERROR_NOT_SUPPORTED:
+                       _tbm_set_last_result(TBM_ERROR_NONE);
+                       break;
+               default:
+                       TBM_WRN("fail to lock. error:%d", error);
+                       break;
+       }
 }
 
 static int
@@ -221,7 +192,7 @@ _tbm_bo_lock(tbm_bo bo, int device, int opt)
                return 1;
 
        if (bo->lock_cnt < 0) {
-               TBM_LOG_E("error bo:%p LOCK_CNT=%d\n",
+               TBM_ERR("error bo:%p LOCK_CNT=%d\n",
                        bo, bo->lock_cnt);
                return 0;
        }
@@ -231,29 +202,29 @@ _tbm_bo_lock(tbm_bo bo, int device, int opt)
        switch (bo->bufmgr->bo_lock_type) {
        case TBM_BUFMGR_BO_LOCK_TYPE_ONCE:
                if (bo->lock_cnt == 0) {
-                       _tbm_bo_mutex_unlock();
+                       _tbm_bufmgr_mutex_unlock();
                        ret = _bo_lock(bo, device, opt);
-                       _tbm_bo_mutex_lock();
+                       _tbm_bufmgr_mutex_lock();
                        if (ret)
                                bo->lock_cnt++;
                } else
                        ret = 1;
                break;
        case TBM_BUFMGR_BO_LOCK_TYPE_ALWAYS:
-               _tbm_bo_mutex_unlock();
+               _tbm_bufmgr_mutex_unlock();
                ret = _bo_lock(bo, device, opt);
-               _tbm_bo_mutex_lock();
+               _tbm_bufmgr_mutex_lock();
                if (ret)
                        bo->lock_cnt++;
                break;
        default:
-               TBM_LOG_E("error bo:%p bo_lock_type[%d] is wrong.\n",
+               TBM_ERR("error bo:%p bo_lock_type[%d] is wrong.\n",
                                bo, bo->bufmgr->bo_lock_type);
                ret = 0;
                break;
        }
 
-       TBM_DBG_LOCK(">> LOCK bo:%p(%d->%d)\n", bo, old, bo->lock_cnt);
+       TBM_DBG(">> LOCK bo:%p(%d->%d)\n", bo, old, bo->lock_cnt);
 
        return ret;
 }
@@ -284,7 +255,7 @@ _tbm_bo_unlock(tbm_bo bo)
                }
                break;
        default:
-               TBM_LOG_E("error bo:%p bo_lock_type[%d] is wrong.\n",
+               TBM_ERR("error bo:%p bo_lock_type[%d] is wrong.\n",
                                bo, bo->bufmgr->bo_lock_type);
                break;
        }
@@ -292,109 +263,72 @@ _tbm_bo_unlock(tbm_bo bo)
        if (bo->lock_cnt < 0)
                bo->lock_cnt = 0;
 
-       TBM_DBG_LOCK(">> UNLOCK bo:%p(%d->%d)\n", bo, old, bo->lock_cnt);
+       TBM_DBG(">> UNLOCK bo:%p(%d->%d)\n", bo, old, bo->lock_cnt);
 }
 
 static int
-_tbm_bo_is_valid(tbm_bo bo)
+_tbm_bo_magic_check(tbm_bo bo)
 {
-       tbm_bufmgr bufmgr = NULL;
-       tbm_bo old_data = NULL;
-
-       if (bo == NULL) {
-               TBM_LOG_E("error: bo is NULL.\n");
+       if (bo->magic != TBM_BO_MAGIC)
                return 0;
-       }
 
-       bufmgr = tbm_bufmgr_get();
-       if (bufmgr == NULL) {
-               TBM_LOG_E("error: bufmgr is NULL.\n");
+       return 1;
+}
+
+static int
+_tbm_bo_is_valid(tbm_bo bo)
+{
+       if (!bo) {
+               TBM_ERR("error: bo is NULL.\n");
                return 0;
        }
 
-       if (LIST_IS_EMPTY(&bufmgr->bo_list)) {
-               TBM_LOG_E("error: bo->bo->bufmgr->bo_list is EMPTY.\n");
+       if (!_tbm_bo_magic_check(bo)) {
+               TBM_ERR("error: No valid bo(%p).\n", bo);
                return 0;
        }
 
-       LIST_FOR_EACH_ENTRY(old_data, &bufmgr->bo_list, item_link) {
-               if (old_data == bo)
-                       return 1;
-       }
+       return 1;
+}
 
-       TBM_LOG_E("error: No valid bo(%p).\n", bo);
+static void
+_tbm_bo_deinit(tbm_bo bo)
+{
+       bo->magic = 0;
 
-       return 0;
+       bo->bufmgr->bo_cnt--;
+       LIST_DEL(&bo->item_link);
 }
 
 tbm_bo
 tbm_bo_alloc(tbm_bufmgr bufmgr, int size, int flags)
 {
-       void *bo_priv;
        tbm_bo bo;
 
-       _tbm_bo_mutex_lock();
-
-       TBM_BO_RETURN_VAL_IF_FAIL(TBM_BUFMGR_IS_VALID(bufmgr), NULL);
-       TBM_BO_RETURN_VAL_IF_FAIL(size > 0, NULL);
-
-       bo = calloc(1, sizeof(struct _tbm_bo));
+       bo = tbm_bufmgr_internal_alloc_bo(bufmgr, size, flags);
        if (!bo) {
                /* LCOV_EXCL_START */
-               TBM_LOG_E("error: fail to create of tbm_bo size(%d) flag(%s)\n",
-                               size, _tbm_flag_to_str(flags));
-               _tbm_set_last_result(TBM_BO_ERROR_HEAP_ALLOC_FAILED);
-               _tbm_bo_mutex_unlock();
+               TBM_ERR("tbm_bufmgr_internal_alloc_bo failed.");
                return NULL;
                /* LCOV_EXCL_STOP */
        }
 
-       _tbm_util_check_bo_cnt(bufmgr);
-
-       bo->bufmgr = bufmgr;
-
-       bo_priv = bo->bufmgr->backend->bo_alloc(bo, size, flags);
-       if (!bo_priv) {
-               /* LCOV_EXCL_START */
-               TBM_LOG_E("error: fail to create of tbm_bo size(%d) flag(%s)\n",
-                               size, _tbm_flag_to_str(flags));
-               _tbm_set_last_result(TBM_BO_ERROR_BO_ALLOC_FAILED);
-               free(bo);
-               _tbm_bo_mutex_unlock();
-               return NULL;
-               /* LCOV_EXCL_STOP */
-       }
-
-       bo->bufmgr->bo_cnt++;
-
-       bo->ref_cnt = 1;
-       bo->flags = flags;
-       bo->priv = bo_priv;
-
-       TBM_TRACE("bo(%p) size(%d) refcnt(%d), flag(%s)\n", bo, size, bo->ref_cnt,
-                       _tbm_flag_to_str(bo->flags));
-
-       LIST_INITHEAD(&bo->user_data_list);
-
-       LIST_ADD(&bo->item_link, &bo->bufmgr->bo_list);
-
-       _tbm_bo_mutex_unlock();
-
        return bo;
 }
 
 tbm_bo
 tbm_bo_ref(tbm_bo bo)
 {
-       _tbm_bo_mutex_lock();
+       _tbm_bufmgr_mutex_lock();
+       _tbm_set_last_result(TBM_ERROR_NONE);
 
        TBM_BO_RETURN_VAL_IF_FAIL(_tbm_bo_is_valid(bo), NULL);
 
        bo->ref_cnt++;
 
-       TBM_TRACE("bo(%p) ref_cnt(%d)\n", bo, bo->ref_cnt);
+       TBM_TRACE_BO("bo(%p) ref_cnt(%d)\n", bo, bo->ref_cnt);
 
-       _tbm_bo_mutex_unlock();
+       _tbm_bufmgr_mutex_unlock();
 
        return bo;
 }
@@ -402,14 +336,15 @@ tbm_bo_ref(tbm_bo bo)
 void
 tbm_bo_unref(tbm_bo bo)
 {
-       _tbm_bo_mutex_lock();
+       _tbm_bufmgr_mutex_lock();
+       _tbm_set_last_result(TBM_ERROR_NONE);
 
        TBM_BO_RETURN_IF_FAIL(_tbm_bo_is_valid(bo));
 
-       TBM_TRACE("bo(%p) ref_cnt(%d)\n", bo, bo->ref_cnt - 1);
+       TBM_TRACE_BO("bo(%p) ref_cnt(%d)\n", bo, bo->ref_cnt - 1);
 
        if (bo->ref_cnt <= 0) {
-               _tbm_bo_mutex_unlock();
+               _tbm_bufmgr_mutex_unlock();
                return;
        }
 
@@ -417,32 +352,33 @@ tbm_bo_unref(tbm_bo bo)
        if (bo->ref_cnt == 0)
                _tbm_bo_free(bo);
 
-       _tbm_bo_mutex_unlock();
+       _tbm_bufmgr_mutex_unlock();
 }
 
 tbm_bo_handle
 tbm_bo_map(tbm_bo bo, int device, int opt)
 {
        tbm_bo_handle bo_handle;
+       tbm_error_e error;
 
-       _tbm_bo_mutex_lock();
+       _tbm_bufmgr_mutex_lock();
+       _tbm_set_last_result(TBM_ERROR_NONE);
 
        TBM_BO_RETURN_VAL_IF_FAIL(_tbm_bo_is_valid(bo), (tbm_bo_handle) NULL);
 
        if (!_tbm_bo_lock(bo, device, opt)) {
-               _tbm_set_last_result(TBM_BO_ERROR_LOCK_FAILED);
-               TBM_LOG_E("error: fail to lock bo:%p)\n", bo);
-               _tbm_bo_mutex_unlock();
+               TBM_ERR("error: fail to lock bo:%p)\n", bo);
+               _tbm_bufmgr_mutex_unlock();
                return (tbm_bo_handle) NULL;
        }
 
-       bo_handle = bo->bufmgr->backend->bo_map(bo, device, opt);
+       bo_handle = tbm_bo_data_map(bo->bo_data, device, opt, &error);
        if (bo_handle.ptr == NULL) {
                /* LCOV_EXCL_START */
-               _tbm_set_last_result(TBM_BO_ERROR_MAP_FAILED);
-               TBM_LOG_E("error: fail to map bo:%p\n", bo);
+               TBM_ERR("error: fail to map bo:%p error:%d\n", bo, error);
+               _tbm_set_last_result(error);
                _tbm_bo_unlock(bo);
-               _tbm_bo_mutex_unlock();
+               _tbm_bufmgr_mutex_unlock();
                return (tbm_bo_handle) NULL;
                /* LCOV_EXCL_STOP */
        }
@@ -450,9 +386,9 @@ tbm_bo_map(tbm_bo bo, int device, int opt)
        /* increase the map_count */
        bo->map_cnt++;
 
-       TBM_TRACE("bo(%p) map_cnt(%d)\n", bo, bo->map_cnt);
+       TBM_TRACE_BO("bo(%p) map_cnt(%d)\n", bo, bo->map_cnt);
 
-       _tbm_bo_mutex_unlock();
+       _tbm_bufmgr_mutex_unlock();
 
        return bo_handle;
 }
@@ -460,57 +396,60 @@ tbm_bo_map(tbm_bo bo, int device, int opt)
 int
 tbm_bo_unmap(tbm_bo bo)
 {
-       int ret;
+       tbm_error_e error;
 
-       _tbm_bo_mutex_lock();
+       _tbm_bufmgr_mutex_lock();
+       _tbm_set_last_result(TBM_ERROR_NONE);
 
        TBM_BO_RETURN_VAL_IF_FAIL(_tbm_bo_is_valid(bo), 0);
        TBM_BO_RETURN_VAL_IF_FAIL(bo->map_cnt > 0, 0);
 
-       ret = bo->bufmgr->backend->bo_unmap(bo);
-       if (!ret) {
+       error = tbm_bo_data_unmap(bo->bo_data);
+       if (error != TBM_ERROR_NONE) {
                /* LCOV_EXCL_START */
-               TBM_LOG_E("error: bo(%p) map_cnt(%d)\n", bo, bo->map_cnt);
-               _tbm_set_last_result(TBM_BO_ERROR_UNMAP_FAILED);
-               _tbm_bo_mutex_unlock();
-               return ret;
+               TBM_ERR("error: bo(%p) map_cnt(%d) error(%d)\n", bo, bo->map_cnt, error);
+               _tbm_set_last_result(error);
+               _tbm_bufmgr_mutex_unlock();
+               return 0;
                /* LCOV_EXCL_STOP */
        }
 
        /* decrease the map_count */
        bo->map_cnt--;
 
-       TBM_TRACE("bo(%p) map_cnt(%d)\n", bo, bo->map_cnt);
+       TBM_TRACE_BO("bo(%p) map_cnt(%d)\n", bo, bo->map_cnt);
 
        _tbm_bo_unlock(bo);
 
-       _tbm_bo_mutex_unlock();
+       _tbm_bufmgr_mutex_unlock();
 
-       return ret;
+       return 1;
 }
 
 tbm_bo_handle
 tbm_bo_get_handle(tbm_bo bo, int device)
 {
        tbm_bo_handle bo_handle;
+       tbm_error_e error;
 
-       _tbm_bo_mutex_lock();
+       _tbm_bufmgr_mutex_lock();
+       _tbm_set_last_result(TBM_ERROR_NONE);
 
        TBM_BO_RETURN_VAL_IF_FAIL(_tbm_bo_is_valid(bo), (tbm_bo_handle) NULL);
 
-       bo_handle = bo->bufmgr->backend->bo_get_handle(bo, device);
+       bo_handle = tbm_bo_data_get_handle(bo->bo_data, device, &error);
        if (bo_handle.ptr == NULL) {
                /* LCOV_EXCL_START */
-               _tbm_set_last_result(TBM_BO_ERROR_GET_HANDLE_FAILED);
-               TBM_LOG_E("error: bo(%p) bo_handle(%p)\n", bo, bo_handle.ptr);
-               _tbm_bo_mutex_unlock();
-               return (tbm_bo_handle) NULL;
+               TBM_ERR("error: bo(%p) bo_handle(%p) error(%d)\n", bo, bo_handle.ptr, error);
+               _tbm_set_last_result(error);
+               _tbm_bufmgr_mutex_unlock();
+               return (tbm_bo_handle)NULL;
                /* LCOV_EXCL_STOP */
        }
 
-       TBM_TRACE("bo(%p) bo_handle(%p)\n", bo, bo_handle.ptr);
+       TBM_TRACE_BO("bo(%p) bo_handle(%p)\n", bo, bo_handle.ptr);
 
-       _tbm_bo_mutex_unlock();
+       _tbm_bufmgr_mutex_unlock();
 
        return bo_handle;
 }
@@ -519,31 +458,26 @@ tbm_key
 tbm_bo_export(tbm_bo bo)
 {
        tbm_key ret;
+       tbm_error_e error;
 
-       _tbm_bo_mutex_lock();
+       _tbm_bufmgr_mutex_lock();
+       _tbm_set_last_result(TBM_ERROR_NONE);
 
        TBM_BO_RETURN_VAL_IF_FAIL(_tbm_bo_is_valid(bo), 0);
 
-       if (!bo->bufmgr->backend->bo_export) {
-               /* LCOV_EXCL_START */
-               _tbm_bo_mutex_unlock();
-               return 0;
-               /* LCOV_EXCL_STOP */
-       }
-
-       ret = bo->bufmgr->backend->bo_export(bo);
+       ret = tbm_bo_data_export_key(bo->bo_data, &error);
        if (!ret) {
                /* LCOV_EXCL_START */
-               _tbm_set_last_result(TBM_BO_ERROR_EXPORT_FAILED);
-               TBM_LOG_E("error: bo(%p) tbm_key(%d)\n", bo, ret);
-               _tbm_bo_mutex_unlock();
+               TBM_ERR("tbm_moule_bo_export_key failed. bo:%p tbm_key:%d error:%d", bo, ret, error);
+               _tbm_set_last_result(error);
+               _tbm_bufmgr_mutex_unlock();
                return ret;
                /* LCOV_EXCL_STOP */
        }
 
-       TBM_TRACE("bo(%p) tbm_key(%u)\n", bo, ret);
+       TBM_TRACE_BO("bo:%p tbm_key:%u", bo, ret);
 
-       _tbm_bo_mutex_unlock();
+       _tbm_bufmgr_mutex_unlock();
 
        return ret;
 }
@@ -551,111 +485,44 @@ tbm_bo_export(tbm_bo bo)
 tbm_fd
 tbm_bo_export_fd(tbm_bo bo)
 {
-       tbm_fd ret;
+       tbm_fd fd;
+       tbm_error_e error;
 
-       _tbm_bo_mutex_lock();
+       _tbm_bufmgr_mutex_lock();
+       _tbm_set_last_result(TBM_ERROR_NONE);
 
        TBM_BO_RETURN_VAL_IF_FAIL(_tbm_bo_is_valid(bo), -1);
 
-       if (!bo->bufmgr->backend->bo_export_fd) {
+       fd = tbm_bo_data_export_fd(bo->bo_data, &error);
+       if (fd < 0) {
                /* LCOV_EXCL_START */
-               _tbm_bo_mutex_unlock();
-               return -1;
+               TBM_ERR("tbm_bo_data_export_fd filed. bo:%p tbm_fd:%d error:%d", bo, fd, error);
+               _tbm_set_last_result(error);
+               _tbm_bufmgr_mutex_unlock();
+               return fd;
                /* LCOV_EXCL_STOP */
        }
 
-       ret = bo->bufmgr->backend->bo_export_fd(bo);
-       if (ret < 0) {
-               /* LCOV_EXCL_START */
-               _tbm_set_last_result(TBM_BO_ERROR_EXPORT_FD_FAILED);
-               TBM_LOG_E("error: bo(%p) tbm_fd(%d)\n", bo, ret);
-               _tbm_bo_mutex_unlock();
-               return ret;
-               /* LCOV_EXCL_STOP */
-       }
+       TBM_TRACE_BO("bo:%p tbm_fd:%d", bo, fd);
 
-       TBM_TRACE("bo(%p) tbm_fd(%d)\n", bo, ret);
+       _tbm_bufmgr_mutex_unlock();
 
-       _tbm_bo_mutex_unlock();
-
-       return ret;
+       return fd;
 }
 
 tbm_bo
 tbm_bo_import(tbm_bufmgr bufmgr, unsigned int key)
 {
        tbm_bo bo;
-       tbm_bo bo2 = NULL;
-       void *bo_priv;
-
-       _tbm_bo_mutex_lock();
-
-       TBM_BO_RETURN_VAL_IF_FAIL(TBM_BUFMGR_IS_VALID(bufmgr), NULL);
-
-       if (!bufmgr->backend->bo_import) {
-               /* LCOV_EXCL_START */
-               _tbm_bo_mutex_unlock();
-               return NULL;
-               /* LCOV_EXCL_STOP */
-       }
 
-       _tbm_util_check_bo_cnt(bufmgr);
-
-       bo = calloc(1, sizeof(struct _tbm_bo));
+       bo = tbm_bufmgr_internal_import_bo_with_key(bufmgr, key);
        if (!bo) {
                /* LCOV_EXCL_START */
-               TBM_LOG_E("error: fail to import of tbm_bo by key(%d)\n", key);
-               _tbm_bo_mutex_unlock();
+               TBM_ERR("tbm_bufmgr_internal_import_bo_with_key failed");
                return NULL;
                /* LCOV_EXCL_STOP */
        }
 
-       bo->bufmgr = bufmgr;
-
-       bo_priv = bo->bufmgr->backend->bo_import(bo, key);
-       if (!bo_priv) {
-               /* LCOV_EXCL_START */
-               TBM_LOG_E("error: fail to import of tbm_bo by key(%d)\n", key);
-               _tbm_set_last_result(TBM_BO_ERROR_IMPORT_FAILED);
-               free(bo);
-               _tbm_bo_mutex_unlock();
-               return NULL;
-               /* LCOV_EXCL_STOP */
-       }
-
-       if (!LIST_IS_EMPTY(&bo->bufmgr->bo_list)) {
-               LIST_FOR_EACH_ENTRY(bo2, &bo->bufmgr->bo_list, item_link) {
-                       if (bo2->priv == bo_priv) {
-                               TBM_TRACE("find bo(%p) ref(%d) key(%d) flag(%s) in list\n",
-                                               bo2, bo2->ref_cnt, key,
-                                               _tbm_flag_to_str(bo2->flags));
-                               bo2->ref_cnt++;
-                               free(bo);
-                               _tbm_bo_mutex_unlock();
-                               return bo2;
-                       }
-               }
-       }
-
-       bo->bufmgr->bo_cnt++;
-
-       bo->ref_cnt = 1;
-       bo->priv = bo_priv;
-
-       if (bo->bufmgr->backend->bo_get_flags)
-               bo->flags = bo->bufmgr->backend->bo_get_flags(bo);
-       else
-               bo->flags = TBM_BO_DEFAULT;
-
-       TBM_TRACE("import new bo(%p) ref(%d) key(%d) flag(%s) in list\n",
-                         bo, bo->ref_cnt, key, _tbm_flag_to_str(bo->flags));
-
-       LIST_INITHEAD(&bo->user_data_list);
-
-       LIST_ADD(&bo->item_link, &bo->bufmgr->bo_list);
-
-       _tbm_bo_mutex_unlock();
-
        return bo;
 }
 
@@ -663,78 +530,15 @@ tbm_bo
 tbm_bo_import_fd(tbm_bufmgr bufmgr, tbm_fd fd)
 {
        tbm_bo bo;
-       tbm_bo bo2 = NULL;
-       void *bo_priv;
-
-       _tbm_bo_mutex_lock();
 
-       TBM_BO_RETURN_VAL_IF_FAIL(TBM_BUFMGR_IS_VALID(bufmgr), NULL);
-
-       if (!bufmgr->backend->bo_import_fd) {
-               /* LCOV_EXCL_START */
-               _tbm_bo_mutex_unlock();
-               return NULL;
-               /* LCOV_EXCL_STOP */
-       }
-
-       _tbm_util_check_bo_cnt(bufmgr);
-
-       bo = calloc(1, sizeof(struct _tbm_bo));
+       bo = tbm_bufmgr_internal_import_bo_with_fd(bufmgr, fd);
        if (!bo) {
                /* LCOV_EXCL_START */
-               TBM_LOG_E("error: fail to import tbm_bo by tbm_fd(%d)\n", fd);
-               _tbm_bo_mutex_unlock();
+               TBM_ERR("tbm_bufmgr_internal_import_fd failed.");
                return NULL;
                /* LCOV_EXCL_STOP */
        }
 
-       bo->bufmgr = bufmgr;
-
-       bo_priv = bo->bufmgr->backend->bo_import_fd(bo, fd);
-       if (!bo_priv) {
-               /* LCOV_EXCL_START */
-               TBM_LOG_E("error: fail to import tbm_bo by tbm_fd(%d)\n", fd);
-               _tbm_set_last_result(TBM_BO_ERROR_IMPORT_FD_FAILED);
-               free(bo);
-               _tbm_bo_mutex_unlock();
-               return NULL;
-               /* LCOV_EXCL_STOP */
-       }
-
-       if (!LIST_IS_EMPTY(&bo->bufmgr->bo_list)) {
-
-               LIST_FOR_EACH_ENTRY(bo2, &bo->bufmgr->bo_list, item_link) {
-                       if (bo2->priv == bo_priv) {
-                               TBM_TRACE("find bo(%p) ref(%d) fd(%d) flag(%s) in list\n",
-                                               bo2, bo2->ref_cnt, fd,
-                                               _tbm_flag_to_str(bo2->flags));
-                               bo2->ref_cnt++;
-                               free(bo);
-                               _tbm_bo_mutex_unlock();
-                               return bo2;
-                       }
-               }
-       }
-
-       bo->bufmgr->bo_cnt++;
-
-       bo->ref_cnt = 1;
-       bo->priv = bo_priv;
-
-       if (bo->bufmgr->backend->bo_get_flags)
-               bo->flags = bo->bufmgr->backend->bo_get_flags(bo);
-       else
-               bo->flags = TBM_BO_DEFAULT;
-
-       TBM_TRACE("import bo(%p) ref(%d) fd(%d) flag(%s)\n",
-                       bo, bo->ref_cnt, fd, _tbm_flag_to_str(bo->flags));
-
-       LIST_INITHEAD(&bo->user_data_list);
-
-       LIST_ADD(&bo->item_link, &bo->bufmgr->bo_list);
-
-       _tbm_bo_mutex_unlock();
-
        return bo;
 }
 
@@ -742,16 +546,19 @@ int
 tbm_bo_size(tbm_bo bo)
 {
        int size;
+       tbm_error_e error;
 
-       _tbm_bo_mutex_lock();
+       _tbm_bufmgr_mutex_lock();
+       _tbm_set_last_result(TBM_ERROR_NONE);
 
        TBM_BO_RETURN_VAL_IF_FAIL(_tbm_bo_is_valid(bo), 0);
 
-       size = bo->bufmgr->backend->bo_size(bo);
+       size = tbm_bo_data_get_size(bo->bo_data, &error);
+       _tbm_set_last_result(error);
 
-       TBM_TRACE("bo(%p) size(%d)\n", bo, size);
+       TBM_TRACE_BO("bo(%p) size(%d)\n", bo, size);
 
-       _tbm_bo_mutex_unlock();
+       _tbm_bufmgr_mutex_unlock();
 
        return size;
 }
@@ -759,24 +566,26 @@ tbm_bo_size(tbm_bo bo)
 int
 tbm_bo_locked(tbm_bo bo)
 {
-       _tbm_bo_mutex_lock();
+       _tbm_bufmgr_mutex_lock();
+       _tbm_set_last_result(TBM_ERROR_NONE);
 
        TBM_BO_RETURN_VAL_IF_FAIL(_tbm_bo_is_valid(bo), 0);
 
        if (bo->bufmgr->bo_lock_type == TBM_BUFMGR_BO_LOCK_TYPE_NEVER) {
-               TBM_LOG_E("bo(%p) lock_cnt(%d)\n", bo, bo->lock_cnt);
-               _tbm_bo_mutex_unlock();
+               TBM_ERR("bo(%p) lock_cnt(%d)\n", bo, bo->lock_cnt);
+               _tbm_set_last_result(TBM_ERROR_INVALID_OPERATION);
+               _tbm_bufmgr_mutex_unlock();
                return 0;
        }
 
        if (bo->lock_cnt > 0) {
-               TBM_TRACE("error: bo(%p) lock_cnt(%d)\n", bo, bo->lock_cnt);
-               _tbm_bo_mutex_unlock();
+               TBM_TRACE_BO("error: bo(%p) lock_cnt(%d)\n", bo, bo->lock_cnt);
+               _tbm_bufmgr_mutex_unlock();
                return 1;
        }
 
-       TBM_TRACE("bo(%p) lock_cnt(%d)\n", bo, bo->lock_cnt);
-       _tbm_bo_mutex_unlock();
+       TBM_TRACE_BO("bo(%p) lock_cnt(%d)\n", bo, bo->lock_cnt);
+       _tbm_bufmgr_mutex_unlock();
 
        return 0;
 }
@@ -784,31 +593,44 @@ tbm_bo_locked(tbm_bo bo)
 int
 tbm_bo_swap(tbm_bo bo1, tbm_bo bo2)
 {
+       tbm_error_e error1, error2;
+       int size1 = -1, size2 = -2;
        void *temp;
 
-       _tbm_bo_mutex_lock();
+       _tbm_bufmgr_mutex_lock();
+       _tbm_set_last_result(TBM_ERROR_NONE);
 
        TBM_BO_RETURN_VAL_IF_FAIL(_tbm_bo_is_valid(bo1), 0);
        TBM_BO_RETURN_VAL_IF_FAIL(_tbm_bo_is_valid(bo2), 0);
 
-       TBM_TRACE("before: bo1(%p) bo2(%p)\n", bo1, bo2);
+       TBM_TRACE_BO("before: bo1(%p) bo2(%p)\n", bo1, bo2);
 
-       if (bo1->bufmgr->backend->bo_size(bo1) != bo2->bufmgr->backend->bo_size(bo2)) {
-               _tbm_set_last_result(TBM_BO_ERROR_SWAP_FAILED);
-               TBM_LOG_E("error: bo1(%p) bo2(%p)\n", bo1, bo2);
-               _tbm_bo_mutex_unlock();
-               return 0;
+       size1 = tbm_bo_data_get_size(bo1->bo_data, &error1);
+       _tbm_set_last_result(error1);
+       size2 = tbm_bo_data_get_size(bo2->bo_data, &error2);
+       _tbm_set_last_result(error2);
+
+       if (size1 != size2) {
+               TBM_ERR("error: bo1 size(%d) and bo2 size(%d) is different.", size1, size2);
+               _tbm_set_last_result(TBM_ERROR_INVALID_OPERATION);
+               goto fail;
        }
 
-       TBM_TRACE("after: bo1(%p) bo2(%p)\n", bo1, bo2);
+       temp = bo1->bo_data;
+       bo1->bo_data = bo2->bo_data;
+       bo2->bo_data = temp;
 
-       temp = bo1->priv;
-       bo1->priv = bo2->priv;
-       bo2->priv = temp;
+       TBM_TRACE_BO("after: bo1(%p) bo2(%p)\n", bo1, bo2);
 
-       _tbm_bo_mutex_unlock();
+       _tbm_bufmgr_mutex_unlock();
 
        return 1;
+
+fail:
+       TBM_ERR("error: bo1(%p) bo2(%p)\n", bo1, bo2);
+       _tbm_bufmgr_mutex_unlock();
+
+       return 0;
 }
 
 int
@@ -817,30 +639,33 @@ tbm_bo_add_user_data(tbm_bo bo, unsigned long key,
 {
        tbm_user_data *data;
 
-       _tbm_bo_mutex_lock();
+       _tbm_bufmgr_mutex_lock();
+       _tbm_set_last_result(TBM_ERROR_NONE);
 
        TBM_BO_RETURN_VAL_IF_FAIL(_tbm_bo_is_valid(bo), 0);
 
        /* check if the data according to the key exist if so, return false. */
        data = user_data_lookup(&bo->user_data_list, key);
        if (data) {
-               TBM_TRACE("warning: user data already exist key(%ld)\n", key);
-               _tbm_bo_mutex_unlock();
+               TBM_TRACE_BO("warning: user data already exist key(%ld)\n", key);
+               _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER);
+               _tbm_bufmgr_mutex_unlock();
                return 0;
        }
 
        data = user_data_create(key, data_free_func);
        if (!data) {
-               TBM_LOG_E("error: bo(%p) key(%lu)\n", bo, key);
-               _tbm_bo_mutex_unlock();
+               TBM_ERR("error: bo(%p) key(%lu)\n", bo, key);
+               _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER);
+               _tbm_bufmgr_mutex_unlock();
                return 0;
        }
 
-       TBM_TRACE("bo(%p) key(%lu) data(%p)\n", bo, key, data->data);
+       TBM_TRACE_BO("bo(%p) key(%lu) data(%p)\n", bo, key, data->data);
 
        LIST_ADD(&data->item_link, &bo->user_data_list);
 
-       _tbm_bo_mutex_unlock();
+       _tbm_bufmgr_mutex_unlock();
 
        return 1;
 }
@@ -850,28 +675,31 @@ tbm_bo_delete_user_data(tbm_bo bo, unsigned long key)
 {
        tbm_user_data *old_data;
 
-       _tbm_bo_mutex_lock();
+       _tbm_bufmgr_mutex_lock();
+       _tbm_set_last_result(TBM_ERROR_NONE);
 
        TBM_BO_RETURN_VAL_IF_FAIL(_tbm_bo_is_valid(bo), 0);
 
        if (LIST_IS_EMPTY(&bo->user_data_list)) {
-               TBM_TRACE("error: bo(%p) key(%lu)\n", bo, key);
-               _tbm_bo_mutex_unlock();
+               TBM_TRACE_BO("bo(%p) key(%lu)\n", bo, key);
+               _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER);
+               _tbm_bufmgr_mutex_unlock();
                return 0;
        }
 
        old_data = user_data_lookup(&bo->user_data_list, key);
        if (!old_data) {
-               TBM_TRACE("error: bo(%p) key(%lu)\n", bo, key);
-               _tbm_bo_mutex_unlock();
+               TBM_TRACE_BO("bo(%p) key(%lu)\n", bo, key);
+               _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER);
+               _tbm_bufmgr_mutex_unlock();
                return 0;
        }
 
-       TBM_TRACE("bo(%p) key(%lu) data(%p)\n", bo, key, old_data->data);
+       TBM_TRACE_BO("bo(%p) key(%lu) data(%p)\n", bo, key, old_data->data);
 
        user_data_delete(old_data);
 
-       _tbm_bo_mutex_unlock();
+       _tbm_bufmgr_mutex_unlock();
 
        return 1;
 }
@@ -881,20 +709,23 @@ tbm_bo_set_user_data(tbm_bo bo, unsigned long key, void *data)
 {
        tbm_user_data *old_data;
 
-       _tbm_bo_mutex_lock();
+       _tbm_bufmgr_mutex_lock();
+       _tbm_set_last_result(TBM_ERROR_NONE);
 
        TBM_BO_RETURN_VAL_IF_FAIL(_tbm_bo_is_valid(bo), 0);
 
        if (LIST_IS_EMPTY(&bo->user_data_list)) {
-               TBM_TRACE("error: bo(%p) key(%lu)\n", bo, key);
-               _tbm_bo_mutex_unlock();
+               TBM_TRACE_BO("error: bo(%p) key(%lu)\n", bo, key);
+               _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER);
+               _tbm_bufmgr_mutex_unlock();
                return 0;
        }
 
        old_data = user_data_lookup(&bo->user_data_list, key);
        if (!old_data) {
-               TBM_TRACE("error: bo(%p) key(%lu)\n", bo, key);
-               _tbm_bo_mutex_unlock();
+               TBM_TRACE_BO("error: bo(%p) key(%lu)\n", bo, key);
+               _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER);
+               _tbm_bufmgr_mutex_unlock();
                return 0;
        }
 
@@ -902,9 +733,9 @@ tbm_bo_set_user_data(tbm_bo bo, unsigned long key, void *data)
                old_data->free_func(old_data->data);
        old_data->data = data;
 
-       TBM_TRACE("bo(%p) key(%lu) data(%p)\n", bo, key, old_data->data);
+       TBM_TRACE_BO("bo(%p) key(%lu) data(%p)\n", bo, key, old_data->data);
 
-       _tbm_bo_mutex_unlock();
+       _tbm_bufmgr_mutex_unlock();
 
        return 1;
 }
@@ -914,55 +745,50 @@ tbm_bo_get_user_data(tbm_bo bo, unsigned long key, void **data)
 {
        tbm_user_data *old_data;
 
-       _tbm_bo_mutex_lock();
+       _tbm_bufmgr_mutex_lock();
+       _tbm_set_last_result(TBM_ERROR_NONE);
 
        TBM_BO_RETURN_VAL_IF_FAIL(_tbm_bo_is_valid(bo), 0);
 
        if (!data || LIST_IS_EMPTY(&bo->user_data_list)) {
-               TBM_TRACE("error: bo(%p) key(%lu)\n", bo, key);
-               _tbm_bo_mutex_unlock();
+               TBM_TRACE_BO("error: bo(%p) key(%lu)\n", bo, key);
+               _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER);
+               _tbm_bufmgr_mutex_unlock();
                return 0;
        }
 
        old_data = user_data_lookup(&bo->user_data_list, key);
        if (!old_data) {
-               TBM_TRACE("error: bo(%p) key(%lu)\n", bo, key);
                *data = NULL;
-               _tbm_bo_mutex_unlock();
+               TBM_TRACE_BO("error: bo(%p) key(%lu)\n", bo, key);
+               _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER);
+               _tbm_bufmgr_mutex_unlock();
                return 0;
        }
 
        *data = old_data->data;
 
-       TBM_TRACE("bo(%p) key(%lu) data(%p)\n", bo, key, old_data->data);
+       TBM_TRACE_BO("bo(%p) key(%lu) data(%p)\n", bo, key, old_data->data);
 
-       _tbm_bo_mutex_unlock();
+       _tbm_bufmgr_mutex_unlock();
 
        return 1;
 }
 
-/* LCOV_EXCL_START */
-tbm_error_e
-tbm_get_last_error(void)
-{
-       return tbm_last_error;
-}
-/* LCOV_EXCL_STOP */
-
 int
 tbm_bo_get_flags(tbm_bo bo)
 {
        int flags;
 
-       _tbm_bo_mutex_lock();
+       _tbm_bufmgr_mutex_lock();
 
        TBM_BO_RETURN_VAL_IF_FAIL(_tbm_bo_is_valid(bo), 0);
 
        flags = bo->flags;
 
-       TBM_TRACE("bo(%p)\n", bo);
+       TBM_TRACE_BO("bo(%p)\n", bo);
 
-       _tbm_bo_mutex_unlock();
+       _tbm_bufmgr_mutex_unlock();
 
        return flags;
 }
@@ -972,13 +798,13 @@ tbm_bo_get_flags(tbm_bo bo)
 int
 _tbm_bo_set_surface(tbm_bo bo, tbm_surface_h surface)
 {
-       _tbm_bo_mutex_lock();
+       _tbm_bufmgr_mutex_lock();
 
        TBM_BO_RETURN_VAL_IF_FAIL(_tbm_bo_is_valid(bo), 0);
 
        bo->surface = surface;
 
-       _tbm_bo_mutex_unlock();
+       _tbm_bufmgr_mutex_unlock();
 
        return 1;
 }
@@ -990,26 +816,24 @@ _tbm_bo_free(tbm_bo bo)
        if (!LIST_IS_EMPTY(&bo->user_data_list)) {
                tbm_user_data *old_data = NULL, *tmp;
 
-               LIST_FOR_EACH_ENTRY_SAFE(old_data, tmp,
-                               &bo->user_data_list, item_link) {
+               LIST_FOR_EACH_ENTRY_SAFE(old_data, tmp, &bo->user_data_list, item_link) {
                        TBM_DBG("free user_data\n");
                        user_data_delete(old_data);
                }
+
+               LIST_INITHEAD(&bo->user_data_list); // TODO: remove this. build-break when it is removed.
        }
 
        while (bo->lock_cnt > 0) {
-               TBM_LOG_E("error lock_cnt:%d\n", bo->lock_cnt);
+               TBM_ERR("error lock_cnt:%d\n", bo->lock_cnt);
                _bo_unlock(bo);
                bo->lock_cnt--;
        }
 
-       /* call the bo_free */
-       bo->bufmgr->backend->bo_free(bo);
-       bo->priv = NULL;
+       tbm_bo_data_free(bo->bo_data, bo->get_from_surface_data);
 
-       bo->bufmgr->bo_cnt--;
+       _tbm_bo_deinit(bo);
 
-       LIST_DEL(&bo->item_link);
        free(bo);
 }
 /* LCOV_EXCL_STOP */