Series is
Reviewed-by: Ian Romanick <ian.d.roman...@intel.com>
On 12/12/2014 11:15 PM, Kenneth Graunke wrote:
> q->Ready means that the results are in, and core Mesa is free to return
> them to the application. gen6_queryobj_get_results() is a natural place
> to set that flag; doing so means callers don't have to.
>
> The older non-hardware-context aware code couldn't do this, because we
> had to call brw_queryobj_get_results() to gather intermediate results
> when we ran out of space for snapshots in the query buffer. We only
> gather complete results in the Gen6+ code, however.
>
> Signed-off-by: Kenneth Graunke <kenn...@whitecape.org>
> Cc: Chris Wilson <ch...@chris-wilson.co.uk>
> Cc: Eero Tamminen <eero.t.tammi...@intel.com>
> ---
> src/mesa/drivers/dri/i965/gen6_queryobj.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/gen6_queryobj.c
> b/src/mesa/drivers/dri/i965/gen6_queryobj.c
> index 130236e..3013513 100644
> --- a/src/mesa/drivers/dri/i965/gen6_queryobj.c
> +++ b/src/mesa/drivers/dri/i965/gen6_queryobj.c
> @@ -195,6 +195,8 @@ gen6_queryobj_get_results(struct gl_context *ctx,
> */
> drm_intel_bo_unreference(query->bo);
> query->bo = NULL;
> +
> + query->Base.Ready = true;
> }
>
> /**
> @@ -305,7 +307,6 @@ static void gen6_wait_query(struct gl_context *ctx,
> struct gl_query_object *q)
> struct brw_query_object *query = (struct brw_query_object *)q;
>
> gen6_queryobj_get_results(ctx, query);
> - query->Base.Ready = true;
> }
>
> /**
> @@ -331,7 +332,6 @@ static void gen6_check_query(struct gl_context *ctx,
> struct gl_query_object *q)
>
> if (query->bo == NULL || !drm_intel_bo_busy(query->bo)) {
> gen6_queryobj_get_results(ctx, query);
> - query->Base.Ready = true;
> }
> }
>
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
