[Mesa-dev] [Bug 104214] Dota crashes when switching from game to desktop

https://bugs.freedesktop.org/show_bug.cgi?id=104214 --- Comment #39 from Florian Bruhin --- FWIW the crashes I've seen in mpv and on YouTube when resizing the window also seem to be gone with the package from comment 34. -- You are receiving this mail because: You are the QA Contact for the bug

Re: [Mesa-dev] [PATCH] dri_util: remove ALLOW_RGB10_CONFIGS option (v2)

On 01/11/2018 08:14 AM, Mario Kleiner wrote: On 01/10/2018 07:04 AM, Tapani Pälli wrote: Hi Marek; This one works but only if you add DRI_CONF_ALLOW_RGB10_CONFIGS("false") to the DRI_CONF_SECTION_MISCELLANEOUS section in intel_screen. With that change: Reviewed-by: Tapani Pälli With t

Re: [Mesa-dev] [PATCH 2/6] i965: Track format and aux usage in the render cache

On Wed, 2018-01-10 at 11:22 -0800, Jason Ekstrand wrote: > This lets us perform render cache flushes whenever a surface goes > from > being used with one aux+format to a different aux+format. > > This is the "proper" fix for https://bugs.freedesktop.org/102435. > ee57b15ec764736e2d5360beaef9fb2045

Re: [Mesa-dev] [RFC PATCH 1/2] r600/sb: Set flags for GROUP_BARRIER instruction and force it into slot X

Am Donnerstag, den 11.01.2018, 08:18 +1000 schrieb Dave Airlie: > > > The number of writes is the same though, and as far as I can tell > > from the TGSI, the values written to LDS before the barrier are not > > read back within the shader - which makes me wonder whether the > > barrier is actuall

Re: [Mesa-dev] [RFC PATCH 1/2] r600/sb: Set flags for GROUP_BARRIER instruction and force it into slot X

Am Donnerstag, den 11.01.2018, 08:09 +0100 schrieb Gert Wollny: > Am Mittwoch, den 10.01.2018, 15:27 -0500 schrieb Ilia Mirkin: > [...] > > > > If your hardware executes all the vertices in parallel, then a > > barrier should be unnecessary. > > My first try for this patch did not include forcing

Re: [Mesa-dev] [PATCH 2/2] radv: Implement VK_EXT_discard_rectangles.

The Sea Islands 3d/Compute Register Reference Guide: http://developer.amd.com/wordpress/media/2013/10/CIK_3D_registers_v2.pdf Which says: "OpenGL Clip boolean function. The `inside` flags for each of the four clip rectangles form a 4-bit binary number. The corresponding bit in this 16-bit number

[Mesa-dev] [Bug 104214] Dota crashes when switching from game to desktop

https://bugs.freedesktop.org/show_bug.cgi?id=104214 --- Comment #40 from Andriy Khulap --- I was able to reproduce the Bug 104301 on the following system: Intel(R) HD Graphics 530 (Skylake GT2) (0x191b) Ubuntu 16.04 LTS (Kernel 4.4.0) Mesa 17.3.0 and latest from git. mpv 0.14.0-git-e1993d5 Debu

Re: [Mesa-dev] [PATCH 2/2] ac/nir: fix translation of nir_op_frsq for doubles

On 01/11/2018 07:06 AM, Timothy Arceri wrote: Without this we en up with the llvm error message: typo: end up Series is: Reviewed-by: Samuel Pitoiset "Both operands to a binary operator are not of the same type!" --- src/amd/common/ac_nir_to_llvm.c | 3 ++- 1 file changed, 2 insertio

[Mesa-dev] [Bug 104214] Dota crashes when switching from game to desktop

https://bugs.freedesktop.org/show_bug.cgi?id=104214 --- Comment #41 from Andriy Khulap --- BTW my quick fix was removing the event processing from dri3_fence_await(). (added by a727c804a2c17db306c68e259ae845aa6382d3b1 loader/dri3: Process event after each fence wait) -- You are receiving this m

Re: [Mesa-dev] [PATCH] radv: trim buffer load result (fixes dota2)

How can this be reproduced? I have just launched the benchmark and dota2 doesn't hit any LLVM assert. On 01/11/2018 03:11 AM, Dave Airlie wrote: From: Dave Airlie Running dota2 since the below commit crashes with an llvm assert. Trim the vector like the other user. This possible could also b

[Mesa-dev] [Bug 104214] Dota crashes when switching from game to desktop

https://bugs.freedesktop.org/show_bug.cgi?id=104214 --- Comment #42 from Cyril --- (In reply to Evangelos Foutras from comment #34) > I built mesa/lib32-mesa packages for Arch Linux which include the patch from > comment 29 (not sure if lib32-mesa is needed but it can't hurt either): > > https:/

[Mesa-dev] [PATCH v2 1/2] drm-uapi: Update vc4 header with perfmon related definitions

Signed-off-by: Boris Brezillon --- include/drm-uapi/vc4_drm.h | 67 ++ 1 file changed, 67 insertions(+) diff --git a/include/drm-uapi/vc4_drm.h b/include/drm-uapi/vc4_drm.h index 3415a4b71884..3686f451d779 100644 --- a/include/drm-uapi/vc4_drm.h +++ b/

[Mesa-dev] [PATCH v2 2/2] broadcom/vc4: Add support for HW perfmon

The V3D engine provides several perf counters. Implement ->get_driver_query_[group_]info() so that these counters are exposed through the GL_AMD_performance_monitor extension. Signed-off-by: Boris Brezillon --- Changes in v2 (all reported by Eric): - Add missing "TLB-quads-passing-z-and-stencil-t

[Mesa-dev] [PATCH v2 0/2] broadcom/v4: Expose VC4 HW perf counters

Hello, This series makes use of the VC4 perfmon ioctls to expose HW perf counters through the GL_AMD_performance_monitor interface. This patches depends on their kernel counterparts and should not be applied until the kernel patches have reached the drm tree. Regards, Boris Changes in v2: - Dr

Re: [Mesa-dev] [PATCH] radv: don't emit unneeded vertex state.

I thought GL_vs_VK wasn't really important to optimize, but heh? Looks good to me! Reviewed-by: Samuel Pitoiset On 01/10/2018 11:14 PM, Dave Airlie wrote: From: Dave Airlie If the number of instances hasn't changed and we've already emitted it, don't emit it again. If the vertex shader is

Re: [Mesa-dev] [PATCH 2/2] radv/winsys: skip padded empty buffers in chain.

Series is: Reviewed-by: Samuel Pitoiset On 01/11/2018 05:56 AM, Dave Airlie wrote: From: Dave Airlie When chaining buffers if we see an empty one (just padded) skip over it. --- src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.c | 19 +++ 1 file changed, 15 insertions(+), 4 del

Re: [Mesa-dev] [PATCH] radv/winsys: replace bo list searchs with a hash table.

mmh, last time I tried to use a hash table for the amdgpu winsys of RadeonSI it was *worse*, so not sure if that really helps. Though, I don't have any numbers to show you because it was a while ago. On 01/11/2018 04:53 AM, Dave Airlie wrote: From: Dave Airlie This should make the merging of

[Mesa-dev] [Bug 104214] Dota crashes when switching from game to desktop

https://bugs.freedesktop.org/show_bug.cgi?id=104214 Evangelos Foutras changed: What|Removed |Added CC||spam3910...@gmail.com --- Comment #

Re: [Mesa-dev] [PATCH] radv: set no-signed-zeros-fp-math

Apparently we are not quite sure if this is compatible with Vulkan or not. On 01/11/2018 12:30 AM, Marek Olšák wrote: I also recommend porting lp_create_builder, which might also improve things. Marek On Wed, Jan 10, 2018 at 10:31 PM, Samuel Pitoiset wrote: This is an optimisation that is re

Re: [Mesa-dev] [PATCH 5/6] i965/draw: Do resolves properly for textures used by TXF

On Wed, 2018-01-10 at 11:22 -0800, Jason Ekstrand wrote: > --- >  src/mesa/drivers/dri/i965/brw_draw.c | 41 > >  1 file changed, 41 insertions(+) > > diff --git a/src/mesa/drivers/dri/i965/brw_draw.c > b/src/mesa/drivers/dri/i965/brw_draw.c > index 4945dec..9fd

[Mesa-dev] [Bug 104214] Dota crashes when switching from game to desktop

https://bugs.freedesktop.org/show_bug.cgi?id=104214 --- Comment #44 from Thomas Hellström --- I sent out the patch for review on the mesa-dev list now. If anybody wants a Reported-by: or Tested-by: tag, please reply to that email, Thanks, Thomas -- You are receiving this mail because: You are

Re: [Mesa-dev] [PATCH] radv: trim buffer load result (fixes dota2)

On 01/11/2018 10:20 AM, Samuel Pitoiset wrote: How can this be reproduced? I have just launched the benchmark and dota2 doesn't hit any LLVM assert. dota2: /home/hakzsam/programming/llvm/lib/IR/Instructions.cpp:2565: static llvm::CastInst* llvm::CastInst::Create(llvm::Instruction::CastOps,

Re: [Mesa-dev] [PATCH 2/2] radv: Implement VK_EXT_discard_rectangles.

Now that I reread your email, no, that is not how we do it. For each bit i, we interpret the i itself as a bitmask. So e.g. if we have 3 rectangles in inclusive mode, the bits set would be any i, that has at least one of its 3 LSBs set (e.g. 1,2,3,4,5,6,7,9,10,11,12,13,14,15). In exclusive mode wi

Re: [Mesa-dev] [PATCH 5/6] i965/draw: Do resolves properly for textures used by TXF

On Thu, 2018-01-11 at 10:48 +0100, Iago Toral wrote: > On Wed, 2018-01-10 at 11:22 -0800, Jason Ekstrand wrote: > > --- > >  src/mesa/drivers/dri/i965/brw_draw.c | 41 > > > >  1 file changed, 41 insertions(+) > > > > diff --git a/src/mesa/drivers/dri/i965/brw_d

Re: [Mesa-dev] [PATCH] radv: set no-signed-zeros-fp-math

On Wed, Jan 10, 2018 at 10:31 PM, Samuel Pitoiset wrote: > + LLVMAddTargetDependentFunctionAttr(main_function, > + "no-signed-zeros-fp-math", > + "true"); > + Is this option safe to enable for Vulkan? The Vulk

[Mesa-dev] [PATCH] ac/nir: fix translation of nir_op_frcp for doubles

Without this we end up with the llvm error message: "Both operands to a binary operator are not of the same type!" --- This is an additional fix on top of this series: https://patchwork.freedesktop.org/series/36313/ src/amd/common/ac_nir_to_llvm.c | 3 ++- 1 file changed, 2 insertions(+), 1

Re: [Mesa-dev] [PATCH 2/2] radv: inline push constants where possible.

Hi Dave, This seems to cause some breakage when both push constants and dynamic descriptors are used. I've commented 2 fixes inline below needed to avoid a crash, but with those F1 2017 will still hang pretty quick before the main menu, not sure why so far. Mad Max is OK but that doesn't use dyna

[Mesa-dev] [PATCH] loader/dri3: Avoid freeing renderbuffers in use

Upon reception of an event that lowered the number of active back buffers, the code would immediately try to free all back buffers with an id equal to or higher than the new number of active back buffers. However, that could lead to an active or to-be-active back buffer being freed, since the old

Re: [Mesa-dev] [PATCH 0/6] i965: Misc sRGB and CCS fixes

I sent minor comments to for patch 2, with or without those patches 1-4 are: Reviewed-by: Iago Toral Quiroga I don't think I understand patch 5 well enough and because of that I still don't see how patch 6 achieves its purpose. Iago On Wed, 2018-01-10 at 11:22 -0800, Jason Ekstrand wrote: > T

[Mesa-dev] [PATCH] etnaviv: dirty TS state when framebuffer has changed

When switching between framebuffers with and without TS, the TS state needs to be flushed to the command stream even if the derived state isn't changed. Signed-off-by: Lucas Stach --- src/gallium/drivers/etnaviv/etnaviv_state.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

Re: [Mesa-dev] [PATCH] loader/dri3: Avoid freeing renderbuffers in use

Tested-by: Andriy.Khulap Fixes Bug 104301 (https://bugs.freedesktop.org/show_bug.cgi?id=104301) on Intel(R) HD Graphics 530 (Skylake GT2) (0x191b) Ubuntu 16.04 Kernel 4.4.0 Mesa 17.4.0-devel (git-4f1e500) mpv 0.14.0-git-e1993d5 *Andriy Khulap | Software Engineer* Mobile: +38-067-607-27-80 Skype

Re: [Mesa-dev] [PATCH] loader/dri3: Avoid freeing renderbuffers in use

Tested-by: Vadym Shovkoplias This patch also fixes similar issue with SHENZHEN I/O game ( https://bugs.freedesktop.org/show_bug.cgi?id=104392) Tested on HP Zbook. 2018-01-11 11:53 GMT+02:00 Thomas Hellstrom : > Upon reception of an event that lowered the number of active back buffers, > the cod

[Mesa-dev] [PATCH] anv: VkDescriptorSetLayoutBinding can have descriptorCount == 0

From Vulkan spec: "descriptorCount is the number of descriptors contained in the binding, accessed in a shader as an array. If descriptorCount is zero this binding entry is reserved and the resource must not be accessed from any stage via this binding within any pipeline using the set layout." Fi

Re: [Mesa-dev] [RFC PATCH 1/2] r600/sb: Set flags for GROUP_BARRIER instruction and force it into slot X

On Thu, Jan 11, 2018 at 3:51 AM, Gert Wollny wrote: > Am Donnerstag, den 11.01.2018, 08:09 +0100 schrieb Gert Wollny: >> Am Mittwoch, den 10.01.2018, 15:27 -0500 schrieb Ilia Mirkin: >> [...] >> > >> > If your hardware executes all the vertices in parallel, then a >> > barrier should be unnecessar

[Mesa-dev] [PATCH 2/2] ac/nir: fix translation of nir_op_fsign for doubles

Without this we end up with the llvm error message: "Both operands to a binary operator are not of the same type!" --- src/amd/common/ac_nir_to_llvm.c | 26 +++--- 1 file changed, 19 insertions(+), 7 deletions(-) diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/a

[Mesa-dev] [PATCH 1/2] ac: add f64_0 to the llvm build context

--- src/amd/common/ac_llvm_build.c | 1 + src/amd/common/ac_llvm_build.h | 1 + 2 files changed, 2 insertions(+) diff --git a/src/amd/common/ac_llvm_build.c b/src/amd/common/ac_llvm_build.c index 75cc0dc63c..f0a1788eaf 100644 --- a/src/amd/common/ac_llvm_build.c +++ b/src/amd/common/ac_llvm_build

[Mesa-dev] [PATCH] radv: Initialize DCC on transition from preinitialized.

Looks like the decompress does not handle invalid encodings well, which happens with random memory. Of course apps should not use it with random memory, but they are allowed to Fixes: 44fcf58744 "radv: Disable DCC for GENERAL layout and compute transfer dest." --- src/amd/vulkan/radv_cmd_bu

Re: [Mesa-dev] [PATCH] anv: VkDescriptorSetLayoutBinding can have descriptorCount == 0

Reviewed-by: Lionel Landwerlin Probably Cc stable too. Thanks! On 11/01/18 11:09, Samuel Iglesias Gonsálvez wrote: From Vulkan spec: "descriptorCount is the number of descriptors contained in the binding, accessed in a shader as an array. If descriptorCount is zero this binding entry is rese

Re: [Mesa-dev] [PATCH 04/10] freedreno: implement user constant buffers

On Wed, Jan 10, 2018 at 4:41 PM, Marek Olšák wrote: > On Wed, Jan 10, 2018 at 9:36 PM, Rob Clark wrote: >> On Wed, Jan 10, 2018 at 3:33 PM, Rob Clark wrote: >>> On Wed, Jan 10, 2018 at 2:49 PM, Marek Olšák wrote: From: Marek Olšák --- src/gallium/drivers/freedreno/freedren

Re: [Mesa-dev] [PATCH v2] isl: don't crash when creating a huge image

This patch is still unreviewed. Sam On 14/11/17 09:45, Samuel Iglesias Gonsálvez wrote: > The HW has some limits but, according to the spec, we can create > the image as it has not yet any memory backing it. This patch > logs a debug error and set the size to the UINT64_MAX in order to > avoid a

Re: [Mesa-dev] [PATCH 2/2] radv: Implement VK_EXT_discard_rectangles.

Actually it is not calculated in this way. - The `inside` flags for each of the four clip rectangles form a 4-bit binary number. The corresponding bit in this 16-bit number specifies whether the pixel is visible 1: we can only set clip rules for 4 rectangles at the same time. (b0011 means when

[Mesa-dev] [PATCH v2] i965/meta-util: Convert the clear color through the surf format

When programming the fast clear color there was previously a chunk of code to try to make the color match the constraints of the surface format such as by filling in missing components and handling luminance formats. These cases are not handled by the hardware. There are some additional possible re

Re: [Mesa-dev] [PATCH 04/10] freedreno: implement user constant buffers

On Jan 11, 2018 1:45 PM, "Rob Clark" wrote: On Wed, Jan 10, 2018 at 4:41 PM, Marek Olšák wrote: > On Wed, Jan 10, 2018 at 9:36 PM, Rob Clark wrote: >> On Wed, Jan 10, 2018 at 3:33 PM, Rob Clark wrote: >>> On Wed, Jan 10, 2018 at 2:49 PM, Marek Olšák wrote: From: Marek Olšák --

Re: [Mesa-dev] [PATCH 2/2] radv: Implement VK_EXT_discard_rectangles.

Right, that matches with what I wrote on how to use it? e.g. to write out my first example (3 rects, inclusive): 0/b unset, as none of the first 3 bits are set 1/b0001 set 2/b0010 set 3/b0011 set 4/b0100 set 5/b0101 set 6/b0110 set 7/b0111 set 8/b1000 unset, as none of the first 3 bi

Re: [Mesa-dev] [PATCH] dri_util: remove ALLOW_RGB10_CONFIGS option (v2)

On Jan 11, 2018 7:14 AM, "Mario Kleiner" wrote: On 01/10/2018 07:04 AM, Tapani Pälli wrote: > Hi Marek; > > This one works but only if you add > > DRI_CONF_ALLOW_RGB10_CONFIGS("false") > > to the DRI_CONF_SECTION_MISCELLANEOUS section in intel_screen. With that > change: Reviewed-by: Tapani Päll

Re: [Mesa-dev] [PATCH 2/2] radv: Implement VK_EXT_discard_rectangles.

Oh. I see. I was distracted by the hard coded 16! Thanks. Best Regards, David -Original Message- From: Bas Nieuwenhuizen [mailto:b...@basnieuwenhuizen.nl] Sent: Thursday, January 11, 2018 9:29 PM To: Mao, David Cc: mesa-dev@lists.freedesktop.org Subject: Re: [Mesa-dev] [PATCH 2/2] radv

[Mesa-dev] [PATCH] amd/common: fix loading InstanceID for tess on < GFX9

InstanceID is in VGPR2, not 1. One more failure that CTS didn't catch up... Reported-by: Alex Smith Signed-off-by: Samuel Pitoiset --- src/amd/common/ac_nir_to_llvm.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_ni

Re: [Mesa-dev] [PATCH] amd/common: fix loading InstanceID for tess on < GFX9

On 01/11/2018 02:56 PM, Samuel Pitoiset wrote: InstanceID is in VGPR2, not 1. One more failure that CTS didn't catch up... Reported-by: Alex Smith Signed-off-by: Samuel Pitoiset --- src/amd/common/ac_nir_to_llvm.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/s

Re: [Mesa-dev] r600 sb tessellation support

Hi Dave, Am Mittwoch, den 10.01.2018, 16:48 +1000 schrieb Dave Airlie: > This is an attempt to add tessellation support to the SB backend. > I tried to dig a bit more in the failing piglits, specifically  "1in-1out" that passed with your WIP branch form Jan/9. Now, with sb it fails by drawing

Re: [Mesa-dev] [PATCH 0/6] i965: Misc sRGB and CCS fixes

Hi, This series increases performance except in SynMark DrvState, where drop is acceptable & expected [1]. BXT J4205 improvements: 1-2% in GfxBench Manhattan 3.1 1% in GfxBench CarChase & SynMark Deferred KBL GT2 improvements: 2-3% in GfxBench Manhattan 3.1 1-2% in GfxBench CarChase & Syn

[Mesa-dev] [PATCH v2] r600: Allow egd_tables.py to run with python3 too

diff --git a/src/gallium/drivers/r600/egd_tables.py b/src/gallium/drivers/r600/egd_tables.py index d7b78c7fb1..7bda44ce83 100644 --- a/src/gallium/drivers/r600/egd_tables.py +++ b/src/gallium/drivers/r600/egd_tables.py @@ -1,4 +1,4 @@ - +#!/usr/bin/python3 CopyRight = ''' /* * Copyright 2015 A

[Mesa-dev] [Bug 104558] Building Vulkan Radeon driver fails

https://bugs.freedesktop.org/show_bug.cgi?id=104558 --- Comment #2 from Paul Menzel --- (In reply to Bas Nieuwenhuizen from comment #1) > This looks like it fails to link with LLVM. What LLVM version do you have > installed? I have LLVM 4.0.0 installed built with `-DLLVM_BUILD_LLVM_DYLIB=ON` fro

Re: [Mesa-dev] [PATCH] mesa: require at least 14 UBOs for GL 4.3

On 01/10/2018 06:14 PM, srol...@vmware.com wrote: From: Roland Scheidegger ARB_ubo requires 12 UBOs (per stage) at least, but this limit has been raised by GL 4.3 to 14, so don't advertize GL 4.3 without it (only checking the vertex stage since all drivers probably have the same limit anyway fo

[Mesa-dev] [Bug 104558] Building Vulkan Radeon driver fails

https://bugs.freedesktop.org/show_bug.cgi?id=104558 --- Comment #3 from Paul Menzel --- Created attachment 136663 --> https://bugs.freedesktop.org/attachment.cgi?id=136663&action=edit Output of `./configure …` The errors below are shown. ``` […] llvm-config: error: missing: /usr/lib/libLLVMDe

Re: [Mesa-dev] [PATCH] loader/dri3: Avoid freeing renderbuffers in use

On 2018-01-11 10:53 AM, Thomas Hellstrom wrote: > Upon reception of an event that lowered the number of active back buffers, > the code would immediately try to free all back buffers with an id equal to or > higher than the new number of active back buffers. > > However, that could lead to an acti

[Mesa-dev] [PATCH] mesa: enable ASTC/ETC1 compressed textures

Enable these kind of texture when proper extensions are available. Fixes KHR-GLES2.texture_3d.* with these textures. --- src/mesa/main/teximage.c | 26 +++--- 1 file changed, 19 insertions(+), 7 deletions(-) diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c ind

[Mesa-dev] [Bug 104558] Building Vulkan Radeon driver fails

https://bugs.freedesktop.org/show_bug.cgi?id=104558 --- Comment #4 from Michel Dänzer --- (In reply to Paul Menzel from comment #2) > I have LLVM 4.0.0 installed built with `-DLLVM_BUILD_LLVM_DYLIB=ON` from > Linux from Scratch [1]. Does adding -DLLVM_LINK_LLVM_DYLIB=ON for the LLVM build, then

Re: [Mesa-dev] [PATCH 5/6] i965/draw: Do resolves properly for textures used by TXF

On Thu, Jan 11, 2018 at 1:48 AM, Iago Toral wrote: > On Wed, 2018-01-10 at 11:22 -0800, Jason Ekstrand wrote: > > --- > > src/mesa/drivers/dri/i965/brw_draw.c | 41 > > > > 1 file changed, 41 insertions(+) > > > > diff --git a/src/mesa/drivers/dri/i965/brw_dr

[Mesa-dev] [PATCH] ac: replace llvm.AMDGPU.kilp by llvm.amdgcn.kill with LLVM 6

This also replaces llvm.AMDGPU.kilp by llvm.AMDGPU.kill with LLVM < 6. Similar to RadeonSI codepath. Signed-off-by: Samuel Pitoiset --- src/amd/common/ac_nir_to_llvm.c | 22 -- 1 file changed, 12 insertions(+), 10 deletions(-) diff --git a/src/amd/common/ac_nir_to_llvm.c b/s

Re: [Mesa-dev] [PATCH] glsl/linker: link-error using the same name in unnamed block and outside

On Wed, 2018-01-10 at 14:04 +0200, Tapani Pälli wrote: > > On 01/08/2018 03:19 PM, Juan A. Suarez Romero wrote: > > Please, could someone take a look at this patch? Thanks in advance. > > > > This looks correct to me. Is there some dEQP/Piglit test for this situation? Yes. It fixes KHR-GL*.sha

[Mesa-dev] [PATCH] i965/screen: Allow drirc to set 'allow_rgb10_configs' again.

Since setup of ALLOW_RGB10_CONFIGS was moved to i965's own brw_config_options.xml, this was hard-coded to false and could not be overriden by drirc. Add some parsing into i965's private screen->optionCache to enable drirc again. Fixes: b391fb26df9f1b ("dri_util: remove ALLOW_RGB10_CONFIGS option (

Re: [Mesa-dev] [PATCH] dri_util: remove ALLOW_RGB10_CONFIGS option (v2)

On 01/11/2018 02:39 PM, Marek Olšák wrote: On Jan 11, 2018 7:14 AM, "Mario Kleiner" > wrote: On 01/10/2018 07:04 AM, Tapani Pälli wrote: Hi Marek; This one works but only if you add DRI_CONF_ALLOW_RGB10_CONFIGS("false")

Re: [Mesa-dev] [PATCH] anv: Import mako templates only during execution of anv_extensions

I also prefer Jason's approach of moving just the creation of the mako Template inside the if __main__ block, but leaving the actual string at the top level. Alternatively we could split anv_extensions.py into two files: one that has the constant data, and one that generates anv_extensions.c. I th

Re: [Mesa-dev] [PATCH v2] anv: Import mako templates only during execution of anv_extensions

Reviewed-by: Dylan Baker Quoting Andres Gomez (2018-01-10 13:11:51) > anv_extensions usage from anv_icd was bringing the unwanted dependency > of mako templates for the latter. We don't want that since it will > force the dependency even for distributable tarballs which was not > needed until now

Re: [Mesa-dev] [PATCH 1/7] gallium: Refactor out vl_put_screen and vl_get_screen

Please make sure the new .c file is added in the meson build as well before pushing. Dylan Quoting Christian König (2018-01-10 05:03:44) > My last status was that Leo wanted to commit this. > > Leo are you still waiting for something? > > Christian. > > Am 10.01.2018 um 10:29 schrieb Julien Is

Re: [Mesa-dev] [PATCH] r600: add support for ARB_shader_clock.

I can't test it with my 6870 HD, piglit clock2x32  need OGL 4.3 (wich I force), and then it fails with  evergreen_state.c:4762: evergreen_emit_atomic_buffer_setup:  Assertion `resource' failed. Compute shader piglits with atomics work though. The patch itself looks fine, so Reviewed-by

Re: [Mesa-dev] meson: vdpau broken on r600

On Tuesday, 2018-01-09 10:09:16 -0800, Dylan Baker wrote: > I'm not sure off the top of my head. I don't have an r600 anymore, but I have > an > SI and a nouveau machine, so I'll see if I can reproduce the problem there and > fix it. I can confirm, the symbol is missing when building with meson:

Re: [Mesa-dev] [PATCH] dri_util: remove ALLOW_RGB10_CONFIGS option (v2)

On 01/11/2018 09:14 AM, Tapani Pälli wrote: Yes, but as it broke regular visuals (on some of our testing machines as well) we needed a fast fix for this. While this is an issue, I think the visual corruption has higher priority than this. This can be fixed meanwhile or afterwards when things

Re: [Mesa-dev] [PATCH 2/2] radv/winsys: skip padded empty buffers in chain.

On Thu, Jan 11, 2018 at 5:56 AM, Dave Airlie wrote: > From: Dave Airlie > > When chaining buffers if we see an empty one (just padded) > skip over it. > --- > src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.c | 19 +++ > 1 file changed, 15 insertions(+), 4 deletions(-) > > diff --git

Re: [Mesa-dev] [PATCH] i965/screen: Allow drirc to set 'allow_rgb10_configs' again.

Reviewed-by: Marek Olšák Marek On Thu, Jan 11, 2018 at 5:00 PM, Mario Kleiner wrote: > Since setup of ALLOW_RGB10_CONFIGS was moved to i965's own > brw_config_options.xml, this was hard-coded to false and > could not be overriden by drirc. Add some parsing into > i965's private screen->optionCa

Re: [Mesa-dev] [RFC libdrm 0/5] Move alloc_handle_t from gralloc impls.

Heya, On 12/22/17 1:09 PM, Tomasz Figa wrote: On Fri, Dec 22, 2017 at 10:09 AM, Gurchetan Singh wrote: So the plan is for alloc_handle_t to not be sub-classed by the implementations, but have all necessary information that an implementation would need? If so, how do we reconcile the implement

[Mesa-dev] [PATCH 3/3] ac: add 'const' qualifiers to the shader info pass

For clarification purposes. Signed-off-by: Samuel Pitoiset --- src/amd/common/ac_shader_info.c | 17 ++--- src/amd/common/ac_shader_info.h | 2 +- 2 files changed, 11 insertions(+), 8 deletions(-) diff --git a/src/amd/common/ac_shader_info.c b/src/amd/common/ac_shader_info.c index

[Mesa-dev] [PATCH 1/3] nir: add a 'const' qualifier to nir_ssa_def_components_read()

To avoid compilation warnings and because this helper shouldn't update anything. Signed-off-by: Samuel Pitoiset --- src/compiler/nir/nir.c | 2 +- src/compiler/nir/nir.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/compiler/nir/nir.c b/src/compiler/nir/nir.c index 6

[Mesa-dev] [PATCH 2/3] ac: remove unused ac_nir_compiler_options from gather_info_input_decl()

Signed-off-by: Samuel Pitoiset --- src/amd/common/ac_shader_info.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/amd/common/ac_shader_info.c b/src/amd/common/ac_shader_info.c index 443980c7d1..9dc85fb74f 100644 --- a/src/amd/common/ac_shader_info.c +++ b/src/amd/co

Re: [Mesa-dev] [PATCH 3/3] ac: add 'const' qualifiers to the shader info pass

Reviewed-by: Bas Nieuwenhuizen for the series On Thu, Jan 11, 2018 at 10:07 PM, Samuel Pitoiset wrote: > For clarification purposes. > > Signed-off-by: Samuel Pitoiset > --- > src/amd/common/ac_shader_info.c | 17 ++--- > src/amd/common/ac_shader_info.h | 2 +- > 2 files changed,

Re: [Mesa-dev] [PATCH] ac/nir: fix translation of nir_op_frcp for doubles

Reviewed-by: Samuel Pitoiset On 01/11/2018 11:14 AM, Timothy Arceri wrote: Without this we end up with the llvm error message: "Both operands to a binary operator are not of the same type!" --- This is an additional fix on top of this series: https://patchwork.freedesktop.org/series/3631

Re: [Mesa-dev] [PATCH] radv: set no-signed-zeros-fp-math

On 01/11/2018 11:08 AM, Józef Kucia wrote: On Wed, Jan 10, 2018 at 10:31 PM, Samuel Pitoiset wrote: + LLVMAddTargetDependentFunctionAttr(main_function, + "no-signed-zeros-fp-math", + "true"); + Is this o

Re: [Mesa-dev] [PATCH 2/2] ac/nir: fix translation of nir_op_fsign for doubles

Series is: Reviewed-by: Samuel Pitoiset On 01/11/2018 12:35 PM, Timothy Arceri wrote: Without this we end up with the llvm error message: "Both operands to a binary operator are not of the same type!" --- src/amd/common/ac_nir_to_llvm.c | 26 +++--- 1 file changed, 19 i

Re: [Mesa-dev] [PATCH 2/2] radv: inline push constants where possible.

On 01/11/2018 04:03 AM, Dave Airlie wrote: From: Dave Airlie Instead of putting the push constants into the upload buffer, if we have space in the sgprs we can upload the per-stage constants into the shaders directly. This saves a few reads from memory in the meta shaders, we should also be

Re: [Mesa-dev] [PATCH 17/29] anv/cmd_buffer: Move the color portion of clear_subpass into begin_subpass

On Fri, Dec 08, 2017 at 04:16:04PM +0200, Pohjolainen, Topi wrote: > On Mon, Nov 27, 2017 at 07:06:07PM -0800, Jason Ekstrand wrote: > > This doesn't really change much now but it will give us more/better > > control over clears in the future. The one interesting functional > > change here is that

[Mesa-dev] [Bug 104558] Building Vulkan Radeon driver fails

https://bugs.freedesktop.org/show_bug.cgi?id=104558 Paul Menzel changed: What|Removed |Added Status|NEEDINFO|NEW --- Comment #5 from Paul Menzel ---

Re: [Mesa-dev] [PATCH v2] isl: don't crash when creating a huge image

Sorry. It's taken a bit of time but the WG has a decision and that is that we are supposed to throw VK_ERROR_OUT_OF_DEVICE_MEMORY in this case. I believe you had an alternate version of the patch that did something like that. If so, we should revive it. On Thu, Jan 11, 2018 at 5:04 AM, Samuel I

[Mesa-dev] [PATCH] nouveau: Support fence FDs

From: Thierry Reding Implements fence FDs based on new libdrm API and the accompanying IOCTL. Signed-off-by: Thierry Reding --- For the kernel patches that add the new IOCTL, see the series at: https://patchwork.freedesktop.org/series/36361/ The libdrm patch that adds the new API is h

[Mesa-dev] [Bug 104214] Dota crashes when switching from game to desktop

https://bugs.freedesktop.org/show_bug.cgi?id=104214 --- Comment #45 from Mark Janes --- Thomas: do you have any ideas on how we could catch this category of bug in automated testing? We have comprehensive automated tests for GL/Vulkan apis, but not much for dri3. -- You are receiving this mai

[Mesa-dev] [PATCH] r600: fix relocs for PIPE_QUERY_SO_OVERFLOW_ANY_PREDICATE query

From: Roland Scheidegger The command parser is very sad if we don't emit the relocs per hw query... However, don't enable it. It mostly works, but piglit arb_transform_feedback_overflow_query-basic shows 2 failures (it's really the same case for the hw), conditional_render_any and conditional_re

Re: [Mesa-dev] [PATCH] loader/dri3: Avoid freeing renderbuffers in use

On Thursday, January 11, 2018 1:53:48 AM PST Thomas Hellstrom wrote: > Upon reception of an event that lowered the number of active back buffers, > the code would immediately try to free all back buffers with an id equal to or > higher than the new number of active back buffers. > > However, that

Re: [Mesa-dev] [RFC libdrm 0/5] Move alloc_handle_t from gralloc impls.

> > So I had a look into implementing this, and using function pointers is > problematic due to this struct being passed between processes which would > prevent mesa calling a function assigned in gbm_gralloc for example. Which function? In theory, anything with a dependency on libdrm and a gral

[Mesa-dev] [PATCH 01/10] glsl/tests: reimplement warnings-test in python

This reimplements the test in python with a shell script wrapper that allows autotools to continue to run the test without realizing that anything has changed. Using python has two advantages, first it's portable so this test can be run on windows as well as Linux since it just requires python, no

[Mesa-dev] [PATCH 04/10] meson: enable optimization-test

Signed-off-by: Dylan Baker --- src/compiler/glsl/tests/meson.build | 5 + 1 file changed, 5 insertions(+) diff --git a/src/compiler/glsl/tests/meson.build b/src/compiler/glsl/tests/meson.build index 11fb4fb..d5e6b5f 100644 --- a/src/compiler/glsl/tests/meson.build +++ b/src/compiler/glsl/te

[Mesa-dev] [PATCH 06/10] glcpp/tests: Convert shell scripts to a python script

This ports glcpp-test.sh and glcpp-test-cr-lf.sh to a python script that accepts arguments for each line ending type. This should allow for better reporting to users. Signed-off-by: Dylan Baker --- src/compiler/glsl/glcpp/tests/glcpp-test-cr-lf.sh | 148 +-- src/compiler/glsl/glcpp/tests

Re: [Mesa-dev] meson: vdpau broken on r600

I couldn't reproduce with nouveau, vdpauinfo seems hardcoded to look for vdpau_nvidia, and just ignores vdpau_nouveau. I have a patch that adds the symbol, but it feels ugly. I'm sending it and we can test it, if it does fix it I guess we should figure out why we need to keep adding --Wl,-whole-a

[Mesa-dev] [PATCH 05/10] glsl/tests: Remove unused compare_ir.py script

Signed-off-by: Dylan Baker --- src/compiler/glsl/tests/compare_ir.py | 58 + 1 file changed, 58 deletions(-) delete mode 100644 src/compiler/glsl/tests/compare_ir.py diff --git a/src/compiler/glsl/tests/compare_ir.py b/src/compiler/glsl/tests/compare_ir.py deleted f

[Mesa-dev] [PATCH 03/10] glsl/tests: Convert optimization-test.sh to pure python

This patch converts optimization-test.sh to python, in this process it removes external shell dependencies including diff. It replaces the python script that generates shell scripts with a python library that generates test cases and runs them using subprocess. Signed-off-by: Dylan Baker --- src

Re: [Mesa-dev] [PATCH v2] r600: Allow egd_tables.py to run with python3 too

This is somewhat insufficient, it might run okay with python3, but the behavior will be different between the two. Please add `from __future__ import print_function`, so that the behavior will be the same, not just the syntax. Also, the StringTable, InteTable, Field, and Reg classes is also an old

[Mesa-dev] [PATCH 00/10] Finish unit tests for meson build

This series adds the rest of the unit tests in the mesa tree, including the gallium test binaries. The relevant tests are wired into the `ninja test` command, and all tests pass. A big part of this series is converting tests from shell based to a pure python base. These are mostly straight forward

[Mesa-dev] [Test Patch] Meson: ensure vdpau has proper symbols exposed

Signed-off-by: Dylan Baker --- src/gallium/targets/vdpau/meson.build | 16 +--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/src/gallium/targets/vdpau/meson.build b/src/gallium/targets/vdpau/meson.build index 67f1469fb0f..432a32a7340 100644 --- a/src/gallium/targets

[Mesa-dev] [PATCH 07/10] meson: enable glcpp test

Signed-off-by: Dylan Baker --- src/compiler/glsl/glcpp/meson.build | 18 +- 1 file changed, 17 insertions(+), 1 deletion(-) diff --git a/src/compiler/glsl/glcpp/meson.build b/src/compiler/glsl/glcpp/meson.build index 17ee1a2..9f869da 100644 --- a/src/compiler/glsl/glcpp/meson.bu

[Mesa-dev] [PATCH 10/10] meson: build gallium unit tests

Signed-off-by: Dylan Baker --- src/gallium/tests/meson.build | 1 +- src/gallium/tests/unit/meson.build | 31 +++- 2 files changed, 32 insertions(+) create mode 100644 src/gallium/tests/unit/meson.build diff --git a/src/gallium/tests/meson.build b/src/gallium/t

[Mesa-dev] [PATCH 02/10] meson: run glsl compiler warnings test

Signed-off-by: Dylan Baker --- src/compiler/glsl/tests/meson.build | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/src/compiler/glsl/tests/meson.build b/src/compiler/glsl/tests/meson.build index 27f3407..11fb4fb 100644 --- a/src/compiler/glsl/tests/meson.build +++

[Mesa-dev] [PATCH 08/10] meson: Remove TODO about mesa/main tests

They're already done. Signed-off-by: Dylan Baker --- src/mesa/meson.build | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/mesa/meson.build b/src/mesa/meson.build index ab6bc27..ce612fb 100644 --- a/src/mesa/meson.build +++ b/src/mesa/meson.build @@ -18,8 +18,6 @@ # OUT OF OR IN CONNECT

  1   2   >