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

2017-01-26 Thread Christian König
Am 26.01.2017 um 12:16 schrieb Peter Frühberger: Hi Christian, 2017-01-26 12:00 GMT+01:00 Christian König >: Hi Peter, Am 25.01.2017 um 19:45 schrieb Peter Frühberger: Peter, Rainer any idea what I'm missing

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

2017-01-26 Thread Christian König
Am 26.01.2017 um 12:01 schrieb Samuel Pitoiset: On 01/26/2017 03:45 AM, Michel Dänzer wrote: 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

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

2017-01-26 Thread Mark Thompson
On 26/01/17 11:00, Christian König wrote: > Hi Peter, > > Am 25.01.2017 um 19:45 schrieb Peter Frühberger: >> >> >> Peter, Rainer any idea what I'm missing here? Do you guys use some >> modified ffmpeg for Kodi or how does that work for you? >> >> >> do you set the format correctly, e.g.:

[Mesa-dev] [PATCH 13/17] gallium/radeon: clean up r600_query_init_backend_mask

2017-01-26 Thread Marek Olšák
From: Marek Olšák This just needs to be done for r600g in the screen. We don't need an IB submission for every new context created for GCN. --- src/gallium/drivers/r600/r600_pipe.c | 2 +- src/gallium/drivers/radeon/r600_pipe_common.c | 1 +

[Mesa-dev] [PATCH 06/17] radeonsi: unbind disabled shader stages to prevent useless L2 prefetches

2017-01-26 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_state_shaders.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/src/gallium/drivers/radeonsi/si_state_shaders.c b/src/gallium/drivers/radeonsi/si_state_shaders.c index 010ce15..b3616dc 100644 ---

[Mesa-dev] [PATCH 07/17] radeonsi: atomize L2 prefetches

2017-01-26 Thread Marek Olšák
From: Marek Olšák to move the big conditional statement out of draw_vbo --- src/gallium/drivers/radeonsi/si_cp_dma.c| 39 + src/gallium/drivers/radeonsi/si_descriptors.c | 2 ++ src/gallium/drivers/radeonsi/si_hw_context.c| 3 ++

[Mesa-dev] [PATCH 17/17] radeonsi: fold info->indirect conditionals into the last one in draw_vbo

2017-01-26 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_state_draw.c | 25 + 1 file changed, 13 insertions(+), 12 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c b/src/gallium/drivers/radeonsi/si_state_draw.c index

[Mesa-dev] [PATCH 10/17] radeonsi: state atom IDs don't have to be off by one

2017-01-26 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_pipe.h | 4 ++-- src/gallium/drivers/radeonsi/si_state.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_pipe.h b/src/gallium/drivers/radeonsi/si_pipe.h index

[Mesa-dev] [PATCH 16/17] radeonsi: atomize the scratch buffer state

2017-01-26 Thread Marek Olšák
From: Marek Olšák The update frequency is very low. Difference: Only account for the size when allocating a new one. --- src/gallium/drivers/radeonsi/si_cp_dma.c| 3 ++- src/gallium/drivers/radeonsi/si_hw_context.c| 2 +-

[Mesa-dev] [PATCH 14/17] winsys/amdgpu: fix ADDR_REGISTER_VALUE::backendDisables

2017-01-26 Thread Marek Olšák
From: Marek Olšák This would be a fix if the value was used anywhere. --- src/gallium/winsys/amdgpu/drm/amdgpu_surface.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_surface.c

Re: [Mesa-dev] [PATCH] anv: don't assert on out of memory descriptor pool in debug mode

2017-01-26 Thread Jason Ekstrand
On Jan 26, 2017 6:03 AM, "Lionel Landwerlin" wrote: On 26/01/17 13:43, Eric Engestrom wrote: > On Thursday, 2017-01-26 11:27:00 +, Lionel Landwerlin wrote: > >> Fixes: >> dEQP-VK.api.descriptor_pool.out_of_pool_memory >> >> Signed-off-by: Lionel Landwerlin

[Mesa-dev] [PATCH 15/17] gallium/radeon: remove r600_common_context::max_db

2017-01-26 Thread Marek Olšák
From: Marek Olšák this cleanup is based on the vulkan driver, which seems to do the same thing --- src/gallium/drivers/radeon/r600_pipe_common.c | 7 --- src/gallium/drivers/radeon/r600_pipe_common.h | 2 -- src/gallium/drivers/radeon/r600_query.c | 28

[Mesa-dev] [PATCH 03/17] gallium/radeon: allow VRAM-only placements again on APUs & recent amdgpu

2017-01-26 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeon/r600_buffer_common.c | 4 1 file changed, 4 insertions(+) diff --git a/src/gallium/drivers/radeon/r600_buffer_common.c b/src/gallium/drivers/radeon/r600_buffer_common.c index c6f4d0d..da6f020 100644 ---

[Mesa-dev] [PATCH 11/17] radeonsi: move VGT_VERTEX_REUSE_BLOCK_CNTL into shader states for Polaris

2017-01-26 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_hw_context.c| 1 - src/gallium/drivers/radeonsi/si_pipe.h | 1 - src/gallium/drivers/radeonsi/si_state_draw.c| 19 --- src/gallium/drivers/radeonsi/si_state_shaders.c | 43

[Mesa-dev] [PATCH 08/17] gallium/radeon: merge dirty_fb_counter and dirty_tex_descriptor_counter

2017-01-26 Thread Marek Olšák
From: Marek Olšák to simplify things in draw_vbo a little --- src/gallium/drivers/r600/r600_state_common.c | 8 src/gallium/drivers/radeon/r600_pipe_common.h | 21 - src/gallium/drivers/radeon/r600_texture.c | 18 +-

[Mesa-dev] [PATCH 09/17] radeonsi: use a bitmask for looping over dirty PM4 states

2017-01-26 Thread Marek Olšák
From: Marek Olšák also move it to draw_vbo, because it should be 0 in most cases --- src/gallium/drivers/radeonsi/si_pipe.h | 1 + src/gallium/drivers/radeonsi/si_pm4.c| 16 +--- src/gallium/drivers/radeonsi/si_pm4.h| 1 -

[Mesa-dev] [PATCH 05/17] radeonsi: also prefetch compute shaders

2017-01-26 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_compute.c | 12 1 file changed, 12 insertions(+) diff --git a/src/gallium/drivers/radeonsi/si_compute.c b/src/gallium/drivers/radeonsi/si_compute.c index fe29fb1..d05c488 100644 ---

[Mesa-dev] [PATCH 12/17] radeonsi: precompute IA_MULTI_VGT_PARAM values into a table

2017-01-26 Thread Marek Olšák
From: Marek Olšák The perf difference is very small: 0.99% -> 0.40% for the time spent in si_get_ia_multi_vgt_param when si_draw_vbo is 20%. Pretty much nothing. --- src/gallium/drivers/radeonsi/si_pipe.c | 1 + src/gallium/drivers/radeonsi/si_pipe.h |

[Mesa-dev] [PATCH 02/17] radeonsi: don't set +fp64-denormals

2017-01-26 Thread Marek Olšák
From: Marek Olšák it's the default and the name will change to +fp64-fp16-denormals. --- src/gallium/drivers/radeonsi/si_pipe.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/radeonsi/si_pipe.c

[Mesa-dev] [PATCH 01/17] radeonsi: remove si_shader_context::param_tess_offchip

2017-01-26 Thread Marek Olšák
From: Marek Olšák we don't use on-chip tess. --- src/gallium/drivers/radeonsi/si_shader.c | 6 +++--- src/gallium/drivers/radeonsi/si_shader_internal.h | 5 - 2 files changed, 3 insertions(+), 8 deletions(-) diff --git

Re: [Mesa-dev] [PATCH 7/9] scons: swr: remove explicit __STDC_.*_MACROS defines

2017-01-26 Thread Kyriazis, George
Signed-off: George Kyriazis > -Original Message- > From: Emil Velikov [mailto:emil.l.veli...@gmail.com] > Sent: Thursday, January 26, 2017 7:24 AM > To: mesa-dev@lists.freedesktop.org > Cc: emil.l.veli...@gmail.com; Kyriazis, George

[Mesa-dev] [PATCH 04/17] radeonsi: update dirty_level_mask only after the first draw after FB change

2017-01-26 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_pipe.h | 1 + src/gallium/drivers/radeonsi/si_state.c | 2 ++ src/gallium/drivers/radeonsi/si_state_draw.c | 52 +++- 3 files changed, 31 insertions(+), 24 deletions(-) diff --git

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

2017-01-26 Thread Christian König
Hi Peter, Am 25.01.2017 um 19:45 schrieb Peter Frühberger: Peter, Rainer any idea what I'm missing here? Do you guys use some modified ffmpeg for Kodi or how does that work for you? do you set the format correctly, e.g.:

[Mesa-dev] [PATCH 1/8] nir: silence implicit conversion to 64bit

2017-01-26 Thread Emil Velikov
From: Emil Velikov MSVC warns about implicit conversion as below. Annotate the literal appropriately to silence the warning. nir_gather_info.c(249) : warning C4334: '<<' : result of 32-bit shift implicitly converted to 64 bits (was 64-bit shift intended?)

[Mesa-dev] [PATCH 5/8] st/mesa: use correct return statement for a void function

2017-01-26 Thread Emil Velikov
From: Emil Velikov Analogous to previous commit. Signed-off-by: Emil Velikov --- src/mesa/state_tracker/st_atifs_to_tgsi.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/mesa/state_tracker/st_atifs_to_tgsi.c

[Mesa-dev] [PATCH 8/8] st/xa: automake: remove duplicate -Wall

2017-01-26 Thread Emil Velikov
From: Emil Velikov Already handled by configure.ac Signed-off-by: Emil Velikov --- src/gallium/state_trackers/xa/Makefile.am | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/xa/Makefile.am

[Mesa-dev] [PATCH 2/8] nir: add extra const notation in compare_blocks()

2017-01-26 Thread Emil Velikov
From: Emil Velikov MSVC warns about different const qualifiers. Add the extra const to silence it. nir_phi_builder.c(244) : warning C4090: 'initializing' : different 'const' qualifiers nir_phi_builder.c(245) : warning C4090: 'initializing' : different 'const'

[Mesa-dev] [PATCH 4/8] mesa: use correct return statement for a void function

2017-01-26 Thread Emil Velikov
From: Emil Velikov Using return foo() is incorrect even if foo itself returns void. Spotted by AppVeyor, as below: teximage.c(3653) : warning C4098: 'copyteximage' : 'void' function returning a value Signed-off-by: Emil Velikov ---

Re: [Mesa-dev] [PATCH 1/9] android: always set __STDC_*_MACROS for C++ sources

2017-01-26 Thread Eric Engestrom
On Thursday, 2017-01-26 13:24:02 +, Emil Velikov wrote: > From: Emil Velikov > > Various parts of the code depend on the macros being defined. > > Just set those unconditionally, only where needed (c++ sources) so that > we can drop the workarounds through the

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

2017-01-26 Thread Samuel Pitoiset
On 01/26/2017 03:45 AM, Michel Dänzer wrote: 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

[Mesa-dev] [PATCH v2] anv: fix descriptor pool internal size allocation

2017-01-26 Thread Lionel Landwerlin
The size of the pool is slightly smaller than the size of the structure containing the whole pool. We need to take that into account on when setting up the internals. Fixes a crash due to out of bound memory access in: dEQP-VK.api.descriptor_pool.out_of_pool_memory v2: Drop debug traces

[Mesa-dev] [PATCH] anv: don't assert on out of memory descriptor pool in debug mode

2017-01-26 Thread Lionel Landwerlin
Fixes: dEQP-VK.api.descriptor_pool.out_of_pool_memory Signed-off-by: Lionel Landwerlin --- src/intel/vulkan/anv_util.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/intel/vulkan/anv_util.c b/src/intel/vulkan/anv_util.c index 6408ac87c7..5fbc7cc113

Re: [Mesa-dev] [PATCH 1/8] nir: silence implicit conversion to 64bit

2017-01-26 Thread Lionel Landwerlin
Reviewed-by: Lionel Landwerlin On 26/01/17 13:18, Emil Velikov wrote: From: Emil Velikov MSVC warns about implicit conversion as below. Annotate the literal appropriately to silence the warning. nir_gather_info.c(249) : warning

Re: [Mesa-dev] [PATCH 2/8] nir: add extra const notation in compare_blocks()

2017-01-26 Thread Lionel Landwerlin
Reviewed-by: Lionel Landwerlin On 26/01/17 13:18, Emil Velikov wrote: From: Emil Velikov MSVC warns about different const qualifiers. Add the extra const to silence it. nir_phi_builder.c(244) : warning C4090: 'initializing' :

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

2017-01-26 Thread Samuel Pitoiset
On 01/26/2017 12:07 PM, Christian König wrote: Am 26.01.2017 um 12:01 schrieb Samuel Pitoiset: On 01/26/2017 03:45 AM, Michel Dänzer wrote: 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,

[Mesa-dev] [PATCH] anv: fix descriptor pool internal size allocation

2017-01-26 Thread Lionel Landwerlin
The size of the pool is slightly smaller than the size of the structure containing the whole pool. We need to take that into account on when setting up the internals. Fixes a crash due to out of bound memory access in: dEQP-VK.api.descriptor_pool.out_of_pool_memory Signed-off-by: Lionel

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

2017-01-26 Thread Mark Thompson
On 25/01/17 14:41, 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. > > My main problem at the moment is that I can't

Re: [Mesa-dev] [PATCH] anv: don't assert on out of memory descriptor pool in debug mode

2017-01-26 Thread Eric Engestrom
On Thursday, 2017-01-26 11:27:00 +, Lionel Landwerlin wrote: > Fixes: >dEQP-VK.api.descriptor_pool.out_of_pool_memory > > Signed-off-by: Lionel Landwerlin Reviewed-by: Eric Engestrom But is there any reason not to add support

[Mesa-dev] [PATCH] configure.ac: Require LLVM for r300 only on x86 and x86_64

2017-01-26 Thread Andreas Boll
b3119a3 introduced a strict LLVM requirement for r300 on all architectures and thus configure fails on architectures where LLVM is not available or buggy. r300 doesn't strictly require LLVM, but for performance reasons we highly recommend LLVM usage. So require it at least on x86 and x86_64

[Mesa-dev] [PATCH RFC 1/3] docs: add EGL_MESA_drm_image_formats extension proposal

2017-01-26 Thread Nicolai Hähnle
From: Nicolai Hähnle --- docs/specs/EGL_MESA_drm_image_formats.txt | 98 +++ docs/specs/enums.txt | 5 ++ 2 files changed, 103 insertions(+) create mode 100644 docs/specs/EGL_MESA_drm_image_formats.txt diff --git

[Mesa-dev] [PATCH mesa 1/2] anv: add missing core errors in vk_errorf()

2017-01-26 Thread Eric Engestrom
Signed-off-by: Eric Engestrom --- src/intel/vulkan/anv_util.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/intel/vulkan/anv_util.c b/src/intel/vulkan/anv_util.c index 5fbc7cc113..289d01ecb0 100644 --- a/src/intel/vulkan/anv_util.c +++

Re: [Mesa-dev] [PATCH mesa 1/2] anv: add missing core errors in vk_errorf()

2017-01-26 Thread Lionel Landwerlin
Both patches look good to me : Reviewed-by: Lionel Landwerlin On 26/01/17 13:48, Eric Engestrom wrote: Signed-off-by: Eric Engestrom --- src/intel/vulkan/anv_util.c | 3 +++ 1 file changed, 3 insertions(+) diff --git

Re: [Mesa-dev] [PATCH] anv: don't assert on out of memory descriptor pool in debug mode

2017-01-26 Thread Lionel Landwerlin
On 26/01/17 13:43, Eric Engestrom wrote: On Thursday, 2017-01-26 11:27:00 +, Lionel Landwerlin wrote: Fixes: dEQP-VK.api.descriptor_pool.out_of_pool_memory Signed-off-by: Lionel Landwerlin Reviewed-by: Eric Engestrom But is

[Mesa-dev] [PATCH RFC 2/3] eglmesaext: add new enums for EGL_MESA_drm_image_formats

2017-01-26 Thread Nicolai Hähnle
From: Nicolai Hähnle --- include/EGL/eglmesaext.h | 7 +++ 1 file changed, 7 insertions(+) diff --git a/include/EGL/eglmesaext.h b/include/EGL/eglmesaext.h index 405d0e9..9b0de56 100644 --- a/include/EGL/eglmesaext.h +++ b/include/EGL/eglmesaext.h @@ -83,15 +83,22

[Mesa-dev] [RFC] EGL extension for additional DRM_BUFFER_FORMATs

2017-01-26 Thread Nicolai Hähnle
Hi all, this is mostly motivated by the need to support more screen depths than just 24/32-bit RGB in Glamor. The extension is simple enough, just adding three more enums that are accepted. I _think_ every DRI driver that exposes EGL_MESA_drm_image should be able to support this new extension as

[Mesa-dev] [PATCH RFC 3/3] egl: add support for EGL_MESA_drm_image_formats

2017-01-26 Thread Nicolai Hähnle
From: Nicolai Hähnle XXX do we need a separate enable bit? --- src/egl/drivers/dri2/egl_dri2.c | 21 - src/egl/main/eglapi.c | 2 ++ 2 files changed, 22 insertions(+), 1 deletion(-) diff --git a/src/egl/drivers/dri2/egl_dri2.c

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

2017-01-26 Thread Mark Thompson
On 25/01/17 14:42, Christian König wrote: > 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 ++--- >

[Mesa-dev] [PATCH 3/8] svga: remove const qualifier from SVGA3D_vgpu10_GenMips() prototype

2017-01-26 Thread Emil Velikov
From: Emil Velikov Does not match the function definition or how it's used. Triggers the following warning in AppVeyor svga_cmd_vgpu10.c(1301) : warning C4028: formal parameter 2 different from declaration Cc: Charmaine Lee Signed-off-by:

[Mesa-dev] [PATCH 6/8] mesa: move variable declaration to where its used

2017-01-26 Thread Emil Velikov
From: Emil Velikov The variable replacement was unused when building w/o ENABLE_SHADER_CACHE. Since we can mix variable declarations and code, move it to where its used. Fixes: 9f8dc3bf03e "utils: build sha1/disk cache only with Android/Autoconf" Signed-off-by: Emil

[Mesa-dev] [PATCH 7/8] st/mesa: explicitly handle/list 64bit ops in visit_expression()

2017-01-26 Thread Emil Velikov
From: Emil Velikov Follow the approach set in the file and handle all the ops, as otherwise the compiler throws a bunch of lovely warnings. Note that some versions of GCC have -Wswitch implied by -Wall, latter of which set in out autoconf and scons builds.

[Mesa-dev] [PATCH mesa 2/2] anv: add missing extension errors in vk_errorf()

2017-01-26 Thread Eric Engestrom
Signed-off-by: Eric Engestrom --- Feel free to reject this if you prefer to add extension errors when each extension is implemented :) --- src/intel/vulkan/anv_util.c | 5 + 1 file changed, 5 insertions(+) diff --git a/src/intel/vulkan/anv_util.c

[Mesa-dev] [PATCH 5/9] gallivm: remove explicit __STDC_.*_MACROS defines

2017-01-26 Thread Emil Velikov
From: Emil Velikov Correctly handled by the build systems. Cc: Roland Scheidegger Signed-off-by: Emil Velikov --- src/gallium/auxiliary/gallivm/lp_bld_misc.cpp | 8 1 file changed, 8 deletions(-) diff --git

[Mesa-dev] [PATCH 6/9] gallium: remove explicit __STDC_.*_MACROS defines

2017-01-26 Thread Emil Velikov
From: Emil Velikov Analogous to previous commits. Signed-off-by: Emil Velikov --- src/gallium/include/pipe/p_compiler.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/gallium/include/pipe/p_compiler.h

[Mesa-dev] [PATCH 3/9] autoconf: set all __STDC_*_MACROS

2017-01-26 Thread Emil Velikov
From: Emil Velikov Analogous to previous commit(s), with a minor detail - here we set the macros when building both C and C++ sources. Resolving that is a more challenging task that we'll sort out another day. Signed-off-by: Emil Velikov

[Mesa-dev] [PATCH 4/9] glsl: remove explicit __STDC_FORMAT_MACROS define

2017-01-26 Thread Emil Velikov
From: Emil Velikov Correctly handled by all the build systems. Signed-off-by: Emil Velikov --- src/compiler/glsl/glsl_parser_extras.cpp | 1 - src/compiler/glsl/ir_builder_print_visitor.cpp | 1 -

[Mesa-dev] [PATCH 9/9] mesa: remove explicit __STDC_FORMAT_MACROS define

2017-01-26 Thread Emil Velikov
From: Emil Velikov Analogous to previous commits. Signed-off-by: Emil Velikov --- src/mesa/x86/gen_matypes.c | 4 1 file changed, 4 deletions(-) diff --git a/src/mesa/x86/gen_matypes.c b/src/mesa/x86/gen_matypes.c index

[Mesa-dev] [PATCH 8/9] nouveau: remove explicit __STDC_FORMAT_MACROS define

2017-01-26 Thread Emil Velikov
From: Emil Velikov Already handled by the build. Signed-off-by: Emil Velikov --- src/gallium/drivers/nouveau/codegen/nv50_ir_print.cpp | 1 - 1 file changed, 1 deletion(-) diff --git

[Mesa-dev] [PATCH 7/9] scons: swr: remove explicit __STDC_.*_MACROS defines

2017-01-26 Thread Emil Velikov
From: Emil Velikov Analogous to previous commits. Cc: George Kyriazis Signed-off-by: Emil Velikov --- src/gallium/drivers/swr/SConscript | 5 - 1 file changed, 5 deletions(-) diff --git

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

2017-01-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97879 --- Comment #59 from Andreas Hartmetz --- Another random guess: Maybe shader compilation is a problem insofar that it blocks in a different GL API call than in other drivers, which causes some lock to be held for much

[Mesa-dev] [PATCH 2/9] scons: always set __STDC_*_MACROS for C++ sources

2017-01-26 Thread Emil Velikov
From: Emil Velikov Analogous to previous commit - just set the lot once throughout. Cc: Jose Fonseca Signed-off-by: Emil Velikov --- Jose, Appveyor seemed happy, but please check that it doesn't explode elsewhere for

[Mesa-dev] [PATCH 1/9] android: always set __STDC_*_MACROS for C++ sources

2017-01-26 Thread Emil Velikov
From: Emil Velikov Various parts of the code depend on the macros being defined. Just set those unconditionally, only where needed (c++ sources) so that we can drop the workarounds through the code. Cc: Rob Herring Cc: Chih-Wei Huang

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

2017-01-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97879 --- Comment #60 from Iaroslav Andrusyak --- if it help rocket league in wine(gallium-nine) works fine without hangs -- You are receiving this mail because: You are the assignee for the

[Mesa-dev] [PATCH mesa 1/2] radv: add missing core errors in vk_errorf()

2017-01-26 Thread Eric Engestrom
Signed-off-by: Eric Engestrom --- Similar to the anvil series: https://lists.freedesktop.org/archives/mesa-dev/2017-January/142187.html --- src/amd/vulkan/radv_util.c | 4 1 file changed, 4 insertions(+) diff --git a/src/amd/vulkan/radv_util.c

[Mesa-dev] [PATCH mesa 2/2] radv: add missing extension errors in vk_errorf()

2017-01-26 Thread Eric Engestrom
Signed-off-by: Eric Engestrom --- Feel free to reject this if you prefer to add extension errors when each extension is implemented :) --- src/amd/vulkan/radv_util.c | 8 1 file changed, 8 insertions(+) diff --git a/src/amd/vulkan/radv_util.c

[Mesa-dev] [PATCH] anv: Advertise API version 1.0.39

2017-01-26 Thread Jason Ekstrand
I'm pretty sure we've kept up with the bug fixes. --- 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 f40e9b2..27f06ac 100644 --- a/src/intel/vulkan/anv_device.c +++

Re: [Mesa-dev] [PATCH] configure.ac: Require LLVM for r300 only on x86 and x86_64

2017-01-26 Thread Marek Olšák
Reviewed-by: Marek Olšák On Jan 26, 2017 12:13 PM, "Andreas Boll" wrote: > b3119a3 introduced a strict LLVM requirement for r300 on all > architectures and thus configure fails on architectures where LLVM is > not available or buggy. > > r300

Re: [Mesa-dev] [PATCH 5/8] glsl: Rewrite atan2 implementation to fix accuracy and handling of zero/infinity.

2017-01-26 Thread Ian Romanick
On 01/25/2017 12:55 PM, Francisco Jerez wrote: > Ian Romanick writes: > >> It's a real bummer that we have two implementations of this function >> that are basically written in assembly... I'm not sure what else you'd >> call generating IR by hand. The code review and

Re: [Mesa-dev] [PATCH 2/2] spirv: handle undefined components for OpVectorShuffle

2017-01-26 Thread Jason Ekstrand
On Thu, Jan 26, 2017 at 9:08 AM, Lionel Landwerlin < lionel.g.landwer...@intel.com> wrote: > Fixes: >dEQP-VK.spirv_assembly.instruction.compute.opspecconstantop.vector_ > related >dEQP-VK.spirv_assembly.instruction.graphics.opspecconstantop.vector_ > related* > > Signed-off-by: Lionel

Re: [Mesa-dev] [PATCH] i965: Share the workaround bo between all contexts

2017-01-26 Thread Chad Versace
On Thu 26 Jan 2017, Chris Wilson wrote: > Since the workaround bo is used strictly as a write-only buffer, we need > only allocate one per screen and use the same one from all contexts. > > (The caveat here is during extension initialisation, where we write into > and read back register values

[Mesa-dev] [PATCH] st/va encode handle ntsc framerate rate control

2017-01-26 Thread Andy Furniss
Tested with ffmpeg and gst-vaapi. Without this bits per frame is set way too low. Signed-off-by: Andy Furniss --- src/gallium/state_trackers/va/picture.c | 32 1 file changed, 24 insertions(+), 8 deletions(-) diff --git

Re: [Mesa-dev] [PATCH 1/8] nir: silence implicit conversion to 64bit

2017-01-26 Thread Ian Romanick
I keep seeing patches like this... is it time to move BITFIELD64_* from mtypes.h to somewhere in util for more general use? On 01/26/2017 05:18 AM, Emil Velikov wrote: > From: Emil Velikov > > MSVC warns about implicit conversion as below. Annotate the literal >

Re: [Mesa-dev] [PATCH 1/8] nir: silence implicit conversion to 64bit

2017-01-26 Thread Jason Ekstrand
Ugh... windows defines long to be 32-bit on 32-bit platforms Yeah. Reviewed-by: Jason Ekstrand On Thu, Jan 26, 2017 at 6:05 AM, Lionel Landwerlin < lionel.g.landwer...@intel.com> wrote: > Reviewed-by: Lionel Landwerlin > > On 26/01/17

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

2017-01-26 Thread Marek Olšák
For the series: Reviewed-by: Marek Olšák On Jan 25, 2017 5:50 PM, "Nicolai Hähnle" wrote: Reviewed-by: Nicolai Hähnle On 25.01.2017 16:56, Samuel Pitoiset wrote: > This new query returns the current visible usage of VRAM

Re: [Mesa-dev] [PATCH 1/8] nir: silence implicit conversion to 64bit

2017-01-26 Thread Matt Turner
On Thu, Jan 26, 2017 at 10:22 AM, Jason Ekstrand wrote: > Ugh... windows defines long to be 32-bit on 32-bit platforms Yeah. long is 32-bit on 32-bit x86/Linux too... I think you mean they define long to be 32-bits on 64-bit platforms.

Re: [Mesa-dev] [PATCH mesa] docs/repository: fix name of main branch

2017-01-26 Thread Matt Turner
On Thu, Jan 26, 2017 at 10:11 AM, Eric Engestrom wrote: > This is git, not svn :P > > Signed-off-by: Eric Engestrom Reviewed-by: Matt Turner > --- > I also noticed some difference between this file and the one on >

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

2017-01-26 Thread Christian König
Am 26.01.2017 um 13:14 schrieb Mark Thompson: On 26/01/17 11:00, Christian König wrote: Hi Peter, Am 25.01.2017 um 19:45 schrieb Peter Frühberger: Peter, Rainer any idea what I'm missing here? Do you guys use some modified ffmpeg for Kodi or how does that work for you? do you set

Re: [Mesa-dev] [PATCH 1/2] spirv: handle OpUndef as part of the variable parsing pass

2017-01-26 Thread Jason Ekstrand
Thanks! Reviewed-by: Jason Ekstrand Please cc stable for both patches. --Jason On Thu, Jan 26, 2017 at 9:08 AM, Lionel Landwerlin < lionel.g.landwer...@intel.com> wrote: > Looking at the following bit of SPIRV shader : > > ... > %zero= OpConstant %i32 0 >

Re: [Mesa-dev] [PATCH 15/17] gallium/radeon: remove r600_common_context::max_db

2017-01-26 Thread Gustaw Smolarczyk
2017-01-26 17:04 GMT+01:00 Marek Olšák : > From: Marek Olšák > > this cleanup is based on the vulkan driver, which seems to do the same thing Is this also ok for r600g? If I'm right, the amdgpu-pro Vulkan driver doesn't have any support for pre-GCN

Re: [Mesa-dev] [PATCH] clover: Fix build against clang SVN >= r293097

2017-01-26 Thread Francisco Jerez
Michel Dänzer writes: > From: Michel Dänzer > > Signed-off-by: Michel Dänzer Reviewed-by: Francisco Jerez > --- > > Not sure if PropagateAttrs should be set to true or false. Setting it to > true

Re: [Mesa-dev] [PATCH 15/17] gallium/radeon: remove r600_common_context::max_db

2017-01-26 Thread Alex Deucher
On Thu, Jan 26, 2017 at 12:39 PM, Gustaw Smolarczyk wrote: > 2017-01-26 17:04 GMT+01:00 Marek Olšák : >> From: Marek Olšák >> >> this cleanup is based on the vulkan driver, which seems to do the same thing > > Is this also ok for

Re: [Mesa-dev] [PATCH 5/9] gallivm: remove explicit __STDC_.*_MACROS defines

2017-01-26 Thread Roland Scheidegger
Am 26.01.2017 um 14:24 schrieb Emil Velikov: > From: Emil Velikov > > Correctly handled by the build systems. > > Cc: Roland Scheidegger > Signed-off-by: Emil Velikov > --- >

[Mesa-dev] [PATCH 1/2] spirv: handle OpUndef as part of the variable parsing pass

2017-01-26 Thread Lionel Landwerlin
Looking at the following bit of SPIRV shader : ... %zero= OpConstant %i32 0 %ivec3_0 = OpConstantComposite %ivec3 %zero %zero %zero %vec3_undef = OpUndef %ivec3 %sc_0= OpSpecConstant %i32 0 %sc_1= OpSpecConstant %i32 0 %sc_2= OpSpecConstant %i32 0 ... Our

[Mesa-dev] [PATCH mesa] docs/repository: fix name of main branch

2017-01-26 Thread Eric Engestrom
This is git, not svn :P Signed-off-by: Eric Engestrom --- I also noticed some difference between this file and the one on mesa3d.org; it might be worth making sure everything is sync'ed between the two (most likely just push the version on git to the webserver). ---

[Mesa-dev] [PATCH 2/8] llvmpipe: use draw_get_option_use_llvm() instead of open coding it

2017-01-26 Thread Emil Velikov
From: Emil Velikov Cc: Roland Scheidegger Cc: Jose Fonseca Signed-off-by: Emil Velikov --- src/gallium/drivers/llvmpipe/lp_screen.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff

[Mesa-dev] [PATCH 5/8] i915g: use draw_get_option_use_llvm() instead of open coding it

2017-01-26 Thread Emil Velikov
From: Emil Velikov Currently one can build i915g without LLVM thus the current handling is wrong. Whether using i915g w/o LLVM is a good idea or not is a question for another time. Cc: Stéphane Marchesin Signed-off-by: Emil Velikov

[Mesa-dev] [PATCH 6/8] i915g: let .get_name() append LLVM if built with LLVM

2017-01-26 Thread Emil Velikov
From: Emil Velikov Provides quick and direct feedback to the user/developer. Cc: Stéphane Marchesin Signed-off-by: Emil Velikov --- src/gallium/drivers/i915/i915_screen.c | 7 ++- 1 file changed, 6

[Mesa-dev] [PATCH 1/8] virgl: remove unused draw_context.h include

2017-01-26 Thread Emil Velikov
From: Emil Velikov Signed-off-by: Emil Velikov --- src/gallium/drivers/virgl/virgl_screen.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/gallium/drivers/virgl/virgl_screen.c b/src/gallium/drivers/virgl/virgl_screen.c index

[Mesa-dev] [PATCH 8/8] nouveau: let .get_name() append LLVM if built with LLVM

2017-01-26 Thread Emil Velikov
From: Emil Velikov Analogous to previous two commits. Afaict only nv30 uses draw, so if people prefer we can restrict this print only to those devices. Signed-off-by: Emil Velikov --- Unrelated: Wasn't there a kernel/libdrm helper which

Re: [Mesa-dev] [PATCH v2] anv: fix descriptor pool internal size allocation

2017-01-26 Thread Jason Ekstrand
Thanks for catching this! Reviewed-by: Jason Ekstrand Cc: "17.0 13.0" On Thu, Jan 26, 2017 at 3:18 AM, Lionel Landwerlin < lionel.g.landwer...@intel.com> wrote: > The size of the pool is slightly smaller than the size of the > structure

Re: [Mesa-dev] [PATCH 7/8] st/mesa: explicitly handle/list 64bit ops in visit_expression()

2017-01-26 Thread Marek Olšák
Reviewed-by: Marek Olšák On Jan 26, 2017 2:22 PM, "Emil Velikov" wrote: > From: Emil Velikov > > Follow the approach set in the file and handle all the ops, as otherwise > the compiler throws a bunch of lovely

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

2017-01-26 Thread Ian Romanick
On 01/25/2017 01:42 PM, Francisco Jerez wrote: > 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 >>>

Re: [Mesa-dev] [PATCH] i965: Share the workaround bo between all contexts

2017-01-26 Thread Chris Wilson
On Thu, Jan 26, 2017 at 09:39:51AM -0800, Chad Versace wrote: > On Thu 26 Jan 2017, Chris Wilson wrote: > > Since the workaround bo is used strictly as a write-only buffer, we need > > only allocate one per screen and use the same one from all contexts. > > > > (The caveat here is during

[Mesa-dev] [PATCH 3/8] softpipe: set softpipe_screen::use_llvm when draw is build with LLVM

2017-01-26 Thread Emil Velikov
From: Emil Velikov Currently we can build draw without LLVM thus honouring SOFTPIPE_USE_LLVM is misleading even if most of the code nicely falls-back to no-op in the lack of LLVM. That does not seem to be the case in softpipe_draw_vbo() where extra prepare

[Mesa-dev] [PATCH 0/8] LLVM requirement for drivers using draw

2017-01-26 Thread Emil Velikov
Hi all, Here's a few small fixes/functionality improvements when dealing with LLVM. Most notably the series adds "LLVM" string [when applicable] to the .get_name() callback for drivers that use draw. Thus developer can respond accordingly - be that "rebuild with LLVM or enjoy the bad

[Mesa-dev] [PATCH 4/8] softpipe: let .get_name() append LLVM if built with LLVM

2017-01-26 Thread Emil Velikov
From: Emil Velikov Provides quick and direct feedback to the user/developer. Cc: Roland Scheidegger Cc: Jose Fonseca Signed-off-by: Emil Velikov --- We can move all the ifdef to a helper, if

[Mesa-dev] [PATCH 7/8] r300: let .get_name() append LLVM if built with LLVM

2017-01-26 Thread Emil Velikov
From: Emil Velikov Provides quick and direct feedback to the user/developer. Cc: Marek Olšák Signed-off-by: Emil Velikov --- src/gallium/drivers/r300/r300_screen.c | 10 +- 1 file changed, 9 insertions(+), 1

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

2017-01-26 Thread Andy Furniss
Andy Furniss wrote: 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,

[Mesa-dev] [PATCH 2/2] spirv: handle undefined components for OpVectorShuffle

2017-01-26 Thread Lionel Landwerlin
Fixes: dEQP-VK.spirv_assembly.instruction.compute.opspecconstantop.vector_related dEQP-VK.spirv_assembly.instruction.graphics.opspecconstantop.vector_related* Signed-off-by: Lionel Landwerlin --- src/compiler/spirv/spirv_to_nir.c | 52

Re: [Mesa-dev] [PATCH 1/9] android: always set __STDC_*_MACROS for C++ sources

2017-01-26 Thread Jose Fonseca
On 26/01/17 13:24, Emil Velikov wrote: From: Emil Velikov Various parts of the code depend on the macros being defined. Just set those unconditionally, only where needed (c++ sources) so that we can drop the workarounds through the code. Cc: Rob Herring

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

2017-01-26 Thread Marek Olšák
Reviewed-by: Marek Olšák On Jan 26, 2017 4:36 AM, "Ilia Mirkin" wrote: > 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 >

  1   2   >