RE: [PATCH] drm/ttm: fix deref of bo->ttm without holding the lock v2

2021-06-08 Thread Huang, Ray
Looks good for me. Reviewed-by: Huang Rui -Original Message- From: Christian König Sent: Monday, June 7, 2021 8:52 PM To: dri-devel@lists.freedesktop.org; Pan, Xinhui ; Das, Nirmoy ; Huang, Ray Cc: Thomas Hellström Subject: Re: [PATCH] drm/ttm: fix deref of bo->ttm without hold

Re: [PATCH] drm/ttm: fix deref of bo->ttm without holding the lock v2

2021-06-07 Thread Christian König
@lists.freedesktop.org ; Pan, Xinhui ; Das, Nirmoy ; Huang, Ray *Cc:* Thomas Hellström *Subject:* Re: [PATCH] drm/ttm: fix deref of bo->ttm without holding the lock v2 Am 28.05.21 um 15:06 schrieb Thomas Hellström: > > On 5/28/21 3:00 PM, Christian König wrote: >> We need t

Re: [PATCH] drm/ttm: fix deref of bo->ttm without holding the lock v2

2021-06-07 Thread Pan, Xinhui
[AMD Official Use Only] Looks good to me. From: Christian König Sent: Monday, June 7, 2021 8:52:21 PM To: dri-devel@lists.freedesktop.org ; Pan, Xinhui ; Das, Nirmoy ; Huang, Ray Cc: Thomas Hellström Subject: Re: [PATCH] drm/ttm: fix deref of bo->ttm with

Re: [PATCH] drm/ttm: fix deref of bo->ttm without holding the lock v2

2021-06-07 Thread Christian König
Am 28.05.21 um 15:06 schrieb Thomas Hellström: On 5/28/21 3:00 PM, Christian König wrote: We need to grab the resv lock first before doing that check. v2 (chk): simplify the change for -fixes Signed-off-by: Christian König Signed-off-by: Thomas Hellström Hmm, OK, but this doesn't fix t

Re: [PATCH] drm/ttm: fix deref of bo->ttm without holding the lock v2

2021-05-28 Thread Thomas Hellström
On 5/28/21 3:00 PM, Christian König wrote: We need to grab the resv lock first before doing that check. v2 (chk): simplify the change for -fixes Signed-off-by: Christian König Signed-off-by: Thomas Hellström Hmm, OK, but this doesn't fix the swapped-bo-not-on-lru and unpopulating from sw

[PATCH] drm/ttm: fix deref of bo->ttm without holding the lock v2

2021-05-28 Thread Christian König
We need to grab the resv lock first before doing that check. v2 (chk): simplify the change for -fixes Signed-off-by: Christian König Signed-off-by: Thomas Hellström --- drivers/gpu/drm/ttm/ttm_bo.c | 5 - drivers/gpu/drm/ttm/ttm_device.c | 8 +--- 2 files changed, 5 insertions(+),