[igt-dev] [PATCH i-g-t 6/6] tests/kms_async_flips: Test all modifiers

[Murthy, Arun R]

> -----Original Message-----
> From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> Sent: Friday, February 3, 2023 6:00 PM
> To: Murthy, Arun R <arun.r.murthy at intel.com>
> Cc: igt-dev at lists.freedesktop.org
> Subject: Re: [PATCH i-g-t 6/6] tests/kms_async_flips: Test all modifiers
> 
> On Thu, Feb 02, 2023 at 11:23:09AM +0000, Murthy, Arun R wrote:
> > > -----Original Message-----
> > > From: Ville Syrjala <ville.syrjala at linux.intel.com>
> > > Sent: Tuesday, January 31, 2023 5:37 PM
> > > To: igt-dev at lists.freedesktop.org
> > > Cc: Murthy, Arun R <arun.r.murthy at intel.com>
> > > Subject: [PATCH i-g-t 6/6] tests/kms_async_flips: Test all modifiers
> > >
> > > From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> > >
> > > Run the basic async flip test for all modifiers reported by the
> > > plane (for the
> > > XRGB8888 format). The driver may not support async flip with all
> > > modifiers so we allow this to fail.
> >
> > Support for adding X-tile and linear for all platforms is being added
> > as sub-test in https://patchwork.freedesktop.org/series/103137/
> 
> I presume you mean:
> https://patchwork.freedesktop.org/patch/515577/?series=107871&rev=11
> 
> That does nothing to get rid of the i915isms, nor does do anything to test all
> the other modifiers (both now and in the future).
> 
Yes it doesn't test on all other modifiers but adds support for x-tiled and linear.
Testing all supported modifiers is a good one and have no comments on that.

Thanks and Regards,
Arun R Murthy
-------------------
> --
> Ville Syrjälä
> Intel