[PATCH 1/3] drm/radeon: take exclusive_lock in read mode during ring tests

2014-08-18 Thread Maarten Lankhorst
Hey, On 18-08-14 18:10, Alex Deucher wrote: > On Mon, Aug 18, 2014 at 12:07 PM, Christian K?nig > wrote: >>> Yeah, looks like a bug. I think the attached patch should fix it. >> >> Sounds logical and the patch is Reviewed-by: Christian K?nig >> >> >> Going to apply Maartens patch on top and tes

[PATCH 1/3] drm/radeon: take exclusive_lock in read mode during ring tests

2014-08-18 Thread Christian König
> Yeah, looks like a bug. I think the attached patch should fix it. Sounds logical and the patch is Reviewed-by: Christian K?nig Going to apply Maartens patch on top and test that one a bit to make sure it works as expected. Regards, Christian. Am 18.08.2014 um 18:03 schrieb Alex Deucher: > O

[PATCH 1/3] drm/radeon: take exclusive_lock in read mode during ring tests, v2

2014-08-18 Thread Maarten Lankhorst
This is needed for the next commit, because the lockup detection will need the read lock to run. Signed-off-by: Maarten Lankhorst --- Changes since v1: Changed order of resuming a bit according to ckoenig's feedback. Reset handling seems just as unreliable as before this commit, but at least it

[PATCH 1/3] drm/radeon: take exclusive_lock in read mode during ring tests

2014-08-18 Thread Christian König
Am 18.08.2014 um 16:45 schrieb Maarten Lankhorst: > This is needed for the next commit, because the lockup detection > will need the read lock to run. > > Signed-off-by: Maarten Lankhorst > --- > drivers/gpu/drm/radeon/radeon.h| 2 +- > drivers/gpu/drm/radeon/radeon_device.c | 61 > ++

[PATCH 1/3] drm/radeon: take exclusive_lock in read mode during ring tests

2014-08-18 Thread Alex Deucher
On Mon, Aug 18, 2014 at 4:02 PM, Maarten Lankhorst wrote: > Hey, > > On 18-08-14 18:10, Alex Deucher wrote: >> On Mon, Aug 18, 2014 at 12:07 PM, Christian K?nig >> wrote: Yeah, looks like a bug. I think the attached patch should fix it. >>> >>> Sounds logical and the patch is Reviewed-by: C

[PATCH 1/3] drm/radeon: take exclusive_lock in read mode during ring tests

2014-08-18 Thread Maarten Lankhorst
This is needed for the next commit, because the lockup detection will need the read lock to run. Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/radeon/radeon.h| 2 +- drivers/gpu/drm/radeon/radeon_device.c | 61 -- 2 files changed, 37 insertions(+),

[PATCH 1/3] drm/radeon: take exclusive_lock in read mode during ring tests

2014-08-18 Thread Alex Deucher
On Mon, Aug 18, 2014 at 12:07 PM, Christian K?nig wrote: >> Yeah, looks like a bug. I think the attached patch should fix it. > > Sounds logical and the patch is Reviewed-by: Christian K?nig > > > Going to apply Maartens patch on top and test that one a bit to make sure it > works as expected.

[PATCH 1/3] drm/radeon: take exclusive_lock in read mode during ring tests

2014-08-18 Thread Alex Deucher
On Mon, Aug 18, 2014 at 11:02 AM, Christian K?nig wrote: > Am 18.08.2014 um 16:45 schrieb Maarten Lankhorst: > >> This is needed for the next commit, because the lockup detection >> will need the read lock to run. >> >> Signed-off-by: Maarten Lankhorst >> --- >> drivers/gpu/drm/radeon/radeon.h