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 <samuel.pitoi...@gmail.com>
---
 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 9fe415c4e0..34561d69c0 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.17.0

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to