IIRC if level N is unaligned then num_dcc_levels <= N+1, so level N+1
is not DCC compressed?
On Wed, Apr 18, 2018 at 12:53 PM, Marek Olšák <mar...@gmail.com> wrote:
> On Wed, Apr 18, 2018 at 5:54 AM, Nicolai Hähnle <nhaeh...@gmail.com> wrote:
>>
>> On 17.04.2018 02:41, Marek Olšák wrote:
>>>
>>> From: Marek Olšák <marek.ol...@amd.com>
>>>
>>> ---
>>> src/amd/common/ac_surface.c | 14 +++++++++++++-
>>> 1 file changed, 13 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/src/amd/common/ac_surface.c b/src/amd/common/ac_surface.c
>>> index 7558dd91e34..c209b209da2 100644
>>> --- a/src/amd/common/ac_surface.c
>>> +++ b/src/amd/common/ac_surface.c
>>> @@ -344,24 +344,36 @@ static int gfx6_compute_level(ADDR_HANDLE addrlib,
>>> AddrDccIn->tileInfo = *AddrSurfInfoOut->pTileInfo;
>>> AddrDccIn->tileIndex = AddrSurfInfoOut->tileIndex;
>>> AddrDccIn->macroModeIndex =
>>> AddrSurfInfoOut->macroModeIndex;
>>> ret = AddrComputeDccInfo(addrlib,
>>> AddrDccIn,
>>> AddrDccOut);
>>> if (ret == ADDR_OK) {
>>> surf_level->dcc_offset = surf->dcc_size;
>>> - surf_level->dcc_fast_clear_size =
>>> AddrDccOut->dccFastClearSize;
>>> surf->num_dcc_levels = level + 1;
>>> surf->dcc_size = surf_level->dcc_offset +
>>> AddrDccOut->dccRamSize;
>>> surf->dcc_alignment = MAX2(surf->dcc_alignment,
>>> AddrDccOut->dccRamBaseAlign);
>>> +
>>> + /* If the DCC size of a subresource (1 mip level
>>> or 1 slice)
>>> + * is not aligned, the DCC memory layout is not
>>> contiguous for
>>> + * that subresource, which means we can't use
>>> fast clear.
>>> + *
>>> + * We only do fast clears for whole mipmap
>>> levels. If we did
>>> + * per-slice fast clears, the same restriction
>>> would apply.
>>> + * (i.e. only compute the slice size and see if
>>> it's aligned)
>>> + */
>>> + if (level == config->info.levels - 1 ||
>>> AddrDccOut->dccRamSizeAligned)
>>
>>
>> Are you sure that the last level is always okay? It seems possible that
>> the last and second-to-last level end up interleaved due to swizzling.
>>
>> The catch is that you'd probably only ever notice this with a truncated
>> mip-tree.
>
>
> This condition was copied from amdvlk. I guess that if the second-to-last
> level is unaligned, the last level would also be unaligned.
>
> We never use fast clear for non-zero level. For now, we only have to make
> sure that level 0 is contiguous.
>
> Marek
>
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
