From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 1/1] demos: Add intel_plane_rotate test to verify rotation of planes and crtc Date: Tue, 4 Feb 2014 14:08:27 +0200 Message-ID: <20140204120827.GF3891@intel.com> References: <1391195628-9067-1-git-send-email-sagar.a.kamble@intel.com> <20140204111824.GM17001@phenom.ffwll.local> <20140204112530.GA31692@strange.amr.corp.intel.com> <1391513747.21527.8.camel@sagar-desktop> <20140204114646.GA25630@strange.amr.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by gabe.freedesktop.org (Postfix) with ESMTP id 8F208FA856 for ; Tue, 4 Feb 2014 04:08:51 -0800 (PST) Content-Disposition: inline In-Reply-To: <20140204114646.GA25630@strange.amr.corp.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces@lists.freedesktop.org Errors-To: intel-gfx-bounces@lists.freedesktop.org To: Damien Lespiau Cc: intel-gfx@lists.freedesktop.org, Sagar Arun Kamble List-Id: intel-gfx@lists.freedesktop.org On Tue, Feb 04, 2014 at 11:46:46AM +0000, Damien Lespiau wrote: > On Tue, Feb 04, 2014 at 05:05:47PM +0530, Sagar Arun Kamble wrote: > > On Tue, 2014-02-04 at 11:25 +0000, Damien Lespiau wrote: > > > On Tue, Feb 04, 2014 at 12:18:24PM +0100, Daniel Vetter wrote: > > > > On Sat, Feb 01, 2014 at 12:43:48AM +0530, sagar.a.kamble@intel.com = wrote: > > > > > From: Sagar Kamble > > > > > = > > > > > This test will verify the 180 degree rotation of sprite and crtc = planes. > > > > > It will allow user to control rotation separately for crtc and sp= rite > > > > > planes. > > > > > = > > > > > Signed-off-by: Sagar Kamble > > > > = > > > > What I actually want for upstreaming is a fully automated testcase = as part > > > > of our i-g-t testsuite which uses the CRC support. Having testing t= ools > > > > for developers is neat, but if it's not automated it's essentially > > > > untested. > > > > = > > > > I don't mind merging this if there's someone else who'll ack it tho= ugh. > > > = > > > While having such a tool helps development for sure, let's try to tak= e a > > > better approach on this. > > > = > > > I'm currently reworking how we write display tests to not have 1000 > > > lines code to setup the display pipeline in every test, leave me a few > > > days and we should be in a much better place to write kms tests. Of > > > course, we also really want a CRC enabled test. > > > = > > = > > I am trying to enable CRC approach for this test however I have > > following opens: > > 1. How do I calculate CRC for reference image(blend of primary and > > overlay planes) prepared by i-g-t test. > = > We can't do that, the algorithms (that can change on a per platform > basis) for computing the CRCs isn't public. What we can do on the other > hand is comparing two CRCs and figure out if what we are testing is the > same as a reference frame. > = > For the 180 rotation case, we can, for instance: > = > - draw a 180 rotated fb with cairo, grab that CRC > - draw a 0 rotated fb, rotate it with hardware, grab that CRC > - the two CRCs should be equal > = > For the fb we can paint the upper half in red and the lower half in > blue. The 180 rotated version would do the opposite. Or mauybe even each corner in different color to make sure it gets 180 degree rotated and not just flipped across one axis. Not really useful for now since we only have 180 degree rotation, but we might get more options at some point so might as well make the test ready for it. -- = Ville Syrj=E4l=E4 Intel OTC