Re-send: [PATCH] modesetting: allow switching from software to hardware cursors.

2016-03-09 Thread Michael Thayer
Currently if modesetting ever fails to set a hardware cursor it will switch to using a software cursor and never go back. Change this to try a hardware cursor first every time a new one is set. This is needed because hardware may be able to handle some cursors in harware and others not, or

Re: [PATCH] [RFC] glamor: implement write-only prepares

2016-03-09 Thread Michel Dänzer
On 10.03.2016 10:51, Dave Airlie wrote: > From: Dave Airlie > > For some putimages we know we won't ever care about the data we readback, > we are going to trash it with the putimage contents. So implement > GLAMOR_ACCESS_WO mode. > > This will avoid doing the readbacks. Use

[PATCH] [RFC] glamor: implement write-only prepares

2016-03-09 Thread Dave Airlie
From: Dave Airlie For some putimages we know we won't ever care about the data we readback, we are going to trash it with the putimage contents. So implement GLAMOR_ACCESS_WO mode. This will avoid doing the readbacks. Use it for putimages that are copy with a solid

Re: [PATCH xserver 1/5] xfree86/modes: Refactor xf86_use_hw_cursor_argb to use xf86_use_hw_cursor

2016-03-09 Thread Keith Packard
Michel Dänzer writes: > Sounds good to me. > > One thing that might be nice to do as part of your patch would be moving > the xf86_config->cursor = NULL assignment from xf86_cursors_init to > xf86_hide_cursors. That way xf86_config->cursor would always be NULL > while the

Re: [PATCH] xv: Build left-shift constants from ints not longs

2016-03-09 Thread Adam Jackson
On Wed, 2016-03-09 at 10:59 -0500, Olivier Fourdan wrote: > Reviewed-by: Olivier Fourdan remote: I: patch #76365 updated using rev 723881f31d65353e80660e6d8cd9785e6ec1b430. remote: I: 1 patch(es) updated to state Accepted. To

Re: [PATCH xserver] glamor: do not build Xv support when --disable-xv

2016-03-09 Thread Adam Jackson
On Wed, 2016-03-09 at 16:45 +0100, Olivier Fourdan wrote: > Signed-off-by: Olivier Fourdan Merged this and the xwayland xv support: remote: E: failed to find patch for rev da7724d3d277c6c8a814881785b716896802629a. remote: I: patch #76376 updated using rev

Re: [PULL] hw/xwin cleanup patches

2016-03-09 Thread Adam Jackson
On Wed, 2016-03-09 at 16:31 +, Jon Turney wrote: > The following changes since commit 24042b4e367803dd64f3fcdc1bef7b2bf36c4145: > >   modesetting: Allow CRTC transforms to actually take effect (2016-03-09 > 16:46:13 +0900) > > are available in the git repository at: > >  

[PULL] hw/xwin cleanup patches

2016-03-09 Thread Jon Turney
The following changes since commit 24042b4e367803dd64f3fcdc1bef7b2bf36c4145: modesetting: Allow CRTC transforms to actually take effect (2016-03-09 16:46:13 +0900) are available in the git repository at: git://people.freedesktop.org/~jturney/xserver for you to fetch changes up to

Re: [PATCH] xv: Build left-shift constants from ints not longs

2016-03-09 Thread Olivier Fourdan
- Original Message - > Looks good to me. > > Funny that we have: > #define XvNumReasons (XvLastReason + 1) > and a few lines below it: > #define XvAnyReasonMask ((1L< Anyway, > > Reviewed-by: Olivier

Re: [PATCH] xv: Build left-shift constants from ints not longs

2016-03-09 Thread Olivier Fourdan
- Original Message - > We typically store these in ints in server, leading to warnings like: > > xwayland-glamor-xv.c: In function ‘xwl_glamor_xv_add_adaptors’: > xwayland-glamor-xv.c:339:16: warning: large integer implicitly truncated > to unsigned type [-Woverflow] > pa->type =

[PATCH xserver] glamor: do not build Xv support when --disable-xv

2016-03-09 Thread Olivier Fourdan
Signed-off-by: Olivier Fourdan --- glamor/Makefile.am | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/glamor/Makefile.am b/glamor/Makefile.am index c488029..c631c53 100644 --- a/glamor/Makefile.am +++ b/glamor/Makefile.am @@ -46,10 +46,14 @@

[PATCH xserver v4] xwayland: add glamor Xv adaptor

2016-03-09 Thread Olivier Fourdan
This adds an Xv adaptor using glamor. Signed-off-by: Olivier Fourdan --- v2: Plug leak of Xv adaptor and glamor private ports array on closure v3: Get the adaptor size from the GL_MAX_TEXTURE_SIZE v4: Fix double comma and unsigned int warning, fix build with --disable-xv

Re: [PATCH] Add Compose sequence for U+1F4A9.

2016-03-09 Thread Adam Jackson
On Wed, 2016-03-09 at 14:35 +0100, Daniel Albers wrote: > Signed-off-by: Daniel Albers Merged, thanks: remote: I: patch #76361 updated using rev e1011b9e2f6c82255959cf3cc1d8cda402ded0a9. remote: I: 1 patch(es) updated to state Accepted. To

Re: [PATCH xserver v3] xwayland: add glamor Xv adaptor

2016-03-09 Thread Adam Jackson
On Tue, 2016-03-08 at 10:15 -0500, Olivier Fourdan wrote: > Hi Adam, > > I had posted a revisited version using GL_MAX_TEXTURE_SIZE to > determine the encoder size as per your review last week, is there > anything else that needs rework? This adds a warning, but I don't think that's your fault:

[PATCH] xv: Build left-shift constants from ints not longs

2016-03-09 Thread Adam Jackson
We typically store these in ints in server, leading to warnings like: xwayland-glamor-xv.c: In function ‘xwl_glamor_xv_add_adaptors’: xwayland-glamor-xv.c:339:16: warning: large integer implicitly truncated to unsigned type [-Woverflow] pa->type = XvWindowMask | XvInputMask | XvImageMask;;

Re: [PULL] Reviewed cursor changes

2016-03-09 Thread Adam Jackson
On Wed, 2016-03-09 at 16:51 +0900, Michel Dänzer wrote: > Hi Keith / Adam, > > > the following changes since commit a3e681eafa5355b8bb3b099d47983f14f0d5e197: > >   glamor: Source pictures are always depth 32 (2016-03-08 15:17:19 -0500) > > are available in the git repository at: > >  

[PATCH] Add Compose sequence for U+1F4A9.

2016-03-09 Thread Daniel Albers
Signed-off-by: Daniel Albers --- nls/en_US.UTF-8/Compose.pre | 2 ++ 1 file changed, 2 insertions(+) diff --git a/nls/en_US.UTF-8/Compose.pre b/nls/en_US.UTF-8/Compose.pre index 2041757..d88fe36 100644 --- a/nls/en_US.UTF-8/Compose.pre +++ b/nls/en_US.UTF-8/Compose.pre @@ -278,6

Re: [PATCH xserver] xwayland: Make it possible to force CLOCK_MONOTONIC

2016-03-09 Thread Daniel Stone
Hi, On 9 March 2016 at 04:27, Jonas Ådahl wrote: > By default the X server will try CLOCK_MONOTONIC_COARSE before > CLOCK_MONOTONIC. This causes various issues for Wayland display > servers who may get their internal timestamps from the CLOCK_MONOTONIC, > since it may very well

Re: [PATCH xserver 4/4] xwayland: sync event queue to check compositor reply

2016-03-09 Thread Olivier Fourdan
- Original Message - > - Original Message - > > Read and dispatch pending Wayland events to make sure we do nto miss a > > possible reply from the compositor prior to discard a key repeat. > > > > Signed-off-by: Olivier Fourdan > > --- > >

[PATCH xserver 3/4 v2] xwayland: refactor Wayland event handling

2016-03-09 Thread Olivier Fourdan
To be able to reuse some code. Signed-off-by: Olivier Fourdan --- v2: call prepre_read() before read() so we don't end up in a deadlock hw/xwayland/xwayland.c | 44 hw/xwayland/xwayland.h | 2 ++ 2 files changed, 34

Re: [PATCH xserver 4/4] xwayland: sync event queue to check compositor reply

2016-03-09 Thread Olivier Fourdan
- Original Message - > Read and dispatch pending Wayland events to make sure we do nto miss a > possible reply from the compositor prior to discard a key repeat. > > Signed-off-by: Olivier Fourdan > --- > hw/xwayland/xwayland-input.c | 3 +++ > 1 file changed, 3

[PATCH xserver 4/4] xwayland: sync event queue to check compositor reply

2016-03-09 Thread Olivier Fourdan
Read and dispatch pending Wayland events to make sure we do nto miss a possible reply from the compositor prior to discard a key repeat. Signed-off-by: Olivier Fourdan --- hw/xwayland/xwayland-input.c | 3 +++ 1 file changed, 3 insertions(+) diff --git

[PATCH xserver 3/4] xwayland: refactor Wayland event handling

2016-03-09 Thread Olivier Fourdan
To be able to reuse some code. Signed-off-by: Olivier Fourdan --- hw/xwayland/xwayland.c | 44 hw/xwayland/xwayland.h | 2 ++ 2 files changed, 34 insertions(+), 12 deletions(-) diff --git a/hw/xwayland/xwayland.c

[PATCH xserver 2/4 v3] xwayland: add a server sync before repeating keys

2016-03-09 Thread Olivier Fourdan
Key repeat is handled by the X server, but input events need to be processed and forwarded by the Wayland compositor first. Make sure the Wayland compositor is actually processing events, to avoid repeating keys in Xwayland while the Wayland compositor cannot deal with input events for whatever

[PATCH xserver 1/4 v2] xkb: add hook to allow/deny AccessX key repeat

2016-03-09 Thread Olivier Fourdan
The xserver generates the key repeat by itself. But when used with another server processing inputs first (e.g. a Wayland compositor), the other server may be busy dealing with some other things and not queue up key release events in time. Add a vfunc in XkbSrvInfo to possibly add a check before