Module: Mesa Branch: master Commit: 539de78633c45598e0b1e3b7763ea318f9200c32 URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=539de78633c45598e0b1e3b7763ea318f9200c32
Author: George Kyriazis <george.kyria...@intel.com> Date: Fri Feb 16 11:14:50 2018 -0600 swr/rast: Fix index buffer overfetch issue for non-indexed draws 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.chern...@intel.com> --- src/gallium/drivers/swr/rasterizer/core/frontend.cpp | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/src/gallium/drivers/swr/rasterizer/core/frontend.cpp b/src/gallium/drivers/swr/rasterizer/core/frontend.cpp index 1c4b522e45..c2be5d7bd1 100644 --- a/src/gallium/drivers/swr/rasterizer/core/frontend.cpp +++ b/src/gallium/drivers/swr/rasterizer/core/frontend.cpp @@ -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 _______________________________________________ mesa-commit mailing list mesa-commit@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-commit