Re: [PATCH v2 0/5] amd/display: improve atomic cursor checks

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2021-03-11 9:57 a.m., Simon Ser wrote:
> On Wednesday, March 10th, 2021 at 6:20 PM, Michel Dänzer <michel@xxxxxxxxxxx> wrote:
>> On 2021-03-10 3:50 p.m., Simon Ser wrote:
>>
>>> Changes in v2: drop "amd/display: fail on cursor plane without an
>>> underlying plane". This retains the current behavior instead.
>>
>> Patches 2 & 3 (and possibly 4? not sure) would still cause the same
>> issue in the scenario discussed in follow-ups to the dropped patch
>> (disabling an ARGB overlay plane with a YUV primary plane and the
>> cursor plane enabled), wouldn't they?
> 
> Yes, but that shouldn't be too much of an issue in practice, since
> user-space using YUV and overlays also uses atomic. Dropping the patch
> avoids the issue to be triggered with legacy user-space.

The last scenario discussed is about atomic KMS user space trying to disable the overlay plane. Daniel seems to agree that not being able to disable an overlay plane is too surprising.


-- 
Earthling Michel Dänzer               |               https://redhat.com
Libre software enthusiast             |             Mesa and X developer
_______________________________________________
amd-gfx mailing list
amd-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/amd-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux