drm/i915: Deal with video overlay on GPU reset
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Wed, 26 Nov 2014 15:07:29 +0000 (17:07 +0200)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Wed, 3 Dec 2014 08:35:24 +0000 (09:35 +0100)
Clear the video overlay state on GPU reset. Any pending overlay request
in the ring has been nuked, and the display itself gets reset. So we
pretty much lose all state here. Adjust the software state to match so
that the next "putimage" will restore things to working order.

v2: Ass a locking check into intel_overlay_release_old_vid() (Daniel)

Cc: Daniel Vetter <daniel@ffwll.ch>
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
[danvet: s/0/NULL/ to appease sparse, reported by 0-day tester.]
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/i915_drv.c
drivers/gpu/drm/i915/intel_drv.h
drivers/gpu/drm/i915/intel_overlay.c

index 1e9c136a874cf1741e09aecc33c6cea37e26e1d4..71be3c930a0ee458bce4f784a72fe58d36a80665 100644 (file)
@@ -838,6 +838,8 @@ int i915_reset(struct drm_device *dev)
                return ret;
        }
 
+       intel_overlay_reset(dev_priv);
+
        /* Ok, now get things going again... */
 
        /*
index 1a9c70fed91818fdbc87417231718d15af678dfc..abb2cf4162b2026be5ae6a842c824ba869fc1059 100644 (file)
@@ -1081,6 +1081,7 @@ int intel_overlay_put_image(struct drm_device *dev, void *data,
                            struct drm_file *file_priv);
 int intel_overlay_attrs(struct drm_device *dev, void *data,
                        struct drm_file *file_priv);
+void intel_overlay_reset(struct drm_i915_private *dev_priv);
 
 
 /* intel_panel.c */
index c1abf49ea5b85e1a136c5494b43bea55636eaa33..973c9de3b87da4c129a4d99602a8308902067f04 100644 (file)
@@ -394,6 +394,8 @@ static int intel_overlay_release_old_vid(struct intel_overlay *overlay)
        struct intel_engine_cs *ring = &dev_priv->ring[RCS];
        int ret;
 
+       WARN_ON(!mutex_is_locked(&dev->struct_mutex));
+
        /* Only wait if there is actually an old frame to release to
         * guarantee forward progress.
         */
@@ -424,6 +426,22 @@ static int intel_overlay_release_old_vid(struct intel_overlay *overlay)
        return 0;
 }
 
+void intel_overlay_reset(struct drm_i915_private *dev_priv)
+{
+       struct intel_overlay *overlay = dev_priv->overlay;
+
+       if (!overlay)
+               return;
+
+       intel_overlay_release_old_vid(overlay);
+
+       overlay->last_flip_req = NULL;
+       overlay->old_xscale = 0;
+       overlay->old_yscale = 0;
+       overlay->crtc = NULL;
+       overlay->active = false;
+}
+
 struct put_image_params {
        int format;
        short dst_x;