[waylandws_client]: Added Multithread support #4
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
create wl_event_queue in eglCreateWindowSurface() to make it thread specific Authored-by: Ajay Kumar Sahoo <[email protected]> Re-based the code changes for wayland-wsegl v3.1.1. Signed-off-by: Rajendraprasad K J <[email protected]>
- drawable->surface->frame_sync & drawable->callback can exists even if drawable->window points to NULL For eg- in case of resize. For the resize case if we avoid destroying the callback then done events gets dispatched in freed queue at later point of time. - Also for proper cleanup reallocate a new surface in case of resize while setting up new window drawable and free up the old one from inside WSEGLc_DeleteDrawable. Signed-off-by: Sunidhi Dixit <[email protected]>
- For throtling the frames use a generic callback holder member instead of drawable->callback and drawable->surface->frame_sync to give a chance for the compositor to handle the commit and send a release event before checking for a free wl_buffer. Signed-off-by: Sunidhi Dixit <[email protected]>
There is no usecase in which aggressive sync mode is useful hence removing it. Also with this change there is no point in keeping drawable->callback. Signed-off-by: Sunidhi Dixit <[email protected]>
Removed the redundant free_buffer_queue, free_buffer and free_buffer_unused. Added a new member in struct kms_buffer i.e. is_free which is true if the buffer is free to be used or false if the buffer is already been used as back buffer. Signed-off-by: Sunidhi Dixit <[email protected]>
sdikshit786
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Multithreaded usage is not completely possible because rcar EGL uses one wayland queue. Wayland queue should be per EGL surface.
In earlier implementation, there was a single global wayland event queue created in wsegl backend during eglInitialize() . In the current implementation we moved the event queue creation to eglCreateWindowSurface() and each window drawable has its own wayland event queue.