[PATCH] evdev: Handle SYN_DROPPED, keep transaction buffer

2013-04-05 Thread Martin Minarik
Each evdev device keeps a key press bitmap, the incoming events are filtered to the following constraints: 1. only notify releases for previously pressed keys 2. only notify presses for previously released keys 3. on device destroy, notify a releases for all pressed keys Notes: 1. For example wh

Re: [patches] Add a color management framework to weston

2013-04-05 Thread Graeme Gill
Bill Spitzak wrote: Maybe the client should be able to query the color space of the output and can set the color space of it's buffer to the same one, this would indicate that no color correction is needed. The compositor could still color-correct it from one output's color space to another's if

Re: [patches] Add a color management framework to weston

2013-04-05 Thread Graeme Gill
Thomas Daede wrote: I am not sure that doing the color conversion in the compositor is the correct place. Some of it has to be there to support vcgt, but for more general conversions, it gets complicated quickly. Hi, The expectation is that vcgt is per output (ie., it's loaded into the display

[PATCH weston] Make backends always specify output repaint time

2013-04-05 Thread Jonas Ådahl
Most backends relies on gettimeofday(2) for output repaint timestamps but this is not a requirement. Before this patch repaints coming from idle_repaint() always used gettimeofday(2) for timestamps. For backends not using that time source this could cause large jumps between timestamps. To fix thi

Re: [patches] Add a color management framework to weston

2013-04-05 Thread Bill Spitzak
Richard Hughes wrote: For GIMP I was going to just allow the app to opt-out a sub-surface from any kind of color management. i.e. do it all in the app. Maybe the client should be able to query the color space of the output and can set the color space of it's buffer to the same one, this would

Re: [patches] Add a color management framework to weston

2013-04-05 Thread Richard Hughes
On 5 April 2013 20:23, Thomas Daede wrote: > Color correction is most important for artists doing work in something > like the GIMP. Programs like this (as of GIMP 3.0, at least) generally > work in higher bit depths For GIMP I was going to just allow the app to opt-out a sub-surface from any kin

Re: [patches] Add a color management framework to weston

2013-04-05 Thread Thomas Daede
I am not sure that doing the color conversion in the compositor is the correct place. Some of it has to be there to support vcgt, but for more general conversions, it gets complicated quickly. Color correction is most important for artists doing work in something like the GIMP. Programs like this

Re: [PATCH weston v3 1/3] protocol: Add wl_notification_daemon interface

2013-04-05 Thread Bill Spitzak
Emilio Pozuelo Monfort wrote: Hi, Why are you reinventing org.freedesktop.Notifications? Couldn't this be a client that implements the org.freedesktop.Notifications dbus interface and draws notifications instead? It looks like he is trying to define the protocol the receiver of these dbus n

Re: [patches] Add a color management framework to weston

2013-04-05 Thread Bill Spitzak
Richard Hughes wrote: I've got to move onto other stuff next week, but I'd be really interested if anyone has inputs on how the sub-surface gamut mapping using shaders is going to work. The main complication seems to be that it's per-output, rather than per-surface. I've played with 1bit masks i

Re: [patches] Add a color management framework to weston

2013-04-05 Thread Richard Hughes
On 5 April 2013 09:53, Richard Hughes wrote: > * GLib really wants to return all signals (colord_device_changed_cb) > on the main thread, not on the thread that's running the loop. As discussed on IRC, it seems I was wrong. I've attached both patches (v6) for review. I've switched the coldplug on

Re: glmark2 ported to wayland by Rafał Mielniczuk

2013-04-05 Thread darxus
On 04/05, Rafal Mielniczuk wrote: > To fix the problem with the missing header file you may try setting > CPLUS_INCLUDE_PATH=$WLD/include environment variable Great, thanks, that worked. Added build instructions to http://wayland.freedesktop.org/extras.html -- "Life is either a daring adventure

Re: [PATCH weston v3 1/3] protocol: Add wl_notification_daemon interface

2013-04-05 Thread Emilio Pozuelo Monfort
Hi, Why are you reinventing org.freedesktop.Notifications? Couldn't this be a client that implements the org.freedesktop.Notifications dbus interface and draws notifications instead? Regards, Emilio On 04/03/2013 07:13 PM, Quentin Glidic wrote: > From: Quentin Glidic > > Signed-off-by: Quentin

Re: glmark2 ported to wayland by Rafał Mielniczuk

2013-04-05 Thread darxus
On 04/05, Rafal Mielniczuk wrote: > To fix the problem with the missing header file you may try setting > CPLUS_INCLUDE_PATH=$WLD/include environment variable Thanks. I've been told needing C_INCLUDE_PATH set is a bug, so I'd think this is too? Looks like running glmark2-es2-wayland on DRM took

Re: [patches] Add a color management framework to weston

2013-04-05 Thread Graeme Gill
Bill Spitzak wrote: These gamma ramps are an attempt to simulate a color conversion from some color space (sRGB?) to the color space of the output device. No, they are not. Gamma ramp setting generally accomplishes one or more of: 1) Setting the brightness to a calibrated level. 2) Setting th

Re: glmark2 ported to wayland by Rafał Mielniczuk

2013-04-05 Thread Rafal Mielniczuk
On 05.04.2013 06:37, dar...@chaosreigns.com wrote: > An open source GLES2 / GL benchmark. > > It's been accepted upstream: > https://code.launchpad.net/~kruk87/glmark2/glmark2-wayland/+merge/156907 > https://launchpad.net/glmark2 > https://afrantzis.wordpress.com/2011/12/16/glmark2-more-than-a-benc

Re: [patches] Add a color management framework to weston

2013-04-05 Thread Richard Hughes
On 4 April 2013 17:11, John Kåre Alsaker wrote: > You should remove the destroy and user_data fields from weston_color_profile. Fixed. > weston_cms_create_profile and weston_cms_load_profile should just > return a pointer to weston_color_profile. Fixed. > Plugins should just exit if compositor

Re: [patches] Add a color management framework to weston

2013-04-05 Thread Richard Hughes
On 4 April 2013 21:30, Matthias Clasen wrote: > Just ftr (and without actually looking at the patch): glib signals > don't need an event loop, they're entirely synchronous. Right, but in this case the event is coming from the GDBus thread, which I assume is g_idle_add'ing the signal it back to th