[Mesa-dev] [Bug 102502] [bisected] Kodi crashes since commit 707d2e8b - gallium: fold u_trim_pipe_prim call from st/mesa to drivers

2017-09-01 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=102502 Vinson Lee changed: What|Removed |Added Version|git |17.2 Keywords|

Re: [Mesa-dev] [PATCH 1/2] gallium: Add PIPE_CAP_HALFS

2017-09-01 Thread Roland Scheidegger
Am 02.09.2017 um 02:55 schrieb Jan Vesely: > Denotes native half precision float operations capability > > Signed-off-by: Jan Vesely > --- > I can change the spelling to HALVES, but simplified english sounded more > appropriate. My spelling of choice would be "FP16"... Roland > > src/gallium

[Mesa-dev] [Bug 102496] Frontbuffer rendering corruption on mesa master

2017-09-01 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=102496 --- Comment #1 from Dieter Nützel --- I can second this on radeonsi/RX580. 'olympic -sb' show no corruption per se, but no animation (ring movement), here. With commit eceb6710024716433069d705fbd873d6d136c2cc reverted 'olympic -sb' is fine, aga

[Mesa-dev] [PATCH 2/2] clover: Query and export half precision support

2017-09-01 Thread Jan Vesely
Signed-off-by: Jan Vesely --- src/gallium/state_trackers/clover/api/device.cpp | 14 +++--- src/gallium/state_trackers/clover/core/device.cpp | 5 + src/gallium/state_trackers/clover/core/device.hpp | 1 + 3 files changed, 17 insertions(+), 3 deletions(-) diff --git a/src/gallium

[Mesa-dev] [PATCH 1/2] gallium: Add PIPE_CAP_HALFS

2017-09-01 Thread Jan Vesely
Denotes native half precision float operations capability Signed-off-by: Jan Vesely --- I can change the spelling to HALVES, but simplified english sounded more appropriate. src/gallium/docs/source/screen.rst | 1 + src/gallium/drivers/etnaviv/etnaviv_screen.c | 1 + src/galli

[Mesa-dev] [PATCH v4 1/1] clover: Wait for requested operation if blocking flag is set

2017-09-01 Thread Jan Vesely
v2: wait in map_buffer and map_image as well v3: use event::wait instead of wait (skips fence wait for hard_event) v4: use wait_signalled() Signed-off-by: Jan Vesely --- Hi Francisco, once again sorry for the delay, and thanks for you patience. This patch applies on top of the two you attached d

Re: [Mesa-dev] [PATCH 2/2] genxml: Make Border Color Pointer an address on Gen4-5, not an offset.

2017-09-01 Thread Jason Ekstrand
On Fri, Sep 1, 2017 at 4:15 PM, Kenneth Graunke wrote: > --- > src/intel/genxml/gen4.xml | 2 +- > src/intel/genxml/gen45.xml| 2 +- > src/intel/genxml/gen5.xml | 2 +- > src/mesa/drivers/dri/i965/genX_state_upload.c | 10 --

[Mesa-dev] [PATCH] st/mesa: skip draw calls with pipe_draw_info::count == 0

2017-09-01 Thread Marek Olšák
From: Marek Olšák Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102502 Cc: 17.2 --- src/mesa/state_tracker/st_draw.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/mesa/state_tracker/st_draw.c b/src/mesa/state_tracker/st_draw.c index fe03a4a..2fe7070 100

[Mesa-dev] [PATCH 1/2] i965: Inline emit_reloc in __genx_combine_address

2017-09-01 Thread Kenneth Graunke
One less layer of baklava. --- src/mesa/drivers/dri/i965/genX_state_upload.c | 17 + 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/src/mesa/drivers/dri/i965/genX_state_upload.c b/src/mesa/drivers/dri/i965/genX_state_upload.c index b15829fb57c..4eb1a79bcd4 100644 -

[Mesa-dev] [PATCH 2/2] genxml: Make Border Color Pointer an address on Gen4-5, not an offset.

2017-09-01 Thread Kenneth Graunke
--- src/intel/genxml/gen4.xml | 2 +- src/intel/genxml/gen45.xml| 2 +- src/intel/genxml/gen5.xml | 2 +- src/mesa/drivers/dri/i965/genX_state_upload.c | 10 -- 4 files changed, 7 insertions(+), 9 deletions(-) diff --git a/src

[Mesa-dev] [Bug 90438] glxinfo should have a quiet mode

2017-09-01 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=90438 Bryan Quigley changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

Re: [Mesa-dev] [PATCH mesa] util: improve compiler guard

2017-09-01 Thread Jose Fonseca
On 01/09/17 15:54, Eric Engestrom wrote: On Thursday, 2017-08-31 13:28:25 -0500, Rob Herring wrote: On Thu, Aug 31, 2017 at 11:54 AM, Eric Engestrom wrote: Glibc 2.26 has dropped xlocale.h, but the functions needed (strtod_l() and strdof_l()) can be found in stdlib.h. Improve the detection met

Re: [Mesa-dev] [PATCH mesa] util: improve compiler guard

2017-09-01 Thread Laurent Carlier
Le jeudi 31 août 2017, 18:54:48 CEST Eric Engestrom a écrit : Reviewed-by: Laurent Carlier > Glibc 2.26 has dropped xlocale.h, but the functions needed (strtod_l() > and strdof_l()) can be found in stdlib.h. > Improve the detection method to allow newer builds to still make use of > the locale-s

Re: [Mesa-dev] [PATCH] st/mesa: fix view template initialization in try_pbo_readpixels

2017-09-01 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Fri, Sep 1, 2017 at 1:53 AM, wrote: > From: Roland Scheidegger > > I think this is what the code was meant to do, albeit as far as I can tell > the redundant initialization some analyzers complain about should work as > well just fine (only the first layer wi

Re: [Mesa-dev] [Mesa-stable] [PATCH] i965: Fix crash in fallback GTT mapping.

2017-09-01 Thread Anuj Phogat
On Fri, Sep 1, 2017 at 10:02 AM, Kenneth Graunke wrote: > We can't perf_debug without a context. > > Cc: mesa-sta...@lists.freedesktop.org > --- > src/mesa/drivers/dri/i965/brw_bufmgr.c | 6 -- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_bu

[Mesa-dev] [PATCH] i965: Fix crash in fallback GTT mapping.

2017-09-01 Thread Kenneth Graunke
We can't perf_debug without a context. Cc: mesa-sta...@lists.freedesktop.org --- src/mesa/drivers/dri/i965/brw_bufmgr.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_bufmgr.c b/src/mesa/drivers/dri/i965/brw_bufmgr.c index d33754d9497..adf

Re: [Mesa-dev] [PATCH 2/2] swr: Report format max_samples=1 to maintain support for "fake" msaa.

2017-09-01 Thread Kyriazis, George
Discussed with Bruce. Reviewed-By: George Kyriazis mailto:george.kyria...@intel.com>> On Aug 31, 2017, at 11:09 PM, Brian Paul mailto:bri...@vmware.com>> wrote: I'm not familiar with this driver so I'll give this an Acked-by: Brian Paul mailto:bri...@vmware.com>> Roland gave me his R-b for t

Re: [Mesa-dev] [PATCH mesa] aubinator: remove duplicate initialisation

2017-09-01 Thread Lionel Landwerlin
Rb On 01/09/17 16:56, Eric Engestrom wrote: Signed-off-by: Eric Engestrom --- src/intel/tools/aubinator_error_decode.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/intel/tools/aubinator_error_decode.c b/src/intel/tools/aubinator_error_decode.c index 636f56a336..ed4d6f662c 100644 -

[Mesa-dev] [PATCH mesa] aubinator: remove duplicate initialisation

2017-09-01 Thread Eric Engestrom
Signed-off-by: Eric Engestrom --- src/intel/tools/aubinator_error_decode.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/intel/tools/aubinator_error_decode.c b/src/intel/tools/aubinator_error_decode.c index 636f56a336..ed4d6f662c 100644 --- a/src/intel/tools/aubinator_error_decode.c +++

Re: [Mesa-dev] [PATCH v2] aubinator: implement a rolling window of programs

2017-09-01 Thread Eric Engestrom
On Friday, 2017-09-01 16:40:17 +0100, Lionel Landwerlin wrote: > If we have more programs than what we can store, > aubinator_error_decode will assert. Instead let's have a rolling > window of programs. > > v2: Fix overflowing issues (Eric Engestrom) > > Signed-off-by: Lionel Landwerlin I don't

Re: [Mesa-dev] [PATCH] aubinator: implement a rolling window of programs

2017-09-01 Thread Lionel Landwerlin
On 01/09/17 15:37, Eric Engestrom wrote: >8 With that, you can keep the assert (although it would only fire it someone were to manually increment `num_programs`) and the MIN() becomes unnecessary. Actually I just realized the MIN() doesn't work once num_programs goes back to 0... Th

[Mesa-dev] [PATCH v2] aubinator: implement a rolling window of programs

2017-09-01 Thread Lionel Landwerlin
If we have more programs than what we can store, aubinator_error_decode will assert. Instead let's have a rolling window of programs. v2: Fix overflowing issues (Eric Engestrom) Signed-off-by: Lionel Landwerlin --- src/intel/tools/aubinator_error_decode.c | 24 1 file c

Re: [Mesa-dev] [PATCH] aubinator: implement a rolling window of programs

2017-09-01 Thread Eric Engestrom
On Friday, 2017-09-01 15:45:37 +0100, Lionel Landwerlin wrote: > On 01/09/17 15:37, Eric Engestrom wrote: > > On Friday, 2017-09-01 10:38:33 +0100, Lionel Landwerlin wrote: > > > If we have more programs than what we can store, > > > aubinator_error_decode will assert. Instead let's have a rolling

Re: [Mesa-dev] [PATCH] intel: genxml: index fields using their safe name

2017-09-01 Thread Jason Ekstrand
I didn't do this before precisely to force us to keep the XML consistent. On September 1, 2017 2:38:27 AM Lionel Landwerlin wrote: With a slightly different capitalization of an additional space, 2 fields having the same safe name might not be grouped together. This leads to multiple *_bits

Re: [Mesa-dev] [PATCH mesa] util: improve compiler guard

2017-09-01 Thread Eric Engestrom
On Thursday, 2017-08-31 13:28:25 -0500, Rob Herring wrote: > On Thu, Aug 31, 2017 at 11:54 AM, Eric Engestrom > wrote: > > Glibc 2.26 has dropped xlocale.h, but the functions needed (strtod_l() > > and strdof_l()) can be found in stdlib.h. > > Improve the detection method to allow newer builds to

[Mesa-dev] [Bug 99591] Segmentation fault when running vulkaninfo with RADV Radeon Vulkan driver

2017-09-01 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99591 --- Comment #15 from Matias N. Goldberg --- "This might be related to building LLVM with -DBUILD_SHARED_LIBS=ON. Does it also happen when building LLVM with -DLLVM_BUILD_LLVM_DYLIB=ON instead?" I just re-read your comment and it seems I misunder

Re: [Mesa-dev] [PATCH] aubinator: implement a rolling window of programs

2017-09-01 Thread Lionel Landwerlin
On 01/09/17 15:37, Eric Engestrom wrote: On Friday, 2017-09-01 10:38:33 +0100, Lionel Landwerlin wrote: If we have more programs than what we can store, aubinator_error_decode will assert. Instead let's have a rolling window of programs. Signed-off-by: Lionel Landwerlin --- src/intel/tools/a

[Mesa-dev] [Bug 99591] Segmentation fault when running vulkaninfo with RADV Radeon Vulkan driver

2017-09-01 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99591 --- Comment #14 from Matias N. Goldberg --- Forgot to say: BUILD_SHARED_LIBS *IS* enabled. -- You are receiving this mail because: You are the assignee for the bug. You are the QA Contact for the bug.

[Mesa-dev] [Bug 99591] Segmentation fault when running vulkaninfo with RADV Radeon Vulkan driver

2017-09-01 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99591 --- Comment #13 from Matias N. Goldberg --- I updated to latest and greatest LLVM from SVN today: svn info -r HEAD Path: trunk URL: http://llvm.org/svn/llvm-project/llvm/trunk Relative URL: ^/llvm/trunk Repository Root: http://llvm.org/svn/llvm-p

Re: [Mesa-dev] [PATCH] radv: fix a memleak when compiling the GS copy shader

2017-09-01 Thread Eric Engestrom
On Friday, 2017-09-01 15:40:52 +0100, Eric Engestrom wrote: > On Friday, 2017-09-01 14:07:43 +0200, Samuel Pitoiset wrote: > > Found by inspection. > > > > Signed-off-by: Samuel Pitoiset > > --- > > src/amd/vulkan/radv_pipeline.c | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a

Re: [Mesa-dev] [PATCH] radv: fix a memleak when compiling the GS copy shader

2017-09-01 Thread Eric Engestrom
On Friday, 2017-09-01 14:07:43 +0200, Samuel Pitoiset wrote: > Found by inspection. > > Signed-off-by: Samuel Pitoiset > --- > src/amd/vulkan/radv_pipeline.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_pipeline.c > index f2d1b

Re: [Mesa-dev] [PATCH] aubinator: implement a rolling window of programs

2017-09-01 Thread Eric Engestrom
On Friday, 2017-09-01 10:38:33 +0100, Lionel Landwerlin wrote: > If we have more programs than what we can store, > aubinator_error_decode will assert. Instead let's have a rolling > window of programs. > > Signed-off-by: Lionel Landwerlin > --- > src/intel/tools/aubinator_error_decode.c | 16 ++

[Mesa-dev] [PATCH mesa] glx: remove dead code

2017-09-01 Thread Eric Engestrom
These fields were added in 2d94601582 but never used; hasPresent was never set, while the other ones were set but never read. Fixes: 2d94601582e4f0fcaf8c "Add DRI3+Present loader" Cc: Keith Packard Signed-off-by: Eric Engestrom --- Keith, did you mean to do something with these, or are they just

Re: [Mesa-dev] [PATCH 1/2] radeonsi: set a per-buffer flag that disables inter-process sharing (v4)

2017-09-01 Thread Marek Olšák
Well, this patch shouldn't change behavior. Are you sure it's not some random thing? I know, for example, that SI isn't very stable with piglit on radeon either. Marek On Sep 1, 2017 12:28 PM, "Michel Dänzer" wrote: > On 01/09/17 07:23 PM, Nicolai Hähnle wrote: > > On 01.09.2017 11:58, Michel D

[Mesa-dev] [PATCH] radv: fix a memleak when compiling the GS copy shader

2017-09-01 Thread Samuel Pitoiset
Found by inspection. Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_pipeline.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_pipeline.c index f2d1b491b7..858cc54085 100644 --- a/src/amd/vulkan/radv_pipeline.c +++ b/src/amd/vul

Re: [Mesa-dev] [PATCH] i965: skip varyings without slot

2017-09-01 Thread Juan A. Suarez Romero
On Thu, 2017-06-29 at 14:43 +1000, Timothy Arceri wrote: > On 27/06/17 21:20, Juan A. Suarez Romero wrote: > > On Tue, 2017-06-27 at 09:29 +1000, Timothy Arceri wrote: > > > On 16/06/17 18:12, Juan A. Suarez Romero wrote: > > > > > > > Commit 00620782c9 (i965: use nir_shader_gather_info() over > >

Re: [Mesa-dev] [PATCH 1/2] radeonsi: set a per-buffer flag that disables inter-process sharing (v4)

2017-09-01 Thread Christian König
Am 01.09.2017 um 12:28 schrieb Michel Dänzer: On 01/09/17 07:23 PM, Nicolai Hähnle wrote: On 01.09.2017 11:58, Michel Dänzer wrote: On 29/08/17 11:47 PM, Christian König wrote: From: Marek Olšák For lower overhead in the CS ioctl. Winsys allocators are not used with interprocess-sharable res

Re: [Mesa-dev] [PATCH 1/2] radeonsi: set a per-buffer flag that disables inter-process sharing (v4)

2017-09-01 Thread Michel Dänzer
On 01/09/17 07:23 PM, Nicolai Hähnle wrote: > On 01.09.2017 11:58, Michel Dänzer wrote: >> On 29/08/17 11:47 PM, Christian König wrote: >>> From: Marek Olšák >>> >>> For lower overhead in the CS ioctl. >>> Winsys allocators are not used with interprocess-sharable resources. >>> >>> v2: It shouldn'

Re: [Mesa-dev] [PATCH 5/5] i965: Move BATCH_SZ define into intel_batchbuffer.c.

2017-09-01 Thread Chris Wilson
Quoting Kenneth Graunke (2017-08-31 21:18:00) > It's only used in one file. Series is Reviewed-by: Chris Wilson with just a quibble on patch 3. -Chris ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinf

Re: [Mesa-dev] [PATCH 3/5] i965: Rename brw_bo::offset64 to presumed_offset.

2017-09-01 Thread Chris Wilson
Quoting Kenneth Graunke (2017-08-31 21:17:58) > We can drop the meaningless "64" suffix - libdrm_intel originally had > an "offset" field that was an "unsigned long" which was the wrong size, > and we couldn't remove/alter that field without breaking ABI, so we had > to add a uint64_t "offset64" fi

Re: [Mesa-dev] [PATCH 1/2] radeonsi: set a per-buffer flag that disables inter-process sharing (v4)

2017-09-01 Thread Nicolai Hähnle
On 01.09.2017 11:58, Michel Dänzer wrote: On 29/08/17 11:47 PM, Christian König wrote: From: Marek Olšák For lower overhead in the CS ioctl. Winsys allocators are not used with interprocess-sharable resources. v2: It shouldn't crash anymore, but the kernel will reject the new flag. v3 (christ

Re: [Mesa-dev] [PATCH 1/2] radeonsi: set a per-buffer flag that disables inter-process sharing (v4)

2017-09-01 Thread Michel Dänzer
On 29/08/17 11:47 PM, Christian König wrote: > From: Marek Olšák > > For lower overhead in the CS ioctl. > Winsys allocators are not used with interprocess-sharable resources. > > v2: It shouldn't crash anymore, but the kernel will reject the new flag. > v3 (christian): Rename the flag, avoid se

[Mesa-dev] [PATCH] aubinator: implement a rolling window of programs

2017-09-01 Thread Lionel Landwerlin
If we have more programs than what we can store, aubinator_error_decode will assert. Instead let's have a rolling window of programs. Signed-off-by: Lionel Landwerlin --- src/intel/tools/aubinator_error_decode.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a

[Mesa-dev] [PATCH] intel: genxml: index fields using their safe name

2017-09-01 Thread Lionel Landwerlin
With a slightly different capitalization of an additional space, 2 fields having the same safe name might not be grouped together. This leads to multiple *_bits() functions with the same name. We can prevent this by indexing fields with their safe name. Signed-off-by: Lionel Landwerlin --- src/i