Re: Thread affinity

2012-03-11 Thread Jørgen Lind
Thanks for taking your time to look at the patch :) > Code is copy-pasted back in (why attach?), so it will probably look > weird and the reply-to markers are missing. Looks fine to me. > @@ -212,6 +220,14 @@ wl_proxy_marshal(struct wl_proxy *proxy, uint32_t > opcode, ...) > wl_clos

RE: [PATCH] compositor: Fix pick an incorrect surface due to dirty surface_list

2012-03-11 Thread wuzhiwen
Hi Kristian, >-Original Message- >From: Kristian Hoegsberg [mailto:hoegsb...@gmail.com] >Sent: Monday, March 12, 2012 4:23 AM >To: zhiwen...@linux.intel.com >Cc: wayland-devel@lists.freedesktop.org >Subject: Re: [PATCH] compositor: Fix pick an incorrect surface due to dirty >surface_list

[PATCH 5/5] RFC: add eglWaitGL()

2012-03-11 Thread Rob Clark
From: Rob Clark For deferred rendering GPUs, some magic under the hood of eglSwapBuffers() is responsible for making sure rendering has actually started. (glFlush() doesn't do the trick.) I've managed to make weston drm compositor work on sgx/pvr stack (plus omapdrm providing the KMS bits) by g

[PATCH 4/5] drm: fix issue copying supported plane formats

2012-03-11 Thread Rob Clark
From: Rob Clark Originally on the first 1/4th of the plane formats were copied, leading to problems if AR24/XR24 appears later in the list. --- src/compositor-drm.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/compositor-drm.c b/src/compositor-drm.c index cf9a937

[PATCH 3/5] drm: plane src coords are 16.16 fixed point

2012-03-11 Thread Rob Clark
From: Rob Clark --- src/compositor-drm.c |9 - 1 files changed, 4 insertions(+), 5 deletions(-) diff --git a/src/compositor-drm.c b/src/compositor-drm.c index 500ec15..cf9a937 100644 --- a/src/compositor-drm.c +++ b/src/compositor-drm.c @@ -538,13 +538,12 @@ drm_output_prepare_overl

[PATCH 2/5] drm: use page-flip for sprites, in case no vblank event

2012-03-11 Thread Rob Clark
From: Rob Clark If the output does not support vblank events (possibly the case with USB or DSI command mode displays which do not have a traditional vsync), use instead the page_flip handler to know when the display is finished scanning out a sprite. --- src/compositor-drm.c | 17

[PATCH 1/5] clients/window: fix build issues w/ gles

2012-03-11 Thread Rob Clark
From: Rob Clark --- clients/window.c | 13 - 1 files changed, 12 insertions(+), 1 deletions(-) diff --git a/clients/window.c b/clients/window.c index b09de5b..5465c2d 100644 --- a/clients/window.c +++ b/clients/window.c @@ -41,7 +41,12 @@ #include +#ifdef USE_CAIRO_GLESV2 +#

Re: [PATCH] compositor: Fix pick an incorrect surface due to dirty surface_list

2012-03-11 Thread Kristian Hoegsberg
On Fri, Mar 09, 2012 at 11:57:36AM +0800, zhiwen...@linux.intel.com wrote: > From: Alex Wu > > When a new toplevel window is activated and positioned under the > cursor, the pointer focus still on the previous window. The botton > event can not deliver to the new window. The root cause is that >

Re: [PATCH] compositor-drm: use udev_device_get_property_value() helper

2012-03-11 Thread Kristian Høgsberg
On Sun, Mar 11, 2012 at 3:07 PM, David Herrmann wrote: > On Sun, Mar 11, 2012 at 8:05 PM, David Herrmann > wrote: >> Udev provides a convenient helper. Use it instead of working with the >> property-list directly. >> >> Signed-off-by: David Herrmann >> --- >>  src/compositor-drm.c |   10 ---

Re: [PATCH] compositor-drm: use udev_device_get_property_value() helper

2012-03-11 Thread David Herrmann
On Sun, Mar 11, 2012 at 8:05 PM, David Herrmann wrote: > Udev provides a convenient helper. Use it instead of working with the > property-list directly. > > Signed-off-by: David Herrmann > --- >  src/compositor-drm.c |   10 -- >  1 file changed, 4 insertions(+), 6 deletions(-) > > diff --

[PATCH] compositor-drm: use udev_device_get_property_value() helper

2012-03-11 Thread David Herrmann
Udev provides a convenient helper. Use it instead of working with the property-list directly. Signed-off-by: David Herrmann --- src/compositor-drm.c | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/src/compositor-drm.c b/src/compositor-drm.c index 27d4512..ba376eb

[PATCH] compositor-drm: ignore hotplug-events from other devices

2012-03-11 Thread David Herrmann
If we have multiple video devices on the system (card0, card1), we should ignore hotplug events for cards that we do not use. This avoids calling update_outputs() if the event was not generated by our device so we avoid refreshing the DRM information if it didn't change. Signed-off-by: David Herrm

Re: [RFC] compositor: Simple key repeat implementation.

2012-03-11 Thread Joakim Sindholt
On Fri, 2012-03-09 at 10:53 +, Daniel Stone wrote: > Hi, > > On 8 March 2012 18:20, Joakim Sindholt wrote: > > I think that wayland should be consistent across the board and act > > *only* as a display server and otherwise implement only the minimum set > > of features necessary for proper op