Re: [Mesa-dev] [PATCH 7/7] i965: Fix asynchronous mappings on !LLC platforms.

2017-07-07 Thread Chris Wilson
Quoting Kenneth Graunke (2017-07-07 07:08:16) > On Thursday, July 6, 2017 10:51:49 PM PDT Kenneth Graunke wrote: > > On Wednesday, July 5, 2017 2:24:55 PM PDT Chris Wilson wrote: > > > Quoting Kenneth Graunke (2017-07-05 21:56:54) > > > > --- > > > > src/mesa/drivers/dri/i965/brw_bufmgr.c | 15

Re: [Mesa-dev] [PATCH 7/7] i965: Fix asynchronous mappings on !LLC platforms.

2017-07-07 Thread Daniel Vetter
On Thu, Jul 06, 2017 at 10:51:49PM -0700, Kenneth Graunke wrote: > On Wednesday, July 5, 2017 2:24:55 PM PDT Chris Wilson wrote: > > Quoting Kenneth Graunke (2017-07-05 21:56:54) > > In the meantime, s/else if (!bo->cache_coherent)/if (!bo->cache_coherent)/ > > Oh? I can do that. I figured that

Re: [Mesa-dev] [PATCH 7/7] i965: Fix asynchronous mappings on !LLC platforms.

2017-07-07 Thread Chris Wilson
Quoting Kenneth Graunke (2017-07-07 06:51:49) > On Wednesday, July 5, 2017 2:24:55 PM PDT Chris Wilson wrote: > > Quoting Kenneth Graunke (2017-07-05 21:56:54) > > > --- > > > src/mesa/drivers/dri/i965/brw_bufmgr.c | 15 +-- > > > 1 file changed, 13 insertions(+), 2 deletions(-) > > >

Re: [Mesa-dev] [PATCH 7/7] i965: Fix asynchronous mappings on !LLC platforms.

2017-07-07 Thread Kenneth Graunke
On Thursday, July 6, 2017 10:51:49 PM PDT Kenneth Graunke wrote: > On Wednesday, July 5, 2017 2:24:55 PM PDT Chris Wilson wrote: > > Quoting Kenneth Graunke (2017-07-05 21:56:54) > > > --- > > > src/mesa/drivers/dri/i965/brw_bufmgr.c | 15 +-- > > > 1 file changed, 13 insertions(+), 2

Re: [Mesa-dev] [PATCH 7/7] i965: Fix asynchronous mappings on !LLC platforms.

2017-07-06 Thread Kenneth Graunke
On Wednesday, July 5, 2017 2:24:55 PM PDT Chris Wilson wrote: > Quoting Kenneth Graunke (2017-07-05 21:56:54) > > --- > > src/mesa/drivers/dri/i965/brw_bufmgr.c | 15 +-- > > 1 file changed, 13 insertions(+), 2 deletions(-) > > > > diff --git a/src/mesa/drivers/dri/i965/brw_bufmgr.c

Re: [Mesa-dev] [PATCH 7/7] i965: Fix asynchronous mappings on !LLC platforms.

2017-07-05 Thread Chris Wilson
Quoting Kenneth Graunke (2017-07-05 21:56:54) > --- > src/mesa/drivers/dri/i965/brw_bufmgr.c | 15 +-- > 1 file changed, 13 insertions(+), 2 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_bufmgr.c > b/src/mesa/drivers/dri/i965/brw_bufmgr.c > index

[Mesa-dev] [PATCH 7/7] i965: Fix asynchronous mappings on !LLC platforms.

2017-07-05 Thread Kenneth Graunke
When using a read-only CPU mapping, we may encounter stale buffer contents. For example, the Piglit primitive-restart test offers the following scenario: 1. Read data via a CPU map. 2. Destroy that buffer. 3. Create a new buffer - obtaining the same one via the BO cache. 4. Call