[Mesa-dev] [PATCH 00/15] Import libtxc_dxtn's S3TC code into Mesa!

2017-10-02 Thread Matt Turner
The patent on S3TC has now expired! Rejoice! This series imports the code from Roland's libtxc_dxtn into Mesa and cleans up some mess dealing with the possibility of S3TC not being available. EXT_texture_compression_s3tc and ANGLE_texture_compression_dxt are now always enabled where supported. N

[Mesa-dev] [PATCH 03/15] mesa: Remove commented-out DXTn fetch code

2017-10-02 Thread Matt Turner
Has been disabled for 12 years. --- src/mesa/main/texcompress_s3tc_tmp.h | 80 1 file changed, 80 deletions(-) diff --git a/src/mesa/main/texcompress_s3tc_tmp.h b/src/mesa/main/texcompress_s3tc_tmp.h index b67b7a299e..57bbf7e0ae 100644 --- a/src/mesa/main/tex

[Mesa-dev] [PATCH 01/15] mesa: Import libtxc_dxtn sources

2017-10-02 Thread Matt Turner
Straight import of the libtxc_dxtn sources. Will be squashed with next commit that reorganizes them into a more familiar form --- src/mesa/Makefile.sources | 3 + src/mesa/main/txc_compress_dxtn.c | 843 ++ src/mesa/main/txc_dxtn.h | 55 +++

[Mesa-dev] [PATCH 02/15] mesa: Combine libtxc_dxtn sources into texcompress_s3tc_tmp.h

2017-10-02 Thread Matt Turner
TODO: Squash with previous commit --- src/mesa/Makefile.sources | 3 - src/mesa/main/texcompress_s3tc.c | 1 + ...{txc_compress_dxtn.c => texcompress_s3tc_tmp.h} | 249 - src/mesa/main/txc_dxtn.h | 55 ---

[Mesa-dev] [PATCH 04/15] mesa: Remove prototypes and mark S3TC functions static

2017-10-02 Thread Matt Turner
This file will be #included, so the functions should be static. --- src/mesa/main/texcompress_s3tc_tmp.h | 23 +-- 1 file changed, 5 insertions(+), 18 deletions(-) diff --git a/src/mesa/main/texcompress_s3tc_tmp.h b/src/mesa/main/texcompress_s3tc_tmp.h index 57bbf7e0ae..3abce

[Mesa-dev] [PATCH 05/15] mesa: Remove fprintf referring to libdxtn

2017-10-02 Thread Matt Turner
When this file is included by Gallium, the fprintf causes it to fail to compile. This is an unreachable error case, and we shouldn't be calling fprintf directly. --- src/mesa/main/texcompress_s3tc_tmp.h | 1 - 1 file changed, 1 deletion(-) diff --git a/src/mesa/main/texcompress_s3tc_tmp.h b/src/

[Mesa-dev] [PATCH 06/15] mesa: Call DXTn functions directly

2017-10-02 Thread Matt Turner
--- src/mesa/main/texcompress_s3tc.c | 117 +-- 1 file changed, 25 insertions(+), 92 deletions(-) diff --git a/src/mesa/main/texcompress_s3tc.c b/src/mesa/main/texcompress_s3tc.c index e08ed23d3a..e74e4c402a 100644 --- a/src/mesa/main/texcompress_s3tc.c +++ b/s

[Mesa-dev] [PATCH 07/15] mesa: Drop function pointer checks in s3tc code

2017-10-02 Thread Matt Turner
Now never null! --- src/mesa/main/texcompress_s3tc.c | 193 --- 1 file changed, 60 insertions(+), 133 deletions(-) diff --git a/src/mesa/main/texcompress_s3tc.c b/src/mesa/main/texcompress_s3tc.c index e74e4c402a..14067f0e99 100644 --- a/src/mesa/main/texcompre

[Mesa-dev] [PATCH 09/15] mesa/st: Drop has_lib_dxtc argument from st_init_extensions()

2017-10-02 Thread Matt Turner
--- src/mesa/state_tracker/st_context.c| 2 +- src/mesa/state_tracker/st_extensions.c | 8 +--- src/mesa/state_tracker/st_extensions.h | 3 +-- src/mesa/state_tracker/st_manager.c| 2 +- 4 files changed, 4 insertions(+), 11 deletions(-) diff --git a/src/mesa/state_tracker/st_context.c

[Mesa-dev] [PATCH 11/15] gallium: Remove util_format_s3tc_init()

2017-10-02 Thread Matt Turner
--- src/gallium/auxiliary/util/u_format_s3tc.c | 79 ++-- src/gallium/auxiliary/util/u_format_s3tc.h | 4 -- src/gallium/drivers/freedreno/freedreno_screen.c | 2 - src/gallium/drivers/i915/i915_screen.c | 2 - src/gallium/drivers/llvmpipe/lp_screen.c

[Mesa-dev] [PATCH 13/15] mesa: Delete now unused dlopen.h

2017-10-02 Thread Matt Turner
--- src/mesa/Makefile.sources| 1 - src/mesa/main/dlopen.h | 97 src/mesa/main/texcompress_s3tc.c | 1 - 3 files changed, 99 deletions(-) delete mode 100644 src/mesa/main/dlopen.h diff --git a/src/mesa/Makefile.sources b/src/mesa/Makef

[Mesa-dev] [PATCH 12/15] mesa: Remove force_s3tc_enable driconf variable

2017-10-02 Thread Matt Turner
--- src/gallium/auxiliary/pipe-loader/driinfo_gallium.h | 1 - src/gallium/include/state_tracker/st_api.h | 1 - src/gallium/state_trackers/dri/dri_screen.c | 2 -- src/gallium/state_trackers/osmesa/osmesa.c | 1 - src/mesa/drivers/dri/i915/intel_screen.c| 1 -

[Mesa-dev] [PATCH 08/15] mesa: Drop Mesa_DXTn from gl_context

2017-10-02 Thread Matt Turner
--- src/mesa/drivers/dri/i915/intel_extensions.c | 5 + src/mesa/drivers/dri/i965/intel_extensions.c | 4 +--- src/mesa/drivers/dri/nouveau/nv10_context.c | 6 ++ src/mesa/drivers/dri/nouveau/nv20_context.c | 6 ++ src/mesa/drivers/dri/r200/r200_context.c | 10 ++--- src/

[Mesa-dev] [PATCH 15/15] travis: Remove libtxc_dxtn from the build

2017-10-02 Thread Matt Turner
--- .travis.yml | 14 -- 1 file changed, 14 deletions(-) diff --git a/.travis.yml b/.travis.yml index d9a8bf5a9d..088295e133 100644 --- a/.travis.yml +++ b/.travis.yml @@ -20,7 +20,6 @@ env: - XCBPROTO_VERSION=xcb-proto-1.11 - LIBXCB_VERSION=libxcb-1.11 - LIBXSHMFENCE_V

[Mesa-dev] [PATCH 10/15] gallium: Remove util_format_s3tc_enabled

2017-10-02 Thread Matt Turner
--- src/gallium/auxiliary/util/u_format.c | 4 -- src/gallium/auxiliary/util/u_format_s3tc.c| 54 +-- src/gallium/auxiliary/util/u_format_s3tc.h| 2 - src/gallium/drivers/llvmpipe/lp_screen.c | 4 -- src/gallium/drivers/llvmpipe/lp_test_format.c | 5

[Mesa-dev] [PATCH 14/15] build: Remove HAVE_DLOPEN

2017-10-02 Thread Matt Turner
--- Android.common.mk | 1 - configure.ac| 4 ++-- meson.build | 1 - src/mesa/SConscript | 4 4 files changed, 2 insertions(+), 8 deletions(-) diff --git a/Android.common.mk b/Android.common.mk index 4d5daf8e9d..483f6c5be2 100644 --- a/Android.common.mk +++ b/Android.commo

Re: [Mesa-dev] [PATCH 4/7] i965: Record the presence of the kernel scheduler

2017-10-02 Thread Joonas Lahtinen
On Fri, 2017-09-29 at 12:52 -0700, Kenneth Graunke wrote: > On Friday, September 29, 2017 3:25:07 AM PDT Chris Wilson wrote: > > Mention to the debug log if the kernel scheduler is enabled; and in > > particular if it has preemption enabled. > > > > Signed-off-by: Chris Wilson > > Cc: Joonas Laht

Re: [Mesa-dev] [PATCH] radv: Check for GFX9 for 1D arrays in image_size intrinsic.

2017-10-02 Thread Juan A. Suarez Romero
Small clarification: my understanding is that this patch is intended only for 17.2 stable release, right? As we already have 979978ee06867a5 in master for this. On Sun, 2017-10-01 at 10:07 +0200, Bas Nieuwenhuizen wrote: > Only on GFX9 we implement them as 2D images. > > This fixes: > dEQP-VK.

Re: [Mesa-dev] [PATCH 2/4] gallivm: allow additional llc options

2017-10-02 Thread Nicolai Hähnle
On 28.09.2017 20:09, Ben Crocker wrote: In init_native_targets, allow the passing of additional options to the LLC compiler via new GALLIVM_LLC_OPTIONS environmental control. This option is available only #ifdef DEBUG, initially. At top, add #include for LLVMParseCommandLineOptions() declaration

Re: [Mesa-dev] [PATCH 01/19] radv: skip the decompression color pass when CMASK/FMASK/DCC are disabled

2017-10-02 Thread Samuel Pitoiset
On 10/01/2017 09:30 PM, Dave Airlie wrote: On 1 Oct. 2017 22:03, "Bas Nieuwenhuizen" > wrote: Why add this? It sounds like extra code for no reason? According to the way we initialize DCC/CMASK/FMASK, I think we can end up by not allocating DCC and CM

Re: [Mesa-dev] [PATCH 02/19] radv: do not allocate CMASK for unsupported 128 bit formats

2017-10-02 Thread Samuel Pitoiset
On 10/01/2017 07:11 AM, Bas Nieuwenhuizen wrote: Do we know if this does not have any improvement for 128 bit MSAA, even without fast clear? Ah right, missed this point. I have no ideas though. On Fri, Sep 29, 2017 at 5:48 PM, Samuel Pitoiset wrote: Signed-off-by: Samuel Pitoiset ---

Re: [Mesa-dev] [PATCH 08/19] radv: remove the resummarize decompress depth state

2017-10-02 Thread Samuel Pitoiset
On 10/01/2017 11:50 PM, Bas Nieuwenhuizen wrote: I'd like to keep this due to potential use for fixing the htile bugs. See e.g. https://bugs.freedesktop.org/show_bug.cgi?id=102488 Okay, makes sense to keep it. On Fri, Sep 29, 2017 at 5:48 PM, Samuel Pitoiset wrote: It's unused. Signed-

Re: [Mesa-dev] [PATCH] gallium/u_tests: test sync_file fences

2017-10-02 Thread Nicolai Hähnle
On 29.09.2017 14:26, Marek Olšák wrote: From: Marek Olšák This should be sufficient for testing all kernel/libdrm/radeonsi codepaths that are used by radeonsi. Reviewed-by: Nicolai Hähnle --- src/gallium/auxiliary/util/u_tests.c | 101 +++ 1 file changed

Re: [Mesa-dev] [PATCH 09/19] radv: remove radv_layout_is_htile_compressed()

2017-10-02 Thread Samuel Pitoiset
On 10/01/2017 11:49 PM, Bas Nieuwenhuizen wrote: I'd like to keep this because they specify different concepts: has_htile is whether the htile metadata is valid in combination with the image data, i.e. we can enable htile reads if we want, and don't have to . is_htile_compressed is whether we

Re: [Mesa-dev] [PATCH 6/7] i965: Reported supported context priorities to EGL/DRI

2017-10-02 Thread Chris Wilson
Quoting Kenneth Graunke (2017-09-29 22:59:56) > On Friday, September 29, 2017 1:03:14 PM PDT Chris Wilson wrote: > > Quoting Kenneth Graunke (2017-09-29 20:55:53) > > > On Friday, September 29, 2017 3:25:09 AM PDT Chris Wilson wrote: > > > > Hook up the RendererQuery for __DRI2_RENDERER_HAS_CONTEXT

Re: [Mesa-dev] [PATCH v2] meson: change vulkan icd config to - instead of _

2017-10-02 Thread Eric Engestrom
On Saturday, 2017-09-30 01:00:56 +, Dylan Baker wrote: > Just to be consistent. > > v2: - update meson.build too > > Signed-off-by: Dylan Baker > --- > meson.build | 2 +- > meson_options.txt | 4 ++-- > 2 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/meson.build b/

Re: [Mesa-dev] [PATCH 01/11] radeonsi: use ac helpers for bitcasts

2017-10-02 Thread Nicolai Hähnle
On 29.09.2017 16:49, Marek Olšák wrote: From: Marek Olšák --- src/gallium/drivers/radeonsi/si_shader.c | 147 - src/gallium/drivers/radeonsi/si_shader_tgsi_alu.c | 14 +- src/gallium/drivers/radeonsi/si_shader_tgsi_mem.c | 38 +++--- .../drivers/radeonsi/s

Re: [Mesa-dev] [PATCH v3 0/2] build system: Unify c++11 detection and used [was: configure+mesa/st:check -std=c++11 support and enable tests accordingly]

2017-10-02 Thread Gert Wollny
Ping ... Am Dienstag, den 19.09.2017, 13:35 +0200 schrieb Gert Wollny: > Was: configure+mesa/st:  > > As Emil pointed out, adding a test for C++11 for only one test is not > very nice, and since swr and clover already did distinct checks for > this, I propose to unify these tests and base comp

Re: [Mesa-dev] [PATCH 05/11] radeonsi: use ctx->ac.builder

2017-10-02 Thread Nicolai Hähnle
On 29.09.2017 16:49, Marek Olšák wrote: From: Marek Olšák --- src/gallium/drivers/radeonsi/si_shader.c | 401 ++--- src/gallium/drivers/radeonsi/si_shader_tgsi_alu.c | 161 - src/gallium/drivers/radeonsi/si_shader_tgsi_mem.c | 114 +++--- .../drivers/rad

[Mesa-dev] [PATCH] radv: make radv_dynamic_state_copy() static

2017-10-02 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_cmd_buffer.c | 2 +- src/amd/vulkan/radv_private.h| 4 2 files changed, 1 insertion(+), 5 deletions(-) diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c index 19b3b24a4b..61ea11c12a 100644 --- a/src/am

[Mesa-dev] [PATCH mesa] egl/wayland: simplify LIBGL_ALWAYS_SOFTWARE logic

2017-10-02 Thread Eric Engestrom
Signed-off-by: Eric Engestrom --- src/egl/drivers/dri2/platform_wayland.c | 12 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/src/egl/drivers/dri2/platform_wayland.c b/src/egl/drivers/dri2/platform_wayland.c index ae3347756bc1a4629b69..011dddfabf422f30cb6c 100644 ---

Re: [Mesa-dev] [PATCH] st/mesa: don't use pipe_surface for passing information about EGLImage

2017-10-02 Thread Nicolai Hähnle
On 29.09.2017 23:28, Marek Olšák wrote: From: Marek Olšák Use st_egl_image instead. radeonsi doesn't like when we create a pipe_surface with PIPE_FORMAT_NV12. This fixes NV12 texturing on radeonsi using kmscube. Cc: 17.1 17.2 This could probably benefit from testing with other drivers befo

Re: [Mesa-dev] [PATCH 03/11] tgsi/scan: scan address operands

2017-10-02 Thread Nicolai Hähnle
On 29.09.2017 14:25, Marek Olšák wrote: From: Marek Olšák --- src/gallium/auxiliary/tgsi/tgsi_scan.c | 45 +- 1 file changed, 44 insertions(+), 1 deletion(-) diff --git a/src/gallium/auxiliary/tgsi/tgsi_scan.c b/src/gallium/auxiliary/tgsi/tgsi_scan.c index 2

Re: [Mesa-dev] [PATCH] egl/surfaceless: Use KMS swrast fallback

2017-10-02 Thread Eric Engestrom
On Friday, 2017-09-29 23:57:32 +, gurchetansi...@chromium.org wrote: > From: Gurchetan Singh > > The kms_swrast extension is an actively developed software fallback, > and platform_surfaceless can use it if there are no available > hardware drivers. > --- > src/egl/drivers/dri2/platform_surf

Re: [Mesa-dev] [PATCH 07/11] glsl_to_tgsi: enable copy propagation for tessellation shaders

2017-10-02 Thread Nicolai Hähnle
On 29.09.2017 14:25, Marek Olšák wrote: From: Marek Olšák just don't propagate output reads --- src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp b/src/mesa/state_tracker/st_glsl_to

Re: [Mesa-dev] [PATCH 09/11] glsl_to_tgsi: each reladdr object should have only one parent

2017-10-02 Thread Nicolai Hähnle
On 29.09.2017 14:25, Marek Olšák wrote: From: Marek Olšák required by rename_temp_registers. --- src/mesa/state_tracker/st_glsl_to_tgsi_private.cpp | 66 -- src/mesa/state_tracker/st_glsl_to_tgsi_private.h | 4 ++ 2 files changed, 65 insertions(+), 5 deletions(-) dif

Re: [Mesa-dev] [PATCH 11/11] glsl_to_tgsi: skip UARL for 1D registers if the driver doesn't need it

2017-10-02 Thread Nicolai Hähnle
On 29.09.2017 14:25, Marek Olšák wrote: From: Marek Olšák --- src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 45 +++- src/mesa/state_tracker/st_glsl_to_tgsi_private.h | 8 + 2 files changed, 44 insertions(+), 9 deletions(-) diff --git a/src/mesa/state_tracker/s

Re: [Mesa-dev] [PATCH 05/15] mesa: Remove fprintf referring to libdxtn

2017-10-02 Thread Nicolai Hähnle
On 02.10.2017 08:59, Matt Turner wrote: When this file is included by Gallium, the fprintf causes it to fail to compile. This is an unreachable error case, and we shouldn't be calling fprintf directly. --- src/mesa/main/texcompress_s3tc_tmp.h | 1 - 1 file changed, 1 deletion(-) diff --git a/

Re: [Mesa-dev] [PATCH 06/15] mesa: Call DXTn functions directly

2017-10-02 Thread Nicolai Hähnle
On 02.10.2017 08:59, Matt Turner wrote: --- src/mesa/main/texcompress_s3tc.c | 117 +-- 1 file changed, 25 insertions(+), 92 deletions(-) diff --git a/src/mesa/main/texcompress_s3tc.c b/src/mesa/main/texcompress_s3tc.c index e08ed23d3a..e74e4c402a 100644 --

Re: [Mesa-dev] [PATCH 00/15] Import libtxc_dxtn's S3TC code into Mesa!

2017-10-02 Thread Nicolai Hähnle
On 02.10.2017 08:59, Matt Turner wrote: The patent on S3TC has now expired! Rejoice! Hooray! :) There will be much partying at the Khronos Munich local chapter meeting next week :) I've sent some minor comments on patches 5 & 6. Apart from that, the series is Reviewed-by: Nicolai Hähnle

Re: [Mesa-dev] [PATCH mesa] egl/wayland: simplify LIBGL_ALWAYS_SOFTWARE logic

2017-10-02 Thread Nicolai Hähnle
On 02.10.2017 12:35, Eric Engestrom wrote: Signed-off-by: Eric Engestrom Reviewed-by: Nicolai Hähnle --- src/egl/drivers/dri2/platform_wayland.c | 12 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/src/egl/drivers/dri2/platform_wayland.c b/src/egl/drivers/dr

Re: [Mesa-dev] [PATCH mesa] egl/wayland: simplify LIBGL_ALWAYS_SOFTWARE logic

2017-10-02 Thread Emil Velikov
On 2 October 2017 at 11:35, Eric Engestrom wrote: > Signed-off-by: Eric Engestrom > --- > src/egl/drivers/dri2/platform_wayland.c | 12 > 1 file changed, 4 insertions(+), 8 deletions(-) > Reviewed-by: Emil Velikov -Emil ___ mesa-dev mail

Re: [Mesa-dev] [PATCH] egl/surfaceless: Use KMS swrast fallback

2017-10-02 Thread Emil Velikov
Hi Gurchetan, On 30 September 2017 at 00:57, gurchetansi...@chromium.org wrote: > From: Gurchetan Singh > > The kms_swrast extension is an actively developed software fallback, > and platform_surfaceless can use it if there are no available > hardware drivers. > --- > src/egl/drivers/dri2/platf

Re: [Mesa-dev] [PATCH] radv: Check for GFX9 for 1D arrays in image_size intrinsic.

2017-10-02 Thread Bas Nieuwenhuizen
Yes, that is correct. On 2 Oct 2017 11:20, "Juan A. Suarez Romero" wrote: > Small clarification: my understanding is that this patch is intended > only for 17.2 stable release, right? > > As we already have 979978ee06867a5 in master for this. > > > > On Sun, 2017-10-01 at 10:07 +0200, Bas Nieuwe

Re: [Mesa-dev] [PATCH v3 1/2] configure+mesa/st: unify check for -std=c++11 support and enable accordingly

2017-10-02 Thread Emil Velikov
Hi Gert, On 19 September 2017 at 12:35, Gert Wollny wrote: > Unify the CXX feature tests for C++11 support that is required for SWR, > clover, and mesa/st/tests. > Surely we'll have capable compiler in 90+% of the time, still asking for C++11 for single test is an overkill. Then again, let's get

[Mesa-dev] [OS]Mesa MinGW AppVeyor Repository

2017-10-02 Thread Trevor Sandy
Hi George, You might be interested to know I have set up a CI env on AppVeyor for MinGW builds. Here is the GitHub repo: https://github.com/trevorsandy/osmesa_mingw_av. Follow the badge in the README to the AppVeyor instance. You can demo your patches quite easily - see the README.md on how to e

Re: [Mesa-dev] [PATCH 01/15] mesa: Import libtxc_dxtn sources

2017-10-02 Thread Emil Velikov
On 2 October 2017 at 07:59, Matt Turner wrote: > Straight import of the libtxc_dxtn sources. Will be squashed with next > commit that reorganizes them into a more familiar form The code builds and works like a charm, so I won't bother squashing the commits. Please add a reference to the repo/comm

Re: [Mesa-dev] [PATCH 06/15] mesa: Call DXTn functions directly

2017-10-02 Thread Emil Velikov
On 2 October 2017 at 07:59, Matt Turner wrote: > --- > src/mesa/main/texcompress_s3tc.c | 117 > +-- > 1 file changed, 25 insertions(+), 92 deletions(-) > > diff --git a/src/mesa/main/texcompress_s3tc.c > b/src/mesa/main/texcompress_s3tc.c > index e08ed23d3a.

Re: [Mesa-dev] [PATCH 03/15] mesa: Remove commented-out DXTn fetch code

2017-10-02 Thread Emil Velikov
On 2 October 2017 at 07:59, Matt Turner wrote: > Has been disabled for 12 years. > --- > src/mesa/main/texcompress_s3tc_tmp.h | 80 > > 1 file changed, 80 deletions(-) > I would have made this 2/15 since it's a bit meh to move dead code. -Emil __

Re: [Mesa-dev] [PATCH 06/15] mesa: Call DXTn functions directly

2017-10-02 Thread Emil Velikov
On 2 October 2017 at 14:05, Emil Velikov wrote: >> - if (ext_tx_compress_dxtn) { >> - (*ext_tx_compress_dxtn)(3, srcWidth, srcHeight, pixels, >> + if (tx_compress_dxtn) { >> + (*tx_compress_dxtn)(3, srcWidth, srcHeight, pixels, > Please use something like the following. It will drop

Re: [Mesa-dev] [PATCH 10/15] gallium: Remove util_format_s3tc_enabled

2017-10-02 Thread Emil Velikov
On 2 October 2017 at 07:59, Matt Turner wrote: > +#include "../../../mesa/main/texcompress_s3tc_tmp.h" > This feels a bit dirty. If people are not too keen on the idea, another approach is to have the code in src/util/ -Emil ___ mesa-dev mailing list m

Re: [Mesa-dev] [PATCH 00/15] Import libtxc_dxtn's S3TC code into Mesa!

2017-10-02 Thread Emil Velikov
On 2 October 2017 at 07:59, Matt Turner wrote: > The patent on S3TC has now expired! Rejoice! > It's time to open that 20 year old bottle of wine ;-) > This series imports the code from Roland's libtxc_dxtn into Mesa and cleans up > some mess dealing with the possibility of S3TC not being availab

Re: [Mesa-dev] [PATCH 01/11] radeonsi: use ac helpers for bitcasts

2017-10-02 Thread Marek Olšák
This reply of yours doesn't contain anything. (clipped message?) Marek ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH v3 1/2] configure+mesa/st: unify check for -std=c++11 support and enable accordingly

2017-10-02 Thread Gert Wollny
Am Montag, den 02.10.2017, 13:37 +0100 schrieb Emil Velikov: > Hi Gert, > > On 19 September 2017 at 12:35, Gert Wollny > wrote: > > Unify the CXX feature tests for C++11 support that is required for > > SWR, clover, and mesa/st/tests. > > > > Surely we'll have capable compiler in 90+% of the ti

Re: [Mesa-dev] [OS]Mesa MinGW AppVeyor Repository

2017-10-02 Thread Emil Velikov
Hi Trevor, On 2 October 2017 at 13:48, Trevor Sandy wrote: > Hi George, > > You might be interested to know I have set up a CI env on AppVeyor for MinGW > builds. > > Here is the GitHub repo: https://github.com/trevorsandy/osmesa_mingw_av. > Follow the badge in the README to the AppVeyor instance

Re: [Mesa-dev] [OS]Mesa MinGW AppVeyor Repository

2017-10-02 Thread Kyriazis, George
Emil, I’m working with Trevor to upstream the correct versions of the swr patches. Thanks, George > On Oct 2, 2017, at 9:04 AM, Emil Velikov wrote: > > Hi Trevor, > > On 2 October 2017 at 13:48, Trevor Sandy wrote: >> Hi George, >> >> You might be interested to know I have set up a CI env

Re: [Mesa-dev] [OS]Mesa MinGW AppVeyor Repository

2017-10-02 Thread Trevor (CIMdata)
Hello Emil, Yes, I did see the AppVeyor integraton for MSVC. It could be quite easy to add the MinGW scope to the metrix. I don’t really know how to ‘upsteam’ the patches I use mostly because I am not the author. Do I just submit a PR? Anyway to help George with his patch – which fixes the MinG

Re: [Mesa-dev] [PATCH v3 1/2] configure+mesa/st: unify check for -std=c++11 support and enable accordingly

2017-10-02 Thread Emil Velikov
On 2 October 2017 at 15:04, Gert Wollny wrote: > Am Montag, den 02.10.2017, 13:37 +0100 schrieb Emil Velikov: >> Hi Gert, >> >> On 19 September 2017 at 12:35, Gert Wollny >> wrote: >> > Unify the CXX feature tests for C++11 support that is required for >> > SWR, clover, and mesa/st/tests. >> > >>

[Mesa-dev] [Bug 103062] Error starting game FS17

2017-10-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103062 Bug ID: 103062 Summary: Error starting game FS17 Product: Mesa Version: 17.2 Hardware: x86-64 (AMD64) OS: Linux (All) Status: NEW Severity: normal

[Mesa-dev] [Bug 103062] Error starting game FS17

2017-10-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103062 --- Comment #1 from alexander --- And the proprietary driver cannot be installed (does Not work). -- You are receiving this mail because: You are the assignee for the bug. You are the QA Contact for the bug.

[Mesa-dev] [Bug 103062] Error starting game FS17

2017-10-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103062 alexander changed: What|Removed |Added QA Contact|mesa-dev@lists.freedesktop. |intel-3d-bugs@lists.freedes

Re: [Mesa-dev] [PATCH 2/4] gallivm: allow additional llc options

2017-10-02 Thread Ben Crocker
- Original Message - > From: "Nicolai Hähnle" > To: "Ben Crocker" , mesa-dev@lists.freedesktop.org > Cc: "Emil Velikov" , "17.2" > > Sent: Monday, October 2, 2017 5:40:31 AM > Subject: Re: [Mesa-dev] [PATCH 2/4] gallivm: allow additional llc options > > On 28.09.2017 20:09, Ben Crocker

Re: [Mesa-dev] [OS]Mesa MinGW AppVeyor Repository

2017-10-02 Thread Emil Velikov
On 2 October 2017 at 15:16, Trevor (CIMdata) wrote: > Hello Emil, > > Yes, I did see the AppVeyor integraton for MSVC. It could be quite easy to > add the MinGW scope to the metrix. > > > > I don’t really know how to ‘upsteam’ the patches I use mostly because I am > not the author. Do I just submi

[Mesa-dev] [Bug 103062] Error starting game FS17

2017-10-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103062 --- Comment #2 from Emil Velikov --- What hardware are you using? Can you attach the FS17 log (as plain text). Implementing GL_NV_texture_compression_vtc in Mesa might be problematic since the extension IP status is "NVIDIA Proprietary". Then

Re: [Mesa-dev] [OS]Mesa MinGW AppVeyor Repository

2017-10-02 Thread Trevor Sandy
Many thanks for the details - and the reminder on passively contributing to wasting developers' time :-) For the content I created, I will proceed accordingly. I've reached out to the repo owner where I got the patches I use to validate the MinGW build to see his opinion. If there is no conflict,

Re: [Mesa-dev] [Mesa-stable] [PATCH] st/dri: don't expose modifiers in EGL if the driver doesn't implement them

2017-10-02 Thread Emil Velikov
On 27 September 2017 at 20:27, Juan A. Suarez Romero wrote: > On Wed, 2017-09-27 at 18:09 +0100, Emil Velikov wrote: >> On 27 September 2017 at 17:28, Marek Olšák wrote: >> > On Wed, Sep 27, 2017 at 6:22 PM, Emil Velikov >> > wrote: >> > > On 27 September 2017 at 16:00, Daniel Stone wrote: >>

Re: [Mesa-dev] [PATCH v3 1/2] configure+mesa/st: unify check for -std=c++11 support and enable accordingly

2017-10-02 Thread Gert Wollny
Am Montag, den 02.10.2017, 15:16 +0100 schrieb Emil Velikov: > > > While rebuilding most of Mesa + its tests (as illustrated in 2/2) > only for a single test is very wasteful. > > Let's not do that, pretty please? I'm not quite sure what you mean by that. IHMO as long as mesa is supposed to sup

[Mesa-dev] [Bug 102847] swr fail to build with llvm-5.0.0

2017-10-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=102847 --- Comment #4 from Juan A. Suarez --- Patch https://patchwork.freedesktop.org/patch/177834/ has been landed upstream. Does it fix this issue? -- You are receiving this mail because: You are the QA Contact for the bug. You are the assignee fo

Re: [Mesa-dev] [PATCH v2 2/3] vulkan/wsi: Report the correct min/maxImageCount

2017-10-02 Thread Jason Ekstrand
On Sun, Oct 1, 2017 at 10:32 PM, Neil Roberts wrote: > Jason Ekstrand writes: > > > Hey, Neil! > > Hey Jason :) > > > Yeah... That's a bit unfortunate. The problem is that we have no way of > > returning a different number of images depending on the mode. In theory, > > we could start out at 2

Re: [Mesa-dev] [PATCH] swr/rast: do not crash on NULL strings returned by getenv

2017-10-02 Thread Emil Velikov
On 19 September 2017 at 21:08, Rowley, Timothy O wrote: > I have a bit of a preference for Eric’s version. > Ack. Amended and pushed to master. -Emil ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo

Re: [Mesa-dev] [PATCH 4/5] meson: convert gtest to an internal dependency

2017-10-02 Thread Eric Engestrom
On Sunday, 2017-10-01 03:58:27 +, Dylan Baker wrote: > In truth gtest is an external dependency that upstream expects you to > "vendor" into your own tree. As such, it makes sense to treat it more > like a dependency than an internal library, and collect it's > requirements together in a depend

Re: [Mesa-dev] [PATCH] st/dri: don't expose modifiers in EGL if the driver doesn't implement them

2017-10-02 Thread Daniel Stone
Hi Andy, On 27 September 2017 at 20:34, Andy Furniss wrote: > Marek Olšák wrote: >> Sorry too late, I pushed it. >> >> I don't know if stable is affected. > > It regresses things starting on radeonsi using weston eg. > > mpv - > > [vo/opengl/wayland] error occurred on the display fd: closing file

[Mesa-dev] [Bug 103062] Error starting game FS17

2017-10-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103062 --- Comment #3 from alexander --- Created attachment 134620 --> https://bugs.freedesktop.org/attachment.cgi?id=134620&action=edit Log FS17 Wine FS17 incorrectly defines my CPU. In fact AMD-A8 6410 with AMD Radeon R5 Graphics 4 core. I have an

[Mesa-dev] [Bug 103062] Error starting game FS17

2017-10-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103062 --- Comment #4 from alexander --- (In reply to Emil Velikov from comment #2) > Implementing GL_NV_texture_compression_vtc in Mesa might be problematic > since the extension IP status is "NVIDIA Proprietary". How to upgrade the version of the Sha

Re: [Mesa-dev] [PATCH 7/7] wayland-egl: rework and simplify wl_egl_window initialization

2017-10-02 Thread Emil Velikov
On 29 September 2017 at 21:40, Miguel Angel Vico wrote: >> Miguel I believe the comment correctly describes the design plan, while >> addressing Dan's comment that things look a bit ugly. > > Yes. Thank you. > > Also, the whole series: > > Reviewed-by: Miguel A. Vico > Smashing, thanks guys. Push

Re: [Mesa-dev] [PATCH 5/5] travis: add meson build for vulkan drivers.

2017-10-02 Thread Eric Engestrom
On Sunday, 2017-10-01 03:58:28 +, Dylan Baker wrote: > Signed-off-by: Dylan Baker > --- > .travis.yml | 36 > 1 file changed, 36 insertions(+) > > diff --git a/.travis.yml b/.travis.yml > index d9a8bf5a9d4..e8fff1b66ab 100644 > --- a/.travis.yml > +++ b/.

Re: [Mesa-dev] [PATCH 4/7] i965: Record the presence of the kernel scheduler

2017-10-02 Thread Kenneth Graunke
On Monday, October 2, 2017 1:52:12 AM PDT Joonas Lahtinen wrote: > On Fri, 2017-09-29 at 12:52 -0700, Kenneth Graunke wrote: > > On Friday, September 29, 2017 3:25:07 AM PDT Chris Wilson wrote: > > > Mention to the debug log if the kernel scheduler is enabled; and in > > > particular if it has pree

Re: [Mesa-dev] [PATCH 6/7] i965: Reported supported context priorities to EGL/DRI

2017-10-02 Thread Kenneth Graunke
On Monday, October 2, 2017 2:44:13 AM PDT Chris Wilson wrote: > Quoting Kenneth Graunke (2017-09-29 22:59:56) > > On Friday, September 29, 2017 1:03:14 PM PDT Chris Wilson wrote: > > > Quoting Kenneth Graunke (2017-09-29 20:55:53) > > > > On Friday, September 29, 2017 3:25:09 AM PDT Chris Wilson wr

[Mesa-dev] [PATCH] egl/wayland: Don't use dmabuf with no modifiers

2017-10-02 Thread Daniel Stone
The dmabuf interface requires a valid modifier to be sent. If we don't explicitly get a modifier from the driver, we can't know what to send; it must be inferred from legacy side-channels (or assumed to linear, if none exists). If we have no modifier, then we can only have a single-plane format an

[Mesa-dev] [Bug 103062] Error starting game FS17

2017-10-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=103062 --- Comment #5 from Emil Velikov --- (In reply to alexander from comment #4) > How to upgrade the version of the Shader? That's not it works. Can you provide a more complete log/output from the game? Alternatively getting some connection betwee

Re: [Mesa-dev] [PATCH] egl/wayland: Don't use dmabuf with no modifiers

2017-10-02 Thread Emil Velikov
On 2 October 2017 at 16:55, Daniel Stone wrote: > The dmabuf interface requires a valid modifier to be sent. If we don't > explicitly get a modifier from the driver, we can't know what to send; > it must be inferred from legacy side-channels (or assumed to linear, if > none exists). > > If we have

Re: [Mesa-dev] [PATCH] egl/wayland: Don't use dmabuf with no modifiers

2017-10-02 Thread Daniel Stone
Hey Emil, On 2 October 2017 at 17:08, Emil Velikov wrote: > On 2 October 2017 at 16:55, Daniel Stone wrote: >> The dmabuf interface requires a valid modifier to be sent. If we don't >> explicitly get a modifier from the driver, we can't know what to send; >> it must be inferred from legacy side-

Re: [Mesa-dev] [PATCH 5/5] travis: add meson build for vulkan drivers.

2017-10-02 Thread Emil Velikov
On 2 October 2017 at 16:51, Eric Engestrom wrote: > On Sunday, 2017-10-01 03:58:28 +, Dylan Baker wrote: >> Signed-off-by: Dylan Baker >> --- >> .travis.yml | 36 >> 1 file changed, 36 insertions(+) >> >> diff --git a/.travis.yml b/.travis.yml >> index d9

[Mesa-dev] [ANNOUNCE] mesa 17.2.2

2017-10-02 Thread Juan A. Suarez Romero
Mesa 17.2.2 is now available. In this release we have: In build and integration system, we add a dependency on libunwind when running make distcheck, as this is optional for libgallium but we want to catch any problem. As consequence, also force LLVM 3.3 in Travis when building Gallium ST Other,

Re: [Mesa-dev] [PATCH v2] meson: change vulkan icd config to - instead of _

2017-10-02 Thread Dylan Baker
Quoting Eric Engestrom (2017-10-02 02:49:51) > On Saturday, 2017-09-30 01:00:56 +, Dylan Baker wrote: > > Just to be consistent. > > > > v2: - update meson.build too > > > > Signed-off-by: Dylan Baker > > --- > > meson.build | 2 +- > > meson_options.txt | 4 ++-- > > 2 files changed,

[Mesa-dev] [PATCH v2 1/2] egl/wayland: Check queryImage return for wl_buffer

2017-10-02 Thread Daniel Stone
When creating a wl_buffer from a DRIImage, we extract all the DRIImage information via queryImage. Check whether or not it actually succeeds, either bailing out if the query was critical, or providing sensible fallbacks for information which was not available in older DRIImage versions. Fixes: a65

[Mesa-dev] [PATCH v2 2/2] egl/wayland: Don't use dmabuf with no modifiers

2017-10-02 Thread Daniel Stone
The dmabuf interface requires a valid modifier to be sent. If we don't explicitly get a modifier from the driver, we can't know what to send; it must be inferred from legacy side-channels (or assumed to linear, if none exists). If we have no modifier, then we can only have a single-plane format an

Re: [Mesa-dev] [PATCH 10/15] gallium: Remove util_format_s3tc_enabled

2017-10-02 Thread Nicolai Hähnle
On 02.10.2017 15:17, Emil Velikov wrote: On 2 October 2017 at 07:59, Matt Turner wrote: +#include "../../../mesa/main/texcompress_s3tc_tmp.h" This feels a bit dirty. If people are not too keen on the idea, another approach is to have the code in src/util/ I like that idea. Cheers, Nicolai

Re: [Mesa-dev] [PATCH 4/5] meson: convert gtest to an internal dependency

2017-10-02 Thread Dylan Baker
Quoting Eric Engestrom (2017-10-02 08:33:14) > On Sunday, 2017-10-01 03:58:27 +, Dylan Baker wrote: > > In truth gtest is an external dependency that upstream expects you to > > "vendor" into your own tree. As such, it makes sense to treat it more > > like a dependency than an internal library,

Re: [Mesa-dev] [PATCH] radv: make radv_dynamic_state_copy() static

2017-10-02 Thread Bas Nieuwenhuizen
Reviewed-by: Bas Nieuwenhuizen On 2 Oct 2017 12:26, "Samuel Pitoiset" wrote: > Signed-off-by: Samuel Pitoiset > --- > src/amd/vulkan/radv_cmd_buffer.c | 2 +- > src/amd/vulkan/radv_private.h| 4 > 2 files changed, 1 insertion(+), 5 deletions(-) > > diff --git a/src/amd/vulkan/radv_cm

Re: [Mesa-dev] [PATCH 5/5] travis: add meson build for vulkan drivers.

2017-10-02 Thread Dylan Baker
Quoting Emil Velikov (2017-10-02 09:13:49) > On 2 October 2017 at 16:51, Eric Engestrom wrote: > > On Sunday, 2017-10-01 03:58:28 +, Dylan Baker wrote: > >> Signed-off-by: Dylan Baker > >> --- > >> .travis.yml | 36 > >> 1 file changed, 36 insertions(+) >

Re: [Mesa-dev] [PATCH v2 1/2] egl/wayland: Check queryImage return for wl_buffer

2017-10-02 Thread Andy Furniss
These are still a regression for me over git revert a65db0ad1c weston-simple-egl works as does mpv opengl windowed. The issue is that kodi or fullscreen mpv is mostly junk, IIRC from an old bug this is to do with direct scan out and tiling. Daniel Stone wrote: When creating a wl_buffer from

Re: [Mesa-dev] [PATCH 3/3] st/va: add RGB support to vlVaPutSurface

2017-10-02 Thread Andy Furniss
Tested-by: Andy Furniss Leo Liu wrote: --- src/gallium/state_trackers/va/surface.c | 14 +- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/va/surface.c b/src/gallium/state_trackers/va/surface.c index 643cdcd54a..4c2f4b5452 100644 --- a/s

Re: [Mesa-dev] [PATCH v2 2/3] vulkan/wsi: Report the correct min/maxImageCount

2017-10-02 Thread Neil Roberts
Jason Ekstrand writes: > I wish... Unfortunately, the spec says: > > Let *n* be the total number of images in the swapchain, *m* be the > value of VkSurfaceCapabilitiesKHR::minImageCount, and *a* be the > number of presentable images that the application has currently > acquired (i.e. images acq

Re: [Mesa-dev] [OS]Mesa MinGW AppVeyor Repository

2017-10-02 Thread Eric Engestrom
On Monday, 2017-10-02 14:16:02 +, Trevor (CIMdata) wrote: > Hello Emil, > Yes, I did see the AppVeyor integraton for MSVC. It could be quite > easy to add the MinGW scope to the metrix. > > I don’t really know how to ‘upsteam’ the patches I use mostly because > I am not the author. Do I just s

[Mesa-dev] [Bug 102847] swr fail to build with llvm-5.0.0

2017-10-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=102847 Laurent carlier changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

[Mesa-dev] [Bug 101832] [PATCH][regression][bisect] Xorg fails to start after f50aa21456d82c8cb6fbaa565835f1acc1720a5d

2017-10-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101832 Laurent carlier changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

Re: [Mesa-dev] [PATCH 05/15] mesa: Remove fprintf referring to libdxtn

2017-10-02 Thread Matt Turner
On Mon, Oct 2, 2017 at 4:20 AM, Nicolai Hähnle wrote: > On 02.10.2017 08:59, Matt Turner wrote: >> >> When this file is included by Gallium, the fprintf causes it to fail to >> compile. This is an unreachable error case, and we shouldn't be calling >> fprintf directly. >> --- >> src/mesa/main/te

[Mesa-dev] [PATCH] travis: add meson build for vulkan drivers.

2017-10-02 Thread Dylan Baker
v2: - use -isystem`pwd` instead of cp to include fake linux header (Eric, Emil) Signed-off-by: Dylan Baker --- .travis.yml | 36 1 file changed, 36 insertions(+) diff --git a/.travis.yml b/.travis.yml index d9a8bf5a9d4..ca766917f25 100644 --- a/.travis.yml +

Re: [Mesa-dev] [PATCH 06/15] mesa: Call DXTn functions directly

2017-10-02 Thread Matt Turner
On Mon, Oct 2, 2017 at 4:23 AM, Nicolai Hähnle wrote: > On 02.10.2017 08:59, Matt Turner wrote: >> >> --- >> src/mesa/main/texcompress_s3tc.c | 117 >> +-- >> 1 file changed, 25 insertions(+), 92 deletions(-) >> >> diff --git a/src/mesa/main/texcompress_s3tc.

  1   2   >