On 16/09/15 08:53, Jordan Justen wrote:
> Subject line is too long.
>
> On 2015-09-10 06:35:43, Iago Toral Quiroga wrote:
>> From: Samuel Iglesias Gonsalvez
>>
>> They are used to calculate the offset, array stride of uniform/shader storage
>> buffer variables. Take into account this info to ge
Subject line is too long.
On 2015-09-10 06:35:43, Iago Toral Quiroga wrote:
> From: Samuel Iglesias Gonsalvez
>
> They are used to calculate the offset, array stride of uniform/shader storage
> buffer variables. Take into account this info to get the right value for
> std430.
Commit message li
Reviewed-by: Iago Toral Quiroga
On Tue, 2015-09-15 at 19:33 -0400, Rob Clark wrote:
> From: Rob Clark
>
> v2: split out moving of FILE *fp into state structure into it's own
> (more complete patch) to reduce the noise in this one
>
> Signed-off-by: Rob Clark
> ---
> src/glsl/nir/nir_print.c
Thanks for all the explanations, this is:
Reviewed-by: Iago Toral Quiroga
On Tue, 2015-09-15 at 19:33 -0400, Rob Clark wrote:
> From: Rob Clark
>
> Signed-off-by: Rob Clark
> ---
> src/Makefile.am | 1 +
> src/glsl/shader_enums.c | 204
> +++
On 16/09/15 01:39, Jason Ekstrand wrote:
> On Tue, Sep 15, 2015 at 3:23 PM, Alejandro Piñeiro
> wrote:
>> Until know, nir_intrinsic_load_uniform was using a default type. This
> "now"
>> caused some type mismatch between movs and alu operations, so copy
>> propagation optimization was
On Tue, 2015-09-15 at 22:02 -0700, Jason Ekstrand wrote:
> On Tue, Sep 15, 2015 at 9:42 PM, Timothy Arceri <
> t_arc...@yahoo.com.au> wrote:
> > On Tue, 2015-09-15 at 19:21 -0700, Jason Ekstrand wrote:
> > > On Tue, Sep 15, 2015 at 5:18 PM, Timothy Arceri <
> > > t_arc...@yahoo.com.au> wrote:
> > >
On 15/09/15 20:25, Kristian Høgsberg wrote:
> On Fri, Sep 11, 2015 at 08:12:13AM +0200, Iago Toral wrote:
>> On Thu, 2015-09-10 at 15:17 -0400, Ilia Mirkin wrote:
>>> On Thu, Sep 10, 2015 at 2:52 PM, Ian Romanick wrote:
On 09/10/2015 10:45 AM, Ilia Mirkin wrote:
> On Thu, Sep 10, 2015 a
On 15/09/15 20:37, Kristian Høgsberg wrote:
> On Thu, Sep 10, 2015 at 03:35:18PM +0200, Iago Toral Quiroga wrote:
>> This is the same we do for other things like uniforms because it ensures
>> optimal performance.
>>
>> Reviewed-by: Jordan Justen
>> ---
>> src/mesa/drivers/dri/i965/brw_context.
On 15/09/15 21:03, Jordan Justen wrote:
> On 2015-09-10 22:48:55, Samuel Iglesias Gonsálvez wrote:
>> On 10/09/15 20:13, Jordan Justen wrote:
>>> On 2015-09-10 06:35:41, Iago Toral Quiroga wrote:
From: Samuel Iglesias Gonsalvez
They are used to calculate size, base alignment and a
On Tue, Sep 15, 2015 at 9:42 PM, Timothy Arceri wrote:
> On Tue, 2015-09-15 at 19:21 -0700, Jason Ekstrand wrote:
>> On Tue, Sep 15, 2015 at 5:18 PM, Timothy Arceri <
>> t_arc...@yahoo.com.au> wrote:
>> > On Tue, 2015-09-15 at 11:51 -0700, Jason Ekstrand wrote:
>> > > On Tue, Sep 15, 2015 at 12:51
On Tue, 2015-09-15 at 19:21 -0700, Jason Ekstrand wrote:
> On Tue, Sep 15, 2015 at 5:18 PM, Timothy Arceri <
> t_arc...@yahoo.com.au> wrote:
> > On Tue, 2015-09-15 at 11:51 -0700, Jason Ekstrand wrote:
> > > On Tue, Sep 15, 2015 at 12:51 AM, Timothy Arceri <
> > > t_arc...@yahoo.com.au> wrote:
> >
As a bonus we get indirect support for arrays of arrays for free.
V5: move uniform location offset calculation into this file and structure
it in a more readable way, plus a couple of small clean-ups all suggestions
by Jason.
V4: fix struct member location caclulation, use nir_ssa_def rather than
Move this code out of bufferobj.c since it's not strongly connected to
buffer objects.
---
src/mapi/glapi/gen/gl_genexec.py | 1 +
src/mesa/Makefile.sources| 2 +
src/mesa/main/bufferobj.c| 378 ---
src/mesa/main/bufferobj.h| 10 -
src/m
On Tue, Sep 15, 2015 at 5:18 PM, Timothy Arceri wrote:
> On Tue, 2015-09-15 at 11:51 -0700, Jason Ekstrand wrote:
>> On Tue, Sep 15, 2015 at 12:51 AM, Timothy Arceri <
>> t_arc...@yahoo.com.au> wrote:
>> > From: Timothy
>> >
>> > Cc: Jason Ekstrand
>> > ---
>> > src/glsl/glsl_types.cpp | 20 +++
On Tue, Sep 15, 2015 at 9:50 PM, Connor Abbott wrote:
> On Tue, Sep 15, 2015 at 9:42 PM, Rob Clark wrote:
>> On Tue, Sep 15, 2015 at 8:48 PM, Connor Abbott wrote:
>>> On Tue, Sep 15, 2015 at 7:33 PM, Rob Clark wrote:
From: Rob Clark
Rename print_var_state to print_state, and stu
On Tue, Sep 15, 2015 at 9:42 PM, Rob Clark wrote:
> On Tue, Sep 15, 2015 at 8:48 PM, Connor Abbott wrote:
>> On Tue, Sep 15, 2015 at 7:33 PM, Rob Clark wrote:
>>> From: Rob Clark
>>>
>>> Rename print_var_state to print_state, and stuff FILE ptr into the state
>>> object. This avoids passing ar
On Tue, Sep 15, 2015 at 8:48 PM, Connor Abbott wrote:
> On Tue, Sep 15, 2015 at 7:33 PM, Rob Clark wrote:
>> From: Rob Clark
>>
>> Rename print_var_state to print_state, and stuff FILE ptr into the state
>> object. This avoids passing around an extra parameter everywhere.
>>
>> Signed-off-by: R
On Tue, Sep 15, 2015 at 7:33 PM, Rob Clark wrote:
> From: Rob Clark
>
> v2: split out moving of FILE *fp into state structure into it's own
> (more complete patch) to reduce the noise in this one
>
> Signed-off-by: Rob Clark
Reviewed-by: Connor Abbott
__
On Tue, Sep 15, 2015 at 7:33 PM, Rob Clark wrote:
> From: Rob Clark
>
> Rename print_var_state to print_state, and stuff FILE ptr into the state
> object. This avoids passing around an extra parameter everywhere.
>
> Signed-off-by: Rob Clark
> ---
> src/glsl/nir/nir_print.c | 95
> +++
On Tue, 2015-09-15 at 14:01 -0700, Jason Ekstrand wrote:
> On Tue, Sep 15, 2015 at 11:52 AM, Jason Ekstrand <
> ja...@jlekstrand.net> wrote:
> > On Tue, Sep 15, 2015 at 12:51 AM, Timothy Arceri <
> > t_arc...@yahoo.com.au> wrote:
> > > As a bonus we get indirect support for arrays of arrays for fre
On Tue, 2015-09-15 at 11:23 -0700, Jason Ekstrand wrote:
> On Tue, Sep 15, 2015 at 12:51 AM, Timothy Arceri <
> t_arc...@yahoo.com.au> wrote:
> > This allows the correct offset to be easily calculated for indirect
> > indexing when a struct array contains multiple samplers, or any
> > crazy
> > nes
On Tue, 2015-09-15 at 11:51 -0700, Jason Ekstrand wrote:
> On Tue, Sep 15, 2015 at 12:51 AM, Timothy Arceri <
> t_arc...@yahoo.com.au> wrote:
> > From: Timothy
> >
> > Cc: Jason Ekstrand
> > ---
> > src/glsl/glsl_types.cpp | 20
> > src/glsl/glsl_types.h | 7 +++
> >
On Tue, Sep 15, 2015 at 3:23 PM, Alejandro Piñeiro wrote:
> Until know, nir_intrinsic_load_uniform was using a default type. This
"now"
> caused some type mismatch between movs and alu operations, so copy
> propagation optimization was not applied to remove unneeded movs if
> negate modi
On Wed, 16 Sep 2015 01:32:10 +0200, Ilia Mirkin
wrote:
This fixes failures with the newly-submitted max-size texture buffer
piglit test for GPUs exposing >= 128M max texels.
Signed-off-by: Ilia Mirkin
Cc: "10.6 11.0"
---
src/mesa/state_tracker/st_atom_texture.c | 2 +-
1 file changed, 1 i
This is what the hardware supports, there never was any sort of 64K
limit.
Signed-off-by: Ilia Mirkin
Cc: "10.6 11.0"
---
Need to double-check this one on nv50, but online sources suggest that
it's a 128M element limit there as well.
src/gallium/drivers/nouveau/nv50/nv50_screen.c | 2 +-
src/
From: Rob Clark
Extracted out from previous patchset, rebased, and updated the
nir-print to split out the print_state refactoring (which is now
a bit more complete) from the use of shader_enum-to-name helpers.
Rob Clark (3):
glsl: shader-enum to name debug fxns
nir/print: bit of state refact
From: Rob Clark
Rename print_var_state to print_state, and stuff FILE ptr into the state
object. This avoids passing around an extra parameter everywhere.
Signed-off-by: Rob Clark
---
src/glsl/nir/nir_print.c | 95 +++-
1 file changed, 54 insertions
From: Rob Clark
Signed-off-by: Rob Clark
---
src/Makefile.am | 1 +
src/glsl/shader_enums.c | 204 ++
src/glsl/shader_enums.h | 53
src/mesa/Makefile.sources | 4 +-
4 files changed, 261 insertions(+), 1 deletion(-)
cr
From: Rob Clark
v2: split out moving of FILE *fp into state structure into it's own
(more complete patch) to reduce the noise in this one
Signed-off-by: Rob Clark
---
src/glsl/nir/nir_print.c | 45 ++---
1 file changed, 42 insertions(+), 3 deletions(-)
This fixes failures with the newly-submitted max-size texture buffer
piglit test for GPUs exposing >= 128M max texels.
Signed-off-by: Ilia Mirkin
Cc: "10.6 11.0"
---
src/mesa/state_tracker/st_atom_texture.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/state_track
Until know, nir_intrinsic_load_uniform was using a default type. This
caused some type mismatch between movs and alu operations, so copy
propagation optimization was not applied to remove unneeded movs if
negate modifier was involved. This was first detected on minus
(negate+add) operations with un
Reviewed-by: Jason Ekstrand
On Tue, Sep 15, 2015 at 2:48 PM, Jordan Justen
wrote:
> When preparing the barrier payload, the instructions should operate in
> simd8 mode since we only use 1 payload register.
>
> fs_inst::regs_read is also updated to indicate that it only reads one
> register for S
On Fri, Aug 28, 2015 at 7:50 AM, Nanley Chery wrote:
> From: Nanley Chery
>
> Adds S3TC and PALETTE formats.
>
> Signed-off-by: Nanley Chery
> ---
> src/mesa/main/texcompress.c | 14 ++
> 1 file changed, 14 insertions(+)
>
> diff --git a/src/mesa/main/texcompress.c b/src/mesa/main/
On 09/15/2015 09:23 PM, Jason Ekstrand wrote:
> On Tue, Sep 15, 2015 at 4:47 AM, Eduardo Lima Mitev wrote:
>> For consistency and efficiency, the (sub)texture dimension error check
>> should go before the validation of format, type and internal format.
>
> You mentioned in another patch that this
On Fri, Aug 28, 2015 at 7:50 AM, Nanley Chery wrote:
> From: Nanley Chery
>
> Reuse utility functions instead of reimplementing the same logic.
>
> * _mesa_is_compressed_format() performs the required checking to
> determine format support in the current context.
> * _mesa_gl_compressed_format
When preparing the barrier payload, the instructions should operate in
simd8 mode since we only use 1 payload register.
fs_inst::regs_read is also updated to indicate that it only reads one
register for SHADER_OPCODE_BARRIER.
These issues were flagged by:
commit cadd7dd384b33a779d46bd664f456bed4
From: Rob Clark
Signed-off-by: Rob Clark
---
src/glsl/nir/nir.h| 3 +++
src/glsl/nir/nir_opt_algebraic.py | 1 +
2 files changed, 4 insertions(+)
diff --git a/src/glsl/nir/nir.h b/src/glsl/nir/nir.h
index f0acd75..284fccd 100644
--- a/src/glsl/nir/nir.h
+++ b/src/glsl/nir/nir.h
On Tue, Sep 15, 2015 at 11:52 AM, Jason Ekstrand wrote:
> On Tue, Sep 15, 2015 at 12:51 AM, Timothy Arceri
> wrote:
>> As a bonus we get indirect support for arrays of arrays for free.
>>
>> V4: fix struct member location caclulation, use nir_ssa_def rather than
>> nir_src for the indirect as su
On Tue, Sep 15, 2015 at 1:10 PM, Erik Faye-Lund wrote:
>> I guess I'd care about it more once I supported MSAA ;-)
>>
>> (and tbh, all this effort was mostly just to get neverball to render
>> correctly :-P)
>
> Oh, I just meant that it might be neat to add to the commit message. I
> wouldn't go a
The idea here is not that it gives register coalescing a little bit of a
helping hand. It doesn't actually fix the coalescing problems, but it
seems to help a good bit.
Shader-db results for vec4 programs on Haswell:
total instructions in shared programs: 1746280 -> 1683959 (-3.57%)
instru
v2 (Jason Ekstrand):
- Handle non-SSA sources and destinations
---
src/glsl/Makefile.sources| 1 +
src/glsl/nir/nir.h | 1 +
src/glsl/nir/nir_move_vec_src_uses_to_dest.c | 189 +++
3 files changed, 191 insertions(+)
create
The provided indices have the very nice property that if A dominates B then
A->index <= B->index. We should document that somewhere.
Reviewed-by: Kenneth Graunke
---
src/glsl/nir/nir.c | 8
1 file changed, 8 insertions(+)
diff --git a/src/glsl/nir/nir.c b/src/glsl/nir/nir.c
index 1ff8
Reviewed-by: Kenneth Graunke
---
src/glsl/nir/nir.c | 18 ++
src/glsl/nir/nir.h | 4
2 files changed, 22 insertions(+)
diff --git a/src/glsl/nir/nir.c b/src/glsl/nir/nir.c
index 96cf94c..1ff8f21 100644
--- a/src/glsl/nir/nir.c
+++ b/src/glsl/nir/nir.c
@@ -1435,6 +1435,24 @@
On Mon, Sep 14, 2015 at 11:17 AM, Nanley Chery
wrote:
> From: Nanley Chery
>
> Instead of case statements, use _mesa_get_format_layout() to
> determine if a GL format is part of a family of compressed formats.
>
> v2. restrict LATC formats to API_OPENGL_COMPAT (Ilia).
> rename the variable m
C++ gets cranky if we take references of temporaries. This isn't a problem
yet in master because nir_builder is never used from C++. However, it will
be in the future so we should fix it now.
Cc: Rob Clark
---
src/glsl/nir/nir_builder.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
On Tue, Sep 15, 2015 at 4:47 AM, Eduardo Lima Mitev wrote:
> For consistency and efficiency, the (sub)texture dimension error check
> should go before the validation of format, type and internal format.
You mentioned in another patch that this fixes a bug or, at the very
least, prevents one. Wha
On Tue, Sep 15, 2015 at 12:02 PM, Eduardo Lima Mitev wrote:
> Patch 10.1, 10.2 and this one are:
>
> Reviewed-by: Eduardo Lima Mitev
>
> In the case of patch 10.1 and 10.2, though they look good to me, I would
> let Connor Abbott give his OK, since he is in CC for those, and also my
> experience
On 2015-09-10 22:48:55, Samuel Iglesias Gonsálvez wrote:
> On 10/09/15 20:13, Jordan Justen wrote:
> > On 2015-09-10 06:35:41, Iago Toral Quiroga wrote:
> >> From: Samuel Iglesias Gonsalvez
> >>
> >> They are used to calculate size, base alignment and array stride values
> >> for a glsl_type follo
Patch 10.1, 10.2 and this one are:
Reviewed-by: Eduardo Lima Mitev
In the case of patch 10.1 and 10.2, though they look good to me, I would
let Connor Abbott give his OK, since he is in CC for those, and also my
experience with NIR constant expressions is rather limited.
Eduardo
On 09/11/2015
On Tue, Sep 15, 2015 at 12:51 AM, Timothy Arceri wrote:
> As a bonus we get indirect support for arrays of arrays for free.
>
> V4: fix struct member location caclulation, use nir_ssa_def rather than
> nir_src for the indirect as suggested by Jason
>
> V3: Use nir_instr_rewrite_src() with empty sr
On Tue, Sep 15, 2015 at 12:51 AM, Timothy Arceri wrote:
> From: Timothy
>
> Cc: Jason Ekstrand
> ---
> src/glsl/glsl_types.cpp | 20
> src/glsl/glsl_types.h | 7 +++
> 2 files changed, 27 insertions(+)
>
> diff --git a/src/glsl/glsl_types.cpp b/src/glsl/glsl_types.cp
On Thu, Sep 10, 2015 at 03:35:18PM +0200, Iago Toral Quiroga wrote:
> This is the same we do for other things like uniforms because it ensures
> optimal performance.
>
> Reviewed-by: Jordan Justen
> ---
> src/mesa/drivers/dri/i965/brw_context.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff -
On Fri, Sep 11, 2015 at 08:12:13AM +0200, Iago Toral wrote:
> On Thu, 2015-09-10 at 15:17 -0400, Ilia Mirkin wrote:
> > On Thu, Sep 10, 2015 at 2:52 PM, Ian Romanick wrote:
> > > On 09/10/2015 10:45 AM, Ilia Mirkin wrote:
> > >> On Thu, Sep 10, 2015 at 9:35 AM, Iago Toral Quiroga
> > >> wrote:
>
On Tue, Sep 15, 2015 at 12:51 AM, Timothy Arceri wrote:
> This allows the correct offset to be easily calculated for indirect
> indexing when a struct array contains multiple samplers, or any crazy
> nesting.
>
> The indices for the folling struct will now look like this:
> Sampler index: 0 Name:
On Tue, Sep 15, 2015 at 6:39 AM, Francisco Jerez wrote:
> Jason Ekstrand writes:
>
>> In certain conditions, we have to do bounds-checking in the shader for
>> image_load_store. The way this works for image loads is that we do the
>> load anyway and then emit a series of slecects, one per compon
On Tue, Sep 15, 2015 at 12:24:00PM +0300, Abdiel Janulgue wrote:
> Here's a more comprehensive shader-db run:
>
> total instructions in shared programs: 6394485 -> 6374865 (-0.31%)
> instructions in affected programs: 261322 -> 241702 (-7.51%)
> helped:3210
> HU
On Mon, Sep 14, 2015 at 4:35 PM, Jason Ekstrand wrote:
> Cc: Kristian Høgsberg
Reviewed-by: Kristian Høgsberg
> ---
> src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
> b/src/
On Thu, Sep 10, 2015 at 2:38 PM, Ville Syrjälä <
ville.syrj...@linux.intel.com> wrote:
> On Thu, Sep 10, 2015 at 12:20:10PM -0700, Chad Versace wrote:
> > On Wed 19 Aug 2015, Anuj Phogat wrote:
> > > V2:
> > > - Do the tile width/height computations in the new helper
> > > function and use it la
On Mon, Sep 14, 2015 at 4:35 PM, Jason Ekstrand wrote:
> Cc: Kristian Høgsberg
Reviewed-by: Kristian Høgsberg
> ---
> src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
> b/src/mesa/drivers/dri/
Hello! I noticed an inefficiency in libGL.so, so I thought I'd take a
stab at fixing it. This is my first patch submitted to mesa-dev, so
if I'm doing anything dumb, let me know. I can't use git send-email,
but I've formatted the patch using git format-patch, which should
hopefully produce simi
On Tue, Sep 15, 2015 at 7:12 PM, Ilia Mirkin wrote:
> On Tue, Sep 15, 2015 at 1:09 PM, Erik Faye-Lund wrote:
>> On Tue, Sep 15, 2015 at 6:49 PM, Ilia Mirkin wrote:
>>> However having a piglit test that covers this would be neat... I guess
>>> you could clip a pixel in half and make sure that the
On Tue, Sep 15, 2015 at 6:49 PM, Ilia Mirkin wrote:
> On Tue, Sep 15, 2015 at 12:39 PM, Erik Faye-Lund wrote:
>> On Mon, Sep 14, 2015 at 11:53 AM, Erik Faye-Lund wrote:
>>> On Sun, Sep 13, 2015 at 5:51 PM, Rob Clark wrote:
From: Rob Clark
The vertex shader lowering adds calculat
On Tue, Sep 15, 2015 at 1:09 PM, Erik Faye-Lund wrote:
> On Tue, Sep 15, 2015 at 6:49 PM, Ilia Mirkin wrote:
>> However having a piglit test that covers this would be neat... I guess
>> you could clip a pixel in half and make sure that the resolved result
>> is some in-between color? Lots of impl
On Tue, Sep 15, 2015 at 6:49 PM, Rob Clark wrote:
> On Tue, Sep 15, 2015 at 12:39 PM, Erik Faye-Lund wrote:
>> On Mon, Sep 14, 2015 at 11:53 AM, Erik Faye-Lund wrote:
>>> On Sun, Sep 13, 2015 at 5:51 PM, Rob Clark wrote:
From: Rob Clark
The vertex shader lowering adds calculatio
On Tue, Sep 15, 2015 at 6:49 PM, Ilia Mirkin wrote:
> However having a piglit test that covers this would be neat... I guess
> you could clip a pixel in half and make sure that the resolved result
> is some in-between color? Lots of implementation-dependent stuff going
> on in there though.
I thi
On 15 September 2015 at 14:27, Boyan Ding wrote:
> 2015-09-15 1:35 GMT+08:00 Emil Velikov :
>> From: Matthew Waters
>>
>> As of version 15 of the EGL_KHR_create_context spec, debug contexts
>> are allowed for ES contexts. We should allow creation instead of
>> erroring.
>>
>> While we're here pr
On Tue, Sep 15, 2015 at 12:49 PM, Rob Clark wrote:
> On Tue, Sep 15, 2015 at 12:39 PM, Erik Faye-Lund wrote:
>> On Mon, Sep 14, 2015 at 11:53 AM, Erik Faye-Lund wrote:
>>> On Sun, Sep 13, 2015 at 5:51 PM, Rob Clark wrote:
From: Rob Clark
The vertex shader lowering adds calculati
On Tue, Sep 15, 2015 at 12:39 PM, Erik Faye-Lund wrote:
> On Mon, Sep 14, 2015 at 11:53 AM, Erik Faye-Lund wrote:
>> On Sun, Sep 13, 2015 at 5:51 PM, Rob Clark wrote:
>>> From: Rob Clark
>>>
>>> The vertex shader lowering adds calculation for CLIPDIST, if needed
>>> (ie. user-clip-planes), and
On Tue, Sep 15, 2015 at 12:39 PM, Erik Faye-Lund wrote:
> On Mon, Sep 14, 2015 at 11:53 AM, Erik Faye-Lund wrote:
>> On Sun, Sep 13, 2015 at 5:51 PM, Rob Clark wrote:
>>> From: Rob Clark
>>>
>>> The vertex shader lowering adds calculation for CLIPDIST, if needed
>>> (ie. user-clip-planes), and
On Tue, Sep 15, 2015 at 9:52 AM, Iago Toral wrote:
> On Tue, 2015-09-15 at 08:07 -0400, Rob Clark wrote:
>> On Tue, Sep 15, 2015 at 2:55 AM, Iago Toral wrote:
>> > On Sun, 2015-09-13 at 11:51 -0400, Rob Clark wrote:
>> >> From: Rob Clark
>> >>
>> >> Signed-off-by: Rob Clark
>> >> ---
>> >> src
On Mon, Sep 14, 2015 at 11:53 AM, Erik Faye-Lund wrote:
> On Sun, Sep 13, 2015 at 5:51 PM, Rob Clark wrote:
>> From: Rob Clark
>>
>> The vertex shader lowering adds calculation for CLIPDIST, if needed
>> (ie. user-clip-planes), and the frag shader lowering adds conditional
>> kills based on CLIP
No driver changes needed for softpipe/llvmpipe - things just work.
v2: Whitespace fixes.
Signed-off-by: Emil Velikov
Reviewed-by: Boyan Ding
---
docs/relnotes/11.1.0.html | 1 +
src/egl/drivers/dri2/platform_drm.c | 14 +++---
src/egl/drivers/dri2/platform_wayland.c
On 14 September 2015 at 22:33, Marcin Ślusarz wrote:
> On Mon, Sep 14, 2015 at 06:33:16PM +0100, Emil Velikov wrote:
>> Hi Marcin,
>>
>> On 13 September 2015 at 17:55, Marcin Ślusarz
>> wrote:
>> > Useful when locally installed mesa has more quirks than the system one.
>> > ---
>> > src/mesa/dr
Hi Tapani,
On 15 September 2015 at 08:13, Tapani Pälli wrote:
> OpenGL ES 3.0 spec 3.7.2 "Transfer of Pixel Rectangles" specifies
> DEPTH_COMPONENT, UNSIGNED_INT as a valid couple, validation for
> internal format is checked by is_float_depth().
>
> Fix regression caused by 81d2fd9 in following C
On Tue, Sep 15, 2015 at 12:13 AM, Tapani Pälli
wrote:
> OpenGL ES 3.0 spec 3.7.2 "Transfer of Pixel Rectangles" specifies
> DEPTH_COMPONENT, UNSIGNED_INT as a valid couple, validation for
> internal format is checked by is_float_depth().
>
> Fix regression caused by 81d2fd9 in following CTS test:
On Sun, Sep 13, 2015 at 11:51 AM, Rob Clark wrote:
> From: Rob Clark
>
> Signed-off-by: Rob Clark
> ---
> src/glsl/nir/nir_print.c | 73
> ++--
> 1 file changed, 59 insertions(+), 14 deletions(-)
>
> diff --git a/src/glsl/nir/nir_print.c b/src/glsl/n
2015-09-15 1:35 GMT+08:00 Emil Velikov :
> No driver changes needed for softpipe/llvmpipe - things just work.
>
> Signed-off-by: Emil Velikov
> ---
> docs/relnotes/11.1.0.html | 1 +
> src/egl/drivers/dri2/platform_drm.c | 12 ++--
> src/egl/drivers/dri2/platform_waylan
On Sep 15, 2015 6:39 AM, "Francisco Jerez" wrote:
>
> Jason Ekstrand writes:
>
> > In certain conditions, we have to do bounds-checking in the shader for
> > image_load_store. The way this works for image loads is that we do the
> > load anyway and then emit a series of slecects, one per compone
On Tue, 2015-09-15 at 08:27 -0400, Rob Clark wrote:
> On Tue, Sep 15, 2015 at 3:18 AM, Iago Toral wrote:
> > On Sun, 2015-09-13 at 11:51 -0400, Rob Clark wrote:
> >> From: Rob Clark
> >>
> >> Signed-off-by: Rob Clark
> >> ---
> >> src/glsl/nir/nir_print.c | 73
> >>
EGL_KHR_image_base only creates the mechanism about EGLimage without
providing real functionality, so it is safe to expose unconditionally
since egl/main should handle it well. It also saves some platforms from
manually exposing it.
Signed-off-by: Boyan Ding
---
src/egl/drivers/dri2/egl_dri2.c
From: Ian Romanick
v2: Fix '- nr' typo noticed by Marius.
Signed-off-by: Ian Romanick
---
src/mesa/tnl_dd/t_dd_dmatmp.h | 198 --
1 file changed, 95 insertions(+), 103 deletions(-)
diff --git a/src/mesa/tnl_dd/t_dd_dmatmp.h b/src/mesa/tnl_dd/t_dd_dmatmp
Reviewed-by: Eduardo Lima Mitev
On 09/11/2015 05:52 PM, Jason Ekstrand wrote:
> v2 (Jason Ekstrand):
> - Use nir_instr_rewrite_dest
> - Pass impl directly into lower_vec_to_movs
> ---
> src/glsl/nir/nir_lower_vec_to_movs.c | 21 +++--
> 1 file changed, 15 insertions(+), 6 deleti
On 09/14/2015 07:04 PM, Brian Paul wrote:
> On 09/14/2015 07:01 PM, Ian Romanick wrote:
>> From: Ian Romanick
>>
>> Signed-off-by: Ian Romanick
>> ---
>> src/mesa/tnl_dd/t_dd_dmatmp.h | 92
>> +--
>> 1 file changed, 46 insertions(+), 46 deletions(-)
>>
>
On 09/14/2015 07:04 PM, Brian Paul wrote:
> On 09/14/2015 07:01 PM, Ian Romanick wrote:
>> I looked at t_dd_dmatmp.h after the previous discussions about fixing
>> the "count" problem. It was a mess. The first 5 patches fix the bug.
>> The remaining 19 patches delete dead code and make the file a
On 09/10/2015 02:50 AM, Jason Ekstrand wrote:
> The old pass blindly inserted a bunch of moves into the shader with no
> concern for whether or not it was really needed. This adds code to try and
> coalesce into the destination of the instruction providing the value.
>
> Shader-db results for vec
On 09/15/2015 06:18 AM, Predut, Marius wrote:
>
>
>> -Original Message-
>> From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On Behalf Of
>> Ian Romanick
>> Sent: Tuesday, September 15, 2015 3:01 AM
>> To: mesa-dev@lists.freedesktop.org
>> Cc: Romanick, Ian D
>> Subject: [Mes
On Tue, 2015-09-15 at 08:07 -0400, Rob Clark wrote:
> On Tue, Sep 15, 2015 at 2:55 AM, Iago Toral wrote:
> > On Sun, 2015-09-13 at 11:51 -0400, Rob Clark wrote:
> >> From: Rob Clark
> >>
> >> Signed-off-by: Rob Clark
> >> ---
> >> src/Makefile.am | 1 +
> >> src/glsl/shader_enums.c
Jason Ekstrand writes:
> In certain conditions, we have to do bounds-checking in the shader for
> image_load_store. The way this works for image loads is that we do the
> load anyway and then emit a series of slecects, one per component, that
Strictly speaking the load is predicated so it's not
2015-09-15 1:35 GMT+08:00 Emil Velikov :
> From: Matthew Waters
>
> As of version 15 of the EGL_KHR_create_context spec, debug contexts
> are allowed for ES contexts. We should allow creation instead of
> erroring.
>
> While we're here provide a more comprehensive checking for the other two
> fla
Various pieces of code to create compressed textures will first
generate an uncompressed RGBA texture into a temporary buffer,
and then read from that buffer while creating the final compressed
texture in the requested format.
The code reading from the temporary buffer assumes the buffer is
forma
> -Original Message-
> From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On Behalf Of
> Ian Romanick
> Sent: Tuesday, September 15, 2015 3:01 AM
> To: mesa-dev@lists.freedesktop.org
> Cc: Romanick, Ian D
> Subject: [Mesa-dev] [PATCH 15/24] t_dd_dmatmp.h: Indentation and forma
https://bugs.freedesktop.org/show_bug.cgi?id=91889
--- Comment #25 from Albert Freeman ---
Correction: I didn't walk through every call in the application, only util I
realised what was happening with the UI.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are th
https://bugs.freedesktop.org/show_bug.cgi?id=91889
--- Comment #24 from Albert Freeman ---
Yep, you are absolutely right, I just walked through every single gl call in
the application (skipping redundant frames and shader compilation). The entire
UI is drawn outside the apitrace capture and just
Reviewed-by: Eduardo Lima Mitev
On 09/11/2015 05:52 PM, Jason Ekstrand wrote:
> Previously, we were passing the shader around, we were just calling it
> "mem_ctx". However, the nir_shader is (and must be for the purposes of
> mark-and-sweep) the mem_ctx so we might as well pass it around explici
On Tue, Sep 15, 2015 at 3:18 AM, Iago Toral wrote:
> On Sun, 2015-09-13 at 11:51 -0400, Rob Clark wrote:
>> From: Rob Clark
>>
>> Signed-off-by: Rob Clark
>> ---
>> src/glsl/nir/nir_print.c | 73
>> ++--
>> 1 file changed, 59 insertions(+), 14 deleti
On Tue, Sep 15, 2015 at 2:55 AM, Iago Toral wrote:
> On Sun, 2015-09-13 at 11:51 -0400, Rob Clark wrote:
>> From: Rob Clark
>>
>> Signed-off-by: Rob Clark
>> ---
>> src/Makefile.am | 1 +
>> src/glsl/shader_enums.c | 202
>> ++
>> src/g
For consistency and efficiency, the (sub)texture dimension error check
should go before the validation of format, type and internal format.
---
src/mesa/main/teximage.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximag
When validating format+type+internalFormat for texture pixel operations
on GLES3, the effective internal format should be used if the one
specified is an unsized internal format. Page 127, section "3.8 Texturing"
of the GLES 3.0.4 spec says:
"if internalformat is a base internal format, the ef
Hello Ray,
On 12 September 2015 at 20:25, Ray Strode wrote:
> Hi Emil,
>
> Is this still on your radar?
>
Yes it still is. The bugreport mentioned does cover at least three
distinct topics, one of which prodded me to complete a libdrm helper
or two. With those done, and another pair of eyes check
Reviewed-by: Tapani Pälli
On 09/10/2015 04:36 PM, Iago Toral Quiroga wrote:
From: Samuel Iglesias Gonsalvez
Signed-off-by: Samuel Iglesias Gonsalvez
---
src/mesa/main/tests/enum_strings.cpp | 15 +++
1 file changed, 15 insertions(+)
diff --git a/src/mesa/main/tests/enum_strin
1 - 100 of 123 matches
Mail list logo
