Am 13.12.2017 um 03:06 schrieb He, Roger:
That is a bug fix, isn't it? If yes maybe add CC:stable and commit it first before all other patches.
Fortunately so far there is no issue directly resulted from that.
Yeah, but that is irrelevant. Patches are classified as fix if they fix
something, not if the bug was ever hit.
Not sure if the code is already upstream, but we should still make sure
that we send this patch to the appropriate places.
Christian.
Thanks
Roger(Hongbo.He)
-----Original Message-----
From: Christian König [mailto:ckoenig.leichtzumerken@xxxxxxxxx]
Sent: Tuesday, December 12, 2017 6:37 PM
To: He, Roger <Hongbo.He@xxxxxxx>; amd-gfx@xxxxxxxxxxxxxxxxxxxxx; dri-devel@xxxxxxxxxxxxxxxxxxxxx
Subject: Re: [PATCH 4/6] drm/ttm: init locked again to prevent incorrect unlock
Am 12.12.2017 um 10:33 schrieb Roger He:
Change-Id: I8db51d843955f5db14bb4bbff892eaedbd9f0abe
Signed-off-by: Roger He <Hongbo.He@xxxxxxx>
Reviewed-by: Christian König <christian.koenig@xxxxxxx>
That is a bug fix, isn't it? If yes maybe add CC:stable and commit it first before all other patches.
Christian.
---
drivers/gpu/drm/ttm/ttm_bo.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/ttm/ttm_bo.c
b/drivers/gpu/drm/ttm/ttm_bo.c index 17fe8be..eb8c568 100644
--- a/drivers/gpu/drm/ttm/ttm_bo.c
+++ b/drivers/gpu/drm/ttm/ttm_bo.c
@@ -735,6 +735,7 @@ static int ttm_mem_evict_first(struct ttm_bo_device *bdev,
place)) {
if (locked)
reservation_object_unlock(bo->resv);
+ locked = false;
continue;
}
break;
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel
