[PATCH xf86-input-libinput] Correct the horizontal scroll property name

2016-09-12 Thread Peter Hutterer
Clear typo. Not bothering to be backwards compatible here, anything that uses the #define will update on rebuild, anyone using the string directly should've told me about the typo... Signed-off-by: Peter Hutterer --- Just sending this to the list as a sort-of heads-up.

Re: [PATCH xserver] randr: Fix crtc_bounds when using rotation combined with reflection

2016-09-12 Thread Michel Dänzer
On 12/09/16 07:53 PM, Hans de Goede wrote: > Before this commit crtc_bounds() did not take reflection into account, > when using reflection with 0 / 180 degree rotation this was not an > issue because of the default in the switch-case doing the right thing. > > But when using 90 / 270 degree

Re: [PATCH 2/3] Xext/shm: Better support cases where O_CLOEXEC is not defined

2016-09-12 Thread Julien Cristau
On Mon, Sep 12, 2016 at 10:53:39 -0700, Jeremy Huddleston Sequoia wrote: > > > On Sep 12, 2016, at 04:33, Julien Cristau wrote: > > > > On Sun, Sep 11, 2016 at 20:01:50 -0700, Jeremy Huddleston Sequoia wrote: > > > >> Signed-off-by: Jeremy Huddleston Sequoia

[PATCH xserver v2] xwayland: handle EAGAIN and EINTR gracefully

2016-09-12 Thread Olivier Fourdan
wl_display_flush() can fail with EAGAIN and Xwayland would make this a fatal error. Handle the usual EAGAIN and EINTR gracefully so that Xwayland doesn't die for so little. Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1278159 Signed-off-by: Olivier Fourdan --- v2:

Re: [PATCH 2/3] Xext/shm: Better support cases where O_CLOEXEC is not defined

2016-09-12 Thread Jeremy Huddleston Sequoia
> On Sep 12, 2016, at 04:33, Julien Cristau wrote: > > On Sun, Sep 11, 2016 at 20:01:50 -0700, Jeremy Huddleston Sequoia wrote: > >> Signed-off-by: Jeremy Huddleston Sequoia >> --- >> Xext/shm.c | 6 +- >> 1 file changed, 5 insertions(+), 1

Re: [PATCH:xserver] OsSigHandler should not show rtld errors for unrelated signals

2016-09-12 Thread Aaron Plattner
On 09/10/2016 09:08 PM, Alan Coopersmith wrote: > If RTLD_DI_SETSIGNAL is set to let us turn runtime linker/loader errors > into catchable signals, then we should only show the errors when catching > that signal, instead of tossing out red herrings to distract people with > unrelated crashes long

[PATCH xserver] xwayland: handle EAGAIN and EINTR gracefully

2016-09-12 Thread Olivier Fourdan
wl_display_flush() can fail with EAGAIN and Xwayland would make this a fatal error. Handle the usual EAGAIN and EINTR gracefully so that Xwayland doesn't die for so little. Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1278159 Signed-off-by: Olivier Fourdan ---

Re: modesetting: allow switching from software to hardware cursors.

2016-09-12 Thread Hans de Goede
Hi Michael, Since the posting of this patch, the underlying code has changed, e.g. the modesetting driver does use the load_cursor_argb_check hook already now. Can you check if this patch is still necessary, and if so rebase it please ? Thanks & Regards, Hans On 06-03-16 12:56, Michael

Re: [PATCH 3/3] os/inputthread: Fix setting of cloexec on file descriptors

2016-09-12 Thread Julien Cristau
On Sun, Sep 11, 2016 at 20:01:51 -0700, Jeremy Huddleston Sequoia wrote: > O_CLOEXEC is not a file bit. It is not setable with F_SETFL. One must use it > when calling open(2). To set it cloexec on an existing fd, F_SETFD and > FD_CLOEXEC must be used. > > This also fixes a build failure

Re: [PATCH 2/3] Xext/shm: Better support cases where O_CLOEXEC is not defined

2016-09-12 Thread Julien Cristau
On Sun, Sep 11, 2016 at 20:01:50 -0700, Jeremy Huddleston Sequoia wrote: > Signed-off-by: Jeremy Huddleston Sequoia > --- > Xext/shm.c | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) > Does anything have O_TMPFILE but not O_CLOEXEC? Cheers, Julien > diff

Re: [PATCH 1/3] Xext/shm: Fix usage of F_GETFD to match standard

2016-09-12 Thread Julien Cristau
On Sun, Sep 11, 2016 at 20:01:49 -0700, Jeremy Huddleston Sequoia wrote: > flags = fcntl(fd, F_GETFD) is compliant. > > fcntl(fd, F_GETFD, ) is non-compliant (Linux extension?) > > cf: http://pubs.opengroup.org/onlinepubs/9699919799/functions/fcntl.html > Signed-off-by: Jeremy Huddleston

[PATCH xserver v9] xf86Cursor: Deal with rotation on GPU screens using a hw-cursor

2016-09-12 Thread Hans de Goede
When a slave-output is rotated the transformation is done on the blit from master to slave GPU, so crtc->transform_in_use is not set, but we still need to adjust the mouse position for things to work. This commit modifies xf86_crtc_transform_cursor_position to not rely on

Re: [PATCH v8 xserver 6/7] xf86Cursor: Deal with rotation on GPU screens using a hw-cursor

2016-09-12 Thread Hans de Goede
Hi, On 12-09-16 10:39, Michel Dänzer wrote: On 12/09/16 05:24 PM, Hans de Goede wrote: On 12-09-16 09:47, Michel Dänzer wrote: On 09/09/16 09:50 PM, Hans De Goede wrote: When a slave-output is rotated the transformation is done on the blit from master to slave GPU, so crtc->transform_in_use

[PATCH xserver] randr: Fix crtc_bounds when using rotation combined with reflection

2016-09-12 Thread Hans de Goede
Before this commit crtc_bounds() did not take reflection into account, when using reflection with 0 / 180 degree rotation this was not an issue because of the default in the switch-case doing the right thing. But when using 90 / 270 degree rotation we would also end up in the default which is

Re: [PATCH v8 xserver 6/7] xf86Cursor: Deal with rotation on GPU screens using a hw-cursor

2016-09-12 Thread Hans de Goede
Hi, On 12-09-16 10:39, Michel Dänzer wrote: On 12/09/16 05:24 PM, Hans de Goede wrote: On 12-09-16 09:47, Michel Dänzer wrote: On 09/09/16 09:50 PM, Hans De Goede wrote: When a slave-output is rotated the transformation is done on the blit from master to slave GPU, so crtc->transform_in_use

Re: [PATCH v8 xserver 6/7] xf86Cursor: Deal with rotation on GPU screens using a hw-cursor

2016-09-12 Thread Michel Dänzer
On 12/09/16 05:24 PM, Hans de Goede wrote: > On 12-09-16 09:47, Michel Dänzer wrote: >> On 09/09/16 09:50 PM, Hans De Goede wrote: >>> When a slave-output is rotated the transformation is done on the blit >>> from master to slave GPU, so crtc->transform_in_use is not set, but we >>> still need to

Re: [PATCH v8 xserver 6/7] xf86Cursor: Deal with rotation on GPU screens using a hw-cursor

2016-09-12 Thread Hans de Goede
Hi, On 12-09-16 09:47, Michel Dänzer wrote: On 09/09/16 09:50 PM, Hans De Goede wrote: When a slave-output is rotated the transformation is done on the blit from master to slave GPU, so crtc->transform_in_use is not set, but we still need to adjust the mouse position for things to work. This

Re: [PATCH font/arabic-misc] Typo: font.dir -> fonts.dir

2016-09-12 Thread Matthieu Herrb
On Mon, Sep 12, 2016 at 09:41:22AM +0200, Hans de Goede wrote: > Hi, > > On 11-09-16 22:20, Alan Coopersmith wrote: > >Reviewed-by: Alan Coopersmith > >for all the fonts with this typo. > > Ack, also: > > Reviewed-by: Hans de Goede > > So

Re: [PATCH v8 xserver 6/7] xf86Cursor: Deal with rotation on GPU screens using a hw-cursor

2016-09-12 Thread Michel Dänzer
On 09/09/16 09:50 PM, Hans De Goede wrote: > When a slave-output is rotated the transformation is done on the blit > from master to slave GPU, so crtc->transform_in_use is not set, but we > still need to adjust the mouse position for things to work. > > This commit modifies

Re: [PATCH font/arabic-misc] Typo: font.dir -> fonts.dir

2016-09-12 Thread Hans de Goede
Hi, On 11-09-16 22:20, Alan Coopersmith wrote: Reviewed-by: Alan Coopersmith for all the fonts with this typo. Ack, also: Reviewed-by: Hans de Goede So Matthieu do you have xorg repo commit / push rights ? If not how are we going to deal