* [PATCH] drm/i915: Tune down link train error messages due to IO failure @ 2019-05-23 13:08 Imre Deak 2019-05-23 16:52 ` ✓ Fi.CI.BAT: success for " Patchwork ` (2 more replies) 0 siblings, 3 replies; 8+ messages in thread From: Imre Deak @ 2019-05-23 13:08 UTC (permalink / raw) To: intel-gfx An IO failure happens if the sink is unplugged. This scenario shouldn't be logged with error level, since it is not a user visible problem. Converting the corresponding error messages to be debug messages also ensures that we don't fail some CI test incorrectly that tests the unplugging while a mode is still active on the given connector (like the kms_chamelium tests imititating such an unplug during being suspended). Leave those link training error messages that are due to an actual transfer rate error (and not an IO error) as-is, as that kind of problem is related to the user's configuration (cable, mode, monitor) and so it is really user visible. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=110390 Signed-off-by: Imre Deak <imre.deak@intel.com> --- drivers/gpu/drm/i915/intel_dp_link_training.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dp_link_training.c b/drivers/gpu/drm/i915/intel_dp_link_training.c index 9b1fccea966b..392ebeb89ce8 100644 --- a/drivers/gpu/drm/i915/intel_dp_link_training.c +++ b/drivers/gpu/drm/i915/intel_dp_link_training.c @@ -169,7 +169,7 @@ intel_dp_link_training_clock_recovery(struct intel_dp *intel_dp) if (!intel_dp_reset_link_train(intel_dp, DP_TRAINING_PATTERN_1 | DP_LINK_SCRAMBLING_DISABLE)) { - DRM_ERROR("failed to enable link training\n"); + DRM_DEBUG_KMS("failed to enable link training\n"); return false; } @@ -193,7 +193,7 @@ intel_dp_link_training_clock_recovery(struct intel_dp *intel_dp) drm_dp_link_train_clock_recovery_delay(intel_dp->dpcd); if (!intel_dp_get_link_status(intel_dp, link_status)) { - DRM_ERROR("failed to get link status\n"); + DRM_DEBUG_KMS("failed to get link status\n"); return false; } @@ -217,7 +217,7 @@ intel_dp_link_training_clock_recovery(struct intel_dp *intel_dp) /* Update training set as requested by target */ intel_get_adjust_train(intel_dp, link_status); if (!intel_dp_update_link_train(intel_dp)) { - DRM_ERROR("failed to update link training\n"); + DRM_DEBUG_KMS("failed to update link training\n"); return false; } @@ -295,7 +295,7 @@ intel_dp_link_training_channel_equalization(struct intel_dp *intel_dp) /* channel equalization */ if (!intel_dp_set_link_train(intel_dp, training_pattern)) { - DRM_ERROR("failed to start channel equalization\n"); + DRM_DEBUG_KMS("failed to start channel equalization\n"); return false; } @@ -303,7 +303,7 @@ intel_dp_link_training_channel_equalization(struct intel_dp *intel_dp) drm_dp_link_train_channel_eq_delay(intel_dp->dpcd); if (!intel_dp_get_link_status(intel_dp, link_status)) { - DRM_ERROR("failed to get link status\n"); + DRM_DEBUG_KMS("failed to get link status\n"); break; } @@ -327,7 +327,7 @@ intel_dp_link_training_channel_equalization(struct intel_dp *intel_dp) /* Update training set as requested by target */ intel_get_adjust_train(intel_dp, link_status); if (!intel_dp_update_link_train(intel_dp)) { - DRM_ERROR("failed to update link training\n"); + DRM_DEBUG_KMS("failed to update link training\n"); break; } } -- 2.17.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 8+ messages in thread
* ✓ Fi.CI.BAT: success for drm/i915: Tune down link train error messages due to IO failure 2019-05-23 13:08 [PATCH] drm/i915: Tune down link train error messages due to IO failure Imre Deak @ 2019-05-23 16:52 ` Patchwork 2019-05-24 16:27 ` [PATCH] " Martin Peres 2019-05-25 2:03 ` ✓ Fi.CI.IGT: success for " Patchwork 2 siblings, 0 replies; 8+ messages in thread From: Patchwork @ 2019-05-23 16:52 UTC (permalink / raw) To: Imre Deak; +Cc: intel-gfx == Series Details == Series: drm/i915: Tune down link train error messages due to IO failure URL : https://patchwork.freedesktop.org/series/61039/ State : success == Summary == CI Bug Log - changes from CI_DRM_6132 -> Patchwork_13079 ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/ Possible new issues ------------------- Here are the unknown changes that may have been introduced in Patchwork_13079: ### IGT changes ### #### Suppressed #### The following results come from untrusted machines, tests, or statuses. They do not affect the overall result. * igt@kms_flip@basic-flip-vs-wf_vblank: - {fi-icl-u3}: [DMESG-FAIL][1] ([fdo#110718]) -> [FAIL][2] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/fi-icl-u3/igt@kms_flip@basic-flip-vs-wf_vblank.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/fi-icl-u3/igt@kms_flip@basic-flip-vs-wf_vblank.html Known issues ------------ Here are the changes found in Patchwork_13079 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@i915_selftest@live_hangcheck: - fi-skl-iommu: [PASS][3] -> [INCOMPLETE][4] ([fdo#108602] / [fdo#108744]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/fi-skl-iommu/igt@i915_selftest@live_hangcheck.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/fi-skl-iommu/igt@i915_selftest@live_hangcheck.html #### Possible fixes #### * igt@gem_ctx_create@basic-files: - {fi-icl-dsi}: [INCOMPLETE][5] ([fdo#107713] / [fdo#109100]) -> [PASS][6] [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/fi-icl-dsi/igt@gem_ctx_create@basic-files.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/fi-icl-dsi/igt@gem_ctx_create@basic-files.html * igt@gem_exec_reloc@basic-write-gtt-noreloc: - {fi-icl-u3}: [DMESG-WARN][7] ([fdo#107724]) -> [PASS][8] [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/fi-icl-u3/igt@gem_exec_reloc@basic-write-gtt-noreloc.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/fi-icl-u3/igt@gem_exec_reloc@basic-write-gtt-noreloc.html * igt@gem_exec_suspend@basic-s3: - fi-blb-e6850: [INCOMPLETE][9] ([fdo#107718]) -> [PASS][10] [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/fi-blb-e6850/igt@gem_exec_suspend@basic-s3.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/fi-blb-e6850/igt@gem_exec_suspend@basic-s3.html * igt@kms_chamelium@common-hpd-after-suspend: - fi-kbl-7500u: [DMESG-WARN][11] ([fdo#102505] / [fdo#103558] / [fdo#105079] / [fdo#105602]) -> [PASS][12] [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/fi-kbl-7500u/igt@kms_chamelium@common-hpd-after-suspend.html [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/fi-kbl-7500u/igt@kms_chamelium@common-hpd-after-suspend.html - {fi-icl-u2}: [DMESG-WARN][13] ([fdo#102505] / [fdo#110390] / [fdo#110718]) -> [PASS][14] [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/fi-icl-u2/igt@kms_chamelium@common-hpd-after-suspend.html [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/fi-icl-u2/igt@kms_chamelium@common-hpd-after-suspend.html - {fi-cml-u2}: [DMESG-WARN][15] ([fdo#102505]) -> [PASS][16] [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/fi-cml-u2/igt@kms_chamelium@common-hpd-after-suspend.html [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/fi-cml-u2/igt@kms_chamelium@common-hpd-after-suspend.html * igt@kms_flip@basic-plain-flip: - {fi-icl-u3}: [DMESG-WARN][17] ([fdo#110718]) -> [PASS][18] [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/fi-icl-u3/igt@kms_flip@basic-plain-flip.html [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/fi-icl-u3/igt@kms_flip@basic-plain-flip.html #### Warnings #### * igt@i915_selftest@live_hangcheck: - fi-apl-guc: [INCOMPLETE][19] ([fdo#103927] / [fdo#110624]) -> [DMESG-FAIL][20] ([fdo#110620]) [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/fi-apl-guc/igt@i915_selftest@live_hangcheck.html [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/fi-apl-guc/igt@i915_selftest@live_hangcheck.html * igt@runner@aborted: - fi-apl-guc: [FAIL][21] ([fdo#110624]) -> [FAIL][22] ([fdo#110622]) [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/fi-apl-guc/igt@runner@aborted.html [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/fi-apl-guc/igt@runner@aborted.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#102505]: https://bugs.freedesktop.org/show_bug.cgi?id=102505 [fdo#103558]: https://bugs.freedesktop.org/show_bug.cgi?id=103558 [fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927 [fdo#105079]: https://bugs.freedesktop.org/show_bug.cgi?id=105079 [fdo#105602]: https://bugs.freedesktop.org/show_bug.cgi?id=105602 [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713 [fdo#107718]: https://bugs.freedesktop.org/show_bug.cgi?id=107718 [fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724 [fdo#108602]: https://bugs.freedesktop.org/show_bug.cgi?id=108602 [fdo#108730]: https://bugs.freedesktop.org/show_bug.cgi?id=108730 [fdo#108744]: https://bugs.freedesktop.org/show_bug.cgi?id=108744 [fdo#109100]: https://bugs.freedesktop.org/show_bug.cgi?id=109100 [fdo#110390]: https://bugs.freedesktop.org/show_bug.cgi?id=110390 [fdo#110620]: https://bugs.freedesktop.org/show_bug.cgi?id=110620 [fdo#110622]: https://bugs.freedesktop.org/show_bug.cgi?id=110622 [fdo#110624]: https://bugs.freedesktop.org/show_bug.cgi?id=110624 [fdo#110718]: https://bugs.freedesktop.org/show_bug.cgi?id=110718 Participating hosts (54 -> 46) ------------------------------ Missing (8): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 fi-byt-clapper fi-bdw-samus Build changes ------------- * Linux: CI_DRM_6132 -> Patchwork_13079 CI_DRM_6132: 78850b480c542b2e10da5a93afac2e13307909cb @ git://anongit.freedesktop.org/gfx-ci/linux IGT_5010: 631f3ac2e78c8d6332afc693bf290ae23d8d5685 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools Patchwork_13079: d43e42399d099994c556ea4077ffc96779030fe8 @ git://anongit.freedesktop.org/gfx-ci/linux == Linux commits == d43e42399d09 drm/i915: Tune down link train error messages due to IO failure == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/ _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/i915: Tune down link train error messages due to IO failure 2019-05-23 13:08 [PATCH] drm/i915: Tune down link train error messages due to IO failure Imre Deak 2019-05-23 16:52 ` ✓ Fi.CI.BAT: success for " Patchwork @ 2019-05-24 16:27 ` Martin Peres 2019-05-24 16:49 ` Imre Deak 2019-05-25 2:03 ` ✓ Fi.CI.IGT: success for " Patchwork 2 siblings, 1 reply; 8+ messages in thread From: Martin Peres @ 2019-05-24 16:27 UTC (permalink / raw) To: Imre Deak, intel-gfx On 23/05/2019 16:08, Imre Deak wrote: > An IO failure happens if the sink is unplugged. This scenario shouldn't > be logged with error level, since it is not a user visible problem. > Converting the corresponding error messages to be debug messages also > ensures that we don't fail some CI test incorrectly that tests the > unplugging while a mode is still active on the given connector (like the > kms_chamelium tests imititating such an unplug during being suspended). > > Leave those link training error messages that are due to an actual > transfer rate error (and not an IO error) as-is, as that kind of problem > is related to the user's configuration (cable, mode, monitor) and so it > is really user visible. Are there some scenarios that would be warranting a DRM_ERROR? If so, then we should decide on the verbosity based on wether the connector is connected or disconnected, don't you think? Martin > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=110390 > Signed-off-by: Imre Deak <imre.deak@intel.com> > --- > drivers/gpu/drm/i915/intel_dp_link_training.c | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_dp_link_training.c b/drivers/gpu/drm/i915/intel_dp_link_training.c > index 9b1fccea966b..392ebeb89ce8 100644 > --- a/drivers/gpu/drm/i915/intel_dp_link_training.c > +++ b/drivers/gpu/drm/i915/intel_dp_link_training.c > @@ -169,7 +169,7 @@ intel_dp_link_training_clock_recovery(struct intel_dp *intel_dp) > if (!intel_dp_reset_link_train(intel_dp, > DP_TRAINING_PATTERN_1 | > DP_LINK_SCRAMBLING_DISABLE)) { > - DRM_ERROR("failed to enable link training\n"); > + DRM_DEBUG_KMS("failed to enable link training\n"); > return false; > } > > @@ -193,7 +193,7 @@ intel_dp_link_training_clock_recovery(struct intel_dp *intel_dp) > drm_dp_link_train_clock_recovery_delay(intel_dp->dpcd); > > if (!intel_dp_get_link_status(intel_dp, link_status)) { > - DRM_ERROR("failed to get link status\n"); > + DRM_DEBUG_KMS("failed to get link status\n"); > return false; > } > > @@ -217,7 +217,7 @@ intel_dp_link_training_clock_recovery(struct intel_dp *intel_dp) > /* Update training set as requested by target */ > intel_get_adjust_train(intel_dp, link_status); > if (!intel_dp_update_link_train(intel_dp)) { > - DRM_ERROR("failed to update link training\n"); > + DRM_DEBUG_KMS("failed to update link training\n"); > return false; > } > > @@ -295,7 +295,7 @@ intel_dp_link_training_channel_equalization(struct intel_dp *intel_dp) > /* channel equalization */ > if (!intel_dp_set_link_train(intel_dp, > training_pattern)) { > - DRM_ERROR("failed to start channel equalization\n"); > + DRM_DEBUG_KMS("failed to start channel equalization\n"); > return false; > } > > @@ -303,7 +303,7 @@ intel_dp_link_training_channel_equalization(struct intel_dp *intel_dp) > > drm_dp_link_train_channel_eq_delay(intel_dp->dpcd); > if (!intel_dp_get_link_status(intel_dp, link_status)) { > - DRM_ERROR("failed to get link status\n"); > + DRM_DEBUG_KMS("failed to get link status\n"); > break; > } > > @@ -327,7 +327,7 @@ intel_dp_link_training_channel_equalization(struct intel_dp *intel_dp) > /* Update training set as requested by target */ > intel_get_adjust_train(intel_dp, link_status); > if (!intel_dp_update_link_train(intel_dp)) { > - DRM_ERROR("failed to update link training\n"); > + DRM_DEBUG_KMS("failed to update link training\n"); > break; > } > } > _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/i915: Tune down link train error messages due to IO failure 2019-05-24 16:27 ` [PATCH] " Martin Peres @ 2019-05-24 16:49 ` Imre Deak 2019-05-24 18:32 ` Manasi Navare 0 siblings, 1 reply; 8+ messages in thread From: Imre Deak @ 2019-05-24 16:49 UTC (permalink / raw) To: Martin Peres; +Cc: intel-gfx On Fri, May 24, 2019 at 07:27:54PM +0300, Martin Peres wrote: > On 23/05/2019 16:08, Imre Deak wrote: > > An IO failure happens if the sink is unplugged. This scenario shouldn't > > be logged with error level, since it is not a user visible problem. > > Converting the corresponding error messages to be debug messages also > > ensures that we don't fail some CI test incorrectly that tests the > > unplugging while a mode is still active on the given connector (like the > > kms_chamelium tests imititating such an unplug during being suspended). > > > > Leave those link training error messages that are due to an actual > > transfer rate error (and not an IO error) as-is, as that kind of problem > > is related to the user's configuration (cable, mode, monitor) and so it > > is really user visible. > > Are there some scenarios that would be warranting a DRM_ERROR? If so, > then we should decide on the verbosity based on wether the connector is > connected or disconnected, don't you think? I don't think we should depend on the correctness of setting the message level correctly based on the live state, since the live state itself is flaky/unreliable based on the timing of the unplug vs. the settling of the live state flag. For instance we could see the AUX IO failing already but live state still indicating that the sink is plugged, so we'd print again an error instead of the correct tuned-down note. > > Martin > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=110390 > > Signed-off-by: Imre Deak <imre.deak@intel.com> > > --- > > drivers/gpu/drm/i915/intel_dp_link_training.c | 12 ++++++------ > > 1 file changed, 6 insertions(+), 6 deletions(-) > > > > diff --git a/drivers/gpu/drm/i915/intel_dp_link_training.c b/drivers/gpu/drm/i915/intel_dp_link_training.c > > index 9b1fccea966b..392ebeb89ce8 100644 > > --- a/drivers/gpu/drm/i915/intel_dp_link_training.c > > +++ b/drivers/gpu/drm/i915/intel_dp_link_training.c > > @@ -169,7 +169,7 @@ intel_dp_link_training_clock_recovery(struct intel_dp *intel_dp) > > if (!intel_dp_reset_link_train(intel_dp, > > DP_TRAINING_PATTERN_1 | > > DP_LINK_SCRAMBLING_DISABLE)) { > > - DRM_ERROR("failed to enable link training\n"); > > + DRM_DEBUG_KMS("failed to enable link training\n"); > > return false; > > } > > > > @@ -193,7 +193,7 @@ intel_dp_link_training_clock_recovery(struct intel_dp *intel_dp) > > drm_dp_link_train_clock_recovery_delay(intel_dp->dpcd); > > > > if (!intel_dp_get_link_status(intel_dp, link_status)) { > > - DRM_ERROR("failed to get link status\n"); > > + DRM_DEBUG_KMS("failed to get link status\n"); > > return false; > > } > > > > @@ -217,7 +217,7 @@ intel_dp_link_training_clock_recovery(struct intel_dp *intel_dp) > > /* Update training set as requested by target */ > > intel_get_adjust_train(intel_dp, link_status); > > if (!intel_dp_update_link_train(intel_dp)) { > > - DRM_ERROR("failed to update link training\n"); > > + DRM_DEBUG_KMS("failed to update link training\n"); > > return false; > > } > > > > @@ -295,7 +295,7 @@ intel_dp_link_training_channel_equalization(struct intel_dp *intel_dp) > > /* channel equalization */ > > if (!intel_dp_set_link_train(intel_dp, > > training_pattern)) { > > - DRM_ERROR("failed to start channel equalization\n"); > > + DRM_DEBUG_KMS("failed to start channel equalization\n"); > > return false; > > } > > > > @@ -303,7 +303,7 @@ intel_dp_link_training_channel_equalization(struct intel_dp *intel_dp) > > > > drm_dp_link_train_channel_eq_delay(intel_dp->dpcd); > > if (!intel_dp_get_link_status(intel_dp, link_status)) { > > - DRM_ERROR("failed to get link status\n"); > > + DRM_DEBUG_KMS("failed to get link status\n"); > > break; > > } > > > > @@ -327,7 +327,7 @@ intel_dp_link_training_channel_equalization(struct intel_dp *intel_dp) > > /* Update training set as requested by target */ > > intel_get_adjust_train(intel_dp, link_status); > > if (!intel_dp_update_link_train(intel_dp)) { > > - DRM_ERROR("failed to update link training\n"); > > + DRM_DEBUG_KMS("failed to update link training\n"); > > break; > > } > > } > > > _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/i915: Tune down link train error messages due to IO failure 2019-05-24 16:49 ` Imre Deak @ 2019-05-24 18:32 ` Manasi Navare 2019-05-24 19:04 ` Imre Deak 0 siblings, 1 reply; 8+ messages in thread From: Manasi Navare @ 2019-05-24 18:32 UTC (permalink / raw) To: Imre Deak; +Cc: intel-gfx On Fri, May 24, 2019 at 07:49:07PM +0300, Imre Deak wrote: > On Fri, May 24, 2019 at 07:27:54PM +0300, Martin Peres wrote: > > On 23/05/2019 16:08, Imre Deak wrote: > > > An IO failure happens if the sink is unplugged. This scenario shouldn't > > > be logged with error level, since it is not a user visible problem. > > > Converting the corresponding error messages to be debug messages also > > > ensures that we don't fail some CI test incorrectly that tests the > > > unplugging while a mode is still active on the given connector (like the > > > kms_chamelium tests imititating such an unplug during being suspended). > > > > > > Leave those link training error messages that are due to an actual > > > transfer rate error (and not an IO error) as-is, as that kind of problem > > > is related to the user's configuration (cable, mode, monitor) and so it > > > is really user visible. > > > > Are there some scenarios that would be warranting a DRM_ERROR? If so, > > then we should decide on the verbosity based on wether the connector is > > connected or disconnected, don't you think? > > I don't think we should depend on the correctness of setting the message > level correctly based on the live state, since the live state itself is > flaky/unreliable based on the timing of the unplug vs. the settling of > the live state flag. For instance we could see the AUX IO failing > already but live state still indicating that the sink is plugged, so > we'd print again an error instead of the correct tuned-down note. > But in case of AUX failures which we see a lot when AUX power is accidently off or eDP is still not ON or sink is unplugged while the link training was in process, doesnt it make sense to call it as a DRM_ERROR and not just a DEBUG_KMS. For Eg: the CI tests failed due to AUX timeouts should be the actual failures which would be filtered now if we tune it down to DEBUG_KMS. Manasi > > > > Martin > > > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=110390 > > > Signed-off-by: Imre Deak <imre.deak@intel.com> > > > --- > > > drivers/gpu/drm/i915/intel_dp_link_training.c | 12 ++++++------ > > > 1 file changed, 6 insertions(+), 6 deletions(-) > > > > > > diff --git a/drivers/gpu/drm/i915/intel_dp_link_training.c b/drivers/gpu/drm/i915/intel_dp_link_training.c > > > index 9b1fccea966b..392ebeb89ce8 100644 > > > --- a/drivers/gpu/drm/i915/intel_dp_link_training.c > > > +++ b/drivers/gpu/drm/i915/intel_dp_link_training.c > > > @@ -169,7 +169,7 @@ intel_dp_link_training_clock_recovery(struct intel_dp *intel_dp) > > > if (!intel_dp_reset_link_train(intel_dp, > > > DP_TRAINING_PATTERN_1 | > > > DP_LINK_SCRAMBLING_DISABLE)) { > > > - DRM_ERROR("failed to enable link training\n"); > > > + DRM_DEBUG_KMS("failed to enable link training\n"); > > > return false; > > > } > > > > > > @@ -193,7 +193,7 @@ intel_dp_link_training_clock_recovery(struct intel_dp *intel_dp) > > > drm_dp_link_train_clock_recovery_delay(intel_dp->dpcd); > > > > > > if (!intel_dp_get_link_status(intel_dp, link_status)) { > > > - DRM_ERROR("failed to get link status\n"); > > > + DRM_DEBUG_KMS("failed to get link status\n"); > > > return false; > > > } > > > > > > @@ -217,7 +217,7 @@ intel_dp_link_training_clock_recovery(struct intel_dp *intel_dp) > > > /* Update training set as requested by target */ > > > intel_get_adjust_train(intel_dp, link_status); > > > if (!intel_dp_update_link_train(intel_dp)) { > > > - DRM_ERROR("failed to update link training\n"); > > > + DRM_DEBUG_KMS("failed to update link training\n"); > > > return false; > > > } > > > > > > @@ -295,7 +295,7 @@ intel_dp_link_training_channel_equalization(struct intel_dp *intel_dp) > > > /* channel equalization */ > > > if (!intel_dp_set_link_train(intel_dp, > > > training_pattern)) { > > > - DRM_ERROR("failed to start channel equalization\n"); > > > + DRM_DEBUG_KMS("failed to start channel equalization\n"); > > > return false; > > > } > > > > > > @@ -303,7 +303,7 @@ intel_dp_link_training_channel_equalization(struct intel_dp *intel_dp) > > > > > > drm_dp_link_train_channel_eq_delay(intel_dp->dpcd); > > > if (!intel_dp_get_link_status(intel_dp, link_status)) { > > > - DRM_ERROR("failed to get link status\n"); > > > + DRM_DEBUG_KMS("failed to get link status\n"); > > > break; > > > } > > > > > > @@ -327,7 +327,7 @@ intel_dp_link_training_channel_equalization(struct intel_dp *intel_dp) > > > /* Update training set as requested by target */ > > > intel_get_adjust_train(intel_dp, link_status); > > > if (!intel_dp_update_link_train(intel_dp)) { > > > - DRM_ERROR("failed to update link training\n"); > > > + DRM_DEBUG_KMS("failed to update link training\n"); > > > break; > > > } > > > } > > > > > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/i915: Tune down link train error messages due to IO failure 2019-05-24 18:32 ` Manasi Navare @ 2019-05-24 19:04 ` Imre Deak 2019-05-24 21:57 ` Manasi Navare 0 siblings, 1 reply; 8+ messages in thread From: Imre Deak @ 2019-05-24 19:04 UTC (permalink / raw) To: Manasi Navare; +Cc: intel-gfx On Fri, May 24, 2019 at 11:32:36AM -0700, Manasi Navare wrote: > On Fri, May 24, 2019 at 07:49:07PM +0300, Imre Deak wrote: > > On Fri, May 24, 2019 at 07:27:54PM +0300, Martin Peres wrote: > > > On 23/05/2019 16:08, Imre Deak wrote: > > > > An IO failure happens if the sink is unplugged. This scenario shouldn't > > > > be logged with error level, since it is not a user visible problem. > > > > Converting the corresponding error messages to be debug messages also > > > > ensures that we don't fail some CI test incorrectly that tests the > > > > unplugging while a mode is still active on the given connector (like the > > > > kms_chamelium tests imititating such an unplug during being suspended). > > > > > > > > Leave those link training error messages that are due to an actual > > > > transfer rate error (and not an IO error) as-is, as that kind of problem > > > > is related to the user's configuration (cable, mode, monitor) and so it > > > > is really user visible. > > > > > > Are there some scenarios that would be warranting a DRM_ERROR? If so, > > > then we should decide on the verbosity based on wether the connector is > > > connected or disconnected, don't you think? > > > > I don't think we should depend on the correctness of setting the message > > level correctly based on the live state, since the live state itself is > > flaky/unreliable based on the timing of the unplug vs. the settling of > > the live state flag. For instance we could see the AUX IO failing > > already but live state still indicating that the sink is plugged, so > > we'd print again an error instead of the correct tuned-down note. > > But in case of AUX failures which we see a lot when AUX power > is accidently off or eDP is still not ON or sink is unplugged while > the link training was in process, doesnt it make sense to > call it as a DRM_ERROR and not just a DEBUG_KMS. Logging such AUX fails as DEBUG makes sense, since it will be up to a developer to look through the logs if needed; for instance if it leads to failing the actual link training logged with DRM_ERROR or failing the detection when you expect the sink to be there. Otoh, it's a bad thing to log such AUX fails as DRM_ERROR as will result in failing tests incorrectly when the sink is disconnected and the AUX fails happen as expected. > For Eg: the CI tests failed due to AUX timeouts should be the actual failures > which would be filtered now if we tune it down to DEBUG_KMS. These will lead to actually failing the detection/link training, which you can depend on failing the tests instead of depending on error messages in dmesg. > > Manasi > > > > > > > Martin > > > > > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=110390 > > > > Signed-off-by: Imre Deak <imre.deak@intel.com> > > > > --- > > > > drivers/gpu/drm/i915/intel_dp_link_training.c | 12 ++++++------ > > > > 1 file changed, 6 insertions(+), 6 deletions(-) > > > > > > > > diff --git a/drivers/gpu/drm/i915/intel_dp_link_training.c b/drivers/gpu/drm/i915/intel_dp_link_training.c > > > > index 9b1fccea966b..392ebeb89ce8 100644 > > > > --- a/drivers/gpu/drm/i915/intel_dp_link_training.c > > > > +++ b/drivers/gpu/drm/i915/intel_dp_link_training.c > > > > @@ -169,7 +169,7 @@ intel_dp_link_training_clock_recovery(struct intel_dp *intel_dp) > > > > if (!intel_dp_reset_link_train(intel_dp, > > > > DP_TRAINING_PATTERN_1 | > > > > DP_LINK_SCRAMBLING_DISABLE)) { > > > > - DRM_ERROR("failed to enable link training\n"); > > > > + DRM_DEBUG_KMS("failed to enable link training\n"); > > > > return false; > > > > } > > > > > > > > @@ -193,7 +193,7 @@ intel_dp_link_training_clock_recovery(struct intel_dp *intel_dp) > > > > drm_dp_link_train_clock_recovery_delay(intel_dp->dpcd); > > > > > > > > if (!intel_dp_get_link_status(intel_dp, link_status)) { > > > > - DRM_ERROR("failed to get link status\n"); > > > > + DRM_DEBUG_KMS("failed to get link status\n"); > > > > return false; > > > > } > > > > > > > > @@ -217,7 +217,7 @@ intel_dp_link_training_clock_recovery(struct intel_dp *intel_dp) > > > > /* Update training set as requested by target */ > > > > intel_get_adjust_train(intel_dp, link_status); > > > > if (!intel_dp_update_link_train(intel_dp)) { > > > > - DRM_ERROR("failed to update link training\n"); > > > > + DRM_DEBUG_KMS("failed to update link training\n"); > > > > return false; > > > > } > > > > > > > > @@ -295,7 +295,7 @@ intel_dp_link_training_channel_equalization(struct intel_dp *intel_dp) > > > > /* channel equalization */ > > > > if (!intel_dp_set_link_train(intel_dp, > > > > training_pattern)) { > > > > - DRM_ERROR("failed to start channel equalization\n"); > > > > + DRM_DEBUG_KMS("failed to start channel equalization\n"); > > > > return false; > > > > } > > > > > > > > @@ -303,7 +303,7 @@ intel_dp_link_training_channel_equalization(struct intel_dp *intel_dp) > > > > > > > > drm_dp_link_train_channel_eq_delay(intel_dp->dpcd); > > > > if (!intel_dp_get_link_status(intel_dp, link_status)) { > > > > - DRM_ERROR("failed to get link status\n"); > > > > + DRM_DEBUG_KMS("failed to get link status\n"); > > > > break; > > > > } > > > > > > > > @@ -327,7 +327,7 @@ intel_dp_link_training_channel_equalization(struct intel_dp *intel_dp) > > > > /* Update training set as requested by target */ > > > > intel_get_adjust_train(intel_dp, link_status); > > > > if (!intel_dp_update_link_train(intel_dp)) { > > > > - DRM_ERROR("failed to update link training\n"); > > > > + DRM_DEBUG_KMS("failed to update link training\n"); > > > > break; > > > > } > > > > } > > > > > > > > > _______________________________________________ > > Intel-gfx mailing list > > Intel-gfx@lists.freedesktop.org > > https://lists.freedesktop.org/mailman/listinfo/intel-gfx _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/i915: Tune down link train error messages due to IO failure 2019-05-24 19:04 ` Imre Deak @ 2019-05-24 21:57 ` Manasi Navare 0 siblings, 0 replies; 8+ messages in thread From: Manasi Navare @ 2019-05-24 21:57 UTC (permalink / raw) To: Imre Deak; +Cc: intel-gfx On Fri, May 24, 2019 at 10:04:10PM +0300, Imre Deak wrote: > On Fri, May 24, 2019 at 11:32:36AM -0700, Manasi Navare wrote: > > On Fri, May 24, 2019 at 07:49:07PM +0300, Imre Deak wrote: > > > On Fri, May 24, 2019 at 07:27:54PM +0300, Martin Peres wrote: > > > > On 23/05/2019 16:08, Imre Deak wrote: > > > > > An IO failure happens if the sink is unplugged. This scenario shouldn't > > > > > be logged with error level, since it is not a user visible problem. > > > > > Converting the corresponding error messages to be debug messages also > > > > > ensures that we don't fail some CI test incorrectly that tests the > > > > > unplugging while a mode is still active on the given connector (like the > > > > > kms_chamelium tests imititating such an unplug during being suspended). > > > > > > > > > > Leave those link training error messages that are due to an actual > > > > > transfer rate error (and not an IO error) as-is, as that kind of problem > > > > > is related to the user's configuration (cable, mode, monitor) and so it > > > > > is really user visible. > > > > > > > > Are there some scenarios that would be warranting a DRM_ERROR? If so, > > > > then we should decide on the verbosity based on wether the connector is > > > > connected or disconnected, don't you think? > > > > > > I don't think we should depend on the correctness of setting the message > > > level correctly based on the live state, since the live state itself is > > > flaky/unreliable based on the timing of the unplug vs. the settling of > > > the live state flag. For instance we could see the AUX IO failing > > > already but live state still indicating that the sink is plugged, so > > > we'd print again an error instead of the correct tuned-down note. > > > > But in case of AUX failures which we see a lot when AUX power > > is accidently off or eDP is still not ON or sink is unplugged while > > the link training was in process, doesnt it make sense to > > call it as a DRM_ERROR and not just a DEBUG_KMS. > > Logging such AUX fails as DEBUG makes sense, since it will be up to > a developer to look through the logs if needed; for instance if it leads > to failing the actual link training logged with DRM_ERROR or failing the > detection when you expect the sink to be there. > > Otoh, it's a bad thing to log such AUX fails as DRM_ERROR as will result > in failing tests incorrectly when the sink is disconnected and the AUX > fails happen as expected. > > > For Eg: the CI tests failed due to AUX timeouts should be the actual failures > > which would be filtered now if we tune it down to DEBUG_KMS. > > These will lead to actually failing the detection/link training, which > you can depend on failing the tests instead of depending on error > messages in dmesg. Okay makes sense. So basically rely on the final result of training only to decide if its a real failure or not. Got it. Looks good to me in that case. Reviewed-by: Manasi Navare <manasi.d.navare@intel.com> Manasi > > > > > Manasi > > > > > > > > > > Martin > > > > > > > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=110390 > > > > > Signed-off-by: Imre Deak <imre.deak@intel.com> > > > > > --- > > > > > drivers/gpu/drm/i915/intel_dp_link_training.c | 12 ++++++------ > > > > > 1 file changed, 6 insertions(+), 6 deletions(-) > > > > > > > > > > diff --git a/drivers/gpu/drm/i915/intel_dp_link_training.c b/drivers/gpu/drm/i915/intel_dp_link_training.c > > > > > index 9b1fccea966b..392ebeb89ce8 100644 > > > > > --- a/drivers/gpu/drm/i915/intel_dp_link_training.c > > > > > +++ b/drivers/gpu/drm/i915/intel_dp_link_training.c > > > > > @@ -169,7 +169,7 @@ intel_dp_link_training_clock_recovery(struct intel_dp *intel_dp) > > > > > if (!intel_dp_reset_link_train(intel_dp, > > > > > DP_TRAINING_PATTERN_1 | > > > > > DP_LINK_SCRAMBLING_DISABLE)) { > > > > > - DRM_ERROR("failed to enable link training\n"); > > > > > + DRM_DEBUG_KMS("failed to enable link training\n"); > > > > > return false; > > > > > } > > > > > > > > > > @@ -193,7 +193,7 @@ intel_dp_link_training_clock_recovery(struct intel_dp *intel_dp) > > > > > drm_dp_link_train_clock_recovery_delay(intel_dp->dpcd); > > > > > > > > > > if (!intel_dp_get_link_status(intel_dp, link_status)) { > > > > > - DRM_ERROR("failed to get link status\n"); > > > > > + DRM_DEBUG_KMS("failed to get link status\n"); > > > > > return false; > > > > > } > > > > > > > > > > @@ -217,7 +217,7 @@ intel_dp_link_training_clock_recovery(struct intel_dp *intel_dp) > > > > > /* Update training set as requested by target */ > > > > > intel_get_adjust_train(intel_dp, link_status); > > > > > if (!intel_dp_update_link_train(intel_dp)) { > > > > > - DRM_ERROR("failed to update link training\n"); > > > > > + DRM_DEBUG_KMS("failed to update link training\n"); > > > > > return false; > > > > > } > > > > > > > > > > @@ -295,7 +295,7 @@ intel_dp_link_training_channel_equalization(struct intel_dp *intel_dp) > > > > > /* channel equalization */ > > > > > if (!intel_dp_set_link_train(intel_dp, > > > > > training_pattern)) { > > > > > - DRM_ERROR("failed to start channel equalization\n"); > > > > > + DRM_DEBUG_KMS("failed to start channel equalization\n"); > > > > > return false; > > > > > } > > > > > > > > > > @@ -303,7 +303,7 @@ intel_dp_link_training_channel_equalization(struct intel_dp *intel_dp) > > > > > > > > > > drm_dp_link_train_channel_eq_delay(intel_dp->dpcd); > > > > > if (!intel_dp_get_link_status(intel_dp, link_status)) { > > > > > - DRM_ERROR("failed to get link status\n"); > > > > > + DRM_DEBUG_KMS("failed to get link status\n"); > > > > > break; > > > > > } > > > > > > > > > > @@ -327,7 +327,7 @@ intel_dp_link_training_channel_equalization(struct intel_dp *intel_dp) > > > > > /* Update training set as requested by target */ > > > > > intel_get_adjust_train(intel_dp, link_status); > > > > > if (!intel_dp_update_link_train(intel_dp)) { > > > > > - DRM_ERROR("failed to update link training\n"); > > > > > + DRM_DEBUG_KMS("failed to update link training\n"); > > > > > break; > > > > > } > > > > > } > > > > > > > > > > > > _______________________________________________ > > > Intel-gfx mailing list > > > Intel-gfx@lists.freedesktop.org > > > https://lists.freedesktop.org/mailman/listinfo/intel-gfx _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 8+ messages in thread
* ✓ Fi.CI.IGT: success for drm/i915: Tune down link train error messages due to IO failure 2019-05-23 13:08 [PATCH] drm/i915: Tune down link train error messages due to IO failure Imre Deak 2019-05-23 16:52 ` ✓ Fi.CI.BAT: success for " Patchwork 2019-05-24 16:27 ` [PATCH] " Martin Peres @ 2019-05-25 2:03 ` Patchwork 2 siblings, 0 replies; 8+ messages in thread From: Patchwork @ 2019-05-25 2:03 UTC (permalink / raw) To: Imre Deak; +Cc: intel-gfx == Series Details == Series: drm/i915: Tune down link train error messages due to IO failure URL : https://patchwork.freedesktop.org/series/61039/ State : success == Summary == CI Bug Log - changes from CI_DRM_6132_full -> Patchwork_13079_full ==================================================== Summary ------- **WARNING** Minor unknown changes coming with Patchwork_13079_full need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in Patchwork_13079_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_13079_full: ### IGT changes ### #### Warnings #### * igt@prime_vgem@wait-bsd1: - shard-snb: [INCOMPLETE][1] ([fdo#105411]) -> [FAIL][2] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-snb5/igt@prime_vgem@wait-bsd1.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-snb4/igt@prime_vgem@wait-bsd1.html Known issues ------------ Here are the changes found in Patchwork_13079_full that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_persistent_relocs@forked-interruptible-thrashing: - shard-iclb: [PASS][3] -> [INCOMPLETE][4] ([fdo#107713]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-iclb4/igt@gem_persistent_relocs@forked-interruptible-thrashing.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-iclb7/igt@gem_persistent_relocs@forked-interruptible-thrashing.html * igt@i915_pm_rpm@system-suspend: - shard-skl: [PASS][5] -> [INCOMPLETE][6] ([fdo#104108] / [fdo#107807]) [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-skl7/igt@i915_pm_rpm@system-suspend.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-skl9/igt@i915_pm_rpm@system-suspend.html - shard-kbl: [PASS][7] -> [INCOMPLETE][8] ([fdo#103665] / [fdo#107807]) [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-kbl6/igt@i915_pm_rpm@system-suspend.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-kbl4/igt@i915_pm_rpm@system-suspend.html * igt@kms_cursor_legacy@2x-cursor-vs-flip-atomic: - shard-hsw: [PASS][9] -> [SKIP][10] ([fdo#109271]) +25 similar issues [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-hsw4/igt@kms_cursor_legacy@2x-cursor-vs-flip-atomic.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-hsw5/igt@kms_cursor_legacy@2x-cursor-vs-flip-atomic.html * igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size: - shard-skl: [PASS][11] -> [FAIL][12] ([fdo#102670]) [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-skl3/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size.html [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-skl6/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size.html * igt@kms_flip@flip-vs-suspend-interruptible: - shard-skl: [PASS][13] -> [INCOMPLETE][14] ([fdo#109507]) [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-skl2/igt@kms_flip@flip-vs-suspend-interruptible.html [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-skl5/igt@kms_flip@flip-vs-suspend-interruptible.html * igt@kms_flip@plain-flip-fb-recreate-interruptible: - shard-glk: [PASS][15] -> [FAIL][16] ([fdo#100368]) [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-glk5/igt@kms_flip@plain-flip-fb-recreate-interruptible.html [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-glk8/igt@kms_flip@plain-flip-fb-recreate-interruptible.html * igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-indfb-draw-mmap-gtt: - shard-iclb: [PASS][17] -> [FAIL][18] ([fdo#103167]) [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-iclb4/igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-indfb-draw-mmap-gtt.html [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-iclb7/igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-indfb-draw-mmap-gtt.html * igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-shrfb-draw-mmap-wc: - shard-glk: [PASS][19] -> [INCOMPLETE][20] ([fdo#103359] / [k.org#198133]) [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-glk5/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-shrfb-draw-mmap-wc.html [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-glk8/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-shrfb-draw-mmap-wc.html * igt@kms_plane_alpha_blend@pipe-b-constant-alpha-min: - shard-skl: [PASS][21] -> [FAIL][22] ([fdo#108145]) [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-skl7/igt@kms_plane_alpha_blend@pipe-b-constant-alpha-min.html [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-skl7/igt@kms_plane_alpha_blend@pipe-b-constant-alpha-min.html * igt@kms_psr2_su@page_flip: - shard-iclb: [PASS][23] -> [SKIP][24] ([fdo#109642]) [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-iclb2/igt@kms_psr2_su@page_flip.html [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-iclb6/igt@kms_psr2_su@page_flip.html * igt@kms_setmode@basic: - shard-kbl: [PASS][25] -> [FAIL][26] ([fdo#99912]) [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-kbl2/igt@kms_setmode@basic.html [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-kbl1/igt@kms_setmode@basic.html * igt@kms_vblank@pipe-c-ts-continuation-suspend: - shard-apl: [PASS][27] -> [DMESG-WARN][28] ([fdo#108566]) +3 similar issues [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-apl2/igt@kms_vblank@pipe-c-ts-continuation-suspend.html [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-apl5/igt@kms_vblank@pipe-c-ts-continuation-suspend.html #### Possible fixes #### * igt@gem_softpin@noreloc-s3: - shard-skl: [INCOMPLETE][29] ([fdo#104108]) -> [PASS][30] [29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-skl5/igt@gem_softpin@noreloc-s3.html [30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-skl7/igt@gem_softpin@noreloc-s3.html * igt@gem_workarounds@suspend-resume: - shard-apl: [DMESG-WARN][31] ([fdo#108566]) -> [PASS][32] +5 similar issues [31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-apl3/igt@gem_workarounds@suspend-resume.html [32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-apl6/igt@gem_workarounds@suspend-resume.html * igt@kms_flip_tiling@flip-to-y-tiled: - shard-iclb: [FAIL][33] ([fdo#107931] / [fdo#108134]) -> [PASS][34] [33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-iclb3/igt@kms_flip_tiling@flip-to-y-tiled.html [34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-iclb3/igt@kms_flip_tiling@flip-to-y-tiled.html * igt@kms_flip_tiling@flip-x-tiled: - shard-skl: [FAIL][35] ([fdo#108145] / [fdo#108303]) -> [PASS][36] [35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-skl5/igt@kms_flip_tiling@flip-x-tiled.html [36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-skl1/igt@kms_flip_tiling@flip-x-tiled.html * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-move: - shard-iclb: [FAIL][37] ([fdo#103167]) -> [PASS][38] +1 similar issue [37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-iclb4/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-move.html [38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-iclb7/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-move.html * igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-pri-shrfb-draw-mmap-wc: - shard-hsw: [SKIP][39] ([fdo#109271]) -> [PASS][40] +25 similar issues [39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-hsw5/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-pri-shrfb-draw-mmap-wc.html [40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-hsw6/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-pri-shrfb-draw-mmap-wc.html * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-move: - shard-skl: [FAIL][41] ([fdo#108040]) -> [PASS][42] [41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-skl3/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-move.html [42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-skl6/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-move.html * igt@kms_plane_alpha_blend@pipe-a-coverage-7efc: - shard-skl: [FAIL][43] ([fdo#108145]) -> [PASS][44] +1 similar issue [43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-skl3/igt@kms_plane_alpha_blend@pipe-a-coverage-7efc.html [44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-skl6/igt@kms_plane_alpha_blend@pipe-a-coverage-7efc.html * igt@kms_plane_alpha_blend@pipe-b-coverage-7efc: - shard-skl: [FAIL][45] ([fdo#108145] / [fdo#110403]) -> [PASS][46] [45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-skl3/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html [46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-skl6/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html * igt@kms_plane_lowres@pipe-a-tiling-x: - shard-iclb: [FAIL][47] ([fdo#103166]) -> [PASS][48] [47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-iclb4/igt@kms_plane_lowres@pipe-a-tiling-x.html [48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-iclb7/igt@kms_plane_lowres@pipe-a-tiling-x.html * igt@kms_psr2_su@frontbuffer: - shard-iclb: [SKIP][49] ([fdo#109642]) -> [PASS][50] [49]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-iclb8/igt@kms_psr2_su@frontbuffer.html [50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-iclb2/igt@kms_psr2_su@frontbuffer.html * igt@kms_setmode@basic: - shard-apl: [FAIL][51] ([fdo#99912]) -> [PASS][52] [51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-apl3/igt@kms_setmode@basic.html [52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-apl6/igt@kms_setmode@basic.html #### Warnings #### * igt@gem_mmap_gtt@forked-big-copy-odd: - shard-iclb: [INCOMPLETE][53] ([fdo#107713] / [fdo#109100]) -> [TIMEOUT][54] ([fdo#109673]) +1 similar issue [53]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-iclb2/igt@gem_mmap_gtt@forked-big-copy-odd.html [54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-iclb2/igt@gem_mmap_gtt@forked-big-copy-odd.html * igt@gem_tiled_swapping@non-threaded: - shard-hsw: [FAIL][55] ([fdo#108686]) -> [INCOMPLETE][56] ([fdo#103540]) [55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-hsw1/igt@gem_tiled_swapping@non-threaded.html [56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-hsw2/igt@gem_tiled_swapping@non-threaded.html * igt@kms_flip@2x-flip-vs-expired-vblank: - shard-apl: [SKIP][57] ([fdo#109271]) -> [INCOMPLETE][58] ([fdo#103927]) [57]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-apl1/igt@kms_flip@2x-flip-vs-expired-vblank.html [58]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-apl1/igt@kms_flip@2x-flip-vs-expired-vblank.html * igt@prime_vgem@busy-bsd1: - shard-snb: [FAIL][59] -> [INCOMPLETE][60] ([fdo#105411]) [59]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6132/shard-snb2/igt@prime_vgem@busy-bsd1.html [60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_13079/shard-snb7/igt@prime_vgem@busy-bsd1.html [fdo#100368]: https://bugs.freedesktop.org/show_bug.cgi?id=100368 [fdo#102670]: https://bugs.freedesktop.org/show_bug.cgi?id=102670 [fdo#103166]: https://bugs.freedesktop.org/show_bug.cgi?id=103166 [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167 [fdo#103359]: https://bugs.freedesktop.org/show_bug.cgi?id=103359 [fdo#103540]: https://bugs.freedesktop.org/show_bug.cgi?id=103540 [fdo#103665]: https://bugs.freedesktop.org/show_bug.cgi?id=103665 [fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927 [fdo#104108]: https://bugs.freedesktop.org/show_bug.cgi?id=104108 [fdo#105411]: https://bugs.freedesktop.org/show_bug.cgi?id=105411 [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713 [fdo#107807]: https://bugs.freedesktop.org/show_bug.cgi?id=107807 [fdo#107931]: https://bugs.freedesktop.org/show_bug.cgi?id=107931 [fdo#108040]: https://bugs.freedesktop.org/show_bug.cgi?id=108040 [fdo#108134]: https://bugs.freedesktop.org/show_bug.cgi?id=108134 [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145 [fdo#108303]: https://bugs.freedesktop.org/show_bug.cgi?id=108303 [fdo#108566]: https://bugs.freedesktop.org/show_bug.cgi?id=108566 [fdo#108686]: https://bugs.freedesktop.org/show_bug.cgi?id=108686 [fdo#109100]: https://bugs.freedesktop.org/show_bug.cgi?id=109100 [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271 [fdo#109507]: https://bugs.freedesktop.org/show_bug.cgi?id=109507 [fdo#109642]: https://bugs.freedesktop.org/show_bug.cgi?id=109642 [fdo#109673]: https://bugs.freedesktop.org/show_bug.cgi?id=109673 [fdo#110403]: https://bugs.freedesktop.org/show_bug.cgi?id=110403 [fdo#99912]: https://bugs.freedesktop.org/show_bug.cgi?id=99912 [k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133 Participating hosts (10 -> 10) ------------------------------ No changes in participating hosts Build changes ------------- * Linux: CI_DRM_6132 -> Patchwork_13079 CI_DRM_6132: 78850b480c542b2e10da5a93afac2e13307909cb @ git://anongit.freedesktop.org/gfx-ci/linux IGT_5010: 631f3ac2e78c8d6332afc693bf290ae23d8d5685 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools Patchwork_13079: d43e42399d099994c556ea4077ffc96779030fe8 @ 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_13079/ _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2019-05-25 2:03 UTC | newest] Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-05-23 13:08 [PATCH] drm/i915: Tune down link train error messages due to IO failure Imre Deak 2019-05-23 16:52 ` ✓ Fi.CI.BAT: success for " Patchwork 2019-05-24 16:27 ` [PATCH] " Martin Peres 2019-05-24 16:49 ` Imre Deak 2019-05-24 18:32 ` Manasi Navare 2019-05-24 19:04 ` Imre Deak 2019-05-24 21:57 ` Manasi Navare 2019-05-25 2:03 ` ✓ Fi.CI.IGT: success for " 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.