Re: [Mesa-dev] [PATCH 4/7] i965: Add an end-of-pipe sync helper

2017-06-15 Thread Jason Ekstrand
On Thu, Jun 15, 2017 at 9:14 AM, Chris Wilson wrote: > Quoting Jason Ekstrand (2017-06-15 16:59:19) > > On Thu, Jun 15, 2017 at 4:11 AM, Chris Wilson > wrote: > > The kernel does have a LRI after a flush before signaling the batch > is > > complete. I don't see a need to add another... >

Re: [Mesa-dev] [PATCH 4/7] i965: Add an end-of-pipe sync helper

2017-06-15 Thread Chris Wilson
Quoting Jason Ekstrand (2017-06-15 16:59:19) > On Thu, Jun 15, 2017 at 4:11 AM, Chris Wilson > wrote: > The kernel does have a LRI after a flush before signaling the batch is > complete. I don't see a need to add another... > > The question is whether this posting is required for GPU

Re: [Mesa-dev] [PATCH 4/7] i965: Add an end-of-pipe sync helper

2017-06-15 Thread Jason Ekstrand
On Thu, Jun 15, 2017 at 4:11 AM, Chris Wilson wrote: > Quoting Kenneth Graunke (2017-06-14 21:41:56) > > On Tuesday, June 13, 2017 2:53:24 PM PDT Jason Ekstrand wrote: > > > From: Topi Pohjolainen > > > > > > v2 (Jason Ekstrand): > > > - Take a flags parameter to control the flushes > > > - Re

Re: [Mesa-dev] [PATCH 4/7] i965: Add an end-of-pipe sync helper

2017-06-15 Thread Chris Wilson
Quoting Kenneth Graunke (2017-06-14 21:41:56) > On Tuesday, June 13, 2017 2:53:24 PM PDT Jason Ekstrand wrote: > > From: Topi Pohjolainen > > > > v2 (Jason Ekstrand): > > - Take a flags parameter to control the flushes > > - Refactoring > > > > Signed-off-by: Topi Pohjolainen > > --- > > src

Re: [Mesa-dev] [PATCH 4/7] i965: Add an end-of-pipe sync helper

2017-06-14 Thread Kenneth Graunke
On Tuesday, June 13, 2017 2:53:24 PM PDT Jason Ekstrand wrote: > From: Topi Pohjolainen > > v2 (Jason Ekstrand): > - Take a flags parameter to control the flushes > - Refactoring > > Signed-off-by: Topi Pohjolainen > --- > src/mesa/drivers/dri/i965/brw_context.h | 1 + > src/mesa/drive

[Mesa-dev] [PATCH 4/7] i965: Add an end-of-pipe sync helper

2017-06-13 Thread Jason Ekstrand
From: Topi Pohjolainen v2 (Jason Ekstrand): - Take a flags parameter to control the flushes - Refactoring Signed-off-by: Topi Pohjolainen --- src/mesa/drivers/dri/i965/brw_context.h | 1 + src/mesa/drivers/dri/i965/brw_pipe_control.c | 96 +++- 2 files changed,