Am 31.10.2016 um 09:58 schrieb Huang Rui:
On Mon, Oct 31, 2016 at 03:53:52PM +0800, Christian König wrote:
Am 31.10.2016 um 03:08 schrieb Huang Rui:
Signed-off-by: Huang Rui <[email protected]>
Looks good to me, patch is Reviewed-by: Christian König
<[email protected]>.

BTW: amdgpu_cs_handle_lockup() is a leftover from radeon and can be
removed. Would be nice if you can take care of this as well.

Sure. amdgpu_cs_handle_lockup actually will trigger a gpu reset when r ==
-EDEADLK. I'm not very clear with why we needn't gpu reset on that case
yet, could you explain?

On radeon the fence wait function returned -EDEADLK when it detected that the hardware is locked up.

In the CS and GEM IOCTLs we then catch this error, reseted the hardware and tried again.

Since we never return -EDEADLK any more we can just drop the handling for this.

Regards,
Christian.


Thanks,
Rui


_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to