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 5f489a4..3d107aa 100644 --- a/.travis.yml +++ b/.travis.yml

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 appreciated. Emil I've looked through the enti

[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)

[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 th

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 > fi

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 clear the whole fb instead > o

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 height and we would clear the whole fb ins

[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 does, please attach

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 ioctl one. > > > > Considering the issues we had with libudev -

[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 -GL_AMD_pinned_memory" > > Then check that th

[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 compiler: - please

[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&action=edit Shaders output folfer MESA_SHADER_CAPTURE_PATH output folder. -- You are receiving this mail because: Yo

[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 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: x86-6

[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 preprocessor

[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||i...@freedesktop.org --- Comment #2 fr

[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 UBO

[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 Summary|GLES3 shad

[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 with (the GPU result was

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 > --- > .../drivers

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 > --- > .../drivers/nouveau/codegen/nv50_ir_peephole.cpp | 121 >

[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 Signed-off-by:

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 and > .textureoffset_sampler2darrayshadow_vec

[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 are receiving this mai

[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 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 Resolution|---

[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 b/configure.ac index e1

[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 dri

[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 --- a/configure.ac +++ b/co

[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 bi

[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 insertions(+), 4 deletions(-) diff --git a/configure.ac

[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 || test "x$HAVE_RAD

[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 gallium_require

[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(+), 7 deletions(-) d

[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 deletions(-) dif

[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 @@ -1003,12 +1003,12 @@

[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 deletions(-) dif

[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 --- a/configure.ac +++ b/c

[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 i

[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(+), 70 deletions(-) d

[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 --- a/configure.ac +++ b/configure.ac @@ -2337,10 +233

[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 4133a72..c29604e 10

[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 thi

[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(-) diff --git a/co

[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. Signed-

[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 insertion(+), 1 de

[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 file changed, 4 ins

Re: [Mesa-dev] [PACH v2] glsl: SSBO unsized array declarations, if present, must be declared last

2016-10-15 Thread Timothy Arceri
On Fri, 2016-10-14 at 14:30 +0200, Iago Toral Quiroga wrote: > From the ARB_shader_storage_buffer_object spec: > > "In a shader storage block, the last member may be declared without > an explicit >  size.  In this case, the effective array size is inferred at run- > time from >  the size of the d

Re: [Mesa-dev] [PATCH 1/2] glsl: fail compilation of compute shaders when unsupported

2016-10-15 Thread Timothy Arceri
On Fri, 2016-10-14 at 14:21 +0200, Iago Toral Quiroga wrote: > Generally, we only check for the presence of compute shaders during > parsing when we find any language (like layout qualifiers) that are > specific to compute shaders, however, it is possible to define an > empty compute shader does no

[Mesa-dev] [PATCH 2/2] doc/features.txt: factor out i965/hsw+ & radeonsi as GL45 complete

2016-10-15 Thread Edward O'Callaghan
V2. add i965/hsw+ to list V3. rebased on master. Signed-off-by: Edward O'Callaghan --- docs/features.txt | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/docs/features.txt b/docs/features.txt index 0d6c16a..acea0cf 100644 --- a/docs/features.txt +++ b/docs/f

[Mesa-dev] [PATCH 1/2] radeonsi: Expose GLSL 4.5

2016-10-15 Thread Edward O'Callaghan
Although not strictly CTS conformant, provided we don't actually crash in any of the CTS tests we can advertise 4.5 as others do. Signed-off-by: Edward O'Callaghan --- src/gallium/drivers/radeonsi/si_pipe.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/r