* [CI 1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method
@ 2018-12-11 17:31 Matt Roper
2018-12-11 17:31 ` [CI 2/2] drm/i915: Switch to level-based DDB allocation algorithm (v5) Matt Roper
` (6 more replies)
0 siblings, 7 replies; 11+ messages in thread
From: Matt Roper @ 2018-12-11 17:31 UTC (permalink / raw
To: intel-gfx
The bspec gives an if/else chain for choosing whether to use "method 1"
or "method 2" for calculating the watermark "Selected Result Blocks"
value for a plane. One of the branches of the if chain is:
"Else If ('plane buffer allocation' is known and (plane buffer
allocation / plane blocks per line) >=1)"
Since our driver currently calculates DDB allocations first and the
actual watermark values second, the plane buffer allocation is known at
this point in our code and we include this test in our driver's logic.
However we plan to soon move to a "watermarks first, ddb allocation
second" sequence where we won't know the DDB allocation at this point.
Let's drop this arm of the if/else statement (effectively considering
the DDB allocation unknown) as an independent patch so that any
regressions can be more accurately bisected to either the different
watermark value (in this patch) or the new DDB allocation (in the next
patch).
Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
drivers/gpu/drm/i915/intel_pm.c | 7 -------
1 file changed, 7 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
index 2bba5315b764..bf970cf7b8a5 100644
--- a/drivers/gpu/drm/i915/intel_pm.c
+++ b/drivers/gpu/drm/i915/intel_pm.c
@@ -4761,13 +4761,6 @@ static int skl_compute_plane_wm(const struct intel_crtc_state *cstate,
wp->dbuf_block_size < 1) &&
(wp->plane_bytes_per_line / wp->dbuf_block_size < 1)) {
selected_result = method2;
- } else if (ddb_allocation >=
- fixed16_to_u32_round_up(wp->plane_blocks_per_line)) {
- if (IS_GEN9(dev_priv) &&
- !IS_GEMINILAKE(dev_priv))
- selected_result = min_fixed16(method1, method2);
- else
- selected_result = method2;
} else if (latency >= wp->linetime_us) {
if (IS_GEN9(dev_priv) &&
!IS_GEMINILAKE(dev_priv))
--
2.14.4
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [CI 2/2] drm/i915: Switch to level-based DDB allocation algorithm (v5)
2018-12-11 17:31 [CI 1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method Matt Roper
@ 2018-12-11 17:31 ` Matt Roper
2018-12-11 18:58 ` [CI 2/2] drm/i915: Switch to level-based DDB allocation algorithm (v6) Matt Roper
2018-12-11 17:57 ` ✗ Fi.CI.SPARSE: warning for series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method Patchwork
` (5 subsequent siblings)
6 siblings, 1 reply; 11+ messages in thread
From: Matt Roper @ 2018-12-11 17:31 UTC (permalink / raw
To: intel-gfx
The DDB allocation algorithm currently used by the driver grants each
plane a very small minimum allocation of DDB blocks and then divies up
all of the remaining blocks based on the percentage of the total data
rate that the plane makes up. It turns out that this proportional
allocation approach is overly-generous with the larger planes and can
leave very small planes wthout a big enough allocation to even hit their
level 0 watermark requirements (especially on APL, which has a smaller
DDB in general than other gen9 platforms). Or there can be situations
where the smallest planes hit a lower watermark level than they should
have been able to hit with a more equitable division of DDB blocks, thus
limiting the overall system sleep state that can be achieved.
The bspec now describes an alternate algorithm that can be used to
overcome these types of issues. With the new algorithm, we calculate
all plane watermark values for all wm levels first, then go back and
partition a pipe's DDB space second. The DDB allocation will calculate
what the highest watermark level that can be achieved on *all* active
planes, and then grant the blocks necessary to hit that level to each
plane. Any remaining blocks are then divided up proportionally
according to data rate, similar to the old algorithm.
There was a previous attempt to implement this algorithm a couple years
ago in bb9d85f6e9d ("drm/i915/skl: New ddb allocation algorithm"), but
some regressions were reported, the patch was reverted, and nobody
ever got around to figuring out exactly where the bug was in that
version. Our watermark code has evolved significantly in the meantime,
but we're still getting bug reports caused by the unfair proportional
algorithm, so let's give this another shot.
v2:
- Make sure cursor allocation stays constant and fixed at the end of
the pipe allocation.
- Fix some watermark level iterators that weren't handling the max
level.
v3:
- Ensure we don't leave any DDB blocks unused by using DIV_ROUND_UP+min
to calculate the extra blocks for each plane. (Ville)
- Replace a while() loop with a for() loop to be more consistent with
surrounding code. (Ville)
- Clean unattainable watermark levels with memset rather than directly
clearing the member fields. Also do the same for the transition
watermark values if they can't be achieved. (Ville)
- Drop min_disp_buf_needed calculations in skl_compute_plane_wm() since
the results are no longer needed or used. (Ville)
- Drop skl_latency[0] != 0 sanity check; both watermark methods already
account for an invalid 0 latency by returning FP_16_16_MAX. (Ville)
v4:
- Break DDB allocation loop when total_data_rate=0 rather than
alloc_size=0. If total_data_rate has dropped to 0, all remaining
planes are disabled, which isn't true for alloc_size (we might just
have not had any remaining blocks to hand out). Plus
total_data_rate=0 is the case we need to avoid to a prevent a
div-by-0. (Ville)
- s/DIV_ROUND_UP/DIV64_U64_ROUND_UP/ to prevent 32-bit breakage (Ville)
v5:
- Don't forget to move 'start' pointer forward for UV surface when
setting plane DDB boundaries. (Ville)
Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105458
Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
drivers/gpu/drm/i915/intel_pm.c | 383 +++++++++++++++-------------------------
1 file changed, 140 insertions(+), 243 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
index bf970cf7b8a5..6d074f2e69d3 100644
--- a/drivers/gpu/drm/i915/intel_pm.c
+++ b/drivers/gpu/drm/i915/intel_pm.c
@@ -4301,102 +4301,6 @@ icl_get_total_relative_data_rate(struct intel_crtc_state *intel_cstate,
return total_data_rate;
}
-static uint16_t
-skl_ddb_min_alloc(const struct intel_plane_state *plane_state, const int plane)
-{
- struct drm_framebuffer *fb = plane_state->base.fb;
- uint32_t src_w, src_h;
- uint32_t min_scanlines = 8;
- uint8_t plane_bpp;
-
- if (WARN_ON(!fb))
- return 0;
-
- /* For packed formats, and uv-plane, return 0 */
- if (plane == 1 && fb->format->format != DRM_FORMAT_NV12)
- return 0;
-
- /* For Non Y-tile return 8-blocks */
- if (fb->modifier != I915_FORMAT_MOD_Y_TILED &&
- fb->modifier != I915_FORMAT_MOD_Yf_TILED &&
- fb->modifier != I915_FORMAT_MOD_Y_TILED_CCS &&
- fb->modifier != I915_FORMAT_MOD_Yf_TILED_CCS)
- return 8;
-
- /*
- * Src coordinates are already rotated by 270 degrees for
- * the 90/270 degree plane rotation cases (to match the
- * GTT mapping), hence no need to account for rotation here.
- */
- src_w = drm_rect_width(&plane_state->base.src) >> 16;
- src_h = drm_rect_height(&plane_state->base.src) >> 16;
-
- /* Halve UV plane width and height for NV12 */
- if (plane == 1) {
- src_w /= 2;
- src_h /= 2;
- }
-
- plane_bpp = fb->format->cpp[plane];
-
- if (drm_rotation_90_or_270(plane_state->base.rotation)) {
- switch (plane_bpp) {
- case 1:
- min_scanlines = 32;
- break;
- case 2:
- min_scanlines = 16;
- break;
- case 4:
- min_scanlines = 8;
- break;
- case 8:
- min_scanlines = 4;
- break;
- default:
- WARN(1, "Unsupported pixel depth %u for rotation",
- plane_bpp);
- min_scanlines = 32;
- }
- }
-
- return DIV_ROUND_UP((4 * src_w * plane_bpp), 512) * min_scanlines/4 + 3;
-}
-
-static void
-skl_ddb_calc_min(const struct intel_crtc_state *cstate, int num_active,
- uint16_t *minimum, uint16_t *uv_minimum)
-{
- const struct drm_plane_state *pstate;
- struct drm_plane *plane;
-
- drm_atomic_crtc_state_for_each_plane_state(plane, pstate, &cstate->base) {
- enum plane_id plane_id = to_intel_plane(plane)->id;
- struct intel_plane_state *plane_state = to_intel_plane_state(pstate);
-
- if (plane_id == PLANE_CURSOR)
- continue;
-
- /* slave plane must be invisible and calculated from master */
- if (!pstate->visible || WARN_ON(plane_state->slave))
- continue;
-
- if (!plane_state->linked_plane) {
- minimum[plane_id] = skl_ddb_min_alloc(plane_state, 0);
- uv_minimum[plane_id] =
- skl_ddb_min_alloc(plane_state, 1);
- } else {
- enum plane_id y_plane_id =
- plane_state->linked_plane->id;
-
- minimum[y_plane_id] = skl_ddb_min_alloc(plane_state, 0);
- minimum[plane_id] = skl_ddb_min_alloc(plane_state, 1);
- }
- }
-
- minimum[PLANE_CURSOR] = skl_cursor_allocation(num_active);
-}
-
static int
skl_allocate_pipe_ddb(struct intel_crtc_state *cstate,
struct skl_ddb_allocation *ddb /* out */)
@@ -4406,15 +4310,17 @@ skl_allocate_pipe_ddb(struct intel_crtc_state *cstate,
struct drm_i915_private *dev_priv = to_i915(crtc->dev);
struct intel_crtc *intel_crtc = to_intel_crtc(crtc);
struct skl_ddb_entry *alloc = &cstate->wm.skl.ddb;
- uint16_t alloc_size, start;
- uint16_t minimum[I915_MAX_PLANES] = {};
- uint16_t uv_minimum[I915_MAX_PLANES] = {};
+ struct skl_plane_wm *wm;
+ uint16_t alloc_size, start = 0;
+ uint16_t total[I915_MAX_PLANES] = {};
+ uint16_t uv_total[I915_MAX_PLANES] = {};
u64 total_data_rate;
enum plane_id plane_id;
int num_active;
u64 plane_data_rate[I915_MAX_PLANES] = {};
u64 uv_plane_data_rate[I915_MAX_PLANES] = {};
- uint16_t total_min_blocks = 0;
+ uint16_t blocks = 0;
+ int level;
/* Clear the partitioning for disabled planes. */
memset(cstate->wm.skl.plane_ddb_y, 0, sizeof(cstate->wm.skl.plane_ddb_y));
@@ -4444,81 +4350,134 @@ skl_allocate_pipe_ddb(struct intel_crtc_state *cstate,
if (alloc_size == 0)
return 0;
- skl_ddb_calc_min(cstate, num_active, minimum, uv_minimum);
+ /* Allocate fixed number of blocks for cursor. */
+ total[PLANE_CURSOR] = skl_cursor_allocation(num_active);
+ alloc_size -= total[PLANE_CURSOR];
+ cstate->wm.skl.plane_ddb_y[PLANE_CURSOR].start =
+ alloc->end - total[PLANE_CURSOR];
+ cstate->wm.skl.plane_ddb_y[PLANE_CURSOR].end = alloc->end;
+
+ if (total_data_rate == 0)
+ return 0;
/*
- * 1. Allocate the mininum required blocks for each active plane
- * and allocate the cursor, it doesn't require extra allocation
- * proportional to the data rate.
+ * Find the highest watermark level for which we can satisfy the block
+ * requirement of active planes.
*/
+ for (level = ilk_wm_max_level(dev_priv); level >= 0; level--) {
+ for_each_plane_id_on_crtc(intel_crtc, plane_id) {
+ if (plane_id == PLANE_CURSOR)
+ continue;
- for_each_plane_id_on_crtc(intel_crtc, plane_id) {
- total_min_blocks += minimum[plane_id];
- total_min_blocks += uv_minimum[plane_id];
+ wm = &cstate->wm.skl.optimal.planes[plane_id];
+ blocks += wm->wm[level].plane_res_b;
+ blocks += wm->uv_wm[level].plane_res_b;
+ }
+
+ if (blocks < alloc_size) {
+ alloc_size -= blocks;
+ break;
+ }
}
- if (total_min_blocks > alloc_size) {
+ if (level < 0) {
DRM_DEBUG_KMS("Requested display configuration exceeds system DDB limitations");
- DRM_DEBUG_KMS("minimum required %d/%d\n", total_min_blocks,
- alloc_size);
+ DRM_DEBUG_KMS("minimum required %d/%d\n", blocks,
+ alloc_size);
return -EINVAL;
}
- alloc_size -= total_min_blocks;
- cstate->wm.skl.plane_ddb_y[PLANE_CURSOR].start = alloc->end - minimum[PLANE_CURSOR];
- cstate->wm.skl.plane_ddb_y[PLANE_CURSOR].end = alloc->end;
-
/*
- * 2. Distribute the remaining space in proportion to the amount of
- * data each plane needs to fetch from memory.
- *
- * FIXME: we may not allocate every single block here.
+ * Grant each plane the blocks it requires at the highest achievable
+ * watermark level, plus an extra share of the leftover blocks
+ * proportional to its relative data rate.
*/
- if (total_data_rate == 0)
- return 0;
-
- start = alloc->start;
for_each_plane_id_on_crtc(intel_crtc, plane_id) {
- u64 data_rate, uv_data_rate;
- uint16_t plane_blocks, uv_plane_blocks;
+ u64 rate;
+ u16 extra;
if (plane_id == PLANE_CURSOR)
continue;
- data_rate = plane_data_rate[plane_id];
-
/*
- * allocation for (packed formats) or (uv-plane part of planar format):
- * promote the expression to 64 bits to avoid overflowing, the
- * result is < available as data_rate / total_data_rate < 1
+ * We've accounted for all active planes; remaining planes are
+ * all disabled.
*/
- plane_blocks = minimum[plane_id];
- plane_blocks += div64_u64(alloc_size * data_rate, total_data_rate);
+ if (total_data_rate == 0)
+ break;
- /* Leave disabled planes at (0,0) */
- if (data_rate) {
- cstate->wm.skl.plane_ddb_y[plane_id].start = start;
- cstate->wm.skl.plane_ddb_y[plane_id].end = start + plane_blocks;
- }
+ wm = &cstate->wm.skl.optimal.planes[plane_id];
- start += plane_blocks;
+ rate = plane_data_rate[plane_id];
+ extra = min_t(u16, alloc_size,
+ DIV64_U64_ROUND_UP(alloc_size * rate,
+ total_data_rate));
+ total[plane_id] = wm->wm[level].plane_res_b + extra;
+ alloc_size -= extra;
+ total_data_rate -= rate;
- /* Allocate DDB for UV plane for planar format/NV12 */
- uv_data_rate = uv_plane_data_rate[plane_id];
+ if (total_data_rate == 0)
+ break;
- uv_plane_blocks = uv_minimum[plane_id];
- uv_plane_blocks += div64_u64(alloc_size * uv_data_rate, total_data_rate);
+ rate = uv_plane_data_rate[plane_id];
+ extra = min_t(u16, alloc_size,
+ DIV64_U64_ROUND_UP(alloc_size * rate,
+ total_data_rate));
+ uv_total[plane_id] = wm->uv_wm[level].plane_res_b + extra;
+ alloc_size -= extra;
+ total_data_rate -= rate;
+ }
+ WARN_ON(alloc_size != 0 || total_data_rate != 0);
+
+ /* Set the actual DDB start/end points for each plane */
+ start = alloc->start;
+ for_each_plane_id_on_crtc(intel_crtc, plane_id) {
+ struct skl_ddb_entry *plane_alloc, *uv_plane_alloc;
+
+ if (plane_id == PLANE_CURSOR)
+ continue;
+
+ plane_alloc = &cstate->wm.skl.plane_ddb_y[plane_id];
+ uv_plane_alloc = &cstate->wm.skl.plane_ddb_uv[plane_id];
/* Gen11+ uses a separate plane for UV watermarks */
- WARN_ON(INTEL_GEN(dev_priv) >= 11 && uv_plane_blocks);
+ WARN_ON(INTEL_GEN(dev_priv) >= 11 && uv_total[plane_id]);
+
+ /* Leave disabled planes at (0,0) */
+ if (total[plane_id]) {
+ plane_alloc->start = start;
+ start += total[plane_id];
+ plane_alloc->end = start;
+ }
- if (uv_data_rate) {
- cstate->wm.skl.plane_ddb_uv[plane_id].start = start;
- cstate->wm.skl.plane_ddb_uv[plane_id].end =
- start + uv_plane_blocks;
+ if (uv_total[plane_id]) {
+ uv_plane_alloc->start = start;
+ start += uv_total[plane_id];
+ uv_plane_alloc->end = start;
}
+ }
- start += uv_plane_blocks;
+ /*
+ * When we calculated watermark values we didn't know how high
+ * of a level we'd actually be able to hit, so we just marked
+ * all levels as "enabled." Go back now and disable the ones
+ * that aren't actually possible.
+ */
+ for (level++; level <= ilk_wm_max_level(dev_priv); level++) {
+ for_each_plane_id_on_crtc(intel_crtc, plane_id) {
+ wm = &cstate->wm.skl.optimal.planes[plane_id];
+ memset(&wm->wm[level], 0, sizeof(wm->wm[level]));
+ }
+ }
+
+ /*
+ * Go back and disable the transition watermark if it turns out we
+ * don't have enough DDB blocks for it.
+ */
+ for_each_plane_id_on_crtc(intel_crtc, plane_id) {
+ wm = &cstate->wm.skl.optimal.planes[plane_id];
+ if (wm->trans_wm.plane_res_b > total[plane_id])
+ memset(&wm->trans_wm, 0, sizeof(wm->trans_wm));
}
return 0;
@@ -4715,17 +4674,15 @@ skl_compute_plane_wm_params(const struct intel_crtc_state *cstate,
return 0;
}
-static int skl_compute_plane_wm(const struct intel_crtc_state *cstate,
- const struct intel_plane_state *intel_pstate,
- uint16_t ddb_allocation,
- int level,
- const struct skl_wm_params *wp,
- const struct skl_wm_level *result_prev,
- struct skl_wm_level *result /* out */)
+static void skl_compute_plane_wm(const struct intel_crtc_state *cstate,
+ const struct intel_plane_state *intel_pstate,
+ int level,
+ const struct skl_wm_params *wp,
+ const struct skl_wm_level *result_prev,
+ struct skl_wm_level *result /* out */)
{
struct drm_i915_private *dev_priv =
to_i915(intel_pstate->base.plane->dev);
- const struct drm_plane_state *pstate = &intel_pstate->base;
uint32_t latency = dev_priv->wm.skl_latency[level];
uint_fixed_16_16_t method1, method2;
uint_fixed_16_16_t selected_result;
@@ -4733,10 +4690,6 @@ static int skl_compute_plane_wm(const struct intel_crtc_state *cstate,
struct intel_atomic_state *state =
to_intel_atomic_state(cstate->base.state);
bool apply_memory_bw_wa = skl_needs_memory_bw_wa(state);
- uint32_t min_disp_buf_needed;
-
- if (latency == 0)
- return level == 0 ? -EINVAL : 0;
/* Display WA #1141: kbl,cfl */
if ((IS_KABYLAKE(dev_priv) || IS_COFFEELAKE(dev_priv) ||
@@ -4800,61 +4753,24 @@ static int skl_compute_plane_wm(const struct intel_crtc_state *cstate,
res_blocks = result_prev->plane_res_b;
}
- if (INTEL_GEN(dev_priv) >= 11) {
- if (wp->y_tiled) {
- uint32_t extra_lines;
- uint_fixed_16_16_t fp_min_disp_buf_needed;
-
- if (res_lines % wp->y_min_scanlines == 0)
- extra_lines = wp->y_min_scanlines;
- else
- extra_lines = wp->y_min_scanlines * 2 -
- res_lines % wp->y_min_scanlines;
-
- fp_min_disp_buf_needed = mul_u32_fixed16(res_lines +
- extra_lines,
- wp->plane_blocks_per_line);
- min_disp_buf_needed = fixed16_to_u32_round_up(
- fp_min_disp_buf_needed);
- } else {
- min_disp_buf_needed = DIV_ROUND_UP(res_blocks * 11, 10);
- }
- } else {
- min_disp_buf_needed = res_blocks;
- }
-
- if ((level > 0 && res_lines > 31) ||
- res_blocks >= ddb_allocation ||
- min_disp_buf_needed >= ddb_allocation) {
- /*
- * If there are no valid level 0 watermarks, then we can't
- * support this display configuration.
- */
- if (level) {
- return 0;
- } else {
- struct drm_plane *plane = pstate->plane;
-
- DRM_DEBUG_KMS("Requested display configuration exceeds system watermark limitations\n");
- DRM_DEBUG_KMS("[PLANE:%d:%s] blocks required = %u/%u, lines required = %u/31\n",
- plane->base.id, plane->name,
- res_blocks, ddb_allocation, res_lines);
- return -EINVAL;
- }
- }
-
/* The number of lines are ignored for the level 0 watermark. */
+ if (level > 0 && res_lines > 31)
+ return;
+
+ /*
+ * If res_lines is valid, assume we can use this watermark level
+ * for now. We'll come back and disable it after we calculate the
+ * DDB allocation if it turns out we don't actually have enough
+ * blocks to satisfy it.
+ */
result->plane_res_b = res_blocks;
result->plane_res_l = res_lines;
result->plane_en = true;
-
- return 0;
}
-static int
+static void
skl_compute_wm_levels(const struct intel_crtc_state *cstate,
const struct intel_plane_state *intel_pstate,
- uint16_t ddb_blocks,
const struct skl_wm_params *wm_params,
struct skl_wm_level *levels)
{
@@ -4862,25 +4778,15 @@ skl_compute_wm_levels(const struct intel_crtc_state *cstate,
to_i915(intel_pstate->base.plane->dev);
int level, max_level = ilk_wm_max_level(dev_priv);
struct skl_wm_level *result_prev = &levels[0];
- int ret;
for (level = 0; level <= max_level; level++) {
struct skl_wm_level *result = &levels[level];
- ret = skl_compute_plane_wm(cstate,
- intel_pstate,
- ddb_blocks,
- level,
- wm_params,
- result_prev,
- result);
- if (ret)
- return ret;
+ skl_compute_plane_wm(cstate, intel_pstate, level, wm_params,
+ result_prev, result);
result_prev = result;
}
-
- return 0;
}
static uint32_t
@@ -4908,8 +4814,7 @@ skl_compute_linetime_wm(const struct intel_crtc_state *cstate)
static void skl_compute_transition_wm(const struct intel_crtc_state *cstate,
const struct skl_wm_params *wp,
- struct skl_plane_wm *wm,
- uint16_t ddb_allocation)
+ struct skl_plane_wm *wm)
{
struct drm_device *dev = cstate->base.crtc->dev;
const struct drm_i915_private *dev_priv = to_i915(dev);
@@ -4957,12 +4862,13 @@ static void skl_compute_transition_wm(const struct intel_crtc_state *cstate,
}
- res_blocks += 1;
-
- if (res_blocks < ddb_allocation) {
- wm->trans_wm.plane_res_b = res_blocks;
- wm->trans_wm.plane_en = true;
- }
+ /*
+ * Just assume we can enable the transition watermark. After
+ * computing the DDB we'll come back and disable it if that
+ * assumption turns out to be false.
+ */
+ wm->trans_wm.plane_res_b = res_blocks + 1;
+ wm->trans_wm.plane_en = true;
}
static int skl_build_plane_wm_single(struct intel_crtc_state *crtc_state,
@@ -4970,7 +4876,6 @@ static int skl_build_plane_wm_single(struct intel_crtc_state *crtc_state,
enum plane_id plane_id, int color_plane)
{
struct skl_plane_wm *wm = &crtc_state->wm.skl.optimal.planes[plane_id];
- u16 ddb_blocks = skl_ddb_entry_size(&crtc_state->wm.skl.plane_ddb_y[plane_id]);
struct skl_wm_params wm_params;
int ret;
@@ -4979,12 +4884,8 @@ static int skl_build_plane_wm_single(struct intel_crtc_state *crtc_state,
if (ret)
return ret;
- ret = skl_compute_wm_levels(crtc_state, plane_state,
- ddb_blocks, &wm_params, wm->wm);
- if (ret)
- return ret;
-
- skl_compute_transition_wm(crtc_state, &wm_params, wm, ddb_blocks);
+ skl_compute_wm_levels(crtc_state, plane_state, &wm_params, wm->wm);
+ skl_compute_transition_wm(crtc_state, &wm_params, wm);
return 0;
}
@@ -4994,7 +4895,6 @@ static int skl_build_plane_wm_uv(struct intel_crtc_state *crtc_state,
enum plane_id plane_id)
{
struct skl_plane_wm *wm = &crtc_state->wm.skl.optimal.planes[plane_id];
- u16 ddb_blocks = skl_ddb_entry_size(&crtc_state->wm.skl.plane_ddb_uv[plane_id]);
struct skl_wm_params wm_params;
int ret;
@@ -5006,10 +4906,7 @@ static int skl_build_plane_wm_uv(struct intel_crtc_state *crtc_state,
if (ret)
return ret;
- ret = skl_compute_wm_levels(crtc_state, plane_state,
- ddb_blocks, &wm_params, wm->uv_wm);
- if (ret)
- return ret;
+ skl_compute_wm_levels(crtc_state, plane_state, &wm_params, wm->uv_wm);
return 0;
}
@@ -5521,13 +5418,9 @@ skl_compute_wm(struct intel_atomic_state *state)
if (ret || !changed)
return ret;
- ret = skl_compute_ddb(state);
- if (ret)
- return ret;
-
/*
* Calculate WM's for all pipes that are part of this transaction.
- * Note that the DDB allocation above may have added more CRTC's that
+ * Note that skl_ddb_add_affected_pipes may have added more CRTC's that
* weren't otherwise being modified (and set bits in dirty_pipes) if
* pipe allocations had to change.
*/
@@ -5549,6 +5442,10 @@ skl_compute_wm(struct intel_atomic_state *state)
results->dirty_pipes |= drm_crtc_mask(&crtc->base);
}
+ ret = skl_compute_ddb(state);
+ if (ret)
+ return ret;
+
skl_print_wm_changes(state);
return 0;
--
2.14.4
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 11+ messages in thread
* ✗ Fi.CI.SPARSE: warning for series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method
2018-12-11 17:31 [CI 1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method Matt Roper
2018-12-11 17:31 ` [CI 2/2] drm/i915: Switch to level-based DDB allocation algorithm (v5) Matt Roper
@ 2018-12-11 17:57 ` Patchwork
2018-12-11 18:12 ` ✓ Fi.CI.BAT: success " Patchwork
` (4 subsequent siblings)
6 siblings, 0 replies; 11+ messages in thread
From: Patchwork @ 2018-12-11 17:57 UTC (permalink / raw
To: Matt Roper; +Cc: intel-gfx
== Series Details ==
Series: series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method
URL : https://patchwork.freedesktop.org/series/53901/
State : warning
== Summary ==
$ dim sparse origin/drm-tip
Sparse version: v0.5.2
Commit: drm/i915: Don't use DDB allocation when choosing gen9 watermark method
-drivers/gpu/drm/i915/i915_fixed.h:42:43: warning: expression using sizeof(void)
-drivers/gpu/drm/i915/i915_fixed.h:42:43: warning: expression using sizeof(void)
+drivers/gpu/drm/i915/intel_pm.c:6724:35: warning: expression using sizeof(void)
+drivers/gpu/drm/i915/intel_pm.c:6724:35: warning: expression using sizeof(void)
Commit: drm/i915: Switch to level-based DDB allocation algorithm (v5)
+drivers/gpu/drm/i915/intel_pm.c:4412:25: warning: expression using sizeof(void)
+drivers/gpu/drm/i915/intel_pm.c:4412:25: warning: expression using sizeof(void)
+drivers/gpu/drm/i915/intel_pm.c:4423:25: warning: expression using sizeof(void)
+drivers/gpu/drm/i915/intel_pm.c:4423:25: warning: expression using sizeof(void)
-drivers/gpu/drm/i915/intel_pm.c:6617:24: warning: expression using sizeof(void)
-drivers/gpu/drm/i915/intel_pm.c:6617:24: warning: expression using sizeof(void)
-drivers/gpu/drm/i915/intel_pm.c:6621:35: warning: expression using sizeof(void)
-drivers/gpu/drm/i915/intel_pm.c:6621:35: warning: expression using sizeof(void)
-drivers/gpu/drm/i915/intel_pm.c:6621:35: warning: too many warnings
+drivers/gpu/drm/i915/intel_pm.c:6617:24: warning: too many warnings
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
* ✓ Fi.CI.BAT: success for series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method
2018-12-11 17:31 [CI 1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method Matt Roper
2018-12-11 17:31 ` [CI 2/2] drm/i915: Switch to level-based DDB allocation algorithm (v5) Matt Roper
2018-12-11 17:57 ` ✗ Fi.CI.SPARSE: warning for series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method Patchwork
@ 2018-12-11 18:12 ` Patchwork
2018-12-11 19:14 ` ✗ Fi.CI.SPARSE: warning for series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method (rev2) Patchwork
` (3 subsequent siblings)
6 siblings, 0 replies; 11+ messages in thread
From: Patchwork @ 2018-12-11 18:12 UTC (permalink / raw
To: Matt Roper; +Cc: intel-gfx
== Series Details ==
Series: series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method
URL : https://patchwork.freedesktop.org/series/53901/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_5296 -> Patchwork_11067
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://patchwork.freedesktop.org/api/1.0/series/53901/revisions/1/mbox/
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_11067:
### IGT changes ###
#### Warnings ####
* igt@kms_busy@basic-flip-a:
- {fi-kbl-7567u}: PASS -> SKIP +2
Known issues
------------
Here are the changes found in Patchwork_11067 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_exec_suspend@basic-s3:
- fi-blb-e6850: PASS -> INCOMPLETE [fdo#107718]
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#107718]: https://bugs.freedesktop.org/show_bug.cgi?id=107718
Participating hosts (48 -> 43)
------------------------------
Additional (1): fi-byt-j1900
Missing (6): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-ctg-p8600 fi-pnv-d510
Build changes
-------------
* Linux: CI_DRM_5296 -> Patchwork_11067
CI_DRM_5296: 70751bd8a3f27b035d203ecafcad452f4d7c2c15 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_4745: 3b52e8a5809a4e860350c59476a456745cd9fee0 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_11067: 134ff01059a8bc017c061f3784f0bde602b8bd99 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
134ff01059a8 drm/i915: Switch to level-based DDB allocation algorithm (v5)
76df04365984 drm/i915: Don't use DDB allocation when choosing gen9 watermark method
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_11067/
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
* [CI 2/2] drm/i915: Switch to level-based DDB allocation algorithm (v6)
2018-12-11 17:31 ` [CI 2/2] drm/i915: Switch to level-based DDB allocation algorithm (v5) Matt Roper
@ 2018-12-11 18:58 ` Matt Roper
0 siblings, 0 replies; 11+ messages in thread
From: Matt Roper @ 2018-12-11 18:58 UTC (permalink / raw
To: intel-gfx
The DDB allocation algorithm currently used by the driver grants each
plane a very small minimum allocation of DDB blocks and then divies up
all of the remaining blocks based on the percentage of the total data
rate that the plane makes up. It turns out that this proportional
allocation approach is overly-generous with the larger planes and can
leave very small planes wthout a big enough allocation to even hit their
level 0 watermark requirements (especially on APL, which has a smaller
DDB in general than other gen9 platforms). Or there can be situations
where the smallest planes hit a lower watermark level than they should
have been able to hit with a more equitable division of DDB blocks, thus
limiting the overall system sleep state that can be achieved.
The bspec now describes an alternate algorithm that can be used to
overcome these types of issues. With the new algorithm, we calculate
all plane watermark values for all wm levels first, then go back and
partition a pipe's DDB space second. The DDB allocation will calculate
what the highest watermark level that can be achieved on *all* active
planes, and then grant the blocks necessary to hit that level to each
plane. Any remaining blocks are then divided up proportionally
according to data rate, similar to the old algorithm.
There was a previous attempt to implement this algorithm a couple years
ago in bb9d85f6e9d ("drm/i915/skl: New ddb allocation algorithm"), but
some regressions were reported, the patch was reverted, and nobody
ever got around to figuring out exactly where the bug was in that
version. Our watermark code has evolved significantly in the meantime,
but we're still getting bug reports caused by the unfair proportional
algorithm, so let's give this another shot.
v2:
- Make sure cursor allocation stays constant and fixed at the end of
the pipe allocation.
- Fix some watermark level iterators that weren't handling the max
level.
v3:
- Ensure we don't leave any DDB blocks unused by using DIV_ROUND_UP+min
to calculate the extra blocks for each plane. (Ville)
- Replace a while() loop with a for() loop to be more consistent with
surrounding code. (Ville)
- Clean unattainable watermark levels with memset rather than directly
clearing the member fields. Also do the same for the transition
watermark values if they can't be achieved. (Ville)
- Drop min_disp_buf_needed calculations in skl_compute_plane_wm() since
the results are no longer needed or used. (Ville)
- Drop skl_latency[0] != 0 sanity check; both watermark methods already
account for an invalid 0 latency by returning FP_16_16_MAX. (Ville)
v4:
- Break DDB allocation loop when total_data_rate=0 rather than
alloc_size=0. If total_data_rate has dropped to 0, all remaining
planes are disabled, which isn't true for alloc_size (we might just
have not had any remaining blocks to hand out). Plus
total_data_rate=0 is the case we need to avoid to a prevent a
div-by-0. (Ville)
- s/DIV_ROUND_UP/DIV64_U64_ROUND_UP/ to prevent 32-bit breakage (Ville)
v5:
- Don't forget to move 'start' pointer forward for UV surface when
setting plane DDB boundaries. (Ville)
v6:
- If a watermark level isn't achievable, don't forget to reset blocks=0
when testing the next lower level (otherwise that one will surely
fail as well). (CI)
Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105458
Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
drivers/gpu/drm/i915/intel_pm.c | 384 +++++++++++++++-------------------------
1 file changed, 141 insertions(+), 243 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
index bf970cf7b8a5..a6c7c11d2c0e 100644
--- a/drivers/gpu/drm/i915/intel_pm.c
+++ b/drivers/gpu/drm/i915/intel_pm.c
@@ -4301,102 +4301,6 @@ icl_get_total_relative_data_rate(struct intel_crtc_state *intel_cstate,
return total_data_rate;
}
-static uint16_t
-skl_ddb_min_alloc(const struct intel_plane_state *plane_state, const int plane)
-{
- struct drm_framebuffer *fb = plane_state->base.fb;
- uint32_t src_w, src_h;
- uint32_t min_scanlines = 8;
- uint8_t plane_bpp;
-
- if (WARN_ON(!fb))
- return 0;
-
- /* For packed formats, and uv-plane, return 0 */
- if (plane == 1 && fb->format->format != DRM_FORMAT_NV12)
- return 0;
-
- /* For Non Y-tile return 8-blocks */
- if (fb->modifier != I915_FORMAT_MOD_Y_TILED &&
- fb->modifier != I915_FORMAT_MOD_Yf_TILED &&
- fb->modifier != I915_FORMAT_MOD_Y_TILED_CCS &&
- fb->modifier != I915_FORMAT_MOD_Yf_TILED_CCS)
- return 8;
-
- /*
- * Src coordinates are already rotated by 270 degrees for
- * the 90/270 degree plane rotation cases (to match the
- * GTT mapping), hence no need to account for rotation here.
- */
- src_w = drm_rect_width(&plane_state->base.src) >> 16;
- src_h = drm_rect_height(&plane_state->base.src) >> 16;
-
- /* Halve UV plane width and height for NV12 */
- if (plane == 1) {
- src_w /= 2;
- src_h /= 2;
- }
-
- plane_bpp = fb->format->cpp[plane];
-
- if (drm_rotation_90_or_270(plane_state->base.rotation)) {
- switch (plane_bpp) {
- case 1:
- min_scanlines = 32;
- break;
- case 2:
- min_scanlines = 16;
- break;
- case 4:
- min_scanlines = 8;
- break;
- case 8:
- min_scanlines = 4;
- break;
- default:
- WARN(1, "Unsupported pixel depth %u for rotation",
- plane_bpp);
- min_scanlines = 32;
- }
- }
-
- return DIV_ROUND_UP((4 * src_w * plane_bpp), 512) * min_scanlines/4 + 3;
-}
-
-static void
-skl_ddb_calc_min(const struct intel_crtc_state *cstate, int num_active,
- uint16_t *minimum, uint16_t *uv_minimum)
-{
- const struct drm_plane_state *pstate;
- struct drm_plane *plane;
-
- drm_atomic_crtc_state_for_each_plane_state(plane, pstate, &cstate->base) {
- enum plane_id plane_id = to_intel_plane(plane)->id;
- struct intel_plane_state *plane_state = to_intel_plane_state(pstate);
-
- if (plane_id == PLANE_CURSOR)
- continue;
-
- /* slave plane must be invisible and calculated from master */
- if (!pstate->visible || WARN_ON(plane_state->slave))
- continue;
-
- if (!plane_state->linked_plane) {
- minimum[plane_id] = skl_ddb_min_alloc(plane_state, 0);
- uv_minimum[plane_id] =
- skl_ddb_min_alloc(plane_state, 1);
- } else {
- enum plane_id y_plane_id =
- plane_state->linked_plane->id;
-
- minimum[y_plane_id] = skl_ddb_min_alloc(plane_state, 0);
- minimum[plane_id] = skl_ddb_min_alloc(plane_state, 1);
- }
- }
-
- minimum[PLANE_CURSOR] = skl_cursor_allocation(num_active);
-}
-
static int
skl_allocate_pipe_ddb(struct intel_crtc_state *cstate,
struct skl_ddb_allocation *ddb /* out */)
@@ -4406,15 +4310,17 @@ skl_allocate_pipe_ddb(struct intel_crtc_state *cstate,
struct drm_i915_private *dev_priv = to_i915(crtc->dev);
struct intel_crtc *intel_crtc = to_intel_crtc(crtc);
struct skl_ddb_entry *alloc = &cstate->wm.skl.ddb;
- uint16_t alloc_size, start;
- uint16_t minimum[I915_MAX_PLANES] = {};
- uint16_t uv_minimum[I915_MAX_PLANES] = {};
+ struct skl_plane_wm *wm;
+ uint16_t alloc_size, start = 0;
+ uint16_t total[I915_MAX_PLANES] = {};
+ uint16_t uv_total[I915_MAX_PLANES] = {};
u64 total_data_rate;
enum plane_id plane_id;
int num_active;
u64 plane_data_rate[I915_MAX_PLANES] = {};
u64 uv_plane_data_rate[I915_MAX_PLANES] = {};
- uint16_t total_min_blocks = 0;
+ uint16_t blocks = 0;
+ int level;
/* Clear the partitioning for disabled planes. */
memset(cstate->wm.skl.plane_ddb_y, 0, sizeof(cstate->wm.skl.plane_ddb_y));
@@ -4444,81 +4350,135 @@ skl_allocate_pipe_ddb(struct intel_crtc_state *cstate,
if (alloc_size == 0)
return 0;
- skl_ddb_calc_min(cstate, num_active, minimum, uv_minimum);
+ /* Allocate fixed number of blocks for cursor. */
+ total[PLANE_CURSOR] = skl_cursor_allocation(num_active);
+ alloc_size -= total[PLANE_CURSOR];
+ cstate->wm.skl.plane_ddb_y[PLANE_CURSOR].start =
+ alloc->end - total[PLANE_CURSOR];
+ cstate->wm.skl.plane_ddb_y[PLANE_CURSOR].end = alloc->end;
+
+ if (total_data_rate == 0)
+ return 0;
/*
- * 1. Allocate the mininum required blocks for each active plane
- * and allocate the cursor, it doesn't require extra allocation
- * proportional to the data rate.
+ * Find the highest watermark level for which we can satisfy the block
+ * requirement of active planes.
*/
+ for (level = ilk_wm_max_level(dev_priv); level >= 0; level--) {
+ blocks = 0;
+ for_each_plane_id_on_crtc(intel_crtc, plane_id) {
+ if (plane_id == PLANE_CURSOR)
+ continue;
- for_each_plane_id_on_crtc(intel_crtc, plane_id) {
- total_min_blocks += minimum[plane_id];
- total_min_blocks += uv_minimum[plane_id];
+ wm = &cstate->wm.skl.optimal.planes[plane_id];
+ blocks += wm->wm[level].plane_res_b;
+ blocks += wm->uv_wm[level].plane_res_b;
+ }
+
+ if (blocks < alloc_size) {
+ alloc_size -= blocks;
+ break;
+ }
}
- if (total_min_blocks > alloc_size) {
+ if (level < 0) {
DRM_DEBUG_KMS("Requested display configuration exceeds system DDB limitations");
- DRM_DEBUG_KMS("minimum required %d/%d\n", total_min_blocks,
- alloc_size);
+ DRM_DEBUG_KMS("minimum required %d/%d\n", blocks,
+ alloc_size);
return -EINVAL;
}
- alloc_size -= total_min_blocks;
- cstate->wm.skl.plane_ddb_y[PLANE_CURSOR].start = alloc->end - minimum[PLANE_CURSOR];
- cstate->wm.skl.plane_ddb_y[PLANE_CURSOR].end = alloc->end;
-
/*
- * 2. Distribute the remaining space in proportion to the amount of
- * data each plane needs to fetch from memory.
- *
- * FIXME: we may not allocate every single block here.
+ * Grant each plane the blocks it requires at the highest achievable
+ * watermark level, plus an extra share of the leftover blocks
+ * proportional to its relative data rate.
*/
- if (total_data_rate == 0)
- return 0;
-
- start = alloc->start;
for_each_plane_id_on_crtc(intel_crtc, plane_id) {
- u64 data_rate, uv_data_rate;
- uint16_t plane_blocks, uv_plane_blocks;
+ u64 rate;
+ u16 extra;
if (plane_id == PLANE_CURSOR)
continue;
- data_rate = plane_data_rate[plane_id];
-
/*
- * allocation for (packed formats) or (uv-plane part of planar format):
- * promote the expression to 64 bits to avoid overflowing, the
- * result is < available as data_rate / total_data_rate < 1
+ * We've accounted for all active planes; remaining planes are
+ * all disabled.
*/
- plane_blocks = minimum[plane_id];
- plane_blocks += div64_u64(alloc_size * data_rate, total_data_rate);
+ if (total_data_rate == 0)
+ break;
- /* Leave disabled planes at (0,0) */
- if (data_rate) {
- cstate->wm.skl.plane_ddb_y[plane_id].start = start;
- cstate->wm.skl.plane_ddb_y[plane_id].end = start + plane_blocks;
- }
+ wm = &cstate->wm.skl.optimal.planes[plane_id];
- start += plane_blocks;
+ rate = plane_data_rate[plane_id];
+ extra = min_t(u16, alloc_size,
+ DIV64_U64_ROUND_UP(alloc_size * rate,
+ total_data_rate));
+ total[plane_id] = wm->wm[level].plane_res_b + extra;
+ alloc_size -= extra;
+ total_data_rate -= rate;
- /* Allocate DDB for UV plane for planar format/NV12 */
- uv_data_rate = uv_plane_data_rate[plane_id];
+ if (total_data_rate == 0)
+ break;
- uv_plane_blocks = uv_minimum[plane_id];
- uv_plane_blocks += div64_u64(alloc_size * uv_data_rate, total_data_rate);
+ rate = uv_plane_data_rate[plane_id];
+ extra = min_t(u16, alloc_size,
+ DIV64_U64_ROUND_UP(alloc_size * rate,
+ total_data_rate));
+ uv_total[plane_id] = wm->uv_wm[level].plane_res_b + extra;
+ alloc_size -= extra;
+ total_data_rate -= rate;
+ }
+ WARN_ON(alloc_size != 0 || total_data_rate != 0);
+
+ /* Set the actual DDB start/end points for each plane */
+ start = alloc->start;
+ for_each_plane_id_on_crtc(intel_crtc, plane_id) {
+ struct skl_ddb_entry *plane_alloc, *uv_plane_alloc;
+
+ if (plane_id == PLANE_CURSOR)
+ continue;
+
+ plane_alloc = &cstate->wm.skl.plane_ddb_y[plane_id];
+ uv_plane_alloc = &cstate->wm.skl.plane_ddb_uv[plane_id];
/* Gen11+ uses a separate plane for UV watermarks */
- WARN_ON(INTEL_GEN(dev_priv) >= 11 && uv_plane_blocks);
+ WARN_ON(INTEL_GEN(dev_priv) >= 11 && uv_total[plane_id]);
+
+ /* Leave disabled planes at (0,0) */
+ if (total[plane_id]) {
+ plane_alloc->start = start;
+ start += total[plane_id];
+ plane_alloc->end = start;
+ }
- if (uv_data_rate) {
- cstate->wm.skl.plane_ddb_uv[plane_id].start = start;
- cstate->wm.skl.plane_ddb_uv[plane_id].end =
- start + uv_plane_blocks;
+ if (uv_total[plane_id]) {
+ uv_plane_alloc->start = start;
+ start += uv_total[plane_id];
+ uv_plane_alloc->end = start;
}
+ }
- start += uv_plane_blocks;
+ /*
+ * When we calculated watermark values we didn't know how high
+ * of a level we'd actually be able to hit, so we just marked
+ * all levels as "enabled." Go back now and disable the ones
+ * that aren't actually possible.
+ */
+ for (level++; level <= ilk_wm_max_level(dev_priv); level++) {
+ for_each_plane_id_on_crtc(intel_crtc, plane_id) {
+ wm = &cstate->wm.skl.optimal.planes[plane_id];
+ memset(&wm->wm[level], 0, sizeof(wm->wm[level]));
+ }
+ }
+
+ /*
+ * Go back and disable the transition watermark if it turns out we
+ * don't have enough DDB blocks for it.
+ */
+ for_each_plane_id_on_crtc(intel_crtc, plane_id) {
+ wm = &cstate->wm.skl.optimal.planes[plane_id];
+ if (wm->trans_wm.plane_res_b > total[plane_id])
+ memset(&wm->trans_wm, 0, sizeof(wm->trans_wm));
}
return 0;
@@ -4715,17 +4675,15 @@ skl_compute_plane_wm_params(const struct intel_crtc_state *cstate,
return 0;
}
-static int skl_compute_plane_wm(const struct intel_crtc_state *cstate,
- const struct intel_plane_state *intel_pstate,
- uint16_t ddb_allocation,
- int level,
- const struct skl_wm_params *wp,
- const struct skl_wm_level *result_prev,
- struct skl_wm_level *result /* out */)
+static void skl_compute_plane_wm(const struct intel_crtc_state *cstate,
+ const struct intel_plane_state *intel_pstate,
+ int level,
+ const struct skl_wm_params *wp,
+ const struct skl_wm_level *result_prev,
+ struct skl_wm_level *result /* out */)
{
struct drm_i915_private *dev_priv =
to_i915(intel_pstate->base.plane->dev);
- const struct drm_plane_state *pstate = &intel_pstate->base;
uint32_t latency = dev_priv->wm.skl_latency[level];
uint_fixed_16_16_t method1, method2;
uint_fixed_16_16_t selected_result;
@@ -4733,10 +4691,6 @@ static int skl_compute_plane_wm(const struct intel_crtc_state *cstate,
struct intel_atomic_state *state =
to_intel_atomic_state(cstate->base.state);
bool apply_memory_bw_wa = skl_needs_memory_bw_wa(state);
- uint32_t min_disp_buf_needed;
-
- if (latency == 0)
- return level == 0 ? -EINVAL : 0;
/* Display WA #1141: kbl,cfl */
if ((IS_KABYLAKE(dev_priv) || IS_COFFEELAKE(dev_priv) ||
@@ -4800,61 +4754,24 @@ static int skl_compute_plane_wm(const struct intel_crtc_state *cstate,
res_blocks = result_prev->plane_res_b;
}
- if (INTEL_GEN(dev_priv) >= 11) {
- if (wp->y_tiled) {
- uint32_t extra_lines;
- uint_fixed_16_16_t fp_min_disp_buf_needed;
-
- if (res_lines % wp->y_min_scanlines == 0)
- extra_lines = wp->y_min_scanlines;
- else
- extra_lines = wp->y_min_scanlines * 2 -
- res_lines % wp->y_min_scanlines;
-
- fp_min_disp_buf_needed = mul_u32_fixed16(res_lines +
- extra_lines,
- wp->plane_blocks_per_line);
- min_disp_buf_needed = fixed16_to_u32_round_up(
- fp_min_disp_buf_needed);
- } else {
- min_disp_buf_needed = DIV_ROUND_UP(res_blocks * 11, 10);
- }
- } else {
- min_disp_buf_needed = res_blocks;
- }
-
- if ((level > 0 && res_lines > 31) ||
- res_blocks >= ddb_allocation ||
- min_disp_buf_needed >= ddb_allocation) {
- /*
- * If there are no valid level 0 watermarks, then we can't
- * support this display configuration.
- */
- if (level) {
- return 0;
- } else {
- struct drm_plane *plane = pstate->plane;
-
- DRM_DEBUG_KMS("Requested display configuration exceeds system watermark limitations\n");
- DRM_DEBUG_KMS("[PLANE:%d:%s] blocks required = %u/%u, lines required = %u/31\n",
- plane->base.id, plane->name,
- res_blocks, ddb_allocation, res_lines);
- return -EINVAL;
- }
- }
-
/* The number of lines are ignored for the level 0 watermark. */
+ if (level > 0 && res_lines > 31)
+ return;
+
+ /*
+ * If res_lines is valid, assume we can use this watermark level
+ * for now. We'll come back and disable it after we calculate the
+ * DDB allocation if it turns out we don't actually have enough
+ * blocks to satisfy it.
+ */
result->plane_res_b = res_blocks;
result->plane_res_l = res_lines;
result->plane_en = true;
-
- return 0;
}
-static int
+static void
skl_compute_wm_levels(const struct intel_crtc_state *cstate,
const struct intel_plane_state *intel_pstate,
- uint16_t ddb_blocks,
const struct skl_wm_params *wm_params,
struct skl_wm_level *levels)
{
@@ -4862,25 +4779,15 @@ skl_compute_wm_levels(const struct intel_crtc_state *cstate,
to_i915(intel_pstate->base.plane->dev);
int level, max_level = ilk_wm_max_level(dev_priv);
struct skl_wm_level *result_prev = &levels[0];
- int ret;
for (level = 0; level <= max_level; level++) {
struct skl_wm_level *result = &levels[level];
- ret = skl_compute_plane_wm(cstate,
- intel_pstate,
- ddb_blocks,
- level,
- wm_params,
- result_prev,
- result);
- if (ret)
- return ret;
+ skl_compute_plane_wm(cstate, intel_pstate, level, wm_params,
+ result_prev, result);
result_prev = result;
}
-
- return 0;
}
static uint32_t
@@ -4908,8 +4815,7 @@ skl_compute_linetime_wm(const struct intel_crtc_state *cstate)
static void skl_compute_transition_wm(const struct intel_crtc_state *cstate,
const struct skl_wm_params *wp,
- struct skl_plane_wm *wm,
- uint16_t ddb_allocation)
+ struct skl_plane_wm *wm)
{
struct drm_device *dev = cstate->base.crtc->dev;
const struct drm_i915_private *dev_priv = to_i915(dev);
@@ -4957,12 +4863,13 @@ static void skl_compute_transition_wm(const struct intel_crtc_state *cstate,
}
- res_blocks += 1;
-
- if (res_blocks < ddb_allocation) {
- wm->trans_wm.plane_res_b = res_blocks;
- wm->trans_wm.plane_en = true;
- }
+ /*
+ * Just assume we can enable the transition watermark. After
+ * computing the DDB we'll come back and disable it if that
+ * assumption turns out to be false.
+ */
+ wm->trans_wm.plane_res_b = res_blocks + 1;
+ wm->trans_wm.plane_en = true;
}
static int skl_build_plane_wm_single(struct intel_crtc_state *crtc_state,
@@ -4970,7 +4877,6 @@ static int skl_build_plane_wm_single(struct intel_crtc_state *crtc_state,
enum plane_id plane_id, int color_plane)
{
struct skl_plane_wm *wm = &crtc_state->wm.skl.optimal.planes[plane_id];
- u16 ddb_blocks = skl_ddb_entry_size(&crtc_state->wm.skl.plane_ddb_y[plane_id]);
struct skl_wm_params wm_params;
int ret;
@@ -4979,12 +4885,8 @@ static int skl_build_plane_wm_single(struct intel_crtc_state *crtc_state,
if (ret)
return ret;
- ret = skl_compute_wm_levels(crtc_state, plane_state,
- ddb_blocks, &wm_params, wm->wm);
- if (ret)
- return ret;
-
- skl_compute_transition_wm(crtc_state, &wm_params, wm, ddb_blocks);
+ skl_compute_wm_levels(crtc_state, plane_state, &wm_params, wm->wm);
+ skl_compute_transition_wm(crtc_state, &wm_params, wm);
return 0;
}
@@ -4994,7 +4896,6 @@ static int skl_build_plane_wm_uv(struct intel_crtc_state *crtc_state,
enum plane_id plane_id)
{
struct skl_plane_wm *wm = &crtc_state->wm.skl.optimal.planes[plane_id];
- u16 ddb_blocks = skl_ddb_entry_size(&crtc_state->wm.skl.plane_ddb_uv[plane_id]);
struct skl_wm_params wm_params;
int ret;
@@ -5006,10 +4907,7 @@ static int skl_build_plane_wm_uv(struct intel_crtc_state *crtc_state,
if (ret)
return ret;
- ret = skl_compute_wm_levels(crtc_state, plane_state,
- ddb_blocks, &wm_params, wm->uv_wm);
- if (ret)
- return ret;
+ skl_compute_wm_levels(crtc_state, plane_state, &wm_params, wm->uv_wm);
return 0;
}
@@ -5521,13 +5419,9 @@ skl_compute_wm(struct intel_atomic_state *state)
if (ret || !changed)
return ret;
- ret = skl_compute_ddb(state);
- if (ret)
- return ret;
-
/*
* Calculate WM's for all pipes that are part of this transaction.
- * Note that the DDB allocation above may have added more CRTC's that
+ * Note that skl_ddb_add_affected_pipes may have added more CRTC's that
* weren't otherwise being modified (and set bits in dirty_pipes) if
* pipe allocations had to change.
*/
@@ -5549,6 +5443,10 @@ skl_compute_wm(struct intel_atomic_state *state)
results->dirty_pipes |= drm_crtc_mask(&crtc->base);
}
+ ret = skl_compute_ddb(state);
+ if (ret)
+ return ret;
+
skl_print_wm_changes(state);
return 0;
--
2.14.4
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 11+ messages in thread
* ✗ Fi.CI.SPARSE: warning for series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method (rev2)
2018-12-11 17:31 [CI 1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method Matt Roper
` (2 preceding siblings ...)
2018-12-11 18:12 ` ✓ Fi.CI.BAT: success " Patchwork
@ 2018-12-11 19:14 ` Patchwork
2018-12-11 19:29 ` ✓ Fi.CI.BAT: success " Patchwork
` (2 subsequent siblings)
6 siblings, 0 replies; 11+ messages in thread
From: Patchwork @ 2018-12-11 19:14 UTC (permalink / raw
To: Matt Roper; +Cc: intel-gfx
== Series Details ==
Series: series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method (rev2)
URL : https://patchwork.freedesktop.org/series/53901/
State : warning
== Summary ==
$ dim sparse origin/drm-tip
Sparse version: v0.5.2
Commit: drm/i915: Don't use DDB allocation when choosing gen9 watermark method
-drivers/gpu/drm/i915/i915_fixed.h:42:43: warning: expression using sizeof(void)
-drivers/gpu/drm/i915/i915_fixed.h:42:43: warning: expression using sizeof(void)
+drivers/gpu/drm/i915/intel_pm.c:6724:35: warning: expression using sizeof(void)
+drivers/gpu/drm/i915/intel_pm.c:6724:35: warning: expression using sizeof(void)
Commit: drm/i915: Switch to level-based DDB allocation algorithm (v6)
+drivers/gpu/drm/i915/intel_pm.c:4413:25: warning: expression using sizeof(void)
+drivers/gpu/drm/i915/intel_pm.c:4413:25: warning: expression using sizeof(void)
+drivers/gpu/drm/i915/intel_pm.c:4424:25: warning: expression using sizeof(void)
+drivers/gpu/drm/i915/intel_pm.c:4424:25: warning: expression using sizeof(void)
-drivers/gpu/drm/i915/intel_pm.c:6618:24: warning: expression using sizeof(void)
-drivers/gpu/drm/i915/intel_pm.c:6618:24: warning: expression using sizeof(void)
-drivers/gpu/drm/i915/intel_pm.c:6622:35: warning: expression using sizeof(void)
-drivers/gpu/drm/i915/intel_pm.c:6622:35: warning: expression using sizeof(void)
-drivers/gpu/drm/i915/intel_pm.c:6622:35: warning: too many warnings
+drivers/gpu/drm/i915/intel_pm.c:6618:24: warning: too many warnings
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
* ✓ Fi.CI.BAT: success for series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method (rev2)
2018-12-11 17:31 [CI 1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method Matt Roper
` (3 preceding siblings ...)
2018-12-11 19:14 ` ✗ Fi.CI.SPARSE: warning for series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method (rev2) Patchwork
@ 2018-12-11 19:29 ` Patchwork
2018-12-11 20:41 ` ✓ Fi.CI.IGT: success for series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method Patchwork
2018-12-11 23:48 ` ✓ Fi.CI.IGT: success for series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method (rev2) Patchwork
6 siblings, 0 replies; 11+ messages in thread
From: Patchwork @ 2018-12-11 19:29 UTC (permalink / raw
To: Matt Roper; +Cc: intel-gfx
== Series Details ==
Series: series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method (rev2)
URL : https://patchwork.freedesktop.org/series/53901/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_5296 -> Patchwork_11069
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://patchwork.freedesktop.org/api/1.0/series/53901/revisions/2/mbox/
Known issues
------------
Here are the changes found in Patchwork_11069 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@i915_selftest@live_coherency:
- fi-gdg-551: PASS -> DMESG-FAIL [fdo#107164]
* igt@kms_chamelium@hdmi-hpd-fast:
- {fi-kbl-7500u}: PASS -> FAIL [fdo#108767]
* igt@kms_pipe_crc_basic@read-crc-pipe-b-frame-sequence:
- fi-byt-clapper: PASS -> FAIL [fdo#103191] / [fdo#107362]
* igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
- fi-skl-guc: PASS -> FAIL [fdo#103191] / [fdo#107362]
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
[fdo#107164]: https://bugs.freedesktop.org/show_bug.cgi?id=107164
[fdo#107362]: https://bugs.freedesktop.org/show_bug.cgi?id=107362
[fdo#108767]: https://bugs.freedesktop.org/show_bug.cgi?id=108767
Participating hosts (48 -> 44)
------------------------------
Additional (1): fi-byt-j1900
Missing (5): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-ctg-p8600
Build changes
-------------
* Linux: CI_DRM_5296 -> Patchwork_11069
CI_DRM_5296: 70751bd8a3f27b035d203ecafcad452f4d7c2c15 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_4745: 3b52e8a5809a4e860350c59476a456745cd9fee0 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_11069: 2e1fe301f9eaecd0f3525f75f16c525dac9f0354 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
2e1fe301f9ea drm/i915: Switch to level-based DDB allocation algorithm (v6)
f2e2112ac70a drm/i915: Don't use DDB allocation when choosing gen9 watermark method
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_11069/
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
* ✓ Fi.CI.IGT: success for series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method
2018-12-11 17:31 [CI 1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method Matt Roper
` (4 preceding siblings ...)
2018-12-11 19:29 ` ✓ Fi.CI.BAT: success " Patchwork
@ 2018-12-11 20:41 ` Patchwork
2018-12-11 23:48 ` ✓ Fi.CI.IGT: success for series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method (rev2) Patchwork
6 siblings, 0 replies; 11+ messages in thread
From: Patchwork @ 2018-12-11 20:41 UTC (permalink / raw
To: Matt Roper; +Cc: intel-gfx
== Series Details ==
Series: series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method
URL : https://patchwork.freedesktop.org/series/53901/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_5296_full -> Patchwork_11067_full
====================================================
Summary
-------
**WARNING**
Minor unknown changes coming with Patchwork_11067_full need to be verified
manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_11067_full, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_11067_full:
### IGT changes ###
#### Possible regressions ####
* igt@kms_plane_alpha_blend@pipe-a-constant-alpha-max:
- {shard-iclb}: NOTRUN -> FAIL
* igt@kms_rmfb@rmfb-ioctl:
- {shard-iclb}: PASS -> FAIL +5
#### Warnings ####
* igt@kms_plane_lowres@pipe-b-tiling-none:
- {shard-iclb}: PASS -> SKIP
* igt@pm_rc6_residency@rc6-accuracy:
- shard-snb: PASS -> SKIP
* igt@tools_test@tools_test:
- shard-skl: SKIP -> PASS
Known issues
------------
Here are the changes found in Patchwork_11067_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@i915_selftest@live_contexts:
- {shard-iclb}: NOTRUN -> DMESG-FAIL [fdo#108569]
* igt@kms_busy@extended-modeset-hang-newfb-render-b:
- shard-skl: NOTRUN -> DMESG-WARN [fdo#107956]
- {shard-iclb}: NOTRUN -> DMESG-WARN [fdo#107956]
* igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-b:
- {shard-iclb}: PASS -> DMESG-WARN [fdo#107956]
* igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-c:
- shard-glk: NOTRUN -> DMESG-WARN [fdo#107956]
* igt@kms_chv_cursor_fail@pipe-b-64x64-right-edge:
- shard-skl: PASS -> FAIL [fdo#104671]
* igt@kms_color@pipe-a-degamma:
- shard-apl: PASS -> FAIL [fdo#104782] / [fdo#108145]
* igt@kms_color@pipe-b-degamma:
- shard-apl: PASS -> FAIL [fdo#104782]
* igt@kms_color@pipe-c-ctm-green-to-red:
- shard-skl: PASS -> FAIL [fdo#107201]
* igt@kms_cursor_crc@cursor-256x256-onscreen:
- shard-glk: NOTRUN -> FAIL [fdo#103232]
* igt@kms_cursor_crc@cursor-64x21-random:
- shard-apl: PASS -> FAIL [fdo#103232] +5
* igt@kms_cursor_crc@cursor-64x64-suspend:
- shard-skl: PASS -> INCOMPLETE [fdo#104108] / [fdo#107773] +1
- shard-glk: PASS -> FAIL [fdo#103232]
* igt@kms_fbcon_fbt@fbc-suspend:
- shard-glk: PASS -> FAIL [fdo#103833] / [fdo#105681]
- {shard-iclb}: PASS -> FAIL [fdo#103833] / [fdo#105681]
* igt@kms_fbcon_fbt@psr-suspend:
- shard-skl: NOTRUN -> FAIL [fdo#107882]
* igt@kms_flip@flip-vs-expired-vblank-interruptible:
- shard-skl: PASS -> FAIL [fdo#105363]
* igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-onoff:
- shard-apl: PASS -> FAIL [fdo#103167] +1
* igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-draw-mmap-wc:
- shard-glk: PASS -> FAIL [fdo#103167] +5
* igt@kms_frontbuffer_tracking@fbc-stridechange:
- {shard-iclb}: PASS -> FAIL [fdo#105683] / [fdo#108040]
* igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-draw-mmap-gtt:
- {shard-iclb}: PASS -> FAIL [fdo#103167] +2
* igt@kms_plane@plane-position-covered-pipe-a-planes:
- shard-apl: PASS -> FAIL [fdo#103166] +3
* igt@kms_plane_alpha_blend@pipe-a-constant-alpha-max:
- shard-skl: NOTRUN -> FAIL [fdo#108145] +1
* igt@kms_plane_alpha_blend@pipe-c-alpha-opaque-fb:
- shard-apl: PASS -> FAIL [fdo#108145]
* igt@kms_plane_multiple@atomic-pipe-c-tiling-x:
- shard-glk: PASS -> FAIL [fdo#103166]
* igt@kms_plane_multiple@atomic-pipe-c-tiling-y:
- {shard-iclb}: PASS -> FAIL [fdo#103166]
* {igt@kms_rotation_crc@multiplane-rotation-cropping-top}:
- {shard-iclb}: PASS -> DMESG-WARN [fdo#107724] +1
- shard-glk: PASS -> DMESG-FAIL [fdo#105763] / [fdo#106538]
* igt@perf@polling:
- shard-hsw: PASS -> FAIL [fdo#102252]
* igt@pm_rpm@debugfs-read:
- {shard-iclb}: PASS -> INCOMPLETE [fdo#108840]
* igt@pm_rpm@dpms-lpsp:
- shard-skl: NOTRUN -> INCOMPLETE [fdo#107807]
#### Possible fixes ####
* igt@kms_ccs@pipe-a-crc-sprite-planes-basic:
- shard-apl: FAIL [fdo#105458] / [fdo#106510] -> PASS
* igt@kms_color@pipe-b-ctm-negative:
- shard-skl: FAIL [fdo#107361] -> PASS
* igt@kms_color@pipe-c-degamma:
- shard-apl: FAIL [fdo#104782] -> PASS
* igt@kms_cursor_crc@cursor-256x85-onscreen:
- shard-glk: FAIL [fdo#103232] -> PASS
* igt@kms_cursor_crc@cursor-64x21-sliding:
- shard-apl: FAIL [fdo#103232] -> PASS
* igt@kms_draw_crc@draw-method-rgb565-render-ytiled:
- {shard-iclb}: WARN [fdo#108336] -> PASS +1
* igt@kms_flip@dpms-off-confusion:
- {shard-iclb}: DMESG-WARN [fdo#107724] -> PASS +25
* igt@kms_flip_tiling@flip-changes-tiling:
- {shard-iclb}: DMESG-WARN [fdo#107724] / [fdo#108336] -> PASS +10
* igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-indfb-draw-pwrite:
- shard-skl: FAIL [fdo#105682] -> PASS
- {shard-iclb}: DMESG-FAIL [fdo#107724] -> PASS +6
* igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-mmap-gtt:
- shard-apl: FAIL [fdo#103167] -> PASS +1
- shard-glk: FAIL [fdo#103167] -> PASS +3
* igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-draw-pwrite:
- {shard-iclb}: FAIL [fdo#103167] -> PASS
* igt@kms_frontbuffer_tracking@fbcpsr-rgb565-draw-pwrite:
- shard-skl: FAIL [fdo#103167] -> PASS +1
* igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
- shard-skl: INCOMPLETE [fdo#104108] / [fdo#107773] -> PASS
* igt@kms_plane@pixel-format-pipe-c-planes:
- shard-apl: FAIL [fdo#103166] -> PASS
* igt@kms_plane_alpha_blend@pipe-b-coverage-7efc:
- shard-skl: FAIL [fdo#107815] -> PASS
* igt@kms_rotation_crc@sprite-rotation-90:
- shard-glk: INCOMPLETE [fdo#103359] / [k.org#198133] -> PASS
* igt@kms_setmode@basic:
- shard-apl: FAIL [fdo#99912] -> PASS
* igt@kms_vblank@pipe-c-ts-continuation-suspend:
- {shard-iclb}: INCOMPLETE [fdo#107713] -> PASS
* igt@perf@blocking:
- shard-hsw: FAIL [fdo#102252] -> PASS
* igt@sw_sync@sync_busy_fork_unixsocket:
- {shard-iclb}: INCOMPLETE [fdo#108889] -> PASS
#### Warnings ####
* igt@i915_suspend@shrink:
- shard-skl: DMESG-WARN [fdo#108784] -> INCOMPLETE [fdo#106886]
- shard-kbl: DMESG-WARN [fdo#108784] -> INCOMPLETE [fdo#103665] / [fdo#106886]
* igt@kms_cursor_crc@cursor-256x256-random:
- {shard-iclb}: DMESG-WARN [fdo#107724] / [fdo#108336] -> FAIL [fdo#103232] +3
* igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-draw-mmap-wc:
- {shard-iclb}: DMESG-FAIL [fdo#107724] -> FAIL [fdo#103167] +1
* igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-draw-render:
- {shard-iclb}: DMESG-WARN [fdo#107724] / [fdo#108336] -> FAIL [fdo#103167]
* igt@kms_plane@pixel-format-pipe-a-planes:
- {shard-iclb}: DMESG-WARN [fdo#107724] / [fdo#108336] -> FAIL [fdo#103166]
* {igt@kms_plane@pixel-format-pipe-a-planes-source-clamping}:
- {shard-iclb}: DMESG-WARN [fdo#107724] / [fdo#108336] -> FAIL [fdo#108948]
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#102252]: https://bugs.freedesktop.org/show_bug.cgi?id=102252
[fdo#103166]: https://bugs.freedesktop.org/show_bug.cgi?id=103166
[fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
[fdo#103232]: https://bugs.freedesktop.org/show_bug.cgi?id=103232
[fdo#103359]: https://bugs.freedesktop.org/show_bug.cgi?id=103359
[fdo#103665]: https://bugs.freedesktop.org/show_bug.cgi?id=103665
[fdo#103833]: https://bugs.freedesktop.org/show_bug.cgi?id=103833
[fdo#104108]: https://bugs.freedesktop.org/show_bug.cgi?id=104108
[fdo#104671]: https://bugs.freedesktop.org/show_bug.cgi?id=104671
[fdo#104782]: https://bugs.freedesktop.org/show_bug.cgi?id=104782
[fdo#105363]: https://bugs.freedesktop.org/show_bug.cgi?id=105363
[fdo#105458]: https://bugs.freedesktop.org/show_bug.cgi?id=105458
[fdo#105681]: https://bugs.freedesktop.org/show_bug.cgi?id=105681
[fdo#105682]: https://bugs.freedesktop.org/show_bug.cgi?id=105682
[fdo#105683]: https://bugs.freedesktop.org/show_bug.cgi?id=105683
[fdo#105763]: https://bugs.freedesktop.org/show_bug.cgi?id=105763
[fdo#106510]: https://bugs.freedesktop.org/show_bug.cgi?id=106510
[fdo#106538]: https://bugs.freedesktop.org/show_bug.cgi?id=106538
[fdo#106886]: https://bugs.freedesktop.org/show_bug.cgi?id=106886
[fdo#107201]: https://bugs.freedesktop.org/show_bug.cgi?id=107201
[fdo#107361]: https://bugs.freedesktop.org/show_bug.cgi?id=107361
[fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
[fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724
[fdo#107773]: https://bugs.freedesktop.org/show_bug.cgi?id=107773
[fdo#107807]: https://bugs.freedesktop.org/show_bug.cgi?id=107807
[fdo#107815]: https://bugs.freedesktop.org/show_bug.cgi?id=107815
[fdo#107882]: https://bugs.freedesktop.org/show_bug.cgi?id=107882
[fdo#107956]: https://bugs.freedesktop.org/show_bug.cgi?id=107956
[fdo#108040]: https://bugs.freedesktop.org/show_bug.cgi?id=108040
[fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
[fdo#108336]: https://bugs.freedesktop.org/show_bug.cgi?id=108336
[fdo#108569]: https://bugs.freedesktop.org/show_bug.cgi?id=108569
[fdo#108784]: https://bugs.freedesktop.org/show_bug.cgi?id=108784
[fdo#108840]: https://bugs.freedesktop.org/show_bug.cgi?id=108840
[fdo#108889]: https://bugs.freedesktop.org/show_bug.cgi?id=108889
[fdo#108948]: https://bugs.freedesktop.org/show_bug.cgi?id=108948
[fdo#99912]: https://bugs.freedesktop.org/show_bug.cgi?id=99912
[k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133
Participating hosts (7 -> 7)
------------------------------
No changes in participating hosts
Build changes
-------------
* Linux: CI_DRM_5296 -> Patchwork_11067
CI_DRM_5296: 70751bd8a3f27b035d203ecafcad452f4d7c2c15 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_4745: 3b52e8a5809a4e860350c59476a456745cd9fee0 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_11067: 134ff01059a8bc017c061f3784f0bde602b8bd99 @ 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_11067/
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
* ✓ Fi.CI.IGT: success for series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method (rev2)
2018-12-11 17:31 [CI 1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method Matt Roper
` (5 preceding siblings ...)
2018-12-11 20:41 ` ✓ Fi.CI.IGT: success for series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method Patchwork
@ 2018-12-11 23:48 ` Patchwork
2018-12-12 0:04 ` Matt Roper
6 siblings, 1 reply; 11+ messages in thread
From: Patchwork @ 2018-12-11 23:48 UTC (permalink / raw
To: Matt Roper; +Cc: intel-gfx
== Series Details ==
Series: series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method (rev2)
URL : https://patchwork.freedesktop.org/series/53901/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_5296_full -> Patchwork_11069_full
====================================================
Summary
-------
**WARNING**
Minor unknown changes coming with Patchwork_11069_full need to be verified
manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_11069_full, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_11069_full:
### IGT changes ###
#### Warnings ####
* igt@pm_rc6_residency@rc6-accuracy:
- shard-snb: PASS -> SKIP
* igt@tools_test@tools_test:
- shard-skl: SKIP -> PASS
Known issues
------------
Here are the changes found in Patchwork_11069_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_userptr_blits@readonly-unsync:
- {shard-iclb}: PASS -> INCOMPLETE [fdo#108342]
* igt@i915_selftest@live_contexts:
- {shard-iclb}: NOTRUN -> DMESG-FAIL [fdo#108569]
* igt@kms_busy@extended-modeset-hang-newfb-render-b:
- shard-skl: NOTRUN -> DMESG-WARN [fdo#107956]
- {shard-iclb}: NOTRUN -> DMESG-WARN [fdo#107956]
* igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-b:
- {shard-iclb}: PASS -> DMESG-WARN [fdo#107956]
* igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-c:
- shard-glk: NOTRUN -> DMESG-WARN [fdo#107956]
* igt@kms_chv_cursor_fail@pipe-b-256x256-right-edge:
- {shard-iclb}: PASS -> DMESG-WARN [fdo#107724] / [fdo#108336] +10
* igt@kms_color@pipe-a-degamma:
- shard-apl: PASS -> FAIL [fdo#104782] / [fdo#108145]
* igt@kms_color@pipe-b-degamma:
- shard-apl: PASS -> FAIL [fdo#104782]
* igt@kms_color@pipe-b-gamma:
- shard-skl: PASS -> FAIL [fdo#104782]
* igt@kms_color@pipe-c-ctm-green-to-red:
- shard-skl: PASS -> FAIL [fdo#107201]
* igt@kms_cursor_crc@cursor-64x21-random:
- shard-apl: PASS -> FAIL [fdo#103232] +11
* igt@kms_cursor_crc@cursor-64x64-suspend:
- shard-apl: PASS -> FAIL [fdo#103191] / [fdo#103232]
* igt@kms_draw_crc@draw-method-rgb565-pwrite-ytiled:
- {shard-iclb}: PASS -> FAIL [fdo#103184]
* igt@kms_draw_crc@draw-method-xrgb2101010-render-ytiled:
- {shard-iclb}: PASS -> WARN [fdo#108336] +2
* igt@kms_fbcon_fbt@psr-suspend:
- shard-skl: NOTRUN -> FAIL [fdo#107882]
* igt@kms_flip@flip-vs-expired-vblank-interruptible:
- shard-skl: PASS -> FAIL [fdo#105363]
* igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-indfb-draw-mmap-gtt:
- {shard-iclb}: PASS -> DMESG-FAIL [fdo#107724] +16
* igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-draw-render:
- shard-apl: PASS -> FAIL [fdo#103167] +5
* igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-draw-mmap-wc:
- shard-glk: PASS -> FAIL [fdo#103167] +2
* igt@kms_frontbuffer_tracking@fbc-stridechange:
- {shard-iclb}: PASS -> FAIL [fdo#105683] / [fdo#108040]
* igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-draw-render:
- {shard-iclb}: PASS -> FAIL [fdo#103167] +3
* igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-draw-blt:
- {shard-iclb}: NOTRUN -> FAIL [fdo#103167]
* igt@kms_plane@plane-panning-bottom-right-suspend-pipe-c-planes:
- shard-skl: PASS -> INCOMPLETE [fdo#104108]
* igt@kms_plane@plane-position-covered-pipe-a-planes:
- shard-glk: PASS -> FAIL [fdo#103166]
- {shard-iclb}: NOTRUN -> FAIL [fdo#103166]
* igt@kms_plane_alpha_blend@pipe-a-constant-alpha-max:
- shard-skl: NOTRUN -> FAIL [fdo#108145] +1
* igt@kms_plane_alpha_blend@pipe-b-alpha-opaque-fb:
- shard-glk: PASS -> FAIL [fdo#108145]
* igt@kms_plane_multiple@atomic-pipe-b-tiling-y:
- {shard-iclb}: PASS -> FAIL [fdo#103166] +3
* igt@kms_psr@cursor_mmap_gtt:
- {shard-iclb}: PASS -> DMESG-WARN [fdo#107724] +19
* igt@kms_sysfs_edid_timing:
- shard-skl: NOTRUN -> FAIL [fdo#100047]
* igt@kms_vblank@pipe-a-ts-continuation-dpms-suspend:
- {shard-iclb}: PASS -> INCOMPLETE [fdo#107713]
* igt@perf@short-reads:
- shard-skl: PASS -> FAIL [fdo#103183]
* igt@pm_rpm@modeset-stress-extra-wait:
- {shard-iclb}: PASS -> INCOMPLETE [fdo#108840]
#### Possible fixes ####
* igt@kms_ccs@pipe-a-crc-sprite-planes-basic:
- shard-apl: FAIL [fdo#105458] / [fdo#106510] -> PASS
* igt@kms_color@pipe-c-degamma:
- shard-apl: FAIL [fdo#104782] -> PASS
* igt@kms_cursor_crc@cursor-64x21-sliding:
- shard-apl: FAIL [fdo#103232] -> PASS +1
* igt@kms_draw_crc@draw-method-rgb565-mmap-gtt-xtiled:
- shard-skl: FAIL [fdo#103184] -> PASS
* igt@kms_draw_crc@draw-method-rgb565-render-ytiled:
- {shard-iclb}: WARN [fdo#108336] -> PASS +1
* igt@kms_fbcon_fbt@fbc-suspend:
- shard-kbl: FAIL [fdo#103833] / [fdo#105681] -> PASS
* igt@kms_flip@dpms-off-confusion:
- {shard-iclb}: DMESG-WARN [fdo#107724] -> PASS +21
* igt@kms_flip_tiling@flip-changes-tiling:
- {shard-iclb}: DMESG-WARN [fdo#107724] / [fdo#108336] -> PASS +10
* igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-indfb-draw-pwrite:
- shard-skl: FAIL [fdo#105682] -> PASS
- {shard-iclb}: DMESG-FAIL [fdo#107724] -> PASS +6
* igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-mmap-gtt:
- shard-apl: FAIL [fdo#103167] -> PASS +1
* igt@kms_frontbuffer_tracking@fbcpsr-rgb565-draw-pwrite:
- shard-skl: FAIL [fdo#103167] -> PASS +1
* igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-draw-mmap-wc:
- {shard-iclb}: FAIL [fdo#103167] -> PASS +1
* igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
- shard-skl: INCOMPLETE [fdo#104108] / [fdo#107773] -> PASS
* igt@kms_plane@pixel-format-pipe-c-planes:
- shard-apl: FAIL [fdo#103166] -> PASS +1
* igt@kms_plane_alpha_blend@pipe-a-coverage-7efc:
- shard-skl: FAIL [fdo#107815] / [fdo#108145] -> PASS
* igt@kms_plane_multiple@atomic-pipe-a-tiling-x:
- shard-glk: FAIL [fdo#103166] -> PASS
* {igt@kms_rotation_crc@multiplane-rotation-cropping-top}:
- shard-kbl: DMESG-FAIL [fdo#108950] -> PASS
* igt@kms_rotation_crc@sprite-rotation-90:
- shard-glk: INCOMPLETE [fdo#103359] / [k.org#198133] -> PASS
* igt@kms_setmode@basic:
- shard-kbl: FAIL [fdo#99912] -> PASS
* igt@kms_vblank@pipe-c-ts-continuation-suspend:
- {shard-iclb}: INCOMPLETE [fdo#107713] -> PASS
* igt@perf@blocking:
- shard-hsw: FAIL [fdo#102252] -> PASS
* igt@sw_sync@sync_busy_fork_unixsocket:
- {shard-iclb}: INCOMPLETE [fdo#108889] -> PASS
#### Warnings ####
* igt@gem_cpu_reloc@full:
- shard-skl: INCOMPLETE [fdo#108073] -> TIMEOUT [fdo#108248]
* igt@i915_suspend@shrink:
- shard-skl: DMESG-WARN [fdo#108784] -> INCOMPLETE [fdo#106886]
* igt@kms_ccs@pipe-a-crc-primary-rotation-180:
- {shard-iclb}: FAIL [fdo#107725] -> DMESG-WARN [fdo#107724] / [fdo#108336] +1
* igt@kms_cursor_crc@cursor-256x256-random:
- {shard-iclb}: DMESG-WARN [fdo#107724] / [fdo#108336] -> FAIL [fdo#103232] +3
* igt@kms_cursor_crc@cursor-256x85-onscreen:
- {shard-iclb}: FAIL [fdo#103232] -> DMESG-WARN [fdo#107724] / [fdo#108336]
* igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-draw-mmap-wc:
- {shard-iclb}: DMESG-FAIL [fdo#107724] -> FAIL [fdo#103167] +1
* igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-draw-render:
- {shard-iclb}: DMESG-WARN [fdo#107724] / [fdo#108336] -> FAIL [fdo#103167]
* igt@kms_plane@pixel-format-pipe-a-planes:
- {shard-iclb}: DMESG-WARN [fdo#107724] / [fdo#108336] -> FAIL [fdo#103166]
* {igt@kms_plane@pixel-format-pipe-a-planes-source-clamping}:
- {shard-iclb}: DMESG-WARN [fdo#107724] / [fdo#108336] -> FAIL [fdo#108948]
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#100047]: https://bugs.freedesktop.org/show_bug.cgi?id=100047
[fdo#102252]: https://bugs.freedesktop.org/show_bug.cgi?id=102252
[fdo#103166]: https://bugs.freedesktop.org/show_bug.cgi?id=103166
[fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
[fdo#103183]: https://bugs.freedesktop.org/show_bug.cgi?id=103183
[fdo#103184]: https://bugs.freedesktop.org/show_bug.cgi?id=103184
[fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
[fdo#103232]: https://bugs.freedesktop.org/show_bug.cgi?id=103232
[fdo#103359]: https://bugs.freedesktop.org/show_bug.cgi?id=103359
[fdo#103833]: https://bugs.freedesktop.org/show_bug.cgi?id=103833
[fdo#104108]: https://bugs.freedesktop.org/show_bug.cgi?id=104108
[fdo#104782]: https://bugs.freedesktop.org/show_bug.cgi?id=104782
[fdo#105363]: https://bugs.freedesktop.org/show_bug.cgi?id=105363
[fdo#105458]: https://bugs.freedesktop.org/show_bug.cgi?id=105458
[fdo#105681]: https://bugs.freedesktop.org/show_bug.cgi?id=105681
[fdo#105682]: https://bugs.freedesktop.org/show_bug.cgi?id=105682
[fdo#105683]: https://bugs.freedesktop.org/show_bug.cgi?id=105683
[fdo#106510]: https://bugs.freedesktop.org/show_bug.cgi?id=106510
[fdo#106886]: https://bugs.freedesktop.org/show_bug.cgi?id=106886
[fdo#107201]: https://bugs.freedesktop.org/show_bug.cgi?id=107201
[fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
[fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724
[fdo#107725]: https://bugs.freedesktop.org/show_bug.cgi?id=107725
[fdo#107773]: https://bugs.freedesktop.org/show_bug.cgi?id=107773
[fdo#107815]: https://bugs.freedesktop.org/show_bug.cgi?id=107815
[fdo#107882]: https://bugs.freedesktop.org/show_bug.cgi?id=107882
[fdo#107956]: https://bugs.freedesktop.org/show_bug.cgi?id=107956
[fdo#108040]: https://bugs.freedesktop.org/show_bug.cgi?id=108040
[fdo#108073]: https://bugs.freedesktop.org/show_bug.cgi?id=108073
[fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
[fdo#108248]: https://bugs.freedesktop.org/show_bug.cgi?id=108248
[fdo#108336]: https://bugs.freedesktop.org/show_bug.cgi?id=108336
[fdo#108342]: https://bugs.freedesktop.org/show_bug.cgi?id=108342
[fdo#108569]: https://bugs.freedesktop.org/show_bug.cgi?id=108569
[fdo#108784]: https://bugs.freedesktop.org/show_bug.cgi?id=108784
[fdo#108840]: https://bugs.freedesktop.org/show_bug.cgi?id=108840
[fdo#108889]: https://bugs.freedesktop.org/show_bug.cgi?id=108889
[fdo#108948]: https://bugs.freedesktop.org/show_bug.cgi?id=108948
[fdo#108950]: https://bugs.freedesktop.org/show_bug.cgi?id=108950
[fdo#99912]: https://bugs.freedesktop.org/show_bug.cgi?id=99912
[k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133
Participating hosts (7 -> 7)
------------------------------
No changes in participating hosts
Build changes
-------------
* Linux: CI_DRM_5296 -> Patchwork_11069
CI_DRM_5296: 70751bd8a3f27b035d203ecafcad452f4d7c2c15 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_4745: 3b52e8a5809a4e860350c59476a456745cd9fee0 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_11069: 2e1fe301f9eaecd0f3525f75f16c525dac9f0354 @ 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_11069/
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: ✓ Fi.CI.IGT: success for series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method (rev2)
2018-12-11 23:48 ` ✓ Fi.CI.IGT: success for series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method (rev2) Patchwork
@ 2018-12-12 0:04 ` Matt Roper
2018-12-12 7:45 ` Saarinen, Jani
0 siblings, 1 reply; 11+ messages in thread
From: Matt Roper @ 2018-12-12 0:04 UTC (permalink / raw
To: intel-gfx
The rc6 pass->skip mentioned below doesn't appear to be related to this
series, so pushing to dinq. Thanks to Ville for reviewing.
I also notice that CI indicates a bunch of pre-existing ICL watermark
failures are no longer happening with my series (or the earlier
revisions of my series), so it's possible that we've also fixed
https://bugs.freedesktop.org/show_bug.cgi?id=107724 "by accident" with
this series.
Matt
On Tue, Dec 11, 2018 at 11:48:05PM +0000, Patchwork wrote:
> == Series Details ==
>
> Series: series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method (rev2)
> URL : https://patchwork.freedesktop.org/series/53901/
> State : success
>
> == Summary ==
>
> CI Bug Log - changes from CI_DRM_5296_full -> Patchwork_11069_full
> ====================================================
>
> Summary
> -------
>
> **WARNING**
>
> Minor unknown changes coming with Patchwork_11069_full need to be verified
> manually.
>
> If you think the reported changes have nothing to do with the changes
> introduced in Patchwork_11069_full, please notify your bug team to allow them
> to document this new failure mode, which will reduce false positives in CI.
>
>
>
> Possible new issues
> -------------------
>
> Here are the unknown changes that may have been introduced in Patchwork_11069_full:
>
> ### IGT changes ###
>
> #### Warnings ####
>
> * igt@pm_rc6_residency@rc6-accuracy:
> - shard-snb: PASS -> SKIP
>
> * igt@tools_test@tools_test:
> - shard-skl: SKIP -> PASS
>
>
> Known issues
> ------------
>
> Here are the changes found in Patchwork_11069_full that come from known issues:
>
> ### IGT changes ###
>
> #### Issues hit ####
>
> * igt@gem_userptr_blits@readonly-unsync:
> - {shard-iclb}: PASS -> INCOMPLETE [fdo#108342]
>
> * igt@i915_selftest@live_contexts:
> - {shard-iclb}: NOTRUN -> DMESG-FAIL [fdo#108569]
>
> * igt@kms_busy@extended-modeset-hang-newfb-render-b:
> - shard-skl: NOTRUN -> DMESG-WARN [fdo#107956]
> - {shard-iclb}: NOTRUN -> DMESG-WARN [fdo#107956]
>
> * igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-b:
> - {shard-iclb}: PASS -> DMESG-WARN [fdo#107956]
>
> * igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-c:
> - shard-glk: NOTRUN -> DMESG-WARN [fdo#107956]
>
> * igt@kms_chv_cursor_fail@pipe-b-256x256-right-edge:
> - {shard-iclb}: PASS -> DMESG-WARN [fdo#107724] / [fdo#108336] +10
>
> * igt@kms_color@pipe-a-degamma:
> - shard-apl: PASS -> FAIL [fdo#104782] / [fdo#108145]
>
> * igt@kms_color@pipe-b-degamma:
> - shard-apl: PASS -> FAIL [fdo#104782]
>
> * igt@kms_color@pipe-b-gamma:
> - shard-skl: PASS -> FAIL [fdo#104782]
>
> * igt@kms_color@pipe-c-ctm-green-to-red:
> - shard-skl: PASS -> FAIL [fdo#107201]
>
> * igt@kms_cursor_crc@cursor-64x21-random:
> - shard-apl: PASS -> FAIL [fdo#103232] +11
>
> * igt@kms_cursor_crc@cursor-64x64-suspend:
> - shard-apl: PASS -> FAIL [fdo#103191] / [fdo#103232]
>
> * igt@kms_draw_crc@draw-method-rgb565-pwrite-ytiled:
> - {shard-iclb}: PASS -> FAIL [fdo#103184]
>
> * igt@kms_draw_crc@draw-method-xrgb2101010-render-ytiled:
> - {shard-iclb}: PASS -> WARN [fdo#108336] +2
>
> * igt@kms_fbcon_fbt@psr-suspend:
> - shard-skl: NOTRUN -> FAIL [fdo#107882]
>
> * igt@kms_flip@flip-vs-expired-vblank-interruptible:
> - shard-skl: PASS -> FAIL [fdo#105363]
>
> * igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-indfb-draw-mmap-gtt:
> - {shard-iclb}: PASS -> DMESG-FAIL [fdo#107724] +16
>
> * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-draw-render:
> - shard-apl: PASS -> FAIL [fdo#103167] +5
>
> * igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-draw-mmap-wc:
> - shard-glk: PASS -> FAIL [fdo#103167] +2
>
> * igt@kms_frontbuffer_tracking@fbc-stridechange:
> - {shard-iclb}: PASS -> FAIL [fdo#105683] / [fdo#108040]
>
> * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-draw-render:
> - {shard-iclb}: PASS -> FAIL [fdo#103167] +3
>
> * igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-draw-blt:
> - {shard-iclb}: NOTRUN -> FAIL [fdo#103167]
>
> * igt@kms_plane@plane-panning-bottom-right-suspend-pipe-c-planes:
> - shard-skl: PASS -> INCOMPLETE [fdo#104108]
>
> * igt@kms_plane@plane-position-covered-pipe-a-planes:
> - shard-glk: PASS -> FAIL [fdo#103166]
> - {shard-iclb}: NOTRUN -> FAIL [fdo#103166]
>
> * igt@kms_plane_alpha_blend@pipe-a-constant-alpha-max:
> - shard-skl: NOTRUN -> FAIL [fdo#108145] +1
>
> * igt@kms_plane_alpha_blend@pipe-b-alpha-opaque-fb:
> - shard-glk: PASS -> FAIL [fdo#108145]
>
> * igt@kms_plane_multiple@atomic-pipe-b-tiling-y:
> - {shard-iclb}: PASS -> FAIL [fdo#103166] +3
>
> * igt@kms_psr@cursor_mmap_gtt:
> - {shard-iclb}: PASS -> DMESG-WARN [fdo#107724] +19
>
> * igt@kms_sysfs_edid_timing:
> - shard-skl: NOTRUN -> FAIL [fdo#100047]
>
> * igt@kms_vblank@pipe-a-ts-continuation-dpms-suspend:
> - {shard-iclb}: PASS -> INCOMPLETE [fdo#107713]
>
> * igt@perf@short-reads:
> - shard-skl: PASS -> FAIL [fdo#103183]
>
> * igt@pm_rpm@modeset-stress-extra-wait:
> - {shard-iclb}: PASS -> INCOMPLETE [fdo#108840]
>
>
> #### Possible fixes ####
>
> * igt@kms_ccs@pipe-a-crc-sprite-planes-basic:
> - shard-apl: FAIL [fdo#105458] / [fdo#106510] -> PASS
>
> * igt@kms_color@pipe-c-degamma:
> - shard-apl: FAIL [fdo#104782] -> PASS
>
> * igt@kms_cursor_crc@cursor-64x21-sliding:
> - shard-apl: FAIL [fdo#103232] -> PASS +1
>
> * igt@kms_draw_crc@draw-method-rgb565-mmap-gtt-xtiled:
> - shard-skl: FAIL [fdo#103184] -> PASS
>
> * igt@kms_draw_crc@draw-method-rgb565-render-ytiled:
> - {shard-iclb}: WARN [fdo#108336] -> PASS +1
>
> * igt@kms_fbcon_fbt@fbc-suspend:
> - shard-kbl: FAIL [fdo#103833] / [fdo#105681] -> PASS
>
> * igt@kms_flip@dpms-off-confusion:
> - {shard-iclb}: DMESG-WARN [fdo#107724] -> PASS +21
>
> * igt@kms_flip_tiling@flip-changes-tiling:
> - {shard-iclb}: DMESG-WARN [fdo#107724] / [fdo#108336] -> PASS +10
>
> * igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-indfb-draw-pwrite:
> - shard-skl: FAIL [fdo#105682] -> PASS
> - {shard-iclb}: DMESG-FAIL [fdo#107724] -> PASS +6
>
> * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-mmap-gtt:
> - shard-apl: FAIL [fdo#103167] -> PASS +1
>
> * igt@kms_frontbuffer_tracking@fbcpsr-rgb565-draw-pwrite:
> - shard-skl: FAIL [fdo#103167] -> PASS +1
>
> * igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-draw-mmap-wc:
> - {shard-iclb}: FAIL [fdo#103167] -> PASS +1
>
> * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
> - shard-skl: INCOMPLETE [fdo#104108] / [fdo#107773] -> PASS
>
> * igt@kms_plane@pixel-format-pipe-c-planes:
> - shard-apl: FAIL [fdo#103166] -> PASS +1
>
> * igt@kms_plane_alpha_blend@pipe-a-coverage-7efc:
> - shard-skl: FAIL [fdo#107815] / [fdo#108145] -> PASS
>
> * igt@kms_plane_multiple@atomic-pipe-a-tiling-x:
> - shard-glk: FAIL [fdo#103166] -> PASS
>
> * {igt@kms_rotation_crc@multiplane-rotation-cropping-top}:
> - shard-kbl: DMESG-FAIL [fdo#108950] -> PASS
>
> * igt@kms_rotation_crc@sprite-rotation-90:
> - shard-glk: INCOMPLETE [fdo#103359] / [k.org#198133] -> PASS
>
> * igt@kms_setmode@basic:
> - shard-kbl: FAIL [fdo#99912] -> PASS
>
> * igt@kms_vblank@pipe-c-ts-continuation-suspend:
> - {shard-iclb}: INCOMPLETE [fdo#107713] -> PASS
>
> * igt@perf@blocking:
> - shard-hsw: FAIL [fdo#102252] -> PASS
>
> * igt@sw_sync@sync_busy_fork_unixsocket:
> - {shard-iclb}: INCOMPLETE [fdo#108889] -> PASS
>
>
> #### Warnings ####
>
> * igt@gem_cpu_reloc@full:
> - shard-skl: INCOMPLETE [fdo#108073] -> TIMEOUT [fdo#108248]
>
> * igt@i915_suspend@shrink:
> - shard-skl: DMESG-WARN [fdo#108784] -> INCOMPLETE [fdo#106886]
>
> * igt@kms_ccs@pipe-a-crc-primary-rotation-180:
> - {shard-iclb}: FAIL [fdo#107725] -> DMESG-WARN [fdo#107724] / [fdo#108336] +1
>
> * igt@kms_cursor_crc@cursor-256x256-random:
> - {shard-iclb}: DMESG-WARN [fdo#107724] / [fdo#108336] -> FAIL [fdo#103232] +3
>
> * igt@kms_cursor_crc@cursor-256x85-onscreen:
> - {shard-iclb}: FAIL [fdo#103232] -> DMESG-WARN [fdo#107724] / [fdo#108336]
>
> * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-draw-mmap-wc:
> - {shard-iclb}: DMESG-FAIL [fdo#107724] -> FAIL [fdo#103167] +1
>
> * igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-draw-render:
> - {shard-iclb}: DMESG-WARN [fdo#107724] / [fdo#108336] -> FAIL [fdo#103167]
>
> * igt@kms_plane@pixel-format-pipe-a-planes:
> - {shard-iclb}: DMESG-WARN [fdo#107724] / [fdo#108336] -> FAIL [fdo#103166]
>
> * {igt@kms_plane@pixel-format-pipe-a-planes-source-clamping}:
> - {shard-iclb}: DMESG-WARN [fdo#107724] / [fdo#108336] -> FAIL [fdo#108948]
>
>
> {name}: This element is suppressed. This means it is ignored when computing
> the status of the difference (SUCCESS, WARNING, or FAILURE).
>
> [fdo#100047]: https://bugs.freedesktop.org/show_bug.cgi?id=100047
> [fdo#102252]: https://bugs.freedesktop.org/show_bug.cgi?id=102252
> [fdo#103166]: https://bugs.freedesktop.org/show_bug.cgi?id=103166
> [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
> [fdo#103183]: https://bugs.freedesktop.org/show_bug.cgi?id=103183
> [fdo#103184]: https://bugs.freedesktop.org/show_bug.cgi?id=103184
> [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
> [fdo#103232]: https://bugs.freedesktop.org/show_bug.cgi?id=103232
> [fdo#103359]: https://bugs.freedesktop.org/show_bug.cgi?id=103359
> [fdo#103833]: https://bugs.freedesktop.org/show_bug.cgi?id=103833
> [fdo#104108]: https://bugs.freedesktop.org/show_bug.cgi?id=104108
> [fdo#104782]: https://bugs.freedesktop.org/show_bug.cgi?id=104782
> [fdo#105363]: https://bugs.freedesktop.org/show_bug.cgi?id=105363
> [fdo#105458]: https://bugs.freedesktop.org/show_bug.cgi?id=105458
> [fdo#105681]: https://bugs.freedesktop.org/show_bug.cgi?id=105681
> [fdo#105682]: https://bugs.freedesktop.org/show_bug.cgi?id=105682
> [fdo#105683]: https://bugs.freedesktop.org/show_bug.cgi?id=105683
> [fdo#106510]: https://bugs.freedesktop.org/show_bug.cgi?id=106510
> [fdo#106886]: https://bugs.freedesktop.org/show_bug.cgi?id=106886
> [fdo#107201]: https://bugs.freedesktop.org/show_bug.cgi?id=107201
> [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
> [fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724
> [fdo#107725]: https://bugs.freedesktop.org/show_bug.cgi?id=107725
> [fdo#107773]: https://bugs.freedesktop.org/show_bug.cgi?id=107773
> [fdo#107815]: https://bugs.freedesktop.org/show_bug.cgi?id=107815
> [fdo#107882]: https://bugs.freedesktop.org/show_bug.cgi?id=107882
> [fdo#107956]: https://bugs.freedesktop.org/show_bug.cgi?id=107956
> [fdo#108040]: https://bugs.freedesktop.org/show_bug.cgi?id=108040
> [fdo#108073]: https://bugs.freedesktop.org/show_bug.cgi?id=108073
> [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
> [fdo#108248]: https://bugs.freedesktop.org/show_bug.cgi?id=108248
> [fdo#108336]: https://bugs.freedesktop.org/show_bug.cgi?id=108336
> [fdo#108342]: https://bugs.freedesktop.org/show_bug.cgi?id=108342
> [fdo#108569]: https://bugs.freedesktop.org/show_bug.cgi?id=108569
> [fdo#108784]: https://bugs.freedesktop.org/show_bug.cgi?id=108784
> [fdo#108840]: https://bugs.freedesktop.org/show_bug.cgi?id=108840
> [fdo#108889]: https://bugs.freedesktop.org/show_bug.cgi?id=108889
> [fdo#108948]: https://bugs.freedesktop.org/show_bug.cgi?id=108948
> [fdo#108950]: https://bugs.freedesktop.org/show_bug.cgi?id=108950
> [fdo#99912]: https://bugs.freedesktop.org/show_bug.cgi?id=99912
> [k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133
>
>
> Participating hosts (7 -> 7)
> ------------------------------
>
> No changes in participating hosts
>
>
> Build changes
> -------------
>
> * Linux: CI_DRM_5296 -> Patchwork_11069
>
> CI_DRM_5296: 70751bd8a3f27b035d203ecafcad452f4d7c2c15 @ git://anongit.freedesktop.org/gfx-ci/linux
> IGT_4745: 3b52e8a5809a4e860350c59476a456745cd9fee0 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
> Patchwork_11069: 2e1fe301f9eaecd0f3525f75f16c525dac9f0354 @ 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_11069/
--
Matt Roper
Graphics Software Engineer
IoTG Platform Enabling & Development
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] 11+ messages in thread
* Re: ✓ Fi.CI.IGT: success for series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method (rev2)
2018-12-12 0:04 ` Matt Roper
@ 2018-12-12 7:45 ` Saarinen, Jani
0 siblings, 0 replies; 11+ messages in thread
From: Saarinen, Jani @ 2018-12-12 7:45 UTC (permalink / raw
To: Roper, Matthew D, intel-gfx@lists.freedesktop.org
Hi,
> -----Original Message-----
> From: Intel-gfx [mailto:intel-gfx-bounces@lists.freedesktop.org] On Behalf Of
> Matt Roper
> Sent: keskiviikko 12. joulukuuta 2018 2.05
> To: intel-gfx@lists.freedesktop.org
> Subject: Re: [Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [CI,1/2]
> drm/i915: Don't use DDB allocation when choosing gen9 watermark method
> (rev2)
>
> The rc6 pass->skip mentioned below doesn't appear to be related to this series,
> so pushing to dinq. Thanks to Ville for reviewing.
>
> I also notice that CI indicates a bunch of pre-existing ICL watermark failures are
> no longer happening with my series (or the earlier revisions of my series), so it's
> possible that we've also fixed
> https://bugs.freedesktop.org/show_bug.cgi?id=107724 "by accident" with this
> series.
Thanks Matt! Let's hope so.
>
>
> Matt
>
> On Tue, Dec 11, 2018 at 11:48:05PM +0000, Patchwork wrote:
> > == Series Details ==
> >
> > Series: series starting with [CI,1/2] drm/i915: Don't use DDB allocation when
> choosing gen9 watermark method (rev2)
> > URL : https://patchwork.freedesktop.org/series/53901/
> > State : success
> >
> > == Summary ==
> >
> > CI Bug Log - changes from CI_DRM_5296_full -> Patchwork_11069_full
> > ====================================================
> >
> > Summary
> > -------
> >
> > **WARNING**
> >
> > Minor unknown changes coming with Patchwork_11069_full need to be
> verified
> > manually.
> >
> > If you think the reported changes have nothing to do with the changes
> > introduced in Patchwork_11069_full, please notify your bug team to allow
> them
> > to document this new failure mode, which will reduce false positives in CI.
> >
> >
> >
> > Possible new issues
> > -------------------
> >
> > Here are the unknown changes that may have been introduced in
> Patchwork_11069_full:
> >
> > ### IGT changes ###
> >
> > #### Warnings ####
> >
> > * igt@pm_rc6_residency@rc6-accuracy:
> > - shard-snb: PASS -> SKIP
> >
> > * igt@tools_test@tools_test:
> > - shard-skl: SKIP -> PASS
> >
> >
> > Known issues
> > ------------
> >
> > Here are the changes found in Patchwork_11069_full that come from known
> issues:
> >
> > ### IGT changes ###
> >
> > #### Issues hit ####
> >
> > * igt@gem_userptr_blits@readonly-unsync:
> > - {shard-iclb}: PASS -> INCOMPLETE [fdo#108342]
> >
> > * igt@i915_selftest@live_contexts:
> > - {shard-iclb}: NOTRUN -> DMESG-FAIL [fdo#108569]
> >
> > * igt@kms_busy@extended-modeset-hang-newfb-render-b:
> > - shard-skl: NOTRUN -> DMESG-WARN [fdo#107956]
> > - {shard-iclb}: NOTRUN -> DMESG-WARN [fdo#107956]
> >
> > * igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-b:
> > - {shard-iclb}: PASS -> DMESG-WARN [fdo#107956]
> >
> > * igt@kms_busy@extended-modeset-hang-newfb-with-reset-render-c:
> > - shard-glk: NOTRUN -> DMESG-WARN [fdo#107956]
> >
> > * igt@kms_chv_cursor_fail@pipe-b-256x256-right-edge:
> > - {shard-iclb}: PASS -> DMESG-WARN [fdo#107724] / [fdo#108336] +10
> >
> > * igt@kms_color@pipe-a-degamma:
> > - shard-apl: PASS -> FAIL [fdo#104782] / [fdo#108145]
> >
> > * igt@kms_color@pipe-b-degamma:
> > - shard-apl: PASS -> FAIL [fdo#104782]
> >
> > * igt@kms_color@pipe-b-gamma:
> > - shard-skl: PASS -> FAIL [fdo#104782]
> >
> > * igt@kms_color@pipe-c-ctm-green-to-red:
> > - shard-skl: PASS -> FAIL [fdo#107201]
> >
> > * igt@kms_cursor_crc@cursor-64x21-random:
> > - shard-apl: PASS -> FAIL [fdo#103232] +11
> >
> > * igt@kms_cursor_crc@cursor-64x64-suspend:
> > - shard-apl: PASS -> FAIL [fdo#103191] / [fdo#103232]
> >
> > * igt@kms_draw_crc@draw-method-rgb565-pwrite-ytiled:
> > - {shard-iclb}: PASS -> FAIL [fdo#103184]
> >
> > * igt@kms_draw_crc@draw-method-xrgb2101010-render-ytiled:
> > - {shard-iclb}: PASS -> WARN [fdo#108336] +2
> >
> > * igt@kms_fbcon_fbt@psr-suspend:
> > - shard-skl: NOTRUN -> FAIL [fdo#107882]
> >
> > * igt@kms_flip@flip-vs-expired-vblank-interruptible:
> > - shard-skl: PASS -> FAIL [fdo#105363]
> >
> > * igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-indfb-draw-mmap-gtt:
> > - {shard-iclb}: PASS -> DMESG-FAIL [fdo#107724] +16
> >
> > * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-draw-render:
> > - shard-apl: PASS -> FAIL [fdo#103167] +5
> >
> > * igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-draw-mmap-wc:
> > - shard-glk: PASS -> FAIL [fdo#103167] +2
> >
> > * igt@kms_frontbuffer_tracking@fbc-stridechange:
> > - {shard-iclb}: PASS -> FAIL [fdo#105683] / [fdo#108040]
> >
> > * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-draw-render:
> > - {shard-iclb}: PASS -> FAIL [fdo#103167] +3
> >
> > * igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-draw-blt:
> > - {shard-iclb}: NOTRUN -> FAIL [fdo#103167]
> >
> > * igt@kms_plane@plane-panning-bottom-right-suspend-pipe-c-planes:
> > - shard-skl: PASS -> INCOMPLETE [fdo#104108]
> >
> > * igt@kms_plane@plane-position-covered-pipe-a-planes:
> > - shard-glk: PASS -> FAIL [fdo#103166]
> > - {shard-iclb}: NOTRUN -> FAIL [fdo#103166]
> >
> > * igt@kms_plane_alpha_blend@pipe-a-constant-alpha-max:
> > - shard-skl: NOTRUN -> FAIL [fdo#108145] +1
> >
> > * igt@kms_plane_alpha_blend@pipe-b-alpha-opaque-fb:
> > - shard-glk: PASS -> FAIL [fdo#108145]
> >
> > * igt@kms_plane_multiple@atomic-pipe-b-tiling-y:
> > - {shard-iclb}: PASS -> FAIL [fdo#103166] +3
> >
> > * igt@kms_psr@cursor_mmap_gtt:
> > - {shard-iclb}: PASS -> DMESG-WARN [fdo#107724] +19
> >
> > * igt@kms_sysfs_edid_timing:
> > - shard-skl: NOTRUN -> FAIL [fdo#100047]
> >
> > * igt@kms_vblank@pipe-a-ts-continuation-dpms-suspend:
> > - {shard-iclb}: PASS -> INCOMPLETE [fdo#107713]
> >
> > * igt@perf@short-reads:
> > - shard-skl: PASS -> FAIL [fdo#103183]
> >
> > * igt@pm_rpm@modeset-stress-extra-wait:
> > - {shard-iclb}: PASS -> INCOMPLETE [fdo#108840]
> >
> >
> > #### Possible fixes ####
> >
> > * igt@kms_ccs@pipe-a-crc-sprite-planes-basic:
> > - shard-apl: FAIL [fdo#105458] / [fdo#106510] -> PASS
> >
> > * igt@kms_color@pipe-c-degamma:
> > - shard-apl: FAIL [fdo#104782] -> PASS
> >
> > * igt@kms_cursor_crc@cursor-64x21-sliding:
> > - shard-apl: FAIL [fdo#103232] -> PASS +1
> >
> > * igt@kms_draw_crc@draw-method-rgb565-mmap-gtt-xtiled:
> > - shard-skl: FAIL [fdo#103184] -> PASS
> >
> > * igt@kms_draw_crc@draw-method-rgb565-render-ytiled:
> > - {shard-iclb}: WARN [fdo#108336] -> PASS +1
> >
> > * igt@kms_fbcon_fbt@fbc-suspend:
> > - shard-kbl: FAIL [fdo#103833] / [fdo#105681] -> PASS
> >
> > * igt@kms_flip@dpms-off-confusion:
> > - {shard-iclb}: DMESG-WARN [fdo#107724] -> PASS +21
> >
> > * igt@kms_flip_tiling@flip-changes-tiling:
> > - {shard-iclb}: DMESG-WARN [fdo#107724] / [fdo#108336] -> PASS +10
> >
> > * igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-indfb-draw-pwrite:
> > - shard-skl: FAIL [fdo#105682] -> PASS
> > - {shard-iclb}: DMESG-FAIL [fdo#107724] -> PASS +6
> >
> > * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-mmap-gtt:
> > - shard-apl: FAIL [fdo#103167] -> PASS +1
> >
> > * igt@kms_frontbuffer_tracking@fbcpsr-rgb565-draw-pwrite:
> > - shard-skl: FAIL [fdo#103167] -> PASS +1
> >
> > * igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-draw-mmap-wc:
> > - {shard-iclb}: FAIL [fdo#103167] -> PASS +1
> >
> > * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
> > - shard-skl: INCOMPLETE [fdo#104108] / [fdo#107773] -> PASS
> >
> > * igt@kms_plane@pixel-format-pipe-c-planes:
> > - shard-apl: FAIL [fdo#103166] -> PASS +1
> >
> > * igt@kms_plane_alpha_blend@pipe-a-coverage-7efc:
> > - shard-skl: FAIL [fdo#107815] / [fdo#108145] -> PASS
> >
> > * igt@kms_plane_multiple@atomic-pipe-a-tiling-x:
> > - shard-glk: FAIL [fdo#103166] -> PASS
> >
> > * {igt@kms_rotation_crc@multiplane-rotation-cropping-top}:
> > - shard-kbl: DMESG-FAIL [fdo#108950] -> PASS
> >
> > * igt@kms_rotation_crc@sprite-rotation-90:
> > - shard-glk: INCOMPLETE [fdo#103359] / [k.org#198133] -> PASS
> >
> > * igt@kms_setmode@basic:
> > - shard-kbl: FAIL [fdo#99912] -> PASS
> >
> > * igt@kms_vblank@pipe-c-ts-continuation-suspend:
> > - {shard-iclb}: INCOMPLETE [fdo#107713] -> PASS
> >
> > * igt@perf@blocking:
> > - shard-hsw: FAIL [fdo#102252] -> PASS
> >
> > * igt@sw_sync@sync_busy_fork_unixsocket:
> > - {shard-iclb}: INCOMPLETE [fdo#108889] -> PASS
> >
> >
> > #### Warnings ####
> >
> > * igt@gem_cpu_reloc@full:
> > - shard-skl: INCOMPLETE [fdo#108073] -> TIMEOUT [fdo#108248]
> >
> > * igt@i915_suspend@shrink:
> > - shard-skl: DMESG-WARN [fdo#108784] -> INCOMPLETE [fdo#106886]
> >
> > * igt@kms_ccs@pipe-a-crc-primary-rotation-180:
> > - {shard-iclb}: FAIL [fdo#107725] -> DMESG-WARN [fdo#107724] /
> [fdo#108336] +1
> >
> > * igt@kms_cursor_crc@cursor-256x256-random:
> > - {shard-iclb}: DMESG-WARN [fdo#107724] / [fdo#108336] -> FAIL
> [fdo#103232] +3
> >
> > * igt@kms_cursor_crc@cursor-256x85-onscreen:
> > - {shard-iclb}: FAIL [fdo#103232] -> DMESG-WARN [fdo#107724] /
> [fdo#108336]
> >
> > * igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-draw-mmap-
> wc:
> > - {shard-iclb}: DMESG-FAIL [fdo#107724] -> FAIL [fdo#103167] +1
> >
> > * igt@kms_frontbuffer_tracking@psr-1p-primscrn-spr-indfb-draw-render:
> > - {shard-iclb}: DMESG-WARN [fdo#107724] / [fdo#108336] -> FAIL
> [fdo#103167]
> >
> > * igt@kms_plane@pixel-format-pipe-a-planes:
> > - {shard-iclb}: DMESG-WARN [fdo#107724] / [fdo#108336] -> FAIL
> [fdo#103166]
> >
> > * {igt@kms_plane@pixel-format-pipe-a-planes-source-clamping}:
> > - {shard-iclb}: DMESG-WARN [fdo#107724] / [fdo#108336] -> FAIL
> [fdo#108948]
> >
> >
> > {name}: This element is suppressed. This means it is ignored when computing
> > the status of the difference (SUCCESS, WARNING, or FAILURE).
> >
> > [fdo#100047]: https://bugs.freedesktop.org/show_bug.cgi?id=100047
> > [fdo#102252]: https://bugs.freedesktop.org/show_bug.cgi?id=102252
> > [fdo#103166]: https://bugs.freedesktop.org/show_bug.cgi?id=103166
> > [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
> > [fdo#103183]: https://bugs.freedesktop.org/show_bug.cgi?id=103183
> > [fdo#103184]: https://bugs.freedesktop.org/show_bug.cgi?id=103184
> > [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191
> > [fdo#103232]: https://bugs.freedesktop.org/show_bug.cgi?id=103232
> > [fdo#103359]: https://bugs.freedesktop.org/show_bug.cgi?id=103359
> > [fdo#103833]: https://bugs.freedesktop.org/show_bug.cgi?id=103833
> > [fdo#104108]: https://bugs.freedesktop.org/show_bug.cgi?id=104108
> > [fdo#104782]: https://bugs.freedesktop.org/show_bug.cgi?id=104782
> > [fdo#105363]: https://bugs.freedesktop.org/show_bug.cgi?id=105363
> > [fdo#105458]: https://bugs.freedesktop.org/show_bug.cgi?id=105458
> > [fdo#105681]: https://bugs.freedesktop.org/show_bug.cgi?id=105681
> > [fdo#105682]: https://bugs.freedesktop.org/show_bug.cgi?id=105682
> > [fdo#105683]: https://bugs.freedesktop.org/show_bug.cgi?id=105683
> > [fdo#106510]: https://bugs.freedesktop.org/show_bug.cgi?id=106510
> > [fdo#106886]: https://bugs.freedesktop.org/show_bug.cgi?id=106886
> > [fdo#107201]: https://bugs.freedesktop.org/show_bug.cgi?id=107201
> > [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
> > [fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724
> > [fdo#107725]: https://bugs.freedesktop.org/show_bug.cgi?id=107725
> > [fdo#107773]: https://bugs.freedesktop.org/show_bug.cgi?id=107773
> > [fdo#107815]: https://bugs.freedesktop.org/show_bug.cgi?id=107815
> > [fdo#107882]: https://bugs.freedesktop.org/show_bug.cgi?id=107882
> > [fdo#107956]: https://bugs.freedesktop.org/show_bug.cgi?id=107956
> > [fdo#108040]: https://bugs.freedesktop.org/show_bug.cgi?id=108040
> > [fdo#108073]: https://bugs.freedesktop.org/show_bug.cgi?id=108073
> > [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
> > [fdo#108248]: https://bugs.freedesktop.org/show_bug.cgi?id=108248
> > [fdo#108336]: https://bugs.freedesktop.org/show_bug.cgi?id=108336
> > [fdo#108342]: https://bugs.freedesktop.org/show_bug.cgi?id=108342
> > [fdo#108569]: https://bugs.freedesktop.org/show_bug.cgi?id=108569
> > [fdo#108784]: https://bugs.freedesktop.org/show_bug.cgi?id=108784
> > [fdo#108840]: https://bugs.freedesktop.org/show_bug.cgi?id=108840
> > [fdo#108889]: https://bugs.freedesktop.org/show_bug.cgi?id=108889
> > [fdo#108948]: https://bugs.freedesktop.org/show_bug.cgi?id=108948
> > [fdo#108950]: https://bugs.freedesktop.org/show_bug.cgi?id=108950
> > [fdo#99912]: https://bugs.freedesktop.org/show_bug.cgi?id=99912
> > [k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133
> >
> >
> > Participating hosts (7 -> 7)
> > ------------------------------
> >
> > No changes in participating hosts
> >
> >
> > Build changes
> > -------------
> >
> > * Linux: CI_DRM_5296 -> Patchwork_11069
> >
> > CI_DRM_5296: 70751bd8a3f27b035d203ecafcad452f4d7c2c15 @
> git://anongit.freedesktop.org/gfx-ci/linux
> > IGT_4745: 3b52e8a5809a4e860350c59476a456745cd9fee0 @
> git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
> > Patchwork_11069: 2e1fe301f9eaecd0f3525f75f16c525dac9f0354 @
> 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_11069/
>
> --
> Matt Roper
> Graphics Software Engineer
> IoTG Platform Enabling & Development
> Intel Corporation
> (916) 356-2795
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2018-12-12 7:45 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-12-11 17:31 [CI 1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method Matt Roper
2018-12-11 17:31 ` [CI 2/2] drm/i915: Switch to level-based DDB allocation algorithm (v5) Matt Roper
2018-12-11 18:58 ` [CI 2/2] drm/i915: Switch to level-based DDB allocation algorithm (v6) Matt Roper
2018-12-11 17:57 ` ✗ Fi.CI.SPARSE: warning for series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method Patchwork
2018-12-11 18:12 ` ✓ Fi.CI.BAT: success " Patchwork
2018-12-11 19:14 ` ✗ Fi.CI.SPARSE: warning for series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method (rev2) Patchwork
2018-12-11 19:29 ` ✓ Fi.CI.BAT: success " Patchwork
2018-12-11 20:41 ` ✓ Fi.CI.IGT: success for series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method Patchwork
2018-12-11 23:48 ` ✓ Fi.CI.IGT: success for series starting with [CI,1/2] drm/i915: Don't use DDB allocation when choosing gen9 watermark method (rev2) Patchwork
2018-12-12 0:04 ` Matt Roper
2018-12-12 7:45 ` Saarinen, Jani
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.