Re: [PATCH] Avoid unnecessarily re-allocating texture buffer when the size hasn't changed.

2013-06-04 Thread Kristian Høgsberg
On Tue, May 28, 2013 at 05:28:49PM -0700, Sinclair Yeh wrote: v3: * Removed unnecessary parentheses * Added check for switching from EGL image to SHM buffer * Moved shader assignment out of IF condition v2: Fixed the wrong comparison v1: Depending on specific DRI driver implementation,

Re: [PATCH 00/15] weston scaling support

2013-06-04 Thread John Kåre Alsaker
On Mon, Jun 3, 2013 at 10:21 PM, Bill Spitzak spit...@gmail.com wrote: + Clients get input in exact pixels (no rounding errors). + Clients doesn't have to transform input events in order to match the sizes used by buffers. You said pels don't match buffer pixels. Therefore a transformation

Re: [PATCH 00/15] weston scaling support

2013-06-04 Thread Pekka Paalanen
On Mon, 3 Jun 2013 16:21:48 +0200 John Kåre Alsaker john.kare.alsa...@gmail.com wrote: On Mon, Jun 3, 2013 at 2:22 PM, Pekka Paalanen ppaala...@gmail.com wrote: On Tue, 28 May 2013 19:27:35 +0200 John Kåre Alsaker john.kare.alsa...@gmail.com wrote: + Conceptually simple to implement.

Re: [PATCH 00/15] weston scaling support

2013-06-04 Thread Pekka Paalanen
On Mon, 03 Jun 2013 18:41:51 -0700 Bill Spitzak spit...@gmail.com wrote: Jason Ekstrand wrote: I think I'm begining to see what you and John have been suggesting. While I think Alexander's proposal will work well enough for the standard case, I think this also has merit. If I were to

Re: Compiling weston now needs colord

2013-06-04 Thread Pekka Paalanen
On Mon, 3 Jun 2013 11:44:25 -0700 Othman, Ossama ossama.oth...@intel.com wrote: Hi, On Thu, May 30, 2013 at 12:42 AM, sardemff7+wayl...@sardemff7.net wrote: On 30/05/2013 08:24, Michael Hasselmann wrote: On Tue, 2013-05-28 at 22:16 -0700, Bill Spitzak wrote: Running autogen.sh in

[PATCH weston 0/5] Small patches

2013-06-04 Thread Quentin Glidic
From: Quentin Glidic sardemff7+...@sardemff7.net The headless-backend patch may not be really needed, but it helped a lot running tests with it, which is the final goal of one of my series of patches. Quentin Glidic (4): tests: Add .weston extension to clients tests weston: Allow relative

[PATCH weston 2/5] weston: Allow relative paths for modules

2013-06-04 Thread Quentin Glidic
From: Quentin Glidic sardemff7+...@sardemff7.net This is to be used by tests or when developping to load modules directly from the build tree Signed-off-by: Quentin Glidic sardemff7+...@sardemff7.net --- man/weston.man | 5 +++-- src/compositor.c | 9 ++--- 2 files changed, 9

[PATCH weston 3/5] headless-backend: Init fake pointer and keyboard

2013-06-04 Thread Quentin Glidic
From: U. Artie Eoff ullysses.a.e...@intel.com Signed-off-by: Quentin Glidic sardemff7+...@sardemff7.net --- src/compositor-headless.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/compositor-headless.c b/src/compositor-headless.c index e250b98..b8eba50 100644 ---

[PATCH weston 1/5] tests: Add .weston extension to clients tests

2013-06-04 Thread Quentin Glidic
From: Quentin Glidic sardemff7+...@sardemff7.net We can then add tests which do not use Weston in the test suite. Signed-off-by: Quentin Glidic sardemff7+...@sardemff7.net --- tests/.gitignore | 7 +-- tests/Makefile.am | 41 ++--- 2 files changed, 23

[PATCH weston 4/5] weston.pc: Add dependencies for pixman-1 and xkbcommon

2013-06-04 Thread Quentin Glidic
From: Quentin Glidic sardemff7+...@sardemff7.net Signed-off-by: Quentin Glidic sardemff7+...@sardemff7.net --- src/weston.pc.in | 1 + 1 file changed, 1 insertion(+) diff --git a/src/weston.pc.in b/src/weston.pc.in index 828cb1f..6d5ca19 100644 --- a/src/weston.pc.in +++ b/src/weston.pc.in @@

Re: Compiling weston now needs colord

2013-06-04 Thread Emilio Pozuelo Monfort
On 04/06/13 09:29, Pekka Paalanen wrote: Optionality is good for end users and distros building stuff, but making it automatic makes it easy for developers to just ignore testing some bits. :-) Then it would be the disabled case that wouldn't get testing, and when a user or a distro disabled a

Re: [PATCH] clients/nested: Fix compilation

2013-06-04 Thread Armin K.
On 06/04/2013 01:05 PM, Armin K wrote: --- clients/Makefile.am | 2 +- clients/nested.c| 3 +++ configure.ac| 1 + 3 files changed, 5 insertions(+), 1 deletion(-) diff --git a/clients/Makefile.am b/clients/Makefile.am index 1379c4b..d84f575 100644 --- a/clients/Makefile.am +++

[PATCH] clients/nested: Fix compilation

2013-06-04 Thread Armin K
--- clients/Makefile.am | 2 +- clients/nested.c| 3 +++ configure.ac| 1 + 3 files changed, 5 insertions(+), 1 deletion(-) diff --git a/clients/Makefile.am b/clients/Makefile.am index 1379c4b..d84f575 100644 --- a/clients/Makefile.am +++ b/clients/Makefile.am @@ -124,7 +124,7 @@

Re: [PATCH] clients/nested: Fix compilation

2013-06-04 Thread sardemff7+wayland
On 04/06/2013 13:05, Armin K wrote: --- clients/Makefile.am | 2 +- clients/nested.c| 3 +++ configure.ac| 1 + 3 files changed, 5 insertions(+), 1 deletion(-) diff --git a/clients/Makefile.am b/clients/Makefile.am index 1379c4b..d84f575 100644 --- a/clients/Makefile.am +++

Re: [PATCH] clients/nested: Fix compilation

2013-06-04 Thread Armin K.
On 06/04/2013 01:19 PM, sardemff7+wayl...@sardemff7.net wrote: On 04/06/2013 13:05, Armin K wrote: diff --git a/clients/nested.c b/clients/nested.c index baaff64..132724d 100644 --- a/clients/nested.c +++ b/clients/nested.c @@ -34,8 +34,11 @@ #include EGL/egl.h #include EGL/eglext.h +

Re: [PATCH] clients/nested: Fix compilation

2013-06-04 Thread Armin K.
On 06/04/2013 01:19 PM, Armin K. wrote: On 06/04/2013 01:19 PM, sardemff7+wayl...@sardemff7.net wrote: On 04/06/2013 13:05, Armin K wrote: diff --git a/clients/nested.c b/clients/nested.c index baaff64..132724d 100644 --- a/clients/nested.c +++ b/clients/nested.c @@ -34,8 +34,11 @@ #include

Re: [PATCH] clients/nested: Fix compilation

2013-06-04 Thread sardemff7+wayland
On 04/06/2013 13:19, Armin K. wrote: On 06/04/2013 01:19 PM, sardemff7+wayl...@sardemff7.net wrote: On 04/06/2013 13:05, Armin K wrote: diff --git a/clients/nested.c b/clients/nested.c index baaff64..132724d 100644 --- a/clients/nested.c +++ b/clients/nested.c @@ -34,8 +34,11 @@ #include

Re: [PATCH] clients/nested: Fix compilation

2013-06-04 Thread Armin K.
On 06/04/2013 01:36 PM, sardemff7+wayl...@sardemff7.net wrote: On 04/06/2013 13:19, Armin K. wrote: On 06/04/2013 01:19 PM, sardemff7+wayl...@sardemff7.net wrote: On 04/06/2013 13:05, Armin K wrote: diff --git a/clients/nested.c b/clients/nested.c index baaff64..132724d 100644 ---

Re: [PATCH wayland-web] raspberrypi: mention Raspbian packages

2013-06-04 Thread Pekka Paalanen
On Mon, 03 Jun 2013 15:57:33 +0200 Emilio Pozuelo Monfort poch...@gmail.com wrote: On 03/06/13 15:50, ppaala...@gmail.com wrote: From: Pekka Paalanen pekka.paala...@collabora.co.uk Signed-off-by: Pekka Paalanen pekka.paala...@collabora.co.uk Cc: Daniel Stone dan...@fooishbar.org Cc:

[PATCH weston] build: Add declaration checks to check for required syscall flags

2013-06-04 Thread Rob Bradford
From: Rob Bradford r...@linux.intel.com The required flags are relatively new and some older enterprise distributions do not feature them. https://bugs.freedesktop.org/show_bug.cgi?id=63360 --- configure.ac | 9 + 1 file changed, 9 insertions(+) diff --git a/configure.ac b/configure.ac

[PATCH wayland] build: Add declaration checks to check for required syscall flags

2013-06-04 Thread Rob Bradford
From: Rob Bradford r...@linux.intel.com The required flags are relatively new and some older enterprise distributions do not feature them. https://bugs.freedesktop.org/show_bug.cgi?id=63360 --- configure.ac | 11 +++ 1 file changed, 11 insertions(+) diff --git a/configure.ac

[PATCH 1/2] compositor-drm: Cache the DPMS property on drm_output

2013-06-04 Thread Ander Conselvan de Oliveira
From: Ander Conselvan de Oliveira ander.conselvan.de.olive...@intel.com This avoids one drmModeGetConnector() call every time the DPMS mode is set. That call can take hundreds of milliseconds due to DDC. --- src/compositor-drm.c | 21 +++-- 1 file changed, 7 insertions(+), 14

[PATCH 2/2] compositor-drm: Force DPMS state to ON after drmModeSetCrtc()

2013-06-04 Thread Ander Conselvan de Oliveira
From: Ander Conselvan de Oliveira ander.conselvan.de.olive...@intel.com The kernel is supposed to set this when drmModeSetCrtc() is called but at least the i915 driver wouldn't do that in all cases. A fix for this should be released with kernel 3.10, but we work around the issue in older kernels

[PATCH weston 2/2] weston-launch: Only clear the environment of the child process

2013-06-04 Thread Ander Conselvan de Oliveira
In order for the systemd notification to succeed, sd_notify() needs the value of the NOTIFY_SOCKET environment variable, but the environment was cleared before entering the launcher's main loop. Work around this by only clearing the environment of the child process. --- src/weston-launch.c |

[PATCH weston 1/2] compositor: Add systemd notification support

2013-06-04 Thread Ander Conselvan de Oliveira
Call sd_notify() when the server is ready to receive connections. This allows weston to be called as systemd service with the 'notify' type, in turn allowing systemd to delay execution of processes that need Weston until it is ready to receive connections. The call to sd_notify() is made through

[PATCH weston 0/2] [RFC] Systemd notification support

2013-06-04 Thread Ander Conselvan de Oliveira
Hi, I was looking into a way of having Weston signal when it is available for clients to connect, and using systemd seems like a good way to achieve this. The idea here is to have Weston inform SystemD that it is ready, by calling sd_notify() just before entering the main loop. SystemD expects

[PATCH weston] shared: fix build of libshared-cairo

2013-06-04 Thread ppaalanen
From: Pekka Paalanen pekka.paala...@collabora.co.uk Another case of missing wayland-util.h, as we didn't pass any libwayland CFLAGS. This is triggerable on a system, where libwayland is installed in a custom prefix, and pixman, cairo, libpng, and webp are either not installed or are installed in

Re: [PATCH wayland-web] raspberrypi: mention Raspbian packages

2013-06-04 Thread darxus
Thanks, pushed. On 06/04, ppaala...@gmail.com wrote: From: Pekka Paalanen pekka.paala...@collabora.co.uk Also mention, that using the user 'pi' should save from pain of setting up permissions for devices. Run apt-get update, too. Signed-off-by: Pekka Paalanen

Re: [PATCH weston 0/2] [RFC] Systemd notification support

2013-06-04 Thread Tom Gundersen
Hi Ander, On Tue, Jun 4, 2013 at 3:43 PM, Ander Conselvan de Oliveira ander.conselvan.de.olive...@intel.com wrote: I was looking into a way of having Weston signal when it is available for clients to connect, and using systemd seems like a good way to achieve this. Wouldn't it be even better

[PATCH v2 wfits] test_dayselector: Add user test

2013-06-04 Thread Brian Lovin
Signed-off-by: Brian Lovin brian.j.lo...@intel.com --- src/test/efl/test_dayselector.cpp | 112 +- 1 file changed, 111 insertions(+), 1 deletion(-) diff --git a/src/test/efl/test_dayselector.cpp b/src/test/efl/test_dayselector.cpp index 254991f..172926d

[PATCH weston] add fullscreen to .gitignore

2013-06-04 Thread U. Artie Eoff
From: U. Artie Eoff ullysses.a.e...@intel.com Signed-off-by: U. Artie Eoff ullysses.a.e...@intel.com --- clients/.gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/clients/.gitignore b/clients/.gitignore index f925ce6..94082f8 100644 --- a/clients/.gitignore +++ b/clients/.gitignore

RE: [PATCH v2 wfits] test_dayselector: Add user test

2013-06-04 Thread Eoff, Ullysses A
Thanks, committed. U. Artie -Original Message- From: wayland-devel-bounces+ullysses.a.eoff=intel@lists.freedesktop.org [mailto:wayland-devel- bounces+ullysses.a.eoff=intel@lists.freedesktop.org] On Behalf Of Brian Lovin Sent: Tuesday, June 04, 2013 9:58 AM To:

Re: [PATCH wayland-web] raspberrypi: mention Raspbian packages

2013-06-04 Thread Daniel Stone
Thanks! For what it's worth, weston-launch isn't actually in the package you get right now, due to comedy version number mishaps (first forgetting how Debian deals with date numbering, then the person with the board I asked to fix it forgetting that it was now 2013 and not 2012), but that'll be

Re: [PATCH 00/15] weston scaling support

2013-06-04 Thread Bill Spitzak
Pekka Paalanen wrote: Bill Spitzak spit...@gmail.com wrote: I always figured that the transformation of a subsurface is multiplied by the transformation of the parent surface. For the scaler proposal this means the output rectangle is in parent-surface buffer pixels. You are confusing

Re: [PATCH 00/15] weston scaling support

2013-06-04 Thread Bill Spitzak
John Kåre Alsaker wrote: And I'm worried that high-resolution pointing devices will be ignored by clients that immediately round to the nearest buffer pixel. Presumably they wouldn't use the high-resolution for anything then. I'm not sure how this relates to the topic either. A

Re: [PATCH 00/15] weston scaling support

2013-06-04 Thread John Kåre Alsaker
On Tue, Jun 4, 2013 at 10:24 PM, Bill Spitzak spit...@gmail.com wrote: John Kåre Alsaker wrote: And I'm worried that high-resolution pointing devices will be ignored by clients that immediately round to the nearest buffer pixel. Presumably they wouldn't use the high-resolution for

Re: [PATCH 00/15] weston scaling support

2013-06-04 Thread Bill Spitzak
John Kåre Alsaker wrote: This is still a problem without any high-DPI stuff. You could just as easily have an input device with a resolution 3 times higher than the display. I am worried that most clients will work around the problem of the scale proposal by rounding all numbers from 3.5 to

Re: [PATCH 00/15] weston scaling support

2013-06-04 Thread John Kåre Alsaker
On Wed, Jun 5, 2013 at 3:11 AM, Bill Spitzak spit...@gmail.com wrote: John Kåre Alsaker wrote: This is still a problem without any high-DPI stuff. You could just as easily have an input device with a resolution 3 times higher than the display. I am worried that most clients will work

Re: [PATCH weston] build: Add declaration checks to check for required syscall flags

2013-06-04 Thread Kristian Høgsberg
On Tue, Jun 04, 2013 at 01:23:01PM +0100, Rob Bradford wrote: From: Rob Bradford r...@linux.intel.com The required flags are relatively new and some older enterprise distributions do not feature them. Looks fine, thanks. https://bugs.freedesktop.org/show_bug.cgi?id=63360 ---

Re: [PATCH wayland] build: Add declaration checks to check for required syscall flags

2013-06-04 Thread Kristian Høgsberg
On Tue, Jun 04, 2013 at 01:27:49PM +0100, Rob Bradford wrote: From: Rob Bradford r...@linux.intel.com The required flags are relatively new and some older enterprise distributions do not feature them. Applied with TFD_CLOEXEC fixup as the other patch, thanks. Kristian

Re: [PATCH weston] shared: fix build of libshared-cairo

2013-06-04 Thread Kristian Høgsberg
On Tue, Jun 04, 2013 at 06:47:39PM +0300, ppaala...@gmail.com wrote: From: Pekka Paalanen pekka.paala...@collabora.co.uk Another case of missing wayland-util.h, as we didn't pass any libwayland CFLAGS. This is triggerable on a system, where libwayland is installed in a custom prefix, and

Re: [PATCH weston] add fullscreen to .gitignore

2013-06-04 Thread Kristian Høgsberg
On Tue, Jun 04, 2013 at 10:10:56AM -0700, U. Artie Eoff wrote: From: U. Artie Eoff ullysses.a.e...@intel.com Added, thanks. Kristian Signed-off-by: U. Artie Eoff ullysses.a.e...@intel.com --- clients/.gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/clients/.gitignore

Re: [PATCH] clients/nested: Fix compilation

2013-06-04 Thread Kristian Høgsberg
On Tue, Jun 04, 2013 at 01:05:22PM +0200, Armin K wrote: --- clients/Makefile.am | 2 +- clients/nested.c| 3 +++ configure.ac| 1 + 3 files changed, 5 insertions(+), 1 deletion(-) Thanks, applied. I edited the fix a bit to just not compile the nested compositor example at all

Re: [PATCH 1/2] compositor-drm: Cache the DPMS property on drm_output

2013-06-04 Thread Kristian Høgsberg
On Tue, Jun 04, 2013 at 06:31:26PM +0200, David Herrmann wrote: Hi Ander On Tue, Jun 4, 2013 at 3:24 PM, Ander Conselvan de Oliveira conselv...@gmail.com wrote: From: Ander Conselvan de Oliveira ander.conselvan.de.olive...@intel.com This avoids one drmModeGetConnector() call every time

Re: [PATCH 1/2] compositor-drm: Cache the DPMS property on drm_output

2013-06-04 Thread Kristian Høgsberg
On Tue, Jun 04, 2013 at 04:24:04PM +0300, Ander Conselvan de Oliveira wrote: From: Ander Conselvan de Oliveira ander.conselvan.de.olive...@intel.com This avoids one drmModeGetConnector() call every time the DPMS mode is set. That call can take hundreds of milliseconds due to DDC. This looks

Re: [PATCH wayland] protocol: Add a name event to give seat name

2013-06-04 Thread Kristian Høgsberg
On Fri, May 31, 2013 at 06:08:23PM +0100, Rob Bradford wrote: From: Rob Bradford r...@linux.intel.com This provides the ability for a client to differentiate events from different seats in a multiple seat environment. That looks fine, applied. For v2 of wl_seat, I do have one other items I

Re: [PATCH weston 06/10] udev-seat: Make the udev_input structure an embedded structure

2013-06-04 Thread Kristian Høgsberg
On Fri, May 31, 2013 at 06:09:55PM +0100, Rob Bradford wrote: From: Rob Bradford r...@linux.intel.com And as a result of this stop iterating through the compositor seat list (of one item) and instead access the udev_input structure directly. I think we should move udev_input to udev-seat.c

Re: [PATCH weston 10/10] udev-seat: Use udev rules to support multiple seats

2013-06-04 Thread Kristian Høgsberg
On Fri, May 31, 2013 at 06:09:59PM +0100, Rob Bradford wrote: From: Rob Bradford r...@linux.intel.com By labelling devices with ENV{WL_SEAT} in udev rules the devices will be pulled into multiple weston seats. As a result you can get multiple independent seats under the DRM and fbdev

Re: [PATCH wayland 1/4] Add a side field and some sanity checks to wl_map.

2013-06-04 Thread Kristian Høgsberg
On Sat, Jun 01, 2013 at 05:40:52PM -0500, Jason Ekstrand wrote: The original wl_map implementation did no checking to ensures that ids fell on the correct side of the WL_SERVER_ID_START line. This meant that a client could send the server a server ID and it would happily try to use it. Also,

Re: [PATCH wayland 2/4] Add support for flags in the wl_map API and add a WL_MAP_ENTRY_LEGACY flag

2013-06-04 Thread Kristian Høgsberg
On Sat, Jun 01, 2013 at 05:40:53PM -0500, Jason Ekstrand wrote: The implementation in this commit allows for one bit worth of flags. If more flags are desired at a future date, then the wl_map implementation will have to change but the wl_map API will not. Looks better with the accessor

Re: [PATCH wayland 4/4] Make wl_resource opaque

2013-06-04 Thread Kristian Høgsberg
On Sat, Jun 01, 2013 at 05:40:55PM -0500, Jason Ekstrand wrote: Signed-off-by: Jason Ekstrand ja...@jlekstrand.net --- src/wayland-server.c | 25 + src/wayland-server.h | 40 2 files changed, 33 insertions(+), 32 deletions(-)

Re: [PATCH wayland] protocol: Add a name event to give seat name

2013-06-04 Thread Kristian Høgsberg
On Mon, Jun 03, 2013 at 01:19:47PM +0100, Rob Bradford wrote: From: Rob Bradford r...@linux.intel.com This provides the ability for a client to differentiate events from different seats in a multiple seat environment. v2: handled versioning correctly based on feedback from Daniel and Ander

Re: [PATCH] Update EFL build page

2013-06-04 Thread Kristian Høgsberg
On Fri, May 31, 2013 at 11:04:45AM +0100, Christopher Michael wrote: Hi All, I have attached a patch to update the EFL build instructions for Wayland. Would someone be kind enough to push this upstream please ? Pushed, thanks. Kristian Kind Regards, devilhorns From

Re: [PATCH weston 16/18] configure.ac: Enable AC_USE_SYSTEM_EXTENSIONS

2013-06-04 Thread Kristian Høgsberg
On Wed, May 22, 2013 at 06:03:19PM +0300, ppaala...@gmail.com wrote: From: Daniel Stone dan...@fooishbar.org AC_USE_SYSTEM_EXTENSIONS enables _XOPEN_SOURCE, _GNU_SOURCE and similar macros to expose the largest extent of functionality supported by the underlying system. This is required