[Mesa-dev] [PATCH v3 1/7] radeonsi: Allocate buffers for DCC.
Marek Olšák
maraeo at gmail.com
Thu Oct 22 01:41:37 PDT 2015
On Wed, Oct 21, 2015 at 9:56 AM, Axel Davy <axel.davy at ens.fr> wrote:
> On 21/10/2015 00:10, Bas Nieuwenhuizen wrote:
>>
>>
>> DCC is disabled for textures that can be shared as sharing the
>> DCC buffers has not been implemented yet.
>>
>>
>> + surf->dcc_enabled = !(surf->flags & RADEON_SURF_Z_OR_SBUFFER) &&
>> + !(surf->flags & RADEON_SURF_SCANOUT) &&
>> + !compressed && AddrDccIn.numSamples <= 1;
>> +
>>
>
> Testing if a surface is scanout is not enough to avoid shared surfaces.
>
> In practice, it may be true currently mesa, and glamor via gbm, would use
> the scanout flag for shared
> buffers. It seems however a bit weak to rely on that.
>
> I suggest rather to use the pipe shared bind flag.
>
> I noticed in some case of imported surfaces the bind flag is not
> advertised, I'm going to send a patch to fix that.
I think that using any BIND flag is weak. The best thing would be to
make some decisions in texture_from_handle and texture_get_handle
functions, not based on flags.
Marek
More information about the mesa-dev
mailing list