[PATCH 16/16] drm/i915: Make sure watermarks work correctly with bigjoiner as well.
Maarten Lankhorst
maarten.lankhorst at linux.intel.com
Tue Aug 6 08:51:59 UTC 2019
For bigjoiner, we cannot do drm_atomic_crtc_state_for_each_plane_state()
on the crtc, because planes don't match the drm core state.
We need a separate master_plane_state for all the properties,
and a slave_plane_state for the rectangles/visibility etc.
This is similar to how we handle the Y plane, because it won't be
directly calculated either. Instead it's calculated from the master
crtc.
Iterate over the plane_states from the bigjoiner master crtc, and
obtain the plane_state by getting the correct plane from
master_plane_state->bigjoiner_plane and then obtaining its plane_state.
Signed-off-by: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
---
drivers/gpu/drm/i915/intel_pm.c | 221 ++++++++++++++++++++++----------
1 file changed, 150 insertions(+), 71 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
index 2a2b2d2f830e..965d5be99d36 100644
--- a/drivers/gpu/drm/i915/intel_pm.c
+++ b/drivers/gpu/drm/i915/intel_pm.c
@@ -811,8 +811,10 @@ static int intel_wm_num_levels(struct drm_i915_private *dev_priv)
return dev_priv->wm.max_level + 1;
}
-static bool intel_wm_plane_visible(const struct intel_crtc_state *crtc_state,
- const struct intel_plane_state *plane_state)
+static bool
+intel_wm_plane_visible(const struct intel_crtc_state *crtc_state,
+ const struct intel_plane_state *master_plane_state,
+ const struct intel_plane_state *plane_state)
{
struct intel_plane *plane = to_intel_plane(plane_state->base.plane);
@@ -829,7 +831,7 @@ static bool intel_wm_plane_visible(const struct intel_crtc_state *crtc_state,
* around this problem with the watermark code.
*/
if (plane->id == PLANE_CURSOR)
- return plane_state->base.fb != NULL;
+ return master_plane_state->base.fb != NULL;
else
return plane_state->base.visible;
}
@@ -1113,7 +1115,7 @@ static u16 g4x_compute_wm(const struct intel_crtc_state *crtc_state,
if (latency == 0)
return USHRT_MAX;
- if (!intel_wm_plane_visible(crtc_state, plane_state))
+ if (!intel_wm_plane_visible(crtc_state, plane_state, plane_state))
return 0;
cpp = plane_state->base.fb->format->cpp[0];
@@ -1211,7 +1213,7 @@ static bool g4x_raw_plane_wm_compute(struct intel_crtc_state *crtc_state,
bool dirty = false;
int level;
- if (!intel_wm_plane_visible(crtc_state, plane_state)) {
+ if (!intel_wm_plane_visible(crtc_state, plane_state, plane_state)) {
dirty |= g4x_raw_plane_wm_set(crtc_state, 0, plane_id, 0);
if (plane_id == PLANE_PRIMARY)
dirty |= g4x_raw_fbc_wm_set(crtc_state, 0, 0);
@@ -1621,7 +1623,7 @@ static u16 vlv_compute_wm_level(const struct intel_crtc_state *crtc_state,
if (dev_priv->wm.pri_latency[level] == 0)
return USHRT_MAX;
- if (!intel_wm_plane_visible(crtc_state, plane_state))
+ if (!intel_wm_plane_visible(crtc_state, plane_state, plane_state))
return 0;
cpp = plane_state->base.fb->format->cpp[0];
@@ -1788,7 +1790,7 @@ static bool vlv_raw_plane_wm_compute(struct intel_crtc_state *crtc_state,
int level;
bool dirty = false;
- if (!intel_wm_plane_visible(crtc_state, plane_state)) {
+ if (!intel_wm_plane_visible(crtc_state, plane_state, plane_state)) {
dirty |= vlv_raw_plane_wm_set(crtc_state, 0, plane_id, 0);
goto out;
}
@@ -2503,7 +2505,7 @@ static u32 ilk_compute_pri_wm(const struct intel_crtc_state *crtc_state,
if (mem_value == 0)
return U32_MAX;
- if (!intel_wm_plane_visible(crtc_state, plane_state))
+ if (!intel_wm_plane_visible(crtc_state, plane_state, plane_state))
return 0;
cpp = plane_state->base.fb->format->cpp[0];
@@ -2535,7 +2537,7 @@ static u32 ilk_compute_spr_wm(const struct intel_crtc_state *crtc_state,
if (mem_value == 0)
return U32_MAX;
- if (!intel_wm_plane_visible(crtc_state, plane_state))
+ if (!intel_wm_plane_visible(crtc_state, plane_state, plane_state))
return 0;
cpp = plane_state->base.fb->format->cpp[0];
@@ -2561,7 +2563,7 @@ static u32 ilk_compute_cur_wm(const struct intel_crtc_state *crtc_state,
if (mem_value == 0)
return U32_MAX;
- if (!intel_wm_plane_visible(crtc_state, plane_state))
+ if (!intel_wm_plane_visible(crtc_state, plane_state, plane_state))
return 0;
cpp = plane_state->base.fb->format->cpp[0];
@@ -2578,7 +2580,7 @@ static u32 ilk_compute_fbc_wm(const struct intel_crtc_state *crtc_state,
{
int cpp;
- if (!intel_wm_plane_visible(crtc_state, plane_state))
+ if (!intel_wm_plane_visible(crtc_state, plane_state, plane_state))
return 0;
cpp = plane_state->base.fb->format->cpp[0];
@@ -4058,6 +4060,7 @@ void skl_ddb_get_hw_state(struct drm_i915_private *dev_priv,
*/
static uint_fixed_16_16_t
skl_plane_downscale_amount(const struct intel_crtc_state *crtc_state,
+ const struct intel_plane_state *master_plane_state,
const struct intel_plane_state *plane_state)
{
struct intel_plane *plane = to_intel_plane(plane_state->base.plane);
@@ -4065,7 +4068,8 @@ skl_plane_downscale_amount(const struct intel_crtc_state *crtc_state,
uint_fixed_16_16_t fp_w_ratio, fp_h_ratio;
uint_fixed_16_16_t downscale_h, downscale_w;
- if (WARN_ON(!intel_wm_plane_visible(crtc_state, plane_state)))
+ if (WARN_ON(!intel_wm_plane_visible(crtc_state,
+ master_plane_state, plane_state)))
return u32_to_fixed16(0);
/* n.b., src is 16.16 fixed point, dst is whole integer */
@@ -4074,10 +4078,10 @@ skl_plane_downscale_amount(const struct intel_crtc_state *crtc_state,
* Cursors only support 0/180 degree rotation,
* hence no need to account for rotation here.
*/
- src_w = plane_state->base.src_w >> 16;
- src_h = plane_state->base.src_h >> 16;
- dst_w = plane_state->base.crtc_w;
- dst_h = plane_state->base.crtc_h;
+ src_w = master_plane_state->base.src_w >> 16;
+ src_h = master_plane_state->base.src_h >> 16;
+ dst_w = master_plane_state->base.crtc_w;
+ dst_h = master_plane_state->base.crtc_h;
} else {
/*
* Src coordinates are already rotated by 270 degrees for
@@ -4131,36 +4135,64 @@ skl_pipe_downscale_amount(const struct intel_crtc_state *crtc_state)
return pipe_downscale;
}
+static const struct intel_plane_state *
+get_slave_plane_state(const struct intel_crtc_state *crtc_state,
+ const struct intel_plane_state *master_plane_state)
+{
+ struct drm_plane *slave;
+
+ if (crtc_state->bigjoiner_master_crtc == NULL)
+ return master_plane_state;
+
+ slave = &master_plane_state->bigjoiner_plane->base;
+ return to_intel_plane_state(
+ __drm_atomic_get_current_plane_state(crtc_state->uapi.state,
+ slave));
+}
+
int skl_check_pipe_max_pixel_rate(struct intel_crtc *intel_crtc,
struct intel_crtc_state *crtc_state)
{
struct drm_i915_private *dev_priv = to_i915(intel_crtc->base.dev);
- struct drm_atomic_state *state = crtc_state->uapi.state;
+ struct intel_atomic_state *state =
+ to_intel_atomic_state(crtc_state->uapi.state);
struct drm_plane *plane;
const struct drm_plane_state *drm_plane_state;
int crtc_clock, dotclk;
u32 pipe_max_pixel_rate;
uint_fixed_16_16_t pipe_downscale;
uint_fixed_16_16_t max_downscale = u32_to_fixed16(1);
+ const struct intel_crtc_state *master_crtc_state = crtc_state;
+
+ if (crtc_state->bigjoiner_master_crtc != NULL)
+ master_crtc_state = intel_atomic_get_new_crtc_state(state,
+ crtc_state->bigjoiner_master_crtc);
if (!crtc_state->hw.enable)
return 0;
- drm_atomic_crtc_state_for_each_plane_state(plane, drm_plane_state, &crtc_state->uapi) {
+ drm_atomic_crtc_state_for_each_plane_state(plane, drm_plane_state,
+ &master_crtc_state->uapi) {
uint_fixed_16_16_t plane_downscale;
uint_fixed_16_16_t fp_9_div_8 = div_fixed16(9, 8);
int bpp;
- const struct intel_plane_state *plane_state =
+ const struct intel_plane_state *master_plane_state =
to_intel_plane_state(drm_plane_state);
+ const struct intel_plane_state *plane_state =
+ get_slave_plane_state(crtc_state, master_plane_state);
- if (!intel_wm_plane_visible(crtc_state, plane_state))
+ if (!intel_wm_plane_visible(crtc_state,
+ master_plane_state, plane_state))
continue;
- if (WARN_ON(!plane_state->base.fb))
+ if (WARN_ON(!master_plane_state->base.fb))
return -EINVAL;
- plane_downscale = skl_plane_downscale_amount(crtc_state, plane_state);
- bpp = plane_state->base.fb->format->cpp[0] * 8;
+ plane_downscale = skl_plane_downscale_amount(crtc_state,
+ master_plane_state,
+ plane_state);
+
+ bpp = master_plane_state->base.fb->format->cpp[0] * 8;
if (bpp == 64)
plane_downscale = mul_fixed16(plane_downscale,
fp_9_div_8);
@@ -4172,7 +4204,7 @@ int skl_check_pipe_max_pixel_rate(struct intel_crtc *intel_crtc,
pipe_downscale = mul_fixed16(pipe_downscale, max_downscale);
crtc_clock = crtc_state->hw.adjusted_mode.crtc_clock;
- dotclk = to_intel_atomic_state(state)->cdclk.logical.cdclk;
+ dotclk = state->cdclk.logical.cdclk;
if (IS_GEMINILAKE(dev_priv) || INTEL_GEN(dev_priv) >= 10)
dotclk *= 2;
@@ -4189,6 +4221,7 @@ int skl_check_pipe_max_pixel_rate(struct intel_crtc *intel_crtc,
static u64
skl_plane_relative_data_rate(const struct intel_crtc_state *crtc_state,
+ const struct intel_plane_state *master_plane_state,
const struct intel_plane_state *plane_state,
const int plane)
{
@@ -4202,7 +4235,7 @@ skl_plane_relative_data_rate(const struct intel_crtc_state *crtc_state,
if (!plane_state->base.visible)
return 0;
- fb = plane_state->base.fb;
+ fb = master_plane_state->base.fb;
if (intel_plane->id == PLANE_CURSOR)
return 0;
@@ -4225,7 +4258,9 @@ skl_plane_relative_data_rate(const struct intel_crtc_state *crtc_state,
data_rate = width * height;
- down_scale_amount = skl_plane_downscale_amount(crtc_state, plane_state);
+ down_scale_amount = skl_plane_downscale_amount(crtc_state,
+ master_plane_state,
+ plane_state);
rate = mul_round_up_u32_fixed16(data_rate, down_scale_amount);
@@ -4247,19 +4282,22 @@ skl_get_total_relative_data_rate(struct intel_crtc_state *crtc_state,
return 0;
/* Calculate and cache data rate for each plane */
- drm_atomic_crtc_state_for_each_plane_state(plane, drm_plane_state, &crtc_state->uapi) {
+ drm_atomic_crtc_state_for_each_plane_state(plane, drm_plane_state,
+ &crtc_state->uapi) {
enum plane_id plane_id = to_intel_plane(plane)->id;
const struct intel_plane_state *plane_state =
to_intel_plane_state(drm_plane_state);
u64 rate;
/* packed/y */
- rate = skl_plane_relative_data_rate(crtc_state, plane_state, 0);
+ rate = skl_plane_relative_data_rate(crtc_state, plane_state,
+ plane_state, 0);
plane_data_rate[plane_id] = rate;
total_data_rate += rate;
/* uv-plane */
- rate = skl_plane_relative_data_rate(crtc_state, plane_state, 1);
+ rate = skl_plane_relative_data_rate(crtc_state, plane_state,
+ plane_state, 1);
uv_plane_data_rate[plane_id] = rate;
total_data_rate += rate;
}
@@ -4271,6 +4309,9 @@ static u64
icl_get_total_relative_data_rate(struct intel_crtc_state *crtc_state,
u64 *plane_data_rate)
{
+ struct intel_atomic_state *state =
+ to_intel_atomic_state(crtc_state->uapi.state);
+ const struct intel_crtc_state *master_crtc_state = crtc_state;
struct drm_plane *plane;
const struct drm_plane_state *drm_plane_state;
u64 total_data_rate = 0;
@@ -4278,40 +4319,54 @@ icl_get_total_relative_data_rate(struct intel_crtc_state *crtc_state,
if (WARN_ON(!crtc_state->uapi.state))
return 0;
+ if (crtc_state->bigjoiner_master_crtc != NULL)
+ master_crtc_state = intel_atomic_get_new_crtc_state(state,
+ crtc_state->bigjoiner_master_crtc);
+
/* Calculate and cache data rate for each plane */
- drm_atomic_crtc_state_for_each_plane_state(plane, drm_plane_state, &crtc_state->uapi) {
- const struct intel_plane_state *plane_state =
+ drm_atomic_crtc_state_for_each_plane_state(plane, drm_plane_state,
+ &master_crtc_state->uapi) {
+ const struct intel_plane_state *master_plane_state =
to_intel_plane_state(drm_plane_state);
- enum plane_id plane_id = to_intel_plane(plane)->id;
+ enum plane_id plane_id;
u64 rate;
+ const struct intel_plane_state *plane_state =
+ get_slave_plane_state(crtc_state, master_plane_state);
+ plane_id = to_intel_plane(plane_state->base.plane)->id;
if (!plane_state->linked_plane) {
- rate = skl_plane_relative_data_rate(crtc_state, plane_state, 0);
+ rate = skl_plane_relative_data_rate(crtc_state,
+ master_plane_state,
+ plane_state, 0);
plane_data_rate[plane_id] = rate;
total_data_rate += rate;
} else {
enum plane_id y_plane_id;
/*
- * The slave plane might not iterate in
- * drm_atomic_crtc_state_for_each_plane_state(),
- * and needs the master plane state which may be
- * NULL if we try get_new_plane_state(), so we
- * always calculate from the master.
- */
+ * The slave plane might not iterate in
+ * drm_atomic_crtc_state_for_each_plane_state(),
+ * and needs the master plane state which may be
+ * NULL if we try get_new_plane_state(), so we
+ * always calculate from the master.
+ */
if (plane_state->slave)
continue;
/* Y plane rate is calculated on the slave */
- rate = skl_plane_relative_data_rate(crtc_state, plane_state, 0);
+ rate = skl_plane_relative_data_rate(crtc_state,
+ master_plane_state,
+ plane_state, 0);
y_plane_id = plane_state->linked_plane->id;
plane_data_rate[y_plane_id] = rate;
total_data_rate += rate;
- rate = skl_plane_relative_data_rate(crtc_state, plane_state, 1);
+ rate = skl_plane_relative_data_rate(crtc_state,
+ master_plane_state,
+ plane_state, 1);
plane_data_rate[plane_id] = rate;
total_data_rate += rate;
- }
+ }
}
return total_data_rate;
@@ -4602,13 +4657,15 @@ intel_get_linetime_us(const struct intel_crtc_state *crtc_state)
static u32
skl_adjusted_plane_pixel_rate(const struct intel_crtc_state *crtc_state,
+ const struct intel_plane_state *master_plane_state,
const struct intel_plane_state *plane_state)
{
u64 adjusted_pixel_rate;
uint_fixed_16_16_t downscale_amount;
/* Shouldn't reach here on disabled planes... */
- if (WARN_ON(!intel_wm_plane_visible(crtc_state, plane_state)))
+ if (WARN_ON(!intel_wm_plane_visible(crtc_state,
+ master_plane_state, plane_state)))
return 0;
/*
@@ -4616,7 +4673,9 @@ skl_adjusted_plane_pixel_rate(const struct intel_crtc_state *crtc_state,
* with additional adjustments for plane-specific scaling.
*/
adjusted_pixel_rate = crtc_state->pixel_rate;
- downscale_amount = skl_plane_downscale_amount(crtc_state, plane_state);
+ downscale_amount = skl_plane_downscale_amount(crtc_state,
+ master_plane_state,
+ plane_state);
return mul_round_up_u32_fixed16(adjusted_pixel_rate,
downscale_amount);
@@ -4715,15 +4774,16 @@ skl_compute_wm_params(const struct intel_crtc_state *crtc_state,
static int
skl_compute_plane_wm_params(const struct intel_crtc_state *crtc_state,
+ const struct intel_plane_state *master_plane_state,
const struct intel_plane_state *plane_state,
struct skl_wm_params *wp, int color_plane)
{
struct intel_plane *plane = to_intel_plane(plane_state->base.plane);
- const struct drm_framebuffer *fb = plane_state->base.fb;
+ const struct drm_framebuffer *fb = master_plane_state->base.fb;
int width;
if (plane->id == PLANE_CURSOR) {
- width = plane_state->base.crtc_w;
+ width = master_plane_state->base.crtc_w;
} else {
/*
* Src coordinates are already rotated by 270 degrees for
@@ -4735,8 +4795,8 @@ skl_compute_plane_wm_params(const struct intel_crtc_state *crtc_state,
return skl_compute_wm_params(crtc_state, width,
fb->format, fb->modifier,
- plane_state->base.rotation,
- skl_adjusted_plane_pixel_rate(crtc_state, plane_state),
+ master_plane_state->base.rotation,
+ skl_adjusted_plane_pixel_rate(crtc_state, master_plane_state, plane_state),
wp, color_plane);
}
@@ -4971,6 +5031,7 @@ static void skl_compute_transition_wm(const struct intel_crtc_state *crtc_state,
}
static int skl_build_plane_wm_single(struct intel_crtc_state *crtc_state,
+ const struct intel_plane_state *master_plane_state,
const struct intel_plane_state *plane_state,
enum plane_id plane_id, int color_plane)
{
@@ -4978,8 +5039,8 @@ static int skl_build_plane_wm_single(struct intel_crtc_state *crtc_state,
struct skl_wm_params wm_params;
int ret;
- ret = skl_compute_plane_wm_params(crtc_state, plane_state,
- &wm_params, color_plane);
+ ret = skl_compute_plane_wm_params(crtc_state, master_plane_state,
+ plane_state, &wm_params, color_plane);
if (ret)
return ret;
@@ -5001,7 +5062,7 @@ static int skl_build_plane_wm_uv(struct intel_crtc_state *crtc_state,
/* uv plane watermarks must also be validated for NV12/Planar */
ret = skl_compute_plane_wm_params(crtc_state, plane_state,
- &wm_params, 1);
+ plane_state, &wm_params, 1);
if (ret)
return ret;
@@ -5018,10 +5079,10 @@ static int skl_build_plane_wm(struct intel_crtc_state *crtc_state,
enum plane_id plane_id = plane->id;
int ret;
- if (!intel_wm_plane_visible(crtc_state, plane_state))
+ if (!intel_wm_plane_visible(crtc_state, plane_state, plane_state))
return 0;
- ret = skl_build_plane_wm_single(crtc_state, plane_state,
+ ret = skl_build_plane_wm_single(crtc_state, plane_state, plane_state,
plane_id, 0);
if (ret)
return ret;
@@ -5037,6 +5098,7 @@ static int skl_build_plane_wm(struct intel_crtc_state *crtc_state,
}
static int icl_build_plane_wm(struct intel_crtc_state *crtc_state,
+ const struct intel_plane_state *master_plane_state,
const struct intel_plane_state *plane_state)
{
enum plane_id plane_id = to_intel_plane(plane_state->base.plane)->id;
@@ -5047,27 +5109,28 @@ static int icl_build_plane_wm(struct intel_crtc_state *crtc_state,
return 0;
if (plane_state->linked_plane) {
- const struct drm_framebuffer *fb = plane_state->base.fb;
+ const struct drm_framebuffer *fb = master_plane_state->base.fb;
enum plane_id y_plane_id = plane_state->linked_plane->id;
- WARN_ON(!intel_wm_plane_visible(crtc_state, plane_state));
+ WARN_ON(!intel_wm_plane_visible(crtc_state,
+ master_plane_state,
+ plane_state));
WARN_ON(!fb->format->is_yuv ||
fb->format->num_planes == 1);
- ret = skl_build_plane_wm_single(crtc_state, plane_state,
- y_plane_id, 0);
+ ret = skl_build_plane_wm_single(crtc_state, master_plane_state,
+ plane_state, y_plane_id, 0);
if (ret)
return ret;
- ret = skl_build_plane_wm_single(crtc_state, plane_state,
- plane_id, 1);
- if (ret)
- return ret;
- } else if (intel_wm_plane_visible(crtc_state, plane_state)) {
- ret = skl_build_plane_wm_single(crtc_state, plane_state,
- plane_id, 0);
+ ret = skl_build_plane_wm_single(crtc_state, master_plane_state,
+ plane_state, plane_id, 1);
if (ret)
return ret;
+ } else if (intel_wm_plane_visible(crtc_state,
+ master_plane_state, plane_state)) {
+ ret = skl_build_plane_wm_single(crtc_state, master_plane_state,
+ plane_state, plane_id, 0);
}
return 0;
@@ -5077,10 +5140,17 @@ static int skl_build_pipe_wm(struct intel_crtc_state *crtc_state)
{
struct drm_i915_private *dev_priv = to_i915(crtc_state->uapi.crtc->dev);
struct skl_pipe_wm *pipe_wm = &crtc_state->wm.skl.optimal;
- struct drm_plane *plane;
+ const struct intel_crtc_state *master_crtc_state = crtc_state;
+ struct intel_atomic_state *state =
+ to_intel_atomic_state(crtc_state->uapi.state);
const struct drm_plane_state *drm_plane_state;
+ struct drm_plane *plane;
int ret;
+ if (crtc_state->bigjoiner_master_crtc != NULL)
+ master_crtc_state = intel_atomic_get_new_crtc_state(state,
+ crtc_state->bigjoiner_master_crtc);
+
/*
* We'll only calculate watermarks for planes that are actually
* enabled, so make sure all other planes are set as disabled.
@@ -5088,14 +5158,23 @@ static int skl_build_pipe_wm(struct intel_crtc_state *crtc_state)
memset(pipe_wm->planes, 0, sizeof(pipe_wm->planes));
drm_atomic_crtc_state_for_each_plane_state(plane, drm_plane_state,
- &crtc_state->uapi) {
- const struct intel_plane_state *plane_state =
+ &master_crtc_state->uapi) {
+ const struct intel_plane_state *master_plane_state =
to_intel_plane_state(drm_plane_state);
- if (INTEL_GEN(dev_priv) >= 11)
- ret = icl_build_plane_wm(crtc_state, plane_state);
- else
- ret = skl_build_plane_wm(crtc_state, plane_state);
+ if (INTEL_GEN(dev_priv) >= 11) {
+ const struct intel_plane_state *plane_state =
+ get_slave_plane_state(crtc_state,
+ master_plane_state);
+
+ ret = icl_build_plane_wm(crtc_state,
+ master_plane_state,
+ plane_state);
+ } else {
+ ret = skl_build_plane_wm(crtc_state,
+ master_plane_state);
+ }
+
if (ret)
return ret;
}
--
2.20.1
More information about the Intel-gfx-trybot
mailing list