Re: [PATCH v5 00/13] PRIME Synchronization

Sorry for all of the self-replies. I spent more time looking into what might be causing this (despite not being able to reproduce it), and found that in the Present extension's present_check_flip(), it doesn't use flipping if there are slave outputs. So, that's why it's not allowing page

[PATCH xserver v2 2/3] xfree86: use the xf86CheckPrivs() helper for modulepath/logfile

v2: Rebase against updated xf86CheckPrivs() helper. Signed-off-by: Emil Velikov --- hw/xfree86/common/xf86Init.c | 50 1 file changed, 23 insertions(+), 27 deletions(-) diff --git a/hw/xfree86/common/xf86Init.c

[PATCH xserver v2 1/3] xfree86: factor out the check priviliges and print a big warning

Current message was quite off "file specified must be a relative path" and alike. Just factor it out and use "path/file" as needed. v2: Rework error message, drop "Using default", print actual arg value. Signed-off-by: Emil Velikov --- hw/xfree86/common/xf86Init.c |

[PATCH xserver v2 3/3] xfree86: drop unneeded strdup for modulepath/logfile

The destination variable is never freed, thus we even plug some memory leaks. v2: Rebase against updated xf86CheckPrivs() helper. Signed-off-by: Emil Velikov --- hw/xfree86/common/xf86Init.c | 14 ++ 1 file changed, 2 insertions(+), 12 deletions(-) diff

Re: [PATCH xserver 1/3] xfree86: factor out the check priviliges and print a big warning

On 2 May 2016 at 14:45, Adam Jackson wrote: > On Sun, 2016-05-01 at 18:34 +0100, Emil Velikov wrote: >> Ping ? > > This faithfully preserves the original behavior, but... > >> > +static void >> > +xf86CheckPrivs(const char *option, const char *arg, const char *path_file, >> > +

Re: [PATCH 2/2] modesetting: port clean start code from amdgpu. (v2)

On Mon, 2016-05-02 at 17:12 -0400, Alex Deucher wrote: > On Mon, May 2, 2016 at 4:54 PM, Dave Airlie wrote: > > From: Dave Airlie > > > > Both radeon and amdgpu don't set the mode until the first blockhandler, > > this means everything should be rendered

Re: [PATCH libX11 3/5] fix: warning: pointer targets in passing argument 2 of '_XSend' differ in signedness [-Wpointer-sign]

On Sat, Apr 30, 2016 at 11:08:34PM +0200, walter harms wrote: > > > Am 29.04.2016 03:03, schrieb Alan Coopersmith: > > On 04/28/16 04:43 PM, Peter Hutterer wrote: > >> fixing this isn't going to do much, and adding fake statements neither. > >> to fix the compiler warnings you could do something

Re: [PATCH xserver] prime: clean up slave bo properly. (v2)

-1 would be consistent with the invalid handle passed back from glamor_fd_from_pixmap() when it fails. It would probably be good to put PixmapUnshareSlavePixmap above the "Unref the pixmap twice" comment to avoid separating the comment from the lines it is describing. Other than that, this looks

Re: [PATCH 2/2] modesetting: port clean start code from amdgpu. (v2)

On Mon, May 2, 2016 at 4:54 PM, Dave Airlie wrote: > From: Dave Airlie > > Both radeon and amdgpu don't set the mode until the first blockhandler, > this means everything should be rendered on the screen correctly by then. > > This ports this code, it also

[PATCH 1/2] modesetting: add support for background none.

From: Dave Airlie This adds support using glamor for background None. loosely based off the amdgpu code. relies on the glamor_finish code. Acked-by: Eric Anholt Signed-off-by: Dave Airlie --- hw/xfree86/drivers/modesetting/driver.c

[PATCH 2/2] modesetting: port clean start code from amdgpu. (v2)

From: Dave Airlie Both radeon and amdgpu don't set the mode until the first blockhandler, this means everything should be rendered on the screen correctly by then. This ports this code, it also removes the tail call of EnterVT from ScreenInit, it really isn't necessary and

Re: [PATCH vesa] Use default refresh by default

On Thu, 2016-04-28 at 13:56 +0200, Stefan Dirsch wrote: > This seems to be a feature that isn't tested well. Too many VBE 3.0 > BIOSes are incorrectly implemented. remote: I: patch #84256 updated using rev 0400b332a90f53b7a1361f832b60f1d4e90fd6e7. remote: I: 1 patch(es) updated to state

Re: [PATCH xproto v2] Don't let XFD_SETSIZE exceed FD_SETSIZE

On Mon, 2016-04-25 at 23:30 +0200, Julien Cristau wrote: > On Mon, Apr 25, 2016 at 17:12:08 +0100, James Clarke wrote: > > > This fixes 2c94cdb453bc641246cc8b9a876da9799bee1ce7 on the Hurd, as > > FD_SETSIZE > > is only 256, and so anyone using XFD_SETSIZE to iterate over the contents of > > an

Re: [PATCH xserver 1/3] xfree86: factor out the check priviliges and print a big warning

On Sun, 2016-05-01 at 18:34 +0100, Emil Velikov wrote: > Ping ? This faithfully preserves the original behavior, but... > > +static void > > +xf86CheckPrivs(const char *option, const char *arg, const char *path_file, > > +   const char *dfault) > > +{ > > +if (xf86PrivsElevated()

Re: [PATCH libpciaccess 1/2] vgaarb: add a the trailing NULL character on read(vgaarb_fd)

On Sun, 2016-05-01 at 18:33 +0100, Emil Velikov wrote: > Somewhat of an annual ping. Anyone, please ? remote: I: patch #43169 updated using rev af2fdf1bdc4532410f49fc0854ae4c0f9086cce9. remote: I: patch #43168 updated using rev 201d1a7623c83f611761f67d4411c3c266f8f37a. remote: I: 2 patch(es)

Re: [PATCH] edid-decode: parse displayid blocks

On Mon, 2016-05-02 at 08:29 +1000, Dave Airlie wrote: > From: Dave Airlie remote: I: patch #84932 updated using rev 34049859fe0d94173eb0bc0e358ce760a1472afc. remote: I: 1 patch(es) updated to state Accepted. To ssh://git.freedesktop.org/git/xorg/app/edid-decode

Re: [PATCH xserver 2/2] glamor: use the lowercase xnf.*alloc API

On 1 May 2016 at 21:24, Rémi Cardona wrote: > Le 17/04/2016 à 18:24, Emil Velikov a écrit : >> The current lower/upper case mix of the xnf API feels quite strange: >> >> ati/amdgpu/intel/nouveau... ddx seems to explicitly use the lower case >> xnfalloc as does the xserver (with

[PATCH][RESEND no2] xf86-video-dummy: fix pointer limits when resizing

Hi, This patch seems to have got lost, so I am resending it, again. This fixes a real bug. Comments? Pointer events are clipped to the original screen dimensions with the dummy driver when using RandR to switch to a higher resolution at runtime. I don't know if this is the correct fix for