Re: [PATCH i-g-t] i-g-t: kms_plane_scaling: Enhanced scaling tests

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Somehow I forgot to send out my irc feedback to the m-l.

On Wed, Nov 22, 2017 at 9:15 AM, Vidya Srinivas
<vidya.srinivas@xxxxxxxxx> wrote:
> +igt_main
>  {
>         data_t data = {};
>
> @@ -308,11 +765,26 @@ igt_simple_main
>         data.drm_fd = drm_open_driver(DRIVER_INTEL);
>         igt_require_pipe_crc(data.drm_fd);
>         igt_display_init(&data.display, data.drm_fd);
> +       igt_require(data.display.is_atomic);
>         data.devid = intel_get_drm_devid(data.drm_fd);
>
>         data.num_scalers = intel_gen(data.devid) >= 9 ? 2 : 0;
>
> -       test_plane_scaling(&data);
> -
> +       //test_plane_scaling(&data);
> +       igt_subtest_f("scaler_with_pixel_format") {
> +               test_scaler_with_pixel_format(&data);
> +       }
> +       igt_subtest_f("scaler_with_rotation") {
> +               test_scaler_with_rotation(&data);
> +       }
> +       igt_subtest_f("scaler_with_multiple_planes") {
> +               test_scaler_with_multiple_planes(&data);
> +       }
> +       igt_subtest_f("scaler_with_clipping_clamping") {
> +               test_scaler_with_clipping_clamping_scenario(&data);
> +       }
> +       igt_subtest_f("scaler_with_multi_pipe_plane") {
> +               test_scaler_with_multi_pipe_plane(&data);
> +       }

Commenting out the existing testcase and replacing it with new ones
entirely, without explaining what's wrong with the old one, or
removing it (we have git for source control, not comments), and how
new tests are better isn't how we do things. Presumably the existing
tests once worked, so the first step should be to fix that up first
and explain why the changes are necessary. When this code was typed 2
years ago someone put some thought into it, throwing all that work
away isn't good.

And _then_ (in follow-up patches) fix the gaps in test coverage.

Thanks, Daniel

>         igt_display_fini(&data.display);
>  }
> --
> 2.7.4
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx



-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux