For testing GPU recovery of gl renderer

Signed-off-by: Trigger Huang <Trigger.Huang@amd.com>

Wei Zhao authored 2 weeks ago and Trigger Huang committed 1 week ago

During GPU recovery process, scanout fb will be destroied by
eglDestroySurface(). Explicitly set them to  NULL to prevent these
invalid fbs from being further processed.

Signed-off-by: Wei Zhao <Wei.Zhao@amd.com>
Signed-off-by: Trigger Huang <Trigger.Huang@amd.com>

Restore all the dma buffers in GPU recovery, then the applications
which created dam buffers will not be terminated.

Signed-off-by: Trigger Huang <Trigger.Huang@amd.com>

Save all the DMA buffers for restore.

Signed-off-by: Trigger Huang <Trigger.Huang@amd.com>

Mark all the SHM buffer as dirty to avoid garbage caused by original
damage information in GPU recovery

Signed-off-by: Trigger Huang <Trigger.Huang@amd.com>

Call gl_renderer_attach_shm() to recreate all the SHM buffer states
in the new EGL context

Signed-off-by: Trigger Huang <Trigger.Huang@amd.com>

Save all SHM buffers properly so that they can be restored during
GPU recovery

Signed-off-by: Trigger Huang <Trigger.Huang@amd.com>

For each frame, if the ERROR_RENDERER_LOST error occurs, we destroy
the context (which has already become invalid due to a GPU reset) and
all associated resources, then recreate them to ensure validity within
the new context.

Signed-off-by: Trigger Huang <Trigger.Huang@amd.com>

Return RENDERER_LOST if GPU reset happened in function
gl_renderer_repaint_output() for further process

Signed-off-by: Trigger Huang <Trigger.Huang@amd.com>

Supports returning an error code during the repaint process of renderer

Signed-off-by: Trigger Huang <Trigger.Huang@amd.com>

When reinitializing the renderer, skip non-GL steps in
gl_renderer_destroy() and gl_renderer_display_create().
All the GL & EGL related resources will be recreated

Signed-off-by: Trigger Huang <Trigger.Huang@amd.com>

Split the renderer init/destroy into two parts: one to init/destroy
(E)GL, and one to init/destroy the renderer state around it.
Introduce the following two new helper functions to centralize the
handling of GL-related operations
        - gl_renderer_init_gl_egl_context
        - gl_renderer_destroy_gl_egl_context

Signed-off-by: Trigger Huang <Trigger.Huang@amd.com>

Tell the gl renderer if the compositor is reinitializing the renderer.

Signed-off-by: Trigger Huang <Trigger.Huang@amd.com>

Import API glGetGraphicsResetStatusEXT to get GPU reset status if
supported

Signed-off-by: Trigger Huang <Trigger.Huang@amd.com>

Enable robust context when creating the EGL context, this will make
sure that weston will not be killed by the GL driver after a GPU
reset has happened.

Signed-off-by: Trigger Huang <Trigger.Huang@amd.com>

Setup a custom assertion handler that increases an assertion counter
instead of aborting on failure. This allows to run all the sub tests
defined for a test and to correctly report which sub test failed.

Signed-off-by: Loïc Molinari <loic.molinari@collabora.com>

NDEBUG doesn't need to be catched anymore now that the test suite
don't use libc's abort() anymore.

Signed-off-by: Loïc Molinari <loic.molinari@collabora.com>

This framework isn't used anymore.

Signed-off-by: Loïc Molinari <loic.molinari@collabora.com>

Make the NULL assertions more explicit.

weston_assert_ptr_is_null() -> weston_assert_ptr_null()
weston_assert_ptr()         -> weston_assert_ptr_not_null()

Signed-off-by: Loïc Molinari <loic.molinari@collabora.com>

This commit gets rid of libc's abort() usage in the test suite using
test asserts instead.

Asserts run in the server as plugins aren't converted because they are
shared between server and client.

Co-authored-by: Daniel Stone <daniels@collabora.com>
Signed-off-by: Loïc Molinari <loic.molinari@collabora.com>

Signed-off-by: Loïc Molinari <loic.molinari@collabora.com>

Signed-off-by: Loïc Molinari <loic.molinari@collabora.com>

It's much more common to have bits defined as enums than having a
position, so let's just directly take a bit as argument to the bit
asserts. For the few cases where only the position is available, it's
easy to get the right bit using a shift.

The is_pow2_64() helper functions have been added in order to validate
the bit passed as argument and prevent programming errors.

Signed-off-by: Loïc Molinari <loic.molinari@collabora.com>

This commit introduces various Weston based asserts for the testing
framework. The goal is to ease test writing and improve readability.

Using custom asserts instead of relying on libc's assert() will also
help providing NDEBUG builds. This is currently not possible because
setting NDEBUG would disable all the asserts in the tests.

Signed-off-by: Loïc Molinari <loic.molinari@collabora.com>

Robert Mader authored 2 weeks ago and Leandro Ribeiro committed 2 weeks ago

For a list of changes that got squashed into this commit see
https://gitlab.freedesktop.org/rmader/weston/-/commits/color-management-protocol-wp-v1-bkp



Co-authored-by: Leandro Ribeiro <leandro.ribeiro@collabora.com>
Signed-off-by: Robert Mader <robert.mader@collabora.com>
Signed-off-by: Leandro Ribeiro <leandro.ribeiro@collabora.com>

LI Qingwu authored 3 weeks ago and Michael Olbrich committed 3 weeks ago

re-aligned the ivi_layout_interface structure in ivi-layout.c.
purely a formatting update, no functional changes were made.

Signed-off-by: LI Qingwu <Qing-wu.Li@leica-geosystems.com.cn>

LI Qingwu authored 3 weeks ago and Michael Olbrich committed 3 weeks ago

Implement new ivi_layout_interface method and signal to forward
desktop client ping timeout events for unresponsive client handling.

Signed-off-by: LI Qingwu <Qing-wu.Li@leica-geosystems.com.cn>

Théo Maillart authored 3 weeks ago and Marius Vlad committed 3 weeks ago

A window freeze can occur in specific use cases like when using a kiosk
shell. It scales surfaces to fit the compositor size and the transformation
to buffer space can, in some cases, round the size one pixel higher than
the texture size, making glTexSubImage2D() refuse the size argument and
generate a GL_INVALID_VALUE without updating the texture.
This commit ensures the GL renderer doesn't exceed the texture size.

Here is how the issue was reproduced:
	$ weston --renderer=gl --backend=wayland --width=1920 --height=1080 --shell=kiosk
	$ gst-launch-1.0 videotestsrc ! video/x-raw,height=590,width=500 ! waylandsink display=wayland-1

Signed-off-by: Théo Maillart <tmaillart@freebox.fr>

Derek Foreman authored 1 month ago and Marius Vlad committed 3 weeks ago

When in VRR mode, repainting without a wait might happen instantly,
so do it.

This is likely not the best long term solution, but is ok for now
while weston's vrr support is experimental.

Signed-off-by: Derek Foreman <derek.foreman@collabora.com>

Derek Foreman authored 1 month ago and Marius Vlad committed 3 weeks ago

Allow weston.ini to enable game VRR.

Signed-off-by: Derek Foreman <derek.foreman@collabora.com>

Derek Foreman authored 1 month ago and Marius Vlad committed 3 weeks ago

Add functions to query output VRR capabilities and set the current VRR
mode.

Signed-off-by: Derek Foreman <derek.foreman@collabora.com>

Derek Foreman authored 1 month ago and Marius Vlad committed 3 weeks ago

Set up the VRR_ENABLED instead of always clearing it to 0.

Signed-off-by: Derek Foreman <derek.foreman@collabora.com>

Derek Foreman authored 1 month ago and Marius Vlad committed 3 weeks ago

Query the vrr_capable property for heads, and expose it to frontends.

Currently we only support game mode VRR, but expose VRR capabilities as
an enum in anticipation of supporting QMS VRR in the future.

Signed-off-by: Derek Foreman <derek.foreman@collabora.com>

Robert Mader authored 3 weeks ago and Marius Vlad committed 3 weeks ago

Disable tests as they require wayland-scanner 1.23.

Signed-off-by: Robert Mader <robert.mader@collabora.com>

Loïc Molinari authored 3 months ago and Daniel Stone committed 3 weeks ago

Signed-off-by: Loïc Molinari <loic.molinari@collabora.com>

Loïc Molinari authored 3 months ago and Daniel Stone committed 3 weeks ago

Signed-off-by: Loïc Molinari <loic.molinari@collabora.com>

Loïc Molinari authored 3 months ago and Daniel Stone committed 3 weeks ago

This is for consistency reasons too.

Signed-off-by: Loïc Molinari <loic.molinari@collabora.com>

Loïc Molinari authored 3 months ago and Daniel Stone committed 3 weeks ago

This is mostly to improve consistency.

Signed-off-by: Loïc Molinari <loic.molinari@collabora.com>

Loïc Molinari authored 3 months ago and Daniel Stone committed 3 weeks ago

Formats using SHADER_VARIANT_Y_XUXV can now use SHADER_VARIANT_Y_UV
instead with swizzle variations in order to simplify the fragment
shader.

Signed-off-by: Loïc Molinari <loic.molinari@collabora.com>

Loïc Molinari authored 3 months ago and Daniel Stone committed 3 weeks ago

The sampler type, which was initially meant for the GL renderer, is
currently used to derive the color model of pixel formats. This commit
gets rid of this legacy field and replaces it with an explicit one
indicating the color model.

Note that it also fixes the issue that formats wouldn't be considered
YUV if ENABLE_EGL isn't defined because the sampler type field in that
particular case would always be 0.

Signed-off-by: Loïc Molinari <loic.molinari@collabora.com>