From 0defc5554780a444c9e2009178dc88f97740a174 Mon Sep 17 00:00:00 2001 From: Samuel Pitoiset Date: Wed, 9 May 2018 23:52:53 +0200 Subject: [PATCH] radv: move handling nosisched option in a better place It's a per-application optimization, so it makes more sense to do that in radv_handle_per_app_options(). Signed-off-by: Samuel Pitoiset Reviewed-by: Bas Nieuwenhuizen --- src/amd/vulkan/radv_device.c | 18 ++++++------------ 1 file changed, 6 insertions(+), 12 deletions(-) diff --git a/src/amd/vulkan/radv_device.c b/src/amd/vulkan/radv_device.c index 08795dd..a7f4a5a 100644 --- a/src/amd/vulkan/radv_device.c +++ b/src/amd/vulkan/radv_device.c @@ -428,10 +428,12 @@ radv_handle_per_app_options(struct radv_instance *instance, 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; + if (!(instance->debug_flags & RADV_DEBUG_NO_SISCHED)) { + /* Force enable LLVM sisched for Talos because it looks + * safe and it gives few more FPS. + */ + instance->perftest_flags |= RADV_PERFTEST_SISCHED; + } } } @@ -508,14 +510,6 @@ VkResult radv_CreateInstance( radv_handle_per_app_options(instance, pCreateInfo->pApplicationInfo); - if (instance->debug_flags & RADV_DEBUG_NO_SISCHED) { - /* Disable sisched when the user requests it, this is mostly - * useful when the driver force-enable sisched for the given - * application. - */ - instance->perftest_flags &= ~RADV_PERFTEST_SISCHED; - } - *pInstance = radv_instance_to_handle(instance); return VK_SUCCESS; -- 2.7.4