Re: [Mesa-dev] [PATCH v4 14/18] anv: Add a helper to extract clear color from the attachment.

2018-03-15 Thread Jordan Justen
On 2018-03-08 08:49:07, Rafael Antognolli wrote: > Extract the code from color_attachment_compute_aux_usage, so we can > later reuse it to update the clear color state buffer. > > Signed-off-by: Rafael Antognolli > --- > src/intel/vulkan/anv_private.h | 20 > src/intel/v

Re: [Mesa-dev] [PATCH 1/5] radeonsi: remove fast color clear for single-sample buffers

2018-03-15 Thread Dieter Nützel
Wow, if we compare Polaris 20 with Ryzen 3 2200G, AMD Vega 8 on Phoronix https://www.phoronix.com/scan.php?page=article&item=ryzen3-2200g-vega8&num=3 Even TessMark against NVIDIA GeForce GTX 1080: http://openbenchmarking.org/prospect/1606047-HA-PASCALGRA41/946288150b2d292b67300c7fea4e9a47f6bb3f4

Re: [Mesa-dev] [PATCH v2] egl/android: Implement the eglSwapinterval for Android.

2018-03-15 Thread Wu, Zhongmin
Friendly ping -Original Message- From: Wu, Zhongmin Sent: Wednesday, March 7, 2018 9:16 To: Eric Engestrom ; Emil Velikov Cc: ML mesa-dev ; Tomasz Figa ; Rob Herring ; Liu, Zhiquan ; Long, Zhifang ; Kondapally, Kalyan ; Palli, Tapani ; Xu, Randy ; Bhardwaj, MunishX ; Kps, Harish Kr

Re: [Mesa-dev] [PATCH] ac/nir: Add workaround for GFX9 buffer views.

2018-03-15 Thread Samuel Pitoiset
On 03/09/2018 12:38 AM, Bas Nieuwenhuizen wrote: On GFX9 whether the buffer size is interpreted as elements or bytes depends on whether IDXEN is enabled in the instruction. If the index is a constant zero, LLVM optimizes IDXEN to 0. Now the size in elements is interpreted in bytes which of cou

Re: [Mesa-dev] [PATCH v2] egl/android: Implement the eglSwapinterval for Android.

2018-03-15 Thread Tomasz Figa
On Thu, Jan 18, 2018 at 4:39 PM, Zhongmin Wu wrote: > Implement the eglSwapinterval for Android platform to > enable the async mode for some GFX benchmarks such as > Daimler C217, CityBench. > > Signed-off-by: Zhongmin Wu > --- > src/egl/drivers/dri2/platform_android.c | 21 +

Re: [Mesa-dev] [PATCH 1/5] radeonsi: remove fast color clear for single-sample buffers

2018-03-15 Thread Samuel Pitoiset
Reviewed-by: Samuel Pitoiset On 03/11/2018 07:11 PM, Marek Olšák wrote: From: Marek Olšák This should improve the score for the GpuTest Triangle benchmark. Vulkan doesn't use this either. --- src/gallium/drivers/radeon/r600_pipe_common.h | 1 - src/gallium/drivers/radeon/r600_texture.c

Re: [Mesa-dev] FLAG-DAY: NIR derefs

2018-03-15 Thread Alejandro Piñeiro
Hi, On 14/03/18 21:32, Jason Ekstrand wrote: > All, > > Connor and I along with several others have been discussing for a > while changing the way NIR dereferences work.  In particular, adding a > new nir_deref_instr type where the first one in the chain takes a > variable and is followed by a ser

[Mesa-dev] [Bug 105507] Crash when destroying a newly resized EGLsurface with wayland egl (dri2)

2018-03-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105507 --- Comment #3 from Johan Helsing --- I initially thought it was all the tests in tst_QOpenGLWindow, but it's only tst_QOpenGLWindow::basic. However, it will soon to be split into tst_QOpenGLWindow::resize if https://codereview.qt-project.org/#/

[Mesa-dev] [Bug 105507] Crash when destroying a newly resized EGLsurface with wayland egl (dri2)

2018-03-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105507 --- Comment #4 from Pekka Paalanen --- (In reply to Daniel Stone from comment #2) > You could place any orphaned wl_buffers on an per-surface list instead, and > spin at destruction until that emptied. What would guarantee that the last committ

[Mesa-dev] [Bug 105507] Crash when destroying a newly resized EGLsurface with wayland egl (dri2)

2018-03-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105507 --- Comment #5 from Daniel Stone --- (In reply to Pekka Paalanen from comment #4) > (In reply to Daniel Stone from comment #2) > > You could place any orphaned wl_buffers on an per-surface list instead, and > > spin at destruction until that emp

Re: [Mesa-dev] soft fp64 support - main body (glsl/gallium)

2018-03-15 Thread Gert Wollny
Am Mittwoch, den 14.03.2018, 21:05 +1100 schrieb Timothy Arceri: > > > Anyway I'm not trying to convince Dave as I'm sure he has a good idea > of the work involve. Just thought I'd mention this stuff in case > anyone else has considered working on NIR for r600 and are scared by > Daves time esti

Re: [Mesa-dev] [PATCH 0/6] Shader cache; transform feedback; i965 program-binary

2018-03-15 Thread Tapani Pälli
Patches 1-3: Reviewed-by: Tapani Pälli (also verified with xfb using app) On 03/14/2018 09:26 AM, Jordan Justen wrote: git://people.freedesktop.org/~jljusten/mesa shader-cache-xform-fb+prog-bin-v1 Patches 1 - 3 remove the restriction preventing the disk shader cache from being enabled if tran

Re: [Mesa-dev] [PATCH 2/6] spirv/radv: add AMD_gcn_shader capability, remove current extensions

2018-03-15 Thread Daniel Schürmann
With AMD_gcn_shader renamed to gcn_shader, this patch is Reviewed-by:/Daniel Schürmann >/ On 08.03.2018 16:00, Alejandro Piñeiro wrote: So now, during spirv_to_nir, it uses the capability instead of the extension. Note that we are really

[Mesa-dev] [Bug 105507] Crash when destroying a newly resized EGLsurface with wayland egl (dri2)

2018-03-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105507 --- Comment #6 from Pekka Paalanen --- (In reply to Daniel Stone from comment #5) > Correct, so the only correct (though incredibly counter-intuitive) course of > action is to destroy the wl_surface before destroying the EGLSurface. > Alternatel

Re: [Mesa-dev] [PATCH 2/6] spirv/radv: add AMD_gcn_shader capability, remove current extensions

2018-03-15 Thread Bas Nieuwenhuizen
On Thu, Mar 15, 2018 at 10:46 AM, Daniel Schürmann wrote: > With AMD_gcn_shader renamed to gcn_shader, this patch is > > Reviewed-by: Daniel Schürmann Note that the mailaddress is mangled a bit. Reviewed-by: Bas Nieuwenhuizen > > On 08.03.2018 16:00, Alejandro Piñeiro wrote: > > So now, during

Re: [Mesa-dev] [PATCH 1/2] anv/pipeline: fail if tcs/tes compile fail

2018-03-15 Thread Lionel Landwerlin
Reviewed-by: Lionel Landwerlin On 15/03/18 00:20, Caio Marcelo de Oliveira Filho wrote: --- src/intel/vulkan/anv_pipeline.c | 16 +--- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/src/intel/vulkan/anv_pipeline.c b/src/intel/vulkan/anv_pipeline.c index 9cfd16df2a.

Re: [Mesa-dev] [PATCH 1/2] anv/pipeline: fail if tcs/tes compile fail

2018-03-15 Thread Lionel Landwerlin
Fixes: e50d4807a35e679 ("anv: Compile TCS/TES shaders.") On 15/03/18 10:14, Lionel Landwerlin wrote: Reviewed-by: Lionel Landwerlin On 15/03/18 00:20, Caio Marcelo de Oliveira Filho wrote: ---   src/intel/vulkan/anv_pipeline.c | 16 +---   1 file changed, 9 insertions(+), 7 deletio

Re: [Mesa-dev] [PATCH 2/2] anv/pipeline: set active_stages early

2018-03-15 Thread Lionel Landwerlin
Looks good, just one tiny nit below. I assume allowing to conditionally run certain passes isn't implemented yet? Thanks! - Lionel On 15/03/18 00:20, Caio Marcelo de Oliveira Filho wrote: Since the intermediate states of active_stages are not used, i.e. active_stages is read only after all st

Re: [Mesa-dev] [PATCH 2/3] meson: Use the same version for all libdrm checks

2018-03-15 Thread Eric Engestrom
On Tuesday, 2018-03-13 11:52:48 -0700, Dylan Baker wrote: > Currently each driver specifies it's own version, and core libdrm > specifies a version. In the most common case this is fine, since there > will be exactly one libdrm installed on a system, but if there are more > than one it's possible t

Re: [Mesa-dev] [PATCH] meson: fix pipe-loaders after omx changes

2018-03-15 Thread Eric Engestrom
On Tuesday, 2018-03-13 16:05:18 -0700, Dylan Baker wrote: > with_gallium_omx used to be a boolean, but now it's a string. That means > it needs to be compared to 'disabled' instead of false. > > CC: Rob Clark > Fixes: 34e852d5b50772199797ea839fc8d6b3805633ff >("meson: Re-add auto option f

[Mesa-dev] [PATCH] mesa: RGB9_E5 invalid for CopyTexSubImage* in GLES

2018-03-15 Thread Juan A. Suarez Romero
According to OpenGL ES 3.2, section 8.6, CopyTexSubImage* should return an INVALID_OPERATION if the internalformat of the texture is RGB9_E5. This fixes dEQP-GLES31.functional.debug.negative_coverage.*.copytexsubimage2d_texture_internalformat. --- src/mesa/main/teximage.c | 14 ++ 1 f

Re: [Mesa-dev] [PATCH] egl/dri2: move wayland header inclusion where applicable

2018-03-15 Thread Eric Engestrom
On Wednesday, 2018-03-14 17:42:16 +, Emil Velikov wrote: > From: Emil Velikov > > Instead of indirectly pulling the wayland headers everywhere, use > forward declarations and #include only as needed. > > Should effectively fix build errors like the following: > > make[5]: Entering directory

[Mesa-dev] [Bug 104926] swrast: Mesa 17.3.3 produces: HW cursor for format 875713089 not supported

2018-03-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104926 Frederic Crozat changed: What|Removed |Added CC||f...@crozat.net --- Comment #2 from F

[Mesa-dev] [Bug 105320] Storage texel buffer access produces wrong results (RX Vega)

2018-03-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105320 libgra...@gmail.com changed: What|Removed |Added CC||libgra...@gmail.com -- You are re

[Mesa-dev] [Bug 105506] Vulkan MSAA is broken on SI

2018-03-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105506 Turo Lamminen changed: What|Removed |Added Status|RESOLVED|REOPENED Resolution|NOTOURBUG

[Mesa-dev] [PATCH 1/2] tgsi: move tgsi_processor_to_shader_stage() to a header

2018-03-15 Thread Emil Velikov
From: Emil Velikov This way we can utilise it with later patches. Signed-off-by: Emil Velikov --- A fair bit of polish can be applied here - name, unsigned -> enum etc. I've opted to keep it as-is since it's been in the current shape for a while. --- src/gallium/auxiliary/nir/tgsi_to_nir.c

[Mesa-dev] [PATCH 2/2] st/mesa: simplify st_init_limits() via tgsi_processor_to_shader_stage

2018-03-15 Thread Emil Velikov
From: Emil Velikov Reuse the tgis helper and remove a bunch of duplicated code. Signed-off-by: Emil Velikov --- src/mesa/state_tracker/st_extensions.c | 45 +- 1 file changed, 6 insertions(+), 39 deletions(-) diff --git a/src/mesa/state_tracker/st_extensions.c

[Mesa-dev] [Bug 105506] Vulkan MSAA is broken on SI

2018-03-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105506 Turo Lamminen changed: What|Removed |Added Attachment #138108|0 |1 is obsolete|

Re: [Mesa-dev] [PATCH 2/2] st/mesa: simplify st_init_limits() via tgsi_processor_to_shader_stage

2018-03-15 Thread Brian Paul
On 03/15/2018 07:12 AM, Emil Velikov wrote: From: Emil Velikov Reuse the tgis helper and remove a bunch of duplicated code. Signed-off-by: Emil Velikov --- src/mesa/state_tracker/st_extensions.c | 45 +- 1 file changed, 6 insertions(+), 39 deletions(-) diff

Re: [Mesa-dev] [PATCH 1/2] tgsi: move tgsi_processor_to_shader_stage() to a header

2018-03-15 Thread Brian Paul
On 03/15/2018 07:12 AM, Emil Velikov wrote: From: Emil Velikov This way we can utilise it with later patches. Signed-off-by: Emil Velikov --- A fair bit of polish can be applied here - name, unsigned -> enum etc. I've opted to keep it as-is since it's been in the current shape for a while. --

[Mesa-dev] [PATCH 0/4] mesa/i965: Add support for INTEL_blackhole_render

2018-03-15 Thread Lionel Landwerlin
Hi all, This series implements a new extension that just landed in the Khronos repository : https://github.com/KhronosGroup/OpenGL-Registry/commit/a4f25a7ec733d397d8ea78794a066ea519493688 The purpose of this extension is mostly to enable a feature that is used in GPA [1], allowing to disable all

[Mesa-dev] [PATCH 1/4] mesa: add INTEL_blackhole_render

2018-03-15 Thread Lionel Landwerlin
Signed-off-by: Lionel Landwerlin --- include/GL/glext.h | 5 + include/GLES2/gl2ext.h | 5 + src/mapi/glapi/registry/gl.xml | 8 +++- src/mesa/main/clear.c| 2 +- src/mesa/main/enable.c | 10 ++ src/mesa/main/extensions_table

[Mesa-dev] [PATCH 2/4] i965: pipecontrol: add LRI write immediate flag

2018-03-15 Thread Lionel Landwerlin
Signed-off-by: Lionel Landwerlin --- src/mesa/drivers/dri/i965/brw_pipe_control.h | 1 + 1 file changed, 1 insertion(+) diff --git a/src/mesa/drivers/dri/i965/brw_pipe_control.h b/src/mesa/drivers/dri/i965/brw_pipe_control.h index 651cd4d3e70..4c58e16660f 100644 --- a/src/mesa/drivers/dri/i965/

[Mesa-dev] [PATCH 3/4] i965: enable INTEL_blackhole_render on haswell+

2018-03-15 Thread Lionel Landwerlin
Signed-off-by: Lionel Landwerlin --- src/mesa/drivers/dri/i965/brw_context.h | 2 ++ src/mesa/drivers/dri/i965/brw_defines.h | 8 ++- src/mesa/drivers/dri/i965/brw_misc_state.c| 33 +++ src/mesa/drivers/dri/i965/brw_state.h | 1 + src/mesa/dr

[Mesa-dev] [PATCH 4/4] intel: genxml: add INSTPM/CS_DEBUG_MODE2 registers

2018-03-15 Thread Lionel Landwerlin
Signed-off-by: Lionel Landwerlin --- src/intel/genxml/gen10.xml | 6 ++ src/intel/genxml/gen11.xml | 6 ++ src/intel/genxml/gen6.xml | 7 +++ src/intel/genxml/gen7.xml | 7 +++ src/intel/genxml/gen75.xml | 7 +++ src/intel/genxml/gen8.xml | 7 +++ src/intel/genxml/gen9.

[Mesa-dev] [FLAG-DAY-PREP 2/9] freedreno/ir3: use nir_intrinsic_var() accessor

2018-03-15 Thread Rob Clark
--- src/gallium/drivers/freedreno/ir3/ir3_compiler_nir.c | 8 src/gallium/drivers/freedreno/ir3/ir3_nir.c | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/src/gallium/drivers/freedreno/ir3/ir3_compiler_nir.c b/src/gallium/drivers/freedreno/ir3/ir3_compiler_

[Mesa-dev] [FLAG-DAY-PREP 3/9] amd: use nir_intrinsic_var() accessor

2018-03-15 Thread Rob Clark
--- src/amd/common/ac_nir_to_llvm.c | 70 +++- src/amd/vulkan/radv_shader_info.c| 11 ++--- src/gallium/drivers/radeonsi/si_shader_nir.c | 4 +- 3 files changed, 44 insertions(+), 41 deletions(-) diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/

[Mesa-dev] [FLAG-DAY-PREP 5/9] intel: use nir_intrinsic_var() accessor

2018-03-15 Thread Rob Clark
--- src/intel/compiler/brw_fs_nir.cpp | 4 ++-- src/intel/vulkan/anv_nir_apply_pipeline_layout.c | 2 +- src/intel/vulkan/anv_nir_lower_input_attachments.c | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/intel/compiler/brw_fs_nir.cpp b/src/intel/compi

[Mesa-dev] [FLAG-DAY-PREP 9/9] amd: use nir_tex_*_var() accessors

2018-03-15 Thread Rob Clark
--- src/amd/common/ac_nir_to_llvm.c | 6 -- src/amd/vulkan/radv_shader_info.c | 11 +++ 2 files changed, 11 insertions(+), 6 deletions(-) diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c index 8df00fc3c8c..37e66f081de 100644 --- a/src/amd/common/ac_nir

[Mesa-dev] [FLAG-DAY-PREP 1/9] nir: start to abstract deref chains/instructions

2018-03-15 Thread Rob Clark
Prep work to convert from deref chains to deref instruction chains, add a accessors so we can start moving code away from direct access to deref chain just for the sake of finding the nir_variable. After conversion to deref instructions, this would be converted to chase the deref instruction ssa l

[Mesa-dev] [FLAG-DAY-PREP 0/9] nir: accessors for deref vars

2018-03-15 Thread Rob Clark
Since a fair bit of access to nir_deref_var's is just to chase down the nir_variable for an intrinsic (or texture/sampler vars for tex instructions), we can make the flag day slightly less painful with some accessors that could be re-implemented for deref instruction chains. It will still be a fla

[Mesa-dev] [FLAG-DAY-PREP 7/9] nir: use nir_tex_*_var() accessors

2018-03-15 Thread Rob Clark
--- src/compiler/nir/nir_instr_set.c | 2 +- src/compiler/nir/nir_lower_samplers.c| 6 -- src/compiler/nir/nir_remove_dead_variables.c | 14 ++ 3 files changed, 11 insertions(+), 11 deletions(-) diff --git a/src/compiler/nir/nir_instr_set.c b/src/compiler/nir

[Mesa-dev] [FLAG-DAY-PREP 8/9] intel: use nir_tex_*_var() accessors

2018-03-15 Thread Rob Clark
--- src/intel/vulkan/anv_nir_apply_pipeline_layout.c | 18 +++--- src/intel/vulkan/anv_nir_lower_ycbcr_textures.c | 2 +- 2 files changed, 12 insertions(+), 8 deletions(-) diff --git a/src/intel/vulkan/anv_nir_apply_pipeline_layout.c b/src/intel/vulkan/anv_nir_apply_pipeline_layout

[Mesa-dev] [FLAG-DAY-PREP 6/9] nir: use nir_intrinsic_var() accessor

2018-03-15 Thread Rob Clark
--- src/compiler/nir/nir_gather_info.c | 2 +- src/compiler/nir/nir_inline_functions.c| 6 +++--- src/compiler/nir/nir_linking_helpers.c | 10 -- src/compiler/nir/nir_lower_alpha_test.c| 2 +- src/compiler/nir/nir_lower_atomics.c

[Mesa-dev] [FLAG-DAY-PREP 4/9] mesa/st: use nir_intrinsic_var() accessor

2018-03-15 Thread Rob Clark
--- src/mesa/state_tracker/st_nir_lower_builtin.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/state_tracker/st_nir_lower_builtin.c b/src/mesa/state_tracker/st_nir_lower_builtin.c index fca0bef6653..29570eaf29b 100644 --- a/src/mesa/state_tracker/st_nir_lower_built

Re: [Mesa-dev] [PATCH 3/4] i965: enable INTEL_blackhole_render on haswell+

2018-03-15 Thread Chris Wilson
Quoting Lionel Landwerlin (2018-03-15 14:24:32) > diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c > b/src/mesa/drivers/dri/i965/intel_extensions.c > index 73a6c73f537..12d22e12eee 100644 > --- a/src/mesa/drivers/dri/i965/intel_extensions.c > +++ b/src/mesa/drivers/dri/i965/intel_extensio

Re: [Mesa-dev] FLAG-DAY: NIR derefs

2018-03-15 Thread Jason Ekstrand
On Thu, Mar 15, 2018 at 1:12 AM, Alejandro Piñeiro wrote: > Hi, > On 14/03/18 21:32, Jason Ekstrand wrote: > > All, > > Connor and I along with several others have been discussing for a while > changing the way NIR dereferences work. In particular, adding a new > nir_deref_instr type where the f

Re: [Mesa-dev] [RFC] nir: Add a deref instruction type

2018-03-15 Thread Rob Clark
On Thu, Mar 15, 2018 at 1:33 AM, Jason Ekstrand wrote: > This commit adds a new instruction type to NIR for handling derefs. > Nothing uses it yet but this adds the data structure as well as all of > the code to validate, print, clone, and [de]serialize them. > > Cc: Rob Clark > Cc: Connor Abbott

Re: [Mesa-dev] [PATCH v4 10/18] i965/miptree: Add new BO for clear color.

2018-03-15 Thread Rafael Antognolli
On Wed, Mar 14, 2018 at 05:26:08PM -0700, Jordan Justen wrote: > What about a subject like this? > > i965/miptree: Add new clear color BO for winsys aux buffers It's definitely better. I'll update it. > On 2018-03-08 08:49:03, Rafael Antognolli wrote: > > Add an extra BO to store clear color whe

Re: [Mesa-dev] [PATCH v4 14/18] anv: Add a helper to extract clear color from the attachment.

2018-03-15 Thread Rafael Antognolli
On Thu, Mar 15, 2018 at 12:20:19AM -0700, Jordan Justen wrote: > On 2018-03-08 08:49:07, Rafael Antognolli wrote: > > Extract the code from color_attachment_compute_aux_usage, so we can > > later reuse it to update the clear color state buffer. > > > > Signed-off-by: Rafael Antognolli > > --- > >

Re: [Mesa-dev] FLAG-DAY: NIR derefs

2018-03-15 Thread Alejandro Piñeiro
On 15/03/18 15:39, Jason Ekstrand wrote: > On Thu, Mar 15, 2018 at 1:12 AM, Alejandro Piñeiro > mailto:apinhe...@igalia.com>> wrote: > > Hi, > > On 14/03/18 21:32, Jason Ekstrand wrote: >> All, >> >> Connor and I along with several others have been discussing for a >> while chan

Re: [Mesa-dev] [RFC] nir: Add a deref instruction type

2018-03-15 Thread Bas Nieuwenhuizen
On Thu, Mar 15, 2018 at 6:33 AM, Jason Ekstrand wrote: > This commit adds a new instruction type to NIR for handling derefs. > Nothing uses it yet but this adds the data structure as well as all of > the code to validate, print, clone, and [de]serialize them. > > Cc: Rob Clark > Cc: Connor Abbott

Re: [Mesa-dev] [PATCH 1/4] mesa: add INTEL_blackhole_render

2018-03-15 Thread Emil Velikov
HI Lionel, On 15 March 2018 at 14:24, Lionel Landwerlin wrote: > Signed-off-by: Lionel Landwerlin > --- > include/GL/glext.h | 5 + > include/GLES2/gl2ext.h | 5 + > src/mapi/glapi/registry/gl.xml | 8 +++- Please update these by pulling the latest files

Re: [Mesa-dev] [PATCH 7/7] radv: dump LLVM IR when AMD_shader_info is used

2018-03-15 Thread Bas Nieuwenhuizen
On Wed, Mar 14, 2018 at 12:27 PM, Samuel Pitoiset wrote: > Signed-off-by: Samuel Pitoiset > --- > src/amd/vulkan/radv_shader.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/src/amd/vulkan/radv_shader.c b/src/amd/vulkan/radv_shader.c > index 327ae9fe3d..1ece13657c 100644 > --- a/src/am

Re: [Mesa-dev] [PATCH 1/7] radv: only display options that are enabled

2018-03-15 Thread Bas Nieuwenhuizen
One comment on patch 7, and the LLVM dump ordering issue we debugged earlier (where we should dump before compiling), otherwise the series is Reviewed-by: Bas Nieuwenhuizen On Wed, Mar 14, 2018 at 12:27 PM, Samuel Pitoiset wrote: > Signed-off-by: Samuel Pitoiset > --- > src/amd/vulkan/radv_de

Re: [Mesa-dev] [PATCH 1/4] mesa: add INTEL_blackhole_render

2018-03-15 Thread Lionel Landwerlin
On 15/03/18 16:03, Emil Velikov wrote: HI Lionel, On 15 March 2018 at 14:24, Lionel Landwerlin wrote: Signed-off-by: Lionel Landwerlin --- include/GL/glext.h | 5 + include/GLES2/gl2ext.h | 5 + src/mapi/glapi/registry/gl.xml | 8 +++- Please updat

Re: [Mesa-dev] [PATCH 4/4] intel: genxml: add INSTPM/CS_DEBUG_MODE2 registers

2018-03-15 Thread Emil Velikov
On 15 March 2018 at 14:24, Lionel Landwerlin wrote: > Signed-off-by: Lionel Landwerlin > --- > src/intel/genxml/gen10.xml | 6 ++ > src/intel/genxml/gen11.xml | 6 ++ > src/intel/genxml/gen6.xml | 7 +++ > src/intel/genxml/gen7.xml | 7 +++ > src/intel/genxml/gen75.xml | 7

Re: [Mesa-dev] [PATCH 7/7] radv: dump LLVM IR when AMD_shader_info is used

2018-03-15 Thread Samuel Pitoiset
On 03/15/2018 05:04 PM, Bas Nieuwenhuizen wrote: On Wed, Mar 14, 2018 at 12:27 PM, Samuel Pitoiset wrote: Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_shader.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/amd/vulkan/radv_shader.c b/src/amd/vulkan/radv_shader.c index 3

Re: [Mesa-dev] [RFC] nir: Add a deref instruction type

2018-03-15 Thread Jason Ekstrand
On Thu, Mar 15, 2018 at 7:46 AM, Rob Clark wrote: > On Thu, Mar 15, 2018 at 1:33 AM, Jason Ekstrand > wrote: > > This commit adds a new instruction type to NIR for handling derefs. > > Nothing uses it yet but this adds the data structure as well as all of > > the code to validate, print, clone,

Re: [Mesa-dev] [PATCH 1/7] radv: only display options that are enabled

2018-03-15 Thread Samuel Pitoiset
On 03/15/2018 05:06 PM, Bas Nieuwenhuizen wrote: One comment on patch 7, and the LLVM dump ordering issue we debugged earlier (where we should dump before compiling), otherwise the series is Yes, v2 was here: https://cgit.freedesktop.org/~hakzsam/mesa/commit/?h=radv_llvm_ir&id=904e9e336cbe94

[Mesa-dev] [Bug 104926] swrast: Mesa 17.3.3 produces: HW cursor for format 875713089 not supported

2018-03-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=104926 --- Comment #3 from Michal Srb --- I was able to reproduce the `gbm_device_is_format_supported` failure. The call sequence is following: * gbm_device_is_format_supported * gbm_dri_is_format_supported (gbm->is_format_supported) - `dri->image->

Re: [Mesa-dev] [RFC] nir: Add a deref instruction type

2018-03-15 Thread Jason Ekstrand
On Thu, Mar 15, 2018 at 8:22 AM, Bas Nieuwenhuizen wrote: > On Thu, Mar 15, 2018 at 6:33 AM, Jason Ekstrand > wrote: > > This commit adds a new instruction type to NIR for handling derefs. > > Nothing uses it yet but this adds the data structure as well as all of > > the code to validate, print,

[Mesa-dev] [PATCH] st/dri: don't set queryDmaBufFormats/queryDmaBufModifiers if the driver does not implement it

2018-03-15 Thread Michal Srb
This is equivalent to commit a65db0ad1c3, but for dri_kms_init_screen. Without this gbm_dri_is_format_supported always returns false. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=104926 --- src/gallium/state_trackers/dri/dri2.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-)

Re: [Mesa-dev] [PATCH 2/2] st/mesa: simplify st_init_limits() via tgsi_processor_to_shader_stage

2018-03-15 Thread Marek Olšák
For the series: Reviewed-by: Marek Olšák Marek On Thu, Mar 15, 2018 at 9:12 AM, Emil Velikov wrote: > From: Emil Velikov > > Reuse the tgis helper and remove a bunch of duplicated code. > > Signed-off-by: Emil Velikov > --- > src/mesa/state_tracker/st_extensions.c | 45 > +

[Mesa-dev] [PATCH 3/3] anv: silence unused variable warning

2018-03-15 Thread Lionel Landwerlin
Fixes: 59b0ea0c748 ("anv: Stop returning VK_ERROR_INCOMPATIBLE_DRIVER") Signed-off-by: Lionel Landwerlin --- src/intel/vulkan/anv_device.c | 7 --- 1 file changed, 7 deletions(-) diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.c index d8c4e986316..82b237e76d8 100644

[Mesa-dev] [PATCH 1/3] anv: silence unused function warning on gen11

2018-03-15 Thread Lionel Landwerlin
[84/227] Compiling C object 'src/intel/vulkan/libanv_gen110@sta/genX_blorp_exec.c.o'. ../src/intel/vulkan/genX_blorp_exec.c:68:1: warning: ‘blorp_get_surface_base_address’ defined but not used [-Wunused-function] blorp_get_surface_base_address(struct blorp_batch *batch) ^~~~

[Mesa-dev] [PATCH 0/3] intel: silence some warnings

2018-03-15 Thread Lionel Landwerlin
A few trivial compiler warnings we can get rid of :) Cheers, Lionel Landwerlin (3): anv: silence unused function warning on gen11 i965: silence unused function warning anv: silence unused variable warning src/intel/blorp/blorp_genX_exec.h | 2 +- src/intel/vulkan/anv_device.c

[Mesa-dev] [PATCH 2/3] i965: silence unused function warning

2018-03-15 Thread Lionel Landwerlin
[123/227] Compiling C object 'src/mesa/drivers/dri/i965/libi965_gen110@sta/genX_blorp_exec.c.o'. ../src/mesa/drivers/dri/i965/genX_blorp_exec.c:99:1: warning: ‘blorp_get_surface_base_address’ defined but not used [-Wunused-function] blorp_get_surface_base_address(struct blorp_batch *batch) ^~~~

Re: [Mesa-dev] Releasing 18.0

2018-03-15 Thread Marek Olšák
Hi, I'd like to have these in 18.0. People can fetch them here: git://people.freedesktop.org/~mareko/mesa for-18.0 They are already in our AMD internal 18.0 branch, so they get quite a lot of testing. pick 0554172 radeonsi: align command buffer starting address to fix some Raven hangs pick e0

Re: [Mesa-dev] [PATCH 0/3] intel: silence some warnings

2018-03-15 Thread Rafael Antognolli
On Thu, Mar 15, 2018 at 04:49:07PM +, Lionel Landwerlin wrote: > A few trivial compiler warnings we can get rid of :) Trivial review too :) Reviewed-by: Rafael Antognolli > Cheers, > > Lionel Landwerlin (3): > anv: silence unused function warning on gen11 > i965: silence unused functio

Re: [Mesa-dev] [PATCH 2/3] meson: Use the same version for all libdrm checks

2018-03-15 Thread Dylan Baker
Quoting Eric Engestrom (2018-03-15 04:52:41) > On Tuesday, 2018-03-13 11:52:48 -0700, Dylan Baker wrote: > > Currently each driver specifies it's own version, and core libdrm > > specifies a version. In the most common case this is fine, since there > > will be exactly one libdrm installed on a sys

Re: [Mesa-dev] [Mesa-stable] [PATCH] Revert "mesa: do not trigger _NEW_TEXTURE_STATE in glActiveTexture()"

2018-03-15 Thread Marek Olšák
Sounds good to me. Marek On Wed, Mar 14, 2018 at 6:26 AM, Timothy Arceri wrote: > > > On 14/03/18 19:46, Samuel Pitoiset wrote: > >> This reverts commit f314a532fdc7af8381586144d2631d9968331f05. >> >> This appears to introduce some blinking textures in UT2004. Not >> sure exactly what's the roo

Re: [Mesa-dev] [RFC] nir: Add a deref instruction type

2018-03-15 Thread Rob Clark
On Thu, Mar 15, 2018 at 11:51 AM, Jason Ekstrand wrote: > On Thu, Mar 15, 2018 at 7:46 AM, Rob Clark wrote: >> >> On Thu, Mar 15, 2018 at 1:33 AM, Jason Ekstrand >> wrote: >> > This commit adds a new instruction type to NIR for handling derefs. >> > Nothing uses it yet but this adds the data str

Re: [Mesa-dev] Releasing 18.0

2018-03-15 Thread Dylan Baker
Quoting Kenneth Graunke (2018-03-14 15:24:29) > Hi Emil, all, > > I think we've closed the last of the Mesa 18.0 blocker bugs. It looks > like there are some patches nominated for the 18.0 branch still (fixing > some of those issues), but assuming things are merged, I think we're > ready to relea

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

2018-03-15 Thread Michel Dänzer
From: Marek Olšák (Ported from radeonsi commit f70f6baaa3bb0f8b280ac2eaea69bbffaf7de840) Allows cached BOs to be reused in more cases. Bugzilla: https://bugs.freedesktop.org/105171 Signed-off-by: Michel Dänzer --- src/gallium/drivers/r600/r600_buffer_common.c | 16 +--- src/galliu

[Mesa-dev] [Bug 105171] performance regression (3x slower) running glamor with PutImage workload (radeonsi)

2018-03-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105171 --- Comment #11 from Michel Dänzer --- https://patchwork.freedesktop.org/patch/210907/ helps for this benchmark with the r600 driver, but radeonsi already has the same code... Clemens, are you still seeing the problem with current Mesa Git mast

Re: [Mesa-dev] [PATCH 1/4] mesa: add INTEL_blackhole_render

2018-03-15 Thread Emil Velikov
On 15 March 2018 at 16:08, Lionel Landwerlin wrote: > On 15/03/18 16:03, Emil Velikov wrote: >> >> HI Lionel, >> >> On 15 March 2018 at 14:24, Lionel Landwerlin >> wrote: >>> >>> Signed-off-by: Lionel Landwerlin >>> --- >>> include/GL/glext.h | 5 + >>> include/GLES2/gl2ext

[Mesa-dev] 2018 X.Org Foundation Election Candidates

2018-03-15 Thread Rob Clark
To all X.Org Foundation Members: The election for the X.Org Foundation Board of Directors will begin on 22 March 2018. We have 6 candidates who are running for 4 seats. They are (in alphabetical order): Eric Anholt Robert Foss Bryce Harrington Keith Packard Laurent Pinchart Harry Wentland Attac

Re: [Mesa-dev] Releasing 18.0

2018-03-15 Thread Michel Dänzer
On 2018-03-15 06:01 PM, Marek Olšák wrote: > > pick a3ac60e ac: move address space definitions to common code > pick 572d46d ac: Use the renumbered const address space for LLVM 7. > # Both are convenient for AMD devs testing 18.0 with LLVM master. The > latter depends on the former. It's not real

[Mesa-dev] [Bug 105529] u_debug_stack.c:268: error: #pragma GCC diagnostic not allowed inside functions

2018-03-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105529 Bug ID: 105529 Summary: u_debug_stack.c:268: error: #pragma GCC diagnostic not allowed inside functions Product: Mesa Version: git Hardware: x86-64 (AMD64)

Re: [Mesa-dev] Releasing 18.0

2018-03-15 Thread Marek Olšák
On Thu, Mar 15, 2018 at 2:19 PM, Michel Dänzer wrote: > On 2018-03-15 06:01 PM, Marek Olšák wrote: > > > > pick a3ac60e ac: move address space definitions to common code > > pick 572d46d ac: Use the renumbered const address space for LLVM 7. > > # Both are convenient for AMD devs testing 18.0 wit

Re: [Mesa-dev] Releasing 18.0

2018-03-15 Thread Michel Dänzer
On 2018-03-15 07:25 PM, Marek Olšák wrote: > On Thu, Mar 15, 2018 at 2:19 PM, Michel Dänzer > wrote: > > On 2018-03-15 06:01 PM, Marek Olšák wrote: > > > > pick a3ac60e ac: move address space definitions to common code > > pick 572d46d ac: Use the renumb

Re: [Mesa-dev] [PATCH] r600: consolidate PIPE_BIND_SHARED/SCANOUT handling

2018-03-15 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Thu, Mar 15, 2018 at 2:04 PM, Michel Dänzer wrote: > From: Marek Olšák > > (Ported from radeonsi commit f70f6baaa3bb0f8b280ac2eaea69bbffaf7de840) > > Allows cached BOs to be reused in more cases. > > Bugzilla: https://bugs.freedesktop.org/105171 > Signed-off-

Re: [Mesa-dev] FLAG-DAY: NIR derefs

2018-03-15 Thread Jason Ekstrand
For those interested in following along, I've pushed a bunch of stuff to a branch: https://cgit.freedesktop.org/~jekstrand/mesa/log/?h=wip/nir-deref-instr I will continue force-pushing that branch as I go. My current approach to doing things incrementally is to try and do it by type of dereferen

Re: [Mesa-dev] Releasing 18.0

2018-03-15 Thread Marek Olšák
On Thu, Mar 15, 2018 at 2:34 PM, Michel Dänzer wrote: > On 2018-03-15 07:25 PM, Marek Olšák wrote: > > On Thu, Mar 15, 2018 at 2:19 PM, Michel Dänzer > > wrote: > > > > On 2018-03-15 06:01 PM, Marek Olšák wrote: > > > > > > pick a3ac60e ac: move address spa

Re: [Mesa-dev] [RFC] nir: Add a deref instruction type

2018-03-15 Thread Jason Ekstrand
On Thu, Mar 15, 2018 at 9:50 AM, Rob Clark wrote: > On Thu, Mar 15, 2018 at 11:51 AM, Jason Ekstrand > wrote: > > On Thu, Mar 15, 2018 at 7:46 AM, Rob Clark wrote: > >> > >> On Thu, Mar 15, 2018 at 1:33 AM, Jason Ekstrand > >> wrote: > >> > This commit adds a new instruction type to NIR for ha

[Mesa-dev] Want to participate in the development

2018-03-15 Thread Kuntal Majumder
Hi I am Kuntal, wanted to participate in the development of mesa. Is there any list of tasks for someone who is just starting out to get acquainted with the code? Thanks Kuntal M ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://li

[Mesa-dev] [PATCH v2 1/2] anv/pipeline: fail if TCS/TES compile fail

2018-03-15 Thread Caio Marcelo de Oliveira Filho
v2: Add Fixes tag. (Lionel) Fixes: e50d4807a35e679 ("anv: Compile TCS/TES shaders.") Reviewed-by: Lionel Landwerlin --- Lionel, I don't have commit access. Could you apply this patch? src/intel/vulkan/anv_pipeline.c | 16 +--- 1 file changed, 9 insertions(+), 7 deletions(-) diff -

[Mesa-dev] [PATCH v2 2/2] anv/pipeline: set active_stages early

2018-03-15 Thread Caio Marcelo de Oliveira Filho
Since the intermediate states of active_stages are not used, i.e. active_stages is read only after all stages were set into it, just set its value before compiling the shaders. This will allow to conditionally run certain passes based on what other shaders are being used, e.g. a certain pass might

[Mesa-dev] [PATCH] radeonsi: don't reallocate on DMABUF export if local BOs are disabled

2018-03-15 Thread Marek Olšák
From: Marek Olšák --- src/amd/common/ac_gpu_info.c | 2 ++ src/amd/common/ac_gpu_info.h | 1 + src/gallium/drivers/radeon/r600_texture.c | 4 +++- src/gallium/winsys/amdgpu/drm/amdgpu_bo.c | 7 +++ 4 files changed, 9 insertions(+), 5 deletions(-) diff --git a/src/a

[Mesa-dev] [PATCH] radv/query: handle multiview queries properly.

2018-03-15 Thread Dave Airlie
From: Dave Airlie For multiview we need to emit a number of sequential queries depending on the view mask. This avoids dEQP-VK.multiview.queries.15 waiting forever on the CPU for query results that are never coming. This doesn't make this test pass though, but I can now finish my CTS run. Sign

Re: [Mesa-dev] [PATCH v4 17/18] anv: Make blorp update the clear color.

2018-03-15 Thread Jordan Justen
On 2018-03-08 08:49:10, Rafael Antognolli wrote: > Instead of updating the clear color in anv before a resolve, just let > blorp handle that for us during fast clears. > > Signed-off-by: Rafael Antognolli > --- > src/intel/vulkan/anv_blorp.c | 69 > +++--- >

[Mesa-dev] [Bug 105436] Blinking textures in UT2004 [bisected]

2018-03-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105436 Samuel Pitoiset changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

[Mesa-dev] [Bug 105171] performance regression (3x slower) running glamor with PutImage workload (radeonsi)

2018-03-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105171 Marek Olšák changed: What|Removed |Added Resolution|--- |WORKSFORME Status|NEW

[Mesa-dev] [Bug 105171] performance regression (3x slower) running glamor with PutImage workload (radeonsi)

2018-03-15 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105171 Clemens Eisserer changed: What|Removed |Added Resolution|WORKSFORME |--- Status|RESOLVED

[Mesa-dev] [PATCH] loader: Include include/drm-uapi in the autotools build

2018-03-15 Thread Jason Ekstrand
We're already including it in the meson build. This fixes build issues on systems which have a drm_fourcc.h that doesn't have modifiers. --- src/loader/Makefile.am | 1 + 1 file changed, 1 insertion(+) diff --git a/src/loader/Makefile.am b/src/loader/Makefile.am index a0e52c1..7625256 100644 ---

Re: [Mesa-dev] [PATCH] loader: Include include/drm-uapi in the autotools build

2018-03-15 Thread Dylan Baker
Quoting Jason Ekstrand (2018-03-15 14:29:55) > We're already including it in the meson build. This fixes build issues > on systems which have a drm_fourcc.h that doesn't have modifiers. > --- > src/loader/Makefile.am | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/src/loader/Makefile.am

[Mesa-dev] [PATCH v2 2/4] anv/gen10: Enable object level preemption.

2018-03-15 Thread Rafael Antognolli
Set bit when initializing a device. Signed-off-by: Rafael Antognolli --- src/intel/vulkan/genX_state.c | 18 ++ 1 file changed, 18 insertions(+) diff --git a/src/intel/vulkan/genX_state.c b/src/intel/vulkan/genX_state.c index c6e54046910..3b5ac10b4cd 100644 --- a/src/intel/vulka

[Mesa-dev] [PATCH v2 3/4] i965/gen10+: Enable object level preemption.

2018-03-15 Thread Rafael Antognolli
Set bit when initializing context. Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/brw_context.h | 2 ++ src/mesa/drivers/dri/i965/brw_defines.h | 5 + src/mesa/drivers/dri/i965/brw_state.h| 3 ++- src/mesa/drivers/dri/i965/brw_state_upload.c | 25

[Mesa-dev] [PATCH v2 1/4] intel/genxml: Add register for object preemption.

2018-03-15 Thread Rafael Antognolli
Signed-off-by: Rafael Antognolli --- src/intel/genxml/gen10.xml | 8 src/intel/genxml/gen11.xml | 8 src/intel/genxml/gen9.xml | 8 3 files changed, 24 insertions(+) diff --git a/src/intel/genxml/gen10.xml b/src/intel/genxml/gen10.xml index 2d36957c2a5..f6773074a61 10

  1   2   >