[Mesa-dev] [PATCH 1/2] radeonsi: always use compute rings for clover on CI and newer (v2)

Michel Dänzer michel at daenzer.net
Tue Apr 2 08:57:37 UTC 2019


On 2019-04-02 12:39 a.m., Marek Olšák wrote:
> On Mon, Apr 1, 2019 at 1:28 PM Jan Vesely <jan.vesely at rutgers.edu> wrote:
>> On Mon, 2019-04-01 at 12:30 -0400, Marek Olšák wrote:
>>> Does the attached patch fix the copy-buffer test?
>>
>> it does thanks.
>> Won't the compute only context still need some synchronization?
>> Is there anything else to guarantee that the data is in place after
>> return from resource_copy_region ?
> 
> The synchronization is the same on gfx and compute rings.

BTW, did you see https://bugs.freedesktop.org/show_bug.cgi?id=110214#c24
? It does indicate some kind of synchronization issue between
si_resource_copy_region using a compute ring and other operations using
a GFX ring.


-- 
Earthling Michel Dänzer               |              https://www.amd.com
Libre software enthusiast             |             Mesa and X developer


More information about the mesa-dev mailing list