Re: [PATCH i-g-t 3/4] igt_kms: Do not reset plane position on assigning a fb

From: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
To: "Konduru, Chandra" <chandra.konduru@intel.com>,
	"Intel-gfx@lists.freedesktop.org"
	<Intel-gfx@lists.freedesktop.org>
Subject: Re: [PATCH i-g-t 3/4] igt_kms: Do not reset plane position on assigning a fb
Date: Thu, 07 May 2015 10:15:06 +0100	[thread overview]
Message-ID: <554B2D1A.6070007@linux.intel.com> (raw)
In-Reply-To: <76A9B330A4D78C4D99CB292C4CC06C0E36FC71BE@fmsmsx101.amr.corp.intel.com>

On 05/06/2015 08:02 PM, Konduru, Chandra wrote:
>
>> diff --git a/lib/igt_kms.c b/lib/igt_kms.c index b5ba273..0665d70 100644
>> --- a/lib/igt_kms.c
>> +++ b/lib/igt_kms.c
>> @@ -1765,9 +1765,7 @@ void igt_plane_set_fb(igt_plane_t *plane, struct
>> igt_fb *fb)
>>   	plane->fb = fb;
>>   	/* hack to keep tests working that don't call igt_plane_set_size() */
>>   	if (fb) {
>> -		/* set default plane pos/size as fb size */
>> -		plane->crtc_x = 0;
>> -		plane->crtc_y = 0;
> Reason for doing this is to make existing kms tests to run without modification.
> Otherwise every existing test has to explicitly call igt_plane_set_position to make
> sure it works. Can you make sure removing above 2 lines doesn't break existing tests?

Hm, before your patch tests could set a plane position, change the fb, 
and plane position would remain the same. After your patch changing the 
fb resets the plane position. My patch reverts this behaviour to old one.

So I am not sure in what cases resetting plane position on fb changes 
helps and which tests?

Which tests do you think should be run?

Regards,

Tvrtko
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx