Commit a1de91e5 authored by Matt Roper's avatar Matt Roper

drm/i915/gen9: Cache plane data rates in CRTC state

This will be important when we start calculating CRTC data rates for
in-flight CRTC states since it will allow us to calculate the total data
rate without needing to grab the plane state for any planes that aren't
updated by the transaction.
Signed-off-by: default avatarMatt Roper <matthew.d.roper@intel.com>
Reviewed-by: default avatarMaarten Lankhorst <maarten.lankhorst@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1463061971-19638-4-git-send-email-matthew.d.roper@intel.com
parent e7649b54
...@@ -427,6 +427,10 @@ struct intel_crtc_wm_state { ...@@ -427,6 +427,10 @@ struct intel_crtc_wm_state {
struct { struct {
/* gen9+ only needs 1-step wm programming */ /* gen9+ only needs 1-step wm programming */
struct skl_pipe_wm optimal; struct skl_pipe_wm optimal;
/* cached plane data rate */
unsigned plane_data_rate[I915_MAX_PLANES];
unsigned plane_y_data_rate[I915_MAX_PLANES];
} skl; } skl;
}; };
......
...@@ -2940,6 +2940,14 @@ skl_plane_relative_data_rate(const struct intel_crtc_state *cstate, ...@@ -2940,6 +2940,14 @@ skl_plane_relative_data_rate(const struct intel_crtc_state *cstate,
struct intel_plane_state *intel_pstate = to_intel_plane_state(pstate); struct intel_plane_state *intel_pstate = to_intel_plane_state(pstate);
struct drm_framebuffer *fb = pstate->fb; struct drm_framebuffer *fb = pstate->fb;
uint32_t width = 0, height = 0; uint32_t width = 0, height = 0;
unsigned format = fb ? fb->pixel_format : DRM_FORMAT_XRGB8888;
if (!intel_pstate->visible)
return 0;
if (pstate->plane->type == DRM_PLANE_TYPE_CURSOR)
return 0;
if (y && format != DRM_FORMAT_NV12)
return 0;
width = drm_rect_width(&intel_pstate->src) >> 16; width = drm_rect_width(&intel_pstate->src) >> 16;
height = drm_rect_height(&intel_pstate->src) >> 16; height = drm_rect_height(&intel_pstate->src) >> 16;
...@@ -2948,17 +2956,17 @@ skl_plane_relative_data_rate(const struct intel_crtc_state *cstate, ...@@ -2948,17 +2956,17 @@ skl_plane_relative_data_rate(const struct intel_crtc_state *cstate,
swap(width, height); swap(width, height);
/* for planar format */ /* for planar format */
if (fb->pixel_format == DRM_FORMAT_NV12) { if (format == DRM_FORMAT_NV12) {
if (y) /* y-plane data rate */ if (y) /* y-plane data rate */
return width * height * return width * height *
drm_format_plane_cpp(fb->pixel_format, 0); drm_format_plane_cpp(format, 0);
else /* uv-plane data rate */ else /* uv-plane data rate */
return (width / 2) * (height / 2) * return (width / 2) * (height / 2) *
drm_format_plane_cpp(fb->pixel_format, 1); drm_format_plane_cpp(format, 1);
} }
/* for packed formats */ /* for packed formats */
return width * height * drm_format_plane_cpp(fb->pixel_format, 0); return width * height * drm_format_plane_cpp(format, 0);
} }
/* /*
...@@ -2967,32 +2975,34 @@ skl_plane_relative_data_rate(const struct intel_crtc_state *cstate, ...@@ -2967,32 +2975,34 @@ skl_plane_relative_data_rate(const struct intel_crtc_state *cstate,
* 3 * 4096 * 8192 * 4 < 2^32 * 3 * 4096 * 8192 * 4 < 2^32
*/ */
static unsigned int static unsigned int
skl_get_total_relative_data_rate(const struct intel_crtc_state *cstate) skl_get_total_relative_data_rate(struct intel_crtc_state *cstate)
{ {
struct intel_crtc *intel_crtc = to_intel_crtc(cstate->base.crtc); struct intel_crtc *intel_crtc = to_intel_crtc(cstate->base.crtc);
struct drm_device *dev = intel_crtc->base.dev; struct drm_device *dev = intel_crtc->base.dev;
const struct intel_plane *intel_plane; const struct intel_plane *intel_plane;
unsigned int total_data_rate = 0; unsigned int rate, total_data_rate = 0;
/* Calculate and cache data rate for each plane */
for_each_intel_plane_on_crtc(dev, intel_crtc, intel_plane) { for_each_intel_plane_on_crtc(dev, intel_crtc, intel_plane) {
const struct drm_plane_state *pstate = intel_plane->base.state; const struct drm_plane_state *pstate = intel_plane->base.state;
int id = skl_wm_plane_id(intel_plane);
if (pstate->fb == NULL) /* packed/uv */
continue; rate = skl_plane_relative_data_rate(cstate, pstate, 0);
cstate->wm.skl.plane_data_rate[id] = rate;
if (intel_plane->base.type == DRM_PLANE_TYPE_CURSOR) /* y-plane */
continue; rate = skl_plane_relative_data_rate(cstate, pstate, 1);
cstate->wm.skl.plane_y_data_rate[id] = rate;
}
/* packed/uv */ /* Calculate CRTC's total data rate from cached values */
total_data_rate += skl_plane_relative_data_rate(cstate, for_each_intel_plane_on_crtc(dev, intel_crtc, intel_plane) {
pstate, int id = skl_wm_plane_id(intel_plane);
0);
if (pstate->fb->pixel_format == DRM_FORMAT_NV12) /* packed/uv */
/* y-plane */ total_data_rate += cstate->wm.skl.plane_data_rate[id];
total_data_rate += skl_plane_relative_data_rate(cstate, total_data_rate += cstate->wm.skl.plane_y_data_rate[id];
pstate,
1);
} }
return total_data_rate; return total_data_rate;
...@@ -3056,6 +3066,8 @@ skl_allocate_pipe_ddb(struct intel_crtc_state *cstate, ...@@ -3056,6 +3066,8 @@ skl_allocate_pipe_ddb(struct intel_crtc_state *cstate,
* FIXME: we may not allocate every single block here. * FIXME: we may not allocate every single block here.
*/ */
total_data_rate = skl_get_total_relative_data_rate(cstate); total_data_rate = skl_get_total_relative_data_rate(cstate);
if (total_data_rate == 0)
return;
start = alloc->start; start = alloc->start;
for_each_intel_plane_on_crtc(dev, intel_crtc, intel_plane) { for_each_intel_plane_on_crtc(dev, intel_crtc, intel_plane) {
...@@ -3070,7 +3082,7 @@ skl_allocate_pipe_ddb(struct intel_crtc_state *cstate, ...@@ -3070,7 +3082,7 @@ skl_allocate_pipe_ddb(struct intel_crtc_state *cstate,
if (plane->type == DRM_PLANE_TYPE_CURSOR) if (plane->type == DRM_PLANE_TYPE_CURSOR)
continue; continue;
data_rate = skl_plane_relative_data_rate(cstate, pstate, 0); data_rate = cstate->wm.skl.plane_data_rate[id];
/* /*
* allocation for (packed formats) or (uv-plane part of planar format): * allocation for (packed formats) or (uv-plane part of planar format):
...@@ -3089,20 +3101,16 @@ skl_allocate_pipe_ddb(struct intel_crtc_state *cstate, ...@@ -3089,20 +3101,16 @@ skl_allocate_pipe_ddb(struct intel_crtc_state *cstate,
/* /*
* allocation for y_plane part of planar format: * allocation for y_plane part of planar format:
*/ */
if (pstate->fb->pixel_format == DRM_FORMAT_NV12) { y_data_rate = cstate->wm.skl.plane_y_data_rate[id];
y_data_rate = skl_plane_relative_data_rate(cstate,
pstate, y_plane_blocks = y_minimum[id];
1); y_plane_blocks += div_u64((uint64_t)alloc_size * y_data_rate,
y_plane_blocks = y_minimum[id]; total_data_rate);
y_plane_blocks += div_u64((uint64_t)alloc_size * y_data_rate,
total_data_rate);
ddb->y_plane[pipe][id].start = start;
ddb->y_plane[pipe][id].end = start + y_plane_blocks;
start += y_plane_blocks;
}
ddb->y_plane[pipe][id].start = start;
ddb->y_plane[pipe][id].end = start + y_plane_blocks;
start += y_plane_blocks;
} }
} }
...@@ -3879,10 +3887,28 @@ void skl_wm_get_hw_state(struct drm_device *dev) ...@@ -3879,10 +3887,28 @@ void skl_wm_get_hw_state(struct drm_device *dev)
struct drm_i915_private *dev_priv = dev->dev_private; struct drm_i915_private *dev_priv = dev->dev_private;
struct skl_ddb_allocation *ddb = &dev_priv->wm.skl_hw.ddb; struct skl_ddb_allocation *ddb = &dev_priv->wm.skl_hw.ddb;
struct drm_crtc *crtc; struct drm_crtc *crtc;
struct intel_crtc *intel_crtc;
skl_ddb_get_hw_state(dev_priv, ddb); skl_ddb_get_hw_state(dev_priv, ddb);
list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) list_for_each_entry(crtc, &dev->mode_config.crtc_list, head)
skl_pipe_wm_get_hw_state(crtc); skl_pipe_wm_get_hw_state(crtc);
/* Calculate plane data rates */
for_each_intel_crtc(dev, intel_crtc) {
struct intel_crtc_state *cstate = intel_crtc->config;
struct intel_plane *intel_plane;
for_each_intel_plane_on_crtc(dev, intel_crtc, intel_plane) {
const struct drm_plane_state *pstate =
intel_plane->base.state;
int id = skl_wm_plane_id(intel_plane);
cstate->wm.skl.plane_data_rate[id] =
skl_plane_relative_data_rate(cstate, pstate, 0);
cstate->wm.skl.plane_y_data_rate[id] =
skl_plane_relative_data_rate(cstate, pstate, 1);
}
}
} }
static void ilk_pipe_wm_get_hw_state(struct drm_crtc *crtc) static void ilk_pipe_wm_get_hw_state(struct drm_crtc *crtc)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment