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

[Andre Heider]

On 20/03/2019 23:47, Axel Davy wrote:
> On 20/03/2019 21:38, 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
> 
> 
> Could have been an || inside the same if, but maybe it is easier to read 
> that way.

Yeah, I chose this way just for cosmetically reasons ;)

> Reviewed-by: Axel Davy <davyaxel0 at gmail.com>

Thanks!