[PATCH 0/2] doc: Switch to generated diagrams with hot-linked areas.

2014-12-12 Thread Jon A. Cruz
These changes switch from static png images for diagrams to generated diagrams with hot-linked areas. It adds xslt to do the 'heavy lifting' of adding useful diagrams and automate linking with graphviz output consistent with the way Doxygen does. This should now make it easy to add additional diag

[PATCH 2/2] doc: Create hot-linked areas in documents.

2014-12-12 Thread Jon A. Cruz
Added xslt processing to give DocBook output diagram image maps/hot-linked areas consistent with those automatically generated by Doxygen. Signed-off-by: Jon A. Cruz --- doc/doxygen/Makefile.am | 10 +++-- doc/publican/Makefile.am | 21 -- doc/publican/merge-ma

[PATCH 1/2] doc: Switch from static image files to generated diagrams.

2014-12-12 Thread Jon A. Cruz
Switches diagrams from using static PNG images to instead generate them via simple graphviz DOT markup files. Signed-off-by: Jon A. Cruz --- configure.ac | 13 + doc/doxygen/Makefile.am| 15 -- doc/doxygen/dot/wayland-ar

[PATCH 7/7] clients: add simple-dmabuf client

2014-12-12 Thread Louis-Francis Ratté-Boulianne
From: George Kiagiadakis Signed-off-by: George Kiagiadakis Signed-off-by: Louis-Francis Ratté-Boulianne --- .gitignore | 1 + Makefile.am | 15 ++ clients/simple-dmabuf.c | 578 configure.ac| 9 + 4 file

[PATCH 4/7] compositor-x11: init linux_dmabuf support

2014-12-12 Thread Louis-Francis Ratté-Boulianne
From: Pekka Paalanen --- src/compositor-x11.c | 5 + 1 file changed, 5 insertions(+) diff --git a/src/compositor-x11.c b/src/compositor-x11.c index a760f33..7b15ba9 100644 --- a/src/compositor-x11.c +++ b/src/compositor-x11.c @@ -51,6 +51,7 @@ #include "pixman-renderer.h" #include "../sha

[PATCH 1/7] protocol: add linux_dmabuf extension RFCv1

2014-12-12 Thread Louis-Francis Ratté-Boulianne
From: Pekka Paalanen An experimental (hence the 'z' prefix) linux_dmabuf Wayland protocol extension for creating dmabuf-based wl_buffers in a generic manner. This does not include proper dmabuf metadata negotiation because there is no way to communicate all dmabuf constraints from the compositor

[PATCH 5/7] compositor-drm: init linux_dmabuf support

2014-12-12 Thread Louis-Francis Ratté-Boulianne
From: Pekka Paalanen --- src/compositor-drm.c | 5 + 1 file changed, 5 insertions(+) diff --git a/src/compositor-drm.c b/src/compositor-drm.c index 9b4d4dc..44fc912 100644 --- a/src/compositor-drm.c +++ b/src/compositor-drm.c @@ -50,6 +50,7 @@ #include "libinput-seat.h" #include "launcher

[PATCH 0/7] Generic dmabuf protocol RFCv1

2014-12-12 Thread Louis-Francis Ratté-Boulianne
This serie of patches by Pekka and George contains an experimental implementation of "zlinux_dmabuf" protocol. See these links for more information about the design and failed previous attempts: http://lists.freedesktop.org/archives/wayland-devel/2014-June/015362.html https://bugs.freedesktop.org

[PATCH 6/7] compositor-drm: dmabuf GBM import

2014-12-12 Thread Louis-Francis Ratté-Boulianne
From: Pekka Paalanen --- src/compositor-drm.c | 26 ++ 1 file changed, 22 insertions(+), 4 deletions(-) diff --git a/src/compositor-drm.c b/src/compositor-drm.c index 44fc912..d3e8b34 100644 --- a/src/compositor-drm.c +++ b/src/compositor-drm.c @@ -839,7 +839,9 @@ drm_ou

[PATCH 3/7] gl-renderer: add dmabuf import

2014-12-12 Thread Louis-Francis Ratté-Boulianne
From: Pekka Paalanen Squashed with: gl-renderer: error if dmabuf exposed unsupported gl_renderer: always use GL_TEXTURE_EXTERNAL_OES with dmabuf --- src/gl-renderer.c | 103 ++ 1 file changed, 103 insertions(+) diff --git a/src/gl-renderer.c

[PATCH 2/7] dmabuf: implement linux_dmabuf extension

2014-12-12 Thread Louis-Francis Ratté-Boulianne
From: Pekka Paalanen Signed-off-by: Pekka Paalanen Signed-off-by: Louis-Francis Ratté-Boulianne --- Makefile.am| 2 + src/linux-dmabuf.c | 322 + src/linux-dmabuf.h | 45 3 files changed, 369 insertions(+) create mode 100

Re: [PATCH weston] compositor-x11: Move the x11 event handler to the display loop

2014-12-12 Thread Derek Foreman
I just noticed that the follow patch *exactly* undoes commit 22ba60e Is there any other reason that commit was necessary, or was it intended to be cosmetic? On 12/12/14 02:12 PM, Derek Foreman wrote: > While it conceptually makes sense to put the x11 event handler > in the compositor "input" loop

Re: [PATCH wayland] doc: Fill in high level description for Surfaces

2014-12-12 Thread Daniel Stone
Hi, On Friday, December 12, 2014, Bill Spitzak wrote: > I think talking about a surface having multiple buffers is misleading. > > A wl_surface only knows about one buffer after commit: the last buffer > attached to it. The wording being proposed here makes it sound like the > surface keeps trac

[PATCH weston] compositor-x11: Move the x11 event handler to the display loop

2014-12-12 Thread Derek Foreman
While it conceptually makes sense to put the x11 event handler in the compositor "input" loop, the input loop is actually dispatched in the middle of the frame repaint. When the X11 event results in closing the compositor, this can cause the current output to be destroyed just prior to trying to p

Re: [PATCH wayland] doc: Fill in high level description for Surfaces

2014-12-12 Thread Bill Spitzak
I think talking about a surface having multiple buffers is misleading. A wl_surface only knows about one buffer after commit: the last buffer attached to it. The wording being proposed here makes it sound like the surface keeps track of the previous buffer and expects or requires it to be used

Re: [PATCH wayland] doc: Fill in high level description for Surfaces

2014-12-12 Thread Giulio Camuffo
2014-12-12 21:37 GMT+02:00 Bryce Harrington : > On Thu, Dec 11, 2014 at 09:49:28AM +, Daniel Stone wrote: >> Hi, >> >> On Thursday, December 11, 2014, Giulio Camuffo >> wrote: >> >> > 2014-12-11 4:04 GMT+02:00 Bryce Harrington > > >: >> > > + A surface manages a rectangular grid of pixels

[PATCH wayland v2] doc: Fill in high level description for Surfaces

2014-12-12 Thread Bryce Harrington
Signed-off-by: Bryce Harrington --- doc/publican/sources/Protocol.xml | 34 ++ 1 file changed, 30 insertions(+), 4 deletions(-) diff --git a/doc/publican/sources/Protocol.xml b/doc/publican/sources/Protocol.xml index b79b6be..9c6cb3b 100644 --- a/doc/publican/s

Re: [PATCH wayland] doc: Fill in high level description for Surfaces

2014-12-12 Thread Bryce Harrington
On Thu, Dec 11, 2014 at 09:49:28AM +, Daniel Stone wrote: > Hi, > > On Thursday, December 11, 2014, Giulio Camuffo > wrote: > > > 2014-12-11 4:04 GMT+02:00 Bryce Harrington > >: > > > + A surface manages a rectangular grid of pixels that clients create > > > + for displaying their

Re: [PATCH wayland] doc: Fill in high level description for Surfaces

2014-12-12 Thread Bryce Harrington
On Thu, Dec 11, 2014 at 10:28:25AM +0200, Giulio Camuffo wrote: > 2014-12-11 4:04 GMT+02:00 Bryce Harrington : > > > > Signed-off-by: Bryce Harrington > > --- > > doc/publican/sources/Protocol.xml | 20 +++- > > 1 file changed, 15 insertions(+), 5 deletions(-) > > > > diff --git

Re: Where should project Weston go?

2014-12-12 Thread Bryce Harrington
On Fri, Dec 12, 2014 at 05:01:12PM +, Daniel Stone wrote: > Hi, > > On 12 December 2014 at 16:10, Pekka Paalanen wrote: > > > > On Fri, 12 Dec 2014 14:17:58 +0200 > > Giulio Camuffo wrote: > > > 2014-12-12 13:29 GMT+02:00 Daniel Stone : > > > > Oooh, whilst we're talking about requirements f

Re: Where should project Weston go?

2014-12-12 Thread Daniel Stone
Hi, On 12 December 2014 at 16:10, Pekka Paalanen wrote: > > On Fri, 12 Dec 2014 14:17:58 +0200 > Giulio Camuffo wrote: > > 2014-12-12 13:29 GMT+02:00 Daniel Stone : > > > Oooh, whilst we're talking about requirements for merging stuff ... how > > > about enforcing decent Doxygen bits as a hard r

Re: [PATCH weston v1 11/17] add get_geometry & geometry request/event

2014-12-12 Thread Pekka Paalanen
On Fri, 5 Dec 2014 14:36:44 +0100 Marek Chalupa wrote: > --- > protocol/wayland-test.xml | 11 +++ > tests/weston-test-client-helper.c | 18 ++ > tests/weston-test-client-helper.h | 6 ++ > tests/weston-test.c | 23 +++ > 4

Re: Where should project Weston go?

2014-12-12 Thread Pekka Paalanen
On Fri, 12 Dec 2014 14:17:58 +0200 Giulio Camuffo wrote: > 2014-12-12 13:29 GMT+02:00 Daniel Stone : > > > > Oooh, whilst we're talking about requirements for merging stuff ... how > > about enforcing decent Doxygen bits as a hard requirement for merging > > anything big-ish? Be that internal int

Re: Time in wayland presentation extension

2014-12-12 Thread Pekka Paalanen
On Fri, 12 Dec 2014 13:48:44 +0100 Dan Oscarsson wrote: > tis 2014-12-09 klockan 17:32 +0200 skrev Pekka Paalanen: > > > I have expected FreeSync, Adaptive-Sync, Nvidia G-sync to actually allow > > > the video player (not the compositor) to play video and get each frame > > > to be displayed at e

Re: [PATCH weston v1 09/17] tests: do attach only with normal surfaces

2014-12-12 Thread Pekka Paalanen
On Fri, 5 Dec 2014 14:36:42 +0100 Marek Chalupa wrote: > When we call move_client on toytoolkit client when intializing, then the > wl_buffer is NULL and we don't want that. Actually, toytoolkit clients > don't need attaching and commiting the surface at all, because it has > been already done.

Re: [PATCH weston v1 08/17] add toytoolkit_client_create

2014-12-12 Thread Pekka Paalanen
On Fri, 5 Dec 2014 14:36:41 +0100 Marek Chalupa wrote: > --- > tests/weston-test-client-helper.c | 345 > -- > tests/weston-test-client-helper.h | 3 + > 2 files changed, 333 insertions(+), 15 deletions(-) > > diff --git a/tests/weston-test-client-helper.

Re: [PATCH weston v1 06/17] clients: add window_get_decoration_size

2014-12-12 Thread Pekka Paalanen
On Fri, 5 Dec 2014 14:36:39 +0100 Marek Chalupa wrote: > In tests we need to know about the decoration size. Moreover > it's a small clean-up. > > Signed-off-by: Marek Chalupa > --- > clients/window.c | 37 ++--- > clients/window.h | 2 ++ > 2 files changed, 2

Re: [PATCH weston v1 05/17] clients: add missing getters for input objects

2014-12-12 Thread Pekka Paalanen
On Fri, 5 Dec 2014 14:36:38 +0100 Marek Chalupa wrote: > We need get wayland objects from toytoolkit display > (there already are display_get_output, input_get_seat, ...), > so that we can use them in the tests. This patch adds: > > display_get_input() > input_get_wl_pointer() > input_get

Re: [PATCH weston v1 04/17] tests: use wl_* objects instead of test structs

2014-12-12 Thread Pekka Paalanen
On Fri, 5 Dec 2014 14:36:37 +0100 Marek Chalupa wrote: > When storing client state i. e. focus, we use our structs. > For example when we got pointer focus, we store into pointer->focus > the type 'struct surface'. We did it via user data in wl_* object, > but toytoolkit has its own data in the

Re: [PATCH weston v1 03/17] tests: add toytoolkit into struct client

2014-12-12 Thread Pekka Paalanen
On Fri, 5 Dec 2014 14:36:36 +0100 Marek Chalupa wrote: > Allow store information about toytoolkit widget in client. > Now we'll be able to get client behave as a toytoolkit client. > > Signed-off-by: Marek Chalupa > --- > tests/weston-test-client-helper.h | 9 + > 1 file changed, 9 in

Re: [PATCH weston v1 01/17] tests: simplify move_client and make it more generic

2014-12-12 Thread Pekka Paalanen
Hi Artie, you wrote the initial test plugin code in commit 65e7e7a65bf17f7b886653bfee850ff728bbd904. Would you happen to remember why the move_surface request requires a configure call (i.e. attach & commit of wl_surface) to apply the move? Was it perhaps to test that the configure hook gets call

Re: [PATCH weston 1/3] window: use roundtrip instead of dispatch after get_registry

2014-12-12 Thread Marek Chalupa
On 12 December 2014 at 13:24, Pekka Paalanen wrote: > > On Fri, 5 Dec 2014 13:49:40 +0100 > Marek Chalupa wrote: > > > wl_display_dispatch() just dispatches events that are in > > default and display queues and if there are no events, > > then it will wait for them. But only dispatching > > the

Re: [PATCH weston v2] clipboard: set fd member of clipboard_source

2014-12-12 Thread Pekka Paalanen
On Wed, 10 Dec 2014 12:07:58 +0100 Marek Chalupa wrote: > The fd member of clipboard_source structure was not set > but was used in close(). > > v2. don't do unnecessary changes > > Signed-off-by: Marek Chalupa > --- > src/clipboard.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/

Re: [PATCH weston v2] screenshooter: fix various memory handling

2014-12-12 Thread Pekka Paalanen
On Wed, 10 Dec 2014 11:50:46 +0100 Marek Chalupa wrote: > There were unchecked malloc and no free for this memory. > Also simplify error handling in one function. > > v2. remove check if memory is NULL, according to man pages, > free(NULL) is a no-op > > Signed-off-by: Marek Chalupa > ---

Re: [PATCH weston 1/3] window: use roundtrip instead of dispatch after get_registry

2014-12-12 Thread Pekka Paalanen
On Fri, 5 Dec 2014 13:49:40 +0100 Marek Chalupa wrote: > wl_display_dispatch() just dispatches events that are in > default and display queues and if there are no events, > then it will wait for them. But only dispatching > the events doesn't guarantee that we got all the global announcements, >

Re: [PATCH v2 libinput 1/2] evdev: Ensure the libevdev object receives the new fd on resume

2014-12-12 Thread Hans de Goede
Hi, On 10-12-14 02:47, Peter Hutterer wrote: From: Carlos Garnacho Otherwise, input_events will be attempted to read from the wrong place, which also leaves the right/current fd with pending data to be read, making the epoll fd wake up constantly. Signed-off-by: Carlos Garnacho Reviewed-by:

Re: Where should project Weston go?

2014-12-12 Thread Giulio Camuffo
2014-12-12 13:29 GMT+02:00 Daniel Stone : > Hi, > > On 10 December 2014 at 23:21, Bryce Harrington > wrote: >> >> An idea I've been kicking around would be to *require* Reviewed-by on >> all patches. Now, you'd probably think this would *slow* things down, >> but consider this: If as a patch sub

Re: [PATCH 3/3] touchpad: Use TOUCHPAD_MIN_SAMPLES in tp_get_delta

2014-12-12 Thread Hans de Goede
Hi, On 10-12-14 04:34, Peter Hutterer wrote: On Tue, Dec 09, 2014 at 12:47:11PM +0100, Hans de Goede wrote: Use TOUCHPAD_MIN_SAMPLES in tp_get_delta rather then hardcoding "4". s/then/than/ Also remove the superfluous TOUCHPAD_MIN_SAMPLES check before calling tp_get_delta in tp_get_pointer_

Re: Where should project Weston go?

2014-12-12 Thread Daniel Stone
Hi, On 10 December 2014 at 23:21, Bryce Harrington wrote: > > An idea I've been kicking around would be to *require* Reviewed-by on > all patches. Now, you'd probably think this would *slow* things down, > but consider this: If as a patch submitter I know I have to get > someone's review, then

Re: Where should project Weston go?

2014-12-12 Thread Pekka Paalanen
On Wed, 10 Dec 2014 12:36:19 -0800 Bryce Harrington wrote: > On Wed, Dec 10, 2014 at 04:00:43PM +, Daniel Stone wrote: > > A platform that people can develop > > desktop environments on? Sure! I guess a lot of those would be fine as > > Weston plugins, though ultimately we'd need something li