[igt-dev] [PATCH i-g-t] tests/kms: Move Intel specific kms tests to i915 directory
Modem, Bhanuprakash
bhanuprakash.modem at intel.com
Tue Aug 3 05:11:25 UTC 2021
> From: Mark Yacoub <markyacoub at chromium.org>
> Sent: Monday, August 2, 2021 9:20 PM
> To: Modem, Bhanuprakash <bhanuprakash.modem at intel.com>; Siqueira, Rodrigo
> <Rodrigo.Siqueira at amd.com>
> Cc: Development mailing list for IGT GPU Tools <igt-
> dev at lists.freedesktop.org>; Latvala, Petri <petri.latvala at intel.com>;
> Srinivas, Vidya <vidya.srinivas at intel.com>; B S, Karthik
> <karthik.b.s at intel.com>
> Subject: Re: [PATCH i-g-t] tests/kms: Move Intel specific kms tests to i915
> directory
>
> d
>
> On Fri, Jul 30, 2021 at 3:21 AM Bhanuprakash Modem
> <bhanuprakash.modem at intel.com> wrote:
> >
> > As KMS tests on IGT are officially supported on multiple SoCs
> > and there is an active development on it. So, KMS tests are
> > meant to be generic and if we need to test few things specific
> > to Intel, so we can use igt_require_intel(). But if the whole
> > test is Intel specific, then the best place for the test would
> > be tests/i915.
> >
> > This patch will move all Intel specific kms tests to tests/i915
> > directory. And update the tests those are generic but still open
> > the driver as 'drm_open_driver_master(DRIVER_INTEL);'. This patch
> > will also do some minor cleanups.
> >
> > Cc: Mark Yacoub <markyacoub at chromium.org>
> > Cc: Petri Latvala <petri.latvala at intel.com>
> > Cc: Vidya Srinivas <vidya.srinivas at intel.com>
> > Cc: Karthik B S <karthik.b.s at intel.com>
> > Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem at intel.com>
> > ---
> > tests/{ => i915}/kms_big_fb.c | 0
> > tests/{ => i915}/kms_busy.c | 0
> > tests/{ => i915}/kms_ccs.c | 0
> > tests/{ => i915}/kms_cdclk.c | 0
> > tests/{ => i915}/kms_draw_crc.c | 0
> > tests/{ => i915}/kms_fence_pin_leak.c | 0
> > tests/{ => i915}/kms_flip_scaled_crc.c | 0
> > tests/{ => i915}/kms_flip_tiling.c | 0
> > tests/{ => i915}/kms_frontbuffer_tracking.c | 0
> > tests/{ => i915}/kms_invalid_dotclock.c | 0
> > tests/{ => i915}/kms_legacy_colorkey.c | 0
> > tests/{ => i915}/kms_pipe_b_c_ivb.c | 0
> > tests/{ => i915}/kms_psr.c | 0
> > tests/{ => i915}/kms_psr2_sf.c | 0
> > tests/{ => i915}/kms_psr2_su.c | 0
> > tests/kms_3d.c | 5 ++--
> > tests/kms_big_joiner.c | 2 +-
> > tests/kms_cursor_edge_walk.c | 9 ++++---
> > tests/kms_fbcon_fbt.c | 3 ++-
> > tests/kms_force_connector_basic.c | 4 +--
> > tests/kms_hdmi_inject.c | 14 +++++-----
> > tests/kms_mmap_write_crc.c | 8 +++---
> > tests/kms_pwrite_crc.c | 8 +++---
> > tests/kms_tv_load_detect.c | 2 +-
> > tests/kms_universal_plane.c | 10 ++++---
> > tests/meson.build | 30 ++++++++++-----------
> > 26 files changed, 50 insertions(+), 45 deletions(-)
> > rename tests/{ => i915}/kms_big_fb.c (100%)
> > rename tests/{ => i915}/kms_busy.c (100%)
> > rename tests/{ => i915}/kms_ccs.c (100%)
> > rename tests/{ => i915}/kms_cdclk.c (100%)
> > rename tests/{ => i915}/kms_draw_crc.c (100%)
> > rename tests/{ => i915}/kms_fence_pin_leak.c (100%)
> > rename tests/{ => i915}/kms_flip_scaled_crc.c (100%)
> > rename tests/{ => i915}/kms_flip_tiling.c (100%)
> > rename tests/{ => i915}/kms_frontbuffer_tracking.c (100%)
> > rename tests/{ => i915}/kms_invalid_dotclock.c (100%)
> > rename tests/{ => i915}/kms_legacy_colorkey.c (100%)
> > rename tests/{ => i915}/kms_pipe_b_c_ivb.c (100%)
> > rename tests/{ => i915}/kms_psr.c (100%)
> > rename tests/{ => i915}/kms_psr2_sf.c (100%)
> > rename tests/{ => i915}/kms_psr2_su.c (100%)
> >
> > diff --git a/tests/kms_big_fb.c b/tests/i915/kms_big_fb.c
> > similarity index 100%
> > rename from tests/kms_big_fb.c
> > rename to tests/i915/kms_big_fb.c
> > diff --git a/tests/kms_busy.c b/tests/i915/kms_busy.c
> > similarity index 100%
> > rename from tests/kms_busy.c
> > rename to tests/i915/kms_busy.c
> > diff --git a/tests/kms_ccs.c b/tests/i915/kms_ccs.c
> > similarity index 100%
> > rename from tests/kms_ccs.c
> > rename to tests/i915/kms_ccs.c
> > diff --git a/tests/kms_cdclk.c b/tests/i915/kms_cdclk.c
> > similarity index 100%
> > rename from tests/kms_cdclk.c
> > rename to tests/i915/kms_cdclk.c
> > diff --git a/tests/kms_draw_crc.c b/tests/i915/kms_draw_crc.c
> > similarity index 100%
> > rename from tests/kms_draw_crc.c
> > rename to tests/i915/kms_draw_crc.c
> > diff --git a/tests/kms_fence_pin_leak.c b/tests/i915/kms_fence_pin_leak.c
> > similarity index 100%
> > rename from tests/kms_fence_pin_leak.c
> > rename to tests/i915/kms_fence_pin_leak.c
> > diff --git a/tests/kms_flip_scaled_crc.c b/tests/i915/kms_flip_scaled_crc.c
> > similarity index 100%
> > rename from tests/kms_flip_scaled_crc.c
> > rename to tests/i915/kms_flip_scaled_crc.c
> > diff --git a/tests/kms_flip_tiling.c b/tests/i915/kms_flip_tiling.c
> > similarity index 100%
> > rename from tests/kms_flip_tiling.c
> > rename to tests/i915/kms_flip_tiling.c
> > diff --git a/tests/kms_frontbuffer_tracking.c
> b/tests/i915/kms_frontbuffer_tracking.c
> > similarity index 100%
> > rename from tests/kms_frontbuffer_tracking.c
> > rename to tests/i915/kms_frontbuffer_tracking.c
> > diff --git a/tests/kms_invalid_dotclock.c
> b/tests/i915/kms_invalid_dotclock.c
> > similarity index 100%
> > rename from tests/kms_invalid_dotclock.c
> > rename to tests/i915/kms_invalid_dotclock.c
> > diff --git a/tests/kms_legacy_colorkey.c b/tests/i915/kms_legacy_colorkey.c
> > similarity index 100%
> > rename from tests/kms_legacy_colorkey.c
> > rename to tests/i915/kms_legacy_colorkey.c
> > diff --git a/tests/kms_pipe_b_c_ivb.c b/tests/i915/kms_pipe_b_c_ivb.c
> > similarity index 100%
> > rename from tests/kms_pipe_b_c_ivb.c
> > rename to tests/i915/kms_pipe_b_c_ivb.c
> > diff --git a/tests/kms_psr.c b/tests/i915/kms_psr.c
> > similarity index 100%
> > rename from tests/kms_psr.c
> > rename to tests/i915/kms_psr.c
> > diff --git a/tests/kms_psr2_sf.c b/tests/i915/kms_psr2_sf.c
> > similarity index 100%
> > rename from tests/kms_psr2_sf.c
> > rename to tests/i915/kms_psr2_sf.c
> > diff --git a/tests/kms_psr2_su.c b/tests/i915/kms_psr2_su.c
> > similarity index 100%
> > rename from tests/kms_psr2_su.c
> > rename to tests/i915/kms_psr2_su.c
> > diff --git a/tests/kms_3d.c b/tests/kms_3d.c
> > index 2e438b430..c29d1d57c 100644
> > --- a/tests/kms_3d.c
> > +++ b/tests/kms_3d.c
> > @@ -34,7 +34,7 @@ igt_simple_main
> > const struct edid *edid;
> > int mode_count, connector_id;
> >
> > - drm_fd = drm_open_driver_master(DRIVER_INTEL);
> > + drm_fd = drm_open_driver_master(DRIVER_ANY);
> >
> > res = drmModeGetResources(drm_fd);
> > igt_require(res);
> > @@ -44,7 +44,8 @@ igt_simple_main
> > /* find an hdmi connector */
> > for (int i = 0; i < res->count_connectors; i++) {
> > connector = drmModeGetConnectorCurrent(drm_fd, res-
> >connectors[i]);
> > - if (connector->connector_type == DRM_MODE_CONNECTOR_HDMIA)
> > + if (connector->connector_type == DRM_MODE_CONNECTOR_HDMIA ||
> > + connector->connector_type == DRM_MODE_CONNECTOR_HDMIB)
> > break;
> > drmModeFreeConnector(connector);
> > connector = NULL;
> > diff --git a/tests/kms_big_joiner.c b/tests/kms_big_joiner.c
> > index 4f1f3152e..45c5b2a2a 100644
> > --- a/tests/kms_big_joiner.c
> > +++ b/tests/kms_big_joiner.c
> > @@ -258,7 +258,7 @@ igt_main
> > uint16_t width = 0, height = 0;
> >
> > igt_fixture {
> > - data.drm_fd = drm_open_driver_master(DRIVER_INTEL);
> > + data.drm_fd = drm_open_driver_master(DRIVER_ANY);
> > kmstest_set_vt_graphics_mode();
> >
> > igt_display_require(&data.display, data.drm_fd);
> > diff --git a/tests/kms_cursor_edge_walk.c b/tests/kms_cursor_edge_walk.c
> > index e9902f5e7..66781eec6 100644
> > --- a/tests/kms_cursor_edge_walk.c
> > +++ b/tests/kms_cursor_edge_walk.c
> > @@ -94,8 +94,8 @@ static void cursor_move(data_t *data, int x, int y, int i)
> > * fails). So let's accept a failure from the ioctl in that case.
> > */
> > igt_assert(drmModeMoveCursor(data->drm_fd, crtc_id, x, y) == 0 ||
> > - (IS_CHERRYVIEW(data->devid) && data->pipe == PIPE_C &&
> > - x < 0 && x > -data->curw));
> > + (data->devid && IS_CHERRYVIEW(data->devid) &&
> > + data->pipe == PIPE_C && x < 0 && x > -data->curw));
> > igt_wait_for_vblank(data->drm_fd,
> > display->pipes[data->pipe].crtc_offset);
> > }
> > @@ -312,9 +312,10 @@ igt_main_args("", long_opts, help_str, opt_handler,
> &data)
> > igt_fixture {
> > int ret;
> >
> > - data.drm_fd = drm_open_driver_master(DRIVER_INTEL);
> > + data.drm_fd = drm_open_driver_master(DRIVER_ANY);
> >
> > - data.devid = intel_get_drm_devid(data.drm_fd);
> > + if (is_i915_device(data.drm_fd))
> > + data.devid = intel_get_drm_devid(data.drm_fd);
> >
> > ret = drmGetCap(data.drm_fd, DRM_CAP_CURSOR_WIDTH,
> &max_curw);
> > igt_assert(ret == 0 || errno == EINVAL);
> > diff --git a/tests/kms_fbcon_fbt.c b/tests/kms_fbcon_fbt.c
> > index 98c452c4f..ff9c5f7bc 100644
> > --- a/tests/kms_fbcon_fbt.c
> > +++ b/tests/kms_fbcon_fbt.c
> > @@ -337,6 +337,7 @@ static void subtest(struct drm_info *drm, struct feature
> *feature, bool suspend)
> > igt_device_set_master(drm->fd);
> > kmstest_set_vt_graphics_mode();
> >
> > + igt_require_intel(drm->fd);
> Should this be moved to i915 as well?
> > igt_require(feature->supported_on_chipset(drm->fd, drm-
> >debugfs_fd));
> >
> > disable_features(drm->fd, drm->debugfs_fd);
> > @@ -378,7 +379,7 @@ static void setup_environment(struct drm_info *drm)
> > {
> > int i;
> >
> > - drm->fd = drm_open_driver_master(DRIVER_INTEL);
> > + drm->fd = drm_open_driver_master(DRIVER_ANY);
> you added igt_require_intel(drm->fd); in subtest() to changing to
> DRIVER_ANY doesn't make the test run on other devices.
> > igt_require(drm->fd >= 0);
> > drm->debugfs_fd = igt_debugfs_dir(drm->fd);
> > igt_require(drm->debugfs_fd >= 0);
> > diff --git a/tests/kms_force_connector_basic.c
> b/tests/kms_force_connector_basic.c
> > index 3b0146fe9..683d36720 100644
> > --- a/tests/kms_force_connector_basic.c
> > +++ b/tests/kms_force_connector_basic.c
> > @@ -37,7 +37,7 @@ static void reset_connectors(void)
> > drmModeRes *res;
> > drmModeConnector *connector = NULL;
> >
> > - drm_fd = drm_open_driver_master(DRIVER_INTEL);
> > + drm_fd = drm_open_driver_master(DRIVER_ANY);
> > res = drmModeGetResources(drm_fd);
> >
> > for (int i = 0; i < res->count_connectors; i++) {
> > @@ -86,7 +86,7 @@ igt_main_args("", long_opts, help_str, opt_handler, NULL)
> > igt_fixture {
> > unsigned connector_id = 0;
> >
> > - drm_fd = drm_open_driver_master(DRIVER_INTEL);
> > + drm_fd = drm_open_driver_master(DRIVER_ANY);
> >
> > res = drmModeGetResources(drm_fd);
> > igt_require(res);
> > diff --git a/tests/kms_hdmi_inject.c b/tests/kms_hdmi_inject.c
> > index ad2dde569..13a05eb1e 100644
> > --- a/tests/kms_hdmi_inject.c
> > +++ b/tests/kms_hdmi_inject.c
> > @@ -63,7 +63,8 @@ get_connector(int drm_fd, drmModeRes *res)
> > connector =
> > drmModeGetConnectorCurrent(drm_fd, res-
> >connectors[i]);
> >
> > - if (connector->connector_type == DRM_MODE_CONNECTOR_HDMIA)
> > + if (connector->connector_type == DRM_MODE_CONNECTOR_HDMIA ||
> > + connector->connector_type == DRM_MODE_CONNECTOR_HDMIB)
> > break;
> >
> > drmModeFreeConnector(connector);
> > @@ -82,12 +83,13 @@ hdmi_inject_4k(int drm_fd, drmModeConnector *connector)
> > int fb_id;
> > struct igt_fb fb;
> > uint8_t found_4k_mode = 0;
> > - uint32_t devid;
> >
> > - devid = intel_get_drm_devid(drm_fd);
> > + if (is_i915_device(drm_fd)) {
> > + uint32_t devid = intel_get_drm_devid(drm_fd);
> >
> > - /* 4K requires at least HSW */
> > - igt_require(IS_HASWELL(devid) || intel_display_ver(devid) >= 8);
> > + /* 4K requires at least HSW */
> > + igt_require(IS_HASWELL(devid) || intel_display_ver(devid) >=
> 8);
> > + }
> >
> > edid = igt_kms_get_4k_edid();
> > kmstest_force_edid(drm_fd, connector, edid);
> > @@ -195,7 +197,7 @@ igt_main
> > drmModeConnector *connector;
> >
> > igt_fixture {
> > - drm_fd = drm_open_driver_master(DRIVER_INTEL);
> > + drm_fd = drm_open_driver_master(DRIVER_ANY);
> >
> > res = drmModeGetResources(drm_fd);
> > igt_require(res);
> > diff --git a/tests/kms_mmap_write_crc.c b/tests/kms_mmap_write_crc.c
> > index 2e323a209..0c697c246 100644
> > --- a/tests/kms_mmap_write_crc.c
> > +++ b/tests/kms_mmap_write_crc.c
> > @@ -51,7 +51,6 @@ typedef struct {
> > enum pipe pipe;
> > igt_crc_t ref_crc;
> > igt_pipe_crc_t *pipe_crc;
> > - uint32_t devid;
> > } data_t;
> >
> > static int ioctl_sync = true;
> > @@ -103,6 +102,7 @@ static void test(data_t *data)
> > igt_display_commit(display);
> >
> > /* make sure caching mode has become UC/WT */
> > + igt_require_intel(data->drm_fd);
> same comment as above. here and other places where the same thing is done.
> Just curious if this is meant to allow other SoCs to expand or it's
> WIP to have it removed.
Intension of this patch is to move i915 specific KMS tests to tests/i915 dir,
and make other KMS tests to support all SOCs. As of now only i915 is using
these tests, so I have added this check. Whoever wants to expand/use these
tests can remove this check.
> > caching = gem_get_caching(data->drm_fd, fb->gem_handle);
> > igt_assert(caching == I915_CACHING_NONE || caching ==
> I915_CACHING_DISPLAY);
> >
> > @@ -204,7 +204,7 @@ static void cleanup_crtc(data_t *data)
> >
> > igt_plane_set_fb(data->primary, NULL);
> >
> > - igt_output_set_pipe(output, PIPE_ANY);
> > + igt_output_set_pipe(output, PIPE_NONE);
> > igt_display_commit(display);
> >
> > igt_remove_fb(data->drm_fd, &data->fb[0]);
> > @@ -272,9 +272,7 @@ igt_main_args("n", NULL, NULL, opt_handler, NULL)
> > int i;
> >
> > igt_fixture {
> > - data.drm_fd = drm_open_driver_master(DRIVER_INTEL);
> > -
> > - data.devid = intel_get_drm_devid(data.drm_fd);
> > + data.drm_fd = drm_open_driver_master(DRIVER_ANY);
> >
> > kmstest_set_vt_graphics_mode();
> >
> > diff --git a/tests/kms_pwrite_crc.c b/tests/kms_pwrite_crc.c
> > index dc32a070c..a1642df6f 100644
> > --- a/tests/kms_pwrite_crc.c
> > +++ b/tests/kms_pwrite_crc.c
> > @@ -42,7 +42,6 @@ typedef struct {
> > enum pipe pipe;
> > igt_crc_t ref_crc;
> > igt_pipe_crc_t *pipe_crc;
> > - uint32_t devid;
> > } data_t;
> >
> > static void test(data_t *data)
> > @@ -77,6 +76,7 @@ static void test(data_t *data)
> > 0, 0, fb->width << 16, fb->height << 16);
> >
> > /* make sure caching mode has become UC/WT */
> > + igt_require_intel(data->drm_fd);
> > caching = gem_get_caching(data->drm_fd, fb->gem_handle);
> > igt_assert(caching == I915_CACHING_NONE || caching ==
> I915_CACHING_DISPLAY);
> >
> > @@ -141,7 +141,7 @@ static void cleanup_crtc(data_t *data)
> >
> > igt_plane_set_fb(data->primary, NULL);
> >
> > - igt_output_set_pipe(output, PIPE_ANY);
> > + igt_output_set_pipe(output, PIPE_NONE);
> > igt_display_commit(display);
> >
> > igt_remove_fb(data->drm_fd, &data->fb[0]);
> > @@ -174,9 +174,7 @@ static data_t data;
> >
> > igt_simple_main
> > {
> > - data.drm_fd = drm_open_driver_master(DRIVER_INTEL);
> > -
> > - data.devid = intel_get_drm_devid(data.drm_fd);
> > + data.drm_fd = drm_open_driver_master(DRIVER_ANY);
> >
> > kmstest_set_vt_graphics_mode();
> >
> > diff --git a/tests/kms_tv_load_detect.c b/tests/kms_tv_load_detect.c
> > index caac3aecb..80ced9918 100644
> > --- a/tests/kms_tv_load_detect.c
> > +++ b/tests/kms_tv_load_detect.c
> > @@ -34,7 +34,7 @@ igt_main
> > drmModeConnector *tv_connector = NULL, *temp;
> >
> > igt_fixture {
> > - drm_fd = drm_open_driver_master(DRIVER_INTEL);
> > + drm_fd = drm_open_driver_master(DRIVER_ANY);
> >
> > res = drmModeGetResources(drm_fd);
> > igt_require(res);
> > diff --git a/tests/kms_universal_plane.c b/tests/kms_universal_plane.c
> > index c9a9cd47a..ea9bd9a56 100644
> > --- a/tests/kms_universal_plane.c
> > +++ b/tests/kms_universal_plane.c
> > @@ -107,7 +107,7 @@ functional_test_fini(functional_test_t *test,
> igt_output_t *output)
> > igt_remove_fb(test->data->drm_fd, &test->red_fb);
> > igt_remove_fb(test->data->drm_fd, &test->yellow_fb);
> >
> > - igt_output_set_pipe(output, PIPE_ANY);
> > + igt_output_set_pipe(output, PIPE_NONE);
> > igt_display_commit2(&test->data->display, COMMIT_LEGACY);
> > }
> >
> > @@ -382,6 +382,7 @@ sanity_test_pipe(data_t *data, enum pipe pipe,
> igt_output_t *output)
> > * doesn't cover CRTC (should fail on pre-gen9 and succeed on
> > * gen9+).
> > */
> > + igt_require_intel(data->drm_fd);
> > igt_plane_set_fb(primary, &test.undersized_fb);
> > expect = (data->display_ver < 9) ? -EINVAL : 0;
> > igt_assert(igt_display_try_commit2(&data->display, COMMIT_UNIVERSAL)
> == expect);
> > @@ -579,6 +580,7 @@ cursor_leak_test_pipe(data_t *data, enum pipe pipe,
> igt_output_t *output)
> >
> > igt_require_pipe(display, pipe);
> > igt_require(display->has_cursor_plane);
> > + igt_require_intel(data->drm_fd);
> >
> > igt_output_set_pipe(output, pipe);
> > mode = igt_output_get_mode(output);
> > @@ -704,6 +706,7 @@ gen9_test_pipe(data_t *data, enum pipe pipe,
> igt_output_t *output)
> >
> > int ret = 0;
> >
> > + igt_require_intel(data->drm_fd);
> > igt_skip_on(data->display_ver < 9);
> > igt_require_pipe(&data->display, pipe);
> >
> > @@ -797,8 +800,9 @@ igt_main
> > enum pipe pipe;
> >
> > igt_fixture {
> > - data.drm_fd = drm_open_driver_master(DRIVER_INTEL);
> > - data.display_ver =
> intel_display_ver(intel_get_drm_devid(data.drm_fd));
> > + data.drm_fd = drm_open_driver_master(DRIVER_ANY);
> > + if (is_i915_device(data.drm_fd))
> > + data.display_ver =
> intel_display_ver(intel_get_drm_devid(data.drm_fd));
> >
> > kmstest_set_vt_graphics_mode();
> >
> > diff --git a/tests/meson.build b/tests/meson.build
> > index 1bdfddbb2..f4f33ca60 100644
> > --- a/tests/meson.build
> > +++ b/tests/meson.build
> > @@ -19,11 +19,7 @@ test_progs = [
> > 'kms_atomic',
> > 'kms_atomic_interruptible',
> > 'kms_atomic_transition',
> > - 'kms_big_fb',
> > 'kms_big_joiner' ,
> > - 'kms_busy',
> > - 'kms_ccs',
> > - 'kms_cdclk',
> > 'kms_concurrent',
> > 'kms_content_protection',
> > 'kms_cursor_crc',
> > @@ -32,26 +28,18 @@ test_progs = [
> > 'kms_dither',
> > 'kms_dp_aux_dev',
> > 'kms_dp_tiled_display',
> > - 'kms_draw_crc',
> > 'kms_dsc',
> > 'kms_fbcon_fbt',
> > - 'kms_fence_pin_leak',
> > 'kms_flip',
> > 'kms_flip_event_leak',
> > - 'kms_flip_scaled_crc',
> > - 'kms_flip_tiling',
> > 'kms_force_connector_basic',
> > - 'kms_frontbuffer_tracking',
> > 'kms_getfb',
> > 'kms_hdmi_inject',
> > 'kms_hdr',
> > - 'kms_invalid_dotclock',
> > 'kms_lease',
> > - 'kms_legacy_colorkey',
> > 'kms_mmap_write_crc',
> > 'kms_multipipe_modeset',
> > 'kms_panel_fitting',
> > - 'kms_pipe_b_c_ivb',
> > 'kms_pipe_crc_basic',
> > 'kms_plane',
> > 'kms_plane_alpha_blend',
> > @@ -62,9 +50,6 @@ test_progs = [
> > 'kms_prime',
> > 'kms_prop_blob',
> > 'kms_properties',
> > - 'kms_psr',
> > - 'kms_psr2_su',
> > - 'kms_psr2_sf',
> > 'kms_pwrite_crc',
> > 'kms_rmfb',
> > 'kms_rotation_crc',
> > @@ -235,6 +220,21 @@ i915_progs = [
> > 'i915_query',
> > 'i915_selftest',
> > 'i915_suspend',
> > + 'kms_big_fb',
> Just curious, now that kms tests moved to i915, do you wanna keep it
> with the same prefix kms_ instead of adding i915_?
Even though these tests moved i915, still I can see these tests are KMS
tests only :-)
Perti, any suggestions?
- Bhanu
> > + 'kms_busy',
> > + 'kms_ccs',
> > + 'kms_cdclk',
> > + 'kms_draw_crc',
> > + 'kms_fence_pin_leak',
> > + 'kms_flip_scaled_crc',
> > + 'kms_flip_tiling',
> > + 'kms_frontbuffer_tracking',
> > + 'kms_invalid_dotclock',
> > + 'kms_legacy_colorkey',
> > + 'kms_pipe_b_c_ivb',
> > + 'kms_psr',
> > + 'kms_psr2_su',
> > + 'kms_psr2_sf',
> > 'sysfs_clients',
> > 'sysfs_defaults',
> > 'sysfs_heartbeat_interval',
> > --
> > 2.32.0
> >
More information about the igt-dev
mailing list