[PATCH v2 0/5] amd/display: improve atomic cursor checks
Michel Dänzer
michel at daenzer.net
Wed Mar 17 08:32:47 UTC 2021
[ Adding Daniel again ]
On 2021-03-17 9:21 a.m., Simon Ser wrote:
> On Thursday, March 11th, 2021 at 3:13 PM, Michel Dänzer <michel at daenzer.net> wrote:
>
>>> I'm not a fan of adding kernel hacks like setting up a transparent FB, when
>>> user-space can just avoid the failure with atomic test-only commits (and e.g.
>>> use the overlay to display the cursor image instead of the cursor plane).
>>
>> I'm not a fan of requiring each atomic client to handle this complexity.
>
> That's just how atomic works though. User-space is expected to incrementally
> build the atomic request, bailing out if something doesn't work along the way.
Being unable to disable a plane which is currently enabled is quite different from being unable to enable a plane which is currently disabled. How is user space supposed to react to that, other than maybe disabling everything and starting from scratch?
> Doing it the old way (ie. issuing singular atomic commits, ie. using the atomic
> API just like the legacy API is used) won't work in many situations anyways.
This isn't about that, not sure why you keep bringing it up.
--
Earthling Michel Dänzer | https://redhat.com
Libre software enthusiast | Mesa and X developer
More information about the amd-gfx
mailing list