* [igt-dev] [PATCH i-g-t v2 0/3] tests/i915/perf: verify non zero reason field in OA reports
@ 2020-11-25 11:25 Lionel Landwerlin
2020-11-25 11:25 ` [igt-dev] [PATCH i-g-t v2 1/3] tests/i915/perf: constify report checks functions Lionel Landwerlin
` (4 more replies)
0 siblings, 5 replies; 11+ messages in thread
From: Lionel Landwerlin @ 2020-11-25 11:25 UTC (permalink / raw)
To: igt-dev
Hi all,
In the process of checking those reason values, I found out about the
reports with the same timestamps.
Thanks a lot to Umesh for suggesting those changes.
Cheers,
Lionel Landwerlin (3):
tests/i915/perf: constify report checks functions
tests/i915/perf: deal with 0 delta reports
tests/i915/perf: verify reason field in OA reports is never 0
tests/i915/perf.c | 201 ++++++++++++++++++++++++++++++++++------------
1 file changed, 151 insertions(+), 50 deletions(-)
--
2.29.2
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 11+ messages in thread
* [igt-dev] [PATCH i-g-t v2 1/3] tests/i915/perf: constify report checks functions
2020-11-25 11:25 [igt-dev] [PATCH i-g-t v2 0/3] tests/i915/perf: verify non zero reason field in OA reports Lionel Landwerlin
@ 2020-11-25 11:25 ` Lionel Landwerlin
2020-11-25 21:43 ` Umesh Nerlige Ramappa
2020-11-25 11:25 ` [igt-dev] [PATCH i-g-t v2 2/3] tests/i915/perf: deal with 0 delta reports Lionel Landwerlin
` (3 subsequent siblings)
4 siblings, 1 reply; 11+ messages in thread
From: Lionel Landwerlin @ 2020-11-25 11:25 UTC (permalink / raw)
To: igt-dev
Those don't need to modify the reports they inspect.
Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
---
tests/i915/perf.c | 20 ++++++++++++--------
1 file changed, 12 insertions(+), 8 deletions(-)
diff --git a/tests/i915/perf.c b/tests/i915/perf.c
index caeabd623..2fbea0f0c 100644
--- a/tests/i915/perf.c
+++ b/tests/i915/perf.c
@@ -206,9 +206,10 @@ static bool *undefined_a_counters;
static uint64_t oa_exp_1_millisec;
static igt_render_copyfunc_t render_copy = NULL;
-static uint32_t (*read_report_ticks)(uint32_t *report,
+static uint32_t (*read_report_ticks)(const uint32_t *report,
enum drm_i915_oa_format format);
-static void (*sanity_check_reports)(uint32_t *oa_report0, uint32_t *oa_report1,
+static void (*sanity_check_reports)(const uint32_t *oa_report0,
+ const uint32_t *oa_report1,
enum drm_i915_oa_format format);
static void
@@ -354,7 +355,7 @@ sysfs_read(const char *path)
* but it's not included in all of the formats.
*/
static uint32_t
-hsw_read_report_ticks(uint32_t *report, enum drm_i915_oa_format format)
+hsw_read_report_ticks(const uint32_t *report, enum drm_i915_oa_format format)
{
uint32_t *c = (uint32_t *)(((uint8_t *)report) + get_oa_format(format).c_off);
@@ -364,13 +365,13 @@ hsw_read_report_ticks(uint32_t *report, enum drm_i915_oa_format format)
}
static uint32_t
-gen8_read_report_ticks(uint32_t *report, enum drm_i915_oa_format format)
+gen8_read_report_ticks(const uint32_t *report, enum drm_i915_oa_format format)
{
return report[3];
}
static void
-gen8_read_report_clock_ratios(uint32_t *report,
+gen8_read_report_clock_ratios(const uint32_t *report,
uint32_t *slice_freq_mhz,
uint32_t *unslice_freq_mhz)
{
@@ -554,7 +555,8 @@ emit_report_perf_count(struct intel_bb *ibb,
}
static void
-hsw_sanity_check_render_basic_reports(uint32_t *oa_report0, uint32_t *oa_report1,
+hsw_sanity_check_render_basic_reports(const uint32_t *oa_report0,
+ const uint32_t *oa_report1,
enum drm_i915_oa_format fmt)
{
uint32_t time_delta = timebase_scale(oa_report1[1] - oa_report0[1]);
@@ -639,7 +641,8 @@ hsw_sanity_check_render_basic_reports(uint32_t *oa_report0, uint32_t *oa_report1
}
static uint64_t
-gen8_read_40bit_a_counter(uint32_t *report, enum drm_i915_oa_format fmt, int a_id)
+gen8_read_40bit_a_counter(const uint32_t *report,
+ enum drm_i915_oa_format fmt, int a_id)
{
struct oa_format format = get_oa_format(fmt);
uint8_t *a40_high = (((uint8_t *)report) + format.a40_high_off);
@@ -757,7 +760,8 @@ accumulator_print(struct accumulator *accumulator, const char *title)
/* The TestOa metric set is designed so */
static void
-gen8_sanity_check_test_oa_reports(uint32_t *oa_report0, uint32_t *oa_report1,
+gen8_sanity_check_test_oa_reports(const uint32_t *oa_report0,
+ const uint32_t *oa_report1,
enum drm_i915_oa_format fmt)
{
struct oa_format format = get_oa_format(fmt);
--
2.29.2
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [igt-dev] [PATCH i-g-t v2 2/3] tests/i915/perf: deal with 0 delta reports
2020-11-25 11:25 [igt-dev] [PATCH i-g-t v2 0/3] tests/i915/perf: verify non zero reason field in OA reports Lionel Landwerlin
2020-11-25 11:25 ` [igt-dev] [PATCH i-g-t v2 1/3] tests/i915/perf: constify report checks functions Lionel Landwerlin
@ 2020-11-25 11:25 ` Lionel Landwerlin
2020-11-25 22:12 ` Umesh Nerlige Ramappa
2020-11-25 11:25 ` [igt-dev] [PATCH i-g-t v2 3/3] tests/i915/perf: verify reason field in OA reports is never 0 Lionel Landwerlin
` (2 subsequent siblings)
4 siblings, 1 reply; 11+ messages in thread
From: Lionel Landwerlin @ 2020-11-25 11:25 UTC (permalink / raw)
To: igt-dev
More thorough testing shows that it is possible to get a timer report
followed by a context-switch report with the same timestamp.
Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
---
tests/i915/perf.c | 91 +++++++++++++++++++++++++----------------------
1 file changed, 49 insertions(+), 42 deletions(-)
diff --git a/tests/i915/perf.c b/tests/i915/perf.c
index 2fbea0f0c..811067aad 100644
--- a/tests/i915/perf.c
+++ b/tests/i915/perf.c
@@ -778,15 +778,14 @@ gen8_sanity_check_test_oa_reports(const uint32_t *oa_report0,
uint32_t b;
uint32_t ref;
+ igt_debug("report type: %s->%s %p/%p\n",
+ gen8_read_report_reason(oa_report0),
+ gen8_read_report_reason(oa_report1),
+ oa_report0, oa_report1);
- igt_assert_neq(time_delta, 0);
- igt_assert_neq(clock_delta, 0);
-
- freq = ((uint64_t)clock_delta * 1000) / time_delta;
+ freq = time_delta ? ((uint64_t)clock_delta * 1000) / time_delta : 0;
igt_debug("freq = %"PRIu64"\n", freq);
- igt_assert(freq <= gt_max_freq_mhz);
-
igt_debug("clock delta = %"PRIu32"\n", clock_delta);
max_delta = clock_delta * intel_perf->devinfo.n_eus;
@@ -823,42 +822,50 @@ gen8_sanity_check_test_oa_reports(const uint32_t *oa_report0,
* multiple of the gpu clock
*/
if (format.n_b) {
- b = rpt1_b[0] - rpt0_b[0];
- igt_debug("B0: delta = %"PRIu32"\n", b);
- igt_assert_eq(b, 0);
-
- b = rpt1_b[1] - rpt0_b[1];
- igt_debug("B1: delta = %"PRIu32"\n", b);
- igt_assert_eq(b, clock_delta);
-
- b = rpt1_b[2] - rpt0_b[2];
- igt_debug("B2: delta = %"PRIu32"\n", b);
- igt_assert_eq(b, clock_delta);
-
- b = rpt1_b[3] - rpt0_b[3];
- ref = clock_delta / 2;
- igt_debug("B3: delta = %"PRIu32"\n", b);
- igt_assert(b >= ref - 1 && b <= ref + 1);
-
- b = rpt1_b[4] - rpt0_b[4];
- ref = clock_delta / 3;
- igt_debug("B4: delta = %"PRIu32"\n", b);
- igt_assert(b >= ref - 1 && b <= ref + 1);
-
- b = rpt1_b[5] - rpt0_b[5];
- ref = clock_delta / 3;
- igt_debug("B5: delta = %"PRIu32"\n", b);
- igt_assert(b >= ref - 1 && b <= ref + 1);
-
- b = rpt1_b[6] - rpt0_b[6];
- ref = clock_delta / 6;
- igt_debug("B6: delta = %"PRIu32"\n", b);
- igt_assert(b >= ref - 1 && b <= ref + 1);
-
- b = rpt1_b[7] - rpt0_b[7];
- ref = clock_delta * 2 / 3;
- igt_debug("B7: delta = %"PRIu32"\n", b);
- igt_assert(b >= ref - 1 && b <= ref + 1);
+ if (clock_delta > 0) {
+ b = rpt1_b[0] - rpt0_b[0];
+ igt_debug("B0: delta = %"PRIu32"\n", b);
+ igt_assert_eq(b, 0);
+
+ b = rpt1_b[1] - rpt0_b[1];
+ igt_debug("B1: delta = %"PRIu32"\n", b);
+ igt_assert_eq(b, clock_delta);
+
+ b = rpt1_b[2] - rpt0_b[2];
+ igt_debug("B2: delta = %"PRIu32"\n", b);
+ igt_assert_eq(b, clock_delta);
+
+ b = rpt1_b[3] - rpt0_b[3];
+ ref = clock_delta / 2;
+ igt_debug("B3: delta = %"PRIu32"\n", b);
+ igt_assert(b >= ref - 1 && b <= ref + 1);
+
+ b = rpt1_b[4] - rpt0_b[4];
+ ref = clock_delta / 3;
+ igt_debug("B4: delta = %"PRIu32"\n", b);
+ igt_assert(b >= ref - 1 && b <= ref + 1);
+
+ b = rpt1_b[5] - rpt0_b[5];
+ ref = clock_delta / 3;
+ igt_debug("B5: delta = %"PRIu32"\n", b);
+ igt_assert(b >= ref - 1 && b <= ref + 1);
+
+ b = rpt1_b[6] - rpt0_b[6];
+ ref = clock_delta / 6;
+ igt_debug("B6: delta = %"PRIu32"\n", b);
+ igt_assert(b >= ref - 1 && b <= ref + 1);
+
+ b = rpt1_b[7] - rpt0_b[7];
+ ref = clock_delta * 2 / 3;
+ igt_debug("B7: delta = %"PRIu32"\n", b);
+ igt_assert(b >= ref - 1 && b <= ref + 1);
+ } else {
+ for (int j = 0; j < format.n_b; j++) {
+ b = rpt1_b[j] - rpt0_b[j];
+ igt_debug("B%i: delta = %"PRIu32"\n", j, b);
+ igt_assert_eq(b, 0);
+ }
+ }
}
for (int j = 0; j < format.n_c; j++) {
--
2.29.2
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [igt-dev] [PATCH i-g-t v2 3/3] tests/i915/perf: verify reason field in OA reports is never 0
2020-11-25 11:25 [igt-dev] [PATCH i-g-t v2 0/3] tests/i915/perf: verify non zero reason field in OA reports Lionel Landwerlin
2020-11-25 11:25 ` [igt-dev] [PATCH i-g-t v2 1/3] tests/i915/perf: constify report checks functions Lionel Landwerlin
2020-11-25 11:25 ` [igt-dev] [PATCH i-g-t v2 2/3] tests/i915/perf: deal with 0 delta reports Lionel Landwerlin
@ 2020-11-25 11:25 ` Lionel Landwerlin
2020-11-25 22:35 ` Umesh Nerlige Ramappa
2020-11-25 12:25 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/i915/perf: verify non zero reason field in OA reports Patchwork
2020-11-25 13:57 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
4 siblings, 1 reply; 11+ messages in thread
From: Lionel Landwerlin @ 2020-11-25 11:25 UTC (permalink / raw)
To: igt-dev
We're about to remove the filtering in i915 on 0 reason fields because
we assume this was a possibility, but it turned out to be a corruption
in the tail pointer register that made us read cleared data.
This test is here to verify that our assumption hold that the HW never
produces such reports.
v2: Fix len checking (Umesh)
Count report lost events (Umesh)
Check report sanity (Umesh)
Limit test to 3 times the OA buffer (Umesh)
Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
---
tests/i915/perf.c | 90 +++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 90 insertions(+)
diff --git a/tests/i915/perf.c b/tests/i915/perf.c
index 811067aad..2d442df96 100644
--- a/tests/i915/perf.c
+++ b/tests/i915/perf.c
@@ -2550,6 +2550,89 @@ test_buffer_fill(void)
__perf_close(stream_fd);
}
+static void
+test_non_zero_reason(void)
+{
+ /* ~10 micro second period */
+ int oa_exponent = max_oa_exponent_for_period_lte(10000);
+ uint64_t properties[] = {
+ /* Include OA reports in samples */
+ DRM_I915_PERF_PROP_SAMPLE_OA, true,
+
+ /* OA unit configuration */
+ DRM_I915_PERF_PROP_OA_METRICS_SET, test_set->perf_oa_metrics_set,
+ DRM_I915_PERF_PROP_OA_FORMAT, test_set->perf_oa_format,
+ DRM_I915_PERF_PROP_OA_EXPONENT, oa_exponent,
+ };
+ struct drm_i915_perf_open_param param = {
+ .flags = I915_PERF_FLAG_FD_CLOEXEC,
+ .num_properties = sizeof(properties) / 16,
+ .properties_ptr = to_user_pointer(properties),
+ };
+ struct drm_i915_perf_record_header *header;
+ uint32_t buf_size = 3 * 65536 * (256 + sizeof(struct drm_i915_perf_record_header));
+ uint8_t *buf = malloc(buf_size);
+ uint32_t total_len = 0, reports_lost;
+ const uint32_t *last_report;
+ int len;
+
+ igt_assert(buf);
+
+ igt_debug("Ready to read about %u bytes\n", buf_size);
+
+ load_helper_init();
+ load_helper_run(HIGH);
+
+ stream_fd = __perf_open(drm_fd, ¶m, true /* prevent_pm */);
+
+ while (total_len < (buf_size - sizeof(struct drm_i915_perf_record_header)) &&
+ ((len = read(stream_fd, &buf[total_len], buf_size - total_len)) > 0 ||
+ (len == -1 && errno == EINTR))) {
+ if (len > 0)
+ total_len += len;
+ }
+
+ __perf_close(stream_fd);
+
+ load_helper_stop();
+ load_helper_fini();
+
+ igt_debug("Got %u bytes\n", total_len);
+
+ last_report = NULL;
+ reports_lost = 0;
+ for (uint32_t offset = 0; offset < total_len; offset += header->size) {
+ header = (void *) (buf + offset);
+
+ switch (header->type) {
+ case DRM_I915_PERF_RECORD_OA_REPORT_LOST:
+ reports_lost++;
+ break;
+ case DRM_I915_PERF_RECORD_SAMPLE: {
+ const uint32_t *report = (void *) (header + 1);
+ uint32_t reason = (report[0] >> OAREPORT_REASON_SHIFT) &
+ OAREPORT_REASON_MASK;
+
+ igt_assert_neq(reason, 0);
+
+ if (last_report) {
+ sanity_check_reports(last_report, report,
+ test_set->perf_oa_format);
+ }
+ last_report = report;
+ break;
+ }
+ case DRM_I915_PERF_RECORD_OA_BUFFER_LOST:
+ igt_assert(!"unexpected overflow");
+ break;
+ }
+ }
+
+ igt_debug("Got %u report lost events\n", reports_lost);
+
+ free(buf);
+}
+
static void
test_enable_disable(void)
{
@@ -4893,6 +4976,13 @@ igt_main
igt_subtest("buffer-fill")
test_buffer_fill();
+ igt_describe("Test that reason field in OA reports is never 0 on Gen8+");
+ igt_subtest("non-zero-reason") {
+ /* Reason field is only available on Gen8+ */
+ igt_require(intel_gen(devid) >= 8);
+ test_non_zero_reason();
+ }
+
igt_subtest("disabled-read-error")
test_disabled_read_error();
igt_subtest("non-sampling-read-error")
--
2.29.2
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for tests/i915/perf: verify non zero reason field in OA reports
2020-11-25 11:25 [igt-dev] [PATCH i-g-t v2 0/3] tests/i915/perf: verify non zero reason field in OA reports Lionel Landwerlin
` (2 preceding siblings ...)
2020-11-25 11:25 ` [igt-dev] [PATCH i-g-t v2 3/3] tests/i915/perf: verify reason field in OA reports is never 0 Lionel Landwerlin
@ 2020-11-25 12:25 ` Patchwork
2020-11-25 13:57 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
4 siblings, 0 replies; 11+ messages in thread
From: Patchwork @ 2020-11-25 12:25 UTC (permalink / raw)
To: Lionel Landwerlin; +Cc: igt-dev
[-- Attachment #1.1: Type: text/plain, Size: 4694 bytes --]
== Series Details ==
Series: tests/i915/perf: verify non zero reason field in OA reports
URL : https://patchwork.freedesktop.org/series/84258/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_9389 -> IGTPW_5226
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/index.html
Known issues
------------
Here are the changes found in IGTPW_5226 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@debugfs_test@read_all_entries:
- fi-tgl-y: [PASS][1] -> [DMESG-WARN][2] ([i915#402]) +2 similar issues
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/fi-tgl-y/igt@debugfs_test@read_all_entries.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/fi-tgl-y/igt@debugfs_test@read_all_entries.html
* igt@i915_module_load@reload:
- fi-tgl-u2: [PASS][3] -> [DMESG-WARN][4] ([i915#1982] / [k.org#205379])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/fi-tgl-u2/igt@i915_module_load@reload.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/fi-tgl-u2/igt@i915_module_load@reload.html
* igt@i915_selftest@live@execlists:
- fi-icl-y: [PASS][5] -> [INCOMPLETE][6] ([i915#1037] / [i915#2276])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/fi-icl-y/igt@i915_selftest@live@execlists.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/fi-icl-y/igt@i915_selftest@live@execlists.html
#### Possible fixes ####
* igt@kms_chamelium@dp-crc-fast:
- fi-cml-u2: [DMESG-WARN][7] ([i915#1982]) -> [PASS][8]
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/fi-cml-u2/igt@kms_chamelium@dp-crc-fast.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/fi-cml-u2/igt@kms_chamelium@dp-crc-fast.html
* igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic:
- fi-byt-j1900: [DMESG-WARN][9] ([i915#1982]) -> [PASS][10]
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/fi-byt-j1900/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/fi-byt-j1900/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-atomic.html
* igt@kms_cursor_legacy@basic-flip-before-cursor-atomic:
- fi-icl-u2: [DMESG-WARN][11] ([i915#1982]) -> [PASS][12] +1 similar issue
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/fi-icl-u2/igt@kms_cursor_legacy@basic-flip-before-cursor-atomic.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/fi-icl-u2/igt@kms_cursor_legacy@basic-flip-before-cursor-atomic.html
* igt@kms_flip@basic-flip-vs-dpms@d-edp1:
- fi-tgl-y: [DMESG-WARN][13] ([i915#1982]) -> [PASS][14]
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/fi-tgl-y/igt@kms_flip@basic-flip-vs-dpms@d-edp1.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/fi-tgl-y/igt@kms_flip@basic-flip-vs-dpms@d-edp1.html
* igt@prime_self_import@basic-with_one_bo:
- fi-tgl-y: [DMESG-WARN][15] ([i915#402]) -> [PASS][16] +1 similar issue
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/fi-tgl-y/igt@prime_self_import@basic-with_one_bo.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/fi-tgl-y/igt@prime_self_import@basic-with_one_bo.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[i915#1037]: https://gitlab.freedesktop.org/drm/intel/issues/1037
[i915#1982]: https://gitlab.freedesktop.org/drm/intel/issues/1982
[i915#2276]: https://gitlab.freedesktop.org/drm/intel/issues/2276
[i915#2417]: https://gitlab.freedesktop.org/drm/intel/issues/2417
[i915#402]: https://gitlab.freedesktop.org/drm/intel/issues/402
[k.org#205379]: https://bugzilla.kernel.org/show_bug.cgi?id=205379
Participating hosts (43 -> 39)
------------------------------
Missing (4): fi-ilk-m540 fi-bsw-cyan fi-bdw-samus fi-hsw-4200u
Build changes
-------------
* CI: CI-20190529 -> None
* IGT: IGT_5870 -> IGTPW_5226
CI-20190529: 20190529
CI_DRM_9389: b0c2cf3ad04abd9e7a44abe12e736bb5ab587393 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_5226: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/index.html
IGT_5870: 08b13995b85df26a77212e4fb21fd772976ef33c @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
== Testlist changes ==
+igt@perf@non-zero-reason
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/index.html
[-- Attachment #1.2: Type: text/html, Size: 5765 bytes --]
[-- Attachment #2: Type: text/plain, Size: 154 bytes --]
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 11+ messages in thread
* [igt-dev] ✓ Fi.CI.IGT: success for tests/i915/perf: verify non zero reason field in OA reports
2020-11-25 11:25 [igt-dev] [PATCH i-g-t v2 0/3] tests/i915/perf: verify non zero reason field in OA reports Lionel Landwerlin
` (3 preceding siblings ...)
2020-11-25 12:25 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/i915/perf: verify non zero reason field in OA reports Patchwork
@ 2020-11-25 13:57 ` Patchwork
2020-11-25 14:08 ` Lionel Landwerlin
4 siblings, 1 reply; 11+ messages in thread
From: Patchwork @ 2020-11-25 13:57 UTC (permalink / raw)
To: Lionel Landwerlin; +Cc: igt-dev
[-- Attachment #1.1: Type: text/plain, Size: 26175 bytes --]
== Series Details ==
Series: tests/i915/perf: verify non zero reason field in OA reports
URL : https://patchwork.freedesktop.org/series/84258/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_9389_full -> IGTPW_5226_full
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/index.html
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in IGTPW_5226_full:
### IGT changes ###
#### Possible regressions ####
* {igt@perf@non-zero-reason} (NEW):
- shard-iclb: NOTRUN -> [FAIL][1]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb7/igt@perf@non-zero-reason.html
#### Suppressed ####
The following results come from untrusted machines, tests, or statuses.
They do not affect the overall result.
* {igt@gem_userptr_blits@vma-merge}:
- shard-snb: [FAIL][2] ([i915#1635]) -> [FAIL][3]
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-snb4/igt@gem_userptr_blits@vma-merge.html
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-snb6/igt@gem_userptr_blits@vma-merge.html
New tests
---------
New tests have been introduced between CI_DRM_9389_full and IGTPW_5226_full:
### New IGT tests (1) ###
* igt@perf@non-zero-reason:
- Statuses : 1 fail(s) 4 pass(s) 2 skip(s)
- Exec time: [0.0, 41.66] s
Known issues
------------
Here are the changes found in IGTPW_5226_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_eio@kms:
- shard-kbl: [PASS][4] -> [DMESG-WARN][5] ([i915#165] / [i915#78]) +1 similar issue
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-kbl2/igt@gem_eio@kms.html
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-kbl2/igt@gem_eio@kms.html
* igt@gem_exec_reloc@basic-many-active@rcs0:
- shard-glk: [PASS][6] -> [FAIL][7] ([i915#2389])
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-glk2/igt@gem_exec_reloc@basic-many-active@rcs0.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-glk2/igt@gem_exec_reloc@basic-many-active@rcs0.html
* igt@gem_workarounds@suspend-resume:
- shard-glk: [PASS][8] -> [DMESG-WARN][9] ([i915#2635])
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-glk9/igt@gem_workarounds@suspend-resume.html
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-glk6/igt@gem_workarounds@suspend-resume.html
- shard-iclb: [PASS][10] -> [INCOMPLETE][11] ([i915#1185])
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb3/igt@gem_workarounds@suspend-resume.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb2/igt@gem_workarounds@suspend-resume.html
- shard-apl: [PASS][12] -> [DMESG-WARN][13] ([i915#1635] / [i915#2635])
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-apl4/igt@gem_workarounds@suspend-resume.html
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-apl4/igt@gem_workarounds@suspend-resume.html
- shard-kbl: [PASS][14] -> [INCOMPLETE][15] ([i915#155])
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-kbl3/igt@gem_workarounds@suspend-resume.html
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-kbl7/igt@gem_workarounds@suspend-resume.html
- shard-hsw: [PASS][16] -> [INCOMPLETE][17] ([i915#2637])
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-hsw1/igt@gem_workarounds@suspend-resume.html
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-hsw5/igt@gem_workarounds@suspend-resume.html
* igt@i915_pm_rpm@modeset-lpsp-stress-no-wait:
- shard-iclb: [PASS][18] -> [SKIP][19] ([i915#579])
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb5/igt@i915_pm_rpm@modeset-lpsp-stress-no-wait.html
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb6/igt@i915_pm_rpm@modeset-lpsp-stress-no-wait.html
* igt@kms_cursor_crc@pipe-a-cursor-64x21-onscreen:
- shard-kbl: [PASS][20] -> [DMESG-WARN][21] ([i915#78]) +2 similar issues
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-kbl7/igt@kms_cursor_crc@pipe-a-cursor-64x21-onscreen.html
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-kbl2/igt@kms_cursor_crc@pipe-a-cursor-64x21-onscreen.html
* igt@kms_cursor_edge_walk@pipe-a-64x64-left-edge:
- shard-glk: [PASS][22] -> [DMESG-WARN][23] ([i915#1982]) +6 similar issues
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-glk3/igt@kms_cursor_edge_walk@pipe-a-64x64-left-edge.html
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-glk1/igt@kms_cursor_edge_walk@pipe-a-64x64-left-edge.html
* igt@kms_cursor_legacy@2x-long-cursor-vs-flip-atomic:
- shard-hsw: [PASS][24] -> [FAIL][25] ([i915#96])
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-hsw1/igt@kms_cursor_legacy@2x-long-cursor-vs-flip-atomic.html
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-hsw1/igt@kms_cursor_legacy@2x-long-cursor-vs-flip-atomic.html
* igt@kms_cursor_legacy@flip-vs-cursor-legacy:
- shard-apl: [PASS][26] -> [DMESG-WARN][27] ([i915#1635] / [i915#1982]) +1 similar issue
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-apl8/igt@kms_cursor_legacy@flip-vs-cursor-legacy.html
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-apl6/igt@kms_cursor_legacy@flip-vs-cursor-legacy.html
* igt@kms_cursor_legacy@flip-vs-cursor-varying-size:
- shard-tglb: [PASS][28] -> [FAIL][29] ([i915#2346])
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-tglb2/igt@kms_cursor_legacy@flip-vs-cursor-varying-size.html
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-tglb3/igt@kms_cursor_legacy@flip-vs-cursor-varying-size.html
* igt@kms_flip@flip-vs-blocking-wf-vblank@a-dp1:
- shard-kbl: [PASS][30] -> [DMESG-WARN][31] ([i915#1982]) +5 similar issues
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-kbl4/igt@kms_flip@flip-vs-blocking-wf-vblank@a-dp1.html
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-kbl3/igt@kms_flip@flip-vs-blocking-wf-vblank@a-dp1.html
* igt@kms_flip@flip-vs-expired-vblank@a-edp1:
- shard-tglb: [PASS][32] -> [FAIL][33] ([i915#2598]) +1 similar issue
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-tglb2/igt@kms_flip@flip-vs-expired-vblank@a-edp1.html
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-tglb8/igt@kms_flip@flip-vs-expired-vblank@a-edp1.html
* igt@kms_flip@flip-vs-suspend-interruptible@c-hdmi-a1:
- shard-hsw: [PASS][34] -> [INCOMPLETE][35] ([i915#2295])
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-hsw2/igt@kms_flip@flip-vs-suspend-interruptible@c-hdmi-a1.html
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-hsw6/igt@kms_flip@flip-vs-suspend-interruptible@c-hdmi-a1.html
* igt@kms_frontbuffer_tracking@psr-farfromfence:
- shard-tglb: [PASS][36] -> [DMESG-WARN][37] ([i915#1982]) +4 similar issues
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-tglb5/igt@kms_frontbuffer_tracking@psr-farfromfence.html
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-tglb3/igt@kms_frontbuffer_tracking@psr-farfromfence.html
* igt@kms_psr@psr2_cursor_plane_onoff:
- shard-iclb: [PASS][38] -> [SKIP][39] ([fdo#109441]) +2 similar issues
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb2/igt@kms_psr@psr2_cursor_plane_onoff.html
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb3/igt@kms_psr@psr2_cursor_plane_onoff.html
* igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm:
- shard-glk: [PASS][40] -> [SKIP][41] ([fdo#109271])
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-glk4/igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm.html
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-glk6/igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm.html
- shard-hsw: [PASS][42] -> [SKIP][43] ([fdo#109271])
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-hsw8/igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm.html
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-hsw5/igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm.html
- shard-iclb: [PASS][44] -> [SKIP][45] ([fdo#109278])
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb8/igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm.html
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb2/igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm.html
- shard-kbl: [PASS][46] -> [SKIP][47] ([fdo#109271])
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-kbl7/igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm.html
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-kbl7/igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm.html
- shard-apl: [PASS][48] -> [SKIP][49] ([fdo#109271] / [i915#1635])
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-apl6/igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm.html
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-apl4/igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm.html
* igt@perf@polling-parameterized:
- shard-apl: [PASS][50] -> [FAIL][51] ([i915#1542] / [i915#1635])
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-apl7/igt@perf@polling-parameterized.html
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-apl7/igt@perf@polling-parameterized.html
#### Possible fixes ####
* igt@device_reset@unbind-reset-rebind:
- shard-glk: [INCOMPLETE][52] ([i915#2283] / [i915#2405]) -> [PASS][53]
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-glk2/igt@device_reset@unbind-reset-rebind.html
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-glk5/igt@device_reset@unbind-reset-rebind.html
- shard-apl: [INCOMPLETE][54] ([i915#1635] / [i915#2283] / [i915#2405]) -> [PASS][55]
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-apl6/igt@device_reset@unbind-reset-rebind.html
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-apl6/igt@device_reset@unbind-reset-rebind.html
- shard-kbl: [INCOMPLETE][56] ([i915#2283] / [i915#2405]) -> [PASS][57]
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-kbl6/igt@device_reset@unbind-reset-rebind.html
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-kbl3/igt@device_reset@unbind-reset-rebind.html
- shard-tglb: [INCOMPLETE][58] ([i915#1602] / [i915#750]) -> [PASS][59]
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-tglb3/igt@device_reset@unbind-reset-rebind.html
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-tglb3/igt@device_reset@unbind-reset-rebind.html
- shard-iclb: [INCOMPLETE][60] ([i915#2283] / [i915#2405]) -> [PASS][61]
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb8/igt@device_reset@unbind-reset-rebind.html
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb3/igt@device_reset@unbind-reset-rebind.html
* igt@feature_discovery@psr2:
- shard-iclb: [SKIP][62] ([i915#658]) -> [PASS][63]
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb1/igt@feature_discovery@psr2.html
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb2/igt@feature_discovery@psr2.html
* igt@gem_exec_whisper@basic-fds-forked:
- shard-glk: [DMESG-WARN][64] ([i915#118] / [i915#95]) -> [PASS][65]
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-glk4/igt@gem_exec_whisper@basic-fds-forked.html
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-glk3/igt@gem_exec_whisper@basic-fds-forked.html
* igt@i915_pm_backlight@fade_with_suspend:
- shard-iclb: [DMESG-WARN][66] -> [PASS][67]
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb7/igt@i915_pm_backlight@fade_with_suspend.html
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb7/igt@i915_pm_backlight@fade_with_suspend.html
* igt@i915_pm_dc@dc6-psr:
- shard-iclb: [FAIL][68] ([i915#454]) -> [PASS][69]
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb8/igt@i915_pm_dc@dc6-psr.html
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb3/igt@i915_pm_dc@dc6-psr.html
* igt@i915_pm_rpm@gem-execbuf-stress:
- shard-glk: [SKIP][70] ([fdo#109271]) -> [PASS][71]
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-glk6/igt@i915_pm_rpm@gem-execbuf-stress.html
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-glk5/igt@i915_pm_rpm@gem-execbuf-stress.html
- shard-apl: [SKIP][72] ([fdo#109271] / [i915#1635]) -> [PASS][73]
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-apl1/igt@i915_pm_rpm@gem-execbuf-stress.html
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-apl6/igt@i915_pm_rpm@gem-execbuf-stress.html
- shard-kbl: [SKIP][74] ([fdo#109271]) -> [PASS][75]
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-kbl1/igt@i915_pm_rpm@gem-execbuf-stress.html
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-kbl3/igt@i915_pm_rpm@gem-execbuf-stress.html
- shard-hsw: [SKIP][76] ([fdo#109271]) -> [PASS][77]
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-hsw4/igt@i915_pm_rpm@gem-execbuf-stress.html
[77]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-hsw2/igt@i915_pm_rpm@gem-execbuf-stress.html
* igt@kms_busy@basic-modeset-pipe-c:
- shard-hsw: [DMESG-WARN][78] ([i915#44]) -> [PASS][79]
[78]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-hsw4/igt@kms_busy@basic-modeset-pipe-c.html
[79]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-hsw2/igt@kms_busy@basic-modeset-pipe-c.html
* igt@kms_cursor_edge_walk@pipe-c-256x256-left-edge:
- shard-apl: [DMESG-WARN][80] ([i915#1635] / [i915#1982]) -> [PASS][81] +4 similar issues
[80]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-apl3/igt@kms_cursor_edge_walk@pipe-c-256x256-left-edge.html
[81]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-apl2/igt@kms_cursor_edge_walk@pipe-c-256x256-left-edge.html
* igt@kms_cursor_legacy@basic-flip-before-cursor-legacy:
- shard-glk: [DMESG-WARN][82] ([i915#1982]) -> [PASS][83] +4 similar issues
[82]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-glk1/igt@kms_cursor_legacy@basic-flip-before-cursor-legacy.html
[83]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-glk8/igt@kms_cursor_legacy@basic-flip-before-cursor-legacy.html
* igt@kms_fbcon_fbt@fbc:
- shard-glk: [FAIL][84] ([i915#64]) -> [PASS][85]
[84]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-glk6/igt@kms_fbcon_fbt@fbc.html
[85]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-glk7/igt@kms_fbcon_fbt@fbc.html
* igt@kms_flip@flip-vs-rmfb-interruptible@c-dp1:
- shard-kbl: [DMESG-WARN][86] ([i915#62]) -> [PASS][87]
[86]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-kbl2/igt@kms_flip@flip-vs-rmfb-interruptible@c-dp1.html
[87]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-kbl2/igt@kms_flip@flip-vs-rmfb-interruptible@c-dp1.html
* igt@kms_frontbuffer_tracking@fbc-indfb-scaledprimary:
- shard-kbl: [FAIL][88] ([i915#49]) -> [PASS][89]
[88]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-kbl1/igt@kms_frontbuffer_tracking@fbc-indfb-scaledprimary.html
[89]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-kbl4/igt@kms_frontbuffer_tracking@fbc-indfb-scaledprimary.html
- shard-apl: [FAIL][90] ([i915#1635] / [i915#49]) -> [PASS][91]
[90]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-apl1/igt@kms_frontbuffer_tracking@fbc-indfb-scaledprimary.html
[91]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-apl2/igt@kms_frontbuffer_tracking@fbc-indfb-scaledprimary.html
* igt@kms_frontbuffer_tracking@fbc-stridechange:
- shard-tglb: [DMESG-WARN][92] ([i915#1982]) -> [PASS][93] +1 similar issue
[92]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-tglb3/igt@kms_frontbuffer_tracking@fbc-stridechange.html
[93]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-tglb2/igt@kms_frontbuffer_tracking@fbc-stridechange.html
* igt@kms_psr@psr2_sprite_blt:
- shard-iclb: [SKIP][94] ([fdo#109441]) -> [PASS][95] +2 similar issues
[94]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb1/igt@kms_psr@psr2_sprite_blt.html
[95]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb2/igt@kms_psr@psr2_sprite_blt.html
* igt@kms_vblank@pipe-b-wait-forked-busy:
- shard-kbl: [DMESG-WARN][96] ([i915#1982]) -> [PASS][97] +4 similar issues
[96]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-kbl1/igt@kms_vblank@pipe-b-wait-forked-busy.html
[97]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-kbl2/igt@kms_vblank@pipe-b-wait-forked-busy.html
* igt@perf_pmu@module-unload:
- shard-apl: [DMESG-WARN][98] ([i915#1635] / [i915#1982] / [i915#262]) -> [PASS][99]
[98]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-apl3/igt@perf_pmu@module-unload.html
[99]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-apl1/igt@perf_pmu@module-unload.html
#### Warnings ####
* igt@device_reset@unbind-reset-rebind:
- shard-hsw: [INCOMPLETE][100] ([i915#2283] / [i915#2405]) -> [WARN][101] ([i915#2283])
[100]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-hsw6/igt@device_reset@unbind-reset-rebind.html
[101]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-hsw2/igt@device_reset@unbind-reset-rebind.html
* igt@gem_workarounds@suspend-resume:
- shard-tglb: [DMESG-WARN][102] ([i915#2411]) -> [DMESG-WARN][103] ([i915#1436] / [i915#1602] / [i915#1887] / [i915#2411])
[102]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-tglb1/igt@gem_workarounds@suspend-resume.html
[103]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-tglb5/igt@gem_workarounds@suspend-resume.html
* igt@i915_pm_dc@dc3co-vpb-simulation:
- shard-iclb: [SKIP][104] ([i915#588]) -> [SKIP][105] ([i915#658])
[104]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb2/igt@i915_pm_dc@dc3co-vpb-simulation.html
[105]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb3/igt@i915_pm_dc@dc3co-vpb-simulation.html
* igt@i915_pm_rc6_residency@rc6-fence:
- shard-iclb: [WARN][106] ([i915#1804] / [i915#2684]) -> [WARN][107] ([i915#2684])
[106]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb3/igt@i915_pm_rc6_residency@rc6-fence.html
[107]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb5/igt@i915_pm_rc6_residency@rc6-fence.html
* igt@i915_pm_rpm@modeset-lpsp-stress-no-wait:
- shard-tglb: [DMESG-WARN][108] ([i915#2411]) -> [SKIP][109] ([i915#579])
[108]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-tglb5/igt@i915_pm_rpm@modeset-lpsp-stress-no-wait.html
[109]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-tglb1/igt@i915_pm_rpm@modeset-lpsp-stress-no-wait.html
* igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm:
- shard-tglb: [DMESG-WARN][110] ([i915#2411]) -> [SKIP][111] ([i915#2648])
[110]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-tglb7/igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm.html
[111]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-tglb5/igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm.html
* igt@runner@aborted:
- shard-hsw: [FAIL][112] ([i915#2283] / [i915#2295]) -> [FAIL][113] ([fdo#109271] / [i915#2295])
[112]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-hsw6/igt@runner@aborted.html
[113]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-hsw5/igt@runner@aborted.html
- shard-iclb: ([FAIL][114], [FAIL][115], [FAIL][116]) ([i915#1814] / [i915#2283] / [i915#2295] / [i915#483]) -> ([FAIL][117], [FAIL][118]) ([i915#2295])
[114]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb7/igt@runner@aborted.html
[115]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb8/igt@runner@aborted.html
[116]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb3/igt@runner@aborted.html
[117]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb2/igt@runner@aborted.html
[118]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb3/igt@runner@aborted.html
- shard-apl: ([FAIL][119], [FAIL][120]) ([i915#1635] / [i915#2295]) -> ([FAIL][121], [FAIL][122]) ([fdo#109271] / [i915#1635] / [i915#2295])
[119]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-apl3/igt@runner@aborted.html
[120]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-apl6/igt@runner@aborted.html
[121]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-apl4/igt@runner@aborted.html
[122]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-apl6/igt@runner@aborted.html
- shard-tglb: ([FAIL][123], [FAIL][124], [FAIL][125]) ([i915#2295]) -> ([FAIL][126], [FAIL][127]) ([i915#1602] / [i915#2295])
[123]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-tglb5/igt@runner@aborted.html
[124]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-tglb6/igt@runner@aborted.html
[125]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-tglb3/igt@runner@aborted.html
[126]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-tglb8/igt@runner@aborted.html
[127]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-tglb5/igt@runner@aborted.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#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
[fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
[i915#118]: https://gitlab.freedesktop.org/drm/intel/issues/118
[i915#1185]: https://gitlab.freedesktop.org/drm/intel/issues/1185
[i915#1436]: https://gitlab.freedesktop.org/drm/intel/issues/1436
[i915#1542]: https://gitlab.freedesktop.org/drm/intel/issues/1542
[i915#155]: https://gitlab.freedesktop.org/drm/intel/issues/155
[i915#1602]: https://gitlab.freedesktop.org/drm/intel/issues/1602
[i915#1635]: https://gitlab.freedesktop.org/drm/intel/issues/1635
[i915#165]: https://gitlab.freedesktop.org/drm/intel/issues/165
[i915#1804]: https://gitlab.freedesktop.org/drm/intel/issues/1804
[i915#1814]: https://gitlab.freedesktop.org/drm/intel/issues/1814
[i915#1887]: https://gitlab.freedesktop.org/drm/intel/issues/1887
[i915#1982]: https://gitlab.freedesktop.org/drm/intel/issues/1982
[i915#2283]: https://gitlab.freedesktop.org/drm/intel/issues/2283
[i915#2295]: https://gitlab.freedesktop.org/drm/intel/issues/2295
[i915#2346]: https://gitlab.freedesktop.org/drm/intel/issues/2346
[i915#2389]: https://gitlab.freedesktop.org/drm/intel/issues/2389
[i915#2405]: https://gitlab.freedesktop.org/drm/intel/issues/2405
[i915#2411]: https://gitlab.freedesktop.org/drm/intel/issues/2411
[i915#2598]: https://gitlab.freedesktop.org/drm/intel/issues/2598
[i915#262]: https://gitlab.freedesktop.org/drm/intel/issues/262
[i915#2635]: https://gitlab.freedesktop.org/drm/intel/issues/2635
[i915#2637]: https://gitlab.freedesktop.org/drm/intel/issues/2637
[i915#2648]: https://gitlab.freedesktop.org/drm/intel/issues/2648
[i915#2684]: https://gitlab.freedesktop.org/drm/intel/issues/2684
[i915#44]: https://gitlab.freedesktop.org/drm/intel/issues/44
[i915#454]: https://gitlab.freedesktop.org/drm/intel/issues/454
[i915#483]: https://gitlab.freedesktop.org/drm/intel/issues/483
[i915#49]: https://gitlab.freedesktop.org/drm/intel/issues/49
[i915#579]: https://gitlab.freedesktop.org/drm/intel/issues/579
[i915#588]: https://gitlab.freedesktop.org/drm/intel/issues/588
[i915#62]: https://gitlab.freedesktop.org/drm/intel/issues/62
[i915#64]: https://gitlab.freedesktop.org/drm/intel/issues/64
[i915#658]: https://gitlab.freedesktop.org/drm/intel/issues/658
[i915#750]: https://gitlab.freedesktop.org/drm/intel/issues/750
[i915#78]: https://gitlab.freedesktop.org/drm/intel/issues/78
[i915#95]: https://gitlab.freedesktop.org/drm/intel/issues/95
[i915#96]: https://gitlab.freedesktop.org/drm/intel/issues/96
Participating hosts (10 -> 8)
------------------------------
Missing (2): pig-skl-6260u pig-glk-j5005
Build changes
-------------
* CI: CI-20190529 -> None
* IGT: IGT_5870 -> IGTPW_5226
* Piglit: piglit_4509 -> None
CI-20190529: 20190529
CI_DRM_9389: b0c2cf3ad04abd9e7a44abe12e736bb5ab587393 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_5226: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/index.html
IGT_5870: 08b13995b85df26a77212e4fb21fd772976ef33c @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/index.html
[-- Attachment #1.2: Type: text/html, Size: 31925 bytes --]
[-- Attachment #2: Type: text/plain, Size: 154 bytes --]
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [igt-dev] ✓ Fi.CI.IGT: success for tests/i915/perf: verify non zero reason field in OA reports
2020-11-25 13:57 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
@ 2020-11-25 14:08 ` Lionel Landwerlin
0 siblings, 0 replies; 11+ messages in thread
From: Lionel Landwerlin @ 2020-11-25 14:08 UTC (permalink / raw)
To: igt-dev
[-- Attachment #1.1: Type: text/plain, Size: 31556 bytes --]
On 25/11/2020 15:57, Patchwork wrote:
> Project List - Patchwork *Patch Details*
> *Series:* tests/i915/perf: verify non zero reason field in OA reports
> *URL:* https://patchwork.freedesktop.org/series/84258/
> <https://patchwork.freedesktop.org/series/84258/>
> *State:* success
> *Details:*
> https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/index.html
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/index.html>
>
>
> CI Bug Log - changes from CI_DRM_9389_full -> IGTPW_5226_full
>
>
> Summary
>
> *SUCCESS*
>
> No regressions found.
>
> External URL:
> https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/index.html
>
>
> Possible new issues
>
> Here are the unknown changes that may have been introduced in
> IGTPW_5226_full:
>
>
> IGT changes
>
>
> Possible regressions
>
> * {igt@perf@non-zero-reason} (NEW):
> o shard-iclb: NOTRUN -> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb7/igt@perf@non-zero-reason.html>
>
Like I mentioned in a previous mail, I think we should backport :
commit 0305613dbcf42b6b27ddf516fea2738dfbfdb7c0
Author: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Date: Tue Nov 17 15:01:24 2020 +0200
drm/i915/perf: workaround register corruption in OATAILPTR
It seems that currently report invalid data to userspace.
-Lionel
> *
>
>
> Suppressed
>
> The following results come from untrusted machines, tests, or statuses.
> They do not affect the overall result.
>
> * {igt@gem_userptr_blits@vma-merge}:
> o shard-snb: FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-snb4/igt@gem_userptr_blits@vma-merge.html>
> (i915#1635
> <https://gitlab.freedesktop.org/drm/intel/issues/1635>) ->
> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-snb6/igt@gem_userptr_blits@vma-merge.html>
>
>
> New tests
>
> New tests have been introduced between CI_DRM_9389_full and
> IGTPW_5226_full:
>
>
> New IGT tests (1)
>
> * igt@perf@non-zero-reason:
> o Statuses : 1 fail(s) 4 pass(s) 2 skip(s)
> o Exec time: [0.0, 41.66] s
>
>
> Known issues
>
> Here are the changes found in IGTPW_5226_full that come from known issues:
>
>
> IGT changes
>
>
> Issues hit
>
> *
>
> igt@gem_eio@kms:
>
> o shard-kbl: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-kbl2/igt@gem_eio@kms.html>
> -> DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-kbl2/igt@gem_eio@kms.html>
> (i915#165
> <https://gitlab.freedesktop.org/drm/intel/issues/165> /
> i915#78 <https://gitlab.freedesktop.org/drm/intel/issues/78>)
> +1 similar issue
> *
>
> igt@gem_exec_reloc@basic-many-active@rcs0:
>
> o shard-glk: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-glk2/igt@gem_exec_reloc@basic-many-active@rcs0.html>
> -> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-glk2/igt@gem_exec_reloc@basic-many-active@rcs0.html>
> (i915#2389 <https://gitlab.freedesktop.org/drm/intel/issues/2389>)
> *
>
> igt@gem_workarounds@suspend-resume:
>
> o
>
> shard-glk: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-glk9/igt@gem_workarounds@suspend-resume.html>
> -> DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-glk6/igt@gem_workarounds@suspend-resume.html>
> (i915#2635 <https://gitlab.freedesktop.org/drm/intel/issues/2635>)
>
> o
>
> shard-iclb: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb3/igt@gem_workarounds@suspend-resume.html>
> -> INCOMPLETE
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb2/igt@gem_workarounds@suspend-resume.html>
> (i915#1185 <https://gitlab.freedesktop.org/drm/intel/issues/1185>)
>
> o
>
> shard-apl: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-apl4/igt@gem_workarounds@suspend-resume.html>
> -> DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-apl4/igt@gem_workarounds@suspend-resume.html>
> (i915#1635
> <https://gitlab.freedesktop.org/drm/intel/issues/1635> /
> i915#2635 <https://gitlab.freedesktop.org/drm/intel/issues/2635>)
>
> o
>
> shard-kbl: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-kbl3/igt@gem_workarounds@suspend-resume.html>
> -> INCOMPLETE
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-kbl7/igt@gem_workarounds@suspend-resume.html>
> (i915#155 <https://gitlab.freedesktop.org/drm/intel/issues/155>)
>
> o
>
> shard-hsw: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-hsw1/igt@gem_workarounds@suspend-resume.html>
> -> INCOMPLETE
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-hsw5/igt@gem_workarounds@suspend-resume.html>
> (i915#2637 <https://gitlab.freedesktop.org/drm/intel/issues/2637>)
>
> *
>
> igt@i915_pm_rpm@modeset-lpsp-stress-no-wait:
>
> o shard-iclb: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb5/igt@i915_pm_rpm@modeset-lpsp-stress-no-wait.html>
> -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb6/igt@i915_pm_rpm@modeset-lpsp-stress-no-wait.html>
> (i915#579 <https://gitlab.freedesktop.org/drm/intel/issues/579>)
> *
>
> igt@kms_cursor_crc@pipe-a-cursor-64x21-onscreen:
>
> o shard-kbl: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-kbl7/igt@kms_cursor_crc@pipe-a-cursor-64x21-onscreen.html>
> -> DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-kbl2/igt@kms_cursor_crc@pipe-a-cursor-64x21-onscreen.html>
> (i915#78 <https://gitlab.freedesktop.org/drm/intel/issues/78>)
> +2 similar issues
> *
>
> igt@kms_cursor_edge_walk@pipe-a-64x64-left-edge:
>
> o shard-glk: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-glk3/igt@kms_cursor_edge_walk@pipe-a-64x64-left-edge.html>
> -> DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-glk1/igt@kms_cursor_edge_walk@pipe-a-64x64-left-edge.html>
> (i915#1982
> <https://gitlab.freedesktop.org/drm/intel/issues/1982>) +6
> similar issues
> *
>
> igt@kms_cursor_legacy@2x-long-cursor-vs-flip-atomic:
>
> o shard-hsw: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-hsw1/igt@kms_cursor_legacy@2x-long-cursor-vs-flip-atomic.html>
> -> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-hsw1/igt@kms_cursor_legacy@2x-long-cursor-vs-flip-atomic.html>
> (i915#96 <https://gitlab.freedesktop.org/drm/intel/issues/96>)
> *
>
> igt@kms_cursor_legacy@flip-vs-cursor-legacy:
>
> o shard-apl: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-apl8/igt@kms_cursor_legacy@flip-vs-cursor-legacy.html>
> -> DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-apl6/igt@kms_cursor_legacy@flip-vs-cursor-legacy.html>
> (i915#1635
> <https://gitlab.freedesktop.org/drm/intel/issues/1635> /
> i915#1982
> <https://gitlab.freedesktop.org/drm/intel/issues/1982>) +1
> similar issue
> *
>
> igt@kms_cursor_legacy@flip-vs-cursor-varying-size:
>
> o shard-tglb: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-tglb2/igt@kms_cursor_legacy@flip-vs-cursor-varying-size.html>
> -> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-tglb3/igt@kms_cursor_legacy@flip-vs-cursor-varying-size.html>
> (i915#2346 <https://gitlab.freedesktop.org/drm/intel/issues/2346>)
> *
>
> igt@kms_flip@flip-vs-blocking-wf-vblank@a-dp1:
>
> o shard-kbl: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-kbl4/igt@kms_flip@flip-vs-blocking-wf-vblank@a-dp1.html>
> -> DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-kbl3/igt@kms_flip@flip-vs-blocking-wf-vblank@a-dp1.html>
> (i915#1982
> <https://gitlab.freedesktop.org/drm/intel/issues/1982>) +5
> similar issues
> *
>
> igt@kms_flip@flip-vs-expired-vblank@a-edp1:
>
> o shard-tglb: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-tglb2/igt@kms_flip@flip-vs-expired-vblank@a-edp1.html>
> -> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-tglb8/igt@kms_flip@flip-vs-expired-vblank@a-edp1.html>
> (i915#2598
> <https://gitlab.freedesktop.org/drm/intel/issues/2598>) +1
> similar issue
> *
>
> igt@kms_flip@flip-vs-suspend-interruptible@c-hdmi-a1:
>
> o shard-hsw: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-hsw2/igt@kms_flip@flip-vs-suspend-interruptible@c-hdmi-a1.html>
> -> INCOMPLETE
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-hsw6/igt@kms_flip@flip-vs-suspend-interruptible@c-hdmi-a1.html>
> (i915#2295 <https://gitlab.freedesktop.org/drm/intel/issues/2295>)
> *
>
> igt@kms_frontbuffer_tracking@psr-farfromfence:
>
> o shard-tglb: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-tglb5/igt@kms_frontbuffer_tracking@psr-farfromfence.html>
> -> DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-tglb3/igt@kms_frontbuffer_tracking@psr-farfromfence.html>
> (i915#1982
> <https://gitlab.freedesktop.org/drm/intel/issues/1982>) +4
> similar issues
> *
>
> igt@kms_psr@psr2_cursor_plane_onoff:
>
> o shard-iclb: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb2/igt@kms_psr@psr2_cursor_plane_onoff.html>
> -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb3/igt@kms_psr@psr2_cursor_plane_onoff.html>
> (fdo#109441
> <https://bugs.freedesktop.org/show_bug.cgi?id=109441>) +2
> similar issues
> *
>
> igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm:
>
> o
>
> shard-glk: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-glk4/igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm.html>
> -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-glk6/igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm.html>
> (fdo#109271 <https://bugs.freedesktop.org/show_bug.cgi?id=109271>)
>
> o
>
> shard-hsw: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-hsw8/igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm.html>
> -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-hsw5/igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm.html>
> (fdo#109271 <https://bugs.freedesktop.org/show_bug.cgi?id=109271>)
>
> o
>
> shard-iclb: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb8/igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm.html>
> -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb2/igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm.html>
> (fdo#109278 <https://bugs.freedesktop.org/show_bug.cgi?id=109278>)
>
> o
>
> shard-kbl: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-kbl7/igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm.html>
> -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-kbl7/igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm.html>
> (fdo#109271 <https://bugs.freedesktop.org/show_bug.cgi?id=109271>)
>
> o
>
> shard-apl: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-apl6/igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm.html>
> -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-apl4/igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm.html>
> (fdo#109271
> <https://bugs.freedesktop.org/show_bug.cgi?id=109271> /
> i915#1635 <https://gitlab.freedesktop.org/drm/intel/issues/1635>)
>
> *
>
> igt@perf@polling-parameterized:
>
> o shard-apl: PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-apl7/igt@perf@polling-parameterized.html>
> -> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-apl7/igt@perf@polling-parameterized.html>
> (i915#1542
> <https://gitlab.freedesktop.org/drm/intel/issues/1542> /
> i915#1635 <https://gitlab.freedesktop.org/drm/intel/issues/1635>)
>
>
> Possible fixes
>
> *
>
> igt@device_reset@unbind-reset-rebind:
>
> o
>
> shard-glk: INCOMPLETE
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-glk2/igt@device_reset@unbind-reset-rebind.html>
> (i915#2283
> <https://gitlab.freedesktop.org/drm/intel/issues/2283> /
> i915#2405
> <https://gitlab.freedesktop.org/drm/intel/issues/2405>) ->
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-glk5/igt@device_reset@unbind-reset-rebind.html>
>
> o
>
> shard-apl: INCOMPLETE
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-apl6/igt@device_reset@unbind-reset-rebind.html>
> (i915#1635
> <https://gitlab.freedesktop.org/drm/intel/issues/1635> /
> i915#2283
> <https://gitlab.freedesktop.org/drm/intel/issues/2283> /
> i915#2405
> <https://gitlab.freedesktop.org/drm/intel/issues/2405>) ->
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-apl6/igt@device_reset@unbind-reset-rebind.html>
>
> o
>
> shard-kbl: INCOMPLETE
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-kbl6/igt@device_reset@unbind-reset-rebind.html>
> (i915#2283
> <https://gitlab.freedesktop.org/drm/intel/issues/2283> /
> i915#2405
> <https://gitlab.freedesktop.org/drm/intel/issues/2405>) ->
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-kbl3/igt@device_reset@unbind-reset-rebind.html>
>
> o
>
> shard-tglb: INCOMPLETE
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-tglb3/igt@device_reset@unbind-reset-rebind.html>
> (i915#1602
> <https://gitlab.freedesktop.org/drm/intel/issues/1602> /
> i915#750
> <https://gitlab.freedesktop.org/drm/intel/issues/750>) -> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-tglb3/igt@device_reset@unbind-reset-rebind.html>
>
> o
>
> shard-iclb: INCOMPLETE
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb8/igt@device_reset@unbind-reset-rebind.html>
> (i915#2283
> <https://gitlab.freedesktop.org/drm/intel/issues/2283> /
> i915#2405
> <https://gitlab.freedesktop.org/drm/intel/issues/2405>) ->
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb3/igt@device_reset@unbind-reset-rebind.html>
>
> *
>
> igt@feature_discovery@psr2:
>
> o shard-iclb: SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb1/igt@feature_discovery@psr2.html>
> (i915#658
> <https://gitlab.freedesktop.org/drm/intel/issues/658>) -> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb2/igt@feature_discovery@psr2.html>
> *
>
> igt@gem_exec_whisper@basic-fds-forked:
>
> o shard-glk: DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-glk4/igt@gem_exec_whisper@basic-fds-forked.html>
> (i915#118
> <https://gitlab.freedesktop.org/drm/intel/issues/118> /
> i915#95 <https://gitlab.freedesktop.org/drm/intel/issues/95>)
> -> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-glk3/igt@gem_exec_whisper@basic-fds-forked.html>
> *
>
> igt@i915_pm_backlight@fade_with_suspend:
>
> o shard-iclb: DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb7/igt@i915_pm_backlight@fade_with_suspend.html>
> -> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb7/igt@i915_pm_backlight@fade_with_suspend.html>
> *
>
> igt@i915_pm_dc@dc6-psr:
>
> o shard-iclb: FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb8/igt@i915_pm_dc@dc6-psr.html>
> (i915#454
> <https://gitlab.freedesktop.org/drm/intel/issues/454>) -> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb3/igt@i915_pm_dc@dc6-psr.html>
> *
>
> igt@i915_pm_rpm@gem-execbuf-stress:
>
> o
>
> shard-glk: SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-glk6/igt@i915_pm_rpm@gem-execbuf-stress.html>
> (fdo#109271
> <https://bugs.freedesktop.org/show_bug.cgi?id=109271>) -> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-glk5/igt@i915_pm_rpm@gem-execbuf-stress.html>
>
> o
>
> shard-apl: SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-apl1/igt@i915_pm_rpm@gem-execbuf-stress.html>
> (fdo#109271
> <https://bugs.freedesktop.org/show_bug.cgi?id=109271> /
> i915#1635
> <https://gitlab.freedesktop.org/drm/intel/issues/1635>) ->
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-apl6/igt@i915_pm_rpm@gem-execbuf-stress.html>
>
> o
>
> shard-kbl: SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-kbl1/igt@i915_pm_rpm@gem-execbuf-stress.html>
> (fdo#109271
> <https://bugs.freedesktop.org/show_bug.cgi?id=109271>) -> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-kbl3/igt@i915_pm_rpm@gem-execbuf-stress.html>
>
> o
>
> shard-hsw: SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-hsw4/igt@i915_pm_rpm@gem-execbuf-stress.html>
> (fdo#109271
> <https://bugs.freedesktop.org/show_bug.cgi?id=109271>) -> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-hsw2/igt@i915_pm_rpm@gem-execbuf-stress.html>
>
> *
>
> igt@kms_busy@basic-modeset-pipe-c:
>
> o shard-hsw: DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-hsw4/igt@kms_busy@basic-modeset-pipe-c.html>
> (i915#44 <https://gitlab.freedesktop.org/drm/intel/issues/44>)
> -> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-hsw2/igt@kms_busy@basic-modeset-pipe-c.html>
> *
>
> igt@kms_cursor_edge_walk@pipe-c-256x256-left-edge:
>
> o shard-apl: DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-apl3/igt@kms_cursor_edge_walk@pipe-c-256x256-left-edge.html>
> (i915#1635
> <https://gitlab.freedesktop.org/drm/intel/issues/1635> /
> i915#1982
> <https://gitlab.freedesktop.org/drm/intel/issues/1982>) ->
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-apl2/igt@kms_cursor_edge_walk@pipe-c-256x256-left-edge.html>
> +4 similar issues
> *
>
> igt@kms_cursor_legacy@basic-flip-before-cursor-legacy:
>
> o shard-glk: DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-glk1/igt@kms_cursor_legacy@basic-flip-before-cursor-legacy.html>
> (i915#1982
> <https://gitlab.freedesktop.org/drm/intel/issues/1982>) ->
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-glk8/igt@kms_cursor_legacy@basic-flip-before-cursor-legacy.html>
> +4 similar issues
> *
>
> igt@kms_fbcon_fbt@fbc:
>
> o shard-glk: FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-glk6/igt@kms_fbcon_fbt@fbc.html>
> (i915#64 <https://gitlab.freedesktop.org/drm/intel/issues/64>)
> -> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-glk7/igt@kms_fbcon_fbt@fbc.html>
> *
>
> igt@kms_flip@flip-vs-rmfb-interruptible@c-dp1:
>
> o shard-kbl: DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-kbl2/igt@kms_flip@flip-vs-rmfb-interruptible@c-dp1.html>
> (i915#62 <https://gitlab.freedesktop.org/drm/intel/issues/62>)
> -> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-kbl2/igt@kms_flip@flip-vs-rmfb-interruptible@c-dp1.html>
> *
>
> igt@kms_frontbuffer_tracking@fbc-indfb-scaledprimary:
>
> o
>
> shard-kbl: FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-kbl1/igt@kms_frontbuffer_tracking@fbc-indfb-scaledprimary.html>
> (i915#49 <https://gitlab.freedesktop.org/drm/intel/issues/49>)
> -> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-kbl4/igt@kms_frontbuffer_tracking@fbc-indfb-scaledprimary.html>
>
> o
>
> shard-apl: FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-apl1/igt@kms_frontbuffer_tracking@fbc-indfb-scaledprimary.html>
> (i915#1635
> <https://gitlab.freedesktop.org/drm/intel/issues/1635> /
> i915#49 <https://gitlab.freedesktop.org/drm/intel/issues/49>)
> -> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-apl2/igt@kms_frontbuffer_tracking@fbc-indfb-scaledprimary.html>
>
> *
>
> igt@kms_frontbuffer_tracking@fbc-stridechange:
>
> o shard-tglb: DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-tglb3/igt@kms_frontbuffer_tracking@fbc-stridechange.html>
> (i915#1982
> <https://gitlab.freedesktop.org/drm/intel/issues/1982>) ->
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-tglb2/igt@kms_frontbuffer_tracking@fbc-stridechange.html>
> +1 similar issue
> *
>
> igt@kms_psr@psr2_sprite_blt:
>
> o shard-iclb: SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb1/igt@kms_psr@psr2_sprite_blt.html>
> (fdo#109441
> <https://bugs.freedesktop.org/show_bug.cgi?id=109441>) -> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb2/igt@kms_psr@psr2_sprite_blt.html>
> +2 similar issues
> *
>
> igt@kms_vblank@pipe-b-wait-forked-busy:
>
> o shard-kbl: DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-kbl1/igt@kms_vblank@pipe-b-wait-forked-busy.html>
> (i915#1982
> <https://gitlab.freedesktop.org/drm/intel/issues/1982>) ->
> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-kbl2/igt@kms_vblank@pipe-b-wait-forked-busy.html>
> +4 similar issues
> *
>
> igt@perf_pmu@module-unload:
>
> o shard-apl: DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-apl3/igt@perf_pmu@module-unload.html>
> (i915#1635
> <https://gitlab.freedesktop.org/drm/intel/issues/1635> /
> i915#1982
> <https://gitlab.freedesktop.org/drm/intel/issues/1982> /
> i915#262
> <https://gitlab.freedesktop.org/drm/intel/issues/262>) -> PASS
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-apl1/igt@perf_pmu@module-unload.html>
>
>
> Warnings
>
> *
>
> igt@device_reset@unbind-reset-rebind:
>
> o shard-hsw: INCOMPLETE
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-hsw6/igt@device_reset@unbind-reset-rebind.html>
> (i915#2283
> <https://gitlab.freedesktop.org/drm/intel/issues/2283> /
> i915#2405
> <https://gitlab.freedesktop.org/drm/intel/issues/2405>) ->
> WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-hsw2/igt@device_reset@unbind-reset-rebind.html>
> (i915#2283 <https://gitlab.freedesktop.org/drm/intel/issues/2283>)
> *
>
> igt@gem_workarounds@suspend-resume:
>
> o shard-tglb: DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-tglb1/igt@gem_workarounds@suspend-resume.html>
> (i915#2411
> <https://gitlab.freedesktop.org/drm/intel/issues/2411>) ->
> DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-tglb5/igt@gem_workarounds@suspend-resume.html>
> (i915#1436
> <https://gitlab.freedesktop.org/drm/intel/issues/1436> /
> i915#1602
> <https://gitlab.freedesktop.org/drm/intel/issues/1602> /
> i915#1887
> <https://gitlab.freedesktop.org/drm/intel/issues/1887> /
> i915#2411 <https://gitlab.freedesktop.org/drm/intel/issues/2411>)
> *
>
> igt@i915_pm_dc@dc3co-vpb-simulation:
>
> o shard-iclb: SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb2/igt@i915_pm_dc@dc3co-vpb-simulation.html>
> (i915#588
> <https://gitlab.freedesktop.org/drm/intel/issues/588>) -> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb3/igt@i915_pm_dc@dc3co-vpb-simulation.html>
> (i915#658 <https://gitlab.freedesktop.org/drm/intel/issues/658>)
> *
>
> igt@i915_pm_rc6_residency@rc6-fence:
>
> o shard-iclb: WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb3/igt@i915_pm_rc6_residency@rc6-fence.html>
> (i915#1804
> <https://gitlab.freedesktop.org/drm/intel/issues/1804> /
> i915#2684
> <https://gitlab.freedesktop.org/drm/intel/issues/2684>) ->
> WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb5/igt@i915_pm_rc6_residency@rc6-fence.html>
> (i915#2684 <https://gitlab.freedesktop.org/drm/intel/issues/2684>)
> *
>
> igt@i915_pm_rpm@modeset-lpsp-stress-no-wait:
>
> o shard-tglb: DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-tglb5/igt@i915_pm_rpm@modeset-lpsp-stress-no-wait.html>
> (i915#2411
> <https://gitlab.freedesktop.org/drm/intel/issues/2411>) ->
> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-tglb1/igt@i915_pm_rpm@modeset-lpsp-stress-no-wait.html>
> (i915#579 <https://gitlab.freedesktop.org/drm/intel/issues/579>)
> *
>
> igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm:
>
> o shard-tglb: DMESG-WARN
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-tglb7/igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm.html>
> (i915#2411
> <https://gitlab.freedesktop.org/drm/intel/issues/2411>) ->
> SKIP
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-tglb5/igt@kms_vblank@pipe-c-ts-continuation-dpms-rpm.html>
> (i915#2648 <https://gitlab.freedesktop.org/drm/intel/issues/2648>)
> *
>
> igt@runner@aborted:
>
> o
>
> shard-hsw: FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-hsw6/igt@runner@aborted.html>
> (i915#2283
> <https://gitlab.freedesktop.org/drm/intel/issues/2283> /
> i915#2295
> <https://gitlab.freedesktop.org/drm/intel/issues/2295>) ->
> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-hsw5/igt@runner@aborted.html>
> (fdo#109271
> <https://bugs.freedesktop.org/show_bug.cgi?id=109271> /
> i915#2295 <https://gitlab.freedesktop.org/drm/intel/issues/2295>)
>
> o
>
> shard-iclb: (FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb7/igt@runner@aborted.html>,
> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb8/igt@runner@aborted.html>,
> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-iclb3/igt@runner@aborted.html>)
> (i915#1814
> <https://gitlab.freedesktop.org/drm/intel/issues/1814> /
> i915#2283
> <https://gitlab.freedesktop.org/drm/intel/issues/2283> /
> i915#2295
> <https://gitlab.freedesktop.org/drm/intel/issues/2295> /
> i915#483
> <https://gitlab.freedesktop.org/drm/intel/issues/483>) ->
> (FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb2/igt@runner@aborted.html>,
> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-iclb3/igt@runner@aborted.html>)
> (i915#2295 <https://gitlab.freedesktop.org/drm/intel/issues/2295>)
>
> o
>
> shard-apl: (FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-apl3/igt@runner@aborted.html>,
> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-apl6/igt@runner@aborted.html>)
> (i915#1635
> <https://gitlab.freedesktop.org/drm/intel/issues/1635> /
> i915#2295
> <https://gitlab.freedesktop.org/drm/intel/issues/2295>) ->
> (FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-apl4/igt@runner@aborted.html>,
> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-apl6/igt@runner@aborted.html>)
> (fdo#109271
> <https://bugs.freedesktop.org/show_bug.cgi?id=109271> /
> i915#1635
> <https://gitlab.freedesktop.org/drm/intel/issues/1635> /
> i915#2295 <https://gitlab.freedesktop.org/drm/intel/issues/2295>)
>
> o
>
> shard-tglb: (FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-tglb5/igt@runner@aborted.html>,
> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-tglb6/igt@runner@aborted.html>,
> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9389/shard-tglb3/igt@runner@aborted.html>)
> (i915#2295
> <https://gitlab.freedesktop.org/drm/intel/issues/2295>) ->
> (FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-tglb8/igt@runner@aborted.html>,
> FAIL
> <https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/shard-tglb5/igt@runner@aborted.html>)
> (i915#1602
> <https://gitlab.freedesktop.org/drm/intel/issues/1602> /
> i915#2295 <https://gitlab.freedesktop.org/drm/intel/issues/2295>)
>
> {name}: This element is suppressed. This means it is ignored when
> computing
> the status of the difference (SUCCESS, WARNING, or FAILURE).
>
>
> Participating hosts (10 -> 8)
>
> Missing (2): pig-skl-6260u pig-glk-j5005
>
>
> Build changes
>
> * CI: CI-20190529 -> None
> * IGT: IGT_5870 -> IGTPW_5226
> * Piglit: piglit_4509 -> None
>
> CI-20190529: 20190529
> CI_DRM_9389: b0c2cf3ad04abd9e7a44abe12e736bb5ab587393 @
> git://anongit.freedesktop.org/gfx-ci/linux
> IGTPW_5226: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5226/index.html
> IGT_5870: 08b13995b85df26a77212e4fb21fd772976ef33c @
> git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
> piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @
> git://anongit.freedesktop.org/piglit
>
[-- Attachment #1.2: Type: text/html, Size: 47423 bytes --]
[-- Attachment #2: Type: text/plain, Size: 154 bytes --]
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [igt-dev] [PATCH i-g-t v2 1/3] tests/i915/perf: constify report checks functions
2020-11-25 11:25 ` [igt-dev] [PATCH i-g-t v2 1/3] tests/i915/perf: constify report checks functions Lionel Landwerlin
@ 2020-11-25 21:43 ` Umesh Nerlige Ramappa
0 siblings, 0 replies; 11+ messages in thread
From: Umesh Nerlige Ramappa @ 2020-11-25 21:43 UTC (permalink / raw)
To: Lionel Landwerlin; +Cc: igt-dev
On Wed, Nov 25, 2020 at 01:25:38PM +0200, Lionel Landwerlin wrote:
>Those don't need to modify the reports they inspect.
>
>Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Reviewed-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
Thanks,
Umesh
>---
> tests/i915/perf.c | 20 ++++++++++++--------
> 1 file changed, 12 insertions(+), 8 deletions(-)
>
>diff --git a/tests/i915/perf.c b/tests/i915/perf.c
>index caeabd623..2fbea0f0c 100644
>--- a/tests/i915/perf.c
>+++ b/tests/i915/perf.c
>@@ -206,9 +206,10 @@ static bool *undefined_a_counters;
> static uint64_t oa_exp_1_millisec;
>
> static igt_render_copyfunc_t render_copy = NULL;
>-static uint32_t (*read_report_ticks)(uint32_t *report,
>+static uint32_t (*read_report_ticks)(const uint32_t *report,
> enum drm_i915_oa_format format);
>-static void (*sanity_check_reports)(uint32_t *oa_report0, uint32_t *oa_report1,
>+static void (*sanity_check_reports)(const uint32_t *oa_report0,
>+ const uint32_t *oa_report1,
> enum drm_i915_oa_format format);
>
> static void
>@@ -354,7 +355,7 @@ sysfs_read(const char *path)
> * but it's not included in all of the formats.
> */
> static uint32_t
>-hsw_read_report_ticks(uint32_t *report, enum drm_i915_oa_format format)
>+hsw_read_report_ticks(const uint32_t *report, enum drm_i915_oa_format format)
> {
> uint32_t *c = (uint32_t *)(((uint8_t *)report) + get_oa_format(format).c_off);
>
>@@ -364,13 +365,13 @@ hsw_read_report_ticks(uint32_t *report, enum drm_i915_oa_format format)
> }
>
> static uint32_t
>-gen8_read_report_ticks(uint32_t *report, enum drm_i915_oa_format format)
>+gen8_read_report_ticks(const uint32_t *report, enum drm_i915_oa_format format)
> {
> return report[3];
> }
>
> static void
>-gen8_read_report_clock_ratios(uint32_t *report,
>+gen8_read_report_clock_ratios(const uint32_t *report,
> uint32_t *slice_freq_mhz,
> uint32_t *unslice_freq_mhz)
> {
>@@ -554,7 +555,8 @@ emit_report_perf_count(struct intel_bb *ibb,
> }
>
> static void
>-hsw_sanity_check_render_basic_reports(uint32_t *oa_report0, uint32_t *oa_report1,
>+hsw_sanity_check_render_basic_reports(const uint32_t *oa_report0,
>+ const uint32_t *oa_report1,
> enum drm_i915_oa_format fmt)
> {
> uint32_t time_delta = timebase_scale(oa_report1[1] - oa_report0[1]);
>@@ -639,7 +641,8 @@ hsw_sanity_check_render_basic_reports(uint32_t *oa_report0, uint32_t *oa_report1
> }
>
> static uint64_t
>-gen8_read_40bit_a_counter(uint32_t *report, enum drm_i915_oa_format fmt, int a_id)
>+gen8_read_40bit_a_counter(const uint32_t *report,
>+ enum drm_i915_oa_format fmt, int a_id)
> {
> struct oa_format format = get_oa_format(fmt);
> uint8_t *a40_high = (((uint8_t *)report) + format.a40_high_off);
>@@ -757,7 +760,8 @@ accumulator_print(struct accumulator *accumulator, const char *title)
>
> /* The TestOa metric set is designed so */
> static void
>-gen8_sanity_check_test_oa_reports(uint32_t *oa_report0, uint32_t *oa_report1,
>+gen8_sanity_check_test_oa_reports(const uint32_t *oa_report0,
>+ const uint32_t *oa_report1,
> enum drm_i915_oa_format fmt)
> {
> struct oa_format format = get_oa_format(fmt);
>--
>2.29.2
>
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [igt-dev] [PATCH i-g-t v2 2/3] tests/i915/perf: deal with 0 delta reports
2020-11-25 11:25 ` [igt-dev] [PATCH i-g-t v2 2/3] tests/i915/perf: deal with 0 delta reports Lionel Landwerlin
@ 2020-11-25 22:12 ` Umesh Nerlige Ramappa
0 siblings, 0 replies; 11+ messages in thread
From: Umesh Nerlige Ramappa @ 2020-11-25 22:12 UTC (permalink / raw)
To: Lionel Landwerlin; +Cc: igt-dev
On Wed, Nov 25, 2020 at 01:25:39PM +0200, Lionel Landwerlin wrote:
>More thorough testing shows that it is possible to get a timer report
>followed by a context-switch report with the same timestamp.
>
>Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
nit below.
Reviewed-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
Thanks,
Umesh
>---
> tests/i915/perf.c | 91 +++++++++++++++++++++++++----------------------
> 1 file changed, 49 insertions(+), 42 deletions(-)
>
>diff --git a/tests/i915/perf.c b/tests/i915/perf.c
>index 2fbea0f0c..811067aad 100644
>--- a/tests/i915/perf.c
>+++ b/tests/i915/perf.c
>@@ -778,15 +778,14 @@ gen8_sanity_check_test_oa_reports(const uint32_t *oa_report0,
> uint32_t b;
> uint32_t ref;
>
>+ igt_debug("report type: %s->%s %p/%p\n",
>+ gen8_read_report_reason(oa_report0),
>+ gen8_read_report_reason(oa_report1),
>+ oa_report0, oa_report1);
>
>- igt_assert_neq(time_delta, 0);
>- igt_assert_neq(clock_delta, 0);
>-
>- freq = ((uint64_t)clock_delta * 1000) / time_delta;
>+ freq = time_delta ? ((uint64_t)clock_delta * 1000) / time_delta : 0;
> igt_debug("freq = %"PRIu64"\n", freq);
Maybe one more check:
igt_assert(freq || (report_reason0 != report_reason1));
>
>- igt_assert(freq <= gt_max_freq_mhz);
>-
> igt_debug("clock delta = %"PRIu32"\n", clock_delta);
>
> max_delta = clock_delta * intel_perf->devinfo.n_eus;
>@@ -823,42 +822,50 @@ gen8_sanity_check_test_oa_reports(const uint32_t *oa_report0,
> * multiple of the gpu clock
> */
> if (format.n_b) {
>- b = rpt1_b[0] - rpt0_b[0];
>- igt_debug("B0: delta = %"PRIu32"\n", b);
>- igt_assert_eq(b, 0);
>-
>- b = rpt1_b[1] - rpt0_b[1];
>- igt_debug("B1: delta = %"PRIu32"\n", b);
>- igt_assert_eq(b, clock_delta);
>-
>- b = rpt1_b[2] - rpt0_b[2];
>- igt_debug("B2: delta = %"PRIu32"\n", b);
>- igt_assert_eq(b, clock_delta);
>-
>- b = rpt1_b[3] - rpt0_b[3];
>- ref = clock_delta / 2;
>- igt_debug("B3: delta = %"PRIu32"\n", b);
>- igt_assert(b >= ref - 1 && b <= ref + 1);
>-
>- b = rpt1_b[4] - rpt0_b[4];
>- ref = clock_delta / 3;
>- igt_debug("B4: delta = %"PRIu32"\n", b);
>- igt_assert(b >= ref - 1 && b <= ref + 1);
>-
>- b = rpt1_b[5] - rpt0_b[5];
>- ref = clock_delta / 3;
>- igt_debug("B5: delta = %"PRIu32"\n", b);
>- igt_assert(b >= ref - 1 && b <= ref + 1);
>-
>- b = rpt1_b[6] - rpt0_b[6];
>- ref = clock_delta / 6;
>- igt_debug("B6: delta = %"PRIu32"\n", b);
>- igt_assert(b >= ref - 1 && b <= ref + 1);
>-
>- b = rpt1_b[7] - rpt0_b[7];
>- ref = clock_delta * 2 / 3;
>- igt_debug("B7: delta = %"PRIu32"\n", b);
>- igt_assert(b >= ref - 1 && b <= ref + 1);
>+ if (clock_delta > 0) {
>+ b = rpt1_b[0] - rpt0_b[0];
>+ igt_debug("B0: delta = %"PRIu32"\n", b);
>+ igt_assert_eq(b, 0);
>+
>+ b = rpt1_b[1] - rpt0_b[1];
>+ igt_debug("B1: delta = %"PRIu32"\n", b);
>+ igt_assert_eq(b, clock_delta);
>+
>+ b = rpt1_b[2] - rpt0_b[2];
>+ igt_debug("B2: delta = %"PRIu32"\n", b);
>+ igt_assert_eq(b, clock_delta);
>+
>+ b = rpt1_b[3] - rpt0_b[3];
>+ ref = clock_delta / 2;
>+ igt_debug("B3: delta = %"PRIu32"\n", b);
>+ igt_assert(b >= ref - 1 && b <= ref + 1);
>+
>+ b = rpt1_b[4] - rpt0_b[4];
>+ ref = clock_delta / 3;
>+ igt_debug("B4: delta = %"PRIu32"\n", b);
>+ igt_assert(b >= ref - 1 && b <= ref + 1);
>+
>+ b = rpt1_b[5] - rpt0_b[5];
>+ ref = clock_delta / 3;
>+ igt_debug("B5: delta = %"PRIu32"\n", b);
>+ igt_assert(b >= ref - 1 && b <= ref + 1);
>+
>+ b = rpt1_b[6] - rpt0_b[6];
>+ ref = clock_delta / 6;
>+ igt_debug("B6: delta = %"PRIu32"\n", b);
>+ igt_assert(b >= ref - 1 && b <= ref + 1);
>+
>+ b = rpt1_b[7] - rpt0_b[7];
>+ ref = clock_delta * 2 / 3;
>+ igt_debug("B7: delta = %"PRIu32"\n", b);
>+ igt_assert(b >= ref - 1 && b <= ref + 1);
>+ } else {
>+ for (int j = 0; j < format.n_b; j++) {
>+ b = rpt1_b[j] - rpt0_b[j];
>+ igt_debug("B%i: delta = %"PRIu32"\n", j, b);
>+ igt_assert_eq(b, 0);
>+ }
>+ }
> }
>
> for (int j = 0; j < format.n_c; j++) {
>--
>2.29.2
>
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [igt-dev] [PATCH i-g-t v2 3/3] tests/i915/perf: verify reason field in OA reports is never 0
2020-11-25 11:25 ` [igt-dev] [PATCH i-g-t v2 3/3] tests/i915/perf: verify reason field in OA reports is never 0 Lionel Landwerlin
@ 2020-11-25 22:35 ` Umesh Nerlige Ramappa
2020-11-26 11:40 ` Lionel Landwerlin
0 siblings, 1 reply; 11+ messages in thread
From: Umesh Nerlige Ramappa @ 2020-11-25 22:35 UTC (permalink / raw)
To: Lionel Landwerlin; +Cc: igt-dev
On Wed, Nov 25, 2020 at 01:25:40PM +0200, Lionel Landwerlin wrote:
>We're about to remove the filtering in i915 on 0 reason fields because
>we assume this was a possibility, but it turned out to be a corruption
>in the tail pointer register that made us read cleared data.
>
>This test is here to verify that our assumption hold that the HW never
>produces such reports.
>
>v2: Fix len checking (Umesh)
> Count report lost events (Umesh)
> Check report sanity (Umesh)
> Limit test to 3 times the OA buffer (Umesh)
>
>Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
>---
> tests/i915/perf.c | 90 +++++++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 90 insertions(+)
>
>diff --git a/tests/i915/perf.c b/tests/i915/perf.c
>index 811067aad..2d442df96 100644
>--- a/tests/i915/perf.c
>+++ b/tests/i915/perf.c
>@@ -2550,6 +2550,89 @@ test_buffer_fill(void)
> __perf_close(stream_fd);
> }
>
>+static void
>+test_non_zero_reason(void)
>+{
>+ /* ~10 micro second period */
>+ int oa_exponent = max_oa_exponent_for_period_lte(10000);
>+ uint64_t properties[] = {
>+ /* Include OA reports in samples */
>+ DRM_I915_PERF_PROP_SAMPLE_OA, true,
>+
>+ /* OA unit configuration */
>+ DRM_I915_PERF_PROP_OA_METRICS_SET, test_set->perf_oa_metrics_set,
>+ DRM_I915_PERF_PROP_OA_FORMAT, test_set->perf_oa_format,
>+ DRM_I915_PERF_PROP_OA_EXPONENT, oa_exponent,
>+ };
>+ struct drm_i915_perf_open_param param = {
>+ .flags = I915_PERF_FLAG_FD_CLOEXEC,
>+ .num_properties = sizeof(properties) / 16,
>+ .properties_ptr = to_user_pointer(properties),
>+ };
>+ struct drm_i915_perf_record_header *header;
>+ uint32_t buf_size = 3 * 65536 * (256 + sizeof(struct drm_i915_perf_record_header));
>+ uint8_t *buf = malloc(buf_size);
>+ uint32_t total_len = 0, reports_lost;
>+ const uint32_t *last_report;
>+ int len;
>+
>+ igt_assert(buf);
nit: Looks like sanity check will log the report0/report1 pointers. We
could log the buf pointer here so we get a sense of the report index
that failed.
Either ways, this is
Reviewed-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
Thanks for adding this test,
Umesh
>+
>+ igt_debug("Ready to read about %u bytes\n", buf_size);
>+
>+ load_helper_init();
>+ load_helper_run(HIGH);
>+
>+ stream_fd = __perf_open(drm_fd, ¶m, true /* prevent_pm */);
>+
>+ while (total_len < (buf_size - sizeof(struct drm_i915_perf_record_header)) &&
>+ ((len = read(stream_fd, &buf[total_len], buf_size - total_len)) > 0 ||
>+ (len == -1 && errno == EINTR))) {
>+ if (len > 0)
>+ total_len += len;
>+ }
>+
>+ __perf_close(stream_fd);
>+
>+ load_helper_stop();
>+ load_helper_fini();
>+
>+ igt_debug("Got %u bytes\n", total_len);
>+
>+ last_report = NULL;
>+ reports_lost = 0;
>+ for (uint32_t offset = 0; offset < total_len; offset += header->size) {
>+ header = (void *) (buf + offset);
>+
>+ switch (header->type) {
>+ case DRM_I915_PERF_RECORD_OA_REPORT_LOST:
>+ reports_lost++;
>+ break;
>+ case DRM_I915_PERF_RECORD_SAMPLE: {
>+ const uint32_t *report = (void *) (header + 1);
>+ uint32_t reason = (report[0] >> OAREPORT_REASON_SHIFT) &
>+ OAREPORT_REASON_MASK;
>+
>+ igt_assert_neq(reason, 0);
>+
>+ if (last_report) {
>+ sanity_check_reports(last_report, report,
>+ test_set->perf_oa_format);
>+ }
>+ last_report = report;
>+ break;
>+ }
>+ case DRM_I915_PERF_RECORD_OA_BUFFER_LOST:
>+ igt_assert(!"unexpected overflow");
>+ break;
>+ }
>+ }
>+
>+ igt_debug("Got %u report lost events\n", reports_lost);
>+
>+ free(buf);
>+}
>+
> static void
> test_enable_disable(void)
> {
>@@ -4893,6 +4976,13 @@ igt_main
> igt_subtest("buffer-fill")
> test_buffer_fill();
>
>+ igt_describe("Test that reason field in OA reports is never 0 on Gen8+");
>+ igt_subtest("non-zero-reason") {
>+ /* Reason field is only available on Gen8+ */
>+ igt_require(intel_gen(devid) >= 8);
>+ test_non_zero_reason();
>+ }
>+
> igt_subtest("disabled-read-error")
> test_disabled_read_error();
> igt_subtest("non-sampling-read-error")
>--
>2.29.2
>
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [igt-dev] [PATCH i-g-t v2 3/3] tests/i915/perf: verify reason field in OA reports is never 0
2020-11-25 22:35 ` Umesh Nerlige Ramappa
@ 2020-11-26 11:40 ` Lionel Landwerlin
0 siblings, 0 replies; 11+ messages in thread
From: Lionel Landwerlin @ 2020-11-26 11:40 UTC (permalink / raw)
To: Umesh Nerlige Ramappa; +Cc: igt-dev
On 26/11/2020 00:35, Umesh Nerlige Ramappa wrote:
> On Wed, Nov 25, 2020 at 01:25:40PM +0200, Lionel Landwerlin wrote:
>> We're about to remove the filtering in i915 on 0 reason fields because
>> we assume this was a possibility, but it turned out to be a corruption
>> in the tail pointer register that made us read cleared data.
>>
>> This test is here to verify that our assumption hold that the HW never
>> produces such reports.
>>
>> v2: Fix len checking (Umesh)
>> Count report lost events (Umesh)
>> Check report sanity (Umesh)
>> Limit test to 3 times the OA buffer (Umesh)
>>
>> Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
>> ---
>> tests/i915/perf.c | 90 +++++++++++++++++++++++++++++++++++++++++++++++
>> 1 file changed, 90 insertions(+)
>>
>> diff --git a/tests/i915/perf.c b/tests/i915/perf.c
>> index 811067aad..2d442df96 100644
>> --- a/tests/i915/perf.c
>> +++ b/tests/i915/perf.c
>> @@ -2550,6 +2550,89 @@ test_buffer_fill(void)
>> __perf_close(stream_fd);
>> }
>>
>> +static void
>> +test_non_zero_reason(void)
>> +{
>> + /* ~10 micro second period */
>> + int oa_exponent = max_oa_exponent_for_period_lte(10000);
>> + uint64_t properties[] = {
>> + /* Include OA reports in samples */
>> + DRM_I915_PERF_PROP_SAMPLE_OA, true,
>> +
>> + /* OA unit configuration */
>> + DRM_I915_PERF_PROP_OA_METRICS_SET,
>> test_set->perf_oa_metrics_set,
>> + DRM_I915_PERF_PROP_OA_FORMAT, test_set->perf_oa_format,
>> + DRM_I915_PERF_PROP_OA_EXPONENT, oa_exponent,
>> + };
>> + struct drm_i915_perf_open_param param = {
>> + .flags = I915_PERF_FLAG_FD_CLOEXEC,
>> + .num_properties = sizeof(properties) / 16,
>> + .properties_ptr = to_user_pointer(properties),
>> + };
>> + struct drm_i915_perf_record_header *header;
>> + uint32_t buf_size = 3 * 65536 * (256 + sizeof(struct
>> drm_i915_perf_record_header));
>> + uint8_t *buf = malloc(buf_size);
>> + uint32_t total_len = 0, reports_lost;
>> + const uint32_t *last_report;
>> + int len;
>> +
>> + igt_assert(buf);
>
> nit: Looks like sanity check will log the report0/report1 pointers. We
> could log the buf pointer here so we get a sense of the report index
> that failed.
Thanks, I did not meant to leave the pointer to the report in the
previous change, I was just checking those same timestamp reports ;)
I just pushed the 2 previous commits, waiting for the Gen11 fix to go
before I land this patch.
>
> Either ways, this is
>
> Reviewed-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
>
> Thanks for adding this test,
> Umesh
>> +
>> + igt_debug("Ready to read about %u bytes\n", buf_size);
>> +
>> + load_helper_init();
>> + load_helper_run(HIGH);
>> +
>> + stream_fd = __perf_open(drm_fd, ¶m, true /* prevent_pm */);
>> +
>> + while (total_len < (buf_size - sizeof(struct
>> drm_i915_perf_record_header)) &&
>> + ((len = read(stream_fd, &buf[total_len], buf_size -
>> total_len)) > 0 ||
>> + (len == -1 && errno == EINTR))) {
>> + if (len > 0)
>> + total_len += len;
>> + }
>> +
>> + __perf_close(stream_fd);
>> +
>> + load_helper_stop();
>> + load_helper_fini();
>> +
>> + igt_debug("Got %u bytes\n", total_len);
>> +
>> + last_report = NULL;
>> + reports_lost = 0;
>> + for (uint32_t offset = 0; offset < total_len; offset +=
>> header->size) {
>> + header = (void *) (buf + offset);
>> +
>> + switch (header->type) {
>> + case DRM_I915_PERF_RECORD_OA_REPORT_LOST:
>> + reports_lost++;
>> + break;
>> + case DRM_I915_PERF_RECORD_SAMPLE: {
>> + const uint32_t *report = (void *) (header + 1);
>> + uint32_t reason = (report[0] >> OAREPORT_REASON_SHIFT) &
>> + OAREPORT_REASON_MASK;
>> +
>> + igt_assert_neq(reason, 0);
>> +
>> + if (last_report) {
>> + sanity_check_reports(last_report, report,
>> + test_set->perf_oa_format);
>> + }
>> + last_report = report;
>> + break;
>> + }
>> + case DRM_I915_PERF_RECORD_OA_BUFFER_LOST:
>> + igt_assert(!"unexpected overflow");
>> + break;
>> + }
>> + }
>> +
>> + igt_debug("Got %u report lost events\n", reports_lost);
>> +
>> + free(buf);
>> +}
>> +
>> static void
>> test_enable_disable(void)
>> {
>> @@ -4893,6 +4976,13 @@ igt_main
>> igt_subtest("buffer-fill")
>> test_buffer_fill();
>>
>> + igt_describe("Test that reason field in OA reports is never 0 on
>> Gen8+");
>> + igt_subtest("non-zero-reason") {
>> + /* Reason field is only available on Gen8+ */
>> + igt_require(intel_gen(devid) >= 8);
>> + test_non_zero_reason();
>> + }
>> +
>> igt_subtest("disabled-read-error")
>> test_disabled_read_error();
>> igt_subtest("non-sampling-read-error")
>> --
>> 2.29.2
>>
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2020-11-26 11:41 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-25 11:25 [igt-dev] [PATCH i-g-t v2 0/3] tests/i915/perf: verify non zero reason field in OA reports Lionel Landwerlin
2020-11-25 11:25 ` [igt-dev] [PATCH i-g-t v2 1/3] tests/i915/perf: constify report checks functions Lionel Landwerlin
2020-11-25 21:43 ` Umesh Nerlige Ramappa
2020-11-25 11:25 ` [igt-dev] [PATCH i-g-t v2 2/3] tests/i915/perf: deal with 0 delta reports Lionel Landwerlin
2020-11-25 22:12 ` Umesh Nerlige Ramappa
2020-11-25 11:25 ` [igt-dev] [PATCH i-g-t v2 3/3] tests/i915/perf: verify reason field in OA reports is never 0 Lionel Landwerlin
2020-11-25 22:35 ` Umesh Nerlige Ramappa
2020-11-26 11:40 ` Lionel Landwerlin
2020-11-25 12:25 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/i915/perf: verify non zero reason field in OA reports Patchwork
2020-11-25 13:57 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
2020-11-25 14:08 ` Lionel Landwerlin
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.