* [Intel-gfx] [PATCH 0/4] drm/i915/guc: Improve CTB error handling @ 2021-07-01 15:55 Michal Wajdeczko 2021-07-01 15:55 ` [Intel-gfx] [PATCH 1/4] drm/i915/guc: Verify result from CTB (de)register action Michal Wajdeczko ` (5 more replies) 0 siblings, 6 replies; 15+ messages in thread From: Michal Wajdeczko @ 2021-07-01 15:55 UTC (permalink / raw) To: intel-gfx, dri-devel There was a gap in handling MMIO result from CTB (de)registration and while fixing it improve some other error reports. Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com> Michal Wajdeczko (4): drm/i915/guc: Verify result from CTB (de)register action drm/i915/guc: Print error name on CTB (de)registration failure drm/i915/guc: Print error name on CTB send failure drm/i915/guc: Move and improve error message for missed CTB reply drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c | 30 ++++++++++++++--------- 1 file changed, 18 insertions(+), 12 deletions(-) -- 2.25.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
* [Intel-gfx] [PATCH 1/4] drm/i915/guc: Verify result from CTB (de)register action 2021-07-01 15:55 [Intel-gfx] [PATCH 0/4] drm/i915/guc: Improve CTB error handling Michal Wajdeczko @ 2021-07-01 15:55 ` Michal Wajdeczko 2021-08-18 14:21 ` Daniel Vetter 2021-07-01 15:55 ` [Intel-gfx] [PATCH 2/4] drm/i915/guc: Print error name on CTB (de)registration failure Michal Wajdeczko ` (4 subsequent siblings) 5 siblings, 1 reply; 15+ messages in thread From: Michal Wajdeczko @ 2021-07-01 15:55 UTC (permalink / raw) To: intel-gfx, dri-devel; +Cc: Dan Carpenter In commit b839a869dfc9 ("drm/i915/guc: Add support for data reporting in GuC responses") we missed the hypothetical case that GuC might return positive non-zero value as success data. While that would be lucky treated as error case, and at the end will result in reporting valid -EIO, in the meantime this value will be passed to ERR_PTR that could be misleading. v2: rebased Reported-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com> Cc: Dan Carpenter <dan.carpenter@oracle.com> --- drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c index 43409044528e..a26bb55c0898 100644 --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c @@ -148,12 +148,15 @@ static int guc_action_register_ct_buffer(struct intel_guc *guc, u32 type, FIELD_PREP(HOST2GUC_REGISTER_CTB_REQUEST_MSG_2_DESC_ADDR, desc_addr), FIELD_PREP(HOST2GUC_REGISTER_CTB_REQUEST_MSG_3_BUFF_ADDR, buff_addr), }; + int ret; GEM_BUG_ON(type != GUC_CTB_TYPE_HOST2GUC && type != GUC_CTB_TYPE_GUC2HOST); GEM_BUG_ON(size % SZ_4K); /* CT registration must go over MMIO */ - return intel_guc_send_mmio(guc, request, ARRAY_SIZE(request), NULL, 0); + ret = intel_guc_send_mmio(guc, request, ARRAY_SIZE(request), NULL, 0); + + return ret > 0 ? -EPROTO : ret; } static int ct_register_buffer(struct intel_guc_ct *ct, u32 type, @@ -177,11 +180,14 @@ static int guc_action_deregister_ct_buffer(struct intel_guc *guc, u32 type) FIELD_PREP(GUC_HXG_REQUEST_MSG_0_ACTION, GUC_ACTION_HOST2GUC_DEREGISTER_CTB), FIELD_PREP(HOST2GUC_DEREGISTER_CTB_REQUEST_MSG_1_TYPE, type), }; + int ret; GEM_BUG_ON(type != GUC_CTB_TYPE_HOST2GUC && type != GUC_CTB_TYPE_GUC2HOST); /* CT deregistration must go over MMIO */ - return intel_guc_send_mmio(guc, request, ARRAY_SIZE(request), NULL, 0); + ret = intel_guc_send_mmio(guc, request, ARRAY_SIZE(request), NULL, 0); + + return ret > 0 ? -EPROTO : ret; } static int ct_deregister_buffer(struct intel_guc_ct *ct, u32 type) -- 2.25.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [PATCH 1/4] drm/i915/guc: Verify result from CTB (de)register action 2021-07-01 15:55 ` [Intel-gfx] [PATCH 1/4] drm/i915/guc: Verify result from CTB (de)register action Michal Wajdeczko @ 2021-08-18 14:21 ` Daniel Vetter 0 siblings, 0 replies; 15+ messages in thread From: Daniel Vetter @ 2021-08-18 14:21 UTC (permalink / raw) To: Michal Wajdeczko; +Cc: intel-gfx, dri-devel, Dan Carpenter On Thu, Jul 01, 2021 at 05:55:10PM +0200, Michal Wajdeczko wrote: > In commit b839a869dfc9 ("drm/i915/guc: Add support for data > reporting in GuC responses") we missed the hypothetical case > that GuC might return positive non-zero value as success data. > > While that would be lucky treated as error case, and at the > end will result in reporting valid -EIO, in the meantime this > value will be passed to ERR_PTR that could be misleading. > > v2: rebased > > Reported-by: Dan Carpenter <dan.carpenter@oracle.com> > Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com> > Cc: Dan Carpenter <dan.carpenter@oracle.com> Return value where all integers are possible is always a bit fragile, especially here where the meaning additionally depends upon whether you supply a reply buffer or not. Would be good to document this with some kerneldoc, but maybe the CTB interface is a bit too unclear here and that's not worth it (there's at least a ton of functions/variants that just arent used). Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> > --- > drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c | 10 ++++++++-- > 1 file changed, 8 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c > index 43409044528e..a26bb55c0898 100644 > --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c > +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c > @@ -148,12 +148,15 @@ static int guc_action_register_ct_buffer(struct intel_guc *guc, u32 type, > FIELD_PREP(HOST2GUC_REGISTER_CTB_REQUEST_MSG_2_DESC_ADDR, desc_addr), > FIELD_PREP(HOST2GUC_REGISTER_CTB_REQUEST_MSG_3_BUFF_ADDR, buff_addr), > }; > + int ret; > > GEM_BUG_ON(type != GUC_CTB_TYPE_HOST2GUC && type != GUC_CTB_TYPE_GUC2HOST); > GEM_BUG_ON(size % SZ_4K); > > /* CT registration must go over MMIO */ > - return intel_guc_send_mmio(guc, request, ARRAY_SIZE(request), NULL, 0); > + ret = intel_guc_send_mmio(guc, request, ARRAY_SIZE(request), NULL, 0); > + > + return ret > 0 ? -EPROTO : ret; > } > > static int ct_register_buffer(struct intel_guc_ct *ct, u32 type, > @@ -177,11 +180,14 @@ static int guc_action_deregister_ct_buffer(struct intel_guc *guc, u32 type) > FIELD_PREP(GUC_HXG_REQUEST_MSG_0_ACTION, GUC_ACTION_HOST2GUC_DEREGISTER_CTB), > FIELD_PREP(HOST2GUC_DEREGISTER_CTB_REQUEST_MSG_1_TYPE, type), > }; > + int ret; > > GEM_BUG_ON(type != GUC_CTB_TYPE_HOST2GUC && type != GUC_CTB_TYPE_GUC2HOST); > > /* CT deregistration must go over MMIO */ > - return intel_guc_send_mmio(guc, request, ARRAY_SIZE(request), NULL, 0); > + ret = intel_guc_send_mmio(guc, request, ARRAY_SIZE(request), NULL, 0); > + > + return ret > 0 ? -EPROTO : ret; > } > > static int ct_deregister_buffer(struct intel_guc_ct *ct, u32 type) > -- > 2.25.1 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ^ permalink raw reply [flat|nested] 15+ messages in thread
* [Intel-gfx] [PATCH 2/4] drm/i915/guc: Print error name on CTB (de)registration failure 2021-07-01 15:55 [Intel-gfx] [PATCH 0/4] drm/i915/guc: Improve CTB error handling Michal Wajdeczko 2021-07-01 15:55 ` [Intel-gfx] [PATCH 1/4] drm/i915/guc: Verify result from CTB (de)register action Michal Wajdeczko @ 2021-07-01 15:55 ` Michal Wajdeczko 2021-08-18 14:20 ` Daniel Vetter 2021-07-01 15:55 ` [Intel-gfx] [PATCH 3/4] drm/i915/guc: Print error name on CTB send failure Michal Wajdeczko ` (3 subsequent siblings) 5 siblings, 1 reply; 15+ messages in thread From: Michal Wajdeczko @ 2021-07-01 15:55 UTC (permalink / raw) To: intel-gfx, dri-devel Instead of plain error value (%d) print more user friendly error name (%pe). Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com> --- drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c index a26bb55c0898..18d52c39f0c2 100644 --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c @@ -167,8 +167,8 @@ static int ct_register_buffer(struct intel_guc_ct *ct, u32 type, err = guc_action_register_ct_buffer(ct_to_guc(ct), type, desc_addr, buff_addr, size); if (unlikely(err)) - CT_ERROR(ct, "Failed to register %s buffer (err=%d)\n", - guc_ct_buffer_type_to_str(type), err); + CT_ERROR(ct, "Failed to register %s buffer (%pe)\n", + guc_ct_buffer_type_to_str(type), ERR_PTR(err)); return err; } @@ -195,8 +195,8 @@ static int ct_deregister_buffer(struct intel_guc_ct *ct, u32 type) int err = guc_action_deregister_ct_buffer(ct_to_guc(ct), type); if (unlikely(err)) - CT_ERROR(ct, "Failed to deregister %s buffer (err=%d)\n", - guc_ct_buffer_type_to_str(type), err); + CT_ERROR(ct, "Failed to deregister %s buffer (%pe)\n", + guc_ct_buffer_type_to_str(type), ERR_PTR(err)); return err; } -- 2.25.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [PATCH 2/4] drm/i915/guc: Print error name on CTB (de)registration failure 2021-07-01 15:55 ` [Intel-gfx] [PATCH 2/4] drm/i915/guc: Print error name on CTB (de)registration failure Michal Wajdeczko @ 2021-08-18 14:20 ` Daniel Vetter 2021-08-18 15:11 ` Michal Wajdeczko 0 siblings, 1 reply; 15+ messages in thread From: Daniel Vetter @ 2021-08-18 14:20 UTC (permalink / raw) To: Michal Wajdeczko; +Cc: intel-gfx, dri-devel On Thu, Jul 01, 2021 at 05:55:11PM +0200, Michal Wajdeczko wrote: > Instead of plain error value (%d) print more user friendly error > name (%pe). > > Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com> > --- > drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c > index a26bb55c0898..18d52c39f0c2 100644 > --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c > +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c > @@ -167,8 +167,8 @@ static int ct_register_buffer(struct intel_guc_ct *ct, u32 type, > err = guc_action_register_ct_buffer(ct_to_guc(ct), type, > desc_addr, buff_addr, size); > if (unlikely(err)) > - CT_ERROR(ct, "Failed to register %s buffer (err=%d)\n", > - guc_ct_buffer_type_to_str(type), err); > + CT_ERROR(ct, "Failed to register %s buffer (%pe)\n", > + guc_ct_buffer_type_to_str(type), ERR_PTR(err)); errname() is what you want here, not this convoluted jumping through hoops to fake an error pointer. With that: Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> > return err; > } > > @@ -195,8 +195,8 @@ static int ct_deregister_buffer(struct intel_guc_ct *ct, u32 type) > int err = guc_action_deregister_ct_buffer(ct_to_guc(ct), type); > > if (unlikely(err)) > - CT_ERROR(ct, "Failed to deregister %s buffer (err=%d)\n", > - guc_ct_buffer_type_to_str(type), err); > + CT_ERROR(ct, "Failed to deregister %s buffer (%pe)\n", > + guc_ct_buffer_type_to_str(type), ERR_PTR(err)); > return err; > } > > -- > 2.25.1 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [PATCH 2/4] drm/i915/guc: Print error name on CTB (de)registration failure 2021-08-18 14:20 ` Daniel Vetter @ 2021-08-18 15:11 ` Michal Wajdeczko 2021-08-18 16:35 ` Daniel Vetter 0 siblings, 1 reply; 15+ messages in thread From: Michal Wajdeczko @ 2021-08-18 15:11 UTC (permalink / raw) To: Daniel Vetter; +Cc: intel-gfx, dri-devel On 18.08.2021 16:20, Daniel Vetter wrote: > On Thu, Jul 01, 2021 at 05:55:11PM +0200, Michal Wajdeczko wrote: >> Instead of plain error value (%d) print more user friendly error >> name (%pe). >> >> Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com> >> --- >> drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c | 8 ++++---- >> 1 file changed, 4 insertions(+), 4 deletions(-) >> >> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c >> index a26bb55c0898..18d52c39f0c2 100644 >> --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c >> +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c >> @@ -167,8 +167,8 @@ static int ct_register_buffer(struct intel_guc_ct *ct, u32 type, >> err = guc_action_register_ct_buffer(ct_to_guc(ct), type, >> desc_addr, buff_addr, size); >> if (unlikely(err)) >> - CT_ERROR(ct, "Failed to register %s buffer (err=%d)\n", >> - guc_ct_buffer_type_to_str(type), err); >> + CT_ERROR(ct, "Failed to register %s buffer (%pe)\n", >> + guc_ct_buffer_type_to_str(type), ERR_PTR(err)); > > errname() is what you want here, not this convoluted jumping through hoops > to fake an error pointer. nope, I was already trying that shortcut, but errname() is not exported so we fail with ERROR: modpost: "errname" [drivers/gpu/drm/i915/i915.ko] undefined! so unless we add that export we must follow initial approach [1] -Michal [1] https://cgit.freedesktop.org/drm/drm-tip/commit/?id=57f5677e535ba24b8926a7125be2ef8d7f09323c > > With that: Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> >> return err; >> } >> >> @@ -195,8 +195,8 @@ static int ct_deregister_buffer(struct intel_guc_ct *ct, u32 type) >> int err = guc_action_deregister_ct_buffer(ct_to_guc(ct), type); >> >> if (unlikely(err)) >> - CT_ERROR(ct, "Failed to deregister %s buffer (err=%d)\n", >> - guc_ct_buffer_type_to_str(type), err); >> + CT_ERROR(ct, "Failed to deregister %s buffer (%pe)\n", >> + guc_ct_buffer_type_to_str(type), ERR_PTR(err)); >> return err; >> } >> >> -- >> 2.25.1 >> >> _______________________________________________ >> Intel-gfx mailing list >> Intel-gfx@lists.freedesktop.org >> https://lists.freedesktop.org/mailman/listinfo/intel-gfx > ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [PATCH 2/4] drm/i915/guc: Print error name on CTB (de)registration failure 2021-08-18 15:11 ` Michal Wajdeczko @ 2021-08-18 16:35 ` Daniel Vetter 2021-08-18 19:12 ` Michal Wajdeczko 0 siblings, 1 reply; 15+ messages in thread From: Daniel Vetter @ 2021-08-18 16:35 UTC (permalink / raw) To: Michal Wajdeczko; +Cc: intel-gfx, dri-devel On Wed, Aug 18, 2021 at 5:11 PM Michal Wajdeczko <michal.wajdeczko@intel.com> wrote: > > > > On 18.08.2021 16:20, Daniel Vetter wrote: > > On Thu, Jul 01, 2021 at 05:55:11PM +0200, Michal Wajdeczko wrote: > >> Instead of plain error value (%d) print more user friendly error > >> name (%pe). > >> > >> Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com> > >> --- > >> drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c | 8 ++++---- > >> 1 file changed, 4 insertions(+), 4 deletions(-) > >> > >> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c > >> index a26bb55c0898..18d52c39f0c2 100644 > >> --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c > >> +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c > >> @@ -167,8 +167,8 @@ static int ct_register_buffer(struct intel_guc_ct *ct, u32 type, > >> err = guc_action_register_ct_buffer(ct_to_guc(ct), type, > >> desc_addr, buff_addr, size); > >> if (unlikely(err)) > >> - CT_ERROR(ct, "Failed to register %s buffer (err=%d)\n", > >> - guc_ct_buffer_type_to_str(type), err); > >> + CT_ERROR(ct, "Failed to register %s buffer (%pe)\n", > >> + guc_ct_buffer_type_to_str(type), ERR_PTR(err)); > > > > errname() is what you want here, not this convoluted jumping through hoops > > to fake an error pointer. > > nope, I was already trying that shortcut, but errname() is not exported > so we fail with > > ERROR: modpost: "errname" [drivers/gpu/drm/i915/i915.ko] undefined! > > so unless we add that export we must follow initial approach [1] Then we export that function. This is all open source, we can actually fix things up, there should _never_ be a need to hack around things like this. And yes I'm keenly aware that there's a pile of i915-gem code which outright flaunts this principle, but that doesn't mean we should continue with that. scripts/get_maintainers.pl can help with finding all the people you need to cc on that export patch. -Daniel > > -Michal > > [1] > https://cgit.freedesktop.org/drm/drm-tip/commit/?id=57f5677e535ba24b8926a7125be2ef8d7f09323c > > > > > With that: Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> > >> return err; > >> } > >> > >> @@ -195,8 +195,8 @@ static int ct_deregister_buffer(struct intel_guc_ct *ct, u32 type) > >> int err = guc_action_deregister_ct_buffer(ct_to_guc(ct), type); > >> > >> if (unlikely(err)) > >> - CT_ERROR(ct, "Failed to deregister %s buffer (err=%d)\n", > >> - guc_ct_buffer_type_to_str(type), err); > >> + CT_ERROR(ct, "Failed to deregister %s buffer (%pe)\n", > >> + guc_ct_buffer_type_to_str(type), ERR_PTR(err)); > >> return err; > >> } > >> > >> -- > >> 2.25.1 > >> > >> _______________________________________________ > >> Intel-gfx mailing list > >> Intel-gfx@lists.freedesktop.org > >> https://lists.freedesktop.org/mailman/listinfo/intel-gfx > > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [PATCH 2/4] drm/i915/guc: Print error name on CTB (de)registration failure 2021-08-18 16:35 ` Daniel Vetter @ 2021-08-18 19:12 ` Michal Wajdeczko 2021-08-19 8:35 ` Daniel Vetter 0 siblings, 1 reply; 15+ messages in thread From: Michal Wajdeczko @ 2021-08-18 19:12 UTC (permalink / raw) To: Daniel Vetter; +Cc: intel-gfx, dri-devel On 18.08.2021 18:35, Daniel Vetter wrote: > On Wed, Aug 18, 2021 at 5:11 PM Michal Wajdeczko > <michal.wajdeczko@intel.com> wrote: >> >> >> >> On 18.08.2021 16:20, Daniel Vetter wrote: >>> On Thu, Jul 01, 2021 at 05:55:11PM +0200, Michal Wajdeczko wrote: >>>> Instead of plain error value (%d) print more user friendly error >>>> name (%pe). >>>> >>>> Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com> >>>> --- >>>> drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c | 8 ++++---- >>>> 1 file changed, 4 insertions(+), 4 deletions(-) >>>> >>>> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c >>>> index a26bb55c0898..18d52c39f0c2 100644 >>>> --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c >>>> +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c >>>> @@ -167,8 +167,8 @@ static int ct_register_buffer(struct intel_guc_ct *ct, u32 type, >>>> err = guc_action_register_ct_buffer(ct_to_guc(ct), type, >>>> desc_addr, buff_addr, size); >>>> if (unlikely(err)) >>>> - CT_ERROR(ct, "Failed to register %s buffer (err=%d)\n", >>>> - guc_ct_buffer_type_to_str(type), err); >>>> + CT_ERROR(ct, "Failed to register %s buffer (%pe)\n", >>>> + guc_ct_buffer_type_to_str(type), ERR_PTR(err)); >>> >>> errname() is what you want here, not this convoluted jumping through hoops >>> to fake an error pointer. >> >> nope, I was already trying that shortcut, but errname() is not exported >> so we fail with >> >> ERROR: modpost: "errname" [drivers/gpu/drm/i915/i915.ko] undefined! >> >> so unless we add that export we must follow initial approach [1] > > Then we export that function. This is all open source, we can actually > fix things up, there should _never_ be a need to hack around things > like this. simple export might be not sufficient, as this function returns NULL for unrecognized error codes, and it might be hard to print that code in plain format, as it %pe does it for us for free. is that big problem to use ERR_PTR? I'm not the only/first one see drivers/net/can/usb/etas_es58x/es58x_core.c drivers/net/ethernet/freescale/enetc/enetc_pf.c drivers/net/phy/phylink.c ... > > And yes I'm keenly aware that there's a pile of i915-gem code which > outright flaunts this principle, but that doesn't mean we should > continue with that. scripts/get_maintainers.pl can help with finding > all the people you need to cc on that export patch. I'm perfectly fine with updating/fixing shared code (did that before, have few more ideas on my todo-list) but in this case I'm not so sure -Michal > -Daniel > >> >> -Michal >> >> [1] >> https://cgit.freedesktop.org/drm/drm-tip/commit/?id=57f5677e535ba24b8926a7125be2ef8d7f09323c >> >>> >>> With that: Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> >>>> return err; >>>> } >>>> >>>> @@ -195,8 +195,8 @@ static int ct_deregister_buffer(struct intel_guc_ct *ct, u32 type) >>>> int err = guc_action_deregister_ct_buffer(ct_to_guc(ct), type); >>>> >>>> if (unlikely(err)) >>>> - CT_ERROR(ct, "Failed to deregister %s buffer (err=%d)\n", >>>> - guc_ct_buffer_type_to_str(type), err); >>>> + CT_ERROR(ct, "Failed to deregister %s buffer (%pe)\n", >>>> + guc_ct_buffer_type_to_str(type), ERR_PTR(err)); >>>> return err; >>>> } >>>> >>>> -- >>>> 2.25.1 >>>> >>>> _______________________________________________ >>>> Intel-gfx mailing list >>>> Intel-gfx@lists.freedesktop.org >>>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx >>> > > > ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [PATCH 2/4] drm/i915/guc: Print error name on CTB (de)registration failure 2021-08-18 19:12 ` Michal Wajdeczko @ 2021-08-19 8:35 ` Daniel Vetter 0 siblings, 0 replies; 15+ messages in thread From: Daniel Vetter @ 2021-08-19 8:35 UTC (permalink / raw) To: Michal Wajdeczko; +Cc: Daniel Vetter, intel-gfx, dri-devel On Wed, Aug 18, 2021 at 09:12:32PM +0200, Michal Wajdeczko wrote: > > > On 18.08.2021 18:35, Daniel Vetter wrote: > > On Wed, Aug 18, 2021 at 5:11 PM Michal Wajdeczko > > <michal.wajdeczko@intel.com> wrote: > >> > >> > >> > >> On 18.08.2021 16:20, Daniel Vetter wrote: > >>> On Thu, Jul 01, 2021 at 05:55:11PM +0200, Michal Wajdeczko wrote: > >>>> Instead of plain error value (%d) print more user friendly error > >>>> name (%pe). > >>>> > >>>> Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com> > >>>> --- > >>>> drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c | 8 ++++---- > >>>> 1 file changed, 4 insertions(+), 4 deletions(-) > >>>> > >>>> diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c > >>>> index a26bb55c0898..18d52c39f0c2 100644 > >>>> --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c > >>>> +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c > >>>> @@ -167,8 +167,8 @@ static int ct_register_buffer(struct intel_guc_ct *ct, u32 type, > >>>> err = guc_action_register_ct_buffer(ct_to_guc(ct), type, > >>>> desc_addr, buff_addr, size); > >>>> if (unlikely(err)) > >>>> - CT_ERROR(ct, "Failed to register %s buffer (err=%d)\n", > >>>> - guc_ct_buffer_type_to_str(type), err); > >>>> + CT_ERROR(ct, "Failed to register %s buffer (%pe)\n", > >>>> + guc_ct_buffer_type_to_str(type), ERR_PTR(err)); > >>> > >>> errname() is what you want here, not this convoluted jumping through hoops > >>> to fake an error pointer. > >> > >> nope, I was already trying that shortcut, but errname() is not exported > >> so we fail with > >> > >> ERROR: modpost: "errname" [drivers/gpu/drm/i915/i915.ko] undefined! > >> > >> so unless we add that export we must follow initial approach [1] > > > > Then we export that function. This is all open source, we can actually > > fix things up, there should _never_ be a need to hack around things > > like this. > > simple export might be not sufficient, as this function returns NULL for > unrecognized error codes, and it might be hard to print that code in > plain format, as it %pe does it for us for free. "(%s:%i)", errname(ret), ret Would work, but maybe not so pretty. Otoh %pe for unrecognized integers is also not very pretty. > is that big problem to use ERR_PTR? I'm not the only/first one > > see > drivers/net/can/usb/etas_es58x/es58x_core.c > drivers/net/ethernet/freescale/enetc/enetc_pf.c > drivers/net/phy/phylink.c > ... Ah yeah, looking through grep more than half the users do this pattern. Which still feels extremely silly, because it's not a pointer we're printing, and when it's not an errno we should probably print it as an integer or something. But also meh. On both patches, as-is: Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> > > And yes I'm keenly aware that there's a pile of i915-gem code which > > outright flaunts this principle, but that doesn't mean we should > > continue with that. scripts/get_maintainers.pl can help with finding > > all the people you need to cc on that export patch. > > I'm perfectly fine with updating/fixing shared code (did that before, > have few more ideas on my todo-list) but in this case I'm not so sure I think an %ie extension or something like that for printk would make some sense. There's absolute enormous amounts of this kind of casting going on, and it just doesn't make sense to me. It would be pretty easy way to get like 100 patches into the kernel to clean it all up :-) -Daniel > > -Michal > > > -Daniel > > > >> > >> -Michal > >> > >> [1] > >> https://cgit.freedesktop.org/drm/drm-tip/commit/?id=57f5677e535ba24b8926a7125be2ef8d7f09323c > >> > >>> > >>> With that: Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> > >>>> return err; > >>>> } > >>>> > >>>> @@ -195,8 +195,8 @@ static int ct_deregister_buffer(struct intel_guc_ct *ct, u32 type) > >>>> int err = guc_action_deregister_ct_buffer(ct_to_guc(ct), type); > >>>> > >>>> if (unlikely(err)) > >>>> - CT_ERROR(ct, "Failed to deregister %s buffer (err=%d)\n", > >>>> - guc_ct_buffer_type_to_str(type), err); > >>>> + CT_ERROR(ct, "Failed to deregister %s buffer (%pe)\n", > >>>> + guc_ct_buffer_type_to_str(type), ERR_PTR(err)); > >>>> return err; > >>>> } > >>>> > >>>> -- > >>>> 2.25.1 > >>>> > >>>> _______________________________________________ > >>>> Intel-gfx mailing list > >>>> Intel-gfx@lists.freedesktop.org > >>>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx > >>> > > > > > > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ^ permalink raw reply [flat|nested] 15+ messages in thread
* [Intel-gfx] [PATCH 3/4] drm/i915/guc: Print error name on CTB send failure 2021-07-01 15:55 [Intel-gfx] [PATCH 0/4] drm/i915/guc: Improve CTB error handling Michal Wajdeczko 2021-07-01 15:55 ` [Intel-gfx] [PATCH 1/4] drm/i915/guc: Verify result from CTB (de)register action Michal Wajdeczko 2021-07-01 15:55 ` [Intel-gfx] [PATCH 2/4] drm/i915/guc: Print error name on CTB (de)registration failure Michal Wajdeczko @ 2021-07-01 15:55 ` Michal Wajdeczko 2021-08-18 14:22 ` Daniel Vetter 2021-07-01 15:55 ` [Intel-gfx] [PATCH 4/4] drm/i915/guc: Move and improve error message for missed CTB reply Michal Wajdeczko ` (2 subsequent siblings) 5 siblings, 1 reply; 15+ messages in thread From: Michal Wajdeczko @ 2021-07-01 15:55 UTC (permalink / raw) To: intel-gfx, dri-devel Instead of plain error value (%d) print more user friendly error name (%pe). Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com> --- drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c index 18d52c39f0c2..8110586ce1fd 100644 --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c @@ -580,8 +580,8 @@ int intel_guc_ct_send(struct intel_guc_ct *ct, const u32 *action, u32 len, ret = ct_send(ct, action, len, response_buf, response_buf_size, &status); if (unlikely(ret < 0)) { - CT_ERROR(ct, "Sending action %#x failed (err=%d status=%#X)\n", - action[0], ret, status); + CT_ERROR(ct, "Sending action %#x failed (%pe) status=%#X\n", + action[0], ERR_PTR(ret), status); } else if (unlikely(ret)) { CT_DEBUG(ct, "send action %#x returned %d (%#x)\n", action[0], ret, ret); -- 2.25.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [PATCH 3/4] drm/i915/guc: Print error name on CTB send failure 2021-07-01 15:55 ` [Intel-gfx] [PATCH 3/4] drm/i915/guc: Print error name on CTB send failure Michal Wajdeczko @ 2021-08-18 14:22 ` Daniel Vetter 0 siblings, 0 replies; 15+ messages in thread From: Daniel Vetter @ 2021-08-18 14:22 UTC (permalink / raw) To: Michal Wajdeczko; +Cc: intel-gfx, dri-devel On Thu, Jul 01, 2021 at 05:55:12PM +0200, Michal Wajdeczko wrote: > Instead of plain error value (%d) print more user friendly error > name (%pe). > > Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com> > --- > drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c > index 18d52c39f0c2..8110586ce1fd 100644 > --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c > +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c > @@ -580,8 +580,8 @@ int intel_guc_ct_send(struct intel_guc_ct *ct, const u32 *action, u32 len, > > ret = ct_send(ct, action, len, response_buf, response_buf_size, &status); > if (unlikely(ret < 0)) { > - CT_ERROR(ct, "Sending action %#x failed (err=%d status=%#X)\n", > - action[0], ret, status); > + CT_ERROR(ct, "Sending action %#x failed (%pe) status=%#X\n", > + action[0], ERR_PTR(ret), status); errname(), not this, with that: Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> > } else if (unlikely(ret)) { > CT_DEBUG(ct, "send action %#x returned %d (%#x)\n", > action[0], ret, ret); > -- > 2.25.1 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ^ permalink raw reply [flat|nested] 15+ messages in thread
* [Intel-gfx] [PATCH 4/4] drm/i915/guc: Move and improve error message for missed CTB reply 2021-07-01 15:55 [Intel-gfx] [PATCH 0/4] drm/i915/guc: Improve CTB error handling Michal Wajdeczko ` (2 preceding siblings ...) 2021-07-01 15:55 ` [Intel-gfx] [PATCH 3/4] drm/i915/guc: Print error name on CTB send failure Michal Wajdeczko @ 2021-07-01 15:55 ` Michal Wajdeczko 2021-08-18 14:24 ` Daniel Vetter 2021-07-01 20:31 ` [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/guc: Improve CTB error handling Patchwork 2021-07-02 1:31 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork 5 siblings, 1 reply; 15+ messages in thread From: Michal Wajdeczko @ 2021-07-01 15:55 UTC (permalink / raw) To: intel-gfx, dri-devel If we timeout waiting for a CT reply we print very simple error message. Improve that and by moving error reporting to the caller we can use CT_ERROR instead of DRM_ERROR and report just fence as error code will be reported later anyway. Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com> --- drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c index 8110586ce1fd..f488a51e1ebe 100644 --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c @@ -490,9 +490,6 @@ static int wait_for_ct_request_update(struct ct_request *req, u32 *status) err = wait_for(done, 10); #undef done - if (unlikely(err)) - DRM_ERROR("CT: fence %u err %d\n", req->fence, err); - *status = req->status; return err; } @@ -536,8 +533,11 @@ static int ct_send(struct intel_guc_ct *ct, intel_guc_notify(ct_to_guc(ct)); err = wait_for_ct_request_update(&request, status); - if (unlikely(err)) + if (unlikely(err)) { + CT_ERROR(ct, "No response for request %#x (fence %u)\n", + action[0], request.fence); goto unlink; + } if (FIELD_GET(GUC_HXG_MSG_0_TYPE, *status) != GUC_HXG_TYPE_RESPONSE_SUCCESS) { err = -EIO; -- 2.25.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [Intel-gfx] [PATCH 4/4] drm/i915/guc: Move and improve error message for missed CTB reply 2021-07-01 15:55 ` [Intel-gfx] [PATCH 4/4] drm/i915/guc: Move and improve error message for missed CTB reply Michal Wajdeczko @ 2021-08-18 14:24 ` Daniel Vetter 0 siblings, 0 replies; 15+ messages in thread From: Daniel Vetter @ 2021-08-18 14:24 UTC (permalink / raw) To: Michal Wajdeczko; +Cc: intel-gfx, dri-devel On Thu, Jul 01, 2021 at 05:55:13PM +0200, Michal Wajdeczko wrote: > If we timeout waiting for a CT reply we print very simple error > message. Improve that and by moving error reporting to the caller > we can use CT_ERROR instead of DRM_ERROR and report just fence > as error code will be reported later anyway. > > Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com> Look reasonable. Btw for within the driver we generally never document static inline functions with full kerneldoc. That's overkill and they get stale real fast. What would be useful to document is the interface with the driver at large (i.e. non-static functions), especially for something that's used all over like CTB will be. But then we're back to responsibilities and especialy aroung gpu reset, so not sure whether documenting the current code before that's sorted is the best idea. Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> > --- > drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c > index 8110586ce1fd..f488a51e1ebe 100644 > --- a/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c > +++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c > @@ -490,9 +490,6 @@ static int wait_for_ct_request_update(struct ct_request *req, u32 *status) > err = wait_for(done, 10); > #undef done > > - if (unlikely(err)) > - DRM_ERROR("CT: fence %u err %d\n", req->fence, err); > - > *status = req->status; > return err; > } > @@ -536,8 +533,11 @@ static int ct_send(struct intel_guc_ct *ct, > intel_guc_notify(ct_to_guc(ct)); > > err = wait_for_ct_request_update(&request, status); > - if (unlikely(err)) > + if (unlikely(err)) { > + CT_ERROR(ct, "No response for request %#x (fence %u)\n", > + action[0], request.fence); > goto unlink; > + } > > if (FIELD_GET(GUC_HXG_MSG_0_TYPE, *status) != GUC_HXG_TYPE_RESPONSE_SUCCESS) { > err = -EIO; > -- > 2.25.1 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ^ permalink raw reply [flat|nested] 15+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/guc: Improve CTB error handling 2021-07-01 15:55 [Intel-gfx] [PATCH 0/4] drm/i915/guc: Improve CTB error handling Michal Wajdeczko ` (3 preceding siblings ...) 2021-07-01 15:55 ` [Intel-gfx] [PATCH 4/4] drm/i915/guc: Move and improve error message for missed CTB reply Michal Wajdeczko @ 2021-07-01 20:31 ` Patchwork 2021-07-02 1:31 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork 5 siblings, 0 replies; 15+ messages in thread From: Patchwork @ 2021-07-01 20:31 UTC (permalink / raw) To: Michal Wajdeczko; +Cc: intel-gfx [-- Attachment #1.1: Type: text/plain, Size: 2595 bytes --] == Series Details == Series: drm/i915/guc: Improve CTB error handling URL : https://patchwork.freedesktop.org/series/92118/ State : success == Summary == CI Bug Log - changes from CI_DRM_10299 -> Patchwork_20511 ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/index.html Known issues ------------ Here are the changes found in Patchwork_20511 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@amdgpu/amd_basic@cs-gfx: - fi-kbl-soraka: NOTRUN -> [SKIP][1] ([fdo#109271]) +2 similar issues [1]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/fi-kbl-soraka/igt@amdgpu/amd_basic@cs-gfx.html * igt@amdgpu/amd_basic@query-info: - fi-bsw-kefka: NOTRUN -> [SKIP][2] ([fdo#109271]) +32 similar issues [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/fi-bsw-kefka/igt@amdgpu/amd_basic@query-info.html * igt@kms_chamelium@hdmi-edid-read: - fi-bsw-kefka: NOTRUN -> [SKIP][3] ([fdo#109271] / [fdo#111827]) +8 similar issues [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/fi-bsw-kefka/igt@kms_chamelium@hdmi-edid-read.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#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827 [i915#3717]: https://gitlab.freedesktop.org/drm/intel/issues/3717 Participating hosts (36 -> 35) ------------------------------ Additional (1): fi-bsw-kefka Missing (2): fi-bsw-cyan fi-bdw-samus Build changes ------------- * Linux: CI_DRM_10299 -> Patchwork_20511 CI-20190529: 20190529 CI_DRM_10299: 40a1952a06a94218e1bb751a3091a1c105173f1a @ git://anongit.freedesktop.org/gfx-ci/linux IGT_6126: 098d0b82276fa0595e3d0fd745885f9d2147035d @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git Patchwork_20511: db4a4850eae15de316aed98dc87ea3e0a9f60088 @ git://anongit.freedesktop.org/gfx-ci/linux == Linux commits == db4a4850eae1 drm/i915/guc: Move and improve error message for missed CTB reply 8b841ad277be drm/i915/guc: Print error name on CTB send failure 9d728956d405 drm/i915/guc: Print error name on CTB (de)registration failure 7a4060b10fcd drm/i915/guc: Verify result from CTB (de)register action == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/index.html [-- Attachment #1.2: Type: text/html, Size: 3346 bytes --] [-- Attachment #2: Type: text/plain, Size: 160 bytes --] _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
* [Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/guc: Improve CTB error handling 2021-07-01 15:55 [Intel-gfx] [PATCH 0/4] drm/i915/guc: Improve CTB error handling Michal Wajdeczko ` (4 preceding siblings ...) 2021-07-01 20:31 ` [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/guc: Improve CTB error handling Patchwork @ 2021-07-02 1:31 ` Patchwork 5 siblings, 0 replies; 15+ messages in thread From: Patchwork @ 2021-07-02 1:31 UTC (permalink / raw) To: Michal Wajdeczko; +Cc: intel-gfx [-- Attachment #1.1: Type: text/plain, Size: 25982 bytes --] == Series Details == Series: drm/i915/guc: Improve CTB error handling URL : https://patchwork.freedesktop.org/series/92118/ State : failure == Summary == CI Bug Log - changes from CI_DRM_10299_full -> Patchwork_20511_full ==================================================== Summary ------- **FAILURE** Serious unknown changes coming with Patchwork_20511_full absolutely need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in Patchwork_20511_full, please notify your bug team to allow them to document this new failure mode, which will reduce false positives in CI. Possible new issues ------------------- Here are the unknown changes that may have been introduced in Patchwork_20511_full: ### IGT changes ### #### Possible regressions #### * igt@gem_exec_reloc@basic-parallel: - shard-iclb: [PASS][1] -> [TIMEOUT][2] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-iclb5/igt@gem_exec_reloc@basic-parallel.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-iclb4/igt@gem_exec_reloc@basic-parallel.html * igt@kms_big_fb@y-tiled-max-hw-stride-64bpp-rotate-0-async-flip: - shard-skl: NOTRUN -> [FAIL][3] +3 similar issues [3]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl9/igt@kms_big_fb@y-tiled-max-hw-stride-64bpp-rotate-0-async-flip.html * igt@sysfs_timeslice_duration@timeout@vecs0: - shard-iclb: [PASS][4] -> [FAIL][5] [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-iclb4/igt@sysfs_timeslice_duration@timeout@vecs0.html [5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-iclb1/igt@sysfs_timeslice_duration@timeout@vecs0.html Known issues ------------ Here are the changes found in Patchwork_20511_full that come from known issues: ### IGT changes ### #### Issues hit #### * igt@core_auth@getclient-simple: - shard-glk: [PASS][6] -> [DMESG-WARN][7] ([i915#118] / [i915#95]) +1 similar issue [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-glk4/igt@core_auth@getclient-simple.html [7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-glk5/igt@core_auth@getclient-simple.html * igt@gem_create@create-clear: - shard-glk: [PASS][8] -> [FAIL][9] ([i915#3160]) [8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-glk4/igt@gem_create@create-clear.html [9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-glk5/igt@gem_create@create-clear.html * igt@gem_ctx_persistence@legacy-engines-hostile@render: - shard-glk: [PASS][10] -> [FAIL][11] ([i915#2410]) [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-glk8/igt@gem_ctx_persistence@legacy-engines-hostile@render.html [11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-glk6/igt@gem_ctx_persistence@legacy-engines-hostile@render.html * igt@gem_exec_fair@basic-deadline: - shard-skl: NOTRUN -> [FAIL][12] ([i915#2846]) [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl7/igt@gem_exec_fair@basic-deadline.html * igt@gem_exec_fair@basic-none@vcs1: - shard-iclb: NOTRUN -> [FAIL][13] ([i915#2842]) [13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-iclb2/igt@gem_exec_fair@basic-none@vcs1.html * igt@gem_huc_copy@huc-copy: - shard-apl: NOTRUN -> [SKIP][14] ([fdo#109271] / [i915#2190]) [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-apl6/igt@gem_huc_copy@huc-copy.html * igt@gen9_exec_parse@bb-large: - shard-apl: NOTRUN -> [FAIL][15] ([i915#3296]) [15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-apl6/igt@gen9_exec_parse@bb-large.html * igt@i915_selftest@live@hangcheck: - shard-snb: [PASS][16] -> [INCOMPLETE][17] ([i915#2782]) [16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-snb5/igt@i915_selftest@live@hangcheck.html [17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-snb6/igt@i915_selftest@live@hangcheck.html * igt@i915_suspend@debugfs-reader: - shard-skl: [PASS][18] -> [INCOMPLETE][19] ([i915#198]) [18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-skl1/igt@i915_suspend@debugfs-reader.html [19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl4/igt@i915_suspend@debugfs-reader.html * igt@i915_suspend@sysfs-reader: - shard-apl: [PASS][20] -> [DMESG-WARN][21] ([i915#180]) [20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-apl8/igt@i915_suspend@sysfs-reader.html [21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-apl8/igt@i915_suspend@sysfs-reader.html * igt@kms_color@pipe-b-ctm-0-25: - shard-skl: [PASS][22] -> [DMESG-WARN][23] ([i915#1982]) [22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-skl8/igt@kms_color@pipe-b-ctm-0-25.html [23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl8/igt@kms_color@pipe-b-ctm-0-25.html * igt@kms_color_chamelium@pipe-a-ctm-limited-range: - shard-apl: NOTRUN -> [SKIP][24] ([fdo#109271] / [fdo#111827]) +25 similar issues [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-apl7/igt@kms_color_chamelium@pipe-a-ctm-limited-range.html * igt@kms_color_chamelium@pipe-d-degamma: - shard-skl: NOTRUN -> [SKIP][25] ([fdo#109271] / [fdo#111827]) +12 similar issues [25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl2/igt@kms_color_chamelium@pipe-d-degamma.html * igt@kms_cursor_crc@pipe-c-cursor-256x85-random: - shard-apl: NOTRUN -> [FAIL][26] ([i915#3444]) [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-apl2/igt@kms_cursor_crc@pipe-c-cursor-256x85-random.html * igt@kms_cursor_legacy@flip-vs-cursor-toggle: - shard-skl: [PASS][27] -> [FAIL][28] ([i915#2346]) +1 similar issue [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-skl4/igt@kms_cursor_legacy@flip-vs-cursor-toggle.html [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl7/igt@kms_cursor_legacy@flip-vs-cursor-toggle.html * igt@kms_flip@flip-vs-expired-vblank@a-edp1: - shard-skl: [PASS][29] -> [FAIL][30] ([i915#79]) +1 similar issue [29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-skl6/igt@kms_flip@flip-vs-expired-vblank@a-edp1.html [30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl3/igt@kms_flip@flip-vs-expired-vblank@a-edp1.html * igt@kms_flip@flip-vs-suspend-interruptible@c-dp1: - shard-apl: NOTRUN -> [DMESG-WARN][31] ([i915#180]) +1 similar issue [31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-apl6/igt@kms_flip@flip-vs-suspend-interruptible@c-dp1.html * igt@kms_flip@plain-flip-fb-recreate-interruptible@a-edp1: - shard-skl: [PASS][32] -> [FAIL][33] ([i915#2122]) +2 similar issues [32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-skl7/igt@kms_flip@plain-flip-fb-recreate-interruptible@a-edp1.html [33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl2/igt@kms_flip@plain-flip-fb-recreate-interruptible@a-edp1.html * igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytilegen12rcccs: - shard-skl: NOTRUN -> [SKIP][34] ([fdo#109271] / [i915#2672]) [34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl2/igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytilegen12rcccs.html - shard-apl: NOTRUN -> [SKIP][35] ([fdo#109271] / [i915#2672]) [35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-apl3/igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytilegen12rcccs.html * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-indfb-msflip-blt: - shard-skl: NOTRUN -> [SKIP][36] ([fdo#109271]) +133 similar issues [36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl8/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-indfb-msflip-blt.html * igt@kms_pipe_crc_basic@hang-read-crc-pipe-d: - shard-skl: NOTRUN -> [SKIP][37] ([fdo#109271] / [i915#533]) [37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl2/igt@kms_pipe_crc_basic@hang-read-crc-pipe-d.html * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-d: - shard-apl: NOTRUN -> [SKIP][38] ([fdo#109271] / [i915#533]) +3 similar issues [38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-apl7/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-d.html * igt@kms_plane_alpha_blend@pipe-a-alpha-transparent-fb: - shard-skl: NOTRUN -> [FAIL][39] ([i915#265]) [39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl9/igt@kms_plane_alpha_blend@pipe-a-alpha-transparent-fb.html * igt@kms_plane_alpha_blend@pipe-b-alpha-transparent-fb: - shard-apl: NOTRUN -> [FAIL][40] ([i915#265]) [40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-apl7/igt@kms_plane_alpha_blend@pipe-b-alpha-transparent-fb.html * igt@kms_plane_alpha_blend@pipe-c-alpha-7efc: - shard-apl: NOTRUN -> [FAIL][41] ([fdo#108145] / [i915#265]) [41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-apl2/igt@kms_plane_alpha_blend@pipe-c-alpha-7efc.html * igt@kms_plane_alpha_blend@pipe-c-coverage-7efc: - shard-skl: [PASS][42] -> [FAIL][43] ([fdo#108145] / [i915#265]) +1 similar issue [42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-skl3/igt@kms_plane_alpha_blend@pipe-c-coverage-7efc.html [43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl6/igt@kms_plane_alpha_blend@pipe-c-coverage-7efc.html * igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-2: - shard-skl: NOTRUN -> [SKIP][44] ([fdo#109271] / [i915#658]) +1 similar issue [44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl9/igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-2.html * igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-5: - shard-apl: NOTRUN -> [SKIP][45] ([fdo#109271] / [i915#658]) +4 similar issues [45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-apl7/igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-5.html * igt@kms_psr@psr2_no_drrs: - shard-iclb: [PASS][46] -> [SKIP][47] ([fdo#109441]) [46]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-iclb2/igt@kms_psr@psr2_no_drrs.html [47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-iclb3/igt@kms_psr@psr2_no_drrs.html * igt@kms_writeback@writeback-check-output: - shard-apl: NOTRUN -> [SKIP][48] ([fdo#109271] / [i915#2437]) [48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-apl7/igt@kms_writeback@writeback-check-output.html * igt@nouveau_crc@pipe-b-ctx-flip-skip-current-frame: - shard-apl: NOTRUN -> [SKIP][49] ([fdo#109271]) +291 similar issues [49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-apl2/igt@nouveau_crc@pipe-b-ctx-flip-skip-current-frame.html * igt@sysfs_clients@recycle-many: - shard-apl: NOTRUN -> [SKIP][50] ([fdo#109271] / [i915#2994]) +2 similar issues [50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-apl2/igt@sysfs_clients@recycle-many.html * igt@sysfs_clients@split-50: - shard-skl: NOTRUN -> [SKIP][51] ([fdo#109271] / [i915#2994]) [51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl9/igt@sysfs_clients@split-50.html #### Possible fixes #### * igt@gem_ctx_persistence@legacy-engines-hostile@render: - shard-skl: [FAIL][52] ([i915#2410]) -> [PASS][53] [52]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-skl1/igt@gem_ctx_persistence@legacy-engines-hostile@render.html [53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl4/igt@gem_ctx_persistence@legacy-engines-hostile@render.html * igt@gem_ctx_persistence@many-contexts: - shard-tglb: [FAIL][54] ([i915#2410]) -> [PASS][55] [54]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-tglb5/igt@gem_ctx_persistence@many-contexts.html [55]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-tglb2/igt@gem_ctx_persistence@many-contexts.html * igt@gem_exec_fair@basic-deadline: - shard-glk: [FAIL][56] ([i915#2846]) -> [PASS][57] [56]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-glk8/igt@gem_exec_fair@basic-deadline.html [57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-glk2/igt@gem_exec_fair@basic-deadline.html * igt@gem_exec_fair@basic-pace-solo@rcs0: - shard-glk: [FAIL][58] ([i915#2842]) -> [PASS][59] +1 similar issue [58]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-glk3/igt@gem_exec_fair@basic-pace-solo@rcs0.html [59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-glk7/igt@gem_exec_fair@basic-pace-solo@rcs0.html * igt@gem_exec_fair@basic-pace@rcs0: - shard-kbl: [SKIP][60] ([fdo#109271]) -> [PASS][61] [60]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-kbl6/igt@gem_exec_fair@basic-pace@rcs0.html [61]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-kbl2/igt@gem_exec_fair@basic-pace@rcs0.html - shard-tglb: [FAIL][62] ([i915#2842]) -> [PASS][63] [62]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-tglb7/igt@gem_exec_fair@basic-pace@rcs0.html [63]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-tglb1/igt@gem_exec_fair@basic-pace@rcs0.html * igt@gem_exec_fair@basic-pace@vcs1: - shard-kbl: [FAIL][64] ([i915#2842]) -> [PASS][65] +2 similar issues [64]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-kbl6/igt@gem_exec_fair@basic-pace@vcs1.html [65]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-kbl2/igt@gem_exec_fair@basic-pace@vcs1.html * igt@gem_mmap_gtt@cpuset-medium-copy-xy: - shard-iclb: [FAIL][66] ([i915#307]) -> [PASS][67] [66]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-iclb1/igt@gem_mmap_gtt@cpuset-medium-copy-xy.html [67]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-iclb7/igt@gem_mmap_gtt@cpuset-medium-copy-xy.html * igt@gen9_exec_parse@allowed-single: - shard-skl: [DMESG-WARN][68] ([i915#1436] / [i915#716]) -> [PASS][69] [68]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-skl3/igt@gen9_exec_parse@allowed-single.html [69]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl9/igt@gen9_exec_parse@allowed-single.html * igt@kms_async_flips@alternate-sync-async-flip: - shard-skl: [FAIL][70] ([i915#2521]) -> [PASS][71] [70]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-skl6/igt@kms_async_flips@alternate-sync-async-flip.html [71]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl3/igt@kms_async_flips@alternate-sync-async-flip.html * igt@kms_cursor_crc@pipe-b-cursor-64x21-sliding: - shard-skl: [FAIL][72] ([i915#3444]) -> [PASS][73] [72]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-skl1/igt@kms_cursor_crc@pipe-b-cursor-64x21-sliding.html [73]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl4/igt@kms_cursor_crc@pipe-b-cursor-64x21-sliding.html * igt@kms_cursor_edge_walk@pipe-b-64x64-right-edge: - shard-skl: [DMESG-WARN][74] ([i915#1982]) -> [PASS][75] +1 similar issue [74]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-skl1/igt@kms_cursor_edge_walk@pipe-b-64x64-right-edge.html [75]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl4/igt@kms_cursor_edge_walk@pipe-b-64x64-right-edge.html * igt@kms_flip@plain-flip-ts-check@a-hdmi-a1: - shard-glk: [FAIL][76] ([i915#2122]) -> [PASS][77] [76]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-glk9/igt@kms_flip@plain-flip-ts-check@a-hdmi-a1.html [77]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-glk8/igt@kms_flip@plain-flip-ts-check@a-hdmi-a1.html * igt@kms_frontbuffer_tracking@psr-farfromfence-mmap-gtt: - shard-skl: [FAIL][78] ([i915#2546]) -> [PASS][79] [78]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-skl9/igt@kms_frontbuffer_tracking@psr-farfromfence-mmap-gtt.html [79]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl5/igt@kms_frontbuffer_tracking@psr-farfromfence-mmap-gtt.html * igt@kms_hdr@bpc-switch-suspend: - shard-skl: [FAIL][80] ([i915#1188]) -> [PASS][81] [80]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-skl10/igt@kms_hdr@bpc-switch-suspend.html [81]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl10/igt@kms_hdr@bpc-switch-suspend.html * igt@kms_plane_alpha_blend@pipe-b-coverage-7efc: - shard-skl: [FAIL][82] ([fdo#108145] / [i915#265]) -> [PASS][83] [82]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-skl4/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html [83]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl7/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html * igt@kms_plane_alpha_blend@pipe-d-alpha-basic: - shard-tglb: [DMESG-WARN][84] ([i915#2868]) -> [PASS][85] [84]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-tglb5/igt@kms_plane_alpha_blend@pipe-d-alpha-basic.html [85]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-tglb2/igt@kms_plane_alpha_blend@pipe-d-alpha-basic.html * igt@kms_psr@psr2_dpms: - shard-iclb: [SKIP][86] ([fdo#109441]) -> [PASS][87] [86]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-iclb5/igt@kms_psr@psr2_dpms.html [87]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-iclb2/igt@kms_psr@psr2_dpms.html #### Warnings #### * igt@gem_exec_fair@basic-none-rrul@rcs0: - shard-iclb: [FAIL][88] ([i915#2842]) -> [FAIL][89] ([i915#2852]) [88]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-iclb8/igt@gem_exec_fair@basic-none-rrul@rcs0.html [89]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-iclb6/igt@gem_exec_fair@basic-none-rrul@rcs0.html * igt@i915_pm_dc@dc9-dpms: - shard-skl: [INCOMPLETE][90] ([i915#198]) -> [SKIP][91] ([fdo#109271]) [90]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-skl9/igt@i915_pm_dc@dc9-dpms.html [91]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl7/igt@i915_pm_dc@dc9-dpms.html * igt@i915_pm_rc6_residency@rc6-idle: - shard-tglb: [FAIL][92] ([i915#2681]) -> [WARN][93] ([i915#2681] / [i915#2684]) [92]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-tglb2/igt@i915_pm_rc6_residency@rc6-idle.html [93]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-tglb2/igt@i915_pm_rc6_residency@rc6-idle.html * igt@kms_dp_dsc@basic-dsc-enable-edp: - shard-iclb: [DMESG-WARN][94] ([i915#1226]) -> [SKIP][95] ([fdo#109349]) [94]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-iclb2/igt@kms_dp_dsc@basic-dsc-enable-edp.html [95]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-iclb3/igt@kms_dp_dsc@basic-dsc-enable-edp.html * igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-3: - shard-iclb: [SKIP][96] ([i915#2920]) -> [SKIP][97] ([i915#658]) +2 similar issues [96]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-iclb2/igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-3.html [97]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-iclb8/igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-3.html * igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-3: - shard-iclb: [SKIP][98] ([i915#658]) -> [SKIP][99] ([i915#2920]) [98]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-iclb8/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-3.html [99]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-iclb2/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-3.html * igt@runner@aborted: - shard-apl: ([FAIL][100], [FAIL][101], [FAIL][102], [FAIL][103], [FAIL][104]) ([fdo#109271] / [i915#180] / [i915#1814] / [i915#3002] / [i915#3363]) -> ([FAIL][105], [FAIL][106], [FAIL][107]) ([i915#180] / [i915#3002] / [i915#3363]) [100]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-apl1/igt@runner@aborted.html [101]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-apl1/igt@runner@aborted.html [102]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-apl8/igt@runner@aborted.html [103]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-apl2/igt@runner@aborted.html [104]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-apl8/igt@runner@aborted.html [105]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-apl8/igt@runner@aborted.html [106]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-apl6/igt@runner@aborted.html [107]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-apl1/igt@runner@aborted.html - shard-skl: ([FAIL][108], [FAIL][109], [FAIL][110]) ([i915#1436] / [i915#3002] / [i915#3363]) -> ([FAIL][111], [FAIL][112], [FAIL][113]) ([i915#2029] / [i915#3002] / [i915#3363]) [108]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-skl3/igt@runner@aborted.html [109]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-skl9/igt@runner@aborted.html [110]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10299/shard-skl7/igt@runner@aborted.html [111]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl9/igt@runner@aborted.html [112]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl10/igt@runner@aborted.html [113]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_20511/shard-skl3/igt@runner@aborted.html [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145 [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271 [fdo#109349]: https://bugs.freedesktop.org/show_bug.cgi?id=109349 [fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441 [fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827 [i915#118]: https://gitlab.freedesktop.org/drm/intel/issues/118 [i915#1188]: https://gitlab.freedesktop.org/drm/intel/issues/1188 [i915#1226]: https://gitlab.freedesktop.org/drm/intel/issues/1226 [i915#1436]: https://gitlab.freedesktop.org/drm/intel/issues/1436 [i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180 [i915#1814]: https://gitlab.freedesktop.org/drm/intel/issues/1814 [i915#198]: https://gitlab.freedesktop.org/drm/intel/issues/198 [i915#1982]: https://gitlab.freedesktop.org/drm/intel/issues/1982 [i915#2029]: https://gitlab.freedesktop.org/drm/intel/issues/2029 [i915#2122]: https://gitlab.freedesktop.org/drm/intel/issues/2122 [i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190 [i915#2346]: https://gitlab.freedesktop.org/drm/intel/issues/2346 [i915#2410]: https://gitlab.freedesktop.org/drm/intel/issues/2410 [i915#2437]: https://gitlab.freedesktop.org/drm/intel/issues/2437 [i915#2521]: https://gitlab.freedesktop.org/drm/intel/issues/2521 [i915#2546]: https://gitlab.freedesktop.org/drm/intel/issues/2546 [i915#265]: https://gitlab.freedesktop.org/drm/intel/issues/265 [i915#2672]: https://gitlab.freedesktop.org/drm/intel/issues/2672 [i915#2681]: https://gitlab.freedesktop.org/drm/intel/issues/2681 [i915#2684]: https://gitlab.freedesktop.org/drm/intel/issues/2684 [i915#2782]: https://gitlab.freedesktop.org/drm/intel/issues/2782 [i915#2842]: https://gitlab.freedesktop.org/drm/intel/issues/2842 [i915#2846]: https://gitlab.freedesktop.org/drm/intel/issues/2846 [i915#2852]: https://gitlab.freedesktop.org/drm/intel/issues/2852 [i915#2868]: https://gitlab.freedesktop.org/drm/intel/issues/2868 [i915#2920]: https://gitlab.freedesktop.org/drm/intel/issues/2920 [i915#2994]: https://gitlab.freedesktop.org/drm/intel/issues/2994 [i915#3002]: https://gitlab.freedesktop.org/drm/intel/issues/3002 [i915#307]: https://gitlab.freedesktop.org/drm/intel/issues/307 [i915#3160]: https://gitlab.freedesktop.org/drm/intel/issues/3160 [i915#3296]: https://gitlab.freedesktop.org/drm/intel/issues/3296 [i915#3363]: https://gitlab.freedesktop.org/drm/intel/issues/3363 [i915#3444]: https://gitlab.freedesktop.org/drm/intel/issues/3444 [i915#533]: https://gitlab.freedesktop.org/drm/intel/issues/533 [i915#658]: https://gitlab.freedesktop.org/drm/intel/issues/658 [i915#716]: https://gitlab.freedesktop.org/drm/intel/issues/716 [i915#79]: https://gitlab.freedesktop.org/drm/intel/issues/79 [i915#95]: https://gitlab.freedesktop.org/drm/intel/issues/95 Participating hosts (10 -> 10) ------------------------------ No changes in participating hosts Build changes ------------- * Linux: CI_DRM_10299 -> Patchwork_20511 CI-20190529: 20190529 CI_DRM_10299: 40a1952a06a94218e1bb751a3091a1c105173f1a @ git://anongit.freedesktop.org/gfx-ci/linux IGT_6126: 098d0b82276fa0595e3d0fd745885f9d2147035d @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git Patchwork_20511: db4a4850eae15de316aed98dc87ea3e0a9f60088 @ 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_20511/index.html [-- Attachment #1.2: Type: text/html, Size: 32050 bytes --] [-- Attachment #2: Type: text/plain, Size: 160 bytes --] _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2021-08-19 8:35 UTC | newest] Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-07-01 15:55 [Intel-gfx] [PATCH 0/4] drm/i915/guc: Improve CTB error handling Michal Wajdeczko 2021-07-01 15:55 ` [Intel-gfx] [PATCH 1/4] drm/i915/guc: Verify result from CTB (de)register action Michal Wajdeczko 2021-08-18 14:21 ` Daniel Vetter 2021-07-01 15:55 ` [Intel-gfx] [PATCH 2/4] drm/i915/guc: Print error name on CTB (de)registration failure Michal Wajdeczko 2021-08-18 14:20 ` Daniel Vetter 2021-08-18 15:11 ` Michal Wajdeczko 2021-08-18 16:35 ` Daniel Vetter 2021-08-18 19:12 ` Michal Wajdeczko 2021-08-19 8:35 ` Daniel Vetter 2021-07-01 15:55 ` [Intel-gfx] [PATCH 3/4] drm/i915/guc: Print error name on CTB send failure Michal Wajdeczko 2021-08-18 14:22 ` Daniel Vetter 2021-07-01 15:55 ` [Intel-gfx] [PATCH 4/4] drm/i915/guc: Move and improve error message for missed CTB reply Michal Wajdeczko 2021-08-18 14:24 ` Daniel Vetter 2021-07-01 20:31 ` [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/guc: Improve CTB error handling Patchwork 2021-07-02 1:31 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).