Re: [Mesa-dev] [PATCH] gbm: add support for loading third-party backend

2017-01-25 Thread Michel Dänzer
On 24/01/17 12:36 PM, Qiang Yu wrote: > Third-party can put their backend to a directory configured with > '--with-gbm-backenddir' and create a /etc/gbm.conf.d/*.conf file > which contains the backend so file name to overwrite the default > builtin DRI backend. > > The /etc/gbm.conf.d/*.conf will

Re: [Mesa-dev] i965/gen6+: Yet another blorp path - tex_(sub)image2d

2017-01-25 Thread Kenneth Graunke
On Tuesday, December 20, 2016 4:45:28 PM PST Topi Pohjolainen wrote: > This series introduces new use of brw_blorp_blit_miptrees()/ > brw_blorp_copy_miptrees(). Initial intention was to enable compression > on SKL already at the time of upload. That, however, didn't help > benchmarks but quite

Re: [Mesa-dev] [PATCH] gallium/radeon: add a new HUD query for the number of mapped buffers

2017-01-25 Thread Christian König
Am 25.01.2017 um 03:56 schrieb Michel Dänzer: On 25/01/17 12:05 AM, Marek Olšák wrote: On Tue, Jan 24, 2017 at 2:17 PM, Christian König wrote: Am 24.01.2017 um 11:44 schrieb Samuel Pitoiset: On 01/24/2017 11:38 AM, Nicolai Hähnle wrote: On 24.01.2017 11:34, Samuel

Re: [Mesa-dev] [PATCH] mesa: Fix copy-and-paste bug in _mesa_(Program|)Uniform[1234](i|ui)64vARB functions

2017-01-25 Thread Nicolai Hähnle
On 25.01.2017 01:17, Ian Romanick wrote: From: Ian Romanick All of the functions were passing 1 to _mesa_uniform instead of passing count. Fixes 16 unsed parameter warnings like: main/uniforms.c: In function ‘_mesa_Uniform1i64vARB’: main/uniforms.c:1692:47: warning:

Re: [Mesa-dev] [PATCH 0/5] anv: Implement VK_KHR_get_physical_device_properties2 (v2)

2017-01-25 Thread Chad Versace
On Wed 25 Jan 2017, Lionel Landwerlin wrote: > Looks good to me : > > Reviewed-by: Lionel Landwerlin > > On 25/01/17 20:12, Chad Versace wrote: > > Untested, but builds. I wanted to get feedback on this approach before > > going through the trouble of fetching and

Re: [Mesa-dev] [PATCH 2/3] spirv: add default handler for new enums

2017-01-25 Thread Bas Nieuwenhuizen
Series is Reviewed-by: Bas Nieuwenhuizen On Wed, Jan 25, 2017, at 06:05, Lionel Landwerlin wrote: > Signed-off-by: Lionel Landwerlin > --- > src/compiler/spirv/spirv_to_nir.c | 12 > src/compiler/spirv/vtn_variables.c | 3

[Mesa-dev] [PATCH 2/3] spirv: add default handler for new enums

2017-01-25 Thread Lionel Landwerlin
Signed-off-by: Lionel Landwerlin --- src/compiler/spirv/spirv_to_nir.c | 12 src/compiler/spirv/vtn_variables.c | 3 +++ 2 files changed, 15 insertions(+) diff --git a/src/compiler/spirv/spirv_to_nir.c b/src/compiler/spirv/spirv_to_nir.c index

[Mesa-dev] [PATCH 1/3] spirv: fix typos

2017-01-25 Thread Lionel Landwerlin
Signed-off-by: Lionel Landwerlin --- src/compiler/spirv/spirv_to_nir.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/compiler/spirv/spirv_to_nir.c b/src/compiler/spirv/spirv_to_nir.c index aecda17271..968502c5fd 100644 ---

[Mesa-dev] [PATCH 2/7] vl/video_buffer: add support for P016

2017-01-25 Thread Christian König
From: Christian König Just simply the description of the planes. Signed-off-by: Christian König --- src/gallium/auxiliary/vl/vl_video_buffer.c | 10 ++ 1 file changed, 10 insertions(+) diff --git

[Mesa-dev] [PATCH 5/7] st/vdpau: always use the prefered surface format for decode

2017-01-25 Thread Christian König
From: Christian König That allows us to switch between different surface formats for different codecs. Signed-off-by: Christian König --- src/gallium/state_trackers/vdpau/decode.c | 11 ++- 1 file changed, 6 insertions(+), 5

[Mesa-dev] [PATCH 3/7] radeon/uvd: enable 10bit HEVC decode

2017-01-25 Thread Christian König
From: Christian König Just use whatever the state tracker allocated. Signed-off-by: Christian König --- src/gallium/drivers/radeon/radeon_uvd.c | 5 ++--- src/gallium/drivers/radeon/radeon_video.c | 11 ++- 2 files changed, 12

[Mesa-dev] [PATCH 4/7] st/vdpau: add support for 16bit interop

2017-01-25 Thread Christian König
From: Christian König That should make it possible to use the 16bit surfaces in OpenGL as well. Signed-off-by: Christian König --- src/gallium/include/state_tracker/vdpau_dmabuf.h | 2 ++ src/gallium/include/state_tracker/vdpau_funcs.h |

[Mesa-dev] [PATCH 1/7] gallium: add P016 format

2017-01-25 Thread Christian König
From: Christian König Same layout as NV12, but 16bit per channel instead of 8. Signed-off-by: Christian König --- src/gallium/auxiliary/util/u_format.csv | 2 ++ src/gallium/auxiliary/util/u_format_yuv.c | 19 +++

[Mesa-dev] [PATCH 6/7] st/va: cleanup error handling in vlVaCreateSurfaces2

2017-01-25 Thread Christian König
From: Christian König No need to have that twice. Signed-off-by: Christian König --- src/gallium/state_trackers/va/surface.c | 52 + 1 file changed, 27 insertions(+), 25 deletions(-) diff --git

[Mesa-dev] [PATCH 7/7] st/va: start of P016 support

2017-01-25 Thread Christian König
From: Christian König Most likely only partially correct, but at least a start. Signed-off-by: Christian König --- src/gallium/state_trackers/va/config.c | 9 ++--- src/gallium/state_trackers/va/image.c | 10 ++

[Mesa-dev] 10bit HEVC decoding for RadeonSI

2017-01-25 Thread Christian König
Hi guys, ok this is completely work in progress and untested except for a compile run. Most of the stuff necessary should be there for VDPAU, but I'm honestly not sure how to approach VAAPI. My main problem at the moment is that I can't get mpv/ffmpeg to decode Main10 content using VDPAU and

[Mesa-dev] Mesa 13.0.4 release candidate

2017-01-25 Thread Emil Velikov
Hello list, The candidate for the Mesa 13.0.4 is now available. Currently we have: - 61 queued - 18 nominated (outstanding) - and 2 rejected patch(es) With this series we have - multiple fixes for the i965 and radeonsi drivers. An odd glitch in glxgears and freedreno was resolved, with the

Re: [Mesa-dev] [PATCH] gallium/radeon: add a new HUD query for the number of mapped buffers

2017-01-25 Thread Samuel Pitoiset
On 01/25/2017 03:55 PM, Christian König wrote: Am 25.01.2017 um 15:19 schrieb Samuel Pitoiset: On 01/25/2017 03:56 AM, Michel Dänzer wrote: On 25/01/17 12:05 AM, Marek Olšák wrote: On Tue, Jan 24, 2017 at 2:17 PM, Christian König wrote: Am 24.01.2017 um 11:44

Re: [Mesa-dev] [PATCH] gallium/radeon: add a new HUD query for the number of mapped buffers

2017-01-25 Thread Samuel Pitoiset
On 01/25/2017 03:56 AM, Michel Dänzer wrote: On 25/01/17 12:05 AM, Marek Olšák wrote: On Tue, Jan 24, 2017 at 2:17 PM, Christian König wrote: Am 24.01.2017 um 11:44 schrieb Samuel Pitoiset: On 01/24/2017 11:38 AM, Nicolai Hähnle wrote: On 24.01.2017 11:34, Samuel

Re: [Mesa-dev] [PATCH] gallium/radeon: add a new HUD query for the number of mapped buffers

2017-01-25 Thread Christian König
Am 25.01.2017 um 16:01 schrieb Samuel Pitoiset: On 01/25/2017 03:55 PM, Christian König wrote: Am 25.01.2017 um 15:19 schrieb Samuel Pitoiset: On 01/25/2017 03:56 AM, Michel Dänzer wrote: On 25/01/17 12:05 AM, Marek Olšák wrote: On Tue, Jan 24, 2017 at 2:17 PM, Christian König

[Mesa-dev] [ANNOUNCE] mesa 17.0.0-rc2

2017-01-25 Thread Emil Velikov
The second release candidate for Mesa 17.0.0 is now available. Andres Rodriguez (2): vulkan/wsi: clarify the severity of lack of DRI3 v2 radv: fix include order for installed headers v2 Bruce Cherniak (1): swr: Prune empty nodes in CalculateProcessorTopology. Dave Airlie (1):

Re: [Mesa-dev] [PATCH 2/2] spirv: bump headers to SPIRV 1.1

2017-01-25 Thread Lionel Landwerlin
On 25/01/17 13:13, Bas Nieuwenhuizen wrote: On Wed, Jan 25, 2017, at 05:06, Lionel Landwerlin wrote: Just to confirm, are you fine with the header update triggering warnings fixed by the following commit? Can't you update the switches before updating the header? That would avoid any warnings

[Mesa-dev] [PATCH v2 3/3] spirv: bump headers to SPIRV 1.1

2017-01-25 Thread Lionel Landwerlin
Signed-off-by: Lionel Landwerlin --- src/compiler/spirv/GLSL.std.450.h | 12 -- src/compiler/spirv/spirv.h| 77 --- src/compiler/spirv/spirv_info.c | 6 +++ 3 files changed, 86 insertions(+), 9 deletions(-) diff

Re: [Mesa-dev] [PATCH] gallium/radeon: add a new HUD query for the number of mapped buffers

2017-01-25 Thread Samuel Pitoiset
On 01/25/2017 04:01 PM, Samuel Pitoiset wrote: On 01/25/2017 03:55 PM, Christian König wrote: Am 25.01.2017 um 15:19 schrieb Samuel Pitoiset: On 01/25/2017 03:56 AM, Michel Dänzer wrote: On 25/01/17 12:05 AM, Marek Olšák wrote: On Tue, Jan 24, 2017 at 2:17 PM, Christian König

Re: [Mesa-dev] [PATCH] gallium/radeon: add a new HUD query for the number of mapped buffers

2017-01-25 Thread Christian König
Am 25.01.2017 um 15:19 schrieb Samuel Pitoiset: On 01/25/2017 03:56 AM, Michel Dänzer wrote: On 25/01/17 12:05 AM, Marek Olšák wrote: On Tue, Jan 24, 2017 at 2:17 PM, Christian König wrote: Am 24.01.2017 um 11:44 schrieb Samuel Pitoiset: On 01/24/2017 11:38 AM,

[Mesa-dev] [PATCH] anv/lower_input_attachments: honor sample index parameter to subpassLoad()

2017-01-25 Thread Iago Toral Quiroga
According to GL_KHR_vulkan_glsl, the signature of subpassLoad() is: gvec4 subpassLoad(gsubpassInput subpass); gvec4 subpassLoad(gsubpassInputMS subpass, int sample); So the multisampled case always receives an explicit sample index that we should use. The current implementation was ignoring

Re: [Mesa-dev] [PATCH 2/2] spirv: bump headers to SPIRV 1.1

2017-01-25 Thread Bas Nieuwenhuizen
On Wed, Jan 25, 2017, at 05:06, Lionel Landwerlin wrote: > Just to confirm, are you fine with the header update triggering warnings > fixed by the following commit? Can't you update the switches before updating the header? That would avoid any warnings unless I'm missing stuff. > > Thanks!

Re: [Mesa-dev] [PATCH 2/2] spirv: bump headers to SPIRV 1.1

2017-01-25 Thread Bas Nieuwenhuizen
Can you split the switch changes to a different commit? With that, Reviewed-by: Bas Nieuwenhuizen for the series. On Wed, Jan 25, 2017, at 04:39, Lionel Landwerlin wrote: > Signed-off-by: Lionel Landwerlin > --- >

Re: [Mesa-dev] [PATCH] vulkan/wsi: Lower the maximum image sizes

2017-01-25 Thread Lionel Landwerlin
Reviewed-by: Lionel Landwerlin On 25/01/17 01:16, Jason Ekstrand wrote: --- src/vulkan/wsi/wsi_common_wayland.c | 3 ++- src/vulkan/wsi/wsi_common_x11.c | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git

Re: [Mesa-dev] [PATCH 2/2] vulkan/wsi/wayland: Handle VK_INCOMPLETE for GetPresentModes

2017-01-25 Thread Lionel Landwerlin
Reviewed-by: Lionel Landwerlin On 25/01/17 00:44, Jason Ekstrand wrote: --- src/vulkan/wsi/wsi_common_wayland.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/vulkan/wsi/wsi_common_wayland.c

Re: [Mesa-dev] [PATCH 1/2] vulkan/wsi/wayland: Handle VK_INCOMPLETE for GetFormats

2017-01-25 Thread Lionel Landwerlin
Reviewed-by: Lionel Landwerlin On 25/01/17 00:44, Jason Ekstrand wrote: --- src/vulkan/wsi/wsi_common_wayland.c | 16 +--- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/src/vulkan/wsi/wsi_common_wayland.c

Re: [Mesa-dev] [PATCH 1/4] mesa: Trivial clean-ups in uniform_query.cpp

2017-01-25 Thread Manolova, Plamena
Looks good to me :) Reviewed-by: Plamena Manolova < plamena.manol...@intel.com> On Wed, Jan 25, 2017 at 1:30 AM, Ian Romanick wrote: > From: Ian Romanick > > This is C++, so we can mix code and declarations. Doing so allows > constification. > >

Re: [Mesa-dev] [PATCH 2/2] spirv: bump headers to SPIRV 1.1

2017-01-25 Thread Lionel Landwerlin
Just to confirm, are you fine with the header update triggering warnings fixed by the following commit? Thanks! On 25/01/17 12:45, Bas Nieuwenhuizen wrote: Can you split the switch changes to a different commit? With that, Reviewed-by: Bas Nieuwenhuizen for the

[Mesa-dev] [PATCH] i965/fs: don't lower different type size conversions on opt_peephole_sel()

2017-01-25 Thread Samuel Iglesias Gonsálvez
Don't lower a type conversion between different type sizes because SEL does't support them, SEL without conditional modifier just do a raw move. Signed-off-by: Samuel Iglesias Gonsálvez --- src/mesa/drivers/dri/i965/brw_fs_sel_peephole.cpp | 2 ++ 1 file changed, 2

Re: [Mesa-dev] [PATCH] gbm: add support for loading third-party backend

2017-01-25 Thread Yu, Qiang
Right, the order can only be overwrite with GBM_BACKEND environment variable. If we make the DRI backend also configurable by the conf file or give a "priority" property to a backend in conf and assign a default priority to DRI backend, your worry can be addressed. Regards, Qiang

[Mesa-dev] [PATCH 0/2] Spirv: update headers to 1.1

2017-01-25 Thread Lionel Landwerlin
Hi all, Vulkan 1.0.39 was released earlier this week. The specification list a number of new extensions. In particular the VK_KHR_shader_draw_parameters extension requires the SPV_KHR_shader_draw_parameters extension which seems to be part of SPIRV 1.1. Here are patches to update the headers as

[Mesa-dev] [PATCH 1/2] spirv: fix typos

2017-01-25 Thread Lionel Landwerlin
Signed-off-by: Lionel Landwerlin --- src/compiler/spirv/spirv_to_nir.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/compiler/spirv/spirv_to_nir.c b/src/compiler/spirv/spirv_to_nir.c index aecda17271..968502c5fd 100644 ---

[Mesa-dev] [PATCH 2/2] spirv: bump headers to SPIRV 1.1

2017-01-25 Thread Lionel Landwerlin
Signed-off-by: Lionel Landwerlin --- src/compiler/spirv/GLSL.std.450.h | 12 -- src/compiler/spirv/spirv.h | 77 +++--- src/compiler/spirv/spirv_info.c| 6 +++ src/compiler/spirv/spirv_to_nir.c | 12 ++

[Mesa-dev] [Bug 98223] dEQP GLES3.1 program_interface_query failures w/ error "could not find target resource"

2017-01-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98223 Kenneth Graunke changed: What|Removed |Added Status|NEW |RESOLVED

[Mesa-dev] [PATCH 2/2] gallium/radeon: add VRAM-vis-usage HUD query

2017-01-25 Thread Samuel Pitoiset
This new query returns the current visible usage of VRAM accessed by the CPU. It will return 0 on radeon because it's unimplemented. Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/radeon/r600_query.c | 7 +++

[Mesa-dev] [PATCH] anv: set command buffer to NULL when allocations fail

2017-01-25 Thread Lionel Landwerlin
The spec section 5.2 says: "vkAllocateCommandBuffers can be used to create multiple command buffers. If the creation of any of those command buffers fails, the implementation must destroy all successfully created command buffer objects from this command, set all entries of the

Re: [Mesa-dev] [PATCH] anv/lower_input_attachments: honor sample index parameter to subpassLoad()

2017-01-25 Thread Jason Ekstrand
Oops... Reviewed-by: Jason Ekstrand Cc: "17.0" On Wed, Jan 25, 2017 at 7:28 AM, Iago Toral Quiroga wrote: > According to GL_KHR_vulkan_glsl, the signature of subpassLoad() is: > > gvec4 subpassLoad(gsubpassInput

[Mesa-dev] [PATCH 1/2] gallium/radeon: query the CPU accessible size of VRAM

2017-01-25 Thread Samuel Pitoiset
R600_DEBUG="info" can be used to display that size, as well as the total amount of VRAM/GTT. Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/radeon/r600_pipe_common.c | 1 + src/gallium/drivers/radeon/radeon_winsys.h| 1 +

Re: [Mesa-dev] [PATCH mesa 1/2] egl: update headers from registry

2017-01-25 Thread Emil Velikov
On 24 January 2017 at 18:07, Eric Engestrom wrote: > Khronos introduced a new macro (suggested by Google) to avoid using > C-style casts in C++ code, as those generate warnings. > > Khronos Bugzilla: https://cvs.khronos.org/bugzilla/show_bug.cgi?id=16113 >

Re: [Mesa-dev] [PATCH] gallium/radeon: add a new HUD query for the number of mapped buffers

2017-01-25 Thread Christian König
Am 25.01.2017 um 16:12 schrieb Samuel Pitoiset: On 01/25/2017 04:01 PM, Samuel Pitoiset wrote: On 01/25/2017 03:55 PM, Christian König wrote: Am 25.01.2017 um 15:19 schrieb Samuel Pitoiset: On 01/25/2017 03:56 AM, Michel Dänzer wrote: On 25/01/17 12:05 AM, Marek Olšák wrote: On Tue,

Re: [Mesa-dev] [PATCH] anv: set command buffer to NULL when allocations fail

2017-01-25 Thread Jason Ekstrand
I thought I looked for this when doing maintenance1... oh well. Reviewed-by: Jason Ekstrand Cc: "13.0 17.0" On Wed, Jan 25, 2017 at 8:29 AM, Lionel Landwerlin < lionel.g.landwer...@intel.com> wrote: > The spec section 5.2 says: > >

Re: [Mesa-dev] [PATCH] anv: set command buffer to NULL when allocations fail

2017-01-25 Thread Jason Ekstrand
On Wed, Jan 25, 2017 at 8:29 AM, Lionel Landwerlin < lionel.g.landwer...@intel.com> wrote: > The spec section 5.2 says: > >"vkAllocateCommandBuffers can be used to create multiple command >buffers. If the creation of any of those command buffers fails, the >implementation must destroy

Re: [Mesa-dev] [PATCH 2/2] spirv: bump headers to SPIRV 1.1

2017-01-25 Thread Jason Ekstrand
On Wed, Jan 25, 2017 at 4:45 AM, Bas Nieuwenhuizen wrote: > Can you split the switch changes to a different commit? With that, > Agreed, Reviewed-by: Jason Ekstrand > Reviewed-by: Bas Nieuwenhuizen > > for the

[Mesa-dev] [Bug 97879] [amdgpu] Rocket League: long hangs (several seconds) when loading assets (models/textures/shaders?)

2017-01-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97879 --- Comment #55 from Timothee Besset --- Rocket League uses an async I/O thread to stream in content in the background. That thread works off of a queue and only does disk reads and decompression (it doesn't issue any OpenGL

Re: [Mesa-dev] Can we support llvm-4.0 with mesa-13.0.x?

2017-01-25 Thread Roland Scheidegger
mesa 17.0 is scheduled to be released earlier than llvm-4.0, so imho no, not worth the effort. I certainly won't stop anyone from porting the necessary changes and doing some minimal testing, though... Roland Am 25.01.2017 um 17:47 schrieb Nicolai Hähnle: > I think we can consider it, but note

[Mesa-dev] Can we support llvm-4.0 with mesa-13.0.x?

2017-01-25 Thread Matt Turner
To make distributions lives easier, can we please support llvm-4.0 in the mesa-13.0 branch? It currently fails to compile: https://bugs.gentoo.org/show_bug.cgi?id=606608 ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org

Re: [Mesa-dev] [PATCH] gallium/radeon: add a new HUD query for the number of mapped buffers

2017-01-25 Thread Samuel Pitoiset
On 01/25/2017 05:05 PM, Christian König wrote: Am 25.01.2017 um 16:12 schrieb Samuel Pitoiset: On 01/25/2017 04:01 PM, Samuel Pitoiset wrote: On 01/25/2017 03:55 PM, Christian König wrote: Am 25.01.2017 um 15:19 schrieb Samuel Pitoiset: On 01/25/2017 03:56 AM, Michel Dänzer wrote:

Re: [Mesa-dev] Can we support llvm-4.0 with mesa-13.0.x?

2017-01-25 Thread Nicolai Hähnle
I think we can consider it, but note that it's not entirely trivial. In addition to compile fixes, there were changes in intrinsics that are relevant for radeonsi. Nicolai On 25.01.2017 17:19, Matt Turner wrote: To make distributions lives easier, can we please support llvm-4.0 in the

Re: [Mesa-dev] [PATCH 2/2] gallium/radeon: add VRAM-vis-usage HUD query

2017-01-25 Thread Nicolai Hähnle
Reviewed-by: Nicolai Hähnle On 25.01.2017 16:56, Samuel Pitoiset wrote: This new query returns the current visible usage of VRAM accessed by the CPU. It will return 0 on radeon because it's unimplemented. Signed-off-by: Samuel Pitoiset ---

Re: [Mesa-dev] [PATCH mesa 1/2] egl: update headers from registry

2017-01-25 Thread Chad Versace
On Tue 24 Jan 2017, Eric Engestrom wrote: > Khronos introduced a new macro (suggested by Google) to avoid using > C-style casts in C++ code, as those generate warnings. > > Khronos Bugzilla: https://cvs.khronos.org/bugzilla/show_bug.cgi?id=16113 > Signed-off-by: Eric Engestrom

[Mesa-dev] [PATCH] anv: Revive struct anv_common (v2)

2017-01-25 Thread Chad Versace
The struct was deleted by: commit efe9d1cde3340d3a9d17e5560b609a4fb839d43d Author: Edward O'Callaghan Subject: anv: Clean up some unused variables Unlike the original anv_common, the new one has a non-const pNext pointer because we will use it for the output

Re: [Mesa-dev] [PATCH 5/5] anv: Implement VK_KHR_get_physical_device_properties2

2017-01-25 Thread Jason Ekstrand
Having read through this code, I think I like this approach. It seems to be the most easily extensible thing we can do. Series is Reviewed-by: Jason Ekstrand On Wed, Jan 25, 2017 at 12:12 PM, Chad Versace wrote: > --- >

Re: [Mesa-dev] [PATCH 5/5] anv: Implement VK_KHR_get_physical_device_properties2

2017-01-25 Thread Jason Ekstrand
On Wed, Jan 25, 2017 at 12:12 PM, Chad Versace wrote: > --- > src/intel/vulkan/anv_device.c | 90 ++ > > src/intel/vulkan/anv_formats.c | 53 + > src/intel/vulkan/anv_private.h | 18 + > 3 files

Re: [Mesa-dev] [PATCH 4/8] i965/fs: Fix nir_op_fsign of absolute value.

2017-01-25 Thread Francisco Jerez
Ian Romanick writes: > On 01/24/2017 03:26 PM, Francisco Jerez wrote: >> This does point at the front-end emitting silly code that could have >> been optimized out, but the current fsign implementation would emit >> bogus IR if abs was set for the argument (because it would

Re: [Mesa-dev] [PATCH 2/5] anv: Revive struct anv_common

2017-01-25 Thread Chad Versace
On Wed 25 Jan 2017, Jason Ekstrand wrote: > On Wed, Jan 25, 2017 at 12:12 PM, Chad Versace > wrote: > > The struct was deleted by: >   commit efe9d1cde3340d3a9d17e5560b609a4fb839d43d >   Author: Edward O'Callaghan >  

Re: [Mesa-dev] [PATCH 0/5] anv: Implement VK_KHR_get_physical_device_properties2 (v2)

2017-01-25 Thread Lionel Landwerlin
Looks good to me : Reviewed-by: Lionel Landwerlin On 25/01/17 20:12, Chad Versace wrote: Untested, but builds. I wanted to get feedback on this approach before going through the trouble of fetching and building the newest CTS. If the review feedback is good,

Re: [Mesa-dev] 10bit HEVC decoding for RadeonSI

2017-01-25 Thread Andy Furniss
Christian König wrote: Hi guys, ok this is completely work in progress and untested except for a compile run. Most of the stuff necessary should be there for VDPAU, but I'm honestly not sure how to approach VAAPI. These regress R9 285 8bit h264 vaapi decode with mpv, [vd] Pixel formats

Re: [Mesa-dev] 10bit HEVC decoding for RadeonSI

2017-01-25 Thread Peter Frühberger
Hi Christian, 2017-01-25 15:41 GMT+01:00 Christian König : > Hi guys, > > ok this is completely work in progress and untested except for a compile > run. > > Most of the stuff necessary should be there for VDPAU, but I'm honestly > not sure how to approach VAAPI. > > My

[Mesa-dev] [Bug 97879] [amdgpu] Rocket League: long hangs (several seconds) when loading assets (models/textures/shaders?)

2017-01-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97879 --- Comment #56 from Grazvydas Ignotas --- Is the async I/O thread doing a lot of memory allocations? If so, perhaps the malloc implementation is slowed down by having large amount of small allocs/frees from shader compiles

Re: [Mesa-dev] [PATCH] vulkan: Don't install vk_platform.h or vulkan.h.

2017-01-25 Thread Chad Versace
On Tue 24 Jan 2017, Jason Ekstrand wrote: > On Tue, Jan 24, 2017 at 11:25 AM, Emil Velikov > wrote: > > On 24 January 2017 at 18:02, Jason Ekstrand wrote: > > On Tue, Jan 24, 2017 at 9:03 AM, Matt Turner wrote: >

Re: [Mesa-dev] [PATCH 0/4] Micro optimizations for glUniform and glUniformMatrix

2017-01-25 Thread Jordan Justen
Series Reviewed-by: Jordan Justen On 2017-01-24 17:30:11, Ian Romanick wrote: > These are some patches that I wrote ages ago... the initial versions > pre-date Mesa's ARB_gpu_shader_fp64 support. This was part of a larger > effort that got bogged down and eventually

Re: [Mesa-dev] [PATCH 5/5] anv: Implement VK_KHR_get_physical_device_properties2

2017-01-25 Thread Chad Versace
On Wed 25 Jan 2017, Jason Ekstrand wrote: > > > On Wed, Jan 25, 2017 at 12:12 PM, Chad Versace > wrote: > > +VkResult vkGetPhysicalDeviceImageFormatProperties2KHR( > > > This needs to be anv_ Oops. ___ mesa-dev

[Mesa-dev] [PATCH 1/2] radv: Don't allow any operations on non-supported depth/stencil formats.

2017-01-25 Thread Bas Nieuwenhuizen
We really use the depth block for the blits. Signed-off-by: Bas Nieuwenhuizen --- src/amd/vulkan/radv_formats.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/amd/vulkan/radv_formats.c b/src/amd/vulkan/radv_formats.c index

[Mesa-dev] [PATCH 2/2] radv: Expose transfer format features.

2017-01-25 Thread Bas Nieuwenhuizen
Signed-off-by: Bas Nieuwenhuizen Tested-by: Andres Rodriguez --- src/amd/vulkan/radv_formats.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/src/amd/vulkan/radv_formats.c b/src/amd/vulkan/radv_formats.c index f56f67c400e..c968cefc9c7

Re: [Mesa-dev] [PATCH] anv: set command buffer to NULL when allocations fail

2017-01-25 Thread Chad Versace
On Wed 25 Jan 2017, Lionel Landwerlin wrote: > The spec section 5.2 says: > >"vkAllocateCommandBuffers can be used to create multiple command >buffers. If the creation of any of those command buffers fails, the >implementation must destroy all successfully created command buffer >

[Mesa-dev] [Bug 97879] [amdgpu] Rocket League: long hangs (several seconds) when loading assets (models/textures/shaders?)

2017-01-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97879 Clément Guérin changed: What|Removed |Added CC||geecko@free.fr

Re: [Mesa-dev] [PATCH 1/7] blorp: Handle the RGB workaround more like other workarounds

2017-01-25 Thread Jason Ekstrand
On Wed, Jan 25, 2017 at 10:46 PM, Pohjolainen, Topi < topi.pohjolai...@gmail.com> wrote: > On Tue, Jan 24, 2017 at 03:45:48PM -0800, Jason Ekstrand wrote: > > The previous version was sort-of strapped on in that it just adjusted > > the blit rectangle and trusted in the fact that we would use

Re: [Mesa-dev] [PATCH 2/7] intel/blorp: Handle more exotic destination formats

2017-01-25 Thread Pohjolainen, Topi
On Tue, Jan 24, 2017 at 03:45:49PM -0800, Jason Ekstrand wrote: > This commit adds support for using both R24_UNORM_X8_TYPELESS and > R9G9B9E5_SHAREDEXP as destination formats even though the hardware does > not support rendering to them. This is done by using a different format > and emitting

Re: [Mesa-dev] [PATCH 4/7] intel/blorp: Silently convert RGBX destination formats to RGBA

2017-01-25 Thread Pohjolainen, Topi
On Tue, Jan 24, 2017 at 03:45:51PM -0800, Jason Ekstrand wrote: > --- > src/intel/blorp/blorp_blit.c | 4 > 1 file changed, 4 insertions(+) Patches 3 and 4 are: Reviewed-by: Topi Pohjolainen Patch 2 looks good also. I just need to study the E9-math some more.

[Mesa-dev] [PATCH] i965: Unbind deleted shaders from brw_context, fixing malloc heisenbug.

2017-01-25 Thread Kenneth Graunke
Applications may delete a shader program, create a new one, and bind it before the next draw. With terrible luck, malloc may randomly return a chunk of memory for the new gl_program that happened to be the exact same pointer as our previously bound gl_program. In this case, our logic to detect

Re: [Mesa-dev] [PATCH 7/7] i965/blorp: Remove a pile of blorp_blit restrictions

2017-01-25 Thread Pohjolainen, Topi
On Tue, Jan 24, 2017 at 03:45:54PM -0800, Jason Ekstrand wrote: > Previously, blorp could only blit into something that was renderable. > Thanks to recent additions to blorp, it can now blit into basically > anything so long as it isn't compressed. > --- > src/mesa/drivers/dri/i965/brw_blorp.c |

Re: [Mesa-dev] [PATCH 1/7] blorp: Handle the RGB workaround more like other workarounds

2017-01-25 Thread Pohjolainen, Topi
On Tue, Jan 24, 2017 at 03:45:48PM -0800, Jason Ekstrand wrote: > The previous version was sort-of strapped on in that it just adjusted > the blit rectangle and trusted in the fact that we would use texelFetch > and round to the nearest integer to ensure that the component positions > matched.

Re: [Mesa-dev] [PATCH 1/7] blorp: Handle the RGB workaround more like other workarounds

2017-01-25 Thread Pohjolainen, Topi
On Wed, Jan 25, 2017 at 10:51:53PM -0800, Jason Ekstrand wrote: >On Wed, Jan 25, 2017 at 10:46 PM, Pohjolainen, Topi ><[1]topi.pohjolai...@gmail.com> wrote: > >On Tue, Jan 24, 2017 at 03:45:48PM -0800, Jason Ekstrand wrote: >> The previous version was sort-of strapped on in that

Re: [Mesa-dev] [PATCH] i965: Unbind deleted shaders from brw_context, fixing malloc heisenbug.

2017-01-25 Thread Pohjolainen, Topi
On Wed, Jan 25, 2017 at 11:14:32PM -0800, Kenneth Graunke wrote: > Applications may delete a shader program, create a new one, and bind it > before the next draw. With terrible luck, malloc may randomly return a > chunk of memory for the new gl_program that happened to be the exact > same pointer

Re: [Mesa-dev] [PATCH] i965: Unbind deleted shaders from brw_context, fixing malloc heisenbug.

2017-01-25 Thread Jason Ekstrand
Ouch... Reviewed-by: Jason Ekstrand On Wed, Jan 25, 2017 at 11:14 PM, Kenneth Graunke wrote: > Applications may delete a shader program, create a new one, and bind it > before the next draw. With terrible luck, malloc may randomly return a > chunk

[Mesa-dev] [Bug 97879] [amdgpu] Rocket League: long hangs (several seconds) when loading assets (models/textures/shaders?)

2017-01-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97879 --- Comment #58 from Vladislav Egorov --- Correction - in fact I only tested radeon+radeonsi pair. I will try amdgpu+radeonsi later. -- You are receiving this mail because: You are the assignee for the

[Mesa-dev] [Bug 97967] glsl/tests/cache-test regression

2017-01-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97967 --- Comment #10 from Vinson Lee --- Can someone provide a patch with the print statements wanted to test? -- You are receiving this mail because: You are the assignee for the bug. You are the QA Contact for the

[Mesa-dev] [PATCH] nvc0/ir: fix robustness guarantees for constbuf loads on kepler+ compute

2017-01-25 Thread Ilia Mirkin
Kepler and up unfortunately only support up to 8 constbufs. We work around this by loading from constbufs as if they were storage buffers. However we were not consistently applying limits to loads from these buffers. Make sure to do the same thing we do for storage buffers. Fixes

Re: [Mesa-dev] [PATCH] nvc0/ir: fix robustness guarantees for constbuf loads on kepler+ compute

2017-01-25 Thread Ilia Mirkin
On Wed, Jan 25, 2017 at 11:51 PM, Ilia Mirkin wrote: > Kepler and up unfortunately only support up to 8 constbufs. We work > around this by loading from constbufs as if they were storage buffers. > However we were not consistently applying limits to loads from these >

[Mesa-dev] [PATCH v2 1/2] mesa: make glFramebuffer* check immutable texture level bounds

2017-01-25 Thread Ilia Mirkin
When a texture is immutable, we can't tack on extra levels after-the-fact like we could with glTexImage. So check against that level limit and return an error if it's surpassed. The spec is a little unclear in that it says to check if "level is not a supported texture level", however that is

[Mesa-dev] [PATCH v2 2/2] mesa: use same is_color_attachment trick to discern error cases

2017-01-25 Thread Ilia Mirkin
All the other calls to retrieve the attachment have been covered except this one - return the proper error for attachment points that are valid enums but out of bound for the driver. Fixes GL45-CTS.geometry_shader.layered_fbo.fb_texture_invalid_attachment Signed-off-by: Ilia Mirkin

[Mesa-dev] [Bug 97879] [amdgpu] Rocket League: long hangs (several seconds) when loading assets (models/textures/shaders?)

2017-01-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97879 --- Comment #57 from Vladislav Egorov --- Simple way to reproduce the bug - launch the game, then go to "Options" menu and get multi-second (>5s) freeze. I've tried to launch Rocket League on several GPUs. Graphics

[Mesa-dev] [PATCH] st/mesa: MAX_VARYING is the max supported number of patch varyings, not min

2017-01-25 Thread Ilia Mirkin
This fixes GL45-CTS.tessellation_shader.tessellation_shader_tessellation.max_in_out_attributes likely among others on nouveau. We only support 30 patch varyings (as 2 vec4 slots end up being used for tess level settings), but were getting 32 exposed. Signed-off-by: Ilia Mirkin

Re: [Mesa-dev] [PATCH] gallium/radeon: add a new HUD query for the number of mapped buffers

2017-01-25 Thread Michel Dänzer
On 25/01/17 11:19 PM, Samuel Pitoiset wrote: > On 01/25/2017 03:56 AM, Michel Dänzer wrote: >> On 25/01/17 12:05 AM, Marek Olšák wrote: >>> On Tue, Jan 24, 2017 at 2:17 PM, Christian König >>> wrote: Am 24.01.2017 um 11:44 schrieb Samuel Pitoiset: > On 01/24/2017

[Mesa-dev] [PATCH 1/2] i965: fix up asserts in brw_inst_set_jip()

2017-01-25 Thread Timothy Arceri
We are casting from a signed 32bit int to an unsigned 16bit int so shift 15 bits rather than 16. --- src/mesa/drivers/dri/i965/brw_inst.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_inst.h b/src/mesa/drivers/dri/i965/brw_inst.h index

[Mesa-dev] [PATCH 2/2] i965: add assert to while_jumps_before_offset()

2017-01-25 Thread Timothy Arceri
jip should always be negative here as its the result of do instruction - while instruction. --- src/mesa/drivers/dri/i965/brw_eu_emit.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c b/src/mesa/drivers/dri/i965/brw_eu_emit.c index 257757f..f4bec33

[Mesa-dev] [PATCH] nv50/ir: always return 0 when trying to read thread id along unit dim

2017-01-25 Thread Ilia Mirkin
Many many many compute shaders only define a 1- or 2-dimensional block, but then continue to use system values that take the full 3d into account (like gl_LocalInvocationIndex, etc). So for the special case that a dimension is exactly 1, we know that the thread id along that axis will always be 0,

[Mesa-dev] [PATCH] nvc0: increase number of ubo binding points

2017-01-25 Thread Ilia Mirkin
Apparently GL 4.5 requires 14 of these (there's a "*" in the spec, but it's unclear what it refers to). We need to expose an extra binding point for the "program parameters", which means this must be 15. Remove the last vestige of the "use c14 for immediates" idea. Fixes

Re: [Mesa-dev] [PATCH] vulkan: Don't install vk_platform.h or vulkan.h.

2017-01-25 Thread Matt Turner
On Mon, Jan 23, 2017 at 4:54 PM, Matt Turner wrote: > These files belong to the vulkan loader. I forgot to add the tag, but this (commit 045f38a5075) should go to the 17.0 branch. Emil, can you see to that? ___ mesa-dev mailing list

[Mesa-dev] [Bug 93375] unsupported call to function erf when running GROMACS with OpenCL

2017-01-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=93375 Vedran Miletić changed: What|Removed |Added Status|NEW |RESOLVED

Re: [Mesa-dev] [PATCH 2/2] anv: enable VK_KHR_shader_draw_parameters

2017-01-25 Thread Jason Ekstrand
Is there something we need to do here to plumb gl_DrawId through correctly? I'm pretty sure we have exactly zero code for that. On Wed, Jan 25, 2017 at 10:55 AM, Lionel Landwerlin < lionel.g.landwer...@intel.com> wrote: > Enables 10 tests from: > >dEQP-VK.draw.shader_draw_parameters.* > >

[Mesa-dev] [Bug 93375] unsupported call to function erf when running GROMACS with OpenCL

2017-01-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=93375 Vedran Miletić changed: What|Removed |Added Assignee|mesa-dev@lists.freedesktop. |ved...@miletic.net

Re: [Mesa-dev] [PATCH 23/34] i965: Add logic for allocating BO with CCS

2017-01-25 Thread Pohjolainen, Topi
On Mon, Jan 23, 2017 at 10:21:46PM -0800, Ben Widawsky wrote: > This patch provides the support (and comments) for allocating the BO > with space for the CCS buffer just underneath it. > > This patch was originally titled: > "i965: Create correctly sized mcs for an image" > > In order to make

Re: [Mesa-dev] [PATCH 32/34] i965/miptree: Remove dead code assertion

2017-01-25 Thread Pohjolainen, Topi
On Mon, Jan 23, 2017 at 10:21:55PM -0800, Ben Widawsky wrote: > We no longer allocate a miptree for the mcs_buf, so this is not a useful > assertion. > > While here, move the CCS disabling so that we only conditionally shut it > off. This helps us dtrt later when CCS is used in more places. > >

Re: [Mesa-dev] [PATCH 19/34] i965/miptree: Allocate mcs_buf for an image's CCS_E

2017-01-25 Thread Pohjolainen, Topi
On Mon, Jan 23, 2017 at 10:21:42PM -0800, Ben Widawsky wrote: > This code will disable actually creating these buffers for the scanout, > but it puts the allocation in place. > > Primarily this patch is split out for review, it can be squashed in > later if preferred. > > v2: > assert(mt->offset

  1   2   >