tbm_bufmgr: add tbm_bufmgr_internal_import_bo_with_fd and use it
[platform/core/uifw/libtbm.git] / src / tbm_bo.c
index 95d7478..39aac76 100644 (file)
@@ -34,15 +34,12 @@ 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_ERR("'%s' failed.\n", #cond);\
-               _tbm_bo_mutex_unlock();\
+               _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER);\
+               _tbm_bufmgr_mutex_unlock();\
                return;\
        } \
 }
@@ -50,54 +47,13 @@ static void _tbm_bo_mutex_unlock(void);
 #define TBM_BO_RETURN_VAL_IF_FAIL(cond, val) {\
        if (!(cond)) {\
                TBM_ERR("'%s' failed.\n", #cond);\
-               _tbm_bo_mutex_unlock();\
+               _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_ERR("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_ERR("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_DBG("============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,8 +104,8 @@ 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;
 
@@ -169,6 +113,7 @@ tbm_user_data
        if (!user_data) {
                /* LCOV_EXCL_START */
                TBM_ERR("fail to allocate an user_date\n");
+               _tbm_set_last_result(TBM_ERROR_OUT_OF_MEMORY);
                return NULL;
                /* LCOV_EXCL_STOP */
        }
@@ -194,9 +139,35 @@ static int
 _bo_lock(tbm_bo bo, int device, int opt)
 {
        int ret = 1;
-
-       if (bo->bufmgr->backend->bo_lock)
-               ret = bo->bufmgr->backend->bo_lock(bo, device, opt);
+       tbm_error_e error;
+
+       if (bo->bufmgr->use_hal_tbm) {
+               error = (tbm_error_e)hal_tbm_bo_lock((hal_tbm_bo *)bo->bo_data, device, opt);
+               if (error == TBM_ERROR_NOT_SUPPORTED) {
+                       _tbm_set_last_result(TBM_ERROR_NONE);
+               } else {
+                       if (error != TBM_ERROR_NONE) {
+                               TBM_WRN("fail to lock");
+                               _tbm_set_last_result(error);
+                               ret = 0;
+                       }
+               }
+       } else if (bo->bufmgr->backend_module_data) {
+               if (bo->bufmgr->bo_func->bo_lock) {
+                       error = bo->bufmgr->bo_func->bo_lock(bo->bo_data, device, opt);
+                       if (error != TBM_ERROR_NONE) {
+                               TBM_WRN("fail to lock");
+                               _tbm_set_last_result(error);
+                               ret = 0;
+                       }
+               }
+       } else {
+               if (bo->bufmgr->backend->bo_lock) {
+                       ret = bo->bufmgr->backend->bo_lock(bo, device, opt);
+                       if (!ret)
+                               _tbm_set_last_result(TBM_ERROR_INVALID_OPERATION);
+               }
+       }
 
        return ret;
 }
@@ -204,8 +175,30 @@ _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;
+
+       if (bo->bufmgr->use_hal_tbm) {
+               error = (tbm_error_e)hal_tbm_bo_unlock((hal_tbm_bo *)bo->bo_data);
+               if (error == TBM_ERROR_NOT_SUPPORTED) {
+                       _tbm_set_last_result(TBM_ERROR_NONE);
+               } else {
+                       if (error != TBM_ERROR_NONE) {
+                               TBM_WRN("fail to lock");
+                               _tbm_set_last_result(error);
+                       }
+               }
+       } else if (bo->bufmgr->backend_module_data) {
+               if (bo->bufmgr->bo_func->bo_unlock) {
+                       error = bo->bufmgr->bo_func->bo_unlock(bo->bo_data);
+                       if (error != TBM_ERROR_NONE) {
+                               TBM_WRN("fail to unlock");
+                               _tbm_set_last_result(error);
+                       }
+               }
+       } else {
+               if (bo->bufmgr->backend->bo_unlock)
+                       bo->bufmgr->backend->bo_unlock(bo);
+       }
 }
 
 static int
@@ -231,18 +224,18 @@ _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;
@@ -296,89 +289,65 @@ _tbm_bo_unlock(tbm_bo bo)
 }
 
 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_ERR("error: bo is NULL.\n");
+       if (bo->magic != TBM_BO_MAGIC)
                return 0;
-       }
 
-       bufmgr = tbm_bufmgr_get();
-       if (bufmgr == NULL) {
-               TBM_ERR("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_ERR("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_ERR("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_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_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, &error);
        if (!bo) {
                /* LCOV_EXCL_START */
-               TBM_ERR("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. error:%d", error);
+               _tbm_set_last_result(error);
+               _tbm_bufmgr_mutex_unlock();
                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_ERR("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("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();
+       _tbm_bufmgr_mutex_unlock();
 
        return bo;
 }
@@ -386,7 +355,8 @@ tbm_bo_alloc(tbm_bufmgr bufmgr, int size, int flags)
 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);
 
@@ -394,7 +364,7 @@ tbm_bo_ref(tbm_bo bo)
 
        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 +372,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("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,34 +388,55 @@ 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_ERR("error: fail to lock bo:%p)\n", bo);
-               _tbm_bo_mutex_unlock();
+               _tbm_bufmgr_mutex_unlock();
                return (tbm_bo_handle) NULL;
        }
 
-       bo_handle = bo->bufmgr->backend->bo_map(bo, device, opt);
-       if (bo_handle.ptr == NULL) {
-               /* LCOV_EXCL_START */
-               _tbm_set_last_result(TBM_BO_ERROR_MAP_FAILED);
-               TBM_ERR("error: fail to map bo:%p\n", bo);
-               _tbm_bo_unlock(bo);
-               _tbm_bo_mutex_unlock();
-               return (tbm_bo_handle) NULL;
-               /* LCOV_EXCL_STOP */
+       if (bo->bufmgr->use_hal_tbm) {
+               hal_tbm_bo_handle hbo_handle;
+               hbo_handle = hal_tbm_bo_map((hal_tbm_bo *)bo->bo_data, device, opt, (hal_tbm_error *)&error);
+               if (hbo_handle.ptr == NULL) {
+                       /* LCOV_EXCL_START */
+                       _tbm_set_last_result(error);
+                       TBM_ERR("error: bo(%p) bo_handle(%p) error(%d)\n", bo, hbo_handle.ptr, error);
+                       goto bo_map_fail;
+                       /* LCOV_EXCL_STOP */
+               }
+               memcpy(&bo_handle.ptr, &hbo_handle.ptr, sizeof(tbm_bo_handle));
+       } else if (bo->bufmgr->backend_module_data) {
+               bo_handle = bo->bufmgr->bo_func->bo_map(bo->bo_data, device, opt, &error);
+               if (bo_handle.ptr == NULL) {
+                       /* LCOV_EXCL_START */
+                       _tbm_set_last_result(error);
+                       TBM_ERR("error: fail to map bo:%p error:%d\n", bo, error);
+                       goto bo_map_fail;
+                       /* LCOV_EXCL_STOP */
+               }
+       } else {
+               bo_handle = bo->bufmgr->backend->bo_map(bo, device, opt);
+               if (bo_handle.ptr == NULL) {
+                       /* LCOV_EXCL_START */
+                       _tbm_set_last_result(TBM_ERROR_INVALID_OPERATION);
+                       TBM_ERR("error: fail to map bo:%p\n", bo);
+                       goto bo_map_fail;
+                       /* LCOV_EXCL_STOP */
+               }
        }
 
        /* increase the map_count */
@@ -452,29 +444,57 @@ tbm_bo_map(tbm_bo bo, int device, int opt)
 
        TBM_TRACE_BO("bo(%p) map_cnt(%d)\n", bo, bo->map_cnt);
 
-       _tbm_bo_mutex_unlock();
+       _tbm_bufmgr_mutex_unlock();
 
        return bo_handle;
+
+bo_map_fail:
+       _tbm_bo_unlock(bo);
+       _tbm_bufmgr_mutex_unlock();
+       return (tbm_bo_handle) NULL;
 }
 
 int
 tbm_bo_unmap(tbm_bo bo)
 {
-       int ret;
+       int ret = 1;
+       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) {
-               /* LCOV_EXCL_START */
-               TBM_ERR("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;
-               /* LCOV_EXCL_STOP */
+       if (bo->bufmgr->use_hal_tbm) {
+               error = (hal_tbm_error)hal_tbm_bo_unmap((hal_tbm_bo *)bo->bo_data);
+               if (error != TBM_ERROR_NONE) {
+                       /* LCOV_EXCL_START */
+                       TBM_ERR("error: bo(%p) map_cnt(%d) error(%d)\n", bo, bo->map_cnt, error);
+                       _tbm_set_last_result(error);
+                       ret = 0;
+                       goto done;
+                       /* LCOV_EXCL_STOP */
+               }
+       } else if (bo->bufmgr->backend_module_data) {
+               error = bo->bufmgr->bo_func->bo_unmap(bo->bo_data);
+               if (error != TBM_ERROR_NONE) {
+                       /* LCOV_EXCL_START */
+                       TBM_ERR("error: bo(%p) map_cnt(%d) error(%d)\n", bo, bo->map_cnt, error);
+                       _tbm_set_last_result(error);
+                       ret = 0;
+                       goto done;
+                       /* LCOV_EXCL_STOP */
+               }
+       } else {
+               ret = bo->bufmgr->backend->bo_unmap(bo);
+               if (!ret) {
+                       /* LCOV_EXCL_START */
+                       TBM_ERR("error: bo(%p) map_cnt(%d)\n", bo, bo->map_cnt);
+                       _tbm_set_last_result(TBM_ERROR_INVALID_OPERATION);
+                       goto done;
+                       /* LCOV_EXCL_STOP */
+               }
        }
 
        /* decrease the map_count */
@@ -484,7 +504,8 @@ tbm_bo_unmap(tbm_bo bo)
 
        _tbm_bo_unlock(bo);
 
-       _tbm_bo_mutex_unlock();
+done:
+       _tbm_bufmgr_mutex_unlock();
 
        return ret;
 }
@@ -493,57 +514,115 @@ 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);
-       if (bo_handle.ptr == NULL) {
-               /* LCOV_EXCL_START */
-               _tbm_set_last_result(TBM_BO_ERROR_GET_HANDLE_FAILED);
-               TBM_ERR("error: bo(%p) bo_handle(%p)\n", bo, bo_handle.ptr);
-               _tbm_bo_mutex_unlock();
-               return (tbm_bo_handle) NULL;
-               /* LCOV_EXCL_STOP */
+       if (bo->bufmgr->use_hal_tbm) {
+               hal_tbm_bo_handle hbo_handle;
+               hbo_handle = hal_tbm_bo_get_handle((hal_tbm_bo *)bo->bo_data, device, (hal_tbm_error *)&error);
+               if (hbo_handle.ptr == NULL) {
+                       /* LCOV_EXCL_START */
+                       TBM_ERR("error: bo(%p) bo_handle(%p) error(%d)\n", bo, hbo_handle.ptr, error);
+                       _tbm_set_last_result(error);
+                       goto bo_handle_fail;
+                       /* LCOV_EXCL_STOP */
+               }
+               memcpy(&bo_handle.ptr, &hbo_handle.ptr, sizeof(tbm_bo_handle));
+       } else if (bo->bufmgr->backend_module_data) {
+               bo_handle = bo->bufmgr->bo_func->bo_get_handle(bo->bo_data, device, &error);
+               if (bo_handle.ptr == NULL) {
+                       /* LCOV_EXCL_START */
+                       TBM_ERR("error: bo(%p) bo_handle(%p) error(%d)\n", bo, bo_handle.ptr, error);
+                       _tbm_set_last_result(error);
+                       goto bo_handle_fail;
+                       /* LCOV_EXCL_STOP */
+               }
+       } else {
+               bo_handle = bo->bufmgr->backend->bo_get_handle(bo, device);
+               if (bo_handle.ptr == NULL) {
+                       /* LCOV_EXCL_START */
+                       TBM_ERR("error: bo(%p) bo_handle(%p)\n", bo, bo_handle.ptr);
+                       _tbm_set_last_result(TBM_ERROR_INVALID_OPERATION);
+                       goto bo_handle_fail;
+                       /* LCOV_EXCL_STOP */
+               }
        }
 
        TBM_TRACE_BO("bo(%p) bo_handle(%p)\n", bo, bo_handle.ptr);
 
-       _tbm_bo_mutex_unlock();
+       _tbm_bufmgr_mutex_unlock();
 
        return bo_handle;
+
+bo_handle_fail:
+       _tbm_bufmgr_mutex_unlock();
+       return (tbm_bo_handle) NULL;
 }
 
 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 */
-       }
+       if (bo->bufmgr->use_hal_tbm) {
+               ret = (hal_tbm_key)hal_tbm_bo_export_key((hal_tbm_bo *)bo->bo_data, (hal_tbm_error *)&error);
+               if (!ret) {
+                       /* LCOV_EXCL_START */
+                       TBM_ERR("error: bo(%p) tbm_key(%d) error(%d)\n", bo, ret, error);
+                       _tbm_set_last_result(error);
+                       goto done;
+                       /* LCOV_EXCL_STOP */
+               }
+       } else if (bo->bufmgr->backend_module_data) {
+               if (!bo->bufmgr->bo_func->bo_export_key) {
+                       /* LCOV_EXCL_START */
+                       _tbm_bufmgr_mutex_unlock();
+                       _tbm_set_last_result(TBM_ERROR_NOT_SUPPORTED);
+                       return 0;
+                       /* LCOV_EXCL_STOP */
+               }
 
-       ret = bo->bufmgr->backend->bo_export(bo);
-       if (!ret) {
-               /* LCOV_EXCL_START */
-               _tbm_set_last_result(TBM_BO_ERROR_EXPORT_FAILED);
-               TBM_ERR("error: bo(%p) tbm_key(%d)\n", bo, ret);
-               _tbm_bo_mutex_unlock();
-               return ret;
-               /* LCOV_EXCL_STOP */
+               ret = bo->bufmgr->bo_func->bo_export_key(bo->bo_data, &error);
+               if (!ret) {
+                       /* LCOV_EXCL_START */
+                       TBM_ERR("error: bo(%p) tbm_key(%d) error(%d)\n", bo, ret, error);
+                       _tbm_set_last_result(error);
+                       goto done;
+                       /* LCOV_EXCL_STOP */
+               }
+       } else {
+               if (!bo->bufmgr->backend->bo_export) {
+                       /* LCOV_EXCL_START */
+                       _tbm_bufmgr_mutex_unlock();
+                       _tbm_set_last_result(TBM_ERROR_NOT_SUPPORTED);
+                       return 0;
+                       /* LCOV_EXCL_STOP */
+               }
+
+               ret = bo->bufmgr->backend->bo_export(bo);
+               if (!ret) {
+                       /* LCOV_EXCL_START */
+                       TBM_ERR("error: bo(%p) tbm_key(%d)\n", bo, ret);
+                       _tbm_set_last_result(TBM_ERROR_INVALID_OPERATION);
+                       goto done;
+                       /* LCOV_EXCL_STOP */
+               }
        }
 
        TBM_TRACE_BO("bo(%p) tbm_key(%u)\n", bo, ret);
 
-       _tbm_bo_mutex_unlock();
+done:
+       _tbm_bufmgr_mutex_unlock();
 
        return ret;
 }
@@ -552,31 +631,62 @@ tbm_fd
 tbm_bo_export_fd(tbm_bo bo)
 {
        tbm_fd 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), -1);
 
-       if (!bo->bufmgr->backend->bo_export_fd) {
-               /* LCOV_EXCL_START */
-               _tbm_bo_mutex_unlock();
-               return -1;
-               /* LCOV_EXCL_STOP */
-       }
+       if (bo->bufmgr->use_hal_tbm) {
+               ret = (hal_tbm_fd)hal_tbm_bo_export_fd((hal_tbm_bo *)bo->bo_data, (hal_tbm_error *)&error);
+               if (ret < 0) {
+                       /* LCOV_EXCL_START */
+                       TBM_ERR("error: bo(%p) tbm_fd(%d) error(%d)\n", bo, ret, error);
+                       _tbm_set_last_result(error);
+                       goto done;
+                       /* LCOV_EXCL_STOP */
+               }
+       } else if (bo->bufmgr->backend_module_data) {
+               if (!bo->bufmgr->bo_func->bo_export_fd) {
+                       /* LCOV_EXCL_START */
+                       _tbm_bufmgr_mutex_unlock();
+                       _tbm_set_last_result(TBM_ERROR_NOT_SUPPORTED);
+                       return -1;
+                       /* 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_ERR("error: bo(%p) tbm_fd(%d)\n", bo, ret);
-               _tbm_bo_mutex_unlock();
-               return ret;
-               /* LCOV_EXCL_STOP */
+               ret = bo->bufmgr->bo_func->bo_export_fd(bo->bo_data, &error);
+               if (ret < 0) {
+                       /* LCOV_EXCL_START */
+                       TBM_ERR("error: bo(%p) tbm_fd(%d) error(%d)\n", bo, ret, error);
+                       _tbm_set_last_result(error);
+                       goto done;
+                       /* LCOV_EXCL_STOP */
+               }
+       } else {
+               if (!bo->bufmgr->backend->bo_export_fd) {
+                       /* LCOV_EXCL_START */
+                       _tbm_bufmgr_mutex_unlock();
+                       _tbm_set_last_result(TBM_ERROR_NOT_SUPPORTED);
+                       return -1;
+                       /* LCOV_EXCL_STOP */
+               }
+
+               ret = bo->bufmgr->backend->bo_export_fd(bo);
+               if (ret < 0) {
+                       /* LCOV_EXCL_START */
+                       TBM_ERR("error: bo(%p) tbm_fd(%d)\n", bo, ret);
+                       _tbm_set_last_result(TBM_ERROR_INVALID_OPERATION);
+                       goto done;
+                       /* LCOV_EXCL_STOP */
+               }
        }
 
        TBM_TRACE_BO("bo(%p) tbm_fd(%d)\n", bo, ret);
 
-       _tbm_bo_mutex_unlock();
+done:
+       _tbm_bufmgr_mutex_unlock();
 
        return ret;
 }
@@ -585,76 +695,27 @@ tbm_bo
 tbm_bo_import(tbm_bufmgr bufmgr, unsigned int key)
 {
        tbm_bo bo;
-       tbm_bo bo2 = NULL;
-       void *bo_priv;
+       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_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, &error);
        if (!bo) {
                /* LCOV_EXCL_START */
-               TBM_ERR("error: fail to import of tbm_bo by key(%d)\n", key);
-               _tbm_bo_mutex_unlock();
+               TBM_ERR("tbm_bufmgr_internal_import_key failed. error:%d", error);
+               _tbm_set_last_result(error);
+               _tbm_bufmgr_mutex_unlock();
                return NULL;
                /* LCOV_EXCL_STOP */
        }
 
-       bo->bufmgr = bufmgr;
-
-       bo_priv = bo->bufmgr->backend->bo_import(bo, key);
-       if (!bo_priv) {
-               /* LCOV_EXCL_START */
-               TBM_ERR("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_BO("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_BO("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();
+       _tbm_bufmgr_mutex_unlock();
 
        return bo;
 }
@@ -663,77 +724,29 @@ tbm_bo
 tbm_bo_import_fd(tbm_bufmgr bufmgr, tbm_fd fd)
 {
        tbm_bo bo;
-       tbm_bo bo2 = NULL;
-       void *bo_priv;
+       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_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, &error);
        if (!bo) {
                /* LCOV_EXCL_START */
-               TBM_ERR("error: fail to import tbm_bo by tbm_fd(%d)\n", fd);
-               _tbm_bo_mutex_unlock();
-               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_ERR("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();
+               TBM_ERR("tbm_bufmgr_internal_import_fd failed. error:%d", error);
+               _tbm_set_last_result(error);
+               _tbm_bufmgr_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_BO("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;
+       LIST_INITHEAD(&bo->user_data_list); // TODO: remove this. build-break when it is removed.
 
        TBM_TRACE_BO("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();
+       _tbm_bufmgr_mutex_unlock();
 
        return bo;
 }
@@ -742,16 +755,31 @@ 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);
+       if (bo->bufmgr->use_hal_tbm) {
+               size = hal_tbm_bo_get_size((hal_tbm_bo *)bo->bo_data, (hal_tbm_error *)&error);
+               if (error != TBM_ERROR_NONE) {
+                       TBM_ERR("fail to get the size of the bo_data(%d).", error);
+                       _tbm_set_last_result(TBM_ERROR_NONE);
+               }
+       } else if (bo->bufmgr->backend_module_data) {
+               size = bo->bufmgr->bo_func->bo_get_size(bo->bo_data, &error);
+               if (error != TBM_ERROR_NONE) {
+                       TBM_ERR("fail to get the size of the bo_data(%d).", error);
+                       _tbm_set_last_result(TBM_ERROR_NONE);
+               }
+       } else
+               size = bo->bufmgr->backend->bo_size(bo);
 
        TBM_TRACE_BO("bo(%p) size(%d)\n", bo, size);
 
-       _tbm_bo_mutex_unlock();
+       _tbm_bufmgr_mutex_unlock();
 
        return size;
 }
@@ -759,24 +787,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_ERR("bo(%p) lock_cnt(%d)\n", bo, bo->lock_cnt);
-               _tbm_bo_mutex_unlock();
+               _tbm_set_last_result(TBM_ERROR_INVALID_OPERATION);
+               _tbm_bufmgr_mutex_unlock();
                return 0;
        }
 
        if (bo->lock_cnt > 0) {
                TBM_TRACE_BO("error: bo(%p) lock_cnt(%d)\n", bo, bo->lock_cnt);
-               _tbm_bo_mutex_unlock();
+               _tbm_bufmgr_mutex_unlock();
                return 1;
        }
 
        TBM_TRACE_BO("bo(%p) lock_cnt(%d)\n", bo, bo->lock_cnt);
-       _tbm_bo_mutex_unlock();
+       _tbm_bufmgr_mutex_unlock();
 
        return 0;
 }
@@ -784,20 +814,53 @@ 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_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_ERR("error: bo1(%p) bo2(%p)\n", bo1, bo2);
-               _tbm_bo_mutex_unlock();
-               return 0;
+       if (bo1->bufmgr->use_hal_tbm) {
+               size1 = hal_tbm_bo_get_size((hal_tbm_bo *)bo1->bo_data, (hal_tbm_error *)&error1);
+               if (error1 != TBM_ERROR_NONE) {
+                       TBM_ERR("fail to get the size of bo1.(%d)", error1);
+                       _tbm_set_last_result(error1);
+                       goto fail;
+               }
+               size2 = hal_tbm_bo_get_size((hal_tbm_bo *)bo2->bo_data, (hal_tbm_error *)&error2);
+               if (error2 != TBM_ERROR_NONE) {
+                       TBM_ERR("fail to get the size of bo1.(%d)", error2);
+                       _tbm_set_last_result(error2);
+                       goto fail;
+               }
+       } else if (bo1->bufmgr->backend_module_data) {
+               size1 = bo1->bufmgr->bo_func->bo_get_size(bo1->bo_data, &error1);
+               if (error1 != TBM_ERROR_NONE) {
+                       TBM_ERR("fail to get the size of bo1.(%d)", error1);
+                       _tbm_set_last_result(error1);
+                       goto fail;
+               }
+               size2 = bo2->bufmgr->bo_func->bo_get_size(bo2->bo_data, &error2);
+               if (error2 != TBM_ERROR_NONE) {
+                       TBM_ERR("fail to get the size of bo2.(%d)", error2);
+                       _tbm_set_last_result(error2);
+                       goto fail;
+               }
+       } else {
+               size1 = bo1->bufmgr->backend->bo_size(bo1);
+               size2 = bo2->bufmgr->backend->bo_size(bo2);
+       }
+
+       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_BO("after: bo1(%p) bo2(%p)\n", bo1, bo2);
@@ -806,9 +869,15 @@ tbm_bo_swap(tbm_bo bo1, tbm_bo bo2)
        bo1->priv = bo2->priv;
        bo2->priv = temp;
 
-       _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,7 +886,8 @@ 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);
 
@@ -825,14 +895,16 @@ tbm_bo_add_user_data(tbm_bo bo, unsigned long key,
        data = user_data_lookup(&bo->user_data_list, key);
        if (data) {
                TBM_TRACE_BO("warning: user data already exist key(%ld)\n", key);
-               _tbm_bo_mutex_unlock();
+               _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_ERR("error: bo(%p) key(%lu)\n", bo, key);
-               _tbm_bo_mutex_unlock();
+               _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER);
+               _tbm_bufmgr_mutex_unlock();
                return 0;
        }
 
@@ -840,7 +912,7 @@ tbm_bo_add_user_data(tbm_bo bo, unsigned long key,
 
        LIST_ADD(&data->item_link, &bo->user_data_list);
 
-       _tbm_bo_mutex_unlock();
+       _tbm_bufmgr_mutex_unlock();
 
        return 1;
 }
@@ -850,20 +922,23 @@ 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_BO("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_BO("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;
        }
 
@@ -871,7 +946,7 @@ tbm_bo_delete_user_data(tbm_bo bo, unsigned long key)
 
        user_data_delete(old_data);
 
-       _tbm_bo_mutex_unlock();
+       _tbm_bufmgr_mutex_unlock();
 
        return 1;
 }
@@ -881,20 +956,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_BO("error: bo(%p) key(%lu)\n", bo, key);
-               _tbm_bo_mutex_unlock();
+               _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_BO("error: bo(%p) key(%lu)\n", bo, key);
-               _tbm_bo_mutex_unlock();
+               _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER);
+               _tbm_bufmgr_mutex_unlock();
                return 0;
        }
 
@@ -904,7 +982,7 @@ tbm_bo_set_user_data(tbm_bo bo, unsigned long key, void *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,21 +992,24 @@ 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_BO("error: bo(%p) key(%lu)\n", bo, key);
-               _tbm_bo_mutex_unlock();
+               _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_BO("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;
        }
 
@@ -936,25 +1017,17 @@ tbm_bo_get_user_data(tbm_bo bo, unsigned long key, void **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);
 
@@ -962,7 +1035,7 @@ tbm_bo_get_flags(tbm_bo bo)
 
        TBM_TRACE_BO("bo(%p)\n", bo);
 
-       _tbm_bo_mutex_unlock();
+       _tbm_bufmgr_mutex_unlock();
 
        return flags;
 }
@@ -972,13 +1045,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;
 }
@@ -1004,12 +1077,24 @@ _tbm_bo_free(tbm_bo bo)
        }
 
        /* call the bo_free */
-       bo->bufmgr->backend->bo_free(bo);
-       bo->priv = NULL;
+       if (bo->bufmgr->use_hal_tbm) {
+               // call hal_tbm_bo_free when bo is created by tbm_bo_alloc api.
+               if (!bo->get_from_hal_surface) {
+                       bo->get_from_hal_surface = 0;
 
-       bo->bufmgr->bo_cnt--;
+                       hal_tbm_bo_free(bo->bo_data);
+                       bo->bo_data = NULL;
+               }
+       } else if (bo->bufmgr->backend_module_data) {
+               bo->bufmgr->bo_func->bo_free(bo->bo_data);
+               bo->bo_data = NULL;
+       } else {
+               bo->bufmgr->backend->bo_free(bo);
+               bo->priv = NULL;
+       }
+
+       _tbm_bo_deinit(bo);
 
-       LIST_DEL(&bo->item_link);
        free(bo);
 }
 /* LCOV_EXCL_STOP */