Re: [Mesa-dev] [v2 02/17] i965/blorp: Skip redundant re-fast clear for non-compressed

2016-11-27 Thread Ben Widawsky
On 16-11-24 09:23:31, Jason Ekstrand wrote: [snip] Right, you are correct. This is actually a patch from really early days when I didn't know any better. We might want to drop this for now, there is the 0/1 color thing for sampler engine that we probably need to fix first anyway. Let's

Re: [Mesa-dev] [v2 02/17] i965/blorp: Skip redundant re-fast clear for non-compressed

2016-11-24 Thread Jason Ekstrand
On Nov 24, 2016 8:42 AM, "Ben Widawsky" wrote: > > On 16-11-23 12:04:02, Jason Ekstrand wrote: >> >> On Wed, Nov 23, 2016 at 10:16 AM, Pohjolainen, Topi < >> topi.pohjolai...@gmail.com> wrote: >> >>> On Wed, Nov 23, 2016 at 09:05:39AM -0800, Jason Ekstrand wrote: >>>

Re: [Mesa-dev] [v2 02/17] i965/blorp: Skip redundant re-fast clear for non-compressed

2016-11-24 Thread Ben Widawsky
On 16-11-23 12:04:02, Jason Ekstrand wrote: On Wed, Nov 23, 2016 at 10:16 AM, Pohjolainen, Topi < topi.pohjolai...@gmail.com> wrote: On Wed, Nov 23, 2016 at 09:05:39AM -0800, Jason Ekstrand wrote: >On Wed, Nov 23, 2016 at 1:16 AM, Topi Pohjolainen ><[1]topi.pohjolai...@gmail.com>

Re: [Mesa-dev] [v2 02/17] i965/blorp: Skip redundant re-fast clear for non-compressed

2016-11-23 Thread Jason Ekstrand
On Wed, Nov 23, 2016 at 10:16 AM, Pohjolainen, Topi < topi.pohjolai...@gmail.com> wrote: > On Wed, Nov 23, 2016 at 09:05:39AM -0800, Jason Ekstrand wrote: > >On Wed, Nov 23, 2016 at 1:16 AM, Topi Pohjolainen > ><[1]topi.pohjolai...@gmail.com> wrote: > > > > Originally re-clears where

Re: [Mesa-dev] [v2 02/17] i965/blorp: Skip redundant re-fast clear for non-compressed

2016-11-23 Thread Pohjolainen, Topi
On Wed, Nov 23, 2016 at 09:05:39AM -0800, Jason Ekstrand wrote: >On Wed, Nov 23, 2016 at 1:16 AM, Topi Pohjolainen ><[1]topi.pohjolai...@gmail.com> wrote: > > Originally re-clears where skipped but when lossless compression > was introduced the re-clears where errorneously

Re: [Mesa-dev] [v2 02/17] i965/blorp: Skip redundant re-fast clear for non-compressed

2016-11-23 Thread Jason Ekstrand
On Wed, Nov 23, 2016 at 1:16 AM, Topi Pohjolainen < topi.pohjolai...@gmail.com> wrote: > Originally re-clears where skipped but when lossless compression > was introduced the re-clears where errorneously enabled also for > non-compressed fast clears. > > Signed-off-by: Topi Pohjolainen

[Mesa-dev] [v2 02/17] i965/blorp: Skip redundant re-fast clear for non-compressed

2016-11-23 Thread Topi Pohjolainen
Originally re-clears where skipped but when lossless compression was introduced the re-clears where errorneously enabled also for non-compressed fast clears. Signed-off-by: Topi Pohjolainen CC: Ben Widawsky CC: Kenneth Graunke