Comme=
nt # 60
on bug 10617=
5
from Brandon Wright
> There are larger problems within am=
dgpu_dm's commit tail that if addressed=20
> should resolve this issue for compton I'd imagine.
Honestly, I don't care about compton. I don't think you realize the effects=
of
this issue. It seriously affects performance when the cursor is in motion w=
ith
any page-flipping application. GNOME and KDE, while the window motion is le=
ss
affected, stutter in composited client applications.=20
> This is a nice attempt but it only resolves the =
problem because it relies on
> the blocking behavior in atomic check that amdgpu_dm currently does=20
> (and shouldn't be doing).
>
> Asynchronous updates can and will occur in paral=
lel with other commits on=20
> worker threads. Without the wait in atomic_check you'll see the IGT le=
gacy=20
> cursor tests break with this patch (and there will probably be system =
faults=20
> as well).
You'd have to point this out to me, because I didn't see anything that would
obviously block, unless it's buried in dc_validate_plane.
Since, as you say, atomic_check is blocking for now, why not work around th=
is
issue with a tiny change. If someone ever gets around to doing things the
correct way it's no big deal to remove.