execute mutex unlock before false return

Change-Id: I5016870d1d7e68799c100dff259eb2ae4f408961

don't check path condition if queue_dump off

Change-Id: I425c8376c503285d89b6f113d2808b77648d0095
Signed-off-by: Junkyeong Kim <jk0430.kim@samsung.com>

pakcage version up to 2.0.13

Change-Id: I7ba9cb857b99f13acba7bddb964d6a43cb110996

wrap surface mutex lock in tbm_surface_interal_is_valid

Change-Id: I2c1d6c57b186419f54f7b63899e23c761da833c6

obey the code rule

Change-Id: Ibd0599add7ad8b984719f83d2b176ce73a0c6ea0
Signed-off-by: SooChan Lim <sc1.lim@samsung.com>

added warning log when tbm_bufmgr_init get valid fd

Change-Id: I7115ea121221a70341781bd064effed32884d608

added fucntion for getting drm fd used by backend

Change-Id: I8aa01d67ad49597b99552e3cd6acee7675d89f86

added description in tbm_drm_helper header

Change-Id: I2235b254e205b59b7fb85124267a3178e7a2fd32

package version up to 2.0.12

Change-Id: If02ae9ecbcd230224035f448001851e7eaa0131d

fix memory leak in tbm_surface_internal_set_debug_data()
 - old_data->value is come from strdup(). so it should be free.

Change-Id: I130ac337bbbd00b7f4fa89886a5c392fc81180b8
Signed-off-by: Minsu Han <minsu81.han@samsung.com>

added capture buffer api

Change-Id: I43f6976b05a1e00099b4df0bcd2c6edb8966ce63

pakcage version up to 2.0.11

Change-Id: I10fb0b7ef988b9676de80681eb4944d47b245935

Don't use surface mutex lock in internal function

- process into deadlock when create tbm_surface and call tbm_bufmgr_debug_show

Change-Id: I05d9aa33d27c2e790d44e8365e45c30029bfd498

pakcage version up to 2.0.10

Change-Id: I383aa6034c602034ac608f13179af536898f92cb

return 0 when surf_list is null.

Change-Id: Ia3aafa38a33f534e2758842471c4a798d1991121

package version up to 2.0.9

Change-Id: I479b8c4505804fa8829c5bd2823ac4546c7c92a3

use LIST_FOR_EACH_ENTRY_SAFE at _notify_emit

The item->cb is the outside function of the libtbm.
The tbm user may/can remove the item of the list,
so we have to use the LIST_FOR_EACH_ENTRY_SAFE.

Change-Id: Ib8ccbe0e1145f410001798a54535ee92d8dcc209

surface_queue: unlock mutex lock of queue when call alloc cb

Change-Id: I4f4dd4e9eecd7898cb6f602144a017bcb68bdfb4

null initialization

LIST_FOR_EACH_ENTRY may use the uninitialized data.

Change-Id: I32efcdd051b71ee3d38c8a5d152066bd806284d5

tbm_bufmgr: clean up tbm_bufmgr_debug_***() functions

This patch cleans up tbm_bufmgr_debug_***() functions.

Change-Id: I0745dee5781acc066c4fc5f802157e9ec7f4141c
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>

tbm_bufmgr: clean up tbm_bufmgr_***() functions

This patch cleans up tbm_bufmgr_***() functions.

Change-Id: Id3d70acc960c3fc47d8ccba13a48c0acabb5845a
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>

tbm_bufmgr: clean up tbm_bo_***() functions

This patch cleans up tbm_bo_***() functions.

Change-Id: Ib954922063a3b50f38993a59ce4adc825e379ed7
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>

tbm_bufmgr: clean up tbm_bufmgr_init/deinit()

This patch cleans up tbm_bufmgr_init/deinit().

Change-Id: Ib95d22d98c6029c06a522ae80811dc3a11927721
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>

tbm_bufmgr: clean up load module codes

This patch cleans up load module codes.
- Tries not to use over 2-depth condition.

Change-Id: I0bcd7e0308e85028db4bf7b2c740cf052b1ec245
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>

tbm_bufmgr: clean up internal functions

This patch cleans up internal functions.
- Renames tbm_flag_to_str() as _tbm_flag_to_str() with static.
- Moves last_chk_bo_cnt into _tbm_util_check_bo_cnt.
- Uses LIST_FOR_EACH_ENTRY() if it is enough.
- Uses switch/case for readability in (_)tbm_bo_(un)lock().

Change-Id: Ia41fb6c793e2668574810d2b70b4ee12cb3b34ba
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>

package version up to 2.0.8

Change-Id: I20fe01c5050c6594230833410a8fc05058b2d6e3

set the null initialization

fix issues from the static code analysis tools

Change-Id: I8c8a4cf54eecccddb58d3fc3363eacb8f38446fa

remove the profile condition at spec file

make the symbolic link of tbm-drm-auth-user.path to
basic.target.want

Change-Id: Ie6d0ce9e859f38b8be2c475fbb1b02b7c90c33ac

tbm_bufmgr_backend: clean up header

This patch cleans up tbm_bufmgr_backend.h.

Change-Id: I5b3d7c6c4ce61c467d2d247f26c702e223996e7f
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>

tbm_surface_queue: clean up tbm_surface_queue_can_dequeue / acquire

This patch cleans up tbm_surface_queue_can_dequeue / acqurie
not to use 3-depth 'if' condition.

Change-Id: I7529ec36205eb25f1e772408686018b03a2baa18
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>

tbm_surface_queue: change return code in tbm_surface_queue_release()

This patch changes return code in tbm_surface_queue_release()
from TBM_SURFACE_QUEUE_ERROR_NONE to TBM_SURFACE_QUEUE_ERROR_INVALID_SURFACE
when free_queue is empty.

Change-Id: I980ec7c8f0b8ad7312b88c4a56bfa46010db4c2a
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>

tbm_surface_queue: move output variable setting point

This patch moves output variable setting point,
so it could be set even function is exited by TBM_SURF_QUEUE_RETURN_***.

Change-Id: Ia3190b695d96f7657f6639d46a26ec7336a7d64b
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>

tbm_surface_queue: clean up tbm_surface_queue_init / destroy

This patch cleans up tbm_surface_queue_init / destroy.
- The surface_queue is created by calloc, so memset() is not required
  in tbm_surface_queue_init().
- There are missed _notify_remove_all() calls for dequeue_noti and acquirable_noti
  in tbm_surface_queue_destroy().
- Reorders notifications list as structure defined order.

Change-Id: I2ab5312a4dd686cb55beca5c7601a849fd00412a
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>

tbm_surface_queue: cast impl_data as actual tbm_queue_*** type

This patch casts impl_data as actual tbm_queue_*** type.

Change-Id: I12ecb7d5357280e6266448071d8d31104b83c280
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>

tbm_surface_queue: clean up LIST related codes

This patch cleans up LIST related codes.
- Uses LIST_FOR_EACH_ENTRY if it is enough.
- Removes meaningless variable initialization.

Change-Id: I3ec2a94dc74cebf46a233f37c8b4b0464f04ce54
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>

tbm_surface_queue: declare _tbm_surface_queue_***() as static function

This patch declares _tbm_surface_queue_***() as static function.

Change-Id: I6cf963913cfbfd5bac4bafc6a4e08b69b9f91973
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>

package version up to 2.0.7

Change-Id: I110b60aef479b4fbc88243474eef66d18cff3c78

apply the coding rule

Change-Id: I6128af6c8a35254ea5e0543e126d02b6a5fabdc9

tbm_surface_internal: clean up dump buffer functions

This patch cleans up dump buffer functions.
- Adds missed error routines.
- Adds missed buf_info LIST_DEL routines.

Change-Id: Iccdb1d386eefd15c06337d2a7275421824ef1331
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>

tbm_bufmgr, tbm_surface_internal: clean up LIST relevant codes

This patch cleans up LIST relevant codes.
- LIST_FOR_EACH_ENTRY is enough if there is no LIST_DEL routine inside.
- The user_data_lookup() checks LIST_IS_EMPTY() internally, so removes duplicated check routine.

Change-Id: Id4821fd22014cb5dfa32a77da8f5c26e0dc9569c
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>

tbm_surface_internal: clean up tbm_surface_internal_get_info()

This patch cleans up tbm_surface_internal_get_info().
There is a missed error routine.

Change-Id: I31be6bcb1b974c7ca7e6016b4a0879a766ab3089
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>

tbm_surface_internal: fix header descriptions

This patch fixes header descriptions for error return value.

Change-Id: Id3d02f44cef981fcb716e32b63bd5130d88981b4
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>

tbm_surface_internal: change tbm_surface_internal_get_size() return type

This patch changes tbm_surface_internal_get_size() return type
from int to unsigned int.
The type of size in _tbm_surface_info is uint32_t and this function returns it.

Change-Id: I72a3ae9cda8e77a49e36122d19ae9ca8353cc674
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>

tbm_surface_internal: clean up tbm_surface_internal_query_supported_formats()

This patch cleans up tbm_surface_internal_query_supported_formats().
There are some missed error routines.

Change-Id: I010e301d358849d8f48242b6a3db39a39a3f09c5
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>

tbm_surface_internal: clean up tbm_surface_internal_create_with_bos()

This patch cleans up tbm_surface_internal_create_with_bos().
There are some missed error routines.

Change-Id: Iec22737915e6d3b01ea8f7a9346f3cb812aec5f5
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>

tbm_surface_internal: clean up tbm_surface_internal_create_with_flags()

This patch cleans up tbm_surface_internal_create_with_flags().
- Adds missed error logs, routines.
- Adds _tbm_surface_internal_query_plane_data() error routine.
- Adds tbm_bo_alloc() error routine.

Change-Id: Iaaead4755e9aa3213e88af92b1e971adb17c0a09
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>

packaging: version up to 2.0.6

Change-Id: Ife9b1bd92c0630f3317aaa903f9561074ff997c1

applied ui_fw uid,gid in tbm-drm-auth.service

Change-Id: Ieefb55c191fd4bed6c5cae885c2a5b0cef35c40f

unlock bo untill lock_cnt is zero when free bo

Change-Id: If5dc9b0450974484d9b4bc8155c7740f682ecd40

unlock mutex of surface when tbm_bo_map

Change-Id: I7fa6078d1d39db8cd2afad77093b0cfb22aab256

tbm_surface_internal: clean up tbm_surface_internal_is_valid()

This patch cleans up tbm_surface_internal_is_valid().
It is enough to use LIST_FOR_EACH_ENTRY because there is no deletion routine.

Change-Id: I58e63e9f3de99ec3195df19385d3a675fb3a32f3
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>

don't call reset callback when queue doesn't have attached buffer in flush

Change-Id: I2bccf23fc7b9569ea2e7aee26e06d0e73b0bd993

do not increase bo cnt of bufmgr if bo is resued (import, import_fd)

Change-Id: I16b90bcfda2187959ad5f5275e167d0b9a45a06e

get the size from surf->bos

Change-Id: I4c6223d2bd522ae5e720629dbdc20ad6f5ef651c

fixed deadlock issues

Change-Id: I1023f64872947a43b1f02a6069cec3d9e1b2e307

add api tbm_surface_queue_set_size

Change-Id: I7c0604c955d6e477ba65ab1d042f6a7276db5c2f

don't use tbm api in bufmgr_debug_show

if bufmgr debug show is called in import/fd/alloc, process enter deadlock

Change-Id: I2473cb2537c131cab3472b8be0c716a33611ed5a

package version up to 2.0.5

Change-Id: I20696793d50f3c180f60c559b11d1508f6287da9

checked roundtrip return error

Change-Id: I8788d783cf8878396de063ac4158ea16736520ee

include config.h in all c file and fixed dlog level

Change-Id: I38fcd99e965afc36ede781b6a6250e8197673221

use MONOTONIC clock to get time

Change-Id: Ibd1ccb92ec42079c035caf744def690b58de8b88

print the time of tbm_bufmgr_init

default disabled

Change-Id: I4c18fda4c2c50a4fb0cddb3cd4bab9702ef1d067

change the DBG to TBM_DBG

Change-Id: Ie3dd8bb328ca93df986c435357b3f282b713ae94

Revert "put gtest framework source code"

This reverts commit 2be3334833c3fc56ffb9d48ccb2116a48748b93e.

Change-Id: I53305df736e639394826b6df23477ff49140d64a

fixed export_fd error return value

Change-Id: I08d8061193dd88c7fe37fe0cf62d6d92f8b06db0

fix error return value

Change-Id: Ic59614bb26531f6cb1cdd16f83ec59a589c0e79f

Fix access null pointer

Change-Id: I1516ba4c5fb735116b7b60604fc3c0f3c5022fa9

package version up to 2.0.4

Change-Id: If40a6ffdd10caa053c30cfa7919ef4ac4519fdb5

fix deadlock problem in queue_reset

Change-Id: I925902b3ad82a5d718d7dd0f1c6c50ec66e8ab82

unlock global mutex lock of surface_queue when call alloc cb in default queue

Change-Id: I41d59af74d23fc7c017e46cf61d0961b67f5a58a

put gtest framework source code

I suppressed all warnings by using a flag -w.
This is a temporary solution. Warnings occur because
we build project with g++. In C++ we need to use explicit
types conversion. I will fix warnings in unit test sources,
in next commit. But I can't fix warnings in libtbm sources.

Change-Id: Ia92f76fce72210822c0479f3cbf2b4ce6b4373cc
Signed-off-by: Konstantin Drabeniuk <k.drabeniuk@samsung.com>

increase the size for the null termination

Change-Id: I7e2b00c41e65094cb0951e2aa636ab648c62d829

Fix deadlock problem

Change-Id: I64345d042d399586f85b8e29a4675b22467ea008

package version up to 2.0.3

Change-Id: I6fb1b26a44c03ce40be485d6c32ce3761769dae9

init surface_queue list in bufmgr

Change-Id: I288b7d66a628994c8bb16a050066cf75bdea98b8

return error when enqueue surface but queue is empty

Change-Id: Ic82f7abf2bf3d713769072b6d4265c92b48565d5

fix coding rule

Change-Id: I7cf7fabd2f88e108f8164028e50f0865ca3aab24

enforce the thread safety of the tbm_surface_queue

Change-Id: If52f7ca3d8cde09afa39d1e0ac40323051eade78

enforce the thread safety of the tbm_surface

Change-Id: I08b1a98ef2e49382d709632a6a9beb575fc21fa4

obey the coding rule

Change-Id: Idbf678141871dc9fabc0389fa1f6a7847599800b

enforce the thread safety of the tbm_bufmgr

Change-Id: Ib7ebf70046ec2f7b0b1caa341d1c78c9e340e180

Remove doxygen warning

Change-Id: I4f6ece8b0dbcfc50db7d847975ca5c3f65a534f6

package version up to 2.0.2

Change-Id: If838c35915e80c6e328600196d4790fe290e1c6a

check null before access import/export(fd) backend interface

Change-Id: I2d2b8a74b1713bfbf9a982436eed54e485dcc263

[libtbm] Move unit tests from ws-testcase and change test framework

1) change unit testing framework (Check to gtest);
2) move unit test from the ws-testcase project to the project libtbm;

Change-Id: Ibad976c45030f02799c211b22db3e65534817233
Signed-off-by: Konstantin Drabeniuk <k.drabeniuk@samsung.com>

Add function add/remove dequeue callback in queue

Change-Id: Ic4dc54e72a7a8917df1577a0d74243baa2cd9dd4

Check gBufMgr in _tbm_bo_valid()

Change-Id: I6e6d44c45ebcd4bf91d5e53de52d1c2376a3d99a

package version up to 2.0.1

Change-Id: Idab7eb5bff56b99dc25b1a59ae20d360434e704c

Add check condition for queue

Change-Id: Iee58d24745cd2aa20d0d065a16192ecfaaddb53f

add the api functions (dump_all and queue_dump) for the debugging

Change-Id: If65bf485517fa0f9e5b0aa5d9546ce3ec2b18513
Signed-off-by: Roman Marchenko <r.marchenko@samsung.com>

tbm_sync: API refactoring

Handles to the sync objects are replaced with file descriptor to remove
overhead of import/export.

Change-Id: I1703162b928b405a583930bc85f8c3b0d2a8426b
Signed-off-by: Taekyun Kim <tkq.kim@samsung.com>

Apply tizen coding rule

Change-Id: I83f03009c34b676b81c4e986751cfd331477e3c7

systemd: support for the lazy mount of user data

Change-Id: I23c200e35bc622e291948f3c19a9d49cb7f3aa83

tbm_sync: bug fix tbm_sync_fence_import

save old fd to new fd in tbm_sync_fence

Change-Id: I219340be9baec80b362140a50a49c4fa27fce4c4
Signed-off-by: deasungkim <deasung.kim@samsung.com>

tbm_sync: tbm_sync_timeline/fence_import use dup

for same look with export funcs
and use CLOSE_EXEC flag

Change-Id: I3d41ba20a621f544c55d85c5415ee1262cf75f66
Signed-off-by: deasungkim <deasung.kim@samsung.com>

tbm_sync: bug fix timeline create and fence create

set timeline fd in timeline structure
fix fence create structure(char pointer to char array[32])

Change-Id: I4d6677543ae0d857bd6f60e77cbbbe4ecef17d62
Signed-off-by: deasungkim <deasung.kim@samsung.com>

tbm_sync: implement tbm_sync

added new API: tbm_sync_fence_export, tbm_sync_fence_import
this APIs need to gpu driver
   tbm_sync_timeline_export
        this APIs maybe need to wayland-tbm
not implement: tbm_sync_fence_get_count_val, tbm_sync_timeline_get_cur_count
I don't know how to get that count from kernel
and if manage in one process(by member variable) I think it is meanless(can't share other processes)

Change-Id: Iea1b21fcacdbdca2061fa12ca589c8f5fc40d38b
Signed-off-by: deasungkim <deasung.kim@samsung.com>

add tbm_sync to support for the timeline and the fence

There are two resources to manage the synchronization.
two handle.. tbm_sync_timeline_h, tbm_sync_fence_h

Change-Id: I680ce38370164d17c7b487400b9e29f1f5cac3f6

Apply tizen coding rule

Change-Id: Ib272a11dfdd15dc698c8e9bfeb5ff4e37ef6faa5

add tbm_surface_internal_set_debug_data function for debugging

debug data will be print in tbm_bufmgr_debug_show

Change-Id: If351c9e987eb5a397707aae5cd94f79502daf89d