diff options
| author | Joonyoung Shim <jy0922.shim@samsung.com> | 2012-04-18 00:47:02 -0400 |
|---|---|---|
| committer | Dave Airlie <airlied@redhat.com> | 2012-04-19 09:36:51 -0400 |
| commit | aef6a7eeac6fd867eda415f65a7f0247fb82baf4 (patch) | |
| tree | af52dcf49550f4791e02296e49af1f0c6ca15536 /drivers/gpu | |
| parent | d6b8395262a9d90e4ab11b614a5fb8f858e2b3df (diff) | |
drm: fix page_flip error handling
Free event and restore event_space only when page_flip->flags has
DRM_MODE_PAGE_FLIP_EVENT if page_flip() is failed.
Signed-off-by: Joonyoung Shim <jy0922.shim@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
Reviewed-by: Michel Dänzer <michel.daenzer@amd.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
Diffstat (limited to 'drivers/gpu')
| -rw-r--r-- | drivers/gpu/drm/drm_crtc.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c index d3aaeb6ae236..c79870a75c2f 100644 --- a/drivers/gpu/drm/drm_crtc.c +++ b/drivers/gpu/drm/drm_crtc.c | |||
| @@ -3335,10 +3335,12 @@ int drm_mode_page_flip_ioctl(struct drm_device *dev, | |||
| 3335 | 3335 | ||
| 3336 | ret = crtc->funcs->page_flip(crtc, fb, e); | 3336 | ret = crtc->funcs->page_flip(crtc, fb, e); |
| 3337 | if (ret) { | 3337 | if (ret) { |
| 3338 | spin_lock_irqsave(&dev->event_lock, flags); | 3338 | if (page_flip->flags & DRM_MODE_PAGE_FLIP_EVENT) { |
| 3339 | file_priv->event_space += sizeof e->event; | 3339 | spin_lock_irqsave(&dev->event_lock, flags); |
| 3340 | spin_unlock_irqrestore(&dev->event_lock, flags); | 3340 | file_priv->event_space += sizeof e->event; |
| 3341 | kfree(e); | 3341 | spin_unlock_irqrestore(&dev->event_lock, flags); |
| 3342 | kfree(e); | ||
| 3343 | } | ||
| 3342 | } | 3344 | } |
| 3343 | 3345 | ||
| 3344 | out: | 3346 | out: |
