* [igt-dev] [PATCH i-g-t 0/2] gem_exec_big -- noreloc support
@ 2021-08-11 20:28 Andrzej Turko
2021-08-11 20:28 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_exec_big: Add a path without relocations Andrzej Turko
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Andrzej Turko @ 2021-08-11 20:28 UTC (permalink / raw)
To: igt-dev; +Cc: Andrzej Turko, Zbigniew Kempczyński
Adapt gem_exec_big to run also on platforms
with disabled relocation support.
Cc: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
Andrzej Turko (2):
tests/i915/gem_exec_big: Add a path without relocations
HAX: Run gem_exec_big
tests/i915/gem_exec_big.c | 40 ++++++
tests/intel-ci/fast-feedback.testlist | 175 +-------------------------
2 files changed, 43 insertions(+), 172 deletions(-)
--
2.25.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_exec_big: Add a path without relocations
2021-08-11 20:28 [igt-dev] [PATCH i-g-t 0/2] gem_exec_big -- noreloc support Andrzej Turko
@ 2021-08-11 20:28 ` Andrzej Turko
2021-08-12 3:37 ` Zbigniew Kempczyński
2021-08-11 20:28 ` [igt-dev] [PATCH i-g-t 2/2] HAX: Run gem_exec_big Andrzej Turko
2021-08-11 22:51 ` [igt-dev] ✗ Fi.CI.BAT: failure for gem_exec_big -- noreloc support Patchwork
2 siblings, 1 reply; 5+ messages in thread
From: Andrzej Turko @ 2021-08-11 20:28 UTC (permalink / raw)
To: igt-dev; +Cc: Andrzej Turko, Zbigniew Kempczyński
When running on platforms without relocation support,
verification of their correctness should be skipped.
What remains is exercising submission of large batches.
Signed-off-by: Andrzej Turko <andrzej.turko@linux.intel.com>
Cc: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
---
tests/i915/gem_exec_big.c | 40 +++++++++++++++++++++++++++++++++++++++
1 file changed, 40 insertions(+)
diff --git a/tests/i915/gem_exec_big.c b/tests/i915/gem_exec_big.c
index 1f8c720b6..a10b7dc6a 100644
--- a/tests/i915/gem_exec_big.c
+++ b/tests/i915/gem_exec_big.c
@@ -55,6 +55,35 @@ IGT_TEST_DESCRIPTION("Run a large nop batch to stress test the error capture"
#define FORCE_PREAD_PWRITE 0
static int use_64bit_relocs;
+static bool do_relocs;
+
+static void exec_noreloc(int fd, uint64_t handle, unsigned flags)
+{
+ struct drm_i915_gem_execbuffer2 execbuf;
+ struct drm_i915_gem_exec_object2 gem_exec[1];
+
+ gem_exec[0].handle = handle;
+ gem_exec[0].relocation_count = 0;
+ gem_exec[0].alignment = 0;
+ gem_exec[0].offset = 0;
+ gem_exec[0].flags = EXEC_OBJECT_SUPPORTS_48B_ADDRESS;
+ gem_exec[0].rsvd1 = 0;
+ gem_exec[0].rsvd2 = 0;
+
+ execbuf.buffers_ptr = to_user_pointer(gem_exec);
+ execbuf.buffer_count = 1;
+ execbuf.batch_start_offset = 0;
+ execbuf.batch_len = 8;
+ execbuf.cliprects_ptr = 0;
+ execbuf.num_cliprects = 0;
+ execbuf.DR1 = 0;
+ execbuf.DR4 = 0;
+ execbuf.flags = flags;
+ i915_execbuffer2_set_context_id(execbuf, 0);
+ execbuf.rsvd2 = 0;
+
+ gem_execbuf(fd, &execbuf);
+}
static void exec1(int fd, uint32_t handle, uint64_t reloc_ofs, unsigned flags, char *ptr)
{
@@ -62,6 +91,11 @@ static void exec1(int fd, uint32_t handle, uint64_t reloc_ofs, unsigned flags, c
struct drm_i915_gem_exec_object2 gem_exec[1];
struct drm_i915_gem_relocation_entry gem_reloc[1];
+ if (!do_relocs) {
+ exec_noreloc(fd, handle, flags);
+ return;
+ }
+
gem_reloc[0].offset = reloc_ofs;
gem_reloc[0].delta = 0;
gem_reloc[0].target_handle = handle;
@@ -137,6 +171,11 @@ static void execN(int fd, uint32_t handle, uint64_t batch_size, unsigned flags,
struct drm_i915_gem_relocation_entry *gem_reloc;
uint64_t n, nreloc = batch_size >> 12;
+ if (!do_relocs) {
+ exec_noreloc(fd, handle, flags);
+ return;
+ }
+
gem_reloc = calloc(nreloc, sizeof(*gem_reloc));
igt_assert(gem_reloc);
@@ -305,6 +344,7 @@ igt_main
igt_require_gem(i915);
use_64bit_relocs = intel_gen(intel_get_drm_devid(i915)) >= 8;
+ do_relocs = gem_has_relocations(i915);
}
igt_subtest("single")
--
2.25.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [igt-dev] [PATCH i-g-t 2/2] HAX: Run gem_exec_big
2021-08-11 20:28 [igt-dev] [PATCH i-g-t 0/2] gem_exec_big -- noreloc support Andrzej Turko
2021-08-11 20:28 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_exec_big: Add a path without relocations Andrzej Turko
@ 2021-08-11 20:28 ` Andrzej Turko
2021-08-11 22:51 ` [igt-dev] ✗ Fi.CI.BAT: failure for gem_exec_big -- noreloc support Patchwork
2 siblings, 0 replies; 5+ messages in thread
From: Andrzej Turko @ 2021-08-11 20:28 UTC (permalink / raw)
To: igt-dev; +Cc: Andrzej Turko
Signed-off-by: Andrzej Turko <andrzej.turko@linux.intel.com>
---
tests/intel-ci/fast-feedback.testlist | 175 +-------------------------
1 file changed, 3 insertions(+), 172 deletions(-)
diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist
index fa5006d2e..8227934a4 100644
--- a/tests/intel-ci/fast-feedback.testlist
+++ b/tests/intel-ci/fast-feedback.testlist
@@ -1,175 +1,6 @@
# Keep alphabetically sorted by default
-igt@core_auth@basic-auth
-igt@debugfs_test@read_all_entries
-igt@fbdev@eof
-igt@fbdev@info
-igt@fbdev@nullptr
-igt@fbdev@read
-igt@fbdev@write
-igt@gem_basic@bad-close
-igt@gem_basic@create-close
-igt@gem_basic@create-fd-close
-igt@gem_busy@busy@all
-igt@gem_close_race@basic-process
-igt@gem_close_race@basic-threads
-igt@gem_ctx_create@basic
-igt@gem_ctx_create@basic-files
-igt@gem_ctx_exec@basic
-igt@gem_exec_basic@basic
-igt@gem_exec_create@basic
-igt@gem_exec_fence@basic-busy
-igt@gem_exec_fence@basic-wait
-igt@gem_exec_fence@basic-await
-igt@gem_exec_fence@nb-await
-igt@gem_exec_gttfill@basic
-igt@gem_exec_parallel@engines
-igt@gem_exec_store@basic
-igt@gem_exec_suspend@basic-s0
-igt@gem_exec_suspend@basic-s3
-igt@gem_flink_basic@bad-flink
-igt@gem_flink_basic@bad-open
-igt@gem_flink_basic@basic
-igt@gem_flink_basic@double-flink
-igt@gem_flink_basic@flink-lifetime
-igt@gem_huc_copy@huc-copy
-igt@gem_linear_blits@basic
-igt@gem_mmap@basic
-igt@gem_mmap_gtt@basic
-igt@gem_render_linear_blits@basic
-igt@gem_render_tiled_blits@basic
-igt@gem_ringfill@basic-all
-igt@gem_softpin@allocator-basic
-igt@gem_softpin@allocator-basic-reserve
-igt@gem_sync@basic-all
-igt@gem_sync@basic-each
-igt@gem_tiled_blits@basic
-igt@gem_tiled_fence_blits@basic
-igt@gem_tiled_pread_basic
-igt@gem_wait@busy@all
-igt@gem_wait@wait@all
-igt@i915_getparams_basic@basic-eu-total
-igt@i915_getparams_basic@basic-subslice-total
-igt@i915_hangman@error-state-basic
-igt@kms_addfb_basic@addfb25-bad-modifier
-igt@kms_addfb_basic@addfb25-framebuffer-vs-set-tiling
-igt@kms_addfb_basic@addfb25-modifier-no-flag
-igt@kms_addfb_basic@addfb25-x-tiled-legacy
-igt@kms_addfb_basic@addfb25-x-tiled-mismatch-legacy
-igt@kms_addfb_basic@addfb25-yf-tiled-legacy
-igt@kms_addfb_basic@addfb25-y-tiled-legacy
-igt@kms_addfb_basic@addfb25-y-tiled-small-legacy
-igt@kms_addfb_basic@bad-pitch-0
-igt@kms_addfb_basic@bad-pitch-1024
-igt@kms_addfb_basic@bad-pitch-128
-igt@kms_addfb_basic@bad-pitch-256
-igt@kms_addfb_basic@bad-pitch-32
-igt@kms_addfb_basic@bad-pitch-63
-igt@kms_addfb_basic@bad-pitch-65536
-igt@kms_addfb_basic@bad-pitch-999
-igt@kms_addfb_basic@basic
-igt@kms_addfb_basic@basic-x-tiled-legacy
-igt@kms_addfb_basic@basic-y-tiled-legacy
-igt@kms_addfb_basic@bo-too-small
-igt@kms_addfb_basic@bo-too-small-due-to-tiling
-igt@kms_addfb_basic@clobberred-modifier
-igt@kms_addfb_basic@framebuffer-vs-set-tiling
-igt@kms_addfb_basic@invalid-get-prop
-igt@kms_addfb_basic@invalid-get-prop-any
-igt@kms_addfb_basic@invalid-set-prop
-igt@kms_addfb_basic@invalid-set-prop-any
-igt@kms_addfb_basic@no-handle
-igt@kms_addfb_basic@size-max
-igt@kms_addfb_basic@small-bo
-igt@kms_addfb_basic@tile-pitch-mismatch
-igt@kms_addfb_basic@too-high
-igt@kms_addfb_basic@too-wide
-igt@kms_addfb_basic@unused-handle
-igt@kms_addfb_basic@unused-modifier
-igt@kms_addfb_basic@unused-offsets
-igt@kms_addfb_basic@unused-pitches
-igt@kms_busy@basic
-igt@kms_chamelium@dp-hpd-fast
-igt@kms_chamelium@dp-edid-read
-igt@kms_chamelium@dp-crc-fast
-igt@kms_chamelium@hdmi-hpd-fast
-igt@kms_chamelium@hdmi-edid-read
-igt@kms_chamelium@hdmi-crc-fast
-igt@kms_chamelium@vga-hpd-fast
-igt@kms_chamelium@vga-edid-read
-igt@kms_chamelium@common-hpd-after-suspend
-igt@kms_prop_blob@basic
-igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic
-igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy
-igt@kms_cursor_legacy@basic-flip-after-cursor-atomic
-igt@kms_cursor_legacy@basic-flip-after-cursor-legacy
-igt@kms_cursor_legacy@basic-flip-after-cursor-varying-size
-igt@kms_cursor_legacy@basic-flip-before-cursor-atomic
-igt@kms_cursor_legacy@basic-flip-before-cursor-legacy
-igt@kms_cursor_legacy@basic-flip-before-cursor-varying-size
-igt@kms_flip@basic-flip-vs-dpms
-igt@kms_flip@basic-flip-vs-modeset
-igt@kms_flip@basic-flip-vs-wf_vblank
-igt@kms_flip@basic-plain-flip
-igt@kms_force_connector_basic@force-connector-state
-igt@kms_force_connector_basic@force-edid
-igt@kms_force_connector_basic@force-load-detect
-igt@kms_force_connector_basic@prune-stale-modes
-igt@kms_frontbuffer_tracking@basic
-igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-a
-igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-b
-igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-c
-igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d
-igt@kms_pipe_crc_basic@hang-read-crc-pipe-a
-igt@kms_pipe_crc_basic@nonblocking-crc-pipe-a
-igt@kms_pipe_crc_basic@nonblocking-crc-pipe-a-frame-sequence
-igt@kms_pipe_crc_basic@read-crc-pipe-a
-igt@kms_pipe_crc_basic@read-crc-pipe-b
-igt@kms_pipe_crc_basic@read-crc-pipe-c
-igt@kms_pipe_crc_basic@read-crc-pipe-a-frame-sequence
-igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a
-igt@kms_psr@primary_page_flip
-igt@kms_psr@cursor_plane_move
-igt@kms_psr@sprite_plane_onoff
-igt@kms_psr@primary_mmap_gtt
-igt@kms_setmode@basic-clone-single-crtc
-igt@i915_pm_backlight@basic-brightness
-igt@i915_pm_rpm@basic-pci-d3-state
-igt@i915_pm_rpm@basic-rte
-igt@i915_pm_rps@basic-api
-igt@prime_self_import@basic-llseek-bad
-igt@prime_self_import@basic-llseek-size
-igt@prime_self_import@basic-with_fd_dup
-igt@prime_self_import@basic-with_one_bo
-igt@prime_self_import@basic-with_one_bo_two_files
-igt@prime_self_import@basic-with_two_bos
-igt@prime_vgem@basic-fence-flip
-igt@prime_vgem@basic-fence-mmap
-igt@prime_vgem@basic-fence-read
-igt@prime_vgem@basic-gtt
-igt@prime_vgem@basic-read
-igt@prime_vgem@basic-write
-igt@prime_vgem@basic-userptr
-igt@vgem_basic@setversion
-igt@vgem_basic@create
-igt@vgem_basic@debugfs
-igt@vgem_basic@dmabuf-export
-igt@vgem_basic@dmabuf-fence
-igt@vgem_basic@dmabuf-fence-before
-igt@vgem_basic@dmabuf-mmap
-igt@vgem_basic@mmap
-igt@vgem_basic@second-client
-igt@vgem_basic@sysfs
-# All tests that do module unloading and reloading are executed last.
-# They will sometimes reveal issues of earlier tests leaving the
-# driver in a broken state that is not otherwise noticed in that test.
-
-igt@core_hotunplug@unbind-rebind
-igt@vgem_basic@unload
-igt@i915_module_load@reload
-igt@i915_pm_rpm@module-reload
-
-# Kernel selftests
-igt@i915_selftest@live
-igt@dmabuf@all
+igt@gem_exec_big@single
+igt@gem_exec_big@exhaustive
+igt@meta_test@fail-result
--
2.25.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [igt-dev] ✗ Fi.CI.BAT: failure for gem_exec_big -- noreloc support
2021-08-11 20:28 [igt-dev] [PATCH i-g-t 0/2] gem_exec_big -- noreloc support Andrzej Turko
2021-08-11 20:28 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_exec_big: Add a path without relocations Andrzej Turko
2021-08-11 20:28 ` [igt-dev] [PATCH i-g-t 2/2] HAX: Run gem_exec_big Andrzej Turko
@ 2021-08-11 22:51 ` Patchwork
2 siblings, 0 replies; 5+ messages in thread
From: Patchwork @ 2021-08-11 22:51 UTC (permalink / raw)
To: Andrzej Turko; +Cc: igt-dev
[-- Attachment #1: Type: text/plain, Size: 13243 bytes --]
== Series Details ==
Series: gem_exec_big -- noreloc support
URL : https://patchwork.freedesktop.org/series/93610/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_10470 -> IGTPW_6117
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with IGTPW_6117 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in IGTPW_6117, 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_6117/index.html
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in IGTPW_6117:
### IGT changes ###
#### Possible regressions ####
* {igt@gem_exec_big@exhaustive} (NEW):
- {fi-ehl-2}: NOTRUN -> [SKIP][1]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-ehl-2/igt@gem_exec_big@exhaustive.html
- {fi-tgl-dsi}: NOTRUN -> [SKIP][2]
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-tgl-dsi/igt@gem_exec_big@exhaustive.html
- fi-icl-u2: NOTRUN -> [SKIP][3]
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-icl-u2/igt@gem_exec_big@exhaustive.html
- fi-elk-e7500: NOTRUN -> [TIMEOUT][4]
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-elk-e7500/igt@gem_exec_big@exhaustive.html
- fi-icl-y: NOTRUN -> [SKIP][5]
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-icl-y/igt@gem_exec_big@exhaustive.html
- fi-ilk-650: NOTRUN -> [TIMEOUT][6]
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-ilk-650/igt@gem_exec_big@exhaustive.html
- fi-tgl-1115g4: NOTRUN -> [SKIP][7]
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-tgl-1115g4/igt@gem_exec_big@exhaustive.html
- fi-hsw-4770: NOTRUN -> [FAIL][8] +1 similar issue
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-hsw-4770/igt@gem_exec_big@exhaustive.html
- fi-rkl-guc: NOTRUN -> [SKIP][9]
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-rkl-guc/igt@gem_exec_big@exhaustive.html
- fi-cml-u2: NOTRUN -> [SKIP][10]
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-cml-u2/igt@gem_exec_big@exhaustive.html
- {fi-jsl-1}: NOTRUN -> [SKIP][11]
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-jsl-1/igt@gem_exec_big@exhaustive.html
- fi-bwr-2160: NOTRUN -> [TIMEOUT][12]
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-bwr-2160/igt@gem_exec_big@exhaustive.html
- fi-pnv-d510: NOTRUN -> [TIMEOUT][13]
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-pnv-d510/igt@gem_exec_big@exhaustive.html
* igt@gem_exec_big@single:
- fi-apl-guc: NOTRUN -> [TIMEOUT][14]
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-apl-guc/igt@gem_exec_big@single.html
* igt@meta_test@fail-result (NEW):
- fi-pnv-d510: NOTRUN -> [FAIL][15]
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-pnv-d510/igt@meta_test@fail-result.html
- {fi-jsl-1}: NOTRUN -> [FAIL][16]
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-jsl-1/igt@meta_test@fail-result.html
- fi-icl-u2: NOTRUN -> [FAIL][17]
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-icl-u2/igt@meta_test@fail-result.html
- fi-cml-u2: NOTRUN -> [FAIL][18]
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-cml-u2/igt@meta_test@fail-result.html
- fi-cfl-8700k: NOTRUN -> [FAIL][19]
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-cfl-8700k/igt@meta_test@fail-result.html
- {fi-hsw-gt1}: NOTRUN -> [FAIL][20] +1 similar issue
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-hsw-gt1/igt@meta_test@fail-result.html
- {fi-tgl-dsi}: NOTRUN -> [FAIL][21]
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-tgl-dsi/igt@meta_test@fail-result.html
- fi-snb-2520m: NOTRUN -> [FAIL][22] +1 similar issue
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-snb-2520m/igt@meta_test@fail-result.html
- fi-ivb-3770: NOTRUN -> [FAIL][23] +1 similar issue
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-ivb-3770/igt@meta_test@fail-result.html
- fi-apl-guc: NOTRUN -> [FAIL][24]
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-apl-guc/igt@meta_test@fail-result.html
- fi-bxt-dsi: NOTRUN -> [FAIL][25]
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-bxt-dsi/igt@meta_test@fail-result.html
- fi-elk-e7500: NOTRUN -> [FAIL][26]
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-elk-e7500/igt@meta_test@fail-result.html
- fi-skl-6700k2: NOTRUN -> [FAIL][27]
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-skl-6700k2/igt@meta_test@fail-result.html
- fi-tgl-1115g4: NOTRUN -> [FAIL][28]
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-tgl-1115g4/igt@meta_test@fail-result.html
- fi-kbl-soraka: NOTRUN -> [FAIL][29]
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-kbl-soraka/igt@meta_test@fail-result.html
- fi-cfl-guc: NOTRUN -> [FAIL][30]
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-cfl-guc/igt@meta_test@fail-result.html
- fi-ilk-650: NOTRUN -> [FAIL][31]
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-ilk-650/igt@meta_test@fail-result.html
- fi-rkl-guc: NOTRUN -> [FAIL][32]
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-rkl-guc/igt@meta_test@fail-result.html
- fi-bsw-kefka: NOTRUN -> [FAIL][33] +1 similar issue
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-bsw-kefka/igt@meta_test@fail-result.html
- fi-kbl-x1275: NOTRUN -> [FAIL][34]
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-kbl-x1275/igt@meta_test@fail-result.html
- fi-cfl-8109u: NOTRUN -> [FAIL][35]
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-cfl-8109u/igt@meta_test@fail-result.html
- fi-icl-y: NOTRUN -> [FAIL][36]
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-icl-y/igt@meta_test@fail-result.html
- fi-bdw-5557u: NOTRUN -> [FAIL][37]
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-bdw-5557u/igt@meta_test@fail-result.html
- fi-glk-dsi: NOTRUN -> [FAIL][38]
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-glk-dsi/igt@meta_test@fail-result.html
- fi-bsw-nick: NOTRUN -> [FAIL][39]
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-bsw-nick/igt@meta_test@fail-result.html
- fi-kbl-7567u: NOTRUN -> [FAIL][40]
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-kbl-7567u/igt@meta_test@fail-result.html
- fi-kbl-8809g: NOTRUN -> [FAIL][41]
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-kbl-8809g/igt@meta_test@fail-result.html
- fi-bwr-2160: NOTRUN -> [FAIL][42]
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-bwr-2160/igt@meta_test@fail-result.html
- fi-skl-guc: NOTRUN -> [FAIL][43]
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-skl-guc/igt@meta_test@fail-result.html
- {fi-ehl-2}: NOTRUN -> [FAIL][44]
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-ehl-2/igt@meta_test@fail-result.html
- fi-kbl-guc: NOTRUN -> [FAIL][45]
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-kbl-guc/igt@meta_test@fail-result.html
- fi-bdw-gvtdvm: NOTRUN -> [FAIL][46]
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-bdw-gvtdvm/igt@meta_test@fail-result.html
- fi-kbl-7500u: NOTRUN -> [FAIL][47]
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-kbl-7500u/igt@meta_test@fail-result.html
#### Suppressed ####
The following results come from untrusted machines, tests, or statuses.
They do not affect the overall result.
* igt@gem_exec_big@single:
- {fi-tgl-dsi}: NOTRUN -> [TIMEOUT][48]
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-tgl-dsi/igt@gem_exec_big@single.html
New tests
---------
New tests have been introduced between CI_DRM_10470 and IGTPW_6117:
### New IGT tests (2) ###
* igt@gem_exec_big@exhaustive:
- Statuses : 5 fail(s) 26 skip(s) 4 timeout(s)
- Exec time: [0.05, 121.87] s
* igt@meta_test@fail-result:
- Statuses : 35 fail(s)
- Exec time: [0.01, 0.14] s
Known issues
------------
Here are the changes found in IGTPW_6117 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@amdgpu/amd_cs_nop@sync-compute0:
- fi-bdw-gvtdvm: NOTRUN -> [SKIP][49] ([fdo#109271]) +18 similar issues
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-bdw-gvtdvm/igt@amdgpu/amd_cs_nop@sync-compute0.html
* {igt@gem_exec_big@exhaustive} (NEW):
- fi-apl-guc: NOTRUN -> [SKIP][50] ([fdo#109271]) +18 similar issues
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-apl-guc/igt@gem_exec_big@exhaustive.html
- fi-cfl-8700k: NOTRUN -> [SKIP][51] ([fdo#109271])
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-cfl-8700k/igt@gem_exec_big@exhaustive.html
- fi-skl-6700k2: NOTRUN -> [SKIP][52] ([fdo#109271])
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-skl-6700k2/igt@gem_exec_big@exhaustive.html
- fi-skl-guc: NOTRUN -> [SKIP][53] ([fdo#109271])
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-skl-guc/igt@gem_exec_big@exhaustive.html
- fi-cfl-guc: NOTRUN -> [SKIP][54] ([fdo#109271])
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-cfl-guc/igt@gem_exec_big@exhaustive.html
- fi-kbl-7567u: NOTRUN -> [SKIP][55] ([fdo#109271])
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-kbl-7567u/igt@gem_exec_big@exhaustive.html
- fi-bxt-dsi: NOTRUN -> [SKIP][56] ([fdo#109271])
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-bxt-dsi/igt@gem_exec_big@exhaustive.html
- {fi-dg1-1}: NOTRUN -> [SKIP][57] ([i915#3717])
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-dg1-1/igt@gem_exec_big@exhaustive.html
- fi-kbl-8809g: NOTRUN -> [SKIP][58] ([fdo#109271])
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-kbl-8809g/igt@gem_exec_big@exhaustive.html
- fi-kbl-7500u: NOTRUN -> [SKIP][59] ([fdo#109271])
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-kbl-7500u/igt@gem_exec_big@exhaustive.html
- fi-bsw-nick: NOTRUN -> [SKIP][60] ([fdo#109271])
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-bsw-nick/igt@gem_exec_big@exhaustive.html
- fi-cfl-8109u: NOTRUN -> [SKIP][61] ([fdo#109271])
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-cfl-8109u/igt@gem_exec_big@exhaustive.html
- fi-kbl-guc: NOTRUN -> [SKIP][62] ([fdo#109271])
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-kbl-guc/igt@gem_exec_big@exhaustive.html
- fi-kbl-x1275: NOTRUN -> [SKIP][63] ([fdo#109271])
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-kbl-x1275/igt@gem_exec_big@exhaustive.html
- fi-kbl-soraka: NOTRUN -> [SKIP][64] ([fdo#109271])
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-kbl-soraka/igt@gem_exec_big@exhaustive.html
- fi-glk-dsi: NOTRUN -> [SKIP][65] ([fdo#109271])
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-glk-dsi/igt@gem_exec_big@exhaustive.html
- fi-bdw-5557u: NOTRUN -> [SKIP][66] ([fdo#109271])
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-bdw-5557u/igt@gem_exec_big@exhaustive.html
* igt@meta_test@fail-result (NEW):
- {fi-dg1-1}: NOTRUN -> [FAIL][67] ([i915#3717])
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/fi-dg1-1/igt@meta_test@fail-result.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[i915#3717]: https://gitlab.freedesktop.org/drm/intel/issues/3717
Participating hosts (40 -> 35)
------------------------------
Missing (5): fi-ilk-m540 fi-hsw-4200u fi-bsw-cyan bat-jsl-1 fi-bdw-samus
Build changes
-------------
* CI: CI-20190529 -> None
* IGT: IGT_6170 -> IGTPW_6117
CI-20190529: 20190529
CI_DRM_10470: 8ef1164291b6c56baccf77f4be6e99a86821c684 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_6117: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/index.html
IGT_6170: d4ff2b4ab88d62a1888cb0316e70a1729fe4a685 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6117/index.html
[-- Attachment #2: Type: text/html, Size: 16081 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_exec_big: Add a path without relocations
2021-08-11 20:28 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_exec_big: Add a path without relocations Andrzej Turko
@ 2021-08-12 3:37 ` Zbigniew Kempczyński
0 siblings, 0 replies; 5+ messages in thread
From: Zbigniew Kempczyński @ 2021-08-12 3:37 UTC (permalink / raw)
To: Andrzej Turko; +Cc: igt-dev
On Wed, Aug 11, 2021 at 10:28:10PM +0200, Andrzej Turko wrote:
> When running on platforms without relocation support,
> verification of their correctness should be skipped.
> What remains is exercising submission of large batches.
>
> Signed-off-by: Andrzej Turko <andrzej.turko@linux.intel.com>
> Cc: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
> ---
> tests/i915/gem_exec_big.c | 40 +++++++++++++++++++++++++++++++++++++++
> 1 file changed, 40 insertions(+)
>
> diff --git a/tests/i915/gem_exec_big.c b/tests/i915/gem_exec_big.c
> index 1f8c720b6..a10b7dc6a 100644
> --- a/tests/i915/gem_exec_big.c
> +++ b/tests/i915/gem_exec_big.c
> @@ -55,6 +55,35 @@ IGT_TEST_DESCRIPTION("Run a large nop batch to stress test the error capture"
> #define FORCE_PREAD_PWRITE 0
>
> static int use_64bit_relocs;
> +static bool do_relocs;
> +
> +static void exec_noreloc(int fd, uint64_t handle, unsigned flags)
> +{
> + struct drm_i915_gem_execbuffer2 execbuf;
> + struct drm_i915_gem_exec_object2 gem_exec[1];
> +
> + gem_exec[0].handle = handle;
> + gem_exec[0].relocation_count = 0;
> + gem_exec[0].alignment = 0;
> + gem_exec[0].offset = 0;
> + gem_exec[0].flags = EXEC_OBJECT_SUPPORTS_48B_ADDRESS;
> + gem_exec[0].rsvd1 = 0;
> + gem_exec[0].rsvd2 = 0;
> +
> + execbuf.buffers_ptr = to_user_pointer(gem_exec);
> + execbuf.buffer_count = 1;
> + execbuf.batch_start_offset = 0;
> + execbuf.batch_len = 8;
> + execbuf.cliprects_ptr = 0;
> + execbuf.num_cliprects = 0;
> + execbuf.DR1 = 0;
> + execbuf.DR4 = 0;
> + execbuf.flags = flags;
> + i915_execbuffer2_set_context_id(execbuf, 0);
> + execbuf.rsvd2 = 0;
> +
> + gem_execbuf(fd, &execbuf);
> +}
That makes code duplication I don't really like. Maybe there's less conditional code
which looks worse but we have no ~20 lines of code which differs with single relocation_count
value.
--
Zbigniew
>
> static void exec1(int fd, uint32_t handle, uint64_t reloc_ofs, unsigned flags, char *ptr)
> {
> @@ -62,6 +91,11 @@ static void exec1(int fd, uint32_t handle, uint64_t reloc_ofs, unsigned flags, c
> struct drm_i915_gem_exec_object2 gem_exec[1];
> struct drm_i915_gem_relocation_entry gem_reloc[1];
>
> + if (!do_relocs) {
> + exec_noreloc(fd, handle, flags);
> + return;
> + }
> +
> gem_reloc[0].offset = reloc_ofs;
> gem_reloc[0].delta = 0;
> gem_reloc[0].target_handle = handle;
> @@ -137,6 +171,11 @@ static void execN(int fd, uint32_t handle, uint64_t batch_size, unsigned flags,
> struct drm_i915_gem_relocation_entry *gem_reloc;
> uint64_t n, nreloc = batch_size >> 12;
>
> + if (!do_relocs) {
> + exec_noreloc(fd, handle, flags);
> + return;
> + }
> +
> gem_reloc = calloc(nreloc, sizeof(*gem_reloc));
> igt_assert(gem_reloc);
>
> @@ -305,6 +344,7 @@ igt_main
> igt_require_gem(i915);
>
> use_64bit_relocs = intel_gen(intel_get_drm_devid(i915)) >= 8;
> + do_relocs = gem_has_relocations(i915);
> }
>
> igt_subtest("single")
> --
> 2.25.1
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-08-12 3:37 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-11 20:28 [igt-dev] [PATCH i-g-t 0/2] gem_exec_big -- noreloc support Andrzej Turko
2021-08-11 20:28 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_exec_big: Add a path without relocations Andrzej Turko
2021-08-12 3:37 ` Zbigniew Kempczyński
2021-08-11 20:28 ` [igt-dev] [PATCH i-g-t 2/2] HAX: Run gem_exec_big Andrzej Turko
2021-08-11 22:51 ` [igt-dev] ✗ Fi.CI.BAT: failure for gem_exec_big -- noreloc support Patchwork
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.