{"action":"created","issue":{"url":"https://api.github.com/repos/KhronosGroup/Vulkan-CTS/issues/3","repository_url":"https://api.github.com/repos/KhronosGroup/Vulkan-CTS","labels_url":"https://api.github.com/repos/KhronosGroup/Vulkan-CTS/issues/3/labels{/name}","comments_url":"https://api.github.com/repos/KhronosGroup/Vulkan-CTS/issues/3/comments","events_url":"https://api.github.com/repos/KhronosGroup/Vulkan-CTS/issues/3/events","html_url":"https://github.com/KhronosGroup/Vulkan-CTS/issues/3","id":134801346,"number":3,"title":"API version patch level behaviour","user":{"login":"hlandau","id":2270010,"avatar_url":"https://avatars.githubusercontent.com/u/2270010?v=3","gravatar_id":"","url":"https://api.github.com/users/hlandau","html_url":"https://github.com/hlandau","followers_url":"https://api.github.com/users/hlandau/followers","following_url":"https://api.github.com/users/hlandau/following{/other_user}","gists_url":"https://api.github.com/users/hlandau/gists{/gist_id}","starred_url":"https://api.github.com/users/hlandau/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hlandau/subscriptions","organizations_url":"https://api.github.com/users/hlandau/orgs","repos_url":"https://api.github.com/users/hlandau/repos","events_url":"https://api.github.com/users/hlandau/events{/privacy}","received_events_url":"https://api.github.com/users/hlandau/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2016-02-19T08:06:37Z","updated_at":"2016-02-25T18:24:44Z","closed_at":null,"body":"These tests appear to verify that a request for an instance with an API version of X.Y.W is rejected where W>Z and a driver supports X.Y.Z. Based on my reading of the specification, this is wrong and on the contrary, the conformance tests should ensure that a driver for X.Y.Z accepts a request for X.Y.W for W>Z.\r\n\r\nThe specification states that patch-level version changes should preserve \"full compatibility\". Thus, based on the definition of \"full compatibility\" therein, it is acceptable to provide X.Y.Z in response to a request for X.Y.W for W>Z.\r\n\r\nThis seems like the only sane implementation mode, otherwise newer code compiled for X.Y.W will fail when run against perfectly viable drivers for X.Y.Z. This has already happened at launch with the AMD driver, 1.0.2 vs. 1.0.3, but code changed to specify 1.0.2 works fine.\r\n\r\nhttps://github.com/KhronosGroup/Vulkan-CTS/blob/vulkan-cts-1.0/external/vulkancts/modules/vulkan/api/vktApiDeviceInitializationTests.cpp#L189"},"comment":{"url":"https://api.github.com/repos/KhronosGroup/Vulkan-CTS/issues/comments/188918494","html_url":"https://github.com/KhronosGroup/Vulkan-CTS/issues/3#issuecomment-188918494","issue_url":"https://api.github.com/repos/KhronosGroup/Vulkan-CTS/issues/3","id":188918494,"user":{"login":"phaulos","id":295496,"avatar_url":"https://avatars.githubusercontent.com/u/295496?v=3","gravatar_id":"","url":"https://api.github.com/users/phaulos","html_url":"https://github.com/phaulos","followers_url":"https://api.github.com/users/phaulos/followers","following_url":"https://api.github.com/users/phaulos/following{/other_user}","gists_url":"https://api.github.com/users/phaulos/gists{/gist_id}","starred_url":"https://api.github.com/users/phaulos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/phaulos/subscriptions","organizations_url":"https://api.github.com/users/phaulos/orgs","repos_url":"https://api.github.com/users/phaulos/repos","events_url":"https://api.github.com/users/phaulos/events{/privacy}","received_events_url":"https://api.github.com/users/phaulos/received_events","type":"User","site_admin":false},"created_at":"2016-02-25T18:24:44Z","updated_at":"2016-02-25T18:24:44Z","body":"The issue here is that driver can't know if 1.0.3 is compatible with 1.0.0 if it only implements 1.0.0 so it has no other choice than reject the request.\r\n\r\nApplications should set `apiVersion` to the minimum version they need to run, not to `VK_API_VERSION`."}}