On 07/10/15 01:19, Timothy Arceri wrote:
> On Tue, 2015-10-06 at 15:07 +0200, Samuel Iglesias Gonsálvez wrote:
>> On 06/10/15 12:59, Timothy Arceri wrote:
>>> On Tue, 2015-10-06 at 10:08 +0200, Samuel Iglesias Gonsalvez wrote:
From ARB_program_query_interface:
For the property AR
https://bugs.freedesktop.org/show_bug.cgi?id=92122
--- Comment #12 from Tapani Pälli ---
Updating status, existing patches can be scrapped and I will send new *real*
fix soon.
--
You are receiving this mail because:
You are the QA Contact for the bug.
___
Signed-off-by: Varad Gautam
---
src/egl/drivers/dri2/egl_dri2.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.c
index aff2181..f600d1b 100644
--- a/src/egl/drivers/dri2/egl_dri2.c
+++ b/src/egl/drivers/dri2/egl_
commit c2c2e9a (egl: implement EGL_KHR_gl_colorspace (v2)) leaves
_EGLConfig->SurfaceType set incorrectly before calling _eglLinkConfig(),
and the bad value is passed around to platform_android. set it to zero
as earlier.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91596
Signed-off-by:
On Tue, Oct 6, 2015 at 11:11 PM, oscar bg wrote:
> Hi,
> seeing Mesa support up to GL4.3 is almost complete now would be nice if
> developers can implement some AZDO extensions which were released as
> optional ARB extensions in 2013 with GL4.4 release..
>
> With that GL Mesa implementation should
On Wednesday, October 07, 2015 05:11:21 AM oscar bg wrote:
> Hi,
> seeing Mesa support up to GL4.3 is almost complete now would be nice if
> developers can implement some AZDO extensions which were released as
> optional ARB extensions in 2013 with GL4.4 release..
>
> With that GL Mesa implementat
Hi,
seeing Mesa support up to GL4.3 is almost complete now would be nice if
developers can implement some AZDO extensions which were released as
optional ARB extensions in 2013 with GL4.4 release..
With that GL Mesa implementation should be able to compete better with new
Vulkan API coming soon..
From: Ian Romanick
It may be possible to consolodate this check with the existing check for
uniform and const initializers, but a way that didn't require a huge
amount of restructing was not obvious to me.
Fixes:
ES2-CTS.shaders.negative.initialize
ES3-CTS.shaders.negative.initialize
Boyan Ding writes:
> Similar to 9ffc1049ca (freedreno/ir3: use nir two-sided-color lowering).
> No piglit regression.
Pushed. Thanks!
By the way, if you're interested in getting an fd.o account to push
patches directly (http://www.freedesktop.org/wiki/AccountRequests/), I'd
ack it. I'm also h
The SDM reference is admittedly a little fuzzy on the required fencing, but an
SFENCE has no impact on the streaming loads. MFENCE is required to enforce the
ordering of the streaming loads.
Joe
-Original Message-
From: Chris Wilson [mailto:ch...@chris-wilson.co.uk]
Sent: Tuesday, Oct
On 10/5/2015 7:06 PM, Kristian Høgsberg wrote:
On Mon, Oct 5, 2015 at 7:03 AM, Michel Thierry wrote:
On 9/14/2015 2:54 PM, Michał Winiarski wrote:
On Thu, Sep 03, 2015 at 03:23:58PM +0100, Michel Thierry wrote:
Gen8+ supports 48-bit virtual addresses, but some objects must always be
allocat
https://bugs.freedesktop.org/show_bug.cgi?id=91596
Mauro Rossi changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On 6 October 2015 at 11:26, Marek Olšák wrote:
> Hi,
>
> This is a start of reworking how st/mesa translates and creates shaders. The
> result of this patch series is this:
>
> In LinkShader or ProgramStringNotify, the shader is translated to TGSI as-is.
> There are no shader code modifications
On Tue, 2015-10-06 at 15:07 +0200, Samuel Iglesias Gonsálvez wrote:
> On 06/10/15 12:59, Timothy Arceri wrote:
> > On Tue, 2015-10-06 at 10:08 +0200, Samuel Iglesias Gonsalvez wrote:
> > > From ARB_program_query_interface:
> > >
> > > For the property ARRAY_SIZE, a single integer identifying the
On Tue, Oct 6, 2015 at 6:39 PM, Eric Anholt wrote:
> r<#secure method=pgpmime mode=sign>
> Ilia Mirkin writes:
>
>> Signed-off-by: Ilia Mirkin
>
> It looks like cso_destroy_context should already be making you
> unreference a bunch of this stuff. Am I missing something?
I may have gone a littl
When checking for non-const indexing of interfaces
take into account arrays of arrays
---
src/glsl/ast_array_index.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/glsl/ast_array_index.cpp b/src/glsl/ast_array_index.cpp
index 5927c91..74d403f 100644
--- a/src/glsl/ast_ar
Currently only one ir assignment is removed for each var in a single
dead code optimisation pass. This means if a var has more than one
assignment, then it requires all the glsl optimisations to be run again
for each additional assignment to be removed.
Another pass is also required to remove the v
Reviewed-by: Ian Romanick
---
docs/GL3.txt | 4 ++--
docs/relnotes/11.1.0.html | 1 +
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/docs/GL3.txt b/docs/GL3.txt
index e17e783..ffdc524 100644
--- a/docs/GL3.txt
+++ b/docs/GL3.txt
@@ -149,7 +149,7 @@ GL 4.2, GLSL 4.20:
V2: make array processing functions static
---
src/glsl/lower_named_interface_blocks.cpp | 51 ++-
1 file changed, 37 insertions(+), 14 deletions(-)
diff --git a/src/glsl/lower_named_interface_blocks.cpp
b/src/glsl/lower_named_interface_blocks.cpp
index 01bbdd0..04677
From Section 4.1.9 of the GLSL ES 3.10 spec:
"Arrays are sized either at compile-time or at run-time.
To size an array at compile-time, either the size
must be specified within the brackets as above or
must be inferred from the type of the initializer."
---
src/glsl/ast.h | 1
---
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 44 --
src/mesa/drivers/dri/i965/brw_nir_uniforms.cpp | 2 ++
2 files changed, 30 insertions(+), 16 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
index 2b
Reviewed-by: Ian Romanick
---
src/glsl/ir.cpp | 4 ++--
src/glsl/linker.cpp | 9 +
2 files changed, 7 insertions(+), 6 deletions(-)
diff --git a/src/glsl/ir.cpp b/src/glsl/ir.cpp
index 2c45b9e..ee80abb 100644
--- a/src/glsl/ir.cpp
+++ b/src/glsl/ir.cpp
@@ -1673,8 +1673,8 @@ ir_variab
---
src/glsl/lower_ubo_reference.cpp | 119 ---
1 file changed, 87 insertions(+), 32 deletions(-)
diff --git a/src/glsl/lower_ubo_reference.cpp b/src/glsl/lower_ubo_reference.cpp
index 247620e..7e983d0 100644
--- a/src/glsl/lower_ubo_reference.cpp
+++ b/src/gls
---
src/glsl/link_uniforms.cpp | 77 +-
1 file changed, 49 insertions(+), 28 deletions(-)
diff --git a/src/glsl/link_uniforms.cpp b/src/glsl/link_uniforms.cpp
index e2ee628..d34d612 100644
--- a/src/glsl/link_uniforms.cpp
+++ b/src/glsl/link_uniforms.cp
Reviewed-by: Samuel Iglesias Gonsálvez
Reviewed-by: Ian Romanick
---
src/glsl/glsl_types.cpp | 15 ---
1 file changed, 8 insertions(+), 7 deletions(-)
diff --git a/src/glsl/glsl_types.cpp b/src/glsl/glsl_types.cpp
index a1ee505..9a0f5cd 100644
--- a/src/glsl/glsl_types.cpp
+++ b/src
V3: use a check_*_allowed style function for requirements checking
rather than has_* which doesn't encapsulate the error message
V2: add missing 's' to the extension name in error messages
and add decimal place in version string
Reviewed-by: Marta Lofstedt
---
src/glsl/ast_to_hir.cpp |
Add support for setting the max access of an unsized member
of an interface array of arrays.
For example ifc[j][k].foo[i] where foo is unsized.
Reviewed-by: Ian Romanick
---
src/glsl/ast_array_index.cpp | 16
1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/src/gl
This adds support for setting up the UniformBlock structures for AoA
and also adds support for resizing AoA blocks with a packed layout.
---
src/glsl/link_uniform_block_active_visitor.cpp | 168 -
src/glsl/link_uniform_block_active_visitor.h | 13 +-
src/glsl/link_unifor
This allows the correct offset to be calculated for use in indirect
indexing of samplers.
---
src/glsl/glsl_types.cpp | 17 -
1 file changed, 16 insertions(+), 1 deletion(-)
diff --git a/src/glsl/glsl_types.cpp b/src/glsl/glsl_types.cpp
index 25927f6..a1ee505 100644
--- a/src/glsl
Also add TODO comment about adding proper support
Signed-off-by: Timothy Arceri
Reviewed-by: Ian Romanick
---
src/glsl/ir_set_program_inouts.cpp | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/glsl/ir_set_program_inouts.cpp
b/src/glsl/ir_set_program_inouts.cpp
index b7a0f6e..d7c2
V2: get the correct vector elements value for outputs
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 4 ++--
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 7 +++
2 files changed, 5 insertions(+), 6 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
b/src/mesa/drivers/dri/i965/brw_fs
Reviewed-by: Tapani Pälli
Reviewed-by: Ian Romanick
---
src/glsl/nir/nir_types.cpp | 6 ++
src/glsl/nir/nir_types.h | 2 ++
2 files changed, 8 insertions(+)
diff --git a/src/glsl/nir/nir_types.cpp b/src/glsl/nir/nir_types.cpp
index da9807f..965f423 100644
--- a/src/glsl/nir/nir_types.cpp
---
src/glsl/ast_function.cpp | 43 ++-
src/glsl/lower_subroutine.cpp | 2 +-
2 files changed, 39 insertions(+), 6 deletions(-)
diff --git a/src/glsl/ast_function.cpp b/src/glsl/ast_function.cpp
index cf4e64a..402180b 100644
--- a/src/glsl/ast_function
---
src/glsl/nir/nir_lower_atomics.c | 22 --
1 file changed, 12 insertions(+), 10 deletions(-)
diff --git a/src/glsl/nir/nir_lower_atomics.c b/src/glsl/nir/nir_lower_atomics.c
index 6f9ecc0..46e1376 100644
--- a/src/glsl/nir/nir_lower_atomics.c
+++ b/src/glsl/nir/nir_lower_at
This marks all counters in an AoA as active.
For AoA all but the innermost array are treated as separate
counters/uniforms. The Nvidia binary also goes further and
finds inactive counters in the AoA, in future we should do
this too, however this gets things working for the time being.
This change
Reviewed-by: Samuel Iglesias Gonsálvez
---
src/mesa/drivers/dri/i965/intel_extensions.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c
b/src/mesa/drivers/dri/i965/intel_extensions.c
index 3f9afd1..c1f3d0d 100644
--- a/src/mesa/drivers/dri/i965/in
V3: Fix setting of data.location for struct AoA UBO members
V2: Handle arrays of arrays in the same way structures are handled
The ARB_arrays_of_arrays spec doesn't give very many details on how
AoA uniforms are intended to be implemented. However in the
ARB_program_interface_query spec there are
Reviewed-by: Ian Romanick
---
src/glsl/link_uniform_initializers.cpp | 83 --
1 file changed, 49 insertions(+), 34 deletions(-)
diff --git a/src/glsl/link_uniform_initializers.cpp
b/src/glsl/link_uniform_initializers.cpp
index b84c160..1b22e35 100644
--- a/src/gl
V3: move patch after fixes to ast for AoA and add const to helper
as suggested by Ian
V2: move single dimensional array detection into a helper
Signed-off-by: Timothy Arceri
Reviewed-by: Ian Romanick
---
src/glsl/ast.h | 6 ++
src/glsl/ast_to_hir.cpp | 3 ++-
2 files changed, 8 in
Patches still needing review (11/24):
3, 8, 9, 11, 12, 13, 14, 18, 19, 20, 21
Series can be found in AoA_v7 branch here:
https://github.com/tarceri/Mesa_arrays_of_arrays
Changes in v7:
- Some patches dropped as the old GLSL IR vec4 backend is now gone
- Rebased on code refactors in master
- Some
r<#secure method=pgpmime mode=sign>
Ilia Mirkin writes:
> Signed-off-by: Ilia Mirkin
It looks like cso_destroy_context should already be making you
unreference a bunch of this stuff. Am I missing something?
___
mesa-dev mailing list
mesa-dev@lists.fr
yes, the assertions are still accurate. I haven't been able to get a
stack trace yet. If I don't get to it perhaps Tapani can take a look
this evening.
Ian Romanick writes:
> On 10/06/2015 09:31 AM, Mark Janes wrote:
>> Mark Janes writes:
>>
>>> I tested this patch and found:
>>>
>>> regress
On Tue, 2015-10-06 at 13:24 -0700, Matt Turner wrote:
> On Tue, Oct 6, 2015 at 1:13 PM, Ian Romanick
> wrote:
> > Do you have these on a branch somewhere?
>
> I think it's in his github:
> https://github.com/tarceri/Mesa_arrays_of_arrays/branches
Yes AoA_v7 is the latest rebased on master yeste
On Tue, 2015-10-06 at 13:10 -0700, Ian Romanick wrote:
> On 09/28/2015 07:42 PM, Timothy Arceri wrote:
> > This allows the correct offset to be calculated for use in indirect
> > indexing of samplers.
> > ---
> > src/glsl/glsl_types.cpp | 16 +++-
> > 1 file changed, 15 insertions(+),
On 06/10/15 20:30, Ian Romanick wrote:
On 10/06/2015 12:04 PM, Matt Turner wrote:
On Sat, Oct 3, 2015 at 5:19 PM, Jan Vesely wrote:
Signed-off-by: Jan Vesely
---
include/c11/threads_posix.h | 6 ++
1 file changed, 6 insertions(+)
diff --git a/include/c11/threads_posix.h b/include/c11/
On Tue, Oct 6, 2015 at 1:13 PM, Ian Romanick wrote:
> Do you have these on a branch somewhere?
I think it's in his github:
https://github.com/tarceri/Mesa_arrays_of_arrays/branches
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.fr
This patch is
Reviewed-by: Ian Romanick
On 09/28/2015 07:42 PM, Timothy Arceri wrote:
> Add support for setting the max access of an unsized member
> of an interface array of arrays.
>
> For example ifc[j][k].foo[i] where foo is unsized.
> ---
> src/glsl/ast_array_index.cpp | 16 --
This patch is
Reviewed-by: Ian Romanick
On 09/28/2015 07:42 PM, Timothy Arceri wrote:
> ---
> src/glsl/ir.cpp | 4 ++--
> src/glsl/linker.cpp | 9 +
> 2 files changed, 7 insertions(+), 6 deletions(-)
>
> diff --git a/src/glsl/ir.cpp b/src/glsl/ir.cpp
> index 2c45b9e..ee80abb 100644
On 09/28/2015 07:42 PM, Timothy Arceri wrote:
> Reviewed-by: Samuel Iglesias Gonsálvez
> ---
> src/glsl/glsl_types.cpp | 15 ---
> 1 file changed, 8 insertions(+), 7 deletions(-)
>
> diff --git a/src/glsl/glsl_types.cpp b/src/glsl/glsl_types.cpp
> index 6664a47..cc2f102 100644
> ---
Do you have these on a branch somewhere?
On 09/28/2015 07:42 PM, Timothy Arceri wrote:
> The following are standalone patches I would like to commit asap.
>
> [PATCH 05/27] glsl: remove dead code in a single pass
> [PATCH 10/27] i965: fix atomic buffer index for bindings other than 0
> [PATCH 13/
When this lands we should probably also submit a bug report. It is
likely to be forgotten otherwise. This patch is
Reviewed-by: Ian Romanick
On 09/28/2015 07:42 PM, Timothy Arceri wrote:
> Also add TODO comment about adding proper support
>
> Signed-off-by: Timothy Arceri
> ---
> src/glsl/i
On 09/28/2015 07:42 PM, Timothy Arceri wrote:
> This allows the correct offset to be calculated for use in indirect
> indexing of samplers.
> ---
> src/glsl/glsl_types.cpp | 16 +++-
> 1 file changed, 15 insertions(+), 1 deletion(-)
>
> diff --git a/src/glsl/glsl_types.cpp b/src/glsl/
On 10/06/2015 09:31 AM, Mark Janes wrote:
> Mark Janes writes:
>
>> I tested this patch and found:
>>
>> regressions:
>> spec.arb_shader_storage_buffer_object.layout-std140-write-shader (BDW
>> only)
>> expected[1] = 1.00. Read value: 0.00
>
> In subsequent testing I found this BDW failu
On 10/06/2015 01:00 PM, Ian Romanick wrote:
> On 10/05/2015 11:18 AM, Brian Paul wrote:
>> Before, we were unconditionally assigning the TargetIndex field in
>> _mesa_BindTexture(), even if it was already set properly. Now we
>> initialize TargetIndex wherever we initialize the Target field, in
>>
On 10/05/2015 11:18 AM, Brian Paul wrote:
> Before, we were unconditionally assigning the TargetIndex field in
> _mesa_BindTexture(), even if it was already set properly. Now we
> initialize TargetIndex wherever we initialize the Target field, in
> _mesa_initialize_texture_object(), finish_texture
Typo in the subject: atomic
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
On 10/06/2015 12:43 PM, Emil Velikov wrote:
On 6 October 2015 at 16:39, Kyle Brenneman wrote:
On 10/06/2015 07:34 AM, Emil Velikov wrote:
Hello Kyle,
A few questions/points of discussion:
* What is your take on having a libglvnd 'package', which provides
the headers (and maybe other mater
This patch is
Reviewed-by: Ian Romanick
On 09/28/2015 07:42 PM, Timothy Arceri wrote:
> This marks all counters in an AoA as active.
>
> For AoA all but the innermost array are treated as separate
> counters/uniforms. The Nvidia binary also goes further and
> finds inactive counters in the AoA,
On 10/06/2015 10:31 AM, Mark Janes wrote:
Mark Janes writes:
I tested this patch and found:
regressions:
spec.arb_shader_storage_buffer_object.layout-std140-write-shader (BDW only)
expected[1] = 1.00. Read value: 0.00
In subsequent testing I found this BDW failure to be a pre-exist
On 09/28/2015 07:42 PM, Timothy Arceri wrote:
> Currently only one ir assignment is removed for each var in a single
> dead code optimisation pass. This means if a var has more than one
> assignment, then it requires all the glsl optimisations to be run again
> for each additional assignment to be
On 10/06/2015 12:04 PM, Matt Turner wrote:
> On Sat, Oct 3, 2015 at 5:19 PM, Jan Vesely wrote:
>> Signed-off-by: Jan Vesely
>> ---
>> include/c11/threads_posix.h | 6 ++
>> 1 file changed, 6 insertions(+)
>>
>> diff --git a/include/c11/threads_posix.h b/include/c11/threads_posix.h
>> index 3
On Sat, Oct 3, 2015 at 5:19 PM, Jan Vesely wrote:
> Signed-off-by: Jan Vesely
> ---
> include/c11/threads_posix.h | 6 ++
> 1 file changed, 6 insertions(+)
>
> diff --git a/include/c11/threads_posix.h b/include/c11/threads_posix.h
> index 3def6c4..ce9853b 100644
> --- a/include/c11/threads_p
https://bugs.freedesktop.org/show_bug.cgi?id=92122
Krzysztof A. Sobiecki changed:
What|Removed |Added
CC||sob...@gmail.com
--- Comment #11
On Tue, Oct 6, 2015 at 10:01 AM, Kenneth Graunke wrote:
> Switch statements introduce a bogus loop with an unconditional break at
> the end of the loop, just before the while...so the while is unreachable
> and has no immediate dominator.
>
> Signed-off-by: Kenneth Graunke
> ---
> src/mesa/drive
Thanks for reposting. Look OK to me. I'll commit it later.
Reviewed-by: Brian Paul
On 10/05/2015 03:36 PM, Stefan Dösinger wrote:
Version 3: Simplify the code comment, word wrap commit description.
Version 2: Return GL_FALSE if ARB_shadow is unsupported instead of
pretending to store the v
On 6 October 2015 at 16:39, Kyle Brenneman wrote:
> On 10/06/2015 07:34 AM, Emil Velikov wrote:
>>
>> Hello Kyle,
>>
>> A few questions/points of discussion:
>>
>> * What is your take on having a libglvnd 'package', which provides
>> the headers (and maybe other materials), apart from the librar
This patch is
Reviewed-by: Ian Romanick
On 09/28/2015 07:42 PM, Timothy Arceri wrote:
> ---
> src/glsl/link_uniform_initializers.cpp | 83
> --
> 1 file changed, 49 insertions(+), 34 deletions(-)
>
> diff --git a/src/glsl/link_uniform_initializers.cpp
> b/src/
On 10/06/2015 11:28 AM, Alex Deucher wrote:
> On Tue, Oct 6, 2015 at 2:13 PM, Ian Romanick wrote:
>> From: Ian Romanick
>>
>> Signed-off-by: Ian Romanick
>
> Assuming the pointer is set correctly as per Brian's comments, this patch is:
It looks like r100CreateContext calls _mesa_init_driver_fu
On Tue, Oct 6, 2015 at 2:13 PM, Ian Romanick wrote:
> From: Ian Romanick
>
> Signed-off-by: Ian Romanick
Assuming the pointer is set correctly as per Brian's comments, this patch is:
Reviewed-by: Alex Deucher
> ---
> src/mesa/drivers/dri/radeon/radeon_fbo.c | 7 ---
> 1 file changed, 7 d
On 09/28/2015 07:42 PM, Timothy Arceri wrote:
> V3: Fix setting of data.location for struct AoA UBO members
>
> V2: Handle arrays of arrays in the same way structures are handled
>
> The ARB_arrays_of_arrays spec doesn't give very many details on how
> AoA uniforms are intended to be implemented.
On 10/06/2015 12:13 PM, Ian Romanick wrote:
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/mesa/state_tracker/st_cb_fbo.c | 12
1 file changed, 12 deletions(-)
diff --git a/src/mesa/state_tracker/st_cb_fbo.c
b/src/mesa/state_tracker/st_cb_fbo.c
index 9d06a23..aae18b4
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/mesa/drivers/dri/i915/intel_buffers.h | 1 -
src/mesa/drivers/dri/i915/intel_fbo.c | 14 --
2 files changed, 15 deletions(-)
diff --git a/src/mesa/drivers/dri/i915/intel_buffers.h
b/src/mesa/drivers/dri/i915/intel_buffers
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/mesa/state_tracker/st_cb_fbo.c | 12
1 file changed, 12 deletions(-)
diff --git a/src/mesa/state_tracker/st_cb_fbo.c
b/src/mesa/state_tracker/st_cb_fbo.c
index 9d06a23..aae18b4 100644
--- a/src/mesa/state_tracker/st_cb_fbo.c
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/mesa/drivers/dri/radeon/radeon_fbo.c | 7 ---
1 file changed, 7 deletions(-)
diff --git a/src/mesa/drivers/dri/radeon/radeon_fbo.c
b/src/mesa/drivers/dri/radeon/radeon_fbo.c
index 5eece51..4d75d14 100644
--- a/src/mesa/drivers/dri/rad
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/mesa/drivers/dri/i965/intel_buffers.h | 2 --
src/mesa/drivers/dri/i965/intel_fbo.c | 14 --
2 files changed, 16 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/intel_buffers.h
b/src/mesa/drivers/dri/i965/intel_buffer
On 10/06/2015 12:02 AM, Iago Toral wrote:
> On Wed, 2015-09-30 at 13:58 -0700, Ian Romanick wrote:
>> From: Ian Romanick
>>
>> There are multiple similar implementations of these functions, and a
>> later patch was going to add another.
>>
>> Signed-off-by: Ian Romanick
>> ---
>> src/mesa/driver
On 6 October 2015 at 17:24, Varad Gautam wrote:
> commit c2c2e9a (egl: implement EGL_KHR_gl_colorspace (v2)) leaves
> _EGLConfig->SurfaceType set incorrectly before calling _eglLinkConfig(),
> and the bad value is passed around to platform_android. fix it.
>
Well done Varad, nicely spotted. I'm th
I had also noticed the unused parameter warning, but I had a patch that
just papered over the problem. I'm a bit annoyed that GCC didn't warn
(but I guess it couldn't really) that abs_time might be used uninitialized.
This patch is
Reviewed-by: Ian Romanick
On 10/03/2015 05:19 PM, Jan Vesely w
From: Rob Clark
v2: now building both _static and _dynamic variants of libpipe_loader,
targets can link appropriate version depending on whether they want to
statically link or dynamically load pipe drivers.
TODO probably we should just move inline_{drm,sw}_helper.h into pipe-
loader but there l
Switch statements introduce a bogus loop with an unconditional break at
the end of the loop, just before the while...so the while is unreachable
and has no immediate dominator.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_cfg.cpp | 6 +-
1 file changed, 5 insertions(+), 1
On Sunday, September 20, 2015 07:54:56 PM Rhys Kidd wrote:
> ---
> src/glsl/opt_constant_propagation.cpp | 63
> +--
> 1 file changed, 38 insertions(+), 25 deletions(-)
Hi Rhys!
Thanks for looking into this :) A couple of comments...
Now that acp_entry isn't us
This patch caused regressions on master. More work need to be done
before we skip computing align_w for linear 1d layouts. Dropping this
patch from the series. Rest of the patches still need review.
On Wed, Sep 23, 2015 at 4:53 PM, Anuj Phogat wrote:
> If gen9_use_linear_1d_layout() returns true,
https://bugs.freedesktop.org/show_bug.cgi?id=91596
--- Comment #18 from Varad ---
Hello Mauro,
Could you try [1] on your setup? The flashing seems to go away with freedreno
with the patch.
[1] http://patchwork.freedesktop.org/patch/61151/
--
You are receiving this mail because:
You are the QA
Mark Janes writes:
> I tested this patch and found:
>
> regressions:
> spec.arb_shader_storage_buffer_object.layout-std140-write-shader (BDW only)
> expected[1] = 1.00. Read value: 0.00
In subsequent testing I found this BDW failure to be a pre-existing and
intermittent. It is not due to
commit c2c2e9a (egl: implement EGL_KHR_gl_colorspace (v2)) leaves
_EGLConfig->SurfaceType set incorrectly before calling _eglLinkConfig(),
and the bad value is passed around to platform_android. fix it.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91596
Signed-off-by: Varad Gautam
---
On 10/06/2015 07:34 AM, Emil Velikov wrote:
Hello Kyle,
A few questions/points of discussion:
* What is your take on having a libglvnd 'package', which provides
the headers (and maybe other materials), apart from the libraries ?
I'm basically thinking about OpenGL.h, GLX.h, etc for programs t
On 10/05/2015 05:18 PM, Mark Janes wrote:
I tested this patch and found:
regressions:
spec.arb_shader_storage_buffer_object.layout-std140-write-shader (BDW only)
expected[1] = 1.00. Read value: 0.00
spec.arb_copy_image.arb_copy_image-srgb-copy (assertion)
arb_copy_image-srgb-c
Hello Kyle,
A few questions/points of discussion:
* What is your take on having a libglvnd 'package', which provides
the headers (and maybe other materials), apart from the libraries ?
I'm basically thinking about OpenGL.h, GLX.h, etc for programs that
wish to use new ABI, and glvnd{Foo,Bar}.h w
On 06/10/15 12:59, Timothy Arceri wrote:
> On Tue, 2015-10-06 at 10:08 +0200, Samuel Iglesias Gonsalvez wrote:
>> From ARB_program_query_interface:
>>
>> For the property ARRAY_SIZE, a single integer identifying the
>> number of
>> active array elements of an active variable is written to .
>>
On 06/10/15 13:17, Timothy Arceri wrote:
> On Tue, 2015-10-06 at 10:26 +0200, Samuel Iglesias Gonsalvez wrote:
>> Signed-off-by: Samuel Iglesias Gonsalvez
>> Cc: Timothy Arceri
>> ---
>> src/glsl/link_uniforms.cpp | 2 +-
>> src/glsl/lower_ubo_reference.cpp | 6 +++---
>> 2 files changed
On 06/10/15 13:14, Timothy Arceri wrote:
> On Tue, 2015-10-06 at 10:26 +0200, Samuel Iglesias Gonsalvez wrote:
>> Signed-off-by: Samuel Iglesias Gonsalvez
>> Cc: Timothy Arceri
>
>
> I haven't looked at this too hard yet but just fyi I sent a v6 of my
> patchseries that added UBO AoA support
On 06/10/15 13:24, Timothy Arceri wrote:
> On Tue, 2015-10-06 at 10:26 +0200, Samuel Iglesias Gonsalvez wrote:
>> Signed-off-by: Samuel Iglesias Gonsalvez
>> Cc: Timothy Arceri
>
> Reviewed-by: Timothy Arceri
>
Thanks,
Sam
>> ---
>> src/mesa/main/shader_query.cpp | 2 +-
>> 1 file change
On 06/10/15 13:09, Timothy Arceri wrote:
> On Tue, 2015-10-06 at 10:26 +0200, Samuel Iglesias Gonsalvez wrote:
>> Signed-off-by: Samuel Iglesias Gonsalvez
>> Cc: Timothy Arceri
>
> This patch is Reviewed-by: Timothy Arceri
>
> Feel free to push this before the rest of my patches land.
>
OK
On Tue, 2015-10-06 at 10:26 +0200, Samuel Iglesias Gonsalvez wrote:
> Signed-off-by: Samuel Iglesias Gonsalvez
> Cc: Timothy Arceri
Reviewed-by: Timothy Arceri
> ---
> src/mesa/main/shader_query.cpp | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/mesa/main/shader
On Tue, 2015-10-06 at 10:26 +0200, Samuel Iglesias Gonsalvez wrote:
> Signed-off-by: Samuel Iglesias Gonsalvez
> Cc: Timothy Arceri
> ---
> src/glsl/link_uniforms.cpp | 2 +-
> src/glsl/lower_ubo_reference.cpp | 6 +++---
> 2 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a
On Tue, 2015-10-06 at 10:26 +0200, Samuel Iglesias Gonsalvez wrote:
> Signed-off-by: Samuel Iglesias Gonsalvez
> Cc: Timothy Arceri
I haven't looked at this too hard yet but just fyi I sent a v6 of my
patchseries that added UBO AoA support that this patch might clash
with. Also you remove some
On Tue, 2015-10-06 at 10:26 +0200, Samuel Iglesias Gonsalvez wrote:
> Signed-off-by: Samuel Iglesias Gonsalvez
> Cc: Timothy Arceri
This patch is Reviewed-by: Timothy Arceri
Feel free to push this before the rest of my patches land.
> ---
> src/glsl/glsl_types.cpp | 12 +++-
> 1 file
On Tue, 2015-10-06 at 10:08 +0200, Samuel Iglesias Gonsalvez wrote:
> From ARB_program_query_interface:
>
> For the property ARRAY_SIZE, a single integer identifying the
> number of
> active array elements of an active variable is written to .
> The
> array size returned is in units of the t
Neil Roberts writes:
> If a non-const sample number is given to interpolateAtSample it will
> now generate an indirect send message with the sample ID similar to
> how non-const sampler array indexing works. Previously non-const
> values were ignored and instead it ended up using a constant 0 val
On Mon, Oct 05, 2015 at 12:01:25PM -0700, Jordan Justen wrote:
> On 2015-10-05 11:55:57, Matt Turner wrote:
> > On Thu, Oct 1, 2015 at 11:20 AM, Jordan Justen
> > wrote:
> > > On 2015-10-01 10:11:33, Matt Turner wrote:
> > >> According to the Intel Software Development Manual:
> > >
> > > How abou
1 - 100 of 111 matches
Mail list logo
