[Mesa-dev] [PATCH] st/nine: enable csmt per default on iris

Kenneth Graunke kenneth at whitecape.org
Wed Mar 20 20:55:07 UTC 2019


On Wednesday, March 20, 2019 1:38:40 PM PDT Andre Heider wrote:
> iris is thread safe, enable csmt for a ~5% performace boost.
> 
> Signed-off-by: Andre Heider <a.heider at gmail.com>
> ---
>  src/gallium/state_trackers/nine/device9.c | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/src/gallium/state_trackers/nine/device9.c b/src/gallium/state_trackers/nine/device9.c
> index 24c8ce062b3..db1c3a1d23d 100644
> --- a/src/gallium/state_trackers/nine/device9.c
> +++ b/src/gallium/state_trackers/nine/device9.c
> @@ -266,13 +266,15 @@ NineDevice9_ctor( struct NineDevice9 *This,
>      }
>  
>      /* Initialize CSMT */
> +    /* r600, radeonsi and iris are thread safe. */
>      if (pCTX->csmt_force == 1)
>          This->csmt_active = true;
>      else if (pCTX->csmt_force == 0)
>          This->csmt_active = false;
> -    else
> -        /* r600 and radeonsi are thread safe. */
> -        This->csmt_active = strstr(pScreen->get_name(pScreen), "AMD") != NULL;
> +    else if (strstr(pScreen->get_name(pScreen), "AMD") != NULL)
> +        This->csmt_active = true;
> +    else if (strstr(pScreen->get_name(pScreen), "Intel") != NULL)
> +        This->csmt_active = true;
>  
>      /* We rely on u_upload_mgr using persistent coherent buffers (which don't
>       * require flush to work in multi-pipe_context scenario) for vertex and
> 

Thanks Andre!

Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20190320/0d46481d/attachment.sig>


More information about the mesa-dev mailing list