[Mesa-dev] [PATCH] nvc0/query: write 0 to pipeline_statistics.cs_invocations

[Ilia Mirkin]

Reviewed-by: Ilia Mirkin <imirkin at alum.mit.edu>

On Sun, Aug 27, 2017 at 12:00 PM, Karol Herbst <karolherbst at gmail.com> wrote:
> cs_invocations are currently unsupported, but leaving the field uninitialized
> is even worse.
>
> fixes on nvc0:
>  * KHR-GL45.pipeline_statistics_query_tests_ARB.functional_default_qo_values
>  * KHR-GL45.pipeline_statistics_query_tests_ARB.functional_non_rendering_commands_do_not_affect_queries
>
> Signed-off-by: Karol Herbst <karolherbst at gmail.com>
> ---
>  src/gallium/drivers/nouveau/nvc0/nvc0_query_hw.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_query_hw.c b/src/gallium/drivers/nouveau/nvc0/nvc0_query_hw.c
> index 0991af872b..d8d82de865 100644
> --- a/src/gallium/drivers/nouveau/nvc0/nvc0_query_hw.c
> +++ b/src/gallium/drivers/nouveau/nvc0/nvc0_query_hw.c
> @@ -346,6 +346,7 @@ nvc0_hw_get_query_result(struct nvc0_context *nvc0, struct nvc0_query *q,
>     case PIPE_QUERY_PIPELINE_STATISTICS:
>        for (i = 0; i < 10; ++i)
>           res64[i] = data64[i * 2] - data64[24 + i * 2];
> +      result->pipeline_statistics.cs_invocations = 0;
>        break;
>     case NVC0_HW_QUERY_TFB_BUFFER_OFFSET:
>        res32[0] = hq->data[1];
> --
> 2.14.1
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev