X-Git-Url: http://review.tizen.org/git/?a=blobdiff_plain;f=src%2Ftbm_surface_internal.c;h=6fc9b55d97ee8fd57e32fd693338ec57e7ae1a2a;hb=757968fc683975d882aadf1b45a54e214bbd4783;hp=cf7fb7352d2753ad48b13f3045634a3a5eeb5c9d;hpb=9612bdcdaa6789d0c331fc54b38f9259681bb776;p=platform%2Fcore%2Fuifw%2Flibtbm.git diff --git a/src/tbm_surface_internal.c b/src/tbm_surface_internal.c index cf7fb73..6fc9b55 100644 --- a/src/tbm_surface_internal.c +++ b/src/tbm_surface_internal.c @@ -39,6 +39,7 @@ SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. #include "tbm_surface_internal.h" #include "list.h" #include +#include static tbm_bufmgr g_surface_bufmgr; static pthread_mutex_t tbm_surface_lock; @@ -53,7 +54,8 @@ void _tbm_surface_mutex_unlock(void); /* check condition */ #define TBM_SURFACE_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_surface_mutex_unlock();\ return;\ } \ @@ -61,14 +63,14 @@ void _tbm_surface_mutex_unlock(void); #define TBM_SURFACE_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_surface_mutex_unlock();\ return val;\ } \ } /* LCOV_EXCL_START */ - static double _tbm_surface_internal_get_time(void) { @@ -217,7 +219,6 @@ _tbm_surface_internal_format_to_str(tbm_format format) return "unknwon"; } } -/* LCOV_EXCL_STOP */ static bool _tbm_surface_mutex_init(void) @@ -228,7 +229,7 @@ _tbm_surface_mutex_init(void) return true; if (pthread_mutex_init(&tbm_surface_lock, NULL)) { - TBM_LOG_E("fail: tbm_surface mutex init\n"); + TBM_ERR("fail: pthread_mutex_init for tbm_surface_lock.\n"); return false; } @@ -240,8 +241,10 @@ _tbm_surface_mutex_init(void) void _tbm_surface_mutex_lock(void) { - if (!_tbm_surface_mutex_init()) + if (!_tbm_surface_mutex_init()) { + TBM_ERR("fail: _tbm_surface_mutex_init.\n"); return; + } pthread_mutex_lock(&tbm_surface_lock); } @@ -267,6 +270,31 @@ _deinit_surface_bufmgr(void) tbm_bufmgr_deinit(g_surface_bufmgr); g_surface_bufmgr = NULL; } +/* LCOV_EXCL_STOP */ + +static int +_tbm_surface_internal_is_valid(tbm_surface_h surface) +{ + tbm_surface_h old_data = NULL; + + TBM_RETURN_VAL_IF_FAIL(g_surface_bufmgr, 0); + TBM_RETURN_VAL_IF_FAIL(surface, 0); + + if (!LIST_IS_EMPTY(&g_surface_bufmgr->surf_list)) { + LIST_FOR_EACH_ENTRY(old_data, &g_surface_bufmgr->surf_list, item_link) { + if (old_data == surface) { + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p)\n", surface); + return 1; + } + } + } + + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER); + + TBM_ERR("error: No valid tbm_surface(%p)\n", surface); + + return 0; +} static int _tbm_surface_internal_query_plane_data(tbm_surface_h surface, @@ -276,23 +304,49 @@ _tbm_surface_internal_query_plane_data(tbm_surface_h surface, TBM_RETURN_VAL_IF_FAIL(plane_idx > -1, 0); struct _tbm_surface *surf = (struct _tbm_surface *)surface; - struct _tbm_bufmgr *mgr = surf->bufmgr; + struct _tbm_bufmgr *bufmgr = surf->bufmgr; int ret = 0; + tbm_error_e error; - TBM_RETURN_VAL_IF_FAIL(mgr != NULL, 0); + TBM_RETURN_VAL_IF_FAIL(bufmgr != NULL, 0); TBM_RETURN_VAL_IF_FAIL(surf->info.width > 0, 0); TBM_RETURN_VAL_IF_FAIL(surf->info.height > 0, 0); TBM_RETURN_VAL_IF_FAIL(surf->info.format > 0, 0); - if (!mgr->backend->surface_get_plane_data) - return 0; + if (bufmgr->backend_module_data) { + if (!bufmgr->bufmgr_func->bufmgr_get_plane_data) { + _tbm_set_last_result(TBM_ERROR_NOT_SUPPORTED); + return 0; + } - ret = mgr->backend->surface_get_plane_data(surf->info.width, - surf->info.height, surf->info.format, plane_idx, size, offset, pitch, bo_idx); - if (!ret) - return 0; + error = bufmgr->bufmgr_func->bufmgr_get_plane_data(bufmgr->bufmgr_data, surf->info.format, plane_idx, + surf->info.width, surf->info.height, size, offset, pitch, bo_idx); + if (error != TBM_ERROR_NONE) { + /* LCOV_EXCL_START */ + TBM_ERR("Fail to surface_get_plane_data. surface(%p) error(%d)\n", surface, error); + _tbm_set_last_result(error); + return 0; + /* LCOV_EXCL_STOP */ + } + ret = 1; + } else { + if (!bufmgr->backend->surface_get_plane_data) { + _tbm_set_last_result(TBM_ERROR_NOT_SUPPORTED); + return 0; + } - return 1; + ret = bufmgr->backend->surface_get_plane_data(surf->info.width, + surf->info.height, surf->info.format, plane_idx, size, offset, pitch, bo_idx); + if (!ret) { + /* LCOV_EXCL_START */ + TBM_ERR("Fail to surface_get_plane_data. surface(%p)\n", surface); + _tbm_set_last_result(TBM_ERROR_INVALID_OPERATION); + return 0; + /* LCOV_EXCL_STOP */ + } + } + + return ret; } static void @@ -341,71 +395,27 @@ _tbm_surface_internal_destroy(tbm_surface_h surface) } } -int -tbm_surface_internal_is_valid(tbm_surface_h surface) +/* LCOV_EXCL_START */ +static int +_tbm_surface_check_file_is_symbolic_link(const char* path) { - tbm_surface_h old_data = NULL; + struct stat sb; - if (surface == NULL || g_surface_bufmgr == NULL) { - TBM_TRACE("error: tbm_surface(%p)\n", surface); + if (!path) return 0; - } - - if (!LIST_IS_EMPTY(&g_surface_bufmgr->surf_list)) { - LIST_FOR_EACH_ENTRY(old_data, &g_surface_bufmgr->surf_list, item_link) { - if (old_data == surface) { - TBM_TRACE("tbm_surface(%p)\n", surface); - return 1; - } - } - } - TBM_TRACE("error: tbm_surface(%p)\n", surface); - return 0; -} - -int -tbm_surface_internal_query_supported_formats(uint32_t **formats, - uint32_t *num) -{ - struct _tbm_bufmgr *mgr; - int ret = 0; - bool bufmgr_initialized = false; - - _tbm_surface_mutex_lock(); - - if (!g_surface_bufmgr) { - _init_surface_bufmgr(); - LIST_INITHEAD(&g_surface_bufmgr->surf_list); - bufmgr_initialized = true; - } - - mgr = g_surface_bufmgr; - - if (!mgr->backend->surface_supported_format) - goto fail; - - ret = mgr->backend->surface_supported_format(formats, num); - if (!ret) - goto fail; - TBM_TRACE("tbm_bufmgr(%p) format num(%u)\n", g_surface_bufmgr, *num); - - _tbm_surface_mutex_unlock(); + if (stat(path, &sb) != 0) + return 0; - return ret; + if (S_ISLNK(sb.st_mode)) + return 1; -fail: - if (bufmgr_initialized) { - LIST_DELINIT(&g_surface_bufmgr->surf_list); - _deinit_surface_bufmgr(); - } - _tbm_surface_mutex_unlock(); - TBM_TRACE("error: tbm_bufmgr(%p)\n", g_surface_bufmgr); return 0; } +/* LCOV_EXCL_STOP */ -int -tbm_surface_internal_get_num_planes(tbm_format format) +static int +_tbm_surface_internal_get_num_planes(tbm_format format) { int num_planes = 0; @@ -477,16 +487,15 @@ tbm_surface_internal_get_num_planes(tbm_format format) break; default: + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER); break; } - TBM_TRACE("tbm_format(%s) num_planes(%d)\n", _tbm_surface_internal_format_to_str(format), num_planes); - return num_planes; } -int -tbm_surface_internal_get_bpp(tbm_format format) +static int +_tbm_surface_internal_get_bpp(tbm_format format) { int bpp = 0; @@ -572,10 +581,153 @@ tbm_surface_internal_get_bpp(tbm_format format) bpp = 24; break; default: + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER); break; } - TBM_TRACE("tbm_format(%s) bpp(%d)\n", _tbm_surface_internal_format_to_str(format), bpp); + return bpp; +} + +int +tbm_surface_internal_is_valid(tbm_surface_h surface) +{ + int ret = 0; + + _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); + + /* Return silently if surface is null. */ + if (!surface) { + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER); + _tbm_surface_mutex_unlock(); + return 0; + } + + ret = _tbm_surface_internal_is_valid(surface); + + _tbm_surface_mutex_unlock(); + + return ret; +} + +int +tbm_surface_internal_query_supported_formats(uint32_t **formats, + uint32_t *num) +{ + struct _tbm_bufmgr *bufmgr; + int ret = 0; + bool bufmgr_initialized = false; + tbm_error_e error; + + _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); + + TBM_SURFACE_RETURN_VAL_IF_FAIL(formats, 0); + TBM_SURFACE_RETURN_VAL_IF_FAIL(num, 0); + + if (!g_surface_bufmgr) { + _init_surface_bufmgr(); + LIST_INITHEAD(&g_surface_bufmgr->surf_list); + bufmgr_initialized = true; + } + + bufmgr = g_surface_bufmgr; + + if (bufmgr->backend_module_data) { + if (!bufmgr->bufmgr_func->bufmgr_get_supported_formats) { + _tbm_set_last_result(TBM_ERROR_NOT_SUPPORTED); + goto fail; + } + + error = bufmgr->bufmgr_func->bufmgr_get_supported_formats(bufmgr->bufmgr_data, formats, num); + if (error != TBM_ERROR_NONE) { + /* LCOV_EXCL_START */ + TBM_ERR("Fail to surface_supported_format. error(%d)\n", error); + goto fail; + /* LCOV_EXCL_START */ + } + ret = 1; + } else { + if (!bufmgr->backend->surface_supported_format) { + _tbm_set_last_result(TBM_ERROR_NOT_SUPPORTED); + goto fail; + } + + ret = bufmgr->backend->surface_supported_format(formats, num); + if (!ret) { + /* LCOV_EXCL_START */ + TBM_ERR("Fail to surface_supported_format.\n"); + _tbm_set_last_result(TBM_ERROR_INVALID_OPERATION); + goto fail; + /* LCOV_EXCL_START */ + } + } + + TBM_TRACE_SURFACE_INTERNAL("tbm_bufmgr(%p) format num(%u)\n", g_surface_bufmgr, *num); + + if (bufmgr_initialized) { + LIST_DELINIT(&g_surface_bufmgr->surf_list); + _deinit_surface_bufmgr(); + } + + _tbm_surface_mutex_unlock(); + + return ret; + +/* LCOV_EXCL_START */ +fail: + if (bufmgr_initialized) { + LIST_DELINIT(&g_surface_bufmgr->surf_list); + _deinit_surface_bufmgr(); + } + _tbm_surface_mutex_unlock(); + + TBM_ERR("error: tbm_bufmgr(%p)\n", g_surface_bufmgr); + + return 0; +/* LCOV_EXCL_STOP */ +} + +int +tbm_surface_internal_get_num_planes(tbm_format format) +{ + int num_planes = 0; + + _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); + + num_planes = _tbm_surface_internal_get_num_planes(format); + if (!num_planes) { + TBM_ERR("error: tbm_error(%s)\n", tbm_error_str(tbm_get_last_error())); + _tbm_surface_mutex_unlock(); + return 0; + } + + TBM_TRACE_SURFACE_INTERNAL("tbm_format(%s) num_planes(%d)\n", _tbm_surface_internal_format_to_str(format), num_planes); + + _tbm_surface_mutex_unlock(); + + return num_planes; +} + +int +tbm_surface_internal_get_bpp(tbm_format format) +{ + int bpp = 0; + + _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); + + bpp = _tbm_surface_internal_get_bpp(format); + if (!bpp) { + TBM_ERR("error: tbm_error(%s)\n", tbm_error_str(tbm_get_last_error())); + _tbm_surface_mutex_unlock(); + return 0; + } + + _tbm_surface_mutex_unlock(); + + TBM_TRACE_SURFACE_INTERNAL("tbm_format(%s) bpp(%d)\n", _tbm_surface_internal_format_to_str(format), bpp); return bpp; } @@ -584,10 +736,7 @@ tbm_surface_h tbm_surface_internal_create_with_flags(int width, int height, int format, int flags) { - TBM_RETURN_VAL_IF_FAIL(width > 0, NULL); - TBM_RETURN_VAL_IF_FAIL(height > 0, NULL); - - struct _tbm_bufmgr *mgr; + struct _tbm_bufmgr *bufmgr; struct _tbm_surface *surf = NULL; uint32_t size = 0; uint32_t offset = 0; @@ -596,8 +745,16 @@ tbm_surface_internal_create_with_flags(int width, int height, int bo_idx; int i, j; bool bufmgr_initialized = false; + tbm_bo bo = NULL; + void *bo_priv = NULL; + tbm_backend_bo_data *bo_data = NULL; + tbm_error_e error; _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); + + TBM_SURFACE_RETURN_VAL_IF_FAIL(width > 0, NULL); + TBM_SURFACE_RETURN_VAL_IF_FAIL(height > 0, NULL); if (!g_surface_bufmgr) { _init_surface_bufmgr(); @@ -605,31 +762,42 @@ tbm_surface_internal_create_with_flags(int width, int height, bufmgr_initialized = true; } - mgr = g_surface_bufmgr; - if (!TBM_BUFMGR_IS_VALID(mgr)) { - TBM_LOG_E("The bufmgr is invalid\n"); + bufmgr = g_surface_bufmgr; + if (!TBM_BUFMGR_IS_VALID(bufmgr)) { + TBM_ERR("The bufmgr is invalid\n"); goto check_valid_fail; } surf = calloc(1, sizeof(struct _tbm_surface)); if (!surf) { - TBM_LOG_E("fail to alloc surf\n"); + /* LCOV_EXCL_START */ + TBM_ERR("fail to alloc surf\n"); + _tbm_set_last_result(TBM_ERROR_OUT_OF_MEMORY); goto alloc_surf_fail; + /* LCOV_EXCL_STOP */ } - surf->bufmgr = mgr; + surf->bufmgr = bufmgr; surf->info.width = width; surf->info.height = height; surf->info.format = format; - surf->info.bpp = tbm_surface_internal_get_bpp(format); - surf->info.num_planes = tbm_surface_internal_get_num_planes(format); + surf->info.bpp = _tbm_surface_internal_get_bpp(format); + if (!surf->info.bpp) { + TBM_ERR("fail to get bpp. error(%s)\n", tbm_error_str(tbm_get_last_error())); + goto bpp_fail; + } + surf->info.num_planes = _tbm_surface_internal_get_num_planes(format); + if (!surf->info.num_planes) { + TBM_ERR("fail to get num_planes. error(%s)\n", tbm_error_str(tbm_get_last_error())); + goto num_planes_fail; + } surf->refcnt = 1; /* get size, stride and offset bo_idx */ for (i = 0; i < surf->info.num_planes; i++) { if (!_tbm_surface_internal_query_plane_data(surf, i, &size, &offset, &stride, &bo_idx)) { - TBM_LOG_E("fail to query plane data\n"); + TBM_ERR("fail to query plane data\n"); goto query_plane_data_fail; } @@ -657,81 +825,129 @@ tbm_surface_internal_create_with_flags(int width, int height, bo_size += surf->info.planes[j].size; } - if (mgr->backend->surface_bo_alloc) { - /* LCOV_EXCL_START */ - tbm_bo bo = NULL; - void *bo_priv = NULL; + if (bufmgr->backend_module_data) { + if (bufmgr->bufmgr_func->bufmgr_alloc_bo_with_format) { + /* LCOV_EXCL_START */ + bo = calloc(1, sizeof(struct _tbm_bo)); + if (!bo) { + TBM_ERR("fail to alloc bo struct\n"); + _tbm_set_last_result(TBM_ERROR_OUT_OF_MEMORY); + goto alloc_bo_fail; + } - bo = calloc(1, sizeof(struct _tbm_bo)); - if (!bo) { - TBM_LOG_E("fail to alloc bo struct\n"); - goto alloc_bo_fail; - } + bo->bufmgr = surf->bufmgr; + + pthread_mutex_lock(&surf->bufmgr->lock); + + bo_data = bufmgr->bufmgr_func->bufmgr_alloc_bo_with_format(bufmgr->bufmgr_data, format, i, + width, height, flags, &error); + if (!bo_data) { + TBM_ERR("fail to alloc bo priv. error(%d)\n", error); + _tbm_set_last_result(error); + free(bo); + pthread_mutex_unlock(&surf->bufmgr->lock); + goto alloc_bo_fail; + } + bo->bo_data = bo_data; - bo->bufmgr = surf->bufmgr; + bo->ref_cnt = 1; + bo->flags = flags; + LIST_INITHEAD(&bo->user_data_list); - pthread_mutex_lock(&surf->bufmgr->lock); + LIST_ADD(&bo->item_link, &surf->bufmgr->bo_list); - bo_priv = mgr->backend->surface_bo_alloc(bo, width, height, format, flags, i); - if (!bo_priv) { - TBM_LOG_E("fail to alloc bo priv\n"); - free(bo); pthread_mutex_unlock(&surf->bufmgr->lock); - goto alloc_bo_fail; + + surf->bos[i] = bo; + /* LCOV_EXCL_STOP */ + } else { + surf->bos[i] = tbm_bo_alloc(bufmgr, bo_size, flags); + if (!surf->bos[i]) { + TBM_ERR("fail to alloc bo idx:%d\n", i); + goto alloc_bo_fail; + } } + } else { + if (bufmgr->backend->surface_bo_alloc) { + /* LCOV_EXCL_START */ + bo = calloc(1, sizeof(struct _tbm_bo)); + if (!bo) { + TBM_ERR("fail to alloc bo struct\n"); + _tbm_set_last_result(TBM_ERROR_OUT_OF_MEMORY); + goto alloc_bo_fail; + } - bo->ref_cnt = 1; - bo->flags = flags; - bo->priv = bo_priv; + bo->bufmgr = surf->bufmgr; - LIST_INITHEAD(&bo->user_data_list); + pthread_mutex_lock(&surf->bufmgr->lock); - LIST_ADD(&bo->item_link, &surf->bufmgr->bo_list); + bo_priv = bufmgr->backend->surface_bo_alloc(bo, width, height, format, flags, i); + if (!bo_priv) { + TBM_ERR("fail to alloc bo priv\n"); + _tbm_set_last_result(TBM_ERROR_INVALID_OPERATION); + free(bo); + pthread_mutex_unlock(&surf->bufmgr->lock); + goto alloc_bo_fail; + } + bo->priv = bo_priv; - pthread_mutex_unlock(&surf->bufmgr->lock); + bo->ref_cnt = 1; + bo->flags = flags; + LIST_INITHEAD(&bo->user_data_list); - surf->bos[i] = bo; - /* LCOV_EXCL_STOP */ - } else { - surf->bos[i] = tbm_bo_alloc(mgr, bo_size, flags); - if (!surf->bos[i]) { - TBM_LOG_E("fail to alloc bo idx:%d\n", i); - goto alloc_bo_fail; + LIST_ADD(&bo->item_link, &surf->bufmgr->bo_list); + + pthread_mutex_unlock(&surf->bufmgr->lock); + + surf->bos[i] = bo; + /* LCOV_EXCL_STOP */ + } else { + surf->bos[i] = tbm_bo_alloc(bufmgr, bo_size, flags); + if (!surf->bos[i]) { + TBM_ERR("fail to alloc bo idx:%d\n", i); + goto alloc_bo_fail; + } } } _tbm_bo_set_surface(surf->bos[i], surf); } - TBM_TRACE("width(%d) height(%d) format(%s) flags(%d) tbm_surface(%p)\n", width, height, + TBM_TRACE_SURFACE_INTERNAL("width(%d) height(%d) format(%s) flags(%d) tbm_surface(%p)\n", width, height, _tbm_surface_internal_format_to_str(format), flags, surf); LIST_INITHEAD(&surf->user_data_list); LIST_INITHEAD(&surf->debug_data_list); - LIST_ADD(&surf->item_link, &mgr->surf_list); + LIST_ADD(&surf->item_link, &bufmgr->surf_list); _tbm_surface_mutex_unlock(); return surf; +/* LCOV_EXCL_START */ alloc_bo_fail: for (j = 0; j < i; j++) { if (surf->bos[j]) tbm_bo_unref(surf->bos[j]); } query_plane_data_fail: +bpp_fail: +num_planes_fail: free(surf); alloc_surf_fail: check_valid_fail: - if (bufmgr_initialized) { - LIST_DELINIT(&mgr->surf_list); + if (bufmgr_initialized && bufmgr) { + LIST_DELINIT(&bufmgr->surf_list); _deinit_surface_bufmgr(); } _tbm_surface_mutex_unlock(); - TBM_TRACE("error: width(%d) height(%d) format(%s) flags(%d)\n", + + TBM_ERR("error: width(%d) height(%d) format(%s) flags(%d)\n", width, height, _tbm_surface_internal_format_to_str(format), flags); +/* LCOV_EXCL_STOP */ + return NULL; } @@ -739,16 +955,19 @@ tbm_surface_h tbm_surface_internal_create_with_bos(tbm_surface_info_s *info, tbm_bo *bos, int num) { - TBM_RETURN_VAL_IF_FAIL(bos, NULL); - TBM_RETURN_VAL_IF_FAIL(info, NULL); - TBM_RETURN_VAL_IF_FAIL(num == 1 || info->num_planes == num, NULL); - - struct _tbm_bufmgr *mgr; + struct _tbm_bufmgr *bufmgr; struct _tbm_surface *surf = NULL; int i; bool bufmgr_initialized = false; _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); + + TBM_SURFACE_RETURN_VAL_IF_FAIL(bos, NULL); + TBM_SURFACE_RETURN_VAL_IF_FAIL(info, NULL); + TBM_SURFACE_RETURN_VAL_IF_FAIL(info->num_planes > 0, NULL); + TBM_SURFACE_RETURN_VAL_IF_FAIL(num > 0, NULL); + TBM_SURFACE_RETURN_VAL_IF_FAIL(num == 1 || info->num_planes == num, NULL); if (!g_surface_bufmgr) { _init_surface_bufmgr(); @@ -756,19 +975,35 @@ tbm_surface_internal_create_with_bos(tbm_surface_info_s *info, bufmgr_initialized = true; } - mgr = g_surface_bufmgr; - if (!TBM_BUFMGR_IS_VALID(mgr)) + bufmgr = g_surface_bufmgr; + if (!TBM_BUFMGR_IS_VALID(bufmgr)) { + TBM_ERR("fail to validate the Bufmgr.\n"); + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER); goto check_valid_fail; + } surf = calloc(1, sizeof(struct _tbm_surface)); - if (!surf) + if (!surf) { + /* LCOV_EXCL_START */ + TBM_ERR("fail to allocate struct _tbm_surface.\n"); + _tbm_set_last_result(TBM_ERROR_OUT_OF_MEMORY); goto alloc_surf_fail; + /* LCOV_EXCL_STOP */ + } - surf->bufmgr = mgr; + surf->bufmgr = bufmgr; surf->info.width = info->width; surf->info.height = info->height; surf->info.format = info->format; - surf->info.bpp = info->bpp; + if (info->bpp > 0) + surf->info.bpp = info->bpp; + else { + surf->info.bpp = _tbm_surface_internal_get_bpp(info->format); + if (!surf->info.bpp) { + TBM_ERR("fail to get bpp. error(%s)\n", tbm_error_str(tbm_get_last_error())); + goto bpp_fail; + } + } surf->info.num_planes = info->num_planes; surf->refcnt = 1; @@ -779,8 +1014,16 @@ tbm_surface_internal_create_with_bos(tbm_surface_info_s *info, if (info->planes[i].size > 0) surf->info.planes[i].size = info->planes[i].size; - else - surf->info.planes[i].size += surf->info.planes[i].stride * info->height; + else { + uint32_t size = 0, offset = 0, stride = 0; + int32_t bo_idx = 0; + + if (!_tbm_surface_internal_query_plane_data(surf, i, &size, &offset, &stride, &bo_idx)) { + TBM_ERR("fail to get plane_data. error(%s)\n", tbm_error_str(tbm_get_last_error())); + goto plane_data_fail; + } + surf->info.planes[i].size = size; + } if (num == 1) surf->planes_bo_idx[i] = 0; @@ -801,26 +1044,32 @@ tbm_surface_internal_create_with_bos(tbm_surface_info_s *info, /* create only one bo */ surf->num_bos = num; for (i = 0; i < num; i++) { - if (bos[i] == NULL) + if (bos[i] == NULL) { + TBM_ERR("bos[%d] is null.\n", i); + _tbm_set_last_result(TBM_ERROR_INVALID_OPERATION); goto check_bo_fail; + } surf->bos[i] = tbm_bo_ref(bos[i]); _tbm_bo_set_surface(bos[i], surf); } - TBM_TRACE("tbm_surface(%p) width(%u) height(%u) format(%s) bo_num(%d)\n", surf, + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p) width(%u) height(%u) format(%s) bo_num(%d)\n", surf, info->width, info->height, _tbm_surface_internal_format_to_str(info->format), num); LIST_INITHEAD(&surf->user_data_list); LIST_INITHEAD(&surf->debug_data_list); - LIST_ADD(&surf->item_link, &mgr->surf_list); + LIST_ADD(&surf->item_link, &bufmgr->surf_list); _tbm_surface_mutex_unlock(); return surf; +/* LCOV_EXCL_START */ check_bo_fail: +plane_data_fail: +bpp_fail: for (i = 0; i < num; i++) { if (surf->bos[i]) tbm_bo_unref(surf->bos[i]); @@ -828,14 +1077,17 @@ check_bo_fail: free(surf); alloc_surf_fail: check_valid_fail: - if (bufmgr_initialized) { - LIST_DELINIT(&g_surface_bufmgr->surf_list); + if (bufmgr_initialized && bufmgr) { + LIST_DELINIT(&bufmgr->surf_list); _deinit_surface_bufmgr(); } _tbm_surface_mutex_unlock(); - TBM_TRACE("error: width(%u) height(%u) format(%s) bo_num(%d)\n", + + TBM_ERR("error: width(%u) height(%u) format(%s) bo_num(%d)\n", info->width, info->height, _tbm_surface_internal_format_to_str(info->format), num); +/* LCOV_EXCL_STOP */ + return NULL; } @@ -843,18 +1095,19 @@ void tbm_surface_internal_destroy(tbm_surface_h surface) { _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); - TBM_SURFACE_RETURN_IF_FAIL(tbm_surface_internal_is_valid(surface)); + TBM_SURFACE_RETURN_IF_FAIL(_tbm_surface_internal_is_valid(surface)); surface->refcnt--; if (surface->refcnt > 0) { - TBM_TRACE("reduce a refcnt(%d) of tbm_surface(%p)\n", surface->refcnt, surface); + TBM_TRACE_SURFACE_INTERNAL("reduce a refcnt(%d) of tbm_surface(%p)\n", surface->refcnt, surface); _tbm_surface_mutex_unlock(); return; } - TBM_TRACE("destroy tbm_surface(%p) refcnt(%d)\n", surface, surface->refcnt); + TBM_TRACE_SURFACE_INTERNAL("destroy tbm_surface(%p) refcnt(%d)\n", surface, surface->refcnt); if (surface->refcnt == 0) _tbm_surface_internal_destroy(surface); @@ -866,12 +1119,13 @@ void tbm_surface_internal_ref(tbm_surface_h surface) { _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); - TBM_SURFACE_RETURN_IF_FAIL(tbm_surface_internal_is_valid(surface)); + TBM_SURFACE_RETURN_IF_FAIL(_tbm_surface_internal_is_valid(surface)); surface->refcnt++; - TBM_TRACE("tbm_surface(%p) refcnt(%d)\n", surface, surface->refcnt); + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p) refcnt(%d)\n", surface, surface->refcnt); _tbm_surface_mutex_unlock(); } @@ -880,18 +1134,19 @@ void tbm_surface_internal_unref(tbm_surface_h surface) { _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); - TBM_SURFACE_RETURN_IF_FAIL(tbm_surface_internal_is_valid(surface)); + TBM_SURFACE_RETURN_IF_FAIL(_tbm_surface_internal_is_valid(surface)); surface->refcnt--; if (surface->refcnt > 0) { - TBM_TRACE("reduce a refcnt(%d) of tbm_surface(%p)\n", surface->refcnt, surface); + TBM_TRACE_SURFACE_INTERNAL("reduce a refcnt(%d) of tbm_surface(%p)\n", surface->refcnt, surface); _tbm_surface_mutex_unlock(); return; } - TBM_TRACE("destroy tbm_surface(%p) refcnt(%d)\n", surface, surface->refcnt); + TBM_TRACE_SURFACE_INTERNAL("destroy tbm_surface(%p) refcnt(%d)\n", surface, surface->refcnt); if (surface->refcnt == 0) _tbm_surface_internal_destroy(surface); @@ -906,13 +1161,17 @@ tbm_surface_internal_get_num_bos(tbm_surface_h surface) int num; _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); - TBM_SURFACE_RETURN_VAL_IF_FAIL(tbm_surface_internal_is_valid(surface), 0); + TBM_SURFACE_RETURN_VAL_IF_FAIL(_tbm_surface_internal_is_valid(surface), 0); surf = (struct _tbm_surface *)surface; num = surf->num_bos; - TBM_TRACE("tbm_surface(%p) num_bos(%d)\n", surface, num); + if (!num) + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER); + + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p) num_bos(%d)\n", surface, num); _tbm_surface_mutex_unlock(); @@ -926,14 +1185,15 @@ tbm_surface_internal_get_bo(tbm_surface_h surface, int bo_idx) tbm_bo bo; _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); - TBM_SURFACE_RETURN_VAL_IF_FAIL(tbm_surface_internal_is_valid(surface), NULL); + TBM_SURFACE_RETURN_VAL_IF_FAIL(_tbm_surface_internal_is_valid(surface), NULL); TBM_SURFACE_RETURN_VAL_IF_FAIL(bo_idx > -1, NULL); surf = (struct _tbm_surface *)surface; bo = surf->bos[bo_idx]; - TBM_TRACE("tbm_surface(%p) bo_idx(%d) tbm_bo(%p)\n", surface, bo_idx, bo); + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p) bo_idx(%d) tbm_bo(%p)\n", surface, bo_idx, bo); _tbm_surface_mutex_unlock(); @@ -947,13 +1207,14 @@ tbm_surface_internal_get_size(tbm_surface_h surface) unsigned int size; _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); - TBM_SURFACE_RETURN_VAL_IF_FAIL(tbm_surface_internal_is_valid(surface), 0); + TBM_SURFACE_RETURN_VAL_IF_FAIL(_tbm_surface_internal_is_valid(surface), 0); surf = (struct _tbm_surface *)surface; size = surf->info.size; - TBM_TRACE("tbm_surface(%p) size(%u)\n", surface, size); + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p) size(%u)\n", surface, size); _tbm_surface_mutex_unlock(); @@ -967,14 +1228,16 @@ tbm_surface_internal_get_plane_data(tbm_surface_h surface, int plane_idx, struct _tbm_surface *surf; _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); - TBM_SURFACE_RETURN_VAL_IF_FAIL(tbm_surface_internal_is_valid(surface), 0); + TBM_SURFACE_RETURN_VAL_IF_FAIL(_tbm_surface_internal_is_valid(surface), 0); TBM_SURFACE_RETURN_VAL_IF_FAIL(plane_idx > -1, 0); surf = (struct _tbm_surface *)surface; if (plane_idx >= surf->info.num_planes) { - TBM_TRACE("error: tbm_surface(%p) plane_idx(%d)\n", surface, plane_idx); + TBM_TRACE_SURFACE_INTERNAL("error: tbm_surface(%p) plane_idx(%d)\n", surface, plane_idx); + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER); _tbm_surface_mutex_unlock(); return 0; } @@ -988,7 +1251,7 @@ tbm_surface_internal_get_plane_data(tbm_surface_h surface, int plane_idx, if (pitch) *pitch = surf->info.planes[plane_idx].stride; - TBM_TRACE("tbm_surface(%p) plane_idx(%d) size(%u) offset(%u) pitch(%u)\n", surface, plane_idx, + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p) plane_idx(%d) size(%u) offset(%u) pitch(%u)\n", surface, plane_idx, surf->info.planes[plane_idx].size, surf->info.planes[plane_idx].offset, surf->info.planes[plane_idx].stride); @@ -1003,11 +1266,15 @@ tbm_surface_internal_get_info(tbm_surface_h surface, int opt, { struct _tbm_surface *surf; tbm_bo_handle bo_handles[4]; + int num_bos = 0; + tbm_bo bos[4]; + int planes_bo_idx[TBM_SURF_PLANE_MAX]; int i, j; _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); - TBM_SURFACE_RETURN_VAL_IF_FAIL(tbm_surface_internal_is_valid(surface), 0); + TBM_SURFACE_RETURN_VAL_IF_FAIL(_tbm_surface_internal_is_valid(surface), 0); memset(bo_handles, 0, sizeof(tbm_bo_handle) * 4); @@ -1021,42 +1288,48 @@ tbm_surface_internal_get_info(tbm_surface_h surface, int opt, info->size = surf->info.size; info->num_planes = surf->info.num_planes; + for (i = 0; i < surf->info.num_planes; i++) { + info->planes[i].size = surf->info.planes[i].size; + info->planes[i].offset = surf->info.planes[i].offset; + info->planes[i].stride = surf->info.planes[i].stride; + planes_bo_idx[i] = surf->planes_bo_idx[i]; + } + + for (i = 0; i < surf->num_bos; i++) + bos[i] = surf->bos[i]; + + num_bos = surf->num_bos; + if (map == 1) { - for (i = 0; i < surf->num_bos; i++) { - _tbm_surface_mutex_unlock(); - bo_handles[i] = tbm_bo_map(surf->bos[i], TBM_DEVICE_CPU, opt); - _tbm_surface_mutex_lock(); + _tbm_surface_mutex_unlock(); + for (i = 0; i < num_bos; i++) { + bo_handles[i] = tbm_bo_map(bos[i], TBM_DEVICE_CPU, opt); if (bo_handles[i].ptr == NULL) { for (j = 0; j < i; j++) - tbm_bo_unmap(surf->bos[j]); + tbm_bo_unmap(bos[j]); - TBM_TRACE("error: tbm_surface(%p) opt(%d) map(%d)\n", surface, opt, map); - _tbm_surface_mutex_unlock(); + TBM_ERR("error: tbm_surface(%p) opt(%d) map(%d)\n", surface, opt, map); return 0; } } + _tbm_surface_mutex_lock(); } else { - for (i = 0; i < surf->num_bos; i++) { - bo_handles[i] = tbm_bo_get_handle(surf->bos[i], TBM_DEVICE_CPU); + for (i = 0; i < num_bos; i++) { + bo_handles[i] = tbm_bo_get_handle(bos[i], TBM_DEVICE_CPU); if (bo_handles[i].ptr == NULL) { - TBM_TRACE("error: tbm_surface(%p) opt(%d) map(%d)\n", surface, opt, map); + TBM_ERR("error: tbm_surface(%p) opt(%d) map(%d)\n", surface, opt, map); _tbm_surface_mutex_unlock(); return 0; } } } - for (i = 0; i < surf->info.num_planes; i++) { - info->planes[i].size = surf->info.planes[i].size; - info->planes[i].offset = surf->info.planes[i].offset; - info->planes[i].stride = surf->info.planes[i].stride; - - if (bo_handles[surf->planes_bo_idx[i]].ptr) - info->planes[i].ptr = bo_handles[surf->planes_bo_idx[i]].ptr + - surf->info.planes[i].offset; + for (i = 0; i < info->num_planes; i++) { + if (bo_handles[planes_bo_idx[i]].ptr) + info->planes[i].ptr = bo_handles[planes_bo_idx[i]].ptr + info->planes[i].offset; } - TBM_TRACE("tbm_surface(%p) opt(%d) map(%d)\n", surface, opt, map); + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p) opt(%d) map(%d)\n", surface, opt, map); _tbm_surface_mutex_unlock(); @@ -1070,15 +1343,16 @@ tbm_surface_internal_unmap(tbm_surface_h surface) int i; _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); - TBM_SURFACE_RETURN_IF_FAIL(tbm_surface_internal_is_valid(surface)); + TBM_SURFACE_RETURN_IF_FAIL(_tbm_surface_internal_is_valid(surface)); surf = (struct _tbm_surface *)surface; for (i = 0; i < surf->num_bos; i++) tbm_bo_unmap(surf->bos[i]); - TBM_TRACE("tbm_surface(%p)\n", surface); + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p)\n", surface); _tbm_surface_mutex_unlock(); } @@ -1090,13 +1364,14 @@ tbm_surface_internal_get_width(tbm_surface_h surface) unsigned int width; _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); - TBM_SURFACE_RETURN_VAL_IF_FAIL(tbm_surface_internal_is_valid(surface), 0); + TBM_SURFACE_RETURN_VAL_IF_FAIL(_tbm_surface_internal_is_valid(surface), 0); surf = (struct _tbm_surface *)surface; width = surf->info.width; - TBM_TRACE("tbm_surface(%p) width(%u)\n", surface, width); + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p) width(%u)\n", surface, width); _tbm_surface_mutex_unlock(); @@ -1110,13 +1385,14 @@ tbm_surface_internal_get_height(tbm_surface_h surface) unsigned int height; _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); - TBM_SURFACE_RETURN_VAL_IF_FAIL(tbm_surface_internal_is_valid(surface), 0); + TBM_SURFACE_RETURN_VAL_IF_FAIL(_tbm_surface_internal_is_valid(surface), 0); surf = (struct _tbm_surface *)surface; height = surf->info.height; - TBM_TRACE("tbm_surface(%p) height(%u)\n", surface, height); + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p) height(%u)\n", surface, height); _tbm_surface_mutex_unlock(); @@ -1131,13 +1407,14 @@ tbm_surface_internal_get_format(tbm_surface_h surface) tbm_format format; _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); - TBM_SURFACE_RETURN_VAL_IF_FAIL(tbm_surface_internal_is_valid(surface), 0); + TBM_SURFACE_RETURN_VAL_IF_FAIL(_tbm_surface_internal_is_valid(surface), 0); surf = (struct _tbm_surface *)surface; format = surf->info.format; - TBM_TRACE("tbm_surface(%p) format(%s)\n", surface, _tbm_surface_internal_format_to_str(format)); + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p) format(%s)\n", surface, _tbm_surface_internal_format_to_str(format)); _tbm_surface_mutex_unlock(); @@ -1151,14 +1428,15 @@ tbm_surface_internal_get_plane_bo_idx(tbm_surface_h surface, int plane_idx) int bo_idx; _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); - TBM_SURFACE_RETURN_VAL_IF_FAIL(tbm_surface_internal_is_valid(surface), 0); + TBM_SURFACE_RETURN_VAL_IF_FAIL(_tbm_surface_internal_is_valid(surface), 0); TBM_SURFACE_RETURN_VAL_IF_FAIL(plane_idx > -1, 0); surf = (struct _tbm_surface *)surface; bo_idx = surf->planes_bo_idx[plane_idx]; - TBM_TRACE("tbm_surface(%p) plane_idx(%d) bo_idx(%d)\n", surface, plane_idx, bo_idx); + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p) plane_idx(%d) bo_idx(%d)\n", surface, plane_idx, bo_idx); _tbm_surface_mutex_unlock(); @@ -1172,25 +1450,27 @@ tbm_surface_internal_add_user_data(tbm_surface_h surface, unsigned long key, tbm_user_data *data; _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); - TBM_SURFACE_RETURN_VAL_IF_FAIL(tbm_surface_internal_is_valid(surface), 0); + TBM_SURFACE_RETURN_VAL_IF_FAIL(_tbm_surface_internal_is_valid(surface), 0); /* check if the data according to the key exist if so, return false. */ data = user_data_lookup(&surface->user_data_list, key); if (data) { - TBM_TRACE("warning: user data already exist tbm_surface(%p) key(%lu)\n", surface, key); + TBM_TRACE_SURFACE_INTERNAL("warning: user data already exist tbm_surface(%p) key(%lu)\n", surface, key); + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER); _tbm_surface_mutex_unlock(); return 0; } data = user_data_create(key, data_free_func); if (!data) { - TBM_TRACE("error: tbm_surface(%p) key(%lu)\n", surface, key); + TBM_TRACE_SURFACE_INTERNAL("error: tbm_surface(%p) key(%lu)\n", surface, key); _tbm_surface_mutex_unlock(); return 0; } - TBM_TRACE("tbm_surface(%p) key(%lu) data(%p)\n", surface, key, data); + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p) key(%lu) data(%p)\n", surface, key, data); LIST_ADD(&data->item_link, &surface->user_data_list); @@ -1206,18 +1486,14 @@ tbm_surface_internal_set_user_data(tbm_surface_h surface, unsigned long key, tbm_user_data *old_data; _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); - TBM_SURFACE_RETURN_VAL_IF_FAIL(tbm_surface_internal_is_valid(surface), 0); - - if (LIST_IS_EMPTY(&surface->user_data_list)) { - TBM_TRACE("error: tbm_surface(%p) key(%lu)\n", surface, key); - _tbm_surface_mutex_unlock(); - return 0; - } + TBM_SURFACE_RETURN_VAL_IF_FAIL(_tbm_surface_internal_is_valid(surface), 0); old_data = user_data_lookup(&surface->user_data_list, key); if (!old_data) { - TBM_TRACE("error: tbm_surface(%p) key(%lu)\n", surface, key); + TBM_TRACE_SURFACE_INTERNAL("error: tbm_surface(%p) key(%lu)\n", surface, key); + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER); _tbm_surface_mutex_unlock(); return 0; } @@ -1227,7 +1503,7 @@ tbm_surface_internal_set_user_data(tbm_surface_h surface, unsigned long key, old_data->data = data; - TBM_TRACE("tbm_surface(%p) key(%lu) data(%p)\n", surface, key, old_data->data); + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p) key(%lu) data(%p)\n", surface, key, old_data->data); _tbm_surface_mutex_unlock(); @@ -1241,26 +1517,29 @@ tbm_surface_internal_get_user_data(tbm_surface_h surface, unsigned long key, tbm_user_data *old_data; _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); - TBM_SURFACE_RETURN_VAL_IF_FAIL(tbm_surface_internal_is_valid(surface), 0); + TBM_SURFACE_RETURN_VAL_IF_FAIL(_tbm_surface_internal_is_valid(surface), 0); - if (!data || LIST_IS_EMPTY(&surface->user_data_list)) { - TBM_TRACE("error: tbm_surface(%p) key(%lu)\n", surface, key); + if (!data) { + TBM_ERR("error: tbm_surface(%p) key(%lu)\n", surface, key); + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER); _tbm_surface_mutex_unlock(); return 0; } + *data = NULL; old_data = user_data_lookup(&surface->user_data_list, key); if (!old_data) { - TBM_TRACE("error: tbm_surface(%p) key(%lu)\n", surface, key); - *data = NULL; + TBM_TRACE_SURFACE_INTERNAL("error: tbm_surface(%p) key(%lu)\n", surface, key); + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER); _tbm_surface_mutex_unlock(); return 0; } *data = old_data->data; - TBM_TRACE("tbm_surface(%p) key(%lu) data(%p)\n", surface, key, old_data->data); + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p) key(%lu) data(%p)\n", surface, key, old_data->data); _tbm_surface_mutex_unlock(); @@ -1274,23 +1553,19 @@ tbm_surface_internal_delete_user_data(tbm_surface_h surface, tbm_user_data *old_data = (void *)0; _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); - TBM_SURFACE_RETURN_VAL_IF_FAIL(tbm_surface_internal_is_valid(surface), 0); - - if (LIST_IS_EMPTY(&surface->user_data_list)) { - TBM_TRACE("error: tbm_surface(%p) key(%lu)\n", surface, key); - _tbm_surface_mutex_unlock(); - return 0; - } + TBM_SURFACE_RETURN_VAL_IF_FAIL(_tbm_surface_internal_is_valid(surface), 0); old_data = user_data_lookup(&surface->user_data_list, key); if (!old_data) { - TBM_TRACE("error: tbm_surface(%p) key(%lu)\n", surface, key); + TBM_TRACE_SURFACE_INTERNAL("error: tbm_surface(%p) key(%lu)\n", surface, key); + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER); _tbm_surface_mutex_unlock(); return 0; } - TBM_TRACE("tbm_surface(%p) key(%lu) data(%p)\n", surface, key, old_data->data); + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p) key(%lu) data(%p)\n", surface, key, old_data->data); user_data_delete(old_data); @@ -1312,8 +1587,9 @@ void tbm_surface_internal_set_debug_pid(tbm_surface_h surface, unsigned int pid) { _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); - TBM_SURFACE_RETURN_IF_FAIL(tbm_surface_internal_is_valid(surface)); + TBM_SURFACE_RETURN_IF_FAIL(_tbm_surface_internal_is_valid(surface)); surface->debug_pid = pid; @@ -1326,8 +1602,11 @@ _tbm_surface_internal_debug_data_create(char *key, char *value) tbm_surface_debug_data *debug_data = NULL; debug_data = calloc(1, sizeof(tbm_surface_debug_data)); - if (!debug_data) + if (!debug_data) { + _tbm_set_last_result(TBM_ERROR_OUT_OF_MEMORY); + TBM_ERR("fail to allocate the debug_data."); return NULL; + } if (key) debug_data->key = strdup(key); if (value) debug_data->value = strdup(value); @@ -1343,8 +1622,9 @@ tbm_surface_internal_set_debug_data(tbm_surface_h surface, char *key, char *valu tbm_bufmgr bufmgr = NULL; _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); - TBM_SURFACE_RETURN_VAL_IF_FAIL(tbm_surface_internal_is_valid(surface), 0); + TBM_SURFACE_RETURN_VAL_IF_FAIL(_tbm_surface_internal_is_valid(surface), 0); TBM_SURFACE_RETURN_VAL_IF_FAIL(key, 0); bufmgr = surface->bufmgr; @@ -1352,27 +1632,40 @@ tbm_surface_internal_set_debug_data(tbm_surface_h surface, char *key, char *valu TBM_SURFACE_RETURN_VAL_IF_FAIL(bufmgr, 0); if (!LIST_IS_EMPTY(&surface->debug_data_list)) { - LIST_FOR_EACH_ENTRY_SAFE(old_data, tmp, &surface->debug_data_list, item_link) { - if (!strcmp(old_data->key, key)) { - if (value) - old_data->value = strdup(value); - else - old_data->value = NULL; + LIST_FOR_EACH_ENTRY(old_data, &surface->debug_data_list, item_link) { + if (old_data) { + if (!strcmp(old_data->key, key)) { + if (old_data->value && value && !strncmp(old_data->value, value, strlen(old_data->value))) { + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p) Already exist key(%s) and value(%s)!\n", surface, key, value); + goto add_debug_key_list; + } + + if (old_data->value) + free(old_data->value); + + if (value) + old_data->value = strdup(value); + else + old_data->value = NULL; + + goto add_debug_key_list; + } } } } debug_data = _tbm_surface_internal_debug_data_create(key, value); if (!debug_data) { - TBM_TRACE("error: tbm_surface(%p) key(%s) value(%s)\n", surface, key, value); + TBM_ERR("error: tbm_surface(%p) key(%s) value(%s)\n", surface, key, value); _tbm_surface_mutex_unlock(); return 0; } - TBM_TRACE("tbm_surface(%p) key(%s) value(%s)\n", surface, key, value); + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p) key(%s) value(%s)\n", surface, key, value); LIST_ADD(&debug_data->item_link, &surface->debug_data_list); +add_debug_key_list: if (!LIST_IS_EMPTY(&bufmgr->debug_key_list)) { LIST_FOR_EACH_ENTRY_SAFE(old_data, tmp, &bufmgr->debug_key_list, item_link) { if (!strcmp(old_data->key, key)) { @@ -1393,23 +1686,17 @@ tbm_surface_internal_set_debug_data(tbm_surface_h surface, char *key, char *valu char * _tbm_surface_internal_get_debug_data(tbm_surface_h surface, char *key) { - tbm_surface_debug_data *old_data = NULL, *tmp = NULL; - - _tbm_surface_mutex_lock(); + tbm_surface_debug_data *old_data = NULL; TBM_SURFACE_RETURN_VAL_IF_FAIL(surface, NULL); if (!LIST_IS_EMPTY(&surface->debug_data_list)) { - LIST_FOR_EACH_ENTRY_SAFE(old_data, tmp, &surface->debug_data_list, item_link) { - if (!strcmp(old_data->key, key)) { - _tbm_surface_mutex_unlock(); + LIST_FOR_EACH_ENTRY(old_data, &surface->debug_data_list, item_link) { + if (!strcmp(old_data->key, key)) return old_data->value; - } } } - _tbm_surface_mutex_unlock(); - return NULL; } @@ -1441,13 +1728,19 @@ struct _tbm_surface_dump_info { static tbm_surface_dump_info *g_dump_info = NULL; static const char *dump_postfix[2] = {"png", "yuv"}; +static double scale_factor; static void -_tbm_surface_internal_dump_file_raw(const char *file, void *data1, int size1, void *data2, - int size2, void *data3, int size3) +_tbm_surface_internal_dump_file_raw(const char *file, void *data1, int size1, + void *data2, int size2, void *data3, int size3) { + FILE *fp; unsigned int *blocks; - FILE *fp = fopen(file, "w+"); + + if (_tbm_surface_check_file_is_symbolic_link(file)) + TBM_ERR("%s is symbolic link\n", file); + + fp = fopen(file, "w+"); TBM_RETURN_IF_FAIL(fp != NULL); blocks = (unsigned int *)data1; @@ -1467,58 +1760,99 @@ _tbm_surface_internal_dump_file_raw(const char *file, void *data1, int size1, vo } static void -_tbm_surface_internal_dump_file_png(const char *file, const void *data, int width, - int height) +_tbm_surface_internal_dump_file_png(const char *file, const void *data, int width, int height, int format) { - FILE *fp = fopen(file, "wb"); + unsigned int *blocks = (unsigned int *)data; + FILE *fp; + int pixel_size; + png_bytep *row_pointers; + int depth = 8, y; + + if (_tbm_surface_check_file_is_symbolic_link(file)) + TBM_ERR("%s is symbolic link\n", file); + + fp = fopen(file, "wb"); TBM_RETURN_IF_FAIL(fp != NULL); - int depth = 8; - png_structp pPngStruct = - png_create_write_struct(PNG_LIBPNG_VER_STRING, NULL, NULL, NULL); + png_structp pPngStruct = png_create_write_struct(PNG_LIBPNG_VER_STRING, + NULL, NULL, NULL); if (!pPngStruct) { + TBM_ERR("fail to create a png write structure.\n"); fclose(fp); return; } png_infop pPngInfo = png_create_info_struct(pPngStruct); if (!pPngInfo) { + TBM_ERR("fail to create a png info structure.\n"); png_destroy_write_struct(&pPngStruct, NULL); fclose(fp); return; } png_init_io(pPngStruct, fp); - png_set_IHDR(pPngStruct, - pPngInfo, - width, - height, - depth, - PNG_COLOR_TYPE_RGBA, - PNG_INTERLACE_NONE, - PNG_COMPRESSION_TYPE_DEFAULT, PNG_FILTER_TYPE_DEFAULT); + if (format == TBM_FORMAT_XRGB8888) { + pixel_size = 3; + png_set_IHDR(pPngStruct, + pPngInfo, + width, + height, + depth, + PNG_COLOR_TYPE_RGB, + PNG_INTERLACE_NONE, + PNG_COMPRESSION_TYPE_DEFAULT, PNG_FILTER_TYPE_DEFAULT); + } else { + pixel_size = 4; + png_set_IHDR(pPngStruct, + pPngInfo, + width, + height, + depth, + PNG_COLOR_TYPE_RGBA, + PNG_INTERLACE_NONE, + PNG_COMPRESSION_TYPE_DEFAULT, PNG_FILTER_TYPE_DEFAULT); + } png_set_bgr(pPngStruct); png_write_info(pPngStruct, pPngInfo); - const int pixel_size = 4; // RGBA - png_bytep *row_pointers = - png_malloc(pPngStruct, height * sizeof(png_byte *)); - - unsigned int *blocks = (unsigned int *)data; - int y = 0; - int x = 0; + row_pointers = png_malloc(pPngStruct, height * sizeof(png_byte *)); + if (!row_pointers) { + TBM_ERR("fail to allocate the png row_pointers.\n"); + png_destroy_write_struct(&pPngStruct, &pPngInfo); + fclose(fp); + return; + } - for (; y < height; ++y) { - png_bytep row = - png_malloc(pPngStruct, sizeof(png_byte) * width * pixel_size); + for (y = 0; y < height; ++y) { + png_bytep row; + int x = 0; + + row = png_malloc(pPngStruct, sizeof(png_byte) * width * pixel_size); + if (!row) { + TBM_ERR("fail to allocate the png row.\n"); + for (x = 0; x < y; x++) + png_free(pPngStruct, row_pointers[x]); + png_free(pPngStruct, row_pointers); + png_destroy_write_struct(&pPngStruct, &pPngInfo); + fclose(fp); + return; + } row_pointers[y] = (png_bytep)row; + for (x = 0; x < width; ++x) { unsigned int curBlock = blocks[y * width + x]; - row[x * pixel_size] = (curBlock & 0xFF); - row[1 + x * pixel_size] = (curBlock >> 8) & 0xFF; - row[2 + x * pixel_size] = (curBlock >> 16) & 0xFF; - row[3 + x * pixel_size] = (curBlock >> 24) & 0xFF; + + if (pixel_size == 3) { // XRGB8888 + row[x * pixel_size] = (curBlock & 0xFF); + row[1 + x * pixel_size] = (curBlock >> 8) & 0xFF; + row[2 + x * pixel_size] = (curBlock >> 16) & 0xFF; + } else { // ARGB8888 + row[x * pixel_size] = (curBlock & 0xFF); + row[1 + x * pixel_size] = (curBlock >> 8) & 0xFF; + row[2 + x * pixel_size] = (curBlock >> 16) & 0xFF; + row[3 + x * pixel_size] = (curBlock >> 24) & 0xFF; + } } } @@ -1543,17 +1877,13 @@ tbm_surface_internal_dump_start(char *path, int w, int h, int count) TBM_RETURN_IF_FAIL(count > 0); tbm_surface_dump_buf_info *buf_info = NULL; - tbm_surface_dump_buf_info *tmp; - tbm_bo bo = NULL; - int i; - int buffer_size; tbm_surface_h tbm_surface; tbm_surface_info_s info; - tbm_surface_error_e err; + int buffer_size, i; /* check running */ if (g_dump_info) { - TBM_LOG_W("waring already running the tbm_surface_internal_dump.\n"); + TBM_WRN("waring already running the tbm_surface_internal_dump.\n"); return; } @@ -1567,29 +1897,32 @@ tbm_surface_internal_dump_start(char *path, int w, int h, int count) /* get buffer size */ tbm_surface = tbm_surface_create(w, h, TBM_FORMAT_ARGB8888); if (tbm_surface == NULL) { - TBM_LOG_E("tbm_surface_create fail\n"); + TBM_ERR("tbm_surface_create fail\n"); free(g_dump_info); g_dump_info = NULL; return; } - err = tbm_surface_map(tbm_surface, TBM_SURF_OPTION_READ, &info); - if (err != TBM_SURFACE_ERROR_NONE) { - TBM_LOG_E("tbm_surface_map fail\n"); + + if (TBM_SURFACE_ERROR_NONE != tbm_surface_get_info(tbm_surface, &info)) { + TBM_ERR("tbm_surface_get_info fail\n"); tbm_surface_destroy(tbm_surface); free(g_dump_info); g_dump_info = NULL; return; } - buffer_size = info.planes[0].stride * h; - tbm_surface_unmap(tbm_surface); + buffer_size = info.size; tbm_surface_destroy(tbm_surface); /* create dump lists */ - for (i = 0; i < count; i++) { + for (i = 0; i < count; i++) { + tbm_bo bo = NULL; + buf_info = calloc(1, sizeof(tbm_surface_dump_buf_info)); TBM_GOTO_VAL_IF_FAIL(buf_info, fail); + bo = tbm_bo_alloc(g_surface_bufmgr, buffer_size, TBM_BO_DEFAULT); if (bo == NULL) { + TBM_ERR("fail to allocate the tbm_bo[%d]\n", i); free(buf_info); goto fail; } @@ -1604,19 +1937,25 @@ tbm_surface_internal_dump_start(char *path, int w, int h, int count) g_dump_info->path = path; g_dump_info->link = &g_dump_info->surface_list; - TBM_LOG_I("Dump Start.. path:%s, count:%d\n", g_dump_info->path, count); + scale_factor = 0.0; + + TBM_INFO("Dump Start.. path:%s, count:%d\n", g_dump_info->path, count); return; + fail: /* free resources */ if (!LIST_IS_EMPTY(&g_dump_info->surface_list)) { + tbm_surface_dump_buf_info *tmp; + LIST_FOR_EACH_ENTRY_SAFE(buf_info, tmp, &g_dump_info->surface_list, link) { tbm_bo_unref(buf_info->bo); + LIST_DEL(&buf_info->link); free(buf_info); } } - TBM_LOG_E("Dump Start fail.. path:%s\n", g_dump_info->path); + TBM_ERR("Dump Start fail.. path:%s\n", g_dump_info->path); free(g_dump_info); g_dump_info = NULL; @@ -1625,6 +1964,18 @@ fail: } void +tbm_surface_internal_dump_with_scale_start(char *path, int w, int h, int count, double scale) +{ + if (scale > 0.0) { + w *= scale; + h *= scale; + } + + tbm_surface_internal_dump_start(path, w, h, count); + scale_factor = scale; +} + +void tbm_surface_internal_dump_end(void) { tbm_surface_dump_buf_info *buf_info = NULL, *tmp = NULL; @@ -1633,89 +1984,204 @@ tbm_surface_internal_dump_end(void) if (!g_dump_info) return; + if (LIST_IS_EMPTY(&g_dump_info->surface_list)) { + free(g_dump_info); + g_dump_info = NULL; + return; + } + /* make files */ - if (!LIST_IS_EMPTY(&g_dump_info->surface_list)) { - LIST_FOR_EACH_ENTRY_SAFE(buf_info, tmp, &g_dump_info->surface_list, link) { - char file[2048]; - - if (buf_info->dirty) { - void *ptr1 = NULL; - void *ptr2 = NULL; - - bo_handle = tbm_bo_map(buf_info->bo, TBM_DEVICE_CPU, TBM_OPTION_READ); - if (bo_handle.ptr == NULL) - continue; - - snprintf(file, sizeof(file), "%s/%s", g_dump_info->path, buf_info->name); - TBM_LOG_I("Dump File.. %s generated.\n", file); - - switch (buf_info->info.format) { - case TBM_FORMAT_ARGB8888: - case TBM_FORMAT_XRGB8888: - _tbm_surface_internal_dump_file_png(file, bo_handle.ptr, - buf_info->info.planes[0].stride >> 2, buf_info->info.height); - break; - case TBM_FORMAT_YVU420: - case TBM_FORMAT_YUV420: - ptr1 = bo_handle.ptr + buf_info->info.planes[0].stride * buf_info->info.height; - ptr2 = ptr1 + buf_info->info.planes[1].stride * (buf_info->info.height >> 1); - _tbm_surface_internal_dump_file_raw(file, bo_handle.ptr, - buf_info->info.planes[0].stride * buf_info->info.height, - ptr1, - buf_info->info.planes[1].stride * (buf_info->info.height >> 1), - ptr2, - buf_info->info.planes[2].stride * (buf_info->info.height >> 1)); - break; - case TBM_FORMAT_NV12: - case TBM_FORMAT_NV21: - ptr1 = bo_handle.ptr + buf_info->info.planes[0].stride * buf_info->info.height; - _tbm_surface_internal_dump_file_raw(file, bo_handle.ptr, - buf_info->info.planes[0].stride * buf_info->info.height, - ptr1, - buf_info->info.planes[1].stride * (buf_info->info.height >> 1), - NULL, 0); - break; - case TBM_FORMAT_YUYV: - case TBM_FORMAT_UYVY: - _tbm_surface_internal_dump_file_raw(file, bo_handle.ptr, - buf_info->info.planes[0].stride * buf_info->info.height, - NULL, 0, NULL, 0); - break; - default: - TBM_LOG_E("can't dump %c%c%c%c buffer", FOURCC_STR(buf_info->info.format)); - tbm_bo_unmap(buf_info->bo); - return; - } + LIST_FOR_EACH_ENTRY_SAFE(buf_info, tmp, &g_dump_info->surface_list, link) { + char file[2048]; - tbm_bo_unmap(buf_info->bo); - } else if (buf_info->dirty_shm) { - bo_handle = tbm_bo_map(buf_info->bo, TBM_DEVICE_CPU, TBM_OPTION_READ); - if (bo_handle.ptr == NULL) - continue; + bo_handle = tbm_bo_map(buf_info->bo, TBM_DEVICE_CPU, TBM_OPTION_READ); + if (bo_handle.ptr == NULL) { + tbm_bo_unref(buf_info->bo); + LIST_DEL(&buf_info->link); + free(buf_info); + continue; + } - snprintf(file, sizeof(file), "%s/%s", g_dump_info->path, buf_info->name); - TBM_LOG_I("Dump File.. %s generated.\n", file); + snprintf(file, sizeof(file), "%s/%s", g_dump_info->path, buf_info->name); + TBM_INFO("Dump File.. %s generated.\n", file); - _tbm_surface_internal_dump_file_png(file, bo_handle.ptr, - buf_info->shm_stride >> 2, buf_info->shm_h); + if (buf_info->dirty) { + void *ptr1 = NULL, *ptr2 = NULL; - tbm_bo_unmap(buf_info->bo); + switch (buf_info->info.format) { + case TBM_FORMAT_ARGB8888: + _tbm_surface_internal_dump_file_png(file, bo_handle.ptr, + buf_info->info.planes[0].stride >> 2, + buf_info->info.height, TBM_FORMAT_ARGB8888); + break; + case TBM_FORMAT_XRGB8888: + _tbm_surface_internal_dump_file_png(file, bo_handle.ptr, + buf_info->info.planes[0].stride >> 2, + buf_info->info.height, TBM_FORMAT_XRGB8888); + break; + case TBM_FORMAT_YVU420: + case TBM_FORMAT_YUV420: + ptr1 = bo_handle.ptr + buf_info->info.planes[0].stride * buf_info->info.height; + ptr2 = ptr1 + buf_info->info.planes[1].stride * (buf_info->info.height >> 1); + _tbm_surface_internal_dump_file_raw(file, bo_handle.ptr, + buf_info->info.planes[0].stride * buf_info->info.height, + ptr1, + buf_info->info.planes[1].stride * (buf_info->info.height >> 1), + ptr2, + buf_info->info.planes[2].stride * (buf_info->info.height >> 1)); + break; + case TBM_FORMAT_NV12: + case TBM_FORMAT_NV21: + ptr1 = bo_handle.ptr + buf_info->info.planes[0].stride * buf_info->info.height; + _tbm_surface_internal_dump_file_raw(file, bo_handle.ptr, + buf_info->info.planes[0].stride * buf_info->info.height, + ptr1, + buf_info->info.planes[1].stride * (buf_info->info.height >> 1), + NULL, 0); + break; + case TBM_FORMAT_YUYV: + case TBM_FORMAT_UYVY: + _tbm_surface_internal_dump_file_raw(file, bo_handle.ptr, + buf_info->info.planes[0].stride * buf_info->info.height, + NULL, 0, NULL, 0); + break; + default: + TBM_ERR("can't dump %c%c%c%c buffer", FOURCC_STR(buf_info->info.format)); + break; } - } - } + } else if (buf_info->dirty_shm) + _tbm_surface_internal_dump_file_png(file, bo_handle.ptr, + buf_info->shm_stride >> 2, + buf_info->shm_h, 0); - /* free resources */ - if (!LIST_IS_EMPTY(&g_dump_info->surface_list)) { - LIST_FOR_EACH_ENTRY_SAFE(buf_info, tmp, &g_dump_info->surface_list, link) { - tbm_bo_unref(buf_info->bo); - free(buf_info); - } + tbm_bo_unmap(buf_info->bo); + tbm_bo_unref(buf_info->bo); + LIST_DEL(&buf_info->link); + free(buf_info); } free(g_dump_info); g_dump_info = NULL; - TBM_LOG_I("Dump End..\n"); + TBM_INFO("Dump End..\n"); +} + +static pixman_format_code_t +_tbm_surface_internal_pixman_format_get(tbm_format format) +{ + switch (format) { + case TBM_FORMAT_ARGB8888: + return PIXMAN_a8r8g8b8; + case TBM_FORMAT_XRGB8888: + return PIXMAN_x8r8g8b8; + default: + return 0; + } + + return 0; +} + +/** + * This function supports only if a buffer has below formats. + * - TBM_FORMAT_ARGB8888 + * - TBM_FORMAT_XRGB8888 + */ +static tbm_surface_error_e +_tbm_surface_internal_buffer_scale(void *src_ptr, void *dst_ptr, + int format, int src_stride, int src_w, int src_h, + int dst_stride, int dst_w, int dst_h) +{ + pixman_image_t *src_img = NULL, *dst_img = NULL; + pixman_format_code_t pixman_format; + pixman_transform_t t; + struct pixman_f_transform ft; + double scale_x, scale_y; + + TBM_RETURN_VAL_IF_FAIL(src_ptr != NULL, TBM_SURFACE_ERROR_INVALID_OPERATION); + TBM_RETURN_VAL_IF_FAIL(dst_ptr != NULL, TBM_SURFACE_ERROR_INVALID_OPERATION); + + pixman_format = _tbm_surface_internal_pixman_format_get(format); + TBM_RETURN_VAL_IF_FAIL(pixman_format > 0, TBM_SURFACE_ERROR_INVALID_OPERATION); + + /* src */ + src_img = pixman_image_create_bits(pixman_format, src_w, src_h, + (uint32_t*)src_ptr, src_stride); + TBM_GOTO_VAL_IF_FAIL(src_img != NULL, cant_convert); + + /* dst */ + dst_img = pixman_image_create_bits(pixman_format, dst_w, dst_h, + (uint32_t*)dst_ptr, dst_stride); + TBM_GOTO_VAL_IF_FAIL(dst_img != NULL, cant_convert); + + pixman_f_transform_init_identity(&ft); + + scale_x = (double)src_w / dst_w; + scale_y = (double)src_h / dst_h; + + pixman_f_transform_scale(&ft, NULL, scale_x, scale_y); + pixman_f_transform_translate(&ft, NULL, 0, 0); + pixman_transform_from_pixman_f_transform(&t, &ft); + pixman_image_set_transform(src_img, &t); + + pixman_image_composite(PIXMAN_OP_SRC, src_img, NULL, dst_img, + 0, 0, 0, 0, 0, 0, dst_w, dst_h); + + pixman_image_unref(src_img); + pixman_image_unref(dst_img); + + return TBM_SURFACE_ERROR_NONE; + +cant_convert: + if (src_img) + pixman_image_unref(src_img); + + return TBM_SURFACE_ERROR_INVALID_OPERATION; +} + +#define MAX_BOS 4 // This value is came from bos[4] in struct _tbm_surface +#define KEY_LEN 5 // "_XXXX" +#define KEYS_LEN KEY_LEN * MAX_BOS + +static char *_tbm_surface_internal_get_keys(tbm_surface_h surface) +{ + char *keys, temp_key[KEY_LEN + 1]; + struct _tbm_surface *surf; + int i, num_bos; + tbm_bo bo; + + _tbm_surface_mutex_lock(); + + TBM_SURFACE_RETURN_VAL_IF_FAIL(_tbm_surface_internal_is_valid(surface), NULL); + + surf = (struct _tbm_surface *)surface; + + num_bos = surf->num_bos; + if (num_bos > MAX_BOS) + num_bos = MAX_BOS; + + keys = calloc(KEYS_LEN + 1, sizeof(char)); + if (!keys) { + TBM_ERR("Failed to alloc memory"); + _tbm_surface_mutex_unlock(); + return NULL; + } + + for (i = 0; i < num_bos; i++) { + memset(temp_key, 0x00, KEY_LEN + 1); + bo = surf->bos[i]; + snprintf(temp_key, KEY_LEN, "_%d", tbm_bo_export(bo)); + strncat(keys, temp_key, KEY_LEN); + } + + _tbm_surface_mutex_unlock(); + + return keys; +} + +static void _tbm_surface_internal_put_keys(char *keys) +{ + if (keys) + free(keys); } void @@ -1725,11 +2191,13 @@ tbm_surface_internal_dump_buffer(tbm_surface_h surface, const char *type) TBM_RETURN_IF_FAIL(type != NULL); tbm_surface_dump_buf_info *buf_info; - tbm_surface_info_s info; struct list_head *next_link; + tbm_surface_info_s info; tbm_bo_handle bo_handle; - int ret; const char *postfix; + const char *format = NULL; + char *keys; + int ret; if (!g_dump_info) return; @@ -1748,38 +2216,106 @@ tbm_surface_internal_dump_buffer(tbm_surface_h surface, const char *type) ret = tbm_surface_map(surface, TBM_SURF_OPTION_READ|TBM_SURF_OPTION_WRITE, &info); TBM_RETURN_IF_FAIL(ret == TBM_SURFACE_ERROR_NONE); - if (info.size > buf_info->size) { - TBM_LOG_W("Dump skip. surface over created buffer size(%u, %d)\n", info.size, buf_info->size); - tbm_surface_unmap(surface); - return; + if (scale_factor > 0.0) { + const int bpp = 4; + + if (info.format != TBM_FORMAT_ARGB8888 && info.format != TBM_FORMAT_XRGB8888) { + TBM_WRN("Dump with scale skip. unsupported format(%s)\n", + _tbm_surface_internal_format_to_str(info.format)); + tbm_surface_unmap(surface); + return; + } + + memset(&buf_info->info, 0, sizeof(tbm_surface_info_s)); + + buf_info->info.width = info.width * scale_factor; + buf_info->info.height = info.height * scale_factor; + buf_info->info.format = info.format; + buf_info->info.bpp = _tbm_surface_internal_get_bpp(buf_info->info.format); + if (!buf_info->info.bpp) { + TBM_ERR("fail to get bpp. error(%s)\n", tbm_error_str(tbm_get_last_error())); + tbm_surface_unmap(surface); + return; + } + buf_info->info.num_planes = 1; + buf_info->info.planes[0].stride = buf_info->info.width * bpp; + buf_info->info.size = buf_info->info.width * buf_info->info.height * bpp; + + if (buf_info->info.size > buf_info->size) { + TBM_WRN("Dump with scale skip. surface over created buffer size(%u, %d)\n", + buf_info->info.size, buf_info->size); + tbm_surface_unmap(surface); + return; + } + } else { + if (info.size > buf_info->size) { + TBM_WRN("Dump skip. surface over created buffer size(%u, %d)\n", + info.size, buf_info->size); + tbm_surface_unmap(surface); + return; + } + + /* make the file information */ + memcpy(&buf_info->info, &info, sizeof(tbm_surface_info_s)); } - if (info.format == TBM_FORMAT_ARGB8888 || info.format == TBM_FORMAT_XRGB8888) + if (info.format == TBM_FORMAT_ARGB8888 || info.format == TBM_FORMAT_XRGB8888) { postfix = dump_postfix[0]; - else + format = _tbm_surface_internal_format_to_str(info.format); + } else postfix = dump_postfix[1]; - /* make the file information */ - memcpy(&buf_info->info, &info, sizeof(tbm_surface_info_s)); + keys = _tbm_surface_internal_get_keys(surface); + if (!keys) { + TBM_ERR("fail to get keys"); + tbm_surface_unmap(surface); + return; + } /* dump */ bo_handle = tbm_bo_map(buf_info->bo, TBM_DEVICE_CPU, TBM_OPTION_WRITE); - TBM_RETURN_IF_FAIL(bo_handle.ptr != NULL); + if (!bo_handle.ptr) { + TBM_ERR("fail to map bo"); + _tbm_surface_internal_put_keys(keys); + tbm_surface_unmap(surface); + return; + } memset(bo_handle.ptr, 0x00, buf_info->size); switch (info.format) { case TBM_FORMAT_ARGB8888: case TBM_FORMAT_XRGB8888: - snprintf(buf_info->name, sizeof(buf_info->name), "%10.3f_%03d_%p-%s.%s", + snprintf(buf_info->name, sizeof(buf_info->name), + "%10.3f_%03d%s_%p_%s-%s.%s", _tbm_surface_internal_get_time(), - g_dump_info->count++, surface, type, postfix); - memcpy(bo_handle.ptr, info.planes[0].ptr, info.size); + g_dump_info->count++, keys, surface, format, type, postfix); + + if (scale_factor > 0.0) { + ret = _tbm_surface_internal_buffer_scale(info.planes[0].ptr, + bo_handle.ptr, + buf_info->info.format, + info.planes[0].stride, + info.width, info.height, + buf_info->info.planes[0].stride, + buf_info->info.width, + buf_info->info.height); + if (ret != TBM_SURFACE_ERROR_NONE) { + TBM_ERR("fail to scale buffer"); + tbm_bo_unmap(buf_info->bo); + _tbm_surface_internal_put_keys(keys); + tbm_surface_unmap(surface); + return; + } + } else + memcpy(bo_handle.ptr, info.planes[0].ptr, info.size); break; case TBM_FORMAT_YVU420: case TBM_FORMAT_YUV420: - snprintf(buf_info->name, sizeof(buf_info->name), "%10.3f_%03d-%s_%dx%d_%c%c%c%c.%s", + snprintf(buf_info->name, sizeof(buf_info->name), + "%10.3f_%03d%s-%s_%dx%d_%c%c%c%c.%s", _tbm_surface_internal_get_time(), - g_dump_info->count++, type, info.planes[0].stride, info.height, FOURCC_STR(info.format), postfix); + g_dump_info->count++, keys, type, info.planes[0].stride, + info.height, FOURCC_STR(info.format), postfix); memcpy(bo_handle.ptr, info.planes[0].ptr, info.planes[0].stride * info.height); bo_handle.ptr += info.planes[0].stride * info.height; memcpy(bo_handle.ptr, info.planes[1].ptr, info.planes[1].stride * (info.height >> 1)); @@ -1788,28 +2324,36 @@ tbm_surface_internal_dump_buffer(tbm_surface_h surface, const char *type) break; case TBM_FORMAT_NV12: case TBM_FORMAT_NV21: - snprintf(buf_info->name, sizeof(buf_info->name), "%10.3f_%03d-%s_%dx%d_%c%c%c%c.%s", + snprintf(buf_info->name, sizeof(buf_info->name), + "%10.3f_%03d%s-%s_%dx%d_%c%c%c%c.%s", _tbm_surface_internal_get_time(), - g_dump_info->count++, type, info.planes[0].stride, info.height, FOURCC_STR(info.format), postfix); + g_dump_info->count++, keys, type, info.planes[0].stride, + info.height, FOURCC_STR(info.format), postfix); memcpy(bo_handle.ptr, info.planes[0].ptr, info.planes[0].stride * info.height); bo_handle.ptr += info.planes[0].stride * info.height; memcpy(bo_handle.ptr, info.planes[1].ptr, info.planes[1].stride * (info.height >> 1)); break; case TBM_FORMAT_YUYV: case TBM_FORMAT_UYVY: - snprintf(buf_info->name, sizeof(buf_info->name), "%10.3f_%03d-%s_%dx%d_%c%c%c%c.%s", + snprintf(buf_info->name, sizeof(buf_info->name), + "%10.3f_%03d%s-%s_%dx%d_%c%c%c%c.%s", _tbm_surface_internal_get_time(), - g_dump_info->count++, type, info.planes[0].stride, info.height, FOURCC_STR(info.format), postfix); + g_dump_info->count++, keys, type, info.planes[0].stride, + info.height, FOURCC_STR(info.format), postfix); memcpy(bo_handle.ptr, info.planes[0].ptr, info.planes[0].stride * info.height); break; default: - TBM_LOG_E("can't copy %c%c%c%c buffer", FOURCC_STR(info.format)); + TBM_ERR("can't copy %c%c%c%c buffer", FOURCC_STR(info.format)); tbm_bo_unmap(buf_info->bo); + _tbm_surface_internal_put_keys(keys); + tbm_surface_unmap(surface); return; } tbm_bo_unmap(buf_info->bo); + _tbm_surface_internal_put_keys(keys); + tbm_surface_unmap(surface); buf_info->dirty = 1; @@ -1820,10 +2364,11 @@ tbm_surface_internal_dump_buffer(tbm_surface_h surface, const char *type) g_dump_info->link = next_link; - TBM_LOG_I("Dump %s \n", buf_info->name); + TBM_INFO("Dump %s \n", buf_info->name); } -void tbm_surface_internal_dump_shm_buffer(void *ptr, int w, int h, int stride, const char *type) +void tbm_surface_internal_dump_shm_buffer(void *ptr, int w, int h, int stride, + const char *type) { TBM_RETURN_IF_FAIL(ptr != NULL); TBM_RETURN_IF_FAIL(w > 0); @@ -1834,6 +2379,7 @@ void tbm_surface_internal_dump_shm_buffer(void *ptr, int w, int h, int stride, tbm_surface_dump_buf_info *buf_info; struct list_head *next_link; tbm_bo_handle bo_handle; + int ret, size, dw = 0, dh = 0, dstride = 0; if (!g_dump_info) return; @@ -1849,34 +2395,226 @@ void tbm_surface_internal_dump_shm_buffer(void *ptr, int w, int h, int stride, buf_info = LIST_ENTRY(tbm_surface_dump_buf_info, next_link, link); TBM_RETURN_IF_FAIL(buf_info != NULL); - if (stride * h > buf_info->size) { - TBM_LOG_W("Dump skip. shm buffer over created buffer size(%d, %d)\n", stride * h, buf_info->size); + if (scale_factor > 0.0) { + const int bpp = 4; + + dw = w * scale_factor; + dh = h * scale_factor; + dstride = dw * bpp; + size = dstride * dh; + } else + size = stride * h; + + if (size > buf_info->size) { + TBM_WRN("Dump skip. shm buffer over created buffer size(%d, %d)\n", + size, buf_info->size); return; } /* dump */ bo_handle = tbm_bo_map(buf_info->bo, TBM_DEVICE_CPU, TBM_OPTION_WRITE); TBM_RETURN_IF_FAIL(bo_handle.ptr != NULL); + memset(bo_handle.ptr, 0x00, buf_info->size); memset(&buf_info->info, 0x00, sizeof(tbm_surface_info_s)); snprintf(buf_info->name, sizeof(buf_info->name), "%10.3f_%03d-%s.%s", _tbm_surface_internal_get_time(), g_dump_info->count++, type, dump_postfix[0]); - memcpy(bo_handle.ptr, ptr, stride * h); + if (scale_factor > 0.0) { + ret = _tbm_surface_internal_buffer_scale(ptr, bo_handle.ptr, + TBM_FORMAT_ARGB8888, stride, + w, h, dstride, dw, dh); + if (ret != TBM_SURFACE_ERROR_NONE) { + TBM_ERR("fail to scale buffer"); + tbm_bo_unmap(buf_info->bo); + return; + } + buf_info->shm_stride = dstride; + buf_info->shm_h = dh; + } else { + memcpy(bo_handle.ptr, ptr, size); + buf_info->shm_stride = stride; + buf_info->shm_h = h; + } tbm_bo_unmap(buf_info->bo); buf_info->dirty = 0; buf_info->dirty_shm = 1; - buf_info->shm_stride = stride; - buf_info->shm_h = h; if (g_dump_info->count == 1000) g_dump_info->count = 0; g_dump_info->link = next_link; - TBM_LOG_I("Dump %s \n", buf_info->name); + TBM_INFO("Dump %s \n", buf_info->name); +} + +int +tbm_surface_internal_capture_buffer(tbm_surface_h surface, const char *path, const char *name, const char *type) +{ + TBM_RETURN_VAL_IF_FAIL(surface != NULL, 0); + TBM_RETURN_VAL_IF_FAIL(path != NULL, 0); + TBM_RETURN_VAL_IF_FAIL(name != NULL, 0); + + tbm_surface_info_s info; + const char *postfix; + int ret; + char file[1024]; + + ret = tbm_surface_map(surface, TBM_SURF_OPTION_READ|TBM_SURF_OPTION_WRITE, &info); + TBM_RETURN_VAL_IF_FAIL(ret == TBM_SURFACE_ERROR_NONE, 0); + + if (info.format == TBM_FORMAT_ARGB8888 || info.format == TBM_FORMAT_XRGB8888) + postfix = dump_postfix[0]; + else + postfix = dump_postfix[1]; + + if (strcmp(postfix, type)) { + TBM_ERR("not support type(%s) %c%c%c%c buffer", type, FOURCC_STR(info.format)); + tbm_surface_unmap(surface); + return 0; + } + + snprintf(file, sizeof(file), "%s/%s.%s", path , name, postfix); + + if (!access(file, 0)) { + TBM_ERR("can't capture buffer, exist file %s", file); + tbm_surface_unmap(surface); + return 0; + } + + switch (info.format) { + case TBM_FORMAT_ARGB8888: + _tbm_surface_internal_dump_file_png(file, info.planes[0].ptr, + info.planes[0].stride >> 2, + info.height, TBM_FORMAT_ARGB8888); + break; + case TBM_FORMAT_XRGB8888: + _tbm_surface_internal_dump_file_png(file, info.planes[0].ptr, + info.planes[0].stride >> 2, + info.height, TBM_FORMAT_XRGB8888); + break; + case TBM_FORMAT_YVU420: + case TBM_FORMAT_YUV420: + _tbm_surface_internal_dump_file_raw(file, info.planes[0].ptr, + info.planes[0].stride * info.height, + info.planes[1].ptr, + info.planes[1].stride * (info.height >> 1), + info.planes[2].ptr, + info.planes[2].stride * (info.height >> 1)); + break; + case TBM_FORMAT_NV12: + case TBM_FORMAT_NV21: + _tbm_surface_internal_dump_file_raw(file, info.planes[0].ptr, + info.planes[0].stride * info.height, + info.planes[1].ptr, + info.planes[1].stride * (info.height >> 1), + NULL, 0); + break; + case TBM_FORMAT_YUYV: + case TBM_FORMAT_UYVY: + _tbm_surface_internal_dump_file_raw(file, info.planes[0].ptr, + info.planes[0].stride * info.height, + NULL, 0, NULL, 0); + break; + default: + TBM_ERR("can't dump %c%c%c%c buffer", FOURCC_STR(info.format)); + tbm_surface_unmap(surface); + return 0; + } + + tbm_surface_unmap(surface); + + TBM_TRACE_SURFACE_INTERNAL("Capture %s \n", file); + + return 1; +} + +int +tbm_surface_internal_capture_shm_buffer(void *ptr, int w, int h, int stride, + const char *path, const char *name, const char *type) +{ + TBM_RETURN_VAL_IF_FAIL(ptr != NULL, 0); + TBM_RETURN_VAL_IF_FAIL(w > 0, 0); + TBM_RETURN_VAL_IF_FAIL(h > 0, 0); + TBM_RETURN_VAL_IF_FAIL(stride > 0, 0); + TBM_RETURN_VAL_IF_FAIL(path != NULL, 0); + TBM_RETURN_VAL_IF_FAIL(name != NULL, 0); + + char file[1024]; + + if (strcmp(dump_postfix[0], type)) { + TBM_ERR("Not supported type:%s'", type); + return 0; + } + + snprintf(file, sizeof(file), "%s/%s.%s", path , name, dump_postfix[0]); + + if (!access(file, 0)) { + TBM_ERR("can't capture buffer, exist file %sTBM_FORMAT_XRGB8888", file); + return 0; + } + + _tbm_surface_internal_dump_file_png(file, ptr, w, h, 0); + + TBM_TRACE_SURFACE_INTERNAL("Capture %s \n", file); + + return 1; +} + +int +tbm_surface_internal_set_damage(tbm_surface_h surface, int x, int y, int width, int height) +{ + struct _tbm_surface *surf; + + _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); + + TBM_SURFACE_RETURN_VAL_IF_FAIL(width > 0, 0); + TBM_SURFACE_RETURN_VAL_IF_FAIL(height > 0, 0); + TBM_SURFACE_RETURN_VAL_IF_FAIL(_tbm_surface_internal_is_valid(surface), 0); + + surf = (struct _tbm_surface *)surface; + + surf->damage.x = x; + surf->damage.y = y; + surf->damage.width = width; + surf->damage.height = height; + + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p) x(%d) y(%d) width(%d) height(%d)\n", + surface, x, y, width, height); + + _tbm_surface_mutex_unlock(); + + return 1; +} + +int +tbm_surface_internal_get_damage(tbm_surface_h surface, int *x, int *y, int *width, int *height) +{ + struct _tbm_surface *surf; + + _tbm_surface_mutex_lock(); + _tbm_set_last_result(TBM_ERROR_NONE); + + TBM_SURFACE_RETURN_VAL_IF_FAIL(width > 0, 0); + TBM_SURFACE_RETURN_VAL_IF_FAIL(height > 0, 0); + TBM_SURFACE_RETURN_VAL_IF_FAIL(_tbm_surface_internal_is_valid(surface), 0); + + surf = (struct _tbm_surface *)surface; + + *x = surf->damage.x; + *y = surf->damage.y; + *width = surf->damage.width; + *height = surf->damage.height; + + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p) x(%d) y(%d) width(%d) height(%d)\n", + surface, *x, *y, *width, *height); + + _tbm_surface_mutex_unlock(); + + return 1; } /*LCOV_EXCL_STOP*/