On Sat, Apr 30, 2016 at 7:31 PM, Ilia Mirkin wrote:
> Spotted by Coverity
>
> Signed-off-by: Ilia Mirkin
> ---
> src/gallium/drivers/softpipe/sp_compute.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/softpipe/sp_compute.c
> b/src/gallium/driver
On Sat, Apr 30, 2016 at 7:29 PM, Ilia Mirkin wrote:
> Spotted by Coverity
>
> Signed-off-by: Ilia Mirkin
> ---
> src/mesa/vbo/vbo_exec_array.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/src/mesa/vbo/vbo_exec_array.c b/src/mesa/vbo/vbo_exec_array.c
> index 51f41f3
Spotted by Coverity
Signed-off-by: Ilia Mirkin
---
src/gallium/drivers/softpipe/sp_compute.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/softpipe/sp_compute.c
b/src/gallium/drivers/softpipe/sp_compute.c
index d5b5913..53f7a0b 100644
--- a/src/galli
Spotted by Coverity
Signed-off-by: Ilia Mirkin
---
src/mesa/vbo/vbo_exec_array.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/mesa/vbo/vbo_exec_array.c b/src/mesa/vbo/vbo_exec_array.c
index 51f41f3..870a096 100644
--- a/src/mesa/vbo/vbo_exec_array.c
+++ b/src/mesa/v
https://bugs.freedesktop.org/show_bug.cgi?id=95215
Jamey Sharp changed:
What|Removed |Added
Status|NEW |NEEDINFO
--- Comment #5 from Jamey Sharp
https://bugs.freedesktop.org/show_bug.cgi?id=95215
farmboy0+freedesk...@googlemail.com changed:
What|Removed |Added
CC||farmboy0+freedesktop
https://bugs.freedesktop.org/show_bug.cgi?id=95215
--- Comment #4 from Jamey Sharp ---
(In reply to Ilia Mirkin from comment #3)
> (In reply to Jamey Sharp from comment #2)
> > (In reply to Ilia Mirkin from comment #1)
> > > Sounds like the game is buggy and needs fixing - I presume it runs fine
https://bugs.freedesktop.org/show_bug.cgi?id=95215
--- Comment #3 from Ilia Mirkin ---
(In reply to Jamey Sharp from comment #2)
> (In reply to Ilia Mirkin from comment #1)
> > Sounds like the game is buggy and needs fixing - I presume it runs fine on
> > Catalyst drivers, which also don't suppor
https://bugs.freedesktop.org/show_bug.cgi?id=95215
--- Comment #2 from Jamey Sharp ---
(In reply to Ilia Mirkin from comment #1)
> Sounds like the game is buggy and needs fixing - I presume it runs fine on
> Catalyst drivers, which also don't support the ext.
Hmm. Is apitrace mis-leading us here
Hi,
I'm an update about external/llvm status
android-n-preview-2 switched to llvm 3.8.0, the following branch has AMDGPU
patches,
even if named after marshamallow, changes were made on top llvm 3.8.0,
after aosp switched with android-n-preview2:
https://github.com/maurossi/llvm/tree/marshmallow-
On Sat, Apr 30, 2016 at 3:58 PM, Marek Olšák wrote:
> The script is licensed under GNU GPL v2, which can be a showstopper for Mesa.
this has come up a couple times already, but we already have files in
$mesa/m4 which are GPL.. and this script is not even involved in the
build (let alone installed
https://bugs.freedesktop.org/show_bug.cgi?id=95215
Vladimir Usikov changed:
What|Removed |Added
CC||granti...@gmail.com
--
You are receiv
The script is licensed under GNU GPL v2, which can be a showstopper for Mesa.
Marek
On Sat, Apr 30, 2016 at 9:27 PM, Rob Clark wrote:
> From: Rob Clark
>
> Copied from linux kernel, with minimal changes to script (just to
> recognize mesa src tree rather than linux kernel src tree), and slimmed
From: Rob Clark
Copied from linux kernel, with minimal changes to script (just to
recognize mesa src tree rather than linux kernel src tree), and slimmed
down MAINTAINER file syntax to recognize that we don't really have
subsystem "maintainers" in the same sense as the linux kernel (ie. no
differ
OpenGL 4.5 Core Profile section 7.1, in the documentation for
CompileShader, says: "Changing the source code of a shader object with
ShaderSource does not change its compile status or the compiled shader
code." (I haven't checked older versions of the spec.)
This test creates a shader, compiles it
On Fri, Apr 29, 2016 at 9:59 PM, Thomas H.P. Andersen
wrote:
>
>
> On Fri, Apr 29, 2016 at 8:39 PM, Jason Ekstrand
> wrote:
>
>>
>>
>> On Fri, Apr 29, 2016 at 11:31 AM, Thomas Hindoe Paaboel Andersen <
>> pho...@gmail.com> wrote:
>>
>>> Parenthesis are needed here as ! takes precedence over the
https://bugs.freedesktop.org/show_bug.cgi?id=95215
Kamil Páral changed:
What|Removed |Added
CC||kamil.pa...@gmail.com
--
You are receivin
https://bugs.freedesktop.org/show_bug.cgi?id=95085
--- Comment #17 from Marek Olšák ---
FYI, the DX10_CLAMP shader bit flushes NaNs to 0. All other floating-point
shader opcodes flush denorms to 0. 32-bit denorms can be enabled with a
significant performance cost by setting conf->float_mode (se
On Wednesday, April 13, 2016 3:46:29 PM PDT Matt Turner wrote:
> On Wed, Apr 13, 2016 at 12:07 PM, Matt Turner wrote:
> > ---
>
> i965 is going to stop using this code, but I thought it'd be nice to
> fix this and also to note what is the remaining failure on the dEQP
> ldexp tests.
>
> The GLSL
https://bugs.freedesktop.org/show_bug.cgi?id=93551
--- Comment #12 from Jamey Sharp ---
(In reply to Ernst Sjöstrand from comment #11)
> If I apply the "change shader" fix with RadeonSI I get a bunch of these
> glNamedStringARB errors when running the game with APITrace:
But you can confirm that
https://bugs.freedesktop.org/show_bug.cgi?id=95215
--- Comment #1 from Ilia Mirkin ---
Sounds like the game is buggy and needs fixing - I presume it runs fine on
Catalyst drivers, which also don't support the ext.
--
You are receiving this mail because:
You are the assignee for the bug.
You are
https://bugs.freedesktop.org/show_bug.cgi?id=95215
Jamey Sharp changed:
What|Removed |Added
CC||ern...@gmail.com,
|
https://bugs.freedesktop.org/show_bug.cgi?id=95215
Bug ID: 95215
Summary: ARB_shading_language_include is not implemented
Product: Mesa
Version: git
Hardware: All
OS: All
Status: NEW
Severity: enhancement
On Sat, Apr 30, 2016 at 12:35 PM, Samuel Pitoiset
wrote:
> After the MemoryOpt pass, load and store instructions might have been
> combined and we have to take this in account when detecting if the
> destination register of a texture instruction is used by another
> instruction as srcs/defs.
>
> T
After the MemoryOpt pass, load and store instructions might have been
combined and we have to take this in account when detecting if the
destination register of a texture instruction is used by another
instruction as srcs/defs.
This fixes 11 dEQP-GLES31 subtests:
dEQP-GLES3.functional.shaders.opaq
On 2016-04-30 10:34, Kenneth Graunke wrote:
With my horiz_offset concerns and other minor comments addressed,
Patches 1-33 (except 2, 3, 11, and 22) and 36, 40-55, 57-58 are:
Reviewed-by: Kenneth Graunke
I'll look at the rest soon.
It sounds like Curro is taking a look too, so he may have mor
Reviewed-by: Edward O'Callaghan
On 2016-04-30 01:48, Fabio Pedretti wrote:
---
docs/GL3.txt | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/docs/GL3.txt b/docs/GL3.txt
index bb2bb6e..5a6be41 100644
--- a/docs/GL3.txt
+++ b/docs/GL3.txt
@@ -148,17 +148,17 @@ GL
On Sat, Apr 30, 2016 at 8:26 AM, Marek Olšák wrote:
> On Sat, Apr 30, 2016 at 1:55 PM, Rob Clark wrote:
>> On Sat, Apr 30, 2016 at 5:39 AM, Michel Dänzer wrote:
>>> On 26.04.2016 03:51, Rob Herring wrote:
On Mon, Apr 25, 2016 at 9:25 AM, Emil Velikov
wrote:
> On 25 April 2016 at
On Sat, Apr 30, 2016 at 1:55 PM, Rob Clark wrote:
> On Sat, Apr 30, 2016 at 5:39 AM, Michel Dänzer wrote:
>> On 26.04.2016 03:51, Rob Herring wrote:
>>> On Mon, Apr 25, 2016 at 9:25 AM, Emil Velikov
>>> wrote:
On 25 April 2016 at 13:46, Daniel Stone wrote:
> On 23 April 2016 at 03:08,
https://bugs.freedesktop.org/show_bug.cgi?id=93551
--- Comment #11 from Ernst Sjöstrand ---
If I apply the "change shader" fix with RadeonSI I get a bunch of these
glNamedStringARB errors when running the game with APITrace:
2833: message: major api error 1: GL_INVALID_OPERATION in unsupported f
On Sat, Apr 30, 2016 at 5:39 AM, Michel Dänzer wrote:
> On 26.04.2016 03:51, Rob Herring wrote:
>> On Mon, Apr 25, 2016 at 9:25 AM, Emil Velikov
>> wrote:
>>> On 25 April 2016 at 13:46, Daniel Stone wrote:
On 23 April 2016 at 03:08, Rob Herring wrote:
> On Fri, Apr 22, 2016 at 6:32 PM
On 04/30/2016 01:24 AM, Nicolai Hähnle wrote:
From: Nicolai Hähnle
Funnily enough, some of these were turned into a compile-time error by gcc
with -fsanitize=undefined ("initializer is not a constant").
---
src/gallium/auxiliary/tgsi/tgsi_ureg.c | 2 +-
src/gallium/auxiliary/util/u_p
On 28.04.2016 10:54, Michel Dänzer wrote:
> On 23.04.2016 07:24, Marek Olšák wrote:
>> On Fri, Apr 22, 2016 at 11:28 PM, Nicolai Hähnle wrote:
>>> On 22.04.2016 12:29, Nicolai Hähnle wrote:
On 20.04.2016 23:02, Michel Dänzer wrote:
> On 21.04.2016 02:42, Marek Olšák wrote:
>> On Thu,
On 26.04.2016 03:51, Rob Herring wrote:
> On Mon, Apr 25, 2016 at 9:25 AM, Emil Velikov
> wrote:
>> On 25 April 2016 at 13:46, Daniel Stone wrote:
>>> On 23 April 2016 at 03:08, Rob Herring wrote:
On Fri, Apr 22, 2016 at 6:32 PM, Emil Velikov
wrote:
> Can we take a look at the G
On Fri, Apr 29, 2016 at 3:33 PM, Timothy Arceri
wrote:
> +/* Create a directory named 'path' if it does not already exist.
> + *
> + * Returns: 0 if path already exists as a directory or if created.
> + * -1 in all other cases.
> + */
> +static int
> +mkdir_if_needed(char *path)
> +{
> +
For 1-2, 5-9:
Reviewed-by: Marek Olšák
Marek
On Sat, Apr 30, 2016 at 9:24 AM, Nicolai Hähnle wrote:
> Hi,
>
> this patch series enables the use of gcc's -fsanitize=undefined in Mesa and
> fixes the bulk of reported issues in basic examples like glxgears. However,
> many issues remain, and some
Reviewed-by: Marek Olšák
Marek
On Fri, Apr 29, 2016 at 11:21 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> That format has first_non_void < 0. This fixes a regression in piglit
> arb_shader_image_load_store-semantics that was introduced by commit
> 76b8c5cc602,
> while hopefully still
With my horiz_offset concerns and other minor comments addressed,
Patches 1-33 (except 2, 3, 11, and 22) and 36, 40-55, 57-58 are:
Reviewed-by: Kenneth Graunke
I'll look at the rest soon.
It sounds like Curro is taking a look too, so he may have more feedback.
signature.asc
Description: This
On Friday, April 29, 2016 1:29:46 PM PDT Samuel Iglesias Gonsálvez wrote:
> From: Iago Toral Quiroga
>
> ---
> src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 98 +
+--
> 1 file changed, 81 insertions(+), 17 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_fs_n
Samuel Iglesias Gonsálvez writes:
> Hello,
>
> This patch series continues adding arb_gpu_shader_fp64 support to the
> Intel driver. Specifically, this targets the i965 scalar backend for
> BDW+ hardware (vec4 is still under research and gen7 has its own
> issues which we intend tackle after gen
On Friday, April 29, 2016 1:29:26 PM PDT Samuel Iglesias Gonsálvez wrote:
> From: Connor Abbott
>
> ---
> src/mesa/drivers/dri/i965/Makefile.sources | 1 +
> src/mesa/drivers/dri/i965/brw_fs.cpp| 5 +++
> src/mesa/drivers/dri/i965/brw_fs.h | 1 +
> src/mesa/drive
On 30.04.2016 16:24, Nicolai Hähnle wrote:
> Hi,
>
> this patch series enables the use of gcc's -fsanitize=undefined in Mesa and
> fixes the bulk of reported issues in basic examples like glxgears. However,
> many issues remain, and some of them in the GLSL compiler look like they could
> be real
From: Nicolai Hähnle
---
src/mesa/main/uniforms.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/main/uniforms.c b/src/mesa/main/uniforms.c
index a9308d0..7758640 100644
--- a/src/mesa/main/uniforms.c
+++ b/src/mesa/main/uniforms.c
@@ -78,7 +78,7 @@ _mesa_update_sha
From: Nicolai Hähnle
One of these is an unsigned bitfield, which I suspect is a false positive, but
gcc 5.3.1 complains about it with -fsanitize=undefined.
---
src/gallium/drivers/radeonsi/si_state.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/gallium/drivers/ra
From: Nicolai Hähnle
This macro avoids undefined behaviour that crashes gcc's ubsan.
---
src/compiler/glsl/list.h | 13 +
src/compiler/glsl/opt_dead_code_local.cpp | 7 +--
src/compiler/glsl/opt_tree_grafting.cpp | 5 +
3 files changed, 15 insertions(+),
From: Nicolai Hähnle
This crashes gcc's undefined behaviour sanitizer.
---
src/compiler/glsl/ir.cpp | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/compiler/glsl/ir.cpp b/src/compiler/glsl/ir.cpp
index 750f617..d69ab13 100644
--- a/src/compiler/glsl/ir.cpp
+++ b/src/c
From: Nicolai Hähnle
Since 1 << 31 complains about undefined behaviour; the others are changed
only for consistency.
---
src/mesa/main/mtypes.h | 60 +-
1 file changed, 30 insertions(+), 30 deletions(-)
diff --git a/src/mesa/main/mtypes.h b/src/me
From: Nicolai Hähnle
---
src/gallium/drivers/radeonsi/si_state_shaders.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeonsi/si_state_shaders.c
b/src/gallium/drivers/radeonsi/si_state_shaders.c
index 52b9851..0bfd7e8 100644
--- a/src/gallium/drive
On Friday, April 29, 2016 1:29:20 PM PDT Samuel Iglesias Gonsálvez wrote:
> From: Connor Abbott
>
> v2 (Iago):
> - Squashed bits from 'support double precission constant operands for
> the implementation of 64-bit emit_load_const'.
> - Do not use BRW_REGISTER_TYPE_D for all 32-bit registe
From: Nicolai Hähnle
The old iteration casts sentinel nodes (which are mere exec_nodes) into
whatever type we're looping over, which leads to badness (in fact, gcc's
undefined behaviour sanitizer crashes while trying to verify that we have
the correct type at hand).
These modified looping constr
From: Nicolai Hähnle
Funnily enough, some of these were turned into a compile-time error by gcc
with -fsanitize=undefined ("initializer is not a constant").
---
src/gallium/auxiliary/tgsi/tgsi_ureg.c | 2 +-
src/gallium/auxiliary/util/u_pack_color.h | 2 +-
src/gallium/auxiliary/uti
Hi,
this patch series enables the use of gcc's -fsanitize=undefined in Mesa and
fixes the bulk of reported issues in basic examples like glxgears. However,
many issues remain, and some of them in the GLSL compiler look like they could
be real - see below for a sampling.
By "enables the use" I mea
On Friday, April 29, 2016 1:29:19 PM PDT Samuel Iglesias Gonsálvez wrote:
> From: Connor Abbott
>
> ---
> src/mesa/drivers/dri/i965/brw_fs.cpp | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/
i965/brw_fs.cpp
> index 78f7d40
On Friday, April 29, 2016 1:29:14 PM PDT Samuel Iglesias Gonsálvez wrote:
> From: Connor Abbott
>
> ---
> src/mesa/drivers/dri/i965/brw_shader.cpp | 28
> 1 file changed, 24 insertions(+), 4 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_shader.cpp b/sr
On Friday, April 29, 2016 1:29:08 PM PDT Samuel Iglesias Gonsálvez wrote:
> From: Connor Abbott
>
> ---
> src/mesa/drivers/dri/i965/brw_eu_emit.c | 6 --
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c b/src/mesa/drivers/dri/
i965/b
55 matches
Mail list logo
