swr/rast: Fix index buffer overfetch issue for non-indexed draws
authorGeorge Kyriazis <george.kyriazis@intel.com>
Fri, 16 Feb 2018 17:14:50 +0000 (11:14 -0600)
committerGeorge Kyriazis <george.kyriazis@intel.com>
Wed, 28 Feb 2018 17:42:19 +0000 (11:42 -0600)
Populate pLastIndex, even for the non-indexed case.  An zero pLastIndex
can cause the index offsets inside the fetcher to have non-sensical values
that can be either very large positive or very large negative numbers.

Reviewed-By: Bruce Cherniak <bruce.cherniak@intel.com>
src/gallium/drivers/swr/rasterizer/core/frontend.cpp

index 1c4b522e45ea67c7ce755c63568c791bf2c55e9e..c2be5d7bd11c6d5975f1677cc6d691d7c9efeb28 100644 (file)
@@ -1719,6 +1719,21 @@ void ProcessDraw(
 
             if (i < endVertex)
             {
+                if (!IsIndexedT::value)
+                {
+                    fetchInfo_lo.pLastIndex = fetchInfo_lo.pIndices;
+                    uint32_t offset;
+                    offset = std::min(endVertex-i, (uint32_t) KNOB_SIMD16_WIDTH);
+#if USE_SIMD16_SHADERS
+                    fetchInfo_lo.pLastIndex += offset;
+#else
+                    fetchInfo_lo.pLastIndex += std::min(offset, (uint32_t) KNOB_SIMD_WIDTH);
+                    uint32_t offset2 = std::min(offset, (uint32_t) KNOB_SIMD16_WIDTH)-KNOB_SIMD_WIDTH;
+                    assert(offset >= 0);
+                    fetchInfo_hi.pLastIndex = fetchInfo_hi.pIndices;
+                    fetchInfo_hi.pLastIndex += offset2;
+#endif
+                }
                 // 1. Execute FS/VS for a single SIMD.
                 RDTSC_BEGIN(FEFetchShader, pDC->drawId);
 #if USE_SIMD16_SHADERS