Re: [Mesa-dev] [PATCH] etnaviv: bugfix: Don't do resolve-in-place without valid TS

2017-11-03 Thread Lucas Stach
Hi Andres, Am Freitag, den 03.11.2017, 14:39 +0200 schrieb Andres Gomez: > On Wed, 2017-11-01 at 12:47 +0100, Lucas Stach wrote: > > Am Mittwoch, den 01.11.2017, 12:34 +0100 schrieb Wladimir J. van > > der Laan: > > > On Wed, Nov 01, 2017 at 11:52:55AM +0100, Lucas Stach wrote: > > > > Am Mittwoch

Re: [Mesa-dev] [PATCH] etnaviv: bugfix: Don't do resolve-in-place without valid TS

2017-11-03 Thread Andres Gomez
On Wed, 2017-11-01 at 12:47 +0100, Lucas Stach wrote: > Am Mittwoch, den 01.11.2017, 12:34 +0100 schrieb Wladimir J. van der Laan: > > On Wed, Nov 01, 2017 at 11:52:55AM +0100, Lucas Stach wrote: > > > Am Mittwoch, den 01.11.2017, 11:17 +0100 schrieb Wladimir J. van der Laan: > > > > GC3000 resolve

Re: [Mesa-dev] [PATCH] etnaviv: bugfix: Don't do resolve-in-place without valid TS

2017-11-02 Thread Lucas Stach
Am Mittwoch, den 01.11.2017, 11:21 -0700 schrieb Chris Healy: > Tested-by: Chris Healy > > I tested this with i.MX6qp (and i.MX6q) and found that the rendering > issues I experienced with Wayland/Weston and the i.MX6qp went away > with this patch applied. > > Also, I ran glmark2-es2-drm and saw

Re: [Mesa-dev] [PATCH] etnaviv: bugfix: Don't do resolve-in-place without valid TS

2017-11-01 Thread Chris Healy
Tested-by: Chris Healy I tested this with i.MX6qp (and i.MX6q) and found that the rendering issues I experienced with Wayland/Weston and the i.MX6qp went away with this patch applied. Also, I ran glmark2-es2-drm and saw no regressions. On Wed, Nov 1, 2017 at 3:17 AM, Wladimir J. van der Laan w

Re: [Mesa-dev] [PATCH] etnaviv: bugfix: Don't do resolve-in-place without valid TS

2017-11-01 Thread Wladimir
On Wed, Nov 1, 2017 at 11:35 AM, Eric Engestrom wrote: > On Wednesday, 2017-11-01 11:17:53 +0100, Wladimir J. van der Laan wrote: >> GC3000 resolve-in-place assumes that the TS state is configured. >> If it is not, this will result in MMU errors. This is especially >> apparent when using glGenMipm

Re: [Mesa-dev] [PATCH] etnaviv: bugfix: Don't do resolve-in-place without valid TS

2017-11-01 Thread Lucas Stach
Am Mittwoch, den 01.11.2017, 12:34 +0100 schrieb Wladimir J. van der Laan: > On Wed, Nov 01, 2017 at 11:52:55AM +0100, Lucas Stach wrote: > > Am Mittwoch, den 01.11.2017, 11:17 +0100 schrieb Wladimir J. van der Laan: > > > GC3000 resolve-in-place assumes that the TS state is configured. > > > If it

Re: [Mesa-dev] [PATCH] etnaviv: bugfix: Don't do resolve-in-place without valid TS

2017-11-01 Thread Wladimir J. van der Laan
On Wed, Nov 01, 2017 at 11:52:55AM +0100, Lucas Stach wrote: > Am Mittwoch, den 01.11.2017, 11:17 +0100 schrieb Wladimir J. van der Laan: > > GC3000 resolve-in-place assumes that the TS state is configured. > > If it is not, this will result in MMU errors. This is especially > > apparent when using

Re: [Mesa-dev] [PATCH] etnaviv: bugfix: Don't do resolve-in-place without valid TS

2017-11-01 Thread Lucas Stach
Am Mittwoch, den 01.11.2017, 11:17 +0100 schrieb Wladimir J. van der Laan: > GC3000 resolve-in-place assumes that the TS state is configured. > If it is not, this will result in MMU errors. This is especially > apparent when using glGenMipmaps(). > > Fixes a problem introduced in 78ade659569ee6fe9

Re: [Mesa-dev] [PATCH] etnaviv: bugfix: Don't do resolve-in-place without valid TS

2017-11-01 Thread Eric Engestrom
On Wednesday, 2017-11-01 11:17:53 +0100, Wladimir J. van der Laan wrote: > GC3000 resolve-in-place assumes that the TS state is configured. > If it is not, this will result in MMU errors. This is especially > apparent when using glGenMipmaps(). > > Fixes a problem introduced in 78ade659569ee6fe9bd

[Mesa-dev] [PATCH] etnaviv: bugfix: Don't do resolve-in-place without valid TS

2017-11-01 Thread Wladimir J. van der Laan
GC3000 resolve-in-place assumes that the TS state is configured. If it is not, this will result in MMU errors. This is especially apparent when using glGenMipmaps(). Fixes a problem introduced in 78ade659569ee6fe9bd244170956139f19dd8c6c. Signed-off-by: Wladimir J. van der Laan --- src/gallium/d