Fragment input components limit includes position
authorAlex Walters <alex.walters@imgtec.com>
Tue, 12 Feb 2019 17:50:27 +0000 (17:50 +0000)
committerAlexander Galazin <Alexander.Galazin@arm.com>
Thu, 21 Feb 2019 08:03:17 +0000 (03:03 -0500)
The Vulkan limits maxFragmentInputComponent is inclusive of any
components that are being used up by the built-in interface block.

This means that the test of n components should consist of 4 components
for the position and (n - 4) user defined components.

The test should also check the maxVertexOutputComponent limit as
technically there is no implied >= requirement.

Affects:

dEQP-VK.glsl.limits.near_max.fragment_input.components_*

Component: Vulkan

VK-GL-CTS Issue: 1597

Change-Id: I58adbbbaaa94dd185b024f63519dac7daf5e4b3c
(cherry picked from commit 8b5781b8e7a4564ddec376828a16fd89fb8e3227)

external/vulkancts/modules/vulkan/shaderrender/vktShaderRenderLimitTests.cpp

index 6c32e3c..e4cbfb5 100644 (file)
@@ -166,7 +166,22 @@ void FragmentInputComponentCase::initPrograms (SourceCollections& dst) const
                "        o_color = vec4(1.0, 0.0, 0.0, 1.0);\n"
                "}\n");
 
-       deUint16                                        maxLocations    = (deUint16)deCeilToInt32((float)m_inputComponents / 4u);
+       //
+       // The number of vertex output/fragment input components is *inclusive* of any built-ins being used,
+       // since gl_Position is always output by the shader, this actually means that there are n - 4 components
+       // available as user specified output data.
+       //
+       // [14.1.4. Location Assignment, para 11]
+       //
+       // "The number of input and output locations available for a shader input or output
+       //  interface are limited, and dependent on the shader stage as described in Shader
+       //  Input and Output Locations. All variables in both the built-in interface block
+       //  and the user-defined variable interface count against these limits."
+       //
+       // So, as an example, the '128' component variant of this test will specify 124 user
+       // declared outputs in addition to gl_Position.
+
+       deUint16                                        maxLocations    = (deUint16)deCeilToInt32((float)(m_inputComponents - 4) / 4u);
        string                                          varyingType;
        map<string, string>                     vertexParams;
        map<string, string>                     fragmentParams;
@@ -209,10 +224,21 @@ TestInstance* FragmentInputComponentCase::createInstance (Context& context) cons
        const VkPhysicalDevice                  physDevice                                      = context.getPhysicalDevice();
        const VkPhysicalDeviceLimits    limits                                          = getPhysicalDeviceProperties(vki, physDevice).limits;
        const deUint16                                  maxFragmentInputComponents      = (deUint16)limits.maxFragmentInputComponents;
+       const deUint16                                  maxVertexOutputComponents       = (deUint16)limits.maxVertexOutputComponents;
 
        if (m_inputComponents > maxFragmentInputComponents)
        {
-               const std::string notSupportedStr = "Unsupported number of fragment input components, fragmentInputComponents: " + de::toString(m_inputComponents);
+               const std::string notSupportedStr = "Unsupported number of fragment input components (" +
+                                                                                       de::toString(m_inputComponents) +
+                                                                                       ") maxFragmentInputComponents=" + de::toString(maxFragmentInputComponents);
+               TCU_THROW(NotSupportedError, notSupportedStr.c_str());
+       }
+
+       if (m_inputComponents > maxVertexOutputComponents)
+       {
+               const std::string notSupportedStr = "Unsupported number of user specified vertex output components (" +
+                                                                                       de::toString(m_inputComponents) +
+                                                                                       ") maxVertexOutputComponents=" + de::toString(maxVertexOutputComponents);
                TCU_THROW(NotSupportedError, notSupportedStr.c_str());
        }