Am 28.12.2017 um 02:16 schrieb Alex Deucher: > On Wed, Dec 27, 2017 at 7:51 PM, Dave Airlie <[email protected]> wrote: >> On 23 December 2017 at 17:19, <[email protected]> wrote: >>> From: Roland Scheidegger <[email protected]> >>> >>> piglit doesn't care, but I'm quite confident that the size actually bound >>> as range should be reported and not the base size of the resource. >>> Also, the array in the constant buffer looks overallocated by a factor of 4. >>> For eg, also decrease the size by another factor of 2 by using the same >>> constant slot for both buffer size (required for txq for TBOs) and the >>> number >>> of layers for cube arrays, as these are mutually exclusive. Could of course >>> use >>> some more logic and only actually do this for the samplers/images/buffers >>> where >>> it's required rather than for all, but ah well... >>> (FWIW I believe the txq for TBOs would be fixable on EG without using a >>> constant buffer by using the GET_BUFFER_RESINFO vc fetch, but for cube map >>> arrays we'd still need the buffer as it's unfixable since the hw requires >>> always 0 unfortunately.) >> >> For the series: >> >> Reviewed-by: Dave Airlie <[email protected]> >> from previous experiments the hw doesn't work at all using vc fetch or >> otherwise. > > vc vs tc fetch just determines which cache is used. vc was slower > than tc and was not present on some asics and was eventually removed > altogether.
Not according to the docs for this specific instruction: GET_BUFFER_RESINFO Returns the number of elements in a buffer. This is a vertex fetch instruction and uses a vertex constant. It can be used only by a texture cache, not a vertex cache. Albeit that restriction is only in the r800 isa docs, not NI (r600 doesn't support it in the first place). But if it doesn't work either way, I'll just adjust the comment for the patch... Roland > Alex > >> >> Dave. >> _______________________________________________ >> mesa-dev mailing list >> [email protected] >> https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.freedesktop.org_mailman_listinfo_mesa-2Ddev&d=DwIBaQ&c=uilaK90D4TOVoH58JNXRgQ&r=_QIjpv-UJ77xEQY8fIYoQtr5qv8wKrPJc7v7_-CYAb0&m=VFFmGFm56BHuS8VYK1t0LK0HZCTASA0LdgVXxHJZWYw&s=AbgSWv099aF3iDuTasfbRiWjEQo5f-kXmT9YFf0Nucg&e= _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
