drm/atomic: clear plane's CRTC and FB when shutting down
authorRob Clark <robdclark@gmail.com>
Wed, 26 Nov 2014 23:58:04 +0000 (18:58 -0500)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Thu, 27 Nov 2014 14:39:11 +0000 (15:39 +0100)
Otherwise we'd still end up w/ the plane attached to the CRTC, and
seemingly active, but without an FB.  Which ends up going *boom*
in the drivers.

Slightly modified version of Daniel's irc suggestion.

Note that the big problem isn't drivers going *boom* here (since we
already have the situation of planes being left enabled when the crtc
goes down). The real issue is that the core assumes the primary plane
always goes down when calling ->set_config with a NULL mode. Ignoring
that assumption leads to the legacy state pointers plane->fb/crtc
getting out of sync with atomic, and that then leads to the subsequent
*boom* all over the place.

CC: Daniel Vetter <daniel@ffwll.ch>
Signed-off-by: Rob Clark <robdclark@gmail.com>
[danvet: Drop my opinion of what's going sidewides here into the
commit message as a note.]
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/drm_atomic_helper.c

index 2ee509c92034d278904a5396451b4fc264441bd3..4a78a773151ce71f464d36cc4d1d6d887eea1619 100644 (file)
@@ -1452,11 +1452,24 @@ retry:
                goto fail;
        }
 
+       primary_state = drm_atomic_get_plane_state(state, crtc->primary);
+       if (IS_ERR(primary_state)) {
+               ret = PTR_ERR(primary_state);
+               goto fail;
+       }
+
        if (!set->mode) {
                WARN_ON(set->fb);
                WARN_ON(set->num_connectors);
 
                crtc_state->enable = false;
+
+               ret = drm_atomic_set_crtc_for_plane(state, crtc->primary, NULL);
+               if (ret != 0)
+                       goto fail;
+
+               drm_atomic_set_fb_for_plane(primary_state, NULL);
+
                goto commit;
        }
 
@@ -1466,12 +1479,6 @@ retry:
        crtc_state->enable = true;
        drm_mode_copy(&crtc_state->mode, set->mode);
 
-       primary_state = drm_atomic_get_plane_state(state, crtc->primary);
-       if (IS_ERR(primary_state)) {
-               ret = PTR_ERR(primary_state);
-               goto fail;
-       }
-
        ret = drm_atomic_set_crtc_for_plane(state, crtc->primary, crtc);
        if (ret != 0)
                goto fail;