* [Intel-gfx] [PATCH v3 0/2] Avoid reading OA reports before they land
@ 2023-06-02 20:53 Umesh Nerlige Ramappa
2023-06-02 20:53 ` [Intel-gfx] [PATCH v3 1/2] i915/perf: Drop the aging_tail logic in perf OA Umesh Nerlige Ramappa
` (5 more replies)
0 siblings, 6 replies; 8+ messages in thread
From: Umesh Nerlige Ramappa @ 2023-06-02 20:53 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/
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 | 76 ++++++++++----------------
drivers/gpu/drm/i915/i915_perf_types.h | 12 ----
2 files changed, 28 insertions(+), 60 deletions(-)
--
2.36.1
^ permalink raw reply [flat|nested] 8+ messages in thread* [Intel-gfx] [PATCH v3 1/2] i915/perf: Drop the aging_tail logic in perf OA 2023-06-02 20:53 [Intel-gfx] [PATCH v3 0/2] Avoid reading OA reports before they land Umesh Nerlige Ramappa @ 2023-06-02 20:53 ` Umesh Nerlige Ramappa 2023-06-02 21:53 ` Dixit, Ashutosh 2023-06-02 20:53 ` [Intel-gfx] [PATCH v3 2/2] i915/perf: Do not add ggtt offset to hw_tail Umesh Nerlige Ramappa ` (4 subsequent siblings) 5 siblings, 1 reply; 8+ messages in thread From: Umesh Nerlige Ramappa @ 2023-06-02 20:53 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 Bug: https://gitlab.freedesktop.org/drm/intel/-/issues/7484 Bug: https://gitlab.freedesktop.org/drm/intel/-/issues/7757 Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com> --- drivers/gpu/drm/i915/i915_perf.c | 75 ++++++++++---------------- drivers/gpu/drm/i915/i915_perf_types.h | 12 ----- 2 files changed, 28 insertions(+), 59 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_perf.c b/drivers/gpu/drm/i915/i915_perf.c index 58284156428d..9cb3d395046e 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,27 +567,15 @@ 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; + /* 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; + 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 @@ -596,31 +583,28 @@ static bool oa_buffer_check_unlocked(struct i915_perf_stream *stream) * 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; + * + * 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; @@ -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] 8+ messages in thread
* Re: [Intel-gfx] [PATCH v3 1/2] i915/perf: Drop the aging_tail logic in perf OA 2023-06-02 20:53 ` [Intel-gfx] [PATCH v3 1/2] i915/perf: Drop the aging_tail logic in perf OA Umesh Nerlige Ramappa @ 2023-06-02 21:53 ` Dixit, Ashutosh 0 siblings, 0 replies; 8+ messages in thread From: Dixit, Ashutosh @ 2023-06-02 21:53 UTC (permalink / raw) To: Umesh Nerlige Ramappa; +Cc: intel-gfx On Fri, 02 Jun 2023 13:53:26 -0700, Umesh Nerlige Ramappa wrote: > Hi Umesh, > 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 > > Bug: https://gitlab.freedesktop.org/drm/intel/-/issues/7484 > Bug: https://gitlab.freedesktop.org/drm/intel/-/issues/7757 > Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@intel.com> > --- > drivers/gpu/drm/i915/i915_perf.c | 75 ++++++++++---------------- > drivers/gpu/drm/i915/i915_perf_types.h | 12 ----- > 2 files changed, 28 insertions(+), 59 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i915_perf.c b/drivers/gpu/drm/i915/i915_perf.c > index 58284156428d..9cb3d395046e 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,27 +567,15 @@ 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; > + /* 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; > + 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 > @@ -596,31 +583,28 @@ static bool oa_buffer_check_unlocked(struct i915_perf_stream *stream) > * 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 entire comment above should move to the left (at present the above half is floating to the right). > - * > - * 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; > + * > + * 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; I forgot to mention it earlier, but the above statement is exactly equivalent to: pollin = OA_TAKEN(stream->oa_buffer.tail, stream->oa_buffer.head) >= report_size; So gtt_offset can be removed (because OA_TAKEN is a circular diff). Anyway this is optional, you can sneak it into Patch 1 or Patch 2 if you wish, otherwise leave as is. Otherwise, this is: Reviewed-by: Ashutosh Dixit <ashutosh.dixit@intel.com> > @@ -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 [flat|nested] 8+ messages in thread
* [Intel-gfx] [PATCH v3 2/2] i915/perf: Do not add ggtt offset to hw_tail 2023-06-02 20:53 [Intel-gfx] [PATCH v3 0/2] Avoid reading OA reports before they land Umesh Nerlige Ramappa 2023-06-02 20:53 ` [Intel-gfx] [PATCH v3 1/2] i915/perf: Drop the aging_tail logic in perf OA Umesh Nerlige Ramappa @ 2023-06-02 20:53 ` Umesh Nerlige Ramappa 2023-06-02 22:07 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Avoid reading OA reports before they land Patchwork ` (3 subsequent siblings) 5 siblings, 0 replies; 8+ messages in thread From: Umesh Nerlige Ramappa @ 2023-06-02 20:53 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 9cb3d395046e..0a1f40d21163 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] 8+ messages in thread
* [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Avoid reading OA reports before they land 2023-06-02 20:53 [Intel-gfx] [PATCH v3 0/2] Avoid reading OA reports before they land Umesh Nerlige Ramappa 2023-06-02 20:53 ` [Intel-gfx] [PATCH v3 1/2] i915/perf: Drop the aging_tail logic in perf OA Umesh Nerlige Ramappa 2023-06-02 20:53 ` [Intel-gfx] [PATCH v3 2/2] i915/perf: Do not add ggtt offset to hw_tail Umesh Nerlige Ramappa @ 2023-06-02 22:07 ` Patchwork 2023-06-02 22:07 ` [Intel-gfx] ✗ Fi.CI.SPARSE: " Patchwork ` (2 subsequent siblings) 5 siblings, 0 replies; 8+ messages in thread From: Patchwork @ 2023-06-02 22:07 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/118802/ State : warning == Summary == Error: dim checkpatch failed 4ad91b694aa1 i915/perf: Drop the aging_tail logic in perf OA -:36: WARNING:COMMIT_LOG_USE_LINK: Unknown link reference 'Bug:', use 'Link:' or 'Closes:' instead #36: Bug: https://gitlab.freedesktop.org/drm/intel/-/issues/7484 -:37: WARNING:COMMIT_LOG_USE_LINK: Unknown link reference 'Bug:', use 'Link:' or 'Closes:' instead #37: Bug: https://gitlab.freedesktop.org/drm/intel/-/issues/7757 total: 0 errors, 2 warnings, 0 checks, 143 lines checked c3dbf9b44f05 i915/perf: Do not add ggtt offset to hw_tail ^ permalink raw reply [flat|nested] 8+ messages in thread
* [Intel-gfx] ✗ Fi.CI.SPARSE: warning for Avoid reading OA reports before they land 2023-06-02 20:53 [Intel-gfx] [PATCH v3 0/2] Avoid reading OA reports before they land Umesh Nerlige Ramappa ` (2 preceding siblings ...) 2023-06-02 22:07 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Avoid reading OA reports before they land Patchwork @ 2023-06-02 22:07 ` Patchwork 2023-06-02 22:21 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork 2023-06-05 11:28 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork 5 siblings, 0 replies; 8+ messages in thread From: Patchwork @ 2023-06-02 22:07 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/118802/ 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] 8+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for Avoid reading OA reports before they land 2023-06-02 20:53 [Intel-gfx] [PATCH v3 0/2] Avoid reading OA reports before they land Umesh Nerlige Ramappa ` (3 preceding siblings ...) 2023-06-02 22:07 ` [Intel-gfx] ✗ Fi.CI.SPARSE: " Patchwork @ 2023-06-02 22:21 ` Patchwork 2023-06-05 11:28 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork 5 siblings, 0 replies; 8+ messages in thread From: Patchwork @ 2023-06-02 22:21 UTC (permalink / raw) To: Umesh Nerlige Ramappa; +Cc: intel-gfx [-- Attachment #1: Type: text/plain, Size: 13144 bytes --] == Series Details == Series: Avoid reading OA reports before they land URL : https://patchwork.freedesktop.org/series/118802/ State : success == Summary == CI Bug Log - changes from CI_DRM_13223 -> Patchwork_118802v1 ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/index.html Participating hosts (36 -> 37) ------------------------------ Additional (2): fi-kbl-soraka bat-dg1-5 Missing (1): fi-snb-2520m Known issues ------------ Here are the changes found in Patchwork_118802v1 that come from known issues: ### CI changes ### #### Possible fixes #### * boot: - fi-kbl-8809g: [FAIL][1] ([i915#8293] / [i915#8298]) -> [PASS][2] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/fi-kbl-8809g/boot.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/fi-kbl-8809g/boot.html ### IGT changes ### #### Issues hit #### * igt@core_hotunplug@unbind-rebind: - fi-kbl-8809g: NOTRUN -> [ABORT][3] ([i915#8298] / [i915#8299] / [i915#8397]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/fi-kbl-8809g/igt@core_hotunplug@unbind-rebind.html * igt@gem_exec_suspend@basic-s0@smem: - bat-jsl-3: [PASS][4] -> [ABORT][5] ([i915#5122]) [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/bat-jsl-3/igt@gem_exec_suspend@basic-s0@smem.html [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/bat-jsl-3/igt@gem_exec_suspend@basic-s0@smem.html * igt@gem_huc_copy@huc-copy: - fi-kbl-8809g: NOTRUN -> [SKIP][6] ([fdo#109271] / [i915#2190]) [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/fi-kbl-8809g/igt@gem_huc_copy@huc-copy.html - fi-kbl-soraka: NOTRUN -> [SKIP][7] ([fdo#109271] / [i915#2190]) [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/fi-kbl-soraka/igt@gem_huc_copy@huc-copy.html * igt@gem_lmem_swapping@basic: - fi-kbl-soraka: NOTRUN -> [SKIP][8] ([fdo#109271] / [i915#4613]) +3 similar issues [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/fi-kbl-soraka/igt@gem_lmem_swapping@basic.html * igt@gem_mmap@basic: - bat-dg1-5: NOTRUN -> [SKIP][9] ([i915#4083]) [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/bat-dg1-5/igt@gem_mmap@basic.html * igt@gem_tiled_fence_blits@basic: - bat-dg1-5: NOTRUN -> [SKIP][10] ([i915#4077]) +2 similar issues [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/bat-dg1-5/igt@gem_tiled_fence_blits@basic.html * igt@gem_tiled_pread_basic: - bat-dg1-5: NOTRUN -> [SKIP][11] ([i915#4079]) +1 similar issue [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/bat-dg1-5/igt@gem_tiled_pread_basic.html * igt@i915_module_load@reload: - fi-kbl-soraka: NOTRUN -> [DMESG-WARN][12] ([i915#1982]) [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/fi-kbl-soraka/igt@i915_module_load@reload.html * igt@i915_pm_backlight@basic-brightness: - bat-dg1-5: NOTRUN -> [SKIP][13] ([i915#7561]) [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/bat-dg1-5/igt@i915_pm_backlight@basic-brightness.html * igt@i915_pm_rpm@basic-pci-d3-state: - fi-hsw-4770: [PASS][14] -> [SKIP][15] ([fdo#109271]) [14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/fi-hsw-4770/igt@i915_pm_rpm@basic-pci-d3-state.html [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/fi-hsw-4770/igt@i915_pm_rpm@basic-pci-d3-state.html * igt@i915_pm_rpm@basic-rte: - fi-hsw-4770: [PASS][16] -> [FAIL][17] ([i915#7364]) [16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/fi-hsw-4770/igt@i915_pm_rpm@basic-rte.html [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/fi-hsw-4770/igt@i915_pm_rpm@basic-rte.html * igt@i915_pm_rps@basic-api: - bat-dg1-5: NOTRUN -> [SKIP][18] ([i915#6621]) [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/bat-dg1-5/igt@i915_pm_rps@basic-api.html * igt@i915_selftest@live@gt_pm: - fi-kbl-soraka: NOTRUN -> [DMESG-FAIL][19] ([i915#1886] / [i915#7913]) [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/fi-kbl-soraka/igt@i915_selftest@live@gt_pm.html * igt@i915_selftest@live@hangcheck: - fi-skl-guc: [PASS][20] -> [DMESG-WARN][21] ([i915#8073]) [20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/fi-skl-guc/igt@i915_selftest@live@hangcheck.html [21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/fi-skl-guc/igt@i915_selftest@live@hangcheck.html * igt@i915_suspend@basic-s3-without-i915: - bat-jsl-3: [PASS][22] -> [FAIL][23] ([fdo#103375]) [22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/bat-jsl-3/igt@i915_suspend@basic-s3-without-i915.html [23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/bat-jsl-3/igt@i915_suspend@basic-s3-without-i915.html * igt@kms_addfb_basic@basic-x-tiled-legacy: - bat-dg1-5: NOTRUN -> [SKIP][24] ([i915#4212]) +7 similar issues [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/bat-dg1-5/igt@kms_addfb_basic@basic-x-tiled-legacy.html * igt@kms_addfb_basic@basic-y-tiled-legacy: - bat-dg1-5: NOTRUN -> [SKIP][25] ([i915#4215]) [25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/bat-dg1-5/igt@kms_addfb_basic@basic-y-tiled-legacy.html * igt@kms_addfb_basic@too-high: - fi-kbl-8809g: NOTRUN -> [FAIL][26] ([i915#8296]) +2 similar issues [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/fi-kbl-8809g/igt@kms_addfb_basic@too-high.html * igt@kms_chamelium_frames@hdmi-crc-fast: - fi-kbl-soraka: NOTRUN -> [SKIP][27] ([fdo#109271]) +14 similar issues [27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/fi-kbl-soraka/igt@kms_chamelium_frames@hdmi-crc-fast.html * igt@kms_chamelium_hpd@vga-hpd-fast: - bat-dg1-5: NOTRUN -> [SKIP][28] ([i915#7828]) +8 similar issues [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/bat-dg1-5/igt@kms_chamelium_hpd@vga-hpd-fast.html * igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy: - bat-dg1-5: NOTRUN -> [SKIP][29] ([i915#4103] / [i915#4213]) +1 similar issue [29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/bat-dg1-5/igt@kms_cursor_legacy@basic-busy-flip-before-cursor-legacy.html * igt@kms_force_connector_basic@force-connector-state: - fi-kbl-8809g: NOTRUN -> [DMESG-FAIL][30] ([i915#8299]) [30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/fi-kbl-8809g/igt@kms_force_connector_basic@force-connector-state.html * igt@kms_force_connector_basic@force-edid: - fi-kbl-8809g: NOTRUN -> [CRASH][31] ([i915#8299]) [31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/fi-kbl-8809g/igt@kms_force_connector_basic@force-edid.html * igt@kms_force_connector_basic@force-load-detect: - bat-dg1-5: NOTRUN -> [SKIP][32] ([fdo#109285]) [32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/bat-dg1-5/igt@kms_force_connector_basic@force-load-detect.html * igt@kms_psr@cursor_plane_move: - fi-kbl-8809g: NOTRUN -> [SKIP][33] ([fdo#109271]) +59 similar issues [33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/fi-kbl-8809g/igt@kms_psr@cursor_plane_move.html * igt@kms_psr@sprite_plane_onoff: - bat-dg1-5: NOTRUN -> [SKIP][34] ([i915#1072] / [i915#4078]) +3 similar issues [34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/bat-dg1-5/igt@kms_psr@sprite_plane_onoff.html * igt@kms_setmode@basic-clone-single-crtc: - bat-dg1-5: NOTRUN -> [SKIP][35] ([i915#3555] / [i915#4579]) [35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/bat-dg1-5/igt@kms_setmode@basic-clone-single-crtc.html - fi-kbl-soraka: NOTRUN -> [SKIP][36] ([fdo#109271] / [i915#4579]) [36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/fi-kbl-soraka/igt@kms_setmode@basic-clone-single-crtc.html - fi-kbl-8809g: NOTRUN -> [SKIP][37] ([fdo#109271] / [i915#4579]) [37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/fi-kbl-8809g/igt@kms_setmode@basic-clone-single-crtc.html * igt@prime_vgem@basic-fence-read: - bat-dg1-5: NOTRUN -> [SKIP][38] ([i915#3708]) +3 similar issues [38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/bat-dg1-5/igt@prime_vgem@basic-fence-read.html * igt@prime_vgem@basic-gtt: - bat-dg1-5: NOTRUN -> [SKIP][39] ([i915#3708] / [i915#4077]) +1 similar issue [39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/bat-dg1-5/igt@prime_vgem@basic-gtt.html #### Possible fixes #### * igt@i915_selftest@live@guc: - bat-rpls-2: [DMESG-WARN][40] ([i915#7852]) -> [PASS][41] [40]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/bat-rpls-2/igt@i915_selftest@live@guc.html [41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/bat-rpls-2/igt@i915_selftest@live@guc.html * igt@i915_selftest@live@slpc: - {bat-mtlp-8}: [DMESG-WARN][42] ([i915#6367]) -> [PASS][43] [42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/bat-mtlp-8/igt@i915_selftest@live@slpc.html [43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/bat-mtlp-8/igt@i915_selftest@live@slpc.html - bat-rpls-1: [DMESG-WARN][44] ([i915#6367]) -> [PASS][45] [44]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/bat-rpls-1/igt@i915_selftest@live@slpc.html [45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/bat-rpls-1/igt@i915_selftest@live@slpc.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#103375]: https://bugs.freedesktop.org/show_bug.cgi?id=103375 [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271 [fdo#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285 [i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072 [i915#1886]: https://gitlab.freedesktop.org/drm/intel/issues/1886 [i915#1982]: https://gitlab.freedesktop.org/drm/intel/issues/1982 [i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190 [i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555 [i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708 [i915#4077]: https://gitlab.freedesktop.org/drm/intel/issues/4077 [i915#4078]: https://gitlab.freedesktop.org/drm/intel/issues/4078 [i915#4079]: https://gitlab.freedesktop.org/drm/intel/issues/4079 [i915#4083]: https://gitlab.freedesktop.org/drm/intel/issues/4083 [i915#4103]: https://gitlab.freedesktop.org/drm/intel/issues/4103 [i915#4212]: https://gitlab.freedesktop.org/drm/intel/issues/4212 [i915#4213]: https://gitlab.freedesktop.org/drm/intel/issues/4213 [i915#4215]: https://gitlab.freedesktop.org/drm/intel/issues/4215 [i915#4423]: https://gitlab.freedesktop.org/drm/intel/issues/4423 [i915#4579]: https://gitlab.freedesktop.org/drm/intel/issues/4579 [i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613 [i915#5122]: https://gitlab.freedesktop.org/drm/intel/issues/5122 [i915#6367]: https://gitlab.freedesktop.org/drm/intel/issues/6367 [i915#6621]: https://gitlab.freedesktop.org/drm/intel/issues/6621 [i915#7364]: https://gitlab.freedesktop.org/drm/intel/issues/7364 [i915#7561]: https://gitlab.freedesktop.org/drm/intel/issues/7561 [i915#7828]: https://gitlab.freedesktop.org/drm/intel/issues/7828 [i915#7852]: https://gitlab.freedesktop.org/drm/intel/issues/7852 [i915#7913]: https://gitlab.freedesktop.org/drm/intel/issues/7913 [i915#8073]: https://gitlab.freedesktop.org/drm/intel/issues/8073 [i915#8293]: https://gitlab.freedesktop.org/drm/intel/issues/8293 [i915#8296]: https://gitlab.freedesktop.org/drm/intel/issues/8296 [i915#8298]: https://gitlab.freedesktop.org/drm/intel/issues/8298 [i915#8299]: https://gitlab.freedesktop.org/drm/intel/issues/8299 [i915#8397]: https://gitlab.freedesktop.org/drm/intel/issues/8397 [i915#8497]: https://gitlab.freedesktop.org/drm/intel/issues/8497 Build changes ------------- * Linux: CI_DRM_13223 -> Patchwork_118802v1 CI-20190529: 20190529 CI_DRM_13223: a47222ca5df4240d376b2617d04de42db9e2c0ff @ git://anongit.freedesktop.org/gfx-ci/linux IGT_7318: c2d8ef8b9397d0976959f29dc1dd7c8a698d65fe @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git Patchwork_118802v1: a47222ca5df4240d376b2617d04de42db9e2c0ff @ git://anongit.freedesktop.org/gfx-ci/linux ### Linux commits e5cc018b21ad i915/perf: Do not add ggtt offset to hw_tail 5b6157cb4644 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_118802v1/index.html [-- Attachment #2: Type: text/html, Size: 15558 bytes --] ^ permalink raw reply [flat|nested] 8+ messages in thread
* [Intel-gfx] ✓ Fi.CI.IGT: success for Avoid reading OA reports before they land 2023-06-02 20:53 [Intel-gfx] [PATCH v3 0/2] Avoid reading OA reports before they land Umesh Nerlige Ramappa ` (4 preceding siblings ...) 2023-06-02 22:21 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork @ 2023-06-05 11:28 ` Patchwork 5 siblings, 0 replies; 8+ messages in thread From: Patchwork @ 2023-06-05 11:28 UTC (permalink / raw) To: Umesh Nerlige Ramappa; +Cc: intel-gfx [-- Attachment #1: Type: text/plain, Size: 14253 bytes --] == Series Details == Series: Avoid reading OA reports before they land URL : https://patchwork.freedesktop.org/series/118802/ State : success == Summary == CI Bug Log - changes from CI_DRM_13223_full -> Patchwork_118802v1_full ==================================================== Summary ------- **SUCCESS** No regressions found. Participating hosts (7 -> 7) ------------------------------ No changes in participating hosts Known issues ------------ Here are the changes found in Patchwork_118802v1_full that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_ctx_persistence@idempotent: - shard-snb: NOTRUN -> [SKIP][1] ([fdo#109271] / [i915#1099]) [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/shard-snb6/igt@gem_ctx_persistence@idempotent.html * igt@gen9_exec_parse@allowed-single: - shard-glk: [PASS][2] -> [ABORT][3] ([i915#5566]) [2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/shard-glk9/igt@gen9_exec_parse@allowed-single.html [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/shard-glk1/igt@gen9_exec_parse@allowed-single.html * igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-vga: - shard-snb: NOTRUN -> [SKIP][4] ([fdo#109271] / [i915#4579]) +13 similar issues [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/shard-snb2/igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-vga.html * igt@kms_big_fb@4-tiled-max-hw-stride-32bpp-rotate-180-async-flip: - shard-snb: NOTRUN -> [SKIP][5] ([fdo#109271]) +79 similar issues [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/shard-snb6/igt@kms_big_fb@4-tiled-max-hw-stride-32bpp-rotate-180-async-flip.html * igt@kms_ccs@pipe-c-crc-primary-rotation-180-y_tiled_gen12_rc_ccs_cc: - shard-apl: NOTRUN -> [SKIP][6] ([fdo#109271] / [i915#3886]) [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/shard-apl4/igt@kms_ccs@pipe-c-crc-primary-rotation-180-y_tiled_gen12_rc_ccs_cc.html * igt@kms_ccs@pipe-d-missing-ccs-buffer-y_tiled_gen12_rc_ccs_cc: - shard-apl: NOTRUN -> [SKIP][7] ([fdo#109271]) +4 similar issues [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/shard-apl4/igt@kms_ccs@pipe-d-missing-ccs-buffer-y_tiled_gen12_rc_ccs_cc.html * igt@kms_vrr@flipline: - shard-apl: NOTRUN -> [SKIP][8] ([fdo#109271] / [i915#4579]) [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/shard-apl4/igt@kms_vrr@flipline.html #### Possible fixes #### * igt@drm_fdinfo@most-busy-idle-check-all@rcs0: - {shard-rkl}: [FAIL][9] ([i915#7742]) -> [PASS][10] +1 similar issue [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/shard-rkl-7/igt@drm_fdinfo@most-busy-idle-check-all@rcs0.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/shard-rkl-2/igt@drm_fdinfo@most-busy-idle-check-all@rcs0.html * igt@gem_eio@unwedge-stress: - {shard-dg1}: [FAIL][11] ([i915#5784]) -> [PASS][12] +1 similar issue [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/shard-dg1-18/igt@gem_eio@unwedge-stress.html [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/shard-dg1-17/igt@gem_eio@unwedge-stress.html * igt@gem_exec_fair@basic-none-solo@rcs0: - shard-apl: [FAIL][13] ([i915#2842]) -> [PASS][14] [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/shard-apl1/igt@gem_exec_fair@basic-none-solo@rcs0.html [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/shard-apl1/igt@gem_exec_fair@basic-none-solo@rcs0.html * igt@gem_exec_fair@basic-none@bcs0: - {shard-rkl}: [FAIL][15] ([i915#2842]) -> [PASS][16] +3 similar issues [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/shard-rkl-3/igt@gem_exec_fair@basic-none@bcs0.html [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/shard-rkl-6/igt@gem_exec_fair@basic-none@bcs0.html * igt@gem_exec_fair@basic-pace-solo@rcs0: - shard-glk: [FAIL][17] ([i915#2842]) -> [PASS][18] [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/shard-glk9/igt@gem_exec_fair@basic-pace-solo@rcs0.html [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/shard-glk1/igt@gem_exec_fair@basic-pace-solo@rcs0.html * igt@gem_mmap_offset@clear@smem0: - {shard-dg1}: [DMESG-WARN][19] ([i915#8304]) -> [PASS][20] [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/shard-dg1-17/igt@gem_mmap_offset@clear@smem0.html [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/shard-dg1-15/igt@gem_mmap_offset@clear@smem0.html * igt@i915_pm_dc@dc6-dpms: - {shard-tglu}: [FAIL][21] ([i915#3989] / [i915#454]) -> [PASS][22] [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/shard-tglu-5/igt@i915_pm_dc@dc6-dpms.html [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/shard-tglu-4/igt@i915_pm_dc@dc6-dpms.html * igt@i915_pm_rc6_residency@rc6-idle@rcs0: - {shard-dg1}: [FAIL][23] ([i915#3591]) -> [PASS][24] +1 similar issue [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/shard-dg1-18/igt@i915_pm_rc6_residency@rc6-idle@rcs0.html [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/shard-dg1-18/igt@i915_pm_rc6_residency@rc6-idle@rcs0.html * igt@i915_selftest@perf@engine_cs: - shard-snb: [ABORT][25] ([i915#4528] / [i915#4579]) -> [PASS][26] [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/shard-snb5/igt@i915_selftest@perf@engine_cs.html [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/shard-snb6/igt@i915_selftest@perf@engine_cs.html * igt@kms_async_flips@alternate-sync-async-flip@pipe-a-hdmi-a-2: - shard-glk: [FAIL][27] ([i915#2521]) -> [PASS][28] [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/shard-glk1/igt@kms_async_flips@alternate-sync-async-flip@pipe-a-hdmi-a-2.html [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/shard-glk5/igt@kms_async_flips@alternate-sync-async-flip@pipe-a-hdmi-a-2.html * igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size: - shard-apl: [FAIL][29] ([i915#2346]) -> [PASS][30] +1 similar issue [29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/shard-apl2/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size.html [30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/shard-apl1/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size.html * igt@kms_cursor_legacy@single-bo@pipe-b: - {shard-rkl}: [INCOMPLETE][31] ([i915#8011]) -> [PASS][32] [31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/shard-rkl-7/igt@kms_cursor_legacy@single-bo@pipe-b.html [32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/shard-rkl-1/igt@kms_cursor_legacy@single-bo@pipe-b.html * igt@kms_plane@plane-panning-bottom-right-suspend@pipe-b-planes: - shard-apl: [ABORT][33] ([i915#180]) -> [PASS][34] [33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/shard-apl1/igt@kms_plane@plane-panning-bottom-right-suspend@pipe-b-planes.html [34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/shard-apl4/igt@kms_plane@plane-panning-bottom-right-suspend@pipe-b-planes.html * igt@kms_plane_scaling@intel-max-src-size@pipe-a-hdmi-a-2: - {shard-rkl}: [FAIL][35] ([i915#8292]) -> [PASS][36] [35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/shard-rkl-3/igt@kms_plane_scaling@intel-max-src-size@pipe-a-hdmi-a-2.html [36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/shard-rkl-6/igt@kms_plane_scaling@intel-max-src-size@pipe-a-hdmi-a-2.html #### Warnings #### * igt@kms_hdmi_inject@inject-audio: - shard-glk: [FAIL][37] ([IGT#3]) -> [SKIP][38] ([fdo#109271]) [37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_13223/shard-glk5/igt@kms_hdmi_inject@inject-audio.html [38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_118802v1/shard-glk4/igt@kms_hdmi_inject@inject-audio.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [IGT#3]: https://gitlab.freedesktop.org/drm/igt-gpu-tools/issues/3 [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271 [fdo#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285 [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#112283]: https://bugs.freedesktop.org/show_bug.cgi?id=112283 [i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072 [i915#1099]: https://gitlab.freedesktop.org/drm/intel/issues/1099 [i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180 [i915#2346]: https://gitlab.freedesktop.org/drm/intel/issues/2346 [i915#2437]: https://gitlab.freedesktop.org/drm/intel/issues/2437 [i915#2521]: https://gitlab.freedesktop.org/drm/intel/issues/2521 [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#280]: https://gitlab.freedesktop.org/drm/intel/issues/280 [i915#2842]: https://gitlab.freedesktop.org/drm/intel/issues/2842 [i915#3281]: https://gitlab.freedesktop.org/drm/intel/issues/3281 [i915#3282]: https://gitlab.freedesktop.org/drm/intel/issues/3282 [i915#3299]: https://gitlab.freedesktop.org/drm/intel/issues/3299 [i915#3359]: https://gitlab.freedesktop.org/drm/intel/issues/3359 [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#3591]: https://gitlab.freedesktop.org/drm/intel/issues/3591 [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#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#4083]: https://gitlab.freedesktop.org/drm/intel/issues/4083 [i915#4098]: https://gitlab.freedesktop.org/drm/intel/issues/4098 [i915#4103]: https://gitlab.freedesktop.org/drm/intel/issues/4103 [i915#4213]: https://gitlab.freedesktop.org/drm/intel/issues/4213 [i915#4270]: https://gitlab.freedesktop.org/drm/intel/issues/4270 [i915#4391]: https://gitlab.freedesktop.org/drm/intel/issues/4391 [i915#4528]: https://gitlab.freedesktop.org/drm/intel/issues/4528 [i915#4538]: https://gitlab.freedesktop.org/drm/intel/issues/4538 [i915#454]: https://gitlab.freedesktop.org/drm/intel/issues/454 [i915#4565]: https://gitlab.freedesktop.org/drm/intel/issues/4565 [i915#4579]: https://gitlab.freedesktop.org/drm/intel/issues/4579 [i915#4771]: https://gitlab.freedesktop.org/drm/intel/issues/4771 [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#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#5439]: https://gitlab.freedesktop.org/drm/intel/issues/5439 [i915#5566]: https://gitlab.freedesktop.org/drm/intel/issues/5566 [i915#5723]: https://gitlab.freedesktop.org/drm/intel/issues/5723 [i915#5784]: https://gitlab.freedesktop.org/drm/intel/issues/5784 [i915#6095]: https://gitlab.freedesktop.org/drm/intel/issues/6095 [i915#6334]: https://gitlab.freedesktop.org/drm/intel/issues/6334 [i915#6524]: https://gitlab.freedesktop.org/drm/intel/issues/6524 [i915#6590]: https://gitlab.freedesktop.org/drm/intel/issues/6590 [i915#7116]: https://gitlab.freedesktop.org/drm/intel/issues/7116 [i915#7697]: https://gitlab.freedesktop.org/drm/intel/issues/7697 [i915#7711]: https://gitlab.freedesktop.org/drm/intel/issues/7711 [i915#7742]: https://gitlab.freedesktop.org/drm/intel/issues/7742 [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#8213]: https://gitlab.freedesktop.org/drm/intel/issues/8213 [i915#8292]: https://gitlab.freedesktop.org/drm/intel/issues/8292 [i915#8304]: https://gitlab.freedesktop.org/drm/intel/issues/8304 [i915#8411]: https://gitlab.freedesktop.org/drm/intel/issues/8411 [i915#8414]: https://gitlab.freedesktop.org/drm/intel/issues/8414 [i915#8502]: https://gitlab.freedesktop.org/drm/intel/issues/8502 [i915#8555]: https://gitlab.freedesktop.org/drm/intel/issues/8555 Build changes ------------- * Linux: CI_DRM_13223 -> Patchwork_118802v1 CI-20190529: 20190529 CI_DRM_13223: a47222ca5df4240d376b2617d04de42db9e2c0ff @ git://anongit.freedesktop.org/gfx-ci/linux IGT_7318: c2d8ef8b9397d0976959f29dc1dd7c8a698d65fe @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git Patchwork_118802v1: a47222ca5df4240d376b2617d04de42db9e2c0ff @ 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_118802v1/index.html [-- Attachment #2: Type: text/html, Size: 12052 bytes --] ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2023-06-05 11:28 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2023-06-02 20:53 [Intel-gfx] [PATCH v3 0/2] Avoid reading OA reports before they land Umesh Nerlige Ramappa 2023-06-02 20:53 ` [Intel-gfx] [PATCH v3 1/2] i915/perf: Drop the aging_tail logic in perf OA Umesh Nerlige Ramappa 2023-06-02 21:53 ` Dixit, Ashutosh 2023-06-02 20:53 ` [Intel-gfx] [PATCH v3 2/2] i915/perf: Do not add ggtt offset to hw_tail Umesh Nerlige Ramappa 2023-06-02 22:07 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Avoid reading OA reports before they land Patchwork 2023-06-02 22:07 ` [Intel-gfx] ✗ Fi.CI.SPARSE: " Patchwork 2023-06-02 22:21 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork 2023-06-05 11:28 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.