[Mesa-dev] [PATCH 3/4] st/mesa: add ARB_texture_stencil8 enable

2015-04-04 Thread Dave Airlie
if we support stencil texturing, enable texture_stencil8 there is no requirement to support native S8 for this, the texture can be converted to x24s8 fine. Signed-off-by: Dave Airlie --- src/mesa/state_tracker/st_extensions.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --g

[Mesa-dev] [PATCH 4/4] docs: mark off texture_stencil8

2015-04-04 Thread Dave Airlie
add it to the GL3.txt, I've only enabled it for r600, but I think it should be fine on radeonsi, nv50, nvc0 and I think i965 enabling should only be a line or two. Signed-off-by: Dave Airlie --- docs/GL3.txt | 2 +- docs/relnotes/10.6.0.html | 1 + 2 files changed, 2 insertions(+),

[Mesa-dev] [PATCH 1/4] mesa/teximage: use correct extension for accept stencil texture.

2015-04-04 Thread Dave Airlie
This was using the wrong extension, ARB_stencil_texturing doesn't mention any changes in this area. Signed-off-by: Dave Airlie --- src/mesa/main/teximage.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c index 8d9d7cf..36e0b

[Mesa-dev] [PATCH 2/4] mesa: finish implementing ARB_texture_stencil8

2015-04-04 Thread Dave Airlie
Parts of this were implemented previously, so finish it off. Signed-off-by: Dave Airlie --- src/mesa/main/extensions.c | 1 + src/mesa/main/teximage.c | 3 ++- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c index f212015..e

Re: [Mesa-dev] [PATCH 4/4] mesa: Remove the _WindowMap from gl_viewport_attrib.

2015-04-04 Thread Dave Airlie
On 31 March 2015 at 16:37, wrote: > From: Mathias Froehlich > > The _WindowMap can be dropped from gl_viewport_attrib now. > Simplify gl_viewport_attrib handling where possible. You seemed to have left some warnings behind after this. unused variables for scale and translate. Dave. __

Re: [Mesa-dev] [PATCH 3/4] tnl: Maintain the _WindowMap matrix in TNLcontext v2.

2015-04-04 Thread Mathias Fröhlich
Hi Brian, > > The Patch with unneeded parentheses removed. > > Ok, to push? > > > > Yes. Just pushed, thanks again for the review! Mathias ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH] i965: Implement support for ARB_clip_control.

2015-04-04 Thread Mathias Fröhlich
Hi Kenneth, > Fine by me - thanks again! Pushed, thanks for review! Mathias ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH 1/3] i965: Fix instanced geometry shaders on Gen8+.

2015-04-04 Thread Kenneth Graunke
On Saturday, April 04, 2015 03:05:11 PM Jordan Justen wrote: > On 2015-04-04 03:46:31, Kenneth Graunke wrote: > > Jordan added this in commit 741782b5948bb3d01d699f062a37513c2e73b076 for > > Gen7 platforms. Embarassingly, this was missed for well over a year. > > :( I'm not sure why I didn't enab

Re: [Mesa-dev] [PATCH 1/3] i965: Fix instanced geometry shaders on Gen8+.

2015-04-04 Thread Matt Turner
On Sat, Apr 4, 2015 at 3:46 AM, Kenneth Graunke wrote: > Jordan added this in commit 741782b5948bb3d01d699f062a37513c2e73b076 for > Gen7 platforms. Embarassingly, this was missed for well over a year. Embarrassingly has two r's. ___ mesa-dev mailing li

Re: [Mesa-dev] [PATCH 1/3] i965: Fix instanced geometry shaders on Gen8+.

2015-04-04 Thread Matt Turner
On Sat, Apr 4, 2015 at 3:05 PM, Jordan Justen wrote: > On 2015-04-04 03:46:31, Kenneth Graunke wrote: >> Jordan added this in commit 741782b5948bb3d01d699f062a37513c2e73b076 for >> Gen7 platforms. Embarassingly, this was missed for well over a year. > > :( I'm not sure why I didn't enable gen8 at

Re: [Mesa-dev] [PATCH 4/4] i965: Implement SIMD16 texturing on Gen4.

2015-04-04 Thread Jordan Justen
On 2015-04-04 01:23:28, Kenneth Graunke wrote: > This allows SIMD16 mode to work for a lot more programs. Texturing is > also more efficient in SIMD16 mode than SIMD8. Several messages don't > actually exist in SIMD8 mode, so we did SIMD16 messages and threw away > half of the data. Now we compu

Re: [Mesa-dev] [PATCH 1/3] i965: Fix instanced geometry shaders on Gen8+.

2015-04-04 Thread Jordan Justen
On 2015-04-04 03:46:31, Kenneth Graunke wrote: > Jordan added this in commit 741782b5948bb3d01d699f062a37513c2e73b076 for > Gen7 platforms. Embarassingly, this was missed for well over a year. :( I'm not sure why I didn't enable gen8 at the same time back then. Reviewed-by: Jordan Justen > Fix

Re: [Mesa-dev] [Mesa-stable] [PATCH 1/3] i965: Fix instanced geometry shaders on Gen8+.

2015-04-04 Thread Chris Forbes
For the series: Reviewed-by: Chris Forbes On Sat, Apr 4, 2015 at 11:46 PM, Kenneth Graunke wrote: > Jordan added this in commit 741782b5948bb3d01d699f062a37513c2e73b076 for > Gen7 platforms. Embarassingly, this was missed for well over a year. > > Fixes Piglit's spec/arb_gpu_shader5/invocation

[Mesa-dev] [Bug 89905] scons build broken on 10.5.2 due to activated vega st

2015-04-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=89905 --- Comment #3 from Alexander von Gluck --- Thanks Emil. Yeah.. I originally did the port for scons as it is *much* easier to wrangle vs autotools in big projects. Anyway, if it isn't planned for scons to become the main buildsystem I should do

[Mesa-dev] [Bug 89905] scons build broken on 10.5.2 due to activated vega st

2015-04-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=89905 --- Comment #2 from Emil Velikov --- I see what happened here - I checked the automake build with `make distcheck', but did the scons one only in-tree. So include/VG and src/mapi/mapi.[ch] were around and everything worked like a charm. I'll add

[Mesa-dev] [Bug 89905] scons build broken on 10.5.2 due to activated vega st

2015-04-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=89905 --- Comment #1 from Alexander von Gluck --- Created attachment 114865 --> https://bugs.freedesktop.org/attachment.cgi?id=114865&action=edit patch these fixes should likely be cherry-picked for 10.5.3 as they already exist in master. -- You a

[Mesa-dev] [Bug 89905] scons build broken on 10.5.2 due to activated vega st

2015-04-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=89905 Bug ID: 89905 Summary: scons build broken on 10.5.2 due to activated vega st Product: Mesa Version: 10.5 Hardware: All OS: All Status: NEW Severity: major

[Mesa-dev] [PATCH 1/2] gallivm: don't use control flow when doing indirect constant buffer lookups

2015-04-04 Thread sroland
From: Roland Scheidegger llvm goes crazy when doing that, using way more memory and time, though there's probably more to it - this points to a very much similar issue as fixed in 8a9f5ecdb116d0449d63f7b94efbfa8b205d826f. In any case I've seen a quite plain looking vertex shader with just ~50 sim

[Mesa-dev] [PATCH 2/2] draw: (trivial) don't print the shader twice with GALLIVM_DEBUG=tgsi (or ir)

2015-04-04 Thread sroland
From: Roland Scheidegger Neither the shader nor the key change when doing elts or linear variant, so this was just annoying (probably mildly useful at some point when we printed the IR per function too). --- src/gallium/auxiliary/draw/draw_llvm.c | 10 +- 1 file changed, 5 insertions(+),

Re: [Mesa-dev] [PATCH] nv50: allocate more offset space for occlusion queries

2015-04-04 Thread Samuel Pitoiset
Reviewed-by: Samuel Pitoiset But please, fix the comment in the nv50_query struct, now it is "/* base + i * 32 */". On 04/04/2015 06:00 AM, Ilia Mirkin wrote: Commit 1a170980a09 started writing to q->data[4]/[5] but kept the per-query space at 16, which meant that in some cases we would writ

[Mesa-dev] [PATCH] nir: split out lower_sub from lower_negate

2015-04-04 Thread Rob Clark
From: Rob Clark Originally you had to have one or the other. But actually I don't want either. (Or rather I want whatever is the minimum # of instructions.) TODO: not sure where the best place to insert a check that driver hasn't set *both* lower_negate and lower_sub? Signed-off-by: Rob Clark

[Mesa-dev] [PATCH 3/3] i965: Make sure GS output streams 1-3 aren't rasterized on Haswell+.

2015-04-04 Thread Kenneth Graunke
Presumably no one has noticed this breakage because ARB_transform_feedback3 isn't exposed on Haswell due to the ongoing command streamer shenanigans, and ARB_gpu_shader5 isn't exposed on Broadwell because we keep forgetting about it. Fixes 3 Piglit tests: - spec/arb_gpu_shader5/emitstreamvertex_no

[Mesa-dev] [PATCH 1/3] i965: Fix instanced geometry shaders on Gen8+.

2015-04-04 Thread Kenneth Graunke
Jordan added this in commit 741782b5948bb3d01d699f062a37513c2e73b076 for Gen7 platforms. Embarassingly, this was missed for well over a year. Fixes Piglit's spec/arb_gpu_shader5/invocation-id-{basic,in-separate-gs} with MESA_EXTENSION_OVERRIDE=GL_ARB_gpu_shader5 set. Signed-off-by: Kenneth Graun

[Mesa-dev] [PATCH 2/3] i965: Add forgotten multi-stream code to Gen8 SOL state.

2015-04-04 Thread Kenneth Graunke
Fixes Piglit's arb_gpu_shader5-xfb-streams-without-invocations. Signed-off-by: Kenneth Graunke Cc: mesa-sta...@lists.freedesktop.org --- src/mesa/drivers/dri/i965/gen8_sol_state.c | 9 + 1 file changed, 9 insertions(+) diff --git a/src/mesa/drivers/dri/i965/gen8_sol_state.c b/src/mesa/

[Mesa-dev] [PATCH 1/4] i965/fp: Set coord_components correctly for cube textures.

2015-04-04 Thread Kenneth Graunke
I've no idea why this was 4. It certainly seems wrong. Prevents assertion failures in fp-incomplete-tex with some upcoming patches of mine. Signed-off-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_fs_fp.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) This series is available

[Mesa-dev] [PATCH 4/4] i965: Implement SIMD16 texturing on Gen4.

2015-04-04 Thread Kenneth Graunke
This allows SIMD16 mode to work for a lot more programs. Texturing is also more efficient in SIMD16 mode than SIMD8. Several messages don't actually exist in SIMD8 mode, so we did SIMD16 messages and threw away half of the data. Now we compute real data in both halves. Also, the SIMD16 "sample"

[Mesa-dev] [PATCH 2/4] i965: Respect the no_8 flag on Gen4-5.

2015-04-04 Thread Kenneth Graunke
This flag means to ignore the SIMD8 program and only use the SIMD16 one. It was originally meant for repdata clear shaders, but I plan to use it for other things on Gen4 as well. Signed-off-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_wm_state.c | 55 1

[Mesa-dev] [PATCH 3/4] i965: Use SIMD16 instead of SIMD8 on Gen4 when possible.

2015-04-04 Thread Kenneth Graunke
Gen5+ systems allow you to specify multiple shader programs - both SIMD8 and SIMD16 - and the hardware will automatically dispatch to the most appropriate one, given the number of subspans to be processed. However, that is not the case on Gen4. Instead, you program a single shader. If you enable

Re: [Mesa-dev] [PATCH 1/2] egl/dri2: implement platform_null (v2).

2015-04-04 Thread Jordan Justen
On 2015-04-03 19:18:35, Stéphane Marchesin wrote: > On Fri, Apr 3, 2015 at 1:35 PM, Chad Versace wrote: > > On Tue 17 Feb 2015, Haixia Shi wrote: > >> The NULL platform is for off-screen rendering only. Render node support is > >> required. > > > > Naming it the "NULL" platform seems very odd. It

Re: [Mesa-dev] [PATCH 2/2] xa: support for drivers which use NIR

2015-04-04 Thread Thomas Hellstrom
lgtm. Could you just add a small comment for future reference in in xa_tracker.c that the function _mesa_error_no_memory is used by NIR? Thanks, Thomas On 04/03/2015 08:07 PM, Rob Clark wrote: > From: Rob Clark > > We need to pull in libnir.la and it's dependency libglsl_util.la. Also, >

Re: [Mesa-dev] [PATCH 16/16] android: mesa: fix the path of the SSE4_1 optimisations

2015-04-04 Thread Chih-Wei Huang
2015-03-29 4:54 GMT+08:00 Emil Velikov : > Commit dd6f641303c(mesa: Build with subdir-objects.) removed the SRCDIR > variable, but forgot to update all references of it. > > Cc: "10.5" > Signed-off-by: Emil Velikov > --- > src/mesa/Android.libmesa_dricore.mk | 4 ++-- > 1 file changed, 2 inserti