[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