[PATCH igt 1/2] lib: Add igt_plane_set_size()

[PATCH igt 1/2] lib: Add igt_plane_set_size()

[ville.syrjala]

From: Ville Syrjälä <ville.syrjala@linux.intel.com>

Allow tests to specify the plane size instead of assuming that the
entire FB will be scanned out.

To keep the current tests working without having to sprinkle
igt_plane_set_size() calls all over the place, make
igt_plane_set_fb() reset the plane size to the FB size.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
 lib/igt_kms.c | 34 ++++++++++++++++++++++++++++++----
 lib/igt_kms.h |  3 +++
 2 files changed, 33 insertions(+), 4 deletions(-)

diff --git a/lib/igt_kms.c b/lib/igt_kms.c
index 3a850f1..0547147 100644
--- a/lib/igt_kms.c
+++ b/lib/igt_kms.c
@@ -1269,7 +1269,7 @@ static int igt_drm_plane_commit(igt_plane_t *plane,
 				      fb_id,
 				      0,    /* flags */
 				      plane->crtc_x, plane->crtc_y,
-				      plane->fb->width, plane->fb->height,
+				      plane->crtc_w, plane->crtc_h,
 				      IGT_FIXED(0,0), /* src_x */
 				      IGT_FIXED(0,0), /* src_y */
 				      IGT_FIXED(plane->fb->width,0), /* src_w */
@@ -1314,12 +1314,12 @@ static int igt_cursor_commit_legacy(igt_plane_t *cursor,
 			    igt_output_name(output),
 			    kmstest_pipe_name(output->config.pipe),
 			    gem_handle,
-			    cursor->fb->width, cursor->fb->height);
+			    cursor->crtc_w, cursor->crtc_h);
 
 			ret = drmModeSetCursor(display->drm_fd, crtc_id,
 					       gem_handle,
-					       cursor->fb->width,
-					       cursor->fb->height);
+					       cursor->crtc_w,
+					       cursor->crtc_h);
 		} else {
 			LOG(display,
 			    "%s: SetCursor pipe %s, disabling\n",
@@ -1631,6 +1631,14 @@ void igt_plane_set_fb(igt_plane_t *plane, struct igt_fb *fb)
 	    plane->index, fb ? fb->fb_id : 0);
 
 	plane->fb = fb;
+	/* hack to keep tests working that don't call igt_plane_set_size() */
+	if (fb) {
+		plane->crtc_w = fb->width;
+		plane->crtc_h = fb->height;
+	} else {
+		plane->crtc_w = 0;
+		plane->crtc_h = 0;
+	}
 
 	plane->fb_changed = true;
 }
@@ -1649,6 +1657,24 @@ void igt_plane_set_position(igt_plane_t *plane, int x, int y)
 	plane->position_changed = true;
 }
 
+void igt_plane_set_size(igt_plane_t *plane, int w, int h)
+{
+	igt_pipe_t *pipe = plane->pipe;
+	igt_display_t *display = pipe->display;
+
+	LOG(display, "%s.%d: plane_set_size(%d,%d)\n",
+	    kmstest_pipe_name(pipe->pipe), plane->index, w, h);
+
+	plane->crtc_w = w;
+	plane->crtc_h = h;
+
+	/*
+	 * must be fb_changed so that legacy cursors call
+	 * drmModeSetCursor() instead of drmModeMoveCursor()
+	 */
+	plane->fb_changed = true;
+}
+
 void igt_plane_set_panning(igt_plane_t *plane, int x, int y)
 {
 	igt_pipe_t *pipe = plane->pipe;
diff --git a/lib/igt_kms.h b/lib/igt_kms.h
index 921afef..027b4e0 100644
--- a/lib/igt_kms.h
+++ b/lib/igt_kms.h
@@ -212,6 +212,8 @@ typedef struct {
 
 	/* position within pipe_src_w x pipe_src_h */
 	int crtc_x, crtc_y;
+	/* size within pipe_src_w x pipe_src_h */
+	int crtc_w, crtc_h;
 	/* panning offset within the fb */
 	unsigned int pan_x, pan_y;
 	igt_rotation_t rotation;
@@ -266,6 +268,7 @@ static inline bool igt_plane_supports_rotation(igt_plane_t *plane)
 
 void igt_plane_set_fb(igt_plane_t *plane, struct igt_fb *fb);
 void igt_plane_set_position(igt_plane_t *plane, int x, int y);
+void igt_plane_set_size(igt_plane_t *plane, int w, int h);
 void igt_plane_set_panning(igt_plane_t *plane, int x, int y);
 void igt_plane_set_rotation(igt_plane_t *plane, igt_rotation_t rotation);
 
-- 
1.8.5.5

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

[PATCH igt 2/2] tests/kms_cursor_crc: Add basic non-square cursor testing

[ville.syrjala]

From: Ville Syrjälä <ville.syrjala@linux.intel.com>

To minimally verify that non-square cursors work on the platforms where
they're supported perform the tests first with WxH cursor and then
repeat with WxH/3 cursor.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
 tests/kms_cursor_crc.c | 130 +++++++++++++++++++++++++++++++++++++------------
 1 file changed, 100 insertions(+), 30 deletions(-)

diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c
index 3f53859..718699d 100644
--- a/tests/kms_cursor_crc.c
+++ b/tests/kms_cursor_crc.c
@@ -31,6 +31,7 @@
 #include "drmtest.h"
 #include "igt_debugfs.h"
 #include "igt_kms.h"
+#include "intel_chipset.h"
 
 #ifndef DRM_CAP_CURSOR_WIDTH
 #define DRM_CAP_CURSOR_WIDTH 0x8
@@ -50,22 +51,30 @@ typedef struct {
 	int left, right, top, bottom;
 	int screenw, screenh;
 	int curw, curh; /* cursor size */
-	int cursor_max_size;
+	int cursor_max_w, cursor_max_h;
 	igt_pipe_crc_t *pipe_crc;
+	uint32_t devid;
 } data_t;
 
-static void draw_cursor(cairo_t *cr, int x, int y, int w)
+static void draw_cursor(cairo_t *cr, int x, int y, int cw, int ch)
 {
-	w /= 2;
+	int wl, wr, ht, hb;
+
+	/* deal with odd cursor width/height */
+	wl = cw / 2;
+	wr = (cw + 1) / 2;
+	ht = ch / 2;
+	hb = (ch + 1) / 2;
+
 	/* Cairo doesn't like to be fed numbers that are too wild */
 	if ((x < SHRT_MIN) || (x > SHRT_MAX) || (y < SHRT_MIN) || (y > SHRT_MAX))
 		return;
 	cairo_set_antialias(cr, CAIRO_ANTIALIAS_NONE);
 	/* 4 color rectangles in the corners, RGBY */
-	igt_paint_color_alpha(cr, x, y, w, w, 1.0, 0.0, 0.0, 1.0);
-	igt_paint_color_alpha(cr, x + w, y, w, w, 0.0, 1.0, 0.0, 1.0);
-	igt_paint_color_alpha(cr, x, y + w, w, w, 0.0, 0.0, 1.0, 1.0);
-	igt_paint_color_alpha(cr, x + w, y + w, w, w, 0.5, 0.5, 0.5, 1.0);
+	igt_paint_color_alpha(cr, x,      y,      wl, ht, 1.0, 0.0, 0.0, 1.0);
+	igt_paint_color_alpha(cr, x + wl, y,      wr, ht, 0.0, 1.0, 0.0, 1.0);
+	igt_paint_color_alpha(cr, x,      y + ht, wl, hb, 0.0, 0.0, 1.0, 1.0);
+	igt_paint_color_alpha(cr, x + wl, y + ht, wr, hb, 0.5, 0.5, 0.5, 1.0);
 }
 
 static void cursor_enable(data_t *data)
@@ -76,7 +85,7 @@ static void cursor_enable(data_t *data)
 
 	cursor = igt_output_get_plane(output, IGT_PLANE_CURSOR);
 	igt_plane_set_fb(cursor, &data->fb);
-	igt_display_commit(display);
+	igt_plane_set_size(cursor, data->curw, data->curh);
 }
 
 static void cursor_disable(data_t *data)
@@ -87,7 +96,6 @@ static void cursor_disable(data_t *data)
 
 	cursor = igt_output_get_plane(output, IGT_PLANE_CURSOR);
 	igt_plane_set_fb(cursor, NULL);
-	igt_display_commit(display);
 }
 
 
@@ -110,9 +118,10 @@ static void do_single_test(data_t *data, int x, int y)
 	igt_wait_for_vblank(data->drm_fd, data->pipe);
 	igt_pipe_crc_collect_crc(pipe_crc, &crc);
 	cursor_disable(data);
+	igt_display_commit(display);
 
 	/* Now render the same in software and collect crc */
-	draw_cursor(cr, x, y, data->curw);
+	draw_cursor(cr, x, y, data->curw, data->curh);
 	igt_display_commit(display);
 	igt_wait_for_vblank(data->drm_fd, data->pipe);
 	igt_pipe_crc_collect_crc(pipe_crc, &ref_crc);
@@ -223,6 +232,7 @@ static bool prepare_crtc(data_t *data, igt_output_t *output,
 
 	/* select the pipe we want to use */
 	igt_output_set_pipe(output, data->pipe);
+	cursor_disable(data);
 	igt_display_commit(display);
 
 	if (!output->valid) {
@@ -265,7 +275,6 @@ static bool prepare_crtc(data_t *data, igt_output_t *output,
 	data->screenh = mode->vdisplay;
 	data->curw = cursor_w;
 	data->curh = cursor_h;
-	data->cursor_max_size = cursor_w;
 
 	/* make sure cursor is disabled */
 	cursor_disable(data);
@@ -301,6 +310,9 @@ static void run_test(data_t *data, void (*testfunc)(data_t *), int cursor_w, int
 	enum pipe p;
 	int valid_tests = 0;
 
+	igt_require(cursor_w <= data->cursor_max_w &&
+		    cursor_h <= data->cursor_max_h);
+
 	for_each_connected_output(display, output) {
 		data->output = output;
 		for_each_pipe(display, p) {
@@ -336,16 +348,36 @@ static void create_cursor_fb(data_t *data, int cur_w, int cur_h)
 	cairo_t *cr;
 	uint32_t fb_id;
 
-	fb_id = igt_create_fb(data->drm_fd, cur_w, cur_h,
-			      DRM_FORMAT_ARGB8888, I915_TILING_NONE,
-			      &data->fb);
+	/*
+	 * Make the FB slightly taller and leave the extra
+	 * line opaque white, so that we can see that the
+	 * hardware won't scan beyond what it should (esp.
+	 * with non-square cursors).
+	 */
+	fb_id = igt_create_color_fb(data->drm_fd, cur_w, cur_h + 1,
+				    DRM_FORMAT_ARGB8888, I915_TILING_NONE,
+				    1.0, 1.0, 1.0,
+				    &data->fb);
+
 	igt_assert(fb_id);
 
 	cr = igt_get_cairo_ctx(data->drm_fd, &data->fb);
-	draw_cursor(cr, 0, 0, cur_w);
+	draw_cursor(cr, 0, 0, cur_w, cur_h);
 	igt_assert(cairo_status(cr) == 0);
 }
 
+static bool has_nonsquare_cursors(uint32_t devid)
+{
+	/*
+	 * Test non-square cursors a bit on the platforms
+	 * that support such things.
+	 */
+	return devid == PCI_CHIP_845_G ||
+		devid == PCI_CHIP_I865_G ||
+		(IS_GEN7(devid) && !IS_VALLEYVIEW(devid)) ||
+		(IS_GEN8(devid) && !IS_CHERRYVIEW(devid));
+}
+
 static void test_cursor_size(data_t *data)
 {
 	igt_display_t *display = &data->display;
@@ -353,7 +385,8 @@ static void test_cursor_size(data_t *data)
 	igt_crc_t crc[10], ref_crc;
 	cairo_t *cr;
 	uint32_t fb_id;
-	int i, size, cursor_max_size = data->cursor_max_size;
+	int i, size;
+	int cursor_max_size = data->cursor_max_w;
 	int ret;
 
 	/* Create a maximum size cursor, then change the size in flight to
@@ -395,35 +428,67 @@ static void test_cursor_size(data_t *data)
 	}
 }
 
-static void run_test_generic(data_t *data, int cursor_max_size)
+static void run_test_generic(data_t *data)
 {
 	int cursor_size;
-	for (cursor_size = 64; cursor_size <= 256; cursor_size *= 2) {
+	for (cursor_size = 64; cursor_size <= 512; cursor_size *= 2) {
+		int w = cursor_size;
+		int h = cursor_size;
+
 		igt_fixture
-			igt_require(cursor_max_size >= cursor_size);
+			create_cursor_fb(data, w, h);
+
+		/* Using created cursor FBs to test cursor support */
+		igt_subtest_f("cursor-%dx%d-onscreen", w, h)
+			run_test(data, test_crc_onscreen, w, h);
+		igt_subtest_f("cursor-%dx%d-offscreen", w, h)
+			run_test(data, test_crc_offscreen, w, h);
+		igt_subtest_f("cursor-%dx%d-sliding", w, h)
+			run_test(data, test_crc_sliding, w, h);
+		igt_subtest_f("cursor-%dx%d-random", w, h)
+			run_test(data, test_crc_random, w, h);
 
 		igt_fixture
-			create_cursor_fb(data, cursor_size, cursor_size);
+			igt_remove_fb(data->drm_fd, &data->fb);
+
+		/*
+		 * Test non-square cursors a bit on the platforms
+		 * that support such things. And make it a bit more
+		 * interesting by using a non-pot height.
+		 */
+		h /= 3;
+
+		igt_fixture
+			create_cursor_fb(data, w, h);
 
 		/* Using created cursor FBs to test cursor support */
-		igt_subtest_f("cursor-%d-onscreen", cursor_size)
-			run_test(data, test_crc_onscreen, cursor_size, cursor_size);
-		igt_subtest_f("cursor-%d-offscreen", cursor_size)
-			run_test(data, test_crc_offscreen, cursor_size, cursor_size);
-		igt_subtest_f("cursor-%d-sliding", cursor_size)
-			run_test(data, test_crc_sliding, cursor_size, cursor_size);
-		igt_subtest_f("cursor-%d-random", cursor_size)
-			run_test(data, test_crc_random, cursor_size, cursor_size);
+		igt_subtest_f("cursor-%dx%d-onscreen", w, h) {
+			igt_require(has_nonsquare_cursors(data->devid));
+			run_test(data, test_crc_onscreen, w, h);
+		}
+		igt_subtest_f("cursor-%dx%d-offscreen", w, h) {
+			igt_require(has_nonsquare_cursors(data->devid));
+			run_test(data, test_crc_offscreen, w, h);
+		}
+		igt_subtest_f("cursor-%dx%d-sliding", w, h) {
+			igt_require(has_nonsquare_cursors(data->devid));
+			run_test(data, test_crc_sliding, w, h);
+		}
+		igt_subtest_f("cursor-%dx%d-random", w, h) {
+			igt_require(has_nonsquare_cursors(data->devid));
+			run_test(data, test_crc_random, w, h);
+		}
 
 		igt_fixture
 			igt_remove_fb(data->drm_fd, &data->fb);
 	}
 }
 
+static data_t data;
+
 igt_main
 {
 	uint64_t cursor_width = 64, cursor_height = 64;
-	data_t data = {};
 	int ret;
 
 	igt_skip_on_simulation();
@@ -431,6 +496,8 @@ igt_main
 	igt_fixture {
 		data.drm_fd = drm_open_any();
 
+		data.devid = intel_get_drm_devid(data.drm_fd);
+
 		ret = drmGetCap(data.drm_fd, DRM_CAP_CURSOR_WIDTH, &cursor_width);
 		igt_assert(ret == 0 || errno == EINVAL);
 		/* Not making use of cursor_height since it is same as width, still reading */
@@ -447,10 +514,13 @@ igt_main
 		igt_display_init(&data.display, data.drm_fd);
 	}
 
+	data.cursor_max_w = cursor_width;
+	data.cursor_max_h = cursor_height;
+
 	igt_subtest_f("cursor-size-change")
 		run_test(&data, test_cursor_size, cursor_width, cursor_height);
 
-	run_test_generic(&data, cursor_width);
+	run_test_generic(&data);
 
 	igt_fixture {
 		igt_display_fini(&data.display);
-- 
1.8.5.5

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

Re: [PATCH igt 1/2] lib: Add igt_plane_set_size()

[Daniel Vetter]

On Fri, Sep 12, 2014 at 09:38:13PM +0300, ville.syrjala@linux.intel.com wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
> 
> Allow tests to specify the plane size instead of assuming that the
> entire FB will be scanned out.
> 
> To keep the current tests working without having to sprinkle
> igt_plane_set_size() calls all over the place, make
> igt_plane_set_fb() reset the plane size to the FB size.

Can you please add gtkdoc for just these two functions? We need to start
somewhere ...

Thanks, Daniel
> 
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> ---
>  lib/igt_kms.c | 34 ++++++++++++++++++++++++++++++----
>  lib/igt_kms.h |  3 +++
>  2 files changed, 33 insertions(+), 4 deletions(-)
> 
> diff --git a/lib/igt_kms.c b/lib/igt_kms.c
> index 3a850f1..0547147 100644
> --- a/lib/igt_kms.c
> +++ b/lib/igt_kms.c
> @@ -1269,7 +1269,7 @@ static int igt_drm_plane_commit(igt_plane_t *plane,
>  				      fb_id,
>  				      0,    /* flags */
>  				      plane->crtc_x, plane->crtc_y,
> -				      plane->fb->width, plane->fb->height,
> +				      plane->crtc_w, plane->crtc_h,
>  				      IGT_FIXED(0,0), /* src_x */
>  				      IGT_FIXED(0,0), /* src_y */
>  				      IGT_FIXED(plane->fb->width,0), /* src_w */
> @@ -1314,12 +1314,12 @@ static int igt_cursor_commit_legacy(igt_plane_t *cursor,
>  			    igt_output_name(output),
>  			    kmstest_pipe_name(output->config.pipe),
>  			    gem_handle,
> -			    cursor->fb->width, cursor->fb->height);
> +			    cursor->crtc_w, cursor->crtc_h);
>  
>  			ret = drmModeSetCursor(display->drm_fd, crtc_id,
>  					       gem_handle,
> -					       cursor->fb->width,
> -					       cursor->fb->height);
> +					       cursor->crtc_w,
> +					       cursor->crtc_h);
>  		} else {
>  			LOG(display,
>  			    "%s: SetCursor pipe %s, disabling\n",
> @@ -1631,6 +1631,14 @@ void igt_plane_set_fb(igt_plane_t *plane, struct igt_fb *fb)
>  	    plane->index, fb ? fb->fb_id : 0);
>  
>  	plane->fb = fb;
> +	/* hack to keep tests working that don't call igt_plane_set_size() */
> +	if (fb) {
> +		plane->crtc_w = fb->width;
> +		plane->crtc_h = fb->height;
> +	} else {
> +		plane->crtc_w = 0;
> +		plane->crtc_h = 0;
> +	}
>  
>  	plane->fb_changed = true;
>  }
> @@ -1649,6 +1657,24 @@ void igt_plane_set_position(igt_plane_t *plane, int x, int y)
>  	plane->position_changed = true;
>  }
>  
> +void igt_plane_set_size(igt_plane_t *plane, int w, int h)
> +{
> +	igt_pipe_t *pipe = plane->pipe;
> +	igt_display_t *display = pipe->display;
> +
> +	LOG(display, "%s.%d: plane_set_size(%d,%d)\n",
> +	    kmstest_pipe_name(pipe->pipe), plane->index, w, h);
> +
> +	plane->crtc_w = w;
> +	plane->crtc_h = h;
> +
> +	/*
> +	 * must be fb_changed so that legacy cursors call
> +	 * drmModeSetCursor() instead of drmModeMoveCursor()
> +	 */
> +	plane->fb_changed = true;
> +}
> +
>  void igt_plane_set_panning(igt_plane_t *plane, int x, int y)
>  {
>  	igt_pipe_t *pipe = plane->pipe;
> diff --git a/lib/igt_kms.h b/lib/igt_kms.h
> index 921afef..027b4e0 100644
> --- a/lib/igt_kms.h
> +++ b/lib/igt_kms.h
> @@ -212,6 +212,8 @@ typedef struct {
>  
>  	/* position within pipe_src_w x pipe_src_h */
>  	int crtc_x, crtc_y;
> +	/* size within pipe_src_w x pipe_src_h */
> +	int crtc_w, crtc_h;
>  	/* panning offset within the fb */
>  	unsigned int pan_x, pan_y;
>  	igt_rotation_t rotation;
> @@ -266,6 +268,7 @@ static inline bool igt_plane_supports_rotation(igt_plane_t *plane)
>  
>  void igt_plane_set_fb(igt_plane_t *plane, struct igt_fb *fb);
>  void igt_plane_set_position(igt_plane_t *plane, int x, int y);
> +void igt_plane_set_size(igt_plane_t *plane, int w, int h);
>  void igt_plane_set_panning(igt_plane_t *plane, int x, int y);
>  void igt_plane_set_rotation(igt_plane_t *plane, igt_rotation_t rotation);
>  
> -- 
> 1.8.5.5
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch

Re: [PATCH igt 1/2] lib: Add igt_plane_set_size()

[Daniel Vetter]

On Mon, Sep 15, 2014 at 11:17:11AM +0200, Daniel Vetter wrote:
> On Fri, Sep 12, 2014 at 09:38:13PM +0300, ville.syrjala@linux.intel.com wrote:
> > From: Ville Syrjälä <ville.syrjala@linux.intel.com>
> > 
> > Allow tests to specify the plane size instead of assuming that the
> > entire FB will be scanned out.
> > 
> > To keep the current tests working without having to sprinkle
> > igt_plane_set_size() calls all over the place, make
> > igt_plane_set_fb() reset the plane size to the FB size.
> 
> Can you please add gtkdoc for just these two functions? We need to start
> somewhere ...

Ping for documentation.
-Daniel

> 
> Thanks, Daniel
> > 
> > Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
> > ---
> >  lib/igt_kms.c | 34 ++++++++++++++++++++++++++++++----
> >  lib/igt_kms.h |  3 +++
> >  2 files changed, 33 insertions(+), 4 deletions(-)
> > 
> > diff --git a/lib/igt_kms.c b/lib/igt_kms.c
> > index 3a850f1..0547147 100644
> > --- a/lib/igt_kms.c
> > +++ b/lib/igt_kms.c
> > @@ -1269,7 +1269,7 @@ static int igt_drm_plane_commit(igt_plane_t *plane,
> >  				      fb_id,
> >  				      0,    /* flags */
> >  				      plane->crtc_x, plane->crtc_y,
> > -				      plane->fb->width, plane->fb->height,
> > +				      plane->crtc_w, plane->crtc_h,
> >  				      IGT_FIXED(0,0), /* src_x */
> >  				      IGT_FIXED(0,0), /* src_y */
> >  				      IGT_FIXED(plane->fb->width,0), /* src_w */
> > @@ -1314,12 +1314,12 @@ static int igt_cursor_commit_legacy(igt_plane_t *cursor,
> >  			    igt_output_name(output),
> >  			    kmstest_pipe_name(output->config.pipe),
> >  			    gem_handle,
> > -			    cursor->fb->width, cursor->fb->height);
> > +			    cursor->crtc_w, cursor->crtc_h);
> >  
> >  			ret = drmModeSetCursor(display->drm_fd, crtc_id,
> >  					       gem_handle,
> > -					       cursor->fb->width,
> > -					       cursor->fb->height);
> > +					       cursor->crtc_w,
> > +					       cursor->crtc_h);
> >  		} else {
> >  			LOG(display,
> >  			    "%s: SetCursor pipe %s, disabling\n",
> > @@ -1631,6 +1631,14 @@ void igt_plane_set_fb(igt_plane_t *plane, struct igt_fb *fb)
> >  	    plane->index, fb ? fb->fb_id : 0);
> >  
> >  	plane->fb = fb;
> > +	/* hack to keep tests working that don't call igt_plane_set_size() */
> > +	if (fb) {
> > +		plane->crtc_w = fb->width;
> > +		plane->crtc_h = fb->height;
> > +	} else {
> > +		plane->crtc_w = 0;
> > +		plane->crtc_h = 0;
> > +	}
> >  
> >  	plane->fb_changed = true;
> >  }
> > @@ -1649,6 +1657,24 @@ void igt_plane_set_position(igt_plane_t *plane, int x, int y)
> >  	plane->position_changed = true;
> >  }
> >  
> > +void igt_plane_set_size(igt_plane_t *plane, int w, int h)
> > +{
> > +	igt_pipe_t *pipe = plane->pipe;
> > +	igt_display_t *display = pipe->display;
> > +
> > +	LOG(display, "%s.%d: plane_set_size(%d,%d)\n",
> > +	    kmstest_pipe_name(pipe->pipe), plane->index, w, h);
> > +
> > +	plane->crtc_w = w;
> > +	plane->crtc_h = h;
> > +
> > +	/*
> > +	 * must be fb_changed so that legacy cursors call
> > +	 * drmModeSetCursor() instead of drmModeMoveCursor()
> > +	 */
> > +	plane->fb_changed = true;
> > +}
> > +
> >  void igt_plane_set_panning(igt_plane_t *plane, int x, int y)
> >  {
> >  	igt_pipe_t *pipe = plane->pipe;
> > diff --git a/lib/igt_kms.h b/lib/igt_kms.h
> > index 921afef..027b4e0 100644
> > --- a/lib/igt_kms.h
> > +++ b/lib/igt_kms.h
> > @@ -212,6 +212,8 @@ typedef struct {
> >  
> >  	/* position within pipe_src_w x pipe_src_h */
> >  	int crtc_x, crtc_y;
> > +	/* size within pipe_src_w x pipe_src_h */
> > +	int crtc_w, crtc_h;
> >  	/* panning offset within the fb */
> >  	unsigned int pan_x, pan_y;
> >  	igt_rotation_t rotation;
> > @@ -266,6 +268,7 @@ static inline bool igt_plane_supports_rotation(igt_plane_t *plane)
> >  
> >  void igt_plane_set_fb(igt_plane_t *plane, struct igt_fb *fb);
> >  void igt_plane_set_position(igt_plane_t *plane, int x, int y);
> > +void igt_plane_set_size(igt_plane_t *plane, int w, int h);
> >  void igt_plane_set_panning(igt_plane_t *plane, int x, int y);
> >  void igt_plane_set_rotation(igt_plane_t *plane, igt_rotation_t rotation);
> >  
> > -- 
> > 1.8.5.5
> > 
> > _______________________________________________
> > Intel-gfx mailing list
> > Intel-gfx@lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/intel-gfx
> 
> -- 
> Daniel Vetter
> Software Engineer, Intel Corporation
> +41 (0) 79 365 57 48 - http://blog.ffwll.ch

-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch