[Mesa-dev] [PATCH 4/5] r600g: use a clever alignment for index buffer uploads

Constantine Kharlamov Hi-Angel at yandex.ru
Sat Apr 1 08:55:36 UTC 2017


Thank you, please, could you push them too? I don't have commit access.

On 28.03.2017 22:36, Marek Olšák wrote:
> Actually nevermind.
> 
> Patches 1-3:
> Reviewed-by: Marek Olšák <marek.olsak at amd.com>
> 
> Patches 4-5:
> These have no effect, because r600g hw doesn't use TC for index fetches.
> 
> Marek
> 
> 
> On Tue, Mar 28, 2017 at 9:33 PM, Marek Olšák <maraeo at gmail.com> wrote:
>> BTW, I don't know if tcc_cache_line_size has the correct value for
>> pre-GCN chips.
>>
>> Marek
>>
>> On Sun, Mar 26, 2017 at 5:36 PM, Constantine Kharlamov
>> <Hi-Angel at yandex.ru> wrote:
>>> Stolen from radeonsi
>>>
>>> Signed-off-by: Constantine Kharlamov <Hi-Angel at yandex.ru>
>>> ---
>>>  src/gallium/drivers/r600/r600_state_common.c | 3 ++-
>>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/src/gallium/drivers/r600/r600_state_common.c b/src/gallium/drivers/r600/r600_state_common.c
>>> index 961e04e9db..5293c4034f 100644
>>> --- a/src/gallium/drivers/r600/r600_state_common.c
>>> +++ b/src/gallium/drivers/r600/r600_state_common.c
>>> @@ -1766,7 +1766,8 @@ static void r600_draw_vbo(struct pipe_context *ctx, const struct pipe_draw_info
>>>                                                  info->instance_count > 1 ||
>>>                                                  info->count*ib.index_size > 20)) {
>>>                         u_upload_data(ctx->stream_uploader, 0,
>>> -                                      info->count * ib.index_size, 256,
>>> +                                     info->count * ib.index_size,
>>> +                                     rctx->screen->b.info.tcc_cache_line_size,
>>>                                       ib.user_buffer, &ib.offset, &ib.buffer);
>>>                         ib.user_buffer = NULL;
>>>                 }
>>> --
>>> 2.12.0
>>>
>>> _______________________________________________
>>> mesa-dev mailing list
>>> mesa-dev at lists.freedesktop.org
>>> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
> 


More information about the mesa-dev mailing list