[Mesa-dev] [PATCH v2 3/7] travis: Add clover build using llvm-3.7

2017-09-16 Thread Jan Vesely
v2: Use direct llvm repo link instead of alias Enable Signed-off-by: Jan Vesely --- .travis.yml | 40 1 file changed, 40 insertions(+) diff --git a/.travis.yml b/.travis.yml index 0012d27dc7..638ec2a78a 100644 --- a/.travis.yml +++ b/.travis.yml @@ -

[Mesa-dev] [Bug 102573] fails to build on armel

2017-09-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=102573 --- Comment #4 from Shmerl --- (In reply to Grazvydas Ignotas from comment #3) > On what conditions does this show up? Mesa git builds fine on debian stretch > and it's gcc 6.3.0, gcc is generating __sync_* calls even when __atomic_* is > explic

[Mesa-dev] [PATCH 5/7] travis: Add clover build using llvm-3.9

2017-09-16 Thread Jan Vesely
Use r600,radeonsi instead of i915 Update binutils, new linker is required for llvm-3.9: https://www.ubuntuupdates.org/package/core/trusty/universe/updates/binutils-2.26 Signed-off-by: Jan Vesely --- Let me know if there is a better ay to use the new linker than overloading path .travis.yml | 41

[Mesa-dev] [PATCH 4/7] travis: Add clover build using llvm-3.8

2017-09-16 Thread Jan Vesely
Signed-off-by: Jan Vesely --- .travis.yml | 38 ++ 1 file changed, 38 insertions(+) diff --git a/.travis.yml b/.travis.yml index a2df2073d1..4e5510e800 100644 --- a/.travis.yml +++ b/.travis.yml @@ -191,6 +191,44 @@ matrix: #- libelf-dev #

[Mesa-dev] [PATCH 1/7] clover: add missing include to compat.h

2017-09-16 Thread Jan Vesely
Fixes build issues with llvm-3.6 Fixes: 3115687f9b9830417c408228db2bc679e346bba6 (clover: Fix build after LLVM r313390) Signed-off-by: Jan Vesely --- src/gallium/state_trackers/clover/llvm/compat.hpp | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/state_trackers/clover/llvm/compa

[Mesa-dev] [PATCH 7/7] travis: Add clover build using llvm-5.0

2017-09-16 Thread Jan Vesely
Signed-off-by: Jan Vesely --- .travis.yml | 36 1 file changed, 36 insertions(+) diff --git a/.travis.yml b/.travis.yml index dc05dc0c14..5f72024817 100644 --- a/.travis.yml +++ b/.travis.yml @@ -304,6 +304,42 @@ matrix: - libelf-dev

[Mesa-dev] [PATCH 2/7] travis: Rename clover job to include llvm version

2017-09-16 Thread Jan Vesely
Signed-off-by: Jan Vesely --- .travis.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.travis.yml b/.travis.yml index 435216fc8f..0012d27dc7 100644 --- a/.travis.yml +++ b/.travis.yml @@ -115,7 +115,7 @@ matrix: - libunwind8-dev - env: # NOTE: An

[Mesa-dev] [PATCH 6/7] travsi: Add clover build using llvm-4.0

2017-09-16 Thread Jan Vesely
llvm-4 needs gcc 4.8: http://releases.llvm.org/4.0.1/docs/ReleaseNotes.html#non-comprehensive-list-of-changes-in-this-release Signed-off-by: Jan Vesely --- .travis.yml | 36 1 file changed, 36 insertions(+) diff --git a/.travis.yml b/.travis.yml index 3b1cbd

[Mesa-dev] [PATCH RFC 3/7] travis: Add clover build using llvm-3.7

2017-09-16 Thread Jan Vesely
Disable for now, travis does not allow llvm-3.7 trusty toolchain Signed-off-by: Jan Vesely --- This patch is included just for completeness. I'm not sure what the right approach is here travis is probably not going to include 3.7 toolchain: https://github.com/travis-ci/apt-source-whitelist/issues

Re: [Mesa-dev] [PATCH] gbm: Add gbm_device_get_format_modifier_plane_count to symbols-check

2017-09-16 Thread Jason Ekstrand
Rb On September 16, 2017 10:23:22 AM Andres Gomez wrote: Fixes: 8824141b8d4("gbm: Add a gbm_device_get_format_modifier_plane_count function") Signed-off-by: Andres Gomez Cc: Jason Ekstrand Cc: Emil Velikov Cc: Daniel Stone --- src/gbm/gbm-symbols-check | 1 + 1 file changed, 1 insertion

Re: [Mesa-dev] [PATCH] gbm: Add gbm_device_get_format_modifier_plane_count to test

2017-09-16 Thread Eric Engestrom
On Saturday, 2017-09-16 16:04:15 +, Gert Wollny wrote: > Adding gbm_device_get_format_modifier_plane_count made the > test gbm-symbols-check fail, this patch adds the according > function name to the test. > > Fixes: 8824141b8d48d9120ddbf542d6fb661046c41c62 > (gbm: Add a gbm_device_get_format

[Mesa-dev] [PATCH mesa] docs: update sourcetree following omx rename

2017-09-16 Thread Eric Engestrom
Fixes: 6a8aa11c207b99920b93 "st/omx_bellagio: Rename state tracker and option" Cc: Gurkirpal Singh Signed-off-by: Eric Engestrom --- docs/sourcetree.html | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/sourcetree.html b/docs/sourcetree.html index 9448df58c9..07c8ece6ea 1

Re: [Mesa-dev] [PATCH] travis: replace omx feature flag with omx-bellagio one

2017-09-16 Thread Eric Engestrom
On Saturday, 2017-09-16 17:24:50 +, Andres Gomez wrote: > Fixes: 6a8aa11c207 ("st/omx_bellagio: Rename state tracker and > option") > Signed-off-by: Andres Gomez > Cc: Gurkirpal Singh > Cc: Eric Engestrom Reviewed-by: Eric Engestrom > Cc: Emil Velikov > --- > .travis.yml | 12 ++

Re: [Mesa-dev] Breaks building with llvm-3.6, was: [PATCH 1/1] clover: Fix build after LLVM r313390

2017-09-16 Thread Jan Vesely
On Sat, 2017-09-16 at 18:30 +0200, Gert Wollny wrote: > Am Freitag, den 15.09.2017, 21:41 -0400 schrieb Jan Vesely: > [...] > > > > > > > a/src/gallium/state_trackers/clover/llvm/compat.hpp > > > > > > b/src/gallium/state_trackers/clover/llvm/compat.hpp > > > > > > index b22ecfa6b2..9bca6d82e8 100

[Mesa-dev] [Bug 102809] Rust shadows(?) flash random colours

2017-09-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=102809 Bug ID: 102809 Summary: Rust shadows(?) flash random colours Product: Mesa Version: git Hardware: x86-64 (AMD64) OS: Linux (All) Status: NEW Severity: no

Re: [Mesa-dev] [PATCH] travis: replace omx feature flag with omx-bellagio one

2017-09-16 Thread Christian König
Am 16.09.2017 um 19:23 schrieb Andres Gomez: Fixes: 6a8aa11c207 ("st/omx_bellagio: Rename state tracker and option") Signed-off-by: Andres Gomez Cc: Gurkirpal Singh Cc: Eric Engestrom Cc: Emil Velikov Reviewed-by: Christian König --- .travis.yml | 12 ++-- 1 file changed, 6 in

[Mesa-dev] [PATCH] travis: replace omx feature flag with omx-bellagio one

2017-09-16 Thread Andres Gomez
Fixes: 6a8aa11c207 ("st/omx_bellagio: Rename state tracker and option") Signed-off-by: Andres Gomez Cc: Gurkirpal Singh Cc: Eric Engestrom Cc: Emil Velikov --- .travis.yml | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/.travis.yml b/.travis.yml index 51061d953

[Mesa-dev] [PATCH] gbm: Add gbm_device_get_format_modifier_plane_count to symbols-check

2017-09-16 Thread Andres Gomez
Fixes: 8824141b8d4("gbm: Add a gbm_device_get_format_modifier_plane_count function") Signed-off-by: Andres Gomez Cc: Jason Ekstrand Cc: Emil Velikov Cc: Daniel Stone --- src/gbm/gbm-symbols-check | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gbm/gbm-symbols-check b/src/gbm/gbm-symbo

[Mesa-dev] [PATCH] configure+mesa/st/tests: avoid building with pre c++11 compilers

2017-09-16 Thread Gert Wollny
The tests in mesa/st/tests require C++11. Check whether the compiler supports the -std=c++11 flag and if so, enable the tests. Otherwise skip compiling and running the tests altogether. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102665 --- - Submitter has no write access to mesa-git.

[Mesa-dev] Breaks building with llvm-3.6, was: [PATCH 1/1] clover: Fix build after LLVM r313390

2017-09-16 Thread Gert Wollny
Am Freitag, den 15.09.2017, 21:41 -0400 schrieb Jan Vesely: [...] > > > > > a/src/gallium/state_trackers/clover/llvm/compat.hpp > > > > > b/src/gallium/state_trackers/clover/llvm/compat.hpp > > > > > index b22ecfa6b2..9bca6d82e8 100644 > > > > > --- a/src/gallium/state_trackers/clover/llvm/compat.

[Mesa-dev] [PATCH] gbm: Add gbm_device_get_format_modifier_plane_count to test

2017-09-16 Thread Gert Wollny
Adding gbm_device_get_format_modifier_plane_count made the test gbm-symbols-check fail, this patch adds the according function name to the test. Fixes: 8824141b8d48d9120ddbf542d6fb661046c41c62 (gbm: Add a gbm_device_get_format_modifier_plane_count function) Signed-off-by: Gert Wollny --- Submit

Re: [Mesa-dev] [PATCH] anv: Fix descriptors copying

2017-09-16 Thread Jason Ekstrand
Oh, and Reviewed-by: Jason Ekstrand On September 16, 2017 08:55:16 Jason Ekstrand wrote: Clearly there are no tests for this... If you wanted to go above and beyond, you could write I've. :-) On September 16, 2017 05:24:25 Józef Kucia wrote: Trivial. Cc: mesa-sta...@lists.freedeskto

Re: [Mesa-dev] [PATCH] anv: Fix descriptors copying

2017-09-16 Thread Jason Ekstrand
Clearly there are no tests for this... If you wanted to go above and beyond, you could write I've. :-) On September 16, 2017 05:24:25 Józef Kucia wrote: Trivial. Cc: mesa-sta...@lists.freedesktop.org --- I do not have commit access. --- src/intel/vulkan/anv_descriptor_set.c | 2 +- 1 f

[Mesa-dev] [Bug 102573] fails to build on armel

2017-09-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=102573 --- Comment #3 from Grazvydas Ignotas --- On what conditions does this show up? Mesa git builds fine on debian stretch and it's gcc 6.3.0, gcc is generating __sync_* calls even when __atomic_* is explicitly used, and that is handled. (In reply

[Mesa-dev] [PATCH] anv: Fix descriptors copying

2017-09-16 Thread Józef Kucia
Trivial. Cc: mesa-sta...@lists.freedesktop.org --- I do not have commit access. --- src/intel/vulkan/anv_descriptor_set.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/intel/vulkan/anv_descriptor_set.c b/src/intel/vulkan/anv_descriptor_set.c index 91387c065e..84077982

[Mesa-dev] [Bug 102665] test_glsl_to_tgsi_lifetime.cpp:53:67: error: ‘?=>>=?UTF-8?Q?’ should be ‘?=> >=?UTF-8?Q?’ within a nested template argument list

2017-09-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=102665 --- Comment #7 from Gert Wollny --- Isn't g++4.4 a bit old? It is not maintained since 2012[1]. Well, I can see whether I can disable compiling the tests for these older compilers where c++11 was not available or still experimental. [1] http

Re: [Mesa-dev] [PATCH 00/11] glsl, tgsi, radeonsi: ldexp and frexp bug fixes and features

2017-09-16 Thread Nicolai Hähnle
If you want to test, you can find the series at https://cgit.freedesktop.org/~nh/mesa/log/?h=ldexp On 16.09.2017 13:23, Nicolai Hähnle wrote: Hi all, This series was motivated by radeonsi failing some ldexp tests due to not handling denorms correctly and not handling overflows (which GLSL does

[Mesa-dev] [PATCH 09/11] st/glsl_to_tgsi: use LDEXP when available

2017-09-16 Thread Nicolai Hähnle
From: Nicolai Hähnle --- src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp index cf6e8f8fde1..196b547d3d9 100644 --- a/src/mesa/state_t

[Mesa-dev] [PATCH 08/11] gallium: add LDEXP TGSI instruction and corresponding cap

2017-09-16 Thread Nicolai Hähnle
From: Nicolai Hähnle --- src/gallium/auxiliary/gallivm/lp_bld_limits.h| 1 + src/gallium/auxiliary/tgsi/tgsi_exec.c | 15 +++ src/gallium/auxiliary/tgsi/tgsi_exec.h | 1 + src/gallium/auxiliary/tgsi/tgsi_info.c | 3 ++- src/gallium/auxiliary/tgsi/

[Mesa-dev] [PATCH 04/11] tgsi: clarify the semantics of DFRACEXP

2017-09-16 Thread Nicolai Hähnle
From: Nicolai Hähnle The status quo is quite the mess: 1. tgsi_exec will do a per-channel computation, and store the dst[0] result (significand) correctly for each channel. The dst[1] result (exponent) will be written to the first bit set in the writemask. So per-component calculation o

[Mesa-dev] [PATCH 06/11] gallivm: add support for TGSI instructions with two outputs

2017-09-16 Thread Nicolai Hähnle
From: Nicolai Hähnle --- src/gallium/auxiliary/gallivm/lp_bld_tgsi.c| 18 +- src/gallium/auxiliary/gallivm/lp_bld_tgsi_action.h | 5 + src/gallium/auxiliary/tgsi/tgsi_exec.h | 9 + 3 files changed, 31 insertions(+), 1 deletion(-) diff --git a/sr

[Mesa-dev] [PATCH 10/11] radeonsi: emit LDEXP opcode

2017-09-16 Thread Nicolai Hähnle
From: Nicolai Hähnle The LLVM intrinsic has existed for a long time. The current name was established in LLVM 3.9. --- src/gallium/drivers/radeonsi/si_pipe.c| 2 +- src/gallium/drivers/radeonsi/si_shader_tgsi_alu.c | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a

[Mesa-dev] [PATCH 11/11] radeonsi: emit DLDEXP and DFRACEXP TGSI opcodes

2017-09-16 Thread Nicolai Hähnle
From: Nicolai Hähnle Note: this causes spurious regressions in some current piglit tests, because the tests incorrectly assume that there is no denorm support for doubles. I'm going to send out a fix for those tests as well. --- src/gallium/drivers/radeonsi/si_pipe.c| 2 +- src/gall

[Mesa-dev] [PATCH 07/11] tgsi: infer that dst[1] of DFRACEXP is an integer

2017-09-16 Thread Nicolai Hähnle
From: Nicolai Hähnle --- src/gallium/auxiliary/gallivm/lp_bld_tgsi.c | 2 +- src/gallium/auxiliary/gallivm/lp_bld_tgsi_soa.c | 4 ++-- src/gallium/auxiliary/tgsi/tgsi_info.c | 5 - src/gallium/auxiliary/tgsi/tgsi_info.h | 2 +- src/gallium/drivers/radeon

[Mesa-dev] [PATCH 05/11] gallivm: add dst register index to lp_build_tgsi_context::emit_store

2017-09-16 Thread Nicolai Hähnle
From: Nicolai Hähnle --- src/gallium/auxiliary/gallivm/lp_bld_tgsi.c | 2 +- src/gallium/auxiliary/gallivm/lp_bld_tgsi.h | 1 + src/gallium/auxiliary/gallivm/lp_bld_tgsi_soa.c | 15 +++ src/gallium/drivers/radeonsi/si_shader.c| 14 -- src

[Mesa-dev] [PATCH 01/11] glsl/lower_instruction: handle denorms and overflow in ldexp correctly

2017-09-16 Thread Nicolai Hähnle
From: Nicolai Hähnle GLSL ES requires both, and while GLSL explicitly doesn't require correct overflow handling, it does appear to require handling input inf/denorms correctly. Fixes dEQP-GLES31.functional.shaders.builtin_functions.precision.ldexp.* Cc: mesa-sta...@lists.freedesktop.org --- sr

[Mesa-dev] [PATCH 03/11] tgsi: fix the documentation of DLDEXP

2017-09-16 Thread Nicolai Hähnle
From: Nicolai Hähnle Sourcing the exponent for the zw destination pair from Z is consistent with both tgsi_exec and gallivm. In practice, st_glsl_to_tgsi always generates per-channel instructions anyway. --- src/gallium/docs/source/tgsi.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[Mesa-dev] [PATCH 02/11] tgsi: infer that DLDEXP's second source has an integer type

2017-09-16 Thread Nicolai Hähnle
From: Nicolai Hähnle --- src/gallium/auxiliary/gallivm/lp_bld_tgsi.c | 4 ++-- src/gallium/auxiliary/nir/tgsi_to_nir.c | 7 --- src/gallium/auxiliary/tgsi/tgsi_info.c | 5 - src/gallium/auxiliary/tgsi/tgsi_info.h | 2 +- 4 files changed, 11 insertions(+), 7 deletions(-) di

[Mesa-dev] [PATCH 00/11] glsl, tgsi, radeonsi: ldexp and frexp bug fixes and features

2017-09-16 Thread Nicolai Hähnle
Hi all, This series was motivated by radeonsi failing some ldexp tests due to not handling denorms correctly and not handling overflows (which GLSL doesn't require, but GLSL ES does). The first patch fixes the GLSL IR lowering of ldexp() to handle all cases fully except: 1. Denorms; they're anno

[Mesa-dev] [PATCH 1/2] i965: Plumb brw through to intel_batchbuffer_reset().

2017-09-16 Thread Kenneth Graunke
We'll want to pass this to brw_bo_map in a moment. --- src/mesa/drivers/dri/i965/brw_context.c | 2 +- src/mesa/drivers/dri/i965/intel_batchbuffer.c | 17 + src/mesa/drivers/dri/i965/intel_batchbuffer.h | 3 +-- 3 files changed, 11 insertions(+), 11 deletions(-) diff --git

[Mesa-dev] [PATCH 2/2] i965: Warn for GTT fallbacks when mapping the batch/state buffers.

2017-09-16 Thread Kenneth Graunke
This shouldn't really happen in practice, but I hit it a couple of times when running a driver with a bad memory leak. We may as well hook up the warning, because if it ever triggers, we'll know something is wrong. --- src/mesa/drivers/dri/i965/intel_batchbuffer.c | 4 ++-- 1 file changed, 2 inse