[Mesa-dev] [PATCH] anv: Use separate MOCS settings for external BOs on gen8

2018-07-09 Thread Jason Ekstrand
On all other platforms, it's safe to use the usual PTE settings for both internal and external BOs. On Broadwell, however, we can't get the right caching behavior for scanout without disabling eLLC and we really don't want to do this on everything. In order to do this, we add an anv-specific BO

[Mesa-dev] [Bug 107170] Build fails if building against X server/libxrandr with no leases support

2018-07-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107170 Bug ID: 107170 Summary: Build fails if building against X server/libxrandr with no leases support Product: Mesa Version: git Hardware: Other OS: All

[Mesa-dev] [Bug 106151] [amdgpu][vulkan] GPU hang (Vega 56) while running game (Rise of the Tomb Raider)

2018-07-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=106151 --- Comment #28 from pritzl3...@gmail.com --- (In reply to Samuel Pitoiset from comment #27) > I guess it also hangs with RADV_PERFTEST=nobatchchain ? Yes I just gave it a try and it hangs in the same way. I am now on kernel 4.16.13 Mesa

[Mesa-dev] [PATCH mesa] egl: drop unused _EGL_BUILT_IN_DRIVER_DRI2

2018-07-09 Thread Eric Engestrom
Unused since b174a1ae720cb404738c "egl: Simplify the "driver" interface". Signed-off-by: Eric Engestrom --- src/egl/Android.mk | 1 - src/egl/Makefile.am | 3 +-- src/egl/meson.build | 1 - 3 files changed, 1 insertion(+), 4 deletions(-) diff --git a/src/egl/Android.mk b/src/egl/Android.mk

[Mesa-dev] [PATCH] intel/tools/dump_gpu: Add option to print ppgtt mappings.

2018-07-09 Thread Rafael Antognolli
Using -vv will increase the verbosity, by printing the ppgtt mappings as they get written into the aub file. Cc: Lionel Landwerlin --- src/intel/tools/intel_dump_gpu.c | 25 - src/intel/tools/intel_dump_gpu.in | 6 ++ 2 files changed, 30 insertions(+), 1

[Mesa-dev] [PATCH] radv: using tls to store llvm related info and speed up compiles (v9)

2018-07-09 Thread Dave Airlie
From: Dave Airlie This uses the common compiler passes abstraction to help radv avoid fixed cost compiler overheads. This uses a linked list per thread stored in thread local storage, with an entry in the list for each target machine. This should remove all the fixed overheads setup costs of

Re: [Mesa-dev] [PATCH] radv: using tls to store llvm related info and speed up compiles (v8)

2018-07-09 Thread Bas Nieuwenhuizen
On Mon, Jul 9, 2018 at 11:03 PM, Dave Airlie wrote: > On 9 July 2018 at 05:58, Bas Nieuwenhuizen wrote: >> On Thu, Jul 5, 2018 at 2:03 AM, Dave Airlie wrote: >>> From: Dave Airlie >>> >>> This uses the common compiler passes abstraction to help radv >>> avoid fixed cost compiler overheads.

Re: [Mesa-dev] [PATCH] radv: using tls to store llvm related info and speed up compiles (v8)

2018-07-09 Thread Dave Airlie
On 9 July 2018 at 05:58, Bas Nieuwenhuizen wrote: > On Thu, Jul 5, 2018 at 2:03 AM, Dave Airlie wrote: >> From: Dave Airlie >> >> This uses the common compiler passes abstraction to help radv >> avoid fixed cost compiler overheads. This uses a linked list per >> thread stored in thread local

Re: [Mesa-dev] [PATCH] glsl: Take sampler2DRect and sampler2DRectShadow as reserved

2018-07-09 Thread Eric Anholt
Eric Anholt writes: > [ Unknown signature status ] > zhaowei yuan writes: > >> "sampler2DRect" and "sampler2DRectShadow" are specified as >> reserved from GLSL 1.1 and GLSL ES 1.0 >> >> Signed-off-by: zhaowei yuan >> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=106906 > > It looks

Re: [Mesa-dev] [PATCH] scons: Fix CC env quoting.

2018-07-09 Thread Roland Scheidegger
Am 09.07.2018 um 21:57 schrieb Jose Fonseca: > Handle the case where there are spaces in CC env var (ie, compiler + > options.) > > This updates the CC check to match what newer SCons do internally. > --- > scons/gallium.py | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff

[Mesa-dev] [Bug 107169] [regression] Upgrade from 18.0.4 to 18.1.0 causes severe stuttering in games

2018-07-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107169 Bug ID: 107169 Summary: [regression] Upgrade from 18.0.4 to 18.1.0 causes severe stuttering in games Product: Mesa Version: 18.0 Hardware: x86-64 (AMD64)

[Mesa-dev] [PATCH] radv: using tls to store llvm related info and speed up compiles (v9)

2018-07-09 Thread Dave Airlie
From: Dave Airlie This uses the common compiler passes abstraction to help radv avoid fixed cost compiler overheads. This uses a linked list per thread stored in thread local storage, with an entry in the list for each target machine. This should remove all the fixed overheads setup costs of

Re: [Mesa-dev] [PATCH] nir: delete not needed for reinserted nir_cf_list

2018-07-09 Thread Jason Ekstrand
Yeah, I think this is probably better. Reviewed-by: Jason Ekstrand On Thu, Jul 5, 2018 at 5:50 PM Caio Marcelo de Oliveira Filho < caio.olive...@intel.com> wrote: > It wasn't causing problems since there's nothing to delete, but better > be consistent with the rest of existing codebase. > ---

Re: [Mesa-dev] [PATCH v4] virgl: Add support for glGetMultisample

2018-07-09 Thread Dave Airlie
On 29 June 2018 at 20:57, Erik Faye-Lund wrote: > On Fri, Jun 29, 2018 at 12:52 PM Erik Faye-Lund wrote: >> >> On Fri, Jun 29, 2018 at 12:39 PM Gert Wollny >> wrote: >> > >> > Use caps to obtain the multisample sample positions for up to 16 >> > positions and implement the according Gallium

Re: [Mesa-dev] [PATCH] intel/tools/dump_gpu: Add option to print ppgtt mappings.

2018-07-09 Thread Lionel Landwerlin
Thanks! Reviewed-by: Lionel Landwerlin On 09/07/18 21:28, Rafael Antognolli wrote: Using -vv will increase the verbosity, by printing the ppgtt mappings as they get written into the aub file. Cc: Lionel Landwerlin --- src/intel/tools/intel_dump_gpu.c | 25 -

Re: [Mesa-dev] [PATCH 1/2] intel/batch_decoder: Recurse for all 2nd level batches

2018-07-09 Thread Lionel Landwerlin
Reviewed-by: Lionel Landwerlin On 10/07/18 00:01, Jason Ekstrand wrote: Our attempt to restart the loop with the second level batch worked at one point but got broken at some point. It was too fragile anyway and we're not likely to have enough secondaries to actually overflow the stack so we

Re: [Mesa-dev] [PATCH 2/2] intel/aubinator_error_decode: Allow for more sections

2018-07-09 Thread Lionel Landwerlin
Reviewed-by: Lionel Landwerlin On 10/07/18 00:01, Jason Ekstrand wrote: Error states coming from actual Vulkan applications tend to have fairly long command buffers and lots of chained batches. 30 total BOs isn't nearly enough. This commit bumps it to 256, makes some things use the actual

[Mesa-dev] [Bug 106644] [llvmpipe] Mesa 18.1.2 fails lp_test_format, lp_test_arit, lp_test_blend, lp_test_printf, lp_test_conv tests

2018-07-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=106644 --- Comment #33 from Ben Crocker --- AFAIK, there IS PPC32 LE; we just don't support it in Red Hat products. So, I can't tell you much more. Sorry! What are the chances you can convert to using PPC64LE across the board? -- You are receiving

Re: [Mesa-dev] [PATCH] radv: using tls to store llvm related info and speed up compiles (v9)

2018-07-09 Thread Bas Nieuwenhuizen
Reviewed-by: Bas Nieuwenhuizen On Mon, Jul 9, 2018 at 11:14 PM, Dave Airlie wrote: > From: Dave Airlie > > This uses the common compiler passes abstraction to help radv > avoid fixed cost compiler overheads. This uses a linked list per > thread stored in thread local storage, with an entry in

[Mesa-dev] [PATCH] gallium/u_transfer_helper: Fix MSAA mappings with nonzero x/y.

2018-07-09 Thread Eric Anholt
We created a temporary with box->{width,height} and then tried to map width,height from a nonzero offset when we meant to just map the whole temporary. Fixes segfaults in V3D in dEQP-GLES3.functional.prerequisite.read_pixels with --deqp-egl-config-name=rgbad24s8ms4 and also piglit's

[Mesa-dev] [Bug 106644] [llvmpipe] Mesa 18.1.2 fails lp_test_format, lp_test_arit, lp_test_blend, lp_test_printf, lp_test_conv tests

2018-07-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=106644 --- Comment #32 from Ben Crocker --- I suspect that, if you run your Talos II big-endian, you will have the same results as on your G5; big-endian machines have not had the attention from the LLVM developers that little-endian machines have. I

Re: [Mesa-dev] [PATCH v2 05/11] util/set: add a basic unit test

2018-07-09 Thread Caio Marcelo de Oliveira Filho
On Mon, Jul 09, 2018 at 10:53:19AM -0700, Caio Marcelo de Oliveira Filho wrote: > --- > src/util/Makefile.am| 3 +- > src/util/meson.build| 1 + > src/util/tests/set/Makefile.am | 42 + > src/util/tests/set/meson.build | 30 ++ >

[Mesa-dev] [PATCH 2/2] intel/aubinator_error_decode: Allow for more sections

2018-07-09 Thread Jason Ekstrand
Error states coming from actual Vulkan applications tend to have fairly long command buffers and lots of chained batches. 30 total BOs isn't nearly enough. This commit bumps it to 256, makes some things use the actual number of sections instead of the #define, and adds asserts if we ever go over

[Mesa-dev] [PATCH 1/2] intel/batch_decoder: Recurse for all 2nd level batches

2018-07-09 Thread Jason Ekstrand
Our attempt to restart the loop with the second level batch worked at one point but got broken at some point. It was too fragile anyway and we're not likely to have enough secondaries to actually overflow the stack so we may as well recurse in both cases. --- src/intel/common/gen_batch_decoder.c

Re: [Mesa-dev] [PATCH 1/2] glsl: slim the kill_entry struct used in const propagation

2018-07-09 Thread Eric Anholt
Caio Marcelo de Oliveira Filho writes: > Since 4654439fdd7 "glsl: Use hash tables for > opt_constant_propagation() kill sets." uses a hash_table for storing > kill_entries, so the structs can be simplified. > > Remove the exec_node from kill_entry since it is not used in an > exec_list anymore.

[Mesa-dev] [PATCH] nir: Add flipping of gl_PointCoord.y in nir_lower_wpos_ytransform.

2018-07-09 Thread Eric Anholt
This is controlled by a new nir_shader_compiler_options flag, and fixes dEQP-GLES3.functional.shaders.builtin_variable.pointcoord on V3D. --- src/broadcom/compiler/nir_to_vir.c | 1 + src/compiler/nir/nir.h | 3 ++ src/compiler/nir/nir_lower_wpos_ytransform.c |

Re: [Mesa-dev] [PATCH 32/53] intel/fs: Mark LINTERP opcode as writing accumulator implicitly on pre-Gen7.

2018-07-09 Thread Jason Ekstrand
https://gitlab.freedesktop.org/jekstrand/mesa/commit/e4928d31f13d5501f03979159711c0f4d08005aa On Mon, Jul 9, 2018 at 11:16 AM Dylan Baker wrote: > Quoting Dylan Baker (2018-06-29 08:28:09) > > Quoting Jason Ekstrand (2018-05-24 14:56:14) > > > From: Francisco Jerez > > > > > > --- > > >

[Mesa-dev] [PATCH 2/2] glsl: remove struct kill_entry in constant propagation

2018-07-09 Thread Caio Marcelo de Oliveira Filho
The only value in kill_entry is the writemask, which can be stored in the data pointer of the hash table entry. Suggested by Eric Anholt. --- Kept this as a separate patch since I felt the casting would be controversial. .../glsl/opt_constant_propagation.cpp | 33 --- 1

[Mesa-dev] [PATCH 1/2] glsl: slim the kill_entry struct used in const propagation

2018-07-09 Thread Caio Marcelo de Oliveira Filho
Since 4654439fdd7 "glsl: Use hash tables for opt_constant_propagation() kill sets." uses a hash_table for storing kill_entries, so the structs can be simplified. Remove the exec_node from kill_entry since it is not used in an exec_list anymore. Remove the 'var' from kill_entry since it is now

[Mesa-dev] [PATCH] scons: Fix CC env quoting.

2018-07-09 Thread Jose Fonseca
Handle the case where there are spaces in CC env var (ie, compiler + options.) This updates the CC check to match what newer SCons do internally. --- scons/gallium.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/scons/gallium.py b/scons/gallium.py index

[Mesa-dev] [PATCH] nir/vars_to_ssa: Don't build deref nodes for non-local variables

2018-07-09 Thread Jason Ekstrand
--- src/compiler/nir/nir_lower_vars_to_ssa.c | 18 ++ 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/src/compiler/nir/nir_lower_vars_to_ssa.c b/src/compiler/nir/nir_lower_vars_to_ssa.c index 3f37acaed33..cd679be5be5 100644 ---

[Mesa-dev] [PATCH mesa 3/3] radv: Add new VK_MESA_query_timestamp extension to radv driver [v2]

2018-07-09 Thread Keith Packard
This extension adds a single function to query the current GPU timestamp, just like glGetInteger64v(GL_TIMESTAMP, ). This function is needed to complete the implementation of GOOGLE_display_timing, which needs to be able to correlate GPU and CPU timestamps. Signed-off-by: Keith Packard v2:

[Mesa-dev] [PATCH 0/3] Add MESA_query_timestamp extension (v2)

2018-07-09 Thread Keith Packard
Here's an updated version of the extension to get GPU timestamps. This version returns both CPU and GPU timestamps which could (in theory) be more tightly correlated than just getting the time of one and then the other sequentially. The idea is that we'd eventually add a suitable kernel ioctl to

[Mesa-dev] [PATCH mesa 1/3] vulkan: Define new VK_MESA_query_timestamp extension [v3]

2018-07-09 Thread Keith Packard
This extension adds a single function to query the current GPU timestamp, just like glGetInteger64v(GL_TIMESTAMP, ). This function is needed to complete the implementation of GOOGLE_display_timing, which needs to be able to correlate GPU and CPU timestamps. v2: Adopt Jason Ekstrand's coding

[Mesa-dev] [PATCH mesa 2/3] anv: Add new VK_MESA_query_timestamp extension to anv driver [v3]

2018-07-09 Thread Keith Packard
This extension adds a single function to query the current GPU timestamp, just like glGetInteger64v(GL_TIMESTAMP, ). This function is needed to complete the implementation of GOOGLE_display_timing, which needs to be able to correlate GPU and CPU timestamps. v2: Adopt Jason Ekstrand's coding

Re: [Mesa-dev] [PATCH v5] i965: Fix ETC2/EAC GetCompressed* functions on Gen7 GPUs

2018-07-09 Thread Nanley Chery
On Thu, Jun 14, 2018 at 10:50:57PM +0300, Eleni Maria Stea wrote: > On 06/14/2018 10:27 PM, Nanley Chery wrote: > > > +Jason, Ken > > > > Hello, > > > > I recently did some miptree work relating to the r8stencil_mt and I > > think I now have a more informed opinion about how things should be >

[Mesa-dev] [Bug 107158] When compiling against musl libc, pthread.h is not included in radv_amdgpu_winsys

2018-07-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107158 roncha...@gmail.com changed: What|Removed |Added Status|NEEDINFO|RESOLVED Resolution|---

Re: [Mesa-dev] [PATCH] radv: winsys/amdgpu: include missing pthread.h header

2018-07-09 Thread Bas Nieuwenhuizen
I'd like to nominate this for 18.1 stable releases. On Mon, Jul 2, 2018 at 1:25 AM, Mauro Rossi wrote: > pthread types are used in some files without explicitely including pthread.h. > This leads to compile errors on Android 7.x nougat-x86 > e.g. in

[Mesa-dev] [PATCH 4/4] radv: add support for VK_EXT_conditional_rendering

2018-07-09 Thread Samuel Pitoiset
Inherited commands buffers are not supported. Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_cmd_buffer.c | 29 + src/amd/vulkan/radv_device.c | 7 +++ src/amd/vulkan/radv_extensions.py | 1 + 3 files changed, 37 insertions(+) diff --git

[Mesa-dev] [PATCH 2/4] radv: set the predicate for indirect/indexed draw commands

2018-07-09 Thread Samuel Pitoiset
VK_EXT_conditional_rendering allows to discard draw commands (not only normal draws). Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_cmd_buffer.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/amd/vulkan/radv_cmd_buffer.c

[Mesa-dev] [PATCH 3/4] radv: add support for non-inverted conditional rendering

2018-07-09 Thread Samuel Pitoiset
By default, our internal rendering commands are discarded only if the predicate is non-zero (ie. DRAW_VISIBLE). But VK_EXT_conditional_rendering also allows to discard commands when the predicate is zero, which means we have to use a different flag. Signed-off-by: Samuel Pitoiset ---

[Mesa-dev] [PATCH 1/4] radv: set the predicate for dispatch commands

2018-07-09 Thread Samuel Pitoiset
VK_EXT_conditional_rendering allows to discard dispatch commands. Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_cmd_buffer.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c index

Re: [Mesa-dev] [PATCH 1/6] configure: bump libdrm for AMDGPU to 2.4.92

2018-07-09 Thread Samuel Pitoiset
On 07/04/2018 07:26 PM, Emil Velikov wrote: Hi Samuel, On 4 July 2018 at 14:06, Samuel Pitoiset wrote: This will allow us to submit more IBs at once because the limit has been removed in libdrm 2.4.92. Signed-off-by: Samuel Pitoiset --- configure.ac | 2 +- 1 file changed, 1

Re: [Mesa-dev] [PATCH 5/6] radv/winsys: fix buffer deletion in the sysmem path

2018-07-09 Thread Samuel Pitoiset
Bas, I think patches 4&5 can already be merged? I will investigate about the CTS regressions, probably this week. On 07/04/2018 03:06 PM, Samuel Pitoiset wrote: In case we failed to submit the CS correctly. Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.c |

Re: [Mesa-dev] [PATCH] configure.ac: Check for xcb-randr version and xrandr to enable leasing

2018-07-09 Thread Danylo Piliaiev
Beside just checking for version being >= 1.13 I made sure that configuring with --enable-xlib-lease will produce error if version isn't right. Also due to small changes in autoconf the patch probably won't apply cleanly so if necessary I'll make new patch which will also include bumping

Re: [Mesa-dev] [PATCH] configure.ac: Check for xcb-randr version and xrandr to enable leasing

2018-07-09 Thread Danylo Piliaiev
Hi Lionel, Yes meson probably also has the issue since it checks for xcb-randr 1.12 and not 1.13 . I didn't send further patches since vulkan build got broken once more, see my reply to "[PATCH mesa 2/4] vulkan: add VK_EXT_display_control [v8]". Also I have upgrade to Ubuntu 18.04 and

Re: [Mesa-dev] [PATCH] configure.ac: Check for xcb-randr version and xrandr to enable leasing

2018-07-09 Thread Lionel Landwerlin
Hey Danylo, I didn't really understand why your change didn't just include a >= 1.13 in the if test x"$have_xlease" = xyes; then I guess just adding the revision there (and configure.ac) and bumping the revision in meson.build should be enough. Thanks, - Lionel On 09/07/18 10:16, Danylo

Re: [Mesa-dev] [PATCH 4/4] radv: add support for VK_EXT_conditional_rendering

2018-07-09 Thread Bas Nieuwenhuizen
Where is the code to disable predication for copy/blit commands and then enable it again afterwards? On Mon, Jul 9, 2018 at 11:42 AM, Samuel Pitoiset wrote: > Inherited commands buffers are not supported. > > Signed-off-by: Samuel Pitoiset > --- > src/amd/vulkan/radv_cmd_buffer.c | 29

[Mesa-dev] [PATCH v1 3/3] android: Build kms_swrast for the Android platform

2018-07-09 Thread Robert Foss
From: Rob Herring Signed-off-by: Rob Herring Signed-off-by: Robert Foss Reviewed-by: Emil Velikov --- Changes since RFC: - Added r-b src/gallium/Android.mk | 2 +- src/gallium/auxiliary/pipe-loader/Android.mk | 1 + src/gallium/drivers/softpipe/Android.mk |

[Mesa-dev] [PATCH v1 2/3] platform/android: Enable kms_swrast fallback

2018-07-09 Thread Robert Foss
Add support for the ForceSoftware option, which is togglable on the Android platform through setting the "drm.gpu.force_software" property to a non-zero value. kms_swrast is also enabled as a fallback for when a driver is not able to be loaded for for a drm node that was opened. Signed-off-by:

[Mesa-dev] [PATCH v1 0/3] Android kms_swrast support

2018-07-09 Thread Robert Foss
NOTE: This series has not been tested successfully, and I'm seeing a segfault during the boot process. Which I'm currently looking into. This series implements kms_swrast support for the Android platform. It's available here: https://gitlab.collabora.com/robertfoss/mesa/tree/kms_swrast_v1

[Mesa-dev] [PATCH v1 1/3] egl/android: Add Android property for forcing software rendering

2018-07-09 Thread Robert Foss
In order to simplify Android bringup on new devices, provide the property "drm.gpu.force_software" which forces kms_swrast to be used. Signed-off-by: Robert Foss Reviewed-by: Emil Velikov --- Changes since RFC: - Fixed EGLBoolean comparison - Added r-b src/egl/main/egldriver.c | 10

[Mesa-dev] [PATCH mesa] meson: fix windows build

2018-07-09 Thread Eric Engestrom
The python installer on Windows installs two `python.exe`, one for python2 and one for python3 and no `python2.exe` nor `python3.exe`. This seems utterly broken to me, but luckily for us meson handles this craziness for us, but only since version 0.46. This is higher than we require for now, but

[Mesa-dev] [PATCH] intel/compiler: remove unused function

2018-07-09 Thread Iago Toral Quiroga
--- src/intel/compiler/brw_fs.cpp | 27 --- src/intel/compiler/brw_fs.h | 4 2 files changed, 31 deletions(-) diff --git a/src/intel/compiler/brw_fs.cpp b/src/intel/compiler/brw_fs.cpp index 38a8621f2c..99b21f6d89 100644 --- a/src/intel/compiler/brw_fs.cpp +++

Re: [Mesa-dev] [PATCH] intel/fs: use uint type for per_slot_offset at GS

2018-07-09 Thread Samuel Iglesias Gonsálvez
Reviewed-by: Samuel Iglesias Gonsálvez On 12/06/18 18:51, Jose Maria Casanova Crespo wrote: > This helps us to compact original instruction: > > mul(8) g3<1>D g6<8,8,1>UD 0x0006UD { align1 1Q }; > > So now we emit: > > mul(8) g3<1>UD g6<8,8,1>UD 0x0006UD { align1 1Q compacted }; >

Re: [Mesa-dev] [PATCH] intel/compiler: emit actual barriers for working-group level barriers

2018-07-09 Thread Iago Toral
Any feedback about this? We need this to fix some new CTS tests. On Thu, 2018-06-21 at 13:52 +0200, Iago Toral Quiroga wrote: > Until now we have assumed that we could skip emitting these barriers > in the general case based on empirical testing and a few assumptions > detailed in a comment in

Re: [Mesa-dev] [PATCH 0/3] verify max vertex attrib stride

2018-07-09 Thread Gert Wollny
Am Montag, den 09.07.2018, 09:54 +0200 schrieb Erik Faye-Lund: > On 06. juli 2018 18:43, Roland Scheidegger wrote: > > > > Personally I think it's _much_ better to lie about the supported GL > > version rather than the maximum vertex attrib stride (I don't know > > if dEQP would actually have a

Re: [Mesa-dev] [PATCH] intel/compiler: remove unused function

2018-07-09 Thread Lionel Landwerlin
Reviewed-by: Lionel Landwerlin On 09/07/18 10:49, Iago Toral Quiroga wrote: --- src/intel/compiler/brw_fs.cpp | 27 --- src/intel/compiler/brw_fs.h | 4 2 files changed, 31 deletions(-) diff --git a/src/intel/compiler/brw_fs.cpp

Re: [Mesa-dev] [PATCH 0/3] verify max vertex attrib stride

2018-07-09 Thread Erik Faye-Lund
On 06. juli 2018 18:43, Roland Scheidegger wrote: Am 06.07.2018 um 12:03 schrieb Erik Faye-Lund: OpenGL 4.4 and OpenGL ES 3.1 both require the maximum vertex attrib stride to be at least 2048. If this isn't the case, we shouldn't expose these API versions. Unfortunately, the r600 driver only

Re: [Mesa-dev] [PATCH 1/2] radv: introduce radv_subpass_attachment data structure

2018-07-09 Thread Samuel Pitoiset
On 07/08/2018 09:50 PM, Bas Nieuwenhuizen wrote: On Sun, Jul 8, 2018 at 5:47 PM, Samuel Pitoiset wrote: Needed for VK_KHR_create_renderpass2. Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_cmd_buffer.c | 4 ++-- src/amd/vulkan/radv_meta_clear.c | 4 ++--

Re: [Mesa-dev] [PATCH] anv/pipeline: honor the pipeline_cache_enabled run-time flag

2018-07-09 Thread Iago Toral
On Mon, 2018-07-09 at 08:22 +0200, Iago Toral wrote: > On Fri, 2018-07-06 at 15:50 +0100, Lionel Landwerlin wrote: > > On 04/07/18 09:44, Iago Toral Quiroga wrote: > > > --- > > > src/intel/vulkan/anv_pipeline_cache.c | 37 +++- > > > -- > > > - > > > 1 file changed,

Re: [Mesa-dev] [PATCH] anv/pipeline: honor the pipeline_cache_enabled run-time flag

2018-07-09 Thread Iago Toral
On Fri, 2018-07-06 at 15:50 +0100, Lionel Landwerlin wrote: > On 04/07/18 09:44, Iago Toral Quiroga wrote: > > --- > > src/intel/vulkan/anv_pipeline_cache.c | 37 +++--- > > - > > 1 file changed, 20 insertions(+), 17 deletions(-) > > > > diff --git

Re: [Mesa-dev] [PATCH v3] i965/gen6/gs: Handle case where a GS doesn't allocate VUE

2018-07-09 Thread asimiklit.work
Hello, Sorry for late response. I was on vacation. I am glad that this fix is ok. Thanks a lot for your review. Regards, Andrii. On 2018-06-25 12:11 PM, Iago Toral wrote: Thanks for testing Mark. Andrii, I'll add my Reviewed-by and and push the patch to master later today (I'll also queue it

Re: [Mesa-dev] [PATCH 1/2] radv: introduce radv_subpass_attachment data structure

2018-07-09 Thread Bas Nieuwenhuizen
On Mon, Jul 9, 2018 at 10:19 AM, Samuel Pitoiset wrote: > > > On 07/08/2018 09:50 PM, Bas Nieuwenhuizen wrote: >> >> On Sun, Jul 8, 2018 at 5:47 PM, Samuel Pitoiset >> wrote: >>> >>> Needed for VK_KHR_create_renderpass2. >>> >>> Signed-off-by: Samuel Pitoiset >>> --- >>>

Re: [Mesa-dev] [PATCH] nir/vars_to_ssa: Don't build deref nodes for non-local variables

2018-07-09 Thread Dave Airlie
On 9 July 2018 at 14:55, Jason Ekstrand wrote: > --- > src/compiler/nir/nir_lower_vars_to_ssa.c | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/src/compiler/nir/nir_lower_vars_to_ssa.c > b/src/compiler/nir/nir_lower_vars_to_ssa.c > index 3f37acaed33..ef2019551c6 100644 > ---

[Mesa-dev] Proposing an extension to mesa

2018-07-09 Thread Marathe, Yogesh
Hi, We want to propose an extension to mesa (GL & EGL), is there a separate way to do this or that's though RFC and patches on this list here? We have implemented it for i965 but we believe it can be used for other drivers as well. Please let us know. Regards, Yogesh.

Re: [Mesa-dev] [PATCH v2 4/4] radv: add support for VK_EXT_conditional_rendering

2018-07-09 Thread Jason Ekstrand
On Mon, Jul 9, 2018 at 8:29 AM Samuel Pitoiset wrote: > > On 07/09/2018 05:28 PM, Jason Ekstrand wrote: > > Are there any tests for this anywhere? > > Unfortunately, not yet. > Someone should write some then. :-) I recently wrote tests for VK_EXT_vertex_attribute_divisor and we discovered that

Re: [Mesa-dev] [PATCH v2 4/4] radv: add support for VK_EXT_conditional_rendering

2018-07-09 Thread Samuel Pitoiset
On 07/09/2018 06:10 PM, Jason Ekstrand wrote: On Mon, Jul 9, 2018 at 8:29 AM Samuel Pitoiset mailto:samuel.pitoi...@gmail.com>> wrote: On 07/09/2018 05:28 PM, Jason Ekstrand wrote: > Are there any tests for this anywhere? Unfortunately, not yet. Someone should write some

[Mesa-dev] [Bug 107022] [RADV] The Witcher 3: Trembling of trees

2018-07-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107022 --- Comment #3 from Samuel Pitoiset --- We are not ignoring, we are just busy with something else. I will have a look as soon as possible, don't worry. -- You are receiving this mail because: You are the QA Contact for the bug. You are the

[Mesa-dev] [PATCH v2] spirv: Fix InterpolateAt* instructions for vecs with dynamic index

2018-07-09 Thread Alejandro Piñeiro
From: Neil Roberts If the glsl is something like this: in vec4 some_input; interpolateAtCentroid(some_input[idx]) then it now gets generated as if it were: interpolateAtCentroid(some_input)[idx] This is necessary because the index will get generated as a series of nir_bcsel

Re: [Mesa-dev] [PATCH 0/3] verify max vertex attrib stride

2018-07-09 Thread Roland Scheidegger
Am 09.07.2018 um 09:54 schrieb Erik Faye-Lund: > On 06. juli 2018 18:43, Roland Scheidegger wrote: >> Am 06.07.2018 um 12:03 schrieb Erik Faye-Lund: >>> OpenGL 4.4 and OpenGL ES 3.1 both require the maximum >>> vertex attrib stride to be at least 2048. If this isn't >>> the case, we shouldn't

[Mesa-dev] [PATCH v2 4/4] radv: add support for VK_EXT_conditional_rendering

2018-07-09 Thread Samuel Pitoiset
Inherited commands buffers are not supported. v2: - disable predication for blit and copy commands Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_cmd_buffer.c | 29 + src/amd/vulkan/radv_device.c | 7 +++ src/amd/vulkan/radv_extensions.py | 1 +

[Mesa-dev] [Bug 107022] [RADV] The Witcher 3: Trembling of trees

2018-07-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107022 --- Comment #2 from zefkerri...@gmail.com --- (In reply to ximik from comment #1) > on the video on the embedded link is visible. same problem. card r290x, the > driver of the message 18-18.2 git > > > link

Re: [Mesa-dev] [PATCH v1 2/3] platform/android: Enable kms_swrast fallback

2018-07-09 Thread Eric Engestrom
On Monday, 2018-07-09 13:01:49 +0200, Robert Foss wrote: > Add support for the ForceSoftware option, which is togglable > on the Android platform through setting the "drm.gpu.force_software" > property to a non-zero value. > > kms_swrast is also enabled as a fallback for when a driver is not >

[Mesa-dev] [Bug 107156] earth tessellation bug

2018-07-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=107156 --- Comment #4 from ximik --- No, in the Arch repository there is only the last Mesa-git driver. a rollback to the old version is not possible, and on a stable version of the driver, DXVK does not work. -- You are receiving this mail

Re: [Mesa-dev] [PATCH 0/3] verify max vertex attrib stride

2018-07-09 Thread Roland Scheidegger
Am 09.07.2018 um 11:51 schrieb Gert Wollny: > Am Montag, den 09.07.2018, 09:54 +0200 schrieb Erik Faye-Lund: >> On 06. juli 2018 18:43, Roland Scheidegger wrote: >>> >>> Personally I think it's _much_ better to lie about the supported GL >>> version rather than the maximum vertex attrib stride (I

Re: [Mesa-dev] [PATCH 1/2] radv: introduce radv_subpass_attachment data structure

2018-07-09 Thread Samuel Pitoiset
On 07/09/2018 10:36 AM, Bas Nieuwenhuizen wrote: On Mon, Jul 9, 2018 at 10:19 AM, Samuel Pitoiset wrote: On 07/08/2018 09:50 PM, Bas Nieuwenhuizen wrote: On Sun, Jul 8, 2018 at 5:47 PM, Samuel Pitoiset wrote: Needed for VK_KHR_create_renderpass2. Signed-off-by: Samuel Pitoiset ---

[Mesa-dev] [Bug 106644] [llvmpipe] Mesa 18.1.2 fails lp_test_format, lp_test_arit, lp_test_blend, lp_test_printf, lp_test_conv tests

2018-07-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=106644 --- Comment #30 from Ben Crocker --- An LLVM developer tells me that llvmpipe on PPC32 BE doesn't work, and probably won't ever work. He suggests using swrast instead. -- You are receiving this mail because: You are the QA Contact for the

Re: [Mesa-dev] [PATCH v1 2/3] platform/android: Enable kms_swrast fallback

2018-07-09 Thread Robert Foss
Hey Eric, Thanks for the quick reviews! On 09/07/18 15:45, Eric Engestrom wrote: On Monday, 2018-07-09 13:01:49 +0200, Robert Foss wrote: Add support for the ForceSoftware option, which is togglable on the Android platform through setting the "drm.gpu.force_software" property to a non-zero

Re: [Mesa-dev] [PATCH v1 0/3] Android kms_swrast support

2018-07-09 Thread Mauro Rossi
Hi Robert, Il giorno lun 9 lug 2018 alle ore 13:01 Robert Foss < robert.f...@collabora.com> ha scritto: > > NOTE: This series has not been tested successfully, and I'm seeing a > segfault > during the boot process. Which I'm currently looking into. > What is your testing setup

[Mesa-dev] [PATCH] swrast: Fix eglMakeCurrent(dpy, NULL, NULL, ctx) (v2)

2018-07-09 Thread Adam Jackson
Fixes 14 piglits, mostly in egl_khr_create_context. v2: Also short-circuit the same-context-no-drawables case (Eric Anholt) Fixes: https://github.com/anholt/libepoxy/issues/177 Signed-off-by: Adam Jackson --- src/mesa/drivers/dri/swrast/swrast.c | 37 ++-- 1 file

Re: [Mesa-dev] [PATCH] intel/compiler: emit actual barriers for working-group level barriers

2018-07-09 Thread Jason Ekstrand
Reviewed-by: Jason Ekstrand On Mon, Jul 9, 2018 at 2:09 AM Iago Toral wrote: > Any feedback about this? We need this to fix some new CTS tests. > > On Thu, 2018-06-21 at 13:52 +0200, Iago Toral Quiroga wrote: > > Until now we have assumed that we could skip emitting these barriers > > in the

[Mesa-dev] [PATCH] st/mesa: call resource_changed when binding a EGLImage to a texture

2018-07-09 Thread Lucas Stach
When a EGLImage is newly bound to a texture, we need to make sure the driver is informed that the resource might have changed. Fixes stale texture content on Etnaviv when binding an existing EGLImage to an existing texture object. Signed-off-by: Lucas Stach ---

Re: [Mesa-dev] [PATCH v2] spirv: Fix InterpolateAt* instructions for vecs with dynamic index

2018-07-09 Thread Jason Ekstrand
Reviewed-by: Jason Ekstrand On Mon, Jul 9, 2018 at 5:21 AM Alejandro Piñeiro wrote: > From: Neil Roberts > > If the glsl is something like this: > > in vec4 some_input; > interpolateAtCentroid(some_input[idx]) > > then it now gets generated as if it were: > >

[Mesa-dev] [Bug 106644] [llvmpipe] Mesa 18.1.2 fails lp_test_format, lp_test_arit, lp_test_blend, lp_test_printf, lp_test_conv tests

2018-07-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=106644 --- Comment #31 from erhar...@mailbox.org --- I see. I am in no hurry either, 'cause I don't depend on llvmpipe working on PPC32 BE. Leaves me wondering if there is PPC32 LE? I just reported this 'cause I use PPC32 BE regularly and it looked

Re: [Mesa-dev] [PATCH v2 4/4] radv: add support for VK_EXT_conditional_rendering

2018-07-09 Thread Samuel Pitoiset
On 07/09/2018 05:28 PM, Jason Ekstrand wrote: Are there any tests for this anywhere? Unfortunately, not yet. On Mon, Jul 9, 2018 at 5:56 AM Samuel Pitoiset mailto:samuel.pitoi...@gmail.com>> wrote: Inherited commands buffers are not supported. v2: - disable predication for

Re: [Mesa-dev] [PATCH V2] Check if the window is non-NULL before setting swap interval.

2018-07-09 Thread Erik Faye-Lund
On Thu, Jul 5, 2018 at 7:02 PM Emil Velikov wrote: > > On 5 July 2018 at 17:17, Eric Engestrom wrote: > > On Thursday, 2018-07-05 14:43:02 +0100, Emil Velikov wrote: > >> On 5 July 2018 at 10:53, Eric Engestrom wrote: > >> > On Monday, 2018-07-02 14:12:44 +0530, samiuddi wrote: > >> >> This

Re: [Mesa-dev] [PATCH v1 0/3] Android kms_swrast support

2018-07-09 Thread Robert Foss
Hey Mauro, On 09/07/18 14:27, Mauro Rossi wrote: Hi Robert, Il giorno lun 9 lug 2018 alle ore 13:01 Robert Foss > ha scritto: NOTE: This series has not been tested successfully, and I'm seeing a segfault       during the boot process. Which I'm

[Mesa-dev] [PATCH v3] radv: make sure to wait for CP DMA when needed

2018-07-09 Thread Samuel Pitoiset
This might fix some synchronization issues. I don't know if that will affect performance but it's required for correctness. v3: - wait for CP DMA in CmdPipelineBarrier() - clear the busy value when CP_DMA_SYNC is requested v2: - wait for CP DMA in CmdWaitEvents() - track if CP DMA is used

[Mesa-dev] [PATCH 5/9] etnaviv: create optional 2d pipe in screen

2018-07-09 Thread Lucas Stach
The 2D pipe is useful to implement fast planar and interleaved YUV buffer imports. Not all systems have a 2D capable core, so this is strictly optional. Signed-off-by: Lucas Stach --- src/gallium/drivers/etnaviv/etnaviv_context.c | 6 ++ src/gallium/drivers/etnaviv/etnaviv_context.h | 1 +

[Mesa-dev] [PATCH 9/9] etnaviv: handle YUV textures with the 2D GPU

2018-07-09 Thread Lucas Stach
This allows color space conversion and tiling in a single step, as well as handling multi-planar formats like NV12, which are really useful when dealing with hardware video decoders. Signed-off-by: Lucas Stach --- src/gallium/drivers/etnaviv/etnaviv_clear_blit.c | 2 +-

[Mesa-dev] [PATCH 1/9] etnaviv: improve PIPE_BIND_LINEAR handling

2018-07-09 Thread Lucas Stach
We weren't handling this flag at all, which broke some assumptions made by the users of the resource_create interface. As we can't render to a linear surface and the usefulness of yet another layout transition to handle this case seems limited, we only respect the flag when the resource isn't used

[Mesa-dev] [PATCH 4/9] etnaviv: remember data offset into BO

2018-07-09 Thread Lucas Stach
Imported resources might not start at offset 0 into the buffer object. Make sure to remember the offset that is provided with the handle on import. Signed-off-by: Lucas Stach --- src/gallium/drivers/etnaviv/etnaviv_resource.c | 2 +- src/gallium/drivers/etnaviv/etnaviv_resource.h | 1 + 2 files

[Mesa-dev] [PATCH 0/9] Etnaviv direct YUV support

2018-07-09 Thread Lucas Stach
Driver support for YUV textures is mostly limited to OES_external, which is suboptimal for etnaviv, as this incurs an additional copy blit on each texture bind, due to the way the OES_external extension is defined and the Vivante GPUs inability to sample from linear resources. Also emulating

Re: [Mesa-dev] [PATCH v2 4/4] radv: add support for VK_EXT_conditional_rendering

2018-07-09 Thread Jason Ekstrand
Are there any tests for this anywhere? On Mon, Jul 9, 2018 at 5:56 AM Samuel Pitoiset wrote: > Inherited commands buffers are not supported. > > v2: - disable predication for blit and copy commands > > Signed-off-by: Samuel Pitoiset > --- > src/amd/vulkan/radv_cmd_buffer.c | 29

[Mesa-dev] [Bug 106151] [amdgpu][vulkan] GPU hang (Vega 56) while running game (Rise of the Tomb Raider)

2018-07-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=106151 --- Comment #27 from Samuel Pitoiset --- I guess it also hangs with RADV_PERFTEST=nobatchchain ? -- You are receiving this mail because: You are the QA Contact for the bug. You are the assignee for the

[Mesa-dev] [PATCH 6/9] etnaviv: add 2D GPU YUV->RGB blitter

2018-07-09 Thread Lucas Stach
This adds a blit path using the 2D GPU for a linear YUV to tiled RGB blit. This allows to implement importing of planar YUV textures with a single copy. Signed-off-by: Lucas Stach --- src/gallium/drivers/etnaviv/Makefile.sources |2 + src/gallium/drivers/etnaviv/etnaviv_2d.c | 166 ++

[Mesa-dev] [PATCH 3/9] etnaviv: clear out next pointer when allocating resource

2018-07-09 Thread Lucas Stach
We copy the template resource content into the newly allocated resource. If the template derived from a planar resource this leads to a non reference counted copy of the next resource pointer. Make sure to clear this out when allocating a new resource. Signed-off-by: Lucas Stach ---

[Mesa-dev] [PATCH 7/9] etnaviv: export etna_submit_rs_state

2018-07-09 Thread Lucas Stach
The new 2D YUV blit needs this in some cases, so make it available. Signed-off-by: Lucas Stach --- src/gallium/drivers/etnaviv/etnaviv_rs.c | 2 +- src/gallium/drivers/etnaviv/etnaviv_rs.h | 4 2 files changed, 5 insertions(+), 1 deletion(-) diff --git

[Mesa-dev] [PATCH 8/9] etnaviv: use filter blit for 2D YUV import on old GC320

2018-07-09 Thread Lucas Stach
The GC320 without the 2D tiling feature doesn't support regular blits with YUV input, as well as the tiled output. So on those cores we need need to do a filter blit for the YUV->RGB conversion to a temporary linear buffer and then do a tiling blit into the texture buffer using the RS engine on

  1   2   >