Re: [Mesa-dev] [Mesa-stable] [PATCH 3/3] amd: Apply elf relocations and allow code with relocations

2019-06-13 Thread Jan Vesely
On Thu, 2019-06-13 at 21:20 +0200, Dieter Nützel wrote: > Am 13.06.2019 07:10, schrieb Marek Olšák: > > FYI, I just pushed the new linker. > > > > Marek > > Thank you very much Marek and _Nicolai_ for this GREAT stuff. > It brings back some speed after 1/8 drop with glmark2, lately. > Maybe my am

[Mesa-dev] [PATCH] meson: Add dep_thread dependency.

2019-06-13 Thread Vinson Lee
Fix this build error on Ubuntu 18.04. /usr/bin/ld: src/util/libmesa_util.a(u_cpu_detect.c.o): undefined reference to symbol 'pthread_once@@GLIBC_2.2.5' Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=110663 Suggested-by: Eric Engestrom Signed-off-by: Vinson Lee --- src/gallium/tests/tr

Re: [Mesa-dev] [PATCH 7/7] radeonsi: don't use the low-optimizing compiler on LLVM 9

2019-06-13 Thread Dieter Nützel
For the series Tested-by: Dieter Nützel on Polaris 20 Have a look here, too. https://www.phoronix.com/forums/forum/phoronix/latest-phoronix-articles/1106085-linux-kernel-set-to-expose-hidden-nvidia-hda-controllers-helping-laptop-users?p=1106199#post1106199 Dieter Am 13.06.2019 02:40, schrieb

Re: [Mesa-dev] [Mesa-stable] [PATCH 3/3] amd: Apply elf relocations and allow code with relocations

2019-06-13 Thread Dieter Nützel
Am 13.06.2019 07:10, schrieb Marek Olšák: FYI, I just pushed the new linker. Marek Thank you very much Marek and _Nicolai_ for this GREAT stuff. It brings back some speed after 1/8 drop with glmark2, lately. Maybe my amd-staging-drm-next tree (5.2-rc1) didn't honor the kernel mitigation param

[Mesa-dev] [Bug 99553] Tracker bug for runnning OpenCL applications on Clover

2019-06-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99553 Bug 99553 depends on bug 105113, which changed state. Bug 105113 Summary: [hawaii, radeonsi, clover] Running Piglit cl/program/execute/{,tail-}calls{,-struct,-workitem-id}.cl cause GPU VM error and ring stalled GPU lockup https://bugs.freedeskto

Re: [Mesa-dev] [Mesa-stable] [PATCH 3/3] amd: Apply elf relocations and allow code with relocations

2019-06-13 Thread Jan Vesely
On Thu, 2019-06-13 at 01:10 -0400, Marek Olšák wrote: > FYI, I just pushed the new linker. thanks. I've checked the handling and the current approach works for sections as well (even if not handled explicitly). Jan > > Marek > > On Mon, Jun 3, 2019 at 10:39 PM Jan Vesely wrote: > > > Fixes p

Re: [Mesa-dev] [PATCH 2/2] ac: make ac_compute_cmask() a static function

2019-06-13 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Thu, Jun 13, 2019 at 11:14 AM Samuel Pitoiset wrote: > Signed-off-by: Samuel Pitoiset > --- > src/amd/common/ac_surface.c | 6 +++--- > src/amd/common/ac_surface.h | 4 > 2 files changed, 3 insertions(+), 7 deletions(-) > > diff --git a/src/amd/common/a

Re: [Mesa-dev] [PATCH] gallium: add z24s8_as_r8g8b8a8 format

2019-06-13 Thread Marek Olšák
I'm OK with this. Reviewed-by: Marek Olšák Marek On Thu, Jun 13, 2019 at 1:39 PM Rob Clark wrote: > From: Rob Clark > > This maps to a special format that recent generations of adreno have, > for blitting z24s8. Conceptually it is similar to doing Z and/or S > blits by pretending it is r8g8

Re: [Mesa-dev] [PATCH] include: update GL headers from the registry

2019-06-13 Thread Marek Olšák
Kind ping. Marek On Tue, Jun 11, 2019 at 8:06 PM Marek Olšák wrote: > From: Marek Olšák > > --- > include/GL/glcorearb.h | 261 +++ > include/GL/glext.h | 269 + > include/GLES/gl.h | 21 +--- > include/GLES

[Mesa-dev] [Bug 110697] glXWaitForMscOML and glXWaitVideoSyncSGI may block indefinitely

2019-06-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=110697 --- Comment #13 from QwertyChouskie --- (In reply to Michel Dänzer from comment #12) > (In reply to QwertyChouskie from comment #11) > > Whoops, missed that they were posted in a comment further down. Here they > > are again for reference: > >

Re: [Mesa-dev] [PATCH] ac: add radeon_info::is_amdgpu instead of checking drm_major == 3

2019-06-13 Thread Marek Olšák
It's only a random cleanup. Marek On Thu, Jun 13, 2019 at 2:57 AM Samuel Pitoiset wrote: > Why do you need that? > > On 6/12/19 11:31 PM, Marek Olšák wrote: > > From: Marek Olšák > > > > and clean up > > --- > > src/amd/common/ac_gpu_info.c | 13 -- > > src/amd/common/a

Re: [Mesa-dev] [PATCH] panfrost: Fix general purpose varying handling

2019-06-13 Thread Alyssa Rosenzweig
Reviewed-by: Alyssa Rosenzweig ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH] gallium: add z24s8_as_r8g8b8a8 format

2019-06-13 Thread Rob Clark
From: Rob Clark This maps to a special format that recent generations of adreno have, for blitting z24s8. Conceptually it is similar to doing Z and/or S blits by pretending it is r8g8b8a8 (with appropriate writemask). But it differs when bandwidth compression is used, as z24 is a different type

Re: [Mesa-dev] [PATCH 1/2] panfrost: ci: Update test expectations

2019-06-13 Thread Alyssa Rosenzweig
Both R-b, thank you! Wonder what's up with the RK3288 regressions ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH 1/2] panfrost: ci: Update test expectations

2019-06-13 Thread Tomeu Vizoso
Some tests got fixed since the last update, but also some regressions crept in. To keep the CI green, add the regressions to the expected failures. Signed-off-by: Tomeu Vizoso --- .../drivers/panfrost/ci/expected-failures.txt | 98 +-- 1 file changed, 5 insertions(+), 93 deletio

[Mesa-dev] [PATCH 2/2] panfrost: ci: Exclude some tests from results

2019-06-13 Thread Tomeu Vizoso
These are tests that regressed in RK3288 but still pass on RK3399. So we still have a CI we can rely on, add them to the flip-flop list for now. Signed-off-by: Tomeu Vizoso --- src/gallium/drivers/panfrost/ci/gitlab-ci.yml | 51 ++- 1 file changed, 50 insertions(+), 1 deletion(-

[Mesa-dev] [PATCH] panfrost: Fix general purpose varying handling

2019-06-13 Thread Boris Brezillon
When both the fragment and vertex shaders point to the same varying location they expect to share the same varying slot. Make sure vertex and fragment varyings pointing to the same loc have ->src_offset set to the same value. Signed-off-by: Boris Brezillon --- Found the bug while debugging galliu

[Mesa-dev] [PATCH 2/2] ac: make ac_compute_cmask() a static function

2019-06-13 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/amd/common/ac_surface.c | 6 +++--- src/amd/common/ac_surface.h | 4 2 files changed, 3 insertions(+), 7 deletions(-) diff --git a/src/amd/common/ac_surface.c b/src/amd/common/ac_surface.c index 28256352cb7..7b86cb1e1f2 100644 --- a/src/amd/common/ac_s

[Mesa-dev] [PATCH 1/2] radv: rely on ac_compute_cmask() for CMASK info

2019-06-13 Thread Samuel Pitoiset
Instead of re-computing in the driver. The 3d and cube flags are correctly set, so the same values should returned by ac_compute_surface(). Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_image.c | 45 +++-- 1 file changed, 3 insertions(+), 42 deletions(-)

[Mesa-dev] [Bug 110760] Low FPS in Quake Champions with Vega20

2019-06-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=110760 --- Comment #9 from network...@rkmail.ru --- (In reply to Samuel Pitoiset from comment #8) > This is not a bug and performance for that game is quite acceptable. > Closing. How does this is 'not a bug' and 'acceptable' if much more powerful gpu

[Mesa-dev] [PATCH] radv: implement compressed FMASK texture reads with RADV_PERFTEST=tccompatcmask

2019-06-13 Thread Samuel Pitoiset
This allows us to disable the FMASK decompress pass when transitioning from CB writes to shader reads. This will likely be improved and enabled by default in the future. No CTS regressions on GFX8 but a few number of multisample CTS failures on GFX9 (they look related to the small hint). Signed-

[Mesa-dev] [Bug 110815] Segfault vkCreateDescriptorPool in The-Forge on RADV

2019-06-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=110815 Samuel Pitoiset changed: What|Removed |Added Resolution|--- |NOTOURBUG Status|NEW

[Mesa-dev] [Bug 110760] Low FPS in Quake Champions with Vega20

2019-06-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=110760 Samuel Pitoiset changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Mesa-dev] [PATCH] radv: emit framebuffer state from primary if secondary doesn't herit it

2019-06-13 Thread Samuel Pitoiset
Otherwise fast color/depth clears can't work because they depend on the framebuffer. This fixes the following CTS (when the small hint is disabled): - dEQP-VK.geometry.layered.1d_array.secondary_cmd_buffer - dEQP-VK.geometry.layered.2d_array.secondary_cmd_buffer - dEQP-VK.geometry.layered.cube.sec

Re: [Mesa-dev] mirroring ogl-sample source code in gitlab?

2019-06-13 Thread Erik Faye-Lund
On Wed, 2019-06-12 at 14:03 -0400, Adam Jackson wrote: > On Wed, 2019-06-12 at 10:29 +0200, Erik Faye-Lund wrote: > > On Wed, 2019-06-12 at 10:25 +0200, Erik Faye-Lund wrote: > > > On Tue, 2019-06-11 at 12:41 -0400, Adam Jackson wrote: > > > > On Tue, 2019-06-11 at 11:13 +0200, Erik Faye-Lund wrote

[Mesa-dev] [PATCH] radv: make sure to init the DCC decompress compute path state

2019-06-13 Thread Samuel Pitoiset
This fixes a segfault when forcing DCC decompressions on compute because internal meta objects are not created since the on-demand stuff. Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_meta_fast_clear.c | 8 1 file changed, 8 insertions(+) diff --git a/src/amd/vulkan/radv_meta_

Re: [Mesa-dev] [PATCH] radv: flush and invalidate CB before resetting query pools on GFX9

2019-06-13 Thread Samuel Pitoiset
On 6/13/19 11:14 AM, Bas Nieuwenhuizen wrote: Add DB while we are at it? Pushed with DB added. Otherwise, r-b On Thu, Jun 13, 2019 at 10:51 AM Samuel Pitoiset wrote: We have to emit a CACHE_FLUSH_AND_INV_TS_EVENT to be sure all prior GPU work is done. This fixes the following CTS (when th

Re: [Mesa-dev] [PATCH] radv: flush and invalidate CB before resetting query pools on GFX9

2019-06-13 Thread Bas Nieuwenhuizen
Add DB while we are at it? Otherwise, r-b On Thu, Jun 13, 2019 at 10:51 AM Samuel Pitoiset wrote: > > We have to emit a CACHE_FLUSH_AND_INV_TS_EVENT to be sure all > prior GPU work is done. > > This fixes the following CTS (when the small hint is disabled): > dEQP-VK.query_pool.statistics_query.

[Mesa-dev] [PATCH] radv: flush and invalidate CB before resetting query pools on GFX9

2019-06-13 Thread Samuel Pitoiset
We have to emit a CACHE_FLUSH_AND_INV_TS_EVENT to be sure all prior GPU work is done. This fixes the following CTS (when the small hint is disabled): dEQP-VK.query_pool.statistics_query.reset_before_copy.* Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_query.c | 2 ++ 1 file changed, 2

[Mesa-dev] [Bug 110697] glXWaitForMscOML and glXWaitVideoSyncSGI may block indefinitely

2019-06-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=110697 --- Comment #12 from Michel Dänzer --- (In reply to QwertyChouskie from comment #11) > Whoops, missed that they were posted in a comment further down. Here they > are again for reference: I did follow those steps. -- You are receiving this m

Re: [Mesa-dev] [PATCH 2/7] ac/registers: use better names for disambiguated definitions

2019-06-13 Thread Samuel Pitoiset
Patches 1-2 are: Reviewed-by: Samuel Pitoiset On 6/13/19 2:40 AM, Marek Olšák wrote: From: Marek Olšák --- src/amd/registers/makeregheader.py | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/amd/registers/makeregheader.py b/src/amd/registers/makeregheader.p