https://bugs.freedesktop.org/show_bug.cgi?id=90264
--- Comment #61 from James Jones ---
I'm on paternity leave from January 25th - April 15th.
While I'm out, please contact the following people:
linux-b...@nvidia.com for Linux graphics bug triage issues
Damien Leone
https://bugs.freedesktop.org/show_bug.cgi?id=90264
--- Comment #60 from Furkan ---
Has this been resolved for anybody? I'm running Xorg 1.18.2 and Mesa 11.2rc3. I
still get the same corruption, with both DRI2 and DRI3.
--
You are receiving this mail because:
You are the
On Wed, Mar 23, 2016 at 5:22 PM, Rob Herring wrote:
> On Fri, Mar 4, 2016 at 12:07 PM, Rob Clark wrote:
>> On Fri, Mar 4, 2016 at 12:59 PM, Rob Clark wrote:
>>> So, I've been advocating that for android, gallium drivers use
>>>
https://bugs.freedesktop.org/show_bug.cgi?id=94711
Bug ID: 94711
Summary: Patch for pb_reference to explicitly handle NULL src
Product: Mesa
Version: 11.1
Hardware: Other
OS: All
Status: NEW
Severity:
https://bugs.freedesktop.org/show_bug.cgi?id=94710
Bug ID: 94710
Summary: Patch for dangling disp->DriverData pointer in error
path
Product: Mesa
Version: 11.1
Hardware: Other
OS: All
Status:
Quoting Dylan Baker (2016-03-25 16:01:00)
> Quoting Adam Jackson (2016-03-24 11:10:01)
> > Signed-off-by: Adam Jackson
> > ---
> > src/mapi/glapi/gen/gl_XML.py | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/src/mapi/glapi/gen/gl_XML.py
Both are
Reviewed-by: Jason Ekstrand
On Fri, Mar 25, 2016 at 4:38 PM, Nanley Chery wrote:
> From: Nanley Chery
>
> This function differs from the open-coded implementation in that the
> ImageView's width is determined by
From: Nanley Chery
This is required to create multiple, horizontally adjacent, max-width
images from one blit2d surface. This is also required for more accurate
width specification of surfaces within a larger surface (which is seen
as the smaller surface's enclosing
From: Nanley Chery
This function differs from the open-coded implementation in that the
ImageView's width is determined by the caller and is not unconditionally
set to match the number of texels within the surface's pitch.
Signed-off-by: Nanley Chery
Previously, the pass assumed that the entrypoint would be whatever function
happened to have the name "main". We really shouldn't trust in the
function names.
---
src/compiler/nir/glsl_to_nir.cpp| 2 +-
src/compiler/nir/nir.h | 4 +++-
---
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 32 ++
src/mesa/drivers/dri/i965/brw_vec4_nir.cpp | 32 ++
2 files changed, 64 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
---
src/compiler/nir/nir_builder.h | 16
1 file changed, 16 insertions(+)
diff --git a/src/compiler/nir/nir_builder.h b/src/compiler/nir/nir_builder.h
index 4df79f5..e9c409b 100644
--- a/src/compiler/nir/nir_builder.h
+++ b/src/compiler/nir/nir_builder.h
@@ -399,6 +399,22 @@
Our hardware requires an LOD for all texelFetch commands even if they are
on buffer textures. GLSL IR gives us an LOD of 0 in that case, but the LOD
is really rather meaningless. This commit allows other NIR producers to be
more lazy and not provide one at all.
---
The old version of the pass only worked on globals and locals and always
left inputs, outputs, uniforms, etc. alone.
---
src/compiler/nir/nir.h | 2 +-
src/compiler/nir/nir_remove_dead_variables.c | 33
This correlates directly to the SPIR-V opcode OpQuantizeToF16
---
src/compiler/nir/nir_opcodes.py | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/compiler/nir/nir_opcodes.py b/src/compiler/nir/nir_opcodes.py
index 7c8a322..d6b658d 100644
--- a/src/compiler/nir/nir_opcodes.py
+++
---
src/compiler/nir/nir.h | 11 +++
1 file changed, 11 insertions(+)
diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h
index 45f208a..03c1f76 100644
--- a/src/compiler/nir/nir.h
+++ b/src/compiler/nir/nir.h
@@ -1936,6 +1936,17 @@ typedef struct {
};
} nir_cursor;
---
src/compiler/nir/nir.h | 3 ++
src/compiler/nir/nir_lower_system_values.c | 74 --
2 files changed, 74 insertions(+), 3 deletions(-)
diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h
index 907dc34..d46858e 100644
---
These are all needed for SPIR-V
---
src/compiler/nir/nir_opcodes.py | 16 +++-
src/compiler/nir/nir_opt_algebraic.py | 1 +
2 files changed, 16 insertions(+), 1 deletion(-)
diff --git a/src/compiler/nir/nir_opcodes.py b/src/compiler/nir/nir_opcodes.py
index 553f924..7c8a322
This is used to facilitate the Vulkan binding model where each resource is
described by a (descriptor set, binding, array index) tuple.
---
src/compiler/nir/nir.h| 12
src/compiler/nir/nir_intrinsics.h | 20
src/compiler/nir/nir_print.c | 2 ++
---
src/compiler/Makefile.sources | 6 +++---
src/compiler/nir/Makefile.sources | 6 +++---
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/src/compiler/Makefile.sources b/src/compiler/Makefile.sources
index 1f85172..0aee200 100644
--- a/src/compiler/Makefile.sources
+++
The algorithm used is different from both the naieve suggestion from the
GLSL spec and the one used in GLSL IR today. Unfortunately, the GLSL IR
implementation doesn't handle some of the corner cases correctly and
neither does a naieve f * 2.0^exp implementation. Assuming that hardware
does the
These weren't added before because they are actually calculated values that
are computed from other inputs. However, in order to handle them in
nir_lower_system_values, it's nice for them to have a cannonical locaiton.
---
src/compiler/shader_enums.c | 2 ++
src/compiler/shader_enums.h | 2 ++
2
---
src/compiler/nir/nir_builder.h | 14 ++
1 file changed, 14 insertions(+)
diff --git a/src/compiler/nir/nir_builder.h b/src/compiler/nir/nir_builder.h
index e9c409b..3dc7c25 100644
--- a/src/compiler/nir/nir_builder.h
+++ b/src/compiler/nir/nir_builder.h
@@ -75,6 +75,20 @@
They are compute-shader only and that's where the code for doing atomics on
shared variables lives so it seemes to make sense.
---
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 152 +++
1 file changed, 76 insertions(+), 76 deletions(-)
diff --git
---
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 28
src/mesa/drivers/dri/i965/brw_vec4_nir.cpp | 25 +
2 files changed, 53 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
index
In Vulkan, you have InstanceIndex which begins at the base instance value
rather than the zero-based InstanceID of GL.
---
src/compiler/shader_enums.c | 1 +
src/compiler/shader_enums.h | 7 +++
2 files changed, 8 insertions(+)
diff --git a/src/compiler/shader_enums.c
---
src/compiler/nir_types.cpp | 7 +++
src/compiler/nir_types.h | 3 +++
2 files changed, 10 insertions(+)
diff --git a/src/compiler/nir_types.cpp b/src/compiler/nir_types.cpp
index 3669cfe..70e9cd3 100644
--- a/src/compiler/nir_types.cpp
+++ b/src/compiler/nir_types.cpp
@@ -124,6 +124,13
---
src/compiler/nir/nir_builder.h | 17 +
1 file changed, 17 insertions(+)
diff --git a/src/compiler/nir/nir_builder.h b/src/compiler/nir/nir_builder.h
index b245f48..4df79f5 100644
--- a/src/compiler/nir/nir_builder.h
+++ b/src/compiler/nir/nir_builder.h
@@ -305,6 +305,23 @@
They are no longer in the list of local variables so we need to explicitly
sweep them.
---
src/compiler/nir/nir_sweep.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/compiler/nir/nir_sweep.c b/src/compiler/nir/nir_sweep.c
index 5c62154..b22f0f5 100644
---
This is needed for supporting the Vulkan binding model
---
src/compiler/nir/glsl_to_nir.cpp | 1 +
src/compiler/nir/nir.h | 5 +
2 files changed, 6 insertions(+)
diff --git a/src/compiler/nir/glsl_to_nir.cpp b/src/compiler/nir/glsl_to_nir.cpp
index f6e1a17..efef9f5 100644
---
---
src/compiler/nir/nir.h | 11 +++
1 file changed, 11 insertions(+)
diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h
index d46858e..d39c5fd 100644
--- a/src/compiler/nir/nir.h
+++ b/src/compiler/nir/nir.h
@@ -1816,6 +1816,17 @@ typedef struct nir_shader {
---
src/compiler/Makefile.sources | 1 +
src/compiler/nir/Makefile.sources | 1 +
src/compiler/nir/nir.h | 1 +
src/compiler/nir/nir_gather_info.c | 126 +
4 files changed, 129 insertions(+)
create mode 100644
---
src/compiler/nir/nir_lower_outputs_to_temporaries.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/compiler/nir/nir_lower_outputs_to_temporaries.c
b/src/compiler/nir/nir_lower_outputs_to_temporaries.c
index 71b06b8..80c9af4 100644
---
---
src/compiler/nir/nir.h | 28
1 file changed, 28 insertions(+)
diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h
index 68df76c..57b1a87 100644
--- a/src/compiler/nir/nir.h
+++ b/src/compiler/nir/nir.h
@@ -348,6 +348,34 @@ nir_variable_is_global(const
---
src/compiler/nir/nir.h | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h
index 33bb1fd..68df76c 100644
--- a/src/compiler/nir/nir.h
+++ b/src/compiler/nir/nir.h
@@ -339,6 +339,9 @@ typedef struct nir_variable {
#define
This patch series adds a bunch of random things that are used by the SPIR-V
front-end and by our Vulkan driver. It's in no particular order (other
than ensuring that it builds) and most of the newly added things don't have
a consumer outside our Vulkan driver. With this series and the one for
Quoting Adam Jackson (2016-03-24 11:10:03)
[snip]
> @@ -620,9 +617,10 @@ class PrintGlxReqSize_c(PrintGlxReqSize_common):
> if s == 0: s = 1
>
> sig += "(%u,%u)" % (f.offset_of(p.counter), s)
> -size += '%s%s' % (plus, p.size_string())
> -
Quoting Adam Jackson (2016-03-24 11:10:01)
> Signed-off-by: Adam Jackson
> ---
> src/mapi/glapi/gen/gl_XML.py | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/mapi/glapi/gen/gl_XML.py b/src/mapi/glapi/gen/gl_XML.py
> index 2e7123e..e11f6fc 100644
>
Quoting Adam Jackson (2016-03-24 11:09:56)
> Broken by:
>
> commit 9ace0b542241c77ae82a0835ac8a09e2a7510eaf
> Author: Dylan Baker
> Date: Wed May 20 15:49:11 2015 -0700
>
> glapi: glX_proto_size.py: use argparse instead of getopt
>
> Which changed
Quoting Adam Jackson (2016-03-24 10:58:40)
> Broken by:
>
> commit 9ace0b542241c77ae82a0835ac8a09e2a7510eaf
> Author: Dylan Baker
> Date: Wed May 20 15:49:11 2015 -0700
>
> glapi: glX_proto_size.py: use argparse instead of getopt
>
> Which changed
On Fri, Mar 25, 2016 at 6:12 PM, Dieter Nützel wrote:
> Hello all AMD guys,
>
> does that mean, that Polaris 10 didn't have VCE support?
> Or is it the same for both?
It's the same FW version for both.
Alex
>
> With VCE support and the like Polaris 10 looks like the right
Hello all AMD guys,
does that mean, that Polaris 10 didn't have VCE support?
Or is it the same for both?
With VCE support and the like Polaris 10 looks like the right thing for
me.
Thanks a lot for 'day one' support...;-)
-Dieter
Am 23.03.2016 19:43, schrieb Alex Deucher:
From: Sonny
Note: This patch appears to violate older OpenGL and OpenGLES specs.
The OpenGLES GLSL 3.1 and OpenGL GLSL 4.3 specifications both remove
the requirement for the output and input centroid qualifiers to match.
The deqp
dEQP-GLES3.functional.shaders.linkage.varying.rules.differing_interpolation_2
On 03/24/2016 07:03 PM, Thomas Helland wrote:
Welcome to the community!
Some small comments.
We usually write patch messages in present tense.
So, "Format code in ".
Some more comments below.
On Mar 25, 2016 01:24, "Rovanion Luckey"
svga_shader_expand() will fall back to using non-malloced memory for
emit.buf if malloc fails. We should check if the memory is malloced
before freeing it in the error path of svga_tgsi_vgpu9_translate.
Original patch by Thomas Hindoe Paaboel Andersen .
Remove trivial
On 03/24/2016 04:36 PM, Thomas Hindoe Paaboel Andersen wrote:
svga_shader_expand will fall back to using non-malloced
memory for emit.buf if malloc fails. We should check if
the memory is malloced before freeing it in the error
path of svga_tgsi_vgpu9_translate.
---
All we need to do is check if
Tested and Reviewed-by: Charmaine Lee
From: Brian Paul
Sent: Friday, March 25, 2016 1:08 PM
To: mesa-dev@lists.freedesktop.org
Cc: Charmaine Lee
Subject: [PATCH] st/xa: emit sampler view declarations in shaders
Fixes recent regressions with the VMware gallium driver.
---
src/gallium/state_trackers/xa/xa_tgsi.c | 19 +++
1 file changed, 19 insertions(+)
diff --git a/src/gallium/state_trackers/xa/xa_tgsi.c
b/src/gallium/state_trackers/xa/xa_tgsi.c
index 5d8b807..a50393d 100644
---
On Tue, Mar 22, 2016 at 3:33 PM, Jason Ekstrand
wrote:
> This is mostly a re-send of a patch series I've had floating around in one
> form or a while for quite some time. It's basically the same except that
> the original version was missing a work-around for Sandy Bridge.
On Thu, Mar 24, 2016 at 7:45 PM, Francisco Jerez
wrote:
> Francisco Jerez writes:
>
> > Jason Ekstrand writes:
> >
> >> On Thu, Mar 24, 2016 at 5:50 PM, Kenneth Graunke >
> >> wrote:
> >>
> >>>
On Thu, Mar 24, 2016 at 8:11 PM, Edward O'Callaghan
wrote:
> Signed-off-by: Edward O'Callaghan
> ---
> docs/GL3.txt | 2 +-
> docs/relnotes/11.3.0.html | 1 +
> 2 files changed, 2 insertions(+), 1 deletion(-)
>
> diff
On Fri, Mar 25, 2016 at 8:11 AM, wrote:
> On 2016-03-25 22:20, Ilia Mirkin wrote:
>>
>> On Mar 25, 2016 4:43 AM, wrote:
>>>
>>>
>>> On 2016-03-25 14:02, Ilia Mirkin wrote:
On Thu, Mar 24, 2016 at 8:11 PM, Edward
On 2016-03-25 22:20, Ilia Mirkin wrote:
On Mar 25, 2016 4:43 AM, wrote:
On 2016-03-25 14:02, Ilia Mirkin wrote:
On Thu, Mar 24, 2016 at 8:11 PM, Edward O'Callaghan
wrote:
Using PIPE_FORMAT_NONE to indicate what MSAA modes are
On Mar 25, 2016 4:43 AM, wrote:
>
> On 2016-03-25 14:02, Ilia Mirkin wrote:
>>
>> On Thu, Mar 24, 2016 at 8:11 PM, Edward O'Callaghan
>> wrote:
>>>
>>> Using PIPE_FORMAT_NONE to indicate what MSAA modes are supported
>>> with a
This is a tiny housekeeping patch which does the following:
* Replaced tabs with three spaces.
* Formatted oneline and multiline code comments. Some doxygen
comments weren't marked as such and some code comments were marked
as doxygen comments.
* Spaces between if- and
I'll resend a patch following Jose's suggestion, probably on Sunday. There are
just too many things happening before that.
Pierre
> On 25 Mar 2016, at 00:36, Jose Fonseca wrote:
>
>> On 24/03/16 22:07, Ilia Mirkin wrote:
>>> On Sat, Mar 19, 2016 at 6:30 PM, Jose Fonseca
On 2016-03-25 14:29, Ilia Mirkin wrote:
Please leave this and the next patch out of your series. I'm going to
need to retest everything carefully once the core support is in (and I
get a bit of time).
Done. Be sure to remove my Rb if you made changes and i`ll review it
again.
Thanks,
On 2016-03-25 14:20, Ilia Mirkin wrote:
Instead of introducing buggy code in patch 6/17 and then fixing it up
here, you need to fold this with patch 6 so that it's all done at the
same time.
Yea, can do. Cheers,
On Thu, Mar 24, 2016 at 8:11 PM, Edward O'Callaghan
On 2016-03-25 14:02, Ilia Mirkin wrote:
On Thu, Mar 24, 2016 at 8:11 PM, Edward O'Callaghan
wrote:
Using PIPE_FORMAT_NONE to indicate what MSAA modes are supported
with a framebuffer using no attachment.
Signed-off-by: Edward O'Callaghan
59 matches
Mail list logo