[Mesa-dev] [Bug 99119] swr_fence_work.cpp(42): error: argument of type "std::nullptr_t" is incompatible with parameter of type "unsigned long"

2016-12-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99119 Bug ID: 99119 Summary: swr_fence_work.cpp(42): error: argument of type "std::nullptr_t" is incompatible with parameter of type "unsigned long" Product: Mesa

[Mesa-dev] [PATCH] dri: allow 16bit R/GR images to be exported via drm buffers

2016-12-16 Thread Rainer Hochecker
From: Rainer Hochecker This allows eglCreateImageKHR to access P010 surfaces created by vaapi patch for drm, fourcc: http://paste.ubuntu.com/23638632/ Signed-off-by: Rainer Hochecker --- include/GL/internal/dri_interface.h | 4

[Mesa-dev] [RFC 9/9] drm/i915: Add engine reset count in get-reset-stats ioctl

2016-12-16 Thread Michel Thierry
Users/tests relying on the total reset count will start seeing a smaller number since most of the hangs can be handled by engine reset. Note that if reset engine x, context a running on engine y will be unaware and unaffected. To start the discussion, include just a total engine reset count. If

[Mesa-dev] [PATCH] glsl: Drop bogus is_vertex_input from add_shader_variable().

2016-12-16 Thread Kenneth Graunke
stage_mask is a bitmask of shader stages, so the proper comparison would be (1 << MESA_SHADER_VERTEX), not MESA_SHADER_VERTEX itself. But we only care for structure types, and VS inputs cannot be structs. So we can just drop this entirely. Signed-off-by: Kenneth Graunke

[Mesa-dev] [PATCH] glsl: Fix program interface queries relating to interface blocks.

2016-12-16 Thread Kenneth Graunke
This fixes 555 dEQP tests (using the nougat-cts-dev branch), Piglit's arb_program_interface_query/arb_program_interface_query-resource-query, and GL45-CTS.program_interface_query.separate-programs-{tess-control, tess-eval,geometry}. Only one dEQP program interface failure remains. I would have

Re: [Mesa-dev] New GBM backend for dEQP

2016-12-16 Thread Chad Versace
On Fri 16 Dec 2016, Chad Versace wrote: > Happy Christmas to everyone who's busy squashing dEQP bugs. > > I wrote a new GBM backend for dEQP. [...] > There's probably bugs. No surprises there. You probably want this Mesa patch before using dEQP on GBM. It fixes some EGL failures. Subject:

[Mesa-dev] [PATCH] egl: Check config's surface types in eglCreate*Surface()

2016-12-16 Thread Chad Versace
If the provided EGLConfig does not support the requested surface type, then emit EGL_BAD_MATCH. Fixes dEQP-EGL.functional.negative_api.create_pbuffer_surface on GBM. Cc: "13.0" --- src/egl/main/eglapi.c | 10 ++ 1 file changed, 10 insertions(+) diff

[Mesa-dev] New GBM backend for dEQP

2016-12-16 Thread Chad Versace
Happy Christmas to everyone who's busy squashing dEQP bugs. I wrote a new GBM backend for dEQP. I even submitted it to dEQP's upstream Gerrit. Pyry, dEQP's maintainer, told me over beer earlier this year that he would accept it if I submitted it, and if it wasn't too crazy. So, maybe it'll be

Re: [Mesa-dev] [PATCH 1/2] radeonsi: only set VS_OUT_MISC_SIDE_BUS_ENA when the misc vector is used

2016-12-16 Thread Edward O'Callaghan
This series is, Reviewed-by: Edward O'Callaghan On 12/16/2016 02:42 AM, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > Should have no effect (other than perhaps on power consumption), but > Vulkan does this. > --- >

Re: [Mesa-dev] [PATCH 1/2] spirv/nir: add support for ImageGatherExtended

2016-12-16 Thread Dave Airlie
On 15 December 2016 at 12:23, Ilia Mirkin wrote: > ping > > On Sun, Nov 27, 2016 at 4:22 PM, Ilia Mirkin wrote: >> The strategy is to do the same thing that the GLSL lower_offset_arrays >> pass does - create 4 separate texture gather ops, one per

Re: [Mesa-dev] [PATCH] radv: Don't bail out on pipeline create failure.

2016-12-16 Thread Dave Airlie
On 17 December 2016 at 08:14, Bas Nieuwenhuizen wrote: > The spec says we have to try to create all, and only set failed > pipelines to VK_NULL_HANDLE. If one of them fails, we have to return > an error, but as far as I can see, the spec does not care which of > the

[Mesa-dev] [Bug 99116] Wine program showing only a blackscreen when using mesa

2016-12-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99116 --- Comment #1 from Ilia Mirkin --- Replays fine on Intel SKL GT2 (Mesa 13.0.2 with Intel DDX + DRI3). Probably an issue with st/mesa's logic somewhere. -- You are receiving this mail because: You are the QA Contact for

[Mesa-dev] [Bug 99116] Wine program showing only a blackscreen when using mesa

2016-12-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99116 Fabian Maurer changed: What|Removed |Added CC||dark.shad...@web.de

[Mesa-dev] [Bug 99116] Wine program showing only a blackscreen when using mesa

2016-12-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99116 Bug ID: 99116 Summary: Wine program showing only a blackscreen when using mesa Product: Mesa Version: git Hardware: x86-64 (AMD64) OS: Linux (All)

[Mesa-dev] [PATCH] radv: Don't bail out on pipeline create failure.

2016-12-16 Thread Bas Nieuwenhuizen
The spec says we have to try to create all, and only set failed pipelines to VK_NULL_HANDLE. If one of them fails, we have to return an error, but as far as I can see, the spec does not care which of the suberrors. Fixes dEQP-VK.api.object_management.alloc_callback_fail_multiple.compute_pipeline

Re: [Mesa-dev] [PATCH] Mesa: Return GL error INVALID_OPERATION in case invalid format/type

2016-12-16 Thread Ian Romanick
On 12/15/2016 05:25 PM, Xu, Randy wrote: > Thanks, Matt > > I have run "git config --global user.name "Randy Xu"" and see it in > ~/.gitconfig > [user] > email = randy...@intel.com > name = Randy Xu Perhaps you need quotes around the name? My ~/.gitconfig has: [user]

Re: [Mesa-dev] [PATCH] anv: Fix uniform and storage buffer offset alignment limits.

2016-12-16 Thread Jason Ekstrand
On Fri, Dec 16, 2016 at 1:26 PM, Francisco Jerez wrote: > Jason Ekstrand writes: > > > On Dec 15, 2016 3:22 PM, "Francisco Jerez" > wrote: > > > > This fixes an apparent regression in a bunch of image store vulkan CTS > >

Re: [Mesa-dev] [PATCH] Mesa: Fix error code for glTexImage3D in GLES

2016-12-16 Thread Ian Romanick
On 12/16/2016 12:49 PM, Chad Versace wrote: > On Fri 16 Dec 2016, Chad Versace wrote: >> On Fri 16 Dec 2016, Randy Xu wrote: >>> From: "Xu,Randy" >>> >>> The ES specification says that TexImage3D should return GL_INVALID_OPERATION >>> if the internal format is DEPTH_COMPONENT,

Re: [Mesa-dev] [RFC PATCH] clover: Return correct CL_EVENT_REFERENCE_COUNT

2016-12-16 Thread Francisco Jerez
Vedran Miletić writes: > Current implementation of event handling keeps an extra reference to > the hardware event, in addition to the reference returned via the OpenCL > API. This additional reference is internal and should not be counted > when queried via the

Re: [Mesa-dev] [PATCH] anv: Fix uniform and storage buffer offset alignment limits.

2016-12-16 Thread Francisco Jerez
Jason Ekstrand writes: > On Dec 15, 2016 3:22 PM, "Francisco Jerez" wrote: > > This fixes an apparent regression in a bunch of image store vulkan CTS > tests from commit ad38ba113491869ab0dffed937f7b3dd50e8a735, which > started using OWORD block read

Re: [Mesa-dev] [PATCH] Mesa: Fix error code for glTexImage3D in GLES

2016-12-16 Thread Chad Versace
On Fri 16 Dec 2016, Chad Versace wrote: > On Fri 16 Dec 2016, Randy Xu wrote: > > From: "Xu,Randy" > > > > The ES specification says that TexImage3D should return GL_INVALID_OPERATION > > if the internal format is DEPTH_COMPONENT, DEPTH_-STENCIL or STENCIL_INDEX. > > The

Re: [Mesa-dev] [PATCH] Mesa: Fix error code for glTexImage3D in GLES

2016-12-16 Thread Chad Versace
On Fri 16 Dec 2016, Randy Xu wrote: > From: "Xu,Randy" > > The ES specification says that TexImage3D should return GL_INVALID_OPERATION > if the internal format is DEPTH_COMPONENT, DEPTH_-STENCIL or STENCIL_INDEX. > The current code returns INVALID_ENUM as

Re: [Mesa-dev] [Mesa-stable] [PATCH 2/2] i965/mt: Disable HiZ when sharing depth buffer externally (v2)

2016-12-16 Thread Chad Versace
On Thu 15 Dec 2016, Emil Velikov wrote: > Hi Chad, > > On 10 December 2016 at 00:40, Chad Versace wrote: > > intel_miptree_make_shareable() discarded and disabled CCS. Fix it so > > that it discards and disables HiZ too. > > > > Fixes > >

Re: [Mesa-dev] [PATCH] nir: Remove nir_array from lower_locals_to_regs

2016-12-16 Thread Jason Ekstrand
On Thu, Dec 15, 2016 at 10:52 PM, Thomas Helland wrote: > Could I bother one of you to push it? > I haven't yet acquired commit access. > Done. > 2016-12-16 7:26 GMT+01:00 Jason Ekstrand : > > Thanks! Rb. > > > > On Dec 15, 2016 3:30 PM, "Eric

Re: [Mesa-dev] [PATCH 3/3] nir: Rewrite lower_regs_to_ssa to use the phi builder

2016-12-16 Thread Jason Ekstrand
On Fri, Dec 16, 2016 at 11:08 AM, Jason Ekstrand wrote: > On Thu, Dec 15, 2016 at 6:29 PM, Jason Ekstrand > wrote: > >> On Dec 15, 2016 3:08 PM, "Eric Anholt" wrote: >> >> Jason Ekstrand writes: >> >> > This

Re: [Mesa-dev] [PATCH v3 3/4] nir/phi-builder: Set the value in the block when creating a phi

2016-12-16 Thread Jason Ekstrand
Ken, Eric, Any thoughts on whether or not this should go to stable? It does fix a real bug but the result of the bug is that we end up generating a bunch of duplicate phi nodes instead of just one for the block. In the end it doesn't matter since CSE cleans them up. --Jason On Fri, Dec 16,

Re: [Mesa-dev] [PATCH 3/3] nir: Rewrite lower_regs_to_ssa to use the phi builder

2016-12-16 Thread Jason Ekstrand
On Thu, Dec 15, 2016 at 6:29 PM, Jason Ekstrand wrote: > On Dec 15, 2016 3:08 PM, "Eric Anholt" wrote: > > Jason Ekstrand writes: > > > This keeps some of Connor's original code. However, while I was at it, > > I updated this very

[Mesa-dev] [PATCH v3 3/4] nir/phi-builder: Set the value in the block when creating a phi

2016-12-16 Thread Jason Ekstrand
After we figure out the value that we are going to return, we have a loop that walks up the dominance tree and sets the value in each of the blocks that doesn't have one yet. In the case of the phi, the def is set to NEEDS_PHI not NULL, so the last one where the phi node actually goes never gets

[Mesa-dev] [PATCH v3 4/4] nir: Rewrite lower_regs_to_ssa to use the phi builder

2016-12-16 Thread Jason Ekstrand
This keeps some of Connor's original code. However, while I was at it, I updated this very old pass to a bit more modern NIR. --- src/compiler/nir/nir_lower_regs_to_ssa.c | 589 +-- 1 file changed, 167 insertions(+), 422 deletions(-) diff --git

[Mesa-dev] [PATCH v3 2/4] nir: Add foreach_register helper macros

2016-12-16 Thread Jason Ekstrand
--- src/compiler/nir/nir.h | 5 + 1 file changed, 5 insertions(+) diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h index f1c99ce..9402863 100644 --- a/src/compiler/nir/nir.h +++ b/src/compiler/nir/nir.h @@ -373,6 +373,11 @@ typedef struct nir_register { struct list_head

[Mesa-dev] [PATCH v3 1/4] nir: Rename convert_to_ssa lower_regs_to_ssa

2016-12-16 Thread Jason Ekstrand
This matches the naming of nir_lower_vars_to_ssa, the other to-SSA pass. --- src/compiler/Makefile.sources | 2 +- src/compiler/nir/nir.h | 4 +--- src/compiler/nir/{nir_to_ssa.c => nir_lower_regs_to_ssa.c} | 8

[Mesa-dev] [Bug 99076] dEQP-GLES3.functional.negative_api.texture#teximage3d fails due to wrong Error code

2016-12-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99076 Chad Versace changed: What|Removed |Added CC|

[Mesa-dev] [RFC PATCH] clover: Return correct CL_EVENT_REFERENCE_COUNT

2016-12-16 Thread Vedran Miletić
Current implementation of event handling keeps an extra reference to the hardware event, in addition to the reference returned via the OpenCL API. This additional reference is internal and should not be counted when queried via the clGetEventInfo() function. Fixes Piglit's

Re: [Mesa-dev] [PATCH 2/2] radeonsi: fix gl_ClipDistance and gl_ClipVertex for points

2016-12-16 Thread Marek Olšák
For the series: Reviewed-by: Marek Olšák Marek On Thu, Dec 15, 2016 at 4:42 PM, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > The clipper hardware doesn't consider points as primitives that can be > clipped. Simply setting

Re: [Mesa-dev] radv vs vulkan loader

2016-12-16 Thread Gustaw Smolarczyk
2016-12-16 18:12 GMT+01:00 Gustaw Smolarczyk : > 2016-12-16 17:57 GMT+01:00 Emil Velikov : >> On 16 December 2016 at 15:27, Gustaw Smolarczyk wrote: >>> 2016-12-16 14:50 GMT+01:00 Emil Velikov :

Re: [Mesa-dev] [Mesa-stable] [PATCH] configure: Strip LLVMX86Asm* dependencies.

2016-12-16 Thread Emil Velikov
On 16 December 2016 at 15:36, Nicolai Hähnle wrote: > On 04.12.2016 11:34, Gustaw Smolarczyk wrote: >> >> They are picked automatically by the provided llvm-config flags, but are >> not needed. >> >> Fixes loading radv through a vulkan loader. >> >> Cc: 13.0

Re: [Mesa-dev] [PATCH] configure.ac: Respect LLVM_CFLAGS in LLVM version detection

2016-12-16 Thread Tobias Droste
Hi Tomasz, does this actually fix anything? Because right now llvm-config.h does not include anything and I doubt it will in the future, as it's just a collection of defines. The path to the header file itself is given by llvm-config ($LLVM_INCLUDEDIR). Did you just happen to see this or do

Re: [Mesa-dev] radv vs vulkan loader

2016-12-16 Thread Gustaw Smolarczyk
2016-12-16 17:57 GMT+01:00 Emil Velikov : > On 16 December 2016 at 15:27, Gustaw Smolarczyk wrote: >> 2016-12-16 14:50 GMT+01:00 Emil Velikov : >>> >>> On 5 October 2016 at 23:12, Gustaw Smolarczyk

Re: [Mesa-dev] [PATCH 4/6] configure.ac: Set and use HAVE_GALLIUM_LLVM define

2016-12-16 Thread Tobias Droste
Hey Emil, did you have to time look at this? What should we do? I would argue to first merge what I have now and if you really want to disallow llvm-less gallium with llvm-full radv we could do this later and revert the changes of patch 4. Personally I think it's ok how it is now. Gallium

Re: [Mesa-dev] radv vs vulkan loader

2016-12-16 Thread Emil Velikov
On 16 December 2016 at 15:27, Gustaw Smolarczyk wrote: > 2016-12-16 14:50 GMT+01:00 Emil Velikov : >> >> On 5 October 2016 at 23:12, Gustaw Smolarczyk >> wrote: >> > 2016-10-06 0:05 GMT+02:00 Emil Velikov

Re: [Mesa-dev] [PATCH] configure: Strip LLVMX86Asm* dependencies.

2016-12-16 Thread Nicolai Hähnle
On 04.12.2016 11:34, Gustaw Smolarczyk wrote: They are picked automatically by the provided llvm-config flags, but are not needed. Fixes loading radv through a vulkan loader. Cc: 13.0 --- It's work-around for:

Re: [Mesa-dev] radv vs vulkan loader

2016-12-16 Thread Gustaw Smolarczyk
2016-12-16 14:50 GMT+01:00 Emil Velikov : > On 5 October 2016 at 23:12, Gustaw Smolarczyk > wrote: > > 2016-10-06 0:05 GMT+02:00 Emil Velikov : > >> On 5 October 2016 at 21:45, Gustaw Smolarczyk >

[Mesa-dev] [Bug 99014] clover is broken for Oland in 12.0.4

2016-12-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99014 Vedran Miletić changed: What|Removed |Added Summary|clover is broken in 12.0.4 |clover is broken

[Mesa-dev] [Bug 99014] clover is broken in 12.0.4

2016-12-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99014 --- Comment #9 from Vedran Miletić --- Not reproducible on Hawaii. -- You are receiving this mail because: You are the assignee for the bug. You are the QA Contact for the bug.___

[Mesa-dev] [PATCH 2/2] automake: use shared LLVM for `make distcheck'

2016-12-16 Thread Emil Velikov
From: Emil Velikov When using static link with SWR et al. causes huge sprike in processing power/memory gets leading to hude increase in the test times. Revert to the default value, which is used by the vast majority. Cc: 12.0 13.0

[Mesa-dev] [PATCH 1/2] get-typod-pick-list.sh: add new script

2016-12-16 Thread Emil Velikov
From: Emil Velikov Typos do happen as people nominate patches for stable. This script aims to catch most of those. Due to the subtle nature of things, one has to pay special attention to the output, similar to get-extra-pick-list.sh. At the moment only the following

[Mesa-dev] [PATCH v2 06/25] spirv: fix SpvOpSpecConstantOp with SpvOpVectorShuffle working with double-based vecs

2016-12-16 Thread Juan A. Suarez Romero
From: Samuel Iglesias Gonsálvez We need to pick two 32-bit values per component to perform the right shuffle operation. Signed-off-by: Samuel Iglesias Gonsálvez --- src/compiler/spirv/spirv_to_nir.c | 25 + 1 file changed,

[Mesa-dev] [PATCH v2 05/25] spirv: add DF support to SpvOp*ConstantComposite

2016-12-16 Thread Juan A. Suarez Romero
From: Samuel Iglesias Gonsálvez Signed-off-by: Samuel Iglesias Gonsálvez --- src/compiler/spirv/spirv_to_nir.c | 12 +--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/src/compiler/spirv/spirv_to_nir.c

[Mesa-dev] [PATCH v2 23/25] spirv: enable SpvCapabilityFloat64 only to supported platforms

2016-12-16 Thread Juan A. Suarez Romero
From: Samuel Iglesias Gonsálvez Signed-off-by: Samuel Iglesias Gonsálvez --- src/compiler/spirv/spirv_to_nir.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/compiler/spirv/spirv_to_nir.c

[Mesa-dev] [PATCH v2 21/25] spirv: Add nir_options to vtn_builder

2016-12-16 Thread Juan A. Suarez Romero
From: Samuel Iglesias Gonsálvez Signed-off-by: Samuel Iglesias Gonsálvez --- src/compiler/spirv/spirv_to_nir.c | 1 + src/compiler/spirv/vtn_private.h | 1 + 2 files changed, 2 insertions(+) diff --git a/src/compiler/spirv/spirv_to_nir.c

[Mesa-dev] [PATCH v2 24/25] i965: enable nir_option's native_float64 to supported generations

2016-12-16 Thread Juan A. Suarez Romero
From: Samuel Iglesias Gonsálvez Currently, gen8+ supports ARB_gpu_shader_fp64 in mesa master. Signed-off-by: Samuel Iglesias Gonsálvez --- src/mesa/drivers/dri/i965/brw_compiler.c | 36 +--- 1 file changed, 24

[Mesa-dev] [PATCH v2 25/25] anv: enable shaderFloat64 feature

2016-12-16 Thread Juan A. Suarez Romero
From: Samuel Iglesias Gonsálvez Signed-off-by: Samuel Iglesias Gonsálvez --- src/intel/vulkan/anv_device.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.c index

[Mesa-dev] [PATCH v2 22/25] nir/i965: use two slots from inputs_read for dvec3/dvec4 vertex input attributes

2016-12-16 Thread Juan A. Suarez Romero
So far, input_reads was a bitmap tracking which vertex input locations were being used. In OpenGL, an attribute bigger than a vec4 (like a dvec3 or dvec4) consumes just one location, any other small attribute. So we mark the proper bit in inputs_read, and also the same bit in double_inputs_read

[Mesa-dev] [PATCH v2 20/25] nir: Add flag to detect platforms with native float64 support

2016-12-16 Thread Juan A. Suarez Romero
From: Samuel Iglesias Gonsálvez Signed-off-by: Samuel Iglesias Gonsálvez --- src/compiler/nir/nir.h | 5 + 1 file changed, 5 insertions(+) diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h index 9f3abb7..c369db3 100644 ---

[Mesa-dev] [PATCH v2 19/25] isl: fix VA64 support for double and dvecN vertex attributes

2016-12-16 Thread Juan A. Suarez Romero
From: Samuel Iglesias Gonsálvez We use *64*_PASSTHRU formats to upload vertex attributes of 64 bits to avoid conversions. From the BDW PRM, Volume 2d, page 586 (VERTEX_ELEMENT_STATE): "When SourceElementFormat is set to one of the *64*_PASSTHRU formats, 64-bit

[Mesa-dev] [PATCH v2 18/25] anv/pipeline: get map for double input attributes

2016-12-16 Thread Juan A. Suarez Romero
--- src/intel/vulkan/anv_pipeline.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/intel/vulkan/anv_pipeline.c b/src/intel/vulkan/anv_pipeline.c index 30ac19a..6a141b6 100644 --- a/src/intel/vulkan/anv_pipeline.c +++ b/src/intel/vulkan/anv_pipeline.c @@ -489,6 +489,7 @@

[Mesa-dev] [PATCH v2 16/25] spirv/nir: add (un)packDouble2x32() translation

2016-12-16 Thread Juan A. Suarez Romero
From: Samuel Iglesias Gonsálvez Signed-off-by: Samuel Iglesias Gonsálvez --- src/compiler/spirv/vtn_glsl450.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/compiler/spirv/vtn_glsl450.c b/src/compiler/spirv/vtn_glsl450.c index

[Mesa-dev] [PATCH v2 15/25] spirv/nir: implement DF conversions

2016-12-16 Thread Juan A. Suarez Romero
From: Samuel Iglesias Gonsálvez SPIR-V does not have special opcodes for DF conversions. We need to identify them by checking the bit size of the operand and the result. Signed-off-by: Samuel Iglesias Gonsálvez --- src/compiler/spirv/spirv_to_nir.c

[Mesa-dev] [PATCH v2 17/25] spirv: add support for doubles to OpSpecConstant

2016-12-16 Thread Juan A. Suarez Romero
From: Samuel Iglesias Gonsálvez Signed-off-by: Samuel Iglesias Gonsálvez --- src/amd/vulkan/radv_pipeline.c| 5 +++- src/compiler/spirv/nir_spirv.h| 5 +++- src/compiler/spirv/spirv_to_nir.c | 51 +++

[Mesa-dev] [PATCH v2 14/25] nir: add nir_type_conversion_op()

2016-12-16 Thread Juan A. Suarez Romero
From: Samuel Iglesias Gonsálvez This function returns the nir_op corresponding to the conversion between the given nir_alu_type arguments. This function lacks support for integer-based types with bit_size != 32 and for float16 conversion ops. Signed-off-by: Samuel

[Mesa-dev] [PATCH v2 13/25] nir: add nir_get_nir_type_for_glsl_type()

2016-12-16 Thread Juan A. Suarez Romero
From: Samuel Iglesias Gonsálvez Signed-off-by: Samuel Iglesias Gonsálvez --- src/compiler/nir/nir.h | 19 +++ 1 file changed, 19 insertions(+) diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h index 544d4ba..9310dab 100644

[Mesa-dev] [PATCH v2 09/25] spirv: add double support to _vtn_block_load_store()

2016-12-16 Thread Juan A. Suarez Romero
From: Samuel Iglesias Gonsálvez Signed-off-by: Samuel Iglesias Gonsálvez --- src/compiler/spirv/vtn_variables.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/compiler/spirv/vtn_variables.c b/src/compiler/spirv/vtn_variables.c index

[Mesa-dev] [PATCH v2 12/25] compiler/nir: add glsl_type_is_{float, integer}()

2016-12-16 Thread Juan A. Suarez Romero
From: Samuel Iglesias Gonsálvez Signed-off-by: Samuel Iglesias Gonsálvez --- src/compiler/nir_types.cpp | 15 +++ src/compiler/nir_types.h | 2 ++ 2 files changed, 17 insertions(+) diff --git a/src/compiler/nir_types.cpp

[Mesa-dev] [PATCH v2 11/25] spirv: add support for doubles on OpComposite{Insert, Extract}

2016-12-16 Thread Juan A. Suarez Romero
From: Samuel Iglesias Gonsálvez Signed-off-by: Samuel Iglesias Gonsálvez --- src/compiler/spirv/spirv_to_nir.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/compiler/spirv/spirv_to_nir.c b/src/compiler/spirv/spirv_to_nir.c index

[Mesa-dev] [PATCH v2 10/25] spirv: Enable double floating points when copying variables in _vtn_variable_copy()

2016-12-16 Thread Juan A. Suarez Romero
From: Samuel Iglesias Gonsálvez Signed-off-by: Samuel Iglesias Gonsálvez --- src/compiler/spirv/vtn_variables.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/compiler/spirv/vtn_variables.c b/src/compiler/spirv/vtn_variables.c index

[Mesa-dev] [PATCH v2 08/25] spirv: add double support to _vtn_variable_load_store

2016-12-16 Thread Juan A. Suarez Romero
From: Samuel Iglesias Gonsálvez Signed-off-by: Samuel Iglesias Gonsálvez --- src/compiler/spirv/vtn_variables.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/compiler/spirv/vtn_variables.c b/src/compiler/spirv/vtn_variables.c index

[Mesa-dev] [PATCH v2 07/25] spirv: add double support to SpvOpCompositeExtract

2016-12-16 Thread Juan A. Suarez Romero
From: Samuel Iglesias Gonsálvez Signed-off-by: Samuel Iglesias Gonsálvez --- src/compiler/spirv/spirv_to_nir.c | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/src/compiler/spirv/spirv_to_nir.c

[Mesa-dev] [PATCH v2 03/25] spirv: add support for loading DF constants

2016-12-16 Thread Juan A. Suarez Romero
From: Samuel Iglesias Gonsálvez Signed-off-by: Samuel Iglesias Gonsálvez --- src/compiler/spirv/spirv_to_nir.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/src/compiler/spirv/spirv_to_nir.c

[Mesa-dev] [PATCH v2 04/25] spirv: add DF support to vtn_const_ssa_value()

2016-12-16 Thread Juan A. Suarez Romero
From: Samuel Iglesias Gonsálvez Signed-off-by: Samuel Iglesias Gonsálvez --- src/compiler/spirv/spirv_to_nir.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/compiler/spirv/spirv_to_nir.c

[Mesa-dev] [PATCH v2 02/25] spirv: add definition of double based data types

2016-12-16 Thread Juan A. Suarez Romero
From: Samuel Iglesias Gonsálvez Signed-off-by: Samuel Iglesias Gonsálvez --- src/compiler/spirv/spirv_to_nir.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/compiler/spirv/spirv_to_nir.c

[Mesa-dev] [PATCH v2 01/25] spirv: fix typo in spec_constant_decoration_cb()

2016-12-16 Thread Juan A. Suarez Romero
From: Samuel Iglesias Gonsálvez Signed-off-by: Samuel Iglesias Gonsálvez --- src/compiler/spirv/spirv_to_nir.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/compiler/spirv/spirv_to_nir.c

[Mesa-dev] [PATCH v2 00/25] Enable Float64 capability support for Intel's Vulkan driver

2016-12-16 Thread Juan A. Suarez Romero
This patch series is a second iteration of previous one: https://lists.freedesktop.org/archives/mesa-dev/2016-November/136507.html Main changes are the ones suggested by Jason, and also a refactor of the way inputs_read bitmap is used in NIR. If you want to test these patches, you can clone our

Re: [Mesa-dev] radv vs vulkan loader

2016-12-16 Thread Emil Velikov
On 5 October 2016 at 23:12, Gustaw Smolarczyk wrote: > 2016-10-06 0:05 GMT+02:00 Emil Velikov : >> On 5 October 2016 at 21:45, Gustaw Smolarczyk wrote: >>> Hello, >>> >>> I have encountered a following problem while trying to

Re: [Mesa-dev] [PATCH 16/19] glsl: Silence "unused parameter" warnings in ast_type.cpp

2016-12-16 Thread Andres Gomez
Ugh! Thanks for this. This is Reviewed-by: Andres Gomez Br. On Thu, 2016-12-15 at 20:10 -0800, Ian Romanick wrote: > From: Ian Romanick > > glsl/ast_type.cpp: In function ‘bool validate_point_mode(YYLTYPE*, > _mesa_glsl_parse_state*, const

[Mesa-dev] [PATCH 5/5] st/xvmc: error handling

2016-12-16 Thread Nayan Deshmukh
vl_compositor_set_csc_matrix may return false Signed-off-by: Nayan Deshmukh --- src/gallium/state_trackers/xvmc/attributes.c | 3 ++- src/gallium/state_trackers/xvmc/context.c| 4 +++- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git

[Mesa-dev] [PATCH 4/5] st/va: error handling

2016-12-16 Thread Nayan Deshmukh
handle the cases when vl_compositor_set_csc_matrix(), vl_compositor_init_state() and vl_compositor_init() fail Signed-off-by: Nayan Deshmukh --- src/gallium/state_trackers/va/context.c | 18 +++--- 1 file changed, 15 insertions(+), 3 deletions(-) diff

[Mesa-dev] [PATCH 3/5] st/vdpau: error handling

2016-12-16 Thread Nayan Deshmukh
handle the cases when vl_compositor_set_csc_matrix(), vl_compositor_init_state() and vl_compositor_init() fail Signed-off-by: Nayan Deshmukh --- src/gallium/state_trackers/vdpau/device.c | 8 +- src/gallium/state_trackers/vdpau/mixer.c | 43

[Mesa-dev] [PATCH 1/5] st/xvmc: use goto to make code more readable (v2)

2016-12-16 Thread Nayan Deshmukh
v2: add missing semicolon Signed-off-by: Nayan Deshmukh --- src/gallium/state_trackers/xvmc/context.c | 40 +++ 1 file changed, 19 insertions(+), 21 deletions(-) diff --git a/src/gallium/state_trackers/xvmc/context.c

[Mesa-dev] [PATCH 2/5] vl/compositor: implement error handling

2016-12-16 Thread Nayan Deshmukh
pipe_buffer_map and pipe_buffer_create may return NULL Signed-off-by: Nayan Deshmukh --- src/gallium/auxiliary/vl/vl_compositor.c | 13 +++-- src/gallium/auxiliary/vl/vl_compositor.h | 2 +- 2 files changed, 12 insertions(+), 3 deletions(-) diff --git

Re: [Mesa-dev] [PATCH 6/6] Revert "cso: don't release sampler states that are bound"

2016-12-16 Thread Nicolai Hähnle
Thanks for taking care of this! Patches 4-6 are also: Reviewed-by: Nicolai Hähnle On 16.12.2016 10:52, Michel Dänzer wrote: From: Michel Dänzer This reverts commit 6dc96de303290e8d1fc294da478c4f370be98dea. No longer necessary with the

Re: [Mesa-dev] [PATCH 3/6] cso: Optimize cso_save/restore_fragment_samplers

2016-12-16 Thread Nicolai Hähnle
On 16.12.2016 10:52, Michel Dänzer wrote: From: Michel Dänzer Only copy/memset the pointers that actually need to be. Signed-off-by: Michel Dänzer --- src/gallium/auxiliary/cso_cache/cso_context.c | 21 + 1 file changed, 17

Re: [Mesa-dev] [PATCH 2/6] cso: Store pointers to struct cso_sampler in struct sampler_info

2016-12-16 Thread Nicolai Hähnle
On 16.12.2016 10:52, Michel Dänzer wrote: From: Michel Dänzer Preparation for following changes, no functional change intended. Signed-off-by: Michel Dänzer --- src/gallium/auxiliary/cso_cache/cso_context.c | 33 +++ 1

Re: [Mesa-dev] [PATCH 1/6] cso: Don't restore nr_samplers in cso_restore_fragment_samplers

2016-12-16 Thread Nicolai Hähnle
On 16.12.2016 10:52, Michel Dänzer wrote: From: Michel Dänzer If info->nr_samplers > ctx->nr_fragment_samplers_saved, the assignment would prevent cso_single_sampler_done from unbinding the no longer used samplers from the driver, which could result in use-after-free.

Re: [Mesa-dev] [PATCH] radeonsi: Use build_buffer_load helper function for geometry shaders

2016-12-16 Thread Nicolai Hähnle
On 15.12.2016 21:35, Tom Stellard wrote: Also add a need_range_checks parameter to this function, which can be set to false to enable some additional optimizations. Currently, this will cause the compiler to emit the llvm.SI.buffer.load.dword intrinsic instead of llvm.amdgcn.buffer.load.

Re: [Mesa-dev] [PATCH] radeonsi: Set datalayout on the llvm module

2016-12-16 Thread Nicolai Hähnle
On 15.12.2016 19:11, Tom Stellard wrote: This prevents LLVM from using sext instructions for local memory offsets and allows the backend to fold immediate offsets into the instruction. This also prevents some incorrect code generation for ptrtoint and inttoptr instructions. Reviewed-by:

[Mesa-dev] [PATCH 5/6] cso: Make sanitize_hash safe for samplers

2016-12-16 Thread Michel Dänzer
From: Michel Dänzer Remove currently bound sampler states from the hash table before pruning entries from the hash table, so currently bound states cannot accidentally be deleted by the pruning. Signed-off-by: Michel Dänzer ---

[Mesa-dev] [PATCH 2/6] cso: Store pointers to struct cso_sampler in struct sampler_info

2016-12-16 Thread Michel Dänzer
From: Michel Dänzer Preparation for following changes, no functional change intended. Signed-off-by: Michel Dänzer --- src/gallium/auxiliary/cso_cache/cso_context.c | 33 +++ 1 file changed, 18 insertions(+), 15

[Mesa-dev] [PATCH 0/6] cso: Enforce maximum size of sampler cache again

2016-12-16 Thread Michel Dänzer
From: Michel Dänzer It turned out to be slightly more complicated than I'd expected, but I think I've found a good solution with low (hopefully insignificant) additional overhead. In fact, thanks to the optimization in patch 3, the overhead may be slightly lower in

[Mesa-dev] [PATCH 1/6] cso: Don't restore nr_samplers in cso_restore_fragment_samplers

2016-12-16 Thread Michel Dänzer
From: Michel Dänzer If info->nr_samplers > ctx->nr_fragment_samplers_saved, the assignment would prevent cso_single_sampler_done from unbinding the no longer used samplers from the driver, which could result in use-after-free. This is probably unlikely to happen in

[Mesa-dev] [PATCH 3/6] cso: Optimize cso_save/restore_fragment_samplers

2016-12-16 Thread Michel Dänzer
From: Michel Dänzer Only copy/memset the pointers that actually need to be. Signed-off-by: Michel Dänzer --- src/gallium/auxiliary/cso_cache/cso_context.c | 21 + 1 file changed, 17 insertions(+), 4 deletions(-) diff --git

[Mesa-dev] [PATCH 6/6] Revert "cso: don't release sampler states that are bound"

2016-12-16 Thread Michel Dänzer
From: Michel Dänzer This reverts commit 6dc96de303290e8d1fc294da478c4f370be98dea. No longer necessary with the previous change. Signed-off-by: Michel Dänzer --- src/gallium/auxiliary/cso_cache/cso_cache.c | 4 +--- 1 file changed, 1

[Mesa-dev] [PATCH 4/6] cso: Store hash key in struct cso_sampler

2016-12-16 Thread Michel Dänzer
From: Michel Dänzer Preparation for following changes, no functional change intended. Signed-off-by: Michel Dänzer --- src/gallium/auxiliary/cso_cache/cso_cache.h | 1 + src/gallium/auxiliary/cso_cache/cso_context.c | 1 + 2 files changed, 2

[Mesa-dev] [Bug 99027] gtk+ apps segfault on Xvfb with mesa 13

2016-12-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99027 Timo Aaltonen changed: What|Removed |Added Status|REOPENED|RESOLVED