X-Git-Url: http://review.tizen.org/git/?a=blobdiff_plain;f=src%2Ftbm_bo.c;h=39aac766ce8a337a473a4ceb8caa0441829189a7;hb=e565c41f907df4ed8a2b85d304bf11cc3ba0724d;hp=95d74781252347b0a7bc30c1b65d0d9bae89e551;hpb=24f1cd4d6464b189e5116bd8562151fdce5cd546;p=platform%2Fcore%2Fuifw%2Flibtbm.git

diff --git a/src/tbm_bo.c b/src/tbm_bo.c
index 95d7478..39aac76 100644
--- a/src/tbm_bo.c
+++ b/src/tbm_bo.c
@@ -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 */