Attachment 124708 Details for Bug 96588 – Additional fix.

[patch] Additional fix.

0001-drm-ttm-fix-stupid-parameter-inversion-in-the-pipeli.patch (text/plain), 1.04 KB, created by Christian König on 2016-06-24 19:50:08 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Christian König

Created: 2016-06-24 19:50:08 UTC

Size: 1.04 KB

patch

obsolete

>From b5396c0c8b8342eb88905112960311f12e098084 Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Christian=20K=C3=B6nig?= <christian.koenig@amd.com>
>Date: Fri, 24 Jun 2016 20:46:52 +0200
>Subject: [PATCH 1/2] drm/ttm: fix stupid parameter inversion in the pipeline
> code
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>We want to keep the newest fence, not the oldest one.
>
>Signed-off-by: Christian KÃ¶nig <christian.koenig@amd.com>
>---
> drivers/gpu/drm/ttm/ttm_bo_util.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/drivers/gpu/drm/ttm/ttm_bo_util.c b/drivers/gpu/drm/ttm/ttm_bo_util.c
>index 0c389a5..4da0e78 100644
>--- a/drivers/gpu/drm/ttm/ttm_bo_util.c
>+++ b/drivers/gpu/drm/ttm/ttm_bo_util.c
>@@ -753,7 +753,7 @@ int ttm_bo_pipeline_move(struct ttm_buffer_object *bo,
> 		 */
> 
> 		spin_lock(&from->move_lock);
>-		if (!from->move || fence_is_later(from->move, fence)) {
>+		if (!from->move || fence_is_later(fence, from->move)) {
> 			fence_put(from->move);
> 			from->move = fence_get(fence);
> 		}
>-- 
>2.5.0
>

Actions: View | Splinter Review

Attachments on bug 96588: 124602 | 124607 | 124616 | 124623 | 124672 | 124708

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.