Re: [Mesa-dev] [PATCH v2 05/27] configure: enable the surfaceless platform by default

2017-05-07 Thread Tapani Pälli
Yes please! Acked-by: Tapani Pälli On 05/04/2017 07:29 PM, Emil Velikov wrote: From: Emil Velikov A simple platform that you want to use in a many usecases. See the spec file details. It has no special requirements plus it takes less than

Re: [Mesa-dev] [PATCH] android: i965: add per-gen libmesa_i965_gen{4, 45, 5} static

2017-05-07 Thread Tapani Pälli
Reviewed-by: Tapani Pälli On 05/06/2017 05:03 PM, Mauro Rossi wrote: Needed to fix android building errors: external/mesa/src/mesa/drivers/dri/i965/brw_state_upload.c:148: error: undefined reference to 'gen5_init_atoms'

[Mesa-dev] [PATCH] mesa: Make _mesa_primitive_restart_index a static inline in the header.

2017-05-07 Thread Kenneth Graunke
It's now basically a single expression, so it probably makes sense to have it inlined into the callers. Suggested by Marek. --- src/mesa/main/varray.c | 18 -- src/mesa/main/varray.h | 17 +++-- 2 files changed, 15 insertions(+), 20 deletions(-) diff --git

Re: [Mesa-dev] [PATCH] glsl: destroy function and subroutine hash tables

2017-05-07 Thread Timothy Arceri
Pushed, thanks! On 03/05/17 04:06, Grazvydas Ignotas wrote: Just like other type hash tables are destroyed in _mesa_glsl_release_types(), also destroy the ones for function and subroutine types. Signed-off-by: Grazvydas Ignotas --- src/compiler/glsl_types.cpp | 10

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

2017-05-07 Thread AppVeyor
Build mesa 4289 failed Commit 1f743a0edf by Grazvydas Ignotas on 5/2/2017 6:06 PM: glsl: destroy function and subroutine hash tables\n\nJust like other type hash tables are destroyed in\n_mesa_glsl_release_types(), also destroy the ones for function

[Mesa-dev] [Bug 100782] [GLVND] X fails to start AMDGPU DDX when GLVND compiled into Mesa

2017-05-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100782 Shawn Starr changed: What|Removed |Added Resolution|--- |INVALID

[Mesa-dev] [PATCH] st/xvmc: deal with drivers wanting different texture formats

2017-05-07 Thread Ilia Mirkin
Previously, texture formats were being used unconditionally without checking. However nv30 supports neither RGBX8 nor R4A4/A4R4 formats. Add sufficient fallbacks so that the nv30 driver can have working OSD. Tested on a NV44A/PCI. Signed-off-by: Ilia Mirkin --- I need to

[Mesa-dev] [PATCH] gallivm: Make sure module has the correct data layout when pass manager runs

2017-05-07 Thread Tom Stellard
The datalayout for modules was purposely not being set in order to work around the fact that the ExecutionEngine requires that the module's datalayout matches the datalayout of the TargetMachine that the ExecutionEngine is using. When the pass manager runs on a module with no datalayout, it uses

Re: [Mesa-dev] [PATCH v3 0/4] Disable glthread if libX11 isn't thread-safe

2017-05-07 Thread Axel Davy
Hi, There should be very few X11 calls while rendering (basically only at the beginning or end of a frame). Why not just always run these calls in the main thread (and wait for glthread work to finish) ? That's basically what we do for gallium nine. Yours, Axel On 05/05/2017 17:37,

Re: [Mesa-dev] [PATCH] docs: remove released versions from the calendar

2017-05-07 Thread Eric Engestrom
On Thursday, 2017-05-04 17:37:56 +0100, Emil Velikov wrote: > From: Emil Velikov > > Signed-off-by: Emil Velikov Reviewed-by: Eric Engestrom I guess updating the release calendar could be added as part of

Re: [Mesa-dev] [PATCH v2 07/27] configure: error out when building GLX w/o the X11 platform

2017-05-07 Thread Eric Engestrom
On Thursday, 2017-05-04 17:29:52 +0100, Emil Velikov wrote: > From: Emil Velikov > > Building EGL/Vulkan/other without X11, while GLX is enabled is confusing > and misleading. In practise anyone aiming at the former will also > disable GLX. > > The inverse (some

[Mesa-dev] [PATCH] mesa: add KHR_no_error support for glUseProgram

2017-05-07 Thread Timothy Arceri
V3: use always_inline attribute (Suggested by Nicolai) Cc: Nicolai Hähnle --- src/mapi/glapi/gen/gl_API.xml | 2 +- src/mesa/main/shaderapi.c | 75 +-- src/mesa/main/shaderapi.h | 2 ++ 3 files changed, 54 insertions(+), 25

Re: [Mesa-dev] [PATCH 2/2] docs: extend until the end of August

2017-05-07 Thread Eric Engestrom
On Saturday, 2017-05-06 18:46:13 +0300, Andres Gomez wrote: > Completed the 17.1 cycle and added the beginning of the 17.2 one. > > Cc: Emil Velikov > Cc: Juan A. Suarez Romero > Signed-off-by: Andres Gomez Reviewed-by: Eric

Re: [Mesa-dev] [PATCH] bin/get-fixes-pick-list.sh: don't warn if more than one, go over them

2017-05-07 Thread Eric Engestrom
On Saturday, 2017-05-06 17:09:35 +0300, Andres Gomez wrote: > If an identified commit was having more than one fix, we would warn > about that and only treat the first. > > Now, we don't warn but treat all of them. This was on my todo list, thanks for fixing this :) > > Signed-off-by: Andres

Re: [Mesa-dev] [PATCH] egl: Fix -Wint-to-pointer-cast

2017-05-07 Thread Eric Engestrom
On Friday, 2017-05-05 12:27:03 -0700, Chad Versace wrote: > main/egldisplay.c: In function '_eglParseX11DisplayAttribList': > main/egldisplay.c:491:38: warning: cast to pointer from integer of different > size [-Wint-to-pointer-cast] > display->Options.Platform = (void *)value; > > The

Re: [Mesa-dev] [PATCH 3/3 v2] r600g: get rid of dummy pixel shader

2017-05-07 Thread Dieter Nützel
Am 26.04.2017 14:37, schrieb tournier.elie: Hello, I'm not an r600g person but I will be pleased to have a look to this work. Hello Elie, I know, read most of your posts about your 'former' fp64 lib which was indeed HW-agnostic. I'm currently working on the fp64 for r600g. Hope to be

Re: [Mesa-dev] [PATCH v2 21/27] st/xvmc: add DRI3 support

2017-05-07 Thread Ilia Mirkin
Tested-by: Ilia Mirkin Seems to work with a NV44A (with some hacks to deal with its non-pcie/agp-ness, but that's entirely separate). The "background" area when entering full-screen mode appears to not get cleared with dri3 like it does with dri2, although I think I've seen

[Mesa-dev] [PATCH mesa] egl: avoid dereferencing a null display

2017-05-07 Thread Eric Engestrom
Fixes: ddb99127a6f6c ("egl/x11: Honor the EGL_PLATFORM_X11_SCREEN_EXT attribute") Signed-off-by: Eric Engestrom --- src/egl/main/egldisplay.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/egl/main/egldisplay.c b/src/egl/main/egldisplay.c index

Re: [Mesa-dev] [PATCH 1/7] radv/meta: add srgb conversion to end of resolve shader.

2017-05-07 Thread Dave Airlie
On 5 May 2017 at 01:46, Bas Nieuwenhuizen wrote: > Can't we just use a non-SRGB image view into the destination image? > That should save some operations. We could possibly here, but not in the subpass path, which a later patch enables. I wasn't sure it was worth the

Re: [Mesa-dev] [PATCH 1/5] radv: set base/ranges for push constant loads.

2017-05-07 Thread Bas Nieuwenhuizen
This series is Reviewed-by: Bas Nieuwenhuizen On Fri, May 5, 2017 at 2:59 AM, Dave Airlie wrote: > From: Dave Airlie > > This isn't necessary yet but I'd like to use the range in > some future patches. > > Signed-off-by: Dave

Re: [Mesa-dev] [PATCH 1/7] radv/meta: add srgb conversion to end of resolve shader.

2017-05-07 Thread Bas Nieuwenhuizen
This series is Reviewed-by: Bas Nieuwenhuizen On Thu, May 4, 2017 at 5:27 AM, Dave Airlie wrote: > From: Dave Airlie > > If we are resolving into an srgb dest, we need to convert > to linear so the store does the conversion

[Mesa-dev] [PATCH] util: move ALWAYS_INLINE macro to util/macro.h

2017-05-07 Thread Timothy Arceri
Also added clang check. macro.h is include by p_compiler.h so no other change is needed. --- src/gallium/include/pipe/p_compiler.h | 11 --- src/util/macros.h | 11 +++ 2 files changed, 11 insertions(+), 11 deletions(-) diff --git

Re: [Mesa-dev] [PATCH 7/7] mesa: add KHR_no_error support for glUseProgram

2017-05-07 Thread Timothy Arceri
On 08/05/17 06:29, Nicolai Hähnle wrote: On 04.05.2017 16:28, Nicolai Hähnle wrote: On 04.05.2017 05:34, Timothy Arceri wrote: On 04/05/17 13:31, Dave Airlie wrote: +/* The ARB_separate_shader_object spec says: + * + * "The executable code for an individual shader stage is taken from +

[Mesa-dev] [Bug 100960] Special block from Minecraft mod rendered out of place

2017-05-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100960 --- Comment #2 from Fabian Maurer --- Created attachment 131245 --> https://bugs.freedesktop.org/attachment.cgi?id=131245=edit screenshot(broken) on linux -- You are receiving this mail because: You are the QA Contact

[Mesa-dev] [Bug 100960] Special block from Minecraft mod rendered out of place

2017-05-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100960 --- Comment #1 from Fabian Maurer --- Created attachment 131244 --> https://bugs.freedesktop.org/attachment.cgi?id=131244=edit screenshot (working) on windows -- You are receiving this mail because: You are the QA

[Mesa-dev] [Bug 100960] Special block from Minecraft mod rendered out of place

2017-05-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100960 Bug ID: 100960 Summary: Special block from Minecraft mod rendered out of place Product: Mesa Version: git Hardware: x86-64 (AMD64) OS: Linux (All) Status: NEW

Re: [Mesa-dev] [PATCH 7/7] mesa: add KHR_no_error support for glUseProgram

2017-05-07 Thread Nicolai Hähnle
On 04.05.2017 16:28, Nicolai Hähnle wrote: On 04.05.2017 05:34, Timothy Arceri wrote: On 04/05/17 13:31, Dave Airlie wrote: +/* The ARB_separate_shader_object spec says: + * + * "The executable code for an individual shader stage is taken from + * the current program for that stage.

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

2017-05-07 Thread AppVeyor
Build mesa 4285 completed Commit dab6a2dfd9 by Jose Fonseca on 5/7/2017 4:58 PM: nir: Fix missing snprintf symbol on Windows.\n\nCopy nir_print.c's snprintf definition for now, to unbreak Windows\nbuilds.\n\nWe can and should cleanup all snprintf definitions

Re: [Mesa-dev] [PATCH 02/13] mesa/dri: always link against shared glapi

2017-05-07 Thread Aaron Watry
I ran into the same thing... Do you by chance have '--with-dri-driverdir=' in your configure command? I had '--prefix=/usr/local' and '--with-dri-driverdir=/usr/local/dri', and removing the --with-dri-driverdir fixed my build. --Aaron On Sat, May 6, 2017 at 7:48 PM, Grazvydas Ignotas

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

2017-05-07 Thread AppVeyor
Build mesa 4283 failed Commit c297e68828 by Dave Airlie on 4/28/2017 6:17 AM: radv: set PERF_MOD in sample state like radeonsi.\n\nThis just aligns the code with radeonsi.\n\nReviewed-by: Bas Nieuwenhuizen \nSigned-off-by: Dave Airlie

Re: [Mesa-dev] [PATCH] radv: apply the tess+GS hang workaround to Polaris12 as well

2017-05-07 Thread Bas Nieuwenhuizen
Reviewed-by: Bas Nieuwenhuizen On Sun, May 7, 2017 at 11:54 AM, Dave Airlie wrote: > From: Dave Airlie > > As I pointed out for radeonsi, and AMD confirmed, so fix this > in radv as well. > > Cc: "17.1"

[Mesa-dev] [PATCH] radv: apply the tess+GS hang workaround to Polaris12 as well

2017-05-07 Thread Dave Airlie
From: Dave Airlie As I pointed out for radeonsi, and AMD confirmed, so fix this in radv as well. Cc: "17.1" Signed-off-by: Dave Airlie --- src/amd/vulkan/si_cmd_buffer.c | 3 ++- 1 file changed, 2 insertions(+), 1