Re: [Mesa-dev] [PATCH] glsl: do not show locp information if it is not available

2016-11-03 Thread Tapani Pälli
Please add bug tag: Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97420 On 10/26/2016 02:42 PM, Juan A. Suarez Romero wrote: Ignore source file, line number and column in glcpp_error() and glcpp_warning() if those are not available. It fixes 4 piglit tests: spec/glsl-1.10/compiler/

[Mesa-dev] [PATCH 2/2] draw: use vectorized calculations for fetch

2016-11-03 Thread sroland
From: Roland Scheidegger Instead of doing all the math with scalars, use vectors. This means the overflow math needs to be done manually, albeit that's only really problematic for the stride/index mul, the rest has been pretty much moved outside the shader loop (albeit the mul could actually be o

[Mesa-dev] [PATCH 1/2] gallivm: introduce 32x32->64bit lp_build_mul_32_lohi function

2016-11-03 Thread sroland
From: Roland Scheidegger This is used by shader umul_hi/imul_hi functions (and soon by draw). It's actually useful separating this out on its own, however the real reason for doing it is because we're using an optimized sse2 version, since the code llvm generates is atrocious (since there's no wi

Re: [Mesa-dev] [PATCH v3 8/8] anv: Do relocations in userspace before execbuf ioctl

2016-11-03 Thread Jason Ekstrand
On Thu, Nov 3, 2016 at 3:00 PM, Chris Wilson wrote: > On Thu, Nov 03, 2016 at 02:19:01PM -0700, Jason Ekstrand wrote: > >On Thu, Nov 3, 2016 at 1:53 AM, Chris Wilson <[1] > ch...@chris-wilson.co.uk> > >wrote: > > > > Something to consider is just randomly assigning an address and > u

Re: [Mesa-dev] [PATCH] anv: Document anv_event::semaphore

2016-11-03 Thread Jason Ekstrand
rb On Thu, Nov 3, 2016 at 10:29 AM, Chad Versace wrote: > Explain why it's 64-bit and its valid values. > --- > src/intel/vulkan/anv_private.h | 4 > 1 file changed, 4 insertions(+) > > diff --git a/src/intel/vulkan/anv_private.h b/src/intel/vulkan/anv_ > private.h > index 8f5a95b..0924ccf

Re: [Mesa-dev] [PATCH 1/3] anv/gen8: Stall when needed in Cmd(Set|Reset)Event

2016-11-03 Thread Jason Ekstrand
On Thu, Nov 3, 2016 at 10:42 AM, Chad Versace wrote: > On Wed 02 Nov 2016, Jason Ekstrand wrote: > > --- > > src/intel/vulkan/gen8_cmd_buffer.c | 29 + > > 1 file changed, 29 insertions(+) > > > > diff --git a/src/intel/vulkan/gen8_cmd_buffer.c > b/src/intel/vulkan/ge

Re: [Mesa-dev] [PATCH] nir: Make sure to set the texsrc type in nir drawpixels/bitmap lowering.

2016-11-03 Thread Jason Ekstrand
Reviewed-by: Jason Ekstrand Maybe CC stable? On Thu, Nov 3, 2016 at 4:35 PM, Eric Anholt wrote: > We were leaving an undefined value since the ralloc zeroing changes. > Fixes nir_validate() failures on vc4. > --- > src/compiler/nir/nir_lower_bitmap.c | 1 + > src/compiler/nir/nir_lower_dr

[Mesa-dev] [PATCH 2/2] vulkan/wsi/x11: Implement FIFO mode.

2016-11-03 Thread Jason Ekstrand
This implements VK_PRESENT_MODE_FIFO_KHR for X11. Unfortunately, due to the way the present extension works, we have to manage the queue of presented images in a separate thread. Signed-off-by: Jason Ekstrand --- src/vulkan/wsi/wsi_common_x11.c | 161 +--- 1

[Mesa-dev] [PATCH 1/2] vulkan/wsi: Add a thread-safe queue implementation

2016-11-03 Thread Jason Ekstrand
From: Kevin Strasser In order to support FIFO mode without blocking the application on calls to vkQueuePresentKHR it is necessary to enqueue the request and defer calling the server until the next vblank period. The xcb present api doesn't offer a way to register a callback, so we will have to sp

Re: [Mesa-dev] [PATCH] drivers/common/meta: Disable GL_PRIMITIVE_RESTART in glBlitFramebuffer

2016-11-03 Thread Matt Turner
On Wed, Nov 2, 2016 at 5:16 AM, Andrey Sudnik wrote: > glTexSubImage2D works incorrectly in i965 > when GL_PIXEL_UNPACK_BUFFER is set and > GL_PRIMITIVE_RESTART is enabled. > The patch fixes this issue. Thanks for the patch! Probably need more information about what the actual problem is. Looks

Re: [Mesa-dev] [PATCH 2/2] gallivm: permit use of avx512 instructions on llvm-3.9+

2016-11-03 Thread Roland Scheidegger
Am 04.11.2016 um 04:24 schrieb Rowley, Timothy O: > >> On Nov 3, 2016, at 7:09 PM, Roland Scheidegger wrote: >> >> I'm a bit worried by this. >> We've had some (a lot actually) unpleasant surprises in the past with >> llvm choosing to use instruction sets not appropriate for a given cpu... >> Hen

[Mesa-dev] [Bug 97532] Regression: GLB 2.7 & Glmark-2 GLES versions segfault due to linker precision error (259fc505) on dead variable

2016-11-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97532 Mark Janes changed: What|Removed |Added Assignee|mesa-dev@lists.freedesktop. |eero.t.tammi...@intel.com

Re: [Mesa-dev] [PATCH 2/2] gallivm: permit use of avx512 instructions on llvm-3.9+

2016-11-03 Thread Rowley, Timothy O
> On Nov 3, 2016, at 7:09 PM, Roland Scheidegger wrote: > > I'm a bit worried by this. > We've had some (a lot actually) unpleasant surprises in the past with > llvm choosing to use instruction sets not appropriate for a given cpu... > Hence only setting flags we checked ourselves being availabl

Re: [Mesa-dev] [PATCH 2/2] radv: allow image view to be create before memory is bound

2016-11-03 Thread Dave Airlie
On 4 November 2016 at 11:29, Dave Airlie wrote: > From: Dave Airlie > > The spec doesn't seem to say this is illegal anywhere, it is > unexpected, but I've got an app doing what appears to be this. > > This change splits out setting the base addresses to a separate > function, if the image has no

Re: [Mesa-dev] [PATCH 2/2] radv: allow image view to be create before memory is bound

2016-11-03 Thread Jason Ekstrand
Oh, this is definitely against the spec... From section 11.6 of the 1.0.32 spec: > Non-sparse resources must be bound completely and contiguously to a single > VkDeviceMemory object before the resource is passed as a parameter to any > of the following operations: > >creating image or buffer

[Mesa-dev] [PATCH 2/2] radv: allow image view to be create before memory is bound

2016-11-03 Thread Dave Airlie
From: Dave Airlie The spec doesn't seem to say this is illegal anywhere, it is unexpected, but I've got an app doing what appears to be this. This change splits out setting the base addresses to a separate function, if the image has no memory bound we wait until descriptor update time to bind th

[Mesa-dev] [PATCH 1/2] radv: drop bo ptr from iview.

2016-11-03 Thread Dave Airlie
From: Dave Airlie This is needed to fixup some ordering issues with when memory is bound to images. Signed-off-by: Dave Airlie --- src/amd/vulkan/radv_cmd_buffer.c | 4 ++-- src/amd/vulkan/radv_descriptor_set.c | 2 +- src/amd/vulkan/radv_device.c | 12 ++-- src/amd/vulka

[Mesa-dev] [PATCH] drivers/common/meta: Disable GL_PRIMITIVE_RESTART in glBlitFramebuffer

2016-11-03 Thread Andrey Sudnik
glTexSubImage2D works incorrectly in i965 when GL_PIXEL_UNPACK_BUFFER is set and GL_PRIMITIVE_RESTART is enabled. The patch fixes this issue. Signed-off-by: Andrey Sudnik --- src/mesa/drivers/common/meta.c | 12 src/mesa/drivers/common/meta.h | 6 ++ src/mesa/drivers

[Mesa-dev] [PATCH] mesa: Add linear ETC2/EAC to the compressed format list with ES3 compat.

2016-11-03 Thread Kenneth Graunke
GL_ARB_ES3_compatibility brings ETC2/EAC formats to desktop GL. The meaning of the GL compressed format list is pretty vague - it's supposed to return formats for "general-purpose usage". (GL 4.2 deprecates the list because of this.) Basically everyone interprets this as "linear RGB/RGBA". ETC2

Re: [Mesa-dev] [PATCH 1/2] draw: fix undefined input handling some more...

2016-11-03 Thread Roland Scheidegger
Am 03.11.2016 um 18:44 schrieb Jose Fonseca: > Patch 1 looks good to me. Yes, not using compilation key is a recipe > for problems. Indeed. Thankfully no longer used in generate_fetch(). There's still some state used not from the key, but that's just information belonging the the shader itself, so

Re: [Mesa-dev] [PATCH 1/2] draw: fix undefined input handling some more...

2016-11-03 Thread Roland Scheidegger
Am 03.11.2016 um 11:48 schrieb Nicolai Hähnle: > On 03.11.2016 02:15, srol...@vmware.com wrote: >> From: Roland Scheidegger >> >> Previous fixes were incomplete - some code still iterated through the >> number >> of elements provided by velem layout instead of the number stored in >> the key >> (w

Re: [Mesa-dev] [PATCH 2/2] gallivm: permit use of avx512 instructions on llvm-3.9+

2016-11-03 Thread Roland Scheidegger
I'm a bit worried by this. We've had some (a lot actually) unpleasant surprises in the past with llvm choosing to use instruction sets not appropriate for a given cpu... Hence only setting flags we checked ourselves being available, and disabling everything else. Not sure if this actually still wor

[Mesa-dev] [PATCH] nir: Make sure to set the texsrc type in nir drawpixels/bitmap lowering.

2016-11-03 Thread Eric Anholt
We were leaving an undefined value since the ralloc zeroing changes. Fixes nir_validate() failures on vc4. --- src/compiler/nir/nir_lower_bitmap.c | 1 + src/compiler/nir/nir_lower_drawpixels.c | 1 + 2 files changed, 2 insertions(+) diff --git a/src/compiler/nir/nir_lower_bitmap.c b/src/com

Re: [Mesa-dev] [PATCH 1/2] gallium: detect avx512 cpu features

2016-11-03 Thread Roland Scheidegger
I didn't check the actual bits, but looks good to me. Even includes the new cannonlake avx512 bits :-). Reviewed-by: Roland Scheidegger Am 03.11.2016 um 22:29 schrieb Tim Rowley: > v3: fix check for xmm/ymm test > v2: style code, add avx512 to cpu dump > --- > src/gallium/auxiliary/util/u_cpu_d

Re: [Mesa-dev] [PATCH 1/3] glsl: use the prefixed name of the lexer generated functions

2016-11-03 Thread Timothy Arceri
On Fri, 2016-11-04 at 08:56 +1100, Timothy Arceri wrote: > On Thu, 2016-10-27 at 12:41 +0100, Emil Velikov wrote: > > > > From: Emil Velikov > > > > Flex version 2.6.2 does not expand (define) the yy version of some > > function, thus we fail to compile. > > functions > > > > > > > Strictly

[Mesa-dev] [Bug 98473] Mesa fails to build with flex 2.6.2

2016-11-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98473 --- Comment #4 from Timothy Arceri --- Looks like a flex bug: https://github.com/westes/flex/issues/113 -- You are receiving this mail because: You are the assignee for the bug. You are the QA Contact for the bug.__

[Mesa-dev] [PATCH] i965/compute: Allow ARB_compute_shader in compat profile

2016-11-03 Thread Jordan Justen
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97447 Cc: Evan Odabashian Signed-off-by: Jordan Justen --- src/mesa/drivers/dri/i965/brw_context.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/brw_context.c b/src/mesa/drivers/dri/i965/brw_c

Re: [Mesa-dev] [PATCH 3/3] anv: Rework fences

2016-11-03 Thread Fredrik Höglund
On Thursday 03 November 2016, Jason Ekstrand wrote: > On Thu, Nov 3, 2016 at 2:53 AM, Chris Wilson > wrote: > > > On Wed, Nov 02, 2016 at 05:15:52PM -0700, Jason Ekstrand wrote: > > > @@ -1562,22 +1580,98 @@ VkResult anv_WaitForFences( > > > * best we can do is to clamp the timeout to INT64_

Re: [Mesa-dev] [PATCH v3 8/8] anv: Do relocations in userspace before execbuf ioctl

2016-11-03 Thread Chris Wilson
On Thu, Nov 03, 2016 at 02:19:01PM -0700, Jason Ekstrand wrote: >On Thu, Nov 3, 2016 at 1:53 AM, Chris Wilson <[1]ch...@chris-wilson.co.uk> >wrote: > > Something to consider is just randomly assigning an address and using > it. The kernel will relocate if it wants to, but in futu

[Mesa-dev] [Bug 98279] [vulkan/radeon] dota2 -vulkan hangs the GPU on R9-390

2016-11-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98279 --- Comment #5 from Luke A. Guest --- I have told Dave Airlie about rendering issues on this card via irc before radv was mainlined.This https://youtu.be/zj9TCgdRn2c was the result of the demos, I don't know if this is the same issue you're havin

[Mesa-dev] [PATCH v4 8/8] anv: Do relocations in userspace before execbuf ioctl

2016-11-03 Thread Jason Ekstrand
From: Kristian Høgsberg Kristensen Since our surface state buffer is shared by all batches, the kernel does a full stall and sync with the CPU between batches every time we call execbuf2 because it refuses to do relocations on an active buffer. Doing them in userspace and passing the NO_RELOC fl

Re: [Mesa-dev] [PATCH 1/3] glsl: use the prefixed name of the lexer generated functions

2016-11-03 Thread Timothy Arceri
On Thu, 2016-10-27 at 12:41 +0100, Emil Velikov wrote: > From: Emil Velikov > > Flex version 2.6.2 does not expand (define) the yy version of some > function, thus we fail to compile. functions > > Strictly speaking this might be a flex bug, although expanding the > few > instances is perfectl

Re: [Mesa-dev] [PATCH] st/mesa: fix the layer of VDPAU surface samplers

2016-11-03 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Thu, Nov 3, 2016 at 10:06 PM, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > A latent bug in VDPAU interop was exposed by commit > e5cc84dd43be066c1dd418e32f5ad258e31a150a. > > Before that commit, the st_vdpau code created samplers with > first_layer == last

[Mesa-dev] [Bug 98581] Dota 2 graphics glitch on autocast abilities.

2016-11-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98581 Bug ID: 98581 Summary: Dota 2 graphics glitch on autocast abilities. Product: Mesa Version: 13.0 Hardware: x86-64 (AMD64) OS: Linux (All) Status: NEW Seve

Re: [Mesa-dev] [PATCH 2/3] glx/glvnd: Fix dispatch function names and indices

2016-11-03 Thread Emil Velikov
On 3 November 2016 at 17:59, Adam Jackson wrote: > On Thu, 2016-11-03 at 17:28 +, Emil Velikov wrote: > >> Your earlier reply did not have ack/nack on the topic of dropping it, >> thus the above reads as a justification why one would wants to keep >> it. > > I thought "useless" expressed my op

[Mesa-dev] [PATCH 1/2] gallium: detect avx512 cpu features

2016-11-03 Thread Tim Rowley
v3: fix check for xmm/ymm test v2: style code, add avx512 to cpu dump --- src/gallium/auxiliary/util/u_cpu_detect.c | 26 ++ src/gallium/auxiliary/util/u_cpu_detect.h | 10 ++ 2 files changed, 36 insertions(+) diff --git a/src/gallium/auxiliary/util/u_cpu_detect.c

[Mesa-dev] [PATCH 2/2] gallivm: permit use of avx512 instructions on llvm-3.9+

2016-11-03 Thread Tim Rowley
--- src/gallium/auxiliary/gallivm/lp_bld_misc.cpp | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/gallium/auxiliary/gallivm/lp_bld_misc.cpp b/src/gallium/auxiliary/gallivm/lp_bld_misc.cpp index bd4d4d3..bff2198 100644 --- a/src/gallium/auxiliary/gallivm/lp_bld_misc.cpp +++ b/src/galliu

[Mesa-dev] [PATCH 0/2] avx512 initial usage

2016-11-03 Thread Tim Rowley
Add code to detect avx512 cpu features, and allow llvm to generate avx512 instructions with llvm-3.9+. gallivm will still treat avx512 capable processors as avx2 (256-bit vectors). No piglit regressions with llvmpipe on llvm-3.9.0 and KNL. Tim Rowley (2): gallium: detect avx512 cpu features

Re: [Mesa-dev] [PATCH v3 8/8] anv: Do relocations in userspace before execbuf ioctl

2016-11-03 Thread Jason Ekstrand
On Thu, Nov 3, 2016 at 1:53 AM, Chris Wilson wrote: > On Wed, Nov 02, 2016 at 01:58:43PM -0700, Jason Ekstrand wrote: > > +/** > > + * This function applies the relocation for a command buffer and writes > the > > + * actual addresses into the buffers as per what we were told by the > kernel on >

[Mesa-dev] [Bug 98512] radeon r600 vdpau: Invalid command stream: texture bo too small

2016-11-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98512 --- Comment #13 from Nicolai Hähnle --- I think I understand the problem. https://patchwork.freedesktop.org/patch/119855/ fixes the issue for me, could you please check that it fixes r600 as well? -- You are receiving this mail because: You are

[Mesa-dev] [PATCH] st/mesa: fix the layer of VDPAU surface samplers

2016-11-03 Thread Nicolai Hähnle
From: Nicolai Hähnle A latent bug in VDPAU interop was exposed by commit e5cc84dd43be066c1dd418e32f5ad258e31a150a. Before that commit, the st_vdpau code created samplers with first_layer == last_layer == 1 that the general texture handling code would immediately delete and re-create, because the

Re: [Mesa-dev] [PATCH 3/3] anv: Rework fences

2016-11-03 Thread Chad Versace
On Thu 03 Nov 2016, Jason Ekstrand wrote: > > > On Thu, Nov 3, 2016 at 10:59 AM, Chad Versace > wrote: > > On Wed 02 Nov 2016, Jason Ekstrand wrote: > > @@ -1116,6 +1117,11 @@ VkResult anv_QueueSubmit( > >        result = anv_device_execbuf(device, &fence->execbuf, &fence_bo); >

Re: [Mesa-dev] [PATCH 0/7] GLSL: Lowering small conditional branches

2016-11-03 Thread Ian Romanick
On 10/28/2016 04:13 PM, Marek Olšák wrote: > Hi, > > this series enables lowering small branches for radeonsi, but other > drivers can enable it too. The GLSL compiler computes a cost per > branch and then either lowers the branch if the cost is low enough, > or keeps it. As an aside... When GLS

Re: [Mesa-dev] [PATCH 3/7] glsl/lower_if: don't lower branches touching tess control outputs

2016-11-03 Thread Francisco Jerez
Ian Romanick writes: > On 10/28/2016 04:13 PM, Marek Olšák wrote: >> From: Marek Olšák >> >> --- >> src/compiler/glsl/ir_optimization.h | 3 ++- >> src/compiler/glsl/lower_if_to_cond_assign.cpp | 23 --- >> src/compiler/glsl/test_optpass.cpp| 2 +- >>

Re: [Mesa-dev] [PATCH 4/7] glsl/lower_if: conditionally lower if-branches based on their size

2016-11-03 Thread Ian Romanick
On 10/28/2016 04:13 PM, Marek Olšák wrote: > From: Marek Olšák > > --- > src/compiler/glsl/ir_optimization.h | 2 +- > src/compiler/glsl/lower_if_to_cond_assign.cpp | 55 > --- > 2 files changed, 50 insertions(+), 7 deletions(-) > > diff --git a/src/compiler/

Re: [Mesa-dev] [PATCH 3/7] glsl/lower_if: don't lower branches touching tess control outputs

2016-11-03 Thread Ian Romanick
On 10/28/2016 04:13 PM, Marek Olšák wrote: > From: Marek Olšák > > --- > src/compiler/glsl/ir_optimization.h | 3 ++- > src/compiler/glsl/lower_if_to_cond_assign.cpp | 23 --- > src/compiler/glsl/test_optpass.cpp| 2 +- > src/mesa/drivers/dri/i965/brw_

Re: [Mesa-dev] [PATCH] glsl: do not show locp information if it is not available

2016-11-03 Thread Ian Romanick
On 10/31/2016 07:06 PM, Timothy Arceri wrote: > On Mon, 2016-10-31 at 16:51 +0100, Juan A. Suarez Romero wrote: >> On Wed, 2016-10-26 at 13:42 +0200, Juan A. Suarez Romero wrote: >>> >>> Ignore source file, line number and column in glcpp_error() and >>> glcpp_warning() if those are not available.

Re: [Mesa-dev] [PATCH 1/3] dri: add dri_get_extensions_name(..) helper

2016-11-03 Thread Emil Velikov
On 3 November 2016 at 17:53, Christian Gmeiner wrote: > Hi Emil, > > 2016-11-03 17:02 GMT+01:00 Emil Velikov : >> On 3 November 2016 at 14:25, Christian Gmeiner >> wrote: >>> Signed-off-by: Christian Gmeiner >>> --- >>> include/GL/internal/dri_interface.h | 14 ++ >>> 1 file changed

Re: [Mesa-dev] [PATCH] radeonsi: fix an assertion failure in si_decompress_sampler_color_textures

2016-11-03 Thread Nicolai Hähnle
On 03.11.2016 19:22, Marek Olšák wrote: From: Marek Olšák This fixes a crash in Deus Ex: Mankind Divided. Release builds were unaffected, so it's not too serious. Cc: 11.2 12.0 13.0 Reviewed-by: Nicolai Hähnle --- src/gallium/drivers/radeonsi/si_blit.c | 4 +++- 1 file changed, 3 insert

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

2016-11-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=77449 Bug 77449 depends on bug 97890, which changed state. Bug 97890 Summary: Kerbal Space Program freezes randomly when DRI3 is enabled https://bugs.freedesktop.org/show_bug.cgi?id=97890 What|Removed |Added -

[Mesa-dev] [Bug 97863] [BXT] Webglc is failing a lot of tests related to extensions, textures, uniforms and more

2016-11-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97863 Matt Turner changed: What|Removed |Added Assignee|mesa-dev@lists.freedesktop. |intel-3d-bugs@lists.freedes

Re: [Mesa-dev] [PATCH] radeonsi: fix an assertion failure in si_decompress_sampler_color_textures

2016-11-03 Thread Bas Nieuwenhuizen
Reviewed-by: Bas Nieuwenhuizen On Thu, Nov 3, 2016 at 7:22 PM, Marek Olšák wrote: > From: Marek Olšák > > This fixes a crash in Deus Ex: Mankind Divided. Release builds were > unaffected, so it's not too serious. > > Cc: 11.2 12.0 13.0 > --- > src/gallium/drivers/radeonsi/si_blit.c | 4 +++- >

Re: [Mesa-dev] [PATCH 3/3] anv: Rework fences

2016-11-03 Thread Jason Ekstrand
On Thu, Nov 3, 2016 at 10:59 AM, Chad Versace wrote: > On Wed 02 Nov 2016, Jason Ekstrand wrote: > > Our previous fence implementation was very simple. Fences had two > states: > > signaled and unsignaled. However, this didn't properly handle all of the > > edge-cases that we need to handle. I

[Mesa-dev] [Bug 96684] [swrast] piglit glsl-array-bounds-01 regression

2016-11-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=96684 --- Comment #2 from Emil Velikov --- I can reproduce this here so gave it a quick look: The GLSL IR is identical with _and_ w/o the optimisations (MESA_GLSL=nopt), yet the result changes to pass. Seems like the _mesa_optimize_program invocation

[Mesa-dev] [PATCH] radeonsi: fix an assertion failure in si_decompress_sampler_color_textures

2016-11-03 Thread Marek Olšák
From: Marek Olšák This fixes a crash in Deus Ex: Mankind Divided. Release builds were unaffected, so it's not too serious. Cc: 11.2 12.0 13.0 --- src/gallium/drivers/radeonsi/si_blit.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/radeonsi/si_blit.c

[Mesa-dev] [Bug 98279] [vulkan/radeon] dota2 -vulkan hangs the GPU on R9-390

2016-11-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98279 Diego Vogogna changed: What|Removed |Added CC||ars...@gmail.com -- You are receiving t

[Mesa-dev] [Bug 98279] [vulkan/radeon] dota2 -vulkan hangs the GPU on R9-390

2016-11-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98279 --- Comment #4 from Diego Vogogna --- Getting the same errors in log, amd R9 390, Dota 2 and vulkan cube from sdk tests freeze the system. Running Arch x86_64 linux-git 4.9rc3.r206.g2a26d99-1 with no additional pathces mesa-git 86246.f792f06-1 v

Re: [Mesa-dev] [PATCH 3/3] anv: Rework fences

2016-11-03 Thread Chad Versace
On Wed 02 Nov 2016, Jason Ekstrand wrote: > Our previous fence implementation was very simple. Fences had two states: > signaled and unsignaled. However, this didn't properly handle all of the > edge-cases that we need to handle. In order to handle the case where the > client calls vkGetFenceSta

Re: [Mesa-dev] [PATCH 2/3] glx/glvnd: Fix dispatch function names and indices

2016-11-03 Thread Adam Jackson
On Thu, 2016-11-03 at 17:28 +, Emil Velikov wrote: > Your earlier reply did not have ack/nack on the topic of dropping it, > thus the above reads as a justification why one would wants to keep > it. I thought "useless" expressed my opinion of the value of the feature quite well. But I didn't

Re: [Mesa-dev] [PATCH 0.5/3] anv/blorp: Don't create linear ASTC surfaces for buffers

2016-11-03 Thread Nanley Chery
On Wed, Nov 02, 2016 at 08:00:04PM -0700, Jason Ekstrand wrote: > Reviewed-by: Jason Ekstrand > Thanks! > Wouldn't be a bad idea to do this for all compressed formats. It doesn't > really matter though since blorp will nuke the format the moment it cones > into blorp_copy anyway. > Yeah, sin

Re: [Mesa-dev] [PATCH 1/3] dri: add dri_get_extensions_name(..) helper

2016-11-03 Thread Christian Gmeiner
Hi Emil, 2016-11-03 17:02 GMT+01:00 Emil Velikov : > On 3 November 2016 at 14:25, Christian Gmeiner > wrote: >> Signed-off-by: Christian Gmeiner >> --- >> include/GL/internal/dri_interface.h | 14 ++ >> 1 file changed, 14 insertions(+) >> >> diff --git a/include/GL/internal/dri_inte

Re: [Mesa-dev] [PATCH 2/3] anv/wsi: Set the fence to signaled in AcquireNextImageKHR

2016-11-03 Thread Chad Versace
On Wed 02 Nov 2016, Jason Ekstrand wrote: > --- > src/intel/vulkan/anv_wsi.c | 12 +--- > 1 file changed, 9 insertions(+), 3 deletions(-) > > diff --git a/src/intel/vulkan/anv_wsi.c b/src/intel/vulkan/anv_wsi.c > index 064581d..72f79f1 100644 > --- a/src/intel/vulkan/anv_wsi.c > +++ b/src

Re: [Mesa-dev] [PATCH 1/2] draw: fix undefined input handling some more...

2016-11-03 Thread Jose Fonseca
Patch 1 looks good to me. Yes, not using compilation key is a recipe for problems. Regarding patch 2, I need a bit more time to review. I'd also like the new lp_build_umul... function to be added to lp_bld_arit: there is a single caller note, but that might change in the future, so might as we

[Mesa-dev] [Bug 98471] [TRACKER] Mesa 13.0 release tracker

2016-11-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98471 Emil Velikov changed: What|Removed |Added Depends on|96865 | Referenced Bugs: https://bugs.freedesk

[Mesa-dev] [Bug 96865] [swrast] egl-create-pbuffer-surface regression

2016-11-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=96865 Emil Velikov changed: What|Removed |Added Blocks|98471 | --- Comment #1 from Emil Velikov --- Th

Re: [Mesa-dev] [PATCH 1/3] anv/gen8: Stall when needed in Cmd(Set|Reset)Event

2016-11-03 Thread Chad Versace
On Wed 02 Nov 2016, Jason Ekstrand wrote: > --- > src/intel/vulkan/gen8_cmd_buffer.c | 29 + > 1 file changed, 29 insertions(+) > > diff --git a/src/intel/vulkan/gen8_cmd_buffer.c > b/src/intel/vulkan/gen8_cmd_buffer.c > index 0548a5e..5df2e50 100644 > --- a/src/intel

[Mesa-dev] [PATCH] anv: Document anv_event::semaphore

2016-11-03 Thread Chad Versace
Explain why it's 64-bit and its valid values. --- src/intel/vulkan/anv_private.h | 4 1 file changed, 4 insertions(+) diff --git a/src/intel/vulkan/anv_private.h b/src/intel/vulkan/anv_private.h index 8f5a95b..0924ccf 100644 --- a/src/intel/vulkan/anv_private.h +++ b/src/intel/vulkan/anv_pri

Re: [Mesa-dev] [PATCH 2/3] glx/glvnd: Fix dispatch function names and indices

2016-11-03 Thread Emil Velikov
On 3 November 2016 at 17:12, Adam Jackson wrote: > On Thu, 2016-11-03 at 16:21 +, Emil Velikov wrote: > >> I'm not a fan of keeping legacy stuff with only one (pretty useless) >> user. Not to mention that the glXGetDriverConfig plumbing is missing >> [in here]. But as you feel so strongly in k

Re: [Mesa-dev] [PATCH 3/3] anv: Rework fences

2016-11-03 Thread Jason Ekstrand
On Thu, Nov 3, 2016 at 2:53 AM, Chris Wilson wrote: > On Wed, Nov 02, 2016 at 05:15:52PM -0700, Jason Ekstrand wrote: > > @@ -1562,22 +1580,98 @@ VkResult anv_WaitForFences( > > * best we can do is to clamp the timeout to INT64_MAX. This limits > the > > * maximum timeout from 584 year

[Mesa-dev] [Bug 98578] AMDGPU white glitches in some games

2016-11-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98578 Lukas Jirkovsky changed: What|Removed |Added Assignee|mesa-dev@lists.freedesktop. |dri-devel@lists.freedesktop

[Mesa-dev] [Bug 98578] AMDGPU white glitches in some games

2016-11-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98578 Bug ID: 98578 Summary: AMDGPU white glitches in some games Product: Mesa Version: git Hardware: Other OS: All Status: NEW Severity: normal Prior

[Mesa-dev] [Bug 98578] AMDGPU white glitches in some games

2016-11-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98578 Lukas Jirkovsky changed: What|Removed |Added OS|All |Linux (All) Hardware|Other

Re: [Mesa-dev] [PATCH 2/3] glx/glvnd: Fix dispatch function names and indices

2016-11-03 Thread Adam Jackson
On Thu, 2016-11-03 at 16:21 +, Emil Velikov wrote: > I'm not a fan of keeping legacy stuff with only one (pretty useless) > user. Not to mention that the glXGetDriverConfig plumbing is missing > [in here]. But as you feel so strongly in keeping it ... Maybe I wasn't clear. I don't care at all

Re: [Mesa-dev] [PATCH 15/18] intel/blorp: Always use UINT formats on SKL+

2016-11-03 Thread Pohjolainen, Topi
On Fri, Oct 28, 2016 at 02:17:11AM -0700, Jason Ekstrand wrote: > Many of these UINT formats aren't available prior to Sky Lake so we used > UNORM formats. Using UINT formats is a bit nicer because it guarantees we > don't run into rounding issues. Also, we will need it in the next commit > for h

Re: [Mesa-dev] [PATCH 11/18] util/vk_alloc: Add a vk_zalloc2 helper

2016-11-03 Thread Pohjolainen, Topi
On Fri, Oct 28, 2016 at 02:17:07AM -0700, Jason Ekstrand wrote: > Signed-off-by: Jason Ekstrand With the typo fixed in patch 10, patches 10 and 11 are: Reviewed-by: Topi Pohjolainen I'm working on patch 13. I'm stuck with it pretty much the same way as with your earlier series. I just don't k

Re: [Mesa-dev] [PATCH 0/3] Flex 2.6.2 build fixes

2016-11-03 Thread Emil Velikov
On 27 October 2016 at 12:41, Emil Velikov wrote: > Hi all, > > As pointed out by Mike, building Mesa with latest version of flex is > not possible. > > Upon closer look it seems that the following functions are not expanded > properly (the #define yyfoo ${prefix}foo is missing). > > yylex_init_ext

Re: [Mesa-dev] [PATCH 2/3] glx/glvnd: Fix dispatch function names and indices

2016-11-03 Thread Emil Velikov
On 1 November 2016 at 19:06, Adam Jackson wrote: > On Mon, 2016-10-31 at 20:28 +, Emil Velikov wrote: >> > >> > One user, xdriinfo(1), which is admittedly pretty useless. But the >> > glvnd code in mesa definitely implements dispatch for it (libglvnd >> > itself does not, but is not expected t

Re: [Mesa-dev] [PATCH 1/3] egl: use util/macros.h

2016-11-03 Thread Emil Velikov
On 2 November 2016 at 18:07, Marek Olšák wrote: > From: Marek Olšák > > I need the definition of PUBLIC. > > Cc: 12.0 13.0 > --- > src/egl/drivers/dri2/egl_dri2.h | 2 -- > src/egl/main/egldefines.h | 5 ++--- > 2 files changed, 2 insertions(+), 5 deletions(-) > > diff --git a/src/egl/dri

Re: [Mesa-dev] [PATCH 1/3] dri: add dri_get_extensions_name(..) helper

2016-11-03 Thread Emil Velikov
On 3 November 2016 at 14:25, Christian Gmeiner wrote: > Signed-off-by: Christian Gmeiner > --- > include/GL/internal/dri_interface.h | 14 ++ > 1 file changed, 14 insertions(+) > > diff --git a/include/GL/internal/dri_interface.h > b/include/GL/internal/dri_interface.h > index d0b1b

Re: [Mesa-dev] [PATCH] pipe-loader: add libamd_common for radeonsi

2016-11-03 Thread Emil Velikov
On 3 November 2016 at 14:28, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > This fixes a build regression of commit > 7115e56c21ace07cf04f5073ba73a533e2182099. > Sorry for the breakage, this second location for link dependencies escaped > my build tests. > --- > src/gallium/targets/pipe-loade

[Mesa-dev] [Bug 97967] glsl/tests/cache-test regression

2016-11-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97967 --- Comment #3 from Emil Velikov --- The "Failed to create foo" messages are expected and deliberate. The actual issue is the next lines: Error: Test 'cache_put eviction with MAX_SIZE=1K' failed: Expected=1, Actual=2 Error: Test 'eviction after

[Mesa-dev] [Bug 98512] radeon r600 vdpau: Invalid command stream: texture bo too small

2016-11-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98512 --- Comment #12 from Nicolai Hähnle --- Andy, that's it. With an upgraded mpv, I get the assertion as well. I'm going to look into it. -- You are receiving this mail because: You are the assignee for the bug. You are the QA Contact for the bug.

Re: [Mesa-dev] [PATCH] pipe-loader: add libamd_common for radeonsi

2016-11-03 Thread Dieter Nützel
Tested-by: Dieter Nützel Am 03.11.2016 15:28, schrieb Nicolai Hähnle: From: Nicolai Hähnle This fixes a build regression of commit 7115e56c21ace07cf04f5073ba73a533e2182099. Sorry for the breakage, this second location for link dependencies escaped my build tests. --- src/gallium/targets/pi

[Mesa-dev] [Bug 98512] radeon r600 vdpau: Invalid command stream: texture bo too small

2016-11-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98512 --- Comment #11 from Nicolai Hähnle --- The -vo=opengl makes no difference. The one suspicious thing I'm seeing is: Mesa: User error: GL_INVALID_OPERATION in VDPAUFiniNV but mpv does call VDPAUInitNV later without error, and shows no other comp

[Mesa-dev] [Bug 98512] radeon r600 vdpau: Invalid command stream: texture bo too small

2016-11-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98512 --- Comment #10 from Andy Furniss --- (In reply to Nicolai Hähnle from comment #8) > Hmm, so why don't I see that assertion? Following the linked mpv-player > github issue, I'm running > > $ mpv --hwdec=auto Videos/Serenity.Trailer.H264.mp4 > >

Re: [Mesa-dev] [PATCH] pipe-loader: add libamd_common for radeonsi

2016-11-03 Thread Dieter Nützel
Tested-by: Dieter Nützel Argh, should read, please go ahead and commit. ;-) Dieter Am 03.11.2016 15:28, schrieb Nicolai Hähnle: From: Nicolai Hähnle This fixes a build regression of commit 7115e56c21ace07cf04f5073ba73a533e2182099. Sorry for the breakage, this second location for link depend

Re: [Mesa-dev] [PATCH] st/glsl_to_tgsi: fix dvec[34] loads from SSBO

2016-11-03 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Thu, Nov 3, 2016 at 11:04 AM, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > When splitting up loads, we have to add 16 bytes to the offset for > the high components, just like already happens for stores. > > Fixes arb_gpu_shader_fp64@shader_storage@layout-s

[Mesa-dev] [Bug 98512] radeon r600 vdpau: Invalid command stream: texture bo too small

2016-11-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98512 --- Comment #8 from Nicolai Hähnle --- Hmm, so why don't I see that assertion? Following the linked mpv-player github issue, I'm running $ mpv --hwdec=auto Videos/Serenity.Trailer.H264.mp4 on a radeonsi debug build just fine. The log on the com

[Mesa-dev] [Bug 98512] radeon r600 vdpau: Invalid command stream: texture bo too small

2016-11-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98512 --- Comment #9 from Christian König --- (In reply to Nicolai Hähnle from comment #8) > What flags am I missing? Just an educated guess, but try adding "-vo opengl" to the command line. -- You are receiving this mail because: You are the assign

Re: [Mesa-dev] [PATCH] pipe-loader: add libamd_common for radeonsi

2016-11-03 Thread Mike Lothian
I'm probably not qualified enough to give a Reviewed-by line, but I can confirm this fixes the build for me On Thu, 3 Nov 2016 at 14:28 Nicolai Hähnle wrote: > From: Nicolai Hähnle > > This fixes a build regression of commit > 7115e56c21ace07cf04f5073ba73a533e2182099. > Sorry for the breakage,

[Mesa-dev] [Bug 98512] radeon r600 vdpau: Invalid command stream: texture bo too small

2016-11-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98512 --- Comment #7 from Christian König --- Yeah, on SI we don't have a kernel command stream parser to double check those things. So instead of an error message you probably just get incorrect rendering for maybe one frame or two. -- You are rece

Re: [Mesa-dev] [PATCH 12/18] anv/pass: Precompute some subpass usage information

2016-11-03 Thread Pohjolainen, Topi
On Fri, Oct 28, 2016 at 02:17:08AM -0700, Jason Ekstrand wrote: > Signed-off-by: Jason Ekstrand > --- > src/intel/vulkan/anv_pass.c| 39 --- > src/intel/vulkan/anv_private.h | 11 +++ > 2 files changed, 43 insertions(+), 7 deletions(-) > > diff --g

[Mesa-dev] [Bug 98512] radeon r600 vdpau: Invalid command stream: texture bo too small

2016-11-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98512 --- Comment #6 from Andy Furniss --- (In reply to Nicolai Hähnle from comment #5) > This works just fine on radeonsi, so it's likely an r600-specific problem > that happened to be uncovered by that commit. Have you tried running with a > debug bu

Re: [Mesa-dev] [PATCH 07/18] intel/blorp: Take a fast_clear_op in ccs_resolve

2016-11-03 Thread Pohjolainen, Topi
On Fri, Oct 28, 2016 at 02:17:03AM -0700, Jason Ekstrand wrote: > Eventually, we may want to just have a single blorp_ccs_op function that > does both clears and resolves. For now we'll stick to just making the > ccs_resolve function we have now a bit more configurable. I had a question in patch

[Mesa-dev] [Bug 98512] radeon r600 vdpau: Invalid command stream: texture bo too small

2016-11-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98512 --- Comment #5 from Nicolai Hähnle --- This works just fine on radeonsi, so it's likely an r600-specific problem that happened to be uncovered by that commit. Have you tried running with a debug build of Mesa? -- You are receiving this mail bec

[Mesa-dev] [PATCH] pipe-loader: add libamd_common for radeonsi

2016-11-03 Thread Nicolai Hähnle
From: Nicolai Hähnle This fixes a build regression of commit 7115e56c21ace07cf04f5073ba73a533e2182099. Sorry for the breakage, this second location for link dependencies escaped my build tests. --- src/gallium/targets/pipe-loader/Makefile.am | 1 + 1 file changed, 1 insertion(+) diff --git a/s

Re: [Mesa-dev] [PATCH 04/18] intel/isl: Rework the asserts and fails in isl_surf_get_ccs

2016-11-03 Thread Pohjolainen, Topi
On Fri, Oct 28, 2016 at 02:17:00AM -0700, Jason Ekstrand wrote: > There are some invariants such as number of samples on which we should > assert. However, most other things should silently return false since > they're much easier for isl_surf_get_ccs to check than the caller. We also > update th

[Mesa-dev] [PATCH 3/3] dri: fixup driver names if needed

2016-11-03 Thread Christian Gmeiner
This makes it possible to 'use' the imx-drm driver. Remeber that it is not possible to have sysmbol names in C/C++ with a '-' in it. Signed-off-by: Christian Gmeiner --- include/GL/internal/dri_interface.h | 7 +++ 1 file changed, 7 insertions(+) diff --git a/include/GL/internal/dri_interfa

[Mesa-dev] [PATCH 2/3] dri: make use of dri_get_extensions_name(..) helper

2016-11-03 Thread Christian Gmeiner
Signed-off-by: Christian Gmeiner --- src/egl/drivers/dri2/egl_dri2.c | 5 +++-- src/gbm/backends/dri/gbm_dri.c | 5 +++-- src/glx/dri_common.c| 5 ++--- 3 files changed, 8 insertions(+), 7 deletions(-) diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.c in

[Mesa-dev] [PATCH 1/3] dri: add dri_get_extensions_name(..) helper

2016-11-03 Thread Christian Gmeiner
Signed-off-by: Christian Gmeiner --- include/GL/internal/dri_interface.h | 14 ++ 1 file changed, 14 insertions(+) diff --git a/include/GL/internal/dri_interface.h b/include/GL/internal/dri_interface.h index d0b1bc6..36ba65e 100644 --- a/include/GL/internal/dri_interface.h +++ b/inc

  1   2   >