X-Git-Url: http://review.tizen.org/git/?a=blobdiff_plain;f=src%2Ftbm_bo.c;h=f2870c92b55b3cae850b251ed725611d46a81532;hb=197f7ae3460ee6e7d55bf19c434c9c6495e767fa;hp=ed12dfc8f5d4c40ec122470939377d0ed713e431;hpb=14b37e60f4ce1d5667d6e07f85cbf58ff4dc0d78;p=platform%2Fcore%2Fuifw%2Flibtbm.git diff --git a/src/tbm_bo.c b/src/tbm_bo.c index ed12dfc..f2870c9 100644 --- a/src/tbm_bo.c +++ b/src/tbm_bo.c @@ -35,13 +35,13 @@ SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. #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_ERR("'%s' failed.\n", #cond);\ + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER);\ _tbm_bo_mutex_unlock();\ return;\ } \ @@ -49,18 +49,13 @@ static void _tbm_bo_mutex_unlock(void); #define TBM_BO_RETURN_VAL_IF_FAIL(cond, val) {\ if (!(cond)) {\ - TBM_LOG_E("'%s' failed.\n", #cond);\ + TBM_ERR("'%s' failed.\n", #cond);\ + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER);\ _tbm_bo_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) @@ -71,7 +66,7 @@ _tbm_bo_mutex_init(void) return true; if (pthread_mutex_init(&tbm_bo_lock, NULL)) { - TBM_LOG_E("fail: Cannot pthread_mutex_init for tbm_bo_lock.\n"); + TBM_ERR("fail: Cannot pthread_mutex_init for tbm_bo_lock.\n"); return false; } @@ -84,7 +79,7 @@ static void _tbm_bo_mutex_lock(void) { if (!_tbm_bo_mutex_init()) { - TBM_LOG_E("fail: _tbm_bo_mutex_init()\n"); + TBM_ERR("fail: _tbm_bo_mutex_init()\n"); return; } @@ -137,15 +132,15 @@ _tbm_util_check_bo_cnt(tbm_bufmgr bufmgr) 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_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,15 +155,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 */ } @@ -194,9 +190,24 @@ 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->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 +215,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; + + 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 @@ -221,7 +244,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; } @@ -247,13 +270,13 @@ _tbm_bo_lock(tbm_bo bo, int device, int opt) 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 +307,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,7 +315,7 @@ _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 @@ -302,28 +325,27 @@ _tbm_bo_is_valid(tbm_bo bo) tbm_bo old_data = NULL; if (bo == NULL) { - TBM_LOG_E("error: bo is NULL.\n"); + TBM_ERR("error: bo is NULL.\n"); return 0; } - bufmgr = bo->bufmgr; - - if (!TBM_BUFMGR_IS_VALID(bufmgr)) { - TBM_LOG_E("error: bo->bufmgr is not valid.\n"); + bufmgr = tbm_bufmgr_get(); + if (bufmgr == NULL) { + TBM_ERR("error: bufmgr is NULL.\n"); return 0; } - if (LIST_IS_EMPTY(&bo->bufmgr->bo_list)) { - TBM_LOG_E("error: bo->bo->bufmgr->bo_list is EMPTY.\n"); + if (LIST_IS_EMPTY(&bufmgr->bo_list)) { + TBM_ERR("error: bo->bo->bufmgr->bo_list is EMPTY.\n"); return 0; } - LIST_FOR_EACH_ENTRY(old_data, &bo->bufmgr->bo_list, item_link) { + LIST_FOR_EACH_ENTRY(old_data, &bufmgr->bo_list, item_link) { if (old_data == bo) return 1; } - TBM_LOG_E("error: No valid bo(%p).\n", bo); + TBM_ERR("error: No valid bo(%p).\n", bo); return 0; } @@ -331,10 +353,13 @@ _tbm_bo_is_valid(tbm_bo bo) tbm_bo tbm_bo_alloc(tbm_bufmgr bufmgr, int size, int flags) { - void *bo_priv; tbm_bo bo; + void *bo_priv; + tbm_backend_bo_data *bo_data; + tbm_error_e error; _tbm_bo_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); @@ -342,9 +367,9 @@ tbm_bo_alloc(tbm_bufmgr bufmgr, int size, int flags) bo = calloc(1, sizeof(struct _tbm_bo)); if (!bo) { /* LCOV_EXCL_START */ - TBM_LOG_E("error: fail to create of tbm_bo size(%d) flag(%s)\n", + 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_set_last_result(TBM_ERROR_OUT_OF_MEMORY); _tbm_bo_mutex_unlock(); return NULL; /* LCOV_EXCL_STOP */ @@ -354,25 +379,39 @@ tbm_bo_alloc(tbm_bufmgr bufmgr, int size, int flags) 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 */ + if (bufmgr->backend_module_data) { + bo_data = bufmgr->bufmgr_func->bufmgr_alloc_bo(bufmgr->bufmgr_data, (unsigned int)size, flags, &error); + if (!bo_data) { + /* 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(error); + free(bo); + _tbm_bo_mutex_unlock(); + return NULL; + /* LCOV_EXCL_STOP */ + } + bo->bo_data = bo_data; + } else { + 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_ERROR_INVALID_OPERATION); + free(bo); + _tbm_bo_mutex_unlock(); + return NULL; + /* LCOV_EXCL_STOP */ + } + bo->priv = bo_priv; } 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_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); @@ -388,12 +427,13 @@ tbm_bo tbm_bo_ref(tbm_bo bo) { _tbm_bo_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(); @@ -404,10 +444,11 @@ void tbm_bo_unref(tbm_bo bo) { _tbm_bo_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(); @@ -425,33 +466,47 @@ 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_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_ERR("error: fail to lock bo:%p)\n", bo); _tbm_bo_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_LOG_E("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->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); + _tbm_bo_unlock(bo); + _tbm_bo_mutex_unlock(); + return (tbm_bo_handle) NULL; + /* 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); + _tbm_bo_unlock(bo); + _tbm_bo_mutex_unlock(); + return (tbm_bo_handle) NULL; + /* LCOV_EXCL_STOP */ + } } /* 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(); @@ -461,27 +516,42 @@ tbm_bo_map(tbm_bo bo, int device, int opt) int tbm_bo_unmap(tbm_bo bo) { - int ret; + int ret = 1; + tbm_error_e error; _tbm_bo_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_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; - /* LCOV_EXCL_STOP */ + 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; + _tbm_bo_mutex_unlock(); + return ret; + /* 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); + _tbm_bo_mutex_unlock(); + return ret; + /* 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); @@ -494,22 +564,36 @@ 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_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_LOG_E("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->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); + _tbm_bo_mutex_unlock(); + return (tbm_bo_handle) NULL; + /* 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); + _tbm_bo_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(); @@ -520,29 +604,52 @@ tbm_key tbm_bo_export(tbm_bo bo) { tbm_key ret; + tbm_error_e error; _tbm_bo_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->backend_module_data) { + if (!bo->bufmgr->bo_func->bo_export_key) { + /* LCOV_EXCL_START */ + _tbm_bo_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_LOG_E("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); + _tbm_bo_mutex_unlock(); + return ret; + /* LCOV_EXCL_STOP */ + } + } else { + if (!bo->bufmgr->backend->bo_export) { + /* LCOV_EXCL_START */ + _tbm_bo_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); + _tbm_bo_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)\n", bo, ret); _tbm_bo_mutex_unlock(); @@ -553,29 +660,52 @@ tbm_fd tbm_bo_export_fd(tbm_bo bo) { tbm_fd ret; + tbm_error_e error; _tbm_bo_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->backend_module_data) { + if (!bo->bufmgr->bo_func->bo_export_fd) { + /* LCOV_EXCL_START */ + _tbm_bo_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_LOG_E("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); + _tbm_bo_mutex_unlock(); + return ret; + /* LCOV_EXCL_STOP */ + } + } else { + if (!bo->bufmgr->backend->bo_export_fd) { + /* LCOV_EXCL_START */ + _tbm_bo_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); + _tbm_bo_mutex_unlock(); + return ret; + /* LCOV_EXCL_STOP */ + } } - TBM_TRACE("bo(%p) tbm_fd(%d)\n", bo, ret); + TBM_TRACE_BO("bo(%p) tbm_fd(%d)\n", bo, ret); _tbm_bo_mutex_unlock(); @@ -588,16 +718,30 @@ tbm_bo_import(tbm_bufmgr bufmgr, unsigned int key) tbm_bo bo; tbm_bo bo2 = NULL; void *bo_priv; + tbm_error_e error; + tbm_backend_bo_data *bo_data; _tbm_bo_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 */ + if (bufmgr->backend_module_data) { + if (!bufmgr->bufmgr_func->bufmgr_import_key) { + /* LCOV_EXCL_START */ + _tbm_bo_mutex_unlock(); + _tbm_set_last_result(TBM_ERROR_NOT_SUPPORTED); + return NULL; + /* LCOV_EXCL_STOP */ + } + } else { + if (!bufmgr->backend->bo_import) { + /* LCOV_EXCL_START */ + _tbm_bo_mutex_unlock(); + _tbm_set_last_result(TBM_ERROR_NOT_SUPPORTED); + return NULL; + /* LCOV_EXCL_STOP */ + } } _tbm_util_check_bo_cnt(bufmgr); @@ -605,7 +749,8 @@ tbm_bo_import(tbm_bufmgr bufmgr, unsigned int key) bo = calloc(1, sizeof(struct _tbm_bo)); if (!bo) { /* LCOV_EXCL_START */ - TBM_LOG_E("error: fail to import of tbm_bo by key(%d)\n", key); + TBM_ERR("error: fail to import of tbm_bo by key(%d)\n", key); + _tbm_set_last_result(TBM_ERROR_OUT_OF_MEMORY); _tbm_bo_mutex_unlock(); return NULL; /* LCOV_EXCL_STOP */ @@ -613,42 +758,78 @@ tbm_bo_import(tbm_bufmgr bufmgr, unsigned int key) 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 (bo->bufmgr->backend_module_data) { + bo_data = bo->bufmgr->bufmgr_func->bufmgr_import_key(bufmgr->bufmgr_data, key, &error); + if (!bo_data) { + /* LCOV_EXCL_START */ + TBM_ERR("error: fail to import of tbm_bo by key(%d). error(%d)\n", key, error); + _tbm_set_last_result(error); + 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->bo_data == bo_data) { + 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->bo_data = bo_data; + } else { + 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_ERROR_INVALID_OPERATION); + 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; + 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->priv = bo_priv; } 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; + if (bo->bufmgr->backend_module_data) { + bo->flags = bo->bufmgr->bo_func->bo_get_memory_types(bo->bo_data, &error); + if (error != TBM_ERROR_NONE) { + TBM_ERR("fail to get the bo flags(memory_types)"); + _tbm_set_last_result(error); + bo->flags = TBM_BO_DEFAULT; + } + } else { + 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", + 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); @@ -666,16 +847,30 @@ tbm_bo_import_fd(tbm_bufmgr bufmgr, tbm_fd fd) tbm_bo bo; tbm_bo bo2 = NULL; void *bo_priv; + tbm_backend_bo_data *bo_data; + tbm_error_e error; _tbm_bo_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 */ + if (bufmgr->backend_module_data) { + if (!bufmgr->bufmgr_func->bufmgr_import_fd) { + /* LCOV_EXCL_START */ + _tbm_bo_mutex_unlock(); + _tbm_set_last_result(TBM_ERROR_NOT_SUPPORTED); + return NULL; + /* LCOV_EXCL_STOP */ + } + } else { + if (!bufmgr->backend->bo_import_fd) { + /* LCOV_EXCL_START */ + _tbm_bo_mutex_unlock(); + _tbm_set_last_result(TBM_ERROR_NOT_SUPPORTED); + return NULL; + /* LCOV_EXCL_STOP */ + } } _tbm_util_check_bo_cnt(bufmgr); @@ -683,7 +878,8 @@ tbm_bo_import_fd(tbm_bufmgr bufmgr, tbm_fd fd) bo = calloc(1, sizeof(struct _tbm_bo)); if (!bo) { /* LCOV_EXCL_START */ - TBM_LOG_E("error: fail to import tbm_bo by tbm_fd(%d)\n", fd); + TBM_ERR("error: fail to import tbm_bo by tbm_fd(%d)\n", fd); + _tbm_set_last_result(TBM_ERROR_OUT_OF_MEMORY); _tbm_bo_mutex_unlock(); return NULL; /* LCOV_EXCL_STOP */ @@ -691,43 +887,78 @@ tbm_bo_import_fd(tbm_bufmgr bufmgr, tbm_fd fd) 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 (bo->bufmgr->backend_module_data) { + bo_data = bo->bufmgr->bufmgr_func->bufmgr_import_fd(bufmgr->bufmgr_data, fd, &error); + if (!bo_data) { + /* LCOV_EXCL_START */ + TBM_ERR("error: fail to import tbm_bo by tbm_fd(%d). error(%d)\n", fd, error); + _tbm_set_last_result(error); + free(bo); + _tbm_bo_mutex_unlock(); + return NULL; + /* LCOV_EXCL_STOP */ + } - if (!LIST_IS_EMPTY(&bo->bufmgr->bo_list)) { + if (!LIST_IS_EMPTY(&bo->bufmgr->bo_list)) { + LIST_FOR_EACH_ENTRY(bo2, &bo->bufmgr->bo_list, item_link) { + if (bo2->bo_data == bo_data) { + 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->bo_data = bo_data; + } else { + 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_ERROR_INVALID_OPERATION); + free(bo); + _tbm_bo_mutex_unlock(); + return NULL; + /* LCOV_EXCL_STOP */ + } - 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; + 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->priv = bo_priv; } 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; + if (bo->bufmgr->backend_module_data) { + bo->flags = bo->bufmgr->bo_func->bo_get_memory_types(bo->bo_data, &error); + if (error != TBM_ERROR_NONE) { + TBM_ERR("fail to get the bo flags(memory_types)"); + _tbm_set_last_result(error); + bo->flags = TBM_BO_DEFAULT; + } + } else { + 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", + 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); @@ -743,14 +974,23 @@ int tbm_bo_size(tbm_bo bo) { int size; + tbm_error_e error; _tbm_bo_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->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(%p) size(%d)\n", bo, size); + TBM_TRACE_BO("bo(%p) size(%d)\n", bo, size); _tbm_bo_mutex_unlock(); @@ -761,22 +1001,24 @@ int tbm_bo_locked(tbm_bo bo) { _tbm_bo_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_ERR("bo(%p) lock_cnt(%d)\n", bo, bo->lock_cnt); + _tbm_set_last_result(TBM_ERROR_INVALID_OPERATION); _tbm_bo_mutex_unlock(); return 0; } if (bo->lock_cnt > 0) { - TBM_TRACE("error: bo(%p) lock_cnt(%d)\n", bo, bo->lock_cnt); + TBM_TRACE_BO("error: bo(%p) lock_cnt(%d)\n", bo, bo->lock_cnt); _tbm_bo_mutex_unlock(); return 1; } - TBM_TRACE("bo(%p) lock_cnt(%d)\n", bo, bo->lock_cnt); + TBM_TRACE_BO("bo(%p) lock_cnt(%d)\n", bo, bo->lock_cnt); _tbm_bo_mutex_unlock(); return 0; @@ -785,23 +1027,43 @@ 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_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; + 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("after: bo1(%p) bo2(%p)\n", bo1, bo2); + TBM_TRACE_BO("after: bo1(%p) bo2(%p)\n", bo1, bo2); temp = bo1->priv; bo1->priv = bo2->priv; @@ -810,6 +1072,12 @@ tbm_bo_swap(tbm_bo bo1, tbm_bo bo2) _tbm_bo_mutex_unlock(); return 1; + +fail: + TBM_ERR("error: bo1(%p) bo2(%p)\n", bo1, bo2); + _tbm_bo_mutex_unlock(); + + return 0; } int @@ -819,25 +1087,28 @@ tbm_bo_add_user_data(tbm_bo bo, unsigned long key, tbm_user_data *data; _tbm_bo_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_TRACE_BO("warning: user data already exist key(%ld)\n", key); + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER); _tbm_bo_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_ERR("error: bo(%p) key(%lu)\n", bo, key); + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER); _tbm_bo_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); @@ -852,23 +1123,26 @@ tbm_bo_delete_user_data(tbm_bo bo, unsigned long key) tbm_user_data *old_data; _tbm_bo_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_TRACE_BO("bo(%p) key(%lu)\n", bo, key); + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER); _tbm_bo_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_TRACE_BO("bo(%p) key(%lu)\n", bo, key); + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER); _tbm_bo_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); @@ -883,18 +1157,21 @@ tbm_bo_set_user_data(tbm_bo bo, unsigned long key, void *data) tbm_user_data *old_data; _tbm_bo_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_TRACE_BO("error: bo(%p) key(%lu)\n", bo, key); + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER); _tbm_bo_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_TRACE_BO("error: bo(%p) key(%lu)\n", bo, key); + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER); _tbm_bo_mutex_unlock(); return 0; } @@ -903,7 +1180,7 @@ 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(); @@ -916,40 +1193,35 @@ tbm_bo_get_user_data(tbm_bo bo, unsigned long key, void **data) tbm_user_data *old_data; _tbm_bo_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_TRACE_BO("error: bo(%p) key(%lu)\n", bo, key); + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER); _tbm_bo_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_TRACE_BO("error: bo(%p) key(%lu)\n", bo, key); + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER); _tbm_bo_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(); 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) { @@ -961,7 +1233,7 @@ tbm_bo_get_flags(tbm_bo bo) flags = bo->flags; - TBM_TRACE("bo(%p)\n", bo); + TBM_TRACE_BO("bo(%p)\n", bo); _tbm_bo_mutex_unlock(); @@ -999,14 +1271,19 @@ _tbm_bo_free(tbm_bo bo) } 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; + 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; + } bo->bufmgr->bo_cnt--;