On 16/03/17 03:01 PM, Timothy Arceri wrote:
> glNewList() swaps dispatch tables, and we don't have anything in
> place to handle that in glthread.
This prevents three piglit tests from crashing with mesa_glthread=true.
(Many more left though, as discussed on IRC)
Tested-by: Michel Dänzer
--
E
On 16/03/17 04:23, Francisco Jerez wrote:
> 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.
>>>
On 16/03/17 04:23, Francisco Jerez wrote:
> 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.
>>>
According to dl_iterate_phdr man page first object visited is the
main program where dlpi_name is an empty string. This fixes segfault
on Android when using build-id as identifier.
Signed-off-by: Tapani Pälli
---
src/util/build_id.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/u
radv has failed the SPIR-V isinf tests for a good while, I've talked
to Matt and the llvm backend matches on an ordered feq to infinity,
where as we map the nir feq to unordered.
06:00 < arsenm> airlied: unordered eq for nan input would say true. isinf for
NAN says false
So it loo
glNewList() swaps dispatch tables, and we don't have anything in
place to handle that in glthread.
---
src/mapi/glapi/gen/gl_API.xml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mapi/glapi/gen/gl_API.xml b/src/mapi/glapi/gen/gl_API.xml
index 15d7e4f..c1f0f8f 100644
--- a
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
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: Mac
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
__
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 Ar
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,
Tim
docs/specs/MESA_pr
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.
>>>
>>> Signed-off-by: Samuel Iglesias Gonsálvez
>>> ---
>>
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, Emil Velikov wrote:
>>>
P.S. It would be better to put the mesa
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 better to put the mesa-announce list only in Bcc on
>>> this kind of e-m
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
Configure your notification preferen
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 from
>>> the announce list.
>>
>> listid:mesa-announce.freedesk
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 pick it up - even though
> it's bcc'd, the mai
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 list only in Bcc on
>>> this kind of e-mails,
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 a/src/intel/tools/aubinator.c b/src/int
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 a/src
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, an
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
+++ b/src/intel/tools/aubinator
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 all OSes
> swr: [rasterizer
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 prevent accidental followups there. Reply-to:
>>
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
https://lists.freed
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 for 13.0.6-rc1 I've ad
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)
>> - and 3 rejected patch(es)
>
> I got an email f
% 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 --git a/src/mesa/
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
b/src/mesa/drivers/dri/i965/brw_perfo
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' w
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 a/src/gallium/targets/opencl/Makefile.am
b/src/gallium/targets/opencl/Makefile.am
i
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 | 24 ++
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 do
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 by
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 a/src/util/disk_cache.c b/src
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 +++---
On 16/03/17 11:10, Jason Ekstrand wrote:
On Tue, Mar 14, 2017 at 10:57 PM, Timothy Arceri mailto:tarc...@itsqueeze.com>> 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
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 be
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 VkAllocationCallbacks so we
> can't be sure that both are a
---
.../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
--- a
---
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
src/gallium/d
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
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 a/src/gallium/drivers/swr/ra
---
.../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 a/src/gallium/drivers
---
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
+++ b/s
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
i
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
b/src/gallium/drivers/swr/rasterizer/common/swr_asse
---
.../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
--- a/src/
---
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
--- a/src/
---
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 => codegen}/gen_llvm_ir_macr
---
.../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 --
src/gallium/drivers/swr/rasterizer/
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 del
---
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 | 61
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
b/src/gallium/drivers/s
---
.../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 +-
src/gallium/drivers/swr/rasterizer/core/blend.
---
.../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 a/src/gallium/drivers
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 +++---
.../drivers/swr/ra
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 ++---
.../
---
.../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
b/src/gallium/drivers/swr/rasterizer/archr
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(-)
diff
---
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
b/src/gallium/drivers/swr/rasterizer/co
---
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
--- a/src/gallium/drivers/swr
---
.../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
--- a/src/gal
---
.../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
--- a/src/
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 c
---
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
--- a/src/gall
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 a/src/gallium/drivers/swr/rasterizer/archr
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.
---
src/ga
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 very un-Vulkan thing to do
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 be
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 -
---
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
+++ b/src/intel/vulkan/genX_query.c
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 +
src/intel/vulkan/genX_query.c | 23
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 +++
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 whe
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. T
---
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
+++ b/src/intel/vulkan/genX_quer
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, 18
---
src/intel/genxml/gen7.xml | 44
src/intel/genxml/gen75.xml | 44
src/intel/genxml/gen8.xml | 44
src/intel/genxml/gen9.xml | 44
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
+++ b/src/intel/vulkan/genX_query.c
@@ -132
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 |
---
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 genX(Cm
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 +++
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 insertions(+), 2 de
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| 27 +++
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 incar
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 deletions(-)
diff --g
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
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 904aa66..ad591be 100644
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 +++-
src/util/disk_cache.h
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
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' type redefinition
C:\Pro
Pushed, Thanks!
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
https://bugs.freedesktop.org/show_bug.cgi?id=100202
Jose Fonseca changed:
What|Removed |Added
Resolution|--- |NOTOURBUG
Status|NEW
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(-)
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 insid
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(-)
diff --git a/src/gallium/state_trackers/nine/resource9.
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"
Signed-off-by: Axel Davy
---
src/gallium/state_
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
> --- a/drm-legacy.c
> +++ b/drm-legacy.c
> @@ -33,6 +33,
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 +++
src/gallium/state_trackers/nine/device9
1 - 100 of 186 matches
Mail list logo
