[Mesa-dev] [PATCH 00/11] glthread cleanup, 4 whitelisted games!

[Timothy Arceri]

On 22/06/17 11:02, Marek Olšák wrote:
> Hi,
> 
> This series:
> - cleans up glthread, mainly switches the implementation to u_queue
>    and rearranges it so that it looks like gallium/u_threaded_context
> - adds "performance" counters for monitoring glthread behavior and
>    display them on the gallium HUD
> - decreases the batch size for much better scalability
> - whitelists 4 games
> 
> Here are the games and the performance deltas:
>      Alien Isolation: +17% (it varies depending on the location)
>      Borderlands 2: +50% (it varies depending on the location)
>      BioShock Infinite: +76% (benchmark)
>      Civilization 6: +20% (benchmark)

Nice work on figuring out patch 10. I can confirm with this series I'm 
also seeing a similar boost in the Civ6 benchmark.

Assuming this doesn't regress anything in piglit. Series is:

Reviewed-by: Timothy Arceri <tarceri at itsqueeze.com>


> 
> I would add more games, but not all games benefit, some even regress,
> and I'm not often CPU-bound.
> 
> I'd like to see TF2 and CS:GO on that list eventually, but those are
> the only games where glthread gets automatically disabled (as can be
> observed on the HUD).
> 
> I'd also like to whitelist "Borderlands: The pre-sequel" eventually,
> it should benefit too, but we need to test it first.
> 
> I personally don't plan to spend more time on glthread again in the near
> future. The HUD monitoring can be used for estimating potential
> performance improvement for slower CPUs even if you specifically are
> not CPU-bound.
> 
> Please review.
> 
> Marek
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>