* [PATCH 1/3] drm/dp: Include the AUX CH name in the debug messages @ 2020-01-23 15:45 ` Ville Syrjala 0 siblings, 0 replies; 14+ messages in thread From: Ville Syrjala @ 2020-01-23 15:45 UTC (permalink / raw) To: dri-devel; +Cc: intel-gfx From: Ville Syrjälä <ville.syrjala@linux.intel.com> To make it easier to figure out what caused a particular debug message let's print out aux->name. Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> --- drivers/gpu/drm/drm_dp_helper.c | 44 +++++++++++++++++++-------------- 1 file changed, 25 insertions(+), 19 deletions(-) diff --git a/drivers/gpu/drm/drm_dp_helper.c b/drivers/gpu/drm/drm_dp_helper.c index 5a103e9b3c86..f8cedbc3da5d 100644 --- a/drivers/gpu/drm/drm_dp_helper.c +++ b/drivers/gpu/drm/drm_dp_helper.c @@ -257,7 +257,8 @@ static int drm_dp_dpcd_access(struct drm_dp_aux *aux, u8 request, err = ret; } - DRM_DEBUG_KMS("Too many retries, giving up. First error: %d\n", err); + DRM_DEBUG_KMS("%s: Too many retries, giving up. First error: %d\n", + aux->name, err); ret = err; unlock: @@ -678,10 +679,11 @@ static int drm_dp_i2c_do_msg(struct drm_dp_aux *aux, struct drm_dp_aux_msg *msg) * Avoid spamming the kernel log with timeout errors. */ if (ret == -ETIMEDOUT) - DRM_DEBUG_KMS_RATELIMITED("transaction timed out\n"); + DRM_DEBUG_KMS_RATELIMITED("%s: transaction timed out\n", + aux->name); else - DRM_DEBUG_KMS("transaction failed: %d\n", ret); - + DRM_DEBUG_KMS("%s: transaction failed: %d\n", + aux->name, ret); return ret; } @@ -695,11 +697,12 @@ static int drm_dp_i2c_do_msg(struct drm_dp_aux *aux, struct drm_dp_aux_msg *msg) break; case DP_AUX_NATIVE_REPLY_NACK: - DRM_DEBUG_KMS("native nack (result=%d, size=%zu)\n", ret, msg->size); + DRM_DEBUG_KMS("%s: native nack (result=%d, size=%zu)\n", + aux->name, ret, msg->size); return -EREMOTEIO; case DP_AUX_NATIVE_REPLY_DEFER: - DRM_DEBUG_KMS("native defer\n"); + DRM_DEBUG_KMS("%s: native defer\n", aux->name); /* * We could check for I2C bit rate capabilities and if * available adjust this interval. We could also be @@ -713,7 +716,8 @@ static int drm_dp_i2c_do_msg(struct drm_dp_aux *aux, struct drm_dp_aux_msg *msg) continue; default: - DRM_ERROR("invalid native reply %#04x\n", msg->reply); + DRM_ERROR("%s: invalid native reply %#04x\n", + aux->name, msg->reply); return -EREMOTEIO; } @@ -728,13 +732,13 @@ static int drm_dp_i2c_do_msg(struct drm_dp_aux *aux, struct drm_dp_aux_msg *msg) return ret; case DP_AUX_I2C_REPLY_NACK: - DRM_DEBUG_KMS("I2C nack (result=%d, size=%zu)\n", - ret, msg->size); + DRM_DEBUG_KMS("%s: I2C nack (result=%d, size=%zu)\n", + aux->name, ret, msg->size); aux->i2c_nack_count++; return -EREMOTEIO; case DP_AUX_I2C_REPLY_DEFER: - DRM_DEBUG_KMS("I2C defer\n"); + DRM_DEBUG_KMS("%s: I2C defer\n", aux->name); /* DP Compliance Test 4.2.2.5 Requirement: * Must have at least 7 retries for I2C defers on the * transaction to pass this test @@ -748,12 +752,13 @@ static int drm_dp_i2c_do_msg(struct drm_dp_aux *aux, struct drm_dp_aux_msg *msg) continue; default: - DRM_ERROR("invalid I2C reply %#04x\n", msg->reply); + DRM_ERROR("%s: invalid I2C reply %#04x\n", + aux->name, msg->reply); return -EREMOTEIO; } } - DRM_DEBUG_KMS("too many retries, giving up\n"); + DRM_DEBUG_KMS("%s: Too many retries, giving up\n", aux->name); return -EREMOTEIO; } @@ -782,8 +787,8 @@ static int drm_dp_i2c_drain_msg(struct drm_dp_aux *aux, struct drm_dp_aux_msg *o return err == 0 ? -EPROTO : err; if (err < msg.size && err < ret) { - DRM_DEBUG_KMS("Partial I2C reply: requested %zu bytes got %d bytes\n", - msg.size, err); + DRM_DEBUG_KMS("%s: Partial I2C reply: requested %zu bytes got %d bytes\n", + aux->name, msg.size, err); ret = err; } @@ -962,11 +967,12 @@ static void drm_dp_aux_crc_work(struct work_struct *work) } if (ret == -EAGAIN) { - DRM_DEBUG_KMS("Get CRC failed after retrying: %d\n", - ret); + DRM_DEBUG_KMS("%s: Get CRC failed after retrying: %d\n", + aux->name, ret); continue; } else if (ret) { - DRM_DEBUG_KMS("Failed to get a CRC: %d\n", ret); + DRM_DEBUG_KMS("%s: Failed to get a CRC: %d\n", + aux->name, ret); continue; } @@ -1247,8 +1253,8 @@ int drm_dp_read_desc(struct drm_dp_aux *aux, struct drm_dp_desc *desc, dev_id_len = strnlen(ident->device_id, sizeof(ident->device_id)); - DRM_DEBUG_KMS("DP %s: OUI %*phD dev-ID %*pE HW-rev %d.%d SW-rev %d.%d quirks 0x%04x\n", - is_branch ? "branch" : "sink", + DRM_DEBUG_KMS("%s: DP %s: OUI %*phD dev-ID %*pE HW-rev %d.%d SW-rev %d.%d quirks 0x%04x\n", + aux->name, is_branch ? "branch" : "sink", (int)sizeof(ident->oui), ident->oui, dev_id_len, ident->device_id, ident->hw_rev >> 4, ident->hw_rev & 0xf, -- 2.24.1 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [Intel-gfx] [PATCH 1/3] drm/dp: Include the AUX CH name in the debug messages @ 2020-01-23 15:45 ` Ville Syrjala 0 siblings, 0 replies; 14+ messages in thread From: Ville Syrjala @ 2020-01-23 15:45 UTC (permalink / raw) To: dri-devel; +Cc: intel-gfx From: Ville Syrjälä <ville.syrjala@linux.intel.com> To make it easier to figure out what caused a particular debug message let's print out aux->name. Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> --- drivers/gpu/drm/drm_dp_helper.c | 44 +++++++++++++++++++-------------- 1 file changed, 25 insertions(+), 19 deletions(-) diff --git a/drivers/gpu/drm/drm_dp_helper.c b/drivers/gpu/drm/drm_dp_helper.c index 5a103e9b3c86..f8cedbc3da5d 100644 --- a/drivers/gpu/drm/drm_dp_helper.c +++ b/drivers/gpu/drm/drm_dp_helper.c @@ -257,7 +257,8 @@ static int drm_dp_dpcd_access(struct drm_dp_aux *aux, u8 request, err = ret; } - DRM_DEBUG_KMS("Too many retries, giving up. First error: %d\n", err); + DRM_DEBUG_KMS("%s: Too many retries, giving up. First error: %d\n", + aux->name, err); ret = err; unlock: @@ -678,10 +679,11 @@ static int drm_dp_i2c_do_msg(struct drm_dp_aux *aux, struct drm_dp_aux_msg *msg) * Avoid spamming the kernel log with timeout errors. */ if (ret == -ETIMEDOUT) - DRM_DEBUG_KMS_RATELIMITED("transaction timed out\n"); + DRM_DEBUG_KMS_RATELIMITED("%s: transaction timed out\n", + aux->name); else - DRM_DEBUG_KMS("transaction failed: %d\n", ret); - + DRM_DEBUG_KMS("%s: transaction failed: %d\n", + aux->name, ret); return ret; } @@ -695,11 +697,12 @@ static int drm_dp_i2c_do_msg(struct drm_dp_aux *aux, struct drm_dp_aux_msg *msg) break; case DP_AUX_NATIVE_REPLY_NACK: - DRM_DEBUG_KMS("native nack (result=%d, size=%zu)\n", ret, msg->size); + DRM_DEBUG_KMS("%s: native nack (result=%d, size=%zu)\n", + aux->name, ret, msg->size); return -EREMOTEIO; case DP_AUX_NATIVE_REPLY_DEFER: - DRM_DEBUG_KMS("native defer\n"); + DRM_DEBUG_KMS("%s: native defer\n", aux->name); /* * We could check for I2C bit rate capabilities and if * available adjust this interval. We could also be @@ -713,7 +716,8 @@ static int drm_dp_i2c_do_msg(struct drm_dp_aux *aux, struct drm_dp_aux_msg *msg) continue; default: - DRM_ERROR("invalid native reply %#04x\n", msg->reply); + DRM_ERROR("%s: invalid native reply %#04x\n", + aux->name, msg->reply); return -EREMOTEIO; } @@ -728,13 +732,13 @@ static int drm_dp_i2c_do_msg(struct drm_dp_aux *aux, struct drm_dp_aux_msg *msg) return ret; case DP_AUX_I2C_REPLY_NACK: - DRM_DEBUG_KMS("I2C nack (result=%d, size=%zu)\n", - ret, msg->size); + DRM_DEBUG_KMS("%s: I2C nack (result=%d, size=%zu)\n", + aux->name, ret, msg->size); aux->i2c_nack_count++; return -EREMOTEIO; case DP_AUX_I2C_REPLY_DEFER: - DRM_DEBUG_KMS("I2C defer\n"); + DRM_DEBUG_KMS("%s: I2C defer\n", aux->name); /* DP Compliance Test 4.2.2.5 Requirement: * Must have at least 7 retries for I2C defers on the * transaction to pass this test @@ -748,12 +752,13 @@ static int drm_dp_i2c_do_msg(struct drm_dp_aux *aux, struct drm_dp_aux_msg *msg) continue; default: - DRM_ERROR("invalid I2C reply %#04x\n", msg->reply); + DRM_ERROR("%s: invalid I2C reply %#04x\n", + aux->name, msg->reply); return -EREMOTEIO; } } - DRM_DEBUG_KMS("too many retries, giving up\n"); + DRM_DEBUG_KMS("%s: Too many retries, giving up\n", aux->name); return -EREMOTEIO; } @@ -782,8 +787,8 @@ static int drm_dp_i2c_drain_msg(struct drm_dp_aux *aux, struct drm_dp_aux_msg *o return err == 0 ? -EPROTO : err; if (err < msg.size && err < ret) { - DRM_DEBUG_KMS("Partial I2C reply: requested %zu bytes got %d bytes\n", - msg.size, err); + DRM_DEBUG_KMS("%s: Partial I2C reply: requested %zu bytes got %d bytes\n", + aux->name, msg.size, err); ret = err; } @@ -962,11 +967,12 @@ static void drm_dp_aux_crc_work(struct work_struct *work) } if (ret == -EAGAIN) { - DRM_DEBUG_KMS("Get CRC failed after retrying: %d\n", - ret); + DRM_DEBUG_KMS("%s: Get CRC failed after retrying: %d\n", + aux->name, ret); continue; } else if (ret) { - DRM_DEBUG_KMS("Failed to get a CRC: %d\n", ret); + DRM_DEBUG_KMS("%s: Failed to get a CRC: %d\n", + aux->name, ret); continue; } @@ -1247,8 +1253,8 @@ int drm_dp_read_desc(struct drm_dp_aux *aux, struct drm_dp_desc *desc, dev_id_len = strnlen(ident->device_id, sizeof(ident->device_id)); - DRM_DEBUG_KMS("DP %s: OUI %*phD dev-ID %*pE HW-rev %d.%d SW-rev %d.%d quirks 0x%04x\n", - is_branch ? "branch" : "sink", + DRM_DEBUG_KMS("%s: DP %s: OUI %*phD dev-ID %*pE HW-rev %d.%d SW-rev %d.%d quirks 0x%04x\n", + aux->name, is_branch ? "branch" : "sink", (int)sizeof(ident->oui), ident->oui, dev_id_len, ident->device_id, ident->hw_rev >> 4, ident->hw_rev & 0xf, -- 2.24.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 2/3] drm/i915: Include the AUX CH name in the debug messages 2020-01-23 15:45 ` [Intel-gfx] " Ville Syrjala @ 2020-01-23 15:45 ` Ville Syrjala -1 siblings, 0 replies; 14+ messages in thread From: Ville Syrjala @ 2020-01-23 15:45 UTC (permalink / raw) To: dri-devel; +Cc: intel-gfx From: Ville Syrjälä <ville.syrjala@linux.intel.com> To make it easier to figure out what caused a particular debug message let's print out aux->name. Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> --- drivers/gpu/drm/i915/display/intel_dp.c | 19 +++++++++++-------- 1 file changed, 11 insertions(+), 8 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c index 1795963e1866..941f0f6d55c1 100644 --- a/drivers/gpu/drm/i915/display/intel_dp.c +++ b/drivers/gpu/drm/i915/display/intel_dp.c @@ -1186,7 +1186,7 @@ intel_dp_aux_wait_done(struct intel_dp *intel_dp) trace_i915_reg_rw(false, ch_ctl, status, sizeof(status), true); if (!done) - DRM_ERROR("%s did not complete or timeout within %ums (status 0x%08x)\n", + DRM_ERROR("%s: did not complete or timeout within %ums (status 0x%08x)\n", intel_dp->aux.name, timeout_ms, status); #undef C @@ -1373,8 +1373,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, const u32 status = intel_uncore_read(uncore, ch_ctl); if (status != intel_dp->aux_busy_last_status) { - WARN(1, "dp_aux_ch not started status 0x%08x\n", - status); + WARN(1, "%s: not started status 0x%08x\n", + intel_dp->aux.name, status); intel_dp->aux_busy_last_status = status; } @@ -1435,7 +1435,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, } if ((status & DP_AUX_CH_CTL_DONE) == 0) { - DRM_ERROR("dp_aux_ch not done status 0x%08x\n", status); + DRM_ERROR("%s: not done status 0x%08x\n", + intel_dp->aux.name, status); ret = -EBUSY; goto out; } @@ -1445,7 +1446,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, * Timeouts occur when the sink is not connected */ if (status & DP_AUX_CH_CTL_RECEIVE_ERROR) { - DRM_ERROR("dp_aux_ch receive error status 0x%08x\n", status); + DRM_ERROR("%s: receive error status 0x%08x\n", + intel_dp->aux.name, status); ret = -EIO; goto out; } @@ -1453,7 +1455,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, /* Timeouts occur when the device isn't connected, so they're * "normal" -- don't fill the kernel log with these */ if (status & DP_AUX_CH_CTL_TIME_OUT_ERROR) { - DRM_DEBUG_KMS("dp_aux_ch timeout status 0x%08x\n", status); + DRM_DEBUG_KMS("%s: timeout status 0x%08x\n", + intel_dp->aux.name, status); ret = -ETIMEDOUT; goto out; } @@ -1468,8 +1471,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, * drm layer takes care for the necessary retries. */ if (recv_bytes == 0 || recv_bytes > 20) { - DRM_DEBUG_KMS("Forbidden recv_bytes = %d on aux transaction\n", - recv_bytes); + DRM_DEBUG_KMS("%s: Forbidden recv_bytes = %d on aux transaction\n", + intel_dp->aux.name, recv_bytes); ret = -EBUSY; goto out; } -- 2.24.1 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [Intel-gfx] [PATCH 2/3] drm/i915: Include the AUX CH name in the debug messages @ 2020-01-23 15:45 ` Ville Syrjala 0 siblings, 0 replies; 14+ messages in thread From: Ville Syrjala @ 2020-01-23 15:45 UTC (permalink / raw) To: dri-devel; +Cc: intel-gfx From: Ville Syrjälä <ville.syrjala@linux.intel.com> To make it easier to figure out what caused a particular debug message let's print out aux->name. Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> --- drivers/gpu/drm/i915/display/intel_dp.c | 19 +++++++++++-------- 1 file changed, 11 insertions(+), 8 deletions(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c index 1795963e1866..941f0f6d55c1 100644 --- a/drivers/gpu/drm/i915/display/intel_dp.c +++ b/drivers/gpu/drm/i915/display/intel_dp.c @@ -1186,7 +1186,7 @@ intel_dp_aux_wait_done(struct intel_dp *intel_dp) trace_i915_reg_rw(false, ch_ctl, status, sizeof(status), true); if (!done) - DRM_ERROR("%s did not complete or timeout within %ums (status 0x%08x)\n", + DRM_ERROR("%s: did not complete or timeout within %ums (status 0x%08x)\n", intel_dp->aux.name, timeout_ms, status); #undef C @@ -1373,8 +1373,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, const u32 status = intel_uncore_read(uncore, ch_ctl); if (status != intel_dp->aux_busy_last_status) { - WARN(1, "dp_aux_ch not started status 0x%08x\n", - status); + WARN(1, "%s: not started status 0x%08x\n", + intel_dp->aux.name, status); intel_dp->aux_busy_last_status = status; } @@ -1435,7 +1435,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, } if ((status & DP_AUX_CH_CTL_DONE) == 0) { - DRM_ERROR("dp_aux_ch not done status 0x%08x\n", status); + DRM_ERROR("%s: not done status 0x%08x\n", + intel_dp->aux.name, status); ret = -EBUSY; goto out; } @@ -1445,7 +1446,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, * Timeouts occur when the sink is not connected */ if (status & DP_AUX_CH_CTL_RECEIVE_ERROR) { - DRM_ERROR("dp_aux_ch receive error status 0x%08x\n", status); + DRM_ERROR("%s: receive error status 0x%08x\n", + intel_dp->aux.name, status); ret = -EIO; goto out; } @@ -1453,7 +1455,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, /* Timeouts occur when the device isn't connected, so they're * "normal" -- don't fill the kernel log with these */ if (status & DP_AUX_CH_CTL_TIME_OUT_ERROR) { - DRM_DEBUG_KMS("dp_aux_ch timeout status 0x%08x\n", status); + DRM_DEBUG_KMS("%s: timeout status 0x%08x\n", + intel_dp->aux.name, status); ret = -ETIMEDOUT; goto out; } @@ -1468,8 +1471,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, * drm layer takes care for the necessary retries. */ if (recv_bytes == 0 || recv_bytes > 20) { - DRM_DEBUG_KMS("Forbidden recv_bytes = %d on aux transaction\n", - recv_bytes); + DRM_DEBUG_KMS("%s: Forbidden recv_bytes = %d on aux transaction\n", + intel_dp->aux.name, recv_bytes); ret = -EBUSY; goto out; } -- 2.24.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH 2/3] drm/i915: Include the AUX CH name in the debug messages 2020-01-23 15:45 ` [Intel-gfx] " Ville Syrjala @ 2020-01-23 17:19 ` Matt Roper -1 siblings, 0 replies; 14+ messages in thread From: Matt Roper @ 2020-01-23 17:19 UTC (permalink / raw) To: Ville Syrjala; +Cc: intel-gfx, dri-devel On Thu, Jan 23, 2020 at 05:45:41PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > To make it easier to figure out what caused a particular debug > message let's print out aux->name. > > Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> It might be worth adding commas to some of these messages too while we're at it (e.g., "not started, status"). But either way, this is a really helpful cleanup. Reviewed-by: Matt Roper <matthew.d.roper@intel.com> > --- > drivers/gpu/drm/i915/display/intel_dp.c | 19 +++++++++++-------- > 1 file changed, 11 insertions(+), 8 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c > index 1795963e1866..941f0f6d55c1 100644 > --- a/drivers/gpu/drm/i915/display/intel_dp.c > +++ b/drivers/gpu/drm/i915/display/intel_dp.c > @@ -1186,7 +1186,7 @@ intel_dp_aux_wait_done(struct intel_dp *intel_dp) > trace_i915_reg_rw(false, ch_ctl, status, sizeof(status), true); > > if (!done) > - DRM_ERROR("%s did not complete or timeout within %ums (status 0x%08x)\n", > + DRM_ERROR("%s: did not complete or timeout within %ums (status 0x%08x)\n", > intel_dp->aux.name, timeout_ms, status); > #undef C > > @@ -1373,8 +1373,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, > const u32 status = intel_uncore_read(uncore, ch_ctl); > > if (status != intel_dp->aux_busy_last_status) { > - WARN(1, "dp_aux_ch not started status 0x%08x\n", > - status); > + WARN(1, "%s: not started status 0x%08x\n", > + intel_dp->aux.name, status); > intel_dp->aux_busy_last_status = status; > } > > @@ -1435,7 +1435,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, > } > > if ((status & DP_AUX_CH_CTL_DONE) == 0) { > - DRM_ERROR("dp_aux_ch not done status 0x%08x\n", status); > + DRM_ERROR("%s: not done status 0x%08x\n", > + intel_dp->aux.name, status); > ret = -EBUSY; > goto out; > } > @@ -1445,7 +1446,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, > * Timeouts occur when the sink is not connected > */ > if (status & DP_AUX_CH_CTL_RECEIVE_ERROR) { > - DRM_ERROR("dp_aux_ch receive error status 0x%08x\n", status); > + DRM_ERROR("%s: receive error status 0x%08x\n", > + intel_dp->aux.name, status); > ret = -EIO; > goto out; > } > @@ -1453,7 +1455,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, > /* Timeouts occur when the device isn't connected, so they're > * "normal" -- don't fill the kernel log with these */ > if (status & DP_AUX_CH_CTL_TIME_OUT_ERROR) { > - DRM_DEBUG_KMS("dp_aux_ch timeout status 0x%08x\n", status); > + DRM_DEBUG_KMS("%s: timeout status 0x%08x\n", > + intel_dp->aux.name, status); > ret = -ETIMEDOUT; > goto out; > } > @@ -1468,8 +1471,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, > * drm layer takes care for the necessary retries. > */ > if (recv_bytes == 0 || recv_bytes > 20) { > - DRM_DEBUG_KMS("Forbidden recv_bytes = %d on aux transaction\n", > - recv_bytes); > + DRM_DEBUG_KMS("%s: Forbidden recv_bytes = %d on aux transaction\n", > + intel_dp->aux.name, recv_bytes); > ret = -EBUSY; > goto out; > } > -- > 2.24.1 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Matt Roper Graphics Software Engineer VTT-OSGC Platform Enablement Intel Corporation (916) 356-2795 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Intel-gfx] [PATCH 2/3] drm/i915: Include the AUX CH name in the debug messages @ 2020-01-23 17:19 ` Matt Roper 0 siblings, 0 replies; 14+ messages in thread From: Matt Roper @ 2020-01-23 17:19 UTC (permalink / raw) To: Ville Syrjala; +Cc: intel-gfx, dri-devel On Thu, Jan 23, 2020 at 05:45:41PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > To make it easier to figure out what caused a particular debug > message let's print out aux->name. > > Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> It might be worth adding commas to some of these messages too while we're at it (e.g., "not started, status"). But either way, this is a really helpful cleanup. Reviewed-by: Matt Roper <matthew.d.roper@intel.com> > --- > drivers/gpu/drm/i915/display/intel_dp.c | 19 +++++++++++-------- > 1 file changed, 11 insertions(+), 8 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c > index 1795963e1866..941f0f6d55c1 100644 > --- a/drivers/gpu/drm/i915/display/intel_dp.c > +++ b/drivers/gpu/drm/i915/display/intel_dp.c > @@ -1186,7 +1186,7 @@ intel_dp_aux_wait_done(struct intel_dp *intel_dp) > trace_i915_reg_rw(false, ch_ctl, status, sizeof(status), true); > > if (!done) > - DRM_ERROR("%s did not complete or timeout within %ums (status 0x%08x)\n", > + DRM_ERROR("%s: did not complete or timeout within %ums (status 0x%08x)\n", > intel_dp->aux.name, timeout_ms, status); > #undef C > > @@ -1373,8 +1373,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, > const u32 status = intel_uncore_read(uncore, ch_ctl); > > if (status != intel_dp->aux_busy_last_status) { > - WARN(1, "dp_aux_ch not started status 0x%08x\n", > - status); > + WARN(1, "%s: not started status 0x%08x\n", > + intel_dp->aux.name, status); > intel_dp->aux_busy_last_status = status; > } > > @@ -1435,7 +1435,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, > } > > if ((status & DP_AUX_CH_CTL_DONE) == 0) { > - DRM_ERROR("dp_aux_ch not done status 0x%08x\n", status); > + DRM_ERROR("%s: not done status 0x%08x\n", > + intel_dp->aux.name, status); > ret = -EBUSY; > goto out; > } > @@ -1445,7 +1446,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, > * Timeouts occur when the sink is not connected > */ > if (status & DP_AUX_CH_CTL_RECEIVE_ERROR) { > - DRM_ERROR("dp_aux_ch receive error status 0x%08x\n", status); > + DRM_ERROR("%s: receive error status 0x%08x\n", > + intel_dp->aux.name, status); > ret = -EIO; > goto out; > } > @@ -1453,7 +1455,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, > /* Timeouts occur when the device isn't connected, so they're > * "normal" -- don't fill the kernel log with these */ > if (status & DP_AUX_CH_CTL_TIME_OUT_ERROR) { > - DRM_DEBUG_KMS("dp_aux_ch timeout status 0x%08x\n", status); > + DRM_DEBUG_KMS("%s: timeout status 0x%08x\n", > + intel_dp->aux.name, status); > ret = -ETIMEDOUT; > goto out; > } > @@ -1468,8 +1471,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, > * drm layer takes care for the necessary retries. > */ > if (recv_bytes == 0 || recv_bytes > 20) { > - DRM_DEBUG_KMS("Forbidden recv_bytes = %d on aux transaction\n", > - recv_bytes); > + DRM_DEBUG_KMS("%s: Forbidden recv_bytes = %d on aux transaction\n", > + intel_dp->aux.name, recv_bytes); > ret = -EBUSY; > goto out; > } > -- > 2.24.1 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Matt Roper Graphics Software Engineer VTT-OSGC Platform Enablement Intel Corporation (916) 356-2795 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 2/3] drm/i915: Include the AUX CH name in the debug messages 2020-01-23 17:19 ` [Intel-gfx] " Matt Roper @ 2020-01-27 16:08 ` Ville Syrjälä -1 siblings, 0 replies; 14+ messages in thread From: Ville Syrjälä @ 2020-01-27 16:08 UTC (permalink / raw) To: Matt Roper; +Cc: intel-gfx, dri-devel On Thu, Jan 23, 2020 at 09:19:04AM -0800, Matt Roper wrote: > On Thu, Jan 23, 2020 at 05:45:41PM +0200, Ville Syrjala wrote: > > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > > > To make it easier to figure out what caused a particular debug > > message let's print out aux->name. > > > > Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> > > It might be worth adding commas to some of these messages too while > we're at it (e.g., "not started, status"). But either way, this is a > really helpful cleanup. Good idea. The first one of these already uses "%s: blah (status 0x%x)" as the format, so I went ahead and converted the rest to follow the same style. > > Reviewed-by: Matt Roper <matthew.d.roper@intel.com> 2-3 pushed to dinq. Thanks for the review. > > > > --- > > drivers/gpu/drm/i915/display/intel_dp.c | 19 +++++++++++-------- > > 1 file changed, 11 insertions(+), 8 deletions(-) > > > > diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c > > index 1795963e1866..941f0f6d55c1 100644 > > --- a/drivers/gpu/drm/i915/display/intel_dp.c > > +++ b/drivers/gpu/drm/i915/display/intel_dp.c > > @@ -1186,7 +1186,7 @@ intel_dp_aux_wait_done(struct intel_dp *intel_dp) > > trace_i915_reg_rw(false, ch_ctl, status, sizeof(status), true); > > > > if (!done) > > - DRM_ERROR("%s did not complete or timeout within %ums (status 0x%08x)\n", > > + DRM_ERROR("%s: did not complete or timeout within %ums (status 0x%08x)\n", > > intel_dp->aux.name, timeout_ms, status); > > #undef C > > > > @@ -1373,8 +1373,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, > > const u32 status = intel_uncore_read(uncore, ch_ctl); > > > > if (status != intel_dp->aux_busy_last_status) { > > - WARN(1, "dp_aux_ch not started status 0x%08x\n", > > - status); > > + WARN(1, "%s: not started status 0x%08x\n", > > + intel_dp->aux.name, status); > > intel_dp->aux_busy_last_status = status; > > } > > > > @@ -1435,7 +1435,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, > > } > > > > if ((status & DP_AUX_CH_CTL_DONE) == 0) { > > - DRM_ERROR("dp_aux_ch not done status 0x%08x\n", status); > > + DRM_ERROR("%s: not done status 0x%08x\n", > > + intel_dp->aux.name, status); > > ret = -EBUSY; > > goto out; > > } > > @@ -1445,7 +1446,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, > > * Timeouts occur when the sink is not connected > > */ > > if (status & DP_AUX_CH_CTL_RECEIVE_ERROR) { > > - DRM_ERROR("dp_aux_ch receive error status 0x%08x\n", status); > > + DRM_ERROR("%s: receive error status 0x%08x\n", > > + intel_dp->aux.name, status); > > ret = -EIO; > > goto out; > > } > > @@ -1453,7 +1455,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, > > /* Timeouts occur when the device isn't connected, so they're > > * "normal" -- don't fill the kernel log with these */ > > if (status & DP_AUX_CH_CTL_TIME_OUT_ERROR) { > > - DRM_DEBUG_KMS("dp_aux_ch timeout status 0x%08x\n", status); > > + DRM_DEBUG_KMS("%s: timeout status 0x%08x\n", > > + intel_dp->aux.name, status); > > ret = -ETIMEDOUT; > > goto out; > > } > > @@ -1468,8 +1471,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, > > * drm layer takes care for the necessary retries. > > */ > > if (recv_bytes == 0 || recv_bytes > 20) { > > - DRM_DEBUG_KMS("Forbidden recv_bytes = %d on aux transaction\n", > > - recv_bytes); > > + DRM_DEBUG_KMS("%s: Forbidden recv_bytes = %d on aux transaction\n", > > + intel_dp->aux.name, recv_bytes); > > ret = -EBUSY; > > goto out; > > } > > -- > > 2.24.1 > > > > _______________________________________________ > > dri-devel mailing list > > dri-devel@lists.freedesktop.org > > https://lists.freedesktop.org/mailman/listinfo/dri-devel > > -- > Matt Roper > Graphics Software Engineer > VTT-OSGC Platform Enablement > Intel Corporation > (916) 356-2795 -- Ville Syrjälä Intel _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Intel-gfx] [PATCH 2/3] drm/i915: Include the AUX CH name in the debug messages @ 2020-01-27 16:08 ` Ville Syrjälä 0 siblings, 0 replies; 14+ messages in thread From: Ville Syrjälä @ 2020-01-27 16:08 UTC (permalink / raw) To: Matt Roper; +Cc: intel-gfx, dri-devel On Thu, Jan 23, 2020 at 09:19:04AM -0800, Matt Roper wrote: > On Thu, Jan 23, 2020 at 05:45:41PM +0200, Ville Syrjala wrote: > > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > > > To make it easier to figure out what caused a particular debug > > message let's print out aux->name. > > > > Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> > > It might be worth adding commas to some of these messages too while > we're at it (e.g., "not started, status"). But either way, this is a > really helpful cleanup. Good idea. The first one of these already uses "%s: blah (status 0x%x)" as the format, so I went ahead and converted the rest to follow the same style. > > Reviewed-by: Matt Roper <matthew.d.roper@intel.com> 2-3 pushed to dinq. Thanks for the review. > > > > --- > > drivers/gpu/drm/i915/display/intel_dp.c | 19 +++++++++++-------- > > 1 file changed, 11 insertions(+), 8 deletions(-) > > > > diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c > > index 1795963e1866..941f0f6d55c1 100644 > > --- a/drivers/gpu/drm/i915/display/intel_dp.c > > +++ b/drivers/gpu/drm/i915/display/intel_dp.c > > @@ -1186,7 +1186,7 @@ intel_dp_aux_wait_done(struct intel_dp *intel_dp) > > trace_i915_reg_rw(false, ch_ctl, status, sizeof(status), true); > > > > if (!done) > > - DRM_ERROR("%s did not complete or timeout within %ums (status 0x%08x)\n", > > + DRM_ERROR("%s: did not complete or timeout within %ums (status 0x%08x)\n", > > intel_dp->aux.name, timeout_ms, status); > > #undef C > > > > @@ -1373,8 +1373,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, > > const u32 status = intel_uncore_read(uncore, ch_ctl); > > > > if (status != intel_dp->aux_busy_last_status) { > > - WARN(1, "dp_aux_ch not started status 0x%08x\n", > > - status); > > + WARN(1, "%s: not started status 0x%08x\n", > > + intel_dp->aux.name, status); > > intel_dp->aux_busy_last_status = status; > > } > > > > @@ -1435,7 +1435,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, > > } > > > > if ((status & DP_AUX_CH_CTL_DONE) == 0) { > > - DRM_ERROR("dp_aux_ch not done status 0x%08x\n", status); > > + DRM_ERROR("%s: not done status 0x%08x\n", > > + intel_dp->aux.name, status); > > ret = -EBUSY; > > goto out; > > } > > @@ -1445,7 +1446,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, > > * Timeouts occur when the sink is not connected > > */ > > if (status & DP_AUX_CH_CTL_RECEIVE_ERROR) { > > - DRM_ERROR("dp_aux_ch receive error status 0x%08x\n", status); > > + DRM_ERROR("%s: receive error status 0x%08x\n", > > + intel_dp->aux.name, status); > > ret = -EIO; > > goto out; > > } > > @@ -1453,7 +1455,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, > > /* Timeouts occur when the device isn't connected, so they're > > * "normal" -- don't fill the kernel log with these */ > > if (status & DP_AUX_CH_CTL_TIME_OUT_ERROR) { > > - DRM_DEBUG_KMS("dp_aux_ch timeout status 0x%08x\n", status); > > + DRM_DEBUG_KMS("%s: timeout status 0x%08x\n", > > + intel_dp->aux.name, status); > > ret = -ETIMEDOUT; > > goto out; > > } > > @@ -1468,8 +1471,8 @@ intel_dp_aux_xfer(struct intel_dp *intel_dp, > > * drm layer takes care for the necessary retries. > > */ > > if (recv_bytes == 0 || recv_bytes > 20) { > > - DRM_DEBUG_KMS("Forbidden recv_bytes = %d on aux transaction\n", > > - recv_bytes); > > + DRM_DEBUG_KMS("%s: Forbidden recv_bytes = %d on aux transaction\n", > > + intel_dp->aux.name, recv_bytes); > > ret = -EBUSY; > > goto out; > > } > > -- > > 2.24.1 > > > > _______________________________________________ > > dri-devel mailing list > > dri-devel@lists.freedesktop.org > > https://lists.freedesktop.org/mailman/listinfo/dri-devel > > -- > Matt Roper > Graphics Software Engineer > VTT-OSGC Platform Enablement > Intel Corporation > (916) 356-2795 -- Ville Syrjälä Intel _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH 3/3] drm/i915: Give aux channels a better name 2020-01-23 15:45 ` [Intel-gfx] " Ville Syrjala @ 2020-01-23 15:45 ` Ville Syrjala -1 siblings, 0 replies; 14+ messages in thread From: Ville Syrjala @ 2020-01-23 15:45 UTC (permalink / raw) To: dri-devel; +Cc: intel-gfx From: Ville Syrjälä <ville.syrjala@linux.intel.com> The aux ch is used for more than DDC, so let's give it a better name. For maximum ease let's include both the AUX ch identifier and the port identifier (for cases where the VBT has redefined the relationship of the two). Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> --- drivers/gpu/drm/i915/display/intel_dp.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c index 941f0f6d55c1..b6ce8510f0ae 100644 --- a/drivers/gpu/drm/i915/display/intel_dp.c +++ b/drivers/gpu/drm/i915/display/intel_dp.c @@ -1740,7 +1740,8 @@ intel_dp_aux_init(struct intel_dp *intel_dp) drm_dp_aux_init(&intel_dp->aux); /* Failure to allocate our preferred name is not critical */ - intel_dp->aux.name = kasprintf(GFP_KERNEL, "DPDDC-%c", + intel_dp->aux.name = kasprintf(GFP_KERNEL, "AUX %c/port %c", + aux_ch_name(dig_port->aux_ch), port_name(encoder->port)); intel_dp->aux.transfer = intel_dp_aux_transfer; } -- 2.24.1 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [Intel-gfx] [PATCH 3/3] drm/i915: Give aux channels a better name @ 2020-01-23 15:45 ` Ville Syrjala 0 siblings, 0 replies; 14+ messages in thread From: Ville Syrjala @ 2020-01-23 15:45 UTC (permalink / raw) To: dri-devel; +Cc: intel-gfx From: Ville Syrjälä <ville.syrjala@linux.intel.com> The aux ch is used for more than DDC, so let's give it a better name. For maximum ease let's include both the AUX ch identifier and the port identifier (for cases where the VBT has redefined the relationship of the two). Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> --- drivers/gpu/drm/i915/display/intel_dp.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c index 941f0f6d55c1..b6ce8510f0ae 100644 --- a/drivers/gpu/drm/i915/display/intel_dp.c +++ b/drivers/gpu/drm/i915/display/intel_dp.c @@ -1740,7 +1740,8 @@ intel_dp_aux_init(struct intel_dp *intel_dp) drm_dp_aux_init(&intel_dp->aux); /* Failure to allocate our preferred name is not critical */ - intel_dp->aux.name = kasprintf(GFP_KERNEL, "DPDDC-%c", + intel_dp->aux.name = kasprintf(GFP_KERNEL, "AUX %c/port %c", + aux_ch_name(dig_port->aux_ch), port_name(encoder->port)); intel_dp->aux.transfer = intel_dp_aux_transfer; } -- 2.24.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [Intel-gfx] [PATCH 3/3] drm/i915: Give aux channels a better name 2020-01-23 15:45 ` [Intel-gfx] " Ville Syrjala @ 2020-01-23 17:22 ` Matt Roper -1 siblings, 0 replies; 14+ messages in thread From: Matt Roper @ 2020-01-23 17:22 UTC (permalink / raw) To: Ville Syrjala; +Cc: intel-gfx, dri-devel On Thu, Jan 23, 2020 at 05:45:42PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > The aux ch is used for more than DDC, so let's give it a better > name. For maximum ease let's include both the AUX ch identifier > and the port identifier (for cases where the VBT has redefined > the relationship of the two). > > Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Reviewed-by: Matt Roper <matthew.d.roper@intel.com> > --- > drivers/gpu/drm/i915/display/intel_dp.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c > index 941f0f6d55c1..b6ce8510f0ae 100644 > --- a/drivers/gpu/drm/i915/display/intel_dp.c > +++ b/drivers/gpu/drm/i915/display/intel_dp.c > @@ -1740,7 +1740,8 @@ intel_dp_aux_init(struct intel_dp *intel_dp) > drm_dp_aux_init(&intel_dp->aux); > > /* Failure to allocate our preferred name is not critical */ > - intel_dp->aux.name = kasprintf(GFP_KERNEL, "DPDDC-%c", > + intel_dp->aux.name = kasprintf(GFP_KERNEL, "AUX %c/port %c", > + aux_ch_name(dig_port->aux_ch), > port_name(encoder->port)); > intel_dp->aux.transfer = intel_dp_aux_transfer; > } > -- > 2.24.1 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Matt Roper Graphics Software Engineer VTT-OSGC Platform Enablement Intel Corporation (916) 356-2795 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Intel-gfx] [PATCH 3/3] drm/i915: Give aux channels a better name @ 2020-01-23 17:22 ` Matt Roper 0 siblings, 0 replies; 14+ messages in thread From: Matt Roper @ 2020-01-23 17:22 UTC (permalink / raw) To: Ville Syrjala; +Cc: intel-gfx, dri-devel On Thu, Jan 23, 2020 at 05:45:42PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä <ville.syrjala@linux.intel.com> > > The aux ch is used for more than DDC, so let's give it a better > name. For maximum ease let's include both the AUX ch identifier > and the port identifier (for cases where the VBT has redefined > the relationship of the two). > > Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Reviewed-by: Matt Roper <matthew.d.roper@intel.com> > --- > drivers/gpu/drm/i915/display/intel_dp.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_dp.c b/drivers/gpu/drm/i915/display/intel_dp.c > index 941f0f6d55c1..b6ce8510f0ae 100644 > --- a/drivers/gpu/drm/i915/display/intel_dp.c > +++ b/drivers/gpu/drm/i915/display/intel_dp.c > @@ -1740,7 +1740,8 @@ intel_dp_aux_init(struct intel_dp *intel_dp) > drm_dp_aux_init(&intel_dp->aux); > > /* Failure to allocate our preferred name is not critical */ > - intel_dp->aux.name = kasprintf(GFP_KERNEL, "DPDDC-%c", > + intel_dp->aux.name = kasprintf(GFP_KERNEL, "AUX %c/port %c", > + aux_ch_name(dig_port->aux_ch), > port_name(encoder->port)); > intel_dp->aux.transfer = intel_dp_aux_transfer; > } > -- > 2.24.1 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Matt Roper Graphics Software Engineer VTT-OSGC Platform Enablement Intel Corporation (916) 356-2795 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 14+ messages in thread
* [Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/3] drm/dp: Include the AUX CH name in the debug messages 2020-01-23 15:45 ` [Intel-gfx] " Ville Syrjala ` (2 preceding siblings ...) (?) @ 2020-01-24 2:56 ` Patchwork -1 siblings, 0 replies; 14+ messages in thread From: Patchwork @ 2020-01-24 2:56 UTC (permalink / raw) To: Ville Syrjälä; +Cc: intel-gfx == Series Details == Series: series starting with [1/3] drm/dp: Include the AUX CH name in the debug messages URL : https://patchwork.freedesktop.org/series/72484/ State : success == Summary == CI Bug Log - changes from CI_DRM_7806 -> Patchwork_16242 ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/index.html Known issues ------------ Here are the changes found in Patchwork_16242 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_close_race@basic-threads: - fi-byt-n2820: [PASS][1] -> [INCOMPLETE][2] ([i915#45]) [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/fi-byt-n2820/igt@gem_close_race@basic-threads.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/fi-byt-n2820/igt@gem_close_race@basic-threads.html * igt@gem_exec_suspend@basic-s4-devices: - fi-tgl-y: [PASS][3] -> [INCOMPLETE][4] ([CI#94] / [i915#460] / [i915#472]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/fi-tgl-y/igt@gem_exec_suspend@basic-s4-devices.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/fi-tgl-y/igt@gem_exec_suspend@basic-s4-devices.html * igt@kms_chamelium@dp-crc-fast: - fi-kbl-7500u: [PASS][5] -> [DMESG-WARN][6] ([i915#109]) [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html * igt@kms_frontbuffer_tracking@basic: - fi-hsw-peppy: [PASS][7] -> [DMESG-WARN][8] ([i915#44]) [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/fi-hsw-peppy/igt@kms_frontbuffer_tracking@basic.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/fi-hsw-peppy/igt@kms_frontbuffer_tracking@basic.html #### Possible fixes #### * igt@i915_selftest@live_blt: - fi-bsw-nick: [DMESG-FAIL][9] ([i915#723]) -> [PASS][10] [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/fi-bsw-nick/igt@i915_selftest@live_blt.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/fi-bsw-nick/igt@i915_selftest@live_blt.html - fi-hsw-4770r: [DMESG-FAIL][11] ([i915#563]) -> [PASS][12] [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/fi-hsw-4770r/igt@i915_selftest@live_blt.html [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/fi-hsw-4770r/igt@i915_selftest@live_blt.html * igt@kms_chamelium@hdmi-hpd-fast: - fi-kbl-7500u: [FAIL][13] ([fdo#111407]) -> [PASS][14] [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/fi-kbl-7500u/igt@kms_chamelium@hdmi-hpd-fast.html #### Warnings #### * igt@gem_close_race@basic-threads: - fi-byt-j1900: [TIMEOUT][15] ([fdo#112271] / [i915#816]) -> [INCOMPLETE][16] ([i915#45]) [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/fi-byt-j1900/igt@gem_close_race@basic-threads.html [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/fi-byt-j1900/igt@gem_close_race@basic-threads.html * igt@i915_selftest@live_blt: - fi-hsw-4770: [DMESG-FAIL][17] ([i915#770]) -> [DMESG-FAIL][18] ([i915#553] / [i915#725]) [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/fi-hsw-4770/igt@i915_selftest@live_blt.html [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/fi-hsw-4770/igt@i915_selftest@live_blt.html [CI#94]: https://gitlab.freedesktop.org/gfx-ci/i915-infra/issues/94 [fdo#111407]: https://bugs.freedesktop.org/show_bug.cgi?id=111407 [fdo#112271]: https://bugs.freedesktop.org/show_bug.cgi?id=112271 [i915#109]: https://gitlab.freedesktop.org/drm/intel/issues/109 [i915#44]: https://gitlab.freedesktop.org/drm/intel/issues/44 [i915#45]: https://gitlab.freedesktop.org/drm/intel/issues/45 [i915#460]: https://gitlab.freedesktop.org/drm/intel/issues/460 [i915#472]: https://gitlab.freedesktop.org/drm/intel/issues/472 [i915#553]: https://gitlab.freedesktop.org/drm/intel/issues/553 [i915#563]: https://gitlab.freedesktop.org/drm/intel/issues/563 [i915#723]: https://gitlab.freedesktop.org/drm/intel/issues/723 [i915#725]: https://gitlab.freedesktop.org/drm/intel/issues/725 [i915#770]: https://gitlab.freedesktop.org/drm/intel/issues/770 [i915#816]: https://gitlab.freedesktop.org/drm/intel/issues/816 Participating hosts (52 -> 45) ------------------------------ Additional (1): fi-snb-2520m Missing (8): fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 fi-bdw-samus fi-byt-clapper fi-skl-6600u Build changes ------------- * CI: CI-20190529 -> None * Linux: CI_DRM_7806 -> Patchwork_16242 CI-20190529: 20190529 CI_DRM_7806: 0b551226df5e5b84044705d5fd76571da70f3163 @ git://anongit.freedesktop.org/gfx-ci/linux IGT_5382: 8dbe5ce61baa2d563d4dd7c56a018bb1e1077467 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools Patchwork_16242: 6c82f02126337eccb6531685f600676c60c370fa @ git://anongit.freedesktop.org/gfx-ci/linux == Linux commits == 6c82f0212633 drm/i915: Give aux channels a better name adaab22e5719 drm/i915: Include the AUX CH name in the debug messages e51bae479465 drm/dp: Include the AUX CH name in the debug messages == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/index.html _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 14+ messages in thread
* [Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [1/3] drm/dp: Include the AUX CH name in the debug messages 2020-01-23 15:45 ` [Intel-gfx] " Ville Syrjala ` (3 preceding siblings ...) (?) @ 2020-01-25 20:19 ` Patchwork -1 siblings, 0 replies; 14+ messages in thread From: Patchwork @ 2020-01-25 20:19 UTC (permalink / raw) To: Ville Syrjala; +Cc: intel-gfx == Series Details == Series: series starting with [1/3] drm/dp: Include the AUX CH name in the debug messages URL : https://patchwork.freedesktop.org/series/72484/ State : success == Summary == CI Bug Log - changes from CI_DRM_7806_full -> Patchwork_16242_full ==================================================== Summary ------- **SUCCESS** No regressions found. Known issues ------------ Here are the changes found in Patchwork_16242_full that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_ctx_isolation@vcs1-dirty-create: - shard-iclb: [PASS][1] -> [SKIP][2] ([fdo#109276] / [fdo#112080]) +3 similar issues [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-iclb2/igt@gem_ctx_isolation@vcs1-dirty-create.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-iclb5/igt@gem_ctx_isolation@vcs1-dirty-create.html * igt@gem_eio@unwedge-stress: - shard-snb: [PASS][3] -> [FAIL][4] ([i915#232]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-snb5/igt@gem_eio@unwedge-stress.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-snb5/igt@gem_eio@unwedge-stress.html * igt@gem_exec_reloc@basic-wc-cpu-active: - shard-skl: [PASS][5] -> [DMESG-WARN][6] ([i915#109]) [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-skl8/igt@gem_exec_reloc@basic-wc-cpu-active.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-skl7/igt@gem_exec_reloc@basic-wc-cpu-active.html * igt@gem_exec_reuse@single: - shard-glk: [PASS][7] -> [DMESG-WARN][8] ([i915#118] / [i915#95]) [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-glk5/igt@gem_exec_reuse@single.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-glk4/igt@gem_exec_reuse@single.html * igt@gem_exec_schedule@fifo-bsd: - shard-iclb: [PASS][9] -> [SKIP][10] ([fdo#112146]) +1 similar issue [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-iclb6/igt@gem_exec_schedule@fifo-bsd.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-iclb1/igt@gem_exec_schedule@fifo-bsd.html * igt@gem_exec_schedule@pi-common-bsd: - shard-iclb: [PASS][11] -> [SKIP][12] ([i915#677]) [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-iclb5/igt@gem_exec_schedule@pi-common-bsd.html [12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-iclb4/igt@gem_exec_schedule@pi-common-bsd.html * igt@gem_exec_schedule@promotion-bsd1: - shard-iclb: [PASS][13] -> [SKIP][14] ([fdo#109276]) +7 similar issues [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-iclb1/igt@gem_exec_schedule@promotion-bsd1.html [14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-iclb3/igt@gem_exec_schedule@promotion-bsd1.html * igt@gem_persistent_relocs@forked-faulting-reloc-thrashing: - shard-hsw: [PASS][15] -> [INCOMPLETE][16] ([i915#530] / [i915#61]) [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-hsw7/igt@gem_persistent_relocs@forked-faulting-reloc-thrashing.html [16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-hsw5/igt@gem_persistent_relocs@forked-faulting-reloc-thrashing.html * igt@gem_persistent_relocs@forked-interruptible-faulting-reloc-thrash-inactive: - shard-tglb: [PASS][17] -> [INCOMPLETE][18] ([i915#472]) [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-tglb8/igt@gem_persistent_relocs@forked-interruptible-faulting-reloc-thrash-inactive.html [18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-tglb3/igt@gem_persistent_relocs@forked-interruptible-faulting-reloc-thrash-inactive.html * igt@gem_persistent_relocs@forked-interruptible-faulting-reloc-thrashing: - shard-apl: [PASS][19] -> [INCOMPLETE][20] ([fdo#103927] / [i915#970]) [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-apl4/igt@gem_persistent_relocs@forked-interruptible-faulting-reloc-thrashing.html [20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-apl3/igt@gem_persistent_relocs@forked-interruptible-faulting-reloc-thrashing.html - shard-kbl: [PASS][21] -> [INCOMPLETE][22] ([fdo#103665] / [i915#970]) [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-kbl7/igt@gem_persistent_relocs@forked-interruptible-faulting-reloc-thrashing.html [22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-kbl2/igt@gem_persistent_relocs@forked-interruptible-faulting-reloc-thrashing.html * igt@gem_persistent_relocs@forked-interruptible-thrash-inactive: - shard-apl: [PASS][23] -> [INCOMPLETE][24] ([fdo#103927]) [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-apl2/igt@gem_persistent_relocs@forked-interruptible-thrash-inactive.html [24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-apl1/igt@gem_persistent_relocs@forked-interruptible-thrash-inactive.html * igt@gem_persistent_relocs@forked-interruptible-thrashing: - shard-skl: [PASS][25] -> [INCOMPLETE][26] ([i915#530]) [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-skl6/igt@gem_persistent_relocs@forked-interruptible-thrashing.html [26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-skl7/igt@gem_persistent_relocs@forked-interruptible-thrashing.html * igt@gem_persistent_relocs@forked-thrashing: - shard-iclb: [PASS][27] -> [INCOMPLETE][28] ([fdo#109100] / [i915#140]) [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-iclb8/igt@gem_persistent_relocs@forked-thrashing.html [28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-iclb7/igt@gem_persistent_relocs@forked-thrashing.html * igt@gem_ppgtt@flink-and-close-vma-leak: - shard-glk: [PASS][29] -> [FAIL][30] ([i915#644]) [29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-glk3/igt@gem_ppgtt@flink-and-close-vma-leak.html [30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-glk7/igt@gem_ppgtt@flink-and-close-vma-leak.html * igt@i915_pm_rpm@system-suspend: - shard-skl: [PASS][31] -> [INCOMPLETE][32] ([i915#151] / [i915#69]) [31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-skl2/igt@i915_pm_rpm@system-suspend.html [32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-skl7/igt@i915_pm_rpm@system-suspend.html * igt@i915_pm_rps@reset: - shard-tglb: [PASS][33] -> [FAIL][34] ([i915#413]) [33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-tglb6/igt@i915_pm_rps@reset.html [34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-tglb2/igt@i915_pm_rps@reset.html * igt@i915_suspend@debugfs-reader: - shard-skl: [PASS][35] -> [INCOMPLETE][36] ([i915#69]) +1 similar issue [35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-skl3/igt@i915_suspend@debugfs-reader.html [36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-skl1/igt@i915_suspend@debugfs-reader.html * igt@kms_flip@flip-vs-expired-vblank: - shard-skl: [PASS][37] -> [FAIL][38] ([i915#79]) [37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-skl10/igt@kms_flip@flip-vs-expired-vblank.html [38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-skl3/igt@kms_flip@flip-vs-expired-vblank.html * igt@kms_flip@flip-vs-suspend-interruptible: - shard-apl: [PASS][39] -> [DMESG-WARN][40] ([i915#180]) +1 similar issue [39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-apl7/igt@kms_flip@flip-vs-suspend-interruptible.html [40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-apl6/igt@kms_flip@flip-vs-suspend-interruptible.html * igt@kms_frontbuffer_tracking@fbcpsr-1p-pri-indfb-multidraw: - shard-tglb: [PASS][41] -> [FAIL][42] ([i915#49]) +1 similar issue [41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-tglb7/igt@kms_frontbuffer_tracking@fbcpsr-1p-pri-indfb-multidraw.html [42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-tglb3/igt@kms_frontbuffer_tracking@fbcpsr-1p-pri-indfb-multidraw.html * igt@kms_plane@plane-panning-bottom-right-suspend-pipe-c-planes: - shard-kbl: [PASS][43] -> [DMESG-WARN][44] ([i915#180]) +2 similar issues [43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-kbl6/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-c-planes.html [44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-kbl4/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-c-planes.html * igt@kms_plane_alpha_blend@pipe-a-coverage-7efc: - shard-skl: [PASS][45] -> [FAIL][46] ([fdo#108145]) [45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-skl5/igt@kms_plane_alpha_blend@pipe-a-coverage-7efc.html [46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-skl6/igt@kms_plane_alpha_blend@pipe-a-coverage-7efc.html * igt@kms_psr@psr2_sprite_mmap_gtt: - shard-iclb: [PASS][47] -> [SKIP][48] ([fdo#109441]) +2 similar issues [47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-iclb2/igt@kms_psr@psr2_sprite_mmap_gtt.html [48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-iclb4/igt@kms_psr@psr2_sprite_mmap_gtt.html * igt@kms_psr@suspend: - shard-skl: [PASS][49] -> [INCOMPLETE][50] ([i915#198]) [49]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-skl2/igt@kms_psr@suspend.html [50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-skl4/igt@kms_psr@suspend.html * igt@kms_setmode@basic: - shard-apl: [PASS][51] -> [FAIL][52] ([i915#31]) [51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-apl4/igt@kms_setmode@basic.html [52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-apl6/igt@kms_setmode@basic.html * igt@perf_pmu@busy-vcs1: - shard-iclb: [PASS][53] -> [SKIP][54] ([fdo#112080]) +4 similar issues [53]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-iclb1/igt@perf_pmu@busy-vcs1.html [54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-iclb8/igt@perf_pmu@busy-vcs1.html #### Possible fixes #### * igt@gem_ctx_isolation@rcs0-s3: - shard-kbl: [DMESG-WARN][55] ([i915#180]) -> [PASS][56] +6 similar issues [55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-kbl6/igt@gem_ctx_isolation@rcs0-s3.html [56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-kbl7/igt@gem_ctx_isolation@rcs0-s3.html * igt@gem_ctx_isolation@vcs0-s3: - shard-apl: [DMESG-WARN][57] ([i915#180]) -> [PASS][58] [57]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-apl1/igt@gem_ctx_isolation@vcs0-s3.html [58]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-apl4/igt@gem_ctx_isolation@vcs0-s3.html * igt@gem_ctx_persistence@vcs1-persistence: - shard-iclb: [SKIP][59] ([fdo#109276] / [fdo#112080]) -> [PASS][60] [59]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-iclb5/igt@gem_ctx_persistence@vcs1-persistence.html [60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-iclb4/igt@gem_ctx_persistence@vcs1-persistence.html * igt@gem_ctx_persistence@vecs0-mixed-process: - shard-glk: [FAIL][61] ([i915#679]) -> [PASS][62] [61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-glk3/igt@gem_ctx_persistence@vecs0-mixed-process.html [62]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-glk1/igt@gem_ctx_persistence@vecs0-mixed-process.html * igt@gem_exec_balancer@smoke: - shard-iclb: [SKIP][63] ([fdo#110854]) -> [PASS][64] [63]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-iclb5/igt@gem_exec_balancer@smoke.html [64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-iclb4/igt@gem_exec_balancer@smoke.html * igt@gem_exec_schedule@preemptive-hang-bsd: - shard-iclb: [SKIP][65] ([fdo#112146]) -> [PASS][66] +6 similar issues [65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-iclb1/igt@gem_exec_schedule@preemptive-hang-bsd.html [66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-iclb3/igt@gem_exec_schedule@preemptive-hang-bsd.html * igt@gem_persistent_relocs@forked-interruptible-faulting-reloc-thrashing: - shard-hsw: [INCOMPLETE][67] ([i915#530] / [i915#61]) -> [PASS][68] +1 similar issue [67]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-hsw2/igt@gem_persistent_relocs@forked-interruptible-faulting-reloc-thrashing.html [68]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-hsw2/igt@gem_persistent_relocs@forked-interruptible-faulting-reloc-thrashing.html * igt@gem_persistent_relocs@forked-thrashing: - shard-apl: [INCOMPLETE][69] ([fdo#103927]) -> [PASS][70] [69]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-apl8/igt@gem_persistent_relocs@forked-thrashing.html [70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-apl7/igt@gem_persistent_relocs@forked-thrashing.html - shard-hsw: [INCOMPLETE][71] ([i915#61]) -> [PASS][72] [71]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-hsw5/igt@gem_persistent_relocs@forked-thrashing.html [72]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-hsw7/igt@gem_persistent_relocs@forked-thrashing.html * igt@i915_pm_dc@dc6-psr: - shard-iclb: [FAIL][73] ([i915#454]) -> [PASS][74] [73]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-iclb4/igt@i915_pm_dc@dc6-psr.html [74]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-iclb7/igt@i915_pm_dc@dc6-psr.html * igt@i915_pm_rps@waitboost: - shard-iclb: [FAIL][75] ([i915#413]) -> [PASS][76] [75]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-iclb3/igt@i915_pm_rps@waitboost.html [76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-iclb3/igt@i915_pm_rps@waitboost.html * igt@i915_selftest@mock_requests: - shard-tglb: [INCOMPLETE][77] ([i915#472]) -> [PASS][78] [77]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-tglb6/igt@i915_selftest@mock_requests.html [78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-tglb8/igt@i915_selftest@mock_requests.html * igt@kms_color@pipe-a-ctm-0-5: - shard-skl: [DMESG-WARN][79] ([i915#109]) -> [PASS][80] [79]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-skl10/igt@kms_color@pipe-a-ctm-0-5.html [80]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-skl3/igt@kms_color@pipe-a-ctm-0-5.html * igt@kms_cursor_legacy@flip-vs-cursor-toggle: - shard-skl: [FAIL][81] ([IGT#5] / [i915#697]) -> [PASS][82] [81]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-skl3/igt@kms_cursor_legacy@flip-vs-cursor-toggle.html [82]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-skl2/igt@kms_cursor_legacy@flip-vs-cursor-toggle.html * igt@kms_flip@modeset-vs-vblank-race-interruptible: - shard-glk: [FAIL][83] ([i915#407]) -> [PASS][84] [83]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-glk6/igt@kms_flip@modeset-vs-vblank-race-interruptible.html [84]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-glk8/igt@kms_flip@modeset-vs-vblank-race-interruptible.html * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-mmap-wc: - shard-tglb: [FAIL][85] ([i915#49]) -> [PASS][86] +6 similar issues [85]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-tglb5/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-mmap-wc.html [86]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-tglb6/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-mmap-wc.html * igt@kms_plane_alpha_blend@pipe-b-coverage-7efc: - shard-skl: [FAIL][87] ([fdo#108145] / [i915#265]) -> [PASS][88] [87]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-skl5/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html [88]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-skl8/igt@kms_plane_alpha_blend@pipe-b-coverage-7efc.html * igt@kms_psr2_su@frontbuffer: - shard-iclb: [SKIP][89] ([fdo#109642] / [fdo#111068]) -> [PASS][90] +1 similar issue [89]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-iclb4/igt@kms_psr2_su@frontbuffer.html [90]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-iclb2/igt@kms_psr2_su@frontbuffer.html * igt@kms_psr@psr2_primary_page_flip: - shard-iclb: [SKIP][91] ([fdo#109441]) -> [PASS][92] +1 similar issue [91]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-iclb4/igt@kms_psr@psr2_primary_page_flip.html [92]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-iclb2/igt@kms_psr@psr2_primary_page_flip.html * igt@perf_pmu@busy-check-all-vcs1: - shard-iclb: [SKIP][93] ([fdo#112080]) -> [PASS][94] +2 similar issues [93]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-iclb6/igt@perf_pmu@busy-check-all-vcs1.html [94]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-iclb1/igt@perf_pmu@busy-check-all-vcs1.html * igt@prime_busy@hang-bsd2: - shard-iclb: [SKIP][95] ([fdo#109276]) -> [PASS][96] +12 similar issues [95]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-iclb5/igt@prime_busy@hang-bsd2.html [96]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-iclb4/igt@prime_busy@hang-bsd2.html * igt@prime_mmap_coherency@ioctl-errors: - shard-hsw: [FAIL][97] ([i915#831]) -> [PASS][98] [97]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-hsw2/igt@prime_mmap_coherency@ioctl-errors.html [98]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-hsw7/igt@prime_mmap_coherency@ioctl-errors.html #### Warnings #### * igt@gem_eio@kms: - shard-snb: [INCOMPLETE][99] ([i915#82]) -> [FAIL][100] ([i915#436]) [99]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-snb1/igt@gem_eio@kms.html [100]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-snb5/igt@gem_eio@kms.html * igt@gem_tiled_blits@normal: - shard-hsw: [FAIL][101] ([i915#818]) -> [INCOMPLETE][102] ([i915#61]) [101]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-hsw8/igt@gem_tiled_blits@normal.html [102]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-hsw8/igt@gem_tiled_blits@normal.html * igt@i915_pm_rpm@modeset-lpsp-stress: - shard-snb: [SKIP][103] ([fdo#109271]) -> [INCOMPLETE][104] ([i915#82]) [103]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-snb4/igt@i915_pm_rpm@modeset-lpsp-stress.html [104]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-snb6/igt@i915_pm_rpm@modeset-lpsp-stress.html * igt@kms_atomic_transition@3x-modeset-transitions: - shard-hsw: [SKIP][105] ([fdo#109271] / [i915#439]) -> [SKIP][106] ([fdo#109271]) [105]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7806/shard-hsw5/igt@kms_atomic_transition@3x-modeset-transitions.html [106]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_16242/shard-hsw1/igt@kms_atomic_transition@3x-modeset-transitions.html [IGT#5]: https://gitlab.freedesktop.org/drm/igt-gpu-tools/issues/5 [fdo#103665]: https://bugs.freedesktop.org/show_bug.cgi?id=103665 [fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927 [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145 [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#110854]: https://bugs.freedesktop.org/show_bug.cgi?id=110854 [fdo#111068]: https://bugs.freedesktop.org/show_bug.cgi?id=111068 [fdo#112080]: https://bugs.freedesktop.org/show_bug.cgi?id=112080 [fdo#112146]: https://bugs.freedesktop.org/show_bug.cgi?id=112146 [i915#109]: https://gitlab.freedesktop.org/drm/intel/issues/109 [i915#118]: https://gitlab.freedesktop.org/drm/intel/issues/118 [i915#140]: https://gitlab.freedesktop.org/drm/intel/issues/140 [i915#151]: https://gitlab.freedesktop.org/drm/intel/issues/151 [i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180 [i915#198]: https://gitlab.freedesktop.org/drm/intel/issues/198 [i915#232]: https://gitlab.freedesktop.org/drm/intel/issues/232 [i915#265]: https://gitlab.freedesktop.org/drm/intel/issues/265 [i915#31]: https://gitlab.freedesktop.org/drm/intel/issues/31 [i915#407]: https://gitlab.freedesktop.org/drm/intel/issues/407 [i915#413]: https://gitlab.freedesktop.org/drm/intel/issues/413 [i915#436]: https://gitlab.freedesktop.org/drm/intel/issues/436 [i915#439]: https://gitlab.freedesktop.org/drm/intel/issues/439 [i915#454]: https://gitlab.freedesktop.org/drm/intel/issues/454 [i915#472]: https://gitlab.freedesktop.org/drm/intel/issues/472 [i915#49]: https://gitlab.freedesktop.org/drm/intel/issues/49 [i915#530]: https://gitlab.freedesktop.org/drm/intel/issues/530 [i915#61]: https://gitlab.freedesktop.org/drm/intel/issues/61 [i915#644]: https://gitlab.freedesktop.org/drm/intel/issues/644 [i915#677]: https://gitlab.freedesktop.org/drm/intel/issues/677 [i915#679]: https://gitlab.freedesktop.org/drm/intel/issues/679 [i915#69]: https://gitlab.freedesktop.org/drm/intel/issues/69 [i915#697]: https://gitlab.freedesktop.org/drm/intel/issues/697 [i915#79]: https://gitlab.freedesktop.org/drm/intel/issues/79 [i915#818]: https://gitlab.freedesktop.org/drm/intel/issues/818 [i915#82]: https://gitlab.freedesktop.org/drm/intel/issues/82 [i915#831]: https://gitlab.freedesktop.org/drm/intel/issues/831 [i915#95]: https://gitlab.freedesktop.org/drm/intel/issues/95 [i915#970]: https://gitlab.freedesktop.org/drm/intel/issues/970 Participating hosts (10 -> 10) ------------------------------ No changes in participating hosts Build changes ------------- * CI: CI-20190529 -> None * Linux: CI_DRM_7806 -> Patchwork_16242 CI-20190529: 20190529 CI_DRM_7806: 0b551226df5e5b84044705d5fd76571da70f3163 @ git://anongit.freedesktop.org/gfx-ci/linux IGT_5382: 8dbe5ce61baa2d563d4dd7c56a018bb1e1077467 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools Patchwork_16242: 6c82f02126337eccb6531685f600676c60c370fa @ 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_16242/index.html _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2020-01-27 16:08 UTC | newest] Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-01-23 15:45 [PATCH 1/3] drm/dp: Include the AUX CH name in the debug messages Ville Syrjala 2020-01-23 15:45 ` [Intel-gfx] " Ville Syrjala 2020-01-23 15:45 ` [PATCH 2/3] drm/i915: " Ville Syrjala 2020-01-23 15:45 ` [Intel-gfx] " Ville Syrjala 2020-01-23 17:19 ` Matt Roper 2020-01-23 17:19 ` [Intel-gfx] " Matt Roper 2020-01-27 16:08 ` Ville Syrjälä 2020-01-27 16:08 ` [Intel-gfx] " Ville Syrjälä 2020-01-23 15:45 ` [PATCH 3/3] drm/i915: Give aux channels a better name Ville Syrjala 2020-01-23 15:45 ` [Intel-gfx] " Ville Syrjala 2020-01-23 17:22 ` Matt Roper 2020-01-23 17:22 ` Matt Roper 2020-01-24 2:56 ` [Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/3] drm/dp: Include the AUX CH name in the debug messages Patchwork 2020-01-25 20:19 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.