From c2799574eb104218ac4f6b8a3d64b8a0f8c89525 Mon Sep 17 00:00:00 2001 From: Bas Nieuwenhuizen Date: Wed, 30 May 2018 08:50:03 +0200 Subject: [PATCH] radv: Only expose subgroup shuffles on VI+. MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit The current implementation depends on bpermute, which is VI+. Fixes: f2c6a550611 "radv: enable subgroup capabilities" Reviewed-by: Daniel Schürmann Reviewed-by: Samuel Pitoiset --- src/amd/vulkan/radv_device.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/amd/vulkan/radv_device.c b/src/amd/vulkan/radv_device.c index 61b4fba23f8..93b6e611067 100644 --- a/src/amd/vulkan/radv_device.c +++ b/src/amd/vulkan/radv_device.c @@ -963,9 +963,12 @@ void radv_GetPhysicalDeviceProperties2( VK_SUBGROUP_FEATURE_BASIC_BIT | VK_SUBGROUP_FEATURE_BALLOT_BIT | VK_SUBGROUP_FEATURE_QUAD_BIT | - VK_SUBGROUP_FEATURE_SHUFFLE_BIT | - VK_SUBGROUP_FEATURE_SHUFFLE_RELATIVE_BIT | VK_SUBGROUP_FEATURE_VOTE_BIT; + if (pdevice->rad_info.chip_class >= VI) { + properties->supportedOperations |= + VK_SUBGROUP_FEATURE_SHUFFLE_BIT | + VK_SUBGROUP_FEATURE_SHUFFLE_RELATIVE_BIT; + } properties->quadOperationsInAllStages = true; break; } -- 2.30.2