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

[Marek Olšák]

On Tue, Apr 2, 2019 at 4:57 AM Michel Dänzer <michel at daenzer.net> wrote:

> 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.
>

Only OpenCL uses the compute ring. xterm doesn't invoke OpenCL AFAIK.

Marek
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20190402/3897f297/attachment.html>