Re: [Mesa-dev] Status of radeonsi NIR

2018-04-04 Thread Timothy Arceri
On 04/04/18 21:51, Benedikt Schemmer wrote: Hi Timothy, thanks for looking into this. Dead Island still crashes for me with NIR. Just to make sure, are you building a 32bit version of Mesa? If not you might be running your system Mesa. If you are definitely building Mesa 32bit then I'll

[Mesa-dev] [Bug 105846] Assertion failure @ st_atom_array.c:675 when playing Natural Selection 2

2018-04-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105846 --- Comment #7 from Timothy Arceri --- (In reply to las from comment #6) > Another attempt at sharing my coredump: > https://drive.google.com/open?id=11no6HF0WfEwwlE2IoeMx6aigsJ-504qp > > I'll also try to upload it here

Re: [Mesa-dev] Status of radeonsi NIR

2018-04-04 Thread Benedikt Schemmer
Hi Timothy, thanks for looking into this. Dead Island still crashes for me with NIR. However when I attach apitrace it behaves even more strangely, both TGSI and NIR crash. TGSI gets a little further. Without apitrace I can play the game with TGSI and it reproducibly crashes during shader

[Mesa-dev] [Bug 105871] Discolored KDE panels after updating to Mesa 18.0 on Intel broadwell

2018-04-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105871 --- Comment #5 from sergio.calleg...@gmail.com --- In my case: KDE Frameworks 5.44.0 Qt 5.9.1 (built against 5.9.1) KDE Plasma 5.12.4 kernel 4.13.0-38-generic (ubuntu) Xorg 1.19.5 libdrm 2.4.91 + git mesa Version: 18.0.0 intel driver 2.99.917 +

[Mesa-dev] [Bug 105871] Discolored KDE panels after updating to Mesa 18.0 on Intel broadwell

2018-04-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105871 sergio.calleg...@gmail.com changed: What|Removed |Added CC||sergio.calleg...@gmail.com

Re: [Mesa-dev] [PATCH 1/2] compiler/nir: add nir_intrinsics.[ch] to .gitignore

2018-04-04 Thread Rob Clark
On Wed, Apr 4, 2018 at 3:25 AM, Alejandro Piñeiro wrote: > Generated since > 76dfed8ae2d5c6c509eb2661389be3c6a25077df > "nir: mako all the intrinsics" oh, people still do in-tree builds? I've been spoiled by meson forcing out-of-tree builds ;-) Reviewed-by: Rob Clark

[Mesa-dev] [AppVeyor] mesa master #7374 completed

2018-04-04 Thread AppVeyor
Build mesa 7374 completed Commit 1beb80cb56 by Lionel Landwerlin on 4/3/2018 1:41 PM: intel: compiler: silence compiler warning\n\n../src/intel/compiler/brw_reg.h: In function ‘bool brw_regs_negative_equal(const brw_reg*, const

[Mesa-dev] [Bug 105846] Assertion failure @ st_atom_array.c:675 when playing Natural Selection 2

2018-04-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105846 --- Comment #6 from l...@protonmail.ch --- Another attempt at sharing my coredump: https://drive.google.com/open?id=11no6HF0WfEwwlE2IoeMx6aigsJ-504qp I'll also try to upload it here again, although it will probably take a long time (am

[Mesa-dev] [Bug 105871] Discolored KDE panels after updating to Mesa 18.0 on Intel broadwell

2018-04-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105871 --- Comment #4 from sergio.calleg...@gmail.com --- Disabling compositing, all colors go back to normal. The issue seems to be realated to translucent elements. -- You are receiving this mail because: You are the QA Contact for the bug. You are

[Mesa-dev] [Bug 105846] Assertion failure @ st_atom_array.c:675 when playing Natural Selection 2

2018-04-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105846 --- Comment #5 from Timothy Arceri --- (In reply to las from comment #2) > link to coredump (also uploading as attachment): > https://doc-0o-4k-docs.googleusercontent.com/docs/securesc/ >

[Mesa-dev] [Bug 105871] Discolored KDE panels after updating to Mesa 18.0 on Intel broadwell

2018-04-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=105871 --- Comment #3 from sergio.calleg...@gmail.com --- Seeing exactly the same on kubuntu 17.10 with the padoka ppa (bringing mesa 18.0.0) and the kde ppa on a haswell laptop. Many gray items now are much darker. -- You are receiving this mail

Re: [Mesa-dev] [PATCH 1/4] nir/split_var_copies: handle IMAGE and SAMPLER for bindless vars

2018-04-04 Thread Timothy Arceri
On 04/04/18 20:20, Karol Herbst wrote: On Wed, Apr 4, 2018 at 2:23 AM, Jason Ekstrand wrote: I have a very strong feeling that this isn't the only place where reading/writing IMAGE and SAMPLER variables is going to cause NIR heartburn. For example, we have special cases

Re: [Mesa-dev] [PATCH 1/4] nir/split_var_copies: handle IMAGE and SAMPLER for bindless vars

2018-04-04 Thread Karol Herbst
On Wed, Apr 4, 2018 at 2:23 AM, Jason Ekstrand wrote: > I have a very strong feeling that this isn't the only place where > reading/writing IMAGE and SAMPLER variables is going to cause NIR heartburn. > For example, we have special cases in nir_validate for SUBROUTINE

Re: [Mesa-dev] [PATCH 2/4] nir: add support for bindless_texture samplers

2018-04-04 Thread Karol Herbst
On Wed, Apr 4, 2018 at 2:16 AM, Jason Ekstrand wrote: > On Tue, Apr 3, 2018 at 6:21 AM, Karol Herbst wrote: >> >> Signed-off-by: Karol Herbst >> --- >> src/compiler/glsl/glsl_to_nir.cpp | 17 +++-- >>

Re: [Mesa-dev] [PATCH 02/17] ac/surface: don't set the display flag for obviously unsupported cases

2018-04-04 Thread Michel Dänzer
On 2018-04-04 03:59 AM, Marek Olšák wrote: > From: Marek Olšák > > This enables the tile swizzle for some cases of the displayable micro mode, > and it also fixes an addrlib assertion failure on Vega. > --- > src/amd/common/ac_surface.c | 18 ++ > 1 file

[Mesa-dev] [PATCH 2/3] radv: rename radv_emit_prefetch() to radv_emit_prefetch_L2()

2018-04-04 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_cmd_buffer.c | 30 +++--- 1 file changed, 11 insertions(+), 19 deletions(-) diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c index 684f2a239a..393d9ba4f4

[Mesa-dev] [PATCH 3/3] radv: implement a fast prefetch path for the vertex stage

2018-04-04 Thread Samuel Pitoiset
This allows to start draws as soon as possible. Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_cmd_buffer.c | 40 ++-- 1 file changed, 30 insertions(+), 10 deletions(-) diff --git a/src/amd/vulkan/radv_cmd_buffer.c

[Mesa-dev] [PATCH 1/3] radv: use a mask for VBOs and shaders prefetching

2018-04-04 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_cmd_buffer.c | 75 ++-- src/amd/vulkan/radv_private.h| 3 +- 2 files changed, 51 insertions(+), 27 deletions(-) diff --git a/src/amd/vulkan/radv_cmd_buffer.c

Re: [Mesa-dev] [PATCH 15/17] radeonsi: always prefetch later shaders after the draw packet

2018-04-04 Thread Samuel Pitoiset
On 04/04/2018 03:59 AM, Marek Olšák wrote: From: Marek Olšák so that the draw is started as soon as possible. --- src/gallium/drivers/radeonsi/si_cp_dma.c | 68 ++-- src/gallium/drivers/radeonsi/si_pipe.h | 2 +-

Re: [Mesa-dev] [PATCH 2/2] radv: don't use the SPI barrier management bug workaround

2018-04-04 Thread Bas Nieuwenhuizen
Reviewed-by: Bas Nieuwenhuizen for the series. On Wed, Apr 4, 2018 at 10:55 AM, Samuel Pitoiset wrote: > Ported from RadeonSI. > > Signed-off-by: Samuel Pitoiset > --- > src/amd/vulkan/radv_pipeline.c | 5 + >

Re: [Mesa-dev] [PATCH] radv: do not always disable dual quad mode when chip has RbPlus

2018-04-04 Thread Bas Nieuwenhuizen
Reviewed-by: Bas Nieuwenhuizen On Thu, Mar 29, 2018 at 2:51 PM, Samuel Pitoiset wrote: > For GFX9+ only, RadeonSI does this too. > > Signed-off-by: Samuel Pitoiset > --- > src/amd/vulkan/radv_pipeline.c | 20

Re: [Mesa-dev] [PATCH] intel: compiler: silence compiler warning

2018-04-04 Thread Iago Toral
Reviewed-by: Iago Toral Quiroga On Tue, 2018-04-03 at 15:41 +0100, Lionel Landwerlin wrote: > ../src/intel/compiler/brw_reg.h: In function ‘bool > brw_regs_negative_equal(const brw_reg*, const brw_reg*)’: > ../src/intel/compiler/brw_reg.h:305:1: warning: control reaches end >

Re: [Mesa-dev] [PATCH 1/2] gallium/util: implement util_format_is_yuv

2018-04-04 Thread Christian Gmeiner
2018-03-29 16:15 GMT+02:00 Lucas Stach : > This adds a helper to check if a pipe format is in YUV color space. > Drivers want to know about this, as YUV mostly needs special handling. > > Signed-off-by: Lucas Stach Reviewed-by: Christian Gmeiner

Re: [Mesa-dev] [PATCH 2/2] etnaviv: advertise YUV formats as external only

2018-04-04 Thread Christian Gmeiner
2018-03-29 16:15 GMT+02:00 Lucas Stach : > We only support importing YUV as OES external resources. > This will change in the future, but for now this fixes the > advertised capabilities in eglQueryDmaBufModifiersEXT. > > Signed-off-by: Lucas Stach

[Mesa-dev] [PATCH 2/2] radv: don't use the SPI barrier management bug workaround

2018-04-04 Thread Samuel Pitoiset
Ported from RadeonSI. Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_pipeline.c | 5 + 1 file changed, 5 insertions(+) diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_pipeline.c index 5a44efb78b..89a643a1b1 100644 ---

[Mesa-dev] [PATCH 1/2] radv: mask out high VM address bits in registers where needed

2018-04-04 Thread Samuel Pitoiset
Ported from RadeonSI. Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_cmd_buffer.c | 18 +- src/amd/vulkan/radv_device.c | 2 +- src/amd/vulkan/radv_pipeline.c | 18 +- 3 files changed, 19 insertions(+), 19 deletions(-)

Re: [Mesa-dev] [PATCH 1/2] gallium/util: implement util_format_is_yuv

2018-04-04 Thread Philipp Zabel
On Thu, 2018-03-29 at 16:15 +0200, Lucas Stach wrote: > This adds a helper to check if a pipe format is in YUV color space. > Drivers want to know about this, as YUV mostly needs special handling. > > Signed-off-by: Lucas Stach > --- >

Re: [Mesa-dev] [PATCH 2/2] etnaviv: advertise YUV formats as external only

2018-04-04 Thread Philipp Zabel
On Thu, 2018-03-29 at 16:15 +0200, Lucas Stach wrote: > We only support importing YUV as OES external resources. > This will change in the future, but for now this fixes the > advertised capabilities in eglQueryDmaBufModifiersEXT. > > Signed-off-by: Lucas Stach > --- >

[Mesa-dev] [PATCH v3 4/4] swr: Fix include for createPromoteMemoryToRegisterPass

2018-04-04 Thread Mike Lothian
Include llvm/Transforms/Utils.h with the newest LLVM 7 v2: Include with " " rather than < > (Vinson Lee) v3: Use LLVM_VERSION_MAJOR rather than HAVE_LLVM (George Kyriazis) Signed-of-by: Mike Lothian Tested-by: Vinson Lee ---

Re: [Mesa-dev] [PATCH] i965: Extend the negative 32-bit deltas to 64-bits

2018-04-04 Thread Kenneth Graunke
On Tuesday, April 3, 2018 11:56:27 PM PDT Sergii Romantsov wrote: > Hello, Mark. > I've done: Cc, Tested-by and Reviewed-by also added. I pushed your patch. Thanks so much for tracking this down! signature.asc Description: This is a digitally signed message part.

[Mesa-dev] [PATCH 1/2] compiler/nir: add nir_intrinsics.[ch] to .gitignore

2018-04-04 Thread Alejandro Piñeiro
Generated since 76dfed8ae2d5c6c509eb2661389be3c6a25077df "nir: mako all the intrinsics" --- src/compiler/nir/.gitignore | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/compiler/nir/.gitignore b/src/compiler/nir/.gitignore index 64828eba6d3..dea9ada80fa 100644 ---

[Mesa-dev] [PATCH 2/2] anv: add anv_extensions.h in gitignore

2018-04-04 Thread Alejandro Piñeiro
Generated since: dd088d4bec74f37ffe4cd02626a6a8af93fdebac "anv/extensions: Generate a header file with extension tables" --- src/intel/vulkan/.gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/src/intel/vulkan/.gitignore b/src/intel/vulkan/.gitignore index 4ea978d6e41..4036a1bfe3c

Re: [Mesa-dev] [PATCH] i965: Extend the negative 32-bit deltas to 64-bits

2018-04-04 Thread Sergii Romantsov
Hello, Mark. I've done: Cc, Tested-by and Reviewed-by also added. On Wed, Apr 4, 2018 at 8:16 AM, Mark Janes wrote: > This patch passes Intel's CI suites. > It needs a CC for stable in the commit message. > > Tested-by: Mark Janes > > Sergii

[Mesa-dev] [PATCH v3] i965: Extend the negative 32-bit deltas to 64-bits

2018-04-04 Thread Sergii Romantsov
Gen8+ use 48-bit address relocations so need to extend the sign to 64-bit return value. Without it we have higher bits zeroed and missing the negavive values. Haswell and older use 32-bit deltas so are unaffected by this issue. v2: used int32_t fucntion parameter instead of explicit type

Re: [Mesa-dev] [PATCH] vbo: Use alloca for _vbo_draw_indirect.

2018-04-04 Thread Mathias Fröhlich
Hi, > This adds uses of alloca, without a corresponding include of alloca.h. > > Perhaps something like the attached is needed? Yes, sorry! Reviewed-by: Mathias Fröhlich best Mathias ___ mesa-dev mailing list

Re: [Mesa-dev] [PATCH 02/11] gallium: Use _DrawVAO for edgeflag enabled check.

2018-04-04 Thread Mathias Fröhlich
Hi, On Tuesday, 3 April 2018 22:23:55 CEST Roland Scheidegger wrote: > Not a review, but these aren't gallium changes, but rather state tracker > ones, so should be st/mesa rather than gallium. > (gallium would be changes to the gallium interface.) Ok, I have changed that! Thanks! BTW I have

[Mesa-dev] [AppVeyor] mesa master #7373 failed

2018-04-04 Thread AppVeyor
Build mesa 7373 failed Commit 41ac0b1443 by Iago Toral Quiroga on 4/2/2018 9:39 AM: compiler/spirv: set is_shadow for depth comparitor sampling opcodes\n\nFrom the SPIR-V spec, OpTypeImage:\n\n"Depth is whether or not this image is a depth image. (Note that\n

<    1   2