[PATCH xserver] loader: Handle mod->VersionInfo == NULL

2017-01-25 Thread Michel Dänzer
From: Michel Dänzer This can happen when a module fails to load: Program received signal SIGSEGV, Segmentation fault. UnloadModule (_mod=0x559d9280) at ../../../../hw/xfree86/loader/loadmod.c:848 848 name = mod->VersionInfo->modname; (gdb) bt #0

Re: [PATCH modules v2] autogen: add default patch prefix

2017-01-25 Thread Peter Hutterer
On Fri, Oct 14, 2016 at 12:06:52AM +, Mihail Konev wrote: > Took correct wording from libxtrans. > Removed obsolete patches. all applied (I think). And only a few months late... :) I used your script to regenerate all of them because other commits changed the context. Thanks! Cheers,

Re: [PATCH 0/3 all repositories] autogen.sh: sync with server

2017-01-25 Thread Peter Hutterer
On Tue, Jan 24, 2017 at 10:40:11AM +1000, Peter Hutterer wrote: > Heads up: I'll be pushing the following three patches into all xorg > repositories that need it in the next few days. Speak up now or be forever > silent, etc. etc. Ok, done. Please check your favourite repositories to make sure I

Re: [PATCH 0/3 all repositories] autogen.sh: sync with server

2017-01-25 Thread Peter Hutterer
On Wed, Jan 25, 2017 at 03:22:18AM +, Emil Velikov wrote: > On 25 January 2017 at 02:26, Michel Dänzer wrote: > > On 24/01/17 11:32 PM, Adam Jackson wrote: > >> On Tue, 2017-01-24 at 10:40 +1000, Peter Hutterer wrote: > >>> Heads up: I'll be pushing the following three

Re: [PATCH xserver] dri1: Remove some dead event code

2017-01-25 Thread Michel Dänzer
On 26/01/17 05:08 AM, Adam Jackson wrote: > Signed-off-by: Adam Jackson > --- > hw/xfree86/dri/xf86dri.c | 7 --- > 1 file changed, 7 deletions(-) > > diff --git a/hw/xfree86/dri/xf86dri.c b/hw/xfree86/dri/xf86dri.c > index 68f8b7e..fcae73d 100644 > ---

Re: EXA vs. weirdo graphics chips ( was Re: [PATCH and additional rambling] cache glyphs in the destination format requested to make sure the hardware can use the cached glyphs )

2017-01-25 Thread Michel Dänzer
On 26/01/17 06:47 AM, Michael wrote: > Hello, > > On Wed, 18 Jan 2017 11:24:03 +0900 > Michel Dänzer wrote: > > The Creator is weirder - it's got what Sun calls '3dRAM' - little ALUs > built into the memory chips which can do things like ROPs and alpha >

Re: [RFC] Visual Class for On-Screen HDR Drawables

2017-01-25 Thread Alex Goins
>Either we define the interaction with Render, or automatic redirection >(and backing store) don't work. But again we're painted into a bit of a >corner: > >typedef struct { >CARD16 red B16; >CARD16 redMask B16; >CARD16 green B16; >CARD16 greenMask B16; >

EXA vs. weirdo graphics chips ( was Re: [PATCH and additional rambling] cache glyphs in the destination format requested to make sure the hardware can use the cached glyphs )

2017-01-25 Thread Michael
Hello, On Wed, 18 Jan 2017 11:24:03 +0900 Michel Dänzer wrote: > >>> The Creator is weirder - it's got what Sun calls '3dRAM' - little ALUs > >>> built into the memory chips which can do things like ROPs and alpha > >>> blending. So basically you can set up a few registers,

Re: [PATCH xserver] tests: shuffle around the linking order to please GNU ld

2017-01-25 Thread Mihail Konev
On Wed, Jan 25, 2017 at 09:28:21AM +0500, Mihail Konev wrote: > (The static library was acting like ld's --begin-group ... --end-group > for all its LIBADD-s). Wrong, this would only apply to .a libraries; but *_SYS_LIBS in question are all "-l*"-s. Instead, libtool was remembering the args in

Re: [PATCH v2 xserver 3/4] dri2: refine dri2_probe_driver_name (v2)

2017-01-25 Thread Adam Jackson
On Mon, 2017-01-23 at 06:30 +, Yu, Qiang wrote: > Hi, > > If no further comments, can this patch be merged? remote: I: patch #132425 updated using rev 7617a0a180a2cd3427a8ffa9534152df6a8fecbf. remote: I: 1 patch(es) updated to state Accepted. To ssh://git.freedesktop.org/git/xorg/xserver

[PATCH xserver] dri1: Remove some dead event code

2017-01-25 Thread Adam Jackson
Signed-off-by: Adam Jackson --- hw/xfree86/dri/xf86dri.c | 7 --- 1 file changed, 7 deletions(-) diff --git a/hw/xfree86/dri/xf86dri.c b/hw/xfree86/dri/xf86dri.c index 68f8b7e..fcae73d 100644 --- a/hw/xfree86/dri/xf86dri.c +++ b/hw/xfree86/dri/xf86dri.c @@ -599,14 +599,7 @@

Re: [PATCH xserver v2] tests: shuffle around the linking order to please GNU ld

2017-01-25 Thread Adam Jackson
On Wed, 2017-01-25 at 17:29 +0200, Martin Peres wrote: > Classic GNU ld resolves symbol dependencies only forward, while GOLD > seems to work regardless of the specified library order. > > Suggested-by: Eero Tamminen > Reviewed-by: Mihail Konev >

Re: [PATCH xserver 00/19] Module loader cleanup

2017-01-25 Thread Adam Jackson
On Tue, 2017-01-24 at 00:15 +0100, Julien Cristau wrote: > On Mon, Jan 23, 2017 at 14:32:14 -0500, Adam Jackson wrote: > > > I'd like to move the module loader up to dix. In preparation for > > that, here's > > a bunch of cleanup patches. The first three aren't mine, I just > > think they're > >

Re: [PATCH xserver 06/19] loader: Remove *GetOS

2017-01-25 Thread Adam Jackson
On Mon, 2017-01-23 at 23:44 +0100, Julien Cristau wrote: > On Mon, Jan 23, 2017 at 14:32:20 -0500, Adam Jackson wrote: > > > This API is dumb.  uname(3) exists, feel free to use it, but ideally > > write to the interface not to the OS.  There are a couple of drivers > > using this API, they could

Re: [PATCH xserver 4/4] glx: Move Composite's synthetic visuals to a different select group

2017-01-25 Thread Adam Jackson
On Tue, 2017-01-24 at 11:41 -0800, Eric Anholt wrote: > Nice fix.  The series is: > > Reviewed-by: Eric Anholt remote: I: patch #134086 updated using rev bccbaf7c113b1300071601879002682ebbe8b1c1. remote: I: patch #134087 updated using rev

Re: [PATCH xserver 11/19] xfree86: Remove DriverRec1 compat struct

2017-01-25 Thread Aaron Plattner
On 01/24/2017 04:36 PM, Emil Velikov wrote: On 23 January 2017 at 19:32, Adam Jackson wrote: The idea here is that the driver might have once been old enough to not have the driverFunc slot in DriverRec, with the module ABI not having changed when it was added. That was ages

Re: [PATCH 0/3 all repositories] autogen.sh: sync with server

2017-01-25 Thread Adam Jackson
On Wed, 2017-01-25 at 11:26 +0900, Michel Dänzer wrote: > That would be incorrect for xf86-video-{ati,amdgpu}, for which > patches > are reviewed on the amd-gfx mailing list, where "[PATCH amdgpu]" is > ambiguous (because the same list is also used for reviewing kernel > driver and libdrm

[PATCH xserver v2] tests: shuffle around the linking order to please GNU ld

2017-01-25 Thread Martin Peres
Classic GNU ld resolves symbol dependencies only forward, while GOLD seems to work regardless of the specified library order. Suggested-by: Eero Tamminen Reviewed-by: Mihail Konev Signed-off-by: Martin Peres --- Thanks for

Re: glamor woes with nouveau

2017-01-25 Thread Olivier Fourdan
Hey! So I was wrong (again), this is not glamor_composite_choose_shader() failed which is the problem, the rendering issue comes from glamor_poly_fill_rect_gl() (the plain ackground is garbled, not the actual content, if that makes any sense). If I force glamor_poly_fill_rect_bail() from

Re: [PATCH xserver] tests: shuffle around the linking order to please GNU ld

2017-01-25 Thread Mihail Konev
On Wed, Jan 25, 2017 at 09:28:21AM +0500, Mihail Konev wrote: > On Mon, Jan 23, 2017 at 07:15:46PM +0200, Martin Peres wrote: > > Classic GNU ld resolves symbol dependencies only forward, while GOLD > > seems to work regardless of the specified library order. > > > > I assume that the original