On 2018-04-25 08:39 AM, Emily Deng wrote: > Signed-off-by: Monk Liu <Monk.Liu at amd.com> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c > index 6d55cae..adeca71 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c > @@ -847,6 +847,9 @@ int amdgpu_driver_open_kms(struct drm_device *dev, struct drm_file *file_priv) > struct amdgpu_fpriv *fpriv; > int r, pasid; > > + if (adev->in_gpu_reset) > + return -ECANCELED; This seems like a bad idea, as it would cause basically any userspace which wants to use the GPU to fail to start during a GPU reset. -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer
