From: Mika Kahola <mika.kahola@intel.com>
To: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH i-g-t v2 2/9] tests/kms_plane_scaling: Move the actual test to its own function.
Date: Mon, 15 Jan 2018 10:47:19 +0200 [thread overview]
Message-ID: <1516006039.12453.87.camel@intel.com> (raw)
In-Reply-To: <20180112102116.4670-3-maarten.lankhorst@linux.intel.com>
On Fri, 2018-01-12 at 11:21 +0100, Maarten Lankhorst wrote:
> We will add more subtests in the future, it's more clear if we split
> out the actual test to its own function first.
>
Reviewed-by: Mika Kahola <mika.kahola@intel.com>
> Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> ---
> tests/kms_plane_scaling.c | 226 ++++++++++++++++++++++++----------
> ------------
> 1 file changed, 117 insertions(+), 109 deletions(-)
>
> diff --git a/tests/kms_plane_scaling.c b/tests/kms_plane_scaling.c
> index 403df47e2d3b..37e05158b968 100644
> --- a/tests/kms_plane_scaling.c
> +++ b/tests/kms_plane_scaling.c
> @@ -163,144 +163,151 @@ static void iterate_plane_scaling(data_t *d,
> drmModeModeInfo *mode)
> }
> }
>
> -static void test_plane_scaling(data_t *d)
> +static void
> +test_plane_scaling_on_pipe(data_t *d, enum pipe pipe, igt_output_t
> *output)
> {
> igt_display_t *display = &d->display;
> - igt_output_t *output;
> - enum pipe pipe;
> - int valid_tests = 0;
> + drmModeModeInfo *mode;
> int primary_plane_scaling = 0; /* For now */
>
> - igt_require(d->num_scalers);
> + igt_output_set_pipe(output, pipe);
> + mode = igt_output_get_mode(output);
> +
> + /* allocate fb2 with image size */
> + d->fb_id2 = igt_create_image_fb(d->drm_fd, 0, 0,
> + DRM_FORMAT_XRGB8888,
> + LOCAL_I915_FORMAT_MOD_X_TILE
> D, /* tiled */
> + FILE_NAME, &d->fb2);
> + igt_assert(d->fb_id2);
> +
> + d->fb_id3 = igt_create_pattern_fb(d->drm_fd,
> + mode->hdisplay, mode-
> >vdisplay,
> + DRM_FORMAT_XRGB8888,
> + LOCAL_I915_FORMAT_MOD_X_TI
> LED, /* tiled */
> + &d->fb3);
> + igt_assert(d->fb_id3);
> +
> + /* Set up display with plane 1 */
> + d->plane1 = igt_output_get_plane(output, 0);
> + prepare_crtc(d, output, pipe, d->plane1, mode,
> COMMIT_UNIVERSAL);
> +
> + if (primary_plane_scaling) {
> + /* Primary plane upscaling */
> + igt_fb_set_position(&d->fb1, d->plane1, 100, 100);
> + igt_fb_set_size(&d->fb1, d->plane1, 500, 500);
> + igt_plane_set_position(d->plane1, 0, 0);
> + igt_plane_set_size(d->plane1, mode->hdisplay, mode-
> >vdisplay);
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
>
> - for_each_pipe_with_valid_output(display, pipe, output) {
> - drmModeModeInfo *mode;
> -
> - igt_output_set_pipe(output, pipe);
> -
> - mode = igt_output_get_mode(output);
> -
> - /* allocate fb2 with image size */
> - d->fb_id2 = igt_create_image_fb(d->drm_fd, 0, 0,
> - DRM_FORMAT_XRGB8888,
> - LOCAL_I915_FORMAT_MO
> D_X_TILED, /* tiled */
> - FILE_NAME, &d->fb2);
> - igt_assert(d->fb_id2);
> -
> - d->fb_id3 = igt_create_pattern_fb(d->drm_fd,
> - mode->hdisplay,
> mode->vdisplay,
> - DRM_FORMAT_XRGB888
> 8,
> - LOCAL_I915_FORMAT_
> MOD_X_TILED, /* tiled */
> - &d->fb3);
> - igt_assert(d->fb_id3);
> -
> - /* Set up display with plane 1 */
> - d->plane1 = igt_output_get_plane(output, 0);
> - prepare_crtc(d, output, pipe, d->plane1, mode,
> COMMIT_UNIVERSAL);
> -
> - if (primary_plane_scaling) {
> - /* Primary plane upscaling */
> - igt_fb_set_position(&d->fb1, d->plane1, 100,
> 100);
> - igt_fb_set_size(&d->fb1, d->plane1, 500,
> 500);
> - igt_plane_set_position(d->plane1, 0, 0);
> - igt_plane_set_size(d->plane1, mode-
> >hdisplay, mode->vdisplay);
> - igt_display_commit2(display,
> COMMIT_UNIVERSAL);
> + /* Primary plane 1:1 no scaling */
> + igt_fb_set_position(&d->fb1, d->plane1, 0, 0);
> + igt_fb_set_size(&d->fb1, d->plane1, d->fb1.width, d-
> >fb1.height);
> + igt_plane_set_position(d->plane1, 0, 0);
> + igt_plane_set_size(d->plane1, mode->hdisplay, mode-
> >vdisplay);
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
> + }
>
> - /* Primary plane 1:1 no scaling */
> - igt_fb_set_position(&d->fb1, d->plane1, 0,
> 0);
> - igt_fb_set_size(&d->fb1, d->plane1, d-
> >fb1.width, d->fb1.height);
> - igt_plane_set_position(d->plane1, 0, 0);
> - igt_plane_set_size(d->plane1, mode-
> >hdisplay, mode->vdisplay);
> - igt_display_commit2(display,
> COMMIT_UNIVERSAL);
> - }
> + /* Set up fb2->plane2 mapping. */
> + d->plane2 = igt_output_get_plane(output, 1);
> + igt_plane_set_fb(d->plane2, &d->fb2);
>
> - /* Set up fb2->plane2 mapping. */
> - d->plane2 = igt_output_get_plane(output, 1);
> - igt_plane_set_fb(d->plane2, &d->fb2);
> + /* 2nd plane windowed */
> + igt_fb_set_position(&d->fb2, d->plane2, 100, 100);
> + igt_fb_set_size(&d->fb2, d->plane2, d->fb2.width-200, d-
> >fb2.height-200);
> + igt_plane_set_position(d->plane2, 100, 100);
> + igt_plane_set_size(d->plane2, mode->hdisplay-200, mode-
> >vdisplay-200);
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
>
> - /* 2nd plane windowed */
> - igt_fb_set_position(&d->fb2, d->plane2, 100, 100);
> - igt_fb_set_size(&d->fb2, d->plane2, d->fb2.width-
> 200, d->fb2.height-200);
> - igt_plane_set_position(d->plane2, 100, 100);
> - igt_plane_set_size(d->plane2, mode->hdisplay-200,
> mode->vdisplay-200);
> - igt_display_commit2(display, COMMIT_UNIVERSAL);
> + iterate_plane_scaling(d, mode);
>
> - iterate_plane_scaling(d, mode);
> + /* 2nd plane up scaling */
> + igt_fb_set_position(&d->fb2, d->plane2, 100, 100);
> + igt_fb_set_size(&d->fb2, d->plane2, 500, 500);
> + igt_plane_set_position(d->plane2, 10, 10);
> + igt_plane_set_size(d->plane2, mode->hdisplay-20, mode-
> >vdisplay-20);
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
>
> - /* 2nd plane up scaling */
> - igt_fb_set_position(&d->fb2, d->plane2, 100, 100);
> - igt_fb_set_size(&d->fb2, d->plane2, 500, 500);
> - igt_plane_set_position(d->plane2, 10, 10);
> - igt_plane_set_size(d->plane2, mode->hdisplay-20,
> mode->vdisplay-20);
> - igt_display_commit2(display, COMMIT_UNIVERSAL);
> + /* 2nd plane downscaling */
> + igt_fb_set_position(&d->fb2, d->plane2, 0, 0);
> + igt_fb_set_size(&d->fb2, d->plane2, d->fb2.width, d-
> >fb2.height);
> + igt_plane_set_position(d->plane2, 10, 10);
> + igt_plane_set_size(d->plane2, 500, 500 * d->fb2.height/d-
> >fb2.width);
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
>
> - /* 2nd plane downscaling */
> - igt_fb_set_position(&d->fb2, d->plane2, 0, 0);
> - igt_fb_set_size(&d->fb2, d->plane2, d->fb2.width, d-
> >fb2.height);
> - igt_plane_set_position(d->plane2, 10, 10);
> - igt_plane_set_size(d->plane2, 500, 500 * d-
> >fb2.height/d->fb2.width);
> + if (primary_plane_scaling) {
> + /* Primary plane up scaling */
> + igt_fb_set_position(&d->fb1, d->plane1, 100, 100);
> + igt_fb_set_size(&d->fb1, d->plane1, 500, 500);
> + igt_plane_set_position(d->plane1, 0, 0);
> + igt_plane_set_size(d->plane1, mode->hdisplay, mode-
> >vdisplay);
> igt_display_commit2(display, COMMIT_UNIVERSAL);
> + }
>
> - if (primary_plane_scaling) {
> - /* Primary plane up scaling */
> - igt_fb_set_position(&d->fb1, d->plane1, 100,
> 100);
> - igt_fb_set_size(&d->fb1, d->plane1, 500,
> 500);
> - igt_plane_set_position(d->plane1, 0, 0);
> - igt_plane_set_size(d->plane1, mode-
> >hdisplay, mode->vdisplay);
> - igt_display_commit2(display,
> COMMIT_UNIVERSAL);
> - }
> + /* Set up fb3->plane3 mapping. */
> + d->plane3 = igt_output_get_plane(output, 2);
> + igt_plane_set_fb(d->plane3, &d->fb3);
> +
> + /* 3rd plane windowed - no scaling */
> + igt_fb_set_position(&d->fb3, d->plane3, 100, 100);
> + igt_fb_set_size(&d->fb3, d->plane3, d->fb3.width-300, d-
> >fb3.height-300);
> + igt_plane_set_position(d->plane3, 100, 100);
> + igt_plane_set_size(d->plane3, mode->hdisplay-300, mode-
> >vdisplay-300);
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
>
> - /* Set up fb3->plane3 mapping. */
> - d->plane3 = igt_output_get_plane(output, 2);
> - igt_plane_set_fb(d->plane3, &d->fb3);
> + /* Switch scaler from plane 2 to plane 3 */
> + igt_fb_set_position(&d->fb2, d->plane2, 100, 100);
> + igt_fb_set_size(&d->fb2, d->plane2, d->fb2.width-200, d-
> >fb2.height-200);
> + igt_plane_set_position(d->plane2, 100, 100);
> + igt_plane_set_size(d->plane2, d->fb2.width-200, d-
> >fb2.height-200);
>
> - /* 3rd plane windowed - no scaling */
> - igt_fb_set_position(&d->fb3, d->plane3, 100, 100);
> - igt_fb_set_size(&d->fb3, d->plane3, d->fb3.width-
> 300, d->fb3.height-300);
> - igt_plane_set_position(d->plane3, 100, 100);
> - igt_plane_set_size(d->plane3, mode->hdisplay-300,
> mode->vdisplay-300);
> - igt_display_commit2(display, COMMIT_UNIVERSAL);
> + igt_fb_set_position(&d->fb3, d->plane3, 100, 100);
> + igt_fb_set_size(&d->fb3, d->plane3, d->fb3.width-400, d-
> >fb3.height-400);
> + igt_plane_set_position(d->plane3, 10, 10);
> + igt_plane_set_size(d->plane3, mode->hdisplay-300, mode-
> >vdisplay-300);
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
> +
> + if (primary_plane_scaling) {
> + /* Switch scaler from plane 1 to plane 2 */
> + igt_fb_set_position(&d->fb1, d->plane1, 0, 0);
> + igt_fb_set_size(&d->fb1, d->plane1, d->fb1.width, d-
> >fb1.height);
> + igt_plane_set_position(d->plane1, 0, 0);
> + igt_plane_set_size(d->plane1, mode->hdisplay, mode-
> >vdisplay);
>
> - /* Switch scaler from plane 2 to plane 3 */
> igt_fb_set_position(&d->fb2, d->plane2, 100, 100);
> - igt_fb_set_size(&d->fb2, d->plane2, d->fb2.width-
> 200, d->fb2.height-200);
> + igt_fb_set_size(&d->fb2, d->plane2, d->fb2.width-
> 500,d->fb2.height-500);
> igt_plane_set_position(d->plane2, 100, 100);
> - igt_plane_set_size(d->plane2, d->fb2.width-200, d-
> >fb2.height-200);
> -
> - igt_fb_set_position(&d->fb3, d->plane3, 100, 100);
> - igt_fb_set_size(&d->fb3, d->plane3, d->fb3.width-
> 400, d->fb3.height-400);
> - igt_plane_set_position(d->plane3, 10, 10);
> - igt_plane_set_size(d->plane3, mode->hdisplay-300,
> mode->vdisplay-300);
> + igt_plane_set_size(d->plane2, mode->hdisplay-200,
> mode->vdisplay-200);
> igt_display_commit2(display, COMMIT_UNIVERSAL);
> + }
>
> - if (primary_plane_scaling) {
> - /* Switch scaler from plane 1 to plane 2 */
> - igt_fb_set_position(&d->fb1, d->plane1, 0,
> 0);
> - igt_fb_set_size(&d->fb1, d->plane1, d-
> >fb1.width, d->fb1.height);
> - igt_plane_set_position(d->plane1, 0, 0);
> - igt_plane_set_size(d->plane1, mode-
> >hdisplay, mode->vdisplay);
> -
> - igt_fb_set_position(&d->fb2, d->plane2, 100,
> 100);
> - igt_fb_set_size(&d->fb2, d->plane2, d-
> >fb2.width-500,d->fb2.height-500);
> - igt_plane_set_position(d->plane2, 100, 100);
> - igt_plane_set_size(d->plane2, mode-
> >hdisplay-200, mode->vdisplay-200);
> - igt_display_commit2(display,
> COMMIT_UNIVERSAL);
> - }
> + /* back to single plane mode */
> + igt_plane_set_fb(d->plane2, NULL);
> + igt_plane_set_fb(d->plane3, NULL);
> + igt_display_commit2(display, COMMIT_UNIVERSAL);
>
> - /* back to single plane mode */
> - igt_plane_set_fb(d->plane2, NULL);
> - igt_plane_set_fb(d->plane3, NULL);
> - igt_display_commit2(display, COMMIT_UNIVERSAL);
> + cleanup_crtc(d, output, d->plane1);
> +}
>
> +static void test_plane_scaling(data_t *d, enum pipe pipe)
> +{
> + igt_output_t *output;
> + int valid_tests = 0;
> +
> + igt_require(d->num_scalers);
> +
> + for_each_valid_output_on_pipe(&d->display, pipe, output) {
> + test_plane_scaling_on_pipe(d, pipe, output);
> + igt_output_set_pipe(output, PIPE_ANY);
> valid_tests++;
> - cleanup_crtc(d, output, d->plane1);
> }
> +
> igt_require_f(valid_tests, "no valid crtc/connector
> combinations found\n");
> }
>
> igt_simple_main
> {
> data_t data = {};
> + enum pipe pipe;
>
> igt_skip_on_simulation();
>
> @@ -312,7 +319,8 @@ igt_simple_main
>
> data.num_scalers = intel_gen(data.devid) >= 9 ? 2 : 0;
>
> - test_plane_scaling(&data);
> + for_each_pipe_static(pipe)
> + test_plane_scaling(&data, pipe);
>
> igt_display_fini(&data.display);
> }
--
Mika Kahola - Intel OTC
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2018-01-15 8:46 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-01-12 10:21 [PATCH i-g-t v2 0/9] kms plane scaling tests Maarten Lankhorst
2018-01-12 10:21 ` [PATCH i-g-t v2 1/9] tests/kms_plane: Run test for all supported pixel formats, v2 Maarten Lankhorst
2018-01-12 13:52 ` Mika Kahola
2018-01-12 10:21 ` [PATCH i-g-t v2 2/9] tests/kms_plane_scaling: Move the actual test to its own function Maarten Lankhorst
2018-01-15 8:47 ` Mika Kahola [this message]
2018-01-12 10:21 ` [PATCH i-g-t v2 3/9] tests/kms_plane_scaling: Fix basic scaling test, v2 Maarten Lankhorst
2018-01-15 9:01 ` Mika Kahola
2018-01-12 10:21 ` [PATCH i-g-t v2 4/9] tests/kms_plane_scaling: Convert from simple test to full test Maarten Lankhorst
2018-01-15 9:10 ` Mika Kahola
2018-01-12 10:21 ` [PATCH i-g-t v2 5/9] tests/kms_plane_scaling: Clean up tests to work better with igt_kms Maarten Lankhorst
2018-01-12 10:21 ` [PATCH i-g-t v2 6/9] tests/kms_plane_scaling: test scaling with tiling rotation and pixel formats, v2 Maarten Lankhorst
2018-01-12 10:21 ` [PATCH i-g-t v2 7/9] tests/kms_plane_scaling: test scaler with clipping clamping, v2 Maarten Lankhorst
2018-01-12 10:21 ` [PATCH i-g-t v2 8/9] lib/igt_kms: Add more braces around macros Maarten Lankhorst
2018-01-15 12:01 ` Mika Kahola
2018-01-12 10:21 ` [PATCH i-g-t v2 9/9] tests/kms_plane_scaling: test for multi pipe with scaling Maarten Lankhorst
2018-01-12 13:09 ` ✓ Fi.CI.BAT: success for kms plane scaling tests Patchwork
2018-01-12 13:58 ` ✓ Fi.CI.IGT: " Patchwork
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1516006039.12453.87.camel@intel.com \
--to=mika.kahola@intel.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=maarten.lankhorst@linux.intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.