[Intel-gfx] [RFC] drm/i915/chv: Clip cursor for CHV pipe C HW Cursor pos < 0
Shobhit Kumar
shobhit.kumar at intel.com
Tue Jun 28 12:32:03 UTC 2016
Daniel,
On 06/28/2016 05:57 PM, Shobhit Kumar wrote:
> From: Shobhit Kumar <shobhit.kumar at intel.com>
>
> CHV pipe C hits underrun when we get negative crtc_x values of cursor.
> To avoid this we clip and shift the cursor image by negative crtc_x
> value.
>
> v2: Make a copy of cursor plane state and allocate new gem object and fb
> for clipped cursor and use that in case of negative cursor position
>
This patch is not yet tested and most likely not even complete and is
sent for early review of the approach. Will test it and update the
status here.
Regards
Shobhit
> Signed-off-by: Akshu Agrawal <akshu.agrawal at intel.com>
> Signed-off-by: Shobhit Kumar <shobhit.kumar at intel.com>
> ---
> drivers/gpu/drm/i915/i915_drv.h | 7 ++
> drivers/gpu/drm/i915/intel_display.c | 122 ++++++++++++++++++++++++++++++++++-
> 2 files changed, 128 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
> index 724d34b..1e59c02 100644
> --- a/drivers/gpu/drm/i915/i915_drv.h
> +++ b/drivers/gpu/drm/i915/i915_drv.h
> @@ -2041,6 +2041,13 @@ struct drm_i915_private {
> struct intel_encoder *dig_port_map[I915_MAX_PORTS];
>
> /*
> + * Temporary copy of cursor plane state for CHV PIPE_C
> + * Will be initialized only when crtc_x < 0 as there is a
> + * HW bug causing pipe underrun
> + */
> + struct intel_plane_state *cursor_state;
> +
> + /*
> * NOTE: This is the dri1/ums dungeon, don't add stuff here. Your patch
> * will be rejected. Instead look for a better place.
> */
> diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
> index c3b5dc8..c4988d5 100644
> --- a/drivers/gpu/drm/i915/intel_display.c
> +++ b/drivers/gpu/drm/i915/intel_display.c
> @@ -14456,6 +14456,123 @@ intel_update_cursor_plane(struct drm_plane *plane,
> intel_crtc_update_cursor(crtc, state);
> }
>
> +static void
> +intel_update_chv_pipe_c_cursor_plane(struct drm_plane *plane,
> + const struct intel_crtc_state *crtc_state,
> + const struct intel_plane_state *state)
> +{
> + struct drm_crtc *crtc = crtc_state->base.crtc;
> + struct intel_crtc *intel_crtc = to_intel_crtc(crtc);
> + struct drm_device *dev = plane->dev;
> + struct drm_i915_private *dev_priv = dev->dev_private;
> + struct drm_i915_gem_object *obj = intel_fb_obj(state->base.fb);
> + struct drm_i915_gem_object *cur_obj = NULL, *use_obj = NULL;
> + uint32_t addr;
> + struct intel_plane_state *cursor_state = dev_priv->cursor_state;
> + const struct intel_plane_state *use_state;
> + char __iomem *src, *dst;
> +
> + if (state->visible && state->base.crtc_x < 0) {
> + int bytes_per_pixel = state->base.fb->bits_per_pixel / 8;
> + int x = state->base.crtc_x;
> + int width = state->base.crtc_w;
> + int height = state->base.crtc_h;
> + struct drm_mode_fb_cmd2 mode_cmd = { 0 };
> + int i;
> +
> + if (!cursor_state) {
> + cursor_state = kzalloc(sizeof(*cursor_state), GFP_KERNEL);
> + if (!cursor_state) {
> + use_state = state;
> + use_obj = obj;
> + goto update;
> + }
> +
> + memcpy(cursor_state, state, sizeof(*state));
> + cursor_state->base.crtc_x = 0;
> +
> + /* Allocate new gem object */
> + cur_obj = i915_gem_object_create(dev, obj->base.size);
> + if (IS_ERR(cur_obj)) {
> + kfree(cursor_state);
> + use_state = state;
> + use_obj = obj;
> + goto update;
> + }
> +
> + mode_cmd.width = cursor_state->base.fb->width;
> + mode_cmd.height = cursor_state->base.fb->height;
> + mode_cmd.pitches[0] = cursor_state->base.fb->pitches[0];
> + mode_cmd.pixel_format = cursor_state->base.fb->pixel_format;
> +
> + cursor_state->base.fb = intel_framebuffer_create(dev, &mode_cmd, cur_obj);
> + if (IS_ERR(cursor_state->base.fb)) {
> + drm_gem_object_unreference_unlocked(&cur_obj->base);
> + kfree(cursor_state);
> + use_state = state;
> + use_obj = obj;
> + goto update;
> + }
> +
> + dev_priv->cursor_state = cursor_state;
> + } else
> + cur_obj = intel_fb_obj(cursor_state->base.fb);
> +
> + src = ioremap_wc(dev_priv->ggtt.mappable_base +
> + i915_gem_obj_ggtt_offset(obj),
> + obj->base.size);
> +
> + dst = ioremap_wc(dev_priv->ggtt.mappable_base +
> + i915_gem_obj_ggtt_offset(cur_obj),
> + cur_obj->base.size);
> +
> + /* shift the original cusrsor in to copy buffer offsetting -ive pos */
> + x = -x;
> + for (i = 0; i < height; i++) {
> + src += x * bytes_per_pixel;
> + memcpy(dst, src, (width - x) * bytes_per_pixel);
> + dst += (width - x) * bytes_per_pixel;
> + memset(dst, 0, x * bytes_per_pixel);
> + dst += x * bytes_per_pixel;
> + src += (width -x) * bytes_per_pixel;
> + }
> +
> + iounmap(src);
> + iounmap(dst);
> +
> + use_obj = cur_obj;
> + use_state = cursor_state;
> + } else {
> + /* free the cursor_state if cached */
> + if (cursor_state) {
> + struct intel_framebuffer *intel_fb = to_intel_framebuffer(cursor_state->base.fb);
> + drm_framebuffer_cleanup(cursor_state->base.fb);
> + mutex_lock(&dev->struct_mutex);
> + drm_gem_object_unreference(&intel_fb->obj->base);
> + mutex_unlock(&dev->struct_mutex);
> + kfree(intel_fb);
> +
> + kfree(cursor_state);
> + cursor_state = NULL;
> + }
> +
> + use_obj = obj;
> + use_state = state;
> + }
> +
> +update:
> + if (!use_obj)
> + addr = 0;
> + else if (!INTEL_INFO(dev)->cursor_needs_physical)
> + addr = i915_gem_obj_ggtt_offset(use_obj);
> + else
> + addr = use_obj->phys_handle->busaddr;
> +
> + intel_crtc->cursor_addr = addr;
> +
> + intel_crtc_update_cursor(crtc, use_state);
> +}
> +
> static struct drm_plane *intel_cursor_plane_create(struct drm_device *dev,
> int pipe)
> {
> @@ -14478,7 +14595,10 @@ static struct drm_plane *intel_cursor_plane_create(struct drm_device *dev,
> cursor->plane = pipe;
> cursor->frontbuffer_bit = INTEL_FRONTBUFFER_CURSOR(pipe);
> cursor->check_plane = intel_check_cursor_plane;
> - cursor->update_plane = intel_update_cursor_plane;
> + if (IS_CHERRYVIEW(dev) && pipe == PIPE_C)
> + cursor->update_plane = intel_update_chv_pipe_c_cursor_plane;
> + else
> + cursor->update_plane = intel_update_cursor_plane;
> cursor->disable_plane = intel_disable_cursor_plane;
>
> ret = drm_universal_plane_init(dev, &cursor->base, 0,
>
More information about the Intel-gfx
mailing list