Reviewed-by: Bas Nieuwenhuizen <[email protected]>
On Wed, May 9, 2018 at 11:52 PM, Samuel Pitoiset <[email protected]> wrote: > 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 <[email protected]> > --- > 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 > [email protected] > https://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
