On Wed, Dec 16, 2015 at 3:18 AM, Michel Dänzer wrote:
> On 16.12.2015 06:40, Giuseppe Bilotta wrote:
>> +
>> +Adam Jackson
>> +Adam Jackson
>
> In Adam's case, you put a personal e-mail address first and his
> employer's address last.
D'oh, sorry, failed to check which was the one used in the
On Dec 14, 2015 3:34 PM, "Ian Romanick" wrote:
>
> From: Ian Romanick
>
> nir/nir.h: In function 'nir_src_for_ssa':
> nir/nir.h:552:4: warning: missing initializer for field 'use_link' of
'nir_src'
> [-Wmissing-field-initializers]
> nir_src src = NIR_SRC_INIT;
> ^
> In file included from
Was enabled for i965 and nvc0 by following commits:
c875e3cdd21811ad6669160d59fa39a4526ef872
39f51ec96f00f601b9c4d4e321dacb3af9dc866f
Signed-off-by: Tapani Pälli
---
docs/GL3.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/docs/GL3.txt b/docs/GL3.txt
inde
Patches 1 and 2
Reviewed-by: Tapani Pälli
On 12/16/2015 02:08 AM, Jordan Justen wrote:
Signed-off-by: Jordan Justen
---
src/mesa/main/shaderapi.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/mesa/main/shaderapi.c b/src/mesa/main/shaderapi.c
index ac40891..a732d83 100644
---
Patch makes following changes for interface matching:
- do not try to match builtin variables
- handle swizzle in input name, as example 'a.z' should
match with 'a'
- add matching by location
- check that amount of inputs and outputs matches
These changes make interface matching
On Dec 14, 2015 6:24 AM, "Francisco Jerez" wrote:
>
> Jason Ekstrand writes:
>
> > On Dec 11, 2015 5:44 AM, "Francisco Jerez"
wrote:
> >>
> >> Jason Ekstrand writes:
> >>
> >> > On Dec 10, 2015 6:58 AM, "Francisco Jerez"
> > wrote:
> >> >>
> >> >> Jason Ekstrand writes:
> >> >>
> >> >> > We a
From: Roland Scheidegger
mesa/st doesn't give us a useful swizzle when stencil texturing. Moreover,
it's not even obvious what the swizzle actually should be - the channel which
is used for the fetch (Y) is not the same as the one which must be used for
the border component (X), which is due to a
On Mon, Dec 7, 2015 at 10:50 AM, Matt Turner wrote:
> ---
> With this, we generate the same number of Mesa IR instructions before
> and after my series. all() is the same as well.
Maybe Ian could have a look?
___
mesa-dev mailing list
mesa-dev@lists.fre
Hi,
looking at some piglit failures, I was wondering what is actually the
correct thing to do with stencil texturing. What do you put in the
missing channels?
The GL spec seems to say depth texture mode is only applicable to depth
textures, so what it is then? It looks like nvidia is returning the
On Dec 15, 2015 12:30 AM, "Abdiel Janulgue"
wrote:
>
>
>
> On 12/10/2015 06:23 AM, Jason Ekstrand wrote:
> > While we're at it, we also add support for the possibility that the
> > indirect is, in fact, a constant. This shouldn't happen in the common
case
> > (if it does, that means NIR failed to
On Dec 15, 2015 6:19 PM, "Ilia Mirkin" wrote:
>
>
> On Dec 15, 2015 8:59 PM, "Ian Romanick" wrote:
> >
> > On 12/15/2015 05:08 PM, Ilia Mirkin wrote:
> > > On Tue, Dec 15, 2015 at 7:59 PM, Ian Romanick
wrote:
> > >> On 12/15/2015 04:40 PM, Ilia Mirkin wrote:
> > >>> Hardly a complete review, but
Am 16.12.2015 um 00:05 schrieb Miklós Máté:
> Hi,
>
> This series aims to improve the looks of Star Wars: Knights of the
> Old Republic (via Wine), but features some additional cleanup as
> well. The main component of the series is the implementation of
> GL_ATI_fragment_shader for all Gallium dri
On Dec 15, 2015 8:59 PM, "Ian Romanick" wrote:
>
> On 12/15/2015 05:08 PM, Ilia Mirkin wrote:
> > On Tue, Dec 15, 2015 at 7:59 PM, Ian Romanick
wrote:
> >> On 12/15/2015 04:40 PM, Ilia Mirkin wrote:
> >>> Hardly a complete review, but a handful of comments:
> >>>
> >>> On Tue, Dec 15, 2015 at 6:0
On 16.12.2015 06:40, Giuseppe Bilotta wrote:
> This adds a first tentative .mailmap file, to canonicize contributor
> name/emails in shortlogs and other statistical endeavours.
>
> There's a couple of root and richard entries which I don't know who
> they belong to, and hopefully not too many over
On 12/15/2015 05:08 PM, Ilia Mirkin wrote:
> On Tue, Dec 15, 2015 at 7:59 PM, Ian Romanick wrote:
>> On 12/15/2015 04:40 PM, Ilia Mirkin wrote:
>>> Hardly a complete review, but a handful of comments:
>>>
>>> On Tue, Dec 15, 2015 at 6:05 PM, Miklós Máté wrote:
---
src/mesa/Makefile.sou
On 12/15/2015 05:01 PM, Jordan Justen wrote:
> On 2015-12-15 16:50:39, Ian Romanick wrote:
>> On 12/15/2015 04:08 PM, Jordan Justen wrote:
>>> The OpenGL ARB_compute_shader extension specfication requires at least
>>> 1024 for GL_MAX_COMPUTE_WORK_GROUP_INVOCATIONS, whereas OpenGLES 3.1
>>> only req
This patch is
Reviewed-by: Ian Romanick
Assuming there are no objections, I'll push this in 24 hours.
On 12/15/2015 03:05 PM, Miklós Máté wrote:
> ---
> src/mesa/main/mtypes.h| 2 --
> src/mesa/swrast/s_atifragshader.c | 2 ++
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
>
This patch is
Reviewed-by: Ian Romanick
Assuming there are no objections, I'll push this in 24 hours.
On 12/15/2015 03:05 PM, Miklós Máté wrote:
> ---
> src/mesa/main/atifragshader.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/src/mesa/main/atifragshader.c b/src/mesa/main/atif
Yes... that matches the GL_ARB_fragment_program spec. This patch is
Reviewed-by: Ian Romanick
Assuming there are no objections, I'll push this in 24 hours.
On 12/15/2015 03:05 PM, Miklós Máté wrote:
> ---
> src/mesa/program/prog_statevars.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion
This patch is
Reviewed-by: Ian Romanick
Cc: "11.0 11.1"
Assuming there are no objections, I'll push this in 24 hours.
On 12/15/2015 03:05 PM, Miklós Máté wrote:
> ---
> src/mesa/main/atifragshader.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/mesa/main/atifra
On Sat, Nov 28, 2015 at 8:45 AM, Emil Velikov wrote:
> On 27 November 2015 at 20:45, Jason Ekstrand wrote:
>> On Nov 27, 2015 11:26 AM, "Matt Turner" wrote:
>>> On Fri, Nov 27, 2015 at 6:50 AM, Emil Velikov
>>> wrote:
>>> > On 25 November 2015 at 22:01, Matt Turner wrote:
>>> >> On Wed, Nov 25
On 2015-12-15 17:00:55, Ian Romanick wrote:
> Doesn't this make patch 3 irrelevant? FWIW, I like this better.
This change only updates the way we program some constants. It is for
a local stage_exists array, which we then use later in the same
function when programming context constants.
For exa
On Tue, Dec 15, 2015 at 7:59 PM, Ian Romanick wrote:
> On 12/15/2015 04:40 PM, Ilia Mirkin wrote:
>> Hardly a complete review, but a handful of comments:
>>
>> On Tue, Dec 15, 2015 at 6:05 PM, Miklós Máté wrote:
>>> ---
>>> src/mesa/Makefile.sources | 1 +
>>> src/mesa/state_tr
On 2015-12-15 16:50:39, Ian Romanick wrote:
> On 12/15/2015 04:08 PM, Jordan Justen wrote:
> > The OpenGL ARB_compute_shader extension specfication requires at least
> > 1024 for GL_MAX_COMPUTE_WORK_GROUP_INVOCATIONS, whereas OpenGLES 3.1
> > only required 128.
>
> Does this mean that extensions->
Doesn't this make patch 3 irrelevant? FWIW, I like this better.
On 12/15/2015 04:08 PM, Jordan Justen wrote:
> Previously we were checking the desktop OpenGL ARB_compute_shader
> requirements, but for OpenGLES 3.1, the requirements are lower.
>
> Signed-off-by: Jordan Justen
> Cc: Marta Lofsted
On 12/15/2015 04:40 PM, Ilia Mirkin wrote:
> Hardly a complete review, but a handful of comments:
>
> On Tue, Dec 15, 2015 at 6:05 PM, Miklós Máté wrote:
>> ---
>> src/mesa/Makefile.sources | 1 +
>> src/mesa/state_tracker/st_atifs_to_tgsi.c | 798
>> ++
This allows specifying a GL profile and version so one can get a core-
profile context.
---
docs/relnotes/11.2.0.html| 2 +
include/GL/osmesa.h | 45 -
src/mesa/drivers/osmesa/osmesa.c | 104 ++-
3 files changed, 148 inser
As with the previous commit, except for gallium.
---
src/gallium/state_trackers/osmesa/osmesa.c | 96 +-
1 file changed, 93 insertions(+), 3 deletions(-)
diff --git a/src/gallium/state_trackers/osmesa/osmesa.c
b/src/gallium/state_trackers/osmesa/osmesa.c
index 0f27ba8
---
src/gallium/drivers/svga/svga_state.c | 4
1 file changed, 4 insertions(+)
diff --git a/src/gallium/drivers/svga/svga_state.c
b/src/gallium/drivers/svga/svga_state.c
index 722b369..4479a27 100644
--- a/src/gallium/drivers/svga/svga_state.c
+++ b/src/gallium/drivers/svga/svga_state.c
@@
On 12/15/2015 04:08 PM, Jordan Justen wrote:
> The OpenGL ARB_compute_shader extension specfication requires at least
> 1024 for GL_MAX_COMPUTE_WORK_GROUP_INVOCATIONS, whereas OpenGLES 3.1
> only required 128.
Does this mean that extensions->ARB_compute_shader is not set? I'm a
little bit nervous
Hardly a complete review, but a handful of comments:
On Tue, Dec 15, 2015 at 6:05 PM, Miklós Máté wrote:
> ---
> src/mesa/Makefile.sources | 1 +
> src/mesa/state_tracker/st_atifs_to_tgsi.c | 798
> ++
> src/mesa/state_tracker/st_atifs_to_tgsi.h |
Previously we were checking the desktop OpenGL ARB_compute_shader
requirements, but for OpenGLES 3.1, the requirements are lower.
Signed-off-by: Jordan Justen
Cc: Marta Lofstedt
---
src/mesa/drivers/dri/i965/brw_context.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/s
Signed-off-by: Jordan Justen
---
src/mesa/main/shaderapi.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/mesa/main/shaderapi.c b/src/mesa/main/shaderapi.c
index ac40891..a732d83 100644
--- a/src/mesa/main/shaderapi.c
+++ b/src/mesa/main/shaderapi.c
@@ -1514,6 +1514,8 @@ void GLAPIE
OpenGLES 3.1 cannot be enabled for gen 7 (Ivy Bridge, Haswell) since
they are still missing ARB_stencil_texturing.
Signed-off-by: Jordan Justen
Cc: Ian Romanick
Cc: Marta Lofstedt
---
src/mesa/drivers/dri/i965/intel_screen.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/mesa/dri
Previous OpenGLES 3.1 testing had been done when ARB_compute_shader
was overridden to enabled.
Signed-off-by: Jordan Justen
Cc: Marta Lofstedt
---
src/mesa/main/shaderapi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/main/shaderapi.c b/src/mesa/main/shaderapi.c
The OpenGL ARB_compute_shader extension specfication requires at least
1024 for GL_MAX_COMPUTE_WORK_GROUP_INVOCATIONS, whereas OpenGLES 3.1
only required 128.
Signed-off-by: Jordan Justen
Cc: Ian Romanick
Cc: Marta Lofstedt
---
src/mesa/main/version.c | 9 ++---
1 file changed, 6 insertion
git://people.freedesktop.org/~jljusten/mesa es31-gen8-v1
With this series, gen8+ should be able to create an OpenGLES 3.1
context without any environment variable overrides.
Jordan Justen (5):
main: Add MESA_VERBOSE=api for LinkProgram & UseProgram
main: Allow compute shaders to be compiled w
---
src/mesa/Makefile.sources | 1 +
src/mesa/state_tracker/st_atifs_to_tgsi.c | 798 ++
src/mesa/state_tracker/st_atifs_to_tgsi.h | 49 ++
src/mesa/state_tracker/st_atom_constbuf.c | 14 +
src/mesa/state_tracker/st_cb_drawpixels.c | 1 +
src/mesa/
the state tracker will use it
---
src/mesa/drivers/common/driverfuncs.c | 3 +++
src/mesa/main/atifragshader.c | 13 -
src/mesa/main/dd.h| 6 +-
src/mesa/main/mtypes.h| 1 +
src/mesa/main/state.c | 14 +-
5
---
src/mesa/main/atifragshader.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/main/atifragshader.c b/src/mesa/main/atifragshader.c
index 935ba05..3ddc51d 100644
--- a/src/mesa/main/atifragshader.c
+++ b/src/mesa/main/atifragshader.c
@@ -293,7 +293,7 @@ _mesa_Delete
---
src/mesa/main/atifragshader.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/mesa/main/atifragshader.c b/src/mesa/main/atifragshader.c
index d1c07c5..8b19a35 100644
--- a/src/mesa/main/atifragshader.c
+++ b/src/mesa/main/atifragshader.c
@@ -349,6 +349,9 @@ _mesa_BeginFragmentShader
---
src/mesa/state_tracker/st_extensions.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/state_tracker/st_extensions.c
b/src/mesa/state_tracker/st_extensions.c
index d97dfde..45ceae1 100644
--- a/src/mesa/state_tracker/st_extensions.c
+++ b/src/mesa/state_tracker/st_extensions.c
@@
---
src/mesa/state_tracker/st_atom_sampler.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/mesa/state_tracker/st_atom_sampler.c
b/src/mesa/state_tracker/st_atom_sampler.c
index 4252c27..7d3d8e7 100644
--- a/src/mesa/state_tracker/st_atom_sampler.c
+++ b/src/mesa/sta
this works with radeonsi, but crashes with llvmpipe
---
src/mesa/main/context.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/src/mesa/main/context.c b/src/mesa/main/context.c
index 888c461..dcaf524 100644
--- a/src/mesa/main/context.c
+++ b/src/mesa/main/context.c
---
src/mesa/program/prog_statevars.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/program/prog_statevars.c
b/src/mesa/program/prog_statevars.c
index bdb335e..12490d0 100644
--- a/src/mesa/program/prog_statevars.c
+++ b/src/mesa/program/prog_statevars.c
@@ -474,7 +
---
src/mesa/main/mtypes.h| 2 --
src/mesa/swrast/s_atifragshader.c | 2 ++
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h
index 5c71ac4..99e7912 100644
--- a/src/mesa/main/mtypes.h
+++ b/src/mesa/main/mtypes.h
@@ -2278,8
Hi,
This series aims to improve the looks of Star Wars: Knights of the Old Republic
(via Wine), but features some additional cleanup as well. The main component of
the series is the implementation of GL_ATI_fragment_shader for all Gallium
drivers (though I could only test it with radeonsi, llvm
It was already done in get_mesa_program()
---
src/mesa/program/ir_to_mesa.cpp | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/mesa/program/ir_to_mesa.cpp b/src/mesa/program/ir_to_mesa.cpp
index 8f58f3e..a28cf97 100644
--- a/src/mesa/program/ir_to_mesa.cpp
+++ b/src/mesa/program/ir_to_mesa
Apitrace showed this call to be 5ms (9 times per frame),
but in reality it's about 500us. This shortcut makes it 20us.
---
src/mesa/main/teximage.c | 29 +
1 file changed, 29 insertions(+)
diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c
index ab60a2f..
On 12/15/2015 12:28 AM, Kristian Høgsberg Kristensen wrote:
> We can inspect VS prog_data for iterations i > 0, and only flag
> BRW_NEW_VERTICES when one of our system values change.
>
> This change also flags BRW_NEW_VERTICES in one case we were missing
> before: if we're doing an indirect draw,
On 12/15/2015 12:28 AM, Kristian Høgsberg Kristensen wrote:
> This optimizes a + b - b to just a. Modest shader-db results (BDW):
>
> total instructions in shared programs: 7842452 -> 7841862 (-0.01%)
> instructions in affected programs: 61938 -> 61348 (-0.95%)
> total loops in shared pr
On 12/15/2015 12:28 AM, Kristian Høgsberg Kristensen wrote:
> We have to break open a new vec4 for gl_DrawIDARB. We've used up all
> space in the vec4 we use for SGVS and gl_DrawIDARB has to come from its
> own separate vertex buffer anyway. This is because we point the vb for
> base vertex and ba
This patch is really doing two different things. It changes the
existing SYSTEM_VALUE_BASE_VERTEX to be independent from
SYSTEM_VALUE_VERTEX_ID_ZERO. It also adds SYSTEM_VALUE_BASE_INSTANCE
support.
I was going to let that go, but because the two things happened in one
patch, I overlooked the ex
On 12/15/2015 11:48 AM, Anuj Phogat wrote:
> On Tue, Dec 15, 2015 at 12:28 AM, Kristian Høgsberg Kristensen
> wrote:
>> We already have gl_BaseVertexARB in the .x component of the SGVS vec4
>> and plug gl_BaseInstanceARB into the last free component (.y).
>> ---
>> src/mesa/drivers/dri/i965/brw_c
Only compile-tested.
Cc: Eric Anholt
---
src/gallium/auxiliary/nir/tgsi_to_nir.c | 10 +-
1 file changed, 1 insertion(+), 9 deletions(-)
diff --git a/src/gallium/auxiliary/nir/tgsi_to_nir.c
b/src/gallium/auxiliary/nir/tgsi_to_nir.c
index 5def6d3..122e87b 100644
--- a/src/gallium/auxili
On 12/15/2015 12:28 AM, Kristian Høgsberg Kristensen wrote:
> fs_visitor::emit_vs_system_value() looks like it's trying to handle
> SYSTEM_VALUE_VERTEX_ID, but we should never see that value in the
> backend.
> ---
> src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 1 +
> 1 file changed, 1 insertion
On 12/15/2015 12:28 AM, Kristian Høgsberg Kristensen wrote:
> ---
> src/glsl/builtin_variables.cpp | 5 +
> src/glsl/glsl_parser_extras.cpp | 1 +
> src/glsl/glsl_parser_extras.h | 2 ++
> src/glsl/nir/nir.c | 8
> src/glsl/nir/nir_intrinsics.h
Jason Ekstrand writes:
> On Tue, Dec 15, 2015 at 12:26 PM, Eric Anholt wrote:
>> Jason Ekstrand writes:
>>
>>> The one user of this (i965) only ever calls it while in SSA form.
>>
>> This series is:
>>
>> Reviewed-by: Eric Anholt
>
> Thanks!
>
> Did you happen to run it on something that actua
This adds a first tentative .mailmap file, to canonicize contributor
name/emails in shortlogs and other statistical endeavours.
There's a couple of root and richard entries which I don't know who
they belong to, and hopefully not too many overeager merges.
Signed-off-by: Giuseppe Bilotta
---
.m
On Tue, Dec 15, 2015 at 10:22 PM, Kenneth Graunke wrote:
> On Tuesday, December 15, 2015 02:23:07 PM Giuseppe Bilotta wrote:
>> The only problem with these numbers is actually the lack of a .mailmap
>> to normalize contributor name/emails, which obviously skews the
>> results a little bit towards
On 15.12.2015 16:22, Kenneth Graunke wrote:
On Tuesday, December 15, 2015 02:23:07 PM Giuseppe Bilotta wrote:
The only problem with these numbers is actually the lack of a .mailmap
to normalize contributor name/emails, which obviously skews the
results a little bit towards the lower end. I don't
On Tuesday, December 15, 2015 02:23:07 PM Giuseppe Bilotta wrote:
> The only problem with these numbers is actually the lack of a .mailmap
> to normalize contributor name/emails, which obviously skews the
> results a little bit towards the lower end. I don't suppose someone
> has a .mailmap for Mes
On Tue, 2015-12-15 at 14:32 +0200, Tapani Pälli wrote:
> On 12/15/2015 01:25 AM, Timothy Arceri wrote:
> > On Wed, 2015-12-09 at 00:17 +1100, Timothy Arceri wrote:
> > > In core profile even if re-linking fails rendering shouldn't fail
> > > as
> > > the
> > > previous succesfully linked program wi
On Tue, Dec 15, 2015 at 12:26 PM, Eric Anholt wrote:
> Jason Ekstrand writes:
>
>> The one user of this (i965) only ever calls it while in SSA form.
>
> This series is:
>
> Reviewed-by: Eric Anholt
Thanks!
Did you happen to run it on something that actually uses clip plane
lowering? I'd like
Jason Ekstrand writes:
> The one user of this (i965) only ever calls it while in SSA form.
This series is:
Reviewed-by: Eric Anholt
signature.asc
Description: PGP signature
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freed
On Tue, Dec 15, 2015 at 12:28 AM, Kristian Høgsberg Kristensen
wrote:
> We already have gl_BaseVertexARB in the .x component of the SGVS vec4
> and plug gl_BaseInstanceARB into the last free component (.y).
> ---
> src/mesa/drivers/dri/i965/brw_compiler.h | 2 ++
> src/mesa/drivers/dri/
On Tue, Dec 15, 2015 at 12:28 AM, Kristian Høgsberg Kristensen
wrote:
> The drivers will need this for passing in gl_DrawIDARB. For indirect
> multidraw calls, we get the prim array and prim[i].draw_id == i and is
> redundant. But for non-indirect calls, we get one primitive at a time
> and need t
On Tue, Dec 15, 2015 at 12:28 AM, Kristian Høgsberg Kristensen
wrote:
> ---
> src/glsl/builtin_variables.cpp | 5 +
> src/glsl/glsl_parser_extras.cpp | 1 +
> src/glsl/glsl_parser_extras.h | 2 ++
> src/glsl/nir/nir.c | 8
> src/glsl/nir/nir_intrin
On Tue, Dec 15, 2015 at 12:28 AM, Kristian Høgsberg Kristensen
wrote:
> fs_visitor::emit_vs_system_value() looks like it's trying to handle
> SYSTEM_VALUE_VERTEX_ID, but we should never see that value in the
> backend.
> ---
> src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 1 +
> 1 file changed,
+ mesa-dev of course :)
Thx Sean
On 15 December 2015 at 17:46, Sean V Kelley wrote:
>
> > On Dec 15, 2015, at 1:01 AM, Julien Isorce
> wrote:
> >
> > "image" is not ready yet since it will be set at
> > the end of the function by: *image = *img;
> >
> > Signed-off-by: Julien Isorce
> > ---
>
Reviewed-by: Brian Paul
On 12/15/2015 10:06 AM, srol...@vmware.com wrote:
From: Roland Scheidegger
We just ignored them altogether. While this feature is rather old-fashioned
supporting it is actually rather trivial.
This fixes the associated piglit tests (2 gl-1.0-edgeflag, 2 gl-2.0-edgefl
Jason Ekstrand writes:
> On Dec 15, 2015 3:52 AM, "Iago Toral Quiroga" wrote:
>>
>> We want to make sure that the driver does not disable the FS unit if
>> the shader code only has SSBO writes (i.e. no color or depth output).
>>
>> We could go a step further and check if the shader storage is ac
Ah! I had just also discovered this issue yesterday in some related
work but I didn't get the chance to try the CTS yet! :)
For the subject I had: "Gen 8 requires 64 byte alignment for push
constant data"
On 2015-12-15 03:55:15, Iago Toral Quiroga wrote:
> Page 502 of the Command Reference Broadw
On 12/14/2015 07:36 PM, Jason Ekstrand wrote:
> On Mon, Dec 14, 2015 at 5:12 PM, Ian Romanick wrote:
>> On 12/14/2015 04:39 PM, Ilia Mirkin wrote:
>>> On Mon, Dec 14, 2015 at 7:28 PM, Ian Romanick wrote:
On 12/14/2015 03:38 PM, Ilia Mirkin wrote:
> It's a pretty standard feature of compi
On Tue, Dec 15, 2015 at 9:30 AM, Francisco Jerez wrote:
> Jason Ekstrand writes:
>
>> On Dec 15, 2015 3:52 AM, "Iago Toral Quiroga" wrote:
>>>
>>> We want to make sure that the driver does not disable the FS unit if
>>> the shader code only has SSBO writes (i.e. no color or depth output).
>>>
>>
https://bugs.freedesktop.org/show_bug.cgi?id=92570
--- Comment #3 from Andy Furniss ---
(In reply to Andy Furniss from comment #2)
> If so why not output nv16 or something else 10 bit?
lol at me re-reading this and remembering that nv16 is 8 bit 422.
--
You are receiving this mail because:
Yo
From: Roland Scheidegger
We just ignored them altogether. While this feature is rather old-fashioned
supporting it is actually rather trivial.
This fixes the associated piglit tests (2 gl-1.0-edgeflag, 2 gl-2.0-edgeflag
and all (7) of point-vertex-id).
v2: comment fixes, and make the use of the
Am 15.12.2015 um 17:25 schrieb Brian Paul:
> On 12/14/2015 08:38 PM, srol...@vmware.com wrote:
>> From: Roland Scheidegger
>>
>> We just ignored them altogether. While this feature is rather
>> old-fashioned
>> supporting it is actually rather trivial.
>> This fixes the associated piglit tests (2
On 12/14/2015 08:38 PM, srol...@vmware.com wrote:
From: Roland Scheidegger
We just ignored them altogether. While this feature is rather old-fashioned
supporting it is actually rather trivial.
This fixes the associated piglit tests (2 gl-1.0-edgeflag, 2 gl-2.0-edgeflag
and all (7) of point-vert
On Dec 15, 2015 3:52 AM, "Iago Toral Quiroga" wrote:
>
> We want to make sure that the driver does not disable the FS unit if
> the shader code only has SSBO writes (i.e. no color or depth output).
>
> We could go a step further and check if the shader storage is actually
> used for writing, but d
Hello all,
Mesa 11.1.0 is now available.
I would like to thank all the the developers, testers and everyone for
helping shape this release.
Here are some of the highlights:
- New hardware enablement/support
- a305, freedreno
- stoney, radeonsi
- New drivers
- virgl gallium driver for the
opt_vector_float() transforms several scalar MOV operations to a single
vectorial MOV.
This is done when those MOV covers all the components of the destination
register. So something like:
mov vgrf3.0.xy:D, 0D
mov vgrf3.0.w:D, 1065353216D
mov vgrf3.0.z:D, 0D
is transformed in:
mov vgrf3.0:F, [0
While working on related issue, found out that previous patch (and original
version) were applying incorrectly opt_vector_float in some cases.
Specifically, for this piece of code:
cmp.nz.f0.0 null:F, vgrf6.xyzz:F, vgrf17.xyzz:F
mov vgrf2.0.x:D, 0D
(+f0.0.any4h) mov vgrf2.0.x:D, -1D
mov vgrf2.0.y
Hello all,
when Steam first announced they'd give all present and future games
free to all Mesa contributors with at least 25 commits[1], I was
curious to see how many people would be affected by this choice, so I
ran some statistics on the number of committers (and contributions by
committer) on
Thanks Iago!
This patch does not only fix the ssbo test mentioned below, but a lot of other
GLES 3.1 CTS tests.
> -Original Message-
> From: Iago Toral Quiroga [mailto:ito...@igalia.com]
> Sent: Tuesday, December 15, 2015 12:55 PM
> To: mesa-dev@lists.freedesktop.org
> Cc: Lofstedt, Mart
On 12/15/2015 01:25 AM, Timothy Arceri wrote:
On Wed, 2015-12-09 at 00:17 +1100, Timothy Arceri wrote:
In core profile even if re-linking fails rendering shouldn't fail as
the
previous succesfully linked program will still be available. It also
shouldn't be possible to have an unlinked program a
Yep, I remember when and why this was done for atomic counters.
Patches 1 and 2 are
Reviewed-by: Tapani Pälli
On 12/15/2015 01:51 PM, Iago Toral Quiroga wrote:
Some drivers can disable the FS unit if there is nothing in the shader code
that writes to an output (i.e. color, depth, etc). For dri
Page 502 of the Command Reference Broadwell PRM says that CURBE Total
Data Length must be 64-bit aligned.
Fixes the following CTS tests:
ES31-CTS.shader_storage_buffer_object.basic-atomic-case1-cs
ES31-CTS.shader_storage_buffer_object.basic-operations-case1-cs
ES31-CTS.shader_storage_buffer_object
Some drivers can disable the FS unit if there is nothing in the shader code
that writes to an output (i.e. color, depth, etc). For drivers that check
for these things, this helper function is useful to avoid that optimization
in the case that the shader has shader storage space assigned (since it c
We want to make sure that the driver does not disable the FS unit if
the shader code only has SSBO writes (i.e. no color or depth output).
We could go a step further and check if the shader storage is actually
used for writing, but does not seem worth the trouble. Also, we do the
same thing for at
From: Samuel Iglesias Gonsálvez
Commit 5bb5eeea fix a bug indicating that the surfaces should have the
API buffer size. Hovewer it picked the wrong value.
This patch adds a new variable, which takes into account
glBindBufferRange() values. Fixes the following regressions:
ES31-CTS.shader_storag
On Tue, 2015-12-15 at 12:51 +0200, Tapani Pälli wrote:
> On 12/15/2015 10:56 AM, Timothy Arceri wrote:
> > On Tue, 2015-12-15 at 07:58 +0200, Tapani Pälli wrote:
> > > On 12/15/2015 03:31 AM, Timothy Arceri wrote:
> > > > On Mon, 2015-12-14 at 10:29 +0200, Tapani Pälli wrote:
> > > > > Patch makes
On 15 December 2015 at 10:30, Christian König wrote:
> On 15.12.2015 11:11, Emil Velikov wrote:
>>
>> On 11 December 2015 at 12:33, Christian König
>> wrote:
>>>
>>> From: Christian König
>>>
>>> Avoid referencing NULL pointers.
>>>
>> Lacking any prior knowledge of the sequential patches, I'm a
On 12/15/2015 10:56 AM, Timothy Arceri wrote:
On Tue, 2015-12-15 at 07:58 +0200, Tapani Pälli wrote:
On 12/15/2015 03:31 AM, Timothy Arceri wrote:
On Mon, 2015-12-14 at 10:29 +0200, Tapani Pälli wrote:
Patch makes following changes for interface matching:
- do not try to match builtin va
On 15.12.2015 11:11, Emil Velikov wrote:
On 11 December 2015 at 12:33, Christian König wrote:
From: Christian König
Avoid referencing NULL pointers.
Lacking any prior knowledge of the sequential patches, I'm afraid this
commit message doesn't make any sense. How about "Will be used in the
f
On 15.12.2015 11:08, Emil Velikov wrote:
On 11 December 2015 at 12:33, Christian König wrote:
From: Christian König
Signed-off-by: Christian König
---
src/gallium/state_trackers/va/picture.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/gallium/state_trackers/va/picture.c
b/src/
On 11 December 2015 at 12:33, Christian König wrote:
> From: Christian König
>
> Tested with MPV.
>
Since this patch does not add WEAVE support (since it's used by the
state tracker by default) can we add a small note about it. Any shape
or form will be great.
> Signed-off-by: Christian König
>
On 15.12.2015 11:06, Emil Velikov wrote:
On 11 December 2015 at 12:33, Christian König wrote:
From: Christian König
Signed-off-by: Christian König
---
src/gallium/state_trackers/va/surface.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/gallium/state_trac
On 11 December 2015 at 12:33, Christian König wrote:
> +static void vlVaBoxAdjust(struct pipe_video_buffer *buf, unsigned idx,
> + struct pipe_box *box)
> +{
> + if (buf->interlaced) {
> + box->y /= 2;
> + box->height /= 2;
> + }
> + if (idx > buf->interla
On 11 December 2015 at 12:33, Christian König wrote:
> From: Christian König
>
> Avoid referencing NULL pointers.
>
Lacking any prior knowledge of the sequential patches, I'm afraid this
commit message doesn't make any sense. How about "Will be used in the
follow up patches" or anything alike ?
1 - 100 of 123 matches
Mail list logo
