[PATCH 1/1] Initialize pci_dev

2016-01-21 Thread Pascal Terjan
Else it is read before initialization if XSERVER_LIBPCIACCESS is not defined or location.type is not BUS_PCI. Noticed when I got a segfault where it was 0x1 while doing some tests on arm. Signed-off-by: Pascal Terjan --- src/fbdev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --

Re: [PATCH xserver 0/3] Fixups for GL_RED patch.

2016-01-21 Thread Dave Airlie
On 22 January 2016 at 12:36, Eric Anholt wrote: > Here are my fixes for the regressions in the glamor-core-profile > series. The first commit would go just before the GL_RED change, then > I'd squash the other two into GL_RED. The series is now in > glamor-core-profile of my fdo tree. all 3 of

Re: [PATCH xserver 1/2] glamor: Fix copy-like Render operations between 15 and 16 depth.

2016-01-21 Thread Keith Packard
Eric Anholt writes: > if (op == PictOpSrc) { > +/* We can't do direct copies between different depths at 16bpp > + * because r,g,b are allocated to different bits. > + */ > +if (dst->pDrawable->bitsPerPixel == 16 && > +dst->pDrawable->depth != src-

[PATCH xserver 3/3] squash: glamor: Fix Render blending for alpha-to-red.

2016-01-21 Thread Eric Anholt
Signed-off-by: Eric Anholt --- glamor/glamor_render.c | 36 1 file changed, 32 insertions(+), 4 deletions(-) diff --git a/glamor/glamor_render.c b/glamor/glamor_render.c index c36b345..51718d1 100644 --- a/glamor/glamor_render.c +++ b/glamor/glamor_render.c @

[PATCH xserver 1/3] glamor: Drop duplicated GLAMOR_DEFAULT_PRECISIONs in render accel.

2016-01-21 Thread Eric Anholt
We only need it once at the top of the shader, so just put it there. Signed-off-by: Eric Anholt --- glamor/glamor_render.c | 17 - 1 file changed, 4 insertions(+), 13 deletions(-) diff --git a/glamor/glamor_render.c b/glamor/glamor_render.c index e4beafc..7a50fe9 100644 --- a/gl

[PATCH xserver 2/3] squash: glamor: Fix rendering to a8 textures in core profile.

2016-01-21 Thread Eric Anholt
Signed-off-by: Eric Anholt --- glamor/glamor_priv.h | 10 +- glamor/glamor_render.c | 50 +++--- 2 files changed, 52 insertions(+), 8 deletions(-) diff --git a/glamor/glamor_priv.h b/glamor/glamor_priv.h index 558ed63..f1eed5b 100644 --- a/gl

[PATCH xserver 0/3] Fixups for GL_RED patch.

2016-01-21 Thread Eric Anholt
Here are my fixes for the regressions in the glamor-core-profile series. The first commit would go just before the GL_RED change, then I'd squash the other two into GL_RED. The series is now in glamor-core-profile of my fdo tree. Eric Anholt (3): glamor: Drop duplicated GLAMOR_DEFAULT_PRECISIO

Re: [PATCH libX11 v3] XKB: fix XkbGetKeyboardByName with Xming server

2016-01-21 Thread Peter Hutterer
On Thu, Jan 21, 2016 at 11:54:19AM +0100, Olivier Fourdan wrote: > XkbGetKeyboardByName relies on flags to read the data from the server. > > If the X server sends us the wrong flags or if a subreply is smaller > than it should be, XkbGetKeyboardByName will not read all the available > data and le

Re: [PATCH xserver 2/2] glamor: Drop the composite_with_copy path entirely.

2016-01-21 Thread Dave Airlie
On 22 January 2016 at 07:06, Eric Anholt wrote: > I originally inherited this from the EXA code, without determining > whether it was really needed. Regular composite should end up doing > the same thing, since it's all just shaders anyway. To the extent > that it doesn't, we should fix composit

Re: [PATCH xserver 1/2] glamor: Fix copy-like Render operations between 15 and 16 depth.

2016-01-21 Thread Dave Airlie
On 22 January 2016 at 07:06, Eric Anholt wrote: > Please cherry-pick this to active stable branches. > > Reading and writing to 16-depth pixmaps using PICT_x1r5g5b5 ends up > failing, unless you're doing a straight copy at the same bpp where the > misinterpretation matches on both sides. > > Fixes

[PATCH xserver 2/2] glamor: Drop the composite_with_copy path entirely.

2016-01-21 Thread Eric Anholt
I originally inherited this from the EXA code, without determining whether it was really needed. Regular composite should end up doing the same thing, since it's all just shaders anyway. To the extent that it doesn't, we should fix composite. Signed-off-by: Eric Anholt --- glamor/glamor_render

[PATCH xserver 1/2] glamor: Fix copy-like Render operations between 15 and 16 depth.

2016-01-21 Thread Eric Anholt
Please cherry-pick this to active stable branches. Reading and writing to 16-depth pixmaps using PICT_x1r5g5b5 ends up failing, unless you're doing a straight copy at the same bpp where the misinterpretation matches on both sides. Fixes rendercheck/blend/over and renderhceck/blend/src in piglit.

Re: [PATCH xserver] glx: Fix GLX_EXT_create_context_es2_profile support

2016-01-21 Thread Adam Jackson
On Wed, 2016-01-20 at 15:50 -0500, Adam Jackson wrote: > As of v4 of this extension, any GLES version number may be requested (to > enable GLES3 and later). To comply with this, simply remove the API > version checks and leave it to the DRI driver to validate. This happens > to also enable using GL

[PATCH libX11 v3] XKB: fix XkbGetKeyboardByName with Xming server

2016-01-21 Thread Olivier Fourdan
XkbGetKeyboardByName relies on flags to read the data from the server. If the X server sends us the wrong flags or if a subreply is smaller than it should be, XkbGetKeyboardByName will not read all the available data and leave data in the buffer, which will cause the next _XReply() to fail with:

Re: [PATCH libX11 v2] XKB: fix XkbGetKeyboardByName with Xming server

2016-01-21 Thread Peter Hutterer
On Thu, Jan 21, 2016 at 10:48:05AM +0100, Olivier Fourdan wrote: > XkbGetKeyboardByName relies on flags to read the data from the server. > > If the X server sends us the wrong flags or if a subreply is smaller > than it should be, XkbGetKeyboardByName will not read all the available > data and le

Re: [PATCH libX11 v2] XKB: fix XkbGetKeyboardByName with Xming server

2016-01-21 Thread Daniel Stone
On 21 January 2016 at 09:48, Olivier Fourdan wrote: > XkbGetKeyboardByName relies on flags to read the data from the server. > > If the X server sends us the wrong flags or if a subreply is smaller > than it should be, XkbGetKeyboardByName will not read all the available > data and leave data in t

[PATCH libX11 v2] XKB: fix XkbGetKeyboardByName with Xming server

2016-01-21 Thread Olivier Fourdan
XkbGetKeyboardByName relies on flags to read the data from the server. If the X server sends us the wrong flags or if a subreply is smaller than it should be, XkbGetKeyboardByName will not read all the available data and leave data in the buffer, which will cause the next _XReply() to fail with:

Re: [PATCH libX11] XKB: fix XkbGetKeyboardByName with Xming server

2016-01-21 Thread Daniel Stone
Hi, On 21 January 2016 at 08:58, Olivier Fourdan wrote: > XkbGetKeyboardByName relies on flags to read the data from the server. > > If the X server sends us the wrong flags or if a subreply is smaller > than it should be, XkbGetKeyboardByName will not read all the available > data and leave data

Re: [PATCH libX11] XKB: fix XkbGetKeyboardByName with Xming server

2016-01-21 Thread Olivier Fourdan
+ Peter of course! On 21 January 2016 at 09:58, Olivier Fourdan wrote: > XkbGetKeyboardByName relies on flags to read the data from the server. > > If the X server sends us the wrong flags or if a subreply is smaller > than it should be, XkbGetKeyboardByName will not read all the available > dat

[PATCH libX11] XKB: fix XkbGetKeyboardByName with Xming server

2016-01-21 Thread Olivier Fourdan
XkbGetKeyboardByName relies on flags to read the data from the server. If the X server sends us the wrong flags or if a subreply is smaller than it should be, XkbGetKeyboardByName will not read all the available data and leave data in the buffer, which will cause the next _XReply() to fail with: