[Mesa-dev] [PATCH] genxml: Add XML version tags

There's not much point to having them or not having them but this reduces some pointless diff from the version we can auto-generate --- src/intel/genxml/gen4.xml | 1 + src/intel/genxml/gen45.xml | 1 + src/intel/genxml/gen5.xml | 1 + src/intel/genxml/gen6.xml | 1 + src/intel/genxml/gen7.xml

[Mesa-dev] [Bug 100223] marshal_generated.c:38:10: fatal error: 'X11/Xlib-xcb.h' file not found

https://bugs.freedesktop.org/show_bug.cgi?id=100223 Bug ID: 100223 Summary: marshal_generated.c:38:10: fatal error: 'X11/Xlib-xcb.h' file not found Product: Mesa Version: git Hardware: x86-64 (AMD64) OS:

[Mesa-dev] [AppVeyor] mesa master #3750 completed

Build mesa 3750 completed Commit 04a9ca2700 by Timothy Arceri on 3/16/2017 4:43 AM: mapi: don't include X11/Xlib-xcb.h on non PTHREAD platforms\n\nShould fix the last of the glthread build issues on windows. Configure your notification preferences

Re: [Mesa-dev] [AppVeyor] mesa master #3747 failed

Hello Tim, this is a 'scons' build thing. Haven't have time to look, need some sleep. Dieter Am 16.03.2017 04:17, schrieb AppVeyor: BUILD MESA 3747 FAILED [1] Commit c83562ccaa by Marek Olšák on 2/5/2017 12:20 AM: gallium: implement the backend of threaded GL dispatch\n\nAcked-by: Timothy

[Mesa-dev] [PATCHi V2] spec: MESA_program_binary

In order to add functionality for OES_get_program_binary and ARB_get_program_binary we need a binary format enum. V2: share a single enum for all mesa drivers. Cc: Ian Romanick --- Hi Ian, Did you find out if Mesa still had some available enums assigned to it? Thanks,

Re: [Mesa-dev] [PATCH v3 07/24] i965/fs: generalize the legalization d2x pass

Samuel Iglesias Gonsálvez writes: > On 04/03/17 00:19, Francisco Jerez wrote: >> Samuel Iglesias Gonsálvez writes: >> >>> Add support to SEL instruction and add an assert to detect unsupported >>> instructions than do d2x conversions. >>> >>>

Re: [Mesa-dev] Mesa 17.0.1 release candidate

On 16/03/17 12:20 PM, Emil Velikov wrote: > On 16 March 2017 at 02:57, Matt Turner wrote: >> On Wed, Mar 15, 2017 at 7:55 PM, Emil Velikov >> wrote: >>> On 2 March 2017 at 03:44, Michel Dänzer wrote: On 02/03/17 03:35 AM,

Re: [Mesa-dev] Mesa 17.0.1 release candidate

On 16 March 2017 at 02:57, Matt Turner wrote: > On Wed, Mar 15, 2017 at 7:55 PM, Emil Velikov > wrote: >> On 2 March 2017 at 03:44, Michel Dänzer wrote: >>> On 02/03/17 03:35 AM, Emil Velikov wrote: >> >>> P.S. It would be

[Mesa-dev] [AppVeyor] mesa master #3747 failed

Build mesa 3747 failed Commit c83562ccaa by Marek Olšák on 2/5/2017 12:20 AM: gallium: implement the backend of threaded GL dispatch\n\nAcked-by: Timothy Arceri \nTested-by: Dieter Nützel \nTested-by: Mike Lothian

Re: [Mesa-dev] Mesa 17.0.1 release candidate

On Wed, Mar 15, 2017 at 11:05 PM, Matt Turner wrote: > On Mon, Mar 13, 2017 at 11:16 AM, Ilia Mirkin wrote: >> On Mon, Mar 13, 2017 at 2:13 PM, Matt Turner wrote: >>> I think Bcc'ing also means my gmail filters wouldn't know it came

Re: [Mesa-dev] Mesa 17.0.1 release candidate

On Mon, Mar 13, 2017 at 11:16 AM, Ilia Mirkin wrote: > On Mon, Mar 13, 2017 at 2:13 PM, Matt Turner wrote: >> I think Bcc'ing also means my gmail filters wouldn't know it came from >> the announce list. > > listid:mesa-announce.freedesktop.org should

Re: [Mesa-dev] Mesa 17.0.1 release candidate

On 16/03/17 11:57 AM, Matt Turner wrote: > On Wed, Mar 15, 2017 at 7:55 PM, Emil Velikov > wrote: >> On 2 March 2017 at 03:44, Michel Dänzer wrote: >>> On 02/03/17 03:35 AM, Emil Velikov wrote: >> >>> P.S. It would be better to put the mesa-announce

[Mesa-dev] [PATCH 4/4] aubinator: Use fprintf for output.

This will make it easier to choose an output file. For now, it remains stdout. Signed-off-by: Kenneth Graunke --- src/intel/tools/aubinator.c | 142 +++- 1 file changed, 73 insertions(+), 69 deletions(-) diff --git

[Mesa-dev] [PATCH 2/4] aubinator: Delete redundant NULL check.

handle_struct_decode() is just a wrapper around decode_structure() with a NULL check. But the only caller already does that NULL check. So, just use decode_structure() directly. --- src/intel/tools/aubinator.c | 13 +++-- 1 file changed, 3 insertions(+), 10 deletions(-) diff --git

[Mesa-dev] [PATCH 3/4] aubinator: Reuse decode_structure code for handling commands

The code for decoding structures and commands was almost identical. The only differences are: we print dword headers for commands, and we skip the first one (with the command opcode and lengths). So, generalize decode_structure to add a starting DWord, and a flag for printing the DWord headers,

[Mesa-dev] [PATCH 1/4] aubinator: Fix indentation.

Three space, not four. --- src/intel/tools/aubinator.c | 13 ++--- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/src/intel/tools/aubinator.c b/src/intel/tools/aubinator.c index e2bec8f9494..139637cfefa 100644 --- a/src/intel/tools/aubinator.c +++

Re: [Mesa-dev] [PATCH 00/28] update swr rasterizer

Hi Tim, On 16 March 2017 at 00:12, Tim Rowley wrote: > Hightlights include: lots of simd16 work, assert rework, and autogen > changes (scripts centralized, one file added, two removed). > > Tim Rowley (28): > swr: [rasterizer core] Support sparse numa id values on

Re: [Mesa-dev] Mesa 17.0.1 release candidate

On Wed, Mar 15, 2017 at 7:55 PM, Emil Velikov wrote: > On 2 March 2017 at 03:44, Michel Dänzer wrote: >> On 02/03/17 03:35 AM, Emil Velikov wrote: > >> P.S. It would be better to put the mesa-announce list only in Bcc on >> this kind of e-mails, to

Re: [Mesa-dev] [PATCH 3/3] hud_context: check NULL return from u_upload_alloc

On 13 March 2017 at 16:44, Marek Olšák wrote: > Hi, > > I've pushed the series with small commit message changes. > Hmm crash fixes - do we want these for stable, Marek ? -Emil ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org

Re: [Mesa-dev] Mesa 17.0.1 release candidate

On 2 March 2017 at 03:44, Michel Dänzer wrote: > On 02/03/17 03:35 AM, Emil Velikov wrote: > P.S. It would be better to put the mesa-announce list only in Bcc on > this kind of e-mails, to prevent accidental followups there. Reply-to: > mesa-dev seems useless for that. > So

Re: [Mesa-dev] Mesa 13.0.6 release candidate

On 15 March 2017 at 20:23, Matt Turner wrote: > On Wed, Mar 15, 2017 at 11:44 AM, Emil Velikov > wrote: >> Hello list, >> >> The candidate for the Mesa 13.0.6 is now available. Currently we have: >> - 104 queued >> - 1 nominated (outstanding) >>

[Mesa-dev] [PATCH] i965: avoid using a GNU make pattern rule

% pattern rules are a GNU extension. As there is only one file here avoid patterns and globbing entirely to fix the build on non-GNU make. Signed-off-by: Jonathan Gray --- src/mesa/drivers/dri/i965/Makefile.am | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff

[Mesa-dev] [PATCH] i965: remove uneeded asm/unistd.h include

Fix the build on OpenBSD by removing an uneeded include for asm/unistd.h. Signed-off-by: Jonathan Gray --- src/mesa/drivers/dri/i965/brw_performance_query.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/brw_performance_query.c

Re: [Mesa-dev] Lets try that again. Proposal for merging GL dispatch (aka glthread)

Am 14.03.2017 07:42, schrieb Dieter Nützel: Hello Tim, running this on r600g/Turks XT/6670 _without_ 0016 --- which didn't apply 'cause 'another version' was in? example: src/mapi/glapi/gen/GL3x.xml marshal_fail="_mesa_glthread_is_non_vbo_vertex_attrib_pointer(ctx)"> 'glmark2'

[Mesa-dev] [PATCH] opencl: Add $LIBDRM_LIBS

From: Michel Dänzer Can fail to link otherwise if libdrm is in a non-standard location. Signed-off-by: Michel Dänzer --- src/gallium/targets/opencl/Makefile.am | 1 + 1 file changed, 1 insertion(+) diff --git

Re: [Mesa-dev] [PATCH 7/7] util/disk_cache: also write key_blob to cache entry

On 16/03/17 10:09, Grazvydas Ignotas wrote: This can be used to deal with key hash collisions from different version (should we find that to actually happen) and to find which mesa version produced it the entry. Signed-off-by: Grazvydas Ignotas --- src/util/disk_cache.c |

Re: [Mesa-dev] MESA and KOTOR

I have created a simple stub for wglMakeContextCurrentARB in stw_wgl.c and stw_getprocaddress.c. It simply returns TRUE, but the good thing is that now the game no longer crashes because the function is missing! However I get a divide by zero in glu32.dll, presumably because the stub doesn't

Re: [Mesa-dev] MESA and KOTOR

That's good, can't wait to see your implementation. I have tried to simply return wglMakeCurrent(hReadDC,hglrc); but then I get a crash in gluBuild2DMipmaps (not mesa, glu32.dll). According to the specification, it should work, or at least draw some glitches. Looking at the parameters passed

Re: [Mesa-dev] [PATCH 6/7] util/disk_cache: add a write helper

On 16/03/17 10:09, Grazvydas Ignotas wrote: Simplifies the write code a bit and handles EINTR. Signed-off-by: Grazvydas Ignotas --- src/util/disk_cache.c | 40 ++-- 1 file changed, 26 insertions(+), 14 deletions(-) diff --git

Re: [Mesa-dev] [PATCH 5/7] util/disk_cache: hash pointer size and gpu name into cache keys

On 16/03/17 10:09, Grazvydas Ignotas wrote: This allows to get rid of the arch and gpu name directories. Also, gpu_name is now optional. Signed-off-by: Grazvydas Ignotas --- src/compiler/glsl/tests/cache_test.c | 13 ++ src/util/disk_cache.c| 47

Re: [Mesa-dev] [PATCH 6/4] radv: make use of on-disk cache

On 16/03/17 11:10, Jason Ekstrand wrote: On Tue, Mar 14, 2017 at 10:57 PM, Timothy Arceri > wrote: If the app provided and in-memory pipeline caches don't contain what we are looking for then we fallback to the on-disk cache. Not

Re: [Mesa-dev] [PATCH kmscube] cube-tex: Handle SMOOTH switch case

On Wednesday, 2017-03-15 18:56:53 -0300, Fabio Estevam wrote: > In kmscube.c there is the following logic: > > if (mode == SMOOTH) { > egl = init_cube_smooth(gbm); > } else { > egl = init_cube_tex(gbm, mode); > } > > ,which makes init_cube_tex() to

Re: [Mesa-dev] [PATCH 4/4] radv: make sure gs copy shader is retrieved from the cache with the variant

patch 1-4 are Reviewed-by: Bas Nieuwenhuizen and should be in my opinion pushable before any of the discussed changes to the disk cache. On Wed, Mar 15, 2017 at 5:17 AM, Timothy Arceri wrote: > Apps can limit the size of the cache via

[Mesa-dev] [PATCH 21/28] swr: [rasterizer jitter] Fix LogicOp blend jit after assert changes

--- .../drivers/swr/rasterizer/jitter/blend_jit.cpp| 35 +++--- 1 file changed, 25 insertions(+), 10 deletions(-) diff --git a/src/gallium/drivers/swr/rasterizer/jitter/blend_jit.cpp b/src/gallium/drivers/swr/rasterizer/jitter/blend_jit.cpp index cbcc343..55961d1 100644 ---

[Mesa-dev] [PATCH 28/28] swr: [rasterizer codegen] Rewrite gen_llvm_types.py to use mako

--- src/gallium/drivers/swr/Makefile.am| 9 +- .../swr/rasterizer/codegen/gen_llvm_types.py | 206 ++--- .../swr/rasterizer/codegen/templates/gen_llvm.hpp | 72 +++ 3 files changed, 170 insertions(+), 117 deletions(-) create mode 100644

[Mesa-dev] [PATCH 26/28] swr: [rasterizer codegen] Rewrite gen_llvm_ir_macros.py to use mako

Don't create/use cpp files, header only now. --- src/gallium/drivers/swr/Makefile.am| 61 +-- .../swr/rasterizer/codegen/gen_llvm_ir_macros.py | 411 +++-- .../codegen/templates/gen_builder_template.hpp | 47 +++ .../drivers/swr/rasterizer/jitter/builder.h

[Mesa-dev] [PATCH 18/28] swr: [rasterizer core/common] Fix the native AVX512 build under ICC

--- .../drivers/swr/rasterizer/common/simd16intrin.h | 26 +--- src/gallium/drivers/swr/rasterizer/core/pa.h | 46 -- src/gallium/drivers/swr/rasterizer/core/pa_avx.cpp | 2 +- 3 files changed, 47 insertions(+), 27 deletions(-) diff --git

[Mesa-dev] [PATCH 23/28] swr: [rasterizer core] fix trifan regression from 52f9f54dce

Fixes piglit triangle-rasterization-overdraw. SIMD16 path not working. --- src/gallium/drivers/swr/rasterizer/core/pa.h | 2 ++ src/gallium/drivers/swr/rasterizer/core/pa_avx.cpp | 14 +- 2 files changed, 11 insertions(+), 5 deletions(-) diff --git

[Mesa-dev] [PATCH 19/28] swr: [rasterizer core] Fix typo in SIMD16 code path

--- src/gallium/drivers/swr/rasterizer/core/pa.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/swr/rasterizer/core/pa.h b/src/gallium/drivers/swr/rasterizer/core/pa.h index 2e159da..afeed65 100644 --- a/src/gallium/drivers/swr/rasterizer/core/pa.h +++

[Mesa-dev] [PATCH 14/28] swr: [rasterizer archrast] Fix the early and late depthstencil events

The coverage and stencil mask arguments were reversed. --- src/gallium/drivers/swr/rasterizer/core/backend.cpp | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/gallium/drivers/swr/rasterizer/core/backend.cpp b/src/gallium/drivers/swr/rasterizer/core/backend.cpp

[Mesa-dev] [PATCH 17/28] swr: [rasterizer core] Allow no arguments to SWR_INVALID macro

Turns out this is somewhat tricky with gcc/g++. --- src/gallium/drivers/swr/rasterizer/common/swr_assert.h | 14 +- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/swr/rasterizer/common/swr_assert.h

[Mesa-dev] [PATCH 13/28] swr: [rasterizer core] Implement double pumped SIMD16 TESS

--- .../drivers/swr/rasterizer/core/frontend.cpp | 256 ++--- 1 file changed, 177 insertions(+), 79 deletions(-) diff --git a/src/gallium/drivers/swr/rasterizer/core/frontend.cpp b/src/gallium/drivers/swr/rasterizer/core/frontend.cpp index 761e044..b2f4643 100644 ---

[Mesa-dev] [PATCH 25/28] swr: [rasterizer codegen] Quiet gen_backends.py execution

--- src/gallium/drivers/swr/rasterizer/codegen/gen_backends.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/gallium/drivers/swr/rasterizer/codegen/gen_backends.py b/src/gallium/drivers/swr/rasterizer/codegen/gen_backends.py index cbbc378..d6367ca 100644 ---

[Mesa-dev] [PATCH 24/28] swr: [rasterizer scripts] Put codegen scripts into a separate directory

--- src/gallium/drivers/swr/Makefile.am| 126 ++--- .../{scripts => codegen}/gen_archrast.py | 0 .../{scripts => codegen}/gen_backends.py | 0 .../rasterizer/{scripts => codegen}/gen_knobs.py | 0 .../scripts =>

[Mesa-dev] [PATCH 15/28] swr: [rasterizer] Backend code adjustments

--- .../drivers/swr/rasterizer/common/simdintrin.h | 9 ++ src/gallium/drivers/swr/rasterizer/core/api.cpp| 7 +- .../drivers/swr/rasterizer/core/backend.cpp| 1 + src/gallium/drivers/swr/rasterizer/core/backend.h | 96 --

[Mesa-dev] [PATCH 10/28] swr: [rasterizer archrast/scripts] Further archrast cleanups

Removed redundant data being written out to file --- .../drivers/swr/rasterizer/archrast/archrast.cpp | 264 - .../scripts/templates/ar_eventhandler_h.template | 2 +- .../templates/ar_eventhandlerfile_h.template | 2 +- 3 files changed, 104 insertions(+), 164

[Mesa-dev] [PATCH 27/28] swr: [rasterizer codegen] Fix generation of knobs

--- src/gallium/drivers/swr/Makefile.am| 10 ++-- .../drivers/swr/rasterizer/codegen/gen_backends.py | 2 + .../drivers/swr/rasterizer/codegen/gen_knobs.py| 2 + .../swr/rasterizer/codegen/gen_llvm_ir_macros.py | 2 + .../codegen/templates/backend_template.cpp |

[Mesa-dev] [PATCH 09/28] swr: [rasterizer core] Fix RECT_LIST primitive assembly

The bug would make the 3rd component of attributes on the second triangle of a RECT be invalid. --- src/gallium/drivers/swr/rasterizer/core/pa_avx.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/swr/rasterizer/core/pa_avx.cpp

[Mesa-dev] [PATCH 20/28] swr: [rasterizer] Convert more SWR_ASSERT(false, ...) to SWR_INVALID(...)

--- .../drivers/swr/rasterizer/common/simdintrin.h | 2 +- src/gallium/drivers/swr/rasterizer/core/api.cpp | 2 +- src/gallium/drivers/swr/rasterizer/core/backend.cpp | 2 +- src/gallium/drivers/swr/rasterizer/core/binner.cpp | 2 +-

[Mesa-dev] [PATCH 22/28] swr: [rasterizer core] SIMD16 Frontend WIP - fix tesselation crashes

--- .../drivers/swr/rasterizer/core/frontend.cpp | 19 +- src/gallium/drivers/swr/rasterizer/core/pa.h | 7 ++-- src/gallium/drivers/swr/rasterizer/core/pa_avx.cpp | 40 -- 3 files changed, 35 insertions(+), 31 deletions(-) diff --git

[Mesa-dev] [PATCH 12/28] swr: [rasterizer archrast/core/scripts] Fix archrast multithreading issue

Per pixel stats are cached but were not always being flushed as threads moved from one draw context to the next. Added an explicit flush to allow all archrast objects to flush any cached events. --- .../drivers/swr/rasterizer/archrast/archrast.cpp | 52 +++---

[Mesa-dev] [PATCH 16/28] swr: [rasterizer] Slight assert refactoring

Make asserts more robust. Add SWR_INVALID(...) as a replacement for SWR_ASSERT(0, ...) --- .../drivers/swr/rasterizer/archrast/archrast.cpp | 2 +- .../drivers/swr/rasterizer/archrast/eventmanager.h | 2 +- .../drivers/swr/rasterizer/common/swr_assert.cpp | 331 ++---

[Mesa-dev] [PATCH 05/28] swr: [rasterizer archrast] Add additional API events

--- .../drivers/swr/rasterizer/archrast/events.proto | 36 ++ src/gallium/drivers/swr/rasterizer/core/api.cpp| 12 2 files changed, 48 insertions(+) diff --git a/src/gallium/drivers/swr/rasterizer/archrast/events.proto

[Mesa-dev] [PATCH 11/28] swr: [rasterizer archrast] Remove redundant data from archrast files

If count can be derived from other counts then this can be done in post processing scripts. --- .../drivers/swr/rasterizer/archrast/archrast.cpp | 42 ++--- .../drivers/swr/rasterizer/archrast/events.proto | 198 ++--- 2 files changed, 103 insertions(+), 137 deletions(-)

[Mesa-dev] [PATCH 02/28] swr: [rasterizer core] Finish SIMD16 PA OPT except tesselation

--- src/gallium/drivers/swr/rasterizer/core/pa.h | 84 - src/gallium/drivers/swr/rasterizer/core/pa_avx.cpp | 1625 +--- 2 files changed, 1420 insertions(+), 289 deletions(-) diff --git a/src/gallium/drivers/swr/rasterizer/core/pa.h

[Mesa-dev] [PATCH 08/28] swr: [rasterizer common] Add InterpolateComponentFlat utility

--- src/gallium/drivers/swr/rasterizer/common/simdintrin.h | 13 + 1 file changed, 13 insertions(+) diff --git a/src/gallium/drivers/swr/rasterizer/common/simdintrin.h b/src/gallium/drivers/swr/rasterizer/common/simdintrin.h index 8926e66..ea79902 100644 ---

[Mesa-dev] [PATCH 01/28] swr: [rasterizer core] Support sparse numa id values on all OSes

--- .../drivers/swr/rasterizer/core/threads.cpp| 80 ++ 1 file changed, 53 insertions(+), 27 deletions(-) diff --git a/src/gallium/drivers/swr/rasterizer/core/threads.cpp b/src/gallium/drivers/swr/rasterizer/core/threads.cpp index f1c3030..e11291b 100644 ---

[Mesa-dev] [PATCH 06/28] swr: [rasterizer core] Implement SIMD16 GS and STREAMOUT

--- .../drivers/swr/rasterizer/core/frontend.cpp | 302 + 1 file changed, 251 insertions(+), 51 deletions(-) diff --git a/src/gallium/drivers/swr/rasterizer/core/frontend.cpp b/src/gallium/drivers/swr/rasterizer/core/frontend.cpp index 18728e5..761e044 100644 ---

[Mesa-dev] [PATCH 00/28] update swr rasterizer

Hightlights include: lots of simd16 work, assert rework, and autogen changes (scripts centralized, one file added, two removed). Tim Rowley (28): swr: [rasterizer core] Support sparse numa id values on all OSes swr: [rasterizer core] Finish SIMD16 PA OPT except tesselation swr: [rasterizer

[Mesa-dev] [PATCH 03/28] swr: [rasterizer core] Finish SIMD16 PA OPT including tesselation

--- src/gallium/drivers/swr/rasterizer/core/pa_avx.cpp | 268 +++-- 1 file changed, 247 insertions(+), 21 deletions(-) diff --git a/src/gallium/drivers/swr/rasterizer/core/pa_avx.cpp b/src/gallium/drivers/swr/rasterizer/core/pa_avx.cpp index 22643bd..6fb37e5 100644 ---

[Mesa-dev] [PATCH 07/28] swr: [rasterizer archrast] Fix performance issue with archrast stats

Performance is now 50x faster with archrast now that we're properly filtering out all of the rdtsc begin/end. --- .../drivers/swr/rasterizer/archrast/archrast.cpp | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git

[Mesa-dev] [PATCH 04/28] swr: [rasterizer core/scripts] Autogen backend initialization function(s)

Autogen functions that instantiates different BackendPixelRate templates. Functions get split into separate files after reaching a user defined threshold (currently 512 per file) to speed up compilation. This change will enable the addition of more template flags in the pixel back end. ---

Re: [Mesa-dev] [PATCH 6/4] radv: make use of on-disk cache

On Tue, Mar 14, 2017 at 10:57 PM, Timothy Arceri wrote: > If the app provided and in-memory pipeline caches don't contain > what we are looking for then we fallback to the on-disk cache. > Not my driver so my opinion doesn't count for much here, but this seems like a

[Mesa-dev] [PATCH 05/12] anv/query: Let 32-bit values wrap

From the Vulkan 1.0.39 Specification: "If VK_QUERY_RESULT_64_BIT is not set and the result overflows a 32-bit value, the value may either wrap or saturate." So we can either clamp or wrap. Wrapping is both easier and what the user gets if they use vkCmdCopyQueryPoolResults and we should

[Mesa-dev] [PATCH 11/12] anv/query: Rework store_query_result

The new version is a nice GPU parallel to cpu_write_query_result and it nicely handles things like dealing with 32 vs. 64-bit offsets in the destination buffer. --- src/intel/vulkan/genX_query.c | 39 --- 1 file changed, 24 insertions(+), 15 deletions(-) diff

[Mesa-dev] [PATCH 10/12] anv/query: Break GPU query calculation into a helper

--- src/intel/vulkan/genX_query.c | 30 ++ 1 file changed, 18 insertions(+), 12 deletions(-) diff --git a/src/intel/vulkan/genX_query.c b/src/intel/vulkan/genX_query.c index b8d4c55..6c26e6a 100644 --- a/src/intel/vulkan/genX_query.c +++

[Mesa-dev] [PATCH 12/12] anv: implement pipeline statistics queries

From: Ilia Mirkin In the end, pipeline statistics queries look a lot like occlusion queries only with between 1 and 11 begin/end pairs being generated instead of just the one. --- src/intel/vulkan/anv_device.c | 2 +- src/intel/vulkan/anv_private.h | 3 +

[Mesa-dev] [PATCH 07/12] anv/query: Use a variable-length slot size

Not all queries are the same. Even the two queries we support today require a different amount of data per slot. Once we introduce pipeline statistics queries, the size will vary wildly. --- src/intel/vulkan/anv_private.h | 9 +++- src/intel/vulkan/genX_query.c | 52

[Mesa-dev] [PATCH 03/12] anv/GetQueryPoolResults: Actually implement the spec

The Vulkan spec is fairly clear about when we should and should not write query pool results. We're also supposed to return VK_NOT_READY if VK_QUERY_RESULT_PARTIAL_BIT is not set and we come across any queries which are not yet finished. This fixes rendering corruptions on The Talos Principle

[Mesa-dev] [PATCH 00/12] anv: Queries!

This little patch series does a number of things. The first three patches fix bugs in the current query code. These bugs have been causing rendering corruptions in The Talos Principle. The next 8 patches are various clean-ups and bits of prep-work for implementing pipeline statistics queries.

[Mesa-dev] [PATCH 08/12] anv/query: Add a helper for writing a query pool result

--- src/intel/vulkan/genX_query.c | 33 + 1 file changed, 17 insertions(+), 16 deletions(-) diff --git a/src/intel/vulkan/genX_query.c b/src/intel/vulkan/genX_query.c index 2d8f352..b8d4c55 100644 --- a/src/intel/vulkan/genX_query.c +++

[Mesa-dev] [PATCH 06/12] anv/query: Move the available bits to the front

We're about to make slots variable-length and always having the available bits at the front makes certain operations substantially easier once we do that. --- src/intel/vulkan/anv_private.h | 2 +- src/intel/vulkan/genX_query.c | 43 +- 2 files changed,

[Mesa-dev] [PATCH 09/12] genxml: Add pipeline statistics registers on gen7+

--- src/intel/genxml/gen7.xml | 44 src/intel/genxml/gen75.xml | 44 src/intel/genxml/gen8.xml | 44 src/intel/genxml/gen9.xml | 44

[Mesa-dev] [PATCH 01/12] anv/query: Fix the location of timestamp availability

--- src/intel/vulkan/genX_query.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/intel/vulkan/genX_query.c b/src/intel/vulkan/genX_query.c index 830f867..4e6638a 100644 --- a/src/intel/vulkan/genX_query.c +++ b/src/intel/vulkan/genX_query.c @@ -327,7 +327,7 @@ void

[Mesa-dev] [PATCH 02/12] anv/query: Invalidate the correct range

Cc: "17.0 13.0" --- src/intel/vulkan/genX_query.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/intel/vulkan/genX_query.c b/src/intel/vulkan/genX_query.c index 4e6638a..72ac2cb 100644 --- a/src/intel/vulkan/genX_query.c +++

[Mesa-dev] [PATCH 04/12] genxml: Make MI_STORE_DATA_IMM have a single 64-bit data field

This is way more convenient than having two separate dword fields. --- src/intel/genxml/gen6.xml | 3 +-- src/intel/genxml/gen7.xml | 3 +-- src/intel/genxml/gen75.xml| 3 +-- src/intel/genxml/gen8.xml | 3 +-- src/intel/genxml/gen9.xml | 3 +-- src/intel/vulkan/genX_query.c |

Re: [Mesa-dev] [PATCH 4/7] isl: Validate the calculated row pitch

On Wed, Mar 15, 2017 at 3:34 PM, Nanley Chery wrote: > On Mon, Mar 13, 2017 at 03:28:01PM -0700, Chad Versace wrote: > > Validate that it fits in RENDER_SURFACE_STATE::SurfacePitch or, if it's > > an aux surface, AuxiliarySurfacePitch. > > --- > > src/intel/isl/isl.c | 35

[Mesa-dev] [PATCH 7/7] util/disk_cache: also write key_blob to cache entry

This can be used to deal with key hash collisions from different version (should we find that to actually happen) and to find which mesa version produced it the entry. Signed-off-by: Grazvydas Ignotas --- src/util/disk_cache.c | 24 ++-- 1 file changed, 22

[Mesa-dev] [PATCH v2 1/7] util/disk_cache: use a helper to compute cache keys

This will allow to hash additional data into the cache keys or even change the hashing algorithm easily, should we decide to do so. v2: don't try to compute key (and crash) if cache is disabled Signed-off-by: Grazvydas Ignotas --- src/compiler/glsl/glsl_parser_extras.cpp

[Mesa-dev] [PATCH v2 3/7] util/disk_cache: hash timestamps into the cache keys

Instead of using a directory, hash the timestamps into the cache keys themselves. Since there is no more timestamp directory, there is no more need for deleting the cache of other mesa versions and we rely on eviction to clean up the old cache entries. This solves the problem of using several

[Mesa-dev] [PATCH 2/7] tests/cache_test: use the blob key's actual first byte

There is no need to hardcode it, we can just use blob_key[0]. This is needed because the next patches are going to change how cache keys are computed. Signed-off-by: Grazvydas Ignotas --- src/compiler/glsl/tests/cache_test.c | 8 ++-- 1 file changed, 2 insertions(+), 6

[Mesa-dev] [PATCH 4/7] util/disk_cache: pass timestamps as a blob

We no longer need a string, so can just pass the timestamps as a blob to reduce amount of data that needs to be hashed into the keys. The argument is also renamed to version_blob because the cache doesn't really care if it's timestamps, build id or something else. Signed-off-by: Grazvydas Ignotas

[Mesa-dev] [PATCH 6/7] util/disk_cache: add a write helper

Simplifies the write code a bit and handles EINTR. Signed-off-by: Grazvydas Ignotas --- src/util/disk_cache.c | 40 ++-- 1 file changed, 26 insertions(+), 14 deletions(-) diff --git a/src/util/disk_cache.c b/src/util/disk_cache.c index

[Mesa-dev] [PATCH 5/7] util/disk_cache: hash pointer size and gpu name into cache keys

This allows to get rid of the arch and gpu name directories. Also, gpu_name is now optional. Signed-off-by: Grazvydas Ignotas --- src/compiler/glsl/tests/cache_test.c | 13 ++ src/util/disk_cache.c| 47 +++-

[Mesa-dev] [PATCH v2 0/7] timestamp/arch/gpu dir removal for disk_cache

This is version 2 of the series that removes extra cache directories and hashes that information into cache keys instead. v1: https://lists.freedesktop.org/archives/mesa-dev/2017-March/147632.html Grazvydas Ignotas (7): util/disk_cache: use a helper to compute cache keys tests/cache_test: use

[Mesa-dev] [Bug 100201] llvmpipe Windows scons build with Visual Studio 2015 toolchain and LLVM 4.0 fails

https://bugs.freedesktop.org/show_bug.cgi?id=100201 --- Comment #1 from Jose Fonseca --- Log shows that the problem is not LLVM 4.0 per se, but rather MSVC 2017 headers: C:\Software\Development\projects\mesa\mesa\include\c11/threads.h(44): error C2011: 'xtime': 'struct'

Re: [Mesa-dev] [PATCH] util/disk_cache: do eviction before creating .tmp

Pushed, Thanks! ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [Bug 100202] llvmpipe Windows scons build can't detect Visual Studio 2017 toolchain

https://bugs.freedesktop.org/show_bug.cgi?id=100202 Jose Fonseca changed: What|Removed |Added Resolution|--- |NOTOURBUG

Re: [Mesa-dev] [PATCH 4/7] isl: Validate the calculated row pitch

On Mon, Mar 13, 2017 at 03:28:01PM -0700, Chad Versace wrote: > Validate that it fits in RENDER_SURFACE_STATE::SurfacePitch or, if it's > an aux surface, AuxiliarySurfacePitch. > --- > src/intel/isl/isl.c | 35 +-- > 1 file changed, 29 insertions(+), 6 deletions(-)

[Mesa-dev] [PATCH kmscube] cube-tex: Handle SMOOTH switch case

In kmscube.c there is the following logic: if (mode == SMOOTH) { egl = init_cube_smooth(gbm); } else { egl = init_cube_tex(gbm, mode); } ,which makes init_cube_tex() to be never called on the SMOOTH case. Handle the SMOOTH switch case

[Mesa-dev] [PATCH 2/2] st/nine: Use atomics for available_texture_mem

Resource dtor can be executed in the worker thread. Use atomic to avoid threading safety issues. CC: "17.0" Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/resource9.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

[Mesa-dev] [PATCH 1/2] st/nine: Fix issue with surface and volume dtors and csmt

Surfaces and Volumes can be freed in the worker thread. This fixes these dtor. Especially if they were freed by nine_context_box_upload, the counter was decremented after the destructions, which lead to deadlock. Fixes bugzilla 99246. CC: "17.0"

Re: [Mesa-dev] [PATCH kmscube 7/7] drm-legacy.c: suppress 'unused parameter warnings

Hi Eric, On Tue, Mar 14, 2017 at 10:33 AM, Eric Engestrom wrote: > Signed-off-by: Eric Engestrom > --- > drm-legacy.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drm-legacy.c b/drm-legacy.c > index 2392a3d..8e49075 100644 >

[Mesa-dev] [PATCH] st/nine: Fix user vertex data uploader with csmt

Fix regression caused by abb1c645c476b5dd289ce3efae0594f8796f9cf8 The patch made csmt use context.pipe instead of secondary_pipe, leading to thread safety issues. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 15 +++

[Mesa-dev] [Bug 99246] [d3dadapter+radeonsi & bisect] EVE-Online : hang on wormhole sight

https://bugs.freedesktop.org/show_bug.cgi?id=99246 --- Comment #10 from Mathieu Belanger --- @Axel Davy Yes! Thanks! -- You are receiving this mail because: You are the assignee for the bug. You are the QA Contact for the bug.___

Re: [Mesa-dev] Mesa 13.0.6 release candidate

On Wed, Mar 15, 2017 at 11:44 AM, Emil Velikov wrote: > Hello list, > > The candidate for the Mesa 13.0.6 is now available. Currently we have: > - 104 queued > - 1 nominated (outstanding) > - and 3 rejected patch(es) I got an email from you confirming that you picked

Re: [Mesa-dev] [PATCH] swr: validate backend state numAttributes

Reviewed-by: Bruce Cherniak > On Mar 15, 2017, at 11:51 AM, Rowley, Timothy O > wrote: > > General protection, and prevents us from smashing the stack > on the first clear state validation (a7b8d50bcb). Fixes crash > using icc. > --- >

Re: [Mesa-dev] [PATCH 1/5] util/disk_cache: seed rand

On Wed, Mar 15, 2017 at 8:47 PM, Bas Nieuwenhuizen wrote: > I thought rand() was not thread safe anyway, and hence we can't use it > in mesa as we don't know what application threads call it too? My manpage says "MT-Safe". > > On Tue, Mar 14, 2017 at 3:08 AM, Timothy

Re: [Mesa-dev] [PATCH] util/disk_cache: pass predicate functions file stats directly (v2)

On Wed, Mar 15, 2017 at 3:01 PM, Alan Swanson wrote: > Since switching to LRU eviction the only user of these predicate > functions now resolves directory entry stats itself so pass them > directly saving calling fstat and strlen twice (and the > expensive strlen is

Re: [Mesa-dev] [PATCH 3/3] docs/releasing: document how to squash/announce queued patches

On Wednesday, 2017-03-15 17:34:26 +, Emil Velikov wrote: > From: Emil Velikov > > In the odd case where a patch needs to be fixed, squash the appropriate > fix and document how. Add a note in the pre-release notes, such that > devs can quickly spot it. > >

  1   2   >