* [Intel-gfx] [PATCH v4 0/2] Avoid reading OA reports before they land
@ 2023-06-05 19:39 Umesh Nerlige Ramappa
2023-06-05 19:39 ` [Intel-gfx] [PATCH v4 1/2] i915/perf: Drop the aging_tail logic in perf OA Umesh Nerlige Ramappa
` (6 more replies)
0 siblings, 7 replies; 10+ messages in thread
From: Umesh Nerlige Ramappa @ 2023-06-05 19:39 UTC (permalink / raw)
To: intel-gfx
Fix OA issue seen on DG2 where parts of OA reports are zeroed out or
have stale values. This was due to the fact that rewind logic was not
being run when the tail pointer was aged. The series drops the complex
aging/aged logic and just checks the reports for validity.
rev1 - https://patchwork.freedesktop.org/series/118054/
v2: Drop aging logic completely
v3: Remove unnecessary renames and squash patches
v4: Indentaion fixes
Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com>
Umesh Nerlige Ramappa (2):
i915/perf: Drop the aging_tail logic in perf OA
i915/perf: Do not add ggtt offset to hw_tail
drivers/gpu/drm/i915/i915_perf.c | 92 ++++++++++----------------
drivers/gpu/drm/i915/i915_perf_types.h | 12 ----
2 files changed, 36 insertions(+), 68 deletions(-)
--
2.36.1
^ permalink raw reply [flat|nested] 10+ messages in thread* [Intel-gfx] [PATCH v4 1/2] i915/perf: Drop the aging_tail logic in perf OA 2023-06-05 19:39 [Intel-gfx] [PATCH v4 0/2] Avoid reading OA reports before they land Umesh Nerlige Ramappa @ 2023-06-05 19:39 ` Umesh Nerlige Ramappa 2023-06-05 19:39 ` [Intel-gfx] [PATCH v4 2/2] i915/perf: Do not add ggtt offset to hw_tail Umesh Nerlige Ramappa ` (5 subsequent siblings) 6 siblings, 0 replies; 10+ messages in thread From: Umesh Nerlige Ramappa @ 2023-06-05 19:39 UTC (permalink / raw) To: intel-gfx On DG2, capturing OA reports while running heavy render workloads sometimes results in invalid OA reports where 64-byte chunks inside reports have stale values. Under memory pressure, high OA sampling rates (13.3 us) and heavy render workload, occasionally, the OA HW TAIL pointer does not progress as fast as the sampling rate. When these glitches occur, the TAIL pointer takes approx. 200us to progress. While this is expected behavior from the HW perspective, invalid reports are not expected. In oa_buffer_check_unlocked(), when we execute the if condition, we are updating the oa_buffer.tail to the aging tail and then setting pollin based on this tail value, however, we do not have a chance to rewind and validate the reports prior to setting pollin. The validation happens in a subsequent call to oa_buffer_check_unlocked(). If a read occurs before this validation, then we end up reading reports up until this oa_buffer.tail value which includes invalid reports. Though found on DG2, this affects all platforms. The aging tail logic is no longer necessary since we are explicitly checking for landed reports. Start by dropping the aging tail logic. v2: - Drop extra blank line - Add reason to drop aging logic (Ashutosh) - Add bug links (Ashutosh) - rename aged_tail to read_tail - Squash patches 3 and 1 v3: (Ashutosh) - Remove extra spaces - Remove gtt_offset from the pollin calculation - s/Bug:/Link/ in commit message (checkpatch) Link: https://gitlab.freedesktop.org/drm/intel/-/issues/7484 Link: https://gitlab.freedesktop.org/drm/intel/-/issues/7757 Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com> Reviewed-by: Ashutosh Dixit <ashutosh.dixit@intel.com> --- drivers/gpu/drm/i915/i915_perf.c | 95 +++++++++++--------------- drivers/gpu/drm/i915/i915_perf_types.h | 12 ---- 2 files changed, 38 insertions(+), 69 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_perf.c b/drivers/gpu/drm/i915/i915_perf.c index 58284156428d..a8d43bf1f6d5 100644 --- a/drivers/gpu/drm/i915/i915_perf.c +++ b/drivers/gpu/drm/i915/i915_perf.c @@ -531,8 +531,7 @@ static void oa_context_id_squash(struct i915_perf_stream *stream, u32 *report) * (See description of OA_TAIL_MARGIN_NSEC above for further details.) * * Besides returning true when there is data available to read() this function - * also updates the tail, aging_tail and aging_timestamp in the oa_buffer - * object. + * also updates the tail in the oa_buffer object. * * Note: It's safe to read OA config state here unlocked, assuming that this is * only called while the stream is enabled, while the global OA configuration @@ -544,10 +543,10 @@ static bool oa_buffer_check_unlocked(struct i915_perf_stream *stream) { u32 gtt_offset = i915_ggtt_offset(stream->oa_buffer.vma); int report_size = stream->oa_buffer.format->size; + u32 head, tail, read_tail; unsigned long flags; bool pollin; u32 hw_tail; - u64 now; u32 partial_report_size; /* We have to consider the (unlikely) possibility that read() errors @@ -568,62 +567,47 @@ static bool oa_buffer_check_unlocked(struct i915_perf_stream *stream) /* Subtract partial amount off the tail */ hw_tail = gtt_offset + OA_TAKEN(hw_tail, partial_report_size); - now = ktime_get_mono_fast_ns(); - - if (hw_tail == stream->oa_buffer.aging_tail && - (now - stream->oa_buffer.aging_timestamp) > OA_TAIL_MARGIN_NSEC) { - /* If the HW tail hasn't move since the last check and the HW - * tail has been aging for long enough, declare it the new - * tail. - */ - stream->oa_buffer.tail = stream->oa_buffer.aging_tail; - } else { - u32 head, tail, aged_tail; - - /* NB: The head we observe here might effectively be a little - * out of date. If a read() is in progress, the head could be - * anywhere between this head and stream->oa_buffer.tail. - */ - head = stream->oa_buffer.head - gtt_offset; - aged_tail = stream->oa_buffer.tail - gtt_offset; - - hw_tail -= gtt_offset; - tail = hw_tail; - - /* Walk the stream backward until we find a report with report - * id and timestmap not at 0. Since the circular buffer pointers - * progress by increments of 64 bytes and that reports can be up - * to 256 bytes long, we can't tell whether a report has fully - * landed in memory before the report id and timestamp of the - * following report have effectively landed. - * - * This is assuming that the writes of the OA unit land in - * memory in the order they were written to. - * If not : (╯°□°）╯︵ ┻━┻ - */ - while (OA_TAKEN(tail, aged_tail) >= report_size) { - void *report = stream->oa_buffer.vaddr + tail; + /* NB: The head we observe here might effectively be a little + * out of date. If a read() is in progress, the head could be + * anywhere between this head and stream->oa_buffer.tail. + */ + head = stream->oa_buffer.head - gtt_offset; + read_tail = stream->oa_buffer.tail - gtt_offset; + + hw_tail -= gtt_offset; + tail = hw_tail; + + /* Walk the stream backward until we find a report with report + * id and timestmap not at 0. Since the circular buffer pointers + * progress by increments of 64 bytes and that reports can be up + * to 256 bytes long, we can't tell whether a report has fully + * landed in memory before the report id and timestamp of the + * following report have effectively landed. + * + * This is assuming that the writes of the OA unit land in + * memory in the order they were written to. + * If not : (╯°□°）╯︵ ┻━┻ + */ + while (OA_TAKEN(tail, read_tail) >= report_size) { + void *report = stream->oa_buffer.vaddr + tail; - if (oa_report_id(stream, report) || - oa_timestamp(stream, report)) - break; + if (oa_report_id(stream, report) || + oa_timestamp(stream, report)) + break; - tail = (tail - report_size) & (OA_BUFFER_SIZE - 1); - } + tail = (tail - report_size) & (OA_BUFFER_SIZE - 1); + } - if (OA_TAKEN(hw_tail, tail) > report_size && - __ratelimit(&stream->perf->tail_pointer_race)) - drm_notice(&stream->uncore->i915->drm, - "unlanded report(s) head=0x%x tail=0x%x hw_tail=0x%x\n", - head, tail, hw_tail); + if (OA_TAKEN(hw_tail, tail) > report_size && + __ratelimit(&stream->perf->tail_pointer_race)) + drm_notice(&stream->uncore->i915->drm, + "unlanded report(s) head=0x%x tail=0x%x hw_tail=0x%x\n", + head, tail, hw_tail); - stream->oa_buffer.tail = gtt_offset + tail; - stream->oa_buffer.aging_tail = gtt_offset + hw_tail; - stream->oa_buffer.aging_timestamp = now; - } + stream->oa_buffer.tail = gtt_offset + tail; - pollin = OA_TAKEN(stream->oa_buffer.tail - gtt_offset, - stream->oa_buffer.head - gtt_offset) >= report_size; + pollin = OA_TAKEN(stream->oa_buffer.tail, + stream->oa_buffer.head) >= report_size; spin_unlock_irqrestore(&stream->oa_buffer.ptr_lock, flags); @@ -1727,7 +1711,6 @@ static void gen7_init_oa_buffer(struct i915_perf_stream *stream) gtt_offset | OABUFFER_SIZE_16M); /* Mark that we need updated tail pointers to read from... */ - stream->oa_buffer.aging_tail = INVALID_TAIL_PTR; stream->oa_buffer.tail = gtt_offset; spin_unlock_irqrestore(&stream->oa_buffer.ptr_lock, flags); @@ -1779,7 +1762,6 @@ static void gen8_init_oa_buffer(struct i915_perf_stream *stream) intel_uncore_write(uncore, GEN8_OATAILPTR, gtt_offset & GEN8_OATAILPTR_MASK); /* Mark that we need updated tail pointers to read from... */ - stream->oa_buffer.aging_tail = INVALID_TAIL_PTR; stream->oa_buffer.tail = gtt_offset; /* @@ -1833,7 +1815,6 @@ static void gen12_init_oa_buffer(struct i915_perf_stream *stream) gtt_offset & GEN12_OAG_OATAILPTR_MASK); /* Mark that we need updated tail pointers to read from... */ - stream->oa_buffer.aging_tail = INVALID_TAIL_PTR; stream->oa_buffer.tail = gtt_offset; /* diff --git a/drivers/gpu/drm/i915/i915_perf_types.h b/drivers/gpu/drm/i915/i915_perf_types.h index 66dd5f74de05..fe3a5dae8c22 100644 --- a/drivers/gpu/drm/i915/i915_perf_types.h +++ b/drivers/gpu/drm/i915/i915_perf_types.h @@ -312,18 +312,6 @@ struct i915_perf_stream { */ spinlock_t ptr_lock; - /** - * @aging_tail: The last HW tail reported by HW. The data - * might not have made it to memory yet though. - */ - u32 aging_tail; - - /** - * @aging_timestamp: A monotonic timestamp for when the current aging tail pointer - * was read; used to determine when it is old enough to trust. - */ - u64 aging_timestamp; - /** * @head: Although we can always read back the head pointer register, * we prefer to avoid trusting the HW state, just to avoid any -- 2.36.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [Intel-gfx] [PATCH v4 2/2] i915/perf: Do not add ggtt offset to hw_tail 2023-06-05 19:39 [Intel-gfx] [PATCH v4 0/2] Avoid reading OA reports before they land Umesh Nerlige Ramappa 2023-06-05 19:39 ` [Intel-gfx] [PATCH v4 1/2] i915/perf: Drop the aging_tail logic in perf OA Umesh Nerlige Ramappa @ 2023-06-05 19:39 ` Umesh Nerlige Ramappa 2023-06-05 23:25 ` [Intel-gfx] ✗ Fi.CI.SPARSE: warning for Avoid reading OA reports before they land Patchwork ` (4 subsequent siblings) 6 siblings, 0 replies; 10+ messages in thread From: Umesh Nerlige Ramappa @ 2023-06-05 19:39 UTC (permalink / raw) To: intel-gfx ggtt offset for hw_tail is not required for the calculations, so drop it. Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com> Reviewed-by: Ashutosh Dixit <ashutosh.dixit@intel.com> --- drivers/gpu/drm/i915/i915_perf.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_perf.c b/drivers/gpu/drm/i915/i915_perf.c index a8d43bf1f6d5..0a111b281578 100644 --- a/drivers/gpu/drm/i915/i915_perf.c +++ b/drivers/gpu/drm/i915/i915_perf.c @@ -565,7 +565,7 @@ static bool oa_buffer_check_unlocked(struct i915_perf_stream *stream) partial_report_size %= report_size; /* Subtract partial amount off the tail */ - hw_tail = gtt_offset + OA_TAKEN(hw_tail, partial_report_size); + hw_tail = OA_TAKEN(hw_tail, partial_report_size); /* NB: The head we observe here might effectively be a little * out of date. If a read() is in progress, the head could be @@ -574,7 +574,6 @@ static bool oa_buffer_check_unlocked(struct i915_perf_stream *stream) head = stream->oa_buffer.head - gtt_offset; read_tail = stream->oa_buffer.tail - gtt_offset; - hw_tail -= gtt_offset; tail = hw_tail; /* Walk the stream backward until we find a report with report -- 2.36.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [Intel-gfx] ✗ Fi.CI.SPARSE: warning for Avoid reading OA reports before they land 2023-06-05 19:39 [Intel-gfx] [PATCH v4 0/2] Avoid reading OA reports before they land Umesh Nerlige Ramappa 2023-06-05 19:39 ` [Intel-gfx] [PATCH v4 1/2] i915/perf: Drop the aging_tail logic in perf OA Umesh Nerlige Ramappa 2023-06-05 19:39 ` [Intel-gfx] [PATCH v4 2/2] i915/perf: Do not add ggtt offset to hw_tail Umesh Nerlige Ramappa @ 2023-06-05 23:25 ` Patchwork 2023-06-05 23:44 ` [Intel-gfx] ✗ Fi.CI.BAT: failure " Patchwork ` (3 subsequent siblings) 6 siblings, 0 replies; 10+ messages in thread From: Patchwork @ 2023-06-05 23:25 UTC (permalink / raw) To: Umesh Nerlige Ramappa; +Cc: intel-gfx == Series Details == Series: Avoid reading OA reports before they land URL : https://patchwork.freedesktop.org/series/118886/ State : warning == Summary == Error: dim sparse failed Sparse version: v0.6.2 Fast mode used, each commit won't be checked separately. ^ permalink raw reply [flat|nested] 10+ messages in thread
* [Intel-gfx] ✗ Fi.CI.BAT: failure for Avoid reading OA reports before they land 2023-06-05 19:39 [Intel-gfx] [PATCH v4 0/2] Avoid reading OA reports before they land Umesh Nerlige Ramappa ` (2 preceding siblings ...) 2023-06-05 23:25 ` [Intel-gfx] ✗ Fi.CI.SPARSE: warning for Avoid reading OA reports before they land Patchwork @ 2023-06-05 23:44 ` Patchwork 2023-06-07 19:25 ` Umesh Nerlige Ramappa 2023-06-07 1:30 ` [Intel-gfx] ✗ Fi.CI.SPARSE: warning for Avoid reading OA reports before they land (rev2) Patchwork ` (2 subsequent siblings) 6 siblings, 1 reply; 10+ messages in thread From: Patchwork @ 2023-06-05 23:44 UTC (permalink / raw) To: Umesh Nerlige Ramappa; +Cc: intel-gfx [-- Attachment #1: Type: text/plain, Size: 11056 bytes --] == Series Details == Series: Avoid reading OA reports before they land URL : https://patchwork.freedesktop.org/series/118886/ State : failure == Summary == CI Bug Log - changes from CI_DRM_13232 -> Patchwork_118886v1 ==================================================== Summary ------- **FAILURE** Serious unknown changes coming with Patchwork_118886v1 absolutely need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in Patchwork_118886v1, please notify your bug team to allow them to document this new failure mode, which will reduce false positives in CI. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/index.html Participating hosts (37 -> 37) ------------------------------ Additional (1): bat-rpls-2 Missing (1): fi-snb-2520m Possible new issues ------------------- Here are the unknown changes that may have been introduced in Patchwork_118886v1: ### IGT changes ### #### Possible regressions #### * igt@i915_selftest@live@gt_timelines: - fi-apl-guc: [PASS][1] -> [DMESG-WARN][2] +2 similar issues [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13232/fi-apl-guc/igt@i915_selftest@live@gt_timelines.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/fi-apl-guc/igt@i915_selftest@live@gt_timelines.html #### Warnings #### * igt@kms_psr@sprite_plane_onoff: - bat-rplp-1: [SKIP][3] ([i915#1072]) -> [ABORT][4] [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13232/bat-rplp-1/igt@kms_psr@sprite_plane_onoff.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/bat-rplp-1/igt@kms_psr@sprite_plane_onoff.html Known issues ------------ Here are the changes found in Patchwork_118886v1 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@debugfs_test@basic-hwmon: - bat-rpls-2: NOTRUN -> [SKIP][5] ([i915#7456]) [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/bat-rpls-2/igt@debugfs_test@basic-hwmon.html * igt@fbdev@read: - bat-rpls-2: NOTRUN -> [SKIP][6] ([i915#2582]) +4 similar issues [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/bat-rpls-2/igt@fbdev@read.html * igt@gem_lmem_swapping@verify-random: - bat-rpls-2: NOTRUN -> [SKIP][7] ([i915#4613]) +3 similar issues [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/bat-rpls-2/igt@gem_lmem_swapping@verify-random.html * igt@gem_tiled_pread_basic: - bat-rpls-2: NOTRUN -> [SKIP][8] ([i915#3282]) [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/bat-rpls-2/igt@gem_tiled_pread_basic.html * igt@i915_pm_backlight@basic-brightness: - bat-rpls-2: NOTRUN -> [SKIP][9] ([i915#7561]) [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/bat-rpls-2/igt@i915_pm_backlight@basic-brightness.html * igt@i915_pm_rpm@module-reload: - fi-apl-guc: [PASS][10] -> [DMESG-WARN][11] ([i915#180] / [i915#1982] / [i915#7634]) [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13232/fi-apl-guc/igt@i915_pm_rpm@module-reload.html [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/fi-apl-guc/igt@i915_pm_rpm@module-reload.html * igt@i915_pm_rps@basic-api: - bat-rpls-2: NOTRUN -> [SKIP][12] ([i915#6621]) [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/bat-rpls-2/igt@i915_pm_rps@basic-api.html * igt@i915_selftest@live@execlists: - fi-bsw-nick: [PASS][13] -> [ABORT][14] ([i915#7911] / [i915#7913]) [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13232/fi-bsw-nick/igt@i915_selftest@live@execlists.html [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/fi-bsw-nick/igt@i915_selftest@live@execlists.html * igt@i915_selftest@live@gt_pm: - bat-rpls-2: NOTRUN -> [DMESG-FAIL][15] ([i915#4258] / [i915#7913]) [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/bat-rpls-2/igt@i915_selftest@live@gt_pm.html * igt@i915_selftest@live@requests: - bat-rpls-2: NOTRUN -> [ABORT][16] ([i915#4983] / [i915#7913]) [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/bat-rpls-2/igt@i915_selftest@live@requests.html * igt@i915_selftest@live@reset: - fi-apl-guc: [PASS][17] -> [DMESG-WARN][18] ([i915#7634]) +33 similar issues [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13232/fi-apl-guc/igt@i915_selftest@live@reset.html [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/fi-apl-guc/igt@i915_selftest@live@reset.html * igt@i915_selftest@live@slpc: - bat-rpls-1: NOTRUN -> [DMESG-WARN][19] ([i915#6367]) [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/bat-rpls-1/igt@i915_selftest@live@slpc.html * igt@i915_suspend@basic-s2idle-without-i915: - fi-apl-guc: [PASS][20] -> [DMESG-WARN][21] ([i915#180] / [i915#7634]) [20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13232/fi-apl-guc/igt@i915_suspend@basic-s2idle-without-i915.html [21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/fi-apl-guc/igt@i915_suspend@basic-s2idle-without-i915.html * igt@i915_suspend@basic-s3-without-i915: - bat-rpls-1: NOTRUN -> [ABORT][22] ([i915#6687] / [i915#7978]) [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/bat-rpls-1/igt@i915_suspend@basic-s3-without-i915.html * igt@kms_busy@basic: - bat-rpls-2: NOTRUN -> [SKIP][23] ([i915#1845]) +14 similar issues [23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/bat-rpls-2/igt@kms_busy@basic.html * igt@kms_chamelium_edid@hdmi-edid-read: - bat-rpls-2: NOTRUN -> [SKIP][24] ([i915#7828]) +7 similar issues [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/bat-rpls-2/igt@kms_chamelium_edid@hdmi-edid-read.html * igt@kms_flip@basic-flip-vs-dpms: - bat-rpls-2: NOTRUN -> [SKIP][25] ([i915#3637]) +3 similar issues [25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/bat-rpls-2/igt@kms_flip@basic-flip-vs-dpms.html * igt@kms_force_connector_basic@force-load-detect: - bat-rpls-2: NOTRUN -> [SKIP][26] ([fdo#109285]) [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/bat-rpls-2/igt@kms_force_connector_basic@force-load-detect.html * igt@kms_frontbuffer_tracking@basic: - bat-rpls-2: NOTRUN -> [SKIP][27] ([i915#1849]) [27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/bat-rpls-2/igt@kms_frontbuffer_tracking@basic.html * igt@kms_psr@sprite_plane_onoff: - bat-rpls-2: NOTRUN -> [SKIP][28] ([i915#1072]) +3 similar issues [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/bat-rpls-2/igt@kms_psr@sprite_plane_onoff.html * igt@kms_setmode@basic-clone-single-crtc: - bat-rpls-2: NOTRUN -> [SKIP][29] ([i915#3555] / [i915#4579]) [29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/bat-rpls-2/igt@kms_setmode@basic-clone-single-crtc.html * igt@prime_vgem@basic-fence-flip: - bat-rpls-2: NOTRUN -> [SKIP][30] ([fdo#109295] / [i915#1845] / [i915#3708]) [30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/bat-rpls-2/igt@prime_vgem@basic-fence-flip.html * igt@prime_vgem@basic-fence-read: - bat-rpls-2: NOTRUN -> [SKIP][31] ([fdo#109295] / [i915#3708]) +2 similar issues [31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/bat-rpls-2/igt@prime_vgem@basic-fence-read.html #### Possible fixes #### * igt@i915_selftest@live@reset: - bat-rpls-1: [ABORT][32] ([i915#4983] / [i915#7461] / [i915#7981] / [i915#8347] / [i915#8384]) -> [PASS][33] [32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13232/bat-rpls-1/igt@i915_selftest@live@reset.html [33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/bat-rpls-1/igt@i915_selftest@live@reset.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285 [fdo#109295]: https://bugs.freedesktop.org/show_bug.cgi?id=109295 [i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072 [i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180 [i915#1845]: https://gitlab.freedesktop.org/drm/intel/issues/1845 [i915#1849]: https://gitlab.freedesktop.org/drm/intel/issues/1849 [i915#1982]: https://gitlab.freedesktop.org/drm/intel/issues/1982 [i915#2582]: https://gitlab.freedesktop.org/drm/intel/issues/2582 [i915#3282]: https://gitlab.freedesktop.org/drm/intel/issues/3282 [i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555 [i915#3637]: https://gitlab.freedesktop.org/drm/intel/issues/3637 [i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708 [i915#4258]: https://gitlab.freedesktop.org/drm/intel/issues/4258 [i915#4579]: https://gitlab.freedesktop.org/drm/intel/issues/4579 [i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613 [i915#4983]: https://gitlab.freedesktop.org/drm/intel/issues/4983 [i915#6367]: https://gitlab.freedesktop.org/drm/intel/issues/6367 [i915#6621]: https://gitlab.freedesktop.org/drm/intel/issues/6621 [i915#6687]: https://gitlab.freedesktop.org/drm/intel/issues/6687 [i915#7456]: https://gitlab.freedesktop.org/drm/intel/issues/7456 [i915#7461]: https://gitlab.freedesktop.org/drm/intel/issues/7461 [i915#7561]: https://gitlab.freedesktop.org/drm/intel/issues/7561 [i915#7634]: https://gitlab.freedesktop.org/drm/intel/issues/7634 [i915#7828]: https://gitlab.freedesktop.org/drm/intel/issues/7828 [i915#7911]: https://gitlab.freedesktop.org/drm/intel/issues/7911 [i915#7913]: https://gitlab.freedesktop.org/drm/intel/issues/7913 [i915#7978]: https://gitlab.freedesktop.org/drm/intel/issues/7978 [i915#7981]: https://gitlab.freedesktop.org/drm/intel/issues/7981 [i915#8011]: https://gitlab.freedesktop.org/drm/intel/issues/8011 [i915#8347]: https://gitlab.freedesktop.org/drm/intel/issues/8347 [i915#8384]: https://gitlab.freedesktop.org/drm/intel/issues/8384 [i915#8497]: https://gitlab.freedesktop.org/drm/intel/issues/8497 Build changes ------------- * Linux: CI_DRM_13232 -> Patchwork_118886v1 CI-20190529: 20190529 CI_DRM_13232: 450d228e38403a48aa273ec1e22b463dc64aaae6 @ git://anongit.freedesktop.org/gfx-ci/linux IGT_7319: 2e1bcd49944452b5f9516eecee48e1fa3ae6a636 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git Patchwork_118886v1: 450d228e38403a48aa273ec1e22b463dc64aaae6 @ git://anongit.freedesktop.org/gfx-ci/linux ### Linux commits 692b345cb121 i915/perf: Do not add ggtt offset to hw_tail e7311e914948 i915/perf: Drop the aging_tail logic in perf OA == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/index.html [-- Attachment #2: Type: text/html, Size: 12842 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for Avoid reading OA reports before they land 2023-06-05 23:44 ` [Intel-gfx] ✗ Fi.CI.BAT: failure " Patchwork @ 2023-06-07 19:25 ` Umesh Nerlige Ramappa 0 siblings, 0 replies; 10+ messages in thread From: Umesh Nerlige Ramappa @ 2023-06-07 19:25 UTC (permalink / raw) To: intel-gfx On Mon, Jun 05, 2023 at 11:44:21PM +0000, Patchwork wrote: > Patch Details > >Series: Avoid reading OA reports before they land >URL: [1]https://patchwork.freedesktop.org/series/118886/ >State: failure >Details: [2]https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/index.html > > CI Bug Log - changes from CI_DRM_13232 -> Patchwork_118886v1 > >Summary > > FAILURE > > Serious unknown changes coming with Patchwork_118886v1 absolutely need to > be > verified manually. > > If you think the reported changes have nothing to do with the changes > introduced in Patchwork_118886v1, please notify your bug team to allow > them > to document this new failure mode, which will reduce false positives in > CI. > > External URL: > https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v1/index.html > >Participating hosts (37 -> 37) > > Additional (1): bat-rpls-2 > Missing (1): fi-snb-2520m > >Possible new issues > > Here are the unknown changes that may have been introduced in > Patchwork_118886v1: > > IGT changes > > Possible regressions > > * igt@i915_selftest@live@gt_timelines: > > * fi-apl-guc: [3]PASS -> [4]DMESG-WARN +2 similar issues <3> [309.685038] i915 0000:00:02.0: [drm] *ERROR* Failed to probe lspcon This warning is not related to OA or any use case from this patch. > > Warnings > > * igt@kms_psr@sprite_plane_onoff: > > * bat-rplp-1: [5]SKIP ([6]i915#1072) -> [7]ABORT + John These are not related to OA, but a known lockdep issue. <4>[ 229.036305] ====================================================== <4>[ 229.036320] WARNING: possible circular locking dependency detected <4>[ 229.036334] 6.4.0-rc5-Patchwork_118886v1-g450d228e3840+ #1 Not tainted <4>[ 229.036348] ------------------------------------------------------ <4>[ 229.036362] kworker/0:0H/8 is trying to acquire lock: <4>[ 229.036374] ffff888117b74f48 (>->reset.backoff_srcu){++++}-{0:0}, at: _intel_gt_reset_lock+0x0/0x330 [i915] <4>[ 229.036503] but task is already holding lock: <4>[ 229.036521] ffffc900000d3e60 ((work_completion)(&(&guc->timestamp.work)->work)){+.+.}-{0:0}, at: process_one_work+0x1cc/0x510 <4>[ 229.036548] which lock already depends on the new lock. <4>[ 229.036574] the existing dependency chain (in reverse order) is: <4>[ 229.036598] -> #3 ((work_completion)(&(&guc->timestamp.work)->work)){+.+.}-{0:0}: <4>[ 229.036624] lock_acquire+0xd8/0x2d0 <4>[ 229.036636] __flush_work+0x74/0x530 <4>[ 229.036646] __cancel_work_timer+0x14f/0x1f0 <4>[ 229.036658] intel_guc_submission_reset_prepare+0x81/0x4b0 [i915] <4>[ 229.036799] intel_uc_reset_prepare+0x9c/0x120 [i915] <4>[ 229.036938] reset_prepare+0x21/0x60 [i915] <4>[ 229.037054] intel_gt_reset+0x1dd/0x470 [i915] <4>[ 229.037172] intel_gt_reset_global+0xfb/0x170 [i915] <4>[ 229.037285] intel_gt_handle_error+0x368/0x420 [i915] <4>[ 229.037401] intel_gt_debugfs_reset_store+0x5c/0xc0 [i915] <4>[ 229.037509] i915_wedged_set+0x29/0x40 [i915] <4>[ 229.037600] simple_attr_write_xsigned.constprop.0+0xb4/0x110 <4>[ 229.037616] full_proxy_write+0x52/0x80 <4>[ 229.037627] vfs_write+0xc5/0x4f0 <4>[ 229.037637] ksys_write+0x64/0xe0 <4>[ 229.037646] do_syscall_64+0x3c/0x90 <4>[ 229.037658] entry_SYSCALL_64_after_hwframe+0x72/0xdc <4>[ 229.037672] -> #2 (>->reset.mutex){+.+.}-{3:3}: <4>[ 229.037694] lock_acquire+0xd8/0x2d0 <4>[ 229.037704] i915_gem_shrinker_taints_mutex+0x31/0x50 [i915] <4>[ 229.037835] intel_gt_init_reset+0x65/0x80 [i915] <4>[ 229.037948] intel_gt_common_init_early+0xe1/0x170 [i915] <4>[ 229.038055] intel_root_gt_init_early+0x48/0x60 [i915] <4>[ 229.038158] i915_driver_probe+0x243/0xcd0 [i915] <4>[ 229.038247] i915_pci_probe+0xdc/0x210 [i915] <4>[ 229.038335] pci_device_probe+0x95/0x120 <4>[ 229.038347] really_probe+0x164/0x3c0 <4>[ 229.038358] __driver_probe_device+0x73/0x160 <4>[ 229.038371] driver_probe_device+0x19/0xa0 <4>[ 229.038384] __driver_attach+0xb6/0x180 <4>[ 229.038395] bus_for_each_dev+0x77/0xd0 <4>[ 229.038405] bus_add_driver+0x114/0x210 <4>[ 229.038415] driver_register+0x5b/0x110 <4>[ 229.038425] 0xffffffffa00fd033 <4>[ 229.038439] do_one_initcall+0x57/0x270 <4>[ 229.038450] do_init_module+0x5f/0x220 <4>[ 229.038461] load_module+0x1ca4/0x1f00 <4>[ 229.038472] __do_sys_finit_module+0xb4/0x130 <4>[ 229.038484] do_syscall_64+0x3c/0x90 <4>[ 229.038495] entry_SYSCALL_64_after_hwframe+0x72/0xdc <4>[ 229.038508] -> #1 (fs_reclaim){+.+.}-{0:0}: <4>[ 229.038528] lock_acquire+0xd8/0x2d0 <4>[ 229.038538] fs_reclaim_acquire+0xac/0xe0 <4>[ 229.038550] __kmem_cache_alloc_node+0x30/0x1b0 <4>[ 229.038563] kmalloc_trace+0x24/0xb0 <4>[ 229.039296] kernfs_fop_open+0xc0/0x3d0 <4>[ 229.040028] do_dentry_open+0x14a/0x440 <4>[ 229.040754] path_openat+0x663/0x8a0 <4>[ 229.041480] do_filp_open+0xb1/0x120 <4>[ 229.042030] do_sys_openat2+0x250/0x330 <4>[ 229.042545] do_sys_open+0x43/0x80 <4>[ 229.043107] do_syscall_64+0x3c/0x90 <4>[ 229.043665] entry_SYSCALL_64_after_hwframe+0x72/0xdc <4>[ 229.044221] -> #0 (/-1493934552){...+}-{0:0}: <1>[ 229.045307] BUG: kernel NULL pointer dereference, address: 0000000000000014 <1>[ 229.045852] #PF: supervisor read access in kernel mode <1>[ 229.046390] #PF: error_code(0x0000) - not-present page <6>[ 229.046922] PGD 0 P4D 0 <4>[ 229.047460] Oops: 0000 [#1] PREEMPT SMP NOPTI <4>[ 229.048034] CPU: 0 PID: 8 Comm: kworker/0:0H Not tainted 6.4.0-rc5-Patchwork_118886v1-g450d228e3840+ #1 <4>[ 229.048629] Hardware name: Intel Corporation Raptor Lake Client Platform/RaptorLake-P LP5 RVP, BIOS RPLPFWI1.R00.3257.A00.2207020323 07/02/2022 <4>[ 229.049233] Workqueue: events_highpri guc_timestamp_ping [i915] <4>[ 229.049965] RIP: 0010:print_circular_bug_entry.isra.0+0x44/0x50 <4>[ 229.050571] Code: 53 48 89 f3 89 d6 e8 5b 74 01 00 48 8b 7d 00 e8 d2 f3 ff ff 48 c7 c7 65 21 3c 82 e8 46 74 01 00 48 8b 3b ba 06 00 00 00 5b 5d <8b> 77 14 48 83 c7 18 e9 50 d6 04 00 90 90 90 90 90 90 90 90 90 90 <4>[ 229.051206] RSP: 0018:ffffc900000d3b68 EFLAGS: 00010046 <4>[ 229.051853] RAX: 0000000000000001 RBX: ffff888100d9b3f0 RCX: 0000000000000000 <4>[ 229.052506] RDX: 0000000000000006 RSI: ffffffff823ccb57 RDI: 0000000000000000 <4>[ 229.053151] RBP: ffff888100d9b3c8 R08: 0000000000000000 R09: ffffc900000d3a10 <4>[ 229.053794] R10: 000000000024fd38 R11: 000000000024fda8 R12: 0000000000000000 <4>[ 229.054443] R13: ffffc9000256fd00 R14: ffff888100d9a9c0 R15: ffffffff83f8fd40 <4>[ 229.055094] FS: 0000000000000000(0000) GS:ffff8882a7000000(0000) knlGS:0000000000000000 <4>[ 229.055753] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 <4>[ 229.056409] CR2: 0000000000000014 CR3: 00000001095b2000 CR4: 0000000000f50ef0 <4>[ 229.057069] PKRU: 55555554 <4>[ 229.057727] Call Trace: <4>[ 229.058378] <TASK> <4>[ 229.059023] ? __die_body+0x1a/0x60 <4>[ 229.059671] ? page_fault_oops+0x156/0x450 <4>[ 229.060319] ? do_user_addr_fault+0x65/0xa10 <4>[ 229.060976] ? exc_page_fault+0x68/0x1a0 <4>[ 229.061629] ? asm_exc_page_fault+0x26/0x30 <4>[ 229.062281] ? print_circular_bug_entry.isra.0+0x44/0x50 <4>[ 229.062926] print_circular_bug.isra.0+0x111/0x3f0 <4>[ 229.063536] check_noncircular+0x131/0x150 <4>[ 229.064154] ? arch_stack_walk+0x87/0xf0 <4>[ 229.064759] check_prev_add+0x90/0xc60 <4>[ 229.065363] __lock_acquire+0x19a3/0x25a0 <4>[ 229.065966] ? startup_64_setup_env+0x184/0xaf0 <4>[ 229.066568] lock_acquire+0xd8/0x2d0 <4>[ 229.067173] ? __pfx__intel_gt_reset_lock+0x10/0x10 [i915] <4>[ 229.067881] _intel_gt_reset_lock+0x57/0x330 [i915] <4>[ 229.068586] ? __pfx__intel_gt_reset_lock+0x10/0x10 [i915] <4>[ 229.069288] guc_timestamp_ping+0x35/0x130 [i915] <4>[ 229.070018] process_one_work+0x250/0x510 <4>[ 229.070629] worker_thread+0x4f/0x3a0 <4>[ 229.071235] ? __pfx_worker_thread+0x10/0x10 <4>[ 229.071845] kthread+0xff/0x130 <4>[ 229.072454] ? __pfx_kthread+0x10/0x10 <4>[ 229.073064] ret_from_fork+0x29/0x50 <4>[ 229.073674] </TASK> <4>[ 229.074283] Modules linked in: vgem drm_shmem_helper snd_hda_codec_hdmi snd_hda_intel snd_intel_dspcfg snd_hda_codec snd_hwdep snd_hda_core snd_pcm i915 prime_numbers i2c_algo_bit ttm drm_buddy drm_display_helper drm_kms_helper fuse r8153_ecm cdc_ether usbnet x86_pkg_temp_thermal coretemp kvm_intel kvm e1000e mei_pxp mei_hdcp r8152 irqbypass crct10dif_pclmul crc32_pclmul wmi_bmof mii ghash_clmulni_intel mei_me ptp i2c_i801 mei pps_core i2c_smbus video intel_lpss_pci wmi <4>[ 229.075708] CR2: 0000000000000014 <4>[ 229.076421] ---[ end trace 0000000000000000 ]--- <4>[ 229.373071] RIP: 0010:print_circular_bug_entry.isra.0+0x44/0x50 <4>[ 229.373942] Code: 53 48 89 f3 89 d6 e8 5b 74 01 00 48 8b 7d 00 e8 d2 f3 ff ff 48 c7 c7 65 21 3c 82 e8 46 74 01 00 48 8b 3b ba 06 00 00 00 5b 5d <8b> 77 14 48 83 c7 18 e9 50 d6 04 00 90 90 90 90 90 90 90 90 90 90 <4>[ 229.374830] RSP: 0018:ffffc900000d3b68 EFLAGS: 00010046 <4>[ 229.375578] RAX: 0000000000000001 RBX: ffff888100d9b3f0 RCX: 0000000000000000 <4>[ 229.376235] RDX: 0000000000000006 RSI: ffffffff823ccb57 RDI: 0000000000000000 <4>[ 229.376927] RBP: ffff888100d9b3c8 R08: 0000000000000000 R09: ffffc900000d3a10 <4>[ 229.377649] R10: 000000000024fd38 R11: 000000000024fda8 R12: 0000000000000000 <4>[ 229.378373] R13: ffffc9000256fd00 R14: ffff888100d9a9c0 R15: ffffffff83f8fd40 <4>[ 229.379100] FS: 0000000000000000(0000) GS:ffff8882a7000000(0000) knlGS:0000000000000000 <4>[ 229.379838] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 <4>[ 229.380578] CR2: 0000000000000014 CR3: 00000001095b2000 CR4: 0000000000f50ef0 <4>[ 229.381331] PKRU: 55555554 > >Known issues > ^ permalink raw reply [flat|nested] 10+ messages in thread
* [Intel-gfx] ✗ Fi.CI.SPARSE: warning for Avoid reading OA reports before they land (rev2) 2023-06-05 19:39 [Intel-gfx] [PATCH v4 0/2] Avoid reading OA reports before they land Umesh Nerlige Ramappa ` (3 preceding siblings ...) 2023-06-05 23:44 ` [Intel-gfx] ✗ Fi.CI.BAT: failure " Patchwork @ 2023-06-07 1:30 ` Patchwork 2023-06-07 1:46 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork 2023-06-07 17:40 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork 6 siblings, 0 replies; 10+ messages in thread From: Patchwork @ 2023-06-07 1:30 UTC (permalink / raw) To: Umesh Nerlige Ramappa; +Cc: intel-gfx == Series Details == Series: Avoid reading OA reports before they land (rev2) URL : https://patchwork.freedesktop.org/series/118886/ State : warning == Summary == Error: dim sparse failed Sparse version: v0.6.2 Fast mode used, each commit won't be checked separately. ^ permalink raw reply [flat|nested] 10+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for Avoid reading OA reports before they land (rev2) 2023-06-05 19:39 [Intel-gfx] [PATCH v4 0/2] Avoid reading OA reports before they land Umesh Nerlige Ramappa ` (4 preceding siblings ...) 2023-06-07 1:30 ` [Intel-gfx] ✗ Fi.CI.SPARSE: warning for Avoid reading OA reports before they land (rev2) Patchwork @ 2023-06-07 1:46 ` Patchwork 2023-06-07 17:40 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork 6 siblings, 0 replies; 10+ messages in thread From: Patchwork @ 2023-06-07 1:46 UTC (permalink / raw) To: Umesh Nerlige Ramappa; +Cc: intel-gfx [-- Attachment #1: Type: text/plain, Size: 3944 bytes --] == Series Details == Series: Avoid reading OA reports before they land (rev2) URL : https://patchwork.freedesktop.org/series/118886/ State : success == Summary == CI Bug Log - changes from CI_DRM_13238 -> Patchwork_118886v2 ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/index.html Participating hosts (41 -> 39) ------------------------------ Missing (2): bat-rpls-2 fi-snb-2520m Known issues ------------ Here are the changes found in Patchwork_118886v2 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@i915_selftest@live@execlists: - fi-bsw-nick: [PASS][1] -> [ABORT][2] ([i915#7911] / [i915#7913]) [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13238/fi-bsw-nick/igt@i915_selftest@live@execlists.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/fi-bsw-nick/igt@i915_selftest@live@execlists.html * igt@kms_chamelium_hpd@common-hpd-after-suspend: - bat-adlm-1: NOTRUN -> [SKIP][3] ([i915#7828]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/bat-adlm-1/igt@kms_chamelium_hpd@common-hpd-after-suspend.html * igt@kms_pipe_crc_basic@suspend-read-crc: - bat-adlm-1: NOTRUN -> [SKIP][4] ([i915#1845]) [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/bat-adlm-1/igt@kms_pipe_crc_basic@suspend-read-crc.html #### Possible fixes #### * igt@i915_selftest@live@workarounds: - bat-adlm-1: [INCOMPLETE][5] ([i915#4983] / [i915#7677]) -> [PASS][6] [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13238/bat-adlm-1/igt@i915_selftest@live@workarounds.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/bat-adlm-1/igt@i915_selftest@live@workarounds.html #### Warnings #### * igt@kms_setmode@basic-clone-single-crtc: - bat-rplp-1: [SKIP][7] ([i915#3555] / [i915#4579]) -> [ABORT][8] ([i915#4579] / [i915#8260]) [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13238/bat-rplp-1/igt@kms_setmode@basic-clone-single-crtc.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/bat-rplp-1/igt@kms_setmode@basic-clone-single-crtc.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [i915#1845]: https://gitlab.freedesktop.org/drm/intel/issues/1845 [i915#3546]: https://gitlab.freedesktop.org/drm/intel/issues/3546 [i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555 [i915#4423]: https://gitlab.freedesktop.org/drm/intel/issues/4423 [i915#4579]: https://gitlab.freedesktop.org/drm/intel/issues/4579 [i915#4983]: https://gitlab.freedesktop.org/drm/intel/issues/4983 [i915#6868]: https://gitlab.freedesktop.org/drm/intel/issues/6868 [i915#7677]: https://gitlab.freedesktop.org/drm/intel/issues/7677 [i915#7828]: https://gitlab.freedesktop.org/drm/intel/issues/7828 [i915#7911]: https://gitlab.freedesktop.org/drm/intel/issues/7911 [i915#7913]: https://gitlab.freedesktop.org/drm/intel/issues/7913 [i915#8260]: https://gitlab.freedesktop.org/drm/intel/issues/8260 Build changes ------------- * Linux: CI_DRM_13238 -> Patchwork_118886v2 CI-20190529: 20190529 CI_DRM_13238: 8c0b302811d744b945dcb6d78164a76188914db9 @ git://anongit.freedesktop.org/gfx-ci/linux IGT_7320: 1c96b08a4cde6f2d49824a8cc3303bd860617b52 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git Patchwork_118886v2: 8c0b302811d744b945dcb6d78164a76188914db9 @ git://anongit.freedesktop.org/gfx-ci/linux ### Linux commits eb3f1c965fb0 i915/perf: Do not add ggtt offset to hw_tail a419e80343e6 i915/perf: Drop the aging_tail logic in perf OA == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/index.html [-- Attachment #2: Type: text/html, Size: 4521 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
* [Intel-gfx] ✗ Fi.CI.IGT: failure for Avoid reading OA reports before they land (rev2) 2023-06-05 19:39 [Intel-gfx] [PATCH v4 0/2] Avoid reading OA reports before they land Umesh Nerlige Ramappa ` (5 preceding siblings ...) 2023-06-07 1:46 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork @ 2023-06-07 17:40 ` Patchwork 2023-06-07 19:44 ` Umesh Nerlige Ramappa 6 siblings, 1 reply; 10+ messages in thread From: Patchwork @ 2023-06-07 17:40 UTC (permalink / raw) To: Umesh Nerlige Ramappa; +Cc: intel-gfx [-- Attachment #1: Type: text/plain, Size: 13588 bytes --] == Series Details == Series: Avoid reading OA reports before they land (rev2) URL : https://patchwork.freedesktop.org/series/118886/ State : failure == Summary == CI Bug Log - changes from CI_DRM_13238_full -> Patchwork_118886v2_full ==================================================== Summary ------- **FAILURE** Serious unknown changes coming with Patchwork_118886v2_full absolutely need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in Patchwork_118886v2_full, please notify your bug team to allow them to document this new failure mode, which will reduce false positives in CI. Participating hosts (7 -> 7) ------------------------------ No changes in participating hosts Possible new issues ------------------- Here are the unknown changes that may have been introduced in Patchwork_118886v2_full: ### IGT changes ### #### Possible regressions #### * igt@kms_vblank@pipe-b-accuracy-idle: - shard-glk: [PASS][1] -> [FAIL][2] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13238/shard-glk3/igt@kms_vblank@pipe-b-accuracy-idle.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/shard-glk5/igt@kms_vblank@pipe-b-accuracy-idle.html Known issues ------------ Here are the changes found in Patchwork_118886v2_full that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_ctx_persistence@engines-hang: - shard-snb: NOTRUN -> [SKIP][3] ([fdo#109271] / [i915#1099]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/shard-snb1/igt@gem_ctx_persistence@engines-hang.html * igt@gem_exec_fair@basic-none-share@rcs0: - shard-glk: [PASS][4] -> [FAIL][5] ([i915#2842]) [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13238/shard-glk5/igt@gem_exec_fair@basic-none-share@rcs0.html [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/shard-glk9/igt@gem_exec_fair@basic-none-share@rcs0.html * igt@gem_exec_fair@basic-pace-solo@rcs0: - shard-apl: [PASS][6] -> [FAIL][7] ([i915#2842]) [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13238/shard-apl7/igt@gem_exec_fair@basic-pace-solo@rcs0.html [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/shard-apl1/igt@gem_exec_fair@basic-pace-solo@rcs0.html * igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-vga: - shard-snb: NOTRUN -> [SKIP][8] ([fdo#109271] / [i915#4579]) +6 similar issues [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/shard-snb6/igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-vga.html * igt@kms_big_fb@y-tiled-64bpp-rotate-180: - shard-snb: NOTRUN -> [SKIP][9] ([fdo#109271]) +30 similar issues [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/shard-snb1/igt@kms_big_fb@y-tiled-64bpp-rotate-180.html * igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions: - shard-glk: [PASS][10] -> [FAIL][11] ([i915#2346]) +1 similar issue [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13238/shard-glk7/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions.html [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/shard-glk2/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions.html * igt@kms_vblank@pipe-b-ts-continuation-suspend: - shard-apl: [PASS][12] -> [ABORT][13] ([i915#180] / [i915#8213]) [12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13238/shard-apl7/igt@kms_vblank@pipe-b-ts-continuation-suspend.html [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/shard-apl1/igt@kms_vblank@pipe-b-ts-continuation-suspend.html #### Possible fixes #### * igt@gem_barrier_race@remote-request@rcs0: - {shard-dg1}: [ABORT][14] ([i915#7461] / [i915#8234]) -> [PASS][15] [14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13238/shard-dg1-18/igt@gem_barrier_race@remote-request@rcs0.html [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/shard-dg1-17/igt@gem_barrier_race@remote-request@rcs0.html * igt@gem_exec_fair@basic-deadline: - {shard-rkl}: [FAIL][16] ([i915#2846]) -> [PASS][17] [16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13238/shard-rkl-1/igt@gem_exec_fair@basic-deadline.html [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/shard-rkl-4/igt@gem_exec_fair@basic-deadline.html * igt@gem_exec_fair@basic-none-solo@rcs0: - shard-apl: [FAIL][18] ([i915#2842]) -> [PASS][19] [18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13238/shard-apl4/igt@gem_exec_fair@basic-none-solo@rcs0.html [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/shard-apl4/igt@gem_exec_fair@basic-none-solo@rcs0.html * igt@i915_pm_dc@dc9-dpms: - {shard-tglu}: [SKIP][20] ([i915#4281]) -> [PASS][21] [20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13238/shard-tglu-7/igt@i915_pm_dc@dc9-dpms.html [21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/shard-tglu-9/igt@i915_pm_dc@dc9-dpms.html * igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-hdmi-a: - {shard-rkl}: [SKIP][22] ([i915#1937] / [i915#4579]) -> [PASS][23] [22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13238/shard-rkl-4/igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-hdmi-a.html [23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/shard-rkl-7/igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-hdmi-a.html * igt@i915_pm_rpm@dpms-lpsp: - {shard-rkl}: [SKIP][24] ([i915#1397]) -> [PASS][25] +1 similar issue [24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13238/shard-rkl-3/igt@i915_pm_rpm@dpms-lpsp.html [25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/shard-rkl-7/igt@i915_pm_rpm@dpms-lpsp.html * igt@i915_pm_rpm@modeset-non-lpsp: - {shard-dg1}: [SKIP][26] ([i915#1397]) -> [PASS][27] +1 similar issue [26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13238/shard-dg1-19/igt@i915_pm_rpm@modeset-non-lpsp.html [27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/shard-dg1-14/igt@i915_pm_rpm@modeset-non-lpsp.html * igt@i915_pm_rps@reset: - shard-snb: [INCOMPLETE][28] ([i915#7790]) -> [PASS][29] [28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13238/shard-snb1/igt@i915_pm_rps@reset.html [29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/shard-snb1/igt@i915_pm_rps@reset.html * igt@kms_cursor_legacy@forked-move@pipe-b: - {shard-dg1}: [INCOMPLETE][30] ([i915#8011] / [i915#8347]) -> [PASS][31] [30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13238/shard-dg1-19/igt@kms_cursor_legacy@forked-move@pipe-b.html [31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/shard-dg1-18/igt@kms_cursor_legacy@forked-move@pipe-b.html * igt@kms_fbcon_fbt@fbc-suspend: - shard-apl: [FAIL][32] ([i915#4767]) -> [PASS][33] [32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13238/shard-apl6/igt@kms_fbcon_fbt@fbc-suspend.html [33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/shard-apl2/igt@kms_fbcon_fbt@fbc-suspend.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#109280]: https://bugs.freedesktop.org/show_bug.cgi?id=109280 [fdo#109289]: https://bugs.freedesktop.org/show_bug.cgi?id=109289 [fdo#110189]: https://bugs.freedesktop.org/show_bug.cgi?id=110189 [fdo#111615]: https://bugs.freedesktop.org/show_bug.cgi?id=111615 [fdo#111825]: https://bugs.freedesktop.org/show_bug.cgi?id=111825 [fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827 [i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072 [i915#1099]: https://gitlab.freedesktop.org/drm/intel/issues/1099 [i915#1397]: https://gitlab.freedesktop.org/drm/intel/issues/1397 [i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180 [i915#1937]: https://gitlab.freedesktop.org/drm/intel/issues/1937 [i915#2346]: https://gitlab.freedesktop.org/drm/intel/issues/2346 [i915#2433]: https://gitlab.freedesktop.org/drm/intel/issues/2433 [i915#2437]: https://gitlab.freedesktop.org/drm/intel/issues/2437 [i915#2527]: https://gitlab.freedesktop.org/drm/intel/issues/2527 [i915#2575]: https://gitlab.freedesktop.org/drm/intel/issues/2575 [i915#2587]: https://gitlab.freedesktop.org/drm/intel/issues/2587 [i915#2672]: https://gitlab.freedesktop.org/drm/intel/issues/2672 [i915#2705]: https://gitlab.freedesktop.org/drm/intel/issues/2705 [i915#2842]: https://gitlab.freedesktop.org/drm/intel/issues/2842 [i915#2846]: https://gitlab.freedesktop.org/drm/intel/issues/2846 [i915#3281]: https://gitlab.freedesktop.org/drm/intel/issues/3281 [i915#3282]: https://gitlab.freedesktop.org/drm/intel/issues/3282 [i915#3297]: https://gitlab.freedesktop.org/drm/intel/issues/3297 [i915#3458]: https://gitlab.freedesktop.org/drm/intel/issues/3458 [i915#3539]: https://gitlab.freedesktop.org/drm/intel/issues/3539 [i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555 [i915#3638]: https://gitlab.freedesktop.org/drm/intel/issues/3638 [i915#3689]: https://gitlab.freedesktop.org/drm/intel/issues/3689 [i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708 [i915#3804]: https://gitlab.freedesktop.org/drm/intel/issues/3804 [i915#3886]: https://gitlab.freedesktop.org/drm/intel/issues/3886 [i915#3955]: https://gitlab.freedesktop.org/drm/intel/issues/3955 [i915#3989]: https://gitlab.freedesktop.org/drm/intel/issues/3989 [i915#4077]: https://gitlab.freedesktop.org/drm/intel/issues/4077 [i915#4078]: https://gitlab.freedesktop.org/drm/intel/issues/4078 [i915#4079]: https://gitlab.freedesktop.org/drm/intel/issues/4079 [i915#4083]: https://gitlab.freedesktop.org/drm/intel/issues/4083 [i915#4103]: https://gitlab.freedesktop.org/drm/intel/issues/4103 [i915#4213]: https://gitlab.freedesktop.org/drm/intel/issues/4213 [i915#4258]: https://gitlab.freedesktop.org/drm/intel/issues/4258 [i915#4270]: https://gitlab.freedesktop.org/drm/intel/issues/4270 [i915#4281]: https://gitlab.freedesktop.org/drm/intel/issues/4281 [i915#4391]: https://gitlab.freedesktop.org/drm/intel/issues/4391 [i915#4423]: https://gitlab.freedesktop.org/drm/intel/issues/4423 [i915#4538]: https://gitlab.freedesktop.org/drm/intel/issues/4538 [i915#454]: https://gitlab.freedesktop.org/drm/intel/issues/454 [i915#4579]: https://gitlab.freedesktop.org/drm/intel/issues/4579 [i915#4767]: https://gitlab.freedesktop.org/drm/intel/issues/4767 [i915#4812]: https://gitlab.freedesktop.org/drm/intel/issues/4812 [i915#4833]: https://gitlab.freedesktop.org/drm/intel/issues/4833 [i915#4852]: https://gitlab.freedesktop.org/drm/intel/issues/4852 [i915#4860]: https://gitlab.freedesktop.org/drm/intel/issues/4860 [i915#4880]: https://gitlab.freedesktop.org/drm/intel/issues/4880 [i915#5176]: https://gitlab.freedesktop.org/drm/intel/issues/5176 [i915#5235]: https://gitlab.freedesktop.org/drm/intel/issues/5235 [i915#5286]: https://gitlab.freedesktop.org/drm/intel/issues/5286 [i915#5289]: https://gitlab.freedesktop.org/drm/intel/issues/5289 [i915#5354]: https://gitlab.freedesktop.org/drm/intel/issues/5354 [i915#5784]: https://gitlab.freedesktop.org/drm/intel/issues/5784 [i915#6095]: https://gitlab.freedesktop.org/drm/intel/issues/6095 [i915#6268]: https://gitlab.freedesktop.org/drm/intel/issues/6268 [i915#6433]: https://gitlab.freedesktop.org/drm/intel/issues/6433 [i915#658]: https://gitlab.freedesktop.org/drm/intel/issues/658 [i915#6590]: https://gitlab.freedesktop.org/drm/intel/issues/6590 [i915#7116]: https://gitlab.freedesktop.org/drm/intel/issues/7116 [i915#7461]: https://gitlab.freedesktop.org/drm/intel/issues/7461 [i915#7561]: https://gitlab.freedesktop.org/drm/intel/issues/7561 [i915#7711]: https://gitlab.freedesktop.org/drm/intel/issues/7711 [i915#7742]: https://gitlab.freedesktop.org/drm/intel/issues/7742 [i915#7790]: https://gitlab.freedesktop.org/drm/intel/issues/7790 [i915#7828]: https://gitlab.freedesktop.org/drm/intel/issues/7828 [i915#7975]: https://gitlab.freedesktop.org/drm/intel/issues/7975 [i915#8011]: https://gitlab.freedesktop.org/drm/intel/issues/8011 [i915#8211]: https://gitlab.freedesktop.org/drm/intel/issues/8211 [i915#8213]: https://gitlab.freedesktop.org/drm/intel/issues/8213 [i915#8234]: https://gitlab.freedesktop.org/drm/intel/issues/8234 [i915#8247]: https://gitlab.freedesktop.org/drm/intel/issues/8247 [i915#8347]: https://gitlab.freedesktop.org/drm/intel/issues/8347 [i915#8398]: https://gitlab.freedesktop.org/drm/intel/issues/8398 [i915#8502]: https://gitlab.freedesktop.org/drm/intel/issues/8502 [i915#8516]: https://gitlab.freedesktop.org/drm/intel/issues/8516 Build changes ------------- * Linux: CI_DRM_13238 -> Patchwork_118886v2 CI-20190529: 20190529 CI_DRM_13238: 8c0b302811d744b945dcb6d78164a76188914db9 @ git://anongit.freedesktop.org/gfx-ci/linux IGT_7320: 1c96b08a4cde6f2d49824a8cc3303bd860617b52 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git Patchwork_118886v2: 8c0b302811d744b945dcb6d78164a76188914db9 @ git://anongit.freedesktop.org/gfx-ci/linux piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/index.html [-- Attachment #2: Type: text/html, Size: 10555 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Intel-gfx] ✗ Fi.CI.IGT: failure for Avoid reading OA reports before they land (rev2) 2023-06-07 17:40 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork @ 2023-06-07 19:44 ` Umesh Nerlige Ramappa 0 siblings, 0 replies; 10+ messages in thread From: Umesh Nerlige Ramappa @ 2023-06-07 19:44 UTC (permalink / raw) To: intel-gfx On Wed, Jun 07, 2023 at 05:40:28PM +0000, Patchwork wrote: > Patch Details > >Series: Avoid reading OA reports before they land (rev2) >URL: [1]https://patchwork.freedesktop.org/series/118886/ >State: failure >Details: [2]https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118886v2/index.html > > CI Bug Log - changes from CI_DRM_13238_full -> Patchwork_118886v2_full > >Summary > > FAILURE > > Serious unknown changes coming with Patchwork_118886v2_full absolutely > need to be > verified manually. > > If you think the reported changes have nothing to do with the changes > introduced in Patchwork_118886v2_full, please notify your bug team to > allow them > to document this new failure mode, which will reduce false positives in > CI. > >Participating hosts (7 -> 7) > > No changes in participating hosts > >Possible new issues > > Here are the unknown changes that may have been introduced in > Patchwork_118886v2_full: > > IGT changes > > Possible regressions > > * igt@kms_vblank@pipe-b-accuracy-idle: > > * shard-glk: [3]PASS -> [4]FAIL Unrelated to this patch since no OA use cases in the above test path. Umesh > >Known issues > ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2023-06-07 19:45 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2023-06-05 19:39 [Intel-gfx] [PATCH v4 0/2] Avoid reading OA reports before they land Umesh Nerlige Ramappa 2023-06-05 19:39 ` [Intel-gfx] [PATCH v4 1/2] i915/perf: Drop the aging_tail logic in perf OA Umesh Nerlige Ramappa 2023-06-05 19:39 ` [Intel-gfx] [PATCH v4 2/2] i915/perf: Do not add ggtt offset to hw_tail Umesh Nerlige Ramappa 2023-06-05 23:25 ` [Intel-gfx] ✗ Fi.CI.SPARSE: warning for Avoid reading OA reports before they land Patchwork 2023-06-05 23:44 ` [Intel-gfx] ✗ Fi.CI.BAT: failure " Patchwork 2023-06-07 19:25 ` Umesh Nerlige Ramappa 2023-06-07 1:30 ` [Intel-gfx] ✗ Fi.CI.SPARSE: warning for Avoid reading OA reports before they land (rev2) Patchwork 2023-06-07 1:46 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork 2023-06-07 17:40 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork 2023-06-07 19:44 ` Umesh Nerlige Ramappa
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox