X-Git-Url: http://review.tizen.org/git/?a=blobdiff_plain;f=src%2Ftbm_surface_internal.c;h=8c6fc391112d1d19b029e0e7a164e31f73c68561;hb=e63541a42a8470996ae0dc2209b2febaaa71c390;hp=fd375a9f698a849167142867c7ee91e613d9cf84;hpb=39476e06f2ce99aa0f3ebd0e7a56aa09e78498fd;p=platform%2Fcore%2Fuifw%2Flibtbm.git diff --git a/src/tbm_surface_internal.c b/src/tbm_surface_internal.c index fd375a9..8c6fc39 100644 --- a/src/tbm_surface_internal.c +++ b/src/tbm_surface_internal.c @@ -41,20 +41,36 @@ SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. #include #include +#define TBM_SURFACE_MAGIC 0xBF021234 + static tbm_bufmgr g_surface_bufmgr; -static pthread_mutex_t tbm_surface_lock; +static pthread_mutex_t tbm_surface_lock = PTHREAD_MUTEX_INITIALIZER; void _tbm_surface_mutex_unlock(void); -#define C(b, m) (((b) >> (m)) & 0xFF) -#define B(c, s) ((((unsigned int)(c)) & 0xff) << (s)) -#define FOURCC(a, b, c, d) (B(d, 24) | B(c, 16) | B(b, 8) | B(a, 0)) -#define FOURCC_STR(id) C(id, 0), C(id, 8), C(id, 16), C(id, 24) -#define FOURCC_ID(str) FOURCC(((char*)str)[0], ((char*)str)[1], ((char*)str)[2], ((char*)str)[3]) - /* 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;\ + } \ +} + +#define TBM_SURFACE_RETURN_ERR_IF_FAIL(cond, error_type) {\ + if (!(cond)) {\ + TBM_ERR("'%s' failed.\n", #cond);\ + _tbm_set_last_result(error_type);\ + _tbm_surface_mutex_unlock();\ + return;\ + } \ +} + +#define TBM_SURFACE_RETURN_SET_ERR_IF_FAIL(cond, error, error_type) {\ + if (!(cond)) {\ + TBM_ERR("'%s' failed.\n", #cond);\ + error = error_type;\ + _tbm_set_last_result(error_type);\ _tbm_surface_mutex_unlock();\ return;\ } \ @@ -62,14 +78,34 @@ 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 */ +#define TBM_SURFACE_RETURN_VAL_ERR_IF_FAIL(cond, val, error_type) {\ + if (!(cond)) {\ + TBM_ERR("'%s' failed.\n", #cond);\ + _tbm_set_last_result(error_type);\ + _tbm_surface_mutex_unlock();\ + return val;\ + } \ +} + +#define TBM_SURFACE_RETURN_VAL_SET_ERR_IF_FAIL(cond, val, error, error_type) {\ + if (!(cond)) {\ + TBM_ERR("'%s' failed.\n", #cond);\ + error = error_type;\ + _tbm_set_last_result(error_type);\ + _tbm_surface_mutex_unlock();\ + return val;\ + } \ +} + +/* LCOV_EXCL_START */ static double _tbm_surface_internal_get_time(void) { @@ -218,34 +254,10 @@ _tbm_surface_internal_format_to_str(tbm_format format) return "unknwon"; } } -/* LCOV_EXCL_STOP */ - -static bool -_tbm_surface_mutex_init(void) -{ - static bool tbm_surface_mutex_init = false; - - if (tbm_surface_mutex_init) - return true; - - if (pthread_mutex_init(&tbm_surface_lock, NULL)) { - TBM_LOG_E("fail: pthread_mutex_init for tbm_surface_lock.\n"); - return false; - } - - tbm_surface_mutex_init = true; - - return true; -} void _tbm_surface_mutex_lock(void) { - if (!_tbm_surface_mutex_init()) { - TBM_LOG_E("fail: _tbm_surface_mutex_init.\n"); - return; - } - pthread_mutex_lock(&tbm_surface_lock); } @@ -270,27 +282,33 @@ _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_internal_magic_check(tbm_surface_h surface) { - tbm_surface_h old_data = NULL; + if (surface->magic != TBM_SURFACE_MAGIC) + return 0; - TBM_RETURN_VAL_IF_FAIL(g_surface_bufmgr, 0); - TBM_RETURN_VAL_IF_FAIL(surface, 0); + return 1; +} - 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; - } - } +static int +_tbm_surface_internal_is_valid(tbm_surface_h surface) +{ + if (!surface) { + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER); + TBM_ERR("error: No valid tbm_surface is NULL\n"); + return 0; } - TBM_LOG_E("error: No valid tbm_surface(%p)\n", surface); + if (!_tbm_surface_internal_magic_check(surface)) { + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER); + TBM_ERR("error: No valid tbm_surface(%p)\n", surface); + return 0; + } - return 0; + return 1; } static int @@ -301,21 +319,17 @@ _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; - int ret = 0; + struct _tbm_bufmgr *bufmgr = surf->bufmgr; + 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; - - 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) { - TBM_LOG_E("Fail to surface_get_plane_data. surface(%p)\n", surface); + error = tbm_module_bufmgr_get_plane_data(bufmgr->module, surf->info.format, plane_idx, surf->info.width, surf->info.height, size, offset, pitch, bo_idx); + if (error != TBM_ERROR_NONE) { + _tbm_set_last_result(error); return 0; } @@ -329,6 +343,18 @@ _tbm_surface_internal_destroy(tbm_surface_h surface) tbm_bufmgr bufmgr = surface->bufmgr; tbm_user_data *old_data = NULL, *tmp = NULL; tbm_surface_debug_data *debug_old_data = NULL, *debug_tmp = NULL; + tbm_surface_destroy_func_info *func_info = NULL, *func_next = NULL; + + if (!LIST_IS_EMPTY(&surface->destroy_funcs)) { + LIST_FOR_EACH_ENTRY_SAFE(func_info, func_next, &surface->destroy_funcs, item_link) { + func_info->destroy_func(surface, func_info->user_data); + } + TBM_DBG("free destroy_funcs %p\n", surface); + LIST_FOR_EACH_ENTRY_SAFE(func_info, func_next, &surface->destroy_funcs, item_link) { + LIST_DEL(&func_info->item_link); + free(func_info); + } + } /* destory the user_data_list */ if (!LIST_IS_EMPTY(&surface->user_data_list)) { @@ -351,11 +377,17 @@ _tbm_surface_internal_destroy(tbm_surface_h surface) } LIST_DEL(&surface->item_link); + surface->magic = 0; + + if (surface->hal_surface) { + hal_tbm_surface_free(surface->hal_surface); + surface->hal_surface = NULL; + } free(surface); surface = NULL; - if (LIST_IS_EMPTY(&bufmgr->surf_list)) { + if (bufmgr && LIST_IS_EMPTY(&bufmgr->surf_list)) { LIST_DELINIT(&bufmgr->surf_list); if (!LIST_IS_EMPTY(&bufmgr->debug_key_list)) { @@ -368,73 +400,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) { - int ret = 0; - - _tbm_surface_mutex_lock(); + struct stat sb; - /* Return silently if surface is null. */ - if (!surface) { - _tbm_surface_mutex_unlock(); + if (!path) 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 *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) { - TBM_LOG_E("Fail to surface_supported_format.\n"); - goto fail; - } - - TBM_TRACE("tbm_bufmgr(%p) format num(%u)\n", g_surface_bufmgr, *num); - - _tbm_surface_mutex_unlock(); - - return ret; - -fail: - if (bufmgr_initialized) { - LIST_DELINIT(&g_surface_bufmgr->surf_list); - _deinit_surface_bufmgr(); - } - _tbm_surface_mutex_unlock(); + if (stat(path, &sb) != 0) + return 0; - TBM_LOG_E("error: tbm_bufmgr(%p)\n", g_surface_bufmgr); + if (S_ISLNK(sb.st_mode)) + return 1; 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; @@ -506,16 +492,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; @@ -601,64 +586,54 @@ 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; } -tbm_surface_h -tbm_surface_internal_create_with_flags(int width, int height, - int format, int flags) +static struct _tbm_surface * +_tbm_surface_internal_create_surface(tbm_bufmgr bufmgr, int width, int height, int format, int flags, tbm_error_e *error) { - TBM_RETURN_VAL_IF_FAIL(width > 0, NULL); - TBM_RETURN_VAL_IF_FAIL(height > 0, NULL); - - struct _tbm_bufmgr *mgr; struct _tbm_surface *surf = NULL; - uint32_t size = 0; - uint32_t offset = 0; - uint32_t stride = 0; - uint32_t bo_size = 0; - int bo_idx; - int i, j; - 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 (!TBM_BUFMGR_IS_VALID(mgr)) { - TBM_LOG_E("The bufmgr is invalid\n"); - goto check_valid_fail; - } + uint32_t size = 0, offset = 0, stride = 0, bo_size = 0; + int i, j, bo_idx; 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"); + *error = TBM_ERROR_OUT_OF_MEMORY; goto alloc_surf_fail; + /* LCOV_EXCL_STOP */ } - surf->bufmgr = mgr; + surf->magic = TBM_SURFACE_MAGIC; + 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 from format(%d), error(%s)\n", format, tbm_error_str(*error)); + *error = 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 from format(%d), error(%s)\n", format, tbm_error_str(*error)); + *error = 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"); + if (!_tbm_surface_internal_query_plane_data(surf, i, &size, &offset, &stride, &bo_idx)) { + TBM_ERR("fail to query plane data\n"); + *error = tbm_get_last_error(); goto query_plane_data_fail; } @@ -686,82 +661,517 @@ 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; - - bo = calloc(1, sizeof(struct _tbm_bo)); - if (!bo) { - TBM_LOG_E("fail to alloc bo struct\n"); + surf->bos[i] = tbm_bufmgr_internal_alloc_bo_with_format(bufmgr, format, i, width, height, surf->info.bpp/8, flags, error); + if (!surf->bos[i]) { + surf->bos[i] = tbm_bo_alloc(bufmgr, bo_size, flags); + if (!surf->bos[i]) { + TBM_ERR("fail to alloc bo idx:%d\n", i); + *error = tbm_get_last_error(); goto alloc_bo_fail; } + } + + _tbm_bo_set_surface(surf->bos[i], surf); + } - bo->bufmgr = surf->bufmgr; + *error = TBM_ERROR_NONE; - pthread_mutex_lock(&surf->bufmgr->lock); + return surf; - 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; - } +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: + + return NULL; +} + +static struct _tbm_surface * +_tbm_surface_internal_hal_tbm_create_surface(tbm_bufmgr bufmgr, int width, int height, int format, int flags, tbm_error_e *error) +{ + struct _tbm_surface *surf = NULL; + uint32_t size = 0, offset = 0, stride = 0, bo_size = 0; + int i, j, bo_idx; + hal_tbm_surface *hal_surface = NULL; + hal_tbm_bo **hal_bos = NULL; + int num_bos = 0; + + surf = calloc(1, sizeof(struct _tbm_surface)); + if (!surf) { + /* LCOV_EXCL_START */ + TBM_ERR("fail to alloc surf"); + *error = TBM_ERROR_OUT_OF_MEMORY; + goto alloc_surf_fail; + /* LCOV_EXCL_STOP */ + } + surf->refcnt = 1; - bo->ref_cnt = 1; - bo->flags = flags; - bo->priv = bo_priv; + surf->magic = TBM_SURFACE_MAGIC; + surf->bufmgr = bufmgr; + surf->info.width = width; + surf->info.height = height; + surf->info.format = format; + surf->info.bpp = _tbm_surface_internal_get_bpp(format); + if (!surf->info.bpp) { + TBM_ERR("fail to get bpp from format(%d), error(%s)", format, tbm_error_str(*error)); + *error = tbm_get_last_error(); + goto bpp_fail; + } + surf->flags = flags; - LIST_INITHEAD(&bo->user_data_list); + // get number of planes + surf->info.num_planes = _tbm_surface_internal_get_num_planes(format); + if (!surf->info.num_planes) { + TBM_ERR("fail to get num_planes from format(%d), error(%s)", format, tbm_error_str(*error)); + *error = tbm_get_last_error(); + goto num_planes_fail; + } - LIST_ADD(&bo->item_link, &surf->bufmgr->bo_list); + hal_surface = hal_tbm_bufmgr_alloc_surface(bufmgr->hal_bufmgr, (uint32_t)width, (uint32_t)height, (hal_tbm_format)format, (hal_tbm_bo_memory_type)flags, NULL, 0, (hal_tbm_error *)error); + if (hal_surface) { + // set hal_surface + surf->hal_surface = hal_surface; - pthread_mutex_unlock(&surf->bufmgr->lock); + // set infomation of planes + for (i = 0; i < surf->info.num_planes; i++) { + *error = (tbm_error_e)hal_tbm_surface_get_plane_data(hal_surface, i, &size, &offset, &stride, &bo_idx); + if (*error != TBM_ERROR_NONE) { + goto query_plane_data_fail; + } + surf->info.planes[i].size = size; + surf->info.planes[i].offset = offset; + surf->info.planes[i].stride = stride; + surf->planes_bo_idx[i] = bo_idx; + } + + // set infomation of bos + hal_bos = hal_tbm_surface_get_bos(hal_surface, &num_bos, (hal_tbm_error *)error); + if (!hal_bos) { + TBM_ERR("fail to get bos, error(%s)", tbm_error_str(*error)); + goto get_bos_fail; + } + surf->num_bos = num_bos; + + for (i = 0; i < surf->info.num_planes; i++) { + surf->info.size += surf->info.planes[i].size; - surf->bos[i] = bo; - /* LCOV_EXCL_STOP */ - } else { - surf->bos[i] = tbm_bo_alloc(mgr, bo_size, flags); + if (surf->num_bos < surf->planes_bo_idx[i] + 1) + surf->num_bos = surf->planes_bo_idx[i] + 1; + } + + for (i = 0; i < num_bos; i++) { + surf->bos[i] = tbm_bo_alloc_with_bo_data(bufmgr, (tbm_backend_bo_data *)hal_bos[i], flags); if (!surf->bos[i]) { - TBM_LOG_E("fail to alloc bo idx:%d\n", i); - goto alloc_bo_fail; + TBM_ERR("fail to alloc bo idx:%d", i); + *error = tbm_get_last_error(); + goto get_bo_fail; + } + + _tbm_bo_set_surface(surf->bos[i], surf); + } + } else { + // set infomation of planes + for (i = 0; i < surf->info.num_planes; i++) { + if (!_tbm_surface_internal_query_plane_data(surf, i, &size, &offset, &stride, &bo_idx)) { + TBM_ERR("fail to query plane data"); + *error = tbm_get_last_error(); + goto query_plane_data_fail; } + surf->info.planes[i].size = size; + surf->info.planes[i].offset = offset; + surf->info.planes[i].stride = stride; + surf->planes_bo_idx[i] = bo_idx; } - _tbm_bo_set_surface(surf->bos[i], surf); - } + // count number of bos + surf->num_bos = 1; + for (i = 0; i < surf->info.num_planes; i++) { + surf->info.size += surf->info.planes[i].size; + + if (surf->num_bos < surf->planes_bo_idx[i] + 1) + surf->num_bos = surf->planes_bo_idx[i] + 1; + } - TBM_TRACE("width(%d) height(%d) format(%s) flags(%d) tbm_surface(%p)\n", width, height, - _tbm_surface_internal_format_to_str(format), flags, surf); + // set infomation of bos + for (i = 0; i < surf->num_bos; i++) { + bo_size = 0; + for (j = 0; j < surf->info.num_planes; j++) { + if (surf->planes_bo_idx[j] == i) + bo_size += surf->info.planes[j].size; + } - LIST_INITHEAD(&surf->user_data_list); - LIST_INITHEAD(&surf->debug_data_list); + surf->bos[i] = tbm_bufmgr_internal_alloc_bo_with_format(bufmgr, format, i, width, height, surf->info.bpp / 8, flags, error); + if (*error == TBM_ERROR_NOT_SUPPORTED) { + surf->bos[i] = tbm_bo_alloc(bufmgr, bo_size, flags); + if (!surf->bos[i]) { + TBM_ERR("fail to alloc bo idx:%d", i); + *error = tbm_get_last_error(); + goto get_bo_fail; + } + } - LIST_ADD(&surf->item_link, &mgr->surf_list); + _tbm_bo_set_surface(surf->bos[i], surf); + } + } - _tbm_surface_mutex_unlock(); + *error = TBM_ERROR_NONE; return surf; -alloc_bo_fail: - for (j = 0; j < i; j++) { - if (surf->bos[j]) - tbm_bo_unref(surf->bos[j]); +get_bo_fail: +get_bos_fail: +query_plane_data_fail: + if (hal_surface) { + hal_tbm_surface_free(hal_surface); + } else { + for (j = 0; j < i; j++) { + if (surf->bos[j]) + tbm_bo_unref(surf->bos[j]); + } + } +num_planes_fail: +bpp_fail: + if (surf) + free(surf); +alloc_surf_fail: + + return NULL; +} + +static struct _tbm_surface * +_tbm_surface_internal_hal_tbm_import_surface(tbm_bufmgr bufmgr, int width, int height, int format, tbm_surface_buffer_data *buffer_data, tbm_error_e *error) +{ + struct _tbm_surface *surf = NULL; + uint32_t size = 0, offset = 0, stride = 0; + int i, j, bo_idx; + hal_tbm_surface *hal_surface = NULL; + hal_tbm_bo **hal_bos = NULL; + int num_bos = 0; + int flags; + + surf = calloc(1, sizeof(struct _tbm_surface)); + if (!surf) { + /* LCOV_EXCL_START */ + TBM_ERR("fail to alloc surf"); + *error = TBM_ERROR_OUT_OF_MEMORY; + goto alloc_surf_fail; + /* LCOV_EXCL_STOP */ + } + surf->refcnt = 1; + + surf->magic = TBM_SURFACE_MAGIC; + surf->bufmgr = bufmgr; + surf->info.width = width; + surf->info.height = height; + surf->info.format = format; + surf->info.bpp = _tbm_surface_internal_get_bpp(format); + if (!surf->info.bpp) { + TBM_ERR("fail to get bpp from format(%d), error(%s)", format, tbm_error_str(*error)); + *error = tbm_get_last_error(); + goto bpp_fail; + } + + // get number of planes + surf->info.num_planes = _tbm_surface_internal_get_num_planes(format); + if (!surf->info.num_planes) { + TBM_ERR("fail to get num_planes from format(%d), error(%s)", format, tbm_error_str(*error)); + *error = tbm_get_last_error(); + goto num_planes_fail; + } + + // import surface + hal_surface = hal_tbm_bufmgr_import_surface(bufmgr->hal_bufmgr, + (uint32_t)width, + (uint32_t)height, + (hal_tbm_format)format, + (hal_tbm_surface_buffer_data *)buffer_data, + (hal_tbm_error *)error); + if (!hal_surface) { + TBM_ERR("hal_tbm_bufmgr_import_surface failed.(width:%d height:%d format:%d error:%s)", + width, height, format, tbm_error_str(*error)); + goto import_surface_fail; + } + surf->hal_surface = hal_surface; + + // set infomation of planes + for (i = 0; i < surf->info.num_planes; i++) { + *error = (tbm_error_e)hal_tbm_surface_get_plane_data(hal_surface, i, &size, &offset, &stride, &bo_idx); + if (*error != TBM_ERROR_NONE) { + goto query_plane_data_fail; + } + surf->info.planes[i].size = size; + surf->info.planes[i].offset = offset; + surf->info.planes[i].stride = stride; + surf->planes_bo_idx[i] = bo_idx; + } + + // set infomation of bos + hal_bos = hal_tbm_surface_get_bos(hal_surface, &num_bos, (hal_tbm_error *)error); + if (!hal_bos) { + TBM_ERR("fail to get bos, error(%s)", tbm_error_str(*error)); + goto get_bos_fail; + } + surf->num_bos = num_bos; + + for (i = 0; i < surf->info.num_planes; i++) { + surf->info.size += surf->info.planes[i].size; + + if (surf->num_bos < surf->planes_bo_idx[i] + 1) + surf->num_bos = surf->planes_bo_idx[i] + 1; } + + // get memory_types(bo flags) + flags = (int)hal_tbm_bo_get_memory_types(hal_bos[0], (hal_tbm_error *)error); + if (*error != TBM_ERROR_NONE) { + TBM_ERR("hal_tbm_bo_get_memory_types failed."); + goto get_memory_types_fail; + } + surf->flags = flags; + + for (i = 0; i < num_bos; i++) { + surf->bos[i] = tbm_bo_alloc_with_bo_data(bufmgr, (tbm_backend_bo_data *)hal_bos[i], flags); + if (!surf->bos[i]) { + TBM_ERR("fail to alloc bo idx:%d", i); + *error = tbm_get_last_error(); + goto get_bo_fail; + } + + _tbm_bo_set_surface(surf->bos[i], surf); + } + + *error = TBM_ERROR_NONE; + + return surf; + +get_bo_fail: +get_memory_types_fail: +get_bos_fail: query_plane_data_fail: - free(surf); + if (hal_surface) { + hal_tbm_surface_free(hal_surface); + } else { + for (j = 0; j < i; j++) { + if (surf->bos[j]) + tbm_bo_unref(surf->bos[j]); + } + } +import_surface_fail: +num_planes_fail: +bpp_fail: + if (surf) + free(surf); alloc_surf_fail: + + return NULL; +} + +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; + 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(); + if (!g_surface_bufmgr) { + TBM_ERR("fail bufmgr initialization\n"); + _tbm_set_last_result(TBM_ERROR_INVALID_OPERATION); + goto fail; + } + LIST_INITHEAD(&g_surface_bufmgr->surf_list); + bufmgr_initialized = true; + } + + bufmgr = g_surface_bufmgr; + + error = tbm_module_bufmgr_get_supported_formats(bufmgr->module, formats, num); + if (error != TBM_ERROR_NONE) { + _tbm_set_last_result(error); + goto fail; + } + + 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 1; + +/* 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; +} + +tbm_surface_h +tbm_surface_internal_create_with_flags(int width, int height, + int format, int flags) +{ + struct _tbm_bufmgr *bufmgr; + struct _tbm_surface *surf = NULL; + tbm_error_e error = TBM_ERROR_INVALID_OPERATION; + bool bufmgr_initialized = false; + + _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(); + if (!g_surface_bufmgr) { + TBM_ERR("fail bufmgr initialization\n"); + error = TBM_ERROR_INVALID_OPERATION; + goto check_valid_fail; + } + LIST_INITHEAD(&g_surface_bufmgr->surf_list); + bufmgr_initialized = true; + } + + bufmgr = g_surface_bufmgr; + if (!TBM_BUFMGR_IS_VALID(bufmgr)) { + TBM_ERR("The bufmgr is invalid\n"); + error = TBM_ERROR_INVALID_PARAMETER; + goto check_valid_fail; + } + + if (bufmgr->use_hal_tbm) { + surf = _tbm_surface_internal_hal_tbm_create_surface(bufmgr, width, height, format, flags, &error); + if (!surf) { + TBM_ERR("_tbm_surface_internal_hal_tbm_create_surface failed."); + goto surface_alloc_fail; + } + } else { + surf = _tbm_surface_internal_create_surface(bufmgr, width, height, format, flags, &error); + if (!surf) { + TBM_ERR("_tbm_surface_internal_create_surface failed."); + goto surface_alloc_fail; + } + } + + 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_INITHEAD(&surf->destroy_funcs); + + LIST_ADD(&surf->item_link, &bufmgr->surf_list); + + _tbm_set_last_result(error); + _tbm_surface_mutex_unlock(); + + return surf; + +/* LCOV_EXCL_START */ + +surface_alloc_fail: check_valid_fail: - if (bufmgr_initialized && mgr) { - LIST_DELINIT(&mgr->surf_list); + if (bufmgr_initialized && bufmgr) { + LIST_DELINIT(&bufmgr->surf_list); _deinit_surface_bufmgr(); } + + TBM_ERR("error: width(%d) height(%d) format(%s) flags(%d)\n", + width, height, _tbm_surface_internal_format_to_str(format), flags); + + _tbm_set_last_result(error); _tbm_surface_mutex_unlock(); - TBM_LOG_E("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; } @@ -770,43 +1180,61 @@ 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(); + if (!g_surface_bufmgr) { + TBM_ERR("fail bufmgr initialization\n"); + _tbm_set_last_result(TBM_ERROR_INVALID_OPERATION); + goto check_valid_fail; + } LIST_INITHEAD(&g_surface_bufmgr->surf_list); bufmgr_initialized = true; } - mgr = g_surface_bufmgr; - if (!TBM_BUFMGR_IS_VALID(mgr)) { - TBM_LOG_E("fail to validate the Bufmgr.\n"); + 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) { - TBM_LOG_E("fail to allocate struct _tbm_surface.\n"); + /* 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->magic = TBM_SURFACE_MAGIC; + surf->bufmgr = bufmgr; surf->info.width = info->width; surf->info.height = info->height; surf->info.format = info->format; if (info->bpp > 0) surf->info.bpp = info->bpp; - else - surf->info.bpp = tbm_surface_internal_get_bpp(info->format); + 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; @@ -821,7 +1249,10 @@ tbm_surface_internal_create_with_bos(tbm_surface_info_s *info, uint32_t size = 0, offset = 0, stride = 0; int32_t bo_idx = 0; - _tbm_surface_internal_query_plane_data(surf, i, &size, &offset, &stride, &bo_idx); + 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; } @@ -845,7 +1276,8 @@ tbm_surface_internal_create_with_bos(tbm_surface_info_s *info, surf->num_bos = num; for (i = 0; i < num; i++) { if (bos[i] == NULL) { - TBM_LOG_E("bos[%d] is null.\n", i); + TBM_ERR("bos[%d] is null.\n", i); + _tbm_set_last_result(TBM_ERROR_INVALID_OPERATION); goto check_bo_fail; } @@ -853,19 +1285,23 @@ tbm_surface_internal_create_with_bos(tbm_surface_info_s *info, _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_INITHEAD(&surf->destroy_funcs); - 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]); @@ -873,15 +1309,16 @@ check_bo_fail: free(surf); alloc_surf_fail: check_valid_fail: - if (bufmgr_initialized && mgr) { - LIST_DELINIT(&mgr->surf_list); + if (bufmgr_initialized && bufmgr) { + LIST_DELINIT(&bufmgr->surf_list); _deinit_surface_bufmgr(); } _tbm_surface_mutex_unlock(); - TBM_LOG_E("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; } @@ -890,21 +1327,24 @@ 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)); 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); + else // if (surface->refcnt < 0) + _tbm_set_last_result(TBM_ERROR_INVALID_PARAMETER); _tbm_surface_mutex_unlock(); } @@ -913,12 +1353,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)); 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(); } @@ -927,18 +1368,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)); 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); @@ -953,13 +1395,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); 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(); @@ -973,6 +1419,7 @@ 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(bo_idx > -1, NULL); @@ -980,7 +1427,7 @@ tbm_surface_internal_get_bo(tbm_surface_h surface, int bo_idx) 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(); @@ -994,13 +1441,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); 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(); @@ -1014,6 +1462,7 @@ 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(plane_idx > -1, 0); @@ -1021,7 +1470,8 @@ tbm_surface_internal_get_plane_data(tbm_surface_h surface, int plane_idx, 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; } @@ -1035,7 +1485,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); @@ -1050,9 +1500,13 @@ 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); @@ -1068,42 +1522,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_LOG_E("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_LOG_E("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(); @@ -1117,6 +1577,7 @@ 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)); @@ -1125,7 +1586,7 @@ tbm_surface_internal_unmap(tbm_surface_h 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(); } @@ -1137,13 +1598,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); 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(); @@ -1157,13 +1619,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); 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(); @@ -1178,13 +1641,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); 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(); @@ -1198,6 +1662,7 @@ 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(plane_idx > -1, 0); @@ -1205,7 +1670,7 @@ tbm_surface_internal_get_plane_bo_idx(tbm_surface_h surface, int plane_idx) 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(); @@ -1219,25 +1684,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); /* 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); @@ -1253,12 +1720,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); 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; } @@ -1268,7 +1737,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(); @@ -1282,11 +1751,13 @@ 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); if (!data) { - TBM_LOG_E("error: tbm_surface(%p) key(%lu)\n", surface, key); + 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; } @@ -1294,14 +1765,15 @@ tbm_surface_internal_get_user_data(tbm_surface_h surface, unsigned long key, 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; } *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(); @@ -1315,17 +1787,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); 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); @@ -1347,6 +1821,7 @@ 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)); @@ -1361,8 +1836,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); @@ -1370,14 +1848,48 @@ _tbm_surface_internal_debug_data_create(char *key, char *value) return debug_data; } +static void +_tbm_surface_internal_debug_data_value_update(tbm_surface_debug_data *debug_data, char *value) +{ + if (!debug_data->value && !value) + return; + + if (debug_data->value && value && !strncmp(debug_data->value, value, strlen(debug_data->value))) + return; + + if (debug_data->value) + free(debug_data->value); + + if (value) + debug_data->value = strdup(value); + else + debug_data->value = NULL; +} + +static tbm_surface_debug_data * +_tbm_surface_internal_debug_data_find(struct list_head *list, char *key) +{ + tbm_surface_debug_data *debug_data = NULL; + + if (LIST_IS_EMPTY(list)) + return NULL; + + LIST_FOR_EACH_ENTRY(debug_data, list, item_link) { + if (!strncmp(debug_data->key, key, strlen(debug_data->key))) + return debug_data; + } + + return NULL; +} + int tbm_surface_internal_set_debug_data(tbm_surface_h surface, char *key, char *value) { tbm_surface_debug_data *debug_data = NULL; - tbm_surface_debug_data *old_data = NULL, *tmp = NULL; 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(key, 0); @@ -1386,52 +1898,29 @@ 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(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("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_find(&surface->debug_data_list, key); + if (debug_data) { + _tbm_surface_internal_debug_data_value_update(debug_data, value); + } else { + debug_data = _tbm_surface_internal_debug_data_create(key, value); + if (!debug_data) { + TBM_ERR("error: tbm_surface(%p) key(%s) value(%s)\n", surface, key, value); + _tbm_surface_mutex_unlock(); + return 0; } - } - debug_data = _tbm_surface_internal_debug_data_create(key, value); - if (!debug_data) { - TBM_LOG_E("error: tbm_surface(%p) key(%s) value(%s)\n", surface, key, value); - _tbm_surface_mutex_unlock(); - return 0; + LIST_ADD(&debug_data->item_link, &surface->debug_data_list); } - TBM_TRACE("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)) { - _tbm_surface_mutex_unlock(); - return 1; - } - } + /* add new debug key to list */ + debug_data = _tbm_surface_internal_debug_data_find(&bufmgr->debug_key_list, key); + if (!debug_data) { + debug_data = _tbm_surface_internal_debug_data_create(key, NULL); + if (debug_data) + LIST_ADDTAIL(&debug_data->item_link, &bufmgr->debug_key_list); } - debug_data = _tbm_surface_internal_debug_data_create(key, NULL); - LIST_ADD(&debug_data->item_link, &bufmgr->debug_key_list); + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p) key(%s) value(%s)\n", surface, key, value); _tbm_surface_mutex_unlock(); @@ -1489,10 +1978,15 @@ static void _tbm_surface_internal_dump_file_raw(const char *file, void *data1, int size1, void *data2, int size2, void *data3, int size3) { - FILE *fp = fopen(file, "w+"); - TBM_RETURN_IF_FAIL(fp != NULL); + FILE *fp; unsigned int *blocks; + 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; fwrite(blocks, 1, size1, fp); @@ -1510,47 +2004,73 @@ _tbm_surface_internal_dump_file_raw(const char *file, void *data1, int size1, } 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 stride, int format) { unsigned int *blocks = (unsigned int *)data; - FILE *fp = fopen(file, "wb"); - TBM_RETURN_IF_FAIL(fp != NULL); - const int pixel_size = 4; // RGBA + 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); + png_structp pPngStruct = png_create_write_struct(PNG_LIBPNG_VER_STRING, NULL, NULL, NULL); if (!pPngStruct) { - TBM_LOG_E("fail to create a png write structure.\n"); + TBM_ERR("fail to create a png write structure.\n"); fclose(fp); return; } png_infop pPngInfo = png_create_info_struct(pPngStruct); if (!pPngInfo) { - TBM_LOG_E("fail to create a png info structure.\n"); + TBM_ERR("fail to create a png info structure.\n"); png_destroy_write_struct(&pPngStruct, NULL); fclose(fp); return; } + if (setjmp(png_jmpbuf(pPngStruct))) { + /* if png has problem of writing the file, we get here */ + TBM_ERR("fail to write png file.\n"); + png_destroy_write_struct(&pPngStruct, &pPngInfo); + 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); row_pointers = png_malloc(pPngStruct, height * sizeof(png_byte *)); if (!row_pointers) { - TBM_LOG_E("fail to allocate the png row_pointers.\n"); + TBM_ERR("fail to allocate the png row_pointers.\n"); png_destroy_write_struct(&pPngStruct, &pPngInfo); fclose(fp); return; @@ -1562,7 +2082,7 @@ _tbm_surface_internal_dump_file_png(const char *file, const void *data, int widt row = png_malloc(pPngStruct, sizeof(png_byte) * width * pixel_size); if (!row) { - TBM_LOG_E("fail to allocate the png row.\n"); + 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); @@ -1573,12 +2093,18 @@ _tbm_surface_internal_dump_file_png(const char *file, const void *data, int widt 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; + unsigned int curBlock = blocks[(y * (stride >> 2)) + x]; + + 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; + } } } @@ -1609,7 +2135,7 @@ tbm_surface_internal_dump_start(char *path, int w, int h, int count) /* 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; } @@ -1623,23 +2149,20 @@ 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; } - if (TBM_SURFACE_ERROR_NONE != tbm_surface_map(tbm_surface, - TBM_SURF_OPTION_READ, &info)) { - 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 */ @@ -1651,7 +2174,7 @@ tbm_surface_internal_dump_start(char *path, int w, int h, int count) bo = tbm_bo_alloc(g_surface_bufmgr, buffer_size, TBM_BO_DEFAULT); if (bo == NULL) { - TBM_LOG_E("fail to allocate the tbm_bo[%d]\n", i); + TBM_ERR("fail to allocate the tbm_bo[%d]\n", i); free(buf_info); goto fail; } @@ -1668,7 +2191,7 @@ tbm_surface_internal_dump_start(char *path, int w, int h, int count) scale_factor = 0.0; - TBM_LOG_I("Dump Start.. path:%s, count:%d\n", g_dump_info->path, count); + TBM_INFO("Dump Start.. path:%s, count:%d\n", g_dump_info->path, count); return; @@ -1684,7 +2207,7 @@ fail: } } - 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; @@ -1695,10 +2218,12 @@ fail: void tbm_surface_internal_dump_with_scale_start(char *path, int w, int h, int count, double scale) { - TBM_RETURN_IF_FAIL(scale > 0.0); - - tbm_surface_internal_dump_start(path, (int)(scale * w), (int)(scale * h), count); + if (scale > 0.0) { + w *= scale; + h *= scale; + } + tbm_surface_internal_dump_start(path, w, h, count); scale_factor = scale; } @@ -1730,17 +2255,25 @@ tbm_surface_internal_dump_end(void) } snprintf(file, sizeof(file), "%s/%s", g_dump_info->path, buf_info->name); - TBM_LOG_I("Dump File.. %s generated.\n", file); + TBM_INFO("Dump File.. %s generated.\n", file); if (buf_info->dirty) { void *ptr1 = NULL, *ptr2 = NULL; 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, + buf_info->info.planes[0].stride, + 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); + buf_info->info.height, + buf_info->info.planes[0].stride, + TBM_FORMAT_XRGB8888); break; case TBM_FORMAT_YVU420: case TBM_FORMAT_YUV420: @@ -1769,13 +2302,14 @@ tbm_surface_internal_dump_end(void) NULL, 0, NULL, 0); break; default: - TBM_LOG_E("can't dump %c%c%c%c buffer", FOURCC_STR(buf_info->info.format)); + 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); + buf_info->shm_h, + buf_info->shm_stride, 0); tbm_bo_unmap(buf_info->bo); tbm_bo_unref(buf_info->bo); @@ -1786,7 +2320,7 @@ tbm_surface_internal_dump_end(void) free(g_dump_info); g_dump_info = NULL; - TBM_LOG_I("Dump End..\n"); + TBM_INFO("Dump End..\n"); } static pixman_format_code_t @@ -1861,6 +2395,52 @@ cant_convert: 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 + 1); + } + + _tbm_surface_mutex_unlock(); + + return keys; +} + +static void _tbm_surface_internal_put_keys(char *keys) +{ + if (keys) + free(keys); +} + void tbm_surface_internal_dump_buffer(tbm_surface_h surface, const char *type) { @@ -1872,6 +2452,8 @@ tbm_surface_internal_dump_buffer(tbm_surface_h surface, const char *type) tbm_surface_info_s info; tbm_bo_handle bo_handle; const char *postfix; + const char *format = NULL; + char *keys; int ret; if (!g_dump_info) @@ -1895,7 +2477,7 @@ tbm_surface_internal_dump_buffer(tbm_surface_h surface, const char *type) const int bpp = 4; if (info.format != TBM_FORMAT_ARGB8888 && info.format != TBM_FORMAT_XRGB8888) { - TBM_LOG_W("Dump with scale skip. unsupported format(%s)\n", + TBM_WRN("Dump with scale skip. unsupported format(%s)\n", _tbm_surface_internal_format_to_str(info.format)); tbm_surface_unmap(surface); return; @@ -1906,20 +2488,25 @@ tbm_surface_internal_dump_buffer(tbm_surface_h surface, const char *type) 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); + 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_LOG_W("Dump with scale skip. surface over created buffer size(%u, %d)\n", + 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_LOG_W("Dump skip. surface over created buffer size(%u, %d)\n", + TBM_WRN("Dump skip. surface over created buffer size(%u, %d)\n", info.size, buf_info->size); tbm_surface_unmap(surface); return; @@ -1929,15 +2516,24 @@ tbm_surface_internal_dump_buffer(tbm_surface_h surface, const char *type) 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]; + 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); if (!bo_handle.ptr) { - TBM_LOG_E("fail to map bo"); + TBM_ERR("fail to map bo"); + _tbm_surface_internal_put_keys(keys); tbm_surface_unmap(surface); return; } @@ -1947,9 +2543,9 @@ tbm_surface_internal_dump_buffer(tbm_surface_h surface, const char *type) case TBM_FORMAT_ARGB8888: case TBM_FORMAT_XRGB8888: snprintf(buf_info->name, sizeof(buf_info->name), - "%10.3f_%03d_%p-%s.%s", + "%10.3f_%03d%s_%p_%s-%s.%s", _tbm_surface_internal_get_time(), - g_dump_info->count++, surface, type, postfix); + g_dump_info->count++, keys, surface, format, type, postfix); if (scale_factor > 0.0) { ret = _tbm_surface_internal_buffer_scale(info.planes[0].ptr, @@ -1961,8 +2557,9 @@ tbm_surface_internal_dump_buffer(tbm_surface_h surface, const char *type) buf_info->info.width, buf_info->info.height); if (ret != TBM_SURFACE_ERROR_NONE) { - TBM_LOG_E("fail to scale buffer"); + TBM_ERR("fail to scale buffer"); tbm_bo_unmap(buf_info->bo); + _tbm_surface_internal_put_keys(keys); tbm_surface_unmap(surface); return; } @@ -1972,9 +2569,9 @@ tbm_surface_internal_dump_buffer(tbm_surface_h surface, const char *type) 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", + "%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, + 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; @@ -1985,9 +2582,9 @@ tbm_surface_internal_dump_buffer(tbm_surface_h surface, const char *type) 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", + "%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, + 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; @@ -1996,21 +2593,24 @@ tbm_surface_internal_dump_buffer(tbm_surface_h surface, const char *type) 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", + "%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, + 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; @@ -2021,7 +2621,7 @@ 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, @@ -2063,7 +2663,7 @@ void tbm_surface_internal_dump_shm_buffer(void *ptr, int w, int h, int stride, size = stride * h; if (size > buf_info->size) { - TBM_LOG_W("Dump skip. shm buffer over created buffer size(%d, %d)\n", + TBM_WRN("Dump skip. shm buffer over created buffer size(%d, %d)\n", size, buf_info->size); return; } @@ -2083,7 +2683,7 @@ void tbm_surface_internal_dump_shm_buffer(void *ptr, int w, int h, int stride, TBM_FORMAT_ARGB8888, stride, w, h, dstride, dw, dh); if (ret != TBM_SURFACE_ERROR_NONE) { - TBM_LOG_E("fail to scale buffer"); + TBM_ERR("fail to scale buffer"); tbm_bo_unmap(buf_info->bo); return; } @@ -2105,7 +2705,7 @@ void tbm_surface_internal_dump_shm_buffer(void *ptr, int w, int h, int stride, g_dump_info->link = next_link; - TBM_LOG_I("Dump %s \n", buf_info->name); + TBM_INFO("Dump %s \n", buf_info->name); } int @@ -2129,7 +2729,7 @@ tbm_surface_internal_capture_buffer(tbm_surface_h surface, const char *path, con postfix = dump_postfix[1]; if (strcmp(postfix, type)) { - TBM_LOG_E("not support type(%s) %c%c%c%c buffer", type, FOURCC_STR(info.format)); + TBM_ERR("not support type(%s) %c%c%c%c buffer", type, FOURCC_STR(info.format)); tbm_surface_unmap(surface); return 0; } @@ -2137,17 +2737,25 @@ tbm_surface_internal_capture_buffer(tbm_surface_h surface, const char *path, con snprintf(file, sizeof(file), "%s/%s.%s", path , name, postfix); if (!access(file, 0)) { - TBM_LOG_E("can't capture buffer, exist file %s", file); + 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.width, + info.height, + info.planes[0].stride, + 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); + info.width, + info.height, + info.planes[0].stride, + TBM_FORMAT_XRGB8888); break; case TBM_FORMAT_YVU420: case TBM_FORMAT_YUV420: @@ -2173,14 +2781,14 @@ tbm_surface_internal_capture_buffer(tbm_surface_h surface, const char *path, con NULL, 0, NULL, 0); break; default: - TBM_LOG_E("can't dump %c%c%c%c buffer", FOURCC_STR(info.format)); + 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("Capture %s \n", file); + TBM_TRACE_SURFACE_INTERNAL("Capture %s \n", file); return 1; } @@ -2199,21 +2807,221 @@ tbm_surface_internal_capture_shm_buffer(void *ptr, int w, int h, int stride, char file[1024]; if (strcmp(dump_postfix[0], type)) { - TBM_LOG_E("Not supported type:%s'", 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_LOG_E("can't capture buffer, exist file %s", file); + TBM_ERR("can't capture buffer, exist file %sTBM_FORMAT_XRGB8888", file); return 0; } - snprintf(file, sizeof(file), "%s/%s.%s", path , name, dump_postfix[0]); + _tbm_surface_internal_dump_file_png(file, ptr, w, h, stride, 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(_tbm_surface_internal_is_valid(surface), 0); + + surf = (struct _tbm_surface *)surface; + + if (x) *x = surf->damage.x; + if (y) *y = surf->damage.y; + if (width) *width = surf->damage.width; + if (height) *height = surf->damage.height; + + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p) x(%d) y(%d) width(%d) height(%d)\n", + surface, surf->damage.x, surf->damage.y, surf->damage.width, surf->damage.height); + + _tbm_surface_mutex_unlock(); + + return 1; +} + +int +tbm_surface_internal_add_destroy_handler(tbm_surface_h surface, tbm_surface_internal_destroy_handler func, void *user_data) +{ + struct _tbm_surface *surf; + tbm_surface_destroy_func_info *func_info = 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(func != NULL, 0); + + surf = (struct _tbm_surface *)surface; + LIST_FOR_EACH_ENTRY(func_info, &surf->destroy_funcs, item_link) { + if (func_info->destroy_func == func && func_info->user_data == user_data) { + TBM_ERR("can't add twice"); + _tbm_set_last_result(TBM_ERROR_INVALID_OPERATION); + _tbm_surface_mutex_unlock(); + return 0; + } + } + + func_info = calloc(1, sizeof(tbm_surface_destroy_func_info)); + if (func_info == NULL) { + TBM_ERR("alloc failed"); + _tbm_set_last_result(TBM_ERROR_OUT_OF_MEMORY); + _tbm_surface_mutex_unlock(); + return 0; + } + + func_info->destroy_func = func; + func_info->user_data = user_data; - _tbm_surface_internal_dump_file_png(file, ptr, stride, h); + LIST_ADDTAIL(&func_info->item_link, &surf->destroy_funcs); - TBM_TRACE("Capture %s \n", file); + _tbm_surface_mutex_unlock(); return 1; } + +void +tbm_surface_internal_remove_destroy_handler(tbm_surface_h surface, tbm_surface_internal_destroy_handler func, void *user_data) +{ + struct _tbm_surface *surf; + tbm_surface_destroy_func_info *func_info = NULL, *next = NULL; + + _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(func != NULL); + + surf = (struct _tbm_surface *)surface; + LIST_FOR_EACH_ENTRY_SAFE(func_info, next, &surf->destroy_funcs, item_link) { + if (func_info->destroy_func != func || func_info->user_data != user_data) + continue; + + LIST_DEL(&func_info->item_link); + free(func_info); + + _tbm_surface_mutex_unlock(); + + return; + } + + _tbm_surface_mutex_unlock(); +} + +tbm_surface_buffer_data * +tbm_surface_internal_export(tbm_surface_h surface, tbm_error_e *error) +{ + tbm_surface_buffer_data *buffer_data = NULL; + struct _tbm_surface *surf; + struct _tbm_bufmgr *bufmgr; + + _tbm_surface_mutex_lock(); + + surf = (struct _tbm_surface *)surface; + TBM_SURFACE_RETURN_VAL_SET_ERR_IF_FAIL(_tbm_surface_internal_is_valid(surface), NULL, *error, TBM_ERROR_INVALID_PARAMETER); + + bufmgr = surf->bufmgr; + TBM_SURFACE_RETURN_VAL_SET_ERR_IF_FAIL(bufmgr != NULL, NULL, *error, TBM_ERROR_INVALID_PARAMETER); + + // this function supports when it comes to be use_hal. + TBM_SURFACE_RETURN_VAL_SET_ERR_IF_FAIL(bufmgr->use_hal_tbm, NULL, *error, TBM_ERROR_NOT_SUPPORTED); + + // export a surface + buffer_data = (tbm_surface_buffer_data *)hal_tbm_surface_export((hal_tbm_surface *)surf->hal_surface, + (hal_tbm_error *)error); + TBM_SURFACE_RETURN_VAL_ERR_IF_FAIL(buffer_data != NULL, NULL, *error); + + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p) buffer_data(%p)", surface, buffer_data); + + if (error) + *error = TBM_ERROR_NONE; + + _tbm_set_last_result(TBM_ERROR_NONE); + _tbm_surface_mutex_unlock(); + + return buffer_data; +} + +tbm_surface_h +tbm_surface_internal_import(tbm_surface_info_s *surface_info, tbm_surface_buffer_data *buffer_data, tbm_error_e *error) +{ + struct _tbm_surface *surf; + struct _tbm_bufmgr *bufmgr; + + _tbm_surface_mutex_lock(); + + bufmgr = g_surface_bufmgr; + TBM_SURFACE_RETURN_VAL_SET_ERR_IF_FAIL(bufmgr != NULL, NULL, *error, TBM_ERROR_INVALID_PARAMETER); + + // this function supports when it comes to be use_hal. + TBM_SURFACE_RETURN_VAL_SET_ERR_IF_FAIL(bufmgr->use_hal_tbm, NULL, *error, TBM_ERROR_NOT_SUPPORTED); + + TBM_SURFACE_RETURN_VAL_SET_ERR_IF_FAIL(surface_info != NULL, NULL, *error, TBM_ERROR_INVALID_PARAMETER); + TBM_SURFACE_RETURN_VAL_SET_ERR_IF_FAIL(surface_info->width > 0, NULL, *error, TBM_ERROR_INVALID_PARAMETER); + TBM_SURFACE_RETURN_VAL_SET_ERR_IF_FAIL(surface_info->height > 0, NULL, *error, TBM_ERROR_INVALID_PARAMETER); + TBM_SURFACE_RETURN_VAL_SET_ERR_IF_FAIL(buffer_data != NULL, NULL, *error, TBM_ERROR_INVALID_PARAMETER); + + // import a surface + surf = _tbm_surface_internal_hal_tbm_import_surface(bufmgr, + (int)surface_info->width, + (int)surface_info->height, + (int)surface_info->format, + buffer_data, + error); + TBM_SURFACE_RETURN_VAL_ERR_IF_FAIL(surf != NULL, NULL, *error); + + LIST_INITHEAD(&surf->user_data_list); + LIST_INITHEAD(&surf->debug_data_list); + LIST_INITHEAD(&surf->destroy_funcs); + + LIST_ADD(&surf->item_link, &bufmgr->surf_list); + + TBM_TRACE_SURFACE_INTERNAL("tbm_surface(%p)", surf); + + if (error) + *error = TBM_ERROR_NONE; + + _tbm_set_last_result(TBM_ERROR_NONE); + _tbm_surface_mutex_unlock(); + + return (tbm_surface_h)surf; +} + /*LCOV_EXCL_STOP*/