anv: Disable VK_KHR_16bit_storage
authorJason Ekstrand <jason.ekstrand@intel.com>
Fri, 8 Dec 2017 23:39:00 +0000 (15:39 -0800)
committerJason Ekstrand <jason.ekstrand@intel.com>
Fri, 8 Dec 2017 23:42:55 +0000 (15:42 -0800)
The testing for this extension is currently very poor.  The CTS tests
only test accessing UBOs and SSBOs at dynamic offsets so none of our
constant-offset paths get triggered at all.  Also, there's an assertion
in our handling of nir_intrinsic_load_uniform that offset % 4 == 0 which
is never triggered indicating that nothing every gets loaded from an
offset which is not a dword.  Both push constants and the constant
offset pull paths are complex enough, we really don't want to ship
without tests.  We'll turn the extension back on once we have decent
tests.

src/intel/vulkan/anv_device.c
src/intel/vulkan/anv_extensions.py

index 81a2ed632b2454e18b7d586135d1269cb52e88fc..55b77f90711d1c1a28fa498b69618e563f79e7a0 100644 (file)
@@ -745,8 +745,8 @@ void anv_GetPhysicalDeviceFeatures2KHR(
          VkPhysicalDevice16BitStorageFeaturesKHR *features =
             (VkPhysicalDevice16BitStorageFeaturesKHR *)ext;
 
-         features->storageBuffer16BitAccess = pdevice->info.gen >= 8;
-         features->uniformAndStorageBuffer16BitAccess = pdevice->info.gen >= 8;
+         features->storageBuffer16BitAccess = false;
+         features->uniformAndStorageBuffer16BitAccess = false;
          features->storagePushConstant16 = false;
          features->storageInputOutput16 = false;
          break;
index 3cdf6c173a861354ee0447da5bdbaa5e9f41c2ac..16e5582598e60306411cb3883c9da0e10d92f38c 100644 (file)
@@ -51,7 +51,7 @@ class Extension:
 # and dEQP-VK.api.info.device fail due to the duplicated strings.
 EXTENSIONS = [
     Extension('VK_ANDROID_native_buffer',                 5, 'ANDROID'),
-    Extension('VK_KHR_16bit_storage',                     1, 'device->info.gen >= 8'),
+    Extension('VK_KHR_16bit_storage',                     1, False),
     Extension('VK_KHR_bind_memory2',                      1, True),
     Extension('VK_KHR_dedicated_allocation',              1, True),
     Extension('VK_KHR_descriptor_update_template',        1, True),