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

2016-10-15 Thread Jonathan Gray
% pattern rules are a GNU extension. Convert the use of one to a inference rule to allow this to build on OpenBSD. This is a related change to the one made in e3d43dc5eae5271e2c87bab702aa7409d3dd0b23 Signed-off-by: Jonathan Gray --- src/intel/Makefile.genxml.am | 5 - 1

[Mesa-dev] [PATCH] egl/dri2: add a libname to dlopen for OpenBSD

2016-10-15 Thread Jonathan Gray
On OpenBSD try to dlopen 'libglapi.so', ld.so will find the highest major/minor version and open it in this case. Avoids '#error Unknown glapi provider for this platform' at build time. Signed-off-by: Jonathan Gray --- src/egl/drivers/dri2/egl_dri2.c | 2 +- 1 file changed, 1

[Mesa-dev] [PATCH v4 17/18] configure.ac: Only add default LLVM components if needed

2016-10-15 Thread Tobias Droste
LLVM components are only added when LLVM is needed. This means gallium adds this as soon as "--enable-gallium-llvm" is "yes" and radv + opencl add it explicitly. This slightly changes the "auto" value of "--enable-gallium-llvm": If no LLVM is installed, it is set to "no". If later drivers need

[Mesa-dev] [PATCH v4 14/18] configure.ac: Move LLVM ac_subst closer to usage

2016-10-15 Thread Tobias Droste
This moves llvm_set_environment_variables to its final destination and moves all the LLVM AC_SUBST() below the function call. No functional change. Signed-off-by: Tobias Droste --- configure.ac | 29 ++--- 1 file changed, 14 insertions(+), 15 deletions(-)

[Mesa-dev] [PATCH v4 12/18] configure.ac: Move llvm_set_environment_variables higher.

2016-10-15 Thread Tobias Droste
This moves the function to get the LLVM environment variables higher in the file. It still needs to be below the "--enable-opencl" because it uses $enable_opencl. It can be called without condition now as it only throws errors if openCL is enabled. This patch just prepares the next patch.

[Mesa-dev] [PATCH v4 05/18] configure.ac: Move LLVM version check to the top

2016-10-15 Thread Tobias Droste
A function with the LLVM version checked is moved to the top. The function is called where the old code was. No functional change. Signed-off-by: Tobias Droste --- configure.ac | 144 ++- 1 file changed, 74 insertions(+),

[Mesa-dev] [PATCH v4 08/18] configure.ac: Remove useless oCL LLVM check

2016-10-15 Thread Tobias Droste
This is handled by "llvm_check_version_for" for openCL. Signed-off-by: Tobias Droste Reviewed-by: Emil Velikov --- configure.ac | 4 1 file changed, 4 deletions(-) diff --git a/configure.ac b/configure.ac index c29604e..e1fa7c2 100644 ---

[Mesa-dev] [PATCH v4 07/18] configure.ac: Move llvm-config searching outside the function

2016-10-15 Thread Tobias Droste
There's no harm in always searching llvm-config. This way it's available as soon as possible for all functions. Signed-off-by: Tobias Droste --- configure.ac | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/configure.ac b/configure.ac index

[Mesa-dev] [PATCH v4 10/18] configure.ac: Check gallium LLVM version in gallium_require_llvm

2016-10-15 Thread Tobias Droste
This moves the LLVM version check to the helper function gallium_require_llvm() and uses the llvm_check_version_for() helper instead of open conding the LLVM version check. gallium_require_llvm is functionally the same as before, because "enable_gallium_llvm" is only set to "yes" if the host cpu

[Mesa-dev] [PATCH v4 16/18] configure.ac: Reorder arguments in radeon_llvm_check

2016-10-15 Thread Tobias Droste
Use the same order as llvm_check_version_for. Signed-off-by: Tobias Droste --- configure.ac | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/configure.ac b/configure.ac index 3aa8214..bbc1cb5 100644 --- a/configure.ac +++ b/configure.ac @@

[Mesa-dev] [PATCH v4 06/18] configure.ac: Move LLVM functions to the top

2016-10-15 Thread Tobias Droste
This just moves code around so that all LLVM related stuff is at the top of the file in the correct order. No functional change. Signed-off-by: Tobias Droste --- configure.ac | 151 +-- 1 file changed, 74 insertions(+), 77

[Mesa-dev] [PATCH v4 15/18] configure.ac: Move radv check to the Vulkan section

2016-10-15 Thread Tobias Droste
This moves the LLVM check for radv to the corresponding driver section. No functional change. Signed-off-by: Tobias Droste --- configure.ac | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/configure.ac b/configure.ac index 2f5ed50..3aa8214 100644 ---

[Mesa-dev] [PATCH v4 04/18] configure.ac: Use new helper function for LLVM

2016-10-15 Thread Tobias Droste
Use the new helper function to add LLVM targets and components. The components are added one by one to later find out which component is missing in case there is one. Signed-off-by: Tobias Droste --- configure.ac | 25 ++--- 1 file changed, 18 insertions(+),

[Mesa-dev] [PATCH v4 13/18] configure.ac: Move oCL LLVM checks to the oCL section

2016-10-15 Thread Tobias Droste
The LLVM checks can be anywhere below line 1161 now. Move the openCL LLVM checks to the section with the other openCL checks. No functional change. Signed-off-by: Tobias Droste --- configure.ac | 41 +++-- 1 file changed, 19 insertions(+), 22

[Mesa-dev] [PATCH v4 11/18] configure.ac: Remove swr_llvm_check()

2016-10-15 Thread Tobias Droste
No need for an additional function here. Use the same style for LLVM checks as the other drivers (e.g. r300, llvmpipe) that don't need a load of other checks. Instead of open conding the LLVM version check, use the function used by other drivers. "enable_gallium_llvm" is checked by

[Mesa-dev] [PATCH v4 03/18] configure.ac: Use new llvm_add_default_components

2016-10-15 Thread Tobias Droste
Signed-off-by: Tobias Droste --- configure.ac | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/configure.ac b/configure.ac index 4fc8ef9..3875146 100644 --- a/configure.ac +++ b/configure.ac @@ -2283,11 +2283,7 @@ if test "x$enable_gallium_llvm" = xyes ||

[Mesa-dev] [PATCH v4 01/18] configure.ac: Don't search llvm-config if it's known

2016-10-15 Thread Tobias Droste
This way LLVM_CONFIG can bet set from an env variable if it's outside the $llvm_prefix. This is not a must, but it helps testing. Signed-off-by: Tobias Droste Reviewed-by: Emil Velikov --- configure.ac | 10 ++ 1 file changed, 6

[Mesa-dev] [PATCH v4 02/18] configure.ac: Add helper function for targets/components

2016-10-15 Thread Tobias Droste
Add functions to add and check targets/components. Not used in this patch. Signed-off-by: Tobias Droste --- configure.ac | 36 1 file changed, 36 insertions(+) diff --git a/configure.ac b/configure.ac index 04149da..4fc8ef9 100644 ---

[Mesa-dev] [PATCH v4 00/18] configure.ac: Hopefully the last!

2016-10-15 Thread Tobias Droste
This series leaves out everything from the previous series that was to controversial, requires more discussions or could be done as a follow up. It just cleans up how LLVM version/target/component checks are used. No version or other requirements were changed. I reordered the patches a little

[Mesa-dev] [PATCH v4 09/18] configure.ac: Use short names for r600 und r300

2016-10-15 Thread Tobias Droste
There are no non gallium r300 and r600 drivers anymore. No need to explicilty mention gallium here. Just cosmetics, no functional change. Signed-off-by: Tobias Droste --- configure.ac | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/configure.ac

[Mesa-dev] [PATCH v4 18/18] configure.ac: Add required LLVM versions to the top

2016-10-15 Thread Tobias Droste
Consolidate the required LLVM versions at the top where the other versions for dependencies are listed. This also tries to get the patch version from the LLVM headers. It defaults to 0 if it does not find it. This is good enough since the patch version is in the header since 3.6.0 and the only

[Mesa-dev] [Bug 98242] dEQP mandates preprocessor tests for #line expressions and define defined

2016-10-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98242 Ian Romanick changed: What|Removed |Added Status|NEW |RESOLVED

[Mesa-dev] [Bug 77449] Tracker bug for all bugs related to Steam titles

2016-10-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=77449 Matt Turner changed: What|Removed |Added CC||ern...@gmail.com,

[Mesa-dev] [Bug 97250] Mesa/Clover: openCV library bugs on CL_MEM_USE_HOST_PTR

2016-10-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97250 Matt Turner changed: What|Removed |Added OS|Mac OS X (All) |Linux (All) -- You

Re: [Mesa-dev] [PATCH] glsl: Disable textureOffset(sampler2DArrayShadow, ...) in GLSL ES.

2016-10-15 Thread Ilia Mirkin
Reviewed-by: Ilia Mirkin On Sat, Oct 15, 2016 at 6:44 PM, Kenneth Graunke wrote: > This has apparently never existed in GLSL ES. > > Fixes dEQP-GLES3.functional.shaders.texture_functions.invalid > .textureoffset_sampler2darrayshadow_vec4_ivec2_vertex

[Mesa-dev] [PATCH] glsl: Disable textureOffset(sampler2DArrayShadow, ...) in GLSL ES.

2016-10-15 Thread Kenneth Graunke
This has apparently never existed in GLSL ES. Fixes dEQP-GLES3.functional.shaders.texture_functions.invalid .textureoffset_sampler2darrayshadow_vec4_ivec2_vertex and .textureoffset_sampler2darrayshadow_vec4_ivec2_fragment Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98244

Re: [Mesa-dev] [PATCH] nv50/ir: Split 64-bit integer MAD/MUL operations

2016-10-15 Thread Ilia Mirkin
On Sat, Oct 15, 2016 at 6:24 PM, Pierre Moreau wrote: > Hardware does not support 64-bit integers MAD and MUL operations, so we need > to transform them in 32-bit operations. > > Signed-off-by: Pierre Moreau > --- >

Re: [Mesa-dev] [PATCH] nv50/ir: Split 64-bit integer MAD/MUL operations

2016-10-15 Thread Pierre Moreau
Sorry, there should have been a v2 next to PATCH in the subject… Pierre On 12:24 am - Oct 16 2016, Pierre Moreau wrote: > Hardware does not support 64-bit integers MAD and MUL operations, so we need > to transform them in 32-bit operations. > > Signed-off-by: Pierre Moreau

[Mesa-dev] [PATCH] nv50/ir: Split 64-bit integer MAD/MUL operations

2016-10-15 Thread Pierre Moreau
Hardware does not support 64-bit integers MAD and MUL operations, so we need to transform them in 32-bit operations. Signed-off-by: Pierre Moreau --- .../drivers/nouveau/codegen/nv50_ir_peephole.cpp | 121 + 1 file changed, 121 insertions(+) Tested

[Mesa-dev] [Bug 98244] dEQP: textureOffset(sampler2DArrayShadow, ...) should not exist.

2016-10-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98244 Kenneth Graunke changed: What|Removed |Added Status|NEW |ASSIGNED

[Mesa-dev] [Bug 98243] dEQP mismatched UBO precision qualifiers

2016-10-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98243 Kenneth Graunke changed: What|Removed |Added Summary|GLES3.0 dEQP failure|dEQP mismatched

[Mesa-dev] [Bug 98242] dEQP mandates preprocessor tests for #line expressions and define defined

2016-10-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98242 Kenneth Graunke changed: What|Removed |Added CC|

[Mesa-dev] [Bug 98242] dEQP mandates preprocessor tests for #line expressions and define defined

2016-10-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98242 Kenneth Graunke changed: What|Removed |Added Summary|GLES3.0 dEQP failure|dEQP mandates

[Mesa-dev] [Bug 98275] Segmentation fault when using VAAPI acceleration in VLC after installing oibaf's optimized drivers

2016-10-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98275 Bug ID: 98275 Summary: Segmentation fault when using VAAPI acceleration in VLC after installing oibaf's optimized drivers Product: Mesa Version: unspecified Hardware:

[Mesa-dev] [Bug 98172] Concurrent call to glClientWaitSync results in segfault in one of the waiters.

2016-10-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98172 --- Comment #18 from shinji.suz...@gmail.com --- > Looks like glDeleteSync directly calls (or is a macro of?) I thought that was the case because I got this stack trace; (gdb) where #0 st_delete_sync_object (ctx=0x863890, obj=0x7fffe8001320)

[Mesa-dev] [Bug 97879] [amdgpu] Rocket League: long hangs (several seconds) when loading assets (models/textures/shaders?)

2016-10-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97879 --- Comment #34 from Micael Bergeron --- Created attachment 127319 --> https://bugs.freedesktop.org/attachment.cgi?id=127319=edit Shaders output folfer MESA_SHADER_CAPTURE_PATH output folder. -- You are receiving

[Mesa-dev] [Bug 97879] [amdgpu] Rocket League: long hangs (several seconds) when loading assets (models/textures/shaders?)

2016-10-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97879 --- Comment #33 from Marek Olšák --- The traces don't work. Let's assume it's an apitrace bug. Running the traces can be done with: glretrace --benchmark [trace file] glretrace is part of apitrace. We can at least profile the

[Mesa-dev] [Bug 97879] [amdgpu] Rocket League: long hangs (several seconds) when loading assets (models/textures/shaders?)

2016-10-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97879 --- Comment #32 from Micael Bergeron --- (In reply to Marek Olšák from comment #31) > Please create apitrace while this environment variable is set: > MESA_EXTENSION_OVERRIDE="-GL_ARB_buffer_storage

Re: [Mesa-dev] [PATCH 02/16] loader: slim down loader_get_pci_id_for_fd implementation(s)

2016-10-15 Thread Emil Velikov
On Saturday, 15 October 2016, Jonathan Gray wrote: > On Tue, Oct 11, 2016 at 07:31:46PM +0100, Emil Velikov wrote: > > From: Emil Velikov > > > > > Currently mesa has three code paths in the loader - libudev, manual > > sysfs and drm

[Mesa-dev] [Bug 97879] [amdgpu] Rocket League: long hangs (several seconds) when loading assets (models/textures/shaders?)

2016-10-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97879 --- Comment #31 from Marek Olšák --- Please create apitrace while this environment variable is set: MESA_EXTENSION_OVERRIDE="-GL_ARB_buffer_storage -GL_AMD_pinned_memory" Then check that the apitrace replays correctly. If it

Re: [Mesa-dev] [PATCH] state_tracker: Fix check for scissor enabled when < 0.

2016-10-15 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Oct 15, 2016 1:37 AM, "Eric Anholt" wrote: > DEQP's clear tests like to give us x + w < 0 or y + h < 0. Since we > were comparing to an unsigned, it would get promoted to unsigned and come > out as bignum >= width or

Re: [Mesa-dev] [PATCH] state_tracker: Fix check for scissor enabled when < 0.

2016-10-15 Thread Edward O'Callaghan
Acked-by: Edward O'Callaghan On 10/15/2016 10:37 AM, Eric Anholt wrote: > DEQP's clear tests like to give us x + w < 0 or y + h < 0. Since we > were comparing to an unsigned, it would get promoted to unsigned and come > out as bignum >= width or height and we would

Re: [Mesa-dev] [PATCH 00/10] intel/genxml: Make more stuff booleans

2016-10-15 Thread Kenneth Graunke
On Friday, October 14, 2016 11:11:11 PM PDT Jason Ekstrand wrote: > This is a fairly trivial little cleanup that makes a bunch of 1-bit fields > that used to have the "uint" type have the "bool" type instead. Ideally, a > field should have the "bool" type if and only if, given the name of the >

[Mesa-dev] [Bug 98172] Concurrent call to glClientWaitSync results in segfault in one of the waiters.

2016-10-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98172 --- Comment #17 from shinji.suz...@gmail.com --- I need to take my words back. > dtor should be called only by the thread that eliminated the last reference. Looks like glDeleteSync directly calls (or is a macro of?) st_delete_sync_object. If

[Mesa-dev] [Bug 98271] [radeonsi]Playing videos with vdpau or vaapi hardware acceleration crashes my pc

2016-10-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98271 Bug ID: 98271 Summary: [radeonsi]Playing videos with vdpau or vaapi hardware acceleration crashes my pc Product: Mesa Version: 12.0 Hardware: x86-64 (AMD64)

Re: [Mesa-dev] [PATCH v2 05/16] loader: reimplement loader_get_user_preferred_fd via libdrm

2016-10-15 Thread Axel Davy
On 14/10/2016 22:33, Emil Velikov wrote: On 14 October 2016 at 20:21, Axel Davy wrote: The code looks good. With the minor nitpick fixed, this patch is: Reviewed-by: Axel Davy Thanks. If you can skim through any of the other patches that'll be

Re: [Mesa-dev] [PATCH 11/16] travis: remove no longer needed liudev-dev dependency

2016-10-15 Thread Axel Davy
typo in the commit title: libudev-dev Axel On 11/10/2016 20:31, Emil Velikov wrote: From: Emil Velikov Signed-off-by: Emil Velikov --- .travis.yml | 1 - 1 file changed, 1 deletion(-) diff --git a/.travis.yml b/.travis.yml index

[Mesa-dev] [PATCH 09/10] intel/genxml: Make "Predication enable" a boolean

2016-10-15 Thread Jason Ekstrand
Signed-off-by: Jason Ekstrand --- src/intel/genxml/gen8.xml | 2 +- src/intel/genxml/gen9.xml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/intel/genxml/gen8.xml b/src/intel/genxml/gen8.xml index 07fe014..971c997 100644 ---

[Mesa-dev] [PATCH 07/10] intel/genxml; Make "Tiled Surface" a boolean

2016-10-15 Thread Jason Ekstrand
Signed-off-by: Jason Ekstrand --- src/intel/genxml/gen6.xml | 4 ++-- src/intel/genxml/gen7.xml | 2 +- src/intel/genxml/gen75.xml | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/intel/genxml/gen6.xml b/src/intel/genxml/gen6.xml index

[Mesa-dev] [PATCH 04/10] intel/genxml: Make a couple of STREAMOUT fields booleans

2016-10-15 Thread Jason Ekstrand
Signed-off-by: Jason Ekstrand --- src/intel/genxml/gen7.xml | 4 ++-- src/intel/genxml/gen75.xml | 4 ++-- src/intel/genxml/gen8.xml | 4 ++-- src/intel/genxml/gen9.xml | 4 ++-- 4 files changed, 8 insertions(+), 8 deletions(-) diff --git a/src/intel/genxml/gen7.xml

[Mesa-dev] [PATCH 01/10] intel/genxml: Make "Single Program Flow" a boolean

2016-10-15 Thread Jason Ekstrand
We also get rid of the "(SPF)" a few places. Signed-off-by: Jason Ekstrand --- src/intel/genxml/gen6.xml | 6 +++--- src/intel/genxml/gen7.xml | 8 src/intel/genxml/gen75.xml | 8 src/intel/genxml/gen8.xml | 6 +++--- src/intel/genxml/gen9.xml | 6

[Mesa-dev] [PATCH 05/10] intel/genxml: Make "Stencil Buffer Enable" a boolean

2016-10-15 Thread Jason Ekstrand
Signed-off-by: Jason Ekstrand --- src/intel/genxml/gen75.xml | 2 +- src/intel/genxml/gen8.xml | 2 +- src/intel/genxml/gen9.xml | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/intel/genxml/gen75.xml b/src/intel/genxml/gen75.xml index

[Mesa-dev] [PATCH 02/10] intel/genxml: Make "Vector Mask Enable" a boolean

2016-10-15 Thread Jason Ekstrand
We also get rid of the "(VME)" a few places Signed-off-by: Jason Ekstrand --- src/intel/genxml/gen6.xml | 6 +++--- src/intel/genxml/gen7.xml | 6 +++--- src/intel/genxml/gen75.xml | 6 +++--- 3 files changed, 9 insertions(+), 9 deletions(-) diff --git

[Mesa-dev] [PATCH 06/10] intel/genxml: Make "SO Buffer Enable" fields boolean

2016-10-15 Thread Jason Ekstrand
Signed-off-by: Jason Ekstrand --- src/intel/genxml/gen7.xml | 8 src/intel/genxml/gen75.xml | 8 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/src/intel/genxml/gen7.xml b/src/intel/genxml/gen7.xml index 7d09835..6d88f4a 100644 ---

[Mesa-dev] [PATCH 00/10] intel/genxml: Make more stuff booleans

2016-10-15 Thread Jason Ekstrand
This is a fairly trivial little cleanup that makes a bunch of 1-bit fields that used to have the "uint" type have the "bool" type instead. Ideally, a field should have the "bool" type if and only if, given the name of the field and a true/false value, it's obvious what that value means. I think

[Mesa-dev] [PATCH 03/10] intel/genxml: Make "Include Vertex Handles" and "Include Primitive ID" booleans

2016-10-15 Thread Jason Ekstrand
Signed-off-by: Jason Ekstrand --- src/intel/genxml/gen7.xml | 6 +++--- src/intel/genxml/gen75.xml | 6 +++--- src/intel/genxml/gen8.xml | 6 +++--- src/intel/genxml/gen9.xml | 6 +++--- 4 files changed, 12 insertions(+), 12 deletions(-) diff --git

[Mesa-dev] [PATCH 10/10] intel/genxml: Make some PIPE_CONTROL fields booleans

2016-10-15 Thread Jason Ekstrand
--- src/intel/genxml/gen6.xml | 9 +++-- src/intel/genxml/gen7.xml | 7 ++- src/intel/genxml/gen75.xml | 7 ++- src/intel/genxml/gen8.xml | 7 ++- src/intel/genxml/gen9.xml | 7 ++- 5 files changed, 11 insertions(+), 26 deletions(-) diff --git a/src/intel/genxml/gen6.xml

[Mesa-dev] [PATCH 08/10] intel/genxml; Make "Use Global GTT a boolean

2016-10-15 Thread Jason Ekstrand
We also remove the redundant zero defaults since everything without an explicit default gets zeroed automatically. Signed-off-by: Jason Ekstrand --- src/intel/genxml/gen6.xml | 11 --- src/intel/genxml/gen7.xml | 15 ++- src/intel/genxml/gen75.xml |