diff options
| -rw-r--r-- | drivers/gpu/drm/drm_atomic_helper.c | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/drivers/gpu/drm/drm_atomic_helper.c b/drivers/gpu/drm/drm_atomic_helper.c index 4368581ac1eb..d981d07d50cd 100644 --- a/drivers/gpu/drm/drm_atomic_helper.c +++ b/drivers/gpu/drm/drm_atomic_helper.c | |||
| @@ -1275,6 +1275,17 @@ int drm_atomic_helper_disable_plane(struct drm_plane *plane) | |||
| 1275 | struct drm_plane_state *plane_state; | 1275 | struct drm_plane_state *plane_state; |
| 1276 | int ret = 0; | 1276 | int ret = 0; |
| 1277 | 1277 | ||
| 1278 | /* | ||
| 1279 | * FIXME: Without plane->crtc set we can't get at the implicit legacy | ||
| 1280 | * acquire context. The real fix will be to wire the acquire ctx through | ||
| 1281 | * everywhere we need it, but meanwhile prevent chaos by just skipping | ||
| 1282 | * this noop. The critical case is the cursor ioctls which a) only grab | ||
| 1283 | * crtc/cursor-plane locks (so we need the crtc to get at the right | ||
| 1284 | * acquire context) and b) can try to disable the plane multiple times. | ||
| 1285 | */ | ||
| 1286 | if (!plane->crtc) | ||
| 1287 | return 0; | ||
| 1288 | |||
| 1278 | state = drm_atomic_state_alloc(plane->dev); | 1289 | state = drm_atomic_state_alloc(plane->dev); |
| 1279 | if (!state) | 1290 | if (!state) |
| 1280 | return -ENOMEM; | 1291 | return -ENOMEM; |
