[Mesa-dev] [PATCH 5/6] intel/common: Add gen_device_name_to_pci_device_id

2018-02-10 Thread Jordan Justen
Signed-off-by: Jordan Justen --- src/intel/common/gen_device_info.c | 19 +-- src/intel/common/gen_device_info.h | 1 + 2 files changed, 14 insertions(+), 6 deletions(-) diff --git a/src/intel/common/gen_device_info.c

[Mesa-dev] [PATCH 1/6] intel/vulkan: Support INTEL_NO_HW environment variable

2018-02-10 Thread Jordan Justen
Signed-off-by: Jordan Justen --- src/intel/vulkan/anv_device.c | 3 +++ src/intel/vulkan/anv_private.h | 2 ++ src/intel/vulkan/anv_queue.c | 2 +- 3 files changed, 6 insertions(+), 1 deletion(-) diff --git a/src/intel/vulkan/anv_device.c

[Mesa-dev] [PATCH 2/6] intel/common: Add gen_get_pci_device_id_override

2018-02-10 Thread Jordan Justen
Signed-off-by: Jordan Justen --- src/intel/common/gen_device_info.c | 51 ++ src/intel/common/gen_device_info.h | 1 + 2 files changed, 52 insertions(+) diff --git a/src/intel/common/gen_device_info.c

[Mesa-dev] [PATCH 3/6] i965: Use gen_get_pci_device_id_override

2018-02-10 Thread Jordan Justen
Signed-off-by: Jordan Justen --- src/mesa/drivers/dri/i965/intel_screen.c | 55 ++-- 1 file changed, 3 insertions(+), 52 deletions(-) diff --git a/src/mesa/drivers/dri/i965/intel_screen.c b/src/mesa/drivers/dri/i965/intel_screen.c index

[Mesa-dev] [PATCH 6/6] intel/tools: Use gen_device_name_to_pci_device_id in aubinator

2018-02-10 Thread Jordan Justen
Signed-off-by: Jordan Justen --- src/intel/tools/aubinator.c | 30 ++ 1 file changed, 6 insertions(+), 24 deletions(-) diff --git a/src/intel/tools/aubinator.c b/src/intel/tools/aubinator.c index 92aa208a61b..77bad29051e 100644 ---

[Mesa-dev] [PATCH 4/6] intel/vulkan: Support INTEL_DEVID_OVERRIDE environment variable

2018-02-10 Thread Jordan Justen
Signed-off-by: Jordan Justen --- src/intel/vulkan/anv_device.c | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.c index 59ea073803d..ea6fb58768f 100644 ---

[Mesa-dev] [Bug 105029] simdlib_512_avx512.inl:371:57: error: could not convert ‘_mm512_mask_blend_epi32((__mmask16)(ImmT), a, b)’ from ‘__m512i’ {aka ‘__vector(8) long long int’} to ‘SIMDImpl::SIMD51

2018-02-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105029 Bug ID: 105029 Summary: simdlib_512_avx512.inl:371:57: error: could not convert ‘_mm512_mask_blend_epi32((__mmask16)(ImmT), a, b)’ from ‘__m512i’ {aka ‘__vector(8) long long int’}

[Mesa-dev] [PATCH v2 1/2] gm107/ir: change how SUQ works in preparation for bindless

2018-02-10 Thread Ilia Mirkin
All this information can be retrieved from the TIC directly. Avoid having to dip into the constbuf information about the image. Signed-off-by: Ilia Mirkin --- v1 -> v2: Move GM107 lowering impl into the gm107 lowering pass which I forgot about. (Pointed out by Karol a

Re: [Mesa-dev] radeonsi: copy the NIR enablement debug bit to the shader cache flags

2018-02-10 Thread Marek Olšák
I don't recommend enabling NIR for 18.0. Let's just declare that it's an unsupported unstable configuration. Marek On Feb 10, 2018 3:13 AM, "Dieter Nützel" wrote: Hello, I know that NIR support for radeonsi is new, but should this land in Mesa 18.0.0 final as early

Re: [Mesa-dev] [PATCH 2/2] nvc0: add support for bindless on maxwell+

2018-02-10 Thread Samuel Pitoiset
On 02/10/2018 06:12 AM, Ilia Mirkin wrote: Signed-off-by: Ilia Mirkin --- Tested on GM107 with DOW3 and DiRT Rally (in bindless mode). Seems to work about as well as on Kepler, except I got a few ltc errors in DiRT Rally. Unclear if it's related to this patch though.

Re: [Mesa-dev] [PATCH 10/10] meson: build gallium unit tests

2018-02-10 Thread Eric Anholt
Dylan Baker writes: > Signed-off-by: Dylan Baker > --- > src/gallium/tests/meson.build | 1 +- > src/gallium/tests/unit/meson.build | 31 +++- > 2 files changed, 32 insertions(+) > create mode 100644

Re: [Mesa-dev] [PATCH 2/2] nvc0: add support for bindless on maxwell+

2018-02-10 Thread Ilia Mirkin
On Sat, Feb 10, 2018 at 12:37 PM, Samuel Pitoiset wrote: > > > On 02/10/2018 06:12 AM, Ilia Mirkin wrote: >> >> Signed-off-by: Ilia Mirkin >> --- >> >> Tested on GM107 with DOW3 and DiRT Rally (in bindless mode). Seems to >> work about as well as

Re: [Mesa-dev] [PATCH 10/17] glsl/tests: Fix strict aliasing warning about int64/double.

2018-02-10 Thread Rhys Kidd
On 10 February 2018 at 11:33, Eric Anholt wrote: > Cc: Rhys Kidd > Fixes: 4bf986274728 ("glsl/tests: Add UINT64 and INT64 types") > Reviewed-by: Rhys Kidd > --- > .../glsl/tests/uniform_initializer_utils.cpp | 22 >

Re: [Mesa-dev] [PATCH 2/2] nvc0: add support for bindless on maxwell+

2018-02-10 Thread Samuel Pitoiset
On 02/10/2018 06:41 PM, Ilia Mirkin wrote: On Sat, Feb 10, 2018 at 12:37 PM, Samuel Pitoiset wrote: On 02/10/2018 06:12 AM, Ilia Mirkin wrote: Signed-off-by: Ilia Mirkin --- Tested on GM107 with DOW3 and DiRT Rally (in bindless mode).

[Mesa-dev] [PATCH] intel/blorp: Only double the fast-clear rect alignment on HSW

2018-02-10 Thread Jason Ekstrand
--- src/intel/blorp/blorp_clear.c | 66 --- 1 file changed, 56 insertions(+), 10 deletions(-) diff --git a/src/intel/blorp/blorp_clear.c b/src/intel/blorp/blorp_clear.c index a2dbcd1..63b74e3 100644 --- a/src/intel/blorp/blorp_clear.c +++

Re: [Mesa-dev] [PATCH] intel/blorp: Only double the fast-clear rect alignment on HSW

2018-02-10 Thread Jason Ekstrand
One side comment: The only tests that caught anything were dEQP or CTS tests. Somehow, Piglit didn't catch any of this. This means that we don't have any real data on IVB. I'd be happy to make ISL_DEV_GEN(dev) == 7 instead since the extra alignment won't hurt anything on IVB (no mipmapping).

[Mesa-dev] [PATCH] gm107/ir: avoid using kepler instruction capabilities

2018-02-10 Thread Ilia Mirkin
Split up the op properties table into generation-specific bits, and only use the kepler ones on kepler. Fixes some CTS images tests. Signed-off-by: Ilia Mirkin --- .../nouveau/codegen/nv50_ir_target_nvc0.cpp| 62 ++

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

2018-02-10 Thread Philipp Kerling
Hi, resurrecting this thread again since there's been some progress on the Kodi side. > For the EGL part, see > and > . We recently started testing vaExportSurfaceHandle support,

Re: [Mesa-dev] [PATCH 2/2] nvir/gm107: consider FILE_FLAGS dependencies in SchedDataCalculatorGM107

2018-02-10 Thread Samuel Pitoiset
On 12/05/2017 12:01 PM, Karol Herbst wrote: currently while insterting barriers, writes and reads to FILE_FLAGS aren't considered. This can lead to WaR hazards in some situations. Fixes OpenCL CTS tests on Maxwell+: basic/test_basic.intmath_long.LONG_MAD

Re: [Mesa-dev] [PATCH 1/2] nvir/gm107: iterate over all defs in SchedDataCalculatorGM107::findFirstUse

2018-02-10 Thread Samuel Pitoiset
On 12/05/2017 12:01 PM, Karol Herbst wrote: In the sched data calculator we have to track first use of defs by iterating over all defs of an instruction, not just the first one. Signed-off-by: Karol Herbst --- .../drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp | 26

Re: [Mesa-dev] [PATCH v3 4/8] radeon/uvd:add uvd hevc enc hw ib implementation

2018-02-10 Thread Mark Thompson
On 09/02/18 20:35, James Zhu wrote: > Implement required IBs for UVD HEVC encode. > > Signed-off-by: James Zhu > --- > src/gallium/drivers/radeon/radeon_uvd_enc_1_1.c | 1115 > +++ > 1 file changed, 1115 insertions(+) > create mode 100644

[Mesa-dev] [PATCH 05/17] intel/blorp: Fix compiler warning about num_layers.

2018-02-10 Thread Eric Anholt
The compiler doesn't notice that the condition for num_layers to be undefined already defined it above (as our assert checked in a debug build). Cc: Jason Ekstrand --- src/mesa/drivers/dri/i965/brw_blorp.c | 1 + 1 file changed, 1 insertion(+) diff --git

[Mesa-dev] [PATCH 02/17] gallium/util: Fix compiler warning about unknown pipe_filter_state.

2018-02-10 Thread Eric Anholt
Fixes: 6321b1bd4005 ("gallium/util: replace uint with tgsi enum types"). Cc: Brian Paul --- src/gallium/auxiliary/util/u_blit.h | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/src/gallium/auxiliary/util/u_blit.h b/src/gallium/auxiliary/util/u_blit.h

[Mesa-dev] [PATCH 14/17] i965: Fix compiler warning about write being undefined.

2018-02-10 Thread Eric Anholt
This looks like it should be protected by the assume() about nr_color_regions, but my compiler warns anyway. Cc: Matt Turner --- src/intel/compiler/brw_fs.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/intel/compiler/brw_fs.cpp

[Mesa-dev] [PATCH 10/17] glsl/tests: Fix strict aliasing warning about int64/double.

2018-02-10 Thread Eric Anholt
Cc: Rhys Kidd Fixes: 4bf986274728 ("glsl/tests: Add UINT64 and INT64 types") --- .../glsl/tests/uniform_initializer_utils.cpp | 22 +++--- 1 file changed, 19 insertions(+), 3 deletions(-) diff --git

[Mesa-dev] [PATCH 08/17] loader: Fix compiler warnings about truncating the PCI ID path.

2018-02-10 Thread Eric Anholt
My build was producing: ../src/loader/loader.c:121:67: warning: ‘%1u’ directive output may be truncated writing between 1 and 3 bytes into a region of size 2 [-Wformat-truncation=] and we can avoid this careful calculation by just using asprintf (as we do elsewhere in the file). Cc: Eric

[Mesa-dev] [PATCH 13/17] anv: Silence compiler warnings about uninitialized bind_offset.

2018-02-10 Thread Eric Anholt
This is a legitimate warning: if anv's blorp_alloc_binding_table() throws an error from anv_cmd_buffer_alloc_blorp_binding_table(), we silently continue to use this undefined value. The rest of this code doesn't seem very allocation-error-proof, though, either. Cc: Jason Ekstrand

[Mesa-dev] [PATCH 16/17] anv: Silence warning about heap_size.

2018-02-10 Thread Eric Anholt
We only get VK_SUCCESS if it was initialized, but apparently my compiler doesn't track that far. Cc: Jason Ekstrand --- src/intel/vulkan/anv_device.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/intel/vulkan/anv_device.c

[Mesa-dev] [PATCH 17/17] glsl: Suppress sign-compare warning generated by flex 2.6.1.

2018-02-10 Thread Eric Anholt
I got the following warning: src/compiler/glsl/glsl_lexer.cpp:1521:18: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] for ( n = 0; n < max_size && \ src/compiler/glsl/glsl_lexer.cpp:3448:3: note: in expansion of macro ‘YY_INPUT’ YY_INPUT(

Re: [Mesa-dev] [PATCH v2 0/8] The 2nd version for UVD HEVC encode

2018-02-10 Thread Mark Thompson
On 08/02/18 23:05, Mark Thompson wrote: > On 08/02/18 22:37, Alex Deucher wrote: >> On Thu, Feb 8, 2018 at 5:28 PM, Mark Thompson wrote: >>> On 06/02/18 20:05, James Zhu wrote: The whole series are the updated version. Changes are made mainly based on the comments from

[Mesa-dev] [PATCH 01/17] r200: Remove dead rop table

2018-02-10 Thread Eric Anholt
Fixes: 0aaa27f29187 ("mesa: Pass the translated color logic op dd_function_table::LogicOpcode") Cc: Ian Romanick --- src/mesa/drivers/dri/r200/r200_state.c | 20 1 file changed, 20 deletions(-) diff --git a/src/mesa/drivers/dri/r200/r200_state.c

[Mesa-dev] [PATCH 03/17] gallium/llvmpipe: Fix compiler warnings about ddx/ddy/ddmax.

2018-02-10 Thread Eric Anholt
My gcc doesn't figure out that dims >= 1 (seems reasonable), and doesn't notice that ddmax is used from the same no_rho_opt as its initialization. Cc: Roland Scheidegger --- src/gallium/auxiliary/gallivm/lp_bld_sample.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[Mesa-dev] [PATCH 06/17] ac/nir: Fix compiler warning about uninitialized dw_addr.

2018-02-10 Thread Eric Anholt
Even switching the def's condition to be the same chip revision check as the use, the compiler doesn't figure it out. Just NULL-init it. Fixes: ec53e527421d ("ac/nir: Add ES output to LDS for GFX9.") Cc: Bas Nieuwenhuizen --- src/amd/common/ac_nir_to_llvm.c | 2 +- 1

[Mesa-dev] [PATCH 04/17] egl/gbm: Fix compiler warning about visual matching.

2018-02-10 Thread Eric Anholt
The compiler doesn't know that num_visuals > 0. Fixes: 37a8d907cc16 ("egl/gbm: Ensure EGLConfigs match GBM surface format") Cc: Daniel Stone --- src/egl/drivers/dri2/platform_drm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[Mesa-dev] [PATCH 11/17] glsl/tests: Fix a compiler warning about signed/unsigned loop comparison.

2018-02-10 Thread Eric Anholt
Cc: Ian Romanick Fixes: d32956935edf ("glsl: Walk a list of ir_dereference_array to mark array elements as accessed") --- src/compiler/glsl/tests/array_refcount_test.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[Mesa-dev] [PATCH 12/17] radv: Fix compiler warning about uninitialized 'set'

2018-02-10 Thread Eric Anholt
The compiler doesn't figure out that we only get result == VK_SUCCESS if set got initialized. Cc: Dave Airlie --- src/amd/vulkan/radv_descriptor_set.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/amd/vulkan/radv_descriptor_set.c

[Mesa-dev] [PATCH 09/17] spirv: Silence compiler warning about undefined srcs[0]

2018-02-10 Thread Eric Anholt
It doesn't know that elems >= 1. Cc: Jason Ekstrand --- src/compiler/spirv/spirv_to_nir.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/compiler/spirv/spirv_to_nir.c b/src/compiler/spirv/spirv_to_nir.c index c6df764682ec..f86d45511df8 100644

[Mesa-dev] [PATCH 07/17] glsl: Silence warnings in the uniform initializer test about 16-bit types

2018-02-10 Thread Eric Anholt
They should probably get unit tests implemented, but this cleans up a bunch of warnings in my build for now. Fixes: 59f458cd8703 ("glsl: Add 16-bit types") Cc: Eduardo Lima Mitev --- src/compiler/glsl/tests/uniform_initializer_utils.cpp | 9 + 1 file changed, 9

[Mesa-dev] [PATCH 15/17] i965: Silence compiler warning about promoted_constants.

2018-02-10 Thread Eric Anholt
We only have a cfg != NULL if we went through one of the paths that set it, but my compiler doesn't figure that out. Cc: Jason Ekstrand Fixes: 6411defdcd6f ("intel/cs: Re-run final NIR optimizations for each SIMD size") --- src/intel/compiler/brw_fs.cpp | 2 +- 1 file