All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] drm/amdgpu/sriov: reject kms open if TDR not finished or failed
@ 2018-04-25  6:39 Emily Deng
       [not found] ` <1524638384-31977-1-git-send-email-Emily.Deng-5C7GfCeVMHo@public.gmane.org>
  0 siblings, 1 reply; 12+ messages in thread
From: Emily Deng @ 2018-04-25  6:39 UTC (permalink / raw)
  To: amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW; +Cc: Emily Deng, Monk Liu

Signed-off-by: Monk Liu <Monk.Liu@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;
+
 	file_priv->driver_priv = NULL;
 
 	r = pm_runtime_get_sync(dev->dev);
-- 
2.7.4

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [PATCH] drm/amdgpu/sriov: reject kms open if TDR not finished or failed
       [not found] ` <1524638384-31977-1-git-send-email-Emily.Deng-5C7GfCeVMHo@public.gmane.org>
@ 2018-04-25  7:47   ` Michel Dänzer
       [not found]     ` <5bede6f6-24b2-8a3a-d3d8-9a37ac6d13c8-otUistvHUpPR7s880joybQ@public.gmane.org>
  2018-04-26  7:39   ` Liu, Monk
  1 sibling, 1 reply; 12+ messages in thread
From: Michel Dänzer @ 2018-04-25  7:47 UTC (permalink / raw)
  To: Emily Deng; +Cc: Monk Liu, amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW

On 2018-04-25 08:39 AM, Emily Deng wrote:
> Signed-off-by: Monk Liu <Monk.Liu@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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* RE: [PATCH] drm/amdgpu/sriov: reject kms open if TDR not finished or failed
       [not found]     ` <5bede6f6-24b2-8a3a-d3d8-9a37ac6d13c8-otUistvHUpPR7s880joybQ@public.gmane.org>
@ 2018-04-26  2:03       ` Deng, Emily
       [not found]         ` <CY4PR12MB11254FFDBB4FCCF91CBDE9758F8E0-rpdhrqHFk07v2MZdTKcfDgdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
  0 siblings, 1 reply; 12+ messages in thread
From: Deng, Emily @ 2018-04-26  2:03 UTC (permalink / raw)
  To: Michel Dänzer, Liu, Monk; +Cc: amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW

Hi Michel,
     > 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.
[Emily] Yes, this is what the change want to do, when driver is doing gpu recover or hardware is doing reset,
it doesn't want to be interrupted, and during the reset any driver open kms is meaningless. This behavior could
let the gpu recover and hardware reset back more quickly. After finished the reset, we will set adev->in_gpu_reset=0
again, then could allow the application open the kms.

Best Wishes,
Emily Deng

> -----Original Message-----
> From: Michel Dänzer [mailto:michel@daenzer.net]
> Sent: Wednesday, April 25, 2018 3:48 PM
> To: Deng, Emily <Emily.Deng@amd.com>
> Cc: amd-gfx@lists.freedesktop.org; Liu, Monk <Monk.Liu@amd.com>
> Subject: Re: [PATCH] drm/amdgpu/sriov: reject kms open if TDR not finished
> or failed
> 
> On 2018-04-25 08:39 AM, Emily Deng wrote:
> > Signed-off-by: Monk Liu <Monk.Liu@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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] drm/amdgpu/sriov: reject kms open if TDR not finished or failed
       [not found]         ` <CY4PR12MB11254FFDBB4FCCF91CBDE9758F8E0-rpdhrqHFk07v2MZdTKcfDgdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
@ 2018-04-26  7:33           ` Michel Dänzer
       [not found]             ` <e7756d73-aa65-b351-2b1a-81ceea1e013f-otUistvHUpPR7s880joybQ@public.gmane.org>
  0 siblings, 1 reply; 12+ messages in thread
From: Michel Dänzer @ 2018-04-26  7:33 UTC (permalink / raw)
  To: Deng, Emily, Liu, Monk; +Cc: amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW

On 2018-04-26 04:03 AM, Deng, Emily wrote:
> Hi Michel,
>> 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.
> [Emily] Yes, this is what the change want to do, when driver is doing
> gpu recover or hardware is doing reset, it doesn't want to be
> interrupted, and during the reset any driver open kms is meaningless.

Applications randomly failing to start up during a GPU reset would be
surprising and confusing for the user. The driver needs to handle this
transparently.


-- 
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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* RE: [PATCH] drm/amdgpu/sriov: reject kms open if TDR not finished or failed
       [not found] ` <1524638384-31977-1-git-send-email-Emily.Deng-5C7GfCeVMHo@public.gmane.org>
  2018-04-25  7:47   ` Michel Dänzer
@ 2018-04-26  7:39   ` Liu, Monk
  1 sibling, 0 replies; 12+ messages in thread
From: Liu, Monk @ 2018-04-26  7:39 UTC (permalink / raw)
  To: amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW; +Cc: Deng, Emily

Reviewed-by: Monk

-----Original Message-----
From: amd-gfx [mailto:amd-gfx-bounces@lists.freedesktop.org] On Behalf Of Emily Deng
Sent: 2018年4月25日 14:40
To: amd-gfx@lists.freedesktop.org
Cc: Deng, Emily <Emily.Deng@amd.com>; Liu, Monk <Monk.Liu@amd.com>
Subject: [PATCH] drm/amdgpu/sriov: reject kms open if TDR not finished or failed

Signed-off-by: Monk Liu <Monk.Liu@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;
+
 	file_priv->driver_priv = NULL;
 
 	r = pm_runtime_get_sync(dev->dev);
-- 
2.7.4

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* RE: [PATCH] drm/amdgpu/sriov: reject kms open if TDR not finished or failed
       [not found]             ` <e7756d73-aa65-b351-2b1a-81ceea1e013f-otUistvHUpPR7s880joybQ@public.gmane.org>
@ 2018-04-26  7:57               ` Deng, Emily
       [not found]                 ` <CY4PR12MB112547536CD27B7FE2C330F38F8E0-rpdhrqHFk07v2MZdTKcfDgdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
  0 siblings, 1 reply; 12+ messages in thread
From: Deng, Emily @ 2018-04-26  7:57 UTC (permalink / raw)
  To: Michel Dänzer, Liu, Monk; +Cc: amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW

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.
[Emily] Yes, this is what the change want to do, when driver is doing
gpu recover or hardware is doing reset, it doesn't want to be
 interrupted, and during the reset any driver open kms is meaningless.
 
Applications randomly failing to start up during a GPU reset would be
 surprising and confusing for the user. The driver needs to handle this
 transparently.
[Emily] Yes, you are right, how about to sleep for some time in here to wait the GPU reset 
successfully. After sleep, if it still in gpu reset, then return error to APP?

Best Wishes,
Emily Deng

> -----Original Message-----
> From: Michel Dänzer [mailto:michel@daenzer.net]
> Sent: Thursday, April 26, 2018 3:33 PM
> To: Deng, Emily <Emily.Deng@amd.com>; Liu, Monk <Monk.Liu@amd.com>
> Cc: amd-gfx@lists.freedesktop.org
> Subject: Re: [PATCH] drm/amdgpu/sriov: reject kms open if TDR not finished
> or failed
> 
> On 2018-04-26 04:03 AM, Deng, Emily wrote:
> > Hi Michel,
> >> 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.
> > [Emily] Yes, this is what the change want to do, when driver is doing
> > gpu recover or hardware is doing reset, it doesn't want to be
> > interrupted, and during the reset any driver open kms is meaningless.
> 
> Applications randomly failing to start up during a GPU reset would be
> surprising and confusing for the user. The driver needs to handle this
> transparently.
> 
> 
> --
> 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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] drm/amdgpu/sriov: reject kms open if TDR not finished or failed
       [not found]                 ` <CY4PR12MB112547536CD27B7FE2C330F38F8E0-rpdhrqHFk07v2MZdTKcfDgdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
@ 2018-04-26  9:01                   ` Michel Dänzer
       [not found]                     ` <9d56b1c3-1e51-f3f1-17d6-c1615dea0a05-otUistvHUpPR7s880joybQ@public.gmane.org>
  0 siblings, 1 reply; 12+ messages in thread
From: Michel Dänzer @ 2018-04-26  9:01 UTC (permalink / raw)
  To: Deng, Emily, Liu, Monk; +Cc: amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW

On 2018-04-26 09:57 AM, Deng, Emily wrote:
>>>> 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.
>>> [Emily] Yes, this is what the change want to do, when driver is
>>> doing gpu recover or hardware is doing reset, it doesn't want to
>>> be interrupted, and during the reset any driver open kms is
>>> meaningless.
>> 
>> Applications randomly failing to start up during a GPU reset would
>> be surprising and confusing for the user. The driver needs to
>> handle this transparently.
> [Emily] Yes, you are right, how about to sleep for some time in here
> to wait the GPU reset successfully. After sleep, if it still in gpu
> reset, then return error to APP?

I think the only case where returning an error here *might* be
appropriate is when the driver has tried resetting the GPU, but it has
definitely and irrecoverably failed. But even in that case, it might be
better to let the open succeed, and let userspace figure out what
happened by other means.


-- 
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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* RE: [PATCH] drm/amdgpu/sriov: reject kms open if TDR not finished or failed
       [not found]                     ` <9d56b1c3-1e51-f3f1-17d6-c1615dea0a05-otUistvHUpPR7s880joybQ@public.gmane.org>
@ 2018-04-27  5:29                       ` Deng, Emily
       [not found]                         ` <CY4PR12MB1125653A73D8A22533FB3FBE8F8D0-rpdhrqHFk07v2MZdTKcfDgdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
  0 siblings, 1 reply; 12+ messages in thread
From: Deng, Emily @ 2018-04-27  5:29 UTC (permalink / raw)
  To: Michel Dänzer, Liu, Monk; +Cc: amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW

Sorry, I don't get your point, when the GPU is doing a reset, even let the dkms open, the userspace will
still fail, so during the reset, any driver open kms is meaningless.

Best Wishes,
Emily Deng

> -----Original Message-----
> From: Michel Dänzer [mailto:michel@daenzer.net]
> Sent: Thursday, April 26, 2018 5:01 PM
> To: Deng, Emily <Emily.Deng@amd.com>; Liu, Monk <Monk.Liu@amd.com>
> Cc: amd-gfx@lists.freedesktop.org
> Subject: Re: [PATCH] drm/amdgpu/sriov: reject kms open if TDR not finished
> or failed
> 
> On 2018-04-26 09:57 AM, Deng, Emily wrote:
> >>>> 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.
> >>> [Emily] Yes, this is what the change want to do, when driver is
> >>> doing gpu recover or hardware is doing reset, it doesn't want to be
> >>> interrupted, and during the reset any driver open kms is
> >>> meaningless.
> >>
> >> Applications randomly failing to start up during a GPU reset would be
> >> surprising and confusing for the user. The driver needs to handle
> >> this transparently.
> > [Emily] Yes, you are right, how about to sleep for some time in here
> > to wait the GPU reset successfully. After sleep, if it still in gpu
> > reset, then return error to APP?
> 
> I think the only case where returning an error here *might* be appropriate is
> when the driver has tried resetting the GPU, but it has definitely and
> irrecoverably failed. But even in that case, it might be better to let the open
> succeed, and let userspace figure out what happened by other means.
> 
> 
> --
> 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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] drm/amdgpu/sriov: reject kms open if TDR not finished or failed
       [not found]                         ` <CY4PR12MB1125653A73D8A22533FB3FBE8F8D0-rpdhrqHFk07v2MZdTKcfDgdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
@ 2018-04-27  7:52                           ` Michel Dänzer
       [not found]                             ` <77a5af30-0371-4816-fcf5-0bbcffa4395c-otUistvHUpPR7s880joybQ@public.gmane.org>
  0 siblings, 1 reply; 12+ messages in thread
From: Michel Dänzer @ 2018-04-27  7:52 UTC (permalink / raw)
  To: Deng, Emily, Liu, Monk; +Cc: amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW

On 2018-04-27 07:29 AM, Deng, Emily wrote:
> Sorry, I don't get your point, when the GPU is doing a reset, even
> let the dkms open, the userspace will still fail, so during the
> reset, any driver open kms is meaningless.

That needs to be fixed, then. If the GPU reset succeeds, userspace
shouldn't have to fail.


-- 
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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* RE: [PATCH] drm/amdgpu/sriov: reject kms open if TDR not finished or failed
       [not found]                             ` <77a5af30-0371-4816-fcf5-0bbcffa4395c-otUistvHUpPR7s880joybQ@public.gmane.org>
@ 2018-04-27  8:15                               ` Deng, Emily
       [not found]                                 ` <CY4PR12MB11253279D4404CF482FD7CF08F8D0-rpdhrqHFk07v2MZdTKcfDgdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
  0 siblings, 1 reply; 12+ messages in thread
From: Deng, Emily @ 2018-04-27  8:15 UTC (permalink / raw)
  To: Michel Dänzer, Liu, Monk; +Cc: amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW

No, I mean during GPU reset, the app will fail, it is reasonable. The flag in_gpu_reset indicates the GPU is in reset state, after reset, we will set the flag back.

Best Wishes,
Emily Deng

> -----Original Message-----
> From: Michel Dänzer [mailto:michel@daenzer.net]
> Sent: Friday, April 27, 2018 3:52 PM
> To: Deng, Emily <Emily.Deng@amd.com>; Liu, Monk <Monk.Liu@amd.com>
> Cc: amd-gfx@lists.freedesktop.org
> Subject: Re: [PATCH] drm/amdgpu/sriov: reject kms open if TDR not finished
> or failed
> 
> On 2018-04-27 07:29 AM, Deng, Emily wrote:
> > Sorry, I don't get your point, when the GPU is doing a reset, even let
> > the dkms open, the userspace will still fail, so during the reset, any
> > driver open kms is meaningless.
> 
> That needs to be fixed, then. If the GPU reset succeeds, userspace shouldn't
> have to fail.
> 
> 
> --
> 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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] drm/amdgpu/sriov: reject kms open if TDR not finished or failed
       [not found]                                 ` <CY4PR12MB11253279D4404CF482FD7CF08F8D0-rpdhrqHFk07v2MZdTKcfDgdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
@ 2018-04-27  8:22                                   ` Michel Dänzer
       [not found]                                     ` <fc3f05cd-ed48-c04e-cfcb-59ed5ab224e2-otUistvHUpPR7s880joybQ@public.gmane.org>
  0 siblings, 1 reply; 12+ messages in thread
From: Michel Dänzer @ 2018-04-27  8:22 UTC (permalink / raw)
  To: Deng, Emily, Liu, Monk; +Cc: amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW

On 2018-04-27 10:15 AM, Deng, Emily wrote:
> No, I mean during GPU reset, the app will fail, it is reasonable.

I understand, but I don't agree that it's reasonable. :)


-- 
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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] drm/amdgpu/sriov: reject kms open if TDR not finished or failed
       [not found]                                     ` <fc3f05cd-ed48-c04e-cfcb-59ed5ab224e2-otUistvHUpPR7s880joybQ@public.gmane.org>
@ 2018-05-03 12:01                                       ` Christian König
  0 siblings, 0 replies; 12+ messages in thread
From: Christian König @ 2018-05-03 12:01 UTC (permalink / raw)
  To: Michel Dänzer, Deng, Emily, Liu, Monk
  Cc: amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW

Am 27.04.2018 um 10:22 schrieb Michel Dänzer:
> On 2018-04-27 10:15 AM, Deng, Emily wrote:
>> No, I mean during GPU reset, the app will fail, it is reasonable.
> I understand, but I don't agree that it's reasonable. :)

I agree with Michel here that this is a really bad idea.

Even while the GPU is in reset an application should still be able to 
load userspace and open the driver.

It migth need to reset it's context even if it never used them, but that 
is a completely different story.

Christian.
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2018-05-03 12:01 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-04-25  6:39 [PATCH] drm/amdgpu/sriov: reject kms open if TDR not finished or failed Emily Deng
     [not found] ` <1524638384-31977-1-git-send-email-Emily.Deng-5C7GfCeVMHo@public.gmane.org>
2018-04-25  7:47   ` Michel Dänzer
     [not found]     ` <5bede6f6-24b2-8a3a-d3d8-9a37ac6d13c8-otUistvHUpPR7s880joybQ@public.gmane.org>
2018-04-26  2:03       ` Deng, Emily
     [not found]         ` <CY4PR12MB11254FFDBB4FCCF91CBDE9758F8E0-rpdhrqHFk07v2MZdTKcfDgdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
2018-04-26  7:33           ` Michel Dänzer
     [not found]             ` <e7756d73-aa65-b351-2b1a-81ceea1e013f-otUistvHUpPR7s880joybQ@public.gmane.org>
2018-04-26  7:57               ` Deng, Emily
     [not found]                 ` <CY4PR12MB112547536CD27B7FE2C330F38F8E0-rpdhrqHFk07v2MZdTKcfDgdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
2018-04-26  9:01                   ` Michel Dänzer
     [not found]                     ` <9d56b1c3-1e51-f3f1-17d6-c1615dea0a05-otUistvHUpPR7s880joybQ@public.gmane.org>
2018-04-27  5:29                       ` Deng, Emily
     [not found]                         ` <CY4PR12MB1125653A73D8A22533FB3FBE8F8D0-rpdhrqHFk07v2MZdTKcfDgdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
2018-04-27  7:52                           ` Michel Dänzer
     [not found]                             ` <77a5af30-0371-4816-fcf5-0bbcffa4395c-otUistvHUpPR7s880joybQ@public.gmane.org>
2018-04-27  8:15                               ` Deng, Emily
     [not found]                                 ` <CY4PR12MB11253279D4404CF482FD7CF08F8D0-rpdhrqHFk07v2MZdTKcfDgdYzm3356FpvxpqHgZTriW3zl9H0oFU5g@public.gmane.org>
2018-04-27  8:22                                   ` Michel Dänzer
     [not found]                                     ` <fc3f05cd-ed48-c04e-cfcb-59ed5ab224e2-otUistvHUpPR7s880joybQ@public.gmane.org>
2018-05-03 12:01                                       ` Christian König
2018-04-26  7:39   ` Liu, Monk

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.