Re: [PATCH weston 0/8] Move fade to shell and fix compositor state mess

On Thu, Feb 21, 2013 at 06:35:15PM +0200, Ander Conselvan de Oliveira wrote: > Hi, > > The purpose of this series is to move the fade animation out of core > Weston and improve the compositor->state situation. Currently, when > the compositor goes idle and sends the lock signal, the shell sets up

[PATCH weston] tests: Use&Fix parallel tests

From: Quentin Glidic Also fix automake 1.13 support Signed-off-by: Quentin Glidic --- This patch keeps automake 1.11 compatibility too configure.ac | 2 +- tests/Makefile.am | 6 +- tests/weston-tests-env | 2 ++ 3 files changed, 8 insertions(+), 2 deletions(-) diff --git

Re: [PATCH weston 2/2] tests: Use&Fix parallel tests

Tested, fixed my odd make check problem with automake 1.13.1. Giulio 2013/2/21 Quentin Glidic : > From: Quentin Glidic > > Also fix automake 1.13 support > > Signed-off-by: Quentin Glidic > --- > configure.ac | 2 +- > tests/Makefile.am | 6 +++--- > tests/weston-tests-env | 2 +

[PATCH weston 8/8] gl-renderer: Clean up EGL image and GL texture destruction.

This also ensures we destroy all EGL images if a plane fails to be created. --- src/gl-renderer.c | 42 ++ 1 file changed, 26 insertions(+), 16 deletions(-) diff --git a/src/gl-renderer.c b/src/gl-renderer.c index 778fedd..c2d2490 100644 --- a/src/gl-render

[PATCH weston 7/8] gl-renderer: Remove unneeded glActiveTexture call.

--- src/gl-renderer.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/gl-renderer.c b/src/gl-renderer.c index 7a19b26..778fedd 100644 --- a/src/gl-renderer.c +++ b/src/gl-renderer.c @@ -1772,8 +1772,6 @@ gl_renderer_setup(struct weston_compositor *ec, EGLSurface egl_surface)

[PATCH weston 6/8] gl-renderer: Don't multiply with alpha uniform when it's 1.0.

This eliminates the multiplication of the alpha uniform for the common case of surfaces with 1.0 as alpha. --- src/gl-internal.h | 1 + src/gl-renderer.c | 14 -- src/gl-shaders.c | 6 +++--- 3 files changed, 12 insertions(+), 9 deletions(-) diff --git a/src/gl-internal.h b/src/gl-

[PATCH weston 5/8] gl-renderer: Add support for blending in linear space.

This makes compositing gamma correct by assuming all input surfaces are in the sRGB color space. It can be enabled by setting color-managed to true in the compositor section of weston.ini. It's implemented by converting from sRGB gamma using the new CONVERSION_FROM_SRGB shader attribute and drawi

[PATCH weston 4/8] gl-renderer: Adds the ability to draw damage to a texture and in turn to the framebuffer.

--- src/gl-internal.h | 5 ++ src/gl-renderer.c | 170 -- 2 files changed, 158 insertions(+), 17 deletions(-) diff --git a/src/gl-internal.h b/src/gl-internal.h index f27b221..205c2d9 100644 --- a/src/gl-internal.h +++ b/src/gl-internal.h @@ -

[PATCH weston 3/8] gl-renderer: Add optional support for desktop OpenGL.

This adds support for desktop OpenGL which can be enabled by with ./configure --enable-opengl. Most of the differences in API between OpenGL and OpenGL ES is hidden by the new gl_renderer fields. It also accesses GLES2 extensions by including GLES2/gl2platform.h directly. --- configure.ac

[PATCH weston 2/8] gl-renderer: Add flexible shader generation.

This add a more flexible way of generating shaders. It generates all valid combinations of different input, conversion and output pipelines, which can easily be extended with more if desired. --- src/gl-internal.h | 65 +++-- src/gl-renderer.c | 71 +++--- src/gl-shaders.c | 700 +++

[PATCH weston 1/8] gl-renderer: Move shader functions into it's own file.

This moves all shader functions into it's own file in preparation of a more flexible shader generation. It adds gl-internal.h for shared definitions between the files. --- src/Makefile.am | 2 + src/gl-internal.h | 144 src/gl-renderer.c | 382 ++---

[PATCH weston 2/2] tests: Use&Fix parallel tests

From: Quentin Glidic Also fix automake 1.13 support Signed-off-by: Quentin Glidic --- configure.ac | 2 +- tests/Makefile.am | 6 +++--- tests/weston-tests-env | 2 ++ 3 files changed, 6 insertions(+), 4 deletions(-) diff --git a/configure.ac b/configure.ac index 74eb892..d207e

[PATCH weston 1/2] Makefile.am: SUBDIRS is automagic

From: Quentin Glidic DIST_SUBDIRS should be used for corner cases only, not for conditional SUBDIRS Signed-off-by: Quentin Glidic --- Makefile.am | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/Makefile.am b/Makefile.am index 7b1b1bc..6e38688 100644 --- a/Makefil

Re: [PATCH weston 0/8] Move fade to shell and fix compositor state mess

On Thu, 21 Feb 2013 18:35:15 +0200 Ander Conselvan de Oliveira wrote: > Hi, > > The purpose of this series is to move the fade animation out of core > Weston and improve the compositor->state situation. Currently, when > the compositor goes idle and sends the lock signal, the shell sets up > the

[PATCH weston 8/8] compositor: Rename lock and unlock signals to idle and wake

After the changes introduced in the previous patch, those names are more appropriate. It is up to the shell to decide when to lock the screen. --- src/compositor.c |4 ++-- src/compositor.h |4 ++-- src/shell.c| 24 src/tablet-shell.c |4 ++-- 4 f

[PATCH weston 1/8] compositor: Split util.c into binding.c and animation.c

The few functions not related to bindings or animations were moved into compositor.c. --- src/Makefile.am |3 +- src/animation.c | 275 +++ src/bindings.c | 289 + src/compositor.c | 98 + src/util.c | 637 -

[PATCH weston 5/8] compositor: Move fade animation out of core Weston into shell

Previously, it was impossible to override the fade in/out behavior of Weston using a different shell, since this was implemented in core Weston. This also led to complicated interaction between the shell and the core when displaying lock surfaces and screensavers. This patch starts to solve this i

[PATCH weston 7/8] compositor: Move state tracking complexity into shell

Previously, when coming back from idle the compositor would try to track if the unlock signal needed to be sent, and the shell would change the compositor state in order to track when to display or hide the screensaver. This patch finishes moving this out of the compositor. With this, the composit

[PATCH weston 6/8] shell: Use its own timer for the screensaver timeout

Previously, the shell would change the idle_time of the compositor to the screensaver timeout and wake the compositor again, so that after the timeout the compositor would fade to black and then the shell would receive the lock signal again and would set the dpms state of the outputs to off. Inste

[PATCH weston 4/8] animation: Schedule a repaint only on the outputs the surface is in

The function weston_surface_animation_frame() would schedule a repaint on all outputs, as weston_surface_schedule_repaint() didn't exist when it was implemented. --- src/animation.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/animation.c b/src/animation.c index 9e2ad

[PATCH weston 3/8] animation: When fading, round the surface alpha when close to 0 or 1

The spring code stops when the current value is withing 0.0002 of the target. In that case, round the value to 0.0 or 1.0 to enable the use of fast paths, such as disabling blending in the GL renderer when an opaque region is set. --- src/animation.c |4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH weston 2/8] animation: Make fade more controllable

Add parameters to weston_fade_run() for setting the initial and target values for the fade, as well as a parameter to set the spring constant used for the animation. Also add the weston_fade_update() function, that allows the animation to be changed while it is still running. This will be used to

[PATCH weston 0/8] Move fade to shell and fix compositor state mess

Hi, The purpose of this series is to move the fade animation out of core Weston and improve the compositor->state situation. Currently, when the compositor goes idle and sends the lock signal, the shell sets up the screensaver and makes the compositor active again (after overriding the idle timeou

[PATCH v3] compositor: send a pointer motion if the focus surface moves under the pointer

--- src/compositor.c | 19 +++ 1 file changed, 15 insertions(+), 4 deletions(-) diff --git a/src/compositor.c b/src/compositor.c index 64d0830..025c6fd 100644 --- a/src/compositor.c +++ b/src/compositor.c @@ -776,6 +776,7 @@ weston_device_repick(struct weston_seat *seat) c

Re: [PATCH v2] compositor: send a pointer motion if the focus surface moves under the pointer

On Thu, Feb 21, 2013 at 8:03 AM, Giulio Camuffo wrote: > --- > src/compositor.c | 24 +++- > 1 file changed, 19 insertions(+), 5 deletions(-) > > diff --git a/src/compositor.c b/src/compositor.c > index 64d0830..ec0329b 100644 > --- a/src/compositor.c > +++ b/src/compositor.c

Re: [PATCH wayland 4/7 v3] Add support for custom dispatchers

On Thu, Feb 21, 2013 at 1:44 AM, Pekka Paalanen wrote: > Hi Jason, > > you forgot to CC wayland again. ;-) > > On Wed, 20 Feb 2013 10:23:28 -0600 > Jason Ekstrand wrote: > >> On Wed, Feb 20, 2013 at 3:45 AM, Pekka Paalanen wrote: >> > On Tue, 19 Feb 2013 16:09:21 -0600 >> > Jason Ekstrand wrote

Re: [PATCH v2] compositor: send a pointer motion if the focus surface moves under the pointer

On Thu, Feb 21, 2013 at 8:17 AM, Scott Moreau wrote: > Hi Giulio, > > On Thu, Feb 21, 2013 at 8:03 AM, Giulio Camuffo > wrote: > >> --- >> src/compositor.c | 24 +++- >> 1 file changed, 19 insertions(+), 5 deletions(-) >> >> diff --git a/src/compositor.c b/src/compositor.c >

Re: [PATCH v2] compositor: send a pointer motion if the focus surface moves under the pointer

Hi Giulio, On Thu, Feb 21, 2013 at 8:03 AM, Giulio Camuffo wrote: > --- > src/compositor.c | 24 +++- > 1 file changed, 19 insertions(+), 5 deletions(-) > > diff --git a/src/compositor.c b/src/compositor.c > index 64d0830..ec0329b 100644 > --- a/src/compositor.c > +++ b/src/c

[PATCH v2] compositor: send a pointer motion if the focus surface moves under the pointer

--- src/compositor.c | 24 +++- 1 file changed, 19 insertions(+), 5 deletions(-) diff --git a/src/compositor.c b/src/compositor.c index 64d0830..ec0329b 100644 --- a/src/compositor.c +++ b/src/compositor.c @@ -776,6 +776,7 @@ weston_device_repick(struct weston_seat *seat)

[PATCH] compositor: send a pointer motion if the focus surface moves under the pointer

--- src/compositor.c | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/src/compositor.c b/src/compositor.c index 64d0830..45d7c88 100644 --- a/src/compositor.c +++ b/src/compositor.c @@ -776,6 +776,7 @@ weston_device_repick(struct weston_seat *seat) const s

Re: [PATCH wayland 4/7 v3] Add support for custom dispatchers

Em 21-02-2013 04:44, Pekka Paalanen escreveu: This shouldn't be a problem as it's not crossing a dynamic linking boundary. Let's bump the complexity: add a libfeature to the mix, providing another protocol extension. The app is now using interfaces from libwayland and libfeature. Let's assume:

Re: [PATCH v6] compositor: call configure on surfaces with a null buffer too

On Thu, 21 Feb 2013 11:29:21 +0100 Giulio Camuffo wrote: > This way the shell can know when a surface has been unmapped by > checking the value returned by weston_surface_is_mapped(surface). > The configure handlers have now width and height parameters, so > they do not need anymore to check manu

[PATCH v6] compositor: call configure on surfaces with a null buffer too

This way the shell can know when a surface has been unmapped by checking the value returned by weston_surface_is_mapped(surface). The configure handlers have now width and height parameters, so they do not need anymore to check manually the buffer size. If a surface's buffer is NULL the width and h

Re: [PATCH v5] compositor: call configure on surfaces with a null buffer too

2013/2/21 Pekka Paalanen : > On Wed, 20 Feb 2013 16:28:48 +0100 > Giulio Camuffo wrote: > >> This way the shell can know when a surface has been unmapped by >> checking the value returned by weston_surface_is_mapped(surface). >> The configure handlers have now width and height parameters, so >> th

Re: [PATCH v5] compositor: call configure on surfaces with a null buffer too

On Wed, 20 Feb 2013 16:28:48 +0100 Giulio Camuffo wrote: > This way the shell can know when a surface has been unmapped by > checking the value returned by weston_surface_is_mapped(surface). > The configure handlers have now width and height parameters, so > they do not need anymore to check manu

Re: [PATCH 2/2] compositor: Remove surface->transform.enabled

On Wed, 20 Feb 2013 11:52:18 -0500 Kristian Høgsberg wrote: > We have matrix.type now and can rely on that for optimizing transformations > and choosing fast-paths. > --- > shared/matrix.c | 15 +++ > src/compositor-drm.c | 5 ++--- > src/compositor-rpi.c | 2 +- > src/comp