[PATCH] drm/radeon: Avoid double gpu reset by adding a timeout on IB ring tests.

2016-02-08 Thread Bruno Kant
Le 2016-02-08 15:51, Matthew Dawson a écrit : >> Yeah, 1s works as well. But that 100ms shouldn't be enough sounds like >> another bug to me. > Probably, but it has a tendency to crash my entire system, so I'm not > sure how > to debug it further. Netconsole maybe? I probably won't be able to

[PATCH] drm/radeon: Avoid double gpu reset by adding a timeout on IB ring tests.

2016-02-08 Thread Alex Deucher
On Mon, Feb 8, 2016 at 4:38 AM, Christian König wrote: > Am 07.02.2016 um 22:51 schrieb Matthew Dawson: >> >> When the radeon driver resets a gpu, it attempts to test whether all the >> rings can successfully handle an IB. If these rings fail to respond, the >> process will wait forever. Anoth

[PATCH] drm/radeon: Avoid double gpu reset by adding a timeout on IB ring tests.

2016-02-08 Thread Christian König
Am 07.02.2016 um 22:51 schrieb Matthew Dawson: > When the radeon driver resets a gpu, it attempts to test whether all the > rings can successfully handle an IB. If these rings fail to respond, the > process will wait forever. Another gpu reset can't happen at this point, > as the current reset ho

[PATCH] drm/radeon: Avoid double gpu reset by adding a timeout on IB ring tests.

2016-02-08 Thread Matthew Dawson
On Wednesday, February 3, 2016 1:24:49 PM EST Christian König wrote: > Am 31.01.2016 um 19:50 schrieb Matthew Dawson: > > On Sunday, January 24, 2016 10:49:00 AM EST Christian König wrote: > >> Am 24.01.2016 um 07:18 schrieb Matthew Dawson: > >>> When the radeon driver resets a gpu, it attempts t

[PATCH] drm/radeon: Avoid double gpu reset by adding a timeout on IB ring tests.

2016-02-07 Thread Matthew Dawson
When the radeon driver resets a gpu, it attempts to test whether all the rings can successfully handle an IB. If these rings fail to respond, the process will wait forever. Another gpu reset can't happen at this point, as the current reset holds a lock required to do so. Instead, make all the IB

[PATCH] drm/radeon: Avoid double gpu reset by adding a timeout on IB ring tests.

2016-02-03 Thread Christian König
Am 31.01.2016 um 19:50 schrieb Matthew Dawson: > On Sunday, January 24, 2016 10:49:00 AM EST Christian König wrote: >> Am 24.01.2016 um 07:18 schrieb Matthew Dawson: >>> When the radeon driver resets a gpu, it attempts to test whether all the >>> rings can successfully handle an IB. If these ring

[PATCH] drm/radeon: Avoid double gpu reset by adding a timeout on IB ring tests.

2016-01-31 Thread Matthew Dawson
On Sunday, January 24, 2016 10:49:00 AM EST Christian König wrote: > Am 24.01.2016 um 07:18 schrieb Matthew Dawson: > > When the radeon driver resets a gpu, it attempts to test whether all the > > rings can successfully handle an IB. If these rings fail to respond, the > > process will wait forev

[PATCH] drm/radeon: Avoid double gpu reset by adding a timeout on IB ring tests.

2016-01-24 Thread Christian König
Am 24.01.2016 um 07:18 schrieb Matthew Dawson: > When the radeon driver resets a gpu, it attempts to test whether all the > rings can successfully handle an IB. If these rings fail to respond, the > process will wait forever. Another gpu reset can't happen at this point, > as the current reset ho

[PATCH] drm/radeon: Avoid double gpu reset by adding a timeout on IB ring tests.

2016-01-24 Thread Matthew Dawson
When the radeon driver resets a gpu, it attempts to test whether all the rings can successfully handle an IB. If these rings fail to respond, the process will wait forever. Another gpu reset can't happen at this point, as the current reset holds a lock required to do so. Instead, make all the IB