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