Reviewed-by: Christian Gmeiner <christian.gmei...@gmail.com>
2017-11-05 16:23 GMT+01:00 Christian Gmeiner <christian.gmei...@gmail.com>:
> 2017-11-05 13:38 GMT+01:00 Wladimir <laa...@gmail.com>:
>>>> + if (!ctx->specs.use_blt) {
>>>> + /* This (ab)uses the RS as a plain buffer memset().
>>>> + * Currently uses a fixed row size of 64 bytes. Some
>>>> benchmarking with
>>>> + * different sizes may be in order. */
>>>> + struct etna_bo *ts_bo = etna_resource(surf->base.texture)->ts_bo;
>>>> + etna_compile_rs_state(ctx, &surf->clear_command, &(struct
>>>> rs_state) {
>>>> + .source_format = RS_FORMAT_A8R8G8B8,
>>>> + .dest_format = RS_FORMAT_A8R8G8B8,
>>>> + .dest = ts_bo,
>>>> + .dest_offset = surf->surf.ts_offset,
>>>> + .dest_stride = 0x40,
>>>> + .dest_tiling = ETNA_LAYOUT_TILED,
>>>> + .dither = {0xffffffff, 0xffffffff},
>>>> + .width = 16,
>>>> + .height = etna_align_up(surf->surf.ts_size / 0x40, 4),
>>>> + .clear_value = {ctx->specs.ts_clear_value},
>>>> + .clear_mode = VIVS_RS_CLEAR_CONTROL_MODE_ENABLED1,
>>>> + .clear_bits = 0xffff
>>>> + });
>>>> + }
>>>> } else {
>>>> - etna_rs_gen_clear_surface(ctx, surf, surf->level->clear_value);
>>>> + if (!ctx->specs.use_blt)
>>>> + etna_rs_gen_clear_surface(ctx, surf, surf->level->clear_value);
>>>> }
>>>>
>>>
>>> if (!ctx->specs.use_blt) {
>>> } else {
>>> if (!ctx->specs.use_blt)
>>> ..
>>> }
>>>
>>> Looks funny... btw. do you have a git branch somewhere to look at this
>>> sereis?
>>
>> It looks somewhat funny but I don't think it's wrong.
>> There's an extra } there. Context is:
>>
>> if (surf->surf.ts_size) {
>> if (!ctx->specs.use_blt) {
>> ...
>> }
>> } else {
>> if (!ctx->specs.use_blt)
>> ..
>> }
>>
>> My current working tree can be found at
>> https://github.com/laanwj/mesa/tree/gc7000. It contains these patches
>> (but also some temporary local scaffolding, so be warned).
>
> Then it might not be as useful as I need it.
>
> --
> greets
> --
> Christian Gmeiner, MSc
>
> https://christian-gmeiner.info
--
greets
--
Christian Gmeiner, MSc
https://christian-gmeiner.info
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
