[Mesa-dev] [Bug 102847] swr fail to build with llvm-5.0.0

https://bugs.freedesktop.org/show_bug.cgi?id=102847 Bug ID: 102847 Summary: swr fail to build with llvm-5.0.0 Product: Mesa Version: 17.2 Hardware: x86-64 (AMD64) OS: Linux (All) Status: NEW Severity: norm

Re: [Mesa-dev] [PATCH] glsl/linker: fix output variable overlap check

This seems reasonable, thanks for fixing. Maybe just add a comment in the code also. /* 12 * 4 == (max # of FS outputs) * max components */ Or something similar. Reviewed-by: Timothy Arceri On 18/09/17 19:30, Nicolai Hähnle wrote: From: Nicolai Hähnle Prevent an overflow caused by too man

[Mesa-dev] [PATCH 2/2] glsl: merge loop_controls.cpp with loop_unroll.cpp

Having this separate just makes the code harder to follow, and requires an extra walk of the IR. --- src/compiler/Makefile.sources| 1 - src/compiler/glsl/glsl_parser_extras.cpp | 1 - src/compiler/glsl/loop_analysis.h| 16 - src/compiler/glsl/loop_controls.cpp |

[Mesa-dev] [PATCH 1/2] glsl: move loop analysis helper to loop_analysis.cpp

--- src/compiler/glsl/loop_analysis.cpp | 139 src/compiler/glsl/loop_analysis.h | 7 -- src/compiler/glsl/loop_controls.cpp | 139 3 files changed, 139 insertions(+), 146 deletions(-) diff --git a/src/compiler/glsl/loo

Re: [Mesa-dev] [PATCH] anv: Implement VK_ANDROID_native_buffer (v5)

On 2017年09月19日 05:44, 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 bl

[Mesa-dev] [PATCH] radv: Implement VK_AMD_rasterization_order

Tested with AMD's Anvil OutOfOrderRasterization demo on a RX 560. Signed-off-by: Nicholas Miell --- src/amd/vulkan/radv_device.c | 17 + src/amd/vulkan/radv_pipeline.c | 10 +- 2 files changed, 26 insertions(+), 1 deletion(-) diff --git a/src/amd/vulkan/radv_device.c b

Re: [Mesa-dev] [PATCH 00/15] anv: implement KHR_sampler_ycbcr_conversion

I may review some more tomorrow but I think I'm getting close to done. I would not mind at all seeing a v2 at this point. On September 15, 2017 9:11:51 AM Lionel Landwerlin wrote: Hi all, This series implements KHR_sampler_ycbcr_conversion (and KHR_bind_memory2 as requirement) for the An

Re: [Mesa-dev] [PATCH 4/8] nir: add some helpers for doing linking

On 19/09/17 10:12, Eric Anholt wrote: Timothy Arceri writes: The initial helpers as support for removing unused varyings between stages. --- src/compiler/Makefile.sources | 1 + src/compiler/nir/nir.h | 6 ++ src/compiler/nir/nir_linking_helpers.c | 136 +

Re: [Mesa-dev] [PATCH 4/8] nir: add some helpers for doing linking

Timothy Arceri writes: > The initial helpers as support for removing unused varyings between > stages. > --- > src/compiler/Makefile.sources | 1 + > src/compiler/nir/nir.h | 6 ++ > src/compiler/nir/nir_linking_helpers.c | 136 > + >

Re: [Mesa-dev] [PATCH 1/2] intel/blorp: Support clearing L8_UNORM_SRGB surfaces

On Monday, September 11, 2017 8:51:54 AM PDT Jason Ekstrand wrote: > Vulkan needs to be able to clear any texture you can create. We want to > add support for VK_FORMAT_R8_SRGB and we need to use L8_UNORM_SRGB to do > that so we need to be able to clear it. > --- > src/intel/blorp/blorp_clear.c |

[Mesa-dev] [PATCH] i965: Don't reset aux_state on buffers without modifiers

On gen7-8 where we use CCS_D, we were setting AUX_STATE_AUX_INVALID because that's the obvious aux state when there is no modifier. However, this doesn't work because that aux state isn't compatible with CCS_D. Cc: Chad Versace --- src/intel/isl/isl.h | 4 +++- src/mes

[Mesa-dev] [PATCH] anv: Implement VK_ANDROID_native_buffer (v5)

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 blocks on the sync fd, then puts the VkSemaphore

Re: [Mesa-dev] [PATCH 20/20 v4]] anv: Implement VK_ANDROID_native_buffer (v4)

On Fri 15 Sep 2017, Tapani Pälli wrote: > > > On 09/15/2017 01:06 AM, Chad Versace wrote: > > v3: > >- Delete duplicate #include "anv_private.h". [per Tapani] > >- Try to fix the Android-IA build in Android.vulkan.mk by following > > Tapani's example in > > > >

Re: [Mesa-dev] [PATCH 20/20] anv: Implement VK_ANDROID_native_buffer (v2)

On Fri 15 Sep 2017, zhoucm1 wrote: > > > On 2017年09月14日 07:03, Chad Versace wrote: > > From: Chad Versace > > > > This implementation is correct (afaict), but takes two shortcuts > > regarding the import/export of Android sync fds. > > > >Shortcut 1. When Android calls vkAcquireImageANDROI

Re: [Mesa-dev] [PATCH 12/15] anv: add descriptor support for multiplanar image/sampler

On Fri, Sep 15, 2017 at 3:54 PM, Jason Ekstrand wrote: > > On Fri, Sep 15, 2017 at 7:11 AM, Lionel Landwerlin < > lionel.g.landwer...@intel.com> wrote: > >> Signed-off-by: Lionel Landwerlin >> --- >> src/intel/vulkan/anv_descriptor_set.c| 107 >> +-- >> src/intel

Re: [Mesa-dev] [PATCH 2/2] i965: Fix brw_finish_batch to grow the batchbuffer.

On Monday, September 18, 2017 11:14:35 AM PDT Chris Wilson wrote: > Quoting Kenneth Graunke (2017-09-18 18:56:57) > > brw_finish_batch emits commands needed at the end of every batch buffer, > > including any workarounds. In the past, we freed up some "reserved" > > batch space before calling it,

Re: [Mesa-dev] [PATCH] st/dri: Add an option to always request a front buffer from the loader

On 09/18/2017 12:43 PM, Eric Anholt wrote: Thomas Hellstrom writes: When an application decides to read from the front buffer of a window, typically a fake front is created and initialized with the real front window contents. However, if there was a window manager reparenting operation between

Re: [Mesa-dev] [PATCH] st/dri: Add an option to always request a front buffer from the loader

On 09/18/2017 01:10 PM, Thomas Hellstrom wrote: On 09/18/2017 12:43 PM, Eric Anholt wrote: Thomas Hellstrom writes: When an application decides to read from the front buffer of a window, typically a fake front is created and initialized with the real front window contents. However, if there

Re: [Mesa-dev] [PATCH 1/2] i965: Move MI_BATCHBUFFER_END handling into brw_finish_batch().

On Monday, September 18, 2017 11:03:37 AM PDT Chris Wilson wrote: > Quoting Kenneth Graunke (2017-09-18 18:56:56) > > This is, by definition, finishing the batch. > > --- > > src/mesa/drivers/dri/i965/intel_batchbuffer.c | 14 +++--- > > 1 file changed, 7 insertions(+), 7 deletions(-) > >

Re: [Mesa-dev] [PATCH 1/2] configure: check if -latomic is needed for __atomic_*

Oh, also, these should get a Cc: mesa-stable tag. ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH 1/2] configure: check if -latomic is needed for __atomic_*

On Mon, Sep 18, 2017 at 12:28 PM, Grazvydas Ignotas wrote: > On some platforms, gcc generates library calls when __atomic_* functions > are used, but does not link the required library automatically. Detect > this and add the library when needed. > > This change was tested on armel (library was ad

Re: [Mesa-dev] [PATCH] st/dri: Add an option to always request a front buffer from the loader

Hi. On 09/18/2017 12:18 PM, Keith Packard wrote: Thomas Hellstrom writes: When an application decides to read from the front buffer of a window, typically a fake front is created and initialized with the real front window contents. However, if there was a window manager reparenting operation

Re: [Mesa-dev] [PATCH] r600: fork and import gallium/radeon

It wasn't copy-paste, it was "cp". Marek On Mon, Sep 18, 2017 at 9:42 PM, Grazvydas Ignotas wrote: > Has anyone ran full piglit on these with r600? Copy-paste bugs are > very easy to introduce. > > Gražvydas > > On Mon, Sep 18, 2017 at 7:32 PM, Marek Olšák wrote: >> Latest version with an updat

Re: [Mesa-dev] [PATCH] st/dri: Add an option to always request a front buffer from the loader

Thomas Hellstrom writes: > When an application decides to read from the front buffer of a window, > typically a fake front is created and initialized with the real front window > contents. However, if there was a window manager reparenting operation between > the last swapbuffer and the read the

Re: [Mesa-dev] [PATCH] r600: fork and import gallium/radeon

Has anyone ran full piglit on these with r600? Copy-paste bugs are very easy to introduce. Gražvydas On Mon, Sep 18, 2017 at 7:32 PM, Marek Olšák wrote: > Latest version with an updated commit message: > https://cgit.freedesktop.org/~mareko/mesa/commit/?h=master&id=8671613ed19586ac1eceda48eb067a

[Mesa-dev] [PATCH 2/2] util/u_atomic: remove unnecessaty __atomic functions

They are now provided by -latomic, which should be linked as needed since previous commit. Signed-off-by: Grazvydas Ignotas --- src/util/u_atomic.c | 12 1 file changed, 12 deletions(-) diff --git a/src/util/u_atomic.c b/src/util/u_atomic.c index b32527f..e4bffa8 100644 --- a/src/u

[Mesa-dev] [PATCH 1/2] configure: check if -latomic is needed for __atomic_*

On some platforms, gcc generates library calls when __atomic_* functions are used, but does not link the required library automatically. Detect this and add the library when needed. This change was tested on armel (library was added) and on x86_64 (was not, as expected). Fixes: 8915f0c0 "util: us

Re: [Mesa-dev] [PATCH 11/15] anv: add new formats KHR_sampler_ycbcr_conversion

On Sun, Sep 17, 2017 at 3:31 AM, Lionel Landwerlin < lionel.g.landwer...@intel.com> wrote: > On 15/09/17 23:36, Jason Ekstrand wrote: > > struct anv_format_plane { >> enum isl_format isl_format:16; >> struct isl_swizzle swizzle; >> + >> + /* Whether this plane contains chroma channels *

Re: [Mesa-dev] [PATCH] st/dri: Add an option to always request a front buffer from the loader

Thomas Hellstrom writes: > When an application decides to read from the front buffer of a window, > typically a fake front is created and initialized with the real front window > contents. However, if there was a window manager reparenting operation between > the last swapbuffer and the read the

Re: [Mesa-dev] [PATCH 4/4] i965: Use prepare_external instead of make_shareable in setTexBuffer2

On Mon, Sep 18, 2017 at 10:12 AM, Eric Anholt wrote: > Jason Ekstrand writes: > > > On Mon, Sep 18, 2017 at 9:20 AM, Chad Versace > > wrote: > > > >> On Thu 14 Sep 2017, Eric Anholt wrote: > >> > Jason Ekstrand writes: > >> > > >> > > The setTexBuffer2 hook from GLX is used to implement > glxB

Re: [Mesa-dev] [PATCH 10/15] anv: modify the internal concept of format to express multiple planes

On Fri, Sep 15, 2017 at 7:11 AM, Lionel Landwerlin < lionel.g.landwer...@intel.com> wrote: > A given Vulkan format can now be decomposed into a set of planes. We > now use 'struct anv_format_plane' to represent the format of those > planes. > > Signed-off-by: Lionel Landwerlin > --- > src/intel/

Re: [Mesa-dev] [PATCH mesa 2/2] radv: replace conditional compilation with MAYBE_UNUSED

Both patches are: Reviewed-by: Samuel Pitoiset On 09/18/2017 07:15 PM, Eric Engestrom wrote: Signed-off-by: Eric Engestrom --- src/amd/vulkan/radv_wsi.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/amd/vulkan/radv_wsi.c b/src/amd/vulkan/radv_wsi.c index 8a55

Re: [Mesa-dev] Building Mesa GIT with LLVM prior to 3.9

On Mon, 2017-09-18 at 17:00 +0100, Emil Velikov wrote: > Hi all, > > As you may know, different components in Mesa use LLVM for various purposes. > > In particular: > - 3.3.0 for the aux gallium module, gallivm > - 3.6.0 for the opencl state-tracker, clover > - 3.9.0 for multiple drivers r600,

Re: [Mesa-dev] [PATCH v2 3/7] travis: Add clover build using llvm-3.7

On Mon, 2017-09-18 at 16:46 +0100, Eric Engestrom wrote: > On Monday, 2017-09-18 11:10:37 -0400, Jan Vesely wrote: > > On Mon, 2017-09-18 at 11:13 +0100, Eric Engestrom wrote: > > > On Sunday, 2017-09-17 02:02:33 -0400, Jan Vesely wrote: > > > > v2: Use direct llvm repo link instead of alias > > >

Re: [Mesa-dev] [PATCH v2] configure+mesa/st: check -std=c++11 support and enable tests accordingly

Am Montag, den 18.09.2017, 16:07 +0100 schrieb Emil Velikov: > Hi Gert, > > On 17 September 2017 at 08:32, Gert Wollny > wrote: > > The tests in mesa/st/tests require C++11. Check whether the > > compiler > > supports the -std=c++11 flag and if so, enable the tests. Otherwise > > skip compiling a

Re: [Mesa-dev] [PATCH 2/2] i965: Fix brw_finish_batch to grow the batchbuffer.

Quoting Kenneth Graunke (2017-09-18 18:56:57) > brw_finish_batch emits commands needed at the end of every batch buffer, > including any workarounds. In the past, we freed up some "reserved" > batch space before calling it, so we would never have to flush during > it. This was error prone and eas

Re: [Mesa-dev] [PATCH 1/2] i965: Move MI_BATCHBUFFER_END handling into brw_finish_batch().

Quoting Kenneth Graunke (2017-09-18 18:56:56) > This is, by definition, finishing the batch. > --- > src/mesa/drivers/dri/i965/intel_batchbuffer.c | 14 +++--- > 1 file changed, 7 insertions(+), 7 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/intel_batchbuffer.c > b/src/mesa/dr

[Mesa-dev] [PATCH 1/2] i965: Move MI_BATCHBUFFER_END handling into brw_finish_batch().

This is, by definition, finishing the batch. --- src/mesa/drivers/dri/i965/intel_batchbuffer.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/src/mesa/drivers/dri/i965/intel_batchbuffer.c b/src/mesa/drivers/dri/i965/intel_batchbuffer.c index 7d5a8947e8f..dd584f

[Mesa-dev] [PATCH 2/2] i965: Fix brw_finish_batch to grow the batchbuffer.

brw_finish_batch emits commands needed at the end of every batch buffer, including any workarounds. In the past, we freed up some "reserved" batch space before calling it, so we would never have to flush during it. This was error prone and easy to screw up, so I deleted it a while back in favor o

Re: [Mesa-dev] [PATCH 0/3] tgsi, radeonsi: add CANON opcode for float canonicalization

Am 18.09.2017 um 19:11 schrieb Roland Scheidegger: > Am 18.09.2017 um 17:36 schrieb Nicolai Hähnle: >> On 18.09.2017 17:02, Roland Scheidegger wrote: >>> This looks like a horrendous solution which will break the world - well >>> for us :-). Because integers simply will cease to work, always flushe

[Mesa-dev] [PATCH mesa 2/2] radv: replace conditional compilation with MAYBE_UNUSED

Signed-off-by: Eric Engestrom --- src/amd/vulkan/radv_wsi.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/amd/vulkan/radv_wsi.c b/src/amd/vulkan/radv_wsi.c index 8a551c48bb..c9d4bbce8b 100644 --- a/src/amd/vulkan/radv_wsi.c +++ b/src/amd/vulkan/radv_wsi.c @@ -27,12

[Mesa-dev] [PATCH mesa 1/2] glsl: replace conditional compilation with MAYBE_UNUSED

Suggested-by: Nicolai Hähnle Signed-off-by: Eric Engestrom --- src/compiler/glsl/ir_validate.cpp | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/compiler/glsl/ir_validate.cpp b/src/compiler/glsl/ir_validate.cpp index 50d611500c..e368224a9b 100644 --- a/src/compiler/

Re: [Mesa-dev] [PATCH 4/4] i965: Use prepare_external instead of make_shareable in setTexBuffer2

Jason Ekstrand writes: > On Mon, Sep 18, 2017 at 9:20 AM, Chad Versace > wrote: > >> On Thu 14 Sep 2017, Eric Anholt wrote: >> > Jason Ekstrand writes: >> > >> > > The setTexBuffer2 hook from GLX is used to implement glxBindTexImageEXT >> > > which has tighter restrictions than just "it's share

Re: [Mesa-dev] [PATCH 0/3] tgsi, radeonsi: add CANON opcode for float canonicalization

Am 18.09.2017 um 17:36 schrieb Nicolai Hähnle: > On 18.09.2017 17:02, Roland Scheidegger wrote: >> This looks like a horrendous solution which will break the world - well >> for us :-). Because integers simply will cease to work, always flushed >> to zero (bye bye loop counter...). >> The reason is

Re: [Mesa-dev] [PATCH 4/4] i965: Use prepare_external instead of make_shareable in setTexBuffer2

On Mon, Sep 18, 2017 at 9:20 AM, Chad Versace wrote: > On Thu 14 Sep 2017, Eric Anholt wrote: > > Jason Ekstrand writes: > > > > > The setTexBuffer2 hook from GLX is used to implement glxBindTexImageEXT > > > which has tighter restrictions than just "it's shared". In particular, > > > it says t

Re: [Mesa-dev] [PATCH] ac/surface: handle S8 on gfx9

Reviewed-by: Marek Olšák Marek On Mon, Sep 18, 2017 at 5:21 AM, Dave Airlie wrote: > From: Dave Airlie > > If we don't have a depth piece, we don't get a correct > swizzle mode and we hit an assert in addrlib. > > In case of no depth get the preferrred swizzle mode for > stencil alone. > > Sig

Re: [Mesa-dev] [Mesa-stable] [PATCH 2/2] radeonsi: set MIP_POINT_PRECLAMP to 0

I commented on patch 1. Other than that, the series is: Reviewed-by: Marek Olšák Marek On Mon, Sep 18, 2017 at 4:57 PM, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > This fixes a bug with nearest ("point") mip selection when the fractional > part of max_lod is in (0.5,1). In this case, the

Re: [Mesa-dev] [Mesa-stable] [PATCH 1/2] radeonsi: fix array textures layer coordinate

On Mon, Sep 18, 2017 at 4:57 PM, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > Like for cube map (array) gather, we need to round to nearest on <= VI. > > Fixes tests in dEQP-GLES3.functional.shaders.texture_functions.texture.* > > Cc: mesa-sta...@lists.freedesktop.org > --- > src/gallium/dri

Re: [Mesa-dev] [PATCH] r600: fork and import gallium/radeon

Latest version with an updated commit message: https://cgit.freedesktop.org/~mareko/mesa/commit/?h=master&id=8671613ed19586ac1eceda48eb067a416f9c93be Marek On Mon, Sep 18, 2017 at 6:18 PM, Emil Velikov wrote: > On 18 September 2017 at 17:08, Marek Olšák wrote: >> On Thu, Sep 14, 2017 at 4:52 PM

Re: [Mesa-dev] [PATCH 4/4] i965: Use prepare_external instead of make_shareable in setTexBuffer2

On Thu 14 Sep 2017, Eric Anholt wrote: > Jason Ekstrand writes: > > > The setTexBuffer2 hook from GLX is used to implement glxBindTexImageEXT > > which has tighter restrictions than just "it's shared". In particular, > > it says that any rendering to the image while it is bound causes the > > co

Re: [Mesa-dev] [PATCH] r600: fork and import gallium/radeon

On 18 September 2017 at 17:08, Marek Olšák wrote: > On Thu, Sep 14, 2017 at 4:52 PM, Nicolai Hähnle wrote: >> On 14.09.2017 16:19, Emil Velikov wrote: >>> >>> Hi Marek, >>> >>> On 14 September 2017 at 14:06, Marek Olšák wrote: From: Marek Olšák This marks the end of code sha

Re: [Mesa-dev] [PATCH] r600: fork and import gallium/radeon

On Thu, Sep 14, 2017 at 4:52 PM, Nicolai Hähnle wrote: > On 14.09.2017 16:19, Emil Velikov wrote: >> >> Hi Marek, >> >> On 14 September 2017 at 14:06, Marek Olšák wrote: >>> >>> From: Marek Olšák >>> >>> This marks the end of code sharing between r600 and radeonsi. >>> >> It has the "what" but i

Re: [Mesa-dev] [PATCH] Fixes: radeonsi: allow out-of-order rasterization in commutative blending cases

On 18.09.2017 15:27, Benedikt Schemmer wrote: There is a small typo that prevents compilation. Please kindly review and push. Cheers, Benedikt Signed-off-by: Benedikt Schemmer Ilia pointed this out to me, but I dropped it (I think because the mail didn't go to the mailing list. Or something.

[Mesa-dev] [PATCH] gallium/radeon: consolidate PIPE_BIND_SHARED/SCANOUT handling

From: Marek Olšák --- src/gallium/drivers/radeon/r600_buffer_common.c | 13 - src/gallium/drivers/radeon/r600_texture.c | 4 2 files changed, 4 insertions(+), 13 deletions(-) diff --git a/src/gallium/drivers/radeon/r600_buffer_common.c b/src/gallium/drivers/radeon/r600_

Re: [Mesa-dev] [PATCH RFC 3/7] travis: Add clover build using llvm-3.7

Hi Jan, On 17 September 2017 at 03:05, Jan Vesely wrote: > Disable for now, travis does not allow llvm-3.7 trusty toolchain > I've just fired up a thread about bumping the Clover requirement to 3.9. The reason why I brought it up is that all modern platforms actually using Clover seems to have it

Re: [Mesa-dev] [PATCH 2/4] i965: Reset miptree aux state on update_image_buffer

On Tue 12 Sep 2017, Jason Ekstrand wrote: > When we get a miptree in through glxBindImageEXT, we don't know the > current aux state so we have to assume the worst-case. If the image > gets recreated, everything is fine because miptreecreate_for_dri_image > sets it to the default. However, if our

Re: [Mesa-dev] [PATCH 3/4] i965/tex_image: Reference the renderbuffer miptree in setTexBuffer2

On Tue 12 Sep 2017, Jason Ekstrand wrote: > The old code made a new miptree that referenced the same BO as the > renderbuffer and just trusted in the memory aliasing to work. There are > only two ways in which the new miptree is liable to differ from the one > in the renderbuffer and neither of th

[Mesa-dev] Building Mesa GIT with LLVM prior to 3.9

Hi all, As you may know, different components in Mesa use LLVM for various purposes. In particular: - 3.3.0 for the aux gallium module, gallivm - 3.6.0 for the opencl state-tracker, clover - 3.9.0 for multiple drivers r600, radeonsi, radv and swr AFAICT gallivm is build on Windows, and using

[Mesa-dev] [PATCH v2] glsl: buffer variables can be readonly and writeonly

In GLSL ES 3.10 session 4.9 [Memory Access Qualifiers], it has the following description: "A variable could be qualified as both readonly and writeonly, disallowing both read and write, but still be passed to imageSize() to have the size queried.". This is for image variable, but not for buffer

Re: [Mesa-dev] [PATCH] glsl: buffer variables can be readonly and writeonly

On Mon, 2017-09-18 at 10:50 +0200, Samuel Pitoiset wrote: > > On 09/15/2017 04:56 PM, Juan A. Suarez Romero wrote: > > On Fri, 2017-09-15 at 13:04 +0200, Juan A. Suarez Romero wrote: > > > On Fri, 2017-09-15 at 12:47 +0200, Samuel Pitoiset wrote: > > > > I guess this will break some piglits. > > >

Re: [Mesa-dev] [PATCH v2 3/7] travis: Add clover build using llvm-3.7

On Monday, 2017-09-18 11:10:37 -0400, Jan Vesely wrote: > On Mon, 2017-09-18 at 11:13 +0100, Eric Engestrom wrote: > > On Sunday, 2017-09-17 02:02:33 -0400, Jan Vesely wrote: > > > v2: Use direct llvm repo link instead of alias > > > Enable > > > > > > Signed-off-by: Jan Vesely > > > --- > >

Re: [Mesa-dev] [PATCH] llvmpipe, gallivm: implement lod queries (LODQ opcode)

On 18/09/17 03:53, srol...@vmware.com wrote: From: Roland Scheidegger This uses all the existing code to calculate lod values for mip linear filtering. Though we'll have to disable the simplifications (if we know some parts of the lod calculation won't actually matter for filtering purposes due

Re: [Mesa-dev] [PATCH] gallium: add PIPE_QUERY_OCCLUSION_PREDICATE_CONSERVATIVE

On 18.09.2017 17:22, Jan Vesely wrote: On Wed, 2017-09-13 at 18:53 +0200, Nicolai Hähnle wrote: From: Nicolai Hähnle To be able to properly distinguish between GL_ANY_SAMPLES_PASSED and GL_ANY_SAMPLES_PASSED_CONSERVATIVE. This patch goes through all drivers, having them treat the two query ty

Re: [Mesa-dev] [PATCH 1/7] clover: add missing include to compat.h

Jan Vesely writes: > Fixes build issues with llvm-3.6 > Fixes: 3115687f9b9830417c408228db2bc679e346bba6 (clover: Fix build after > LLVM r313390) > > Signed-off-by: Jan Vesely Reviewed-by: Francisco Jerez > --- > src/gallium/state_trackers/clover/llvm/compat.hpp | 1 + > 1 file changed, 1 ins

Re: [Mesa-dev] [PATCH 1/4] intel/isl: Add a drm_modifier_get_default_aux_state helper

On Tue 12 Sep 2017, Jason Ekstrand wrote: > --- > src/intel/isl/isl.h | 20 > src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 3 +-- > 2 files changed, 21 insertions(+), 2 deletions(-) > > diff --git a/src/intel/isl/isl.h b/src/intel/isl/isl.h > ind

Re: [Mesa-dev] [PATCH 0/3] tgsi, radeonsi: add CANON opcode for float canonicalization

On 18.09.2017 17:02, Roland Scheidegger wrote: This looks like a horrendous solution which will break the world - well for us :-). Because integers simply will cease to work, always flushed to zero (bye bye loop counter...). The reason is that when you translate from something with a untyped regi

Re: [Mesa-dev] [PATCH 1/7] clover: add missing include to compat.h

On Saturday, 2017-09-16 22:05:39 -0400, Jan Vesely wrote: > Fixes build issues with llvm-3.6 > Fixes: 3115687f9b9830417c408228db2bc679e346bba6 (clover: Fix build after > LLVM r313390) > > Signed-off-by: Jan Vesely R-b and pushed this with Gert's t-b, without waiting any longer since the build is

Re: [Mesa-dev] [PATCH] gallium: add PIPE_QUERY_OCCLUSION_PREDICATE_CONSERVATIVE

On Wed, 2017-09-13 at 18:53 +0200, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > To be able to properly distinguish between GL_ANY_SAMPLES_PASSED > and GL_ANY_SAMPLES_PASSED_CONSERVATIVE. > > This patch goes through all drivers, having them treat the two > query types identically, except: >

Re: [Mesa-dev] [PATCH v2 3/7] travis: Add clover build using llvm-3.7

On Mon, 2017-09-18 at 11:13 +0100, Eric Engestrom wrote: > On Sunday, 2017-09-17 02:02:33 -0400, Jan Vesely wrote: > > v2: Use direct llvm repo link instead of alias > > Enable > > > > Signed-off-by: Jan Vesely > > --- > > .travis.yml | 40 > > 1 file

Re: [Mesa-dev] [PATCH v2] configure+mesa/st: check -std=c++11 support and enable tests accordingly

Hi Gert, On 17 September 2017 at 08:32, Gert Wollny wrote: > The tests in mesa/st/tests require C++11. Check whether the compiler > supports the -std=c++11 flag and if so, enable the tests. Otherwise > skip compiling and running the tests altogether. > > Fixes: 7be6d8fe12 ("mesa/st: glsl_to_tgsi

Re: [Mesa-dev] [PATCH 0/3] tgsi, radeonsi: add CANON opcode for float canonicalization

This looks like a horrendous solution which will break the world - well for us :-). Because integers simply will cease to work, always flushed to zero (bye bye loop counter...). The reason is that when you translate from something with a untyped register file to something typed, the obvious solutio

Re: [Mesa-dev] [PATCH 3/3] egl/android: Use per surface out fence

On 18 September 2017 at 15:42, Marathe, Yogesh wrote: > Hi Emil, as discussed, we are leaving this at 3/3 split you've created. > > I've marked my earlier series "Not Applicable" in patchwork. Thanks. > Right, thanks. I'll push the series in an hour or so... with a couple of typos (below) fixed.

[Mesa-dev] [PATCH] Fixes: radeonsi: allow out-of-order rasterization in commutative blending cases

There is a small typo that prevents compilation. Please kindly review and push. Cheers, Benedikt Signed-off-by: Benedikt Schemmer diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_query_hw.c b/src/gallium/drivers/nouveau/nvc0/nvc0_query_hw.c --- a/src/gallium/drivers/nouveau/nvc0/nvc0_query_hw

Re: [Mesa-dev] [PATCH 05/15] vulkan: enum generator: generate extension number defines

No, this is all I had. Tables of tables formats are just in anv : https://github.com/djdeath/mesa/commit/b40dd5245cdaa5d686b8494f0e8c784b2eea4211#diff-8d7721e9d80d390ff503d2c3b4e1adc5L274 On 18/09/17 15:48, Jason Ekstrand wrote: Did you have a patch to the enum generator to actually generate th

[Mesa-dev] [PATCH 1/2] radeonsi: fix array textures layer coordinate

From: Nicolai Hähnle Like for cube map (array) gather, we need to round to nearest on <= VI. Fixes tests in dEQP-GLES3.functional.shaders.texture_functions.texture.* Cc: mesa-sta...@lists.freedesktop.org --- src/gallium/drivers/radeonsi/si_shader_tgsi_mem.c | 12 +++- 1 file changed, 1

[Mesa-dev] [PATCH 2/2] radeonsi: set MIP_POINT_PRECLAMP to 0

From: Nicolai Hähnle This fixes a bug with nearest ("point") mip selection when the fractional part of max_lod is in (0.5,1). In this case, the spec mandates that we still select the mip level ceil(max_lod) in the clamping case. However, MIP_POINT_PRECLAMP will clamp before the mip selection, whi

Re: [Mesa-dev] [PATCH 05/15] vulkan: enum generator: generate extension number defines

Did you have a patch to the enum generator to actually generate the table of tables for the formats? I don't see that in ithe series. On Mon, Sep 18, 2017 at 7:46 AM, Jason Ekstrand wrote: > On Fri, Sep 15, 2017 at 7:10 AM, Lionel Landwerlin < > lionel.g.landwer...@intel.com> wrote: > >> New ex

Re: [Mesa-dev] [PATCH 05/15] vulkan: enum generator: generate extension number defines

On Fri, Sep 15, 2017 at 7:10 AM, Lionel Landwerlin < lionel.g.landwer...@intel.com> wrote: > New extensions can introduce additional enums. Most of the new enums > will have disjoint numbers from the initial enums. For example new > formats introduced by VK_IMG_format_pvrtc : > > VK_FORMAT_ASTC_10

Re: [Mesa-dev] [PATCH 3/3] egl/android: Use per surface out fence

Hi Emil, as discussed, we are leaving this at 3/3 split you've created. I've marked my earlier series "Not Applicable" in patchwork. Thanks. -Yogesh. >-Original Message- >From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On Behalf Of >Emil Velikov >Sent: Friday, September 15

Re: [Mesa-dev] [PATCH] Fixes: radeonsi: allow out-of-order rasterization in commutative blending cases

Wrong commit, correct one is: gallium: add PIPE_QUERY_OCCLUSION_PREDICATE_CONSERVATIVE Am 18.09.2017 um 15:27 schrieb Benedikt Schemmer: > There is a small typo that prevents compilation. > Please kindly review and push. > Cheers, > Benedikt > > Signed-off-by: Benedikt Schemmer > > diff --git a

[Mesa-dev] [PATCH] Fixes: gallium: add PIPE_QUERY_OCCLUSION_PREDICATE_CONSERVATIVE

There is a small typo that prevents compilation. Please kindly review and push. Cheers, Benedikt Signed-off-by: Benedikt Schemmer --- diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_query_hw.c b/src/gallium/drivers/nouveau/nvc0/nvc0_query_hw.c --- a/src/gallium/drivers/nouveau/nvc0/nvc0_que

Re: [Mesa-dev] [PATCH] swr/rast: do not crash on NULL strings returned by getenv

On 18 September 2017 at 11:48, Eric Engestrom wrote: > On Monday, 2017-09-18 11:29:21 +0100, Emil Velikov wrote: >> From: Bernhard Rosenkraenzer >> >> The current convinince function GetEnv feeds the results of getenv >> directly into std::string(). That is a bad idea, since the variable >> may b

Re: [Mesa-dev] [PATCH 3/3] glsl: make loop unrolling more like the nir unrolling path

On 18/09/17 20:50, Nicolai Hähnle wrote: On 14.09.2017 06:47, Timothy Arceri wrote: The old code incorrectly assumed that loop terminators will always be at the start of the loop. It really seems to be just luck that we haven't triggered any bugs here, for example if there is a loop terminator

Re: [Mesa-dev] [PATCH] swr/rast: do not crash on NULL strings returned by getenv

Le lundi 18 septembre 2017, 12:48:45 CEST Eric Engestrom a écrit : > On Monday, 2017-09-18 11:29:21 +0100, Emil Velikov wrote: > > From: Bernhard Rosenkraenzer > > > > The current convinince function GetEnv feeds the results of getenv > > directly into std::string(). That is a bad idea, since the

Re: [Mesa-dev] [PATCH v2 3/7] travis: Add clover build using llvm-3.7

On Sunday, 2017-09-17 02:02:33 -0400, Jan Vesely wrote: > v2: Use direct llvm repo link instead of alias > Enable > > Signed-off-by: Jan Vesely > --- > .travis.yml | 40 > 1 file changed, 40 insertions(+) > > diff --git a/.travis.yml b/.travis.yml >

Re: [Mesa-dev] [PATCH] radv: Check for GFX9 for 1D arrays in image_size intrinsic.

On 15.09.2017 21:46, Bas Nieuwenhuizen wrote: Only on GFX9 we implement them as 2D images. This fixes: dEQP-VK.image.image_size.1d_array.readonly_12x34 dEQP-VK.image.image_size.1d_array.readonly_1x1 dEQP-VK.image.image_size.1d_array.readonly_32x32 dEQP-VK.image.image_size.1d_array.readonly_7x1 d

Re: [Mesa-dev] [PATCH 3/3] glsl: make loop unrolling more like the nir unrolling path

On 14.09.2017 06:47, Timothy Arceri wrote: The old code incorrectly assumed that loop terminators will always be at the start of the loop. It really seems to be just luck that we haven't triggered any bugs here, for example if there is a loop terminator at the start of the loop we might actually

Re: [Mesa-dev] [PATCH 1/3] glsl: don't drop intructions from unreachable terminators continue branch

On 18/09/17 20:09, Nicolai Hähnle wrote: On 14.09.2017 06:47, Timothy Arceri wrote: These instruction will be executed on every iteration of the loop we cannot drop them. ---   src/compiler/glsl/loop_analysis.h   |  7 +++   src/compiler/glsl/loop_controls.cpp | 15 +++   src/com

Re: [Mesa-dev] [PATCH] swr/rast: do not crash on NULL strings returned by getenv

On Monday, 2017-09-18 11:29:21 +0100, Emil Velikov wrote: > From: Bernhard Rosenkraenzer > > The current convinince function GetEnv feeds the results of getenv > directly into std::string(). That is a bad idea, since the variable > may be unset, thus we feed NULL into the C++ construct. > > The

Re: [Mesa-dev] [PATCH] swr/rast: Fix GetEnv() returned value

On Monday, 2017-09-18 11:10:39 +0200, Laurent Carlier wrote: > Return an empty string instead of a NULL pointer. > Patch from Bernhard Rosenkraenzer. I guess attributing the patch to him in git would be nice too: From: Bernhard Rosenkraenzer > > https://bugs.freedesktop.org/show_bug.cgi?id=1018

[Mesa-dev] [PATCH] swr/rast: do not crash on NULL strings returned by getenv

From: Bernhard Rosenkraenzer The current convinince function GetEnv feeds the results of getenv directly into std::string(). That is a bad idea, since the variable may be unset, thus we feed NULL into the C++ construct. The latter of which is not allowed and leads to a crash. Bugzilla: https://

Re: [Mesa-dev] [PATCH 2/3] glsl: check if induction var incremented before use in terminator

On 14.09.2017 06:47, Timothy Arceri wrote: do-while loops can increment the starting value before the condition is checked. e.g. do { ndx++; } while (ndx < 3); This commit changes the code to detect this and reduces the iteration count by 1 if found. With Eric's remark fixes, this

Re: [Mesa-dev] [PATCH 3/3] i965: Force outputs_written to contain varyings needed by stream-out.

For the series: Reviewed-by: Juan A. Suarez Romero On Fri, 2017-09-15 at 10:17 -0700, Kenneth Graunke wrote: > If transform feedback is recording a varying, it needs a slot in the > VUE map, regardless of whether or not the shader writes it. > > Together with the previous patch, this fixes: >

Re: [Mesa-dev] [PATCH 1/3] glsl: don't drop intructions from unreachable terminators continue branch

On 14.09.2017 06:47, Timothy Arceri wrote: These instruction will be executed on every iteration of the loop *instructions Als, the title of the patch is missing an s ;) Cheers, Nicolai we cannot drop them. --- src/compiler/glsl/loop_analysis.h | 7 +++ src/compiler/glsl/loop_con

Re: [Mesa-dev] [PATCH] i965: skip varyings without slot

On Wed, 2017-09-06 at 10:32 +1000, Timothy Arceri wrote: > > On 01/09/17 21:15, Juan A. Suarez Romero wrote: > > On Thu, 2017-06-29 at 14:43 +1000, Timothy Arceri wrote: > > > On 27/06/17 21:20, Juan A. Suarez Romero wrote: > > > > On Tue, 2017-06-27 at 09:29 +1000, Timothy Arceri wrote: > > > > >

Re: [Mesa-dev] [PATCH 1/3] glsl: don't drop intructions from unreachable terminators continue branch

On 14.09.2017 06:47, Timothy Arceri wrote: These instruction will be executed on every iteration of the loop we cannot drop them. --- src/compiler/glsl/loop_analysis.h | 7 +++ src/compiler/glsl/loop_controls.cpp | 15 +++ src/compiler/glsl/loop_unroll.cpp | 7 ---

Re: [Mesa-dev] [PATCH 2/2] radeonsi: reallocate if a non-sharable textures is being shared

Both patches: Reviewed-by: Nicolai Hähnle On 14.09.2017 19:40, Marek Olšák wrote: From: Marek Olšák --- src/gallium/drivers/radeon/r600_texture.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/radeon/r600_texture.c b/src/gallium/drivers/rade

Re: [Mesa-dev] [PATCH 2/3] radv: only copy the dynamic states that changed

On 09/15/2017 05:40 PM, Fredrik Höglund wrote: On Thursday 14 September 2017, Samuel Pitoiset wrote: On 09/13/2017 06:34 PM, Fredrik Höglund wrote: On Wednesday 13 September 2017, Samuel Pitoiset wrote: When binding a new pipeline, we applied all dynamic states without checking if they real

[Mesa-dev] [PATCH 4/4] radv: do not update the number of scissors in vkCmdSetScissor()

The Vulkan spec (1.0.61) says: "The number of scissors used by a pipeline is still specified by the scissorCount member of VkPipelinescissorStateCreateInfo." So, the number of scissors is defined at pipeline creation time and shouldn't be updated when they are set dynamically. Signed-off-

[Mesa-dev] [PATCH 3/4] radv: do not update the number of viewports in vkCmdSetViewport()

The Vulkan spec (1.0.61) says: "The number of viewports used by a pipeline is still specified by the viewportCount member of VkPipelineViewportStateCreateInfo." So, the number of viewports is defined at pipeline creation time and shouldn't be updated when they are set dynamically. Signed-o

  1   2   >