[Mesa-dev] [PATCH 67/65] glsl: disable on disk shader cache when running as root

--- src/compiler/glsl/cache.c | 4 1 file changed, 4 insertions(+) diff --git a/src/compiler/glsl/cache.c b/src/compiler/glsl/cache.c index a6a9e47..dd53f4e 100644 --- a/src/compiler/glsl/cache.c +++ b/src/compiler/glsl/cache.c @@ -141,6 +141,10 @@ cache_create(void) struct stat sb;

Re: [Mesa-dev] [PATCH 1/3] nir/opt_dead_cf: correction of side effect check

On Fri, Apr 29, 2016 at 8:39 PM, Jason Ekstrand wrote: > > > On Fri, Apr 29, 2016 at 11:31 AM, Thomas Hindoe Paaboel Andersen < > pho...@gmail.com> wrote: > >> Parenthesis are needed here as ! takes precedence over the &. The >> check had the opposite effect than intended.

[Mesa-dev] [PATCH V2] glsl: track mesa version shader cache items were created with

Also remove cache item and fallback to full recompile if current Mesa version differs. V2: don't leak buffer --- src/compiler/glsl/shader_cache.cpp | 13 + 1 file changed, 13 insertions(+) diff --git a/src/compiler/glsl/shader_cache.cpp b/src/compiler/glsl/shader_cache.cpp index

[Mesa-dev] [PATCH V2] glsl: add cache to ctx and add sha1 string fields

From: Carl Worth V2: dont leak cache Signed-off-by: Timothy Arceri --- src/mesa/main/context.c | 6 ++ src/mesa/main/mtypes.h | 5 + 2 files changed, 11 insertions(+) diff --git a/src/mesa/main/context.c b/src/mesa/main/context.c

Re: [Mesa-dev] On disk shader cache for i965

Here are the shader-db times after fixing the buffer overflow with patch 66/65. Cache disabled: Thread 1 took 1360.47 seconds and compiled 13015 shaders (not including SIMD16) with 50 GL context switches Thread 3 took 1349.85 seconds and compiled 12848 shaders (not including SIMD16) with 40 GL

[Mesa-dev] [PATCH 66/65] glsl: avoid buffer overflow in cache sha creation

A program may contain multiple shaders from each stage so use ralloc to avoid buffer overflow. --- This fixes the issue I was having running shader-db it now successfully finishes compiling all programs. src/compiler/glsl/shader_cache.cpp | 19 +-- 1 file changed, 9

Re: [Mesa-dev] [PATCH 54/59] i965/fs/lower_simd_width: Fix registers written for split instructions

Samuel Iglesias Gonsálvez writes: > From: Iago Toral Quiroga > > When the original instruction had a stride > 1, the combined registers > written by the split instructions won't amount to the same register space > written by the original instruction

[Mesa-dev] [Bug 95211] scons TypeError: 'tuple' object is not callable

https://bugs.freedesktop.org/show_bug.cgi?id=95211 Bug ID: 95211 Summary: scons TypeError: 'tuple' object is not callable Product: Mesa Version: git Hardware: x86-64 (AMD64) OS: Linux (All) Status: NEW

[Mesa-dev] [Bug 95180] rasterizer/memory/Convert.h:170:9: error: ‘__builtin_isnan’ is not a member of ‘std’

https://bugs.freedesktop.org/show_bug.cgi?id=95180 --- Comment #4 from Vinson Lee --- I am building with GCC 5.3 on Ubuntu 16.10. $ gcc --version gcc (Ubuntu 5.3.1-16ubuntu2) 5.3.1 20160424 Copyright (C) 2015 Free Software Foundation, Inc. This is free software; see the

[Mesa-dev] [Bug 95085] Invalid sampling of second texture in fragment shader that have two samplers with different parameters.

https://bugs.freedesktop.org/show_bug.cgi?id=95085 Nicolai Hähnle changed: What|Removed |Added Status|REOPENED|RESOLVED

[Mesa-dev] [Bug 95085] Invalid sampling of second texture in fragment shader that have two samplers with different parameters.

https://bugs.freedesktop.org/show_bug.cgi?id=95085 --- Comment #15 from Roland Scheidegger --- (In reply to Nicolai Hähnle from comment #14) > The trace uses glVertexAttribPointer instead of glVertexAttribIPointer to > set up a vertex element that will consumed by an uint

[Mesa-dev] [PATCH] anv: Fix make check build error.

CC block_pool_no_free.o In file included from ../../../../src/intel/vulkan/anv_private.h:60:0, from block_pool_no_free.c:26: ../../../../src/intel/vulkan/anv_entrypoints.h:162:69: error: unknown type name ‘VkXcbSurfaceCreateInfoKHR’ VkResult

[Mesa-dev] [Bug 95085] Invalid sampling of second texture in fragment shader that have two samplers with different parameters.

https://bugs.freedesktop.org/show_bug.cgi?id=95085 --- Comment #14 from Nicolai Hähnle --- I think I understand now where the bug is coming from, and I'm not sure if it's a driver or an application bug :) The trace uses glVertexAttribPointer instead of glVertexAttribIPointer

Re: [Mesa-dev] [PATCH 4/7] i965/disasm: Split opcode tables by the generation they were introduced in.

Matt Turner writes: > On Thu, Apr 28, 2016 at 12:46 PM, Francisco Jerez > wrote: >> Matt Turner writes: >> >>> On Thu, Apr 28, 2016 at 12:19 AM, Francisco Jerez >>> wrote: ---

Re: [Mesa-dev] [PATCH] radeonsi: fix PIPE_FORMAT_R11G11B10_FLOAT handling

On 2016-04-29 23:21, Nicolai Hähnle wrote: From: Nicolai Hähnle That format has first_non_void < 0. This fixes a regression in piglit arb_shader_image_load_store-semantics that was introduced by commit 76b8c5cc602, while hopefully still shutting Coverity up (and

[Mesa-dev] [PATCH] radeonsi: fix PIPE_FORMAT_R11G11B10_FLOAT handling

From: Nicolai Hähnle That format has first_non_void < 0. This fixes a regression in piglit arb_shader_image_load_store-semantics that was introduced by commit 76b8c5cc602, while hopefully still shutting Coverity up (and failing in a more obvious way if a similar error

Re: [Mesa-dev] [PATCH] radeonsi: fix synchronization of shader images

Good catch! Reviewed-by: Nicolai Hähnle On 29.04.2016 15:32, Marek Olšák wrote: From: Marek Olšák This fixes the winsys->cs_is_buffer_referenced query, which is used for synchronization before buffers are mapped. ---

[Mesa-dev] [Bug 95085] Invalid sampling of second texture in fragment shader that have two samplers with different parameters.

https://bugs.freedesktop.org/show_bug.cgi?id=95085 Nicolai Hähnle changed: What|Removed |Added Resolution|NOTOURBUG |---

[Mesa-dev] [PATCH] radeonsi: fix synchronization of shader images

From: Marek Olšák This fixes the winsys->cs_is_buffer_referenced query, which is used for synchronization before buffers are mapped. --- src/gallium/drivers/radeonsi/si_descriptors.c | 18 +++--- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git

[Mesa-dev] [Bug 94168] Incorrect rendering when running Populous 3 on wine using DDraw->WineD3D->OpenGL wrapper [apitrace]

https://bugs.freedesktop.org/show_bug.cgi?id=94168 --- Comment #8 from almos --- (In reply to Nicolai Hähnle from comment #7) > When you open the trace with qapitrace, you'll see that appears as one > single big frame. This is a big clue as to what's going on. Somehow, the >

Re: [Mesa-dev] [PATCH 4/7] i965/disasm: Split opcode tables by the generation they were introduced in.

On Thu, Apr 28, 2016 at 12:46 PM, Francisco Jerez wrote: > Matt Turner writes: > >> On Thu, Apr 28, 2016 at 12:19 AM, Francisco Jerez >> wrote: >>> --- >>> src/mesa/drivers/dri/i965/brw_disasm.c | 90 >>>

Re: [Mesa-dev] [PATCH 4/7] i965/disasm: Split opcode tables by the generation they were introduced in.

Ian Romanick writes: > On 04/28/2016 09:46 PM, Francisco Jerez wrote: >> Matt Turner writes: >> >>> On Thu, Apr 28, 2016 at 12:19 AM, Francisco Jerez >>> wrote: --- src/mesa/drivers/dri/i965/brw_disasm.c | 90

[Mesa-dev] [Bug 95085] Invalid sampling of second texture in fragment shader that have two samplers with different parameters.

https://bugs.freedesktop.org/show_bug.cgi?id=95085 Nicolai Hähnle changed: What|Removed |Added Resolution|--- |NOTOURBUG

[Mesa-dev] [PATCH] st/nine: clean up WINAPI definition

As Emil pointed out, only gcc, clang and MSVC compatibility is required. Hence the check for GNUC can be skipped, as __i386__ and __x86_64__ are only defined for gcc/clang, not for MSVC. Remove the #undef which has been there for historic reasons, when wine dlls for nine have been built inside

Re: [Mesa-dev] [PATCH 2/3] mesa: rename lower_fmod to lower_fmod32

Why not just squash 2 and 3 and call it "Separate 32 and 64-bit fmod lowering" or something like that. On Thu, Apr 28, 2016 at 4:19 AM, Samuel Iglesias Gonsálvez < sigles...@igalia.com> wrote: > A later patch will add lower_fmod64 option to NIR. > > Signed-off-by: Samuel Iglesias Gonsálvez

Re: [Mesa-dev] [PATCH 1/3] nir/opt_dead_cf: correction of side effect check

On Fri, Apr 29, 2016 at 11:31 AM, Thomas Hindoe Paaboel Andersen < pho...@gmail.com> wrote: > Parenthesis are needed here as ! takes precedence over the &. The > check had the opposite effect than intended. > --- > Not tested as I do not own the neccesary hardware. > >

[Mesa-dev] [Bug 94168] Incorrect rendering when running Populous 3 on wine using DDraw->WineD3D->OpenGL wrapper [apitrace]

https://bugs.freedesktop.org/show_bug.cgi?id=94168 --- Comment #7 from Nicolai Hähnle --- When you open the trace with qapitrace, you'll see that appears as one single big frame. This is a big clue as to what's going on. Somehow, the game/Wine doesn't trigger the usual swap

[Mesa-dev] [PATCH 3/3] nir/lower_double_ops: fix indentation

--- src/compiler/nir/nir_lower_double_ops.c | 60 - 1 file changed, 30 insertions(+), 30 deletions(-) diff --git a/src/compiler/nir/nir_lower_double_ops.c b/src/compiler/nir/nir_lower_double_ops.c index e5cf094..3f831dc 100644 ---

[Mesa-dev] [PATCH 2/3] nir/opt_dead_cf: fix indentation

--- src/compiler/nir/nir_opt_dead_cf.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/compiler/nir/nir_opt_dead_cf.c b/src/compiler/nir/nir_opt_dead_cf.c index 7450b1c..74af19b 100644 --- a/src/compiler/nir/nir_opt_dead_cf.c +++

[Mesa-dev] [PATCH 1/3] nir/opt_dead_cf: correction of side effect check

Parenthesis are needed here as ! takes precedence over the &. The check had the opposite effect than intended. --- Not tested as I do not own the neccesary hardware. src/compiler/nir/nir_opt_dead_cf.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

Re: [Mesa-dev] [PATCH 3/3] nir/opt_algebraic: lower mod() with doubles operands if lower_fmod64 is enabled

On 2016-04-28 04:19:18, Samuel Iglesias Gonsálvez wrote: > Make this distintion as the drivers might need to lower it inside NIR. > > Signed-off-by: Samuel Iglesias Gonsálvez > --- > src/compiler/nir/nir.h| 2 ++ > src/compiler/nir/nir_opt_algebraic.py | 3

Re: [Mesa-dev] [PATCH] glx: Refactor the configure options for glx implementation choice (v3)

Hi Chuck, Considering that lack of comments it seems like pretty much everyone is happy or that no-one is interested/has time. I'm just giving your patch a few last tests and I'll push it to master. Thanks for the contribution ! Emil ___ mesa-dev

Re: [Mesa-dev] [PATCH] radeonsi: correct NULL-pointer check in si_upload_const_buffer

Reviewed-by: Marek Olšák Marek On Fri, Apr 29, 2016 at 7:13 PM, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > Cc: "11.1 11.2" > --- > src/gallium/drivers/radeonsi/si_descriptors.c | 2 +-

Re: [Mesa-dev] [PATCH] radeonsi: correct NULL-pointer check in si_upload_const_buffer

Reviewed-by: Bas Nieuwenhuizen On Fri, Apr 29, 2016 at 7:13 PM, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > Cc: "11.1 11.2" > --- > src/gallium/drivers/radeonsi/si_descriptors.c | 2

[Mesa-dev] [PATCH] radeonsi: correct NULL-pointer check in si_upload_const_buffer

From: Nicolai Hähnle Cc: "11.1 11.2" --- src/gallium/drivers/radeonsi/si_descriptors.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/radeonsi/si_descriptors.c

[Mesa-dev] [Bug 92850] Segfault loading War Thunder

https://bugs.freedesktop.org/show_bug.cgi?id=92850 Nicolai Hähnle changed: What|Removed |Added Resolution|--- |FIXED

Re: [Mesa-dev] [PATCH 11/59] i965/eu: allow doubles in math instructions

IIRC this is wrong. I think I added it in the beginning before I tested to see if the spec was lying about math instructions not being supported on doubles (it wasn't) and we can delete it now that we lower away all the possible math operations on doubles. On Fri, Apr 29, 2016 at 7:29 AM, Samuel

Re: [Mesa-dev] [PATCH] anv: remove description about GENX_FUNC macro

LGTM fell free to push On Apr 29, 2016 8:39 AM, "Emil Velikov" wrote: > Ping ? > Considering to trivial this is I'm inclined to push it later on today. > > On 21 April 2016 at 13:20, Emil Velikov wrote: > > From: Emil Velikov

Re: [Mesa-dev] [PATCH] mesa: simplify _mesa_Lightfv

Am 29.04.2016 um 18:34 schrieb Brian Paul: On 04/29/2016 08:24 AM, Thomas Faller wrote: Signed-off-by: Thomas Faller --- src/mesa/main/light.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/src/mesa/main/light.c b/src/mesa/main/light.c index 14b4b04..4a8dee3

Re: [Mesa-dev] [PATCH] mesa: simplify _mesa_Lightfv

On 04/29/2016 08:24 AM, Thomas Faller wrote: Signed-off-by: Thomas Faller --- src/mesa/main/light.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/src/mesa/main/light.c b/src/mesa/main/light.c index 14b4b04..4a8dee3 100644 --- a/src/mesa/main/light.c +++

Re: [Mesa-dev] [PATCH] anv: remove description about GENX_FUNC macro

Reviewed-by: Eduardo Lima Mitev On 04/29/2016 05:39 PM, Emil Velikov wrote: > Ping ? > Considering to trivial this is I'm inclined to push it later on today. > > On 21 April 2016 at 13:20, Emil Velikov wrote: >> From: Emil Velikov

[Mesa-dev] [PATCH] GL3: remove radeonsi occurrences in GL 4.2, already specified as "all DONE"

--- docs/GL3.txt | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/docs/GL3.txt b/docs/GL3.txt index bb2bb6e..5a6be41 100644 --- a/docs/GL3.txt +++ b/docs/GL3.txt @@ -148,17 +148,17 @@ GL 4.1, GLSL 4.10 --- all DONE: nvc0, r600, radeonsi GL 4.2, GLSL 4.20 -- all

Re: [Mesa-dev] [PATCH] anv: remove description about GENX_FUNC macro

Ping ? Considering to trivial this is I'm inclined to push it later on today. On 21 April 2016 at 13:20, Emil Velikov wrote: > From: Emil Velikov > > The macro has been gone since commit 1f1cf6fcb0e "anv: Get rid of > GENX_FUNC" > >

Re: [Mesa-dev] [PATCH] st/omx: don't return early in vid_enc_EncodeFrame()

Hi Alex, On 29 April 2016 at 15:30, Alex Deucher wrote: > On Fri, Apr 29, 2016 at 8:56 AM, Emil Velikov > wrote: >> From: Emil Velikov >> >> Earlier commit plugged a memory leak, although it missed a pair of >>

Re: [Mesa-dev] [PATCH] gallium/radeon: fix crash in r600_set_streamout_targets

Reviewed-by: Marek Olšák Marek On Fri, Apr 29, 2016 at 1:58 AM, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > Protect against dereferencing a gap in the targets array. This was triggered > by a test in the Khronos CTS. > >

Re: [Mesa-dev] [PATCH] st/omx: don't return early in vid_enc_EncodeFrame()

On Fri, Apr 29, 2016 at 8:56 AM, Emil Velikov wrote: > From: Emil Velikov > > Earlier commit plugged a memory leak, although it missed a pair of > brackets. Thus we unconditionally returned even in the case of no error. > > Bugzilla:

Re: [Mesa-dev] [android-x86-devel] [RFC 7/7] android: support swrast

[Adding Mauro] On 29 April 2016 at 14:25, Rob Herring wrote: > On Thu, Apr 28, 2016 at 2:35 AM, Chih-Wei Huang > wrote: >> From: WuZhen >> >> System boots up with gles_mesa/softpipe/llvmpipe. >> >> NO_REF_TASK >> tested:

[Mesa-dev] [PATCH] mesa: simplify _mesa_Lightfv

Signed-off-by: Thomas Faller --- src/mesa/main/light.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/src/mesa/main/light.c b/src/mesa/main/light.c index 14b4b04..4a8dee3 100644 --- a/src/mesa/main/light.c +++ b/src/mesa/main/light.c @@ -250,17 +250,7 @@

Re: [Mesa-dev] [android-x86-devel] [RFC 0/7] android: enable llvmpipe for software rendering

Hi, Chih-Wei, Emil, These series of patches was originally developed on the 11.0 branch, and later ported to 11.2. I don't think softpipe was working when I worked on this, Actually the first few patches is from when I tried to make softpipe work. There are some memory issue in mesa that cause

Re: [Mesa-dev] [android-x86-devel] [RFC 0/7] android: enable llvmpipe for software rendering

Thanks for taking the time to review, Emil. We're happy to contribute to the community, I'll will work with Chih-Wei and try to incorporate the comments. 2016-04-29 19:16 GMT+08:00 Emil Velikov : > Hi Zhen Wu, all, > > On 29 April 2016 at 04:12, Zhen Wu

[Mesa-dev] [PATCH 36/65] glsl: add support for serialising atomic and array types

--- src/compiler/glsl/shader_cache.cpp | 15 +-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/src/compiler/glsl/shader_cache.cpp b/src/compiler/glsl/shader_cache.cpp index ee90157..ce23642 100644 --- a/src/compiler/glsl/shader_cache.cpp +++

[Mesa-dev] [PATCH 42/65] glsl: don't cache shader if linking fails

Fixes crash in piglit tests. The assumption is that linking was successful if we see the linked program in the cache. --- src/mesa/program/ir_to_mesa.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/mesa/program/ir_to_mesa.cpp b/src/mesa/program/ir_to_mesa.cpp index

[Mesa-dev] [PATCH 35/65] glsl: add support for serialising image types

--- src/compiler/glsl/shader_cache.cpp | 8 1 file changed, 8 insertions(+) diff --git a/src/compiler/glsl/shader_cache.cpp b/src/compiler/glsl/shader_cache.cpp index 747df78..ee90157 100644 --- a/src/compiler/glsl/shader_cache.cpp +++ b/src/compiler/glsl/shader_cache.cpp @@ -85,6

[Mesa-dev] [PATCH 38/65] glsl: cache interpolation qualifier for frag shader

--- src/compiler/glsl/shader_cache.cpp | 14 ++ 1 file changed, 14 insertions(+) diff --git a/src/compiler/glsl/shader_cache.cpp b/src/compiler/glsl/shader_cache.cpp index 6f09c17..ff94dbf 100644 --- a/src/compiler/glsl/shader_cache.cpp +++ b/src/compiler/glsl/shader_cache.cpp @@

[Mesa-dev] [PATCH 43/65] glsl: make uniform values helper available for use elsewhere

--- src/compiler/glsl/link_uniforms.cpp | 2 +- src/compiler/glsl/linker.h | 3 +++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/src/compiler/glsl/link_uniforms.cpp b/src/compiler/glsl/link_uniforms.cpp index 8180d10..8c27030 100644 ---

[Mesa-dev] [PATCH 32/65] glsl: add support for caching subroutines

--- src/compiler/glsl/shader_cache.cpp | 118 + 1 file changed, 108 insertions(+), 10 deletions(-) diff --git a/src/compiler/glsl/shader_cache.cpp b/src/compiler/glsl/shader_cache.cpp index d2d1381..4599e5a 100644 --- a/src/compiler/glsl/shader_cache.cpp +++

[Mesa-dev] [PATCH 44/65] glsl: cache uniform values

These may be lowered constant arrays or uniform values that we set before linking so we need to cache the actual uniform values. --- src/compiler/glsl/shader_cache.cpp | 31 +++ 1 file changed, 31 insertions(+) diff --git a/src/compiler/glsl/shader_cache.cpp

[Mesa-dev] [PATCH 37/65] glsl: add support for caching atomic buffers

--- src/compiler/glsl/shader_cache.cpp | 87 ++ 1 file changed, 87 insertions(+) diff --git a/src/compiler/glsl/shader_cache.cpp b/src/compiler/glsl/shader_cache.cpp index ce23642..6f09c17 100644 --- a/src/compiler/glsl/shader_cache.cpp +++

[Mesa-dev] [PATCH 31/65] glsl: add cache support for encoding subroutine type

--- src/compiler/glsl/shader_cache.cpp | 7 +++ 1 file changed, 7 insertions(+) diff --git a/src/compiler/glsl/shader_cache.cpp b/src/compiler/glsl/shader_cache.cpp index 9c7b18d..d2d1381 100644 --- a/src/compiler/glsl/shader_cache.cpp +++ b/src/compiler/glsl/shader_cache.cpp @@ -79,6

[Mesa-dev] [PATCH 34/65] glsl: add shader cache support for buffer blocks

--- src/compiler/glsl/shader_cache.cpp | 152 + 1 file changed, 152 insertions(+) diff --git a/src/compiler/glsl/shader_cache.cpp b/src/compiler/glsl/shader_cache.cpp index f9b2157..747df78 100644 --- a/src/compiler/glsl/shader_cache.cpp +++

[Mesa-dev] [PATCH 33/65] glsl: store subroutine remap table in shader cache

--- src/compiler/glsl/shader_cache.cpp | 53 -- 1 file changed, 45 insertions(+), 8 deletions(-) diff --git a/src/compiler/glsl/shader_cache.cpp b/src/compiler/glsl/shader_cache.cpp index 4599e5a..f9b2157 100644 --- a/src/compiler/glsl/shader_cache.cpp +++

Re: [Mesa-dev] gstreamer: v4l2videodec plugin

cc: mesa-dev ML On 04/28/2016 02:33 PM, Stanimir Varbanov wrote: > On 04/15/2016 07:09 PM, Nicolas Dufresne wrote: >> Le vendredi 15 avril 2016 à 11:58 -0400, Rob Clark a écrit : >>> The issue is probably the YUV format, which we cannot really deal >>> with >>> properly in gallium.. it's a

[Mesa-dev] [PATCH 63/65] i965: get scratch on cache restore

--- src/mesa/drivers/dri/i965/brw_shader_cache.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_shader_cache.c b/src/mesa/drivers/dri/i965/brw_shader_cache.c index ac381d1..5d47122 100644 --- a/src/mesa/drivers/dri/i965/brw_shader_cache.c +++

[Mesa-dev] [PATCH 65/65] i965: track mesa version shader cache items were created with

Also remove cache item and fallback to full recompile if current Mesa version differs. --- src/mesa/drivers/dri/i965/brw_shader_cache.c | 12 1 file changed, 12 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_shader_cache.c b/src/mesa/drivers/dri/i965/brw_shader_cache.c

[Mesa-dev] [PATCH 62/65] i965: generate outputs written for use in fs key

This is normally generated at link time so we need to do it ourselves for use with the shader cache. --- src/mesa/drivers/dri/i965/brw_shader_cache.c | 8 1 file changed, 8 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_shader_cache.c

[Mesa-dev] [PATCH 64/65] i965: copy and restore sampler units when doing full relink

If we don't find the program in the cache we need to make sure to store the current sampler units and restore them after falling back to relinking. In this change we also avoid reseting the sampler and image units stored in gl_shader back to 0 during relinking. ---

[Mesa-dev] [PATCH 54/65] glsl: don't lose uniform values when falling back to full compile

Here we skip the recreation of uniform storage if we are relinking after a cache miss. --- src/compiler/glsl/link_uniforms.cpp | 32 +--- src/compiler/glsl/linker.cpp | 3 ++- src/compiler/glsl/linker.h | 2 +-

[Mesa-dev] [PATCH 52/65] i965: always set program id to 0 for on disk cache

We don't care what instance of the program it is we only care if its the correct binary to load so ignore program id for on disk cache --- src/mesa/drivers/dri/i965/brw_shader_cache.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_shader_cache.c

[Mesa-dev] [PATCH 53/65] i965: check memory cache before disk cache

--- src/mesa/drivers/dri/i965/brw_state_upload.c | 4 src/mesa/drivers/dri/i965/brw_vs.c | 21 - src/mesa/drivers/dri/i965/brw_wm.c | 17 ++--- 3 files changed, 30 insertions(+), 12 deletions(-) diff --git

[Mesa-dev] [PATCH 49/65] i965: add image param shader cache support

--- src/mesa/drivers/dri/i965/brw_shader_cache.c | 60 +--- 1 file changed, 55 insertions(+), 5 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_shader_cache.c b/src/mesa/drivers/dri/i965/brw_shader_cache.c index 7945b16..850e1e8 100644 ---

[Mesa-dev] [PATCH 59/65] mesa: extend LinkShader function to include cache fallback param

This allows up to skip certain linking tasks such as building the resource list when we are falling back to a full compile after a cache miss. --- src/mesa/drivers/dri/i965/brw_link.cpp | 3 ++- src/mesa/drivers/dri/i965/brw_shader.h | 4 +++- src/mesa/main/dd.h |

[Mesa-dev] [PATCH 58/65] i965: copy program id to key before upload

On the off chance the program has already been assigned an id --- src/mesa/drivers/dri/i965/brw_shader_cache.c | 4 1 file changed, 4 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_shader_cache.c b/src/mesa/drivers/dri/i965/brw_shader_cache.c index 08d43e7..5de261c 100644 ---

[Mesa-dev] [PATCH 56/65] i965: move binary sha1 gen into its own helper

And call it from write cache as its possible that read cache hasn't been called at this point or the current sha is the wrong one so we need to make sure a fresh sha is generated before writting. --- src/mesa/drivers/dri/i965/brw_shader_cache.c | 54 ++-- 1 file changed,

[Mesa-dev] [PATCH 61/65] i965: move vs outputs written into a helper

We will reuse this for fs key generation for the on disk shader cache. --- src/mesa/drivers/dri/i965/brw_vs.c | 72 ++ src/mesa/drivers/dri/i965/brw_vs.h | 4 +++ 2 files changed, 45 insertions(+), 31 deletions(-) diff --git

[Mesa-dev] [PATCH 50/65] i965: add shader cache support for pull param pointers

--- src/mesa/drivers/dri/i965/brw_shader_cache.c | 52 ++-- 1 file changed, 49 insertions(+), 3 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_shader_cache.c b/src/mesa/drivers/dri/i965/brw_shader_cache.c index 850e1e8..d1336d7 100644 ---

[Mesa-dev] [PATCH 47/65] glsl: add a cache_remove() function

This will be used to remove cache items created with an old version of Mesa from the cache. --- src/compiler/glsl/cache.c | 22 ++ src/compiler/glsl/cache.h | 6 ++ 2 files changed, 28 insertions(+) diff --git a/src/compiler/glsl/cache.c b/src/compiler/glsl/cache.c index

[Mesa-dev] [PATCH 51/65] i965: add cache fallback support

--- src/mesa/drivers/dri/i965/brw_shader_cache.c | 43 ++-- 1 file changed, 41 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_shader_cache.c b/src/mesa/drivers/dri/i965/brw_shader_cache.c index d1336d7..625ad3f 100644 ---

[Mesa-dev] [PATCH 55/65] glsl: skip more uniform initialisation when doing fallback linking

We already pull these values from the metadata cache so no need to recreate them. --- src/compiler/glsl/linker.cpp | 22 ++ src/mesa/main/shaderobj.c| 8 +--- 2 files changed, 19 insertions(+), 11 deletions(-) diff --git a/src/compiler/glsl/linker.cpp

[Mesa-dev] [PATCH 60/65] i965: don't precompile or rebuild resource list on cache fallback

--- src/mesa/drivers/dri/i965/brw_link.cpp | 8 +--- src/mesa/main/shaderobj.c | 2 +- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_link.cpp b/src/mesa/drivers/dri/i965/brw_link.cpp index 97bee3b..240e2db 100644 ---

[Mesa-dev] [PATCH 48/65] glsl: track mesa version shader cache items were created with

Also remove cache item and fallback to full recompile if current Mesa version differs. --- src/compiler/glsl/shader_cache.cpp | 12 1 file changed, 12 insertions(+) diff --git a/src/compiler/glsl/shader_cache.cpp b/src/compiler/glsl/shader_cache.cpp index d3eec83..52301b7 100644

[Mesa-dev] [PATCH 45/65] glsl: cache system values read bitfield

--- src/compiler/glsl/shader_cache.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/compiler/glsl/shader_cache.cpp b/src/compiler/glsl/shader_cache.cpp index 6fe89a7..d3eec83 100644 --- a/src/compiler/glsl/shader_cache.cpp +++ b/src/compiler/glsl/shader_cache.cpp @@ -919,6 +919,7 @@

[Mesa-dev] [PATCH 57/65] glsl,i965: disable shader cache for SSO

Caching SSO requires us to be a little more careful so disable it for now. --- src/compiler/glsl/linker.cpp | 2 +- src/mesa/drivers/dri/i965/brw_shader_cache.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/compiler/glsl/linker.cpp

[Mesa-dev] [PATCH 46/65] glsl: use reproducible name for lowered const arrays

Otherwise we can end up with mismatching names between the cached binary and the cached metadata. --- src/compiler/glsl/ir_optimization.h | 2 +- src/compiler/glsl/linker.cpp | 2 +- src/compiler/glsl/lower_const_arrays_to_uniforms.cpp | 16

[Mesa-dev] [PATCH 40/65] glsl: cache clip distance array size

--- src/compiler/glsl/shader_cache.cpp | 4 1 file changed, 4 insertions(+) diff --git a/src/compiler/glsl/shader_cache.cpp b/src/compiler/glsl/shader_cache.cpp index ff94dbf..d3d329c 100644 --- a/src/compiler/glsl/shader_cache.cpp +++ b/src/compiler/glsl/shader_cache.cpp @@ -898,6 +898,8

[Mesa-dev] [PATCH 41/65] glsl: cache some more image metadata

--- src/compiler/glsl/shader_cache.cpp | 12 1 file changed, 12 insertions(+) diff --git a/src/compiler/glsl/shader_cache.cpp b/src/compiler/glsl/shader_cache.cpp index d3d329c..f8c56ce 100644 --- a/src/compiler/glsl/shader_cache.cpp +++ b/src/compiler/glsl/shader_cache.cpp @@

[Mesa-dev] [PATCH 16/65] i965: don't crash when there is no shader and there are uniforms

--- src/mesa/drivers/dri/i965/brw_shader_cache.c | 35 1 file changed, 25 insertions(+), 10 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_shader_cache.c b/src/mesa/drivers/dri/i965/brw_shader_cache.c index c565e5a..e5aa022 100644 ---

[Mesa-dev] [PATCH 27/65] glsl: compile shaders when program not found in cache

We may have seen the individual shaders before and skipped compiling but they may not have been used together in this combination before. We could probably only compile the shaders which were skipped here but we need to be careful because the source may also have been changed since the last

[Mesa-dev] [PATCH 39/65] glsl: don't crash when dumping shaders if some come from cache

--- src/mesa/drivers/dri/i965/brw_link.cpp | 6 +- src/mesa/main/shaderapi.c | 6 +- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_link.cpp b/src/mesa/drivers/dri/i965/brw_link.cpp index d4bbd8b..7809b4c 100644 ---

[Mesa-dev] [PATCH 30/65] glsl, i965: don't used shader cache when transform feedback enabled

Note in future we may want to use the cache when in shader xfb qualifiers override the API. --- src/compiler/glsl/linker.cpp | 11 +++ src/compiler/glsl/shader_cache.cpp | 7 +++ src/mesa/drivers/dri/i965/brw_shader_cache.c | 7 +++ 3 files changed, 25

[Mesa-dev] [PATCH 25/65] glsl: add basic support for resource list to shader cache

This initially adds support for simple uniforms and varyings. --- src/compiler/glsl/shader_cache.cpp | 117 + 1 file changed, 117 insertions(+) diff --git a/src/compiler/glsl/shader_cache.cpp b/src/compiler/glsl/shader_cache.cpp index fae90ae..63aa450 100644

[Mesa-dev] [PATCH 17/65] glsl: fix uniform remap table cache when explicit locations used

--- src/compiler/glsl/shader_cache.cpp | 24 +++- 1 file changed, 15 insertions(+), 9 deletions(-) diff --git a/src/compiler/glsl/shader_cache.cpp b/src/compiler/glsl/shader_cache.cpp index 98015c4..0c9ae9f 100644 --- a/src/compiler/glsl/shader_cache.cpp +++

[Mesa-dev] [PATCH 29/65] glsl: skip linking when current program has been retrieved from cache

The scenario is a program has been linked for the first time and we cache the program metadata, then glLinkProgram() is called for a second time. Since we will now retrieve the program metadata from cache we need to skip linking. --- src/compiler/glsl/shader_cache.cpp | 1 + 1 file changed, 1

[Mesa-dev] [PATCH 09/65] i965: add new sha1 field to context struct

From: Carl Worth Signed-off-by: Timothy Arceri --- src/mesa/drivers/dri/i965/brw_context.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_context.h b/src/mesa/drivers/dri/i965/brw_context.h index

[Mesa-dev] [PATCH 22/65] glsl: cache more uniform storage values

--- src/compiler/glsl/shader_cache.cpp | 18 -- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/src/compiler/glsl/shader_cache.cpp b/src/compiler/glsl/shader_cache.cpp index 8f2e1ef..092e6e3 100644 --- a/src/compiler/glsl/shader_cache.cpp +++

[Mesa-dev] [PATCH 20/65] glsl: include bindings when creating sha1 for metadata

These bindings change the resulting binary so they are just as important as the shader source. --- src/compiler/glsl/shader_cache.cpp | 21 + 1 file changed, 21 insertions(+) diff --git a/src/compiler/glsl/shader_cache.cpp b/src/compiler/glsl/shader_cache.cpp index

[Mesa-dev] [PATCH 21/65] glsl: don't support caching single vs or fs shaders

We can add support for this later for now skip these programs. --- src/compiler/glsl/shader_cache.cpp | 5 + 1 file changed, 5 insertions(+) diff --git a/src/compiler/glsl/shader_cache.cpp b/src/compiler/glsl/shader_cache.cpp index 575c518..8f2e1ef 100644 ---

[Mesa-dev] [PATCH 26/65] glsl: add shader cache support for samplers

--- src/compiler/glsl/shader_cache.cpp | 25 + 1 file changed, 21 insertions(+), 4 deletions(-) diff --git a/src/compiler/glsl/shader_cache.cpp b/src/compiler/glsl/shader_cache.cpp index 63aa450..6d979ad 100644 --- a/src/compiler/glsl/shader_cache.cpp +++

[Mesa-dev] [PATCH 19/65] glsl: skip shader cache unless program is a vs+fs

--- src/compiler/glsl/shader_cache.cpp | 20 +++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git a/src/compiler/glsl/shader_cache.cpp b/src/compiler/glsl/shader_cache.cpp index 0c9ae9f..9c5095c 100644 --- a/src/compiler/glsl/shader_cache.cpp +++

[Mesa-dev] [PATCH 28/65] glsl: add support for skipping shader cache to compiler and linker

--- src/compiler/glsl/linker.cpp | 5 +++-- src/compiler/glsl/main.cpp | 2 +- src/compiler/glsl/program.h | 3 ++- src/mesa/main/ff_fragment_shader.cpp | 2 +- src/mesa/main/shaderapi.c| 2 +- src/mesa/program/ir_to_mesa.cpp | 5 +++--

[Mesa-dev] [PATCH 23/65] glsl: cache SSBO uniform storage values

--- src/compiler/glsl/shader_cache.cpp | 7 +++ 1 file changed, 7 insertions(+) diff --git a/src/compiler/glsl/shader_cache.cpp b/src/compiler/glsl/shader_cache.cpp index 092e6e3..daa9056 100644 --- a/src/compiler/glsl/shader_cache.cpp +++ b/src/compiler/glsl/shader_cache.cpp @@ -150,6

  1   2   3   >