[RFC PATCH xserver] os: move tempfiles.d/x11.conf from systemd to here

2017-11-08 Thread Peter Hutterer
Let's not rely on some other package to set up and clean up after our tempfiles. Signed-off-by: Peter Hutterer --- Not all of these are created by the server, but moving them to the respective libs etc. makes even less sense. configure.ac | 13 +

Re: [PATCH xserver 3/3] ramdac: Handle master and slave cursors independently

2017-11-08 Thread Alex Goins
Thanks both for the quick feedback. Replies inline. On Wed, 8 Nov 2017, Hans de Goede wrote: > > https://lists.x.org/archives/xorg-devel/2016-September/050973.html implies > > that > > xf86CursorScreenPtr is part of the ABI. Mark xf86CursorPriv.h as such. > > I'm not sure if exporting this is a

[PATCH font-util] ucs2any: Fix parser crash on 32 bit

2017-11-08 Thread Tobias Stoeckmann
It is possible to crash ucs2any or provoke successful return value even though the processing was not successful. The problem lies within a possible integer overflow when adding elements with a key which is too large. You can trigger the issue this way on a 32 bit system: $ cat > source.bdf <<

Re: [PATCH xserver 3/3] ramdac: Handle master and slave cursors independently

2017-11-08 Thread Michel Dänzer
On 08/11/17 05:15 AM, Alex Goins wrote: > Change 7b634067 added HW cursor support for PRIME by removing the > pixmap_dirty_list check from xf86CursorSetCursor() and making the requisite > cursor functions set/check the cursor both on master and slave. > > However, before this change, drivers that

Re: [PATCH] modesetting: fail PreInit() if the device has zero connectors

2017-11-08 Thread Hans de Goede
Hi, On 03-11-17 17:56, Giuseppe Bilotta wrote: On Fri, Nov 3, 2017 at 10:09 AM, Hans de Goede wrote: Weird, on my XPS15 9550 where the nvidia GPU does not have/drives any outputs I do get 2 devices in xrandr --listproviders as expected. You may want to start with figuring

Re: [PATCH xserver 3/3] ramdac: Handle master and slave cursors independently

2017-11-08 Thread Hans de Goede
Hi, On 08-11-17 05:15, Alex Goins wrote: Change 7b634067 added HW cursor support for PRIME by removing the pixmap_dirty_list check from xf86CursorSetCursor() and making the requisite cursor functions set/check the cursor both on master and slave. However, before this change, drivers that did

Re: [PATCH xserver 2/3] ramdac: Add xf86CursorPriv.h to ABI

2017-11-08 Thread Hans de Goede
Hi, On 08-11-17 05:15, Alex Goins wrote: https://lists.x.org/archives/xorg-devel/2016-September/050973.html implies that xf86CursorScreenPtr is part of the ABI. Mark xf86CursorPriv.h as such. I'm not sure if exporting this is a good idea. I assume you are just after

Re: [PATCH xserver 1/3] ramdac: Fix formatting in xf86CheckHWCursor()

2017-11-08 Thread Hans de Goede
Hi, On 08-11-17 05:15, Alex Goins wrote: xf86CheckHWCursor() has spacing that is inconsistent with the rest of the file. Correct this in preparation for subsequent changes. Signed-off-by: Alex Goins LGTM: Acked-by: Hans de Goede Regards, Hans

Re: [PATCH xserver] xkb: small clarification

2017-11-08 Thread Eric Engestrom
On Tuesday, 2017-11-07 18:37:11 +0100, Giuseppe Bilotta wrote: > --- > xkb/xkbUtils.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > On Tue, Nov 7, 2017 at 10:55 AM, Eric Engestrom wrote: > > > I think this patch is good, because it explicitly shows the

Re: [PATCH xserver 03/11] modesetting: Simplify mst path blob parsing

2017-11-08 Thread Daniel Martin
On 7 November 2017 at 14:06, Emil Velikov wrote: > On 7 November 2017 at 09:38, Daniel Martin wrote: >> The kernel guarantees that the MST path property blob of a connector >> has a certain format and this property is immutable - can't be

Re: [PATCH xserver 01/11] xfree86: Fix set but not used warnings in lnx_platform

2017-11-08 Thread Daniel Martin
On 7 November 2017 at 13:57, Emil Velikov wrote: > On 7 November 2017 at 09:38, Daniel Martin wrote: >> ../hw/xfree86/os-support/linux/lnx_platform.c: In function ‘get_drm_info’: >> ../hw/xfree86/os-support/linux/lnx_platform.c:29:16: warning: