* [igt-dev] [PATCH 0/7] removal of igt_skip_on_simulation() from i915/ tests
@ 2019-10-16 7:49 Swati Sharma
2019-10-16 7:49 ` [igt-dev] [PATCH 1/7] tests/i915/gem: remove igt_skip_on_simulation() Swati Sharma
` (12 more replies)
0 siblings, 13 replies; 17+ messages in thread
From: Swati Sharma @ 2019-10-16 7:49 UTC (permalink / raw)
To: igt-dev; +Cc: lucas.demarchi, martin.peres, maarten.lankhorst
Removing igt_skip_on_simulation() since this feature is not supported
anymore by i915 driver. Removing it from gem, kms, i915 specific, tools
and benchmark tests.
Swati Sharma (7):
tests/i915/gem: remove igt_skip_on_simulation()
tests/i915/kms: remove igt_skip_on_simulation()
tests/i915: remove igt_skip_on_simulation()
tests: remove igt_skip_on_simulation()
tools: remove igt_skip_on_simulation()
benchmarks: remove igt_skip_on_simulation()
lib: remove igt_skip_on_simulation
benchmarks/gem_userptr_benchmark.c | 2 --
lib/igt_aux.c | 7 -------
lib/intel_os.c | 2 --
tests/debugfs_test.c | 2 --
tests/i915/gem_busy.c | 2 --
tests/i915/gem_caching.c | 2 --
tests/i915/gem_concurrent_all.c | 2 --
tests/i915/gem_create.c | 2 --
tests/i915/gem_cs_prefetch.c | 2 --
tests/i915/gem_cs_tlb.c | 2 --
tests/i915/gem_ctx_exec.c | 2 --
tests/i915/gem_ctx_switch.c | 6 ------
tests/i915/gem_ctx_thrash.c | 2 --
tests/i915/gem_double_irq_loop.c | 2 --
tests/i915/gem_eio.c | 2 --
tests/i915/gem_exec_async.c | 2 --
tests/i915/gem_exec_balancer.c | 2 --
tests/i915/gem_exec_blt.c | 2 --
tests/i915/gem_exec_capture.c | 2 --
tests/i915/gem_exec_fence.c | 2 --
tests/i915/gem_exec_gttfill.c | 2 --
tests/i915/gem_exec_lut_handle.c | 2 --
tests/i915/gem_exec_schedule.c | 2 --
tests/i915/gem_exec_suspend.c | 2 --
tests/i915/gem_fence_thrash.c | 2 --
tests/i915/gem_fence_upload.c | 2 --
tests/i915/gem_fenced_exec_thrash.c | 2 --
tests/i915/gem_flink_race.c | 2 --
tests/i915/gem_gtt_cpu_tlb.c | 3 ---
tests/i915/gem_gtt_hog.c | 2 --
tests/i915/gem_gtt_speed.c | 2 --
tests/i915/gem_largeobject.c | 2 --
tests/i915/gem_madvise.c | 2 --
tests/i915/gem_mmap_offset_exhaustion.c | 2 --
tests/i915/gem_partial_pwrite_pread.c | 2 --
tests/i915/gem_persistent_relocs.c | 2 --
tests/i915/gem_pread_after_blit.c | 2 --
tests/i915/gem_pwrite_pread.c | 2 --
tests/i915/gem_pwrite_snooped.c | 2 --
tests/i915/gem_read_read_speed.c | 2 --
tests/i915/gem_readwrite.c | 2 --
tests/i915/gem_reloc_vs_gpu.c | 2 --
tests/i915/gem_render_linear_blits.c | 3 ---
tests/i915/gem_render_tiled_blits.c | 3 ---
tests/i915/gem_reset_stats.c | 1 -
tests/i915/gem_ringfill.c | 2 --
tests/i915/gem_set_tiling_vs_blt.c | 2 --
tests/i915/gem_set_tiling_vs_gtt.c | 2 --
tests/i915/gem_set_tiling_vs_pwrite.c | 2 --
tests/i915/gem_shrink.c | 2 --
tests/i915/gem_softpin.c | 2 --
tests/i915/gem_spin_batch.c | 2 --
tests/i915/gem_stolen.c | 2 --
tests/i915/gem_storedw_batches_loop.c | 2 --
tests/i915/gem_sync.c | 2 --
tests/i915/gem_threaded_access_tiled.c | 2 --
tests/i915/gem_tiled_fence_blits.c | 3 ---
tests/i915/gem_tiled_partial_pwrite_pread.c | 2 --
tests/i915/gem_unfence_active_buffers.c | 2 --
tests/i915/gem_unref_active_buffers.c | 2 --
tests/i915/gem_wait.c | 2 --
tests/i915/gem_write_read_ring_switch.c | 2 --
tests/i915/i915_hangman.c | 2 --
tests/i915/i915_pm_backlight.c | 2 --
tests/i915/i915_pm_dc.c | 1 -
tests/i915/i915_pm_rc6_residency.c | 2 --
tests/i915/i915_pm_rps.c | 2 --
tests/i915/i915_pm_sseu.c | 3 ---
tests/i915/i915_suspend.c | 2 --
tests/kms_atomic_interruptible.c | 2 --
tests/kms_atomic_transition.c | 2 --
tests/kms_available_modes_crc.c | 2 --
tests/kms_big_fb.c | 2 --
tests/kms_busy.c | 2 --
tests/kms_chamelium.c | 2 --
tests/kms_color.c | 2 --
tests/kms_concurrent.c | 2 --
tests/kms_content_protection.c | 2 --
tests/kms_crtc_background_color.c | 2 --
tests/kms_cursor_crc.c | 2 --
tests/kms_cursor_edge_walk.c | 2 --
tests/kms_fence_pin_leak.c | 2 --
tests/kms_flip_event_leak.c | 2 --
tests/kms_flip_tiling.c | 2 --
tests/kms_invalid_dotclock.c | 2 --
tests/kms_legacy_colorkey.c | 2 --
tests/kms_mmap_write_crc.c | 2 --
tests/kms_panel_fitting.c | 2 --
tests/kms_pipe_b_c_ivb.c | 2 --
tests/kms_pipe_crc_basic.c | 2 --
tests/kms_plane.c | 2 --
tests/kms_plane_cursor.c | 2 --
tests/kms_plane_lowres.c | 2 --
tests/kms_plane_multiple.c | 2 --
tests/kms_plane_scaling.c | 2 --
tests/kms_properties.c | 2 --
tests/kms_psr.c | 2 --
tests/kms_psr2_su.c | 2 --
tests/kms_pwrite_crc.c | 2 --
tests/kms_rmfb.c | 2 --
tests/kms_rotation_crc.c | 2 --
tests/kms_sequence.c | 2 --
tests/kms_setmode.c | 2 --
tests/kms_universal_plane.c | 2 --
tests/kms_vblank.c | 2 --
tests/kms_vrr.c | 2 --
tests/perf.c | 2 --
tests/prime_busy.c | 1 -
tests/prime_mmap_kms.c | 2 --
tests/prime_udl.c | 2 --
tests/testdisplay.c | 1 -
tests/tools_test.c | 2 --
tools/intel_dp_compliance.c | 2 --
113 files changed, 236 deletions(-)
--
2.23.0
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 17+ messages in thread* [igt-dev] [PATCH 1/7] tests/i915/gem: remove igt_skip_on_simulation() 2019-10-16 7:49 [igt-dev] [PATCH 0/7] removal of igt_skip_on_simulation() from i915/ tests Swati Sharma @ 2019-10-16 7:49 ` Swati Sharma 2019-10-16 7:49 ` [igt-dev] [PATCH 2/7] tests/i915/kms: " Swati Sharma ` (11 subsequent siblings) 12 siblings, 0 replies; 17+ messages in thread From: Swati Sharma @ 2019-10-16 7:49 UTC (permalink / raw) To: igt-dev; +Cc: lucas.demarchi, martin.peres, maarten.lankhorst Removing igt_skip_on_simulation() from all the gem tests since this feature is not supported anymore. Signed-off-by: Swati Sharma <swati2.sharma@intel.com> Signed-off-by: Karthik B S <karthik.b.s@intel.com> --- tests/i915/gem_busy.c | 2 -- tests/i915/gem_caching.c | 2 -- tests/i915/gem_concurrent_all.c | 2 -- tests/i915/gem_create.c | 2 -- tests/i915/gem_cs_prefetch.c | 2 -- tests/i915/gem_cs_tlb.c | 2 -- tests/i915/gem_ctx_exec.c | 2 -- tests/i915/gem_ctx_switch.c | 6 ------ tests/i915/gem_ctx_thrash.c | 2 -- tests/i915/gem_double_irq_loop.c | 2 -- tests/i915/gem_eio.c | 2 -- tests/i915/gem_exec_async.c | 2 -- tests/i915/gem_exec_balancer.c | 2 -- tests/i915/gem_exec_blt.c | 2 -- tests/i915/gem_exec_capture.c | 2 -- tests/i915/gem_exec_fence.c | 2 -- tests/i915/gem_exec_gttfill.c | 2 -- tests/i915/gem_exec_lut_handle.c | 2 -- tests/i915/gem_exec_schedule.c | 2 -- tests/i915/gem_exec_suspend.c | 2 -- tests/i915/gem_fence_thrash.c | 2 -- tests/i915/gem_fence_upload.c | 2 -- tests/i915/gem_fenced_exec_thrash.c | 2 -- tests/i915/gem_flink_race.c | 2 -- tests/i915/gem_gtt_cpu_tlb.c | 3 --- tests/i915/gem_gtt_hog.c | 2 -- tests/i915/gem_gtt_speed.c | 2 -- tests/i915/gem_largeobject.c | 2 -- tests/i915/gem_madvise.c | 2 -- tests/i915/gem_mmap_offset_exhaustion.c | 2 -- tests/i915/gem_partial_pwrite_pread.c | 2 -- tests/i915/gem_persistent_relocs.c | 2 -- tests/i915/gem_pread_after_blit.c | 2 -- tests/i915/gem_pwrite_pread.c | 2 -- tests/i915/gem_pwrite_snooped.c | 2 -- tests/i915/gem_read_read_speed.c | 2 -- tests/i915/gem_readwrite.c | 2 -- tests/i915/gem_reloc_vs_gpu.c | 2 -- tests/i915/gem_render_linear_blits.c | 3 --- tests/i915/gem_render_tiled_blits.c | 3 --- tests/i915/gem_reset_stats.c | 1 - tests/i915/gem_ringfill.c | 2 -- tests/i915/gem_set_tiling_vs_blt.c | 2 -- tests/i915/gem_set_tiling_vs_gtt.c | 2 -- tests/i915/gem_set_tiling_vs_pwrite.c | 2 -- tests/i915/gem_shrink.c | 2 -- tests/i915/gem_softpin.c | 2 -- tests/i915/gem_spin_batch.c | 2 -- tests/i915/gem_stolen.c | 2 -- tests/i915/gem_storedw_batches_loop.c | 2 -- tests/i915/gem_sync.c | 2 -- tests/i915/gem_threaded_access_tiled.c | 2 -- tests/i915/gem_tiled_fence_blits.c | 3 --- tests/i915/gem_tiled_partial_pwrite_pread.c | 2 -- tests/i915/gem_unfence_active_buffers.c | 2 -- tests/i915/gem_unref_active_buffers.c | 2 -- tests/i915/gem_wait.c | 2 -- tests/i915/gem_write_read_ring_switch.c | 2 -- 58 files changed, 123 deletions(-) diff --git a/tests/i915/gem_busy.c b/tests/i915/gem_busy.c index a5535870..4ae88a96 100644 --- a/tests/i915/gem_busy.c +++ b/tests/i915/gem_busy.c @@ -557,7 +557,6 @@ igt_main igt_subtest_f("%shang-%s", e->class == I915_ENGINE_CLASS_RENDER ? "basic-" : "", e->name) { - igt_skip_on_simulation(); gem_quiescent_gpu(fd); basic(fd, e, HANG); } @@ -571,7 +570,6 @@ igt_main __for_each_physical_engine(fd, e) { igt_subtest_f("extended-hang-%s", e->name) { - igt_skip_on_simulation(); igt_require(gem_class_can_store_dword(fd, e->class)); gem_quiescent_gpu(fd); diff --git a/tests/i915/gem_caching.c b/tests/i915/gem_caching.c index ce9e6345..7c1f6410 100644 --- a/tests/i915/gem_caching.c +++ b/tests/i915/gem_caching.c @@ -106,8 +106,6 @@ igt_main uint8_t *cpu_ptr; uint8_t *gtt_ptr; - igt_skip_on_simulation(); - igt_fixture { srandom(0xdeadbeef); diff --git a/tests/i915/gem_concurrent_all.c b/tests/i915/gem_concurrent_all.c index 266995d1..36cd3574 100644 --- a/tests/i915/gem_concurrent_all.c +++ b/tests/i915/gem_concurrent_all.c @@ -1778,8 +1778,6 @@ igt_main char name[80]; int count = 0; - igt_skip_on_simulation(); - if (strstr(igt_test_name(), "all")) all = true; diff --git a/tests/i915/gem_create.c b/tests/i915/gem_create.c index aed7d1ce..1a657be0 100644 --- a/tests/i915/gem_create.c +++ b/tests/i915/gem_create.c @@ -257,8 +257,6 @@ igt_main { int fd = -1; - igt_skip_on_simulation(); - igt_fixture { fd = drm_open_driver(DRIVER_INTEL); } diff --git a/tests/i915/gem_cs_prefetch.c b/tests/i915/gem_cs_prefetch.c index 2b865368..ffc757a1 100644 --- a/tests/i915/gem_cs_prefetch.c +++ b/tests/i915/gem_cs_prefetch.c @@ -141,8 +141,6 @@ igt_main { const struct intel_execution_engine *e; - igt_skip_on_simulation(); - for (e = intel_execution_engines; e->name; e++) igt_subtest_f("%s", e->name) test_ring(e->exec_id | e->flags); diff --git a/tests/i915/gem_cs_tlb.c b/tests/i915/gem_cs_tlb.c index 13de5499..0c1e0949 100644 --- a/tests/i915/gem_cs_tlb.c +++ b/tests/i915/gem_cs_tlb.c @@ -143,8 +143,6 @@ igt_main const struct intel_execution_engine2 *e; int fd = -1; - igt_skip_on_simulation(); - igt_fixture { fd = drm_open_driver(DRIVER_INTEL); igt_require_gem(fd); diff --git a/tests/i915/gem_ctx_exec.c b/tests/i915/gem_ctx_exec.c index 614a9f40..74ca2a3f 100644 --- a/tests/i915/gem_ctx_exec.c +++ b/tests/i915/gem_ctx_exec.c @@ -242,8 +242,6 @@ igt_main igt_subtest("reset-pin-leak") { int i; - igt_skip_on_simulation(); - /* * Use an explicit context to isolate the test from * any major code changes related to the per-file diff --git a/tests/i915/gem_ctx_switch.c b/tests/i915/gem_ctx_switch.c index c071def7..6b233ff0 100644 --- a/tests/i915/gem_ctx_switch.c +++ b/tests/i915/gem_ctx_switch.c @@ -365,9 +365,6 @@ igt_main igt_subtest_f("legacy-%s%s", e->name, p->name) single(fd, light, e2, p->flags, 1, 5); - - igt_skip_on_simulation(); - igt_subtest_f("legacy-%s-heavy%s", e->name, p->name) single(fd, heavy, e2, p->flags, 1, 5); @@ -394,9 +391,6 @@ igt_main igt_subtest_f("%s%s", e2->name, p->name) single(fd, light, e2, p->flags, 1, 5); - - igt_skip_on_simulation(); - igt_subtest_f("%s-heavy%s", e2->name, p->name) single(fd, heavy, e2, p->flags, 1, 5); igt_subtest_f("%s-forked%s", e2->name, p->name) diff --git a/tests/i915/gem_ctx_thrash.c b/tests/i915/gem_ctx_thrash.c index b25f95f1..b8bb0e26 100644 --- a/tests/i915/gem_ctx_thrash.c +++ b/tests/i915/gem_ctx_thrash.c @@ -361,8 +361,6 @@ static void threads(void) igt_main { - igt_skip_on_simulation(); - igt_subtest("single") single("single", false); igt_subtest("engines") diff --git a/tests/i915/gem_double_irq_loop.c b/tests/i915/gem_double_irq_loop.c index b326fc58..b7ff4a84 100644 --- a/tests/i915/gem_double_irq_loop.c +++ b/tests/i915/gem_double_irq_loop.c @@ -93,8 +93,6 @@ igt_simple_main int fd; int devid; - igt_skip_on_simulation(); - fd = drm_open_driver(DRIVER_INTEL); igt_require_gem(fd); gem_require_blitter(fd); diff --git a/tests/i915/gem_eio.c b/tests/i915/gem_eio.c index 892f3657..47534f8d 100644 --- a/tests/i915/gem_eio.c +++ b/tests/i915/gem_eio.c @@ -905,8 +905,6 @@ exit_handler(int sig) igt_main { - igt_skip_on_simulation(); - igt_fixture { fd = drm_open_driver(DRIVER_INTEL); igt_device_drop_master(fd); diff --git a/tests/i915/gem_exec_async.c b/tests/i915/gem_exec_async.c index 9a06af7e..4b0ba0d1 100644 --- a/tests/i915/gem_exec_async.c +++ b/tests/i915/gem_exec_async.c @@ -188,8 +188,6 @@ igt_main const struct intel_execution_engine *e; int fd = -1; - igt_skip_on_simulation(); - igt_fixture { fd = drm_open_driver_master(DRIVER_INTEL); igt_require_gem(fd); diff --git a/tests/i915/gem_exec_balancer.c b/tests/i915/gem_exec_balancer.c index e52f5df9..cf2a4c5d 100644 --- a/tests/i915/gem_exec_balancer.c +++ b/tests/i915/gem_exec_balancer.c @@ -1421,8 +1421,6 @@ igt_main { int i915 = -1; - igt_skip_on_simulation(); - igt_fixture { i915 = drm_open_driver(DRIVER_INTEL); igt_require_gem(i915); diff --git a/tests/i915/gem_exec_blt.c b/tests/i915/gem_exec_blt.c index ffb60d0c..fb2f3693 100644 --- a/tests/i915/gem_exec_blt.c +++ b/tests/i915/gem_exec_blt.c @@ -307,8 +307,6 @@ igt_main int min = -1, max = -1; int fd, sysfs; - igt_skip_on_simulation(); - igt_fixture { fd = drm_open_driver(DRIVER_INTEL); igt_require_gem(fd); diff --git a/tests/i915/gem_exec_capture.c b/tests/i915/gem_exec_capture.c index 4457496d..e6a549f3 100644 --- a/tests/i915/gem_exec_capture.c +++ b/tests/i915/gem_exec_capture.c @@ -531,8 +531,6 @@ igt_main int fd = -1; int dir = -1; - igt_skip_on_simulation(); - igt_fixture { int gen; diff --git a/tests/i915/gem_exec_fence.c b/tests/i915/gem_exec_fence.c index 2f04d7af..9ecb3770 100644 --- a/tests/i915/gem_exec_fence.c +++ b/tests/i915/gem_exec_fence.c @@ -1593,8 +1593,6 @@ igt_main igt_subtest_group { igt_hang_t hang; - igt_skip_on_simulation(); - igt_fixture { hang = igt_allow_hang(i915, 0, 0); } diff --git a/tests/i915/gem_exec_gttfill.c b/tests/i915/gem_exec_gttfill.c index efd612bb..1bf0f0bf 100644 --- a/tests/i915/gem_exec_gttfill.c +++ b/tests/i915/gem_exec_gttfill.c @@ -210,8 +210,6 @@ igt_main const struct intel_execution_engine *e; int device = -1; - igt_skip_on_simulation(); - igt_fixture { device = drm_open_driver(DRIVER_INTEL); igt_require_gem(device); diff --git a/tests/i915/gem_exec_lut_handle.c b/tests/i915/gem_exec_lut_handle.c index 98e6ae5a..b7b69157 100644 --- a/tests/i915/gem_exec_lut_handle.c +++ b/tests/i915/gem_exec_lut_handle.c @@ -101,8 +101,6 @@ igt_simple_main uint32_t reloc_handle; int size; - igt_skip_on_simulation(); - fd = drm_open_driver(DRIVER_INTEL); igt_require_gem(fd); diff --git a/tests/i915/gem_exec_schedule.c b/tests/i915/gem_exec_schedule.c index ddcb1f21..59b640bd 100644 --- a/tests/i915/gem_exec_schedule.c +++ b/tests/i915/gem_exec_schedule.c @@ -1669,8 +1669,6 @@ igt_main const struct intel_execution_engine *e; int fd = -1; - igt_skip_on_simulation(); - igt_fixture { igt_require_sw_sync(); diff --git a/tests/i915/gem_exec_suspend.c b/tests/i915/gem_exec_suspend.c index e43a16e9..bef3a167 100644 --- a/tests/i915/gem_exec_suspend.c +++ b/tests/i915/gem_exec_suspend.c @@ -243,8 +243,6 @@ igt_main igt_hang_t hang; int fd; - igt_skip_on_simulation(); - igt_fixture { fd = drm_open_driver_master(DRIVER_INTEL); igt_require_gem(fd); diff --git a/tests/i915/gem_fence_thrash.c b/tests/i915/gem_fence_thrash.c index 2d7fb2ff..c85c38bb 100644 --- a/tests/i915/gem_fence_thrash.c +++ b/tests/i915/gem_fence_thrash.c @@ -234,8 +234,6 @@ static int run_test(int threads_per_fence, void *f, int tiling, igt_main { - igt_skip_on_simulation(); - igt_subtest("bo-write-verify-none") igt_assert(run_test(0, bo_write_verify, I915_TILING_NONE, 80) == 0); diff --git a/tests/i915/gem_fence_upload.c b/tests/i915/gem_fence_upload.c index f3b0e62f..17580aff 100644 --- a/tests/i915/gem_fence_upload.c +++ b/tests/i915/gem_fence_upload.c @@ -388,8 +388,6 @@ static void wc_contention(void) igt_main { - igt_skip_on_simulation(); - igt_subtest("performance") performance(); igt_subtest("thread-contention") diff --git a/tests/i915/gem_fenced_exec_thrash.c b/tests/i915/gem_fenced_exec_thrash.c index 145b8bf8..e40fa438 100644 --- a/tests/i915/gem_fenced_exec_thrash.c +++ b/tests/i915/gem_fenced_exec_thrash.c @@ -160,8 +160,6 @@ igt_main unsigned int num_fences = 0; int fd = -1; - igt_skip_on_simulation(); - igt_fixture { fd = drm_open_driver(DRIVER_INTEL); igt_require_gem(fd); diff --git a/tests/i915/gem_flink_race.c b/tests/i915/gem_flink_race.c index 322befe9..c1f5d5d5 100644 --- a/tests/i915/gem_flink_race.c +++ b/tests/i915/gem_flink_race.c @@ -197,8 +197,6 @@ static void test_flink_close(void) igt_main { - igt_skip_on_simulation(); - igt_subtest("flink_name") test_flink_name(5); diff --git a/tests/i915/gem_gtt_cpu_tlb.c b/tests/i915/gem_gtt_cpu_tlb.c index cf3c543d..5d39e123 100644 --- a/tests/i915/gem_gtt_cpu_tlb.c +++ b/tests/i915/gem_gtt_cpu_tlb.c @@ -73,11 +73,8 @@ igt_simple_main int fd; int i; uint32_t handle; - uint32_t *ptr; - igt_skip_on_simulation(); - fd = drm_open_driver(DRIVER_INTEL); handle = gem_create(fd, OBJ_SIZE); diff --git a/tests/i915/gem_gtt_hog.c b/tests/i915/gem_gtt_hog.c index ca730649..7f2c26d7 100644 --- a/tests/i915/gem_gtt_hog.c +++ b/tests/i915/gem_gtt_hog.c @@ -163,8 +163,6 @@ igt_simple_main igt_require_gem(fd); close(fd); - igt_skip_on_simulation(); - data.fd = drm_open_driver(DRIVER_INTEL); data.devid = intel_get_drm_devid(data.fd); data.intel_gen = intel_gen(data.devid); diff --git a/tests/i915/gem_gtt_speed.c b/tests/i915/gem_gtt_speed.c index dfa7216c..e3cbc84c 100644 --- a/tests/i915/gem_gtt_speed.c +++ b/tests/i915/gem_gtt_speed.c @@ -112,8 +112,6 @@ igt_simple_main_args("s:", NULL, help_str, opt_handler, NULL) int loop, i, tiling; int fd; - igt_skip_on_simulation(); - igt_assert_f(size != 0, "Invalid object size specified\n"); if (cpu) { diff --git a/tests/i915/gem_largeobject.c b/tests/i915/gem_largeobject.c index 518396fa..fc9296cd 100644 --- a/tests/i915/gem_largeobject.c +++ b/tests/i915/gem_largeobject.c @@ -77,8 +77,6 @@ igt_simple_main { int fd; - igt_skip_on_simulation(); - data = malloc(OBJ_SIZE); igt_assert(data); diff --git a/tests/i915/gem_madvise.c b/tests/i915/gem_madvise.c index 729a4d33..7162a5c3 100644 --- a/tests/i915/gem_madvise.c +++ b/tests/i915/gem_madvise.c @@ -149,8 +149,6 @@ dontneed_before_exec(void) igt_main { - igt_skip_on_simulation(); - igt_subtest("dontneed-before-mmap") dontneed_before_mmap(); diff --git a/tests/i915/gem_mmap_offset_exhaustion.c b/tests/i915/gem_mmap_offset_exhaustion.c index 8c8e3fa2..f243507e 100644 --- a/tests/i915/gem_mmap_offset_exhaustion.c +++ b/tests/i915/gem_mmap_offset_exhaustion.c @@ -78,8 +78,6 @@ igt_simple_main { int fd, i; - igt_skip_on_simulation(); - fd = drm_open_driver(DRIVER_INTEL); /* we have 32bit of address space, so try to fit one MB more diff --git a/tests/i915/gem_partial_pwrite_pread.c b/tests/i915/gem_partial_pwrite_pread.c index 49e086ed..afd43694 100644 --- a/tests/i915/gem_partial_pwrite_pread.c +++ b/tests/i915/gem_partial_pwrite_pread.c @@ -248,8 +248,6 @@ igt_main { srandom(0xdeadbeef); - igt_skip_on_simulation(); - igt_fixture { fd = drm_open_driver(DRIVER_INTEL); igt_require_gem(fd); diff --git a/tests/i915/gem_persistent_relocs.c b/tests/i915/gem_persistent_relocs.c index 2ab7091a..e6226f41 100644 --- a/tests/i915/gem_persistent_relocs.c +++ b/tests/i915/gem_persistent_relocs.c @@ -319,8 +319,6 @@ int fd; #define MAX_BLT_SIZE 128 igt_main { - igt_skip_on_simulation(); - memset(blob, 'A', sizeof(blob)); igt_fixture { diff --git a/tests/i915/gem_pread_after_blit.c b/tests/i915/gem_pread_after_blit.c index cee92127..e12b5d65 100644 --- a/tests/i915/gem_pread_after_blit.c +++ b/tests/i915/gem_pread_after_blit.c @@ -208,8 +208,6 @@ igt_main { NULL, -1 }, }, *t; - igt_skip_on_simulation(); - igt_fixture { fd = drm_open_driver(DRIVER_INTEL); igt_require_gem(fd); diff --git a/tests/i915/gem_pwrite_pread.c b/tests/i915/gem_pwrite_pread.c index 422db34d..9287c677 100644 --- a/tests/i915/gem_pwrite_pread.c +++ b/tests/i915/gem_pwrite_pread.c @@ -274,8 +274,6 @@ igt_main_args("s:", NULL, help_str, opt_handler, NULL) uint32_t buf[20]; int count; - igt_skip_on_simulation(); - if (object_size == 0) object_size = OBJECT_SIZE; object_size = (object_size + 3) & -4; diff --git a/tests/i915/gem_pwrite_snooped.c b/tests/i915/gem_pwrite_snooped.c index 80f590de..14f4c44a 100644 --- a/tests/i915/gem_pwrite_snooped.c +++ b/tests/i915/gem_pwrite_snooped.c @@ -127,8 +127,6 @@ static void test(int w, int h) igt_simple_main { - igt_skip_on_simulation(); - fd = drm_open_driver(DRIVER_INTEL); igt_require_gem(fd); gem_require_blitter(fd); diff --git a/tests/i915/gem_read_read_speed.c b/tests/i915/gem_read_read_speed.c index 5f1356eb..8b5ba8f7 100644 --- a/tests/i915/gem_read_read_speed.c +++ b/tests/i915/gem_read_read_speed.c @@ -180,8 +180,6 @@ igt_main drm_intel_bufmgr *bufmgr = NULL; int fd, i; - igt_skip_on_simulation(); - igt_fixture { int devid; diff --git a/tests/i915/gem_readwrite.c b/tests/i915/gem_readwrite.c index fdf279f1..6b2977c1 100644 --- a/tests/i915/gem_readwrite.c +++ b/tests/i915/gem_readwrite.c @@ -81,8 +81,6 @@ igt_main uint8_t buf[OBJECT_SIZE]; int ret; - igt_skip_on_simulation(); - igt_fixture { fd = drm_open_driver(DRIVER_INTEL); diff --git a/tests/i915/gem_reloc_vs_gpu.c b/tests/i915/gem_reloc_vs_gpu.c index 46f14517..ebc71a68 100644 --- a/tests/i915/gem_reloc_vs_gpu.c +++ b/tests/i915/gem_reloc_vs_gpu.c @@ -297,8 +297,6 @@ int fd; #define MAX_BLT_SIZE 128 igt_main { - igt_skip_on_simulation(); - memset(blob, 'A', sizeof(blob)); igt_fixture { diff --git a/tests/i915/gem_render_linear_blits.c b/tests/i915/gem_render_linear_blits.c index 667ee872..a726652b 100644 --- a/tests/i915/gem_render_linear_blits.c +++ b/tests/i915/gem_render_linear_blits.c @@ -199,9 +199,6 @@ igt_main run_test(fd, 2); } - /* the rest of the tests are too long for simulation */ - igt_skip_on_simulation(); - igt_subtest("aperture-thrash") { count = 3 * gem_aperture_size(fd) / SIZE / 2; intel_require_memory(count, SIZE, CHECK_RAM); diff --git a/tests/i915/gem_render_tiled_blits.c b/tests/i915/gem_render_tiled_blits.c index 12950061..14018329 100644 --- a/tests/i915/gem_render_tiled_blits.c +++ b/tests/i915/gem_render_tiled_blits.c @@ -210,9 +210,6 @@ igt_main run_test(fd, 2); } - /* the rest of the tests are too long for simulation */ - igt_skip_on_simulation(); - igt_subtest("aperture-thrash") { count = 3 * gem_aperture_size(fd) / SIZE / 2; intel_require_memory(count, SIZE, CHECK_RAM); diff --git a/tests/i915/gem_reset_stats.c b/tests/i915/gem_reset_stats.c index ac9af23f..689c4afd 100644 --- a/tests/i915/gem_reset_stats.c +++ b/tests/i915/gem_reset_stats.c @@ -773,7 +773,6 @@ static bool gem_has_reset_stats(int fd) igt_main { const struct intel_execution_engine *e; - igt_skip_on_simulation(); igt_fixture { int fd; diff --git a/tests/i915/gem_ringfill.c b/tests/i915/gem_ringfill.c index c728e1cd..cf85f797 100644 --- a/tests/i915/gem_ringfill.c +++ b/tests/i915/gem_ringfill.c @@ -286,8 +286,6 @@ igt_main e->name, m->suffix) { igt_skip_on(m->flags & NEWFD && master); - if (m->flags & (HANG|SUSPEND|HIBERNATE)) - igt_skip_on_simulation(); run_test(fd, e->exec_id | e->flags, m->flags, m->timeout); } diff --git a/tests/i915/gem_set_tiling_vs_blt.c b/tests/i915/gem_set_tiling_vs_blt.c index 01230547..c317d1ef 100644 --- a/tests/i915/gem_set_tiling_vs_blt.c +++ b/tests/i915/gem_set_tiling_vs_blt.c @@ -225,8 +225,6 @@ igt_main int i; uint32_t tiling, tiling_after; - igt_skip_on_simulation(); - igt_fixture { for (i = 0; i < 1024*256; i++) data[i] = i; diff --git a/tests/i915/gem_set_tiling_vs_gtt.c b/tests/i915/gem_set_tiling_vs_gtt.c index 2611ec55..618c2d05 100644 --- a/tests/i915/gem_set_tiling_vs_gtt.c +++ b/tests/i915/gem_set_tiling_vs_gtt.c @@ -56,8 +56,6 @@ igt_simple_main bool tiling_changed; int tile_height; - igt_skip_on_simulation(); - fd = drm_open_driver(DRIVER_INTEL); if (IS_GEN2(intel_get_drm_devid(fd))) diff --git a/tests/i915/gem_set_tiling_vs_pwrite.c b/tests/i915/gem_set_tiling_vs_pwrite.c index f0126b64..25a28191 100644 --- a/tests/i915/gem_set_tiling_vs_pwrite.c +++ b/tests/i915/gem_set_tiling_vs_pwrite.c @@ -54,8 +54,6 @@ igt_simple_main int i; uint32_t handle; - igt_skip_on_simulation(); - fd = drm_open_driver(DRIVER_INTEL); for (i = 0; i < OBJECT_SIZE/4; i++) diff --git a/tests/i915/gem_shrink.c b/tests/i915/gem_shrink.c index 3db754f5..1f980ece 100644 --- a/tests/i915/gem_shrink.c +++ b/tests/i915/gem_shrink.c @@ -424,8 +424,6 @@ igt_main uint64_t alloc_size = 0; int num_processes = 0; - igt_skip_on_simulation(); - igt_fixture { uint64_t mem_size = intel_get_total_ram_mb(); unsigned int engine; diff --git a/tests/i915/gem_softpin.c b/tests/i915/gem_softpin.c index b9ff532e..e22631ab 100644 --- a/tests/i915/gem_softpin.c +++ b/tests/i915/gem_softpin.c @@ -480,8 +480,6 @@ igt_main { int fd = -1; - igt_skip_on_simulation(); - igt_fixture { fd = drm_open_driver_master(DRIVER_INTEL); igt_require_gem(fd); diff --git a/tests/i915/gem_spin_batch.c b/tests/i915/gem_spin_batch.c index e0623f4c..3222378d 100644 --- a/tests/i915/gem_spin_batch.c +++ b/tests/i915/gem_spin_batch.c @@ -143,8 +143,6 @@ igt_main struct intel_execution_engine2 e2__; int fd = -1; - igt_skip_on_simulation(); - igt_fixture { fd = drm_open_driver(DRIVER_INTEL); igt_require_gem(fd); diff --git a/tests/i915/gem_stolen.c b/tests/i915/gem_stolen.c index e2c52723..d6b30dcb 100644 --- a/tests/i915/gem_stolen.c +++ b/tests/i915/gem_stolen.c @@ -404,8 +404,6 @@ igt_main int fd; uint32_t devid; - igt_skip_on_simulation(); - igt_fixture { fd = drm_open_driver(DRIVER_INTEL); devid = intel_get_drm_devid(fd); diff --git a/tests/i915/gem_storedw_batches_loop.c b/tests/i915/gem_storedw_batches_loop.c index c0343ccb..65aba88a 100644 --- a/tests/i915/gem_storedw_batches_loop.c +++ b/tests/i915/gem_storedw_batches_loop.c @@ -130,8 +130,6 @@ int devid; igt_main { - igt_skip_on_simulation(); - igt_fixture { fd = drm_open_driver(DRIVER_INTEL); igt_require_gem(fd); diff --git a/tests/i915/gem_sync.c b/tests/i915/gem_sync.c index 8c5aaa14..fb28e716 100644 --- a/tests/i915/gem_sync.c +++ b/tests/i915/gem_sync.c @@ -1214,8 +1214,6 @@ igt_main const int ncpus = sysconf(_SC_NPROCESSORS_ONLN); int fd = -1; - igt_skip_on_simulation(); - igt_fixture { fd = drm_open_driver(DRIVER_INTEL); igt_require_gem(fd); diff --git a/tests/i915/gem_threaded_access_tiled.c b/tests/i915/gem_threaded_access_tiled.c index 288324d4..64a00026 100644 --- a/tests/i915/gem_threaded_access_tiled.c +++ b/tests/i915/gem_threaded_access_tiled.c @@ -95,8 +95,6 @@ igt_simple_main unsigned long pitch = 0; int r; - igt_skip_on_simulation(); - fd = drm_open_driver(DRIVER_INTEL); igt_assert(fd >= 0); diff --git a/tests/i915/gem_tiled_fence_blits.c b/tests/i915/gem_tiled_fence_blits.c index ef03c73b..93c79dc2 100644 --- a/tests/i915/gem_tiled_fence_blits.c +++ b/tests/i915/gem_tiled_fence_blits.c @@ -219,9 +219,6 @@ igt_main igt_subtest("basic") run_test (fd, 2); - /* the rest of the tests are too long for simulation */ - igt_skip_on_simulation(); - igt_subtest("normal") { uint64_t count; diff --git a/tests/i915/gem_tiled_partial_pwrite_pread.c b/tests/i915/gem_tiled_partial_pwrite_pread.c index 3d160b0f..9d15cdf1 100644 --- a/tests/i915/gem_tiled_partial_pwrite_pread.c +++ b/tests/i915/gem_tiled_partial_pwrite_pread.c @@ -262,8 +262,6 @@ igt_main { uint32_t tiling_mode = I915_TILING_X; - igt_skip_on_simulation(); - srandom(0xdeadbeef); igt_fixture { diff --git a/tests/i915/gem_unfence_active_buffers.c b/tests/i915/gem_unfence_active_buffers.c index a357ec32..c1670dd5 100644 --- a/tests/i915/gem_unfence_active_buffers.c +++ b/tests/i915/gem_unfence_active_buffers.c @@ -68,8 +68,6 @@ igt_simple_main drm_intel_bo *busy_bo, *test_bo; uint32_t tiling = I915_TILING_X; - igt_skip_on_simulation(); - for (i = 0; i < 1024*256; i++) data[i] = i; diff --git a/tests/i915/gem_unref_active_buffers.c b/tests/i915/gem_unref_active_buffers.c index b5d48206..2adf0f1a 100644 --- a/tests/i915/gem_unref_active_buffers.c +++ b/tests/i915/gem_unref_active_buffers.c @@ -54,8 +54,6 @@ igt_simple_main { int fd, i; - igt_skip_on_simulation(); - fd = drm_open_driver(DRIVER_INTEL); igt_require_gem(fd); gem_require_blitter(fd); diff --git a/tests/i915/gem_wait.c b/tests/i915/gem_wait.c index b5e32219..71f8bb5a 100644 --- a/tests/i915/gem_wait.c +++ b/tests/i915/gem_wait.c @@ -137,8 +137,6 @@ igt_main const struct intel_execution_engine2 *e; int fd = -1; - igt_skip_on_simulation(); - igt_fixture { fd = drm_open_driver_master(DRIVER_INTEL); igt_require_gem(fd); diff --git a/tests/i915/gem_write_read_ring_switch.c b/tests/i915/gem_write_read_ring_switch.c index ef229cc5..a9d0a8ed 100644 --- a/tests/i915/gem_write_read_ring_switch.c +++ b/tests/i915/gem_write_read_ring_switch.c @@ -149,8 +149,6 @@ igt_main }; int i; - igt_skip_on_simulation(); - igt_fixture { fd = drm_open_driver(DRIVER_INTEL); igt_require_gem(fd); -- 2.23.0 _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply related [flat|nested] 17+ messages in thread
* [igt-dev] [PATCH 2/7] tests/i915/kms: remove igt_skip_on_simulation() 2019-10-16 7:49 [igt-dev] [PATCH 0/7] removal of igt_skip_on_simulation() from i915/ tests Swati Sharma 2019-10-16 7:49 ` [igt-dev] [PATCH 1/7] tests/i915/gem: remove igt_skip_on_simulation() Swati Sharma @ 2019-10-16 7:49 ` Swati Sharma 2019-10-16 7:49 ` [igt-dev] [PATCH 3/7] tests/i915: " Swati Sharma ` (10 subsequent siblings) 12 siblings, 0 replies; 17+ messages in thread From: Swati Sharma @ 2019-10-16 7:49 UTC (permalink / raw) To: igt-dev; +Cc: lucas.demarchi, martin.peres, maarten.lankhorst Removing igt_skip_on_simulation() from all the kms tests since this feature is not supported anymore. Signed-off-by: Swati Sharma <swati2.sharma@intel.com> Signed-off-by: Karthik B S <karthik.b.s@intel.com> --- tests/kms_atomic_interruptible.c | 2 -- tests/kms_atomic_transition.c | 2 -- tests/kms_available_modes_crc.c | 2 -- tests/kms_big_fb.c | 2 -- tests/kms_busy.c | 2 -- tests/kms_chamelium.c | 2 -- tests/kms_color.c | 2 -- tests/kms_concurrent.c | 2 -- tests/kms_content_protection.c | 2 -- tests/kms_crtc_background_color.c | 2 -- tests/kms_cursor_crc.c | 2 -- tests/kms_cursor_edge_walk.c | 2 -- tests/kms_fence_pin_leak.c | 2 -- tests/kms_flip_event_leak.c | 2 -- tests/kms_flip_tiling.c | 2 -- tests/kms_invalid_dotclock.c | 2 -- tests/kms_legacy_colorkey.c | 2 -- tests/kms_mmap_write_crc.c | 2 -- tests/kms_panel_fitting.c | 2 -- tests/kms_pipe_b_c_ivb.c | 2 -- tests/kms_pipe_crc_basic.c | 2 -- tests/kms_plane.c | 2 -- tests/kms_plane_cursor.c | 2 -- tests/kms_plane_lowres.c | 2 -- tests/kms_plane_multiple.c | 2 -- tests/kms_plane_scaling.c | 2 -- tests/kms_properties.c | 2 -- tests/kms_psr.c | 2 -- tests/kms_psr2_su.c | 2 -- tests/kms_pwrite_crc.c | 2 -- tests/kms_rmfb.c | 2 -- tests/kms_rotation_crc.c | 2 -- tests/kms_sequence.c | 2 -- tests/kms_setmode.c | 2 -- tests/kms_universal_plane.c | 2 -- tests/kms_vblank.c | 2 -- tests/kms_vrr.c | 2 -- 37 files changed, 74 deletions(-) diff --git a/tests/kms_atomic_interruptible.c b/tests/kms_atomic_interruptible.c index be688638..e5ccdf5b 100644 --- a/tests/kms_atomic_interruptible.c +++ b/tests/kms_atomic_interruptible.c @@ -272,8 +272,6 @@ igt_main igt_output_t *output; enum pipe pipe; - igt_skip_on_simulation(); - igt_fixture { display.drm_fd = drm_open_driver_master(DRIVER_ANY); diff --git a/tests/kms_atomic_transition.c b/tests/kms_atomic_transition.c index 1540daa2..44f01109 100644 --- a/tests/kms_atomic_transition.c +++ b/tests/kms_atomic_transition.c @@ -909,8 +909,6 @@ igt_main enum pipe pipe; int i; - igt_skip_on_simulation(); - igt_fixture { display.drm_fd = drm_open_driver_master(DRIVER_ANY); diff --git a/tests/kms_available_modes_crc.c b/tests/kms_available_modes_crc.c index 786d50e6..d1b7b517 100644 --- a/tests/kms_available_modes_crc.c +++ b/tests/kms_available_modes_crc.c @@ -438,8 +438,6 @@ igt_main { data_t data = {}; - igt_skip_on_simulation(); - igt_fixture { data.gfx_fd = drm_open_driver_master(DRIVER_INTEL); kmstest_set_vt_graphics_mode(); diff --git a/tests/kms_big_fb.c b/tests/kms_big_fb.c index c3498c67..c5d002ca 100644 --- a/tests/kms_big_fb.c +++ b/tests/kms_big_fb.c @@ -605,8 +605,6 @@ igt_main igt_fixture { drmModeResPtr res; - igt_skip_on_simulation(); - data.drm_fd = drm_open_driver_master(DRIVER_INTEL); igt_require(is_i915_device(data.drm_fd)); diff --git a/tests/kms_busy.c b/tests/kms_busy.c index 5ff2b846..e558055b 100644 --- a/tests/kms_busy.c +++ b/tests/kms_busy.c @@ -290,8 +290,6 @@ igt_main const struct intel_execution_engine *e = &intel_execution_engines[1]; enum pipe n; - igt_skip_on_simulation(); - igt_fixture { int fd = drm_open_driver_master(DRIVER_INTEL); diff --git a/tests/kms_chamelium.c b/tests/kms_chamelium.c index 52330098..b35ccf0e 100644 --- a/tests/kms_chamelium.c +++ b/tests/kms_chamelium.c @@ -2615,8 +2615,6 @@ igt_main size_t i; igt_fixture { - igt_skip_on_simulation(); - data.drm_fd = drm_open_driver_master(DRIVER_ANY); data.chamelium = chamelium_init(data.drm_fd); igt_require(data.chamelium); diff --git a/tests/kms_color.c b/tests/kms_color.c index eb85df7d..b4b578a7 100644 --- a/tests/kms_color.c +++ b/tests/kms_color.c @@ -1212,8 +1212,6 @@ igt_main data_t data = {}; enum pipe pipe; - igt_skip_on_simulation(); - igt_fixture { data.drm_fd = drm_open_driver_master(DRIVER_INTEL | DRIVER_AMDGPU); if (is_i915_device(data.drm_fd)) diff --git a/tests/kms_concurrent.c b/tests/kms_concurrent.c index 23b05ea1..a0280d15 100644 --- a/tests/kms_concurrent.c +++ b/tests/kms_concurrent.c @@ -393,8 +393,6 @@ igt_main_args("", long_options, help_str, opt_handler, NULL) { enum pipe pipe; - igt_skip_on_simulation(); - igt_fixture { data.drm_fd = drm_open_driver_master(DRIVER_ANY); kmstest_set_vt_graphics_mode(); diff --git a/tests/kms_content_protection.c b/tests/kms_content_protection.c index fce4b591..82c146d8 100644 --- a/tests/kms_content_protection.c +++ b/tests/kms_content_protection.c @@ -640,8 +640,6 @@ static void test_content_protection_cleanup(void) igt_main { igt_fixture { - igt_skip_on_simulation(); - data.drm_fd = drm_open_driver(DRIVER_ANY); igt_display_require(&data.display, data.drm_fd); diff --git a/tests/kms_crtc_background_color.c b/tests/kms_crtc_background_color.c index 3df3401f..aca7ac3e 100644 --- a/tests/kms_crtc_background_color.c +++ b/tests/kms_crtc_background_color.c @@ -176,8 +176,6 @@ igt_simple_main { data_t data = {}; - igt_skip_on_simulation(); - data.gfx_fd = drm_open_driver(DRIVER_INTEL); igt_require_pipe_crc(data.gfx_fd); igt_display_require(&data.display, data.gfx_fd); diff --git a/tests/kms_cursor_crc.c b/tests/kms_cursor_crc.c index d0fb8f1d..245ff7b2 100644 --- a/tests/kms_cursor_crc.c +++ b/tests/kms_cursor_crc.c @@ -710,8 +710,6 @@ igt_main int ret; enum pipe pipe; - igt_skip_on_simulation(); - igt_fixture { data.drm_fd = drm_open_driver_master(DRIVER_ANY); diff --git a/tests/kms_cursor_edge_walk.c b/tests/kms_cursor_edge_walk.c index 87d17e06..6feb32a8 100644 --- a/tests/kms_cursor_edge_walk.c +++ b/tests/kms_cursor_edge_walk.c @@ -304,8 +304,6 @@ static const char *help_str = igt_main_args("", long_opts, help_str, opt_handler, &data) { - igt_skip_on_simulation(); - igt_fixture { int ret; diff --git a/tests/kms_fence_pin_leak.c b/tests/kms_fence_pin_leak.c index e6c8b33c..8c9e10a5 100644 --- a/tests/kms_fence_pin_leak.c +++ b/tests/kms_fence_pin_leak.c @@ -198,8 +198,6 @@ igt_simple_main drm_intel_context *ctx; data_t data = {}; - igt_skip_on_simulation(); - data.drm_fd = drm_open_driver_master(DRIVER_INTEL); igt_require_gem(data.drm_fd); diff --git a/tests/kms_flip_event_leak.c b/tests/kms_flip_event_leak.c index f29d41fa..06b62d17 100644 --- a/tests/kms_flip_event_leak.c +++ b/tests/kms_flip_event_leak.c @@ -96,8 +96,6 @@ igt_simple_main int valid_tests = 0; enum pipe pipe; - igt_skip_on_simulation(); - data.drm_fd = drm_open_driver_master(DRIVER_ANY); kmstest_set_vt_graphics_mode(); diff --git a/tests/kms_flip_tiling.c b/tests/kms_flip_tiling.c index 7c8accc3..17cf816d 100644 --- a/tests/kms_flip_tiling.c +++ b/tests/kms_flip_tiling.c @@ -151,8 +151,6 @@ igt_output_t *output; igt_main { - igt_skip_on_simulation(); - igt_fixture { data.drm_fd = drm_open_driver_master(DRIVER_INTEL); data.gen = intel_gen(intel_get_drm_devid(data.drm_fd)); diff --git a/tests/kms_invalid_dotclock.c b/tests/kms_invalid_dotclock.c index 275007df..402629ab 100644 --- a/tests/kms_invalid_dotclock.c +++ b/tests/kms_invalid_dotclock.c @@ -126,8 +126,6 @@ static data_t data; igt_simple_main { - igt_skip_on_simulation(); - data.drm_fd = drm_open_driver_master(DRIVER_INTEL); igt_require_intel(data.drm_fd); diff --git a/tests/kms_legacy_colorkey.c b/tests/kms_legacy_colorkey.c index 961aa0a3..de9610a7 100644 --- a/tests/kms_legacy_colorkey.c +++ b/tests/kms_legacy_colorkey.c @@ -45,8 +45,6 @@ static void test_plane(uint32_t plane_id, int expected_ret) igt_simple_main { - igt_skip_on_simulation(); - drm_fd = drm_open_driver_master(DRIVER_INTEL); kmstest_set_vt_graphics_mode(); diff --git a/tests/kms_mmap_write_crc.c b/tests/kms_mmap_write_crc.c index 73606b16..56e0c4d7 100644 --- a/tests/kms_mmap_write_crc.c +++ b/tests/kms_mmap_write_crc.c @@ -271,8 +271,6 @@ igt_main_args("n", NULL, NULL, opt_handler, NULL) { int i; - igt_skip_on_simulation(); - igt_fixture { data.drm_fd = drm_open_driver_master(DRIVER_INTEL); diff --git a/tests/kms_panel_fitting.c b/tests/kms_panel_fitting.c index a942294b..ac24730f 100644 --- a/tests/kms_panel_fitting.c +++ b/tests/kms_panel_fitting.c @@ -252,8 +252,6 @@ igt_main data_t data = {}; igt_fixture { - igt_skip_on_simulation(); - data.drm_fd = drm_open_driver(DRIVER_ANY); igt_display_require(&data.display, data.drm_fd); igt_display_require_output(&data.display); diff --git a/tests/kms_pipe_b_c_ivb.c b/tests/kms_pipe_b_c_ivb.c index fe2250ce..94768eec 100644 --- a/tests/kms_pipe_b_c_ivb.c +++ b/tests/kms_pipe_b_c_ivb.c @@ -255,8 +255,6 @@ igt_main { int devid; - igt_skip_on_simulation(); - igt_fixture { data.drm_fd = drm_open_driver_master(DRIVER_INTEL); devid = intel_get_drm_devid(data.drm_fd); diff --git a/tests/kms_pipe_crc_basic.c b/tests/kms_pipe_crc_basic.c index 36ce624e..d169b7bd 100644 --- a/tests/kms_pipe_crc_basic.c +++ b/tests/kms_pipe_crc_basic.c @@ -173,8 +173,6 @@ igt_main igt_subtest("bad-source") test_bad_source(&data); - igt_skip_on_simulation(); - for_each_pipe_static(pipe) { igt_subtest_f("read-crc-pipe-%s", kmstest_pipe_name(pipe)) test_read_crc(&data, pipe, 0); diff --git a/tests/kms_plane.c b/tests/kms_plane.c index 5f1a73f8..acd2a3b9 100644 --- a/tests/kms_plane.c +++ b/tests/kms_plane.c @@ -831,8 +831,6 @@ igt_main { enum pipe pipe; - igt_skip_on_simulation(); - igt_fixture { data.drm_fd = drm_open_driver_master(DRIVER_ANY); diff --git a/tests/kms_plane_cursor.c b/tests/kms_plane_cursor.c index cbad0041..adcdf5e8 100644 --- a/tests/kms_plane_cursor.c +++ b/tests/kms_plane_cursor.c @@ -306,8 +306,6 @@ igt_main enum pipe pipe; int i; - igt_skip_on_simulation(); - igt_fixture { data.drm_fd = drm_open_driver_master(DRIVER_ANY); diff --git a/tests/kms_plane_lowres.c b/tests/kms_plane_lowres.c index 1a316c84..c6c3f498 100644 --- a/tests/kms_plane_lowres.c +++ b/tests/kms_plane_lowres.c @@ -233,8 +233,6 @@ igt_main data_t data = {}; enum pipe pipe; - igt_skip_on_simulation(); - igt_fixture { data.drm_fd = drm_open_driver_master(DRIVER_ANY); diff --git a/tests/kms_plane_multiple.c b/tests/kms_plane_multiple.c index 255cd34a..899b6c5e 100644 --- a/tests/kms_plane_multiple.c +++ b/tests/kms_plane_multiple.c @@ -433,8 +433,6 @@ igt_main_args("", long_options, help_str, opt_handler, NULL) { enum pipe pipe; - igt_skip_on_simulation(); - igt_fixture { data.drm_fd = drm_open_driver_master(DRIVER_INTEL | DRIVER_AMDGPU); kmstest_set_vt_graphics_mode(); diff --git a/tests/kms_plane_scaling.c b/tests/kms_plane_scaling.c index 0bd36904..35aac8da 100644 --- a/tests/kms_plane_scaling.c +++ b/tests/kms_plane_scaling.c @@ -624,8 +624,6 @@ igt_main data_t data = {}; enum pipe pipe; - igt_skip_on_simulation(); - igt_fixture { data.drm_fd = drm_open_driver_master(DRIVER_INTEL | DRIVER_AMDGPU); igt_display_require(&data.display, data.drm_fd); diff --git a/tests/kms_properties.c b/tests/kms_properties.c index 18cf96e0..b743f9d6 100644 --- a/tests/kms_properties.c +++ b/tests/kms_properties.c @@ -727,8 +727,6 @@ igt_main { igt_display_t display; - igt_skip_on_simulation(); - igt_fixture { display.drm_fd = drm_open_driver_master(DRIVER_ANY); diff --git a/tests/kms_psr.c b/tests/kms_psr.c index 39de0112..232c80aa 100644 --- a/tests/kms_psr.c +++ b/tests/kms_psr.c @@ -433,8 +433,6 @@ igt_main_args("", long_options, help_str, opt_handler, &data) "psr2_" }; - igt_skip_on_simulation(); - igt_fixture { data.drm_fd = drm_open_driver_master(DRIVER_INTEL); data.debugfs_fd = igt_debugfs_dir(data.drm_fd); diff --git a/tests/kms_psr2_su.c b/tests/kms_psr2_su.c index a9f675d1..f2244b29 100644 --- a/tests/kms_psr2_su.c +++ b/tests/kms_psr2_su.c @@ -232,8 +232,6 @@ igt_main { data_t data = {}; - igt_skip_on_simulation(); - igt_fixture { struct itimerspec interval; int r; diff --git a/tests/kms_pwrite_crc.c b/tests/kms_pwrite_crc.c index 9bb5a5da..d6cc0c47 100644 --- a/tests/kms_pwrite_crc.c +++ b/tests/kms_pwrite_crc.c @@ -174,8 +174,6 @@ static data_t data; igt_simple_main { - igt_skip_on_simulation(); - data.drm_fd = drm_open_driver_master(DRIVER_INTEL); data.devid = intel_get_drm_devid(data.drm_fd); diff --git a/tests/kms_rmfb.c b/tests/kms_rmfb.c index a7fcab87..81932e0d 100644 --- a/tests/kms_rmfb.c +++ b/tests/kms_rmfb.c @@ -157,8 +157,6 @@ igt_main { struct rmfb_data data = {}; - igt_skip_on_simulation(); - igt_fixture { data.drm_fd = drm_open_driver_master(DRIVER_ANY); diff --git a/tests/kms_rotation_crc.c b/tests/kms_rotation_crc.c index 19852675..2a7b10e9 100644 --- a/tests/kms_rotation_crc.c +++ b/tests/kms_rotation_crc.c @@ -774,8 +774,6 @@ igt_main data_t data = {}; int gen = 0; - igt_skip_on_simulation(); - igt_fixture { data.gfx_fd = drm_open_driver_master(DRIVER_INTEL); data.devid = intel_get_drm_devid(data.gfx_fd); diff --git a/tests/kms_sequence.c b/tests/kms_sequence.c index 6383b0ca..ff2d73cd 100644 --- a/tests/kms_sequence.c +++ b/tests/kms_sequence.c @@ -295,8 +295,6 @@ igt_main { } }, *m; - igt_skip_on_simulation(); - igt_fixture { fd = drm_open_driver_master(DRIVER_ANY); kmstest_set_vt_graphics_mode(); diff --git a/tests/kms_setmode.c b/tests/kms_setmode.c index 8ace587e..802a9c3d 100644 --- a/tests/kms_setmode.c +++ b/tests/kms_setmode.c @@ -854,8 +854,6 @@ igt_main_args("dt:", NULL, help_str, opt_handler, NULL) }; int i; - igt_skip_on_simulation(); - igt_assert_f(!(dry_run && filter_test_id), "only one of -d and -t is accepted\n"); diff --git a/tests/kms_universal_plane.c b/tests/kms_universal_plane.c index 30c03bbb..676be633 100644 --- a/tests/kms_universal_plane.c +++ b/tests/kms_universal_plane.c @@ -790,8 +790,6 @@ igt_main { enum pipe pipe; - igt_skip_on_simulation(); - igt_fixture { data.drm_fd = drm_open_driver_master(DRIVER_INTEL); data.gen = intel_gen(intel_get_drm_devid(data.drm_fd)); diff --git a/tests/kms_vblank.c b/tests/kms_vblank.c index 79ff97b2..73694433 100644 --- a/tests/kms_vblank.c +++ b/tests/kms_vblank.c @@ -515,8 +515,6 @@ igt_main int fd; data_t data; - igt_skip_on_simulation(); - igt_fixture { fd = drm_open_driver_master(DRIVER_ANY); kmstest_set_vt_graphics_mode(); diff --git a/tests/kms_vrr.c b/tests/kms_vrr.c index 9801b40a..7bc4fc8a 100644 --- a/tests/kms_vrr.c +++ b/tests/kms_vrr.c @@ -392,8 +392,6 @@ igt_main { data_t data = { 0 }; - igt_skip_on_simulation(); - igt_fixture { data.drm_fd = drm_open_driver_master(DRIVER_ANY); -- 2.23.0 _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply related [flat|nested] 17+ messages in thread
* [igt-dev] [PATCH 3/7] tests/i915: remove igt_skip_on_simulation() 2019-10-16 7:49 [igt-dev] [PATCH 0/7] removal of igt_skip_on_simulation() from i915/ tests Swati Sharma 2019-10-16 7:49 ` [igt-dev] [PATCH 1/7] tests/i915/gem: remove igt_skip_on_simulation() Swati Sharma 2019-10-16 7:49 ` [igt-dev] [PATCH 2/7] tests/i915/kms: " Swati Sharma @ 2019-10-16 7:49 ` Swati Sharma 2019-10-16 7:49 ` [igt-dev] [PATCH 4/7] tests: " Swati Sharma ` (9 subsequent siblings) 12 siblings, 0 replies; 17+ messages in thread From: Swati Sharma @ 2019-10-16 7:49 UTC (permalink / raw) To: igt-dev; +Cc: lucas.demarchi, martin.peres, maarten.lankhorst Removing igt_skip_on_simulation() from all i915 specific tests since this feature is not supported anymore. Signed-off-by: Swati Sharma <swati2.sharma@intel.com> Signed-off-by: Karthik B S <karthik.b.s@intel.com> --- tests/i915/i915_hangman.c | 2 -- tests/i915/i915_pm_backlight.c | 2 -- tests/i915/i915_pm_dc.c | 1 - tests/i915/i915_pm_rc6_residency.c | 2 -- tests/i915/i915_pm_rps.c | 2 -- tests/i915/i915_pm_sseu.c | 3 --- tests/i915/i915_suspend.c | 2 -- 7 files changed, 14 deletions(-) diff --git a/tests/i915/i915_hangman.c b/tests/i915/i915_hangman.c index 58141fc9..cdf62ac7 100644 --- a/tests/i915/i915_hangman.c +++ b/tests/i915/i915_hangman.c @@ -259,8 +259,6 @@ igt_main const struct intel_execution_engine2 *e; igt_hang_t hang = {}; - igt_skip_on_simulation(); - igt_fixture { device = drm_open_driver(DRIVER_INTEL); igt_require_gem(device); diff --git a/tests/i915/i915_pm_backlight.c b/tests/i915/i915_pm_backlight.c index 9a5f4c37..83b1ed68 100644 --- a/tests/i915/i915_pm_backlight.c +++ b/tests/i915/i915_pm_backlight.c @@ -187,8 +187,6 @@ igt_main igt_output_t *output; struct igt_fb fb; - igt_skip_on_simulation(); - igt_fixture { enum pipe pipe; bool found = false; diff --git a/tests/i915/i915_pm_dc.c b/tests/i915/i915_pm_dc.c index ce3319b7..d470d391 100644 --- a/tests/i915/i915_pm_dc.c +++ b/tests/i915/i915_pm_dc.c @@ -242,7 +242,6 @@ int main(int argc, char *argv[]) bool has_runtime_pm; data_t data = {}; - igt_skip_on_simulation(); igt_subtest_init(argc, argv); igt_fixture { data.drm_fd = drm_open_driver_master(DRIVER_INTEL); diff --git a/tests/i915/i915_pm_rc6_residency.c b/tests/i915/i915_pm_rc6_residency.c index 1b52e4f5..1b39c870 100644 --- a/tests/i915/i915_pm_rc6_residency.c +++ b/tests/i915/i915_pm_rc6_residency.c @@ -200,8 +200,6 @@ igt_main unsigned int rc6_enabled = 0; unsigned int devid = 0; - igt_skip_on_simulation(); - /* Use drm_open_driver to verify device existence */ igt_fixture { int fd; diff --git a/tests/i915/i915_pm_rps.c b/tests/i915/i915_pm_rps.c index 478c7be7..3b78d326 100644 --- a/tests/i915/i915_pm_rps.c +++ b/tests/i915/i915_pm_rps.c @@ -626,8 +626,6 @@ static void pm_rps_exit_handler(int sig) igt_main { - igt_skip_on_simulation(); - igt_fixture { struct sysfs_file *sysfs_file = sysfs_files; char sysfs_path[80]; diff --git a/tests/i915/i915_pm_sseu.c b/tests/i915/i915_pm_sseu.c index 0b936982..e671e190 100644 --- a/tests/i915/i915_pm_sseu.c +++ b/tests/i915/i915_pm_sseu.c @@ -352,9 +352,6 @@ full_enable(void) const int spin_msec = 10; int ret, spins; - /* Simulation doesn't currently model slice/subslice/EU power gating. */ - igt_skip_on_simulation(); - /* * Gen9 SKL is the first case in which render power gating can leave * slice/subslice/EU in a partially enabled state upon resumption of diff --git a/tests/i915/i915_suspend.c b/tests/i915/i915_suspend.c index 17c68cc1..5a2ec766 100644 --- a/tests/i915/i915_suspend.c +++ b/tests/i915/i915_suspend.c @@ -205,8 +205,6 @@ int fd; igt_main { - igt_skip_on_simulation(); - igt_fixture fd = drm_open_driver(DRIVER_INTEL); -- 2.23.0 _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply related [flat|nested] 17+ messages in thread
* [igt-dev] [PATCH 4/7] tests: remove igt_skip_on_simulation() 2019-10-16 7:49 [igt-dev] [PATCH 0/7] removal of igt_skip_on_simulation() from i915/ tests Swati Sharma ` (2 preceding siblings ...) 2019-10-16 7:49 ` [igt-dev] [PATCH 3/7] tests/i915: " Swati Sharma @ 2019-10-16 7:49 ` Swati Sharma 2019-10-16 7:49 ` [igt-dev] [PATCH 5/7] tools: " Swati Sharma ` (8 subsequent siblings) 12 siblings, 0 replies; 17+ messages in thread From: Swati Sharma @ 2019-10-16 7:49 UTC (permalink / raw) To: igt-dev; +Cc: lucas.demarchi, martin.peres, maarten.lankhorst Removing igt_skip_on_simulation() from tests since this feature is not supported anymore. Signed-off-by: Swati Sharma <swati2.sharma@intel.com> Signed-off-by: Karthik B S <karthik.b.s@intel.com> --- tests/debugfs_test.c | 2 -- tests/perf.c | 2 -- tests/prime_busy.c | 1 - tests/prime_mmap_kms.c | 2 -- tests/prime_udl.c | 2 -- tests/testdisplay.c | 1 - tests/tools_test.c | 2 -- 7 files changed, 12 deletions(-) diff --git a/tests/debugfs_test.c b/tests/debugfs_test.c index f8dd851c..0d4162b3 100644 --- a/tests/debugfs_test.c +++ b/tests/debugfs_test.c @@ -151,8 +151,6 @@ igt_main { int fd = -1, debugfs; - igt_skip_on_simulation(); - igt_fixture { fd = drm_open_driver_master(DRIVER_INTEL); igt_require_gem(fd); diff --git a/tests/perf.c b/tests/perf.c index 5ad8b2db..e4eb8378 100644 --- a/tests/perf.c +++ b/tests/perf.c @@ -4066,8 +4066,6 @@ test_sysctl_defaults(void) igt_main { - igt_skip_on_simulation(); - igt_fixture { struct stat sb; diff --git a/tests/prime_busy.c b/tests/prime_busy.c index 4f237f68..c6b85270 100644 --- a/tests/prime_busy.c +++ b/tests/prime_busy.c @@ -201,7 +201,6 @@ static void test_engine_mode(int fd, } else { - igt_skip_on_simulation(); hang = igt_allow_hang(fd, 0, 0); } } diff --git a/tests/prime_mmap_kms.c b/tests/prime_mmap_kms.c index 1c963e01..29aec499 100644 --- a/tests/prime_mmap_kms.c +++ b/tests/prime_mmap_kms.c @@ -239,8 +239,6 @@ igt_main { gpu_process_t gpu; - igt_skip_on_simulation(); - igt_fixture { gpu.drm_fd = drm_open_driver_master(DRIVER_INTEL); igt_skip_on((check_for_dma_buf_mmap(gpu.drm_fd) != 0)); diff --git a/tests/prime_udl.c b/tests/prime_udl.c index 59f478f7..9b600659 100644 --- a/tests/prime_udl.c +++ b/tests/prime_udl.c @@ -147,8 +147,6 @@ out: igt_simple_main { - igt_skip_on_simulation(); - igt_assert(find_and_open_devices() >= 0); igt_skip_on(udl_fd == -1); diff --git a/tests/testdisplay.c b/tests/testdisplay.c index 2fdc0236..858af7ca 100644 --- a/tests/testdisplay.c +++ b/tests/testdisplay.c @@ -696,7 +696,6 @@ igt_simple_main_args(optstr, long_opts, help_str, opt_handler, NULL) int ret = 0; GIOChannel *stdinchannel; GMainLoop *mainloop; - igt_skip_on_simulation(); enter_exec_path(); diff --git a/tests/tools_test.c b/tests/tools_test.c index 88a6ae96..f253a730 100644 --- a/tests/tools_test.c +++ b/tests/tools_test.c @@ -77,8 +77,6 @@ static bool chdir_to_tools_dir(void) igt_main { - igt_skip_on_simulation(); - igt_fixture { char *path; -- 2.23.0 _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply related [flat|nested] 17+ messages in thread
* [igt-dev] [PATCH 5/7] tools: remove igt_skip_on_simulation() 2019-10-16 7:49 [igt-dev] [PATCH 0/7] removal of igt_skip_on_simulation() from i915/ tests Swati Sharma ` (3 preceding siblings ...) 2019-10-16 7:49 ` [igt-dev] [PATCH 4/7] tests: " Swati Sharma @ 2019-10-16 7:49 ` Swati Sharma 2019-10-16 7:49 ` [igt-dev] [PATCH 6/7] benchmarks: " Swati Sharma ` (7 subsequent siblings) 12 siblings, 0 replies; 17+ messages in thread From: Swati Sharma @ 2019-10-16 7:49 UTC (permalink / raw) To: igt-dev; +Cc: lucas.demarchi, martin.peres, maarten.lankhorst Removing igt_skip_on_simulation() from tools since this feature is not supported anymore. Signed-off-by: Swati Sharma <swati2.sharma@intel.com> --- tools/intel_dp_compliance.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/tools/intel_dp_compliance.c b/tools/intel_dp_compliance.c index 14631a46..5cc3d37a 100644 --- a/tools/intel_dp_compliance.c +++ b/tools/intel_dp_compliance.c @@ -1017,8 +1017,6 @@ int main(int argc, char **argv) {"help", 0, 0, 'h'}, }; - igt_skip_on_simulation(); - enter_exec_path(argv); while ((c = getopt_long(argc, argv, optstr, long_opts, NULL)) != -1) { -- 2.23.0 _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply related [flat|nested] 17+ messages in thread
* [igt-dev] [PATCH 6/7] benchmarks: remove igt_skip_on_simulation() 2019-10-16 7:49 [igt-dev] [PATCH 0/7] removal of igt_skip_on_simulation() from i915/ tests Swati Sharma ` (4 preceding siblings ...) 2019-10-16 7:49 ` [igt-dev] [PATCH 5/7] tools: " Swati Sharma @ 2019-10-16 7:49 ` Swati Sharma 2019-10-16 7:49 ` [igt-dev] [PATCH 7/7] lib: remove igt_skip_on_simulation Swati Sharma ` (6 subsequent siblings) 12 siblings, 0 replies; 17+ messages in thread From: Swati Sharma @ 2019-10-16 7:49 UTC (permalink / raw) To: igt-dev; +Cc: lucas.demarchi, martin.peres, maarten.lankhorst Removing igt_skip_on_simulation() from benchmarks since this feature is not supported anymore. Signed-off-by: Swati Sharma <swati2.sharma@intel.com> Signed-off-by: Karthik B S <karthik.b.s@intel.com> --- benchmarks/gem_userptr_benchmark.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/benchmarks/gem_userptr_benchmark.c b/benchmarks/gem_userptr_benchmark.c index bef2ccc4..b63fdedf 100644 --- a/benchmarks/gem_userptr_benchmark.c +++ b/benchmarks/gem_userptr_benchmark.c @@ -469,8 +469,6 @@ int main(int argc, char **argv) { int fd = -1, ret; - igt_skip_on_simulation(); - igt_subtest_init(argc, argv); fd = drm_open_driver(DRIVER_INTEL); -- 2.23.0 _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply related [flat|nested] 17+ messages in thread
* [igt-dev] [PATCH 7/7] lib: remove igt_skip_on_simulation 2019-10-16 7:49 [igt-dev] [PATCH 0/7] removal of igt_skip_on_simulation() from i915/ tests Swati Sharma ` (5 preceding siblings ...) 2019-10-16 7:49 ` [igt-dev] [PATCH 6/7] benchmarks: " Swati Sharma @ 2019-10-16 7:49 ` Swati Sharma 2019-10-16 8:49 ` [igt-dev] [PATCH 0/7] removal of igt_skip_on_simulation() from i915/ tests Martin Peres ` (5 subsequent siblings) 12 siblings, 0 replies; 17+ messages in thread From: Swati Sharma @ 2019-10-16 7:49 UTC (permalink / raw) To: igt-dev; +Cc: lucas.demarchi, martin.peres, maarten.lankhorst Removing igt_skip_on_simulation() from lib since this feature is not supported anymore. Func() defination hasn't been removed since it can be used by some other drivers (currently amd is using this function). Signed-off-by: Swati Sharma <swati2.sharma@intel.com> Signed-off-by: Karthik B S <karthik.b.s@intel.com> --- lib/igt_aux.c | 7 ------- lib/intel_os.c | 2 -- 2 files changed, 9 deletions(-) diff --git a/lib/igt_aux.c b/lib/igt_aux.c index 578f8579..55632fd0 100644 --- a/lib/igt_aux.c +++ b/lib/igt_aux.c @@ -854,11 +854,6 @@ void igt_system_suspend_autoresume(enum igt_suspend_state state, int power_dir; enum igt_suspend_test orig_test; - /* FIXME: Simulation doesn't like suspend/resume, and not even a lighter - * approach using /sys/power/pm_test to just test our driver's callbacks - * seems to fare better. We need to investigate what's going on. */ - igt_skip_on_simulation(); - igt_require((power_dir = open("/sys/power", O_RDONLY)) >= 0); igt_require(get_supported_suspend_states(power_dir) & (1 << state)); igt_require(test == SUSPEND_TEST_NONE || @@ -905,8 +900,6 @@ void igt_set_autoresume_delay(int delay_secs) int delay_fd; char delay_str[10]; - igt_skip_on_simulation(); - delay_fd = open("/sys/module/suspend/parameters/pm_test_delay", O_RDWR); if (delay_fd >= 0) { diff --git a/lib/intel_os.c b/lib/intel_os.c index 505831c0..8458d39a 100644 --- a/lib/intel_os.c +++ b/lib/intel_os.c @@ -365,8 +365,6 @@ void intel_require_memory(uint64_t count, uint64_t size, unsigned mode) uint64_t required, total; bool sufficient_memory; - igt_skip_on_simulation(); - sufficient_memory = __intel_check_memory(count, size, mode, &required, &total); if (!sufficient_memory) { -- 2.23.0 _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply related [flat|nested] 17+ messages in thread
* Re: [igt-dev] [PATCH 0/7] removal of igt_skip_on_simulation() from i915/ tests 2019-10-16 7:49 [igt-dev] [PATCH 0/7] removal of igt_skip_on_simulation() from i915/ tests Swati Sharma ` (6 preceding siblings ...) 2019-10-16 7:49 ` [igt-dev] [PATCH 7/7] lib: remove igt_skip_on_simulation Swati Sharma @ 2019-10-16 8:49 ` Martin Peres 2019-10-16 9:55 ` [igt-dev] ✗ Fi.CI.BAT: failure for " Patchwork ` (4 subsequent siblings) 12 siblings, 0 replies; 17+ messages in thread From: Martin Peres @ 2019-10-16 8:49 UTC (permalink / raw) To: Swati Sharma, igt-dev; +Cc: lucas.demarchi, martin.peres, maarten.lankhorst On 16/10/2019 10:49, Swati Sharma wrote: > Removing igt_skip_on_simulation() since this feature is not supported > anymore by i915 driver. Removing it from gem, kms, i915 specific, tools > and benchmark tests. Thanks for doing this! The series is: Reviewed-by: Martin Peres <martin.peres@linux.intel.com> > > Swati Sharma (7): > tests/i915/gem: remove igt_skip_on_simulation() > tests/i915/kms: remove igt_skip_on_simulation() > tests/i915: remove igt_skip_on_simulation() > tests: remove igt_skip_on_simulation() > tools: remove igt_skip_on_simulation() > benchmarks: remove igt_skip_on_simulation() > lib: remove igt_skip_on_simulation > > benchmarks/gem_userptr_benchmark.c | 2 -- > lib/igt_aux.c | 7 ------- > lib/intel_os.c | 2 -- > tests/debugfs_test.c | 2 -- > tests/i915/gem_busy.c | 2 -- > tests/i915/gem_caching.c | 2 -- > tests/i915/gem_concurrent_all.c | 2 -- > tests/i915/gem_create.c | 2 -- > tests/i915/gem_cs_prefetch.c | 2 -- > tests/i915/gem_cs_tlb.c | 2 -- > tests/i915/gem_ctx_exec.c | 2 -- > tests/i915/gem_ctx_switch.c | 6 ------ > tests/i915/gem_ctx_thrash.c | 2 -- > tests/i915/gem_double_irq_loop.c | 2 -- > tests/i915/gem_eio.c | 2 -- > tests/i915/gem_exec_async.c | 2 -- > tests/i915/gem_exec_balancer.c | 2 -- > tests/i915/gem_exec_blt.c | 2 -- > tests/i915/gem_exec_capture.c | 2 -- > tests/i915/gem_exec_fence.c | 2 -- > tests/i915/gem_exec_gttfill.c | 2 -- > tests/i915/gem_exec_lut_handle.c | 2 -- > tests/i915/gem_exec_schedule.c | 2 -- > tests/i915/gem_exec_suspend.c | 2 -- > tests/i915/gem_fence_thrash.c | 2 -- > tests/i915/gem_fence_upload.c | 2 -- > tests/i915/gem_fenced_exec_thrash.c | 2 -- > tests/i915/gem_flink_race.c | 2 -- > tests/i915/gem_gtt_cpu_tlb.c | 3 --- > tests/i915/gem_gtt_hog.c | 2 -- > tests/i915/gem_gtt_speed.c | 2 -- > tests/i915/gem_largeobject.c | 2 -- > tests/i915/gem_madvise.c | 2 -- > tests/i915/gem_mmap_offset_exhaustion.c | 2 -- > tests/i915/gem_partial_pwrite_pread.c | 2 -- > tests/i915/gem_persistent_relocs.c | 2 -- > tests/i915/gem_pread_after_blit.c | 2 -- > tests/i915/gem_pwrite_pread.c | 2 -- > tests/i915/gem_pwrite_snooped.c | 2 -- > tests/i915/gem_read_read_speed.c | 2 -- > tests/i915/gem_readwrite.c | 2 -- > tests/i915/gem_reloc_vs_gpu.c | 2 -- > tests/i915/gem_render_linear_blits.c | 3 --- > tests/i915/gem_render_tiled_blits.c | 3 --- > tests/i915/gem_reset_stats.c | 1 - > tests/i915/gem_ringfill.c | 2 -- > tests/i915/gem_set_tiling_vs_blt.c | 2 -- > tests/i915/gem_set_tiling_vs_gtt.c | 2 -- > tests/i915/gem_set_tiling_vs_pwrite.c | 2 -- > tests/i915/gem_shrink.c | 2 -- > tests/i915/gem_softpin.c | 2 -- > tests/i915/gem_spin_batch.c | 2 -- > tests/i915/gem_stolen.c | 2 -- > tests/i915/gem_storedw_batches_loop.c | 2 -- > tests/i915/gem_sync.c | 2 -- > tests/i915/gem_threaded_access_tiled.c | 2 -- > tests/i915/gem_tiled_fence_blits.c | 3 --- > tests/i915/gem_tiled_partial_pwrite_pread.c | 2 -- > tests/i915/gem_unfence_active_buffers.c | 2 -- > tests/i915/gem_unref_active_buffers.c | 2 -- > tests/i915/gem_wait.c | 2 -- > tests/i915/gem_write_read_ring_switch.c | 2 -- > tests/i915/i915_hangman.c | 2 -- > tests/i915/i915_pm_backlight.c | 2 -- > tests/i915/i915_pm_dc.c | 1 - > tests/i915/i915_pm_rc6_residency.c | 2 -- > tests/i915/i915_pm_rps.c | 2 -- > tests/i915/i915_pm_sseu.c | 3 --- > tests/i915/i915_suspend.c | 2 -- > tests/kms_atomic_interruptible.c | 2 -- > tests/kms_atomic_transition.c | 2 -- > tests/kms_available_modes_crc.c | 2 -- > tests/kms_big_fb.c | 2 -- > tests/kms_busy.c | 2 -- > tests/kms_chamelium.c | 2 -- > tests/kms_color.c | 2 -- > tests/kms_concurrent.c | 2 -- > tests/kms_content_protection.c | 2 -- > tests/kms_crtc_background_color.c | 2 -- > tests/kms_cursor_crc.c | 2 -- > tests/kms_cursor_edge_walk.c | 2 -- > tests/kms_fence_pin_leak.c | 2 -- > tests/kms_flip_event_leak.c | 2 -- > tests/kms_flip_tiling.c | 2 -- > tests/kms_invalid_dotclock.c | 2 -- > tests/kms_legacy_colorkey.c | 2 -- > tests/kms_mmap_write_crc.c | 2 -- > tests/kms_panel_fitting.c | 2 -- > tests/kms_pipe_b_c_ivb.c | 2 -- > tests/kms_pipe_crc_basic.c | 2 -- > tests/kms_plane.c | 2 -- > tests/kms_plane_cursor.c | 2 -- > tests/kms_plane_lowres.c | 2 -- > tests/kms_plane_multiple.c | 2 -- > tests/kms_plane_scaling.c | 2 -- > tests/kms_properties.c | 2 -- > tests/kms_psr.c | 2 -- > tests/kms_psr2_su.c | 2 -- > tests/kms_pwrite_crc.c | 2 -- > tests/kms_rmfb.c | 2 -- > tests/kms_rotation_crc.c | 2 -- > tests/kms_sequence.c | 2 -- > tests/kms_setmode.c | 2 -- > tests/kms_universal_plane.c | 2 -- > tests/kms_vblank.c | 2 -- > tests/kms_vrr.c | 2 -- > tests/perf.c | 2 -- > tests/prime_busy.c | 1 - > tests/prime_mmap_kms.c | 2 -- > tests/prime_udl.c | 2 -- > tests/testdisplay.c | 1 - > tests/tools_test.c | 2 -- > tools/intel_dp_compliance.c | 2 -- > 113 files changed, 236 deletions(-) > _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 17+ messages in thread
* [igt-dev] ✗ Fi.CI.BAT: failure for removal of igt_skip_on_simulation() from i915/ tests 2019-10-16 7:49 [igt-dev] [PATCH 0/7] removal of igt_skip_on_simulation() from i915/ tests Swati Sharma ` (7 preceding siblings ...) 2019-10-16 8:49 ` [igt-dev] [PATCH 0/7] removal of igt_skip_on_simulation() from i915/ tests Martin Peres @ 2019-10-16 9:55 ` Patchwork 2019-10-16 10:53 ` Petri Latvala 2019-10-16 11:25 ` [igt-dev] ✗ GitLab.Pipeline: warning for removal of igt_skip_on_simulation() from i915/ tests (rev2) Patchwork ` (3 subsequent siblings) 12 siblings, 1 reply; 17+ messages in thread From: Patchwork @ 2019-10-16 9:55 UTC (permalink / raw) To: Swati Sharma; +Cc: igt-dev == Series Details == Series: removal of igt_skip_on_simulation() from i915/ tests URL : https://patchwork.freedesktop.org/series/68073/ State : failure == Summary == CI Bug Log - changes from CI_DRM_7104 -> IGTPW_3575 ==================================================== Summary ------- **FAILURE** Serious unknown changes coming with IGTPW_3575 absolutely need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in IGTPW_3575, please notify your bug team to allow them to document this new failure mode, which will reduce false positives in CI. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3575/index.html Possible new issues ------------------- Here are the unknown changes that may have been introduced in IGTPW_3575: ### IGT changes ### #### Possible regressions #### * igt@gem_render_linear_blits@basic: - fi-cml-u: [PASS][1] -> [CRASH][2] +1 similar issue [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7104/fi-cml-u/igt@gem_render_linear_blits@basic.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3575/fi-cml-u/igt@gem_render_linear_blits@basic.html * igt@gem_render_tiled_blits@basic: - fi-cml-u2: [PASS][3] -> [CRASH][4] +1 similar issue [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7104/fi-cml-u2/igt@gem_render_tiled_blits@basic.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3575/fi-cml-u2/igt@gem_render_tiled_blits@basic.html * igt@kms_frontbuffer_tracking@basic: - fi-cml-u: [PASS][5] -> [FAIL][6] +5 similar issues [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7104/fi-cml-u/igt@kms_frontbuffer_tracking@basic.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3575/fi-cml-u/igt@kms_frontbuffer_tracking@basic.html - fi-cml-u2: [PASS][7] -> [FAIL][8] +5 similar issues [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7104/fi-cml-u2/igt@kms_frontbuffer_tracking@basic.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3575/fi-cml-u2/igt@kms_frontbuffer_tracking@basic.html #### Suppressed #### The following results come from untrusted machines, tests, or statuses. They do not affect the overall result. * igt@gem_render_linear_blits@basic: - {fi-tgl-u}: NOTRUN -> [CRASH][9] +1 similar issue [9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3575/fi-tgl-u/igt@gem_render_linear_blits@basic.html * igt@gem_render_tiled_blits@basic: - {fi-tgl-u2}: NOTRUN -> [CRASH][10] +1 similar issue [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3575/fi-tgl-u2/igt@gem_render_tiled_blits@basic.html * igt@kms_psr@primary_mmap_gtt: - {fi-tgl-u}: NOTRUN -> [FAIL][11] +4 similar issues [11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3575/fi-tgl-u/igt@kms_psr@primary_mmap_gtt.html * igt@kms_psr@primary_page_flip: - {fi-tgl-u2}: NOTRUN -> [FAIL][12] +4 similar issues [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3575/fi-tgl-u2/igt@kms_psr@primary_page_flip.html Known issues ------------ Here are the changes found in IGTPW_3575 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_flink_basic@flink-lifetime: - fi-icl-u3: [PASS][13] -> [DMESG-WARN][14] ([fdo#107724]) +3 similar issues [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7104/fi-icl-u3/igt@gem_flink_basic@flink-lifetime.html [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3575/fi-icl-u3/igt@gem_flink_basic@flink-lifetime.html #### Possible fixes #### * igt@gem_ctx_create@basic-files: - {fi-tgl-u}: [INCOMPLETE][15] ([fdo#111735]) -> [PASS][16] [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7104/fi-tgl-u/igt@gem_ctx_create@basic-files.html [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3575/fi-tgl-u/igt@gem_ctx_create@basic-files.html * igt@gem_mmap_gtt@basic-write-cpu-read-gtt: - fi-icl-u3: [DMESG-WARN][17] ([fdo#107724]) -> [PASS][18] [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7104/fi-icl-u3/igt@gem_mmap_gtt@basic-write-cpu-read-gtt.html [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3575/fi-icl-u3/igt@gem_mmap_gtt@basic-write-cpu-read-gtt.html * igt@i915_selftest@live_hangcheck: - {fi-icl-u4}: [INCOMPLETE][19] ([fdo#107713] / [fdo#108569]) -> [PASS][20] [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7104/fi-icl-u4/igt@i915_selftest@live_hangcheck.html [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3575/fi-icl-u4/igt@i915_selftest@live_hangcheck.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713 [fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724 [fdo#108569]: https://bugs.freedesktop.org/show_bug.cgi?id=108569 [fdo#111735]: https://bugs.freedesktop.org/show_bug.cgi?id=111735 Participating hosts (53 -> 46) ------------------------------ Missing (7): fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-icl-y fi-byt-clapper fi-bdw-samus Build changes ------------- * CI: CI-20190529 -> None * IGT: IGT_5231 -> IGTPW_3575 CI-20190529: 20190529 CI_DRM_7104: fab6dd8ea079f5f0ec5cda08c8cfe12cabf69157 @ git://anongit.freedesktop.org/gfx-ci/linux IGTPW_3575: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3575/index.html IGT_5231: e293051f8f99c72cb01d21e4b73a5928ea351eb3 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3575/index.html _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [igt-dev] ✗ Fi.CI.BAT: failure for removal of igt_skip_on_simulation() from i915/ tests 2019-10-16 9:55 ` [igt-dev] ✗ Fi.CI.BAT: failure for " Patchwork @ 2019-10-16 10:53 ` Petri Latvala 2019-10-16 11:11 ` Arkadiusz Hiler 0 siblings, 1 reply; 17+ messages in thread From: Petri Latvala @ 2019-10-16 10:53 UTC (permalink / raw) To: igt-dev On Wed, Oct 16, 2019 at 09:55:29AM +0000, Patchwork wrote: > == Series Details == > > Series: removal of igt_skip_on_simulation() from i915/ tests > URL : https://patchwork.freedesktop.org/series/68073/ > State : failure > > == Summary == > > CI Bug Log - changes from CI_DRM_7104 -> IGTPW_3575 > ==================================================== > > Summary > ------- > > **FAILURE** > > Serious unknown changes coming with IGTPW_3575 absolutely need to be > verified manually. > > If you think the reported changes have nothing to do with the changes > introduced in IGTPW_3575, please notify your bug team to allow them > to document this new failure mode, which will reduce false positives in CI. > > External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3575/index.html > > Possible new issues > ------------------- > > Here are the unknown changes that may have been introduced in IGTPW_3575: > > ### IGT changes ### > > #### Possible regressions #### > > * igt@gem_render_linear_blits@basic: > - fi-cml-u: [PASS][1] -> [CRASH][2] +1 similar issue > [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7104/fi-cml-u/igt@gem_render_linear_blits@basic.html > [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3575/fi-cml-u/igt@gem_render_linear_blits@basic.html > > * igt@gem_render_tiled_blits@basic: > - fi-cml-u2: [PASS][3] -> [CRASH][4] +1 similar issue > [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7104/fi-cml-u2/igt@gem_render_tiled_blits@basic.html > [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3575/fi-cml-u2/igt@gem_render_tiled_blits@basic.html Wrong libdrm. Arek, is it broken 1) again, or 2) still? -- Petri Latvala _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [igt-dev] ✗ Fi.CI.BAT: failure for removal of igt_skip_on_simulation() from i915/ tests 2019-10-16 10:53 ` Petri Latvala @ 2019-10-16 11:11 ` Arkadiusz Hiler 0 siblings, 0 replies; 17+ messages in thread From: Arkadiusz Hiler @ 2019-10-16 11:11 UTC (permalink / raw) To: Petri Latvala; +Cc: igt-dev On Wed, Oct 16, 2019 at 01:53:25PM +0300, Petri Latvala wrote: > On Wed, Oct 16, 2019 at 09:55:29AM +0000, Patchwork wrote: > > == Series Details == > > > > Series: removal of igt_skip_on_simulation() from i915/ tests > > URL : https://patchwork.freedesktop.org/series/68073/ > > State : failure > > > > == Summary == > > > > CI Bug Log - changes from CI_DRM_7104 -> IGTPW_3575 > > ==================================================== > > > > Summary > > ------- > > > > **FAILURE** > > > > Serious unknown changes coming with IGTPW_3575 absolutely need to be > > verified manually. > > > > If you think the reported changes have nothing to do with the changes > > introduced in IGTPW_3575, please notify your bug team to allow them > > to document this new failure mode, which will reduce false positives in CI. > > > > External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3575/index.html > > > > Possible new issues > > ------------------- > > > > Here are the unknown changes that may have been introduced in IGTPW_3575: > > > > ### IGT changes ### > > > > #### Possible regressions #### > > > > * igt@gem_render_linear_blits@basic: > > - fi-cml-u: [PASS][1] -> [CRASH][2] +1 similar issue > > [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7104/fi-cml-u/igt@gem_render_linear_blits@basic.html > > [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3575/fi-cml-u/igt@gem_render_linear_blits@basic.html > > > > * igt@gem_render_tiled_blits@basic: > > - fi-cml-u2: [PASS][3] -> [CRASH][4] +1 similar issue > > [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7104/fi-cml-u2/igt@gem_render_tiled_blits@basic.html > > [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3575/fi-cml-u2/igt@gem_render_tiled_blits@basic.html > > > Wrong libdrm. Arek, is it broken 1) again, or 2) still? 3) other Turns out there was a stray conditional "make install" happening outside of normal building path, which is just used for "FAST" build, i.e. pre-merge. This was not altered, which affected pre-merge. I have changed the flow to be more like: if $POSTMERGE; then git -C libdrm pull (cd libdrm; meson build && ninja -C build) report_errors_if_failed $? fi ninja -C libdrm/build install report_errors_if_failed $? I have queued a rerun. Thanks for noticing! -- Cheers, Arek _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 17+ messages in thread
* [igt-dev] ✗ GitLab.Pipeline: warning for removal of igt_skip_on_simulation() from i915/ tests (rev2) 2019-10-16 7:49 [igt-dev] [PATCH 0/7] removal of igt_skip_on_simulation() from i915/ tests Swati Sharma ` (8 preceding siblings ...) 2019-10-16 9:55 ` [igt-dev] ✗ Fi.CI.BAT: failure for " Patchwork @ 2019-10-16 11:25 ` Patchwork 2019-10-16 11:31 ` Arkadiusz Hiler 2019-10-16 11:44 ` [igt-dev] ✓ Fi.CI.BAT: success " Patchwork ` (2 subsequent siblings) 12 siblings, 1 reply; 17+ messages in thread From: Patchwork @ 2019-10-16 11:25 UTC (permalink / raw) To: Swati Sharma; +Cc: igt-dev == Series Details == Series: removal of igt_skip_on_simulation() from i915/ tests (rev2) URL : https://patchwork.freedesktop.org/series/68073/ State : warning == Summary == Did not get list of undocumented tests for this run, something is wrong! Other than that, pipeline status: FAILED. see https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/pipelines/70904 for more details == Logs == For more details see: https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/pipelines/70904 _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [igt-dev] ✗ GitLab.Pipeline: warning for removal of igt_skip_on_simulation() from i915/ tests (rev2) 2019-10-16 11:25 ` [igt-dev] ✗ GitLab.Pipeline: warning for removal of igt_skip_on_simulation() from i915/ tests (rev2) Patchwork @ 2019-10-16 11:31 ` Arkadiusz Hiler 0 siblings, 0 replies; 17+ messages in thread From: Arkadiusz Hiler @ 2019-10-16 11:31 UTC (permalink / raw) To: igt-dev On Wed, Oct 16, 2019 at 11:25:32AM +0000, Patchwork wrote: > == Series Details == > > Series: removal of igt_skip_on_simulation() from i915/ tests (rev2) > URL : https://patchwork.freedesktop.org/series/68073/ > State : warning > > == Summary == > > Did not get list of undocumented tests for this run, something is wrong! > > Other than that, pipeline status: FAILED. > > see https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/pipelines/70904 for more details https://gitlab.freedesktop.org/freedesktop/freedesktop/issues/197 GitLab runner acting up, sorry for the noise _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 17+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for removal of igt_skip_on_simulation() from i915/ tests (rev2) 2019-10-16 7:49 [igt-dev] [PATCH 0/7] removal of igt_skip_on_simulation() from i915/ tests Swati Sharma ` (9 preceding siblings ...) 2019-10-16 11:25 ` [igt-dev] ✗ GitLab.Pipeline: warning for removal of igt_skip_on_simulation() from i915/ tests (rev2) Patchwork @ 2019-10-16 11:44 ` Patchwork 2019-10-16 21:14 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork 2019-11-18 14:46 ` [igt-dev] [PATCH 0/7] removal of igt_skip_on_simulation() from i915/ tests Katarzyna Dec 12 siblings, 0 replies; 17+ messages in thread From: Patchwork @ 2019-10-16 11:44 UTC (permalink / raw) To: Swati Sharma; +Cc: igt-dev == Series Details == Series: removal of igt_skip_on_simulation() from i915/ tests (rev2) URL : https://patchwork.freedesktop.org/series/68073/ State : success == Summary == CI Bug Log - changes from CI_DRM_7105 -> IGTPW_3576 ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/index.html Known issues ------------ Here are the changes found in IGTPW_3576 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@kms_chamelium@hdmi-hpd-fast: - fi-kbl-7500u: [PASS][1] -> [FAIL][2] ([fdo#111407]) [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html * igt@prime_vgem@basic-wait-default: - fi-icl-u3: [PASS][3] -> [DMESG-WARN][4] ([fdo#107724]) +2 similar issues [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/fi-icl-u3/igt@prime_vgem@basic-wait-default.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/fi-icl-u3/igt@prime_vgem@basic-wait-default.html #### Possible fixes #### * igt@gem_ctx_switch@legacy-render: - {fi-icl-guc}: [INCOMPLETE][5] ([fdo#107713] / [fdo#111381]) -> [PASS][6] [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/fi-icl-guc/igt@gem_ctx_switch@legacy-render.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/fi-icl-guc/igt@gem_ctx_switch@legacy-render.html * igt@gem_ctx_switch@rcs0: - fi-icl-u2: [INCOMPLETE][7] ([fdo#107713]) -> [PASS][8] [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/fi-icl-u2/igt@gem_ctx_switch@rcs0.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/fi-icl-u2/igt@gem_ctx_switch@rcs0.html * igt@gem_exec_suspend@basic-s3: - {fi-cml-s}: [DMESG-WARN][9] ([fdo#111764]) -> [PASS][10] [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/fi-cml-s/igt@gem_exec_suspend@basic-s3.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/fi-cml-s/igt@gem_exec_suspend@basic-s3.html * igt@gem_mmap_gtt@basic-read-write-distinct: - fi-icl-u3: [DMESG-WARN][11] ([fdo#107724]) -> [PASS][12] +1 similar issue [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/fi-icl-u3/igt@gem_mmap_gtt@basic-read-write-distinct.html [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/fi-icl-u3/igt@gem_mmap_gtt@basic-read-write-distinct.html * igt@i915_selftest@live_gem_contexts: - fi-cfl-8109u: [DMESG-FAIL][13] -> [PASS][14] [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/fi-cfl-8109u/igt@i915_selftest@live_gem_contexts.html [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/fi-cfl-8109u/igt@i915_selftest@live_gem_contexts.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713 [fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724 [fdo#111381]: https://bugs.freedesktop.org/show_bug.cgi?id=111381 [fdo#111407]: https://bugs.freedesktop.org/show_bug.cgi?id=111407 [fdo#111735]: https://bugs.freedesktop.org/show_bug.cgi?id=111735 [fdo#111764]: https://bugs.freedesktop.org/show_bug.cgi?id=111764 [fdo#111831]: https://bugs.freedesktop.org/show_bug.cgi?id=111831 Participating hosts (52 -> 46) ------------------------------ Additional (1): fi-ivb-3770 Missing (7): fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-icl-y fi-byt-clapper fi-bdw-samus Build changes ------------- * CI: CI-20190529 -> None * IGT: IGT_5231 -> IGTPW_3576 CI-20190529: 20190529 CI_DRM_7105: 46d5b78c89fd86f82347cac18c81a278dbe08841 @ git://anongit.freedesktop.org/gfx-ci/linux IGTPW_3576: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/index.html IGT_5231: e293051f8f99c72cb01d21e4b73a5928ea351eb3 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/index.html _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 17+ messages in thread
* [igt-dev] ✓ Fi.CI.IGT: success for removal of igt_skip_on_simulation() from i915/ tests (rev2) 2019-10-16 7:49 [igt-dev] [PATCH 0/7] removal of igt_skip_on_simulation() from i915/ tests Swati Sharma ` (10 preceding siblings ...) 2019-10-16 11:44 ` [igt-dev] ✓ Fi.CI.BAT: success " Patchwork @ 2019-10-16 21:14 ` Patchwork 2019-11-18 14:46 ` [igt-dev] [PATCH 0/7] removal of igt_skip_on_simulation() from i915/ tests Katarzyna Dec 12 siblings, 0 replies; 17+ messages in thread From: Patchwork @ 2019-10-16 21:14 UTC (permalink / raw) To: Swati Sharma; +Cc: igt-dev == Series Details == Series: removal of igt_skip_on_simulation() from i915/ tests (rev2) URL : https://patchwork.freedesktop.org/series/68073/ State : success == Summary == CI Bug Log - changes from CI_DRM_7105_full -> IGTPW_3576_full ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/index.html Possible new issues ------------------- Here are the unknown changes that may have been introduced in IGTPW_3576_full: ### IGT changes ### #### Suppressed #### The following results come from untrusted machines, tests, or statuses. They do not affect the overall result. * igt@kms_atomic_transition@3x-modeset-transitions-nonblocking-fencing: - {shard-tglb}: NOTRUN -> [SKIP][1] +2 similar issues [1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-tglb6/igt@kms_atomic_transition@3x-modeset-transitions-nonblocking-fencing.html * {igt@kms_cursor_crc@pipe-d-cursor-256x85-sliding}: - {shard-tglb}: NOTRUN -> [FAIL][2] +1 similar issue [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-tglb6/igt@kms_cursor_crc@pipe-d-cursor-256x85-sliding.html Known issues ------------ Here are the changes found in IGTPW_3576_full that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_ctx_isolation@bcs0-s3: - shard-apl: [PASS][3] -> [DMESG-WARN][4] ([fdo#108566]) +3 similar issues [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-apl8/igt@gem_ctx_isolation@bcs0-s3.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-apl3/igt@gem_ctx_isolation@bcs0-s3.html * igt@gem_ctx_shared@exec-single-timeline-bsd: - shard-iclb: [PASS][5] -> [SKIP][6] ([fdo#110841]) [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-iclb6/igt@gem_ctx_shared@exec-single-timeline-bsd.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-iclb2/igt@gem_ctx_shared@exec-single-timeline-bsd.html * igt@gem_exec_balancer@smoke: - shard-iclb: [PASS][7] -> [SKIP][8] ([fdo#110854]) [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-iclb1/igt@gem_exec_balancer@smoke.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-iclb5/igt@gem_exec_balancer@smoke.html * igt@gem_exec_schedule@wide-bsd: - shard-iclb: [PASS][9] -> [SKIP][10] ([fdo#111325]) +3 similar issues [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-iclb6/igt@gem_exec_schedule@wide-bsd.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-iclb2/igt@gem_exec_schedule@wide-bsd.html * igt@gem_userptr_blits@dmabuf-sync: - shard-snb: [PASS][11] -> [DMESG-WARN][12] ([fdo#111870]) +1 similar issue [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-snb6/igt@gem_userptr_blits@dmabuf-sync.html [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-snb2/igt@gem_userptr_blits@dmabuf-sync.html * igt@gem_userptr_blits@map-fixed-invalidate-busy-gup: - shard-hsw: [PASS][13] -> [DMESG-WARN][14] ([fdo#110789] / [fdo#111870]) [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-hsw8/igt@gem_userptr_blits@map-fixed-invalidate-busy-gup.html [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-hsw4/igt@gem_userptr_blits@map-fixed-invalidate-busy-gup.html * igt@gem_userptr_blits@map-fixed-invalidate-overlap-busy: - shard-hsw: [PASS][15] -> [DMESG-WARN][16] ([fdo#111870]) +1 similar issue [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-hsw7/igt@gem_userptr_blits@map-fixed-invalidate-overlap-busy.html [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-hsw1/igt@gem_userptr_blits@map-fixed-invalidate-overlap-busy.html * igt@kms_flip@flip-vs-suspend: - shard-hsw: [PASS][17] -> [INCOMPLETE][18] ([fdo#103540]) [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-hsw8/igt@kms_flip@flip-vs-suspend.html [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-hsw6/igt@kms_flip@flip-vs-suspend.html * igt@kms_flip_tiling@flip-x-tiled: - shard-iclb: [PASS][19] -> [FAIL][20] ([fdo#108303]) [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-iclb6/igt@kms_flip_tiling@flip-x-tiled.html [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-iclb1/igt@kms_flip_tiling@flip-x-tiled.html * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-draw-mmap-wc: - shard-kbl: [PASS][21] -> [FAIL][22] ([fdo#103167]) [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-kbl3/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-draw-mmap-wc.html [22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-kbl7/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-draw-mmap-wc.html - shard-apl: [PASS][23] -> [FAIL][24] ([fdo#103167]) [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-apl4/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-draw-mmap-wc.html [24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-apl5/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-draw-mmap-wc.html * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-blt: - shard-iclb: [PASS][25] -> [FAIL][26] ([fdo#103167]) +2 similar issues [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-iclb2/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-blt.html [26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-iclb2/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-blt.html * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-onoff: - shard-iclb: [PASS][27] -> [INCOMPLETE][28] ([fdo#106978] / [fdo#107713]) [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-iclb5/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-onoff.html [28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-iclb1/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-onoff.html * igt@kms_plane@plane-panning-top-left-pipe-c-planes: - shard-apl: [PASS][29] -> [INCOMPLETE][30] ([fdo#103927]) +3 similar issues [29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-apl3/igt@kms_plane@plane-panning-top-left-pipe-c-planes.html [30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-apl5/igt@kms_plane@plane-panning-top-left-pipe-c-planes.html * igt@kms_plane_lowres@pipe-a-tiling-y: - shard-iclb: [PASS][31] -> [FAIL][32] ([fdo#103166]) [31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-iclb2/igt@kms_plane_lowres@pipe-a-tiling-y.html [32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-iclb6/igt@kms_plane_lowres@pipe-a-tiling-y.html * igt@kms_psr@psr2_primary_mmap_cpu: - shard-iclb: [PASS][33] -> [SKIP][34] ([fdo#109441]) +3 similar issues [33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-iclb2/igt@kms_psr@psr2_primary_mmap_cpu.html [34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-iclb3/igt@kms_psr@psr2_primary_mmap_cpu.html * igt@prime_vgem@fence-wait-bsd2: - shard-iclb: [PASS][35] -> [SKIP][36] ([fdo#109276]) +9 similar issues [35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-iclb4/igt@prime_vgem@fence-wait-bsd2.html [36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-iclb6/igt@prime_vgem@fence-wait-bsd2.html #### Possible fixes #### * igt@gem_eio@in-flight-contexts-immediate: - shard-snb: [FAIL][37] ([fdo#111925]) -> [PASS][38] [37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-snb7/igt@gem_eio@in-flight-contexts-immediate.html [38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-snb6/igt@gem_eio@in-flight-contexts-immediate.html * igt@gem_exec_schedule@preempt-contexts-bsd2: - shard-iclb: [SKIP][39] ([fdo#109276]) -> [PASS][40] +16 similar issues [39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-iclb7/igt@gem_exec_schedule@preempt-contexts-bsd2.html [40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-iclb4/igt@gem_exec_schedule@preempt-contexts-bsd2.html * igt@gem_persistent_relocs@forked-interruptible-thrash-inactive: - shard-hsw: [FAIL][41] -> [PASS][42] [41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-hsw6/igt@gem_persistent_relocs@forked-interruptible-thrash-inactive.html [42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-hsw6/igt@gem_persistent_relocs@forked-interruptible-thrash-inactive.html * igt@gem_persistent_relocs@forked-interruptible-thrashing: - shard-snb: [FAIL][43] -> [PASS][44] [43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-snb1/igt@gem_persistent_relocs@forked-interruptible-thrashing.html [44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-snb6/igt@gem_persistent_relocs@forked-interruptible-thrashing.html * igt@gem_userptr_blits@sync-unmap: - shard-snb: [DMESG-WARN][45] ([fdo#111870]) -> [PASS][46] +1 similar issue [45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-snb2/igt@gem_userptr_blits@sync-unmap.html [46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-snb7/igt@gem_userptr_blits@sync-unmap.html * igt@gem_userptr_blits@sync-unmap-after-close: - shard-hsw: [DMESG-WARN][47] ([fdo#111870]) -> [PASS][48] [47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-hsw8/igt@gem_userptr_blits@sync-unmap-after-close.html [48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-hsw1/igt@gem_userptr_blits@sync-unmap-after-close.html - shard-glk: [DMESG-WARN][49] ([fdo#111870]) -> [PASS][50] [49]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-glk5/igt@gem_userptr_blits@sync-unmap-after-close.html [50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-glk4/igt@gem_userptr_blits@sync-unmap-after-close.html * igt@i915_suspend@sysfs-reader: - shard-apl: [DMESG-WARN][51] ([fdo#108566]) -> [PASS][52] +5 similar issues [51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-apl6/igt@i915_suspend@sysfs-reader.html [52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-apl1/igt@i915_suspend@sysfs-reader.html * igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-b: - shard-hsw: [INCOMPLETE][53] ([fdo#103540]) -> [PASS][54] +1 similar issue [53]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-hsw8/igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-b.html [54]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-hsw5/igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-b.html * igt@kms_busy@extended-modeset-hang-oldfb-with-reset-render-a: - shard-apl: [INCOMPLETE][55] ([fdo#103927]) -> [PASS][56] +2 similar issues [55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-apl6/igt@kms_busy@extended-modeset-hang-oldfb-with-reset-render-a.html [56]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-apl4/igt@kms_busy@extended-modeset-hang-oldfb-with-reset-render-a.html * igt@kms_cursor_crc@pipe-b-cursor-128x42-sliding: - shard-kbl: [FAIL][57] ([fdo#103232]) -> [PASS][58] [57]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-kbl1/igt@kms_cursor_crc@pipe-b-cursor-128x42-sliding.html [58]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-kbl4/igt@kms_cursor_crc@pipe-b-cursor-128x42-sliding.html * igt@kms_flip@flip-vs-suspend: - shard-kbl: [DMESG-WARN][59] ([fdo#103313]) -> [PASS][60] [59]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-kbl1/igt@kms_flip@flip-vs-suspend.html [60]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-kbl3/igt@kms_flip@flip-vs-suspend.html * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render: - shard-iclb: [FAIL][61] ([fdo#103167]) -> [PASS][62] +2 similar issues [61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-iclb8/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render.html [62]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-iclb5/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-cur-indfb-draw-render.html * igt@kms_frontbuffer_tracking@psr-suspend: - {shard-tglb}: [INCOMPLETE][63] ([fdo#111832] / [fdo#111850]) -> [PASS][64] [63]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-tglb1/igt@kms_frontbuffer_tracking@psr-suspend.html [64]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-tglb6/igt@kms_frontbuffer_tracking@psr-suspend.html * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a: - shard-kbl: [INCOMPLETE][65] ([fdo#103665]) -> [PASS][66] [65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-kbl3/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html [66]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-kbl3/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html * igt@kms_plane@plane-position-covered-pipe-c-planes: - shard-apl: [FAIL][67] ([fdo#110038]) -> [PASS][68] [67]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-apl7/igt@kms_plane@plane-position-covered-pipe-c-planes.html [68]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-apl7/igt@kms_plane@plane-position-covered-pipe-c-planes.html - shard-kbl: [FAIL][69] ([fdo#110038]) -> [PASS][70] [69]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-kbl6/igt@kms_plane@plane-position-covered-pipe-c-planes.html [70]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-kbl2/igt@kms_plane@plane-position-covered-pipe-c-planes.html * igt@kms_psr2_su@frontbuffer: - shard-iclb: [SKIP][71] ([fdo#109642] / [fdo#111068]) -> [PASS][72] [71]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-iclb3/igt@kms_psr2_su@frontbuffer.html [72]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-iclb2/igt@kms_psr2_su@frontbuffer.html * igt@kms_psr@psr2_no_drrs: - shard-iclb: [SKIP][73] ([fdo#109441]) -> [PASS][74] +1 similar issue [73]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-iclb1/igt@kms_psr@psr2_no_drrs.html [74]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-iclb2/igt@kms_psr@psr2_no_drrs.html #### Warnings #### * igt@gem_userptr_blits@sync-unmap-after-close: - shard-snb: [DMESG-WARN][75] ([fdo#111870]) -> [DMESG-WARN][76] ([fdo#110789] / [fdo#111870]) [75]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7105/shard-snb5/igt@gem_userptr_blits@sync-unmap-after-close.html [76]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/shard-snb1/igt@gem_userptr_blits@sync-unmap-after-close.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#103166]: https://bugs.freedesktop.org/show_bug.cgi?id=103166 [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167 [fdo#103232]: https://bugs.freedesktop.org/show_bug.cgi?id=103232 [fdo#103313]: https://bugs.freedesktop.org/show_bug.cgi?id=103313 [fdo#103540]: https://bugs.freedesktop.org/show_bug.cgi?id=103540 [fdo#103665]: https://bugs.freedesktop.org/show_bug.cgi?id=103665 [fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927 [fdo#106978]: https://bugs.freedesktop.org/show_bug.cgi?id=106978 [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713 [fdo#108303]: https://bugs.freedesktop.org/show_bug.cgi?id=108303 [fdo#108566]: https://bugs.freedesktop.org/show_bug.cgi?id=108566 [fdo#109276]: https://bugs.freedesktop.org/show_bug.cgi?id=109276 [fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441 [fdo#109642]: https://bugs.freedesktop.org/show_bug.cgi?id=109642 [fdo#110038]: https://bugs.freedesktop.org/show_bug.cgi?id=110038 [fdo#110548]: https://bugs.freedesktop.org/show_bug.cgi?id=110548 [fdo#110789]: https://bugs.freedesktop.org/show_bug.cgi?id=110789 [fdo#110841]: https://bugs.freedesktop.org/show_bug.cgi?id=110841 [fdo#110854]: https://bugs.freedesktop.org/show_bug.cgi?id=110854 [fdo#111068]: https://bugs.freedesktop.org/show_bug.cgi?id=111068 [fdo#111325]: https://bugs.freedesktop.org/show_bug.cgi?id=111325 [fdo#111329]: https://bugs.freedesktop.org/show_bug.cgi?id=111329 [fdo#111832]: https://bugs.freedesktop.org/show_bug.cgi?id=111832 [fdo#111850]: https://bugs.freedesktop.org/show_bug.cgi?id=111850 [fdo#111870]: https://bugs.freedesktop.org/show_bug.cgi?id=111870 [fdo#111925]: https://bugs.freedesktop.org/show_bug.cgi?id=111925 Participating hosts (11 -> 7) ------------------------------ Missing (4): pig-skl-6260u shard-skl pig-hsw-4770r pig-glk-j5005 Build changes ------------- * CI: CI-20190529 -> None * IGT: IGT_5231 -> IGTPW_3576 * Piglit: piglit_4509 -> None CI-20190529: 20190529 CI_DRM_7105: 46d5b78c89fd86f82347cac18c81a278dbe08841 @ git://anongit.freedesktop.org/gfx-ci/linux IGTPW_3576: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/index.html IGT_5231: e293051f8f99c72cb01d21e4b73a5928ea351eb3 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3576/index.html _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [igt-dev] [PATCH 0/7] removal of igt_skip_on_simulation() from i915/ tests 2019-10-16 7:49 [igt-dev] [PATCH 0/7] removal of igt_skip_on_simulation() from i915/ tests Swati Sharma ` (11 preceding siblings ...) 2019-10-16 21:14 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork @ 2019-11-18 14:46 ` Katarzyna Dec 12 siblings, 0 replies; 17+ messages in thread From: Katarzyna Dec @ 2019-11-18 14:46 UTC (permalink / raw) To: Swati Sharma, igt-dev; +Cc: lucas.demarchi, martin.peres, maarten.lankhorst On Wed, Oct 16, 2019 at 01:19:05PM +0530, Swati Sharma wrote: > Removing igt_skip_on_simulation() since this feature is not supported > anymore by i915 driver. Removing it from gem, kms, i915 specific, tools > and benchmark tests. How this series end up? Will we be merging it soon? Kasia > > Swati Sharma (7): > tests/i915/gem: remove igt_skip_on_simulation() > tests/i915/kms: remove igt_skip_on_simulation() > tests/i915: remove igt_skip_on_simulation() > tests: remove igt_skip_on_simulation() > tools: remove igt_skip_on_simulation() > benchmarks: remove igt_skip_on_simulation() > lib: remove igt_skip_on_simulation > > benchmarks/gem_userptr_benchmark.c | 2 -- > lib/igt_aux.c | 7 ------- > lib/intel_os.c | 2 -- > tests/debugfs_test.c | 2 -- > tests/i915/gem_busy.c | 2 -- > tests/i915/gem_caching.c | 2 -- > tests/i915/gem_concurrent_all.c | 2 -- > tests/i915/gem_create.c | 2 -- > tests/i915/gem_cs_prefetch.c | 2 -- > tests/i915/gem_cs_tlb.c | 2 -- > tests/i915/gem_ctx_exec.c | 2 -- > tests/i915/gem_ctx_switch.c | 6 ------ > tests/i915/gem_ctx_thrash.c | 2 -- > tests/i915/gem_double_irq_loop.c | 2 -- > tests/i915/gem_eio.c | 2 -- > tests/i915/gem_exec_async.c | 2 -- > tests/i915/gem_exec_balancer.c | 2 -- > tests/i915/gem_exec_blt.c | 2 -- > tests/i915/gem_exec_capture.c | 2 -- > tests/i915/gem_exec_fence.c | 2 -- > tests/i915/gem_exec_gttfill.c | 2 -- > tests/i915/gem_exec_lut_handle.c | 2 -- > tests/i915/gem_exec_schedule.c | 2 -- > tests/i915/gem_exec_suspend.c | 2 -- > tests/i915/gem_fence_thrash.c | 2 -- > tests/i915/gem_fence_upload.c | 2 -- > tests/i915/gem_fenced_exec_thrash.c | 2 -- > tests/i915/gem_flink_race.c | 2 -- > tests/i915/gem_gtt_cpu_tlb.c | 3 --- > tests/i915/gem_gtt_hog.c | 2 -- > tests/i915/gem_gtt_speed.c | 2 -- > tests/i915/gem_largeobject.c | 2 -- > tests/i915/gem_madvise.c | 2 -- > tests/i915/gem_mmap_offset_exhaustion.c | 2 -- > tests/i915/gem_partial_pwrite_pread.c | 2 -- > tests/i915/gem_persistent_relocs.c | 2 -- > tests/i915/gem_pread_after_blit.c | 2 -- > tests/i915/gem_pwrite_pread.c | 2 -- > tests/i915/gem_pwrite_snooped.c | 2 -- > tests/i915/gem_read_read_speed.c | 2 -- > tests/i915/gem_readwrite.c | 2 -- > tests/i915/gem_reloc_vs_gpu.c | 2 -- > tests/i915/gem_render_linear_blits.c | 3 --- > tests/i915/gem_render_tiled_blits.c | 3 --- > tests/i915/gem_reset_stats.c | 1 - > tests/i915/gem_ringfill.c | 2 -- > tests/i915/gem_set_tiling_vs_blt.c | 2 -- > tests/i915/gem_set_tiling_vs_gtt.c | 2 -- > tests/i915/gem_set_tiling_vs_pwrite.c | 2 -- > tests/i915/gem_shrink.c | 2 -- > tests/i915/gem_softpin.c | 2 -- > tests/i915/gem_spin_batch.c | 2 -- > tests/i915/gem_stolen.c | 2 -- > tests/i915/gem_storedw_batches_loop.c | 2 -- > tests/i915/gem_sync.c | 2 -- > tests/i915/gem_threaded_access_tiled.c | 2 -- > tests/i915/gem_tiled_fence_blits.c | 3 --- > tests/i915/gem_tiled_partial_pwrite_pread.c | 2 -- > tests/i915/gem_unfence_active_buffers.c | 2 -- > tests/i915/gem_unref_active_buffers.c | 2 -- > tests/i915/gem_wait.c | 2 -- > tests/i915/gem_write_read_ring_switch.c | 2 -- > tests/i915/i915_hangman.c | 2 -- > tests/i915/i915_pm_backlight.c | 2 -- > tests/i915/i915_pm_dc.c | 1 - > tests/i915/i915_pm_rc6_residency.c | 2 -- > tests/i915/i915_pm_rps.c | 2 -- > tests/i915/i915_pm_sseu.c | 3 --- > tests/i915/i915_suspend.c | 2 -- > tests/kms_atomic_interruptible.c | 2 -- > tests/kms_atomic_transition.c | 2 -- > tests/kms_available_modes_crc.c | 2 -- > tests/kms_big_fb.c | 2 -- > tests/kms_busy.c | 2 -- > tests/kms_chamelium.c | 2 -- > tests/kms_color.c | 2 -- > tests/kms_concurrent.c | 2 -- > tests/kms_content_protection.c | 2 -- > tests/kms_crtc_background_color.c | 2 -- > tests/kms_cursor_crc.c | 2 -- > tests/kms_cursor_edge_walk.c | 2 -- > tests/kms_fence_pin_leak.c | 2 -- > tests/kms_flip_event_leak.c | 2 -- > tests/kms_flip_tiling.c | 2 -- > tests/kms_invalid_dotclock.c | 2 -- > tests/kms_legacy_colorkey.c | 2 -- > tests/kms_mmap_write_crc.c | 2 -- > tests/kms_panel_fitting.c | 2 -- > tests/kms_pipe_b_c_ivb.c | 2 -- > tests/kms_pipe_crc_basic.c | 2 -- > tests/kms_plane.c | 2 -- > tests/kms_plane_cursor.c | 2 -- > tests/kms_plane_lowres.c | 2 -- > tests/kms_plane_multiple.c | 2 -- > tests/kms_plane_scaling.c | 2 -- > tests/kms_properties.c | 2 -- > tests/kms_psr.c | 2 -- > tests/kms_psr2_su.c | 2 -- > tests/kms_pwrite_crc.c | 2 -- > tests/kms_rmfb.c | 2 -- > tests/kms_rotation_crc.c | 2 -- > tests/kms_sequence.c | 2 -- > tests/kms_setmode.c | 2 -- > tests/kms_universal_plane.c | 2 -- > tests/kms_vblank.c | 2 -- > tests/kms_vrr.c | 2 -- > tests/perf.c | 2 -- > tests/prime_busy.c | 1 - > tests/prime_mmap_kms.c | 2 -- > tests/prime_udl.c | 2 -- > tests/testdisplay.c | 1 - > tests/tools_test.c | 2 -- > tools/intel_dp_compliance.c | 2 -- > 113 files changed, 236 deletions(-) > > -- > 2.23.0 > > _______________________________________________ > igt-dev mailing list > igt-dev@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/igt-dev _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 17+ messages in thread
end of thread, other threads:[~2019-11-18 14:47 UTC | newest] Thread overview: 17+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2019-10-16 7:49 [igt-dev] [PATCH 0/7] removal of igt_skip_on_simulation() from i915/ tests Swati Sharma 2019-10-16 7:49 ` [igt-dev] [PATCH 1/7] tests/i915/gem: remove igt_skip_on_simulation() Swati Sharma 2019-10-16 7:49 ` [igt-dev] [PATCH 2/7] tests/i915/kms: " Swati Sharma 2019-10-16 7:49 ` [igt-dev] [PATCH 3/7] tests/i915: " Swati Sharma 2019-10-16 7:49 ` [igt-dev] [PATCH 4/7] tests: " Swati Sharma 2019-10-16 7:49 ` [igt-dev] [PATCH 5/7] tools: " Swati Sharma 2019-10-16 7:49 ` [igt-dev] [PATCH 6/7] benchmarks: " Swati Sharma 2019-10-16 7:49 ` [igt-dev] [PATCH 7/7] lib: remove igt_skip_on_simulation Swati Sharma 2019-10-16 8:49 ` [igt-dev] [PATCH 0/7] removal of igt_skip_on_simulation() from i915/ tests Martin Peres 2019-10-16 9:55 ` [igt-dev] ✗ Fi.CI.BAT: failure for " Patchwork 2019-10-16 10:53 ` Petri Latvala 2019-10-16 11:11 ` Arkadiusz Hiler 2019-10-16 11:25 ` [igt-dev] ✗ GitLab.Pipeline: warning for removal of igt_skip_on_simulation() from i915/ tests (rev2) Patchwork 2019-10-16 11:31 ` Arkadiusz Hiler 2019-10-16 11:44 ` [igt-dev] ✓ Fi.CI.BAT: success " Patchwork 2019-10-16 21:14 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork 2019-11-18 14:46 ` [igt-dev] [PATCH 0/7] removal of igt_skip_on_simulation() from i915/ tests Katarzyna Dec
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox