On 2018-04-25 08:39 AM, Emily Deng wrote: > Signed-off-by: Monk Liu <monk....@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 _______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
