On 03/05/2018 01:46 PM, Emil Velikov wrote:
Hi Tapani,

On 5 March 2018 at 08:57, Tapani Pälli <[email protected]> wrote:
Just like commit 2ffe395 does for radv.

Fixes following dEQP test on i965:
    dEQP-VK.api.info.android.no_unknown_extensions

v2: make it !ANDROID since this extension is not about
     surfaces/swapchain

CC: 18.0 <[email protected]>
Signed-off-by: Tapani Pälli <[email protected]>
---
  src/intel/vulkan/anv_extensions.py | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/intel/vulkan/anv_extensions.py 
b/src/intel/vulkan/anv_extensions.py
index 8d39038c43..fd0d3337d0 100644
--- a/src/intel/vulkan/anv_extensions.py
+++ b/src/intel/vulkan/anv_extensions.py
@@ -83,7 +83,7 @@ EXTENSIONS = [
      Extension('VK_KHR_wayland_surface',                   6, 
'VK_USE_PLATFORM_WAYLAND_KHR'),
      Extension('VK_KHR_xcb_surface',                       6, 
'VK_USE_PLATFORM_XCB_KHR'),
      Extension('VK_KHR_xlib_surface',                      6, 
'VK_USE_PLATFORM_XLIB_KHR'),
-    Extension('VK_KHX_multiview',                         1, True),
+    Extension('VK_KHX_multiview',                         1, '!ANDROID'),

As reminded by Bas [0], the extensions should not be advertised in
released Mesa altogether.
Hence, I've adapted Jason's old patch for anv and radv on the 17.3 and
18.0 branches [1] [2] [3] [4].

OK cool, I missed this conversation. Then I'll drop the stable tag!

I also have a (horrible) patch to make dEQP-VK.api.info.device.extensions pass [1]. Problem with that test is that it hardcodes a list of 'accepted extensions' and everything outside that list triggers an error. To make it pass we actually need to disable nearly every entry in this list. Maybe a nicer long term solution we should have a separate EXTENSIONS list for Android or specific enable flag for Android ones (instead of disable)?

[1] https://github.com/tpalli/external-mesa/commit/8af6a35fee3a3d18fb947658182d5e4e400f438f


-Emil

[0] https://lists.freedesktop.org/archives/mesa-dev/2018-February/184273.html
https://lists.freedesktop.org/archives/mesa-dev/2018-February/184899.html

[1] 
https://cgit.freedesktop.org/mesa/mesa/commit/?h=17.3&id=87ffdbae1cdd7975047abdcfef740a75bfdbffd5
[2] 
https://cgit.freedesktop.org/mesa/mesa/commit/?h=17.3&id=2b9e16d182a7ce45c4028c440807448a209b08b0
[3] 
https://cgit.freedesktop.org/mesa/mesa/commit/?h=18.0&id=d50d11f84bdcc402a513b26318db849a71680ee8
[4] 
https://cgit.freedesktop.org/mesa/mesa/commit/?h=18.0&id=a307f0696987a4a4f5b0a42ac1613e1a7877927d


// Tapani
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to