On Thu, 2015-07-30 at 16:33 +0300, Francisco Jerez wrote:
> Francisco Jerez writes:
>
> > Iago Toral Quiroga writes:
> >
> >> Previous patches made it so that we do not need to unspill the same vgrf
> >> with every instruction as long as these instructions come right after
> >> the register was
On Thu, 2015-07-30 at 17:14 +0300, Francisco Jerez wrote:
> Francisco Jerez writes:
>
> > Iago Toral Quiroga writes:
> >
> >> When we have code such as this:
> >>
> >> mov vgrf1.0.x:F, vgrf2.:F
> >> mov vgrf3.0.x:F, vgrf1.:F
> >> ...
> >> mov vgrf3.0.x:F, vgrf1.:F
> >>
> >> And vgrf1
On 07/24/2015 08:30 AM, Samuel Iglesias Gonsálvez wrote:
On 23/07/15 08:42, Samuel Iglesias Gonsalvez wrote:
v2:
- Add tessellation shader constants support
Signed-off-by: Samuel Iglesias Gonsalvez
---
src/mesa/main/get.c | 1 +
src/mesa/main/get_hash_params.py | 14 +++
On 2015-07-14 00:46:29, Iago Toral Quiroga wrote:
> ---
> diff --git a/src/glsl/nir/nir_intrinsics.h b/src/glsl/nir/nir_intrinsics.h
> index f264f55..83eeecd 100644
> --- a/src/glsl/nir/nir_intrinsics.h
> +++ b/src/glsl/nir/nir_intrinsics.h
> @@ -176,12 +176,12 @@ LOAD(input, 0, NIR_INTRINSIC_CAN_E
On 07/14/2015 10:46 AM, Iago Toral Quiroga wrote:
From: Samuel Iglesias Gonsalvez
Including TOP_LEVEL_ARRAY_SIZE and TOP_LEVEL_ARRAY_STRIDE queries.
Signed-off-by: Samuel Iglesias Gonsalvez
---
src/glsl/ir_uniform.h| 5 +
src/glsl/link_uniforms.cpp | 17 ++-
src/glsl
On Thu, 2015-07-30 at 16:27 +0300, Francisco Jerez wrote:
> Iago Toral Quiroga writes:
>
> > In theory, GRF array access should have been moved to scratch by the time
> > we got here, so this should never happen. A full piglit run forcing
> > spilling of all registers seems to confirm this. The F
Series Reviewed-by: Jordan Justen
On 2015-07-30 20:16:20, Ben Widawsky wrote:
> This patch adjusts the SKL values to the best known values we have.
>
> v2: Remove HS/DS/CS fields. Adding this makes most sense to add to the
> GEN9_FEATURES macro, however, doing that would require updating BXT val
On Fri, Jul 31, 2015 at 04:37:16PM +1200, Chris Forbes wrote:
> If the indexing expression involves anything modified during the call,
> the fixup code to copy back after the call would use the new values.
>
> This fixes the cases where the first expression node encountered is
> ir_binop_vector_ex
If the indexing expression involves anything modified during the call,
the fixup code to copy back after the call would use the new values.
This fixes the cases where the first expression node encountered is
ir_binop_vector_extract, fixing the piglits:
* shaders@out-parameter-indexing@vs-inout-in
On 31.07.2015 01:29, Marek Olšák wrote:
> From: Marek Olšák
>
> Disabling the FP16 mode didn't help.
>
> If needed, we can use this trick for blits too, but not for scaled blits.
>
> + 4 piglits
> ---
> src/gallium/drivers/radeonsi/si_blit.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 delet
On 31.07.2015 07:06, Zoltan Gilian wrote:
> To circumvent a problem occuring when LINEAR_ALIGNED array mode is
> selected on a TEXTURE_2D RAT.
> This configuration causes MEM_RAT STORE_TYPED to write to incorrect
> locations.
[...]
> @@ -715,10 +716,15 @@ static unsigned r600_choose_tiling(struct
Since we really do not know what may occur in the future, pick a more
conservative value for thread counts until we know better what values are
correct. As far as I can tell, the old values will work fine, but some of the
registers seem to indicate that going even lower is possible and the purpose
This patch adjusts the SKL values to the best known values we have.
v2: Remove HS/DS/CS fields. Adding this makes most sense to add to the
GEN9_FEATURES macro, however, doing that would require updating BXT values, and
Jordan requested I not do that. Conveniently, this request makes a lot of sense
For SKL: These are the production values.
For BXT: These are low estimates to enable platforms.
This patch was originally part of
i965/skl: Add production thread counts and URB size
but was split out at Jordan's request (which I found to be reasonable).
Note on stable inclusion: 10.6 does not ca
seems like one I wrote already..
http://lists.freedesktop.org/archives/mesa-dev/2015-July/088404.html
Dave.
On 31 July 2015 at 12:42, Edward O'Callaghan
wrote:
> Don't trigger assert on some stream emit instructions.
>
> Signed-off-by: Edward O'Callaghan
> ---
> src/gallium/drivers/r600/sb/sb_
Don't trigger assert on some stream emit instructions.
Signed-off-by: Edward O'Callaghan
---
src/gallium/drivers/r600/sb/sb_bc_finalize.cpp | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/gallium/drivers/r600/sb/sb_bc_finalize.cpp
b/src/gallium/drivers/r600/sb/sb_bc_finalize.cpp
index
I believe this assert isn't vaild in generality of possible
stream emit instructions out the sb backend finalizer.
Edward O'Callaghan (1):
r600: Remove assert that not general in some stream instr's
src/gallium/drivers/r600/sb/sb_bc_finalize.cpp | 2 --
1 file changed, 2 deletions(-)
--
2.4.
On 2015-07-29 12:35:24, Ben Widawsky wrote:
> This patch adjusts the SKL values to the best known values we have. It also
> adds
> the missing HS/DS/CS fields. To support this patch I needed to add some
> default
> values to BXT. Those values are just minimal values which we can use for
> enablin
On Thu, Jul 23, 2015 at 4:38 AM, Francisco Jerez wrote:
> Jason Ekstrand writes:
>
>> This all looks correct as far as I can tell. However, I'm very
>> concerned about the number of checks such as
>> has_matching_typed_format() that are built-in to the compiler (via
>> surface_builder) where we
Hello everyone,
We are 1.5 months away from XDC and 20 days away from the proposals
deadline[1]!
If you did not manage to secure funding from your company but still
think you could benefit the community by giving a talk, we encourage you
to send an email to the board of X.Org with your talk
Curro,
What are we still wainting on for the image_load_store extension? I
think I've given you R-B's on all but one or two of the compiler
patches. Is the state setup stuff reviewed? Is there anything else
that needs review?
--Jason
On Thu, Jul 23, 2015 at 6:58 AM, Francisco Jerez wrote:
> Ja
Reviewed-by: Jason Ekstrand
On Thu, Jul 23, 2015 at 10:31 AM, Francisco Jerez wrote:
> v2: Move array coordinate workaround into the surface builder.
> ---
> src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 106
> +++
> 1 file changed, 106 insertions(+)
>
> diff --git a/s
Reviewed-by: Jason Ekstrand
On Thu, Jul 23, 2015 at 10:30 AM, Francisco Jerez wrote:
> Define bitfield packing, unpacking and type conversion operations in
> terms of which the image format conversion code will be implemented.
> These don't directly know about image formats: The packing and
> un
On Thu, Jul 30, 2015 at 3:42 PM, Jason Ekstrand wrote:
> Currently the validation pass only validates that regs_read and
> regs_written are consistent with the sizes of VGRF's. We can add more as
> we find it to be useful.
> ---
> src/mesa/drivers/dri/i965/Makefile.sources| 1 +
> src/mesa/
On 07/30/2015 04:26 PM, Marek Olšák wrote:
From: Marek Olšák
---
src/mesa/state_tracker/st_cb_xformfb.c | 58 ++
src/mesa/state_tracker/st_cb_xformfb.h | 2 +-
src/mesa/state_tracker/st_draw.c | 2 +-
3 files changed, 33 insertions(+), 29 deletions
Currently the validation pass only validates that regs_read and
regs_written are consistent with the sizes of VGRF's. We can add more as
we find it to be useful.
---
src/mesa/drivers/dri/i965/Makefile.sources| 1 +
src/mesa/drivers/dri/i965/brw_fs.cpp | 9
src/mesa/drivers/dri
On 31 July 2015 at 00:26, Marek Olšák wrote:
> From: Marek Olšák
once you rebase onto my last patch, these look fine.
Reviewed-by: Dave Airlie
>
> ---
> src/mesa/state_tracker/st_cb_xformfb.c | 58
> ++
> src/mesa/state_tracker/st_cb_xformfb.h | 2 +-
> src/m
On Thu, Jul 30, 2015 at 2:31 PM, Ben Widawsky
wrote:
> Recommended-by: Kenneth Graunke
> Signed-off-by: Ben Widawsky
> ---
> src/mesa/drivers/dri/i965/Makefile.sources | 1 +
> src/mesa/drivers/dri/i965/gen6_constant_state.c | 189
>
> src/mesa/drivers/dri/i965/
On Tuesday, July 21, 2015 07:54:14 PM Connor Abbott wrote:
> Back when I was getting NIR up and running, I created a bunch of
> functions that dealt with modifying control flow; they allowed you to
> remove control flow nodes (if's, loops, and basic blocks) as well as
> insert newly-created control
To circumvent a problem occuring when LINEAR_ALIGNED array mode is
selected on a TEXTURE_2D RAT.
This configuration causes MEM_RAT STORE_TYPED to write to incorrect
locations.
---
src/gallium/drivers/radeon/r600_texture.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git
The driver may set the pitch of a 2d or 3d mapping.
---
src/gallium/state_trackers/clover/api/transfer.cpp | 13 -
src/gallium/state_trackers/clover/core/resource.hpp | 8
2 files changed, 16 insertions(+), 5 deletions(-)
diff --git a/src/gallium/state_trackers/clover/api/t
Mapping tiled textures requires 2 or 3 dimensional region information.
---
src/gallium/state_trackers/clover/api/transfer.cpp | 33 +++---
1 file changed, 23 insertions(+), 10 deletions(-)
diff --git a/src/gallium/state_trackers/clover/api/transfer.cpp
b/src/gallium/state_tracker
---
src/gallium/state_trackers/clover/core/kernel.cpp | 15 +++
1 file changed, 15 insertions(+)
diff --git a/src/gallium/state_trackers/clover/core/kernel.cpp
b/src/gallium/state_trackers/clover/core/kernel.cpp
index a23cd2b..820a80a 100644
--- a/src/gallium/state_trackers/clover/co
Update:
I no longer thing we should merge this patch. A note in the programming has been
introduced stating that we should not program this state packet without enabling
at least one component.
On Wed, Jul 01, 2015 at 04:06:54PM -0700, Ben Widawsky wrote:
> We don't yet have a use for this state,
Recommended-by: Kenneth Graunke
Signed-off-by: Ben Widawsky
---
src/mesa/drivers/dri/i965/Makefile.sources | 1 +
src/mesa/drivers/dri/i965/gen6_constant_state.c | 189
src/mesa/drivers/dri/i965/gen6_vs_state.c | 88 ---
src/mesa/drivers/dri/i965/ge
This patch replaces the previous patch listed in references.
References: http://patchwork.freedesktop.org/patch/54099/
Recommended-by: Kenneth Graunke
Signed-off-by: Ben Widawsky
---
src/mesa/drivers/dri/i965/gen6_constant_state.c | 104 ++--
1 file changed, 62 insertions(+)
So I'm trying to get these things to line up, especially for nvc0.
Here are the limits exposed by the blob drivers:
http://people.freedesktop.org/~imirkin/glxinfo/glxinfo.html#v=Vendor
and they reflect what the hardware is capable. More specifically, I need
GL_MAX_VERTEX_OUTPUT_COMPONENTS = 128
On Jul 27, 2015 3:39 PM, "Jason Ekstrand" wrote:
>
> On Mon, Jul 27, 2015 at 2:07 PM, Eduardo Lima Mitev
wrote:
> > On 07/25/2015 03:08 AM, Jason Ekstrand wrote:
> >> Alright, I got through it again...
> >>
> >> I asked for a few trivial changes on a few of the patches. With those
> >> fixed, ev
Follow-up to 1b2b0e42ce47bfd1fcb5513ed2c23b9bb7a5a5b8
Signed-off-by: Kai Wasserbäch
---
Please commit this for me, if this is accepted. Thanks.
docs/GL3.txt | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/docs/GL3.txt b/docs/GL3.txt
index 0220f24..8124383 100644
--- a/d
Please, ignore this patch, we have recently sent an updated version to
the mailing list.
The new version is available in:
http://lists.freedesktop.org/archives/mesa-dev/2015-July/090108.html
Regards.
On mar, 2015-07-14 at 09:46 +0200, Iago Toral Quiroga wrote:
> From: Antia Puentes
>
> Commmit
From: Nanley Chery
The ASTC spec was revised as follows:
Revision 2, April 28, 2015 - added CompressedTex{Sub,}Image3D to
commands accepting ASTC format tokens in the New Tokens section [...].
Support only exists in the HDR submode:
Add a second new column "3D Tex." which is empty for
Hi,
I don't know if you will believe it or not, but I have just written a
very similar patch to yours :)
Anyway, yours have come first so have my
Reviewed-by: Bruno Jiménez
Note: I have no permissions in the git repo, you should ask someone to
push this patch for you.
- Bruno
On Thu, 2015-07
Hmm I guess we can't convert it to something else when we don't have a
matching MESA_FORMAT?
Roland
Am 30.07.2015 um 03:48 schrieb Dave Airlie:
> From: Dave Airlie
>
> GL3.3 added GL_ARB_texture_rgb10_a2ui, which specifies
> a lot more things than just rgb10/a2ui.
>
> While playing with ogl co
From: Marek Olšák
Disabling the FP16 mode didn't help.
If needed, we can use this trick for blits too, but not for scaled blits.
+ 4 piglits
---
src/gallium/drivers/radeonsi/si_blit.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeonsi/si_blit.c
On 30 July 2015 at 10:27, Frank Binns wrote:
> This matches similar behaviour for the __DRI_API_OPENGL_CORE bit.
>
> Signed-off-by: Frank Binns
> ---
> src/mesa/drivers/dri/common/dri_util.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/src/mesa/drivers/dri/common/d
On 07/30/2015 07:50 AM, Emil Velikov wrote:
> On 30 July 2015 at 15:22, Emil Velikov wrote:
>> All three of GLX_NV_float_buffer, GLX_EXT_texture_from_pixmap and
>> GLX_MESA_query_renderer have been in glxext.h for a while now.
>>
>> As such we can drop this workaround/hack from the header.
>>
>> S
https://bugs.freedesktop.org/show_bug.cgi?id=91020
--- Comment #5 from Emil Velikov ---
(In reply to George Diamantopoulos from comment #4)
> Created attachment 117106 [details] [review]
> Fix build against EGL implementations that don't support the
> EGL_MESA_screen_surface extension
>
> There
Was sent 2 month ago, still not review/upstream
-Original Message-
From: Predut, Marius
Sent: Thursday, July 30, 2015 7:04 PM
To: mesa-dev@lists.freedesktop.org
Cc: Predut, Marius
Subject: [Mesa-dev][PATCH] i915/aa: fixing anti-aliasing bug for thinnest width
lines
On PNV platform, for
On PNV platform, for 1 pixel line thickness or less,
the general anti-aliasing algorithm gives up, and a garbage line is generated.
Setting a Line Width of 0.0 specifies the rasterization
of the "thinnest" (one-pixel-wide), non-antialiased lines.
Lines rendered with zero Line Width are rasterized u
On 30 July 2015 at 15:22, Emil Velikov wrote:
> All three of GLX_NV_float_buffer, GLX_EXT_texture_from_pixmap and
> GLX_MESA_query_renderer have been in glxext.h for a while now.
>
> As such we can drop this workaround/hack from the header.
>
> Signed-off-by: Emil Velikov
> ---
>
> Not to mention
Patches 1-8 are
Acked-by: Jason Ekstrand
Thanks for cleaning that stuff up.
On Thu, Jul 30, 2015 at 7:14 AM, Ian Romanick wrote:
> All but the last of these patches have been sitting in one tree or
> another for quite some time. All of these files were touched as part of
> other work, but tha
On 07/30/2015 08:22 AM, Emil Velikov wrote:
All three of GLX_NV_float_buffer, GLX_EXT_texture_from_pixmap and
GLX_MESA_query_renderer have been in glxext.h for a while now.
As such we can drop this workaround/hack from the header.
Signed-off-by: Emil Velikov
---
Not to mention that glxext.h h
Ian Romanick writes:
> From: Ian Romanick
>
> Fixes a giant pile of GCC warnings:
>
> builtin_types.cpp:60:1: warning: missing initializer for member
> 'glsl_struct_field::stream' [-Wmissing-field-initializers]
>
> I had to add a default constructor because a non-default constructor
> was added
This wasn't implemented. Piglit tests have yet to be written. Hopefully, they
will just materialize themselves.
Marek
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
From: Marek Olšák
---
src/mesa/drivers/dri/i965/brw_draw.c | 3 ++-
src/mesa/drivers/dri/i965/brw_draw.h | 3 ++-
src/mesa/drivers/dri/i965/brw_meta_fast_clear.c | 2 +-
src/mesa/drivers/dri/i965/brw_primitive_restart.c | 3 ++-
src/mesa/drivers/dri/nouveau/nouvea
From: Marek Olšák
---
src/mesa/state_tracker/st_cb_xformfb.c | 58 ++
src/mesa/state_tracker/st_cb_xformfb.h | 2 +-
src/mesa/state_tracker/st_draw.c | 2 +-
3 files changed, 33 insertions(+), 29 deletions(-)
diff --git a/src/mesa/state_tracker/st_cb_xfor
From: Marek Olšák
---
src/glsl/link_varyings.cpp | 1 +
src/mesa/main/mtypes.h | 5 +
2 files changed, 6 insertions(+)
diff --git a/src/glsl/link_varyings.cpp b/src/glsl/link_varyings.cpp
index 1c52ff3..f7a7b8c 100644
--- a/src/glsl/link_varyings.cpp
+++ b/src/glsl/link_varyings.cpp
@@
From: Ian Romanick
Fixes a giant pile of GCC warnings:
builtin_types.cpp:60:1: warning: missing initializer for member
'glsl_struct_field::stream' [-Wmissing-field-initializers]
I had to add a default constructor because a non-default constructor
was added. Otherwise the only constructor woul
Francisco Jerez writes:
> Iago Toral Quiroga writes:
>
>> When we have code such as this:
>>
>> mov vgrf1.0.x:F, vgrf2.:F
>> mov vgrf3.0.x:F, vgrf1.:F
>> ...
>> mov vgrf3.0.x:F, vgrf1.:F
>>
>> And vgrf1 is chosen for spilling, we can emit this:
>>
>> mov vgrf1.0.x:F, vgrf2.:F
>>
All but the last of these patches have been sitting in one tree or
another for quite some time. All of these files were touched as part of
other work, but that work is stalled a bit. At Ken's suggestion, I'm
sending out the trivial bits just to prune my trees a bit.
_
All three of GLX_NV_float_buffer, GLX_EXT_texture_from_pixmap and
GLX_MESA_query_renderer have been in glxext.h for a while now.
As such we can drop this workaround/hack from the header.
Signed-off-by: Emil Velikov
---
Not to mention that glxext.h honours GLX_GLXEXT_PROTOTYPES and provides
the
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/mesa/drivers/dri/i965/gen6_multisample_state.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/gen6_multisample_state.c
b/src/mesa/drivers/dri/i965/gen6_multisample_state.c
index cf1421e
From: Ian Romanick
It is only ever called from within the same file.
Signed-off-by: Ian Romanick
---
src/mesa/drivers/dri/i965/brw_state.h | 9 -
src/mesa/drivers/dri/i965/gen7_wm_state.c | 2 +-
2 files changed, 1 insertion(+), 10 deletions(-)
diff --git a/src/mesa/drivers/dri/i9
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/mesa/drivers/dri/i965/brw_state.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_state.h
b/src/mesa/drivers/dri/i965/brw_state.h
index 2eff1b5..a99bbbd 100644
--- a/src/mesa/drivers/dri/i965/brw_state.h
++
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/mesa/drivers/dri/i965/brw_draw.c | 102 +--
1 file changed, 51 insertions(+), 51 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_draw.c
b/src/mesa/drivers/dri/i965/brw_draw.c
index b5ef276..9113d0f
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/mesa/drivers/dri/i965/gen7_vs_state.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/gen7_vs_state.c
b/src/mesa/drivers/dri/i965/gen7_vs_state.c
index 4b17d06..00bc6f2 100644
--- a/
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/mesa/drivers/dri/i965/brw_wm.c | 84 +++---
1 file changed, 41 insertions(+), 43 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_wm.c
b/src/mesa/drivers/dri/i965/brw_wm.c
index b590b17..9b90a7c 1006
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/mesa/drivers/dri/i965/brw_draw_upload.c | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_draw_upload.c
b/src/mesa/drivers/dri/i965/brw_draw_upload.c
index c6dd69b..cbfd585 100644
--
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/mesa/drivers/dri/i965/brw_misc_state.c | 49 ++
1 file changed, 23 insertions(+), 26 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_misc_state.c
b/src/mesa/drivers/dri/i965/brw_misc_state.c
index 16b0e
Iago Toral Quiroga writes:
> When we have code such as this:
>
> mov vgrf1.0.x:F, vgrf2.:F
> mov vgrf3.0.x:F, vgrf1.:F
> ...
> mov vgrf3.0.x:F, vgrf1.:F
>
> And vgrf1 is chosen for spilling, we can emit this:
>
> mov vgrf1.0.x:F, vgrf2.:F
> gen4_scratch_write hw_reg0:F, vgrf1.
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 value as suggested by Brian Paul.
This fixes a GL error warning on r200 in Wine.
The GL_ARB_sampler_objects extension does not specify a
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Hi,
Thanks for the quick feedback! I'll resend the patch with the
requested fixes shortly.
Stefan
Am 2015-07-30 um 15:30 schrieb Brian Paul:
> Can you try to shorten the subject line to 75 chars or less? It
> should also start with "mesa:". Perhap
Francisco Jerez writes:
> Iago Toral Quiroga writes:
>
>> Previous patches made it so that we do not need to unspill the same vgrf
>> with every instruction as long as these instructions come right after
>> the register was spilled or unspilled. This means that actually spilling
>> the register
We will need this later on when we implement proper support for precision
qualifiers in the drivers and also to do link time checks for uniforms as
indicated by the spec.
This patch also adds compile-time checks for variables without precision
information (currently, Mesa only checks that a defaul
We will need this to build later patches
---
src/glsl/ast_to_hir.cpp | 71 -
1 file changed, 35 insertions(+), 36 deletions(-)
diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp
index 789b2bc..3ececb7 100644
--- a/src/glsl/ast_to_hir.cpp
Francisco Jerez writes:
> Iago Toral Quiroga writes:
>
>> Previous patches made it so that we do not need to unspill the same vgrf
>> with every instruction as long as these instructions come right after
>> the register was spilled or unspilled. This means that actually spilling
>> the register
On 07/29/2015 09:19 PM, Vinson Lee wrote:
lp_bld_tgsi_soa.c: In function 'lp_emit_immediate_soa':
lp_bld_tgsi_soa.c:3065:18: warning: unused variable 'size' [-Wunused-variable]
const uint size = imm->Immediate.NrTokens - 1;
^
Signed-off-by: Vinson Lee
---
src/galliu
Can you try to shorten the subject line to 75 chars or less? It should
also start with "mesa:". Perhaps:
"mesa: disable texture compare mode error checking for Wine"
On 07/30/2015 07:04 AM, Stefan Dösinger wrote:
Version 2: Return GL_FALSE if ARB_shadow is unsupported instead of pretending
Iago Toral Quiroga writes:
> In theory, GRF array access should have been moved to scratch by the time
> we got here, so this should never happen. A full piglit run forcing
> spilling of all registers seems to confirm this. The FS backend
> does not seem to check for this either.
> ---
> src/mes
Iago Toral Quiroga writes:
> Previous patches made it so that we do not need to unspill the same vgrf
> with every instruction as long as these instructions come right after
> the register was spilled or unspilled. This means that actually spilling
> the register is now cheaper in these scenarios
Iago Toral writes:
> On Thu, 2015-07-30 at 15:58 +0300, Francisco Jerez wrote:
>> Iago Toral Quiroga writes:
>>
>> > ---
>> > src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp | 2 +-
>> > src/mesa/drivers/dri/i965/brw_vec4.cpp| 2 +-
>> > src/mesa/drivers/dri/i965/intel_debug.c
Version 2: Return GL_FALSE if ARB_shadow is unsupported instead of pretending
to store the value as
suggested by Brian Paul.
This fixes a GL error warning on r200 in Wine.
The GL_ARB_sampler_objects extension does not specify a dependency on
GL_ARB_shadow or GL_ARB_depth_texture for this value.
On Thu, 2015-07-30 at 15:58 +0300, Francisco Jerez wrote:
> Iago Toral Quiroga writes:
>
> > ---
> > src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp | 2 +-
> > src/mesa/drivers/dri/i965/brw_vec4.cpp| 2 +-
> > src/mesa/drivers/dri/i965/intel_debug.c | 3 ++-
> > src/mesa
Iago Toral Quiroga writes:
> ---
> src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp | 2 +-
> src/mesa/drivers/dri/i965/brw_vec4.cpp| 2 +-
> src/mesa/drivers/dri/i965/intel_debug.c | 3 ++-
> src/mesa/drivers/dri/i965/intel_debug.h | 5 +++--
> 4 files changed,
With num_direct_uniforms == 0 there's no space allocated in the
param_size array for the one block of direct uniforms -- On the FS
stage this would be a harmless no-op because it would simply re-set
one of the param_size entries allocated for the sampler units to zero,
but on the VS stage it has be
On Thu, 2015-07-30 at 14:21 +0300, Tapani Pälli wrote:
> On 07/30/2015 12:26 PM, Timothy Arceri wrote:
> > On Thu, 2015-07-30 at 11:07 +0300, Tapani Pälli wrote:
> > > On 07/30/2015 08:56 AM, Tapani Pälli wrote:
> > > > On 07/30/2015 08:50 AM, Timothy Arceri wrote:
> > > > > On 30 July 2015 3:27:41
On Thu, 2015-07-30 at 14:16 +0300, Tapani Pälli wrote:
> patches 1,2,3
I'm going to push these so they get as much testing as possible.
>
> and patches for AoA size calculation 11,12,16
>
> Reviewed-by: Tapani Pälli
>
> (will review some more later)
That would be great thanks.
>
> On 07/29
On 07/30/2015 12:26 PM, Timothy Arceri wrote:
On Thu, 2015-07-30 at 11:07 +0300, Tapani Pälli wrote:
On 07/30/2015 08:56 AM, Tapani Pälli wrote:
On 07/30/2015 08:50 AM, Timothy Arceri wrote:
On 30 July 2015 3:27:41 pm AEST, "Tapani Pälli"
wrote:
On 07/30/2015 08:25 AM, Tapani Pälli wrote:
H
patches 1,2,3
and patches for AoA size calculation 11,12,16
Reviewed-by: Tapani Pälli
(will review some more later)
On 07/29/2015 04:56 PM, Timothy Arceri wrote:
If useful the series is in the 'gles8' branch of the repo here:
https://github.com/tarceri/Mesa_arrays_of_arrays.git
Changes in v
On 07/30/2015 12:48 PM, Timothy Arceri wrote:
On Thu, 2015-07-30 at 19:40 +1000, Timothy Arceri wrote:
On Thu, 2015-07-30 at 09:44 +0300, Tapani Pälli wrote:
This fixes a lot of failing tests in:
ES31-CTS.program_interface_query.uniform-types
Signed-off-by: Tapani Pälli
---
src/mesa/mai
It's not a failure, so it can't _fail_ to draw. It should be
interpreted such that the drawing succeeds with the vertex count given
by the transform feedback object, which is 0.
Reviewed-by: Marek Olšák
Marek
On Thu, Jul 30, 2015 at 12:46 PM, Dave Airlie wrote:
> if we get a request to take th
if we get a request to take the count from feedback, but there
is no buffer to take it from, just draw nothing instead of asserting.
This fixes this assert killing the ogl conform, and a piglit
test I've sent.
Signed-off-by: Dave Airlie
---
src/mesa/state_tracker/st_cb_xformfb.c | 6 --
src
Reviewed-by: Samuel Iglesias Gonsálvez
On Wed, 2015-07-29 at 10:53 -0700, Matt Turner wrote:
> Static analysis tools don't like partial object initializations.
> ---
> src/glsl/lower_subroutine.cpp | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/src/glsl/lower_sub
On Wed, 2015-07-29 at 15:21 -0700, Ian Romanick wrote:
> On 07/29/2015 07:01 AM, Samuel Iglesias Gonsalvez wrote:
> > From: Iago Toral Quiroga
> >
> > Currently, we only consider precision qualifiers at compile-time. This patch
> > adds precision information to ir_variable so we can also do link
On Thu, 2015-07-30 at 19:40 +1000, Timothy Arceri wrote:
> On Thu, 2015-07-30 at 09:44 +0300, Tapani Pälli wrote:
> > This fixes a lot of failing tests in:
> >ES31-CTS.program_interface_query.uniform-types
> >
> > Signed-off-by: Tapani Pälli
> > ---
> > src/mesa/main/program_resource.c | 7 +
Update the DRI image interface error codes to reflect the needs of the
EGL_EXT_image_dma_buf_import extension. This means updating the existing error
code documentation and adding a new __DRI_IMAGE_ERROR_BAD_ACCESS error code
so that drivers can correctly reject unsupported pitches and offsets. Hoo
This matches similar behaviour for the __DRI_API_OPENGL_CORE bit.
Signed-off-by: Frank Binns
---
src/mesa/drivers/dri/common/dri_util.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/common/dri_util.c
b/src/mesa/drivers/dri/common/dri_util.c
index 88
Calling eglQuerySurface on a window or pixmap with the EGL_LARGEST_PBUFFER
attribute resulted in the contents of the 'value' parameter being modified.
This is the wrong behaviour according to the EGL spec, which states:
"Querying EGL_LARGEST_PBUFFER for a pbuffer surface returns the
same
On Thu, 2015-07-30 at 09:44 +0300, Tapani Pälli wrote:
> This fixes a lot of failing tests in:
>ES31-CTS.program_interface_query.uniform-types
>
> Signed-off-by: Tapani Pälli
> ---
> src/mesa/main/program_resource.c | 7 +--
> 1 file changed, 5 insertions(+), 2 deletions(-)
>
> diff --g
https://bugs.freedesktop.org/show_bug.cgi?id=91474
--- Comment #3 from Marek Chalupa ---
(In reply to Axel Davy from comment #1)
> Mesa egl does indeed support DRI_PRIME for the wayland backend,
>
> however unless DRI_PRIME is set, it uses the same card that the server.
> I guess either the user
1 - 100 of 111 matches
Mail list logo
