Add a low-level library for using the BLT engine from the etnaviv
driver.
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/gallium/drivers/etnaviv/Makefile.sources | 2 +
src/gallium/drivers/etnaviv/etnaviv_blt.c| 219 +++
src/gallium/d
Add an implemenation of key clear_blit functions using the BLT engine
that replaced the RS on GC7000.
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/gallium/drivers/etnaviv/Makefile.sources | 1 +
src/gallium/drivers/etnaviv/etnaviv_clear_blit.c | 7 +
The HALTI level is an indication of the gross architecture of the GPU.
It determines for significant part what feature level the GPU has, what
state (especially frontend state) is there, and where it is located.
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/gallium/d
When the BLT is involved as source or target, add an extra BLT
enable/disable sequence around the sync sequence.
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/gallium/drivers/etnaviv/etnaviv_emit.c | 15 ++-
1 file changed, 14 insertions(+), 1 deletion(-)
Prepare for BLT-based blitting path by moving RS-based
blitting to its own implementation file.
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/gallium/drivers/etnaviv/Makefile.sources | 1 +
src/gallium/drivers/etnaviv/etnaviv_clear_blit.c
The blob does this, as DRAW_INSTANCED can replace fully all the other
draw commands - the other path is only there for compatibility and
will go away (or at least rot to become buggy due to dis-use) in newer
hardware.
Preparation for GC7000 support.
Signed-off-by: Wladimir J. van der Laan <
This is used by HALTI2+ (GC3000+) when drawing with DRAW_INSTANCED.
It is also necessary when switching between integer and floating point
vertex element formats.
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/gallium/drivers/etnaviv/etnaviv_emit.c | 5 +
src/g
Updated as of etnav_viv commit 8255e4b.
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/gallium/drivers/etnaviv/etnaviv_clear_blit.c | 2 +-
src/gallium/drivers/etnaviv/etnaviv_format.c | 9 +-
src/gallium/drivers/etnaviv/etnaviv_state.c | 2 +-
src/g
. If
you really want to see etnaviv_texture_desc.c in its current state, it's here:
https://github.com/laanwj/mesa/blob/gc7000/src/gallium/drivers/etnaviv/etnaviv_texture_desc.c
Wladimir J. van der Laan (19):
etnaviv: Update from rnndb
etnaviv: Const-correctness etnaviv_emit.h
etnaviv: Put HALTI
The relocation structure is never changed by submitting it.
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/gallium/drivers/etnaviv/etnaviv_emit.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/etnaviv/etnaviv_emit.h
b/src/g
On Mon, Oct 30, 2017 at 02:21:07PM +0100, Lucas Stach wrote:
> I would prefer the shorter expression (!cb->buffer && !cb->user_buffer)
> here, which seems more consistent with the rest of the codebase.
Well, I mimiced the style from other assertion in the function which compares
explicitly with
On Thu, Oct 26, 2017 at 04:16:31PM +0200, Christian Gmeiner wrote:
> I have no problem to postpone this patch until I have my farm up and
> get the first results.
Alternatively, I'd say just disable this for !HALTI0 for now, revert to the old
behavior in that case, then someone who is able to
mats-s3tc
>
> Btw. this enables GL 2.1 :-)
That's all?
Cool, I thought ext_texture_srgb needed rendering to srgb too,
but apparently not.
> Signed-off-by: Christian Gmeiner <christian.gmei...@gmail.com>
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
> ---
>
Prevents an assertion when using GALLIUM_HUD with ioquake3,
when cso_restore_constant_buffer_slot0 restores an empty
constant buffer in slot 0.
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/gallium/drivers/etnaviv/etnaviv_state.c | 2 +-
1 file changed, 1 insertion
Structure code to only flush when we will potentially call cpu_prep. This
prevents spurious flushes in applications that heavily rely on u_uploader.
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/gallium/drivers/etnaviv/etnaviv_transfer.c | 24
Prevents an assertion when using GALLIUM_HUD with ioquake3,
when cso_restore_constant_buffer_slot0 restores an empty
constant buffer in slot 0.
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/gallium/drivers/etnaviv/etnaviv_state.c | 2 +-
1 file changed, 1 insertion
On Sat, Oct 21, 2017 at 10:22:09AM +0200, Christian Gmeiner wrote:
> Hi Wladimir
>
> 2017-10-21 7:28 GMT+02:00 Wladimir J. van der Laan <laa...@gmail.com>:
> > On Fri, Oct 20, 2017 at 10:21:28PM +0200, Christian Gmeiner wrote:
> >> Fixes piglit/bin/oes_texture_f
On Fri, Oct 20, 2017 at 04:04:09PM -0700, Dylan Baker wrote:
> This is all pretty straight-forward, it builds on my system, but I don't have
> any actual hardware to test on.
>
> Christian and/or Wladimir, could one of you build this and do some kind of
> testing to ensure it works?
Sure, if you
On Fri, Oct 20, 2017 at 10:21:21PM +0200, Christian Gmeiner wrote:
> Signed-off-by: Christian Gmeiner <christian.gmei...@gmail.com>
> ---
> src/mesa/main/glformats.c | 19 +++
> 1 file changed, 19 insertions(+)
Reviewed-by: Wladimir J. van der Laan <laa..
On Fri, Oct 20, 2017 at 10:21:27PM +0200, Christian Gmeiner wrote:
> Signed-off-by: Christian Gmeiner <christian.gmei...@gmail.com>
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
Optional suggestions inline.
> ---
> src/gallium/drivers/etnaviv/
On Fri, Oct 20, 2017 at 10:21:26PM +0200, Christian Gmeiner wrote:
> Signed-off-by: Christian Gmeiner <christian.gmei...@gmail.com>
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
> ---
> src/gallium/drivers/etnaviv/etnaviv_screen.c | 4
> 1 file changed, 4
On Fri, Oct 20, 2017 at 10:21:25PM +0200, Christian Gmeiner wrote:
> Signed-off-by: Christian Gmeiner <christian.gmei...@gmail.com>
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
> ---
> src/gallium/drivers/etnaviv/etnaviv_tiling.c | 8 ++--
> 1 file cha
On Fri, Oct 20, 2017 at 10:21:23PM +0200, Christian Gmeiner wrote:
> Also update RS_FORMAT_RB_SWAP.
>
> Signed-off-by: Christian Gmeiner <christian.gmei...@gmail.com>
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
> ---
> src/gallium/drivers/etnaviv/etn
On Fri, Oct 20, 2017 at 10:21:22PM +0200, Christian Gmeiner wrote:
> Signed-off-by: Christian Gmeiner <christian.gmei...@gmail.com>
Reviewed-by: Wladimir J. van der Laan
Verified that HALTI0 has PE TEXTURE_FORMAT_EXT by looking at GC2000 blob
replays.
> ---
> src/gallium/
On Fri, Oct 20, 2017 at 10:21:28PM +0200, Christian Gmeiner wrote:
> Fixes piglit/bin/oes_texture_float half
>
> Signed-off-by: Christian Gmeiner
> ---
> src/gallium/drivers/etnaviv/etnaviv_translate.h | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git
On Fri, Oct 20, 2017 at 10:21:24PM +0200, Christian Gmeiner wrote:
> Use the extended format if an such a format was passed.
>
> Signed-off-by: Christian Gmeiner
> ---
> src/gallium/drivers/etnaviv/etnaviv_state.c | 10 --
> 1 file changed, 8 insertions(+),
On Thu, Oct 19, 2017 at 11:12:44PM +0200, Christian Gmeiner wrote:
> Update to etna_viv commit 6c9c706.
>
> Signed-off-by: Christian Gmeiner <christian.gmei...@gmail.com>
> Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
For whole v2 series:
Reviewed-by: Wladi
> There is one difference - how the sum is interpreted - uint64_t vs. bool
> value. In general the code
Ok in that case it's ok like this, just looked like unnecessary/accidental
duplication.
Regards,
Wladimir
___
mesa-dev mailing list
On Tue, Oct 17, 2017 at 10:38:17PM +0200, Christian Gmeiner wrote:
> Galliums query_type used in APIs is unsigned.
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
> Signed-off-by: Christian Gmeiner <christian.gmei...@gmail.com>
> ---
> src/gallium/drivers/etnavi
On Tue, Oct 17, 2017 at 10:38:15PM +0200, Christian Gmeiner wrote:
> Passes most occlusion query piglits. The following piglits are broken:
> - spec@arb_occlusion_query@occlusion_query_meta_fragments
> - spec@arb_occlusion_query@occlusion_query_meta_save
> - spec@arb_occlusion_query2@render
>
>
On Tue, Oct 17, 2017 at 10:38:16PM +0200, Christian Gmeiner wrote:
> Signed-off-by: Christian Gmeiner <christian.gmei...@gmail.com>
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
> ---
> src/gallium/drivers/etnaviv/etnaviv_screen.c | 3 ++-
> 1 file cha
On Tue, Oct 17, 2017 at 10:38:14PM +0200, Christian Gmeiner wrote:
> No hardware query is supported yet.
>
> Signed-off-by: Christian Gmeiner <christian.gmei...@gmail.com>
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
___
On Tue, Oct 17, 2017 at 10:38:13PM +0200, Christian Gmeiner wrote:
> Update to etna_viv commit 6c9c706.
>
> Signed-off-by: Christian Gmeiner <christian.gmei...@gmail.com>
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
> ---
> src/gallium/drivers/etnav
> Signed-off-by: Lucas Stach <l.st...@pengutronix.de>
Keeping the TS memory config state up to date with our internal state is a good
idea.
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
> ---
> src/gallium/drivers/etnaviv/etnaviv_clear_blit.c | 5 +--
>
rwrite other valid inputs otherwise.
>
> Fixes piglit drawpixels and a bunch of other tests using the st_draw path.
>
> Signed-off-by: Lucas Stach <l.st...@pengutronix.de>
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
__
On Sun, Oct 08, 2017 at 10:00:07PM +0200, Christian Gmeiner wrote:
> Saves us from calling util_query_clear_result(..) in every query
> type implementation.
>
> Signed-off-by: Christian Gmeiner <christian.gmei...@gmail.com>
Reviewed-by: Wladimir J. van der Laan
On Sun, Oct 08, 2017 at 10:00:06PM +0200, Christian Gmeiner wrote:
> We want the same active handling for every query type. So lets
> handle it in the generic layer.
>
> Signed-off-by: Christian Gmeiner <christian.gmei...@gmail.com>
Reviewed-By: Wladimir J. van der Laan
On Wed, Oct 04, 2017 at 11:44:32AM -0400, Rob Clark wrote:
> Signed-off-by: Rob Clark <robdcl...@gmail.com>
Reviewed etnaviv change only.
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
> ---
> src/gallium/drivers/etnaviv/etnaviv_screen.c| 1 +
> src/g
On Sun, Oct 01, 2017 at 01:46:05PM +0200, Christian Gmeiner wrote:
> Okay.. hopefully we do not forget to remove them here when it gets
> used during state emission like PE_ALPHA_COLOR_EXT0. But I
> am fine with that change and will shut up.
Yes, it should be removed then. Though if I understand
Set up new states that the blob started setting for GC3000 consistently.
This makes sure that when another test or driver leaves the GPU in
unpredictable state, these states are set up correctly for our
rendering.
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/g
Setting PA_VIEWPORT_UNK state correctly is necessary to make point sprite
rendering on GC3000 work.
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/gallium/drivers/etnaviv/etnaviv_context.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/gallium/drivers/e
sprite rendering on GC3000 work.
Wladimir J. van der Laan (2):
etnaviv: Fix point sprite rendering on GC3000
etnaviv: Set up unknown GC3000 states
src/gallium/drivers/etnaviv/etnaviv_context.c | 13 +
1 file changed, 13 insertions(+)
--
2.7.4
Support opcodes with bit 6 set in assembler, and assert that only ops
0x00..0x7f are used.
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/gallium/drivers/etnaviv/etnaviv_asm.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/e
A two-component dot product instruction is supported with HALTI2, use it
on hardware that supports it.
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/gallium/drivers/etnaviv/etnaviv_compiler.c | 3 ++-
src/gallium/drivers/etnaviv/etnaviv_disasm.c | 1 +
src/gallium/d
Support opcodes with bit 6 set in assembler, then use that to add the DP2
instruction, supported with HALTI2.
Wladimir J. van der Laan (2):
etnaviv: Support opcodes with bit 6 set in assembler
etnaviv: Add support for DP2 instruction
src/gallium/drivers/etnaviv/etnaviv_asm.c | 5
doing i.MX6qp scanout using PRE, and when rendering to
textures (though here using sampler TS would be even better).
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/gallium/drivers/etnaviv/etnaviv_clear_blit.c | 1 +
src/gallium/drivers/etnaviv/etnaviv_emit.c
On Fri, Sep 29, 2017 at 06:00:13PM +0200, Wladimir J. van der Laan wrote:
> If an RS blit is done with source exactly the same as destination, and
> the hardware supports this, do an in-place resolve.
>
> This is the same as the blob does and potentially saves significant
> bandw
Lucas Stach <l.st...@pengutronix.de>
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
> ---
> src/gallium/drivers/etnaviv/etnaviv_transfer.c | 15 ++-
> 1 file changed, 10 insertions(+), 5 deletions(-)
>
TS would be even better).
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/gallium/drivers/etnaviv/etnaviv_emit.c | 9 -
src/gallium/drivers/etnaviv/etnaviv_rs.c | 16 +---
src/gallium/drivers/etnaviv/etnaviv_rs.h | 1 +
3 files changed, 22 inse
cas Stach <l.st...@pengutronix.de>
Thanks for figuring this one out
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>: Wladimir J. van der
Laan <laa...@gmail.com>
> ---
> src/gallium/drivers/etnaviv/etnaviv_rasterizer.c | 3 +-
> src/gallium/drivers/etnaviv/hw/cmdstream.xml.h
appens a lot with glTexSubImage2D).
>
> Optimize this by only blitting the tile aligned subregion of the
> resource, which the transfer is going to change.
>
> Signed-off-by: Lucas Stach <l.st...@pengutronix.de>
Reviewed-by: Wladimir J. van der Laan <laa...@gmamil.com>
use of the RS
> - split out tile mask computation, to make it easier to read the code
Thanks!
Reviewed-By: Wladimir J. van der Laan <laa...@gmail.com>
> ---
> src/gallium/drivers/etnaviv/etnaviv_clear_blit.c | 86
> +---
> 1 file changed, 78 insertions(
me cases.
Regards,
Wladimir
>
> Signed-off-by: Lucas Stach <l.st...@pengutronix.de>
> Reviewed-by: Philipp Zabel <p.za...@pengutronix.de>
> Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
> ---
> v2: fix invalid vreg assignment
> v3: fix missed negat
sprite rendering on GC3000 work.
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/gallium/drivers/etnaviv/etnaviv_context.c | 13 +
1 file changed, 13 insertions(+)
diff --git a/src/gallium/drivers/etnaviv/etnaviv_context.c
b/src/gallium/drivers/etnaviv/etnaviv_con
Add missing includes after 6ace0b8 (etnaviv: don't enable RT
full-overwrite when logicop is enabled), otherwise the etnaviv driver
won't build because of missing macros.
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/gallium/drivers/etnaviv/etnaviv_blend.c | 2 ++
asked off.
>
> I've only observed the issue with R5G6B5_UNORM surfaces, other 16bpp
> formats seem to work fine.
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
>
> Fixes: d6aa2ba2b2 (etnaviv: replace translate_clear_color with
> util_pack_color)
> Cc: mesa-s
On Tue, Sep 19, 2017 at 06:16:05PM +0200, Christian Gmeiner wrote:
> Also add new define ETNA_SW_QUERY_BASE.
>
> Signed-off-by: Christian Gmeiner <christian.gmei...@gmail.com>
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
> ---
> src/gallium/drivers/etn
ail.com>
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
> ---
> src/gallium/drivers/etnaviv/etnaviv_query.c| 15 +++
> src/gallium/drivers/etnaviv/etnaviv_query_sw.c | 21 +
> src/gallium/drivers/etnaviv/etnaviv_query_sw.h | 4
&
_USE_POINTCOORD is a bit of a misnomer now, as it isn't
> only used for pointcoords, but missing a better name I left it as-is.
>
> Signed-off-by: Lucas Stach <l.st...@pengutronix.de>
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
> ---
> v2: fix invalid vreg assig
nix.de>
> Reviewed-by: Philipp Zabel <p.za...@pengutronix.de>
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
> ---
> src/gallium/drivers/etnaviv/etnaviv_rasterizer.c | 6 +-
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/src/ga
On Fri, Sep 15, 2017 at 06:00:53PM +0200, Lucas Stach wrote:
> This is useful if we only need to copy part of a larger resource, mostly
> when using the RS engine to de-/tile on pipe transfers.
>
> Signed-off-by: Lucas Stach <l.st...@pengutronix.de>
Reviewed-by: Wladimir J.
On Fri, Sep 15, 2017 at 06:00:52PM +0200, Lucas Stach wrote:
> The RS can blit abitrary tile aligned subregions of a resource by
> adjusting the buffer offset.
Thanks for doing this.
Comments inline,
> Signed-off-by: Lucas Stach
> ---
>
On Fri, Aug 25, 2017 at 12:48:22AM -0400, Ilia Mirkin wrote:
> The rest should be possible too, just needs some additional
> investigation. Passes fbo-*-formats piglit tests.
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
> Signed-off-by: Ilia Mirkin <imir...@alum.mit
On Thu, Aug 24, 2017 at 04:12:21PM -0700, Chris Healy wrote:
> When running on Mesa 17.2, I believe this has already been fixed with
> the GC3000 but is still not working with the GC2000. Here's the fix
> I'm thinking addresses the issue with the GC3000:
>
>
On Fri, Aug 04, 2017 at 06:54:12PM +0200, Philipp Zabel wrote:
> On Fri, 2017-08-04 at 18:15 +0200, Wladimir J. van der Laan wrote:
> > On Fri, Aug 04, 2017 at 05:07:54PM +0200, Philipp Zabel wrote:
> > > Look up BOs from the name table using the name parameter instead of
>
On Fri, Aug 04, 2017 at 05:07:54PM +0200, Philipp Zabel wrote:
> Look up BOs from the name table using the name parameter instead of
> req.handle (which at this point is always zero).
Good catch.
Just out of interest: when is this used, what problems does this cause?
Regards,
Wladimir
R8_UNORM textures can be emulated by means of L8 and a swizzle.
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/gallium/drivers/etnaviv/etnaviv_format.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/etnaviv/etnaviv_format.c
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/gallium/drivers/etnaviv/hw/state_3d.xml.h | 14 +-
1 file changed, 9 insertions(+), 5 deletions(-)
diff --git a/src/gallium/drivers/etnaviv/hw/state_3d.xml.h
b/src/gallium/drivers/etnaviv/hw/state_3d.xml.h
rom memory on
demand. Small shaders will be uploaded in the usual way. This mimics the
behavior of the blob.
On GPUs that don't support ICACHE, this patch should make no difference.
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/gallium/drivers/etnaviv/etnaviv_compile
is still fixed and hardcoded. It should
make no difference on hardware that does not have unified uniform
memory.
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/gallium/drivers/etnaviv/etnaviv_emit.c | 19 +++
src/gallium/drivers/etnaviv/etnaviv_internal.
rom memory on
demand. Small shaders will be uploaded in the usual way. This mimics the
behavior of the blob.
On GPUs that don't support ICACHE, this patch should make no difference.
Wladimir J. van der Laan (3):
etnaviv: Update headers from rnndb
etnaviv: Unified uniforms support
etnaviv:
. van der Laan <laa...@gmail.com>
---
src/gallium/drivers/etnaviv/etnaviv_compiler.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/etnaviv/etnaviv_compiler.c
b/src/gallium/drivers/etnaviv/etnaviv_compiler.c
index cc6bfc9..165ab74 100644
--- a/src/g
On Fri, Jul 21, 2017 at 02:43:11PM +0200, Lucas Stach wrote:
> Am Freitag, den 21.07.2017, 12:49 +0200 schrieb Wladimir J. van der
> Laan:
> > Remove the following duplicates from the formats table:
> >
> > - R8G8B8A8_UNORM (V_,_T)
> > - R8G8B8X8_UNORM
Remove the following duplicates from the formats table:
- R8G8B8A8_UNORM (V_,_T)
- R8G8B8X8_UNORM (_T,_T)
- DXT3_RGBA (_T,_T)
Only the first has an effect because the _T overrides the V_ initializer,
the latter two were harmless duplications of the same.
Signed-off-by: Wladimir J. van der Laan
this for sure.
(Only enabled on GC3000+ for now, as the GC2000 ETC2 decoder
implementation is buggy and we don't work around that)
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/gallium/drivers/etnaviv/etnaviv_format.c | 11 +++
src/gallium/drivers/e
A dangling bo object would result in memory corruption while
loading a level in ioquake3_opengl2.
Fixes a problem introduced in 330d0607ed60fd3edca192e54b4246310f06652f.
Fix suggested by Lucas Stach.
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
---
src/gallium/drivers/e
GC3000 has a new LOG instruction, similar to the new SIN and COS instructions.
Generate the new instruction sequence when appropriate; there are
two occasions, as part of LIT and the generator for the LG2
instruction itself.
Signed-off-by: Wladimir J. van der Laan <laa...@gmail.com>
--
On Fri, Jun 23, 2017 at 05:50:27PM +0200, Lucas Stach wrote:
> This allows to create buffers with a specific tiling layout, which is
> primarily
> used by GBM to allocate the EGL back buffers with the correct tiling/modifier
> for use with the scanout engines.
>
> Signed-off-by: Lucas Stach
On Fri, Jun 23, 2017 at 05:50:26PM +0200, Lucas Stach wrote:
> This allows the state trackers to know the tiling layout of the
> resource and pass this through the various userspace protocols.
> Signed-off-by: Lucas Stach <l.st...@pengutronix.de>
Comment inline.
Reviewed-by: Wlad
deronly scanout allocated
> buffers, as the linear modifier is enough to trigger correct handling
> of those buffers.
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
> Signed-off-by: Lucas Stach <l.st...@pengutronix.de>
> ---
> src/galliu
rge performance
> gains for the render to texture cases).
Agreed.
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
Regards,
Wladimir
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
ng
screen specific.
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
> ---
> src/gallium/drivers/etnaviv/etnaviv_resource.c | 14 --
> src/gallium/drivers/etnaviv/etnaviv_screen.c | 19 ---
> src/gallium/drivers/etnaviv/etnaviv_screen.
On Fri, Jun 23, 2017 at 05:50:23PM +0200, Lucas Stach wrote:
> This reworks the logic in etna_update_sampler_source to select the
> newest resource view for updating the texture view. This should make
> the logic easier to follow and fixes texture updates from imported
> dma-bufs.
Comment in-line
the external view seqno, instead of the base resource seqno.
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
> Signed-off-by: Lucas Stach <l.st...@pengutronix.de>
> ---
> src/gallium/drivers/etnaviv/etnaviv_resource.c | 5 -
> 1 file changed, 4 insertions(+),
On Tue, Jun 27, 2017 at 05:31:52PM +0200, Wladimir J. van der Laan wrote:
> > sure if we should advertise the broken formats (on gpus with HALTI0).
FWIW, I did check on both GC2000 and GC3000, so HALTI0 should be ok.
Wladimir
___
mesa-dev mailin
> I can confirm that 11 bit stuff works on the GC2000 . I used piglit's
> oes_compressed_etc2_texture-miptree_gles3 for verification.
> How have you tested these formats?
I rendered the example images from
https://github.com/Ericsson/ETCPACK/tree/master/testing/testvectors_correct/pkm,
which
ender target binding for them.
The SPLIT formats are only supported on hardware with multiple pixel pipes.
Should this be checked, or is it harmless to advertize them unconditionally?
Reviewed-by: Wladimir J. van der Laan
> Signed-off-by: Lucas Stach <l.st...@pengutronix.de>
> --
lution.
Reviewed-by: Wladimir J. van der Laan
> Signed-off-by: Lucas Stach <l.st...@pengutronix.de>
> ---
> src/gallium/drivers/etnaviv/etnaviv_resource.c | 13 +++--
> 1 file changed, 11 insertions(+), 2 deletions(-)
>
> diff --git a/src/gallium/drivers/etnaviv/etnav
On Fri, Jun 23, 2017 at 05:50:19PM +0200, Lucas Stach wrote:
> The resource struct is already allocated at this point and should be
> free properly.
>
> Signed-off-by: Lucas Stach <l.st...@pengutronix.de>
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
>
On Tue, Jun 27, 2017 at 02:54:56PM +0200, Wladimir J. van der Laan wrote:
> Add support for ETC2 compressed textures in the etnaviv driver.
>
> One step closer towards GL ES 3 support.
>
> For now, treat SRGB and RGB formats the same. It looks like these are
> distinguished usi
Add support for ETC2 compressed textures in the etnaviv driver.
One step closer towards GL ES 3 support.
For now, treat SRGB and RGB formats the same. It looks like these are
distinguished using a different bit in sampler state, and not part of
the format, but I have not yet been able to confirm
On Fri, Jun 23, 2017 at 05:50:18PM +0200, Lucas Stach wrote:
> The layer stride information is used in various parts of the driver,
> so it needs to be present regardless if the driver allocated the
> buffer itself or merely imported it from an external source.
Reviewed-by: Wladimir J
On Sat, Jun 24, 2017 at 10:41:58AM +0200, Christian Gmeiner wrote:
> Hi
>
> 2017-06-22 14:39 GMT+02:00 Wladimir J. van der Laan <laa...@gmail.com>:
> > On Wed, Jun 21, 2017 at 10:36:46PM +0200, Christian Gmeiner wrote:
> >> Passes all ext_texture_swizzle piglits.
&g
On Wed, Jun 21, 2017 at 10:36:48PM +0200, Christian Gmeiner wrote:
> Based on a patch from Wladimir J. van der Laan and untested due
> to lack of hardware. Binary blob emits those formats if GPU supports
> HALTI1 (faked with ibvivhook).
Reviewed-by: Wladimir J. van der Laan <laa.
On Wed, Jun 21, 2017 at 10:36:47PM +0200, Christian Gmeiner wrote:
> Passes texwrap GL_ARB_texture_rg piglit (with faked full texture rg support).
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
>
> Signed-off-by: Christian Gmeiner <christian.gmei...@gmail.com>
On Wed, Jun 21, 2017 at 10:36:46PM +0200, Christian Gmeiner wrote:
> Passes all ext_texture_swizzle piglits.
You seem to have dropped the check in etnaviv_screen.c for HALTI0 when texture
formats have implicit non-identity swizzle.
I do think that is necessary as those formats don't work on
On Wed, Jun 21, 2017 at 10:36:45PM +0200, Christian Gmeiner wrote:
> Signed-off-by: Christian Gmeiner <christian.gmei...@gmail.com>
Reviewed-by: Wladimir J. van der Laan <laa...@gmail.com>
> ---
> src/gallium/drivers/etnaviv/etnaviv_format.c | 2 +-
> src
On Mon, Jun 05, 2017 at 09:11:02PM +0200, Lucas Stach wrote:
> Same as with the colormasks, the blend color needs to be swizzled according
> to the rendertarget format.
Yes, indeed.
Reviewed-by: Wladimir J. van der Laan
> Signed-off-by: Lucas Stach <d...@lynxeye.de>
> ---
>
On Tue, May 30, 2017 at 12:50:07PM +0200, Philipp Zabel wrote:
> Before resolving a rendertarget or a depth/stencil resource into a
> texture, flush both the color cache and the depth cache together.
>
> It is unclear whether this is necessary for the following stall to
> work properly, or
t path -
should we count these too?)
Reviewed-By: Wladimir J. van der Laan <laa...@gmail.com>
>
> Signed-off-by: Christian Gmeiner <christian.gmei...@gmail.com>
> ---
> src/gallium/drivers/etnaviv/etnaviv_context.h | 1 +
> src/gallium/drivers/etnaviv/etnaviv_emit.c | 2
101 - 200 of 240 matches
Mail list logo