Re: [Mesa-dev] [PATCH 1/3] etnaviv: Add support for extended texture formats

2017-05-22 Thread Wladimir J. van der Laan
> Thx - will send out updated patches the next days. I quite busy with > lot of stuff atm but > a long weekend is coming :) Yeah np I could update my patches to that too and re-send them if that is more convenient, Wladimir ___ mesa-dev mailing list

Re: [Mesa-dev] [PATCH 0/6] Prehash all the things

2017-05-22 Thread Dieter Nützel
For the series: Tested-by: Dieter Nützel on radeonsi/RX580 Unigine_Heaven-4.0, Unigine_Valley-1.0, Unigine_Superposition-1.0, LS2015 (Wine-staging), Mesa-demos (objviewer) Dieter Am 22.05.2017 20:55, schrieb Thomas Helland: While this doesn't prehash all the things,

Re: [Mesa-dev] [RFC] ARB_gl_spirv and NIR backend for radeonsi

2017-05-22 Thread Timothy Arceri
On 23/05/17 11:45, Rob Clark wrote: On Mon, May 22, 2017 at 9:33 PM, Rob Clark wrote: On Mon, May 22, 2017 at 9:15 PM, Timothy Arceri wrote: On 23/05/17 10:44, Marek Olšák wrote: On Tue, May 23, 2017 at 12:07 AM, Timothy Arceri

Re: [Mesa-dev] [RFC] ARB_gl_spirv and NIR backend for radeonsi

2017-05-22 Thread Jason Ekstrand
On Mon, May 22, 2017 at 6:48 PM, Jason Ekstrand wrote: > On Mon, May 22, 2017 at 6:45 PM, Rob Clark wrote: > >> On Mon, May 22, 2017 at 9:33 PM, Rob Clark wrote: >> > On Mon, May 22, 2017 at 9:15 PM, Timothy Arceri

Re: [Mesa-dev] [RFC] ARB_gl_spirv and NIR backend for radeonsi

2017-05-22 Thread Jason Ekstrand
On Mon, May 22, 2017 at 6:45 PM, Rob Clark wrote: > On Mon, May 22, 2017 at 9:33 PM, Rob Clark wrote: > > On Mon, May 22, 2017 at 9:15 PM, Timothy Arceri > wrote: > >> On 23/05/17 10:44, Marek Olšák wrote: > >>> > >>> On Tue, May

Re: [Mesa-dev] [RFC] ARB_gl_spirv and NIR backend for radeonsi

2017-05-22 Thread Rob Clark
On Mon, May 22, 2017 at 9:33 PM, Rob Clark wrote: > On Mon, May 22, 2017 at 9:15 PM, Timothy Arceri wrote: >> On 23/05/17 10:44, Marek Olšák wrote: >>> >>> On Tue, May 23, 2017 at 12:07 AM, Timothy Arceri >>> wrote: On

Re: [Mesa-dev] [RFC] ARB_gl_spirv and NIR backend for radeonsi

2017-05-22 Thread Rob Clark
On Mon, May 22, 2017 at 9:15 PM, Timothy Arceri wrote: > On 23/05/17 10:44, Marek Olšák wrote: >> >> On Tue, May 23, 2017 at 12:07 AM, Timothy Arceri >> wrote: >>> >>> On 23/05/17 05:02, Marek Olšák wrote: On Sun, May 21, 2017 at 12:48

Re: [Mesa-dev] [RFC] ARB_gl_spirv and NIR backend for radeonsi

2017-05-22 Thread Timothy Arceri
On 23/05/17 10:44, Marek Olšák wrote: On Tue, May 23, 2017 at 12:07 AM, Timothy Arceri wrote: On 23/05/17 05:02, Marek Olšák wrote: On Sun, May 21, 2017 at 12:48 PM, Nicolai Hähnle wrote: Hi all, I've been looking into ARB_gl_spirv for radeonsi.

Re: [Mesa-dev] [RFC] ARB_gl_spirv and NIR backend for radeonsi

2017-05-22 Thread Marek Olšák
On Tue, May 23, 2017 at 12:07 AM, Timothy Arceri wrote: > On 23/05/17 05:02, Marek Olšák wrote: >> >> On Sun, May 21, 2017 at 12:48 PM, Nicolai Hähnle >> wrote: >>> >>> Hi all, >>> >>> I've been looking into ARB_gl_spirv for radeonsi. I don't fancy >>>

[Mesa-dev] [PATCH 2/2] st/dri: Use fence extension in drisw.c

2017-05-22 Thread Gurchetan Singh
This is desirable for synchronization in virtual machines. --- src/gallium/state_trackers/dri/drisw.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/gallium/state_trackers/dri/drisw.c b/src/gallium/state_trackers/dri/drisw.c index b85a73c57d..8fbfa9ecea 100644 ---

[Mesa-dev] [PATCH 1/2] st/dri: move fence implemention into separate file

2017-05-22 Thread Gurchetan Singh
Since the fence implementation is not dri2.c specific, put it in a separate file. This way SW implementations can use this extension too. v2: Don't depend on dri2.c for extensions (Emil) v3: Make this patch only move extension into a separate file (Chad). ---

Re: [Mesa-dev] [PATCH] i965: Add RGBX8888 and RGBA8888 to EGL configure and reorder the list

2017-05-22 Thread Rob Herring
On Mon, May 22, 2017 at 5:47 PM, Mike Lothian wrote: > I'm quite sure on Gentoo lib points to lib64 and we have a lib32 I'm talking about Android. > > On Mon, 22 May 2017 at 22:46 Rob Herring wrote: >> >> On Mon, May 22, 2017 at 9:16 AM, Emil Velikov

Re: [Mesa-dev] [PATCH] Revert "gallium: remove unused PIPE_CC_GCC_VERSION"

2017-05-22 Thread Timothy Arceri
Hi Brain, It's been 7 months since this landed. Why not just send this out with those patches when you are ready to upsteam? On 23/05/17 02:31, Brian Paul wrote: This reverts commit e60928f4c4bd4484821d83f2b16a910ea9f5f9d9. PIPE_CC_GCC_VERSION is used by some of our in-house code which

Re: [Mesa-dev] [PATCH 01/10] mesa: split vertex_array_vertex_buffer() in two

2017-05-22 Thread Dieter Nützel
For the series: Tested-by: Dieter Nützel on radeonsi/RX580 Dieter Am 22.05.2017 07:46, schrieb Timothy Arceri: This will allow us to skip the error checkes when adding KHR_no_error support. Reviewed-by: Nicolai Hähnle ---

Re: [Mesa-dev] [PATCH] mesa: add APPLE_vertex_array_object stubs

2017-05-22 Thread Timothy Arceri
On 23/05/17 03:26, Ian Romanick wrote: WTF? I gave review feedback on IRC, and you said you were going to send a different patch... yet this patch landed. If you're not going to take review feedback, why do you ask for it idr> tarceri: You should be able to just put the functions back in the

Re: [Mesa-dev] [PATCH] i965: Add RGBX8888 and RGBA8888 to EGL configure and reorder the list

2017-05-22 Thread Mike Lothian
I'm quite sure on Gentoo lib points to lib64 and we have a lib32 On Mon, 22 May 2017 at 22:46 Rob Herring wrote: > On Mon, May 22, 2017 at 9:16 AM, Emil Velikov > wrote: > > On 20 May 2017 at 01:16, Rob Herring wrote: > >> On Fri,

Re: [Mesa-dev] [RFC PATCH 00/65] ARB_bindless_texture for RadeonSI

2017-05-22 Thread Dieter Nützel
Hello Samuel, running this on radeonsi/RX580. Didn't saw better numbers for Wine/TS2017 (TrainSimulator 2017) ever before. But didn't feaguered if it (can) use bindless (with Wine/stagging). Do we have a knob to toggle bindless on and off? A key from Feral for DiRT Rally or the like would be

Re: [Mesa-dev] driconf: Add allow_glsl_builtin_variable_redeclaration option

2017-05-22 Thread Dieter Nützel
Am 22.05.2017 23:56, schrieb Samuel Pitoiset: On 05/22/2017 11:49 PM, Dieter Nützel wrote: Hello John, hello Samuel, is 'allow_glsl_builtin_redeclaration=true' needed after this (commit #bf4d7671f423f5d4e451fba81bb9d9ae57f5fe5b) landed? Hi, Yes, because the option is only enabled with

Re: [Mesa-dev] [PATCH] u_format_test: Ignore S3TC errors.

2017-05-22 Thread Eric Anholt
Jose Fonseca writes: > This prevents spurious failures when libtxc-dxtn-s2tc is installed. > > Note: lp_test_format does need any change since we were already ignoring > S3TC failures there. Tested-by: Eric Anholt signature.asc Description: PGP signature

Re: [Mesa-dev] [PATCH] st/dri: Add fence extension to SW path

2017-05-22 Thread Chad Versace
When you resubmit the patch again, please add (v3) to the subject. On Mon 08 May 2017, Gurchetan Singh wrote: > Use the same fence implementation for drisw.c as dri2.c by making > dri2FenceExtension an external variable. Since the fence implementation > is not dri2.c specific, put it in a

Re: [Mesa-dev] [RFC] ARB_gl_spirv and NIR backend for radeonsi

2017-05-22 Thread Timothy Arceri
On 23/05/17 05:02, Marek Olšák wrote: On Sun, May 21, 2017 at 12:48 PM, Nicolai Hähnle wrote: Hi all, I've been looking into ARB_gl_spirv for radeonsi. I don't fancy re-inventing the ~8k LOC of src/compiler/spirv, and there's already a perfectly fine SPIR-V -> NIR -> LLVM

Re: [Mesa-dev] driconf: Add allow_glsl_builtin_variable_redeclaration option

2017-05-22 Thread Samuel Pitoiset
On 05/22/2017 11:49 PM, Dieter Nützel wrote: Hello John, hello Samuel, is 'allow_glsl_builtin_redeclaration=true' needed after this (commit #bf4d7671f423f5d4e451fba81bb9d9ae57f5fe5b) landed? Hi, Yes, because the option is only enabled with 2b878cb8fdb99dd84602553c2e95ef47747f4529.

Re: [Mesa-dev] driconf: Add allow_glsl_builtin_variable_redeclaration option

2017-05-22 Thread Dieter Nützel
Hello John, hello Samuel, is 'allow_glsl_builtin_redeclaration=true' needed after this (commit #bf4d7671f423f5d4e451fba81bb9d9ae57f5fe5b) landed? Thanks, Dieter ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org

Re: [Mesa-dev] [PATCH] i965: Add RGBX8888 and RGBA8888 to EGL configure and reorder the list

2017-05-22 Thread Rob Herring
On Mon, May 22, 2017 at 9:16 AM, Emil Velikov wrote: > On 20 May 2017 at 01:16, Rob Herring wrote: >> On Fri, May 19, 2017 at 12:57 PM, Emil Velikov >> wrote: >>> On 18 May 2017 at 23:01, Rob Herring wrote:

Re: [Mesa-dev] [PATCH] st/nine: Fix a regression and syntax cleanup

2017-05-22 Thread Dieter Nützel
FW: Got it a shot with radeonsi/RX580 under Wine-2.8-staging. Tried it with LS2015, LS2017, TS2017 and PES2015. Tested-by: Dieter Nützel Dieter Am 22.05.2017 00:22, schrieb Axel Davy: A few cleanups and in particular initializing properly the new pipe_draw_info fields.

Re: [Mesa-dev] [PATCH] mesa: Remove GL_APPLE_vertex_array_object stubs

2017-05-22 Thread Timothy Arceri
On 23/05/17 05:26, Ian Romanick wrote: From: Ian Romanick Mark the functions 'exec="skip"' in the XML instead. libGL will still have the functions, but the driver won't try to use them. I verified that this commit works with piglit's 'object-namespace-pollution

Re: [Mesa-dev] [PATCH 2/6] nir: Prehash in instr_set to avoid hashing twice

2017-05-22 Thread Thomas Helland
2017-05-22 20:55 GMT+02:00 Thomas Helland : > This should prove benefitial in the common case of inserting > and not rewriting anything. > > V2: Use hash_instr directly instead of going through the function > pointer embedded in the set Threw some sysprof profiles at

Re: [Mesa-dev] [RFC] ARB_gl_spirv and NIR backend for radeonsi

2017-05-22 Thread Jason Ekstrand
Taking the "just reply to Nicolai" approach like everyeone else... Nicolai, First off, thanks for working on this! I've given it a bit of thought in the context of i965, so I've got a few ideas about how you could overcome some of the issues. On Sun, May 21, 2017 at 3:48 AM, Nicolai Hähnle

Re: [Mesa-dev] [PATCH 1/3] etnaviv: Add support for extended texture formats

2017-05-22 Thread Christian Gmeiner
2017-05-22 19:51 GMT+02:00 Lucas Stach : > Am Dienstag, den 16.05.2017, 21:31 +0200 schrieb Christian Gmeiner: >> Hi Wladimir. >> >> I started working on this topic last week and thought some time on how >> to add those ext texture formats in a clean and nice way. I come up

Re: [Mesa-dev] [PATCH 1/6] glsl: Prehash in refcount hash table to reduce hashing

2017-05-22 Thread Thomas Helland
2017-05-22 21:44 GMT+02:00 Thomas Helland : > 2017-05-22 21:19 GMT+02:00 Ian Romanick : >> On 05/22/2017 11:55 AM, Thomas Helland wrote: >>> _mesa_hash_table_search is one of our hottest function according to perf. >>> Callgrind shows the

Re: [Mesa-dev] [RFC PATCH 02/65] mesa: implement ARB_bindless_texture

2017-05-22 Thread Nicolai Hähnle
On 19.05.2017 18:52, Samuel Pitoiset wrote: Signed-off-by: Samuel Pitoiset --- src/mesa/main/context.c | 3 + src/mesa/main/dd.h | 17 + src/mesa/main/mtypes.h | 34 ++ src/mesa/main/samplerobj.c | 6 + src/mesa/main/shared.c

Re: [Mesa-dev] [PATCH 0/9] glsl: cleanup and fix handling of unnamed struct types

2017-05-22 Thread Nicolai Hähnle
Hey Ian, Do you want more time to look into this? I double-checked that glslang accepts this code pattern. Cheers, Nicolai On 16.05.2017 14:29, Nicolai Hähnle wrote: On 16.05.2017 02:56, Ian Romanick wrote: On 05/15/2017 02:27 AM, Nicolai Hähnle wrote: Hi all, This series aims to

Re: [Mesa-dev] [RFC PATCH 01/65] mapi: add GL_ARB_bindless_texture entry points

2017-05-22 Thread Nicolai Hähnle
On 19.05.2017 18:52, Samuel Pitoiset wrote: Signed-off-by: Samuel Pitoiset --- src/mapi/glapi/gen/ARB_bindless_texture.xml | 100 src/mapi/glapi/gen/Makefile.am | 1 + src/mapi/glapi/gen/gl_API.xml | 4 +-

Re: [Mesa-dev] [PATCH 1/9] mesa: Handle common extension checks with more compact code

2017-05-22 Thread Nanley Chery
On Fri, May 19, 2017 at 04:29:16PM -0700, Ian Romanick wrote: > On 05/19/2017 10:28 AM, Nanley Chery wrote: > > On Fri, May 19, 2017 at 06:38:03AM -0700, Ian Romanick wrote: > >> From: Ian Romanick > >> > >> The previous code handled everything with the general case. I

Re: [Mesa-dev] [PATCH 04/16] anv: Handle transitioning depth from UNDEFINED to other layouts

2017-05-22 Thread Nanley Chery
On Fri, May 19, 2017 at 06:09:49PM -0700, Nanley Chery wrote: > On Fri, May 19, 2017 at 05:18:12PM -0700, Jason Ekstrand wrote: > > On Fri, May 19, 2017 at 4:51 PM, Nanley Chery wrote: > > > > > On Thu, May 18, 2017 at 02:00:51PM -0700, Jason Ekstrand wrote: > > > > --- >

Re: [Mesa-dev] [PATCH 1/6] glsl: Prehash in refcount hash table to reduce hashing

2017-05-22 Thread Thomas Helland
2017-05-22 21:19 GMT+02:00 Ian Romanick : > On 05/22/2017 11:55 AM, Thomas Helland wrote: >> _mesa_hash_table_search is one of our hottest function according to perf. >> Callgrind shows the refcounting as one of the major users of the >> searching functions. We can reduce the

[Mesa-dev] [PATCH] mesa: Remove GL_APPLE_vertex_array_object stubs

2017-05-22 Thread Ian Romanick
From: Ian Romanick Mark the functions 'exec="skip"' in the XML instead. libGL will still have the functions, but the driver won't try to use them. I verified that this commit works with piglit's 'object-namespace-pollution glClear vertex-array' on x64 with a driver

Re: [Mesa-dev] [PATCH 1/6] glsl: Prehash in refcount hash table to reduce hashing

2017-05-22 Thread Ian Romanick
On 05/22/2017 11:55 AM, Thomas Helland wrote: > _mesa_hash_table_search is one of our hottest function according to perf. > Callgrind shows the refcounting as one of the major users of the > searching functions. We can reduce the pain by prehashing, so that we > avoid hashing two times when

Re: [Mesa-dev] [RFC] ARB_gl_spirv and NIR backend for radeonsi

2017-05-22 Thread Ian Romanick
On 05/22/2017 12:02 PM, Marek Olšák wrote: > On Sun, May 21, 2017 at 12:48 PM, Nicolai Hähnle wrote: >> Hi all, >> >> I've been looking into ARB_gl_spirv for radeonsi. I don't fancy re-inventing >> the ~8k LOC of src/compiler/spirv, and there's already a perfectly fine >>

[Mesa-dev] [v2 4/7] i965/gen4: Simplify depth/stencil invalidate check

2017-05-22 Thread Topi Pohjolainen
There is no separate stencil on gen < 6. Signed-off-by: Topi Pohjolainen --- src/mesa/drivers/dri/i965/brw_misc_state.c | 16 +++- 1 file changed, 3 insertions(+), 13 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_misc_state.c

[Mesa-dev] [v2 5/7] i965: Drop depth/stencil miptree pointers in alignment workaround

2017-05-22 Thread Topi Pohjolainen
In brw_workaround_depthstencil_alignment() corresponding renderbuffers are always set to refer to the same temp miptrees. There is no need to carry them in context. Signed-off-by: Topi Pohjolainen --- src/mesa/drivers/dri/i965/brw_context.h| 3 ---

[Mesa-dev] [v2 7/7] i965/gen4: Tell briefly how workaround depth gets reconciled

2017-05-22 Thread Topi Pohjolainen
CC: Kenneth Graunke Signed-off-by: Topi Pohjolainen --- src/mesa/drivers/dri/i965/brw_misc_state.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_misc_state.c

[Mesa-dev] [v2 3/7] i965/gen4: Remove redundant check for depth when rebasing stencil

2017-05-22 Thread Topi Pohjolainen
In case of gen < 6 stencil (if present) is always combined with depth. Both stencil and depth attachments point to the same physical surface. Alignment workaround starts by considering depth and updates stencil accordingly. Current logic continues with stencil and in vain considers the case where

[Mesa-dev] [v2 1/7] i965/gen4: Set depth offset when there is stencil attachment only

2017-05-22 Thread Topi Pohjolainen
Current version fails to set depthstencil.depth_offset when there is only stencil attachment (it does set the intra tile offsets though). Fixes piglits: g45,g965,ilk: depthstencil-render-miplevels 1024 s=z24_s8 g45,ilk:depthstencil-render-miplevels 273 s=z24_s8 CC:

[Mesa-dev] [v2 2/7] i965/gen4: Remove non-existing stencil and hiz buffer setup

2017-05-22 Thread Topi Pohjolainen
Separate stencil and hiz are only enabled for gen6+. Signed-off-by: Topi Pohjolainen --- src/mesa/drivers/dri/i965/brw_context.h| 6 +- src/mesa/drivers/dri/i965/brw_misc_state.c | 120 +++-- 2 files changed, 14 insertions(+), 112

[Mesa-dev] [v2 6/7] i965/gen4: Refactor depth/stencil rebase

2017-05-22 Thread Topi Pohjolainen
Effectively there is the same code twice, once for depth and again for stencil. Signed-off-by: Topi Pohjolainen --- src/mesa/drivers/dri/i965/brw_misc_state.c | 243 - 1 file changed, 63 insertions(+), 180 deletions(-) diff --git

Re: [Mesa-dev] [RFC] ARB_gl_spirv and NIR backend for radeonsi

2017-05-22 Thread Marek Olšák
On Sun, May 21, 2017 at 12:48 PM, Nicolai Hähnle wrote: > Hi all, > > I've been looking into ARB_gl_spirv for radeonsi. I don't fancy re-inventing > the ~8k LOC of src/compiler/spirv, and there's already a perfectly fine > SPIR-V -> NIR -> LLVM compiler pipeline in radv, so I

Re: [Mesa-dev] [PATCH] st/nine: Fix a regression and syntax cleanup

2017-05-22 Thread Edmondo Tommasina
The crash with STALKER: CoC is fixed with this patch. Tested-by: Edmondo Tommasina Thanks edmondo On Mon, May 22, 2017 at 12:22 AM, Axel Davy wrote: > A few cleanups and in particular initializing properly > the new pipe_draw_info fields. > This

Re: [Mesa-dev] [PATCH] u_format_test: Ignore S3TC errors.

2017-05-22 Thread Rhys Kidd
On Mon, May 22, 2017 at 1:07 PM Jose Fonseca wrote: > This prevents spurious failures when libtxc-dxtn-s2tc is installed. > > Note: lp_test_format does need any change s/does/doesn't/ With that commit message fix: Reviewed-by: Rhys Kidd since we

[Mesa-dev] [PATCH 6/6] util: Avoid computing hash twice in string_to_uint_map

2017-05-22 Thread Thomas Helland
This is not a hot path, but when we have the functionality we should take advantage of it. --- src/util/string_to_uint_map.h | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/src/util/string_to_uint_map.h b/src/util/string_to_uint_map.h index e0533ec6ea..3248696e88

[Mesa-dev] [PATCH 2/6] nir: Prehash in instr_set to avoid hashing twice

2017-05-22 Thread Thomas Helland
This should prove benefitial in the common case of inserting and not rewriting anything. V2: Use hash_instr directly instead of going through the function pointer embedded in the set --- src/compiler/nir/nir_instr_set.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git

[Mesa-dev] [PATCH 3/6] glsl: Prehash in constant propagation

2017-05-22 Thread Thomas Helland
This should reduce overhead since we can hash only once. --- src/compiler/glsl/opt_constant_propagation.cpp | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/compiler/glsl/opt_constant_propagation.cpp b/src/compiler/glsl/opt_constant_propagation.cpp index

[Mesa-dev] [PATCH 0/6] Prehash all the things

2017-05-22 Thread Thomas Helland
While this doesn't prehash all the things, it does switch quite a lot of places from doing a search and then a subsequent insert to first hash the key, and then use this hash when searching / inserting. While our new pointer hashing function remedied much of our overhead hashing pointers, there is

[Mesa-dev] [PATCH 1/6] glsl: Prehash in refcount hash table to reduce hashing

2017-05-22 Thread Thomas Helland
_mesa_hash_table_search is one of our hottest function according to perf. Callgrind shows the refcounting as one of the major users of the searching functions. We can reduce the pain by prehashing, so that we avoid hashing two times when inserting in the table. On a short shader-db run (with

[Mesa-dev] [PATCH 4/6] glsl: Prehash in constant var pass to avoid hashing twice

2017-05-22 Thread Thomas Helland
Should help us reduce some hash table overhead. V2: Fix whitespace error --- src/compiler/glsl/opt_constant_variable.cpp | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/compiler/glsl/opt_constant_variable.cpp b/src/compiler/glsl/opt_constant_variable.cpp index

[Mesa-dev] [PATCH 5/6] glsl: Prehash to avoid computing the hash twice

2017-05-22 Thread Thomas Helland
Copy propagation elements is a large user of the hash table. We should not be computing the hash twice when we don't have to. --- src/compiler/glsl/opt_copy_propagation_elements.cpp | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git

Re: [Mesa-dev] [PATCH] u_format_test: Ignore S3TC errors.

2017-05-22 Thread Roland Scheidegger
Reviewed-by: Roland Scheidegger Thanks! Am 22.05.2017 um 19:06 schrieb Jose Fonseca: > This prevents spurious failures when libtxc-dxtn-s2tc is installed. > > Note: lp_test_format does need any change since we were already ignoring > S3TC failures there. > --- >

Re: [Mesa-dev] Mesa 17.1.1 release candidate

2017-05-22 Thread Chuck Atkins
> > The candidate for the Mesa 17.1.1 is now available. Excellent! >From build perspective - SWR now ships it's final generated header, thus > Python/mako is no longer required. > Just what I was looking for, thanks! Is a source tarball available that I can test the build with? I would just

Re: [Mesa-dev] [PATCH 06/24] i965/cnl: Add gen10 specific function declarations

2017-05-22 Thread Anuj Phogat
On Mon, May 22, 2017 at 10:07 AM, Rafael Antognolli wrote: > On Fri, May 12, 2017 at 04:38:10PM -0700, Anuj Phogat wrote: >> These declarations will help the code start compiling >> once we wire up the makefiles for gen10. Later patches >> will start using these

Re: [Mesa-dev] [PATCH 00/13] egl: gbm: Misc cleanups and robustness patches

2017-05-22 Thread Rob Herring
On Thu, May 11, 2017 at 1:57 PM, Emil Velikov wrote: > Hi all, > > I was around the EGL/GBM codebase and I've noticed that we do a few > things, rather badly: > - we have unnessesary layer of abstraction in GBM like the following > gbm_dri_foo -> gbm_drm_foo -> gbm_foo

Re: [Mesa-dev] How does FakeGLX work since Mesa 10?

2017-05-22 Thread Brian Paul
On 05/09/2017 12:07 PM, Tom Hudson wrote: I'm trying to upgrade a Mesa 10 installation to Mesa 17. There have been plenty of changes, but only one breakage that's left me perplexed. https://cgit.freedesktop.org/mesa/mesa/commit/?id=c00b250c8061d042d9905e61b9077462ee91008b In this CL, Mesa

Re: [Mesa-dev] [PATCH 11/13] egl: refactor dri2_create_screen() into three separate functions

2017-05-22 Thread Rob Herring
On Thu, May 11, 2017 at 1:57 PM, Emil Velikov wrote: > Split the create_screen into: > - create screen > - setup/bind extensions > - setup screen > > This will allow us to reuse the latter two on egl/drm. Said platform > does create its own screen and attempts to

Re: [Mesa-dev] [PATCH 0/9][RFC] GLSL preprocessor/parser improvements

2017-05-22 Thread Thomas Helland
2017-05-22 3:41 GMT+02:00 Timothy Arceri : > I suspect you forgot to test with a debug build, there is issues with the > string buffer changes somewhere. > > ralloc.c:203: reralloc_size: Assertion `ralloc_parent(ptr) == ctx' failed. > > #0 0x770eb91f in raise ()

[Mesa-dev] [Bug 100988] glXGetCurrentDisplay() no longer works for FakeGLX contexts?

2017-05-22 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=100988 Brian Paul changed: What|Removed |Added Assignee|mesa-dev@lists.freedesktop. |bri...@vmware.com

Re: [Mesa-dev] [PATCH 1/3] etnaviv: Add support for extended texture formats

2017-05-22 Thread Lucas Stach
Am Dienstag, den 16.05.2017, 21:31 +0200 schrieb Christian Gmeiner: > Hi Wladimir. > > I started working on this topic last week and thought some time on how > to add those ext texture formats in a clean and nice way. I come up > with this patches: > >

Re: [Mesa-dev] [PATCH v3 01/15] st/dri: refactor multi-planar YUV import path

2017-05-22 Thread Lucas Stach
Am Mittwoch, den 10.05.2017, 23:15 +0530 schrieb Varad Gautam: > From: Varad Gautam > > we currently ignore the plane count when converting from > __DRI_IMAGE_FORMAT* tokens to __DRI_IMAGE_FOURCC* for multiplanar > images, and only return the first plane's simplified

Re: [Mesa-dev] [PATCH 2/7] egl/x11: check for dri2_dpy->flush before using the flush extension

2017-05-22 Thread Emil Velikov
On 16 May 2017 at 18:32, Chad Versace wrote: > On Mon 15 May 2017, Emil Velikov wrote: >> From: Emil Velikov >> >> Analogous to previous commit. >> >> Signed-off-by: Emil Velikov >> --- > >> If people prefer I can

[Mesa-dev] [PATCH 2/7] egl/x11: check for dri2_dpy->flush before using the flush extension

2017-05-22 Thread Emil Velikov
From: Emil Velikov Analogous to earlier commit. Note that the dri2_x11_post_sub_buffer and dri2_x11_swap_buffers_region paths already implicitly require __DRI2_FLUSH. The corresponding extensions (NV_post_sub_buffer and NOK_swap_region) are enabled only with DRI2.

[Mesa-dev] [PATCH 1.5/7] egl/drm: flatten codeflow

2017-05-22 Thread Emil Velikov
From: Emil Velikov Rework the code to return early and drop an indentation level. No functional change, yet it should be easier to read. Signed-off-by: Emil Velikov --- src/egl/drivers/dri2/platform_drm.c | 39

[Mesa-dev] [PATCH 3.5/7] egl/x11: flatten codeflow

2017-05-22 Thread Emil Velikov
From: Emil Velikov Signed-off-by: Emil Velikov --- src/egl/drivers/dri2/platform_x11.c | 16 +++- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/src/egl/drivers/dri2/platform_x11.c

[Mesa-dev] [PATCH 1/7] egl/drm: check for dri2_dpy->flush before using the flush extension

2017-05-22 Thread Emil Velikov
From: Emil Velikov The current __DRI_DRI2 imples __DRI2_FLUSH. At the same time, one can use __DRI_IMAGE_DRIVER alongside the latter, so the current check is confusing at best. Check for what we use. v2: Split out from whitespace changes Reviewed-by: Chad Versace

Re: [Mesa-dev] [PATCH v2] drirc: Add allow_glsl_builtin_variable_redeclaration for Dead Island Riptide Definitive Edition

2017-05-22 Thread Marek Olšák
Pushed, thanks. Marek On Sun, May 21, 2017 at 11:46 PM, Benedikt Schemmer wrote: > From: Benedikt Schemmer > > This patch sets the allow_glsl_builtin_variable_redeclaration for Dead > Island Riptide Definitive Edition. > > Tested with Mesa git as of today and Dying

Re: [Mesa-dev] [PATCH 2/9] util: Add tests for the string buffer

2017-05-22 Thread Ian Romanick
New tests should use gtest. Other tests can't be made to produce XML output that can be processed by Jenkins. On 05/21/2017 01:49 PM, Thomas Helland wrote: > More tests could probably be added, but this should cover > concatenation, resizing, clearing, formatted printing, > and checking the

Re: [Mesa-dev] [PATCH] mesa: add APPLE_vertex_array_object stubs

2017-05-22 Thread Ian Romanick
WTF? I gave review feedback on IRC, and you said you were going to send a different patch... yet this patch landed. If you're not going to take review feedback, why do you ask for it? On 05/18/2017 07:50 PM, Timothy Arceri wrote: > APPLE_vertex_array_object support was removed in 7927d0378fc7.

Re: [Mesa-dev] [PATCH 06/24] i965/cnl: Add gen10 specific function declarations

2017-05-22 Thread Rafael Antognolli
On Fri, May 12, 2017 at 04:38:10PM -0700, Anuj Phogat wrote: > These declarations will help the code start compiling > once we wire up the makefiles for gen10. Later patches > will start using these functions for gen10. > > Signed-off-by: Anuj Phogat > --- >

Re: [Mesa-dev] [RFC PATCH 29/65] tgsi: add new Bindless flag to tgsi_instruction_texture

2017-05-22 Thread Marek Olšák
On Fri, May 19, 2017 at 6:52 PM, Samuel Pitoiset wrote: > Old-style images are identified using TGSI_FILE_SAMPLER, but > bindless samplers can be TGSI_FILE_CONSTANT or TGSI_FILE_TEMPORARY. > > To avoid backend compilers to be confused, this adds a new flag > that will

[Mesa-dev] [PATCH] u_format_test: Ignore S3TC errors.

2017-05-22 Thread Jose Fonseca
This prevents spurious failures when libtxc-dxtn-s2tc is installed. Note: lp_test_format does need any change since we were already ignoring S3TC failures there. --- src/gallium/tests/unit/u_format_test.c | 25 + 1 file changed, 25 insertions(+) diff --git

Re: [Mesa-dev] [PATCH] travis: Disable scons tests, since the gallium unit tests fail.

2017-05-22 Thread Jose Fonseca
On 22/05/17 17:29, Eric Anholt wrote: Jose Fonseca writes: I suppose the failures happen with S2TC lib-dxtn implementation. Because I haven't seen them failing with the old implementation, or when no implementation is present (as the tests will just skip.) We could craft

Re: [Mesa-dev] [RFC PATCH 28/65] tc: add ARB_bindless_texture support

2017-05-22 Thread Marek Olšák
On Fri, May 19, 2017 at 6:52 PM, Samuel Pitoiset wrote: > Signed-off-by: Samuel Pitoiset > --- > src/gallium/auxiliary/util/u_threaded_context.c| 147 > + > .../auxiliary/util/u_threaded_context_calls.h | 4 +

Re: [Mesa-dev] [PATCH V2 15/24] i965/cnl: Start using CNL MOCS defines

2017-05-22 Thread Rafael Antognolli
On Tue, May 16, 2017 at 04:01:30PM -0700, Anuj Phogat wrote: > On Tue, May 16, 2017 at 10:34 AM, Anuj Phogat wrote: > > CNL MOCS defines are duplicates of SKL MOCS defines. > > > I can actually drop this patch and continue using SKL MOCS defines for gen10+. > I also noticed

Re: [Mesa-dev] [PATCH] vc4: Remove dead code in vc4_dump_surface_msaa()

2017-05-22 Thread Eric Anholt
Rhys Kidd writes: > Coverity caught the use of dead code copy-paste for > found_colors[] and num_found_colors. Reviewed and pushed. Thanks! signature.asc Description: PGP signature ___ mesa-dev mailing list

Re: [Mesa-dev] [PATCH] glsl: Fix g++ initializer order warning

2017-05-22 Thread Matt Turner
Reviewed-by: Matt Turner ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH V2 00/24] Add Cannonlake support

2017-05-22 Thread Anuj Phogat
On Fri, May 12, 2017 at 4:38 PM, Anuj Phogat wrote: > This series adds support for Cannonlake. > > Changes from V1 to V2: > - Incorporated the review comments from V1. > - Rebased 8 months old CNL branch on top of master > - Wired up Linux and Android build files for gen10

[Mesa-dev] [PATCH] Revert "gallium: remove unused PIPE_CC_GCC_VERSION"

2017-05-22 Thread Brian Paul
This reverts commit e60928f4c4bd4484821d83f2b16a910ea9f5f9d9. PIPE_CC_GCC_VERSION is used by some of our in-house code which hasn't been upstreamed yet. --- src/gallium/include/pipe/p_config.h | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/include/pipe/p_config.h

Re: [Mesa-dev] [PATCH] travis: Disable scons tests, since the gallium unit tests fail.

2017-05-22 Thread Eric Anholt
Jose Fonseca writes: > I suppose the failures happen with S2TC lib-dxtn implementation. > Because I haven't seen them failing with the old implementation, or when > no implementation is present (as the tests will just skip.) > > We could craft the test cases so they aren't

Re: [Mesa-dev] [PATCH] travis: Disable scons tests, since the gallium unit tests fail.

2017-05-22 Thread Eric Anholt
Rhys Kidd writes: > On 21 May 2017 at 19:33, Roland Scheidegger wrote: > >> I suppose the s3tc problems should go away rather sooner than later, but >> isn't it possible to just disable the formats tests? >> > > Yes, see USE_TXC_DXTN that was introduced

Re: [Mesa-dev] [PATCH] docs: Document ASTC extension support for SKL and BXT

2017-05-22 Thread Anuj Phogat
On Fri, May 19, 2017 at 4:42 PM, Nanley Chery wrote: > Cc: Anuj Phogat > Signed-off-by: Nanley Chery > --- > docs/features.txt | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/docs/features.txt

Re: [Mesa-dev] [PATCH] r600: fix compilation error without amdgpu

2017-05-22 Thread Emil Velikov
This patch won't be sufficient since you'll error out at link time, when building w/o radeonsi. There are three possible routes [1]. Let me know which one you prefer :-) -Emil [1] https://lists.freedesktop.org/archives/mesa-dev/2017-May/156429.html ___

Re: [Mesa-dev] [PATCH 2/2] radeon: pass flags that can change shaders to disk_cache_create()

2017-05-22 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Sat, May 20, 2017 at 3:36 AM, Timothy Arceri wrote: > I wasn't sure if I should filter the flags so that we only use > flags that actually change the shader output. To avoid manual > updates we just pass in

Re: [Mesa-dev] i965/gen4: Depth buffer setup cleanup

2017-05-22 Thread Pohjolainen, Topi
On Sun, May 21, 2017 at 06:15:46PM +0300, Pohjolainen, Topi wrote: > On Sat, May 20, 2017 at 10:29:51PM +0300, Topi Pohjolainen wrote: > > First this series removes logic considering separate stencil > > gen < 6 only code paths (where we always use combined > > depth-stencil). > > > > Then patch

Re: [Mesa-dev] [PATCH] r600: fix compilation error without amdgpu

2017-05-22 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Mon, May 22, 2017 at 11:37 AM, Marc Dietrich wrote: > This fixes the following compilation error when amdgpu is not selected. > > CC common/common_libamd_common_la-ac_gpu_info.lo > In file included from

Re: [Mesa-dev] [PATCH] st/nine: Fix a regression and syntax cleanup

2017-05-22 Thread Marek Olšák
I'm not so familiar with this code, but it looks good to me. Marek On Mon, May 22, 2017 at 12:22 AM, Axel Davy wrote: > A few cleanups and in particular initializing properly > the new pipe_draw_info fields. > This should fix the regression caused by >

[Mesa-dev] [PATCH v2 13/13] gbm: manage only the required set of DRI extensions

2017-05-22 Thread Emil Velikov
From: Emil Velikov Currently GBM attempts to know all the extensions that might be required by EGL/DRM [at some later stage]. That is a bit unclear and we often forget to update GBM as EGL gets attention. To avoid that, simply let EGL manage it's own required

[Mesa-dev] [PATCH v2 12/13] egl/drm: use dri2_setup_extensions() over the extensions provided by GBM

2017-05-22 Thread Emil Velikov
From: Emil Velikov Allows us to keep things in sync easier and lets us simplify the interface between the two even further. v2: Don't set GBM's extensions. Cc: Rob Herring Signed-off-by: Emil Velikov ---

Re: [Mesa-dev] Please dont remove support for working hardware

2017-05-22 Thread Jason Ekstrand
No one is suggesting we completely drop support for old hardware. The drivers will still exist and work; they will just live in a legacy branch. We will also ensure that they are installable alongside the modern drivers and the expectation is that distros will continue to package them. The thing

Re: [Mesa-dev] [PATCH v2] mesa: Avoid leaking surface in st_renderbuffer_delete

2017-05-22 Thread Emil Velikov
On 29 April 2017 at 15:37, Bartosz Tomczyk wrote: > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=100741 > Fixes: a5e733c6b52 mesa: drop current draw/read buffer when ctx is released > CC: Rob Clark > v2: add comment in code

Re: [Mesa-dev] [PATCH] i965: Add RGBX8888 and RGBA8888 to EGL configure and reorder the list

2017-05-22 Thread Emil Velikov
On 20 May 2017 at 01:16, Rob Herring wrote: > On Fri, May 19, 2017 at 12:57 PM, Emil Velikov > wrote: >> On 18 May 2017 at 23:01, Rob Herring wrote: >>> On Thu, May 18, 2017 at 5:25 AM, Emil Velikov >>>

Re: [Mesa-dev] [PATCH v13 02/36] egl/main: add support for fourth plane tokens

2017-05-22 Thread Emil Velikov
On 22 May 2017 at 13:47, Daniel Stone wrote: > Hey, > > On 22 May 2017 at 13:43, Emil Velikov wrote: >> On 19 May 2017 at 10:37, Daniel Stone wrote: >>> The EGL_EXT_dma_buf_import_modifiers extension adds support for a >>>

Re: [Mesa-dev] [PATCH v13 13/36] st/dri: support format queries

2017-05-22 Thread Brian Paul
On 05/19/2017 08:48 AM, Lucas Stach wrote: Am Freitag, den 19.05.2017, 10:37 +0100 schrieb Daniel Stone: From: Varad Gautam ask the driver for supported dmabuf formats v2: rebase to master. v3: return false on failure. v4: use pscreen->is_format_supported instead

Re: [Mesa-dev] [PATCH v13 11/36] st/dri: implement createImageWithModifiers in DRIimage

2017-05-22 Thread Emil Velikov
On 19 May 2017 at 10:37, Daniel Stone wrote: > From: Varad Gautam > > adds a pscreen->resource_create_with_modifiers() to create textures > with modifier. > > Signed-off-by: Varad Gautam > Signed-off-by: Daniel Stone

  1   2   >