[Mesa-dev] [PATCH] r600g, radeonsi: add debug option which forces DMA for copy_region and blit

[Grigori Goronzy]

On 08.09.2014 21:07, Axel Davy wrote:
> On 08/09/2014 20:21, Grigori Goronzy wrote :
>> On 08.09.2014 14:50, Axel Davy wrote:
>>> Hi,
>>>
>>> When reading si_dma.c code, it looks like the requested width of the
>>> copy is ignored except for PIPE_BUFFER.
>>> Perhaps that explains the bugs observed ?
>>>
>> It isn't ignored. Partial DMA copies (i.e. operations that do not copy
>> whole lines) are simply not supported right now, and will fall back to
>> resource_copy_region. In fact, it's even stricter: the picthes of source
>> and destination have to match.
>>
>> Grigori
> My point is I don't see a check to verify the width of the copy equals
> the width of the buffers,
> even if I see the pitch test.
>

There's a check for that (src_w != dst_w) in si_dma_copy.

Grigori

> Axel
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20140908/479d4a7f/attachment.sig>