[Mesa-dev] [PATCH 3/4] nine: handle D3DISSUE_END without previous D3DISSUE_BEGIN

[Ilia Mirkin]

On Fri, Nov 21, 2014 at 10:13 PM, David Heidelberg <david at ixit.cz> wrote:
> From: John Ettedgui <john.ettedgui at gmail.com>
>
> This patch fixes black screen with games based on the Unreal Engine 3.
> It was tested that it fixed the issue in Tera Online, Borderlands 2 and Homefront.

What's the actual problem? If there's no begin, then a new query would
remain marked as FRESH. Should _GetData 0 out the result in that case
perhaps instead of insta-returning?

>
> Cc: "10.4" <mesa-stable at lists.freedesktop.org>
> Reviewed-by: Axel Davy <axel.davy at ens.fr>
> Reviewed-by: David Heidelberg <david at ixit.cz>
> Signed-off-by: John Ettedgui <john.ettedgui at gmail.com>
> ---
>  src/gallium/state_trackers/nine/query9.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/state_trackers/nine/query9.c b/src/gallium/state_trackers/nine/query9.c
> index 86762d2..5006d9f 100644
> --- a/src/gallium/state_trackers/nine/query9.c
> +++ b/src/gallium/state_trackers/nine/query9.c
> @@ -188,8 +188,8 @@ NineQuery9_Issue( struct NineQuery9 *This,
>      } else {
>          if (This->state == NINE_QUERY_STATE_RUNNING) {
>              pipe->end_query(pipe, This->pq);
> -            This->state = NINE_QUERY_STATE_ENDED;
>         }
> +       This->state = NINE_QUERY_STATE_ENDED;
>      }
>      return D3D_OK;
>  }
> --
> 2.1.3
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev