Re: [Mesa-dev] [PATCH] anv/query: Busy-wait for available query entries

2017-04-05 Thread Iago Toral
On Wed, 2017-04-05 at 09:31 -0700, Jason Ekstrand wrote: > On Wed, Apr 5, 2017 at 12:24 AM, Iago Toral > wrote: > > On Tue, 2017-04-04 at 19:21 -0700, Jason Ekstrand wrote: > > > Before, we were just looking at whether or not the user wanted us > > to > > > wait and waiting on the BO.  This instea

Re: [Mesa-dev] [PATCH] anv/query: Busy-wait for available query entries

2017-04-05 Thread Chris Wilson
On Wed, Apr 05, 2017 at 09:37:16AM -0700, Jason Ekstrand wrote: >On Wed, Apr 5, 2017 at 1:27 AM, Chris Wilson <[1]ch...@chris-wilson.co.uk> >wrote: > > On Tue, Apr 04, 2017 at 07:21:38PM -0700, Jason Ekstrand wrote: > > Before, we were just looking at whether or not the user want

Re: [Mesa-dev] [PATCH] anv/query: Busy-wait for available query entries

2017-04-05 Thread Jason Ekstrand
On Wed, Apr 5, 2017 at 1:27 AM, Chris Wilson wrote: > On Tue, Apr 04, 2017 at 07:21:38PM -0700, Jason Ekstrand wrote: > > Before, we were just looking at whether or not the user wanted us to > > wait and waiting on the BO. This instead makes us busy-loop on each > > query until it's available.

Re: [Mesa-dev] [PATCH] anv/query: Busy-wait for available query entries

2017-04-05 Thread Jason Ekstrand
On Wed, Apr 5, 2017 at 12:24 AM, Iago Toral wrote: > On Tue, 2017-04-04 at 19:21 -0700, Jason Ekstrand wrote: > > Before, we were just looking at whether or not the user wanted us to > > wait and waiting on the BO. This instead makes us busy-loop on each > > query until it's available. This red

Re: [Mesa-dev] [PATCH] anv/query: Busy-wait for available query entries

2017-04-05 Thread Eero Tamminen
Hi, On 05.04.2017 11:27, Chris Wilson wrote: On Tue, Apr 04, 2017 at 07:21:38PM -0700, Jason Ekstrand wrote: Before, we were just looking at whether or not the user wanted us to wait and waiting on the BO. This instead makes us busy-loop on each query until it's available. This reduces some o

Re: [Mesa-dev] [PATCH] anv/query: Busy-wait for available query entries

2017-04-05 Thread Chris Wilson
On Tue, Apr 04, 2017 at 07:21:38PM -0700, Jason Ekstrand wrote: > Before, we were just looking at whether or not the user wanted us to > wait and waiting on the BO. This instead makes us busy-loop on each > query until it's available. This reduces some of the pipeline bubbles > we were getting an

Re: [Mesa-dev] [PATCH] anv/query: Busy-wait for available query entries

2017-04-05 Thread Iago Toral
On Tue, 2017-04-04 at 19:21 -0700, Jason Ekstrand wrote: > Before, we were just looking at whether or not the user wanted us to > wait and waiting on the BO.  This instead makes us busy-loop on each > query until it's available.  This reduces some of the pipeline > bubbles > we were getting and imp

[Mesa-dev] [PATCH] anv/query: Busy-wait for available query entries

2017-04-04 Thread Jason Ekstrand
Before, we were just looking at whether or not the user wanted us to wait and waiting on the BO. This instead makes us busy-loop on each query until it's available. This reduces some of the pipeline bubbles we were getting and improves performance of The Talos Principle on medium settings (where