[PATCH] drm/amdgpu: cleanup amdgpu_cs_ioctl to make code logicality clear

ray.huang@xxxxxxx (Huang Rui) · Mon, 31 Oct 2016 16:58:07 +0800



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 <ray.huang at amd.com>
> 
> Looks good to me, patch is Reviewed-by: Christian KÃ¶nig 
> <christian.koenig at amd.com>.
> 
> 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?

Thanks,
Rui