[Intel-gfx] [PATCHv3] drm/i915: Support Async Flip on Linear buffers

[Murthy, Arun R]

Ville,
	Gentle reminder!

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

> -----Original Message-----
> From: Murthy, Arun R
> Sent: Monday, September 19, 2022 10:38 AM
> To: 'intel-gfx at lists.freedesktop.org' <intel-gfx at lists.freedesktop.org>
> Cc: Syrjala, Ville <ville.syrjala at intel.com>
> Subject: RE: [PATCHv3] drm/i915: Support Async Flip on Linear buffers
> 
> If no comments, can anyone merge the patch!
> 
> Thanks and Regards,
> Arun R Murthy
> --------------------
> 
> > -----Original Message-----
> > From: Murthy, Arun R
> > Sent: Wednesday, September 14, 2022 4:21 PM
> > To: Murthy, Arun R <arun.r.murthy at intel.com>; intel-
> > gfx at lists.freedesktop.org
> > Cc: Syrjala, Ville <ville.syrjala at intel.com>
> > Subject: RE: [PATCHv3] drm/i915: Support Async Flip on Linear buffers
> >
> > Gentle Reminder!
> > Any comments?
> >
> > Thanks and Regards,
> > Arun R Murthy
> > --------------------
> >
> > > -----Original Message-----
> > > From: Intel-gfx <intel-gfx-bounces at lists.freedesktop.org> On Behalf
> > > Of Murthy, Arun R
> > > Sent: Friday, September 9, 2022 9:17 AM
> > > To: intel-gfx at lists.freedesktop.org
> > > Cc: Syrjala, Ville <ville.syrjala at intel.com>
> > > Subject: Re: [Intel-gfx] [PATCHv3] drm/i915: Support Async Flip on
> > > Linear buffers
> > >
> > > Gentle Reminder!
> > >
> > > > -----Original Message-----
> > > > From: Murthy, Arun R <arun.r.murthy at intel.com>
> > > > Sent: Tuesday, September 6, 2022 9:18 AM
> > > > To: intel-gfx at lists.freedesktop.org
> > > > Cc: ville.syrjala at linux.intel.com; Murthy, Arun R
> > > > <arun.r.murthy at intel.com>
> > > > Subject: [PATCHv3] drm/i915: Support Async Flip on Linear buffers
> > > >
> > > > Starting from Gen12 Async Flip is supported on linear buffers.
> > > > This patch enables support for async on linear buffer.
> > > >
> > > > UseCase: In Hybrid graphics, for hardware unsupported pixel
> > > > formats it will be converted to linear memory and then composed.
> > > >
> > > > v2: Added use case
> > > > v3: Added FIXME for ICL indicating the restrictions
> > > >
> > > > Signed-off-by: Arun R Murthy <arun.r.murthy at intel.com>
> > > > ---
> > > >  drivers/gpu/drm/i915/display/intel_display.c | 14 ++++++++++++++
> > > >  1 file changed, 14 insertions(+)
> > > >
> > > > diff --git a/drivers/gpu/drm/i915/display/intel_display.c
> > > > b/drivers/gpu/drm/i915/display/intel_display.c
> > > > index be7cff722196..1880cfe70a7d 100644
> > > > --- a/drivers/gpu/drm/i915/display/intel_display.c
> > > > +++ b/drivers/gpu/drm/i915/display/intel_display.c
> > > > @@ -6610,6 +6610,20 @@ static int intel_async_flip_check_hw(struct
> > > > intel_atomic_state *state, struct in
> > > >  		 * this selectively if required.
> > > >  		 */
> > > >  		switch (new_plane_state->hw.fb->modifier) {
> > > > +		case DRM_FORMAT_MOD_LINEAR:
> > > > +			/*
> > > > +			 * FIXME: Async on Linear buffer is supported on ICL
> > > > as
> > > > +			 * but with additional alignment and fbc restrictions
> > > > +			 * need to be taken care of. These aren't applicable
> > > > for
> > > > +			 * gen12+.
> > > > +			 */
> > > > +			if (DISPLAY_VER(i915) < 12) {
> > > > +				drm_dbg_kms(&i915->drm,
> > > > +					"[PLANE:%d:%s] Modifier does not
> > > > support async flips\n",
> > > > +					plane->base.base.id, plane-
> > > > >base.name);
> > > > +				return -EINVAL;
> > > > +			}
> > > > +
> > > >  		case I915_FORMAT_MOD_X_TILED:
> > > >  		case I915_FORMAT_MOD_Y_TILED:
> > > >  		case I915_FORMAT_MOD_Yf_TILED:
> > > > --
> > > > 2.25.1