jinbong, Lee [Mon, 18 Sep 2023 06:37:32 +0000 (15:37 +0900)]
Package version up to 1.10.8
Change-Id: Ibc178293f788b1a372ac9f4e7288d7d2aa7b7b5d
jinbong, Lee [Mon, 18 Sep 2023 06:19:41 +0000 (15:19 +0900)]
Enqueued tbm_surface is needed to unref when surface's finalize is called
- If main thread called __tpl_wl_egl_surface_fini(),
then sometimes enqueued buffer can be remained before acquire.
so in that case tbm_surface must be unreferenced.
Change-Id: I3482feca808deec9c238d5ca32ee46c9ebeec401
jinbong, Lee [Mon, 18 Sep 2023 06:13:55 +0000 (15:13 +0900)]
protect wl_egl_buffer->wl_buffer with mutex_lock
- wl_egl_buffer->wl_buffer is must protected when call wl_egl_display && wl_egl_display->wl_tbm_client
Change-Id: I223f1521b1f412914194226ce69a996fc4ef91db
jinbong, Lee [Tue, 29 Aug 2023 11:06:33 +0000 (20:06 +0900)]
Package version up to 1.10.7
Change-Id: I7fc5a01433d47123fca4a008bbfec74dbc9d9c01
jinbong, Lee [Tue, 29 Aug 2023 10:56:54 +0000 (19:56 +0900)]
Remove double free situation
- if (source->fd_type == FD_TYPE_FENCE && source->type == SOURCE_TYPE_DISPOSABLE)
then g_source_remove_unix_fd, g_source_destroy, g_source_unref is called twice for source->gsource.
- it can cause error like that free already freed memory or free with invalid pointer (garbage value)
Change-Id: I44e3b1d50b0710d9cd2ddcf8e8c8f4215acff344
Joonbum Ko [Mon, 24 Jul 2023 07:15:58 +0000 (16:15 +0900)]
Package version up to 1.10.6
Change-Id: Ie0023ea5aaf82238fb8c50fec70e6c4bc676b642
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Tue, 18 Jul 2023 10:40:25 +0000 (19:40 +0900)]
wl_vk: untrack the swapchain_buffers when destroy_swapchain called
Change-Id: I72ed5fdf2fec9f00fa25e5bc87ea26501d1042f9
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Xuelian Bai [Mon, 24 Jul 2023 03:15:17 +0000 (11:15 +0800)]
Fix a typo
This typo will cause frozen for all vulkan app, since display dispatch
didn't work
Change-Id: Ibd3d753e35c0825bb7a6bc008baa4ace2fb076ff
Signed-off-by: Xuelian Bai <xuelian.bai@samsung.com>
Joonbum Ko [Mon, 17 Jul 2023 07:37:55 +0000 (16:37 +0900)]
Package version up to 1.10.5
Change-Id: I1a3e22b04d5b73a5ba2f4dceb80f8202e5e6af93
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Mon, 17 Jul 2023 07:35:16 +0000 (16:35 +0900)]
utils_gthread: do not make data into NULL before the source is destroyed
- Even if thread call the g_source_destroy & unref it, it may not be destroyed
immediately. (waiting for dispatch in g_main_context)
- source->data will be changed to NULL only when it destroyed perfectly.
Change-Id: Iecf8ce7599a0ad92346fe7dc802bfcebbb7cd25c
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Tue, 11 Jul 2023 06:47:59 +0000 (15:47 +0900)]
Package version up to 1.10.4
Change-Id: I480c76e12c88d8e9d972e7903d1fbaae6b39a72c
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Tue, 11 Jul 2023 06:33:48 +0000 (15:33 +0900)]
wl_egl: close fence_fd when gsource finalized
- Even though when main thread have called tpl_gsource_destroy for
fence_waiting_source, which has not yet signaled, gsource may not be
completely destroyed because it might be in dispatch queue in g_main_context.
- In this case, if the fence_waiting_source is dispatched, it can try
to commit with the invalid wl_egl_buffer that has already been freed.
- This commit makes to prevent operating with an invalid wl_egl_buffer
and to close fence_fd properly.
Change-Id: Ibce274b062bf8a6354a76464fca8f1e10b77c7b8
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Mon, 10 Jul 2023 11:06:47 +0000 (20:06 +0900)]
utils_gthread: add enum type to distinguish the fd type
Change-Id: Idca314bb4cb28cae973c56b9bc99a32195924e47
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Mon, 10 Jul 2023 08:24:02 +0000 (17:24 +0900)]
wl_vk: change the event message processing
Change-Id: I6d3b6c7aa7b5289fb1b91a9c3f9cb563499fabd4
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Fri, 7 Jul 2023 04:09:29 +0000 (13:09 +0900)]
log: add & apply TPL_LOG_D instead of TPL_DEBUG
- TPL_LOG_D can print the tag manually.
- The prefix of almost all logs is modified to print wl_egl_surface
to distinguish each wl_egl_surface.
Change-Id: I68a5b450693ed0d9422a9ba72e119f56235d7a32
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Thu, 15 Jun 2023 07:02:37 +0000 (16:02 +0900)]
wl_egl: remove checking activated when dequeue buffer
- Actual buffer will be attached when calling tbm_surface_queue_can_dequeue().
So calling wayland_tbm_client_queue_check_activate() at dequeue_buffer
is not appropriate.
Change-Id: Ie5500aa50f396e4a76160c703932d6c805e93d3d
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Fri, 2 Jun 2023 04:21:49 +0000 (13:21 +0900)]
Package version up to 1.10.3
Change-Id: I4f29af23350c5f5c8ffcea997021256f579f93b1
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Fri, 2 Jun 2023 04:15:03 +0000 (13:15 +0900)]
wl_egl/vk: use new tdm API to handle pending events.
refer to
https://review.tizen.org/gerrit/#/c/platform/core/uifw/libtdm/+/293679/
client: Add new API to handle pending events
fb76bf4
Change-Id: I036bc7b9f704b9f1bd250a6c651d5c90a9d94a65
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Fri, 14 Apr 2023 10:00:36 +0000 (19:00 +0900)]
wl_egl: Initialize vblank_done flag when dequeue timeout occured
- Rarely, even if client cannot receive vblank event from tdm server,
timeout may occur in can_dequeue.
- In this case, if wl_egl_surface->vblank_done flag
is not initialized to TPL_TRUE, problem situation may occur
waiting vblank without calling tdm_client_vblank_wait
after force_flush(can_dequeue_timeout).
Change-Id: If3f8eee13b5ae91a3728f189f53aa25720696b12
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Xuelian Bai [Thu, 23 Mar 2023 06:07:12 +0000 (06:07 +0000)]
Merge "Fix memory leak" into tizen
Xuelian Bai [Tue, 21 Mar 2023 06:56:27 +0000 (14:56 +0800)]
Fix memory leak
Wait until tbm_queue is destroyed, on pixel phone swapchain is destroyed
before tbm_queue is destroyed, then tbm_queue is never destroyed, then
cause memory leak
Change-Id: I897be13684f0e1f901aad751111c1e469414e178
Jinbong Lee [Mon, 20 Mar 2023 07:53:38 +0000 (07:53 +0000)]
Merge changes Ibac91ed8,I7e6a6891,I779718fd into tizen
* changes:
Package version up to 1.10.2
wl_egl: Implement the backend function of fence_sync_is_available
Add new API checks if fence sync is available
jinbong, Lee [Fri, 17 Mar 2023 10:27:02 +0000 (19:27 +0900)]
wl_egl : remive wl_egl_buffer in vblank's waiting buffers when wl_egl_buffer is freed
- Bug fix
. if __cb_wl_egl_buffer_free is called then some wl_egl_buffer can be in waiting buffers for vblank.
so it must be removed it before wl_egl_buffer is freed.
Change-Id: Ia08a7ecc6dcfd8f9d63e644b7cb733740672666b
Joonbum Ko [Fri, 17 Mar 2023 10:45:17 +0000 (19:45 +0900)]
Package version up to 1.10.2
Change-Id: Ibac91ed8fd994c1882d9433bc4dbedfd20cfdf21
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Fri, 27 Jan 2023 11:16:44 +0000 (20:16 +0900)]
wl_egl: Implement the backend function of fence_sync_is_available
Change-Id: I7e6a6891ff12b6869e66cf3a2d5f64098b04cb94
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Fri, 27 Jan 2023 11:08:05 +0000 (20:08 +0900)]
Add new API checks if fence sync is available
/**
* Check the surface can support fence sync mechanism.
*
* It is recommended that checking fence sync is available
* for every frame because the results may change depending on
* frontbuffer rendering is activated or not.
*
* @param surface surface to check fence sync is available.
* @return TPL_TRUE if tpl_surface can support it.
*/
tpl_bool_t
tpl_surface_fence_sync_is_available(tpl_surface_t *surface);
- This API helps DDK to determine whether to deliver the acquire_fence
to signal the render complete when call the surface_enqueue.
- In backend where waiting fence is not implemented,
the result of fixed to TPL_FALSE will be returned.
- The result from the backend with the waiting fence implementation
depends on whether the frontbuffer rendering is activated.
Change-Id: I779718fdc7e8efc7890e17b0d4df4d81974a7907
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Thu, 12 Jan 2023 08:15:40 +0000 (17:15 +0900)]
Package version up to 1.10.1
Change-Id: I8018c7726952577eda15361b938c722e09f7b13c
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Thu, 12 Jan 2023 08:15:20 +0000 (17:15 +0900)]
wayland-egl-tizen: Add description of set_frontbuffer_mode
Change-Id: Ic9e966c1b7b7f4064996765cb5e4f63cd55f813a
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Thu, 12 Jan 2023 07:16:50 +0000 (16:16 +0900)]
wl_egl: support for frontbuffer mode set by surface
- Patch for supporting this API.
wl_egl_window_tizen_set_frontbuffer_mode()
AS-IS:
- If the application, which wants to use frontbuffer rendering,
sets the frontbuffer mode using setenv(), EGL driver checks whether
frontbuffer mode is set using getenv() and if it true calls
tpl_surface_set_frontbuffer_mode().
PROBLEMS:
- The operation using setenv()/getenv() is not thread safe.
- Using env value to set frontbuffer mode cannot manage on/off in runtime.
- Using env value to set frontbuffer mode cannot set by surface.
TO-BE:
- Application would be better to use this API
wl_egl_window_tizen_set_frontbuffer_mode()
- This API makes supports the setting of frontbuffer mode
to the desired surface(window).
- This API gurantee thread safety using tpl_surface object lock.
- Using this API can help application to turn on/off
the frontbuffer mode in runtime.
Change-Id: I608309869dcb9d0bd0ba42c7e54afee6da1b5e04
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Wed, 4 Jan 2023 07:49:41 +0000 (16:49 +0900)]
wl_egl_thread: prepare for the failure of pause_in_idle
- The calling tpl_gthread_pause_in_idle() move from force_flush()
to outside.
- Add locking wl_event_mutex after trying tpl_gthread_pause_in_idle.
- Locking wl_event_mutex is a secondary means of preparing for
the failure of tpl_gthread_pause_in_idle().
If tpl_gthread_pause_in_idle()is successful,
locking wl_event_mutex does not affect.
Change-Id: I35132da013f67921c0f6deecc0909118461f3872
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Wed, 4 Jan 2023 07:29:57 +0000 (16:29 +0900)]
utils_gthread: cancel pause when wait_idle failed
- If the tpl_gthread_wait_idle fails with timeout,
it should be unlock pause_mutex immediately.
Change-Id: If33cc3c0b617ca34bb63d110d927e0dd7c022526
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Tue, 6 Dec 2022 08:51:24 +0000 (17:51 +0900)]
wl_egl: Delete calling wait_idle in surface_fini
- tpl_gthread_wait_idle will be called in tpl_gthread_pause_in_idle()
tpl_gthread_pause_in_idle will be called in buffer_clear()
So, calling wait_idle() is duplicated & meaningless operation
Change-Id: I50b2aa2e73f5a22be5860ab3e6be241def2d862f
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Fri, 2 Dec 2022 07:17:32 +0000 (16:17 +0900)]
utils_gthread: Prevent deadlock caused by wait_idle
Deadlock situation: Multi-threaded and multi surfaces for each thread
T2: call pause_in_idle()->wait_idle()
cond_wait() until idle signaled
T1: call wait_idle()
wait for lock idle_mutex
T2: _thread_idle_cb unlock idle_mutex,
set is_idle to TPL_TRUE
and wait to lock pause_mutex
T1: change is_idle to TPL_FALSE
cond_wait()
T2: cannot exit from while loop
still cond_wait()
- is_idle changing should be set TPL_FALSE after cond_wait finished
Change-Id: I44292e486a1b9f686ec28bc0ae10ddedf94a48e3
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Fri, 2 Dec 2022 01:52:44 +0000 (10:52 +0900)]
Package version up to 1.9.14
Change-Id: Id42e31cefc898f35fb9d84a8f949513a77eac515
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Fri, 2 Dec 2022 01:51:23 +0000 (10:51 +0900)]
utils_gthread: Add missed initializing paused flag
This missed line can cause deadlock issue.
Change-Id: I8d098e79010133dec4d7cc2cec071213cd02ec1a
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
jinbong, Lee [Wed, 30 Nov 2022 06:59:49 +0000 (15:59 +0900)]
Package version up to 1.9.13
Change-Id: I73f1638107fb67870bbc782cf970de7e35e2ff47
jinbong, Lee [Wed, 30 Nov 2022 06:47:33 +0000 (15:47 +0900)]
remove vblank's waiting_buffers in _tbm_queue_force_flush for prevent committing flushed tbm_surface
Change-Id: Ic91aa678659b12c9ce80bce87504deef2e4299af
Joonbum Ko [Wed, 30 Nov 2022 04:47:29 +0000 (13:47 +0900)]
utils_gthread: Change to use g_cond_wait_until to wait idle
- In some cases, signal was sent by idle_callback but
there is a problem that this signal is lost and g_cond_wait
cannot be awakened.
- Above case can cause deadlock problem.
- So set 200ms timeout ms with using g_cond_wait_until instead of
g_cond_wait.
Change-Id: Ib0a549616f74a9fb39e78f7b37b1b8b09e16031d
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Fri, 18 Nov 2022 08:40:21 +0000 (17:40 +0900)]
Package version up to 1.9.12
Change-Id: I24145794da2f4bd1505ee64518a427092c6472de
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Fri, 18 Nov 2022 08:30:17 +0000 (17:30 +0900)]
wl_egl: Set last_enq_buffer before tbm enqueue
- Since the operation from queue_enqueue to surface_commit
can be done at a time slice in thread, last_enq_buffer
must be set to enqueued tbm_surface before tbm queue enqueue
to initialize with NULL when do surface_commit.
Change-Id: I8b753abf44f92d4de4dab43185a7a29cb30c59fd
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Fri, 18 Nov 2022 08:21:49 +0000 (17:21 +0900)]
wl_egl: Use buffers list instead of buffers array
- If surface use array to track buffers,
index management is difficult and difficult
to assign a suitable size.
- The use of list can reduce some of the potential problems
that can occur from complex management.
Change-Id: I9a4574f542cde0ad8d32fed575d03e1a2c861b0b
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Thu, 17 Nov 2022 08:33:11 +0000 (17:33 +0900)]
wl_egl: Apply pause and wait_idle at buffer clear
AS-IS
Using wl_event_mutex lock to pause thread.
Using wl_event_mutex is not good way to pause thread
because locking wl_event_mutex can effect other wayland threads.
TO-BE
Using tpl_gthread_pause_in_idle is better way than before.
Because locking on idle allows thread to handle other tasks.
Change-Id: I47a1b82a91ede648ceb8a8a1e7967fa56950ba00
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Wed, 16 Nov 2022 08:21:08 +0000 (17:21 +0900)]
utils_gthread: Add new functions to pause/continue thread
tpl_gthread_wait_idle
- Create a new idle source and attach it to thread context.
This function will be blocked until idle callback is called.
If the thread has some worksto be handled, wait for
the thread to finish it.
tpl_gthread_pause_in_idle
- Pause the thread when idle callback is called.
tpl_gthread_continue
- Continue the paused thread.
Change-Id: I5a2aaf3b8b9579306246234d252b643b6cc0bf3e
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Thu, 10 Nov 2022 14:57:16 +0000 (23:57 +0900)]
Package version up to 1.9.11
Change-Id: I03d05215651ab24596e24a9a8d011cd7a4940b2f
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Thu, 10 Nov 2022 14:45:49 +0000 (23:45 +0900)]
wl_egl: Print additional INFO logs at buffer_clear
Change-Id: I5cbd4b6b96bc149407969834d4147aeebb3cf2e2
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Thu, 10 Nov 2022 10:36:33 +0000 (19:36 +0900)]
wl_egl: Set timeout for buffer_clear to 500ms
- Some system condition problems may cause signal after 200ms.
Change-Id: I40bdbf7cc862e0026228b85644eda60f15b86dec
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Thu, 27 Oct 2022 23:35:20 +0000 (08:35 +0900)]
Package version up to 1.9.10
Change-Id: I43791c514fa7abd274b238d7cdfbc3489e8278eb
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Thu, 27 Oct 2022 15:02:58 +0000 (00:02 +0900)]
wl_egl: Add defense code for fake signal in buffer_clear
tpl_gcond_timed_wait is changed to be called within the while loop.
It will repeat the process of checking status even if it wakes up
due to Fake Signal.
Change-Id: Ia66810da64ba2830c166f01dcb2f2f4615fc8e4f
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Thu, 27 Oct 2022 11:04:52 +0000 (20:04 +0900)]
wl_vk: Add missed flag setting when surface_init
Change-Id: I45e7d3135a6780a1f08b10fe8913fc9ffe71af7f
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
jinbong, Lee [Wed, 19 Oct 2022 09:58:06 +0000 (18:58 +0900)]
Remove unused int64, int32 hash key for fixing Svace warning
Change-Id: I9bb85b566db975860e4243c39683f40e60601aed
Joonbum Ko [Wed, 5 Oct 2022 01:58:46 +0000 (10:58 +0900)]
Package version up to 1.9.9
Change-Id: I6c7059dae6306e235be00ac9ec8bd1f7d3112136
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Wed, 5 Oct 2022 01:57:23 +0000 (10:57 +0900)]
Add null checking before calling tpl_gsource_destroy.
Change-Id: I431f21b80215abeafc5ab6daa45778f617cc661a
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Wed, 5 Oct 2022 01:49:43 +0000 (10:49 +0900)]
Remove unncessary header
Change-Id: I810ec7c149bbb782d8e04167f66810171226d728
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Fri, 30 Sep 2022 07:35:33 +0000 (16:35 +0900)]
Package version up to 1.9.8
Change-Id: I7a1bda7114b8cf98de0cd8f2f02dfb6b8d2c25fa
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Fri, 30 Sep 2022 07:33:54 +0000 (16:33 +0900)]
Modified to call tpl_gsource_destroy only once.
- tpl_gsource_destroy means 'sending destroy message to thread'.
So it need not to be called in loop.
Change-Id: Ifd5706d738a2385396acaa3723138b2f564de4c2
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Fri, 30 Sep 2022 07:23:50 +0000 (16:23 +0900)]
Changed the order of lock/unlock at buffer_clear.
- The lock/unlock order of buffer->mutex and display->wl_event_mutex
is important. display->mutex must surround buffer->mutex
- Before applying this patch, deadlock issue may be
occured in buffer_clear.
Change-Id: I90d9b36874def4e4c7f0bd36742b3bbd00faab44
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Thu, 29 Sep 2022 12:04:56 +0000 (21:04 +0900)]
Delete wrong g_main_context_unref
- GMainContext is also destroyed when GMainLoop is destroyed.
Therefore, it is the wrong code to do context_unref before loop_unref.
Change-Id: Ib57eab7ba4345565977102abb0df3268d7f42acb
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Thu, 29 Sep 2022 11:45:42 +0000 (20:45 +0900)]
Changed the function name properly.
tpl_cond_timed_wait -> tpl_gcond_timed_wait
Change-Id: Iebe13b6aa7598d652cf3b5968dbc7b2bb96a44e0
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Tue, 27 Sep 2022 11:41:14 +0000 (20:41 +0900)]
Modified the codes related to call tpl_gcond_wait.
- g_cond_wait is sometimes awakened by unknown or stolen signal.
In such cases, unexpected problems may arise. To prevent these problems,
each tpl_gsource has tpl_gmutex and tpl_gcond, and modified to try
tpl_gcond_wait() until gsource_finalized flag becomes true.
- It may be modified with better way.
Change-Id: I1360c0a3888186ba0309fe4d94c5be8e29c6f1b8
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Wed, 28 Sep 2022 11:26:57 +0000 (20:26 +0900)]
Delete g_cond_wait from tpl_gsource_destroy.
- g_cond_wait does not guarantee perfectly.
Therefore, it is recommended that the caller of tpl_gsource_destroy
should call the g_cond_wait to confirm if the destroy is
actually complete.
- https://docs.gtk.org/glib/method.Cond.wait.html
Atomically releases mutex and waits until cond is signalled.
When this function returns, mutex is locked again
and owned by the calling thread.
When using condition variables, it is possible that a spurious
wakeup may occur (ie: g_cond_wait() returns even though g_cond_signal()
was not called). It’s also possible that a stolen wakeup may occur.
This is when g_cond_signal() is called, but another thread acquires
mutex before this thread and modifies the state of the program in such
a way that when g_cond_wait() is able to return,
the expected condition is no longer met.
For this reason, g_cond_wait() must always be used in a loop.
See the documentation for GCond for a complete example.
Change-Id: If3b98b4d79b205d9125558edb75f4b85ef6a3a99
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Thu, 9 Jun 2022 10:13:37 +0000 (19:13 +0900)]
Package version up to 1.9.7
Change-Id: I9a15a0ff3f4accd438af5fe8abd89d71fc1275fa
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Thu, 9 Jun 2022 10:04:11 +0000 (19:04 +0900)]
Fixed to prevent page fault via invalid address.
- If the last_enq_buffer of wl_egl_surface has a wrong pointer address,
page fault may occur inside the tbm_surface_internal_is_valid of libtbm.
- To prevent this problem, it is modified to check in advance from
the list of buffers of wl_egl_surface.
Change-Id: I459b182e9ed435ce93a3a862251869fb9c7829ad
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Thu, 9 Jun 2022 09:43:49 +0000 (18:43 +0900)]
Added internal function to check buffer is validate
- It can be modified flexibly.
- For now, this function can check if given tbm_surface_h
is managed by wl_egl_surface.
Change-Id: Ied59f583666a5f18f15537be6507c83c5277a866
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Jinbong [Mon, 18 Jul 2022 05:53:58 +0000 (14:53 +0900)]
Fix pointer to int cast warning
Change-Id: Id0af40582f900c46745f6c1b8e6864789daeca81
Jinbong [Mon, 18 Jul 2022 05:46:27 +0000 (14:46 +0900)]
Revert "Remove compile warning about casting to pointer from integer of different size"
This reverts commit
b1725298fc58f3e2e63da142bf9687a214f38146.
Jinbong [Mon, 18 Jul 2022 05:14:28 +0000 (14:14 +0900)]
Remove compile warning about casting to pointer from integer of different size
Joonbum Ko [Wed, 8 Jun 2022 10:23:57 +0000 (19:23 +0900)]
Package version up to 1.9.6
Change-Id: I8810e6da7e5c98841c37dee3b6660137b5445b5f
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Wed, 8 Jun 2022 10:23:26 +0000 (19:23 +0900)]
Fix pc.in to follow 64bit build guideline
Change-Id: I32adf0732fab34864cdb7e1cafffdad1b787ac2c
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Tue, 19 Apr 2022 06:04:21 +0000 (15:04 +0900)]
Package version up to 1.9.5
Change-Id: I44d35f42e7a88c8a882e31640ec8783d0496a7ba
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Xuelian Bai [Mon, 11 Apr 2022 07:02:57 +0000 (15:02 +0800)]
Fix issue when reset
Set ref_cnt as 1 before CREATE_QUEUE, or when queue is reused and
ref_cnt is increased, the ref_cnt will be set back to 1.
Set format as default when reset
Change-Id: I0bd90a361d26329cb86393b1eaddf1013af09244
Signed-off-by: Xuelian Bai <xuelian.bai@samsung.com>
Joonbum Ko [Tue, 5 Apr 2022 01:49:15 +0000 (10:49 +0900)]
Package version up to 1.9.4
Change-Id: Ic759425a78f195bd327f44dc3de9f6255a553f7b
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Tue, 5 Apr 2022 01:46:58 +0000 (10:46 +0900)]
fix 64bits build-errors
Change-Id: Ie5a6541b8442806c948c099f69eaa4729292b818
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Tue, 5 Apr 2022 01:42:16 +0000 (10:42 +0900)]
Add missed initializing to trace tpl_surface.
Change-Id: Id023b7633986e6ec59d482f8f77c45fcc7d512e7
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Tue, 22 Feb 2022 08:20:36 +0000 (17:20 +0900)]
Package version up to 1.9.3
Change-Id: I2c1747fc961d1f39166b32f2aa5431db3648f2fc
Joonbum Ko [Tue, 22 Feb 2022 08:20:13 +0000 (17:20 +0900)]
Fix space trim issue at configure.ac
Change-Id: I5c8eaf9bb892af4336658ed3172ed86df353556b
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Tue, 15 Feb 2022 05:33:26 +0000 (14:33 +0900)]
Package version up to 1.9.2
Change-Id: I3c744d64eaa7bfeb2f4dc00fe813e5de09331901
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Tue, 15 Feb 2022 05:06:30 +0000 (14:06 +0900)]
Make clear about use_explicit_sync feature flag.
- if the boolean flag use_explicit_sync is true,
it includes the meaning surface_sync is not null.
- so, it need to be simplified with using only
use_explicit_sync flag.
Change-Id: Icdcc55a76d72b28d9bd38a96de1093ab56edc135
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Wed, 26 Jan 2022 07:04:51 +0000 (16:04 +0900)]
Package version up to 1.9.1
- the minor version for tizen_7.0 is '9'
Change-Id: I21627d64a70f3d9a137ae73106be90d36328ce03
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Wed, 26 Jan 2022 06:26:31 +0000 (15:26 +0900)]
Initialize last_enq_buffer to null when it committed.
- If the last_enq_buffer of wl_egl_surface has been committed,
it is not necessary to have this handle pointer.
Change-Id: I3153b3cc9cb133f1d51321dc7aaa92f57b1e5ed8
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Thu, 20 Jan 2022 08:16:52 +0000 (17:16 +0900)]
Prevents the incorrect signal on gcond of wl_egl_buffer.
As with g_cond_wait() it is possible that a spurious or stolen wakeup
could occur. For that reason, waiting on a condition variable should
always be in a loop, based on an explicitly-checked predicate.
Change-Id: I388e71a48dc91c1636490c856698c4a7f8d3fafd
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Tue, 18 Jan 2022 02:06:30 +0000 (11:06 +0900)]
Package version up to 1.8.23
Change-Id: I39e166a5eba110731a2a3b5e8fa6a14f7b90bdad
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Tue, 18 Jan 2022 06:17:49 +0000 (15:17 +0900)]
Change the handle of last_enq_buffer to tbm_surface_h
- to check validation for the handle of last_enq_buffer.
Change-Id: Ib92b28cd3bc6bfa553fb5de57afd9fc8cfcf0cdc
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Tue, 11 Jan 2022 07:49:33 +0000 (16:49 +0900)]
Package version up to 1.8.22
Change-Id: I0a1ab000f80ab7a3c2aaaab0acfaca6591e95d53
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Fri, 14 Jan 2022 01:51:51 +0000 (10:51 +0900)]
Fixed to use last_enq_buffer instead of last_deq_buffer.
- last_deq_buffer has a risk to be free from thread.
Change-Id: I677704a5b9bbf8a7c405689663a856d83df595f4
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Thu, 13 Jan 2022 08:52:48 +0000 (17:52 +0900)]
Move unref to outside from buffer mutex.
Change-Id: If2c0eb9ee17edf6febc830ec058b274b1eba5431
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Tue, 11 Jan 2022 06:41:03 +0000 (15:41 +0900)]
Call dispatch cb for disposable source when IO error occured.
Change-Id: I8a84a32b5c246c65a9b7a3ab5e5bc0e69afe51ae
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Wed, 5 Jan 2022 07:53:24 +0000 (16:53 +0900)]
Package version up to 1.8.21
Change-Id: I21d36be947889a69c48b7b51c448d95abef6b0c9
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Wed, 5 Jan 2022 07:48:11 +0000 (16:48 +0900)]
Remove finalizer source to before cond_signal
Change-Id: Iaeb1b75ecff8860f10f82a21e8ea2be972e70dad
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Thu, 30 Dec 2021 07:31:34 +0000 (16:31 +0900)]
Package version up to 1.8.20
Change-Id: I27eb1758e92fcafa69fd5203fc85fd7cca1d6fac
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Thu, 30 Dec 2021 06:37:10 +0000 (15:37 +0900)]
Clear all cached buffer when queue_force_flush
Change-Id: Ic232153cfbdea49aea1a9fd862ac81f673f7814c
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Thu, 30 Dec 2021 04:56:12 +0000 (13:56 +0900)]
Add mutex protection when gsource_destroy
Change-Id: I50e841fa3895e6f145249396574e6a8267f1e663
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Thu, 30 Dec 2021 06:19:24 +0000 (15:19 +0900)]
Add log to check when the transform changed
Change-Id: I069fc944356d25845655359cdb8654887b554e8f
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Fri, 24 Dec 2021 01:45:28 +0000 (10:45 +0900)]
Package version up to 1.8.19
Change-Id: I5b0450b14c24e3f8152b46f9482f52a8fce19d71
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Fri, 24 Dec 2021 01:40:41 +0000 (10:40 +0900)]
Initialize last_deq_buffer to NULL when it free.
Problem : DEQ -> ENQ -> ACQ -> VBLANK -> COMMIT -> RELEASE
-> QUEUE_RESET -> last_deq_buffer free
-> try DEQ !!< will be blocked.
Init wl_egl_surface->last_deq_buffer to NULL when it was free
to prevent to access the wrong pointer of last_deq_buffer.
Change-Id: Ic619b6a27a098b3de06c5733cec171538f921165
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Thu, 23 Dec 2021 01:38:38 +0000 (10:38 +0900)]
Package version up to 1.8.18
Change-Id: I61c16ae2ae8798af01bfcf383602f607dce88049
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Thu, 23 Dec 2021 01:41:04 +0000 (10:41 +0900)]
Added mutex for protecting vblank resources.
As-Is :
- vblank resource was protected via surf_mutex.
To-Be :
- It will be protected via its own mutex.
Change-Id: Iff4084a4f8271b8cbe4a7ece308b98915d9641af
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Wed, 22 Dec 2021 08:25:36 +0000 (17:25 +0900)]
Changed to do roundtrip_queue before display_fini.
Change-Id: I998141a3dc63ac944526dbb14876d8d6fa690c02
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Wed, 22 Dec 2021 08:16:12 +0000 (17:16 +0900)]
Remove wl_egl_buffer from vblank list when it freed.
- If the wl_egl_buffer is forcibly free from the buffer_clear(),
the invalid wl_egl_buffer remains in vblank->waiting_buffers.
- This invalid pointer should be removed from the list
when wl_egl_buffer free.
Change-Id: If6b9f58f4160c4426f2b816a22afb9e23b61372a
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Wed, 22 Dec 2021 05:45:09 +0000 (14:45 +0900)]
Modified the new buffer allocation in the RESET situation
As-Is :
- If the tbm_surface_queue_reset occurs more frequently
than VBLANK (16ms), there is a problem that the new tbm_surface
continues to be allocated.
To-Be :
- If there is a dequeue buffer before the RESET occurs,
the new buffer will be allocated after waiting for
dequeued buffer to be commit.
Change-Id: Id256e15e1125e06b362d5b90e7ead7718b6343ad
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>
Joonbum Ko [Tue, 14 Dec 2021 04:46:53 +0000 (13:46 +0900)]
Package version up to 1.8.17
Change-Id: I94cb61d3c1600621d650b4b47ca41758ff8e3d4c
Signed-off-by: Joonbum Ko <joonbum.ko@samsung.com>