[Mesa-dev] [PATCH 6/6] i965: remove brw_lower_texture_gradients

This has been ported to NIR now so we don'tneed to keep the GLSL IR lowering any more. --- src/mesa/drivers/dri/i965/Makefile.sources | 1 - src/mesa/drivers/dri/i965/brw_context.h| 2 - src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 2 +-

[Mesa-dev] [PATCH 5/6] i965/nir: enable lowering of texture gradient for shadow samplers

This gets the lowering on the Vulkan driver too, which is required for hardware that does not have the sample_l_d message (up to IvyBridge). --- src/mesa/drivers/dri/i965/brw_nir.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_nir.c

[Mesa-dev] [PATCH 4/6] nir/lower_tex: add lowering for texture gradient on shadow samplers

This is ported from the Intel lowering pass that we use with GLSL IR. This takes care of lowering texture gradients on shadow samplers other than cube maps. Intel hardware requires this for gen < 8. --- src/compiler/nir/nir.h | 7 +++ src/compiler/nir/nir_lower_tex.c | 40

[Mesa-dev] [PATCH 0/6] i965: port texture gradient lowering to NIR

We need this lowering in Vulkan too do texture gradients with cube map textures (and also with shadow samplers in IvyBridge). Currently, we have the brw_lower_texture_gradients() pass that works on GLSL IR and is called from the OpenGL driver. This pass is actually two passes in one: 1. A

[Mesa-dev] [PATCH 3/6] i965/nir: enable lowering of texture gradient for cube maps

This gets the lowering on the Vulkan driver too. Fixes Vulkan CTS cube map texture gradient tests in: dEQP-VK.glsl.texture_functions.texturegrad.* --- src/mesa/drivers/dri/i965/brw_nir.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/mesa/drivers/dri/i965/brw_nir.c

[Mesa-dev] [PATCH 1/6] nir/lower_tex: generalize get_texture_size()

This was written specifically for RECT samplers. Make it more generic so we can call this from the gradient lowerings too. --- src/compiler/nir/nir_lower_tex.c | 15 ++- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/src/compiler/nir/nir_lower_tex.c

Re: [Mesa-dev] [PATCH] EGL/android: Enhance pbuffer implementation

Hi, On Fri, Nov 25, 2016 at 12:58 PM, Liu Zhiquan wrote: > Some dri drivers will pass multiple bits in buffer_mask parameter > to droid_image_get_buffer(), more than the actual supported buffer > type combination. For such case, will go through all the bits, and > will not

[Mesa-dev] [PATCH v2 1/7] intel/blorp_blit: Create structure for src & dst coordinates

This will be useful for splitting blits into smaller sizes. We also make the coordinates of type double rather than float. Since we will be splitting and scaling the coordinates, we might require extra precision in the calculations. v2: * Use double instead of float. (Jason) Signed-off-by:

[Mesa-dev] [PATCH v2 7/7] i965: Increase max texture to 16k for gen7+

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98297 Signed-off-by: Jordan Justen --- src/mesa/drivers/dri/i965/brw_context.c | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_context.c

[Mesa-dev] [PATCH v2 2/7] intel/blorp_blit: Split blorp blits if they are too large

We rename do_blorp_blit() to try_blorp_blit(), and add a return error if the surface size for the blit is too large. Now, do_blorp_blit() is rewritten to try to split the blit into smaller operations if try_blorp_blit() fails. Note: In this commit, try_blorp_blit() will always attempt to blit and

[Mesa-dev] [PATCH v2 3/7] intel/blorp_blit: Adjust blorp surface parameters for split blits

If try_blorp_blit() previously returned that a blit was too large, shrink_surface_params() will be used to update the surface parameters for the smaller blit so the blit operation can proceed. v2: * Use double instead of float. (Jason) Signed-off-by: Jordan Justen

[Mesa-dev] [PATCH v2 6/7] intel/blorp_blit: Add split_blorp_blit_debug switch

Enabling this debug switch causes surface shrinking to happen by default, and lowers the surface size limit which causes blorp blits to be split. Signed-off-by: Jordan Justen --- src/intel/blorp/blorp_blit.c | 11 --- 1 file changed, 8 insertions(+), 3

[Mesa-dev] [PATCH v2 5/7] intel/blorp_blit: Enable splitting large blorp blits

Detect when the surface sizes are too large for a blorp blit. When it is too large, the blorp blit will be split into a smaller operation and attempted again. For gen7, this fixes the cts test:

[Mesa-dev] [PATCH v2 4/7] intel/blorp_blit: Move RGB=>R conversion to follow blit splitting

In blorp_copy, when RGB surfaces are copied, we convert the destination surface to a Red only surface, but 3 times as wide. This introduces an implicit restriction of "mod 3" for the destination width. It is easier to handle the blorp split buffer offsetting with the original RGB surface, and do

[Mesa-dev] [PATCH v2 0/7] Split blorp blits; Enable 16k render target on gen7+

git://people.freedesktop.org/~jljusten/mesa split-blorp-blit-v2 (These patches have been applied on top of my ISL surface dumping patches which are included in the branch above.) v1: https://lists.freedesktop.org/archives/mesa-dev/2016-November/136235.html v2: * Simplify the logic to shrink

Re: [Mesa-dev] [PATCH 2/2] i965: Add i965 plumbing for ARB_post_depth_coverage for i965 (gen9+).

A couple of notes on existing weirdness here: - Naming of GEN9_PSX_SHADER_NORMAL_COVERAGE_MASK_SHIFT is bizarre (not your fault) - Is BRW_PSICMS_INNER really the right thing for the normal mode? Why not BRW_PSICMS_NORMAL? Perhaps whoever added this stuff can shed some light here? Actual change

[Mesa-dev] [PATCH v2 4/5] intel/blorp_blit: Add option to dump surfaces on blits

Enabling this option causes the source and destination surfaces to be dumped out to debug files. The destination is dumped both before and after the blit operation. Signed-off-by: Jordan Justen --- src/intel/blorp/blorp_blit.c | 18 ++ 1 file changed,

[Mesa-dev] [PATCH v2 5/5] i965/hiz: Add debug dump for depth/hiz buffers around hiz ops

v2: * Use new blorp_surf_dump function. (Jason) Signed-off-by: Jordan Justen Acked-by: Matt Turner Cc: Jason Ekstrand --- src/mesa/drivers/dri/i965/brw_blorp.c | 54 +++ 1 file changed, 36

[Mesa-dev] [PATCH v2 2/5] intel/isl: Add support for saving surface debug info to files

Signed-off-by: Jordan Justen --- src/intel/Makefile.sources | 1 + src/intel/isl/isl.h| 14 +++ src/intel/isl/isl_dump.c | 217 + 3 files changed, 232 insertions(+) create mode 100644 src/intel/isl/isl_dump.c

[Mesa-dev] [PATCH v2 3/5] intel/blorp: Add blorp_surf_dump function

This function takes care of mapping and unmapping the buffers, and then calls isl_surf_dump. Signed-off-by: Jordan Justen --- src/intel/blorp/blorp.c | 35 +++ src/intel/blorp/blorp.h | 6 ++ 2 files changed, 41 insertions(+) diff

[Mesa-dev] [PATCH v2 1/5] intel/blorp: Support map/unmap of blorp_address

Signed-off-by: Jordan Justen --- src/intel/blorp/blorp.h | 8 +++ src/mesa/drivers/dri/i965/brw_blorp.c | 39 +++ 2 files changed, 47 insertions(+) diff --git a/src/intel/blorp/blorp.h b/src/intel/blorp/blorp.h index

Re: [Mesa-dev] [PATCH 1/2] mesa: Add GL and GLSL plumbing for ARB_post_depth_coverage for i965 (gen9+).

On Tue, 2016-11-22 at 23:53 +0200, Plamena Manolova wrote: > This extension allows the fragment shader to control whether values > in > gl_SampleMaskIn[] reflect the coverage after application of the early > depth and stencil tests. > > Signed-off-by: Plamena Manolova

[Mesa-dev] [Bug 97524] Samplers referring to the same texture unit with different types should raise GL_INVALID_OPERATION

https://bugs.freedesktop.org/show_bug.cgi?id=97524 --- Comment #16 from Marek Olšák --- Workaround for radeonsi: CI-VI: If the resource type and instruction mismatch (e.g. a buffer constant with an image instruction, or an image resource with a buffer instruction), the

Re: [Mesa-dev] [PATCH 2/2] i965: Add i965 plumbing for ARB_post_depth_coverage for i965 (gen9+).

On Wed, Nov 30, 2016 at 3:00 PM, Plamena Manolova wrote: > This extension allows the fragment shader to control whether values in > gl_SampleMaskIn[] reflect the coverage after application of the early > depth and stencil tests. > > Signed-off-by: Plamena Manolova

[Mesa-dev] [PATCH] i965: keep gl_program shader info in sync after gather info

It's possible that nir_shader was cloned and it no longer contains a pointer to the shader_info in gl_program. So we need to copy shader_info back to gl_program if that is the case. Fixes a regression with NIR_TEST_CLONE=true Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98840 ---

Re: [Mesa-dev] [PATCH] mesa: only verify that enabled arrays have backing buffers

On Wed, Nov 30, 2016 at 5:04 PM, Ilia Mirkin wrote: > We were previously also verifying that no backing buffers were available > when an array wasn't enabled. This is has no basis in the spec, and it > causes GLupeN64 to fail as a result. > > Fixes: c2e146f487 ("mesa: error

Re: [Mesa-dev] [PATCH 1/2] mesa: Add GL and GLSL plumbing for ARB_post_depth_coverage for i965 (gen9+).

On Wed, Nov 30, 2016 at 2:53 PM, Plamena Manolova wrote: > This extension allows the fragment shader to control whether values in > gl_SampleMaskIn[] reflect the coverage after application of the early > depth and stencil tests. > > Signed-off-by: Plamena Manolova

Re: [Mesa-dev] [PATCH 1/2] mesa: Add GL and GLSL plumbing for ARB_post_depth_coverage for i965 (gen9+).

Excellent, disregard that. Patch looks good. On Thu, Dec 1, 2016 at 3:10 PM, Ilia Mirkin wrote: > On Wed, Nov 30, 2016 at 9:10 PM, Chris Forbes wrote: > > This patch misses adding the #define to the GLSL preprocessor. Other than > > The future is today.

Re: [Mesa-dev] [PATCH 1/2] mesa: Add GL and GLSL plumbing for ARB_post_depth_coverage for i965 (gen9+).

On Wed, Nov 30, 2016 at 9:10 PM, Chris Forbes wrote: > This patch misses adding the #define to the GLSL preprocessor. Other than The future is today. That's no longer necessary :) > that it looks good though, so with that fixed: > > Reviewed-by: Chris Forbes

Re: [Mesa-dev] [PATCH 1/2] mesa: Add GL and GLSL plumbing for ARB_post_depth_coverage for i965 (gen9+).

This patch misses adding the #define to the GLSL preprocessor. Other than that it looks good though, so with that fixed: Reviewed-by: Chris Forbes On Thu, Dec 1, 2016 at 8:53 AM, Plamena Manolova wrote: > This extension allows the fragment

[Mesa-dev] [AppVeyor] mesa master #2856 completed

Build mesa 2856 completed Commit fda1d0187d by Ilia Mirkin on 11/30/2016 5:56 AM: anv: expose support for VK_KHR_sampler_mirror_clamp_to_edge\n\nThis is already supported in genX_state.c, expose the extension string.\n\nSigned-off-by: Ilia Mirkin

Re: [Mesa-dev] [PATCH] anv/cmd_buffer: Actually use the stencil dimension

On Wed, Nov 30, 2016 at 10:20 AM, Nanley Chery wrote: > On Tue, Nov 29, 2016 at 05:41:58PM -0800, Jason Ekstrand wrote: > > In an attempt to fix 3DSTATE_DEPTH_BUFFER for stencil-only cases, I > > accidentally kept setting the SurfaceType to 2D in the stencil-only case > >

[Mesa-dev] [AppVeyor] mesa master #2855 failed

Build mesa 2855 failed Commit 27433b26b1 by Jason Ekstrand on 11/29/2016 10:00 PM: anv/cmd_buffer: Actually use the stencil dimension\n\nIn an attempt to fix 3DSTATE_DEPTH_BUFFER for stencil-only cases, I\naccidentally kept setting the SurfaceType to 2D in the

[Mesa-dev] [Bug 98911] [Ironlake Mobile] Severe Artifacting in Minecraft and weird "screen leftovers" when recording desktop with latest Mesa

https://bugs.freedesktop.org/show_bug.cgi?id=98911 Kenneth Graunke changed: What|Removed |Added Component|Mesa core

Re: [Mesa-dev] Mesa 13.1.0 release plan

Weren't we going to call it 17.0?... On Nov 30, 2016 12:23 PM, "Emil Velikov" wrote: > Hi all, > > With holidays not far off, it might be a nice idea to consider the > branchpoint/release schedule for the next release. > > I will be having limited internet access

[Mesa-dev] [AppVeyor] mesa master #2852 failed

Build mesa 2852 failed Commit 966567aa12 by Timothy Arceri on 11/30/2016 9:52 PM: mesa: reset linked_stages bitmask when re-linking\n\n34953f8907fdd added this bitmask but it wasn't being reset when\na program was relinked. If a stage was removed from the

[Mesa-dev] [PATCH] ac/nir: Only use the first component for SSBO atomics.

Signed-off-by: Bas Nieuwenhuizen --- src/amd/common/ac_nir_to_llvm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c index df121d8..87bb19a 100644 --- a/src/amd/common/ac_nir_to_llvm.c +++

Re: [Mesa-dev] [PATCH 5/5] swr: add streamout buffer offset into pBuffer pointer

Reviewed-by: Tim Rowley > On Nov 29, 2016, at 8:23 PM, Ilia Mirkin > wrote: The buffer_size does not take the offset into account. Just add the offset into the pointer which lines up

Re: [Mesa-dev] [PATCH 2/5] swr: turn off queries around blits

Reviewed-by: Tim Rowley > On Nov 29, 2016, at 8:23 PM, Ilia Mirkin > wrote: Signed-off-by: Ilia Mirkin > ---

Re: [Mesa-dev] [PATCH 4/5] swr: fix assertion for max number of so targets

Reviewed-by: Tim Rowley > On Nov 29, 2016, at 8:23 PM, Ilia Mirkin > wrote: The number has to be less than or equal to the max, not just less than. Signed-off-by: Ilia Mirkin

Re: [Mesa-dev] [PATCH 3/5] swr: properly report max number of SO components

Reviewed-by: Tim Rowley > On Nov 29, 2016, at 8:23 PM, Ilia Mirkin > wrote: The components count the number of individual values, not the number of slots. Signed-off-by: Ilia Mirkin

[Mesa-dev] [PATCH] mesa: only verify that enabled arrays have backing buffers

We were previously also verifying that no backing buffers were available when an array wasn't enabled. This is has no basis in the spec, and it causes GLupeN64 to fail as a result. Fixes: c2e146f487 ("mesa: error out in indirect draw when vertex bindings mismatch") Cc:

[Mesa-dev] [PATCH] mesa: reset linked_stages bitmask when re-linking

34953f8907fdd added this bitmask but it wasn't being reset when a program was relinked. If a stage was removed from the new program then it could case a crash as we expect the linked shader for that stage to not be null. Fixes crashes in:

Re: [Mesa-dev] [PATCH 00/50] GL_ARB_gpu_shader_int64... this time for sure!

On 11/28, Ian Romanick wrote: From: Ian Romanick I believe that I have addressed all of the review feedback from the previous iteration. Many of the patches have been reviewed, and they should be ready to go. Patches marked with "vN" in the subject have changed in a

Re: [Mesa-dev] [PATCH 1/5] swr: don't advertise stream pause/resume

Reviewed-by: Tim Rowley > On Nov 29, 2016, at 8:23 PM, Ilia Mirkin > wrote: There is no support for resuming streamout. Furthermore, this also controls glDrawTransformFeedback

Re: [Mesa-dev] [PATCH] swr: remove warning about multi-layer surfaces

Reviewed-by: Tim Rowley > On Nov 29, 2016, at 8:05 PM, Ilia Mirkin > wrote: We now support clearing these, and actually rendering to multiple layers would require GS support, which

Re: [Mesa-dev] [PATCH] swr: [rasterizer core] don't attempt to load another RTAI when storing

Reviewed-by: Tim Rowley > On Nov 16, 2016, at 9:04 PM, Ilia Mirkin > wrote: Since we don't pass a renderTargetArrayIndex in, and the current hot tile may be for a different index, we

Re: [Mesa-dev] [PATCH 02/23] tgsi: add Stream{X, Y, Z, W} fields to tgsi_declaration_semantic

Am 30.11.2016 um 20:19 schrieb Nicolai Hähnle: > On 30.11.2016 19:06, Roland Scheidegger wrote: >> Am 30.11.2016 um 14:35 schrieb Nicolai Hähnle: >>> From: Nicolai Hähnle >>> >>> This is for geometry shader outputs. Without it, drivers have no way of >>> knowing which

Re: [Mesa-dev] [PATCH] swr: [rasterizer memory] assert when trying to convert an unknown format

Not seeing this assert fire on our tests either. Reviewed-by: Tim Rowley > On Nov 29, 2016, at 8:04 PM, Ilia Mirkin > wrote: Signed-off-by: Ilia Mirkin

Re: [Mesa-dev] [PATCH 27/50] glsl: Add "built-in" function for 64-bit integer sign()

On 11/28, Ian Romanick wrote: From: Ian Romanick These functions are directly available in shaders. A #define is added to detect the presence. This allows these functions to be tested using piglit regardless of whether the driver uses them for lowering. The GLSL

Re: [Mesa-dev] [PATCH 26/50] glsl: Add a lowering pass for 64-bit integer multiplication

On 11/28, Ian Romanick wrote: From: Ian Romanick Signed-off-by: Ian Romanick --- src/compiler/Makefile.glsl.am| 1 + src/compiler/Makefile.sources| 1 + src/compiler/glsl/ir_optimization.h | 6 +

[Mesa-dev] Mesa 13.1.0 release plan

Hi all, With holidays not far off, it might be a nice idea to consider the branchpoint/release schedule for the next release. I will be having limited internet access during 20 Dec - 7 Jan, thus the I'm leaning towards following: Jan 13 2017 - Feature freeze/Release candidate 1 Jan 20 2017 -

Re: [Mesa-dev] [PATCH 25/50] glsl: Add "built-in" functions to do 64x64 => 64 multiplication

On 11/28, Ian Romanick wrote: From: Ian Romanick These functions are directly available in shaders. A #define is added to detect the presence. This allows these functions to be tested using piglit regardless of whether the driver uses them for lowering. The GLSL

[Mesa-dev] [PATCH 2/2] i965: Add i965 plumbing for ARB_post_depth_coverage for i965 (gen9+).

This extension allows the fragment shader to control whether values in gl_SampleMaskIn[] reflect the coverage after application of the early depth and stencil tests. Signed-off-by: Plamena Manolova --- docs/relnotes/13.1.0.html| 1 +

[Mesa-dev] [PATCH 1/2] mesa: Add GL and GLSL plumbing for ARB_post_depth_coverage for i965 (gen9+).

This extension allows the fragment shader to control whether values in gl_SampleMaskIn[] reflect the coverage after application of the early depth and stencil tests. Signed-off-by: Plamena Manolova --- src/compiler/glsl/ast.h | 5 +

Re: [Mesa-dev] [PATCH 23/50] glsl/standalone: Enable ARB_gpu_shader_int64

On 11/28, Ian Romanick wrote: From: Ian Romanick Signed-off-by: Ian Romanick --- src/compiler/glsl/ir_builder_print_visitor.cpp | 16 +--- src/compiler/glsl/standalone_scaffolding.cpp | 1 + 2 files changed, 14 insertions(+), 3

[Mesa-dev] [Bug 98428] Undefined non-weak-symbol in dri-drivers

https://bugs.freedesktop.org/show_bug.cgi?id=98428 Jonathan Dieter changed: What|Removed |Added CC||jdie...@lesbg.com

Re: [Mesa-dev] [PATCH 02/23] tgsi: add Stream{X, Y, Z, W} fields to tgsi_declaration_semantic

On 30.11.2016 19:06, Roland Scheidegger wrote: Am 30.11.2016 um 14:35 schrieb Nicolai Hähnle: From: Nicolai Hähnle This is for geometry shader outputs. Without it, drivers have no way of knowing which stream each output is intended for, and have to conservatively

Re: [Mesa-dev] [PATCH 02/23] tgsi: add Stream{X, Y, Z, W} fields to tgsi_declaration_semantic

Am 30.11.2016 um 14:35 schrieb Nicolai Hähnle: > From: Nicolai Hähnle > > This is for geometry shader outputs. Without it, drivers have no way of > knowing which stream each output is intended for, and have to > conservatively write all outputs to all streams. > >

Re: [Mesa-dev] [PATCH] anv/cmd_buffer: Actually use the stencil dimension

On Tue, Nov 29, 2016 at 05:41:58PM -0800, Jason Ekstrand wrote: > In an attempt to fix 3DSTATE_DEPTH_BUFFER for stencil-only cases, I > accidentally kept setting the SurfaceType to 2D in the stencil-only case > thanks to a copy+paste error. > > Cc: Nanley Chery > --- >

Re: [Mesa-dev] [PATCH] anv: Prefer in-tree headers to out-of-tree headers

On Tue, Nov 29, 2016 at 08:54:26AM -0800, Jason Ekstrand wrote: > On Tue, Nov 29, 2016 at 8:47 AM, Ville Syrjälä < > ville.syrj...@linux.intel.com> wrote: > > > On Tue, Nov 29, 2016 at 08:28:55AM -0800, Jason Ekstrand wrote: > > > On Tue, Nov 29, 2016 at 1:23 AM,

Re: [Mesa-dev] [PATCH] swr: fix range computation for instanced client-side arrays

Reviewed-by: Bruce Cherniak > On Nov 29, 2016, at 8:18 PM, Ilia Mirkin wrote: > > We need to take the instance divisor and number of instances into > account for instanced client-side arrays, rather than the vertex > parameters. > > Loosely

[Mesa-dev] [Bug 98911] [Ironlake Mobile] Severe Artifacting in Minecraft and weird "screen leftovers" when recording desktop with latest Mesa

https://bugs.freedesktop.org/show_bug.cgi?id=98911 Mark Janes changed: What|Removed |Added CC|

Re: [Mesa-dev] [PATCH] anv: Prefer in-tree headers to out-of-tree headers

On Tue 29 Nov 2016, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > Set the include paths to consider in-tree headers before out-of-tree > headers. > > Avoids the build failing due to stale headers being present in > $prefix. Previosuly 'make -ki

Re: [Mesa-dev] [PATCH 07/23] radeonsi: do not kill GS with memory writes

On 30.11.2016 14:35, Nicolai Hähnle wrote: From: Nicolai Hähnle Vertex emits beyond the specified maximum number of vertices are supposed to have no effect, which is why we used to always kill GS that reached the limit. However, if the GS also writes to memory (SSBO,

Re: [Mesa-dev] [PATCH 1/2] mesa: Add GL and GLSL plumbing for ARB_post_depth_coverage for i965 (gen9+).

On Wed, Nov 30, 2016 at 4:36 PM, Ilia Mirkin wrote: > On Wed, Nov 30, 2016 at 9:19 AM, Lionel Landwerlin > wrote: > > Also forgot that (like Ilia suggested for NV_image_formats) you can > update > > docs/relnotes/ to list the new feature. > >

Re: [Mesa-dev] [PATCH 1/2] mesa: Add GL and GLSL plumbing for ARB_post_depth_coverage for i965 (gen9+).

Thank you for reviewing guys! I'll go ahead and make that change. Pam On Wed, Nov 30, 2016 at 3:33 PM, Ilia Mirkin wrote: > On Nov 30, 2016 7:17 AM, "Lionel Landwerlin" < > lionel.g.landwer...@intel.com> wrote: > > > > On 22/11/16 21:53, Plamena Manolova wrote: > >> > >>

Re: [Mesa-dev] [PATCH 7/8] radeonsi: apply the double EVENT_WRITE_EOP workaround to VI as well

On 30.11.2016 15:01, Nicolai Hähnle wrote: On 30.11.2016 02:36, Marek Olšák wrote: From: Marek Olšák Internal docs don't mention it, but they also don't mention that the bug has been fixed (like other CI bugs fixed in VI). Vulkan does this too. Cc: 13.0

Re: [Mesa-dev] [PATCH 1/2] mesa: Add GL and GLSL plumbing for ARB_post_depth_coverage for i965 (gen9+).

On Wed, Nov 30, 2016 at 9:19 AM, Lionel Landwerlin wrote: > Also forgot that (like Ilia suggested for NV_image_formats) you can update > docs/relnotes/ to list the new feature. That would be appropriate for the patch that actually exposes the feature. This one just

Re: [Mesa-dev] [PATCH 1/2] mesa: Add GL and GLSL plumbing for ARB_post_depth_coverage for i965 (gen9+).

Also forgot that (like Ilia suggested for NV_image_formats) you can update docs/relnotes/ to list the new feature. Cheers, - Lionel On 30/11/16 13:33, Ilia Mirkin wrote: On Nov 30, 2016 7:17 AM, "Lionel Landwerlin" >

Re: [Mesa-dev] [AppVeyor] mesa master #2842 failed

I didn't spot any relevant discussion on http://help.appveyor.com/discussions/problems or https://appveyor.statuspage.io/ It must have been something transient. Jose On 30/11/16 03:52, Brian Paul wrote: We've had a few instances in the past where complex macro code caused super long MSVC

[Mesa-dev] [Bug 98911] [Ironlake Mobile] Severe Artifacting in Minecraft and weird "screen leftovers" when recording desktop with latest Mesa

https://bugs.freedesktop.org/show_bug.cgi?id=98911 --- Comment #1 from paed808 --- Here is a video I recorded of the issue. https://www.youtube.com/watch?v=Iatb2PQPSFI -- You are receiving this mail because: You are the assignee for the bug. You are the QA Contact for

Re: [Mesa-dev] [PATCH 8/8] radeonsi: document a CP DMA bug that doesn't need a workaround yet

Apart from the comment on patch 7, the series is Reviewed-by: Nicolai Hähnle On 30.11.2016 02:36, Marek Olšák wrote: From: Marek Olšák This one is easy to miss, because it's not documented in any internal doc. --- src/amd/common/sid.h | 6

Re: [Mesa-dev] [PATCH 7/8] radeonsi: apply the double EVENT_WRITE_EOP workaround to VI as well

On 30.11.2016 02:36, Marek Olšák wrote: From: Marek Olšák Internal docs don't mention it, but they also don't mention that the bug has been fixed (like other CI bugs fixed in VI). Vulkan does this too. Cc: 13.0 ---

Re: [Mesa-dev] [PATCH 0/3] gallium driver for Vivante GPUs

Hi all, this time I send the patchset to the correct ml. Had no coffee when sending it out for the first time :) greets -- Christian Gmeiner, MSc https://soundcloud.com/christian-gmeiner ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org

[Mesa-dev] [PATCH 3/3] imx: gallium driver for imx-drm scanout driver

The imx (stub) driver is needed to get hardware acceleration from etnaviv on a platform using imx-drm kms driver. This adds support for wayland and native kms egl apps. Signed-off-by: Christian Gmeiner --- configure.ac | 12

[Mesa-dev] [PATCH 1/3] gallium: add renderonly library

This a very lightweight library to add basic support for renderonly GPUs. It does all the magic regarding in/exporting buffers etc. This library will likely break android support and hopefully will get replaced with a better solution based on gbm2. Signed-off-by: Christian Gmeiner

[Mesa-dev] [PATCH 0/3] gallium driver for Vivante GPUs

As the original patchstack is now about 300 patches, I have choosen to squash the patches together into three different parts. - renderonly library A lightweight library to add basic infrastructure for renderonly GPUs. With this library it is possible to run wayland or and other kms egl

[Mesa-dev] [PATCH 22/23] radeonsi: shrink each vertex stream to the actually required size

From: Nicolai Hähnle --- src/gallium/drivers/radeonsi/si_shader.c| 26 +++-- src/gallium/drivers/radeonsi/si_state_shaders.c | 37 +++-- 2 files changed, 39 insertions(+), 24 deletions(-) diff --git

[Mesa-dev] [PATCH 17/23] radeonsi: fetch only outputs of current vertex stream from the GSVS ring

From: Nicolai Hähnle --- src/gallium/drivers/radeonsi/si_shader.c | 41 +++- 1 file changed, 25 insertions(+), 16 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_shader.c b/src/gallium/drivers/radeonsi/si_shader.c index

[Mesa-dev] [PATCH 01/23] glsl: remember per-component vertex streams for packed varyings

From: Nicolai Hähnle --- src/compiler/glsl/ir.h | 3 +++ src/compiler/glsl/ir_print_visitor.cpp | 14 -- src/compiler/glsl/lower_packed_varyings.cpp | 9 + 3 files changed, 24 insertions(+), 2 deletions(-) diff --git

[Mesa-dev] [PATCH 21/23] radeonsi: use a single descriptor for the GSVS ring

From: Nicolai Hähnle We can hardcode all of the fields for swizzling in the geometry shader. The advantage is that we use fewer descriptor slots and we no longer have to update any of the (ring) descriptors when the geometry shader changes. ---

[Mesa-dev] [PATCH 23/23] radeonsi: shrink the GSVS ring to account for the reduced item sizes

From: Nicolai Hähnle --- src/gallium/drivers/radeonsi/si_state_shaders.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/radeonsi/si_state_shaders.c b/src/gallium/drivers/radeonsi/si_state_shaders.c index 151ed17..4a7f638 100644

[Mesa-dev] [PATCH 14/23] radeonsi: pull iteration over vertex streams into GS copy shader logic

From: Nicolai Hähnle The iteration is not needed for normal vertex shaders. --- src/gallium/drivers/radeonsi/si_shader.c | 62 +++- 1 file changed, 37 insertions(+), 25 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_shader.c

[Mesa-dev] [PATCH 05/23] st/glsl_to_tgsi: plumb the GS output stream qualifier through to TGSI

From: Nicolai Hähnle Allow drivers to emit GS outputs in a smarter way. --- src/gallium/auxiliary/tgsi/tgsi_ureg.c | 16 +++- src/gallium/auxiliary/tgsi/tgsi_ureg.h | 1 + src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 10 ++ 3 files changed,

[Mesa-dev] [PATCH 16/23] radeonsi: only export from GS copy shader for vertex stream 0

From: Nicolai Hähnle When running the copy shader for vertex streams != 0, the SX does not need any data from us (there is no rasterization for the higher vertex streams, only streamout). --- src/gallium/drivers/radeonsi/si_shader.c | 31 +++

[Mesa-dev] [PATCH 03/23] gallium: extract individual streamout output structure

From: Nicolai Hähnle So that we can pass pointers to individual array entries around. --- src/gallium/include/pipe/p_state.h | 21 + 1 file changed, 13 insertions(+), 8 deletions(-) diff --git a/src/gallium/include/pipe/p_state.h

[Mesa-dev] [PATCH 20/23] radeonsi: pack GS output components for each vertex stream contiguously

From: Nicolai Hähnle Note that the memory layout of one vertex stream inside one "item" (= memory written by one GS wave) on the GSVS ring is: t0v0c0 ... t15v0c0 t0v1c0 ... t15v1c0 ... t0vLc0 ... t15vLc0 t0v0c1 ... t15v0c1 t0v1c1 ... t15v1c1 ... t0vLc1 ... t15vLc1

[Mesa-dev] [PATCH 15/23] radeonsi: do not export VS outputs from vertex streams != 0

From: Nicolai Hähnle This affects for GS copy shaders. When an output is meant for vertex stream != 0, then we don't have to make it available to the pixel shader. There is a minor inefficiency here because the GLSL varying packing pass does not group varyings of the

[Mesa-dev] [PATCH 13/23] radeonsi: group streamout writes by vertex stream

From: Nicolai Hähnle --- src/gallium/drivers/radeonsi/si_shader.c | 32 ++-- 1 file changed, 22 insertions(+), 10 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_shader.c b/src/gallium/drivers/radeonsi/si_shader.c index

[Mesa-dev] [PATCH 11/23] radeonsi: extract writing of a single streamout output

From: Nicolai Hähnle --- src/gallium/drivers/radeonsi/si_shader.c | 91 ++-- 1 file changed, 52 insertions(+), 39 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_shader.c b/src/gallium/drivers/radeonsi/si_shader.c index

[Mesa-dev] [PATCH 09/23] radeonsi: plumb the output vertex_stream through to si_shader_output_values

From: Nicolai Hähnle --- src/gallium/drivers/radeonsi/si_shader.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/radeonsi/si_shader.c b/src/gallium/drivers/radeonsi/si_shader.c index c488f91..b420a7b 100644 ---

[Mesa-dev] [PATCH 07/23] radeonsi: do not kill GS with memory writes

From: Nicolai Hähnle Vertex emits beyond the specified maximum number of vertices are supposed to have no effect, which is why we used to always kill GS that reached the limit. However, if the GS also writes to memory (SSBO, atomics, shader images), then we must keep

[Mesa-dev] [PATCH 18/23] radeonsi: generate an explicit switch instruction over vertex streams

From: Nicolai Hähnle SimplifyCFG generates a switch instruction anyway when all four streams are present, but is simultaneously not smart enough to eliminate some redundant jumps that it generates. The generated assembly is still a bit silly, probably because the

[Mesa-dev] [PATCH 19/23] radeonsi: only write values belonging to the stream when emitting GS vertex

From: Nicolai Hähnle --- src/gallium/drivers/radeonsi/si_shader.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/gallium/drivers/radeonsi/si_shader.c b/src/gallium/drivers/radeonsi/si_shader.c index 0eeff19..b861831 100644 ---

[Mesa-dev] [PATCH 02/23] tgsi: add Stream{X, Y, Z, W} fields to tgsi_declaration_semantic

From: Nicolai Hähnle This is for geometry shader outputs. Without it, drivers have no way of knowing which stream each output is intended for, and have to conservatively write all outputs to all streams. Separate stream numbers for each component are required due to

[Mesa-dev] [PATCH 08/23] radeonsi: rename members of si_shader_output_values

From: Nicolai Hähnle Be a bit more verbose and avoid confusion in future patches. --- src/gallium/drivers/radeonsi/si_shader.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_shader.c

[Mesa-dev] [PATCH 12/23] radeonsi: load the streamout buf descriptors closer to their use

From: Nicolai Hähnle LLVM can still decide to hoist the loads since they're marked invariant. --- src/gallium/drivers/radeonsi/si_shader.c | 25 +++-- 1 file changed, 11 insertions(+), 14 deletions(-) diff --git

[Mesa-dev] [PATCH 06/23] radeonsi: update all GSVS ring descriptors for new buffer allocations

From: Nicolai Hähnle Fixes GL45-CTS.gtf40.GL3Tests.transform_feedback3.transform_feedback3_geometry_instanced. Cc: mesa-sta...@lists.freedesktop.org --- src/gallium/drivers/radeonsi/si_state_shaders.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff

[Mesa-dev] [PATCH 04/23] tgsi/scan: collect information about output vertex streams

From: Nicolai Hähnle --- src/gallium/auxiliary/tgsi/tgsi_scan.c | 9 + src/gallium/auxiliary/tgsi/tgsi_scan.h | 2 ++ 2 files changed, 11 insertions(+) diff --git a/src/gallium/auxiliary/tgsi/tgsi_scan.c b/src/gallium/auxiliary/tgsi/tgsi_scan.c index

  1   2   >