Jason Ekstrand writes:
> On October 27, 2017 18:21:45 Matt Turner wrote:
>
>> ... as can happen with various types like mat4, or else we'll smash the
>> stack writing past the end of components_local[].
>
> Ugh... Ideally, I think you'd want without_matrix() because a matrix is
> just an array
On October 27, 2017 18:21:45 Matt Turner wrote:
... as can happen with various types like mat4, or else we'll smash the
stack writing past the end of components_local[].
Ugh... Ideally, I think you'd want without_matrix() because a matrix is
just an array as far as interfaces go. That said,
... as can happen with various types like mat4, or else we'll smash the
stack writing past the end of components_local[].
Fixes: 5a0d3e1129b7 ("nir: Print the components referenced for split or
packed shader in/outs.")
---
Reproduce with INTEL_DEBUG=vs and
generated_tests/spe
I've submitted an alternative to this approach here [1], more along the
lines of the v1 patch you borrowed from my jenkins branch. Most of the
reasons for that we have discussed already in the office (except for the
last point), but FTR:
- This patch only works around the problem by tweaking the
On Fri, Oct 27, 2017 at 3:16 PM, Nanley Chery wrote:
> On Fri, Oct 27, 2017 at 12:52:30PM -0700, Jason Ekstrand wrote:
> > On Fri, Oct 27, 2017 at 12:24 PM, Nanley Chery
> > wrote:
> >
> > > Only use CCS_E to render to a texture that is CCS_E-compatible with the
> > > original texture's miptree
This makes the dataflow propagation logic of the copy propagation pass
more intelligent in cases where the destination of a copy is known to
be undefined for some incoming CFG edges, building upon the
definedness information provided by the last patch. Helps a few
programs, and avoids a handful sh
Currently the liveness analysis pass would extend a live interval up
to the top of the program when no unconditional and complete
definition of the variable is found that dominates all of its uses.
This can lead to a serious performance problem in shaders containing
many partial writes, like scala
This addresses a long-standing back-end compiler bug that could lead
to cross-channel data corruption in loops executed non-uniformly. In
some cases live variables extending through a loop divergence point
(e.g. a non-uniform break) into a convergence point (e.g. the end of
the loop) wouldn't be c
On 24 October 2017 at 17:14, Emil Velikov wrote:
> From: Emil Velikov
>
> The function is effectively a direct function call into
> libwayland-server.so.
>
> Thus GBM no longer depends on the wayland-drm static library, making the
> build more straight forward. And the resulting binary is a bit s
Vedran Miletić writes:
> v2: remove/inline compat stuff
>
> Reviewed-by: Francisco Jerez
> ---
> .../state_trackers/clover/llvm/codegen/native.cpp | 8 +-
> src/gallium/state_trackers/clover/llvm/compat.hpp | 109
> +
> .../state_trackers/clover/llvm/invocation.cpp
On 23/10/17 05:33, Nicolai Hähnle wrote:
From: Nicolai Hähnle
---
src/util/Makefile.sources | 1 +
src/util/futex.h | 51 +++
src/util/meson.build | 1 +
src/util/simple_mtx.h | 20 +--
4 files changed, 54 i
On Fri, Oct 27, 2017 at 4:00 PM, Kenneth Graunke wrote:
> On Friday, October 27, 2017 3:17:20 PM PDT Anuj Phogat wrote:
>> V2: Use _NEW_BUFFERS in place of _NEW_MULTISAMPLE
>>
>> Signed-off-by: Anuj Phogat
>> Cc: Kenneth Graunke
>> ---
>> src/mesa/drivers/dri/i965/genX_state_upload.c | 13 +
On Friday, October 27, 2017 3:17:20 PM PDT Anuj Phogat wrote:
> V2: Use _NEW_BUFFERS in place of _NEW_MULTISAMPLE
>
> Signed-off-by: Anuj Phogat
> Cc: Kenneth Graunke
> ---
> src/mesa/drivers/dri/i965/genX_state_upload.c | 13 -
> 1 file changed, 12 insertions(+), 1 deletion(-)
>
>
Looks alright too.
Reviewed-by: Roland Scheidegger
Am 28.10.2017 um 00:35 schrieb Brian Paul:
> Fixes: ea53d9a8eb5d4b2 "glapi: include generated headers without path"
> ---
> src/mesa/drivers/osmesa/SConscript | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/src/mesa/drivers/osmesa/SCo
Ah, I see that you fixed that already,
Reviewed-by: Dylan Baker
Quoting Brian Paul (2017-10-27 15:35:09)
> Fixes: ea53d9a8eb5d4b2 "glapi: include generated headers without path"
> ---
> src/mesa/drivers/osmesa/SConscript | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/src/mesa/drivers/
I sent almost the same patch at exactly the same time. I also needed to add this
path to src/map/glapi/Sconscript to get scons building. Maybe that's unrelated?
Dylan
Quoting Brian Paul (2017-10-27 15:35:09)
> Fixes: ea53d9a8eb5d4b2 "glapi: include generated headers without path"
> ---
> src/mes
Fixes: ea53d9a8eb5d4b2 "glapi: include generated headers without path"
---
src/mesa/drivers/osmesa/SConscript | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/drivers/osmesa/SConscript
b/src/mesa/drivers/osmesa/SConscript
index 7280314..9391dc3 100644
--- a/src/mesa/drivers/osmesa/SCo
Build mesa 5970 completed
Commit 05592cebd4 by Brian Paul on 10/27/2017 9:14 PM:
scons: fix scons build to find generated glapitable.h\n\nFixes: ea53d9a8eb5d4b2 "glapi: include generated headers without path"\n\nReviewed-by: Roland Scheidegger
Config
fixes: 5daed06da2c0aaa4 ("osmesa: Include generated headers without path")
cc: Mark Janes
Signed-off-by: Dylan Baker
---
src/mapi/glapi/SConscript | 1 +
src/mesa/drivers/osmesa/SConscript | 4 +++-
2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/mapi/glapi/SConscript
V2: Use _NEW_BUFFERS in place of _NEW_MULTISAMPLE.
Fix indentation.
Signed-off-by: Anuj Phogat
Cc: Kenneth Graunke
Cc: Matt Turner
---
src/mesa/drivers/dri/i965/genX_state_upload.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/genX_state_upload.c
On Thu, Oct 26, 2017 at 11:14 PM, Jordan Justen
wrote:
> Signed-off-by: Jordan Justen
> Reviewed-by: Jason Ekstrand
> Cc: Jason Ekstrand
> Cc: Kenneth Graunke
> ---
>
> * Add comment (Ken)
> * No typedef (Jason)
>
> src/intel/compiler/brw_compiler.h | 20
> 1 file chan
On Fri, Oct 27, 2017 at 12:50:45PM -0700, Jason Ekstrand wrote:
> Thanks for doing this in isl_format_supports_ccs_e instead of changing the
> table!
>
> Reviewed-by: Jason Ekstrand
>
Thank you for the review!
> On Fri, Oct 27, 2017 at 12:24 PM, Nanley Chery
> wrote:
>
> > CannonLake additio
V2: Use _NEW_BUFFERS in place of _NEW_MULTISAMPLE
Signed-off-by: Anuj Phogat
Cc: Kenneth Graunke
---
src/mesa/drivers/dri/i965/genX_state_upload.c | 13 -
1 file changed, 12 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/genX_state_upload.c
b/src/mesa/drivers/
On Fri, Oct 27, 2017 at 12:52:30PM -0700, Jason Ekstrand wrote:
> On Fri, Oct 27, 2017 at 12:24 PM, Nanley Chery
> wrote:
>
> > Only use CCS_E to render to a texture that is CCS_E-compatible with the
> > original texture's miptree (linear) format. This prevents render
> > operations from writing
For the series:
Reviewed-by: Marek Olšák
Marek
On Sun, Oct 22, 2017 at 8:39 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> Some locking is unfortunately required, because well-formed GL programs
> can have multiple threads racing to access the same texture, e.g.: two
> threads/contexts
On Fri, Oct 27, 2017 at 2:50 PM, Matt Turner wrote:
> On Fri, Oct 27, 2017 at 10:50 AM, Anuj Phogat wrote:
>> Signed-off-by: Anuj Phogat
>> ---
>> src/mesa/drivers/dri/i965/genX_state_upload.c | 10 ++
>> 1 file changed, 10 insertions(+)
>>
>> diff --git a/src/mesa/drivers/dri/i965/genX
On Fri, Oct 27, 2017 at 2:44 PM, Kenneth Graunke wrote:
> On Friday, October 27, 2017 10:50:23 AM PDT Anuj Phogat wrote:
>> Signed-off-by: Anuj Phogat
>> ---
>> src/mesa/drivers/dri/i965/genX_state_upload.c | 10 ++
>> 1 file changed, 10 insertions(+)
>>
>> diff --git a/src/mesa/drivers/
Am 27.10.2017 um 23:16 schrieb Brian Paul:
> Fixes: ea53d9a8eb5d4b2 "glapi: include generated headers without path"
> ---
> src/mapi/glapi/SConscript | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/src/mapi/glapi/SConscript b/src/mapi/glapi/SConscript
> index 994778a..08fed6c 100644
> --
On Fri, Oct 27, 2017 at 10:50 AM, Anuj Phogat wrote:
> Signed-off-by: Anuj Phogat
> ---
> src/mesa/drivers/dri/i965/genX_state_upload.c | 10 ++
> 1 file changed, 10 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/i965/genX_state_upload.c
> b/src/mesa/drivers/dri/i965/genX_state_upl
On Friday, October 27, 2017 10:50:23 AM PDT Anuj Phogat wrote:
> Signed-off-by: Anuj Phogat
> ---
> src/mesa/drivers/dri/i965/genX_state_upload.c | 10 ++
> 1 file changed, 10 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/i965/genX_state_upload.c
> b/src/mesa/drivers/dri/i965/genX
Reviewed-by: Roland Scheidegger
Am 27.10.2017 um 23:16 schrieb Brian Paul:
> In the vbuf_render::set_primitive() functions.
> ---
> src/gallium/auxiliary/draw/draw_vbuf.h | 3 ++-
> src/gallium/drivers/i915/i915_prim_vbuf.c| 2 +-
> src/gallium/drivers/llvmpipe/lp_setup_vbuf.c | 2 +-
>
On 10/24/2017 02:42 AM, Muthukumar, Aravindan wrote:
>> -Original Message-
>> From: Ian Romanick [mailto:i...@freedesktop.org]
>> Sent: Friday, October 20, 2017 9:51 AM
>> To: Muthukumar, Aravindan ; mesa-
>> d...@lists.freedesktop.org
>> Cc: J Karanje, Kedar ; Tamminen, Eero T
>>
>> Subje
Fixes: ea53d9a8eb5d4b2 "glapi: include generated headers without path"
---
src/mapi/glapi/SConscript | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mapi/glapi/SConscript b/src/mapi/glapi/SConscript
index 994778a..08fed6c 100644
--- a/src/mapi/glapi/SConscript
+++ b/src/mapi/glapi/SConscri
In the vbuf_render::set_primitive() functions.
---
src/gallium/auxiliary/draw/draw_vbuf.h | 3 ++-
src/gallium/drivers/i915/i915_prim_vbuf.c| 2 +-
src/gallium/drivers/llvmpipe/lp_setup_vbuf.c | 2 +-
src/gallium/drivers/nouveau/nv30/nv30_draw.c | 2 +-
src/gallium/drivers/r300/r300_rend
Having moved gallium_dri.so library to /vendor/lib/dri
also symlinks need to be coherently created using TARGET_OUT_VENDOR instead of
TARGET_OUT
or all non Intel drivers will not be loaded with Android N and earlier,
thus causing SurfaceFlinger SIGABRT
(v2) simplification of post install command
On Fri, Oct 27, 2017 at 12:24 PM, Nanley Chery
wrote:
> Only use CCS_E to render to a texture that is CCS_E-compatible with the
> original texture's miptree (linear) format. This prevents render
> operations from writing data that can't be decoded with the original
> miptree format.
>
> On Gen10,
Reviewed-by: Jordan Justen
On 2017-10-27 09:44:14, Lionel Landwerlin wrote:
> Signed-off-by: Lionel Landwerlin
> ---
> src/intel/common/gen_decoder.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/intel/common/gen_decoder.c b/src/intel/common/gen_decoder.c
> index
Thanks for doing this in isl_format_supports_ccs_e instead of changing the
table!
Reviewed-by: Jason Ekstrand
On Fri, Oct 27, 2017 at 12:24 PM, Nanley Chery
wrote:
> CannonLake additionally supports R11G11B10_FLOAT and four 10-10-10-2
> formats with CCS_E. None of these formats fit within the
On 2017-10-25 21:17:13, Kenneth Graunke wrote:
> Groups containing fields smaller than a DWord were not being decoded
> correctly. For example:
>
>
>
>
>
> gen_field_iterator_next would properly walk over each element of the
> array, incrementing group_iter, and calling iter_gro
On Fri, Oct 27, 2017 at 3:03 AM, Iago Toral wrote:
> This should be squashed into the previous commit
>
Oops... Fixed.
> On Wed, 2017-10-25 at 16:26 -0700, Jason Ekstrand wrote:
> > With the advent of SPIR-V subgroup operations, compute shaders will
> > have
> > to be slightly different depend
On Fri, Oct 27, 2017 at 4:47 AM, Iago Toral wrote:
> On Wed, 2017-10-25 at 16:26 -0700, Jason Ekstrand wrote:
> > The automatic exec size inference can accidentally mess things up if
> > we're not careful. For instance, if we have
> >
> > add(4)g38.2<4>Dg38.1<8,2,4>Dg38.2<8,2,4>D
> >
On Fri, Oct 27, 2017 at 12:35 AM, Iago Toral wrote:
> This sounds good to me, but I guess it is not really fixing anything,
> right? I ask because the subject claims that this patch does something
> that the original code was already supposed to be doing.
>
This patch is a bit of an artifact of
On Fri, Oct 27, 2017 at 2:11 AM, Iago Toral wrote:
> On Wed, 2017-10-25 at 16:26 -0700, Jason Ekstrand wrote:
> > Previously, brw_nir_lower_intrinsics added the param and then emitted
> > a
> > load_uniform intrinsic to load it directly. This commit switches
> > things
> > over to use a specific
On Fri, Oct 27, 2017 at 1:18 AM, Iago Toral wrote:
> On Wed, 2017-10-25 at 16:25 -0700, Jason Ekstrand wrote:
> > The only things that adjust fs_visitor::max_dispatch_width are render
> > target writes which don't happen in compute shaders so they're
> > pointless.
> > ---
> > src/intel/compiler
From: Eduardo Lima Mitev
Some symbols gathered in the symbols table during parsing are needed
later for the compile and link stages, so they are moved along the
process. Currently, only functions and non-temporary variables are
copied between symbol tables. However, the built-in gl_PerVertex
inte
From: Eduardo Lima Mitev
>From GLSL 4.5 spec, section "7.1 Built-In Language Variables", page 130 of
the PDF states:
"If multiple shaders using members of a built-in block belonging to
the same interface are linked together in the same program, they must
all redeclare the built-in
From: Eduardo Lima Mitev
This effectively factorizes a couple of similar routines.
v2 (Neil Roberts): Non-trivial rebase on master
Signed-off-by: Eduardo Lima Mitev
Signed-off-by: Neil Roberts
---
src/compiler/glsl/glsl_parser_extras.cpp | 25 +++--
src/compiler/glsl/link
Hi,
Here is a v2 of Eduardo’s patches to fix the PerVertexValidation CTS
test. They were originally posted here:
https://lists.freedesktop.org/archives/mesa-dev/2017-March/146667.html
The patches needed a non-trivial rebase. The previous discussion was
left with a note saying that there are more
On Fri, Oct 27, 2017 at 12:09 AM, Iago Toral wrote:
>
> On Wed, 2017-10-25 at 16:25 -0700, Jason Ekstrand wrote:
> > ---
> > src/intel/compiler/brw_fs.cpp | 10 --
> > 1 file changed, 8 insertions(+), 2 deletions(-)
> >
> > diff --git a/src/intel/compiler/brw_fs.cpp
> > b/src/intel/compi
CannonLake additionally supports R11G11B10_FLOAT and four 10-10-10-2
formats with CCS_E. None of these formats fit within the current
blorp_copy framework so disable them until support is added.
Signed-off-by: Nanley Chery
Cc: Jason Ekstrand
---
Jason, do you think we modify blorp instead of mo
Only use CCS_E to render to a texture that is CCS_E-compatible with the
original texture's miptree (linear) format. This prevents render
operations from writing data that can't be decoded with the original
miptree format.
On Gen10, with the new CCS_E-enabled formats handled, this enables the
drive
On Thu, Oct 26, 2017 at 11:53 PM, Iago Toral wrote:
> On Wed, 2017-10-25 at 16:25 -0700, Jason Ekstrand wrote:
> > ---
> > src/intel/compiler/brw_fs_nir.cpp | 33 +--
> > --
> > 1 file changed, 21 insertions(+), 12 deletions(-)
> >
> > diff --git a/src/intel/compiler/
On Thu, Oct 26, 2017 at 11:35 PM, Iago Toral wrote:
> On Wed, 2017-10-25 at 16:25 -0700, Jason Ekstrand wrote:
> > Stop retyping the output of shuffle_64bit_data_for_32bit_write. It's
> > always BRW_REGISTER_TYPE_D which is perfectly fine for writing out.
> > Also, when we change get_nir_src to
Well, I went to push this, apparently this was in one of the osmesa
patches as an unrelated change.. :/
So it's fixed.
Dylan
Quoting Dylan Baker (2017-10-27 10:25:33)
> This is needed in cases where xcb isn't actually needed as a dependency,
> but may still be included somewhere.
>
> cc: Er
2017-10-27 14:28 GMT+02:00 Rob Herring :
> On Fri, Oct 27, 2017 at 6:41 AM, Emil Velikov
> wrote:
> > On 26 October 2017 at 23:48, Mauro Rossi wrote:
> >> Having moved gallium_dri.so library to /vendor/lib/dri
> >> also symlinks need to be coherently created using TARGET_OUT_VENDOR
> insted of T
On Fri, Oct 27, 2017 at 7:25 PM, Dylan Baker wrote:
> This is needed in cases where xcb isn't actually needed as a dependency,
> but may still be included somewhere.
>
> cc: Erik Faye-Lund
> cc: Eric Engestrom
> Signed-off-by: Dylan Baker
Tested-by: Erik Faye-Lund
Build mesa 5967 failed
Commit f121a669c7 by Dylan Baker on 10/24/2017 10:52 PM:
meson: build gallium based osmesa\n\nThis has been tested with the osdemo from mesa-demos\n\nv2: - Add SELinux dependency\n- fix typo GALLIUM_LLVM -> GALLIUM_LLVMPIPE\n\nSigned-
Fixes compiling the optional standalone glsl compiler.
Reported-by: DrNick (on irc)
Signed-off-by: Dylan Baker
---
This is not compiled by default, but can be built by:
meson build
ninja -C build src/compiler/glsl/glsl_compiler
src/compiler/glsl/meson.build | 2 +-
1 file changed, 1 insertion(
On Fri, Oct 27, 2017 at 1:43 PM, Andres Gomez wrote:
> Rejected (6)
>
>
> Ilia Mirkin (1):
> glsl: fix derived cs variables
>
> Reason: Commit is too big for stable at this point.
The issue it fixes in regular compute shaders is slightly difficult to
hit (but there are piglits
Quoting Eric Engestrom (2017-10-27 02:28:05)
> On Thursday, 2017-10-26 13:55:35 -0700, Dylan Baker wrote:
> > Quoting Eric Engestrom (2017-10-26 02:40:20)
> > > On Wednesday, 2017-10-25 15:58:23 -0700, Dylan Baker wrote:
> > > > +if with_shared_glapi
> > > > + osmesa_link_with += libglapi
> > > >
On Fri, Oct 27, 2017 at 10:50 AM, Anuj Phogat wrote:
> Signed-off-by: Anuj Phogat
> ---
> src/mesa/drivers/dri/i965/genX_state_upload.c | 10 ++
> 1 file changed, 10 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/i965/genX_state_upload.c
> b/src/mesa/drivers/dri/i965/genX_state_upl
On Friday, 2017-10-27 10:25:33 -0700, Dylan Baker wrote:
> This is needed in cases where xcb isn't actually needed as a dependency,
> but may still be included somewhere.
>
> cc: Erik Faye-Lund
> cc: Eric Engestrom
better indeed :)
Reviewed-by: Eric Engestrom
> Signed-off-by: Dylan Baker
> -
Signed-off-by: Anuj Phogat
---
src/mesa/drivers/dri/i965/genX_state_upload.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/genX_state_upload.c
b/src/mesa/drivers/dri/i965/genX_state_upload.c
index b6e800aa90..2fb6420cee 100644
--- a/src/mesa/drivers/dr
Signed-off-by: Anuj Phogat
---
src/mesa/drivers/dri/i965/genX_state_upload.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/genX_state_upload.c
b/src/mesa/drivers/dri/i965/genX_state_upload.c
index 4ccfd48919..b6e800aa90 100644
--- a/src/mesa/drivers/dr
On Fri, Oct 27, 2017 at 5:03 PM, Andres Gomez wrote:
> Bas, this patch looks like it should have been marked as fixing
> 6ce550453f1 instead of e38685cc62e (?).
Well my reasoning was that the bug got "visible" when we enabled Vega.
The patch you refer to fixed part of it, but not all of it, and t
Hello list,
The candidate for the Mesa 17.2.4 is now available. Currently we have:
- 21 queued
- 10 nominated (outstanding)
- and 6 rejected patches
In the current queue we have:
In Mesa Core we have included a change to prevent KOTOR from breaking
when in combination with the ATI fragment s
Reviewed-by: Mark Janes
Dylan Baker writes:
> This fixes a race condition in the build.
>
> cc: Rob Clark
> Signed-off-by: Dylan Baker
> ---
> src/gallium/drivers/freedreno/meson.build | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/freedreno/meso
Whooo! Thanks for doing this!
Quoting Eric Engestrom (2017-10-27 07:40:17)
> Cc: Rob Herring
> Cc: Tomasz Figa
> Signed-off-by: Eric Engestrom
> ---
> Completely untested!
> It's a step in the right direction though; doesn't hurt non-android,
> and gets android closer to building on meson :)
>
Yeah, I kinda got that feeling. Your approach seems better.
On Oct 27, 2017 19:22, "Dylan Baker" wrote:
This just papers over the actual problem, that dep_xcb isn't declared as an
empty list with the other glx dependencies.
Dylan
Quoting Erik Faye-Lund (2017-10-27 04:55:25)
> This avoids the f
This is needed in cases where xcb isn't actually needed as a dependency,
but may still be included somewhere.
cc: Erik Faye-Lund
cc: Eric Engestrom
Signed-off-by: Dylan Baker
---
meson.build | 1 +
1 file changed, 1 insertion(+)
diff --git a/meson.build b/meson.build
index 875f9d4d294..102b75
On Thursday, October 26, 2017 11:16:06 AM PDT Eric Anholt wrote:
> Gwan-gyeong Mun writes:
>
> > Hi,
> >
> > These Patch v5 series modified with new helper function series [1].
> >
> > These series only have mesa for tizen feature.
> >
> > [1] https://patchwork.freedesktop.org/series/32577/
>
>
This just papers over the actual problem, that dep_xcb isn't declared as an
empty list with the other glx dependencies.
Dylan
Quoting Erik Faye-Lund (2017-10-27 04:55:25)
> This avoids the following build-error when building with emtpy
> vulkan-drivers and without glx=dri:
>
> Meson encountered
This fixes a race condition in the build.
cc: Rob Clark
Signed-off-by: Dylan Baker
---
src/gallium/drivers/freedreno/meson.build | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/freedreno/meson.build
b/src/gallium/drivers/freedreno/meson.build
index 028be
On Friday, October 27, 2017 9:43:45 AM PDT Lionel Landwerlin wrote:
> It seems nobody's using the string hashing function. If you try to
> pass it directly to the hashtable creation function, you'll get
> compiler warning for non matching prototypes. Let's make them match.
>
> Signed-off-by: Lione
On 10/27/2017 06:49 PM, Marek Olšák wrote:
The hw disables depth writes automatically.
Okay, good to know. Thanks!
Marek
On Fri, Oct 27, 2017 at 6:27 PM, Samuel Pitoiset
wrote:
Found by inspection.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_pipeline.c | 8 +++-
1 fi
The hw disables depth writes automatically.
Marek
On Fri, Oct 27, 2017 at 6:27 PM, Samuel Pitoiset
wrote:
> Found by inspection.
>
> Signed-off-by: Samuel Pitoiset
> ---
> src/amd/vulkan/radv_pipeline.c | 8 +++-
> 1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/src/amd/vul
It seems nobody's using the string hashing function. If you try to
pass it directly to the hashtable creation function, you'll get
compiler warning for non matching prototypes. Let's make them match.
Signed-off-by: Lionel Landwerlin
---
src/util/hash_table.c | 3 ++-
src/util/hash_table.h | 2 +-
Signed-off-by: Lionel Landwerlin
---
src/intel/common/gen_decoder.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/intel/common/gen_decoder.c b/src/intel/common/gen_decoder.c
index 395ff02908a..55e7305117c 100644
--- a/src/intel/common/gen_decoder.c
+++ b/src/intel/common
For patches 1-4, 6, 7, assuming you fix the double unlock in patch 7:
Reviewed-by: Marek Olšák
Marek
On Sun, Oct 22, 2017 at 8:33 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> ---
> src/util/Makefile.sources | 1 +
> src/util/futex.h | 51
> +
Found by inspection.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_pipeline.c | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_pipeline.c
index c25642c966..f5ebcda883 100644
--- a/src/amd/vulkan/radv_pipeline
On 10/27/2017 05:21 AM, Emil Velikov wrote:
On 27 October 2017 at 00:57, Dylan Baker wrote:
Which allows the code to be compiled with c99 instead of gnu99.
A little history. This code is guarded by #ifdef __GNUC__, so it's only
compiled with autotools on *nix, SCons with MSVC wont hit that cod
On Friday, 2017-10-27 13:55:25 +0200, Erik Faye-Lund wrote:
> This avoids the following build-error when building with emtpy
> vulkan-drivers and without glx=dri:
>
> Meson encountered an error in file src/vulkan/wsi/meson.build, line 30,
> column 2:
> Unknown variable "dep_xcb".
>
> Signed-off-b
On Fri, Oct 27, 2017 at 11:07 AM, Juan A. Suarez Romero
wrote:
> On Fri, 2017-10-27 at 10:27 -0400, Ilia Mirkin wrote:
>> On Fri, Oct 27, 2017 at 10:03 AM, Juan A. Suarez Romero
>> wrote:
>> > This patch is mostly a patch done by Ilia Mirkin.
>> >
>> > It fixes KHR-GL45.enhanced_layouts.varying_s
On Fri, 2017-10-27 at 10:27 -0400, Ilia Mirkin wrote:
> On Fri, Oct 27, 2017 at 10:03 AM, Juan A. Suarez Romero
> wrote:
> > This patch is mostly a patch done by Ilia Mirkin.
> >
> > It fixes KHR-GL45.enhanced_layouts.varying_structure_locations.
> >
> > CC: Ilia Mirkin
> > Bugzilla: https://bu
Bas, this patch looks like it should have been marked as fixing
6ce550453f1 instead of e38685cc62e (?).
In any case, I was wondering whether it would be interesting to bring
them both to the 17.2 stable queue and whether we would also want
Timothy's preceding patch:
087e010b2b3dd83a539f97203909d6
Cc: Rob Herring
Cc: Tomasz Figa
Signed-off-by: Eric Engestrom
---
Completely untested!
It's a step in the right direction though; doesn't hurt non-android,
and gets android closer to building on meson :)
---
meson.build | 13 +++--
src/egl/meson.build | 5 -
2 files changed
On Thursday, 2017-10-26 10:10:19 -0700, Dylan Baker wrote:
> I have a few tiny nits, but otherwise this seems fine:
> Reviewed-by: Dylan Baker
>
> Quoting Eric Engestrom (2017-10-26 08:40:56)
> > Suggested-by: Jordan Justen
> > Signed-off-by: Eric Engestrom
> > ---
> > bin/git_sha1_gen.py
On 27 October 2017 at 14:53, Lionel Landwerlin
wrote:
> On 27/10/17 14:46, Emil Velikov wrote:
>>
>> On 27 October 2017 at 13:55, Tapani Pälli wrote:
>>>
>>> Patch uses mem_ctx for allocation to ensure param array gets freed
>>> later, in blorp clear case this happens in blorp_params_get_clear_ke
On Fri, Oct 27, 2017 at 10:03 AM, Juan A. Suarez Romero
wrote:
> This patch is mostly a patch done by Ilia Mirkin.
>
> It fixes KHR-GL45.enhanced_layouts.varying_structure_locations.
>
> CC: Ilia Mirkin
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=103098
> Signed-off-by: Juan A. Suare
This patch is mostly a patch done by Ilia Mirkin.
It fixes KHR-GL45.enhanced_layouts.varying_structure_locations.
CC: Ilia Mirkin
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=103098
Signed-off-by: Juan A. Suarez Romero
---
src/compiler/glsl/linker.cpp | 36 +++
On 27/10/17 14:46, Emil Velikov wrote:
On 27 October 2017 at 13:55, Tapani Pälli wrote:
Patch uses mem_ctx for allocation to ensure param array gets freed
later, in blorp clear case this happens in blorp_params_get_clear_kernel.
==6164== 48 bytes in 1 blocks are definitely lost in loss record
On 27 October 2017 at 13:55, Tapani Pälli wrote:
> Patch uses mem_ctx for allocation to ensure param array gets freed
> later, in blorp clear case this happens in blorp_params_get_clear_kernel.
>
> ==6164== 48 bytes in 1 blocks are definitely lost in loss record 61 of 193
> ==6164==at 0x4C2EB6
This allows an app to query shader statistics and get a disassembly of
a shader. RenderDoc git has support for it, so this allows you to view
shader disassembly from a capture.
When this extension is enabled on a device (or when tracing), we now
disable pipeline caching, since we don't get the sha
Reviewed-by: Lionel Landwerlin
On 27/10/17 13:55, Tapani Pälli wrote:
Patch uses mem_ctx for allocation to ensure param array gets freed
later, in blorp clear case this happens in blorp_params_get_clear_kernel.
==6164== 48 bytes in 1 blocks are definitely lost in loss record 61 of 193
==6164==
Patch uses mem_ctx for allocation to ensure param array gets freed
later, in blorp clear case this happens in blorp_params_get_clear_kernel.
==6164== 48 bytes in 1 blocks are definitely lost in loss record 61 of 193
==6164==at 0x4C2EB6B: malloc (vg_replace_malloc.c:299)
==6164==by 0x12E31C
On Fri, Oct 27, 2017 at 5:18 AM, Alejandro Piñeiro wrote:
> From the spec:
>"IMAGE_FORMAT_COMPATIBILITY_TYPE: The matching criteria use for the
> resource when used as an image textures is returned in
> . This is equivalent to calling GetTexParameter"
>
> So we would need to return Non
On 26 October 2017 at 20:18, Rob Herring wrote:
> system/window.h is no longer available by default and is part of
> libnativewindow, so add it to the shared libraries. It has to be conditional
> because the library is only present in O and later.
>
> Really, we should only be depending on vndk/wi
Reviewed-by: Marek Olšák
Marek
On Fri, Oct 27, 2017 at 11:18 AM, Alejandro Piñeiro
wrote:
> From the spec:
>"IMAGE_FORMAT_COMPATIBILITY_TYPE: The matching criteria use for the
> resource when used as an image textures is returned in
> . This is equivalent to calling GetTexParameter"
On Fri, Oct 27, 2017 at 6:41 AM, Emil Velikov wrote:
> On 26 October 2017 at 23:48, Mauro Rossi wrote:
>> Having moved gallium_dri.so library to /vendor/lib/dri
>> also symlinks need to be coherently created using TARGET_OUT_VENDOR insted
>> of TARGET_OUT
>> or all non Intel drivers will not be
On Wed, 2017-10-25 at 16:26 -0700, Jason Ekstrand wrote:
> We have had a feature in codegen for some time that tries to
> automatically infer the execution size of an instruction from the
> width
> of its destination. For things such as fixed function GS, clipper,
> and
> SF programs, this is very
1 - 100 of 130 matches
Mail list logo