Re: [Mesa-dev] [PATCH mesa 2/2] mesa: allow user to set MESA_NO_ERROR=0

On 06/09/17 11:23 PM, Eric Engestrom wrote: > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102530 > Cc: Michel Dänzer > Cc: Alexandre Demers > Signed-off-by: Eric Engestrom > --- > src/mesa/main/context.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/src/

Re: [Mesa-dev] [PATCH 0/9] RadeonSI: Tessellation shader micro-optimizations

On 06.09.2017 19:03, Marek Olšák wrote: Hi, This series seemed like a good idea and results in eliminations of shader instructions. However I haven't been able to find an app where it has a measurable effect. Patches 2, 4-9: Reviewed-by: Nicolai Hähnle Please review. Marek _

Re: [Mesa-dev] [PATCH 1/9] tgsi/scan: add a new pass that analyzes tess factor writes

On 06.09.2017 19:03, Marek Olšák wrote: From: Marek Olšák The pass tries to deduce whether tess factors are always written by invocation 0 (at least). The implication for radeonsi is that it doesn't have to use a barrier near the end of TCS, and doesn't have to use LDS for passing the tess fac

Re: [Mesa-dev] [PATCH mesa 2/2] mesa: allow user to set MESA_NO_ERROR=0

Thanks for fixing this Eric. On 09/06/2017 04:23 PM, Eric Engestrom wrote: Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102530 Cc: Michel Dänzer Cc: Alexandre Demers Signed-off-by: Eric Engestrom --- src/mesa/main/context.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)

Re: [Mesa-dev] [PATCH v2 3/3] docs/release-calendar: update and extend

On Tue, 2017-09-05 at 18:41 +0100, Emil Velikov wrote: > From: Emil Velikov > > v2: Use correct 17.1.10 version, adjust some names. > > Cc: Juan A. Suárez > Cc: Andres Gomez > Signed-off-by: Emil Velikov > Reviewed-by: Eric Engestrom > --- > docs/release-calendar.html | 33 -

Re: [Mesa-dev] [PATCH 23/23] anv: Implement VK_ANDROID_native_buffer

On 09/02/2017 11:17 AM, Chad Versace wrote: This implementation is correct (afaict), but takes two shortcuts regarding the import/export of Android sync fds. Shortcut 1. When Android calls vkAcquireImageANDROID to import a sync fd into a VkSemaphore or VkFence, the driver instead simply b

Re: [Mesa-dev] [PATCH 12/23] intel: Add simple logging façade for Android

Hi, On 06.09.2017 23:25, Chad Versace wrote: On Mon 04 Sep 2017, Eero Tamminen wrote: On 02.09.2017 11:17, Chad Versace wrote: I'm bringing up Vulkan in the Android container of Chrome OS (ARC++). On Android, stdio goes to /dev/null. On Android, remote gdb is even more painful than the usual

Re: [Mesa-dev] [PATCH] radv: use simpler indirect packet 3 if possible.

I'm not really happy with this, what happens if a game actually uses e.g. indirect count in a secondary cmd buf? That said this patch seems correct, so Reviewed-by: Bas Nieuwenhuizen You may want to nominate this for stable (and provide a backport probably..) On Thu, Sep 7, 2017, at 05:03, Da

Re: [Mesa-dev] [PATCH] radv/gfx9: allocate events from uncached VA space

Reviewed-by: Bas Nieuwenhuizen And nominate for stable? On Thu, Sep 7, 2017, at 05:44, Dave Airlie wrote: > From: Dave Airlie > > This copies what amdgpu-pro does, and allocates the memory > for an event with an uncached mtype. > > This fixes hangs with: > dEQP-VK.api.command_buffers.record_s

Re: [Mesa-dev] [PATCH 1/2] radeonsi: enable out-of-order rasterization when possible on VI and GFX9 dGPUs

On 07.09.2017 00:35, Marek Olšák wrote: From: Marek Olšák --- src/gallium/drivers/radeonsi/si_pipe.c | 2 + src/gallium/drivers/radeonsi/si_pipe.h | 1 + src/gallium/drivers/radeonsi/si_state.c | 143 +++- src/gallium/drivers/radeonsi/si_st

Re: [Mesa-dev] [PATCH] radv: use simpler indirect packet 3 if possible.

On 07.09.2017 09:58, Bas Nieuwenhuizen wrote: I'm not really happy with this, what happens if a game actually uses e.g. indirect count in a secondary cmd buf? Note that some packets that require fetching data in the CP cannot be run in secondary command buffers, because the data fetching path

[Mesa-dev] [PATCH] mesa/st: Fix frontbuffer rendering regression

This fixes a regression introduced with commit eceb6710024716433069d705fbd873d6d136c2cc: "mesa/st: Reduce the number of frontbuffer flush calls" where we, after flushing the front buffer marked it as not-rendered-to, the idea being that it should be marked as "rendered-to" again as soon as any rend

Re: [Mesa-dev] [PATCH] nir/spirv: fix crashes when dereferencing a pointer for an OpVariable

This patch is unreviewed. On Tue, 2017-08-29 at 12:04 +0200, Samuel Iglesias Gonsálvez wrote: > When creating a vtn_pointer for an OpVariable, the block_index and > offsets fields are null because there is not ssa to take the data > from. > > However, we can dereference that pointer when processi

Re: [Mesa-dev] [PATCH v2] nir/spirv: fix chain access with different index bit sizes

This patch is unreviewed. On Tue, 2017-08-29 at 08:42 +0200, Samuel Iglesias Gonsálvez wrote: > Currently we support 32-bit indexes/offsets all over the driver, so > we > convert them to that bit size. > > Fixes dEQP-VK.spirv_assembly.instruction.*.indexing.* > > Signed-off-by: Samuel Iglesias G

Re: [Mesa-dev] [PATCH mesa 2/2] mesa: allow user to set MESA_NO_ERROR=0

Hi Eric, can you update the wording in docs/envvars.html to reflect the change. Thanks Emil ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH 1/2] radeonsi: set a per-buffer flag that disables inter-process sharing (v4)

On 01/09/17 07:40 PM, Christian König wrote: > Am 01.09.2017 um 12:28 schrieb Michel Dänzer: >> On 01/09/17 07:23 PM, Nicolai Hähnle wrote: >>> On 01.09.2017 11:58, Michel Dänzer wrote: On 29/08/17 11:47 PM, Christian König wrote: > From: Marek Olšák > > For lower overhead in the

[Mesa-dev] [Bug 102516] regression on glsl shaders on BE architecture mesa-17.2.0-rc6

https://bugs.freedesktop.org/show_bug.cgi?id=102516 --- Comment #1 from intermedi...@hotmail.com --- Tested on 17.2 offical and same issue this come on Mame mame -w ../../../../../src/osd/modules/opengl/gl_shader_tool.cpp:358: GL Error: object 0x1 compilation failed ../../../../../src/osd/modu

Re: [Mesa-dev] [PATCH 1/2] radeonsi: set a per-buffer flag that disables inter-process sharing (v4)

Am 07.09.2017 um 11:23 schrieb Michel Dänzer: On 01/09/17 07:40 PM, Christian König wrote: Am 01.09.2017 um 12:28 schrieb Michel Dänzer: On 01/09/17 07:23 PM, Nicolai Hähnle wrote: On 01.09.2017 11:58, Michel Dänzer wrote: On 29/08/17 11:47 PM, Christian König wrote: From: Marek Olšák For

Re: [Mesa-dev] [PATCH 1/2] radeonsi: set a per-buffer flag that disables inter-process sharing (v4)

On Sep 7, 2017 12:08 PM, "Christian König" wrote: Am 07.09.2017 um 11:23 schrieb Michel Dänzer: > On 01/09/17 07:40 PM, Christian König wrote: > >> Am 01.09.2017 um 12:28 schrieb Michel Dänzer: >> >>> On 01/09/17 07:23 PM, Nicolai Hähnle wrote: >>> On 01.09.2017 11:58, Michel Dänzer wrote:

[Mesa-dev] [PATCH mesa 2/6] dri/radeon: use ARRAY_SIZE macro

Signed-off-by: Eric Engestrom --- src/mesa/drivers/dri/radeon/radeon_tcl.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/radeon/radeon_tcl.c b/src/mesa/drivers/dri/radeon/radeon_tcl.c index 3e2f426160..61ff2311e9 100644 --- a/src/mesa/drivers/dri/rad

[Mesa-dev] [PATCH mesa 5/6] targets/nine: use ARRAY_SIZE macro

Signed-off-by: Eric Engestrom --- src/gallium/targets/d3dadapter9/description.c | 9 + src/gallium/targets/d3dadapter9/drm.c | 3 ++- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/src/gallium/targets/d3dadapter9/description.c b/src/gallium/targets/d3dadapter9/des

[Mesa-dev] [PATCH mesa 3/6] r300: use ARRAY_SIZE macro

Signed-off-by: Eric Engestrom --- src/gallium/drivers/r300/compiler/r300_fragprog_swizzle.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/r300/compiler/r300_fragprog_swizzle.c b/src/gallium/drivers/r300/compiler/r300_fragprog_swizzle.c index 075eac50f

[Mesa-dev] [PATCH mesa 4/6] swr: use ARRAY_SIZE macro

Signed-off-by: Eric Engestrom --- src/gallium/drivers/swr/rasterizer/memory/StoreTile.h | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/gallium/drivers/swr/rasterizer/memory/StoreTile.h b/src/gallium/drivers/swr/rasterizer/memory/StoreTile.h index c3d14e9509..3

[Mesa-dev] [PATCH mesa 1/6] glx: use ARRAY_SIZE macro

Signed-off-by: Eric Engestrom --- src/glx/driwindows_glx.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/glx/driwindows_glx.c b/src/glx/driwindows_glx.c index 02d95e7bfd..85525431bf 100644 --- a/src/glx/driwindows_glx.c +++ b/src/glx/driwindows_glx.c @@ -24,6 +24,7 @@

Re: [Mesa-dev] [PATCH mesa 2/2] mesa: allow user to set MESA_NO_ERROR=0

On Thursday, 2017-09-07 10:13:48 +0100, Emil Velikov wrote: > Hi Eric, can you update the wording in docs/envvars.html to reflect the > change. Yes indeed, I forgot that; done now: 6c2e0527ea7a387d3e19 > > Thanks > Emil ___ mesa-dev mailing list mesa-

[Mesa-dev] [PATCH mesa 6/6] gallium/tests: use ARRAY_SIZE macro

Signed-off-by: Eric Engestrom --- src/gallium/tests/graw/fs-fragcoord.c | 4 +++- src/gallium/tests/graw/fs-frontface.c | 4 +++- src/gallium/tests/graw/fs-write-z.c | 4 +++- 3 files changed, 9 insertions(+), 3 deletions(-) diff --git a/src/gallium/tests/graw/fs-fragcoord.c b/src/gallium/tes

Re: [Mesa-dev] [PATCH 1/2] radeonsi: set a per-buffer flag that disables inter-process sharing (v4)

Am 07.09.2017 um 12:14 schrieb Marek Olšák: On Sep 7, 2017 12:08 PM, "Christian König" > wrote: Am 07.09.2017 um 11:23 schrieb Michel Dänzer: On 01/09/17 07:40 PM, Christian König wrote: Am 01.09.2017 um 12:28 schrieb Michel Dänzer:

Re: [Mesa-dev] [PATCH] radv: use simpler indirect packet 3 if possible.

On 7 Sep. 2017 6:34 pm, "Nicolai Hähnle" wrote: On 07.09.2017 09:58, Bas Nieuwenhuizen wrote: > I'm not really happy with this, what happens if a game actually uses > e.g. indirect count in a secondary cmd buf? > Note that some packets that require fetching data in the CP cannot be run in seco

Re: [Mesa-dev] [PATCH] st/mesa: skip draw calls with pipe_draw_info::count == 0

On Thu, Sep 7, 2017 at 2:27 AM, Timothy Arceri wrote: > On 07/09/17 08:37, Marek Olšák wrote: >> >> Ping. >> >> On Sat, Sep 2, 2017 at 1:34 AM, Marek Olšák wrote: >>> >>> From: Marek Olšák >>> >>> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102502 >>> >>> Cc: 17.2 >>> --- >>> src/m

Re: [Mesa-dev] [PATCH 1/9] tgsi/scan: add a new pass that analyzes tess factor writes

We can also say if gl_TessLevel* is written multiple times, then one these must be true: - there must not be a barrier between the writes - gl_TessLevel* writes aren't inside conditional blocks I see that these games use a barrier in TCS: - Hitman - Grid Autosport - Tomb Raider All constraints ar

Re: [Mesa-dev] [PATCH mesa 1/6] glx: use ARRAY_SIZE macro

On 07/09/2017 11:21, Eric Engestrom wrote: Signed-off-by: Eric Engestrom --- src/glx/driwindows_glx.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/glx/driwindows_glx.c b/src/glx/driwindows_glx.c index 02d95e7bfd..85525431bf 100644 --- a/src/glx/driwindows_glx.c ++

Re: [Mesa-dev] [PATCH 2/2] mesa/st/tests: Fix regressions with libunwind enabled introduced with 7be6d8fe12

On 07.09.2017 08:38, Michel Dänzer wrote: On 07/09/17 04:24 AM, Gert Wollny wrote: Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=102565 Add the according flags to link with libunwind. --- src/mesa/state_tracker/tests/Makefile.am | 1 + 1 file changed, 1 insertion(+) diff --git a/src/m

[Mesa-dev] [Bug 102565] u_debug_stack.c:114: undefined reference to `_Ux86_64_getcontext'

https://bugs.freedesktop.org/show_bug.cgi?id=102565 Nicolai Hähnle changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

Re: [Mesa-dev] [PATCH] nir/spirv: handle if's with same label in both branches

This patch is still unreviewed. Gently ping if someone can take a look. Thanks in advance! J.A. On Thu, 2017-08-24 at 17:16 +0200, Juan A. Suarez Romero wrote: > When a conditional branch has the same labels in the "if" part and in the > "else" part, then we have the same cfg block, an

Re: [Mesa-dev] [PATCH 2/2] mesa/st/tests: Fix regressions with libunwind enabled introduced with 7be6d8fe12

Hi Gert, A couple of small suggestions for the future. On 6 September 2017 at 20:24, Gert Wollny wrote: > Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=102565 > > Add the according flags to link with libunwind. Please keep tags at the bottom of the summary. Fixes: denotes the commit, while

Re: [Mesa-dev] [PATCH 1/9] tgsi/scan: add a new pass that analyzes tess factor writes

On 07.09.2017 12:55, Marek Olšák wrote: We can also say if gl_TessLevel* is written multiple times, then one these must be true: - there must not be a barrier between the writes - gl_TessLevel* writes aren't inside conditional blocks I see that these games use a barrier in TCS: - Hitman - Grid A

Re: [Mesa-dev] [PATCH 1/9] tgsi/scan: add a new pass that analyzes tess factor writes

On 07.09.2017 14:23, Nicolai Hähnle wrote: On 07.09.2017 12:55, Marek Olšák wrote: We can also say if gl_TessLevel* is written multiple times, then one these must be true: - there must not be a barrier between the writes - gl_TessLevel* writes aren't inside conditional blocks I see that these g

Re: [Mesa-dev] [PATCH] radv: use simpler indirect packet 3 if possible.

On 07.09.2017 12:26, Dave Airlie wrote: On 7 Sep. 2017 6:34 pm, "Nicolai Hähnle" > wrote: On 07.09.2017 09 :58, Bas Nieuwenhuizen wrote: I'm not really happy with this, what happens if a game actually uses e.g. indirect count in a secon

Re: [Mesa-dev] [PATCH] radv: use simpler indirect packet 3 if possible.

On Thu, Sep 7, 2017, at 14:52, Nicolai Hähnle wrote: > On 07.09.2017 12:26, Dave Airlie wrote: > > > > > > On 7 Sep. 2017 6:34 pm, "Nicolai Hähnle" > > wrote: > > > > On 07.09.2017 09 :58, Bas Nieuwenhuizen wrote: > > > > I'm not really happy with this,

Re: [Mesa-dev] [PATCH 1/2] radeonsi: enable out-of-order rasterization when possible on VI and GFX9 dGPUs

Am 07.09.2017 00:35, schrieb Marek Olšák: From: Marek Olšák --- src/gallium/drivers/radeonsi/si_pipe.c | 2 + src/gallium/drivers/radeonsi/si_pipe.h | 1 + src/gallium/drivers/radeonsi/si_state.c | 143 +++- src/gallium/drivers/radeonsi/si_st

[Mesa-dev] [PATCH] radv: add an assertion when pushing meta descriptor sets

Just to make sure we are using the set 0, because it's the only one which is saved/restored when doing meta operations. Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_cmd_buffer.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_c

[Mesa-dev] [Bug 102564] swr: GPU Caps Viewer crashes with any 3D demo

https://bugs.freedesktop.org/show_bug.cgi?id=102564 --- Comment #2 from Bruce Cherniak --- > Regression in 17.2.0. v17.1.8 is unaffected. I haven't had a chance to try it yet, but you're saying that 17.1.8 runs without crashing? That's helpful. Thanks. -- You are receiving this mail because:

Re: [Mesa-dev] [PATCH 1/9] tgsi/scan: add a new pass that analyzes tess factor writes

On Thu, Sep 7, 2017 at 2:24 PM, Nicolai Hähnle wrote: > On 07.09.2017 14:23, Nicolai Hähnle wrote: >> >> On 07.09.2017 12:55, Marek Olšák wrote: >>> >>> We can also say if gl_TessLevel* is written multiple times, then one >>> these must be true: >>> - there must not be a barrier between the writes

Re: [Mesa-dev] [PATCH 01/10] i965/screen: Add basic support for rendering 10 bpc/depth 30 framebuffers.

Hi Mario, On 5 September 2017 at 06:01, Mario Kleiner wrote: > Expose formats which are supported at least back to Gen 5 Ironlake, > possibly further. Allow creation of 10 bpc winsys buffers for drawables. > > glxinfo now lists new RGBA 10 10 10 2/0 formats. > Works correctly under DRI2 without c

Re: [Mesa-dev] [PATCH 06/10] dri/common: Add option to disable exposure of 10 bpc color configs.

On 5 September 2017 at 06:01, Mario Kleiner wrote: > A few clients don't like RGB10X2 and RGB10A2 fbconfigs and > visuals. Add a new driconf option 'expose_rgb10_configs' to > allow per application enable/disable. > > The option defaults to enabled. > Most configs tends to be called "force.." or "

Re: [Mesa-dev] [PATCH 1/2] radeonsi: enable out-of-order rasterization when possible on VI and GFX9 dGPUs

On Thu, Sep 7, 2017 at 7:33 AM, Axel Davy wrote: > Hi, > > On 07/09/2017 00:35, Marek Olšák wrote: >> >> + /* Out-of-order rasterization can be enabled for these cases: >> +* >> +* - color-only rendering: >> +* + blending must be enabled and commutative >> +

Re: [Mesa-dev] [PATCH 09/10] egl/x11: Match depth 30 RGB visuals to 32-bit RGBA EGLConfigs.

On 7 September 2017 at 01:51, Mario Kleiner wrote: > On 09/06/2017 03:18 PM, Eric Engestrom wrote: >> >> On Tuesday, 2017-09-05 07:01:13 +0200, Mario Kleiner wrote: >>> >>> Similar to the matching of 24 bit RGB visuals to 32-bit >>> RGBA EGLConfigs. >>> >>> Fixes failure of piglit egl tests to sel

[Mesa-dev] [PATCH] ac/surface: add radeon_surf::has_stencil for convenience

From: Marek Olšák --- src/amd/common/ac_surface.c| 2 ++ src/amd/common/ac_surface.h| 1 + src/amd/vulkan/radv_device.c | 6 +++--- src/gallium/drivers/r600/evergreen_state.c | 2 +- src/gallium/drivers/r600/r600_blit.

Re: [Mesa-dev] [PATCH 10/10] egl/dri2: Add Wayland+EGL support for RGB10 winsys buffers.

Hi Mario, On 5 September 2017 at 06:01, Mario Kleiner wrote: > Successfully tested under Weston 3.0, both with the new > (experimental) dmabuf+modifiers path, and the old buffer > import path. Photometer confirms 10 rgb bits from rendering > to display. > Can you split the patch in to wl_drm and

Re: [Mesa-dev] [PATCH 1/2] radeonsi: enable out-of-order rasterization when possible on VI and GFX9 dGPUs

>> +static bool si_out_of_order_rasterization(struct si_context *sctx) >> +{ >> + struct si_state_blend *blend = sctx->queued.named.blend; >> + struct si_state_dsa *dsa = sctx->queued.named.dsa; >> + >> + if (!sctx->screen->has_out_of_order_rast) >> + return false;

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

https://bugs.freedesktop.org/show_bug.cgi?id=102573 --- Comment #1 from Emil Velikov --- Hmm the code in src/util/u_atomic.c should handle that. The code is omitted when the GCC_64BIT_ATOMICS_SUPPORTED test [in configure.ac] reports "it works". I don't have a armel setup handy, so if anyone anyo

Re: [Mesa-dev] Mesa 17.1.9 release candidate

Hi all, For anyone wondering why a couple of commits are missing from the list, those have been squashed as applicable. Namely: > Charmaine Lee (1): > vbo: fix offset in minmax cache key Squashed with vbo: fix build errors on android > Jason Ekstrand (2): > anv/formats: Nicel

Re: [Mesa-dev] [PATCH 7/7] winsys/amdgpu: use the new raw CS API

BTW, this series is a prep work for Android fences. Marek On Wed, Sep 6, 2017 at 11:34 AM, Marek Olšák wrote: > From: Marek Olšák > > This also cleans things up. > --- > src/gallium/winsys/amdgpu/drm/amdgpu_cs.c | 165 > -- > src/gallium/winsys/amdgpu/drm/amdgpu_cs

Re: [Mesa-dev] [PATCH mesa 6/6] gallium/tests: use ARRAY_SIZE macro

Reviewed-by: Marek Olšák Marek On Thu, Sep 7, 2017 at 12:21 PM, Eric Engestrom wrote: > Signed-off-by: Eric Engestrom > --- > src/gallium/tests/graw/fs-fragcoord.c | 4 +++- > src/gallium/tests/graw/fs-frontface.c | 4 +++- > src/gallium/tests/graw/fs-write-z.c | 4 +++- > 3 files changed,

Re: [Mesa-dev] [PATCH v2 1/4] dri/image: Add a format modifier attributes query

On 5 September 2017 at 19:29, Jason Ekstrand wrote: > On Tue, Sep 5, 2017 at 10:14 AM, Emil Velikov > wrote: >> >> Hi Jason, >> >> On 5 September 2017 at 16:48, Jason Ekstrand wrote: >> >> > + GLboolean (*queryDmaBufFormatModifierAttribs)(__DRIscreen *screen, >> > +

Re: [Mesa-dev] [PATCH] ac/surface: add radeon_surf::has_stencil for convenience

Reviewed-by: Nicolai Hähnle On 07.09.2017 16:42, Marek Olšák wrote: From: Marek Olšák --- src/amd/common/ac_surface.c| 2 ++ src/amd/common/ac_surface.h| 1 + src/amd/vulkan/radv_device.c | 6 +++--- src/gallium/driver

Re: [Mesa-dev] [PATCH 1/2] radeonsi: enable out-of-order rasterization when possible on VI and GFX9 dGPUs

On 07.09.2017 16:56, Marek Olšák wrote: +static bool si_out_of_order_rasterization(struct si_context *sctx) +{ + struct si_state_blend *blend = sctx->queued.named.blend; + struct si_state_dsa *dsa = sctx->queued.named.dsa; + + if (!sctx->screen->has_out_of_order_rast) +

Re: [Mesa-dev] [PATCH 08/10] drirc: Don't expose 10 bpc visuals/configs to gnome-shell.

[+ Dave] We can also ignore gnome-shell in Mesa for now and let the gnome-shell maintainers fix the issue in gnome-shell. Marek On Tue, Sep 5, 2017 at 7:01 AM, Mario Kleiner wrote: > Set 'expose_rgb10_configs' false when gnome-shell is the > client. > > Gnome-Shell/Wayland (= Mutter drm/kms way

Re: [Mesa-dev] [PATCH] mesa/st: Fix frontbuffer rendering regression

Reviewed-by: Bruce Cherniak > On Sep 7, 2017, at 3:53 AM, Thomas Hellstrom wrote: > > This fixes a regression introduced with commit > eceb6710024716433069d705fbd873d6d136c2cc: > "mesa/st: Reduce the number of frontbuffer flush calls" > where we, after flushing the front buffer marked it as no

[Mesa-dev] [PATCH 4/4] egl: use switch statements over if/else chain

From: Emil Velikov Shorter, explicit and consistent with the rest of the co debase. Signed-off-by: Emil Velikov --- src/egl/main/eglapi.c | 59 --- 1 file changed, 28 insertions(+), 31 deletions(-) diff --git a/src/egl/main/eglapi.c b/src/egl/ma

Re: [Mesa-dev] [PATCH] mesa/st: Fix frontbuffer rendering regression

Reviewed-by: Marek Olšák Marek On Thu, Sep 7, 2017 at 10:53 AM, Thomas Hellstrom wrote: > This fixes a regression introduced with commit > eceb6710024716433069d705fbd873d6d136c2cc: > "mesa/st: Reduce the number of frontbuffer flush calls" > where we, after flushing the front buffer marked it as

[Mesa-dev] [PATCH 3/4] egl: use _eglError's 'msg' as an actual message in EGL_KHR_debug

From: Emil Velikov Seemingly, the original intent behind _eglError's 'msg' was aimed to provide a function name. At some point, people started using it the way EGL_KHR_debug's callback() message is meant to be used. Aka providing meaningful information to the developer/user. Swap the funcName/m

[Mesa-dev] [PATCH 2/4] egl: remove unneeded braces around since line if statements

From: Emil Velikov Signed-off-by: Emil Velikov --- src/egl/main/eglcurrent.c | 14 +- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/src/egl/main/eglcurrent.c b/src/egl/main/eglcurrent.c index 50e18e3de89..b197833e88e 100644 --- a/src/egl/main/eglcurrent.c +++ b/src/e

[Mesa-dev] [PATCH 1/4] egl: simplify _eglDebugReport* API

From: Emil Velikov Instead of having three, almost identical but not quite, _eglDebugReport* functions, simply fold them into one. While doing so drop the unnecessary arguments 'command' and 'objectLabel'. Former is identical to funcName, while the latter is already stored (yet unused) in _EGLTh

Re: [Mesa-dev] [PATCH mesa 3/6] r300: use ARRAY_SIZE macro

Reviewed-by: Marek Olšák Marek On Thu, Sep 7, 2017 at 12:21 PM, Eric Engestrom wrote: > Signed-off-by: Eric Engestrom > --- > src/gallium/drivers/r300/compiler/r300_fragprog_swizzle.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/src/gallium/drivers/r300/compiler

Re: [Mesa-dev] [PATCH] mesa/st: Fix frontbuffer rendering regression

Hi Thomas, On 7 September 2017 at 09:53, Thomas Hellstrom wrote: > This fixes a regression introduced with commit > eceb6710024716433069d705fbd873d6d136c2cc: > "mesa/st: Reduce the number of frontbuffer flush calls" > where we, after flushing the front buffer marked it as not-rendered-to, > the i

[Mesa-dev] [PATCH] gallivm: fix gather implementation a bit

From: Roland Scheidegger gather is defined in terms of bilinear filtering, just without the filtering part. However, there's actually some subtle differences required in our implementation, because we use some tricks to simplify coord wrapping for the two coords per direction. For bilinear filter

Re: [Mesa-dev] [PATCH] mesa/st: Fix frontbuffer rendering regression

Hey Thomas, Thomas Hellstrom wrote on 07.09.2017 10:53: > This fixes a regression introduced with commit > eceb6710024716433069d705fbd873d6d136c2cc: > "mesa/st: Reduce the number of frontbuffer flush calls" > where we, after flushing the front buffer marked it as not-rendered-to, > the idea being t

Re: [Mesa-dev] [PATCH v2 1/5] anv: implementation of VK_EXT_debug_report extension

On Mon, Aug 28, 2017 at 10:44 PM, Tapani Pälli wrote: > Patch adds required functionality for extension to manage a list of > application provided callbacks and handle debug reporting from driver > and application side. > > v2: remove useless helper anv_debug_report_call > add locking around

Re: [Mesa-dev] [PATCH v2 1/5] anv: implementation of VK_EXT_debug_report extension

On Thu, Sep 7, 2017 at 9:46 AM, Jason Ekstrand wrote: > On Mon, Aug 28, 2017 at 10:44 PM, Tapani Pälli > wrote: > >> Patch adds required functionality for extension to manage a list of >> application provided callbacks and handle debug reporting from driver >> and application side. >> >> v2: rem

Re: [Mesa-dev] [PATCH v2 2/5] anv: wire up anv_perf_warn macro to do debug reporting

On Mon, Aug 28, 2017 at 10:44 PM, Tapani Pälli wrote: > Signed-off-by: Tapani Pälli > --- > src/intel/vulkan/anv_device.c | 3 +- > src/intel/vulkan/anv_image.c | 12 ++-- > src/intel/vulkan/anv_private.h | 112 ++ > +-- > src/intel/vulkan/anv_u

Re: [Mesa-dev] [PATCH v2 3/5] anv: wire up vk_errorf macro to do debug reporting

On Mon, Aug 28, 2017 at 10:44 PM, Tapani Pälli wrote: > Signed-off-by: Tapani Pälli > --- > src/intel/vulkan/anv_allocator.c | 8 +--- > src/intel/vulkan/anv_device.c| 38 -- > > src/intel/vulkan/anv_formats.c | 3 ++- > src/intel/vulkan/anv_priv

Re: [Mesa-dev] [PATCH v2 0/5] VK_EXT_debug_report

Looks pretty good over-all. Most of the comments are on patch 1. I fully expect v3 to get merged. On Mon, Aug 28, 2017 at 10:44 PM, Tapani Pälli wrote: > Hi; > > Here's v2 of VK_EXT_debug_report with fixes to issues found by Jason > and for some issues that came up with crucible testing, I've

Re: [Mesa-dev] [PATCH 4/4] egl: use switch statements over if/else chain

On Thursday, 2017-09-07 17:03:53 +0100, Emil Velikov wrote: > From: Emil Velikov > > Shorter, explicit and consistent with the rest of the co debase. s/co debase/code base/ in the commit title of patch 2/4: s/since line/single line/ I don't have time to properly review patch 1/4 today, I'll tr

Re: [Mesa-dev] [PATCH] nir/spirv: handle if's with same label in both branches

On Thu, Aug 24, 2017 at 8:16 AM, Juan A. Suarez Romero wrote: > When a conditional branch has the same labels in the "if" part and in the > "else" part, then we have the same cfg block, and it must be handled > once. > > Fixes: > dEQP-VK.spirv_assembly.instruction.compute.conditional_branch.same_

Re: [Mesa-dev] [PATCH] nir/spirv: fix crashes when dereferencing a pointer for an OpVariable

On Tue, Aug 29, 2017 at 3:04 AM, Samuel Iglesias Gonsálvez < sigles...@igalia.com> wrote: > When creating a vtn_pointer for an OpVariable, the block_index and > offsets fields are null because there is not ssa to take the data > from. > > However, we can dereference that pointer when processing an

Re: [Mesa-dev] [PATCH v2] nir/spirv: fix chain access with different index bit sizes

I'd probably go with u2u32 so we don't get sign-extension. Other than that, Reviewed-by: Jason Ekstrand On Mon, Aug 28, 2017 at 11:42 PM, Samuel Iglesias Gonsálvez < sigles...@igalia.com> wrote: > Currently we support 32-bit indexes/offsets all over the driver, so we > convert them to that bit

Re: [Mesa-dev] [PATCH v4] glsl: Add 16-bit types

Reviewed-by: Jason Ekstrand On Sat, Sep 2, 2017 at 12:10 AM, Alejandro Piñeiro wrote: > From: Eduardo Lima Mitev > > Adds new INT16, UINT16 and FLOAT16 base types. > > The corresponding GL types for half floats were reused from the > AMD_gpu_shader_half_float extension. The int16 and uint16 ty

[Mesa-dev] [PATCH] ttn: Fix out-of-bounds accesses since the always-2D-constants change.

Only one of the three checks for dim was updated, so we would try to set a UBO buffer index source value on a nir_load_uniform, and wouldn't actually declare non-UBO uniforms. Fixes: 37dd8e8dee1d ("gallium: all drivers should accept two-dimensional constant buffer indexing") --- src/gallium/auxi

Re: [Mesa-dev] [PATCH 41/47] i965/fs: Add reuse_16bit_conversions_register optimization

On Wed, Sep 6, 2017 at 3:58 PM, Chema Casanova wrote: > Hi Connor and Curro, > > On 28/08/17 12:24, Alejandro Piñeiro wrote: > > On 27/08/17 20:24, Connor Abbott wrote: > >> Hi, > >> > >> On Aug 25, 2017 9:28 AM, "Alejandro Piñeiro" >> > wrote: > >> > >> On 24/08

Re: [Mesa-dev] [PATCH 1/4] egl: simplify _eglDebugReport* API

On 09/07/2017 10:03 AM, Emil Velikov wrote: From: Emil Velikov Instead of having three, almost identical but not quite, _eglDebugReport* functions, simply fold them into one. While doing so drop the unnecessary arguments 'command' and 'objectLabel'. Former is identical to funcName, while the l

Re: [Mesa-dev] [PATCH 1/4] egl: simplify _eglDebugReport* API

On 7 September 2017 at 18:36, Kyle Brenneman wrote: > On 09/07/2017 10:03 AM, Emil Velikov wrote: >> >> From: Emil Velikov >> >> Instead of having three, almost identical but not quite, >> _eglDebugReport* functions, simply fold them into one. >> >> While doing so drop the unnecessary arguments '

[Mesa-dev] [PATCH 3/9] wayland-drm: avoid deprecated use of struct wl_resource

From: Micah Fedke Wayland v1.2 with commit 1488c96a5db ("Add accessor functions for wl_resource and deprecate wl_client_add_resource") paves the way towards making wl_resource opaque. Namely, new helpers were introduced and the struct was annotated as deprecated. Since wayland headers are norma

[Mesa-dev] [PATCH 4/9] configure.ac: define WL_HIDE_DEPRECATED at global scale

From: Emil Velikov Due to GCC feature described in previous commit, the expected deprecation warnings may be missing. Set the WL_HIDE_DEPRECATED macro which will omit the deprecated functionality, resulting in more distinct build issues. Patch is UNTESTED, see the open question below. Cc: Pekk

[Mesa-dev] [PATCH 2/9] wayland-drm: remove unused wayland_drm_buffer_get_{format, buffer}

From: Emil Velikov Unused anywhere throughout the codebase. We could start using it, although that contradicts to an evil plan* of mine. * Only wayland servers will make use of the static library, providing actual distinction between server vs client. Signed-off-by: Emil Velikov --- src/egl/w

[Mesa-dev] [PATCH 1/9] wayland-drm: remove hardcoded enum wl_drm_format

From: Emil Velikov The exact same copy is generated in the client/server protocol header. Signed-off-by: Emil Velikov --- src/egl/wayland/wayland-drm/wayland-drm.h | 64 --- 1 file changed, 64 deletions(-) diff --git a/src/egl/wayland/wayland-drm/wayland-drm.h b/s

[Mesa-dev] [PATCH 6/9] wayland-drm: constify the callbacks struct

From: Emil Velikov Signed-off-by: Emil Velikov --- src/egl/drivers/dri2/egl_dri2.c | 14 +- src/egl/wayland/wayland-drm/wayland-drm.c | 4 ++-- 2 files changed, 7 insertions(+), 11 deletions(-) diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.

[Mesa-dev] [PATCH 5/9] wayland-drm: add wl_display/wl_resource forward declarations

From: Emil Velikov ... making the header self-contained. Signed-off-by: Emil Velikov --- src/egl/wayland/wayland-drm/wayland-drm.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/egl/wayland/wayland-drm/wayland-drm.h b/src/egl/wayland/wayland-drm/wayland-drm.h index 38f711e067b..77e

[Mesa-dev] [PATCH 9/9] automake: adjust wayland-drm comment

From: Emil Velikov Vulkan does not depend on the library or any of the objects created in the process. Signed-off-by: Emil Velikov --- src/Makefile.am | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Makefile.am b/src/Makefile.am index e055cb8e701..e2d53155d52 100644 ---

[Mesa-dev] [PATCH 7/9] wayland-drm: static inline wayland_drm_buffer_get

From: Emil Velikov The function is effectively a direct function call into libwayland-server.so Signed-off-by: Emil Velikov --- src/egl/wayland/wayland-drm/wayland-drm.c | 26 -- src/egl/wayland/wayland-drm/wayland-drm.h | 28 +--- src/gbm/Makefi

[Mesa-dev] [PATCH 8/9] configure.ac: split the wayland client/server confusion

From: Emil Velikov At the moment wayland-clients, such as the Vulkan drivers were over-linking against libwayland-server.so. That went unnoticed, since both client and server code uses the wl*interface symbols, which are present in both libwayland-client.so and libwayland-server.so. I've looked

Re: [Mesa-dev] [PATCH 1/4] egl: simplify _eglDebugReport* API

On 09/07/2017 11:56 AM, Emil Velikov wrote: On 7 September 2017 at 18:36, Kyle Brenneman wrote: On 09/07/2017 10:03 AM, Emil Velikov wrote: From: Emil Velikov Instead of having three, almost identical but not quite, _eglDebugReport* functions, simply fold them into one. While doing so drop

[Mesa-dev] [PATCH 2/2] radeonsi: optimize TCS epilog when invocation 0 writes tess factors

From: Marek Olšák This removes the barrier and LDS stores and loads for tess factors when it's possible. The removal of the barrier seems more important to me though. In one shader, it removes 17 * 4 bytes from the shader binary. --- src/gallium/drivers/radeonsi/si_shader.c | 111 +

[Mesa-dev] [PATCH 1/2] tgsi/scan: add a new pass that analyzes tess factor writes (v2)

From: Marek Olšák The pass tries to deduce whether tess factors are always written by all shader invocations. The implication for radeonsi is that it doesn't have to use a barrier near the end of TCS, and doesn't have to use LDS for passing the tess factors to the epilog. v2: Handle barriers an

[Mesa-dev] [PATCH 1/2] radv/winsys: use amdgpu_bo_va_op_raw.

From: Dave Airlie This is a precursor to the gfx9 fix to use uncached for the event memory. Move to the interface which allows setting the flags, but wrap it to avoid having to copy it around the place. Signed-off-by: Dave Airlie --- src/amd/vulkan/winsys/amdgpu/radv_amdgpu_bo.c | 31 +

[Mesa-dev] [PATCH 2/2] radv/gfx9: allocate events from uncached VA space

From: Dave Airlie This copies what amdgpu-pro does, and allocates the memory for an event with an uncached mtype. This fixes hangs with: dEQP-VK.api.command_buffers.record_simul_use_primary Signed-off-by: Dave Airlie --- src/amd/vulkan/radv_device.c | 2 +- src/amd/vulkan/rad

[Mesa-dev] [PATCH] radv: Actually check for vm faults.

The code can check for vm faults having happened. If we only do it on a hang we don't know when the faults happened. This changes the behavior to when the first VM faults is found, even without a hang. --- src/amd/vulkan/radv_debug.c | 10 +++--- src/amd/vulkan/radv_debug.h | 1 + src/amd/v

[Mesa-dev] [PATCH] Android: Add LLVM support for Android P

The Android version in AOSP master has changed now to P, so we need to add LLVM flags for it. Duplicating the lines because I expect the version will get bumped at some point and diverge from O. Cc: Chih-Wei Huang Signed-off-by: Rob Herring --- Android.mk | 2 ++ 1 file changed, 2 insertions(+)

Re: [Mesa-dev] [PATCH] anv/allocator: Loosen the size restriction on imported BOs

Seems reasonable Reviewed-by: Jason Ekstrand On Wed, Aug 2, 2017 at 3:37 PM, Kevin Strasser wrote: > It's possible for a buffer manager to issue the application a BO that is > larger than what is requested. The out-of-bounds access issue would only > apply in cases where the size of the import

[Mesa-dev] [PATCH v2 1/2] gallium: Add PIPE_SHADER_CAP_FP16

Denotes native half precision float operations capability v2: PIPE_CAP_HALFS -> PIPE_SHADER_CAP_FP16 fix indentation Signed-off-by: Jan Vesely --- I modeled this after PIPE_SHARED_CAP_INTEGERS, but I might have missed a spot. src/gallium/auxiliary/gallivm/lp_bld_limits.h| 2 ++ src/gall

  1   2   >