adapt hal-api-tbm
[platform/core/uifw/libtbm.git] / src / tbm_bufmgr.c
index eaddfc0..c9872df 100644 (file)
@@ -50,7 +50,7 @@ int b_dump_queue;
 static pthread_mutex_t gLock = PTHREAD_MUTEX_INITIALIZER;
 static pthread_mutex_t tbm_bufmgr_lock = PTHREAD_MUTEX_INITIALIZER;
 static double scale_factor = 0;
-static void _tbm_bufmgr_mutex_unlock(void);
+void _tbm_bufmgr_mutex_unlock(void);
 
 //#define TBM_BUFMGR_INIT_TIME
 
@@ -76,6 +76,7 @@ static void _tbm_bufmgr_mutex_unlock(void);
 #define TBM_BUFMGR_RETURN_IF_FAIL(cond) {\
        if (!(cond)) {\
                TBM_ERR("'%s' failed.\n", #cond);\
+               _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER);\
                _tbm_bufmgr_mutex_unlock();\
                return;\
        } \
@@ -84,42 +85,21 @@ static void _tbm_bufmgr_mutex_unlock(void);
 #define TBM_BUFMGR_RETURN_VAL_IF_FAIL(cond, val) {\
        if (!(cond)) {\
                TBM_ERR("'%s' failed.\n", #cond);\
+               _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER);\
                _tbm_bufmgr_mutex_unlock();\
                return val;\
        } \
 }
 
 /* LCOV_EXCL_START */
-static bool
-_tbm_bufmgr_mutex_init(void)
-{
-       static bool tbm_bufmgr_mutex_init = false;
-
-       if (tbm_bufmgr_mutex_init)
-               return true;
-
-       if (pthread_mutex_init(&tbm_bufmgr_lock, NULL)) {
-               TBM_ERR("fail: Cannot pthread_mutex_init for tbm_bufmgr_lock.\n");
-               return false;
-       }
-
-       tbm_bufmgr_mutex_init = true;
-
-       return true;
-}
 
-static void
+void
 _tbm_bufmgr_mutex_lock(void)
 {
-       if (!_tbm_bufmgr_mutex_init()) {
-               TBM_ERR("fail: _tbm_bufmgr_mutex_init()\n");
-               return;
-       }
-
        pthread_mutex_lock(&tbm_bufmgr_lock);
 }
 
-static void
+void
 _tbm_bufmgr_mutex_unlock(void)
 {
        pthread_mutex_unlock(&tbm_bufmgr_lock);
@@ -177,6 +157,8 @@ _tbm_util_get_appname_from_pid(long pid, char *str)
        FILE *fp;
        int len;
 
+       if (pid <= 0) return;
+
        snprintf(fn_cmdline, sizeof(fn_cmdline), "/proc/%ld/cmdline", pid);
 
        fp = fopen(fn_cmdline, "r");
@@ -203,31 +185,83 @@ _tbm_util_get_appname_from_pid(long pid, char *str)
 }
 
 static int
+_tbm_backend_load_hal_tbm(tbm_bufmgr bufmgr)
+{
+       hal_tbm_backend *hal_backend = NULL;
+       hal_tbm_bufmgr_capability capability;
+       hal_tbm_bufmgr *hal_bufmgr;
+       hal_tbm_error ret = HAL_TBM_ERROR_NONE;
+
+       hal_backend = hal_tbm_get_backend(&ret);
+       if (hal_backend == NULL || ret != HAL_TBM_ERROR_NONE) {
+               TBM_ERR("get backend fail");
+               return 0;
+       }
+
+       hal_bufmgr = hal_tbm_backend_get_bufmgr(hal_backend, &ret);
+       if (hal_bufmgr == NULL || ret != HAL_TBM_ERROR_NONE) {
+               TBM_ERR("get hal_bufmgr fail");
+               goto get_backend_fail;
+       }
+
+       capability = hal_tbm_bufmgr_get_capabilities(hal_bufmgr, &ret);
+       if (ret != HAL_TBM_ERROR_NONE) {
+               TBM_ERR("hal_tbm_bufmgr_get_capabilities fail.");
+               goto get_backend_fail;
+       }
+
+       if (capability == HAL_TBM_BUFMGR_CAPABILITY_NONE) {
+               TBM_ERR("The capabilities of the backend module is TBM_BUFMGR_CAPABILITY_NONE.");
+               TBM_ERR("TBM_BUFMGR_CAPABILITY_SHARE_FD is the essential capability.");
+               goto get_backend_fail;
+       }
+       if (!(capability & HAL_TBM_BUFMGR_CAPABILITY_SHARE_FD)) {
+               TBM_ERR("The capabilities of the backend module had no TBM_BUFMGR_CAPABILITY_SHARE_FD.");
+               TBM_ERR("The tbm backend has to get TBM_BUFMGR_CAPABILITY_SHARE_FD. ");
+               goto get_backend_fail;
+       }
+       bufmgr->capabilities = capability;
+
+       bufmgr->hal_backend = hal_backend;
+       bufmgr->hal_bufmgr = hal_bufmgr;
+
+       bufmgr->use_hal_tbm = 1;
+
+       TBM_INFO("use hal tbm");
+
+       return 1;
+
+get_backend_fail:
+       hal_tbm_put_backend(hal_backend);
+       return 0;
+}
+
+static int
 _check_version(TBMModuleVersionInfo *data)
 {
-       int abimaj, abimin;
-       int vermaj, vermin;
+       int backend_module_major, backend_module_minor;
+       int tbm_backend_major, tbm_backend_minor;
 
-       abimaj = GET_ABI_MAJOR(data->abiversion);
-       abimin = GET_ABI_MINOR(data->abiversion);
+       backend_module_major = GET_ABI_MAJOR(data->abiversion);
+       backend_module_minor = GET_ABI_MINOR(data->abiversion);
 
        TBM_DBG("TBM module %s: vendor=\"%s\" ABI=%d,%d\n",
            data->modname ? data->modname : "UNKNOWN!",
-           data->vendor ? data->vendor : "UNKNOWN!", abimaj, abimin);
+           data->vendor ? data->vendor : "UNKNOWN!", backend_module_major, backend_module_minor);
 
-       vermaj = GET_ABI_MAJOR(TBM_ABI_VERSION);
-       vermin = GET_ABI_MINOR(TBM_ABI_VERSION);
+       tbm_backend_major = GET_ABI_MAJOR(TBM_ABI_VERSION);
+       tbm_backend_minor = GET_ABI_MINOR(TBM_ABI_VERSION);
 
        TBM_DBG("TBM ABI version %d.%d\n",
-           vermaj, vermin);
+           tbm_backend_major, tbm_backend_minor);
 
-       if (abimaj != vermaj) {
+       if (backend_module_major != tbm_backend_major) {
                TBM_ERR("TBM module ABI major ver(%d) doesn't match the TBM's ver(%d)\n",
-                       abimaj, vermaj);
+                       backend_module_major, tbm_backend_major);
                return 0;
-       } else if (abimin > vermin) {
+       } else if (backend_module_minor > tbm_backend_minor) {
                TBM_ERR("TBM module ABI minor ver(%d) is newer than the TBM's ver(%d)\n",
-                       abimin, vermin);
+                       backend_module_minor, tbm_backend_minor);
                return 0;
        }
 
@@ -280,8 +314,8 @@ _tbm_backend_load_module(tbm_bufmgr bufmgr, const char *file)
        void *module_data = NULL;
        tbm_backend_module *backend_module_data = NULL;
        tbm_backend_bufmgr_data *bufmgr_data = NULL;
-       int abimaj, abimin;
-       int vermaj, vermin;
+       int backend_module_major, backend_module_minor;
+       int tbm_backend_major, tbm_backend_minor;
        tbm_error_e error;
 
        snprintf(path, sizeof(path), BUFMGR_MODULE_DIR "/%s", file);
@@ -298,25 +332,25 @@ _tbm_backend_load_module(tbm_bufmgr bufmgr, const char *file)
                goto err;
        }
 
-       abimaj = GET_ABI_MAJOR(backend_module_data->abi_version);
-       abimin = GET_ABI_MINOR(backend_module_data->abi_version);
+       tbm_backend_major = GET_ABI_MAJOR(TBM_BACKEND_ABI_LATEST_VERSION);
+       tbm_backend_minor = GET_ABI_MINOR(TBM_BACKEND_ABI_LATEST_VERSION);
+       TBM_INFO("TBM Backend ABI version %d.%d\n", tbm_backend_major, tbm_backend_minor);
 
-       TBM_DBG("TBM module %s: vendor=\"%s\" ABI=%d,%d\n",
+       backend_module_major = GET_ABI_MAJOR(backend_module_data->abi_version);
+       backend_module_minor = GET_ABI_MINOR(backend_module_data->abi_version);
+
+       TBM_INFO("TBM module %s: vendor=\"%s\" Backend ABI version=%d.%d\n",
            backend_module_data->name ? backend_module_data->name : "UNKNOWN!",
            backend_module_data->vendor ? backend_module_data->vendor : "UNKNOWN!",
-               abimaj, abimin);
-
-       vermaj = GET_ABI_MAJOR(TBM_BACKEND_ABI_LATEST_VERSION);
-       vermin = GET_ABI_MINOR(TBM_BACKEND_ABI_LATEST_VERSION);
-       TBM_DBG("TBM ABI version %d.%d\n", vermaj, vermin);
+               backend_module_major, backend_module_minor);
 
-       if (abimaj > vermaj) {
+       if (backend_module_major > tbm_backend_major) {
                TBM_ERR("TBM module ABI major ver(%d) is newer than the TBM's ver(%d)\n",
-                       abimaj, vermaj);
+                       backend_module_major, tbm_backend_major);
                goto err;
-       } else if (abimin > vermin) {
+       } else if (backend_module_minor > tbm_backend_minor) {
                TBM_ERR("TBM module ABI minor ver(%d) is newer than the TBM's ver(%d)\n",
-                       abimin, vermin);
+                       backend_module_minor, tbm_backend_minor);
                goto err;
        }
 
@@ -365,7 +399,7 @@ _tbm_backend_load_module(tbm_bufmgr bufmgr, const char *file)
        bufmgr->backend_module_data = backend_module_data;
        bufmgr->bufmgr_data = bufmgr_data;
 
-       TBM_DBG("Success to load module(%s)\n", file);
+       TBM_INFO("Success to load module(%s)\n", file);
 
        return 1;
 
@@ -445,6 +479,11 @@ _tbm_load_module(tbm_bufmgr bufmgr, int fd)
        struct dirent **namelist;
        int ret = 0, n;
 
+       /* try to load the hal-tbm backend module */
+       ret = _tbm_backend_load_hal_tbm(bufmgr);
+       if (ret)
+               return 1;
+
        /* try to load the new backend module */
        ret = _tbm_backend_load_module(bufmgr, DEFAULT_LIB);
        if (ret)
@@ -496,16 +535,9 @@ _tbm_bufmgr_init(int fd, int server)
        gettimeofday(&start_tv, NULL);
 #endif
 
-       /* LCOV_EXCL_START */
-#ifdef HAVE_DLOG
-       env = getenv("TBM_DLOG");
-       if (env) {
-               bDlog = atoi(env);
-               TBM_DBG("TBM_DLOG=%s\n", env);
-       } else
-               bDlog = 1;
-#endif
+       bDlog = 1;
 
+       /* LCOV_EXCL_START */
 #ifdef TRACE
        env = getenv("TBM_TRACE");
        if (env) {
@@ -517,6 +549,8 @@ _tbm_bufmgr_init(int fd, int server)
 
        pthread_mutex_lock(&gLock);
 
+       _tbm_set_last_result(TBM_ERROR_NONE);
+
        if (fd >= 0) {
                TBM_WRN("!!!!!WARNING:: The tbm_bufmgr_init DOSE NOT use argument fd ANYMORE.\n");
                TBM_WRN("!!!!!WARNING:: IT WILL BE CHANGED like tbm_bufmgr_init(int fd) --> tbm_bufmgr_init(void).\n");
@@ -526,7 +560,7 @@ _tbm_bufmgr_init(int fd, int server)
        /* initialize buffer manager */
        if (gBufMgr) {
                gBufMgr->ref_count++;
-               TBM_INFO("reuse  tbm_bufmgr(%p) ref_count(%d) fd(%d)\n", gBufMgr, gBufMgr->ref_count, gBufMgr->fd);
+               TBM_DBG("reuse  tbm_bufmgr(%p) ref_count(%d) fd(%d)\n", gBufMgr, gBufMgr->ref_count, gBufMgr->fd);
                pthread_mutex_unlock(&gLock);
                return gBufMgr;
        }
@@ -537,6 +571,7 @@ _tbm_bufmgr_init(int fd, int server)
        gBufMgr = calloc(1, sizeof(struct _tbm_bufmgr));
        if (!gBufMgr) {
                TBM_ERR("error: fail to alloc bufmgr fd(%d)\n", fd);
+               _tbm_set_last_result(TBM_ERROR_OUT_OF_MEMORY);
                pthread_mutex_unlock(&gLock);
                return NULL;
        }
@@ -552,6 +587,7 @@ _tbm_bufmgr_init(int fd, int server)
        /* load bufmgr priv from env */
        if (!_tbm_load_module(gBufMgr, gBufMgr->fd)) {
                TBM_ERR("error : Fail to load bufmgr backend\n");
+               _tbm_set_last_result(TBM_ERROR_INVALID_OPERATION);
                free(gBufMgr);
                gBufMgr = NULL;
                pthread_mutex_unlock(&gLock);
@@ -618,6 +654,7 @@ tbm_bufmgr_deinit(tbm_bufmgr bufmgr)
 
        _tbm_bufmgr_mutex_lock();
        pthread_mutex_lock(&gLock);
+       _tbm_set_last_result(TBM_ERROR_NONE);
 
        if (!gBufMgr) {
                TBM_ERR("gBufmgr already destroy: bufmgr:%p\n", bufmgr);
@@ -628,7 +665,7 @@ tbm_bufmgr_deinit(tbm_bufmgr bufmgr)
 
        bufmgr->ref_count--;
        if (bufmgr->ref_count > 0) {
-               TBM_INFO("reduce a ref_count(%d) of tbm_bufmgr(%p)\n", bufmgr->ref_count, bufmgr);
+               TBM_DBG("reduce a ref_count(%d) of tbm_bufmgr(%p)\n", bufmgr->ref_count, bufmgr);
                pthread_mutex_unlock(&gLock);
                _tbm_bufmgr_mutex_unlock();
                return;
@@ -656,28 +693,34 @@ tbm_bufmgr_deinit(tbm_bufmgr bufmgr)
                LIST_DELINIT(&bufmgr->surf_list);
        }
 
-       if (bufmgr->backend_module_data) {
-               /* deinit and backend destroys the backend func and data */
-               bufmgr->backend_module_data->deinit(bufmgr->bufmgr_data);
-               bufmgr->bo_func = NULL;
-               bufmgr->bufmgr_func = NULL;
-               bufmgr->bufmgr_data = NULL;
-               bufmgr->backend_module_data = NULL;
+       if (bufmgr->use_hal_tbm) {
+               hal_tbm_put_backend(bufmgr->hal_backend);
+               bufmgr->hal_backend = NULL;
+               bufmgr->hal_bufmgr = NULL;
+               bufmgr->use_hal_tbm = 0;
        } else {
-               /* destroy bufmgr priv */
-               bufmgr->backend->bufmgr_deinit(bufmgr->backend->priv);
-               bufmgr->backend->priv = NULL;
-               tbm_backend_free(bufmgr->backend);
-               bufmgr->backend = NULL;
-       }
-
-       TBM_INFO("destroy tbm_bufmgr(%p)\n", bufmgr);
-
-       dlclose(bufmgr->module_data);
+               if (bufmgr->backend_module_data) {
+                       /* deinit and backend destroys the backend func and data */
+                       bufmgr->backend_module_data->deinit(bufmgr->bufmgr_data);
+                       bufmgr->bo_func = NULL;
+                       bufmgr->bufmgr_func = NULL;
+                       bufmgr->bufmgr_data = NULL;
+                       bufmgr->backend_module_data = NULL;
+               } else {
+                       /* destroy bufmgr priv */
+                       bufmgr->backend->bufmgr_deinit(bufmgr->backend->priv);
+                       bufmgr->backend->priv = NULL;
+                       tbm_backend_free(bufmgr->backend);
+                       bufmgr->backend = NULL;
+               }
 
+               dlclose(bufmgr->module_data);
+       }
        if (bufmgr->fd > 0)
                close(bufmgr->fd);
 
+       TBM_INFO("destroy tbm_bufmgr(%p)\n", bufmgr);
+
        free(bufmgr);
        gBufMgr = NULL;
 
@@ -691,6 +734,7 @@ tbm_bufmgr_get_capability(tbm_bufmgr bufmgr)
        unsigned int capabilities = TBM_BUFMGR_CAPABILITY_NONE;
 
        _tbm_bufmgr_mutex_lock();
+       _tbm_set_last_result(TBM_ERROR_NONE);
 
        TBM_BUFMGR_RETURN_VAL_IF_FAIL(TBM_BUFMGR_IS_VALID(bufmgr), TBM_BUFMGR_CAPABILITY_NONE);
        TBM_BUFMGR_RETURN_VAL_IF_FAIL(bufmgr == gBufMgr, TBM_BUFMGR_CAPABILITY_NONE);
@@ -713,11 +757,14 @@ tbm_bufmgr_debug_tbm_info_get(tbm_bufmgr bufmgr)
        int c = 0;
        int size;
        tbm_error_e error;
+       long pid = 0;
 
        pthread_mutex_lock(&gLock);
+       _tbm_set_last_result(TBM_ERROR_NONE);
 
        if (!TBM_BUFMGR_IS_VALID(bufmgr) || (bufmgr != gBufMgr)) {
                TBM_ERR("invalid bufmgr\n");
+               _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER);
                pthread_mutex_unlock(&gLock);
                return NULL;
        }
@@ -725,17 +772,19 @@ tbm_bufmgr_debug_tbm_info_get(tbm_bufmgr bufmgr)
        str = malloc(len);
        if (!str) {
                TBM_ERR("Fail to allocate the string.\n");
+               _tbm_set_last_result(TBM_ERROR_OUT_OF_MEMORY);
                pthread_mutex_unlock(&gLock);
                return NULL;
        }
 
        TBM_SNRPRINTF(str, len, c, "\n");
-       _tbm_util_get_appname_from_pid(getpid(), app_name);
+       pid = syscall(SYS_getpid);
+       _tbm_util_get_appname_from_pid(pid, app_name);
        _tbm_util_get_appname_brief(app_name);
-       TBM_SNRPRINTF(str, len, c, "============TBM DEBUG: %s(%d)===========================\n",
-                 app_name, getpid());
+       TBM_SNRPRINTF(str, len, c, "===========================================TBM DEBUG: %s(%ld)===========================================\n",
+                 app_name, pid);
 
-       snprintf(title, 255, "%s", "no  surface     refcnt  width  height  bpp  size    n_b  n_p  flags  format    app_name       ");
+       snprintf(title, 255, "%s", "no  surface     refcnt  width  height  bpp  size    n_b  n_p  flags    format    app_name              ");
 
        if (!LIST_IS_EMPTY(&bufmgr->debug_key_list)) {
                LIST_FOR_EACH_ENTRY(debug_old_data, &bufmgr->debug_key_list, item_link) {
@@ -754,20 +803,24 @@ tbm_bufmgr_debug_tbm_info_get(tbm_bufmgr bufmgr)
 
                LIST_FOR_EACH_ENTRY(surf, &bufmgr->surf_list, item_link) {
                        char data[512] = {0,};
-                       unsigned int pid;
+                       unsigned int surf_pid = 0;
                        int i;
 
-                       pid = _tbm_surface_internal_get_debug_pid(surf);
-                       if (!pid) {
+                       surf_pid = _tbm_surface_internal_get_debug_pid(surf);
+                       if (!surf_pid) {
                                /* if pid is null, set the self_pid */
-                               pid = getpid();
+                               surf_pid = syscall(SYS_getpid);;
                        }
 
                        memset(app_name, 0x0, 255 * sizeof(char));
-                       _tbm_util_get_appname_from_pid(pid, app_name);
-                       _tbm_util_get_appname_brief(app_name);
+                       if (geteuid() == 0) {
+                               _tbm_util_get_appname_from_pid(surf_pid, app_name);
+                               _tbm_util_get_appname_brief(app_name);
+                       } else {
+                               snprintf(app_name, sizeof(app_name), "%d", surf_pid);
+                       }
 
-                       snprintf(data, 255, "%-2d  %-9p    %-4d  %-5u  %-6u  %-3u  %-6u   %-2d   %-2d    %-3d  %-8s  %-15s",
+                       snprintf(data, 255, "%-3d %-11p   %-5d %-6u %-7u %-4u %-7u  %-3d  %-3d %-8d %-9s %-22s",
                                  ++surf_cnt,
                                  surf,
                                  surf->refcnt,
@@ -797,7 +850,11 @@ tbm_bufmgr_debug_tbm_info_get(tbm_bufmgr bufmgr)
                        TBM_SNRPRINTF(str, len, c, "%s\n", data);
 
                        for (i = 0; i < surf->num_bos; i++) {
-                               if (bufmgr->backend_module_data) {
+                               if (bufmgr->use_hal_tbm) {
+                                       size = hal_tbm_bo_get_size((hal_tbm_bo *)surf->bos[i]->bo_data, (hal_tbm_error *)&error);
+                                       if (error != TBM_ERROR_NONE)
+                                               TBM_WRN("fail to get the size of bo.");
+                               } else if (bufmgr->backend_module_data) {
                                        size = bufmgr->bo_func->bo_get_size(surf->bos[i]->bo_data, &error);
                                        if (error != TBM_ERROR_NONE)
                                                TBM_WRN("fail to get the size of bo.");
@@ -814,7 +871,7 @@ tbm_bufmgr_debug_tbm_info_get(tbm_bufmgr bufmgr)
        TBM_SNRPRINTF(str, len, c, "\n");
 
        TBM_SNRPRINTF(str, len, c, "[tbm_bo information]\n");
-       TBM_SNRPRINTF(str, len, c, "no  bo          refcnt  size    lock_cnt  map_cnt  flags  surface     name\n");
+       TBM_SNRPRINTF(str, len, c, "no  bo          refcnt  size    lock_cnt  map_cnt  flags   surface     name\n");
 
        /* show the tbm_bo information in bo_list */
        if (!LIST_IS_EMPTY(&bufmgr->bo_list)) {
@@ -823,7 +880,12 @@ tbm_bufmgr_debug_tbm_info_get(tbm_bufmgr bufmgr)
                tbm_key key = 0;
 
                LIST_FOR_EACH_ENTRY(bo, &bufmgr->bo_list, item_link) {
-                       if (bufmgr->backend_module_data) {
+                       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_WRN("fail to get the size of bo.");
+                               key = (tbm_key)hal_tbm_bo_export_key((hal_tbm_bo *)bo->bo_data, (hal_tbm_error *)&error);
+                       } else if (bufmgr->backend_module_data) {
                                size = bufmgr->bo_func->bo_get_size(bo->bo_data, &error);
                                if (error != TBM_ERROR_NONE)
                                        TBM_WRN("fail to get the size of bo.");
@@ -834,7 +896,7 @@ tbm_bufmgr_debug_tbm_info_get(tbm_bufmgr bufmgr)
                                size = bufmgr->backend->bo_size(bo);
                                key = bufmgr->backend->bo_export(bo);
                        }
-                       TBM_SNRPRINTF(str, len, c, "%-4d%-11p   %-4d  %-6d     %-5d     %-4u    %-3d  %-11p  %-4d\n",
+                       TBM_SNRPRINTF(str, len, c, "%-3d %-11p   %-5d %-7d    %-6d    %-5u %-7d %-11p  %-4d\n",
                                  ++bo_cnt,
                                  bo,
                                  bo->ref_cnt,
@@ -849,7 +911,7 @@ tbm_bufmgr_debug_tbm_info_get(tbm_bufmgr bufmgr)
                TBM_SNRPRINTF(str, len, c, "no tbm_bos.\n");
        TBM_SNRPRINTF(str, len, c, "\n");
 
-       TBM_SNRPRINTF(str, len, c, "===============================================================\n");
+       TBM_SNRPRINTF(str, len, c, "========================================================================================================\n");
 
        pthread_mutex_unlock(&gLock);
 
@@ -871,6 +933,7 @@ void
 tbm_bufmgr_debug_trace(tbm_bufmgr bufmgr, int onoff)
 {
        _tbm_bufmgr_mutex_lock();
+       _tbm_set_last_result(TBM_ERROR_NONE);
 
        TBM_BUFMGR_RETURN_IF_FAIL(TBM_BUFMGR_IS_VALID(bufmgr));
        TBM_BUFMGR_RETURN_IF_FAIL(bufmgr == gBufMgr);
@@ -887,6 +950,7 @@ void
 tbm_bufmgr_debug_set_trace_mask(tbm_bufmgr bufmgr, tbm_bufmgr_debug_trace_mask mask, int set)
 {
        _tbm_bufmgr_mutex_lock();
+       _tbm_set_last_result(TBM_ERROR_NONE);
 
        TBM_BUFMGR_RETURN_IF_FAIL(TBM_BUFMGR_IS_VALID(bufmgr));
        TBM_BUFMGR_RETURN_IF_FAIL(bufmgr == gBufMgr);
@@ -926,6 +990,7 @@ void
 tbm_bufmgr_debug_dump_set_scale(double scale)
 {
        pthread_mutex_lock(&gLock);
+       _tbm_set_last_result(TBM_ERROR_NONE);
        scale_factor = scale;
        pthread_mutex_unlock(&gLock);
 }
@@ -933,13 +998,24 @@ tbm_bufmgr_debug_dump_set_scale(double scale)
 int
 tbm_bufmgr_debug_get_ref_count(void)
 {
-       return (gBufMgr) ? gBufMgr->ref_count : 0;
+       int refcnt;
+
+       pthread_mutex_lock(&gLock);
+
+       _tbm_set_last_result(TBM_ERROR_NONE);
+
+       refcnt = (gBufMgr) ? gBufMgr->ref_count : 0;
+
+       pthread_mutex_unlock(&gLock);
+
+       return refcnt;
 }
 
 int
 tbm_bufmgr_debug_queue_dump(char *path, int count, int onoff)
 {
        pthread_mutex_lock(&gLock);
+       _tbm_set_last_result(TBM_ERROR_NONE);
 
        if (onoff == 0) {
                TBM_DBG("count=%d onoff=%d\n", count, onoff);
@@ -968,6 +1044,7 @@ tbm_bufmgr_debug_queue_dump(char *path, int count, int onoff)
        }
 
        pthread_mutex_unlock(&gLock);
+
        return 1;
 }
 
@@ -977,16 +1054,22 @@ tbm_bufmgr_debug_dump_all(char *path)
        int w, h, count = 0;
        tbm_surface_h surface = NULL;
 
-       TBM_RETURN_VAL_IF_FAIL(path != NULL, 0);
-       TBM_DBG("path=%s\n", path);
-
        pthread_mutex_lock(&gLock);
+       _tbm_set_last_result(TBM_ERROR_NONE);
+
+       if (!path) {
+               TBM_ERR("path is null.\n");
+               pthread_mutex_unlock(&gLock);
+               return 0;
+       }
+
+       TBM_DBG("path=%s\n", path);
 
        count = _tbm_util_get_max_surface_size(&w, &h);
        if (count == 0) {
                TBM_ERR("No tbm_surface.\n");
                pthread_mutex_unlock(&gLock);
-               return 1;
+               return 0;
        }
 
        tbm_surface_internal_dump_with_scale_start(path, w, h, count, scale_factor);
@@ -1016,13 +1099,28 @@ tbm_bufmgr_bind_native_display(tbm_bufmgr bufmgr, void *native_display)
        tbm_error_e error;
 
        _tbm_bufmgr_mutex_lock();
+       _tbm_set_last_result(TBM_ERROR_NONE);
 
        TBM_BUFMGR_RETURN_VAL_IF_FAIL(TBM_BUFMGR_IS_VALID(gBufMgr), 0);
 
-       if (bufmgr->backend_module_data) {
+       if (bufmgr->use_hal_tbm) {
+               error = (tbm_error_e)hal_tbm_bufmgr_bind_native_display(bufmgr->hal_bufmgr, (hal_tbm_native_display *)native_display);
+               if (error == TBM_ERROR_NOT_SUPPORTED) {
+                       TBM_WRN("skip: tbm_bufmgr(%p) native_display(%p)\n", bufmgr, native_display);
+                       _tbm_set_last_result(TBM_ERROR_NOT_SUPPORTED);
+                       _tbm_bufmgr_mutex_unlock();
+                       return 1;
+               } else if (error != TBM_ERROR_NONE) {
+                       TBM_ERR("error: tbm_bufmgr(%p) native_display(%p)\n", bufmgr, native_display);
+                       _tbm_set_last_result(TBM_ERROR_INVALID_OPERATION);
+                       _tbm_bufmgr_mutex_unlock();
+                       return 0;
+               }
+       } else if (bufmgr->backend_module_data) {
                if (!bufmgr->bufmgr_func->bufmgr_bind_native_display) {
                        TBM_WRN("skip: tbm_bufmgr(%p) native_display(%p)\n",
                                        bufmgr, native_display);
+                       _tbm_set_last_result(TBM_ERROR_NOT_SUPPORTED);
                        _tbm_bufmgr_mutex_unlock();
                        return 1;
                }
@@ -1031,6 +1129,7 @@ tbm_bufmgr_bind_native_display(tbm_bufmgr bufmgr, void *native_display)
                if (error != TBM_ERROR_NONE) {
                        TBM_ERR("error: tbm_bufmgr(%p) native_display(%p) error(%d)\n",
                                        bufmgr, native_display, error);
+                       _tbm_set_last_result(error);
                        _tbm_bufmgr_mutex_unlock();
                        return 0;
                }
@@ -1039,6 +1138,7 @@ tbm_bufmgr_bind_native_display(tbm_bufmgr bufmgr, void *native_display)
                if (!bufmgr->backend->bufmgr_bind_native_display) {
                        TBM_WRN("skip: tbm_bufmgr(%p) native_display(%p)\n",
                                        bufmgr, native_display);
+                       _tbm_set_last_result(TBM_ERROR_NOT_SUPPORTED);
                        _tbm_bufmgr_mutex_unlock();
                        return 1;
                }
@@ -1047,6 +1147,7 @@ tbm_bufmgr_bind_native_display(tbm_bufmgr bufmgr, void *native_display)
                if (!ret) {
                        TBM_ERR("error: tbm_bufmgr(%p) native_display(%p)\n",
                                        bufmgr, native_display);
+                       _tbm_set_last_result(TBM_ERROR_INVALID_OPERATION);
                        _tbm_bufmgr_mutex_unlock();
                        return 0;
                }
@@ -1073,6 +1174,7 @@ int
 tbm_bufmgr_set_bo_lock_type(tbm_bufmgr bufmgr, tbm_bufmgr_bo_lock_type bo_lock_type)
 {
        _tbm_bufmgr_mutex_lock();
+       _tbm_set_last_result(TBM_ERROR_NONE);
 
        TBM_BUFMGR_RETURN_VAL_IF_FAIL(TBM_BUFMGR_IS_VALID(gBufMgr), 0);
        TBM_BUFMGR_RETURN_VAL_IF_FAIL(bufmgr == gBufMgr, 0);