[Mesa-dev] [PATCH] radv: force enable LLVM sisched for The Talos Principle

[Michel Dänzer]

On 07/11/17 10:08 AM, Samuel Pitoiset wrote:
> It seems safe and it improves performance by +4% (73->76).
> 
> Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
> ---
>  src/amd/vulkan/radv_device.c | 20 ++++++++++++++++++++
>  1 file changed, 20 insertions(+)
> 
> diff --git a/src/amd/vulkan/radv_device.c b/src/amd/vulkan/radv_device.c
> index 1ecf70d4a9..29bbcc5a43 100644
> --- a/src/amd/vulkan/radv_device.c
> +++ b/src/amd/vulkan/radv_device.c
> @@ -341,6 +341,24 @@ radv_get_perftest_option_name(int id)
>  	return radv_perftest_options[id].string;
>  }
>  
> +static void
> +radv_handle_per_app_options(struct radv_instance *instance,
> +			    const VkApplicationInfo *info)
> +{
> +	const char *name = info ? info->pApplicationName : NULL;
> +
> +	if (!name)
> +		return;
> +
> +	if (!strcmp(name, "Talos - Linux - 32bit") ||
> +	    !strcmp(name, "Talos - Linux - 64bit")) {
> +		/* Force enable LLVM sisched for Talos because it looks safe
> +		 * and it gives few more FPS.
> +		 */
> +		instance->perftest_flags |= RADV_PERFTEST_SISCHED;
> +	}
> +}
> +
>  VkResult radv_CreateInstance(
>  	const VkInstanceCreateInfo*                 pCreateInfo,
>  	const VkAllocationCallbacks*                pAllocator,
> @@ -400,6 +418,8 @@ VkResult radv_CreateInstance(
>  	instance->perftest_flags = parse_debug_string(getenv("RADV_PERFTEST"),
>  						   radv_perftest_options);
>  
> +	radv_handle_per_app_options(instance, pCreateInfo->pApplicationInfo);
> +
>  	*pInstance = radv_instance_to_handle(instance);
>  
>  	return VK_SUCCESS;
> 

There should probably be a way to explicitly disable sisched.


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer