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

This is an ABI break, in that we now pass NULL to a function that hasn't accepted it before. Alex Goins had a different patch for this but it wasn't symmetrical, it freed something in a very different place than it allocated it, this attempts to retain symmetry in the releasing of the backing bo.

Re: [PATCH v5 00/13] PRIME Synchronization

On 13 April 2016 at 14:17, Alex Goins wrote: > Hello all, > > These patches change the xserver to support setting up PRIME with double > buffering, and implement double buffered PRIME sink and source support in > the modesetting driver. In addition to these changes, I've

Re: [PATCH 6/6] prime: clean up slave bo properly.

> This is an ABI break, in that we now pass NULL to a function > that hasn't accepted it before. > > Alex Goins had a different patch for this but it wasn't symmetrical, > it freed something in a very different place than it allocated it, > this attempts to retain symmetry in the releasing of the

[PATCH 5/6] modesetting: set capabilities up after glamor and enable offload caps.

This moves the capabilites setting to after glamor is initialised, and enables the offload caps in cases where they work. This enables DRI2 PRIME support with modesetting. Signed-off-by: Dave Airlie --- hw/xfree86/drivers/modesetting/driver.c | 28

[PATCH 1/6] modesetting: Consistent whitespace in driver.c

From: agoins For some reason a couple of the dirty functions in driver.c used 8 spaces per tab instead of 4 like the rest of the file. Fix this to make it more consistent and give me more room to work in ms_dirty_update in subsequent commits. v1: N/A v2: N/A v3: N/A v4:

[PATCH 6/6] prime: clean up slave bo properly.

This is an ABI break, in that we now pass NULL to a function that hasn't accepted it before. Alex Goins had a different patch for this but it wasn't symmetrical, it freed something in a very different place than it allocated it, this attempts to retain symmetry in the releasing of the backing bo.

[PATCH 2/6] randr/provider: only allow slave gpu to be offload sources.

The other way around makes no sense. Signed-off-by: Dave Airlie --- randr/rrprovider.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/randr/rrprovider.c b/randr/rrprovider.c index bbb8e51..5329f41 100644 --- a/randr/rrprovider.c +++ b/randr/rrprovider.c @@ -325,6

[PATCH 4/6] xf86Crtc: don't set the root window property on slave GPUs.

Slave GPUs don't have a root window to set this on, so don't. This fixes some crashes I saw just playing around. Signed-off-by: Dave Airlie --- hw/xfree86/modes/xf86Crtc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/hw/xfree86/modes/xf86Crtc.c

[PATCH 3/6] modesetting: set driverPrivate to NULL after closing fd.

Otherwise ms_ent_priv will return NULL and things will fall apart. Signed-off-by: Dave Airlie --- hw/xfree86/drivers/modesetting/driver.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/xfree86/drivers/modesetting/driver.c

Re: [PATCH] glamor: add glamor_finish API

On 12 March 2016 at 09:59, Eric Anholt wrote: > Dave Airlie writes: > >> From: Dave Airlie >> >> Some drivers are calling glFinish, they really should be doing this. >> >> This also is needed for some reverse prime scenarios. >> >>

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

On Thu, Apr 28, 2016 at 06:03:05PM -0700, Alan Coopersmith wrote: > 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 like: > > > > #define _xlib_used_internally

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

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 like: #define _xlib_used_internally __attribute__((unused)) and then use that to annotate the variable. No need for fake code

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

On Thu, Apr 28, 2016 at 04:53:46PM +0200, walter harms wrote: > > > Am 27.04.2016 07:04, schrieb Alan Coopersmith: > > On 04/26/16 07:31 AM, walter harms wrote: > >> > >> fix signess warning by casting > >> Signed-off-by: walter harms > >> > >> > >> --- > >> src/ModMap.c | 4

Re: [PATCH libICE] Enable visibility annotations

On 28/04/2016 14:59, Emil Velikov wrote: Are you sure that think this is correct ? Afaict things should work, despite that xserver sets SYMBOL_VISIBILITY=no. Jon, any ideas why xserver disables -fvisibility=hidden ? Should we remove that line or there's something subtle that requires the

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

Am 27.04.2016 07:04, schrieb Alan Coopersmith: > On 04/26/16 07:31 AM, walter harms wrote: >> >> fix signess warning by casting >> Signed-off-by: walter harms >> >> >> --- >> src/ModMap.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git

Re: [PATCH libICE] Enable visibility annotations

On 04/28/2016 04:59 PM, Emil Velikov wrote: On 27 April 2016 at 08:48, Yury Gribov wrote: On 04/26/2016 02:14 PM, Emil Velikov wrote: On 26 April 2016 at 07:54, Yury Gribov wrote: On 04/20/2016 10:03 AM, Yury Gribov wrote: On 04/19/2016 06:35

Re: [PATCH xserver] ad hoc fix for mmap's truncated offset parameter on 32bit

> From: Stefan Dirsch > Date: Thu, 28 Apr 2016 14:35:00 +0200 > > Builtin modesetting driver didn't work on 32bit on cirrus KMS. See > https://bugzilla.suse.com/show_bug.cgi?id=917385 for more details. I think selectively compiling with -D_FILE_OFFSET_BITS=64 is a seriously

Re: [PATCH libICE] Enable visibility annotations

On 27 April 2016 at 08:48, Yury Gribov wrote: > On 04/26/2016 02:14 PM, Emil Velikov wrote: >> >> On 26 April 2016 at 07:54, Yury Gribov wrote: >>> >>> On 04/20/2016 10:03 AM, Yury Gribov wrote: On 04/19/2016 06:35 PM, Yury Gribov wrote:

[PATCH xserver] XDMCP: For IPv6 add IPv6 link local addresses to the end of the list

From: Reinhard Max For IPv6 add a link local addresses to the end of the list passed to the XDMCP servers. Reason: for link local addresses the XDMCP server would need to either know the interface thru a scope identifier or try all available interfaces. If they don't this address

Re: [PATCH] uxa: fix missing includes for fstat

On Thu, Apr 28, 2016 at 11:31:51AM +0200, Julien Cristau wrote: > On Thu, Apr 28, 2016 at 11:06:38 +0200, Stefan Dirsch wrote: > > > On Wed, Apr 27, 2016 at 11:23:10PM +0100, Emil Velikov wrote: > > > P.S. For xorg-devel patches please set the correct prefix $ git config > > > --local

[PATCH xserver] Change include order to avoid conflict with system header

From: Andreas Schwab R_SP is also defined in on m68k. Also remove duplicate definitions. Signed-off-by: Andreas Schwab --- hw/xfree86/int10/xf86x86emu.c | 2 +- hw/xfree86/x86emu/x86emu/regs.h | 8 2 files changed, 1 insertion(+), 9 deletions(-)

Re: [PATCH] fix typo in supported sample sizes and maximum bit rate

On Wed, Apr 27, 2016 at 07:23:15PM +, Reinhard Zitzmann wrote: > git://anongit.freedesktop.org/xorg/app/edid-decode master > You should always CC: the mailing list, so everyone can see the answer. Reviewed-by: Eric Engestrom Thanks for the patch :] > On Tue,

[PATCH xserver] ad hoc fix for mmap's truncated offset parameter on 32bit

Builtin modesetting driver didn't work on 32bit on cirrus KMS. See https://bugzilla.suse.com/show_bug.cgi?id=917385 for more details. Signed-off-by: Stefan Dirsch --- hw/xfree86/drivers/modesetting/dumb_bo.c | 6 ++ 1 file changed, 6 insertions(+) diff --git

[PATCH vesa] Use default refresh by default

This seems to be a feature that isn't tested well. Too many VBE 3.0 BIOSes are incorrectly implemented. Signed-off-by: Stefan Dirsch --- src/vesa.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/vesa.c b/src/vesa.c index 5b9ccb0..e51effe 100644

Re: [xserver-xorg][PATCH 1/1] xwayland-shm: block signals during fallocate

On Thu, 28 Apr 2016 12:49:45 +0300 Ian Ray wrote: > Block signals during the call to posix_fallocate, because that api does > an explicit rollback if it gets EINTR and on slow systems (particularly > with large allocation sizes) the smart scheduler's SIGALRM will prevent >

[xserver-xorg][PATCH 1/1] xwayland-shm: block signals during fallocate

Block signals during the call to posix_fallocate, because that api does an explicit rollback if it gets EINTR and on slow systems (particularly with large allocation sizes) the smart scheduler's SIGALRM will prevent posix_fallocate from ever successfully completing. Signed-off-by: Ian Ray

Re: [PATCH] uxa: fix missing includes for fstat

On Thu, Apr 28, 2016 at 11:06:38 +0200, Stefan Dirsch wrote: > On Wed, Apr 27, 2016 at 11:23:10PM +0100, Emil Velikov wrote: > > P.S. For xorg-devel patches please set the correct prefix $ git config > > --local format.subjectPrefix "PATCH foo". With foo being the > > respective project. > > Ok.

Re: [PATCH] uxa: fix missing includes for fstat

On Wed, Apr 27, 2016 at 11:23:10PM +0100, Emil Velikov wrote: > On 26 April 2016 at 15:07, Stefan Dirsch wrote: > > On Tue, Apr 26, 2016 at 01:51:39PM +0100, Chris Wilson wrote: > >> On Tue, Apr 26, 2016 at 02:42:52PM +0200, Stefan Dirsch wrote: > >> > From: Dominique

Results of the 2016 Election to the X.Org BoD & Vote on the By-Law Changes

The 2016 Election is now over and the results are in. Two questions were up for voting, 4 seats on the Board of Directors and approval of the amended By-Laws to join SPI. The Results of the Board of Director elections: Candidates and their respective points: Egbert Eich 205