* [PATCH v6 0/3] Send a hotplug when edid changes
@ 2019-09-06 8:26 Stanislav Lisovskiy
2019-09-06 8:26 ` [PATCH v6 1/3] drm: Add helper to compare edids Stanislav Lisovskiy
` (5 more replies)
0 siblings, 6 replies; 7+ messages in thread
From: Stanislav Lisovskiy @ 2019-09-06 8:26 UTC (permalink / raw)
To: dri-devel; +Cc: daniel.vetter, intel-gfx, martin.peres
This series introduce to drm a way to determine if something else
except connection_status had changed during probing, which
can be used by other drivers as well. Another i915 specific part
uses this approach to determine if edid had changed without
changing the connection status and send a hotplug event.
Stanislav Lisovskiy (3):
drm: Add helper to compare edids.
drm: Introduce epoch counter to drm_connector
drm/i915: Send hotplug event if edid had changed
drivers/gpu/drm/drm_connector.c | 16 +++++++++
drivers/gpu/drm/drm_edid.c | 36 +++++++++++++++++++
drivers/gpu/drm/drm_probe_helper.c | 38 +++++++++++++++++---
drivers/gpu/drm/i915/display/intel_hotplug.c | 18 +++++++---
include/drm/drm_connector.h | 3 ++
include/drm/drm_edid.h | 9 +++++
6 files changed, 110 insertions(+), 10 deletions(-)
--
2.17.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 7+ messages in thread* [PATCH v6 1/3] drm: Add helper to compare edids. 2019-09-06 8:26 [PATCH v6 0/3] Send a hotplug when edid changes Stanislav Lisovskiy @ 2019-09-06 8:26 ` Stanislav Lisovskiy 2019-09-06 8:26 ` [PATCH v6 2/3] drm: Introduce epoch counter to drm_connector Stanislav Lisovskiy ` (4 subsequent siblings) 5 siblings, 0 replies; 7+ messages in thread From: Stanislav Lisovskiy @ 2019-09-06 8:26 UTC (permalink / raw) To: dri-devel; +Cc: daniel.vetter, intel-gfx, martin.peres Many drivers would benefit from using drm helper to compare edid, rather than bothering with own implementation. v2: Added documentation for this function. Signed-off-by: Stanislav Lisovskiy <stanislav.lisovskiy@intel.com> --- drivers/gpu/drm/drm_edid.c | 33 +++++++++++++++++++++++++++++++++ include/drm/drm_edid.h | 9 +++++++++ 2 files changed, 42 insertions(+) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index 82a4ceed3fcf..0639db9fd23f 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu/drm/drm_edid.c @@ -1362,6 +1362,39 @@ static bool drm_edid_is_zero(const u8 *in_edid, int length) return true; } +/** + * drm_edid_are_equal - compare two edid blobs. + * @edid1: pointer to first blob + * @edid2: pointer to second blob + * This helper can be used during probing to determine if + * edid had changed. + */ +bool drm_edid_are_equal(const struct edid *edid1, const struct edid *edid2) +{ + int edid1_len, edid2_len; + bool edid1_present = edid1 != NULL; + bool edid2_present = edid2 != NULL; + + if (edid1_present != edid2_present) + return false; + + if (edid1) { + + edid1_len = EDID_LENGTH * (1 + edid1->extensions); + edid2_len = EDID_LENGTH * (1 + edid2->extensions); + + if (edid1_len != edid2_len) + return false; + + if (memcmp(edid1, edid2, edid1_len)) + return false; + } + + return true; +} +EXPORT_SYMBOL(drm_edid_are_equal); + + /** * drm_edid_block_valid - Sanity check the EDID block (base or extension) * @raw_edid: pointer to raw EDID block diff --git a/include/drm/drm_edid.h b/include/drm/drm_edid.h index b9719418c3d2..b0c68b2d79ed 100644 --- a/include/drm/drm_edid.h +++ b/include/drm/drm_edid.h @@ -354,6 +354,15 @@ drm_load_edid_firmware(struct drm_connector *connector) } #endif +/** + * drm_edid_are_equal - compare two edid blobs. + * @edid1: pointer to first blob + * @edid2: pointer to second blob + * This helper can be used during probing to determine if + * edid had changed. + */ +bool drm_edid_are_equal(const struct edid *edid1, const struct edid *edid2); + int drm_hdmi_avi_infoframe_from_display_mode(struct hdmi_avi_infoframe *frame, struct drm_connector *connector, -- 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] 7+ messages in thread
* [PATCH v6 2/3] drm: Introduce epoch counter to drm_connector 2019-09-06 8:26 [PATCH v6 0/3] Send a hotplug when edid changes Stanislav Lisovskiy 2019-09-06 8:26 ` [PATCH v6 1/3] drm: Add helper to compare edids Stanislav Lisovskiy @ 2019-09-06 8:26 ` Stanislav Lisovskiy 2019-09-06 8:26 ` [PATCH v6 3/3] drm/i915: Send hotplug event if edid had changed Stanislav Lisovskiy ` (3 subsequent siblings) 5 siblings, 0 replies; 7+ messages in thread From: Stanislav Lisovskiy @ 2019-09-06 8:26 UTC (permalink / raw) To: dri-devel; +Cc: daniel.vetter, intel-gfx, martin.peres This counter will be used by drm_helper_probe_detect caller to determine if anything had changed(including edid, connection status and etc). Hardware specific driver detect hooks are responsible for updating this counter when some change is detected to notify the drm part, which can trigger for example hotplug event. Also now call drm_connector_update_edid_property right after we get edid always to make sure there is a unified way to handle edid change, without having to change tons of source code as currently drm_connector_update_edid_property is called only in certain cases like reprobing and not right after edid is actually updated. v2: Added documentation for the new counter. Rename change_counter to epoch_counter. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105540 Signed-off-by: Stanislav Lisovskiy <stanislav.lisovskiy@intel.com> --- drivers/gpu/drm/drm_connector.c | 16 +++++++++++++ drivers/gpu/drm/drm_edid.c | 3 +++ drivers/gpu/drm/drm_probe_helper.c | 38 ++++++++++++++++++++++++++---- include/drm/drm_connector.h | 3 +++ 4 files changed, 55 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/drm_connector.c b/drivers/gpu/drm/drm_connector.c index 4c766624b20d..98fd236acc57 100644 --- a/drivers/gpu/drm/drm_connector.c +++ b/drivers/gpu/drm/drm_connector.c @@ -246,6 +246,7 @@ int drm_connector_init(struct drm_device *dev, INIT_LIST_HEAD(&connector->modes); mutex_init(&connector->mutex); connector->edid_blob_ptr = NULL; + connector->epoch_counter = 0; connector->tile_blob_ptr = NULL; connector->status = connector_status_unknown; connector->display_info.panel_orientation = @@ -1866,6 +1867,7 @@ int drm_connector_update_edid_property(struct drm_connector *connector, struct drm_device *dev = connector->dev; size_t size = 0; int ret; + const struct edid *old_edid; /* ignore requests to set edid when overridden */ if (connector->override_edid) @@ -1887,6 +1889,20 @@ int drm_connector_update_edid_property(struct drm_connector *connector, else drm_reset_display_info(connector); + if (connector->edid_blob_ptr) { + old_edid = (const struct edid *)connector->edid_blob_ptr->data; + if (old_edid) { + if (!drm_edid_are_equal(edid, old_edid)) { + DRM_DEBUG_KMS("[CONNECTOR:%d:%s] Edid was changed.\n", + connector->base.id, connector->name); + + connector->epoch_counter += 1; + DRM_DEBUG_KMS("Updating change counter to %llu\n", + connector->epoch_counter); + } + } + } + drm_object_property_set_value(&connector->base, dev->mode_config.non_desktop_property, connector->display_info.non_desktop); diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index 0639db9fd23f..48b716abc9ef 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu/drm/drm_edid.c @@ -1803,6 +1803,9 @@ struct edid *drm_get_edid(struct drm_connector *connector, edid = drm_do_get_edid(connector, drm_do_probe_ddc_edid, adapter); if (edid) drm_get_displayid(connector, edid); + + drm_connector_update_edid_property(connector, edid); + return edid; } EXPORT_SYMBOL(drm_get_edid); diff --git a/drivers/gpu/drm/drm_probe_helper.c b/drivers/gpu/drm/drm_probe_helper.c index 351cbc40f0f8..b2b6385cfed3 100644 --- a/drivers/gpu/drm/drm_probe_helper.c +++ b/drivers/gpu/drm/drm_probe_helper.c @@ -287,6 +287,9 @@ drm_helper_probe_detect_ctx(struct drm_connector *connector, bool force) if (WARN_ON(ret < 0)) ret = connector_status_unknown; + if (ret != connector->status) + connector->epoch_counter += 1; + drm_modeset_drop_locks(&ctx); drm_modeset_acquire_fini(&ctx); @@ -320,11 +323,16 @@ drm_helper_probe_detect(struct drm_connector *connector, return ret; if (funcs->detect_ctx) - return funcs->detect_ctx(connector, ctx, force); + ret = funcs->detect_ctx(connector, ctx, force); else if (connector->funcs->detect) - return connector->funcs->detect(connector, force); + ret = connector->funcs->detect(connector, force); else - return connector_status_connected; + ret = connector_status_connected; + + if (ret != connector->status) + connector->epoch_counter += 1; + + return ret; } EXPORT_SYMBOL(drm_helper_probe_detect); @@ -777,6 +785,7 @@ bool drm_helper_hpd_irq_event(struct drm_device *dev) struct drm_connector_list_iter conn_iter; enum drm_connector_status old_status; bool changed = false; + uint64_t old_epoch_counter; if (!dev->mode_config.poll_enabled) return false; @@ -790,20 +799,39 @@ bool drm_helper_hpd_irq_event(struct drm_device *dev) old_status = connector->status; + old_epoch_counter = connector->epoch_counter; + + DRM_DEBUG_KMS("[CONNECTOR:%d:%s] Old epoch counter %llu\n", connector->base.id, + connector->name, + old_epoch_counter); + connector->status = drm_helper_probe_detect(connector, NULL, false); DRM_DEBUG_KMS("[CONNECTOR:%d:%s] status updated from %s to %s\n", connector->base.id, connector->name, drm_get_connector_status_name(old_status), drm_get_connector_status_name(connector->status)); - if (old_status != connector->status) + + DRM_DEBUG_KMS("[CONNECTOR:%d:%s] New epoch counter %llu\n", + connector->base.id, + connector->name, + connector->epoch_counter); + + /* + * Check if epoch counter had changed, meaning that we need + * to send a uevent. + */ + if (old_epoch_counter != connector->epoch_counter) { changed = true; + } } drm_connector_list_iter_end(&conn_iter); mutex_unlock(&dev->mode_config.mutex); - if (changed) + if (changed) { drm_kms_helper_hotplug_event(dev); + DRM_DEBUG_KMS("Sent hotplug event\n"); + } return changed; } diff --git a/include/drm/drm_connector.h b/include/drm/drm_connector.h index 681cb590f952..a3cc7d0927d6 100644 --- a/include/drm/drm_connector.h +++ b/include/drm/drm_connector.h @@ -1288,6 +1288,9 @@ struct drm_connector { /** @override_edid: has the EDID been overwritten through debugfs for testing? */ bool override_edid; + /** @epoch_counter: used to detect any other changes in connector, besides status */ + uint64_t epoch_counter; + #define DRM_CONNECTOR_MAX_ENCODER 3 /** * @encoder_ids: Valid encoders for this connector. Please only use -- 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] 7+ messages in thread
* [PATCH v6 3/3] drm/i915: Send hotplug event if edid had changed 2019-09-06 8:26 [PATCH v6 0/3] Send a hotplug when edid changes Stanislav Lisovskiy 2019-09-06 8:26 ` [PATCH v6 1/3] drm: Add helper to compare edids Stanislav Lisovskiy 2019-09-06 8:26 ` [PATCH v6 2/3] drm: Introduce epoch counter to drm_connector Stanislav Lisovskiy @ 2019-09-06 8:26 ` Stanislav Lisovskiy 2019-09-06 8:46 ` ✗ Fi.CI.CHECKPATCH: warning for Send a hotplug when edid changes (rev7) Patchwork ` (2 subsequent siblings) 5 siblings, 0 replies; 7+ messages in thread From: Stanislav Lisovskiy @ 2019-09-06 8:26 UTC (permalink / raw) To: dri-devel; +Cc: daniel.vetter, intel-gfx, martin.peres Added epoch counter checking to intel_encoder_hotplug in order to be able process all the connector changes, besides connection status. Also now any change in connector would result in epoch counter change, so no multiple checks are needed. v2: Renamed change counter to epoch counter. Fixed type name. v3: Fixed rebase conflict v4: Remove duplicate drm_edid_equal checks from hdmi and dp, lets use only once edid property is getting updated and increment epoch counter from there. Also lets now call drm_connector_update_edid_property right after we get edid always to make sure there is a unified way to handle edid change, without having to change tons of source code as currently drm_connector_update_edid_property is called only in certain cases like reprobing and not right after edid is actually updated. v5: Fixed const modifiers, removed blank line v6: Removed drm specific part from this patch, leaving only i915 specific changes here. Signed-off-by: Stanislav Lisovskiy <stanislav.lisovskiy@intel.com> --- drivers/gpu/drm/i915/display/intel_hotplug.c | 18 +++++++++++++----- 1 file changed, 13 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_hotplug.c b/drivers/gpu/drm/i915/display/intel_hotplug.c index fc29046d48ea..1e3e425dd78d 100644 --- a/drivers/gpu/drm/i915/display/intel_hotplug.c +++ b/drivers/gpu/drm/i915/display/intel_hotplug.c @@ -280,23 +280,31 @@ intel_encoder_hotplug(struct intel_encoder *encoder, { struct drm_device *dev = connector->base.dev; enum drm_connector_status old_status; + u64 old_epoch_counter; + bool ret = false; WARN_ON(!mutex_is_locked(&dev->mode_config.mutex)); old_status = connector->base.status; + old_epoch_counter = connector->base.epoch_counter; + connector->base.status = drm_helper_probe_detect(&connector->base, NULL, false); - if (old_status == connector->base.status) - return INTEL_HOTPLUG_UNCHANGED; + if (old_epoch_counter != connector->base.epoch_counter) + ret = true; - DRM_DEBUG_KMS("[CONNECTOR:%d:%s] status updated from %s to %s\n", + if (ret) { + DRM_DEBUG_KMS("[CONNECTOR:%d:%s] status updated from %s to %s(epoch counter %llu)\n", connector->base.base.id, connector->base.name, drm_get_connector_status_name(old_status), - drm_get_connector_status_name(connector->base.status)); + drm_get_connector_status_name(connector->base.status), + connector->base.epoch_counter); + return INTEL_HOTPLUG_CHANGED; + } - return INTEL_HOTPLUG_CHANGED; + return INTEL_HOTPLUG_UNCHANGED; } static bool intel_encoder_has_hpd_pulse(struct intel_encoder *encoder) -- 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] 7+ messages in thread
* ✗ Fi.CI.CHECKPATCH: warning for Send a hotplug when edid changes (rev7) 2019-09-06 8:26 [PATCH v6 0/3] Send a hotplug when edid changes Stanislav Lisovskiy ` (2 preceding siblings ...) 2019-09-06 8:26 ` [PATCH v6 3/3] drm/i915: Send hotplug event if edid had changed Stanislav Lisovskiy @ 2019-09-06 8:46 ` Patchwork 2019-09-06 9:14 ` ✓ Fi.CI.BAT: success " Patchwork 2019-09-06 10:35 ` ✓ Fi.CI.IGT: " Patchwork 5 siblings, 0 replies; 7+ messages in thread From: Patchwork @ 2019-09-06 8:46 UTC (permalink / raw) To: Stanislav Lisovskiy; +Cc: intel-gfx == Series Details == Series: Send a hotplug when edid changes (rev7) URL : https://patchwork.freedesktop.org/series/62816/ State : warning == Summary == $ dim checkpatch origin/drm-tip c2072e1d23ff drm: Add helper to compare edids. -:32: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written "edid1" #32: FILE: drivers/gpu/drm/drm_edid.c:1375: + bool edid1_present = edid1 != NULL; -:33: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written "edid2" #33: FILE: drivers/gpu/drm/drm_edid.c:1376: + bool edid2_present = edid2 != NULL; -:39: CHECK:BRACES: Blank lines aren't necessary after an open brace '{' #39: FILE: drivers/gpu/drm/drm_edid.c:1382: + if (edid1) { + -:54: CHECK:LINE_SPACING: Please don't use multiple blank lines #54: FILE: drivers/gpu/drm/drm_edid.c:1397: + + total: 0 errors, 0 warnings, 4 checks, 54 lines checked 701f9863dd56 drm: Introduce epoch counter to drm_connector -:55: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #55: FILE: drivers/gpu/drm/drm_connector.c:1897: + DRM_DEBUG_KMS("[CONNECTOR:%d:%s] Edid was changed.\n", + connector->base.id, connector->name); -:59: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #59: FILE: drivers/gpu/drm/drm_connector.c:1901: + DRM_DEBUG_KMS("Updating change counter to %llu\n", + connector->epoch_counter); -:119: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u64' over 'uint64_t' #119: FILE: drivers/gpu/drm/drm_probe_helper.c:788: + uint64_t old_epoch_counter; -:150: WARNING:BRACES: braces {} are not necessary for single statement blocks #150: FILE: drivers/gpu/drm/drm_probe_helper.c:824: + if (old_epoch_counter != connector->epoch_counter) { changed = true; + } -:174: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u64' over 'uint64_t' #174: FILE: include/drm/drm_connector.h:1292: + uint64_t epoch_counter; total: 0 errors, 1 warnings, 4 checks, 128 lines checked 610c89a4b314 drm/i915: Send hotplug event if edid had changed -:61: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis #61: FILE: drivers/gpu/drm/i915/display/intel_hotplug.c:299: + DRM_DEBUG_KMS("[CONNECTOR:%d:%s] status updated from %s to %s(epoch counter %llu)\n", connector->base.base.id, total: 0 errors, 0 warnings, 1 checks, 36 lines checked _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 7+ messages in thread
* ✓ Fi.CI.BAT: success for Send a hotplug when edid changes (rev7) 2019-09-06 8:26 [PATCH v6 0/3] Send a hotplug when edid changes Stanislav Lisovskiy ` (3 preceding siblings ...) 2019-09-06 8:46 ` ✗ Fi.CI.CHECKPATCH: warning for Send a hotplug when edid changes (rev7) Patchwork @ 2019-09-06 9:14 ` Patchwork 2019-09-06 10:35 ` ✓ Fi.CI.IGT: " Patchwork 5 siblings, 0 replies; 7+ messages in thread From: Patchwork @ 2019-09-06 9:14 UTC (permalink / raw) To: Stanislav Lisovskiy; +Cc: intel-gfx == Series Details == Series: Send a hotplug when edid changes (rev7) URL : https://patchwork.freedesktop.org/series/62816/ State : success == Summary == CI Bug Log - changes from CI_DRM_6841 -> Patchwork_14299 ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/ Known issues ------------ Here are the changes found in Patchwork_14299 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_exec_suspend@basic-s4-devices: - fi-blb-e6850: [PASS][1] -> [INCOMPLETE][2] ([fdo#107718]) [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/fi-blb-e6850/igt@gem_exec_suspend@basic-s4-devices.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/fi-blb-e6850/igt@gem_exec_suspend@basic-s4-devices.html * igt@i915_selftest@live_gem_contexts: - fi-kbl-guc: [PASS][3] -> [INCOMPLETE][4] ([fdo#111519]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/fi-kbl-guc/igt@i915_selftest@live_gem_contexts.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/fi-kbl-guc/igt@i915_selftest@live_gem_contexts.html * igt@kms_frontbuffer_tracking@basic: - fi-icl-u2: [PASS][5] -> [FAIL][6] ([fdo#103167]) [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/fi-icl-u2/igt@kms_frontbuffer_tracking@basic.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/fi-icl-u2/igt@kms_frontbuffer_tracking@basic.html #### Possible fixes #### * igt@i915_pm_rpm@basic-rte: - {fi-icl-guc}: [DMESG-WARN][7] -> [PASS][8] [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/fi-icl-guc/igt@i915_pm_rpm@basic-rte.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/fi-icl-guc/igt@i915_pm_rpm@basic-rte.html * igt@i915_selftest@live_execlists: - fi-skl-gvtdvm: [DMESG-FAIL][9] ([fdo#111108]) -> [PASS][10] [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/fi-skl-gvtdvm/igt@i915_selftest@live_execlists.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/fi-skl-gvtdvm/igt@i915_selftest@live_execlists.html * igt@kms_chamelium@dp-crc-fast: - fi-cml-u2: [FAIL][11] ([fdo#110627]) -> [PASS][12] [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/fi-cml-u2/igt@kms_chamelium@dp-crc-fast.html [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/fi-cml-u2/igt@kms_chamelium@dp-crc-fast.html * igt@kms_frontbuffer_tracking@basic: - {fi-icl-u4}: [FAIL][13] ([fdo#103167]) -> [PASS][14] [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/fi-icl-u4/igt@kms_frontbuffer_tracking@basic.html [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/fi-icl-u4/igt@kms_frontbuffer_tracking@basic.html * igt@vgem_basic@unload: - fi-icl-u3: [DMESG-WARN][15] ([fdo#107724]) -> [PASS][16] [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/fi-icl-u3/igt@vgem_basic@unload.html [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/fi-icl-u3/igt@vgem_basic@unload.html {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167 [fdo#107718]: https://bugs.freedesktop.org/show_bug.cgi?id=107718 [fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724 [fdo#110627]: https://bugs.freedesktop.org/show_bug.cgi?id=110627 [fdo#111108]: https://bugs.freedesktop.org/show_bug.cgi?id=111108 [fdo#111519]: https://bugs.freedesktop.org/show_bug.cgi?id=111519 Participating hosts (53 -> 47) ------------------------------ Additional (1): fi-tgl-u Missing (7): fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-icl-y fi-byt-clapper fi-bdw-samus Build changes ------------- * CI: CI-20190529 -> None * Linux: CI_DRM_6841 -> Patchwork_14299 CI-20190529: 20190529 CI_DRM_6841: 5c24bcfb9c6036b32dbfdbc22d773473880ff498 @ git://anongit.freedesktop.org/gfx-ci/linux IGT_5171: 1911564805fe454919e8a5846534a0c1ef376a33 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools Patchwork_14299: 610c89a4b314bc74f621f97988ea5e309399fccd @ git://anongit.freedesktop.org/gfx-ci/linux == Linux commits == 610c89a4b314 drm/i915: Send hotplug event if edid had changed 701f9863dd56 drm: Introduce epoch counter to drm_connector c2072e1d23ff drm: Add helper to compare edids. == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/ _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 7+ messages in thread
* ✓ Fi.CI.IGT: success for Send a hotplug when edid changes (rev7) 2019-09-06 8:26 [PATCH v6 0/3] Send a hotplug when edid changes Stanislav Lisovskiy ` (4 preceding siblings ...) 2019-09-06 9:14 ` ✓ Fi.CI.BAT: success " Patchwork @ 2019-09-06 10:35 ` Patchwork 5 siblings, 0 replies; 7+ messages in thread From: Patchwork @ 2019-09-06 10:35 UTC (permalink / raw) To: Stanislav Lisovskiy; +Cc: intel-gfx == Series Details == Series: Send a hotplug when edid changes (rev7) URL : https://patchwork.freedesktop.org/series/62816/ State : success == Summary == CI Bug Log - changes from CI_DRM_6841_full -> Patchwork_14299_full ==================================================== Summary ------- **SUCCESS** No regressions found. Known issues ------------ Here are the changes found in Patchwork_14299_full that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_ctx_isolation@rcs0-s3: - shard-iclb: [PASS][1] -> [INCOMPLETE][2] ([fdo#107713] / [fdo#109100]) [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-iclb4/igt@gem_ctx_isolation@rcs0-s3.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-iclb3/igt@gem_ctx_isolation@rcs0-s3.html * igt@gem_exec_async@concurrent-writes-bsd: - shard-iclb: [PASS][3] -> [SKIP][4] ([fdo#111325]) +4 similar issues [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-iclb8/igt@gem_exec_async@concurrent-writes-bsd.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-iclb2/igt@gem_exec_async@concurrent-writes-bsd.html * igt@gem_exec_schedule@preempt-queue-bsd1: - shard-iclb: [PASS][5] -> [SKIP][6] ([fdo#109276]) +21 similar issues [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-iclb4/igt@gem_exec_schedule@preempt-queue-bsd1.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-iclb3/igt@gem_exec_schedule@preempt-queue-bsd1.html * igt@i915_suspend@fence-restore-tiled2untiled: - shard-apl: [PASS][7] -> [DMESG-WARN][8] ([fdo#108566]) +5 similar issues [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-apl8/igt@i915_suspend@fence-restore-tiled2untiled.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-apl3/igt@i915_suspend@fence-restore-tiled2untiled.html * igt@kms_flip@dpms-vs-vblank-race: - shard-glk: [PASS][9] -> [FAIL][10] ([fdo#103060]) [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-glk5/igt@kms_flip@dpms-vs-vblank-race.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-glk9/igt@kms_flip@dpms-vs-vblank-race.html * igt@kms_flip@flip-vs-expired-vblank-interruptible: - shard-skl: [PASS][11] -> [FAIL][12] ([fdo#105363]) [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-skl5/igt@kms_flip@flip-vs-expired-vblank-interruptible.html [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-skl6/igt@kms_flip@flip-vs-expired-vblank-interruptible.html * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-pri-shrfb-draw-mmap-gtt: - shard-iclb: [PASS][13] -> [FAIL][14] ([fdo#103167]) [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-iclb8/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-pri-shrfb-draw-mmap-gtt.html [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-iclb2/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-pri-shrfb-draw-mmap-gtt.html * igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-onoff: - shard-skl: [PASS][15] -> [FAIL][16] ([fdo#103167]) [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-skl5/igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-onoff.html [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-skl7/igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-onoff.html * igt@kms_plane_alpha_blend@pipe-a-constant-alpha-min: - shard-skl: [PASS][17] -> [FAIL][18] ([fdo#108145]) [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-skl5/igt@kms_plane_alpha_blend@pipe-a-constant-alpha-min.html [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-skl7/igt@kms_plane_alpha_blend@pipe-a-constant-alpha-min.html * igt@kms_plane_lowres@pipe-a-tiling-x: - shard-iclb: [PASS][19] -> [FAIL][20] ([fdo#103166]) [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-iclb2/igt@kms_plane_lowres@pipe-a-tiling-x.html [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-iclb7/igt@kms_plane_lowres@pipe-a-tiling-x.html * igt@kms_psr@psr2_sprite_mmap_gtt: - shard-iclb: [PASS][21] -> [SKIP][22] ([fdo#109441]) +1 similar issue [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-iclb2/igt@kms_psr@psr2_sprite_mmap_gtt.html [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-iclb5/igt@kms_psr@psr2_sprite_mmap_gtt.html * igt@kms_setmode@basic: - shard-hsw: [PASS][23] -> [FAIL][24] ([fdo#99912]) [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-hsw7/igt@kms_setmode@basic.html [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-hsw7/igt@kms_setmode@basic.html #### Possible fixes #### * igt@gem_ctx_isolation@rcs0-s3: - shard-apl: [DMESG-WARN][25] ([fdo#108566]) -> [PASS][26] +3 similar issues [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-apl7/igt@gem_ctx_isolation@rcs0-s3.html [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-apl8/igt@gem_ctx_isolation@rcs0-s3.html * igt@gem_eio@unwedge-stress: - shard-snb: [FAIL][27] ([fdo#109661]) -> [PASS][28] [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-snb2/igt@gem_eio@unwedge-stress.html [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-snb5/igt@gem_eio@unwedge-stress.html * igt@gem_exec_schedule@preempt-other-chain-bsd: - shard-iclb: [SKIP][29] ([fdo#111325]) -> [PASS][30] +8 similar issues [29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-iclb1/igt@gem_exec_schedule@preempt-other-chain-bsd.html [30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-iclb3/igt@gem_exec_schedule@preempt-other-chain-bsd.html * igt@gem_exec_suspend@basic-s3: - shard-hsw: [FAIL][31] ([fdo#103375]) -> [PASS][32] +5 similar issues [31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-hsw2/igt@gem_exec_suspend@basic-s3.html [32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-hsw4/igt@gem_exec_suspend@basic-s3.html * igt@gem_exec_suspend@basic-s4-devices: - shard-hsw: [FAIL][33] ([fdo#111550]) -> [PASS][34] [33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-hsw2/igt@gem_exec_suspend@basic-s4-devices.html [34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-hsw7/igt@gem_exec_suspend@basic-s4-devices.html * igt@gem_workarounds@suspend-resume-context: - shard-kbl: [INCOMPLETE][35] ([fdo#103665]) -> [PASS][36] [35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-kbl4/igt@gem_workarounds@suspend-resume-context.html [36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-kbl2/igt@gem_workarounds@suspend-resume-context.html * igt@i915_pm_rc6_residency@rc6-accuracy: - shard-kbl: [SKIP][37] ([fdo#109271]) -> [PASS][38] [37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-kbl2/igt@i915_pm_rc6_residency@rc6-accuracy.html [38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-kbl6/igt@i915_pm_rc6_residency@rc6-accuracy.html - shard-snb: [SKIP][39] ([fdo#109271]) -> [PASS][40] [39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-snb5/igt@i915_pm_rc6_residency@rc6-accuracy.html [40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-snb6/igt@i915_pm_rc6_residency@rc6-accuracy.html * igt@i915_pm_rpm@system-suspend: - shard-hsw: [FAIL][41] ([fdo#111548]) -> [PASS][42] +5 similar issues [41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-hsw2/igt@i915_pm_rpm@system-suspend.html [42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-hsw2/igt@i915_pm_rpm@system-suspend.html * igt@kms_cursor_crc@pipe-c-cursor-suspend: - shard-skl: [INCOMPLETE][43] ([fdo#110741]) -> [PASS][44] +1 similar issue [43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-skl4/igt@kms_cursor_crc@pipe-c-cursor-suspend.html [44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-skl4/igt@kms_cursor_crc@pipe-c-cursor-suspend.html * igt@kms_flip@plain-flip-fb-recreate: - shard-skl: [FAIL][45] ([fdo#100368]) -> [PASS][46] [45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-skl6/igt@kms_flip@plain-flip-fb-recreate.html [46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-skl7/igt@kms_flip@plain-flip-fb-recreate.html * igt@kms_frontbuffer_tracking@fbc-stridechange: - shard-iclb: [FAIL][47] ([fdo#103167]) -> [PASS][48] +1 similar issue [47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-iclb7/igt@kms_frontbuffer_tracking@fbc-stridechange.html [48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-iclb8/igt@kms_frontbuffer_tracking@fbc-stridechange.html * igt@kms_plane_alpha_blend@pipe-b-coverage-7efc: - shard-skl: [FAIL][49] ([fdo#108145] / [fdo#110403]) -> [PASS][50] [49]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-skl7/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html [50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-skl3/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html * igt@kms_psr2_su@page_flip: - shard-iclb: [SKIP][51] ([fdo#109642] / [fdo#111068]) -> [PASS][52] [51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-iclb6/igt@kms_psr2_su@page_flip.html [52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-iclb2/igt@kms_psr2_su@page_flip.html * igt@kms_psr@psr2_no_drrs: - shard-iclb: [SKIP][53] ([fdo#109441]) -> [PASS][54] +1 similar issue [53]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-iclb8/igt@kms_psr@psr2_no_drrs.html [54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-iclb2/igt@kms_psr@psr2_no_drrs.html * igt@perf@blocking: - shard-skl: [FAIL][55] ([fdo#110728]) -> [PASS][56] [55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-skl6/igt@perf@blocking.html [56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-skl7/igt@perf@blocking.html * igt@prime_vgem@fence-wait-bsd2: - shard-iclb: [SKIP][57] ([fdo#109276]) -> [PASS][58] +16 similar issues [57]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-iclb6/igt@prime_vgem@fence-wait-bsd2.html [58]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-iclb4/igt@prime_vgem@fence-wait-bsd2.html #### Warnings #### * igt@gem_mocs_settings@mocs-rc6-bsd2: - shard-iclb: [FAIL][59] ([fdo#111330]) -> [SKIP][60] ([fdo#109276]) [59]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-iclb4/igt@gem_mocs_settings@mocs-rc6-bsd2.html [60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-iclb6/igt@gem_mocs_settings@mocs-rc6-bsd2.html * igt@gem_mocs_settings@mocs-settings-bsd2: - shard-iclb: [SKIP][61] ([fdo#109276]) -> [FAIL][62] ([fdo#111330]) [61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-iclb5/igt@gem_mocs_settings@mocs-settings-bsd2.html [62]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-iclb4/igt@gem_mocs_settings@mocs-settings-bsd2.html * igt@i915_pm_rpm@modeset-lpsp-stress: - shard-hsw: [FAIL][63] ([fdo#111548]) -> [SKIP][64] ([fdo#109271]) +1 similar issue [63]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-hsw2/igt@i915_pm_rpm@modeset-lpsp-stress.html [64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-hsw1/igt@i915_pm_rpm@modeset-lpsp-stress.html * igt@perf_pmu@cpu-hotplug: - shard-hsw: [TIMEOUT][65] ([fdo#111546]) -> [INCOMPLETE][66] ([fdo#103540]) [65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6841/shard-hsw2/igt@perf_pmu@cpu-hotplug.html [66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14299/shard-hsw6/igt@perf_pmu@cpu-hotplug.html [fdo#100368]: https://bugs.freedesktop.org/show_bug.cgi?id=100368 [fdo#103060]: https://bugs.freedesktop.org/show_bug.cgi?id=103060 [fdo#103166]: https://bugs.freedesktop.org/show_bug.cgi?id=103166 [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167 [fdo#103375]: https://bugs.freedesktop.org/show_bug.cgi?id=103375 [fdo#103540]: https://bugs.freedesktop.org/show_bug.cgi?id=103540 [fdo#103665]: https://bugs.freedesktop.org/show_bug.cgi?id=103665 [fdo#105363]: https://bugs.freedesktop.org/show_bug.cgi?id=105363 [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713 [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145 [fdo#108566]: https://bugs.freedesktop.org/show_bug.cgi?id=108566 [fdo#109100]: https://bugs.freedesktop.org/show_bug.cgi?id=109100 [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271 [fdo#109276]: https://bugs.freedesktop.org/show_bug.cgi?id=109276 [fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441 [fdo#109642]: https://bugs.freedesktop.org/show_bug.cgi?id=109642 [fdo#109661]: https://bugs.freedesktop.org/show_bug.cgi?id=109661 [fdo#110403]: https://bugs.freedesktop.org/show_bug.cgi?id=110403 [fdo#110728]: https://bugs.freedesktop.org/show_bug.cgi?id=110728 [fdo#110741]: https://bugs.freedesktop.org/show_bug.cgi?id=110741 [fdo#111068]: https://bugs.freedesktop.org/show_bug.cgi?id=111068 [fdo#111325]: https://bugs.freedesktop.org/show_bug.cgi?id=111325 [fdo#111330]: https://bugs.freedesktop.org/show_bug.cgi?id=111330 [fdo#111546]: https://bugs.freedesktop.org/show_bug.cgi?id=111546 [fdo#111548]: https://bugs.freedesktop.org/show_bug.cgi?id=111548 [fdo#111550]: https://bugs.freedesktop.org/show_bug.cgi?id=111550 [fdo#99912]: https://bugs.freedesktop.org/show_bug.cgi?id=99912 Participating hosts (10 -> 10) ------------------------------ No changes in participating hosts Build changes ------------- * CI: CI-20190529 -> None * Linux: CI_DRM_6841 -> Patchwork_14299 CI-20190529: 20190529 CI_DRM_6841: 5c24bcfb9c6036b32dbfdbc22d773473880ff498 @ git://anongit.freedesktop.org/gfx-ci/linux IGT_5171: 1911564805fe454919e8a5846534a0c1ef376a33 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools Patchwork_14299: 610c89a4b314bc74f621f97988ea5e309399fccd @ 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_14299/ _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2019-09-06 10:35 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-09-06 8:26 [PATCH v6 0/3] Send a hotplug when edid changes Stanislav Lisovskiy 2019-09-06 8:26 ` [PATCH v6 1/3] drm: Add helper to compare edids Stanislav Lisovskiy 2019-09-06 8:26 ` [PATCH v6 2/3] drm: Introduce epoch counter to drm_connector Stanislav Lisovskiy 2019-09-06 8:26 ` [PATCH v6 3/3] drm/i915: Send hotplug event if edid had changed Stanislav Lisovskiy 2019-09-06 8:46 ` ✗ Fi.CI.CHECKPATCH: warning for Send a hotplug when edid changes (rev7) Patchwork 2019-09-06 9:14 ` ✓ Fi.CI.BAT: success " Patchwork 2019-09-06 10:35 ` ✓ Fi.CI.IGT: " Patchwork
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.