[Acked] [PATCH artful] qxl: fix framebuffer unpinning

Andy Whitcroft apw at canonical.com
Wed Oct 11 17:18:42 UTC 2017


On Wed, Oct 11, 2017 at 12:12:01PM -0500, Seth Forshee wrote:
> From: Gerd Hoffmann <kraxel at redhat.com>
> 
> BugLink: http://bugs.launchpad.net/bugs/1711358
> 
> qxl_plane_cleanup_fb() unpins the just activated framebuffer
> instead of the old one.  Oops.  Fix it.
> 
> Cc: Gabriel Krisman Bertazi <krisman at collabora.co.uk>
> Fixes: 1277eed5fecb8830c8cc414ad70c1ef640464bc0
> Signed-off-by: Gerd Hoffmann <kraxel at redhat.com>
> Reviewed-by: Gabriel Krisman Bertazi <krisman at collabora.co.uk>
> Link: http://patchwork.freedesktop.org/patch/msgid/20170918074145.2257-1-kraxel@redhat.com
> (cherry picked from commit 5f3d862a736398e7068fa67142133f1713fdee8c)
> Signed-off-by: Seth Forshee <seth.forshee at canonical.com>
> ---
>  drivers/gpu/drm/qxl/qxl_display.c | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/qxl/qxl_display.c b/drivers/gpu/drm/qxl/qxl_display.c
> index 03fe182203ce..274df74a128c 100644
> --- a/drivers/gpu/drm/qxl/qxl_display.c
> +++ b/drivers/gpu/drm/qxl/qxl_display.c
> @@ -701,14 +701,15 @@ static void qxl_plane_cleanup_fb(struct drm_plane *plane,
>  	struct drm_gem_object *obj;
>  	struct qxl_bo *user_bo;
>  
> -	if (!plane->state->fb) {
> -		/* we never executed prepare_fb, so there's nothing to
> +	if (!old_state->fb) {
> +		/*
> +		 * we never executed prepare_fb, so there's nothing to
>  		 * unpin.
>  		 */
>  		return;
>  	}
>  
> -	obj = to_qxl_framebuffer(plane->state->fb)->obj;
> +	obj = to_qxl_framebuffer(old_state->fb)->obj;
>  	user_bo = gem_to_qxl_bo(obj);
>  	qxl_bo_unpin(user_bo);
>  }

Looks to do what is claimed.  Specific to the driver and tested.

Acked-by: Andy Whitcroft <apw at canonical.com>

-apw




More information about the kernel-team mailing list