* [igt-dev] [PATCH i-g-t 0/2] Add two gem-ccs subtests
@ 2022-03-21 11:11 Zbigniew Kempczyński
2022-03-21 11:11 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_ccs: Check ctrl-surf-copy in new context Zbigniew Kempczyński
` (3 more replies)
0 siblings, 4 replies; 16+ messages in thread
From: Zbigniew Kempczyński @ 2022-03-21 11:11 UTC (permalink / raw)
To: igt-dev
Introduce two new subtests - ctrl-surf-copy-new-ctx + suspend-resume.
ctrl-surf-copy-new-ctx checks ccs data is binded to physical mem.
suspend-resume checks S0 suspend state.
v2 Adding separate allocator handle for new context (Zbigniew)
v3 Address review comments (Kamil)
v4 Remove memory leak (Kamil)
Apoorva Singh (2):
tests/i915/gem_ccs: Check ctrl-surf-copy in new context
tests/i915/gem_ccs: Add suspend-resume subtest
tests/i915/gem_ccs.c | 98 +++++++++++++++++++++++++++++++++++++-------
1 file changed, 83 insertions(+), 15 deletions(-)
--
2.32.0
^ permalink raw reply [flat|nested] 16+ messages in thread
* [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_ccs: Check ctrl-surf-copy in new context
2022-03-21 11:11 [igt-dev] [PATCH i-g-t 0/2] Add two gem-ccs subtests Zbigniew Kempczyński
@ 2022-03-21 11:11 ` Zbigniew Kempczyński
2022-03-21 16:00 ` Kamil Konieczny
2022-03-21 11:11 ` [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_ccs: Add suspend-resume subtest Zbigniew Kempczyński
` (2 subsequent siblings)
3 siblings, 1 reply; 16+ messages in thread
From: Zbigniew Kempczyński @ 2022-03-21 11:11 UTC (permalink / raw)
To: igt-dev
From: Apoorva Singh <apoorva1.singh@intel.com>
Verify ccs data is binded to physical memory by using
XY_CTRL_SURF_COPY_BLT command in separate context.
v2: add separate allocator handle for new context (Zbigniew)
Signed-off-by: Apoorva Singh <apoorva1.singh@intel.com>
Cc: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
---
tests/i915/gem_ccs.c | 50 ++++++++++++++++++++++++++++++++++----------
1 file changed, 39 insertions(+), 11 deletions(-)
diff --git a/tests/i915/gem_ccs.c b/tests/i915/gem_ccs.c
index fdf1fe75f5..7e104f6ad8 100644
--- a/tests/i915/gem_ccs.c
+++ b/tests/i915/gem_ccs.c
@@ -39,6 +39,7 @@ struct test_config {
bool compression;
bool inplace;
bool surfcopy;
+ bool new_ctx;
};
static void set_object(struct blt_copy_object *obj,
@@ -234,9 +235,8 @@ static void block_copy(int i915,
const intel_ctx_t *ctx,
const struct intel_execution_engine2 *e,
uint32_t region1, uint32_t region2,
- enum blt_tiling mid_tiling, bool compression,
- bool inplace,
- bool surfcopy)
+ enum blt_tiling mid_tiling,
+ const struct test_config *config)
{
struct blt_copy_data blt = {};
struct blt_block_copy_data_ext ext = {}, *pext = &ext;
@@ -249,7 +249,7 @@ static void block_copy(int i915,
uint32_t run_id = mid_tiling;
uint32_t mid_region = region2, bb;
uint32_t width = param.width, height = param.height;
- enum blt_compression mid_compression = compression;
+ enum blt_compression mid_compression = config->compression;
int mid_compression_format = param.compression_format;
enum blt_compression_type comp_type = COMPRESSION_TYPE_3D;
uint8_t uc_mocs = intel_get_uc_mocs(i915);
@@ -293,8 +293,31 @@ static void block_copy(int i915,
WRITE_PNG(i915, run_id, "src", &blt.src, width, height);
WRITE_PNG(i915, run_id, "mid", &blt.dst, width, height);
- if (surfcopy && pext)
- surf_copy(i915, ctx, e, ahnd, src, mid, dst, run_id);
+ if (config->surfcopy && pext) {
+ const intel_ctx_t *surf_ctx = ctx;
+ uint64_t surf_ahnd = ahnd;
+ struct intel_execution_engine2 surf_e = *e;
+
+ if (config->new_ctx) {
+ intel_ctx_cfg_t cfg = {};
+
+ cfg.num_engines = 1;
+ cfg.engines[0].engine_class = e->class;
+ cfg.engines[0].engine_instance = e->instance;
+ surf_ctx = intel_ctx_create(i915, &cfg);
+ surf_e.flags = 0;
+ ahnd = intel_allocator_open_full(i915, surf_ctx->id, 0, 0,
+ INTEL_ALLOCATOR_SIMPLE,
+ ALLOC_STRATEGY_LOW_TO_HIGH, 0);
+ }
+
+ surf_copy(i915, surf_ctx, &surf_e, surf_ahnd, src, mid, dst, run_id);
+
+ if (surf_ctx != ctx) {
+ intel_ctx_destroy(i915, surf_ctx);
+ put_ahnd(surf_ahnd);
+ }
+ }
memset(&blt, 0, sizeof(blt));
blt.color_depth = CD_32bit;
@@ -303,7 +326,7 @@ static void block_copy(int i915,
set_blt_object(&blt.dst, dst);
set_object_ext(&ext.src, mid_compression_format, width, height, SURFACE_TYPE_2D);
set_object_ext(&ext.dst, 0, width, height, SURFACE_TYPE_2D);
- if (inplace) {
+ if (config->inplace) {
set_object(&blt.dst, mid->handle, dst->size, mid->region, 0,
T_LINEAR, COMPRESSION_DISABLED, comp_type);
blt.dst.ptr = mid->ptr;
@@ -367,10 +390,7 @@ static void block_copy_test(int i915,
param.compression_format, regtxt) {
block_copy(i915, ctx, e,
region1, region2,
- tiling,
- config->compression,
- config->inplace,
- config->surfcopy);
+ tiling, config);
}
free(regtxt);
}
@@ -480,6 +500,14 @@ igt_main_args("bf:pst:W:H:", NULL, help_str, opt_handler, NULL)
block_copy_test(i915, &config, ctx, set);
}
+ igt_subtest_with_dynamic("ctrl-surf-copy-new-ctx") {
+ struct test_config config = { .compression = true,
+ .surfcopy = true,
+ .new_ctx = true };
+
+ block_copy_test(i915, &config, ctx, set);
+ }
+
igt_fixture {
igt_disallow_hang(i915, hang);
close(i915);
--
2.32.0
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_ccs: Add suspend-resume subtest
2022-03-21 11:11 [igt-dev] [PATCH i-g-t 0/2] Add two gem-ccs subtests Zbigniew Kempczyński
2022-03-21 11:11 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_ccs: Check ctrl-surf-copy in new context Zbigniew Kempczyński
@ 2022-03-21 11:11 ` Zbigniew Kempczyński
2022-03-21 16:04 ` Kamil Konieczny
2022-03-21 12:20 ` [igt-dev] ✓ Fi.CI.BAT: success for Add two gem-ccs subtests (rev4) Patchwork
2022-03-21 15:00 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
3 siblings, 1 reply; 16+ messages in thread
From: Zbigniew Kempczyński @ 2022-03-21 11:11 UTC (permalink / raw)
To: igt-dev
From: Apoorva Singh <apoorva1.singh@intel.com>
Verify flatccs data won't be corrupted when device will be put
to S0 (s2idle) state.
v2: addressing review comments (Kamil)
v3: fix memory leak (Kamil)
Signed-off-by: Apoorva Singh <apoorva1.singh@intel.com>
Cc: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
---
tests/i915/gem_ccs.c | 50 +++++++++++++++++++++++++++++++++++++++-----
1 file changed, 45 insertions(+), 5 deletions(-)
diff --git a/tests/i915/gem_ccs.c b/tests/i915/gem_ccs.c
index 7e104f6ad8..1f8f98ce62 100644
--- a/tests/i915/gem_ccs.c
+++ b/tests/i915/gem_ccs.c
@@ -4,6 +4,7 @@
*/
#include <errno.h>
+#include <glib.h>
#include <sys/ioctl.h>
#include <sys/time.h>
#include <malloc.h>
@@ -40,6 +41,7 @@ struct test_config {
bool inplace;
bool surfcopy;
bool new_ctx;
+ bool suspend_resume;
};
static void set_object(struct blt_copy_object *obj,
@@ -162,22 +164,23 @@ static void surf_copy(int i915,
const struct blt_copy_object *src,
const struct blt_copy_object *mid,
const struct blt_copy_object *dst,
- int run_id)
+ int run_id, bool suspend_resume)
{
struct blt_copy_data blt = {};
struct blt_block_copy_data_ext ext = {};
struct blt_ctrl_surf_copy_data surf = {};
- uint32_t bb, ccs, *ccsmap;
- uint64_t bb_size = 4096;
- uint64_t ccssize = mid->size / CCS_RATIO;
+ uint32_t bb, ccs, ccs2, *ccsmap, *ccsmap2;
+ uint64_t bb_size, ccssize = mid->size / CCS_RATIO;
uint32_t *ccscopy;
uint8_t uc_mocs = intel_get_uc_mocs(i915);
int result;
igt_assert(mid->compression);
ccscopy = (uint32_t *) malloc(ccssize);
+ bb_size = 4096;
bb = gem_create_from_pool(i915, &bb_size, REGION_SMEM);
ccs = gem_create(i915, ccssize);
+ ccs2 = gem_create(i915, ccssize);
surf.i915 = i915;
surf.print_bb = param.print_bb;
@@ -193,6 +196,33 @@ static void surf_copy(int i915,
PROT_READ | PROT_WRITE);
memcpy(ccscopy, ccsmap, ccssize);
+ if (suspend_resume) {
+ char *orig, *orig2, *newsum, *newsum2;
+
+ orig = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)ccsmap, surf.dst.size);
+ orig2 = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)mid->ptr, mid->size);
+
+ igt_system_suspend_autoresume(SUSPEND_STATE_FREEZE, SUSPEND_TEST_NONE);
+
+ set_surf_object(&surf.dst, ccs2, REGION_SMEM, ccssize,
+ 0, DIRECT_ACCESS);
+ blt_ctrl_surf_copy(i915, ctx, e, ahnd, &surf);
+ gem_sync(i915, surf.dst.handle);
+
+ ccsmap2 = gem_mmap__device_coherent(i915, ccs2, 0, surf.dst.size,
+ PROT_READ | PROT_WRITE);
+ newsum = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)ccsmap2, surf.dst.size);
+ newsum2 = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)mid->ptr, mid->size);
+
+ munmap(ccsmap2, ccssize);
+ igt_assert(!strcmp(orig, newsum));
+ igt_assert(!strcmp(orig2, newsum2));
+ g_free(orig);
+ g_free(orig2);
+ g_free(newsum);
+ g_free(newsum2);
+ }
+
/* corrupt ccs */
for (int i = 0; i < surf.dst.size / sizeof(uint32_t); i++)
ccsmap[i] = i;
@@ -209,6 +239,7 @@ static void surf_copy(int i915,
set_blt_object(&blt.dst, dst);
set_object_ext(&ext.src, mid->compression_type, mid->x2, mid->y2, SURFACE_TYPE_2D);
set_object_ext(&ext.dst, 0, dst->x2, dst->y2, SURFACE_TYPE_2D);
+ bb_size = 4096;
bb = gem_create_from_pool(i915, &bb_size, REGION_SMEM);
set_batch(&blt.bb, bb, bb_size, REGION_SMEM);
blt_block_copy(i915, ctx, e, ahnd, &blt, &ext);
@@ -311,7 +342,8 @@ static void block_copy(int i915,
ALLOC_STRATEGY_LOW_TO_HIGH, 0);
}
- surf_copy(i915, surf_ctx, &surf_e, surf_ahnd, src, mid, dst, run_id);
+ surf_copy(i915, surf_ctx, &surf_e, ahnd, src, mid, dst, run_id,
+ config->suspend_resume);
if (surf_ctx != ctx) {
intel_ctx_destroy(i915, surf_ctx);
@@ -508,6 +540,14 @@ igt_main_args("bf:pst:W:H:", NULL, help_str, opt_handler, NULL)
block_copy_test(i915, &config, ctx, set);
}
+ igt_subtest_with_dynamic("suspend-resume") {
+ struct test_config config = { .compression = true,
+ .surfcopy = true,
+ .suspend_resume = true };
+
+ block_copy_test(i915, &config, ctx, set);
+ }
+
igt_fixture {
igt_disallow_hang(i915, hang);
close(i915);
--
2.32.0
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for Add two gem-ccs subtests (rev4)
2022-03-21 11:11 [igt-dev] [PATCH i-g-t 0/2] Add two gem-ccs subtests Zbigniew Kempczyński
2022-03-21 11:11 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_ccs: Check ctrl-surf-copy in new context Zbigniew Kempczyński
2022-03-21 11:11 ` [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_ccs: Add suspend-resume subtest Zbigniew Kempczyński
@ 2022-03-21 12:20 ` Patchwork
2022-03-21 15:00 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
3 siblings, 0 replies; 16+ messages in thread
From: Patchwork @ 2022-03-21 12:20 UTC (permalink / raw)
To: Zbigniew Kempczyński; +Cc: igt-dev
[-- Attachment #1: Type: text/plain, Size: 11512 bytes --]
== Series Details ==
Series: Add two gem-ccs subtests (rev4)
URL : https://patchwork.freedesktop.org/series/101472/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11389 -> IGTPW_6804
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/index.html
Participating hosts (35 -> 41)
------------------------------
Additional (10): bat-dg1-6 bat-dg2-8 bat-adlm-1 bat-dg2-9 bat-adlp-6 fi-hsw-4770 bat-rpls-1 bat-rpls-2 bat-jsl-2 bat-jsl-1
Missing (4): fi-bsw-cyan shard-rkl shard-tglu fi-kbl-8809g
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in IGTPW_6804:
### IGT changes ###
#### Suppressed ####
The following results come from untrusted machines, tests, or statuses.
They do not affect the overall result.
* igt@kms_addfb_basic@addfb25-framebuffer-vs-set-tiling:
- {bat-adlm-1}: NOTRUN -> [DMESG-WARN][1] +1 similar issue
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/bat-adlm-1/igt@kms_addfb_basic@addfb25-framebuffer-vs-set-tiling.html
Known issues
------------
Here are the changes found in IGTPW_6804 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@amdgpu/amd_basic@cs-gfx:
- fi-hsw-4770: NOTRUN -> [SKIP][2] ([fdo#109271] / [fdo#109315]) +17 similar issues
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/fi-hsw-4770/igt@amdgpu/amd_basic@cs-gfx.html
* igt@fbdev@info:
- bat-dg1-6: NOTRUN -> [SKIP][3] ([i915#2582]) +4 similar issues
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/bat-dg1-6/igt@fbdev@info.html
* igt@gem_exec_gttfill@basic:
- bat-dg1-6: NOTRUN -> [SKIP][4] ([i915#4086])
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/bat-dg1-6/igt@gem_exec_gttfill@basic.html
* igt@gem_huc_copy@huc-copy:
- fi-hsw-4770: NOTRUN -> [SKIP][5] ([fdo#109271]) +9 similar issues
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/fi-hsw-4770/igt@gem_huc_copy@huc-copy.html
* igt@gem_mmap@basic:
- bat-dg1-6: NOTRUN -> [SKIP][6] ([i915#4083])
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/bat-dg1-6/igt@gem_mmap@basic.html
* igt@gem_tiled_blits@basic:
- bat-dg1-6: NOTRUN -> [SKIP][7] ([i915#4077]) +2 similar issues
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/bat-dg1-6/igt@gem_tiled_blits@basic.html
* igt@gem_tiled_pread_basic:
- bat-dg1-6: NOTRUN -> [SKIP][8] ([i915#4079]) +1 similar issue
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/bat-dg1-6/igt@gem_tiled_pread_basic.html
* igt@i915_pm_backlight@basic-brightness:
- fi-hsw-4770: NOTRUN -> [SKIP][9] ([fdo#109271] / [i915#3012])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/fi-hsw-4770/igt@i915_pm_backlight@basic-brightness.html
- bat-dg1-6: NOTRUN -> [SKIP][10] ([i915#1155])
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/bat-dg1-6/igt@i915_pm_backlight@basic-brightness.html
* igt@i915_selftest@live@hangcheck:
- bat-dg1-6: NOTRUN -> [DMESG-FAIL][11] ([i915#4494] / [i915#4957])
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/bat-dg1-6/igt@i915_selftest@live@hangcheck.html
* igt@kms_addfb_basic@addfb25-x-tiled-legacy:
- bat-dg1-6: NOTRUN -> [SKIP][12] ([i915#4212]) +7 similar issues
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/bat-dg1-6/igt@kms_addfb_basic@addfb25-x-tiled-legacy.html
* igt@kms_addfb_basic@basic-y-tiled-legacy:
- bat-dg1-6: NOTRUN -> [SKIP][13] ([i915#4215])
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/bat-dg1-6/igt@kms_addfb_basic@basic-y-tiled-legacy.html
* igt@kms_busy@basic:
- bat-dg1-6: NOTRUN -> [SKIP][14] ([i915#4303])
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/bat-dg1-6/igt@kms_busy@basic.html
* igt@kms_chamelium@common-hpd-after-suspend:
- fi-hsw-4770: NOTRUN -> [SKIP][15] ([fdo#109271] / [fdo#111827]) +8 similar issues
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/fi-hsw-4770/igt@kms_chamelium@common-hpd-after-suspend.html
* igt@kms_chamelium@hdmi-edid-read:
- bat-dg1-6: NOTRUN -> [SKIP][16] ([fdo#111827]) +8 similar issues
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/bat-dg1-6/igt@kms_chamelium@hdmi-edid-read.html
* igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic:
- bat-dg1-6: NOTRUN -> [SKIP][17] ([i915#4103] / [i915#4213]) +1 similar issue
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/bat-dg1-6/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic.html
* igt@kms_cursor_legacy@basic-flip-after-cursor-legacy:
- bat-dg1-6: NOTRUN -> [SKIP][18] ([i915#4078]) +23 similar issues
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/bat-dg1-6/igt@kms_cursor_legacy@basic-flip-after-cursor-legacy.html
* igt@kms_force_connector_basic@force-load-detect:
- bat-dg1-6: NOTRUN -> [SKIP][19] ([fdo#109285])
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/bat-dg1-6/igt@kms_force_connector_basic@force-load-detect.html
* igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-c:
- bat-dg1-6: NOTRUN -> [SKIP][20] ([i915#4078] / [i915#5341])
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/bat-dg1-6/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-c.html
* igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d:
- fi-hsw-4770: NOTRUN -> [SKIP][21] ([fdo#109271] / [i915#533])
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/fi-hsw-4770/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d.html
* igt@kms_psr@primary_mmap_gtt:
- fi-hsw-4770: NOTRUN -> [SKIP][22] ([fdo#109271] / [i915#1072]) +3 similar issues
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/fi-hsw-4770/igt@kms_psr@primary_mmap_gtt.html
* igt@kms_psr@sprite_plane_onoff:
- bat-dg1-6: NOTRUN -> [SKIP][23] ([i915#1072] / [i915#4078]) +3 similar issues
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/bat-dg1-6/igt@kms_psr@sprite_plane_onoff.html
* igt@kms_setmode@basic-clone-single-crtc:
- bat-dg1-6: NOTRUN -> [SKIP][24] ([i915#3555])
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/bat-dg1-6/igt@kms_setmode@basic-clone-single-crtc.html
* igt@prime_vgem@basic-gtt:
- bat-dg1-6: NOTRUN -> [SKIP][25] ([i915#3708] / [i915#4077]) +1 similar issue
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/bat-dg1-6/igt@prime_vgem@basic-gtt.html
* igt@prime_vgem@basic-userptr:
- bat-dg1-6: NOTRUN -> [SKIP][26] ([i915#3708] / [i915#4873])
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/bat-dg1-6/igt@prime_vgem@basic-userptr.html
* igt@prime_vgem@basic-write:
- bat-dg1-6: NOTRUN -> [SKIP][27] ([i915#3708]) +3 similar issues
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/bat-dg1-6/igt@prime_vgem@basic-write.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
[fdo#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285
[fdo#109308]: https://bugs.freedesktop.org/show_bug.cgi?id=109308
[fdo#109315]: https://bugs.freedesktop.org/show_bug.cgi?id=109315
[fdo#111825]: https://bugs.freedesktop.org/show_bug.cgi?id=111825
[fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
[i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072
[i915#1155]: https://gitlab.freedesktop.org/drm/intel/issues/1155
[i915#1845]: https://gitlab.freedesktop.org/drm/intel/issues/1845
[i915#1849]: https://gitlab.freedesktop.org/drm/intel/issues/1849
[i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190
[i915#2582]: https://gitlab.freedesktop.org/drm/intel/issues/2582
[i915#3003]: https://gitlab.freedesktop.org/drm/intel/issues/3003
[i915#3012]: https://gitlab.freedesktop.org/drm/intel/issues/3012
[i915#3282]: https://gitlab.freedesktop.org/drm/intel/issues/3282
[i915#3291]: https://gitlab.freedesktop.org/drm/intel/issues/3291
[i915#3301]: https://gitlab.freedesktop.org/drm/intel/issues/3301
[i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555
[i915#3576]: https://gitlab.freedesktop.org/drm/intel/issues/3576
[i915#3595]: https://gitlab.freedesktop.org/drm/intel/issues/3595
[i915#3637]: https://gitlab.freedesktop.org/drm/intel/issues/3637
[i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708
[i915#4077]: https://gitlab.freedesktop.org/drm/intel/issues/4077
[i915#4078]: https://gitlab.freedesktop.org/drm/intel/issues/4078
[i915#4079]: https://gitlab.freedesktop.org/drm/intel/issues/4079
[i915#4083]: https://gitlab.freedesktop.org/drm/intel/issues/4083
[i915#4086]: https://gitlab.freedesktop.org/drm/intel/issues/4086
[i915#4103]: https://gitlab.freedesktop.org/drm/intel/issues/4103
[i915#4212]: https://gitlab.freedesktop.org/drm/intel/issues/4212
[i915#4213]: https://gitlab.freedesktop.org/drm/intel/issues/4213
[i915#4215]: https://gitlab.freedesktop.org/drm/intel/issues/4215
[i915#4303]: https://gitlab.freedesktop.org/drm/intel/issues/4303
[i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312
[i915#4391]: https://gitlab.freedesktop.org/drm/intel/issues/4391
[i915#4494]: https://gitlab.freedesktop.org/drm/intel/issues/4494
[i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
[i915#4873]: https://gitlab.freedesktop.org/drm/intel/issues/4873
[i915#4957]: https://gitlab.freedesktop.org/drm/intel/issues/4957
[i915#5190]: https://gitlab.freedesktop.org/drm/intel/issues/5190
[i915#5192]: https://gitlab.freedesktop.org/drm/intel/issues/5192
[i915#5193]: https://gitlab.freedesktop.org/drm/intel/issues/5193
[i915#5195]: https://gitlab.freedesktop.org/drm/intel/issues/5195
[i915#5270]: https://gitlab.freedesktop.org/drm/intel/issues/5270
[i915#5274]: https://gitlab.freedesktop.org/drm/intel/issues/5274
[i915#5275]: https://gitlab.freedesktop.org/drm/intel/issues/5275
[i915#533]: https://gitlab.freedesktop.org/drm/intel/issues/533
[i915#5338]: https://gitlab.freedesktop.org/drm/intel/issues/5338
[i915#5339]: https://gitlab.freedesktop.org/drm/intel/issues/5339
[i915#5341]: https://gitlab.freedesktop.org/drm/intel/issues/5341
[i915#5342]: https://gitlab.freedesktop.org/drm/intel/issues/5342
[i915#5354]: https://gitlab.freedesktop.org/drm/intel/issues/5354
[i915#5356]: https://gitlab.freedesktop.org/drm/intel/issues/5356
Build changes
-------------
* CI: CI-20190529 -> None
* IGT: IGT_6386 -> IGTPW_6804
CI-20190529: 20190529
CI_DRM_11389: 35a8bf3837850c0ecf945c58f5611bcc53c4a43f @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_6804: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/index.html
IGT_6386: 0fcd59ad25b2960c0b654f90dfe4dd9e7c7b874d @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
== Testlist changes ==
+igt@gem_ccs@ctrl-surf-copy-new-ctx
+igt@gem_ccs@suspend-resume
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/index.html
[-- Attachment #2: Type: text/html, Size: 11764 bytes --]
^ permalink raw reply [flat|nested] 16+ messages in thread
* [igt-dev] ✓ Fi.CI.IGT: success for Add two gem-ccs subtests (rev4)
2022-03-21 11:11 [igt-dev] [PATCH i-g-t 0/2] Add two gem-ccs subtests Zbigniew Kempczyński
` (2 preceding siblings ...)
2022-03-21 12:20 ` [igt-dev] ✓ Fi.CI.BAT: success for Add two gem-ccs subtests (rev4) Patchwork
@ 2022-03-21 15:00 ` Patchwork
3 siblings, 0 replies; 16+ messages in thread
From: Patchwork @ 2022-03-21 15:00 UTC (permalink / raw)
To: Zbigniew Kempczyński; +Cc: igt-dev
[-- Attachment #1: Type: text/plain, Size: 30250 bytes --]
== Series Details ==
Series: Add two gem-ccs subtests (rev4)
URL : https://patchwork.freedesktop.org/series/101472/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_11389_full -> IGTPW_6804_full
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/index.html
Participating hosts (13 -> 9)
------------------------------
Missing (4): pig-skl-6260u pig-kbl-iris shard-dg1 pig-glk-j5005
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in IGTPW_6804_full:
### IGT changes ###
#### Suppressed ####
The following results come from untrusted machines, tests, or statuses.
They do not affect the overall result.
* {igt@gem_ccs@suspend-resume}:
- shard-iclb: NOTRUN -> [SKIP][1] +1 similar issue
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb2/igt@gem_ccs@suspend-resume.html
- {shard-tglu}: NOTRUN -> [SKIP][2] +1 similar issue
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglu-6/igt@gem_ccs@suspend-resume.html
- shard-tglb: NOTRUN -> [SKIP][3] +1 similar issue
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb2/igt@gem_ccs@suspend-resume.html
* igt@gem_exec_fence@long-history:
- {shard-rkl}: [PASS][4] -> [INCOMPLETE][5]
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11389/shard-rkl-1/igt@gem_exec_fence@long-history.html
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-rkl-5/igt@gem_exec_fence@long-history.html
* igt@kms_cursor_crc@pipe-a-cursor-32x32-onscreen:
- {shard-rkl}: [SKIP][6] ([fdo#112022]) -> [SKIP][7] +2 similar issues
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11389/shard-rkl-4/igt@kms_cursor_crc@pipe-a-cursor-32x32-onscreen.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-rkl-3/igt@kms_cursor_crc@pipe-a-cursor-32x32-onscreen.html
* igt@kms_cursor_crc@pipe-a-cursor-512x170-random:
- {shard-rkl}: [SKIP][8] ([fdo#112022] / [i915#4070]) -> [SKIP][9] +1 similar issue
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11389/shard-rkl-1/igt@kms_cursor_crc@pipe-a-cursor-512x170-random.html
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-rkl-3/igt@kms_cursor_crc@pipe-a-cursor-512x170-random.html
* igt@kms_cursor_crc@pipe-b-cursor-64x21-random:
- {shard-rkl}: NOTRUN -> [SKIP][10] +17 similar issues
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-rkl-3/igt@kms_cursor_crc@pipe-b-cursor-64x21-random.html
* igt@kms_cursor_crc@pipe-b-cursor-64x64-random:
- {shard-rkl}: [PASS][11] -> [SKIP][12] +2 similar issues
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11389/shard-rkl-6/igt@kms_cursor_crc@pipe-b-cursor-64x64-random.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-rkl-3/igt@kms_cursor_crc@pipe-b-cursor-64x64-random.html
* igt@kms_cursor_edge_walk@pipe-a-128x128-left-edge:
- {shard-rkl}: [SKIP][13] ([i915#1849] / [i915#4070] / [i915#4098]) -> [SKIP][14]
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11389/shard-rkl-1/igt@kms_cursor_edge_walk@pipe-a-128x128-left-edge.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-rkl-3/igt@kms_cursor_edge_walk@pipe-a-128x128-left-edge.html
* igt@kms_frontbuffer_tracking@fbc-1p-primscrn-pri-shrfb-draw-mmap-gtt:
- {shard-rkl}: ([SKIP][15], [SKIP][16]) ([i915#1849] / [i915#4098]) -> [SKIP][17] +3 similar issues
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11389/shard-rkl-4/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-pri-shrfb-draw-mmap-gtt.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11389/shard-rkl-5/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-pri-shrfb-draw-mmap-gtt.html
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-rkl-3/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-pri-shrfb-draw-mmap-gtt.html
* igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-pri-indfb-draw-render:
- {shard-rkl}: [SKIP][18] ([i915#4098]) -> [SKIP][19] +1 similar issue
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11389/shard-rkl-4/igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-pri-indfb-draw-render.html
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-rkl-3/igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-pri-indfb-draw-render.html
* igt@kms_frontbuffer_tracking@psr-2p-primscrn-cur-indfb-draw-render:
- {shard-rkl}: [SKIP][20] ([i915#1849]) -> [SKIP][21] +10 similar issues
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11389/shard-rkl-1/igt@kms_frontbuffer_tracking@psr-2p-primscrn-cur-indfb-draw-render.html
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-rkl-3/igt@kms_frontbuffer_tracking@psr-2p-primscrn-cur-indfb-draw-render.html
* igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
- {shard-rkl}: [SKIP][22] ([i915#1849] / [i915#4098]) -> [SKIP][23] +1 similar issue
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11389/shard-rkl-2/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b.html
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-rkl-3/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b.html
Known issues
------------
Here are the changes found in IGTPW_6804_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@core_hotunplug@unbind-rebind:
- shard-snb: [PASS][24] -> [DMESG-WARN][25] ([i915#4528])
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11389/shard-snb4/igt@core_hotunplug@unbind-rebind.html
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-snb7/igt@core_hotunplug@unbind-rebind.html
* igt@feature_discovery@psr2:
- shard-iclb: [PASS][26] -> [SKIP][27] ([i915#658])
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11389/shard-iclb2/igt@feature_discovery@psr2.html
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb7/igt@feature_discovery@psr2.html
* igt@gem_ctx_param@set-priority-not-supported:
- shard-tglb: NOTRUN -> [SKIP][28] ([fdo#109314])
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb8/igt@gem_ctx_param@set-priority-not-supported.html
- shard-iclb: NOTRUN -> [SKIP][29] ([fdo#109314])
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb4/igt@gem_ctx_param@set-priority-not-supported.html
* igt@gem_ctx_persistence@engines-mixed:
- shard-snb: NOTRUN -> [SKIP][30] ([fdo#109271] / [i915#1099]) +2 similar issues
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-snb5/igt@gem_ctx_persistence@engines-mixed.html
* igt@gem_ctx_sseu@invalid-args:
- shard-tglb: NOTRUN -> [SKIP][31] ([i915#280])
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb6/igt@gem_ctx_sseu@invalid-args.html
* igt@gem_exec_balancer@parallel-out-fence:
- shard-kbl: NOTRUN -> [DMESG-WARN][32] ([i915#5076])
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-kbl1/igt@gem_exec_balancer@parallel-out-fence.html
* igt@gem_exec_create@madvise@smem:
- shard-glk: [PASS][33] -> [DMESG-WARN][34] ([i915#118]) +1 similar issue
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11389/shard-glk6/igt@gem_exec_create@madvise@smem.html
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-glk4/igt@gem_exec_create@madvise@smem.html
* igt@gem_exec_fair@basic-none-share@rcs0:
- shard-tglb: [PASS][35] -> [FAIL][36] ([i915#2842])
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11389/shard-tglb6/igt@gem_exec_fair@basic-none-share@rcs0.html
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb7/igt@gem_exec_fair@basic-none-share@rcs0.html
* igt@gem_exec_fair@basic-none-solo@rcs0:
- shard-kbl: NOTRUN -> [FAIL][37] ([i915#2842]) +2 similar issues
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-kbl7/igt@gem_exec_fair@basic-none-solo@rcs0.html
- shard-glk: NOTRUN -> [FAIL][38] ([i915#2842])
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-glk1/igt@gem_exec_fair@basic-none-solo@rcs0.html
* igt@gem_exec_fair@basic-pace-share@rcs0:
- shard-glk: [PASS][39] -> [FAIL][40] ([i915#2842])
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11389/shard-glk5/igt@gem_exec_fair@basic-pace-share@rcs0.html
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-glk4/igt@gem_exec_fair@basic-pace-share@rcs0.html
* igt@gem_exec_fair@basic-pace-solo@rcs0:
- shard-apl: [PASS][41] -> [FAIL][42] ([i915#2842])
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11389/shard-apl2/igt@gem_exec_fair@basic-pace-solo@rcs0.html
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-apl1/igt@gem_exec_fair@basic-pace-solo@rcs0.html
* igt@gem_exec_fair@basic-pace@vcs1:
- shard-iclb: NOTRUN -> [FAIL][43] ([i915#2842]) +1 similar issue
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb1/igt@gem_exec_fair@basic-pace@vcs1.html
* igt@gem_exec_fair@basic-throttle@rcs0:
- shard-tglb: NOTRUN -> [FAIL][44] ([i915#2842]) +1 similar issue
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb8/igt@gem_exec_fair@basic-throttle@rcs0.html
* igt@gem_exec_flush@basic-batch-kernel-default-cmd:
- shard-tglb: NOTRUN -> [SKIP][45] ([fdo#109313])
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb5/igt@gem_exec_flush@basic-batch-kernel-default-cmd.html
* igt@gem_exec_params@no-vebox:
- shard-tglb: NOTRUN -> [SKIP][46] ([fdo#109283] / [i915#4877])
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb7/igt@gem_exec_params@no-vebox.html
* igt@gem_exec_params@rsvd2-dirt:
- shard-tglb: NOTRUN -> [SKIP][47] ([fdo#109283])
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb6/igt@gem_exec_params@rsvd2-dirt.html
- shard-iclb: NOTRUN -> [SKIP][48] ([fdo#109283])
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb1/igt@gem_exec_params@rsvd2-dirt.html
* igt@gem_huc_copy@huc-copy:
- shard-kbl: NOTRUN -> [SKIP][49] ([fdo#109271] / [i915#2190])
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-kbl7/igt@gem_huc_copy@huc-copy.html
* igt@gem_lmem_swapping@parallel-multi:
- shard-apl: NOTRUN -> [SKIP][50] ([fdo#109271] / [i915#4613]) +2 similar issues
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-apl1/igt@gem_lmem_swapping@parallel-multi.html
* igt@gem_lmem_swapping@parallel-random:
- shard-kbl: NOTRUN -> [SKIP][51] ([fdo#109271] / [i915#4613]) +1 similar issue
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-kbl1/igt@gem_lmem_swapping@parallel-random.html
* igt@gem_pwrite@basic-exhaustion:
- shard-kbl: NOTRUN -> [WARN][52] ([i915#2658])
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-kbl4/igt@gem_pwrite@basic-exhaustion.html
- shard-tglb: NOTRUN -> [WARN][53] ([i915#2658])
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb7/igt@gem_pwrite@basic-exhaustion.html
* igt@gem_pxp@regular-baseline-src-copy-readible:
- shard-tglb: NOTRUN -> [SKIP][54] ([i915#4270]) +5 similar issues
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb2/igt@gem_pxp@regular-baseline-src-copy-readible.html
* igt@gem_pxp@reject-modify-context-protection-off-3:
- shard-iclb: NOTRUN -> [SKIP][55] ([i915#4270]) +2 similar issues
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb8/igt@gem_pxp@reject-modify-context-protection-off-3.html
* igt@gem_render_copy@yf-tiled-to-vebox-linear:
- shard-iclb: NOTRUN -> [SKIP][56] ([i915#768]) +1 similar issue
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb4/igt@gem_render_copy@yf-tiled-to-vebox-linear.html
* igt@gem_userptr_blits@coherency-sync:
- shard-tglb: NOTRUN -> [SKIP][57] ([fdo#110542])
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb2/igt@gem_userptr_blits@coherency-sync.html
- shard-iclb: NOTRUN -> [SKIP][58] ([fdo#109290])
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb8/igt@gem_userptr_blits@coherency-sync.html
* igt@gem_userptr_blits@dmabuf-sync:
- shard-kbl: NOTRUN -> [SKIP][59] ([fdo#109271] / [i915#3323])
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-kbl6/igt@gem_userptr_blits@dmabuf-sync.html
- shard-tglb: NOTRUN -> [SKIP][60] ([i915#3323])
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb7/igt@gem_userptr_blits@dmabuf-sync.html
* igt@gem_userptr_blits@unsync-unmap-after-close:
- shard-tglb: NOTRUN -> [SKIP][61] ([i915#3297])
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb1/igt@gem_userptr_blits@unsync-unmap-after-close.html
- shard-iclb: NOTRUN -> [SKIP][62] ([i915#3297])
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb6/igt@gem_userptr_blits@unsync-unmap-after-close.html
* igt@gen3_render_mixed_blits:
- shard-tglb: NOTRUN -> [SKIP][63] ([fdo#109289]) +2 similar issues
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb6/igt@gen3_render_mixed_blits.html
* igt@gen7_exec_parse@chained-batch:
- shard-iclb: NOTRUN -> [SKIP][64] ([fdo#109289]) +1 similar issue
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb3/igt@gen7_exec_parse@chained-batch.html
* igt@gen9_exec_parse@allowed-all:
- shard-apl: [PASS][65] -> [DMESG-WARN][66] ([i915#1436] / [i915#716])
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11389/shard-apl8/igt@gen9_exec_parse@allowed-all.html
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-apl4/igt@gen9_exec_parse@allowed-all.html
* igt@gen9_exec_parse@cmd-crossing-page:
- shard-iclb: NOTRUN -> [SKIP][67] ([i915#2856])
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb5/igt@gen9_exec_parse@cmd-crossing-page.html
* igt@gen9_exec_parse@shadow-peek:
- shard-tglb: NOTRUN -> [SKIP][68] ([i915#2527] / [i915#2856]) +2 similar issues
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb7/igt@gen9_exec_parse@shadow-peek.html
* igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-dp:
- shard-kbl: NOTRUN -> [SKIP][69] ([fdo#109271] / [i915#1937])
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-kbl1/igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-dp.html
- shard-apl: NOTRUN -> [SKIP][70] ([fdo#109271] / [i915#1937])
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-apl7/igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-dp.html
* igt@i915_pm_rpm@modeset-non-lpsp-stress:
- shard-tglb: NOTRUN -> [SKIP][71] ([fdo#111644] / [i915#1397] / [i915#2411])
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb3/igt@i915_pm_rpm@modeset-non-lpsp-stress.html
- shard-iclb: NOTRUN -> [SKIP][72] ([fdo#110892])
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb3/igt@i915_pm_rpm@modeset-non-lpsp-stress.html
* igt@i915_pm_rpm@modeset-pc8-residency-stress:
- shard-tglb: NOTRUN -> [SKIP][73] ([fdo#109506] / [i915#2411])
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb6/igt@i915_pm_rpm@modeset-pc8-residency-stress.html
- shard-iclb: NOTRUN -> [SKIP][74] ([fdo#109293] / [fdo#109506])
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb1/igt@i915_pm_rpm@modeset-pc8-residency-stress.html
* igt@kms_big_fb@4-tiled-32bpp-rotate-0:
- shard-iclb: NOTRUN -> [SKIP][75] ([i915#5286]) +1 similar issue
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb2/igt@kms_big_fb@4-tiled-32bpp-rotate-0.html
* igt@kms_big_fb@4-tiled-max-hw-stride-64bpp-rotate-0-hflip:
- shard-tglb: NOTRUN -> [SKIP][76] ([i915#5286]) +8 similar issues
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb3/igt@kms_big_fb@4-tiled-max-hw-stride-64bpp-rotate-0-hflip.html
* igt@kms_big_fb@linear-8bpp-rotate-90:
- shard-tglb: NOTRUN -> [SKIP][77] ([fdo#111614]) +5 similar issues
[77]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb5/igt@kms_big_fb@linear-8bpp-rotate-90.html
* igt@kms_big_fb@x-tiled-64bpp-rotate-270:
- shard-iclb: NOTRUN -> [SKIP][78] ([fdo#110725] / [fdo#111614]) +1 similar issue
[78]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb8/igt@kms_big_fb@x-tiled-64bpp-rotate-270.html
* igt@kms_big_fb@x-tiled-max-hw-stride-64bpp-rotate-180-hflip:
- shard-kbl: NOTRUN -> [SKIP][79] ([fdo#109271] / [i915#3777]) +5 similar issues
[79]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-kbl6/igt@kms_big_fb@x-tiled-max-hw-stride-64bpp-rotate-180-hflip.html
* igt@kms_big_fb@y-tiled-max-hw-stride-32bpp-rotate-180-hflip:
- shard-apl: NOTRUN -> [SKIP][80] ([fdo#109271] / [i915#3777]) +1 similar issue
[80]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-apl8/igt@kms_big_fb@y-tiled-max-hw-stride-32bpp-rotate-180-hflip.html
* igt@kms_big_fb@yf-tiled-64bpp-rotate-90:
- shard-iclb: NOTRUN -> [SKIP][81] ([fdo#110723]) +2 similar issues
[81]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb1/igt@kms_big_fb@yf-tiled-64bpp-rotate-90.html
* igt@kms_big_fb@yf-tiled-8bpp-rotate-90:
- shard-tglb: NOTRUN -> [SKIP][82] ([fdo#111615]) +7 similar issues
[82]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb3/igt@kms_big_fb@yf-tiled-8bpp-rotate-90.html
* igt@kms_big_joiner@2x-modeset:
- shard-tglb: NOTRUN -> [SKIP][83] ([i915#2705])
[83]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb2/igt@kms_big_joiner@2x-modeset.html
* igt@kms_ccs@pipe-a-bad-aux-stride-y_tiled_gen12_rc_ccs_cc:
- shard-iclb: NOTRUN -> [SKIP][84] ([fdo#109278] / [i915#3886]) +7 similar issues
[84]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb7/igt@kms_ccs@pipe-a-bad-aux-stride-y_tiled_gen12_rc_ccs_cc.html
* igt@kms_ccs@pipe-a-bad-pixel-format-y_tiled_ccs:
- shard-tglb: NOTRUN -> [SKIP][85] ([i915#3689]) +4 similar issues
[85]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb7/igt@kms_ccs@pipe-a-bad-pixel-format-y_tiled_ccs.html
* igt@kms_ccs@pipe-a-random-ccs-data-yf_tiled_ccs:
- shard-tglb: NOTRUN -> [SKIP][86] ([fdo#111615] / [i915#3689]) +7 similar issues
[86]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb3/igt@kms_ccs@pipe-a-random-ccs-data-yf_tiled_ccs.html
* igt@kms_ccs@pipe-b-bad-rotation-90-y_tiled_gen12_mc_ccs:
- shard-tglb: NOTRUN -> [SKIP][87] ([i915#3689] / [i915#3886]) +6 similar issues
[87]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb1/igt@kms_ccs@pipe-b-bad-rotation-90-y_tiled_gen12_mc_ccs.html
* igt@kms_ccs@pipe-b-crc-sprite-planes-basic-y_tiled_gen12_rc_ccs_cc:
- shard-glk: NOTRUN -> [SKIP][88] ([fdo#109271] / [i915#3886]) +7 similar issues
[88]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-glk9/igt@kms_ccs@pipe-b-crc-sprite-planes-basic-y_tiled_gen12_rc_ccs_cc.html
* igt@kms_ccs@pipe-b-random-ccs-data-y_tiled_gen12_mc_ccs:
- shard-kbl: NOTRUN -> [SKIP][89] ([fdo#109271] / [i915#3886]) +18 similar issues
[89]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-kbl4/igt@kms_ccs@pipe-b-random-ccs-data-y_tiled_gen12_mc_ccs.html
* igt@kms_ccs@pipe-c-bad-rotation-90-y_tiled_gen12_mc_ccs:
- shard-apl: NOTRUN -> [SKIP][90] ([fdo#109271] / [i915#3886]) +15 similar issues
[90]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-apl4/igt@kms_ccs@pipe-c-bad-rotation-90-y_tiled_gen12_mc_ccs.html
* igt@kms_chamelium@hdmi-hpd-enable-disable-mode:
- shard-iclb: NOTRUN -> [SKIP][91] ([fdo#109284] / [fdo#111827]) +4 similar issues
[91]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb7/igt@kms_chamelium@hdmi-hpd-enable-disable-mode.html
- shard-snb: NOTRUN -> [SKIP][92] ([fdo#109271] / [fdo#111827]) +5 similar issues
[92]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-snb5/igt@kms_chamelium@hdmi-hpd-enable-disable-mode.html
* igt@kms_chamelium@hdmi-hpd-storm:
- shard-kbl: NOTRUN -> [SKIP][93] ([fdo#109271] / [fdo#111827]) +39 similar issues
[93]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-kbl4/igt@kms_chamelium@hdmi-hpd-storm.html
* igt@kms_color_chamelium@pipe-a-ctm-0-5:
- shard-apl: NOTRUN -> [SKIP][94] ([fdo#109271] / [fdo#111827]) +15 similar issues
[94]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-apl1/igt@kms_color_chamelium@pipe-a-ctm-0-5.html
* igt@kms_color_chamelium@pipe-d-ctm-0-25:
- shard-glk: NOTRUN -> [SKIP][95] ([fdo#109271] / [fdo#111827]) +5 similar issues
[95]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-glk9/igt@kms_color_chamelium@pipe-d-ctm-0-25.html
* igt@kms_color_chamelium@pipe-d-ctm-max:
- shard-tglb: NOTRUN -> [SKIP][96] ([fdo#109284] / [fdo#111827]) +19 similar issues
[96]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb7/igt@kms_color_chamelium@pipe-d-ctm-max.html
* igt@kms_color_chamelium@pipe-d-ctm-negative:
- shard-iclb: NOTRUN -> [SKIP][97] ([fdo#109278] / [fdo#109284] / [fdo#111827]) +3 similar issues
[97]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb3/igt@kms_color_chamelium@pipe-d-ctm-negative.html
* igt@kms_content_protection@atomic:
- shard-kbl: NOTRUN -> [TIMEOUT][98] ([i915#1319]) +2 similar issues
[98]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-kbl1/igt@kms_content_protection@atomic.html
* igt@kms_content_protection@content_type_change:
- shard-tglb: NOTRUN -> [SKIP][99] ([i915#1063]) +2 similar issues
[99]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb8/igt@kms_content_protection@content_type_change.html
* igt@kms_content_protection@dp-mst-lic-type-0:
- shard-iclb: NOTRUN -> [SKIP][100] ([i915#3116])
[100]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb4/igt@kms_content_protection@dp-mst-lic-type-0.html
- shard-tglb: NOTRUN -> [SKIP][101] ([i915#3116] / [i915#3299])
[101]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb6/igt@kms_content_protection@dp-mst-lic-type-0.html
* igt@kms_content_protection@lic:
- shard-apl: NOTRUN -> [TIMEOUT][102] ([i915#1319]) +2 similar issues
[102]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-apl4/igt@kms_content_protection@lic.html
- shard-iclb: NOTRUN -> [SKIP][103] ([fdo#109300] / [fdo#111066])
[103]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb2/igt@kms_content_protection@lic.html
* igt@kms_cursor_crc@pipe-a-cursor-32x10-offscreen:
- shard-tglb: NOTRUN -> [SKIP][104] ([i915#3359]) +9 similar issues
[104]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb7/igt@kms_cursor_crc@pipe-a-cursor-32x10-offscreen.html
* igt@kms_cursor_crc@pipe-a-cursor-512x512-offscreen:
- shard-iclb: NOTRUN -> [SKIP][105] ([fdo#109278] / [fdo#109279]) +2 similar issues
[105]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb6/igt@kms_cursor_crc@pipe-a-cursor-512x512-offscreen.html
* igt@kms_cursor_crc@pipe-b-cursor-32x32-onscreen:
- shard-tglb: NOTRUN -> [SKIP][106] ([i915#3319]) +3 similar issues
[106]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb7/igt@kms_cursor_crc@pipe-b-cursor-32x32-onscreen.html
* igt@kms_cursor_crc@pipe-c-cursor-512x512-random:
- shard-tglb: NOTRUN -> [SKIP][107] ([fdo#109279] / [i915#3359]) +9 similar issues
[107]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb7/igt@kms_cursor_crc@pipe-c-cursor-512x512-random.html
* igt@kms_cursor_crc@pipe-d-cursor-suspend:
- shard-kbl: NOTRUN -> [SKIP][108] ([fdo#109271]) +417 similar issues
[108]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-kbl4/igt@kms_cursor_crc@pipe-d-cursor-suspend.html
* igt@kms_cursor_legacy@cursora-vs-flipb-atomic:
- shard-iclb: NOTRUN -> [SKIP][109] ([fdo#109274] / [fdo#109278]) +2 similar issues
[109]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb5/igt@kms_cursor_legacy@cursora-vs-flipb-atomic.html
* igt@kms_cursor_legacy@pipe-d-single-bo:
- shard-kbl: NOTRUN -> [SKIP][110] ([fdo#109271] / [i915#533]) +2 similar issues
[110]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-kbl1/igt@kms_cursor_legacy@pipe-d-single-bo.html
* igt@kms_draw_crc@draw-method-rgb565-mmap-cpu-untiled:
- shard-glk: [PASS][111] -> [FAIL][112] ([i915#1888] / [i915#5160])
[111]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11389/shard-glk3/igt@kms_draw_crc@draw-method-rgb565-mmap-cpu-untiled.html
[112]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-glk3/igt@kms_draw_crc@draw-method-rgb565-mmap-cpu-untiled.html
* igt@kms_draw_crc@draw-method-xrgb2101010-render-4tiled:
- shard-tglb: NOTRUN -> [SKIP][113] ([i915#5287])
[113]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb1/igt@kms_draw_crc@draw-method-xrgb2101010-render-4tiled.html
* igt@kms_dsc@basic-dsc-enable:
- shard-iclb: NOTRUN -> [SKIP][114] ([i915#3840])
[114]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb8/igt@kms_dsc@basic-dsc-enable.html
* igt@kms_flip@2x-absolute-wf_vblank:
- shard-tglb: NOTRUN -> [SKIP][115] ([fdo#109274] / [fdo#111825] / [i915#3966])
[115]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb6/igt@kms_flip@2x-absolute-wf_vblank.html
* igt@kms_flip@2x-flip-vs-rmfb:
- shard-tglb: NOTRUN -> [SKIP][116] ([fdo#109274] / [fdo#111825]) +10 similar issues
[116]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb6/igt@kms_flip@2x-flip-vs-rmfb.html
* igt@kms_flip@2x-plain-flip-fb-recreate-interruptible:
- shard-iclb: NOTRUN -> [SKIP][117] ([fdo#109274]) +1 similar issue
[117]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb3/igt@kms_flip@2x-plain-flip-fb-recreate-interruptible.html
* igt@kms_flip@flip-vs-suspend@a-dp1:
- shard-apl: NOTRUN -> [DMESG-WARN][118] ([i915#180])
[118]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-apl1/igt@kms_flip@flip-vs-suspend@a-dp1.html
* igt@kms_flip@flip-vs-suspend@c-dp1:
- shard-kbl: NOTRUN -> [DMESG-WARN][119] ([i915#180]) +4 similar issues
[119]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-kbl7/igt@kms_flip@flip-vs-suspend@c-dp1.html
* igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytileccs-downscaling:
- shard-tglb: NOTRUN -> [SKIP][120] ([i915#2587])
[120]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb3/igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytileccs-downscaling.html
* igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-shrfb-msflip-blt:
- shard-tglb: NOTRUN -> [SKIP][121] ([fdo#109280] / [fdo#111825]) +43 similar issues
[121]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-tglb8/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-shrfb-msflip-blt.html
* igt@kms_frontbuffer_tracking@fbc-suspend:
- shard-kbl: [PASS][122] -> [DMESG-WARN][123] ([i915#180]) +2 similar issues
[122]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11389/shard-kbl3/igt@kms_frontbuffer_tracking@fbc-suspend.html
[123]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-kbl4/igt@kms_frontbuffer_tracking@fbc-suspend.html
- shard-apl: [PASS][124] -> [DMESG-WARN][125] ([i915#180])
[124]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11389/shard-apl3/igt@kms_frontbuffer_tracking@fbc-suspend.html
[125]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-apl6/igt@kms_frontbuffer_tracking@fbc-suspend.html
* igt@kms_frontbuffer_tracking@fbcpsr-2p-shrfb-fliptrack-mmap-gtt:
- shard-iclb: NOTRUN -> [SKIP][126] ([fdo#109280]) +22 similar issues
[126]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb2/igt@kms_frontbuffer_tracking@fbcpsr-2p-shrfb-fliptrack-mmap-gtt.html
- shard-glk: NOTRUN -> [SKIP][127] ([fdo#109271]) +72 similar issues
[127]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-glk5/igt@kms_frontbuffer_tracking@fbcpsr-2p-shrfb-fliptrack-mmap-gtt.html
* igt@kms_pipe_crc_basic@read-crc-pipe-d-frame-sequence:
- shard-snb: NOTRUN -> [SKIP][128] ([fdo#109271]) +147 similar issues
[128]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-snb2/igt@kms_pipe_crc_basic@read-crc-pipe-d-frame-sequence.html
- shard-iclb: NOTRUN -> [SKIP][129] ([fdo#109278]) +22 similar issues
[129]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-iclb6/igt@kms_pipe_crc_basic@read-crc-pipe-d-frame-sequence.html
- shard-glk: NOTRUN -> [SKIP][130] ([fdo#109271] / [i915#533])
[130]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-glk3/igt@kms_pipe_crc_basic@read-crc-pipe-d-frame-sequence.html
- shard-apl: NOTRUN -> [SKIP][131] ([fdo#109271] / [i915#533]) +1 similar issue
[131]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/shard-apl3/igt@kms_pipe_
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6804/index.html
[-- Attachment #2: Type: text/html, Size: 33793 bytes --]
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_ccs: Check ctrl-surf-copy in new context
2022-03-21 11:11 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_ccs: Check ctrl-surf-copy in new context Zbigniew Kempczyński
@ 2022-03-21 16:00 ` Kamil Konieczny
0 siblings, 0 replies; 16+ messages in thread
From: Kamil Konieczny @ 2022-03-21 16:00 UTC (permalink / raw)
To: igt-dev
Hi,
there is only description missed (see below) and one line change
in 2/2 which imho should be here in 1/2.
Dnia 2022-03-21 at 12:11:29 +0100, Zbigniew Kempczyński napisał(a):
> From: Apoorva Singh <apoorva1.singh@intel.com>
>
> Verify ccs data is binded to physical memory by using
> XY_CTRL_SURF_COPY_BLT command in separate context.
>
> v2: add separate allocator handle for new context (Zbigniew)
>
> Signed-off-by: Apoorva Singh <apoorva1.singh@intel.com>
> Cc: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
> ---
> tests/i915/gem_ccs.c | 50 ++++++++++++++++++++++++++++++++++----------
> 1 file changed, 39 insertions(+), 11 deletions(-)
>
> diff --git a/tests/i915/gem_ccs.c b/tests/i915/gem_ccs.c
> index fdf1fe75f5..7e104f6ad8 100644
> --- a/tests/i915/gem_ccs.c
> +++ b/tests/i915/gem_ccs.c
> @@ -39,6 +39,7 @@ struct test_config {
> bool compression;
> bool inplace;
> bool surfcopy;
> + bool new_ctx;
> };
>
> static void set_object(struct blt_copy_object *obj,
> @@ -234,9 +235,8 @@ static void block_copy(int i915,
> const intel_ctx_t *ctx,
> const struct intel_execution_engine2 *e,
> uint32_t region1, uint32_t region2,
> - enum blt_tiling mid_tiling, bool compression,
> - bool inplace,
> - bool surfcopy)
> + enum blt_tiling mid_tiling,
> + const struct test_config *config)
> {
> struct blt_copy_data blt = {};
> struct blt_block_copy_data_ext ext = {}, *pext = &ext;
> @@ -249,7 +249,7 @@ static void block_copy(int i915,
> uint32_t run_id = mid_tiling;
> uint32_t mid_region = region2, bb;
> uint32_t width = param.width, height = param.height;
> - enum blt_compression mid_compression = compression;
> + enum blt_compression mid_compression = config->compression;
> int mid_compression_format = param.compression_format;
> enum blt_compression_type comp_type = COMPRESSION_TYPE_3D;
> uint8_t uc_mocs = intel_get_uc_mocs(i915);
> @@ -293,8 +293,31 @@ static void block_copy(int i915,
> WRITE_PNG(i915, run_id, "src", &blt.src, width, height);
> WRITE_PNG(i915, run_id, "mid", &blt.dst, width, height);
>
> - if (surfcopy && pext)
> - surf_copy(i915, ctx, e, ahnd, src, mid, dst, run_id);
> + if (config->surfcopy && pext) {
> + const intel_ctx_t *surf_ctx = ctx;
> + uint64_t surf_ahnd = ahnd;
> + struct intel_execution_engine2 surf_e = *e;
> +
> + if (config->new_ctx) {
> + intel_ctx_cfg_t cfg = {};
> +
> + cfg.num_engines = 1;
> + cfg.engines[0].engine_class = e->class;
> + cfg.engines[0].engine_instance = e->instance;
> + surf_ctx = intel_ctx_create(i915, &cfg);
> + surf_e.flags = 0;
> + ahnd = intel_allocator_open_full(i915, surf_ctx->id, 0, 0,
> + INTEL_ALLOCATOR_SIMPLE,
> + ALLOC_STRATEGY_LOW_TO_HIGH, 0);
> + }
> +
> + surf_copy(i915, surf_ctx, &surf_e, surf_ahnd, src, mid, dst, run_id);
-------------------------------------------------- ^
Why suspend/resume in 2/2 change this into ahand ? Imho this
change should be done here.
> +
> + if (surf_ctx != ctx) {
> + intel_ctx_destroy(i915, surf_ctx);
> + put_ahnd(surf_ahnd);
> + }
> + }
>
> memset(&blt, 0, sizeof(blt));
> blt.color_depth = CD_32bit;
> @@ -303,7 +326,7 @@ static void block_copy(int i915,
> set_blt_object(&blt.dst, dst);
> set_object_ext(&ext.src, mid_compression_format, width, height, SURFACE_TYPE_2D);
> set_object_ext(&ext.dst, 0, width, height, SURFACE_TYPE_2D);
> - if (inplace) {
> + if (config->inplace) {
> set_object(&blt.dst, mid->handle, dst->size, mid->region, 0,
> T_LINEAR, COMPRESSION_DISABLED, comp_type);
> blt.dst.ptr = mid->ptr;
> @@ -367,10 +390,7 @@ static void block_copy_test(int i915,
> param.compression_format, regtxt) {
> block_copy(i915, ctx, e,
> region1, region2,
> - tiling,
> - config->compression,
> - config->inplace,
> - config->surfcopy);
> + tiling, config);
> }
> free(regtxt);
> }
> @@ -480,6 +500,14 @@ igt_main_args("bf:pst:W:H:", NULL, help_str, opt_handler, NULL)
> block_copy_test(i915, &config, ctx, set);
> }
>
Add description before new test.
> + igt_subtest_with_dynamic("ctrl-surf-copy-new-ctx") {
> + struct test_config config = { .compression = true,
> + .surfcopy = true,
> + .new_ctx = true };
> +
> + block_copy_test(i915, &config, ctx, set);
> + }
> +
> igt_fixture {
> igt_disallow_hang(i915, hang);
> close(i915);
> --
> 2.32.0
>
Regards,
Kamil
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_ccs: Add suspend-resume subtest
2022-03-21 11:11 ` [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_ccs: Add suspend-resume subtest Zbigniew Kempczyński
@ 2022-03-21 16:04 ` Kamil Konieczny
0 siblings, 0 replies; 16+ messages in thread
From: Kamil Konieczny @ 2022-03-21 16:04 UTC (permalink / raw)
To: igt-dev
Hi,
only two comments, see below.
Dnia 2022-03-21 at 12:11:30 +0100, Zbigniew Kempczyński napisał(a):
> From: Apoorva Singh <apoorva1.singh@intel.com>
>
> Verify flatccs data won't be corrupted when device will be put
> to S0 (s2idle) state.
>
> v2: addressing review comments (Kamil)
> v3: fix memory leak (Kamil)
>
> Signed-off-by: Apoorva Singh <apoorva1.singh@intel.com>
> Cc: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
> ---
> tests/i915/gem_ccs.c | 50 +++++++++++++++++++++++++++++++++++++++-----
> 1 file changed, 45 insertions(+), 5 deletions(-)
>
> diff --git a/tests/i915/gem_ccs.c b/tests/i915/gem_ccs.c
> index 7e104f6ad8..1f8f98ce62 100644
> --- a/tests/i915/gem_ccs.c
> +++ b/tests/i915/gem_ccs.c
> @@ -4,6 +4,7 @@
> */
>
> #include <errno.h>
> +#include <glib.h>
> #include <sys/ioctl.h>
> #include <sys/time.h>
> #include <malloc.h>
> @@ -40,6 +41,7 @@ struct test_config {
> bool inplace;
> bool surfcopy;
> bool new_ctx;
> + bool suspend_resume;
> };
>
> static void set_object(struct blt_copy_object *obj,
> @@ -162,22 +164,23 @@ static void surf_copy(int i915,
> const struct blt_copy_object *src,
> const struct blt_copy_object *mid,
> const struct blt_copy_object *dst,
> - int run_id)
> + int run_id, bool suspend_resume)
> {
> struct blt_copy_data blt = {};
> struct blt_block_copy_data_ext ext = {};
> struct blt_ctrl_surf_copy_data surf = {};
> - uint32_t bb, ccs, *ccsmap;
> - uint64_t bb_size = 4096;
> - uint64_t ccssize = mid->size / CCS_RATIO;
> + uint32_t bb, ccs, ccs2, *ccsmap, *ccsmap2;
> + uint64_t bb_size, ccssize = mid->size / CCS_RATIO;
> uint32_t *ccscopy;
> uint8_t uc_mocs = intel_get_uc_mocs(i915);
> int result;
>
> igt_assert(mid->compression);
> ccscopy = (uint32_t *) malloc(ccssize);
> + bb_size = 4096;
> bb = gem_create_from_pool(i915, &bb_size, REGION_SMEM);
> ccs = gem_create(i915, ccssize);
> + ccs2 = gem_create(i915, ccssize);
>
> surf.i915 = i915;
> surf.print_bb = param.print_bb;
> @@ -193,6 +196,33 @@ static void surf_copy(int i915,
> PROT_READ | PROT_WRITE);
> memcpy(ccscopy, ccsmap, ccssize);
>
> + if (suspend_resume) {
> + char *orig, *orig2, *newsum, *newsum2;
> +
> + orig = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)ccsmap, surf.dst.size);
> + orig2 = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)mid->ptr, mid->size);
> +
> + igt_system_suspend_autoresume(SUSPEND_STATE_FREEZE, SUSPEND_TEST_NONE);
> +
> + set_surf_object(&surf.dst, ccs2, REGION_SMEM, ccssize,
> + 0, DIRECT_ACCESS);
> + blt_ctrl_surf_copy(i915, ctx, e, ahnd, &surf);
> + gem_sync(i915, surf.dst.handle);
> +
> + ccsmap2 = gem_mmap__device_coherent(i915, ccs2, 0, surf.dst.size,
> + PROT_READ | PROT_WRITE);
> + newsum = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)ccsmap2, surf.dst.size);
> + newsum2 = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)mid->ptr, mid->size);
> +
> + munmap(ccsmap2, ccssize);
> + igt_assert(!strcmp(orig, newsum));
> + igt_assert(!strcmp(orig2, newsum2));
> + g_free(orig);
> + g_free(orig2);
> + g_free(newsum);
> + g_free(newsum2);
> + }
> +
> /* corrupt ccs */
> for (int i = 0; i < surf.dst.size / sizeof(uint32_t); i++)
> ccsmap[i] = i;
> @@ -209,6 +239,7 @@ static void surf_copy(int i915,
> set_blt_object(&blt.dst, dst);
> set_object_ext(&ext.src, mid->compression_type, mid->x2, mid->y2, SURFACE_TYPE_2D);
> set_object_ext(&ext.dst, 0, dst->x2, dst->y2, SURFACE_TYPE_2D);
> + bb_size = 4096;
> bb = gem_create_from_pool(i915, &bb_size, REGION_SMEM);
> set_batch(&blt.bb, bb, bb_size, REGION_SMEM);
> blt_block_copy(i915, ctx, e, ahnd, &blt, &ext);
> @@ -311,7 +342,8 @@ static void block_copy(int i915,
> ALLOC_STRATEGY_LOW_TO_HIGH, 0);
> }
>
> - surf_copy(i915, surf_ctx, &surf_e, surf_ahnd, src, mid, dst, run_id);
> + surf_copy(i915, surf_ctx, &surf_e, ahnd, src, mid, dst, run_id,
-------------------------------------------------- ^
imho this change to ahand should be in 1/2,
> + config->suspend_resume);
so the only change will be adding suspend_resume option.
>
> if (surf_ctx != ctx) {
> intel_ctx_destroy(i915, surf_ctx);
> @@ -508,6 +540,14 @@ igt_main_args("bf:pst:W:H:", NULL, help_str, opt_handler, NULL)
> block_copy_test(i915, &config, ctx, set);
> }
>
Add description to new test.
> + igt_subtest_with_dynamic("suspend-resume") {
> + struct test_config config = { .compression = true,
> + .surfcopy = true,
> + .suspend_resume = true };
> +
> + block_copy_test(i915, &config, ctx, set);
> + }
> +
> igt_fixture {
> igt_disallow_hang(i915, hang);
> close(i915);
> --
> 2.32.0
>
Regards,
Kamil
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_ccs: Add suspend-resume subtest
2022-03-22 8:25 ` [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_ccs: Add suspend-resume subtest Zbigniew Kempczyński
@ 2022-03-22 11:56 ` Kamil Konieczny
0 siblings, 0 replies; 16+ messages in thread
From: Kamil Konieczny @ 2022-03-22 11:56 UTC (permalink / raw)
To: igt-dev
Dnia 2022-03-22 at 09:25:18 +0100, Zbigniew Kempczyński napisał(a):
> From: Apoorva Singh <apoorva1.singh@intel.com>
>
> Verify flatccs data won't be corrupted when device will be put
> to S0 (s2idle) state.
>
> v2: addressing review comments (Kamil)
> v3: fix memory leak (Kamil)
>
> Signed-off-by: Apoorva Singh <apoorva1.singh@intel.com>
> Cc: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
> ---
> tests/i915/gem_ccs.c | 51 +++++++++++++++++++++++++++++++++++++++-----
> 1 file changed, 46 insertions(+), 5 deletions(-)
>
> diff --git a/tests/i915/gem_ccs.c b/tests/i915/gem_ccs.c
> index b697ea56f9..4ecb3e36ac 100644
> --- a/tests/i915/gem_ccs.c
> +++ b/tests/i915/gem_ccs.c
> @@ -4,6 +4,7 @@
> */
>
> #include <errno.h>
> +#include <glib.h>
> #include <sys/ioctl.h>
> #include <sys/time.h>
> #include <malloc.h>
> @@ -40,6 +41,7 @@ struct test_config {
> bool inplace;
> bool surfcopy;
> bool new_ctx;
> + bool suspend_resume;
> };
>
> static void set_object(struct blt_copy_object *obj,
> @@ -162,22 +164,23 @@ static void surf_copy(int i915,
> const struct blt_copy_object *src,
> const struct blt_copy_object *mid,
> const struct blt_copy_object *dst,
> - int run_id)
> + int run_id, bool suspend_resume)
> {
> struct blt_copy_data blt = {};
> struct blt_block_copy_data_ext ext = {};
> struct blt_ctrl_surf_copy_data surf = {};
> - uint32_t bb, ccs, *ccsmap;
> - uint64_t bb_size = 4096;
> - uint64_t ccssize = mid->size / CCS_RATIO;
> + uint32_t bb, ccs, ccs2, *ccsmap, *ccsmap2;
> + uint64_t bb_size, ccssize = mid->size / CCS_RATIO;
> uint32_t *ccscopy;
> uint8_t uc_mocs = intel_get_uc_mocs(i915);
> int result;
>
> igt_assert(mid->compression);
> ccscopy = (uint32_t *) malloc(ccssize);
> + bb_size = 4096;
> bb = gem_create_from_pool(i915, &bb_size, REGION_SMEM);
> ccs = gem_create(i915, ccssize);
> + ccs2 = gem_create(i915, ccssize);
>
> surf.i915 = i915;
> surf.print_bb = param.print_bb;
> @@ -193,6 +196,33 @@ static void surf_copy(int i915,
> PROT_READ | PROT_WRITE);
> memcpy(ccscopy, ccsmap, ccssize);
>
> + if (suspend_resume) {
> + char *orig, *orig2, *newsum, *newsum2;
> +
> + orig = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)ccsmap, surf.dst.size);
> + orig2 = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)mid->ptr, mid->size);
> +
> + igt_system_suspend_autoresume(SUSPEND_STATE_FREEZE, SUSPEND_TEST_NONE);
> +
> + set_surf_object(&surf.dst, ccs2, REGION_SMEM, ccssize,
> + 0, DIRECT_ACCESS);
> + blt_ctrl_surf_copy(i915, ctx, e, ahnd, &surf);
> + gem_sync(i915, surf.dst.handle);
> +
> + ccsmap2 = gem_mmap__device_coherent(i915, ccs2, 0, surf.dst.size,
> + PROT_READ | PROT_WRITE);
> + newsum = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)ccsmap2, surf.dst.size);
> + newsum2 = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)mid->ptr, mid->size);
> +
> + munmap(ccsmap2, ccssize);
> + igt_assert(!strcmp(orig, newsum));
> + igt_assert(!strcmp(orig2, newsum2));
> + g_free(orig);
> + g_free(orig2);
> + g_free(newsum);
> + g_free(newsum2);
> + }
> +
> /* corrupt ccs */
> for (int i = 0; i < surf.dst.size / sizeof(uint32_t); i++)
> ccsmap[i] = i;
> @@ -209,6 +239,7 @@ static void surf_copy(int i915,
> set_blt_object(&blt.dst, dst);
> set_object_ext(&ext.src, mid->compression_type, mid->x2, mid->y2, SURFACE_TYPE_2D);
> set_object_ext(&ext.dst, 0, dst->x2, dst->y2, SURFACE_TYPE_2D);
> + bb_size = 4096;
> bb = gem_create_from_pool(i915, &bb_size, REGION_SMEM);
> set_batch(&blt.bb, bb, bb_size, REGION_SMEM);
> blt_block_copy(i915, ctx, e, ahnd, &blt, &ext);
> @@ -311,7 +342,8 @@ static void block_copy(int i915,
> ALLOC_STRATEGY_LOW_TO_HIGH, 0);
> }
>
> - surf_copy(i915, surf_ctx, &surf_e, surf_ahnd, src, mid, dst, run_id);
> + surf_copy(i915, surf_ctx, &surf_e, surf_ahnd, src, mid, dst, run_id,
> + config->suspend_resume);
>
> if (surf_ctx != ctx) {
> intel_ctx_destroy(i915, surf_ctx);
> @@ -510,6 +542,15 @@ igt_main_args("bf:pst:W:H:", NULL, help_str, opt_handler, NULL)
> block_copy_test(i915, &config, ctx, set);
> }
>
> + igt_describe("Check flatccs data persists after suspend / resume (S0)");
> + igt_subtest_with_dynamic("suspend-resume") {
> + struct test_config config = { .compression = true,
> + .surfcopy = true,
> + .suspend_resume = true };
> +
> + block_copy_test(i915, &config, ctx, set);
> + }
> +
> igt_fixture {
> igt_disallow_hang(i915, hang);
> close(i915);
> --
> 2.32.0
>
Reviewed-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>
--
Kamil
^ permalink raw reply [flat|nested] 16+ messages in thread
* [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_ccs: Add suspend-resume subtest
2022-03-22 8:25 [igt-dev] [PATCH i-g-t 0/2] Add two gem-ccs subtests Zbigniew Kempczyński
@ 2022-03-22 8:25 ` Zbigniew Kempczyński
2022-03-22 11:56 ` Kamil Konieczny
0 siblings, 1 reply; 16+ messages in thread
From: Zbigniew Kempczyński @ 2022-03-22 8:25 UTC (permalink / raw)
To: igt-dev
From: Apoorva Singh <apoorva1.singh@intel.com>
Verify flatccs data won't be corrupted when device will be put
to S0 (s2idle) state.
v2: addressing review comments (Kamil)
v3: fix memory leak (Kamil)
Signed-off-by: Apoorva Singh <apoorva1.singh@intel.com>
Cc: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
---
tests/i915/gem_ccs.c | 51 +++++++++++++++++++++++++++++++++++++++-----
1 file changed, 46 insertions(+), 5 deletions(-)
diff --git a/tests/i915/gem_ccs.c b/tests/i915/gem_ccs.c
index b697ea56f9..4ecb3e36ac 100644
--- a/tests/i915/gem_ccs.c
+++ b/tests/i915/gem_ccs.c
@@ -4,6 +4,7 @@
*/
#include <errno.h>
+#include <glib.h>
#include <sys/ioctl.h>
#include <sys/time.h>
#include <malloc.h>
@@ -40,6 +41,7 @@ struct test_config {
bool inplace;
bool surfcopy;
bool new_ctx;
+ bool suspend_resume;
};
static void set_object(struct blt_copy_object *obj,
@@ -162,22 +164,23 @@ static void surf_copy(int i915,
const struct blt_copy_object *src,
const struct blt_copy_object *mid,
const struct blt_copy_object *dst,
- int run_id)
+ int run_id, bool suspend_resume)
{
struct blt_copy_data blt = {};
struct blt_block_copy_data_ext ext = {};
struct blt_ctrl_surf_copy_data surf = {};
- uint32_t bb, ccs, *ccsmap;
- uint64_t bb_size = 4096;
- uint64_t ccssize = mid->size / CCS_RATIO;
+ uint32_t bb, ccs, ccs2, *ccsmap, *ccsmap2;
+ uint64_t bb_size, ccssize = mid->size / CCS_RATIO;
uint32_t *ccscopy;
uint8_t uc_mocs = intel_get_uc_mocs(i915);
int result;
igt_assert(mid->compression);
ccscopy = (uint32_t *) malloc(ccssize);
+ bb_size = 4096;
bb = gem_create_from_pool(i915, &bb_size, REGION_SMEM);
ccs = gem_create(i915, ccssize);
+ ccs2 = gem_create(i915, ccssize);
surf.i915 = i915;
surf.print_bb = param.print_bb;
@@ -193,6 +196,33 @@ static void surf_copy(int i915,
PROT_READ | PROT_WRITE);
memcpy(ccscopy, ccsmap, ccssize);
+ if (suspend_resume) {
+ char *orig, *orig2, *newsum, *newsum2;
+
+ orig = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)ccsmap, surf.dst.size);
+ orig2 = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)mid->ptr, mid->size);
+
+ igt_system_suspend_autoresume(SUSPEND_STATE_FREEZE, SUSPEND_TEST_NONE);
+
+ set_surf_object(&surf.dst, ccs2, REGION_SMEM, ccssize,
+ 0, DIRECT_ACCESS);
+ blt_ctrl_surf_copy(i915, ctx, e, ahnd, &surf);
+ gem_sync(i915, surf.dst.handle);
+
+ ccsmap2 = gem_mmap__device_coherent(i915, ccs2, 0, surf.dst.size,
+ PROT_READ | PROT_WRITE);
+ newsum = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)ccsmap2, surf.dst.size);
+ newsum2 = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)mid->ptr, mid->size);
+
+ munmap(ccsmap2, ccssize);
+ igt_assert(!strcmp(orig, newsum));
+ igt_assert(!strcmp(orig2, newsum2));
+ g_free(orig);
+ g_free(orig2);
+ g_free(newsum);
+ g_free(newsum2);
+ }
+
/* corrupt ccs */
for (int i = 0; i < surf.dst.size / sizeof(uint32_t); i++)
ccsmap[i] = i;
@@ -209,6 +239,7 @@ static void surf_copy(int i915,
set_blt_object(&blt.dst, dst);
set_object_ext(&ext.src, mid->compression_type, mid->x2, mid->y2, SURFACE_TYPE_2D);
set_object_ext(&ext.dst, 0, dst->x2, dst->y2, SURFACE_TYPE_2D);
+ bb_size = 4096;
bb = gem_create_from_pool(i915, &bb_size, REGION_SMEM);
set_batch(&blt.bb, bb, bb_size, REGION_SMEM);
blt_block_copy(i915, ctx, e, ahnd, &blt, &ext);
@@ -311,7 +342,8 @@ static void block_copy(int i915,
ALLOC_STRATEGY_LOW_TO_HIGH, 0);
}
- surf_copy(i915, surf_ctx, &surf_e, surf_ahnd, src, mid, dst, run_id);
+ surf_copy(i915, surf_ctx, &surf_e, surf_ahnd, src, mid, dst, run_id,
+ config->suspend_resume);
if (surf_ctx != ctx) {
intel_ctx_destroy(i915, surf_ctx);
@@ -510,6 +542,15 @@ igt_main_args("bf:pst:W:H:", NULL, help_str, opt_handler, NULL)
block_copy_test(i915, &config, ctx, set);
}
+ igt_describe("Check flatccs data persists after suspend / resume (S0)");
+ igt_subtest_with_dynamic("suspend-resume") {
+ struct test_config config = { .compression = true,
+ .surfcopy = true,
+ .suspend_resume = true };
+
+ block_copy_test(i915, &config, ctx, set);
+ }
+
igt_fixture {
igt_disallow_hang(i915, hang);
close(i915);
--
2.32.0
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_ccs: Add suspend-resume subtest
2022-03-22 6:59 [igt-dev] [PATCH i-g-t 0/2] Add two gem-ccs subtests Zbigniew Kempczyński
@ 2022-03-22 6:59 ` Zbigniew Kempczyński
0 siblings, 0 replies; 16+ messages in thread
From: Zbigniew Kempczyński @ 2022-03-22 6:59 UTC (permalink / raw)
To: igt-dev
From: Apoorva Singh <apoorva1.singh@intel.com>
Verify flatccs data won't be corrupted when device will be put
to S0 (s2idle) state.
v2: addressing review comments (Kamil)
v3: fix memory leak (Kamil)
Signed-off-by: Apoorva Singh <apoorva1.singh@intel.com>
Cc: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
---
tests/i915/gem_ccs.c | 51 +++++++++++++++++++++++++++++++++++++++-----
1 file changed, 46 insertions(+), 5 deletions(-)
diff --git a/tests/i915/gem_ccs.c b/tests/i915/gem_ccs.c
index 28711bdc59..2747983925 100644
--- a/tests/i915/gem_ccs.c
+++ b/tests/i915/gem_ccs.c
@@ -4,6 +4,7 @@
*/
#include <errno.h>
+#include <glib.h>
#include <sys/ioctl.h>
#include <sys/time.h>
#include <malloc.h>
@@ -40,6 +41,7 @@ struct test_config {
bool inplace;
bool surfcopy;
bool new_ctx;
+ bool suspend_resume;
};
static void set_object(struct blt_copy_object *obj,
@@ -162,22 +164,23 @@ static void surf_copy(int i915,
const struct blt_copy_object *src,
const struct blt_copy_object *mid,
const struct blt_copy_object *dst,
- int run_id)
+ int run_id, bool suspend_resume)
{
struct blt_copy_data blt = {};
struct blt_block_copy_data_ext ext = {};
struct blt_ctrl_surf_copy_data surf = {};
- uint32_t bb, ccs, *ccsmap;
- uint64_t bb_size = 4096;
- uint64_t ccssize = mid->size / CCS_RATIO;
+ uint32_t bb, ccs, ccs2, *ccsmap, *ccsmap2;
+ uint64_t bb_size, ccssize = mid->size / CCS_RATIO;
uint32_t *ccscopy;
uint8_t uc_mocs = intel_get_uc_mocs(i915);
int result;
igt_assert(mid->compression);
ccscopy = (uint32_t *) malloc(ccssize);
+ bb_size = 4096;
bb = gem_create_from_pool(i915, &bb_size, REGION_SMEM);
ccs = gem_create(i915, ccssize);
+ ccs2 = gem_create(i915, ccssize);
surf.i915 = i915;
surf.print_bb = param.print_bb;
@@ -193,6 +196,33 @@ static void surf_copy(int i915,
PROT_READ | PROT_WRITE);
memcpy(ccscopy, ccsmap, ccssize);
+ if (suspend_resume) {
+ char *orig, *orig2, *newsum, *newsum2;
+
+ orig = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)ccsmap, surf.dst.size);
+ orig2 = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)mid->ptr, mid->size);
+
+ igt_system_suspend_autoresume(SUSPEND_STATE_FREEZE, SUSPEND_TEST_NONE);
+
+ set_surf_object(&surf.dst, ccs2, REGION_SMEM, ccssize,
+ 0, DIRECT_ACCESS);
+ blt_ctrl_surf_copy(i915, ctx, e, ahnd, &surf);
+ gem_sync(i915, surf.dst.handle);
+
+ ccsmap2 = gem_mmap__device_coherent(i915, ccs2, 0, surf.dst.size,
+ PROT_READ | PROT_WRITE);
+ newsum = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)ccsmap2, surf.dst.size);
+ newsum2 = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)mid->ptr, mid->size);
+
+ munmap(ccsmap2, ccssize);
+ igt_assert(!strcmp(orig, newsum));
+ igt_assert(!strcmp(orig2, newsum2));
+ g_free(orig);
+ g_free(orig2);
+ g_free(newsum);
+ g_free(newsum2);
+ }
+
/* corrupt ccs */
for (int i = 0; i < surf.dst.size / sizeof(uint32_t); i++)
ccsmap[i] = i;
@@ -209,6 +239,7 @@ static void surf_copy(int i915,
set_blt_object(&blt.dst, dst);
set_object_ext(&ext.src, mid->compression_type, mid->x2, mid->y2, SURFACE_TYPE_2D);
set_object_ext(&ext.dst, 0, dst->x2, dst->y2, SURFACE_TYPE_2D);
+ bb_size = 4096;
bb = gem_create_from_pool(i915, &bb_size, REGION_SMEM);
set_batch(&blt.bb, bb, bb_size, REGION_SMEM);
blt_block_copy(i915, ctx, e, ahnd, &blt, &ext);
@@ -311,7 +342,8 @@ static void block_copy(int i915,
ALLOC_STRATEGY_LOW_TO_HIGH, 0);
}
- surf_copy(i915, surf_ctx, &surf_e, surf_ahnd, src, mid, dst, run_id);
+ surf_copy(i915, surf_ctx, &surf_e, ahnd, src, mid, dst, run_id,
+ config->suspend_resume);
if (surf_ctx != ctx) {
intel_ctx_destroy(i915, surf_ctx);
@@ -510,6 +542,15 @@ igt_main_args("bf:pst:W:H:", NULL, help_str, opt_handler, NULL)
block_copy_test(i915, &config, ctx, set);
}
+ igt_describe("Check flatccs data persists after suspend / resume (S0)");
+ igt_subtest_with_dynamic("suspend-resume") {
+ struct test_config config = { .compression = true,
+ .surfcopy = true,
+ .suspend_resume = true };
+
+ block_copy_test(i915, &config, ctx, set);
+ }
+
igt_fixture {
igt_disallow_hang(i915, hang);
close(i915);
--
2.32.0
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_ccs: Add suspend-resume subtest
2022-03-21 7:11 [igt-dev] [PATCH i-g-t 0/2] Add two gem-ccs subtests Zbigniew Kempczyński
@ 2022-03-21 7:11 ` Zbigniew Kempczyński
0 siblings, 0 replies; 16+ messages in thread
From: Zbigniew Kempczyński @ 2022-03-21 7:11 UTC (permalink / raw)
To: igt-dev
From: Apoorva Singh <apoorva1.singh@intel.com>
Verify flatccs data won't be corrupted when device will be put
to S0 (s2idle) state.
v2: addressing review comments (Kamil)
Signed-off-by: Apoorva Singh <apoorva1.singh@intel.com>
Cc: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
---
tests/i915/gem_ccs.c | 46 +++++++++++++++++++++++++++++++++++++++-----
1 file changed, 41 insertions(+), 5 deletions(-)
diff --git a/tests/i915/gem_ccs.c b/tests/i915/gem_ccs.c
index 7e104f6ad8..1b58668ad0 100644
--- a/tests/i915/gem_ccs.c
+++ b/tests/i915/gem_ccs.c
@@ -4,6 +4,7 @@
*/
#include <errno.h>
+#include <glib.h>
#include <sys/ioctl.h>
#include <sys/time.h>
#include <malloc.h>
@@ -40,6 +41,7 @@ struct test_config {
bool inplace;
bool surfcopy;
bool new_ctx;
+ bool suspend_resume;
};
static void set_object(struct blt_copy_object *obj,
@@ -162,22 +164,23 @@ static void surf_copy(int i915,
const struct blt_copy_object *src,
const struct blt_copy_object *mid,
const struct blt_copy_object *dst,
- int run_id)
+ int run_id, bool suspend_resume)
{
struct blt_copy_data blt = {};
struct blt_block_copy_data_ext ext = {};
struct blt_ctrl_surf_copy_data surf = {};
- uint32_t bb, ccs, *ccsmap;
- uint64_t bb_size = 4096;
- uint64_t ccssize = mid->size / CCS_RATIO;
+ uint32_t bb, ccs, ccs2, *ccsmap, *ccsmap2;
+ uint64_t bb_size, ccssize = mid->size / CCS_RATIO;
uint32_t *ccscopy;
uint8_t uc_mocs = intel_get_uc_mocs(i915);
int result;
igt_assert(mid->compression);
ccscopy = (uint32_t *) malloc(ccssize);
+ bb_size = 4096;
bb = gem_create_from_pool(i915, &bb_size, REGION_SMEM);
ccs = gem_create(i915, ccssize);
+ ccs2 = gem_create(i915, ccssize);
surf.i915 = i915;
surf.print_bb = param.print_bb;
@@ -193,6 +196,29 @@ static void surf_copy(int i915,
PROT_READ | PROT_WRITE);
memcpy(ccscopy, ccsmap, ccssize);
+ if (suspend_resume) {
+ char *orig, *orig2, *newsum, *newsum2;
+
+ orig = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)ccsmap, surf.dst.size);
+ orig2 = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)mid->ptr, mid->size);
+
+ igt_system_suspend_autoresume(SUSPEND_STATE_FREEZE, SUSPEND_TEST_NONE);
+
+ set_surf_object(&surf.dst, ccs2, REGION_SMEM, ccssize,
+ 0, DIRECT_ACCESS);
+ blt_ctrl_surf_copy(i915, ctx, e, ahnd, &surf);
+ gem_sync(i915, surf.dst.handle);
+
+ ccsmap2 = gem_mmap__device_coherent(i915, ccs2, 0, surf.dst.size,
+ PROT_READ | PROT_WRITE);
+ newsum = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)ccsmap2, surf.dst.size);
+ newsum2 = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)mid->ptr, mid->size);
+
+ munmap(ccsmap2, ccssize);
+ igt_assert(!strcmp(orig, newsum));
+ igt_assert(!strcmp(orig2, newsum2));
+ }
+
/* corrupt ccs */
for (int i = 0; i < surf.dst.size / sizeof(uint32_t); i++)
ccsmap[i] = i;
@@ -209,6 +235,7 @@ static void surf_copy(int i915,
set_blt_object(&blt.dst, dst);
set_object_ext(&ext.src, mid->compression_type, mid->x2, mid->y2, SURFACE_TYPE_2D);
set_object_ext(&ext.dst, 0, dst->x2, dst->y2, SURFACE_TYPE_2D);
+ bb_size = 4096;
bb = gem_create_from_pool(i915, &bb_size, REGION_SMEM);
set_batch(&blt.bb, bb, bb_size, REGION_SMEM);
blt_block_copy(i915, ctx, e, ahnd, &blt, &ext);
@@ -311,7 +338,8 @@ static void block_copy(int i915,
ALLOC_STRATEGY_LOW_TO_HIGH, 0);
}
- surf_copy(i915, surf_ctx, &surf_e, surf_ahnd, src, mid, dst, run_id);
+ surf_copy(i915, surf_ctx, &surf_e, ahnd, src, mid, dst, run_id,
+ config->suspend_resume);
if (surf_ctx != ctx) {
intel_ctx_destroy(i915, surf_ctx);
@@ -508,6 +536,14 @@ igt_main_args("bf:pst:W:H:", NULL, help_str, opt_handler, NULL)
block_copy_test(i915, &config, ctx, set);
}
+ igt_subtest_with_dynamic("suspend-resume") {
+ struct test_config config = { .compression = true,
+ .surfcopy = true,
+ .suspend_resume = true };
+
+ block_copy_test(i915, &config, ctx, set);
+ }
+
igt_fixture {
igt_disallow_hang(i915, hang);
close(i915);
--
2.32.0
^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_ccs: Add suspend-resume subtest
2022-03-18 10:29 ` Kamil Konieczny
@ 2022-03-21 6:18 ` Zbigniew Kempczyński
0 siblings, 0 replies; 16+ messages in thread
From: Zbigniew Kempczyński @ 2022-03-21 6:18 UTC (permalink / raw)
To: Kamil Konieczny, igt-dev, Apoorva Singh
On Fri, Mar 18, 2022 at 11:29:53AM +0100, Kamil Konieczny wrote:
> Hi,
>
> Dnia 2022-03-18 at 08:30:18 +0100, Zbigniew Kempczyński napisał(a):
> > From: Apoorva Singh <apoorva1.singh@intel.com>
> >
> > Verify flatccs data won't be corrupted when device will be put
> > to S0 (s2idle) state.
> >
> > Signed-off-by: Apoorva Singh <apoorva1.singh@intel.com>
> > Cc: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
> > ---
> > tests/i915/gem_ccs.c | 45 +++++++++++++++++++++++++++++++++++++++-----
> > 1 file changed, 40 insertions(+), 5 deletions(-)
> >
> > diff --git a/tests/i915/gem_ccs.c b/tests/i915/gem_ccs.c
> > index 7e104f6ad8..eaa8e043b0 100644
> > --- a/tests/i915/gem_ccs.c
> > +++ b/tests/i915/gem_ccs.c
> > @@ -4,6 +4,7 @@
> > */
> >
> > #include <errno.h>
> > +#include <glib.h>
> > #include <sys/ioctl.h>
> > #include <sys/time.h>
> > #include <malloc.h>
> > @@ -40,6 +41,7 @@ struct test_config {
> > bool inplace;
> > bool surfcopy;
> > bool new_ctx;
> > + bool suspend_resume;
> > };
> >
> > static void set_object(struct blt_copy_object *obj,
> > @@ -162,22 +164,24 @@ static void surf_copy(int i915,
> > const struct blt_copy_object *src,
> > const struct blt_copy_object *mid,
> > const struct blt_copy_object *dst,
> > - int run_id)
> > + int run_id, bool suspend_resume)
> > {
> > struct blt_copy_data blt = {};
> > struct blt_block_copy_data_ext ext = {};
> > struct blt_ctrl_surf_copy_data surf = {};
> > - uint32_t bb, ccs, *ccsmap;
> > - uint64_t bb_size = 4096;
>
> Please keep it here so number of changes will be lower.
>
> > - uint64_t ccssize = mid->size / CCS_RATIO;
> > + uint32_t bb, ccs, ccs2, *ccsmap, *ccsmap2;
> > + uint64_t bb_size, ccssize = mid->size / CCS_RATIO;
>
> No need to move bb_size here.
>
> > uint32_t *ccscopy;
> > uint8_t uc_mocs = intel_get_uc_mocs(i915);
> > int result;
> > + char *orig, *orig2, *new, *new2;
>
> These are used only in suspend_resume block, so move it there.
Ack, makes sense.
>
> >
> > igt_assert(mid->compression);
> > ccscopy = (uint32_t *) malloc(ccssize);
> > + bb_size = 4096;
>
> May be removed (if you keep it above).
My intent was to point the reader that bb_size can alter by the driver
regarding created buffer size. For smem there's 4k, but for lmem can
vary. gem_create_from_pool() is new call, so I try to build some
patterns to avoid mismatch between sizes and allocator calls.
>
> > bb = gem_create_from_pool(i915, &bb_size, REGION_SMEM);
> > ccs = gem_create(i915, ccssize);
> > + ccs2 = gem_create(i915, ccssize);
> >
> > surf.i915 = i915;
> > surf.print_bb = param.print_bb;
> > @@ -193,6 +197,27 @@ static void surf_copy(int i915,
> > PROT_READ | PROT_WRITE);
> > memcpy(ccscopy, ccsmap, ccssize);
> >
> > + if (suspend_resume) {
> > + orig = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)ccsmap, surf.dst.size);
> > + orig2 = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)mid->ptr, mid->size);
> > +
> > + igt_system_suspend_autoresume(SUSPEND_STATE_FREEZE, SUSPEND_TEST_NONE);
> > +
> > + set_surf_object(&surf.dst, ccs2, REGION_SMEM, ccssize,
> > + 0, DIRECT_ACCESS);
> > + blt_ctrl_surf_copy(i915, ctx, e, ahnd, &surf);
> > + gem_sync(i915, surf.dst.handle);
> > +
> > + ccsmap2 = gem_mmap__device_coherent(i915, ccs2, 0, surf.dst.size,
> > + PROT_READ | PROT_WRITE);
> > + new = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)ccsmap2, surf.dst.size);
> > + new2 = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)mid->ptr, mid->size);
> > +
> > + igt_assert(!strcmp(orig, new));
> > + igt_assert(!strcmp(orig2, new2));
>
> May you rename new and new2 into something meaningfull ? Like
> newsum_ccs, newsum_ctrl ? It can help if assert triggers.
> Checksums are pointers allocated by g_compute_checksum so they
> need to be free after use.
Ok, will do.
>
> > + munmap(ccsmap2, ccssize);
>
> Move this before two asserts.
Ack.
>
> > + }
> > +
> > /* corrupt ccs */
> > for (int i = 0; i < surf.dst.size / sizeof(uint32_t); i++)
> > ccsmap[i] = i;
> > @@ -209,6 +234,7 @@ static void surf_copy(int i915,
> > set_blt_object(&blt.dst, dst);
> > set_object_ext(&ext.src, mid->compression_type, mid->x2, mid->y2, SURFACE_TYPE_2D);
> > set_object_ext(&ext.dst, 0, dst->x2, dst->y2, SURFACE_TYPE_2D);
> > + bb_size = 4096;
>
> Looks strange here, remove this.
I'm not sure what bb_size is before gem_create_from_pool() - it may be altered by
previous call (not for smem but lmem likely).
>
> > bb = gem_create_from_pool(i915, &bb_size, REGION_SMEM);
> > set_batch(&blt.bb, bb, bb_size, REGION_SMEM);
> > blt_block_copy(i915, ctx, e, ahnd, &blt, &ext);
> > @@ -311,7 +337,8 @@ static void block_copy(int i915,
> > ALLOC_STRATEGY_LOW_TO_HIGH, 0);
> > }
> >
> > - surf_copy(i915, surf_ctx, &surf_e, surf_ahnd, src, mid, dst, run_id);
> > + surf_copy(i915, surf_ctx, &surf_e, ahnd, src, mid, dst, run_id,
>
> This should be in first patch ?
Hmm, likely not. First (new ctx) changes to use &surf_e. For suspend-resume there's
new option which is additional arg for surf_copy.
Thanks for the review, new series will be sent soon.
--
Zbigniew
>
> Regards,
> Kamil
>
> > + config->suspend_resume);
> >
> > if (surf_ctx != ctx) {
> > intel_ctx_destroy(i915, surf_ctx);
> > @@ -508,6 +535,14 @@ igt_main_args("bf:pst:W:H:", NULL, help_str, opt_handler, NULL)
> > block_copy_test(i915, &config, ctx, set);
> > }
> >
> > + igt_subtest_with_dynamic("suspend-resume") {
> > + struct test_config config = { .compression = true,
> > + .surfcopy = true,
> > + .suspend_resume = true };
> > +
> > + block_copy_test(i915, &config, ctx, set);
> > + }
> > +
> > igt_fixture {
> > igt_disallow_hang(i915, hang);
> > close(i915);
> > --
> > 2.32.0
> >
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_ccs: Add suspend-resume subtest
2022-03-18 7:30 ` [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_ccs: Add suspend-resume subtest Zbigniew Kempczyński
@ 2022-03-18 10:29 ` Kamil Konieczny
2022-03-21 6:18 ` Zbigniew Kempczyński
0 siblings, 1 reply; 16+ messages in thread
From: Kamil Konieczny @ 2022-03-18 10:29 UTC (permalink / raw)
To: igt-dev
Hi,
Dnia 2022-03-18 at 08:30:18 +0100, Zbigniew Kempczyński napisał(a):
> From: Apoorva Singh <apoorva1.singh@intel.com>
>
> Verify flatccs data won't be corrupted when device will be put
> to S0 (s2idle) state.
>
> Signed-off-by: Apoorva Singh <apoorva1.singh@intel.com>
> Cc: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
> ---
> tests/i915/gem_ccs.c | 45 +++++++++++++++++++++++++++++++++++++++-----
> 1 file changed, 40 insertions(+), 5 deletions(-)
>
> diff --git a/tests/i915/gem_ccs.c b/tests/i915/gem_ccs.c
> index 7e104f6ad8..eaa8e043b0 100644
> --- a/tests/i915/gem_ccs.c
> +++ b/tests/i915/gem_ccs.c
> @@ -4,6 +4,7 @@
> */
>
> #include <errno.h>
> +#include <glib.h>
> #include <sys/ioctl.h>
> #include <sys/time.h>
> #include <malloc.h>
> @@ -40,6 +41,7 @@ struct test_config {
> bool inplace;
> bool surfcopy;
> bool new_ctx;
> + bool suspend_resume;
> };
>
> static void set_object(struct blt_copy_object *obj,
> @@ -162,22 +164,24 @@ static void surf_copy(int i915,
> const struct blt_copy_object *src,
> const struct blt_copy_object *mid,
> const struct blt_copy_object *dst,
> - int run_id)
> + int run_id, bool suspend_resume)
> {
> struct blt_copy_data blt = {};
> struct blt_block_copy_data_ext ext = {};
> struct blt_ctrl_surf_copy_data surf = {};
> - uint32_t bb, ccs, *ccsmap;
> - uint64_t bb_size = 4096;
Please keep it here so number of changes will be lower.
> - uint64_t ccssize = mid->size / CCS_RATIO;
> + uint32_t bb, ccs, ccs2, *ccsmap, *ccsmap2;
> + uint64_t bb_size, ccssize = mid->size / CCS_RATIO;
No need to move bb_size here.
> uint32_t *ccscopy;
> uint8_t uc_mocs = intel_get_uc_mocs(i915);
> int result;
> + char *orig, *orig2, *new, *new2;
These are used only in suspend_resume block, so move it there.
>
> igt_assert(mid->compression);
> ccscopy = (uint32_t *) malloc(ccssize);
> + bb_size = 4096;
May be removed (if you keep it above).
> bb = gem_create_from_pool(i915, &bb_size, REGION_SMEM);
> ccs = gem_create(i915, ccssize);
> + ccs2 = gem_create(i915, ccssize);
>
> surf.i915 = i915;
> surf.print_bb = param.print_bb;
> @@ -193,6 +197,27 @@ static void surf_copy(int i915,
> PROT_READ | PROT_WRITE);
> memcpy(ccscopy, ccsmap, ccssize);
>
> + if (suspend_resume) {
> + orig = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)ccsmap, surf.dst.size);
> + orig2 = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)mid->ptr, mid->size);
> +
> + igt_system_suspend_autoresume(SUSPEND_STATE_FREEZE, SUSPEND_TEST_NONE);
> +
> + set_surf_object(&surf.dst, ccs2, REGION_SMEM, ccssize,
> + 0, DIRECT_ACCESS);
> + blt_ctrl_surf_copy(i915, ctx, e, ahnd, &surf);
> + gem_sync(i915, surf.dst.handle);
> +
> + ccsmap2 = gem_mmap__device_coherent(i915, ccs2, 0, surf.dst.size,
> + PROT_READ | PROT_WRITE);
> + new = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)ccsmap2, surf.dst.size);
> + new2 = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)mid->ptr, mid->size);
> +
> + igt_assert(!strcmp(orig, new));
> + igt_assert(!strcmp(orig2, new2));
May you rename new and new2 into something meaningfull ? Like
newsum_ccs, newsum_ctrl ? It can help if assert triggers.
Checksums are pointers allocated by g_compute_checksum so they
need to be free after use.
> + munmap(ccsmap2, ccssize);
Move this before two asserts.
> + }
> +
> /* corrupt ccs */
> for (int i = 0; i < surf.dst.size / sizeof(uint32_t); i++)
> ccsmap[i] = i;
> @@ -209,6 +234,7 @@ static void surf_copy(int i915,
> set_blt_object(&blt.dst, dst);
> set_object_ext(&ext.src, mid->compression_type, mid->x2, mid->y2, SURFACE_TYPE_2D);
> set_object_ext(&ext.dst, 0, dst->x2, dst->y2, SURFACE_TYPE_2D);
> + bb_size = 4096;
Looks strange here, remove this.
> bb = gem_create_from_pool(i915, &bb_size, REGION_SMEM);
> set_batch(&blt.bb, bb, bb_size, REGION_SMEM);
> blt_block_copy(i915, ctx, e, ahnd, &blt, &ext);
> @@ -311,7 +337,8 @@ static void block_copy(int i915,
> ALLOC_STRATEGY_LOW_TO_HIGH, 0);
> }
>
> - surf_copy(i915, surf_ctx, &surf_e, surf_ahnd, src, mid, dst, run_id);
> + surf_copy(i915, surf_ctx, &surf_e, ahnd, src, mid, dst, run_id,
This should be in first patch ?
Regards,
Kamil
> + config->suspend_resume);
>
> if (surf_ctx != ctx) {
> intel_ctx_destroy(i915, surf_ctx);
> @@ -508,6 +535,14 @@ igt_main_args("bf:pst:W:H:", NULL, help_str, opt_handler, NULL)
> block_copy_test(i915, &config, ctx, set);
> }
>
> + igt_subtest_with_dynamic("suspend-resume") {
> + struct test_config config = { .compression = true,
> + .surfcopy = true,
> + .suspend_resume = true };
> +
> + block_copy_test(i915, &config, ctx, set);
> + }
> +
> igt_fixture {
> igt_disallow_hang(i915, hang);
> close(i915);
> --
> 2.32.0
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_ccs: Add suspend-resume subtest
2022-03-18 7:30 [igt-dev] [PATCH i-g-t 0/2] Add two gem-ccs subtests Zbigniew Kempczyński
@ 2022-03-18 7:30 ` Zbigniew Kempczyński
2022-03-18 10:29 ` Kamil Konieczny
0 siblings, 1 reply; 16+ messages in thread
From: Zbigniew Kempczyński @ 2022-03-18 7:30 UTC (permalink / raw)
To: igt-dev
From: Apoorva Singh <apoorva1.singh@intel.com>
Verify flatccs data won't be corrupted when device will be put
to S0 (s2idle) state.
Signed-off-by: Apoorva Singh <apoorva1.singh@intel.com>
Cc: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
---
tests/i915/gem_ccs.c | 45 +++++++++++++++++++++++++++++++++++++++-----
1 file changed, 40 insertions(+), 5 deletions(-)
diff --git a/tests/i915/gem_ccs.c b/tests/i915/gem_ccs.c
index 7e104f6ad8..eaa8e043b0 100644
--- a/tests/i915/gem_ccs.c
+++ b/tests/i915/gem_ccs.c
@@ -4,6 +4,7 @@
*/
#include <errno.h>
+#include <glib.h>
#include <sys/ioctl.h>
#include <sys/time.h>
#include <malloc.h>
@@ -40,6 +41,7 @@ struct test_config {
bool inplace;
bool surfcopy;
bool new_ctx;
+ bool suspend_resume;
};
static void set_object(struct blt_copy_object *obj,
@@ -162,22 +164,24 @@ static void surf_copy(int i915,
const struct blt_copy_object *src,
const struct blt_copy_object *mid,
const struct blt_copy_object *dst,
- int run_id)
+ int run_id, bool suspend_resume)
{
struct blt_copy_data blt = {};
struct blt_block_copy_data_ext ext = {};
struct blt_ctrl_surf_copy_data surf = {};
- uint32_t bb, ccs, *ccsmap;
- uint64_t bb_size = 4096;
- uint64_t ccssize = mid->size / CCS_RATIO;
+ uint32_t bb, ccs, ccs2, *ccsmap, *ccsmap2;
+ uint64_t bb_size, ccssize = mid->size / CCS_RATIO;
uint32_t *ccscopy;
uint8_t uc_mocs = intel_get_uc_mocs(i915);
int result;
+ char *orig, *orig2, *new, *new2;
igt_assert(mid->compression);
ccscopy = (uint32_t *) malloc(ccssize);
+ bb_size = 4096;
bb = gem_create_from_pool(i915, &bb_size, REGION_SMEM);
ccs = gem_create(i915, ccssize);
+ ccs2 = gem_create(i915, ccssize);
surf.i915 = i915;
surf.print_bb = param.print_bb;
@@ -193,6 +197,27 @@ static void surf_copy(int i915,
PROT_READ | PROT_WRITE);
memcpy(ccscopy, ccsmap, ccssize);
+ if (suspend_resume) {
+ orig = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)ccsmap, surf.dst.size);
+ orig2 = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)mid->ptr, mid->size);
+
+ igt_system_suspend_autoresume(SUSPEND_STATE_FREEZE, SUSPEND_TEST_NONE);
+
+ set_surf_object(&surf.dst, ccs2, REGION_SMEM, ccssize,
+ 0, DIRECT_ACCESS);
+ blt_ctrl_surf_copy(i915, ctx, e, ahnd, &surf);
+ gem_sync(i915, surf.dst.handle);
+
+ ccsmap2 = gem_mmap__device_coherent(i915, ccs2, 0, surf.dst.size,
+ PROT_READ | PROT_WRITE);
+ new = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)ccsmap2, surf.dst.size);
+ new2 = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)mid->ptr, mid->size);
+
+ igt_assert(!strcmp(orig, new));
+ igt_assert(!strcmp(orig2, new2));
+ munmap(ccsmap2, ccssize);
+ }
+
/* corrupt ccs */
for (int i = 0; i < surf.dst.size / sizeof(uint32_t); i++)
ccsmap[i] = i;
@@ -209,6 +234,7 @@ static void surf_copy(int i915,
set_blt_object(&blt.dst, dst);
set_object_ext(&ext.src, mid->compression_type, mid->x2, mid->y2, SURFACE_TYPE_2D);
set_object_ext(&ext.dst, 0, dst->x2, dst->y2, SURFACE_TYPE_2D);
+ bb_size = 4096;
bb = gem_create_from_pool(i915, &bb_size, REGION_SMEM);
set_batch(&blt.bb, bb, bb_size, REGION_SMEM);
blt_block_copy(i915, ctx, e, ahnd, &blt, &ext);
@@ -311,7 +337,8 @@ static void block_copy(int i915,
ALLOC_STRATEGY_LOW_TO_HIGH, 0);
}
- surf_copy(i915, surf_ctx, &surf_e, surf_ahnd, src, mid, dst, run_id);
+ surf_copy(i915, surf_ctx, &surf_e, ahnd, src, mid, dst, run_id,
+ config->suspend_resume);
if (surf_ctx != ctx) {
intel_ctx_destroy(i915, surf_ctx);
@@ -508,6 +535,14 @@ igt_main_args("bf:pst:W:H:", NULL, help_str, opt_handler, NULL)
block_copy_test(i915, &config, ctx, set);
}
+ igt_subtest_with_dynamic("suspend-resume") {
+ struct test_config config = { .compression = true,
+ .surfcopy = true,
+ .suspend_resume = true };
+
+ block_copy_test(i915, &config, ctx, set);
+ }
+
igt_fixture {
igt_disallow_hang(i915, hang);
close(i915);
--
2.32.0
^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_ccs: Add suspend-resume subtest
2022-03-17 10:20 ` [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_ccs: Add suspend-resume subtest Zbigniew Kempczyński
@ 2022-03-17 16:46 ` Zbigniew Kempczyński
0 siblings, 0 replies; 16+ messages in thread
From: Zbigniew Kempczyński @ 2022-03-17 16:46 UTC (permalink / raw)
To: igt-dev
On Thu, Mar 17, 2022 at 11:20:02AM +0100, Zbigniew Kempczyński wrote:
> From: Apoorva Singh <apoorva1.singh@intel.com>
>
> Verify flatccs data won't be corrupted when device will be put
> to S0 (s2idle) state.
>
> Signed-off-by: Apoorva Singh <apoorva1.singh@intel.com>
> Cc: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
Looks good for me:
Reviewed-by: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
--
Zbigniew
> ---
> tests/i915/gem_ccs.c | 45 +++++++++++++++++++++++++++++++++++++++-----
> 1 file changed, 40 insertions(+), 5 deletions(-)
>
> diff --git a/tests/i915/gem_ccs.c b/tests/i915/gem_ccs.c
> index 1a21e24597..5bf11c9a8e 100644
> --- a/tests/i915/gem_ccs.c
> +++ b/tests/i915/gem_ccs.c
> @@ -4,6 +4,7 @@
> */
>
> #include <errno.h>
> +#include <glib.h>
> #include <sys/ioctl.h>
> #include <sys/time.h>
> #include <malloc.h>
> @@ -40,6 +41,7 @@ struct test_config {
> bool inplace;
> bool surfcopy;
> bool new_ctx;
> + bool suspend_resume;
> };
>
> static void set_object(struct blt_copy_object *obj,
> @@ -162,22 +164,24 @@ static void surf_copy(int i915,
> const struct blt_copy_object *src,
> const struct blt_copy_object *mid,
> const struct blt_copy_object *dst,
> - int run_id)
> + int run_id, bool suspend_resume)
> {
> struct blt_copy_data blt = {};
> struct blt_block_copy_data_ext ext = {};
> struct blt_ctrl_surf_copy_data surf = {};
> - uint32_t bb, ccs, *ccsmap;
> - uint64_t bb_size = 4096;
> - uint64_t ccssize = mid->size / CCS_RATIO;
> + uint32_t bb, ccs, ccs2, *ccsmap, *ccsmap2;
> + uint64_t bb_size, ccssize = mid->size / CCS_RATIO;
> uint32_t *ccscopy;
> uint8_t uc_mocs = intel_get_uc_mocs(i915);
> int result;
> + char *orig, *orig2, *new, *new2;
>
> igt_assert(mid->compression);
> ccscopy = (uint32_t *) malloc(ccssize);
> + bb_size = 4096;
> bb = gem_create_from_pool(i915, &bb_size, REGION_SMEM);
> ccs = gem_create(i915, ccssize);
> + ccs2 = gem_create(i915, ccssize);
>
> surf.i915 = i915;
> surf.print_bb = param.print_bb;
> @@ -193,6 +197,27 @@ static void surf_copy(int i915,
> PROT_READ | PROT_WRITE);
> memcpy(ccscopy, ccsmap, ccssize);
>
> + if (suspend_resume) {
> + orig = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)ccsmap, surf.dst.size);
> + orig2 = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)mid->ptr, mid->size);
> +
> + igt_system_suspend_autoresume(SUSPEND_STATE_FREEZE, SUSPEND_TEST_NONE);
> +
> + set_surf_object(&surf.dst, ccs2, REGION_SMEM, ccssize,
> + 0, DIRECT_ACCESS);
> + blt_ctrl_surf_copy(i915, ctx, e, ahnd, &surf);
> + gem_sync(i915, surf.dst.handle);
> +
> + ccsmap2 = gem_mmap__device_coherent(i915, ccs2, 0, surf.dst.size,
> + PROT_READ | PROT_WRITE);
> + new = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)ccsmap2, surf.dst.size);
> + new2 = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)mid->ptr, mid->size);
> +
> + igt_assert(!strcmp(orig, new));
> + igt_assert(!strcmp(orig2, new2));
> + munmap(ccsmap2, ccssize);
> + }
> +
> /* corrupt ccs */
> for (int i = 0; i < surf.dst.size / sizeof(uint32_t); i++)
> ccsmap[i] = i;
> @@ -209,6 +234,7 @@ static void surf_copy(int i915,
> set_blt_object(&blt.dst, dst);
> set_object_ext(&ext.src, mid->compression_type, mid->x2, mid->y2, SURFACE_TYPE_2D);
> set_object_ext(&ext.dst, 0, dst->x2, dst->y2, SURFACE_TYPE_2D);
> + bb_size = 4096;
> bb = gem_create_from_pool(i915, &bb_size, REGION_SMEM);
> set_batch(&blt.bb, bb, bb_size, REGION_SMEM);
> blt_block_copy(i915, ctx, e, ahnd, &blt, &ext);
> @@ -299,7 +325,8 @@ static void block_copy(int i915,
> if (config->new_ctx)
> surf_ctx = intel_ctx_create(i915, &ctx->cfg);
>
> - surf_copy(i915, surf_ctx, e, ahnd, src, mid, dst, run_id);
> + surf_copy(i915, surf_ctx, e, ahnd, src, mid, dst, run_id,
> + config->suspend_resume);
>
> if (surf_ctx != ctx)
> intel_ctx_destroy(i915, surf_ctx);
> @@ -494,6 +521,14 @@ igt_main_args("bf:pst:W:H:", NULL, help_str, opt_handler, NULL)
> block_copy_test(i915, &config, ctx, set);
> }
>
> + igt_subtest_with_dynamic("suspend-resume") {
> + struct test_config config = { .compression = true,
> + .surfcopy = true,
> + .suspend_resume = true };
> +
> + block_copy_test(i915, &config, ctx, set);
> + }
> +
> igt_fixture {
> igt_disallow_hang(i915, hang);
> close(i915);
> --
> 2.32.0
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_ccs: Add suspend-resume subtest
2022-03-17 10:20 [igt-dev] [PATCH i-g-t 0/2] Add two gem-ccs subtests Zbigniew Kempczyński
@ 2022-03-17 10:20 ` Zbigniew Kempczyński
2022-03-17 16:46 ` Zbigniew Kempczyński
0 siblings, 1 reply; 16+ messages in thread
From: Zbigniew Kempczyński @ 2022-03-17 10:20 UTC (permalink / raw)
To: igt-dev
From: Apoorva Singh <apoorva1.singh@intel.com>
Verify flatccs data won't be corrupted when device will be put
to S0 (s2idle) state.
Signed-off-by: Apoorva Singh <apoorva1.singh@intel.com>
Cc: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
---
tests/i915/gem_ccs.c | 45 +++++++++++++++++++++++++++++++++++++++-----
1 file changed, 40 insertions(+), 5 deletions(-)
diff --git a/tests/i915/gem_ccs.c b/tests/i915/gem_ccs.c
index 1a21e24597..5bf11c9a8e 100644
--- a/tests/i915/gem_ccs.c
+++ b/tests/i915/gem_ccs.c
@@ -4,6 +4,7 @@
*/
#include <errno.h>
+#include <glib.h>
#include <sys/ioctl.h>
#include <sys/time.h>
#include <malloc.h>
@@ -40,6 +41,7 @@ struct test_config {
bool inplace;
bool surfcopy;
bool new_ctx;
+ bool suspend_resume;
};
static void set_object(struct blt_copy_object *obj,
@@ -162,22 +164,24 @@ static void surf_copy(int i915,
const struct blt_copy_object *src,
const struct blt_copy_object *mid,
const struct blt_copy_object *dst,
- int run_id)
+ int run_id, bool suspend_resume)
{
struct blt_copy_data blt = {};
struct blt_block_copy_data_ext ext = {};
struct blt_ctrl_surf_copy_data surf = {};
- uint32_t bb, ccs, *ccsmap;
- uint64_t bb_size = 4096;
- uint64_t ccssize = mid->size / CCS_RATIO;
+ uint32_t bb, ccs, ccs2, *ccsmap, *ccsmap2;
+ uint64_t bb_size, ccssize = mid->size / CCS_RATIO;
uint32_t *ccscopy;
uint8_t uc_mocs = intel_get_uc_mocs(i915);
int result;
+ char *orig, *orig2, *new, *new2;
igt_assert(mid->compression);
ccscopy = (uint32_t *) malloc(ccssize);
+ bb_size = 4096;
bb = gem_create_from_pool(i915, &bb_size, REGION_SMEM);
ccs = gem_create(i915, ccssize);
+ ccs2 = gem_create(i915, ccssize);
surf.i915 = i915;
surf.print_bb = param.print_bb;
@@ -193,6 +197,27 @@ static void surf_copy(int i915,
PROT_READ | PROT_WRITE);
memcpy(ccscopy, ccsmap, ccssize);
+ if (suspend_resume) {
+ orig = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)ccsmap, surf.dst.size);
+ orig2 = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)mid->ptr, mid->size);
+
+ igt_system_suspend_autoresume(SUSPEND_STATE_FREEZE, SUSPEND_TEST_NONE);
+
+ set_surf_object(&surf.dst, ccs2, REGION_SMEM, ccssize,
+ 0, DIRECT_ACCESS);
+ blt_ctrl_surf_copy(i915, ctx, e, ahnd, &surf);
+ gem_sync(i915, surf.dst.handle);
+
+ ccsmap2 = gem_mmap__device_coherent(i915, ccs2, 0, surf.dst.size,
+ PROT_READ | PROT_WRITE);
+ new = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)ccsmap2, surf.dst.size);
+ new2 = g_compute_checksum_for_data(G_CHECKSUM_SHA1, (void *)mid->ptr, mid->size);
+
+ igt_assert(!strcmp(orig, new));
+ igt_assert(!strcmp(orig2, new2));
+ munmap(ccsmap2, ccssize);
+ }
+
/* corrupt ccs */
for (int i = 0; i < surf.dst.size / sizeof(uint32_t); i++)
ccsmap[i] = i;
@@ -209,6 +234,7 @@ static void surf_copy(int i915,
set_blt_object(&blt.dst, dst);
set_object_ext(&ext.src, mid->compression_type, mid->x2, mid->y2, SURFACE_TYPE_2D);
set_object_ext(&ext.dst, 0, dst->x2, dst->y2, SURFACE_TYPE_2D);
+ bb_size = 4096;
bb = gem_create_from_pool(i915, &bb_size, REGION_SMEM);
set_batch(&blt.bb, bb, bb_size, REGION_SMEM);
blt_block_copy(i915, ctx, e, ahnd, &blt, &ext);
@@ -299,7 +325,8 @@ static void block_copy(int i915,
if (config->new_ctx)
surf_ctx = intel_ctx_create(i915, &ctx->cfg);
- surf_copy(i915, surf_ctx, e, ahnd, src, mid, dst, run_id);
+ surf_copy(i915, surf_ctx, e, ahnd, src, mid, dst, run_id,
+ config->suspend_resume);
if (surf_ctx != ctx)
intel_ctx_destroy(i915, surf_ctx);
@@ -494,6 +521,14 @@ igt_main_args("bf:pst:W:H:", NULL, help_str, opt_handler, NULL)
block_copy_test(i915, &config, ctx, set);
}
+ igt_subtest_with_dynamic("suspend-resume") {
+ struct test_config config = { .compression = true,
+ .surfcopy = true,
+ .suspend_resume = true };
+
+ block_copy_test(i915, &config, ctx, set);
+ }
+
igt_fixture {
igt_disallow_hang(i915, hang);
close(i915);
--
2.32.0
^ permalink raw reply related [flat|nested] 16+ messages in thread
end of thread, other threads:[~2022-03-22 11:56 UTC | newest]
Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-21 11:11 [igt-dev] [PATCH i-g-t 0/2] Add two gem-ccs subtests Zbigniew Kempczyński
2022-03-21 11:11 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_ccs: Check ctrl-surf-copy in new context Zbigniew Kempczyński
2022-03-21 16:00 ` Kamil Konieczny
2022-03-21 11:11 ` [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_ccs: Add suspend-resume subtest Zbigniew Kempczyński
2022-03-21 16:04 ` Kamil Konieczny
2022-03-21 12:20 ` [igt-dev] ✓ Fi.CI.BAT: success for Add two gem-ccs subtests (rev4) Patchwork
2022-03-21 15:00 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
-- strict thread matches above, loose matches on Subject: below --
2022-03-22 8:25 [igt-dev] [PATCH i-g-t 0/2] Add two gem-ccs subtests Zbigniew Kempczyński
2022-03-22 8:25 ` [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_ccs: Add suspend-resume subtest Zbigniew Kempczyński
2022-03-22 11:56 ` Kamil Konieczny
2022-03-22 6:59 [igt-dev] [PATCH i-g-t 0/2] Add two gem-ccs subtests Zbigniew Kempczyński
2022-03-22 6:59 ` [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_ccs: Add suspend-resume subtest Zbigniew Kempczyński
2022-03-21 7:11 [igt-dev] [PATCH i-g-t 0/2] Add two gem-ccs subtests Zbigniew Kempczyński
2022-03-21 7:11 ` [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_ccs: Add suspend-resume subtest Zbigniew Kempczyński
2022-03-18 7:30 [igt-dev] [PATCH i-g-t 0/2] Add two gem-ccs subtests Zbigniew Kempczyński
2022-03-18 7:30 ` [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_ccs: Add suspend-resume subtest Zbigniew Kempczyński
2022-03-18 10:29 ` Kamil Konieczny
2022-03-21 6:18 ` Zbigniew Kempczyński
2022-03-17 10:20 [igt-dev] [PATCH i-g-t 0/2] Add two gem-ccs subtests Zbigniew Kempczyński
2022-03-17 10:20 ` [igt-dev] [PATCH i-g-t 2/2] tests/i915/gem_ccs: Add suspend-resume subtest Zbigniew Kempczyński
2022-03-17 16:46 ` Zbigniew Kempczyński
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.