[PATCH RESEND v13 0/5] Expose modifiers/formats supported by async flips

[Murthy, Arun R]

> -----Original Message-----
> From: Alex Deucher <alexdeucher at gmail.com>
> Sent: Wednesday, April 23, 2025 8:51 PM
> To: Borah, Chaitanya Kumar <chaitanya.kumar.borah at intel.com>; Wentland,
> Harry <Harry.Wentland at amd.com>; Leo (Sunpeng) Li <Sunpeng.Li at amd.com>
> Cc: Murthy, Arun R <arun.r.murthy at intel.com>; dri-
> devel at lists.freedesktop.org; intel-gfx at lists.freedesktop.org; intel-
> xe at lists.freedesktop.org; alexander.deucher at amd.com; Jani Nikula
> <jani.nikula at linux.intel.com>; Syrjala, Ville <ville.syrjala at intel.com>; Ville
> Syrjälä <ville.syrjala at linux.intel.com>; Kumar, Naveen1
> <naveen1.kumar at intel.com>
> Subject: Re: [PATCH RESEND v13 0/5] Expose modifiers/formats supported by
> async flips
> 
> + Harry and Leo
> 
> On Wed, Apr 16, 2025 at 2:33 AM Borah, Chaitanya Kumar
> <chaitanya.kumar.borah at intel.com> wrote:
> >
> > Hello Alexander,
> >
> > > -----Original Message-----
> > > From: Murthy, Arun R <arun.r.murthy at intel.com>
> > > Sent: Monday, April 7, 2025 11:14 AM
> > > To: dri-devel at lists.freedesktop.org;
> > > intel-gfx at lists.freedesktop.org; intel- xe at lists.freedesktop.org
> > > Cc: Jani Nikula <jani.nikula at linux.intel.com>; Borah, Chaitanya
> > > Kumar <chaitanya.kumar.borah at intel.com>; Syrjala, Ville
> > > <ville.syrjala at intel.com>; Murthy, Arun R <arun.r.murthy at intel.com>;
> > > Ville Syrjälä <ville.syrjala at linux.intel.com>; Kumar, Naveen1
> > > <naveen1.kumar at intel.com>
> > > Subject: [PATCH RESEND v13 0/5] Expose modifiers/formats supported
> > > by async flips
> > >
> > > All of the formats/modifiers supported by the plane during
> > > synchronous flips are nor supported by asynchronous flips. The
> > > formats/modifiers exposed to user by IN_FORMATS exposes all
> > > formats/modifiers supported by plane and this list varies for async
> > > flips. If the async flip supported formats/modifiers are exposed to
> > > the user, user based on this list can take decision to proceed or not and
> avoid flip failures during async flips.
> > > Discussion around this can be located @
> > > https://gitlab.freedesktop.org/mesa/mesa/-
> > > /merge_requests/29618#note_2487123
> > > Mutter implementation for IN_FORMARTS_ASYNC under review @
> > > https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/4063
> > > Xorg/modesetting patch
> > > https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1816
> > >
> >
> > We have added a new drm plane property to expose the format modifier pairs
> supported by a plane for async flip.
> > Can you please let us know if this implementation looks good to you? If so,
> kindly provide an Ack.
> >
> > IGT implementation: https://patchwork.freedesktop.org/series/146853/
> 
> Seems reasonable to me, but I'm not as involved in the display side anymore.
> I'd defer to Harry and Leo.  @Wentland, Harry, @Leo
> (Sunpeng) Li can you take a look?
> 
If no other comments on this, can we go ahead with the merge given that we have ACK from the following user space implementation also.
Mutter implementation
https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/4063
Xorg Implementation
https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1816
KDE Kwin Implementation:
https://invent.kde.org/plasma/kwin/-/merge_requests/7277
IGT implementation:
https://patchwork.freedesktop.org/series/146853/

Thanks and Regards,
Arun R Murthy
--------------------