Re: [Mesa-dev] [Mesa-stable] New stable-branch 11.0 candidate pushed

2015-11-08 Thread Emil Velikov
[Dropping Jason, Ken, adding Roland to CC list] Hi Oded, On 8 November 2015 at 09:22, Oded Gabbay wrote: > Hi Emil, > > I would like to propose the following patch to be added to 11.0.5: > 39b4dfe6ab1003863778a25c091c080e098833ec llvmpipe: use simple coeffs > calc for

Re: [Mesa-dev] [PATCH mesa 0/5] nouveau: codegen: Make use of double immediates

2015-11-08 Thread Hans de Goede
Hi, On 07-11-15 01:59, Ilia Mirkin wrote: Hi Hans, All pushed. I made a few additional fixes and improvement to fp64 immediate handling along the way, but all your commits were fine as-is. (Except that they enabled fp64 immediates on nv50 implicitly which is wrong -- there are no

Re: [Mesa-dev] New stable-branch 11.0 candidate pushed

2015-11-08 Thread Emil Velikov
On 7 November 2015 at 19:18, Kenneth Graunke wrote: > On Saturday, November 07, 2015 04:37:05 PM Emil Velikov wrote: >> Jason, Kenneth, >> >> I had to pick a few extra commits as prerequirements to the "nir: >> Properly invalidate metadata in nir_foo" patches. Can you let

Re: [Mesa-dev] [Mesa-stable] New stable-branch 11.0 candidate pushed

2015-11-08 Thread Oded Gabbay
On Sun, Nov 8, 2015 at 1:39 PM, Emil Velikov wrote: > [Dropping Jason, Ken, adding Roland to CC list] > > Hi Oded, > > On 8 November 2015 at 09:22, Oded Gabbay wrote: > > > Hi Emil, > > > > I would like to propose the following patch to be added

Re: [Mesa-dev] [Mesa-stable] New stable-branch 11.0 candidate pushed

2015-11-08 Thread Emil Velikov
Hi Mark, On 7 November 2015 at 23:26, Mark Janes wrote: > Hi Emil, > > I get this regression testing the new branch: > > piglit.spec.oes_compressed_paletted_texture.basic api > > /tmp/build_root/m64/lib/piglit/bin/oes_compressed_paletted_texture-api -auto > -fbo > Trying

Re: [Mesa-dev] [PATCH 3/6] glsl: move layout qualifier validation out of the parser

2015-11-08 Thread Emil Velikov
On 6 November 2015 at 21:13, Timothy Arceri wrote: > On Fri, 2015-11-06 at 13:16 +, Emil Velikov wrote: >> On 5 November 2015 at 11:17, Timothy Arceri wrote: >> > From: Timothy Arceri >> > >> > This is in

Re: [Mesa-dev] [PATCH] st/va: add mpeg4 startcode workaround

2015-11-08 Thread Ilia Mirkin
On Sun, Nov 8, 2015 at 6:41 AM, Christian König wrote: > On 06.11.2015 20:28, Ilia Mirkin wrote: >> >> On Fri, Nov 6, 2015 at 2:15 PM, Christian König >> wrote: >>> >>> On 06.11.2015 20:10, Ilia Mirkin wrote: On Fri, Nov 6, 2015 at 1:48

Re: [Mesa-dev] [PATCH] android: fix LOCAL_C_INCLUDES to find glsl_types.h

2015-11-08 Thread Mauro Rossi
Hi, Sending an update because with the export android_x86 target builds ok, but I'm getting again the "glsl_types.h not found" building error with android_x86_64 target (specifically for 64 bit modules). I'll report as soon I may be able to understand what's going on, added other android-x86

Re: [Mesa-dev] [PATCH] st/va: add mpeg4 startcode workaround

2015-11-08 Thread Emil Velikov
On 8 November 2015 at 11:41, Christian König wrote: > On 06.11.2015 20:28, Ilia Mirkin wrote: >> >> On Fri, Nov 6, 2015 at 2:15 PM, Christian König >> wrote: >>> >>> On 06.11.2015 20:10, Ilia Mirkin wrote: On Fri, Nov 6, 2015 at 1:48

Re: [Mesa-dev] [Mesa-stable] New stable-branch 11.0 candidate pushed

2015-11-08 Thread Roland Scheidegger
I don't have anything against including it in stable (it is pretty harmless), albeit I didn't tag it because I didn't think it was important enough. While technically it makes things more accurate, it doesn't fix anything known except a couple of piglit tests. Roland Am 08.11.2015 um 12:39

[Mesa-dev] [PATCH v2 1/2] main: Don't restrict several KHR_debug enum to desktop GL

2015-11-08 Thread Boyan Ding
In preparation for supporting GL_KHR_debug in OpenGL ES v2: add a missing hunk in _mesa_IsEnabled (Emil) Signed-off-by: Boyan Ding Reviewed-by: Emil Velikov --- src/mesa/main/enable.c| 10 ++ src/mesa/main/getstring.c | 5 + 2

Re: [Mesa-dev] [PATCH 1/6] mesa: add ARB_enhanced_layouts

2015-11-08 Thread Emil Velikov
Hi Tim, On 6 November 2015 at 21:09, Timothy Arceri wrote: > On Fri, 2015-11-06 at 13:02 +, Emil Velikov wrote: >> Hi Tim, >> >> A few comments below >> >> On 5 November 2015 at 11:17, Timothy Arceri wrote: >> > From: Timothy Arceri

Re: [Mesa-dev] [PATCH] android: fix LOCAL_C_INCLUDES to find glsl_types.h

2015-11-08 Thread Emil Velikov
Hi Mauro, On 8 November 2015 at 12:59, Mauro Rossi wrote: > Update2: I'm getting the building error in both x86 target and x86_64 > target. > > I'm relieved it's not arch dependend, I suspect that export will require a > dependency to be declared, > because if i965_dri

Re: [Mesa-dev] The i965 vec4 backend, exec_masks, and 64-bit types

2015-11-08 Thread Connor Abbott
On Tue, Nov 3, 2015 at 8:04 PM, Francisco Jerez wrote: > Francisco Jerez writes: > >> Connor Abbott writes: >> >>> Hi all, >>> >>> While working on FP64 for i965, there's an issue that I thought of >>> with the vec4 backend that

[Mesa-dev] [PATCH 2/7] gallium/radeon: simplify disabling render condition for u_blitter

2015-11-08 Thread Marek Olšák
From: Marek Olšák just disable it by not setting the predication bit --- src/gallium/drivers/r600/r600_blit.c | 12 +--- src/gallium/drivers/r600/r600_state_common.c | 11 ++- src/gallium/drivers/radeon/r600_pipe_common.h | 3 ++-

[Mesa-dev] [PATCH 7/7] gallium/radeon: shorten render_cond variable names

2015-11-08 Thread Marek Olšák
From: Marek Olšák and ..._cond -> ..._invert --- src/gallium/drivers/r600/r600_hw_context.c| 2 +- src/gallium/drivers/r600/r600_state_common.c | 2 +- src/gallium/drivers/radeon/r600_pipe_common.h | 6 +++--- src/gallium/drivers/radeon/r600_query.c | 14

[Mesa-dev] [PATCH 6/7] gallium/radeon: remove predicate_drawing flag

2015-11-08 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/r600/r600_hw_context.c| 2 +- src/gallium/drivers/r600/r600_state_common.c | 2 +- src/gallium/drivers/radeon/r600_pipe_common.h | 1 - src/gallium/drivers/radeon/r600_query.c | 1 -

[Mesa-dev] [PATCH 4/7] gallium/radeon: simplify restoring render condition after flush

2015-11-08 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeon/r600_pipe_common.c | 22 +- src/gallium/drivers/radeon/r600_pipe_common.h | 4 2 files changed, 5 insertions(+), 21 deletions(-) diff --git a/src/gallium/drivers/radeon/r600_pipe_common.c

[Mesa-dev] [PATCH 5/7] gallium/radeon: atomize render condition (SET_PREDICATION)

2015-11-08 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/r600/evergreen_state.c| 1 + src/gallium/drivers/r600/r600_hw_context.c| 1 + src/gallium/drivers/r600/r600_pipe.h | 2 +- src/gallium/drivers/r600/r600_state.c | 1 +

[Mesa-dev] [PATCH 1/7] r600g: don't set predication on non-draw packets

2015-11-08 Thread Marek Olšák
From: Marek Olšák This has no effect. --- src/gallium/drivers/r600/r600_state_common.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/src/gallium/drivers/r600/r600_state_common.c b/src/gallium/drivers/r600/r600_state_common.c index

[Mesa-dev] [PATCH 3/7] gallium/radeon: don't use PREDICATION_OP_CLEAR

2015-11-08 Thread Marek Olšák
From: Marek Olšák Not setting the predication bit is sufficient. --- src/gallium/drivers/radeon/r600_query.c | 60 + 1 file changed, 24 insertions(+), 36 deletions(-) diff --git a/src/gallium/drivers/radeon/r600_query.c

[Mesa-dev] [PATCH 0/7] RadeonSI: Render condition cleanup

2015-11-08 Thread Marek Olšák
I thought this would fix a bug I was hunting, but it didn't. Well, at least it simplifies render condition handling. Please review. Marek ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH 5/5] radeonsi: calculate optimal GS ring sizes to fix GS hangs on Tonga

2015-11-08 Thread Marek Olšák
From: Marek Olšák I discovered that increasing the ESGS ring size fixes GS hangs on Tonga, so let's do it properly. There is now a separate init_config_gs_rings state that is not immutable, because GS rings are resized when needed. This also saves some memory. Most apps

Re: [Mesa-dev] [PATCH 3/6] glsl: move layout qualifier validation out of the parser

2015-11-08 Thread Timothy Arceri
On Sun, 2015-11-08 at 12:07 +, Emil Velikov wrote: > On 6 November 2015 at 21:13, Timothy Arceri wrote: > > On Fri, 2015-11-06 at 13:16 +, Emil Velikov wrote: > > > On 5 November 2015 at 11:17, Timothy Arceri > > > wrote: > > > > From:

[Mesa-dev] [PATCH] gallium/radeon: fix PIPE_QUERY_GPU_FINISHED

2015-11-08 Thread Marek Olšák
From: Marek Olšák Broken by the addition of r600_multi_fence in 3b37155a68acc351cba86a1fa142bd0de2192d4c --- src/gallium/drivers/radeon/r600_query.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/radeon/r600_query.c

[Mesa-dev] ARB_enhanced_layout compile-time-constants V2

2015-11-08 Thread Timothy Arceri
This series adds support for compile time constants and also adds subroutine index qualifier support which was missing for ARB_explicit_uniform_location. V2: Validation of minimum qualifier value moved to the helper functions, all qualifiers will now always have the mimimum value checked. Split

[Mesa-dev] [RFCv2 01/13] gallium: refactor pipe_shader_state to support multiple IR's

2015-11-08 Thread Rob Clark
The goal is to allow the pipe driver to request something other than TGSI, but detect whether what is getting is TGSI vs what it requested. The pipe drivers will always have to support TGSI (and convert that into whatever it is that they prefer), but in some cases we should be able to skip the

[Mesa-dev] [RFCv2 03/13] nir: allow pre-resolved sampler uniform locations

2015-11-08 Thread Rob Clark
From: Rob Clark With TGSI, the ir_variable::data.location gets fixed up to be a stage local location (rather than program global). In this case we need to skip the UniformStorage[location] lookup. --- src/glsl/nir/nir_lower_samplers.c | 23 --- 1

[Mesa-dev] [RFCv2 05/13] gallium/auxiliary: introduce nir_emulate

2015-11-08 Thread Rob Clark
From: Rob Clark NIR equivalent of tgsi_emulate --- src/gallium/auxiliary/Makefile.sources | 2 + src/gallium/auxiliary/nir/nir_emulate.c | 139 src/gallium/auxiliary/nir/nir_emulate.h | 34 3 files changed, 175

[Mesa-dev] [RFCv2 09/13] freedreno/ir3: handle large inputs/outputs

2015-11-08 Thread Rob Clark
From: Rob Clark Internally split them into vec4's.. although perhaps it makes sense to have a generic nir pass which could do this after lower_io? --- .../drivers/freedreno/ir3/ir3_compiler_nir.c | 67 +++--- 1 file changed, 47 insertions(+), 20

[Mesa-dev] [RFCv2 13/13] HACK: freedreno/a4xx: workaround glsl_to_nir hang..

2015-11-08 Thread Rob Clark
From: Rob Clark was getting a hang w/ smaller compmask which happens now with glsl_to_nir since not everything is a vec4 anymore.. --- src/gallium/drivers/freedreno/a4xx/fd4_program.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[Mesa-dev] [RFCv2 12/13] freedreno/ir3: don't ignore local vars

2015-11-08 Thread Rob Clark
From: Rob Clark With glsl_to_nir we end up with local variables, instead of global, for arrays. Note that we'll eventually have to do something more clever, I think, when we support multiple functions, but that will probably take some work in a few places. ---

[Mesa-dev] [RFCv2 08/13] freedreno/ir3: fix const_index handling for uniforms

2015-11-08 Thread Rob Clark
When coming directly from glsl_to_nir (rather than via TGSI where information about, for example, mat4's is lost), both const_index fields will be used (vs. tgsi_to_nir where the 2nd is always zero). For example: decl_var uniform INTERP_QUALIFIER_NONE mat4 ModelViewProjectionMatrix (0, 0)

[Mesa-dev] [RFCv2 04/13] nir: add lowering pass for y-transform

2015-11-08 Thread Rob Clark
From: Rob Clark --- src/glsl/Makefile.sources| 1 + src/glsl/nir/nir.h | 12 ++ src/glsl/nir/nir_lower_wpos_ytransform.c | 320 +++ 3 files changed, 333 insertions(+) create mode 100644

[Mesa-dev] [RFCv2 07/13] freedreno/ir3: add support for NIR as preferred IR

2015-11-08 Thread Rob Clark
For now under debug flag, since only suitable for debugging/testing. --- src/gallium/drivers/freedreno/freedreno_screen.c | 5 - src/gallium/drivers/freedreno/freedreno_util.h | 1 + src/gallium/drivers/freedreno/ir3/ir3_shader.c | 16 3 files changed, 17 insertions(+),

[Mesa-dev] [RFCv2 06/13] mesa/st: add support for NIR as possible driver IR

2015-11-08 Thread Rob Clark
--- src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 410 - src/mesa/state_tracker/st_glsl_to_tgsi.h | 5 + src/mesa/state_tracker/st_program.c| 118 +++-- src/mesa/state_tracker/st_program.h| 6 + 4 files changed, 520 insertions(+), 19

[Mesa-dev] [RFCv2 10/13] freedreno/ir3: support load_front_face intrinsic

2015-11-08 Thread Rob Clark
From: Rob Clark With tgsi_to_nir we get this as a normal input with VARYING_SLOT_FACE. But glsl_to_nir plus nir_lower_system_values this becomes an intrinsic. --- src/gallium/drivers/freedreno/ir3/ir3_compiler_nir.c | 16 ++-- 1 file changed, 14

[Mesa-dev] [RFCv2 02/13] gallium: add NIR as a possible IR

2015-11-08 Thread Rob Clark
--- src/gallium/include/pipe/p_defines.h | 1 + src/gallium/include/pipe/p_state.h | 7 +++ 2 files changed, 8 insertions(+) diff --git a/src/gallium/include/pipe/p_defines.h b/src/gallium/include/pipe/p_defines.h index 0a9d98d..572461f 100644 --- a/src/gallium/include/pipe/p_defines.h

[Mesa-dev] [RFCv2 00/13] gallium: add support for NIR as alternate IR

2015-11-08 Thread Rob Clark
From: Rob Clark Things have progressed somewhat since the initial RFC, to the point that all sorts of common things are working (glmark2, xonotic, stk, etc), and piglit is *mostly* working (~330 regressions or so).. (This is with both VS and FS converted, fwiw,

[Mesa-dev] [RFCv2 11/13] freedreno/ir3: handle tex instrs w/ const offset

2015-11-08 Thread Rob Clark
From: Rob Clark --- src/gallium/drivers/freedreno/ir3/ir3_compiler_nir.c | 16 1 file changed, 16 insertions(+) diff --git a/src/gallium/drivers/freedreno/ir3/ir3_compiler_nir.c b/src/gallium/drivers/freedreno/ir3/ir3_compiler_nir.c index

[Mesa-dev] [PATCH 6/7] radeonsi: prevent recursion in si_context_gfx_flush

2015-11-08 Thread Marek Olšák
From: Marek Olšák The recursion can only occur if you modify need_cs_space to always flush. --- src/gallium/drivers/radeonsi/si_hw_context.c | 7 +++ src/gallium/drivers/radeonsi/si_pipe.h | 1 + 2 files changed, 8 insertions(+) diff --git

[Mesa-dev] [PATCH 7/7] gallium/radeon: inline the r600_rings structure

2015-11-08 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/r600/evergreen_compute.c| 14 ++--- src/gallium/drivers/r600/evergreen_hw_context.c | 10 ++-- src/gallium/drivers/r600/evergreen_state.c | 66 src/gallium/drivers/r600/r600_blit.c| 2

[Mesa-dev] [PATCH 3/7] radeonsi: rename cache flushing flags once more

2015-11-08 Thread Marek Olšák
From: Marek Olšák KCACHE, TC L1 and TC L2 are renamed to: - SMEM L1 - VMEM L1 - GLOBAL L2 You can easily tell what they are used for now. Shaders must deal with coherency issues between both L1s manually, e.g. by setting GLC=1 or by using s_dcache_*. BOTH_ICACHE_KCACHE was

[Mesa-dev] [PATCH 5/7] gallium/radeon: remove the IB flushing flag

2015-11-08 Thread Marek Olšák
From: Marek Olšák Not needed anymore. A similar flag will be introduced in the next commit, which will be private in radeonsi. --- src/gallium/drivers/r600/r600_hw_context.c| 3 --- src/gallium/drivers/radeon/r600_pipe_common.c | 9 ++---

[Mesa-dev] [PATCH 1/7] radeonsi: initialize SX_PS_DOWNCONVERT to 0 on Stoney

2015-11-08 Thread Marek Olšák
From: Marek Olšák otherwise the SX or CB blocks can go bananas --- src/gallium/drivers/radeonsi/si_state.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/gallium/drivers/radeonsi/si_state.c b/src/gallium/drivers/radeonsi/si_state.c index eba9c61..6d97049 100644

[Mesa-dev] [PATCH 3/6] radeonsi: fix a future crash in emit_cb_target_mask

2015-11-08 Thread Marek Olšák
From: Marek Olšák This can't crash currently, but it would crash if clear_buffer from u_blitter were used with a clean context. --- src/gallium/drivers/radeonsi/si_state.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[Mesa-dev] [PATCH 1/6] r600g: fix clear_buffer fallback with offset != 0

2015-11-08 Thread Marek Olšák
From: Marek Olšák Discovered by luck. This code path hasn't been exercised since transform feedback was implemented. --- src/gallium/drivers/r600/r600_blit.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/drivers/r600/r600_blit.c

[Mesa-dev] [PATCH 5/6] radeonsi: add SI_SAVE_FRAGMENT_STATE blitter flag

2015-11-08 Thread Marek Olšák
From: Marek Olšák Buffer clears via transform feedback won't set this. --- src/gallium/drivers/radeonsi/si_blit.c | 44 +++--- 1 file changed, 25 insertions(+), 19 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_blit.c

[Mesa-dev] [PATCH 4/6] gallium/u_blitter: add support for multi-dword clear values in clear_buffer

2015-11-08 Thread Marek Olšák
From: Marek Olšák --- src/gallium/auxiliary/util/u_blitter.c | 25 ++--- 1 file changed, 14 insertions(+), 11 deletions(-) diff --git a/src/gallium/auxiliary/util/u_blitter.c b/src/gallium/auxiliary/util/u_blitter.c index b7b1ece..fccc92c 100644 ---

[Mesa-dev] [PATCH 6/6] radeonsi: add glClearBufferSubData acceleration

2015-11-08 Thread Marek Olšák
From: Marek Olšák Unaligned 8-bit and 16-bit clears are done in software. --- src/gallium/drivers/radeonsi/si_blit.c | 60 ++ 1 file changed, 60 insertions(+) diff --git a/src/gallium/drivers/radeonsi/si_blit.c

[Mesa-dev] [PATCH 4/7] gallium/radeon: move GFX/DMA flushing from add_to_buffer_list to need_cs_space

2015-11-08 Thread Marek Olšák
From: Marek Olšák need_cs_space isn't invoked so often and is called before all commands too. This is a lot cleaner. The code in radeon_add_to_buffer_list always seemed dodgy to me. --- src/gallium/drivers/r600/r600_hw_context.c| 5 +

[Mesa-dev] [PATCH 2/7] radeonsi: set the DISABLE_WR_CONFIRM flag on CI-VI as well

2015-11-08 Thread Marek Olšák
From: Marek Olšák I missed this in commit c3e527f93d4281ad6e2ca165eaf6ff588e4faefa radeonsi: only enable write confirmation on the last CP DMA packet --- src/gallium/drivers/radeonsi/si_cp_dma.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[Mesa-dev] [PATCH 2/6] radeonsi: fix unaligned clear_buffer fallback

2015-11-08 Thread Marek Olšák
From: Marek Olšák This is unreachable currently, but it will be used by unaligned 8-bit and 16-bit fills. --- src/gallium/drivers/radeonsi/si_cp_dma.c | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_cp_dma.c

[Mesa-dev] [PATCH V2 11/12] glsl: add subroutine index qualifier support

2015-11-08 Thread Timothy Arceri
From: Timothy Arceri ARB_explicit_uniform_location allows the index for subroutine functions to be explicitly set in the shader. This patch reduces the restriction on the index qualifier in validate_layout_qualifiers() to allow it to be applied to subroutines and

[Mesa-dev] [PATCH V2 06/12] glsl: remove layout qualifier validation from the parser

2015-11-08 Thread Timothy Arceri
From: Timothy Arceri Now that we have added validation elsewhere we can remove it from the parser. --- src/glsl/glsl_parser.yy | 100 +++- 1 file changed, 23 insertions(+), 77 deletions(-) diff --git

[Mesa-dev] [PATCH V2 04/12] glsl: add layout qualifier validation to the ast for vars

2015-11-08 Thread Timothy Arceri
From: Timothy Arceri This is in preparation for compile-time constant support, a later patch will remove validation from the parser. --- src/glsl/ast.h | 2 ++ src/glsl/ast_to_hir.cpp | 93 ++--- 2 files

[Mesa-dev] [PATCH V2 12/12] docs: mark compile-time constant expressions as done

2015-11-08 Thread Timothy Arceri
From: Timothy Arceri --- docs/GL3.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/GL3.txt b/docs/GL3.txt index 7abdcd8..416b734 100644 --- a/docs/GL3.txt +++ b/docs/GL3.txt @@ -179,7 +179,7 @@ GL 4.4, GLSL 4.40: GL_ARB_buffer_storage

[Mesa-dev] [PATCH V2 01/12] glsl: simplify interface block stream qualifier validation

2015-11-08 Thread Timothy Arceri
From: Timothy Arceri Qualifiers on member variables are redundent all we need to do if check if it matches the stream associated with the block and throw an error if its not. Cc: Samuel Iglesias Gonsalvez Cc: Emil Velikov

[Mesa-dev] [PATCH V2 07/12] glsl: add new type from compile time constants

2015-11-08 Thread Timothy Arceri
From: Timothy Arceri In this patch we introduce a new ast type for holding the new compile-time constant expressions. The main reason for this is that we can no longer do merging of layout qualifiers before they have been converted into GLSL IR so we need to store

[Mesa-dev] [PATCH] nir: fix typo in idiv lowering, causing large-udiv-udiv failures

2015-11-08 Thread Ilia Mirkin
In nv50, and in the python script that Rob circulated, we do: bld.mkCmp(OP_SET, CC_GE, TYPE_U32, (s = bld.getSSA()), TYPE_U32, m, b); Do the same in the nir div lowering pass. This fixes the large-udiv-udiv piglit tests on freedreno. Signed-off-by: Ilia Mirkin Cc:

[Mesa-dev] [PATCH 2/5] radeonsi: move maximum gs stream calculation into create_shader

2015-11-08 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_shader.h| 1 + src/gallium/drivers/radeonsi/si_state_shaders.c | 22 ++ 2 files changed, 7 insertions(+), 16 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_shader.h

[Mesa-dev] [PATCH 1/5] radeonsi: clean up small duplication in si_shader_gs

2015-11-08 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_shader.h| 3 ++- src/gallium/drivers/radeonsi/si_state_shaders.c | 11 ++- 2 files changed, 8 insertions(+), 6 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_shader.h

[Mesa-dev] [PATCH 4/5] radeonsi: rename si_update_gs_rings

2015-11-08 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_state_shaders.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_state_shaders.c b/src/gallium/drivers/radeonsi/si_state_shaders.c index 89d365b..c402ce2 100644 ---

[Mesa-dev] [PATCH 3/5] radeonsi: calculate ESGS_RING_ITEMSIZE in create_shader

2015-11-08 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_shader.h| 1 + src/gallium/drivers/radeonsi/si_state_shaders.c | 3 ++- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/radeonsi/si_shader.h

[Mesa-dev] [PATCH 0/5] RadeonSI: Optimal GS ring sizes, fixing Tonga hangs

2015-11-08 Thread Marek Olšák
This fixes hangs on Tonga when the ESGS ring isn't large enough. It's also a requirement for this not-yet-committed patch: "radeonsi: link ES-GS just like LS-HS" which makes GS hangs easier to reproduce. The ring size equations are based on VGT docs and my discussion with VGT guys. Please

[Mesa-dev] [PATCH V2 03/12] glsl: add helper to check for enhanced layouts support

2015-11-08 Thread Timothy Arceri
From: Timothy Arceri --- src/glsl/glsl_parser_extras.h | 5 + 1 file changed, 5 insertions(+) diff --git a/src/glsl/glsl_parser_extras.h b/src/glsl/glsl_parser_extras.h index 684b917..1d8c1b8 100644 --- a/src/glsl/glsl_parser_extras.h +++

[Mesa-dev] [PATCH V2 10/12] glsl: add support for complie-time constant expressions

2015-11-08 Thread Timothy Arceri
From: Timothy Arceri This patch replaces the old interger constant qualifiers with either the new ast_layout_expression type if the qualifier requires merging or ast_expression if the qualifier can't have mulitple declorations or if all but he newest qualifier is

[Mesa-dev] [PATCH V2 08/12] glsl: add process_qualifier_constant() helper

2015-11-08 Thread Timothy Arceri
From: Timothy Arceri This helper is similar to the function added as part of the ast_layout_expression class but will be used when only the ast_expression type is required for the qualifier. ast_expression is used if qualifier can't have mulitple declorations or if

[Mesa-dev] [PATCH V2 05/12] glsl: add layout qualifier validation for the shader outside the parser

2015-11-08 Thread Timothy Arceri
From: Timothy Arceri This is in preparation for compile-time constant support, a later patch will remove the validation from the shader. The global shader layout qualifiers will now mostly be validated in glsl_parser_extras.cpp. In order to do validation at the

[Mesa-dev] [PATCH V2 02/12] mesa: add ARB_enhanced_layouts

2015-11-08 Thread Timothy Arceri
From: Timothy Arceri --- src/glsl/glcpp/glcpp-parse.y| 1 + src/glsl/glsl_parser_extras.cpp | 1 + src/glsl/glsl_parser_extras.h | 2 ++ src/mesa/main/extensions.c | 1 + src/mesa/main/mtypes.h | 1 + 5 files changed, 6 insertions(+) diff --git

[Mesa-dev] [PATCH V2 09/12] glsl: add validate_stream_qualifier() helper

2015-11-08 Thread Timothy Arceri
From: Timothy Arceri --- src/glsl/ast_to_hir.cpp | 12 1 file changed, 12 insertions(+) diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp index 21a956d..fcf7566 100644 --- a/src/glsl/ast_to_hir.cpp +++ b/src/glsl/ast_to_hir.cpp @@ -2303,6

[Mesa-dev] [Bug 92860] [radeonsi][bisected] st/mesa: implement ARB_copy_image - Corruption in ARK Survival Evolved

2015-11-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=92860 Michel Dänzer changed: What|Removed |Added Component|Drivers/Gallium/radeonsi|Mesa core

Re: [Mesa-dev] [PATCH 1/3] glsl: Parse shared keyword for compute shader variables

2015-11-08 Thread Timothy Arceri
On Sat, 2015-11-07 at 23:16 -0800, Jordan Justen wrote: > On 2015-11-06 19:33:38, Timothy Arceri wrote: > > On Fri, 2015-11-06 at 17:56 -0800, Jordan Justen wrote: > > > Signed-off-by: Jordan Justen > > > --- > > > > > > Notes: > > >

Re: [Mesa-dev] [RFCv2 03/13] nir: allow pre-resolved sampler uniform locations

2015-11-08 Thread Timothy Arceri
On Sun, 2015-11-08 at 15:12 -0500, Rob Clark wrote: > From: Rob Clark > > With TGSI, the ir_variable::data.location gets fixed up to be a stage > local location (rather than program global). In this case we need to > skip the UniformStorage[location] lookup. > --- >

[Mesa-dev] [PATCH] RFC: llvmpipe map scene buffers outside thread.

2015-11-08 Thread Dave Airlie
From: Dave Airlie There might be a reason we do this inside the thread, but I'm not aware of it yet, move stuff around and see if this jogs anyone's memory. Doing this outside the thread at least with front buffer rendering avoids problems with XGetImage failing in the

[Mesa-dev] soft/llvmpipe front buffer access and piglit regressions

2015-11-08 Thread Dave Airlie
So it appears my patch to enable front buffer access on soft/llvmpipe causes some piglit regressions. However these are due to piglit having undefined behaviour where it doesn't create a window but has tests requiring a front buffer. The new code does an XGetImage on the front buffer and when it

Re: [Mesa-dev] [Mesa-stable] New stable-branch 11.0 candidate pushed

2015-11-08 Thread Oded Gabbay
On Sat, Nov 7, 2015 at 6:37 PM, Emil Velikov wrote: > Hello list, > > The candidate for the Mesa 11.0.5 is now available. Currently we have: > - 31 queued > - 13 nominated (outstanding) > - and 6 rejected/obsolete patches > > Currently we have mostly driver related

[Mesa-dev] [PATCH] mesa/copyimage: allow width/height to not be multiples of block

2015-11-08 Thread Ilia Mirkin
For compressed textures, the image size is not necessarily a multiple of the block size (e.g. the last mip levels). For example the RGTC spec calls out this condition as legal, saying to error only when: * is not a multiple of four, and plus * is not equal to TEXTURE_WIDTH; While the

Re: [Mesa-dev] [PATCH] st/va: add mpeg4 startcode workaround

2015-11-08 Thread Christian König
On 06.11.2015 20:28, Ilia Mirkin wrote: On Fri, Nov 6, 2015 at 2:15 PM, Christian König wrote: On 06.11.2015 20:10, Ilia Mirkin wrote: On Fri, Nov 6, 2015 at 1:48 PM, Zhang, Boyuan wrote: Hi Emil, Please see the following information about

Re: [Mesa-dev] [PATCH] gallium/swrast: fix front buffer blitting. (v2)

2015-11-08 Thread Emil Velikov
On 7 November 2015 at 21:57, Dave Airlie wrote: > On 8 November 2015 at 02:47, Emil Velikov wrote: >> Hi Dave, >> >> On 9 October 2015 at 01:38, Dave Airlie wrote: >>> From: Dave Airlie >>> >>> So I've known

Re: [Mesa-dev] [PATCH] android: fix LOCAL_C_INCLUDES to find glsl_types.h

2015-11-08 Thread Mauro Rossi
Update2: I'm getting the building error in both x86 target and x86_64 target. I'm relieved it's not arch dependend, I suspect that export will require a dependency to be declared, because if i965_dri module is built before glsl ones we will have the error. The LOCAL_C_INCLUDES even if not

[Mesa-dev] [RFC PATCH shader-db 2/2] run: request a debug context

2015-11-08 Thread Ilia Mirkin
st/mesa only prints messages in a debug context. Without always enabling the message generation, I don't see a way to hook into the glEnable() to turn it on/off. --- run.c | 1 + 1 file changed, 1 insertion(+) diff --git a/run.c b/run.c index 73e468d..1d8d3b1 100644 --- a/run.c +++ b/run.c @@

[Mesa-dev] [RFC PATCH shader-db 1/2] run: don't expect incoming message to contain a newline

2015-11-08 Thread Ilia Mirkin
It seems a bit odd to expect a debug message to contain a newline -- what if you wanted to include something *after* the message, for example. It makes more sense for the code actually printing to have the newline rather than the string being passed around. --- run.c | 2 +- 1 file changed, 1

Re: [Mesa-dev] [PATCH] gallium/radeon: fix PIPE_QUERY_GPU_FINISHED

2015-11-08 Thread Michel Dänzer
On 09.11.2015 06:43, Marek Olšák wrote: > From: Marek Olšák > > Broken by the addition of r600_multi_fence > in 3b37155a68acc351cba86a1fa142bd0de2192d4c > --- > src/gallium/drivers/radeon/r600_query.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

[Mesa-dev] [PATCH] RFC: llvmpipe map scene buffers outside thread. (v2)

2015-11-08 Thread Dave Airlie
From: Dave Airlie There might be a reason we do this inside the thread, but I'm not aware of it yet, move stuff around and see if this jogs anyone's memory. Doing this outside the thread at least with front buffer rendering avoids problems with XGetImage failing in the