Re: [PATCH v2] drm/xe/uapi: Bring back reset uevent

2024-08-15 Thread Lucas De Marchi
On Fri, Aug 16, 2024 at 09:59:31AM GMT, Aravind Iddamsetty wrote: On 14/08/24 19:54, Lucas De Marchi wrote: On Wed, Aug 14, 2024 at 12:16:41PM GMT, Aravind Iddamsetty wrote: i see that this is even called from xe_guc_exec_queue_lr_cleanup which is for long running queue from various call path

Re: [PATCH v2] drm/xe/uapi: Bring back reset uevent

2024-08-15 Thread Aravind Iddamsetty
On 14/08/24 19:54, Lucas De Marchi wrote: > On Wed, Aug 14, 2024 at 12:16:41PM GMT, Aravind Iddamsetty wrote: > i see that this is even called from xe_guc_exec_queue_lr_cleanup which is > for long running queue > from various call paths need to check in what scenarios those happen. >

Re: [PATCH v2] drm/xe/uapi: Bring back reset uevent

2024-08-14 Thread Lucas De Marchi
On Wed, Aug 14, 2024 at 12:16:41PM GMT, Aravind Iddamsetty wrote: i see that this is even called from xe_guc_exec_queue_lr_cleanup which is for long running queue from various call paths need to check in what scenarios those happen. Should we add a reason for long running queue? Both of your q

Re: [PATCH v2] drm/xe/uapi: Bring back reset uevent

2024-08-14 Thread Aravind Iddamsetty
On 14/08/24 14:42, Raag Jadav wrote: > On Wed, Aug 14, 2024 at 12:16:41PM +0530, Aravind Iddamsetty wrote: >> On 13/08/24 22:24, Rodrigo Vivi wrote: >>> On Tue, Aug 13, 2024 at 04:28:32PM +0300, Raag Jadav wrote: On Mon, Aug 12, 2024 at 03:08:14PM +0530, Aravind Iddamsetty wrote: > On 12

Re: [PATCH v2] drm/xe/uapi: Bring back reset uevent

2024-08-14 Thread Raag Jadav
On Wed, Aug 14, 2024 at 12:16:41PM +0530, Aravind Iddamsetty wrote: > >On 13/08/24 22:24, Rodrigo Vivi wrote: >> On Tue, Aug 13, 2024 at 04:28:32PM +0300, Raag Jadav wrote: >>> On Mon, Aug 12, 2024 at 03:08:14PM +0530, Aravind Iddamsetty wrote: On 12/08/24 13:18, Raag Jadav wrote: > From:

Re: [PATCH v2] drm/xe/uapi: Bring back reset uevent

2024-08-13 Thread Aravind Iddamsetty
On 13/08/24 22:24, Rodrigo Vivi wrote: > On Tue, Aug 13, 2024 at 04:28:32PM +0300, Raag Jadav wrote: >> On Mon, Aug 12, 2024 at 03:08:14PM +0530, Aravind Iddamsetty wrote: >>> On 12/08/24 13:18, Raag Jadav wrote: From: Himal Prasad Ghimiray This was dropped in commit 77a0d4d1cea2

Re: [PATCH v2] drm/xe/uapi: Bring back reset uevent

2024-08-13 Thread Aravind Iddamsetty
On 13/08/24 19:43, Lucas De Marchi wrote: > On Mon, Aug 12, 2024 at 01:18:12PM GMT, Raag Jadav wrote: >> From: Himal Prasad Ghimiray >> >> This was dropped in commit 77a0d4d1cea2 ("drm/xe/uapi: Remove reset >> uevent for now") as part of refactoring. >> >> Now that we have better uapi semantics

Re: [PATCH v2] drm/xe/uapi: Bring back reset uevent

2024-08-13 Thread Rodrigo Vivi
On Tue, Aug 13, 2024 at 04:28:32PM +0300, Raag Jadav wrote: > On Mon, Aug 12, 2024 at 03:08:14PM +0530, Aravind Iddamsetty wrote: > > > > On 12/08/24 13:18, Raag Jadav wrote: > > > From: Himal Prasad Ghimiray > > > > > > This was dropped in commit 77a0d4d1cea2 ("drm/xe/uapi: Remove reset > > > ue

Re: [PATCH v2] drm/xe/uapi: Bring back reset uevent

2024-08-13 Thread Lucas De Marchi
On Mon, Aug 12, 2024 at 01:18:12PM GMT, Raag Jadav wrote: From: Himal Prasad Ghimiray This was dropped in commit 77a0d4d1cea2 ("drm/xe/uapi: Remove reset uevent for now") as part of refactoring. Now that we have better uapi semantics and naming for the uevent, bring it back. With this in place

Re: [PATCH v2] drm/xe/uapi: Bring back reset uevent

2024-08-13 Thread Raag Jadav
On Mon, Aug 12, 2024 at 03:08:14PM +0530, Aravind Iddamsetty wrote: > > On 12/08/24 13:18, Raag Jadav wrote: > > From: Himal Prasad Ghimiray > > > > This was dropped in commit 77a0d4d1cea2 ("drm/xe/uapi: Remove reset > > uevent for now") as part of refactoring. > > > > Now that we have better uap

Re: [PATCH v2] drm/xe/uapi: Bring back reset uevent

2024-08-12 Thread Aravind Iddamsetty
On 12/08/24 13:18, Raag Jadav wrote: > From: Himal Prasad Ghimiray > > This was dropped in commit 77a0d4d1cea2 ("drm/xe/uapi: Remove reset > uevent for now") as part of refactoring. > > Now that we have better uapi semantics and naming for the uevent, > bring it back. With this in place, userspac

Re: [PATCH v2] drm/xe/uapi: Bring back reset uevent

2024-08-12 Thread Ghimiray, Himal Prasad
On 12-08-2024 13:18, Raag Jadav wrote: From: Himal Prasad Ghimiray This was dropped in commit 77a0d4d1cea2 ("drm/xe/uapi: Remove reset uevent for now") as part of refactoring. Now that we have better uapi semantics and naming for the uevent, bring it back. With this in place, userspace will

[PATCH v2] drm/xe/uapi: Bring back reset uevent

2024-08-12 Thread Raag Jadav
From: Himal Prasad Ghimiray This was dropped in commit 77a0d4d1cea2 ("drm/xe/uapi: Remove reset uevent for now") as part of refactoring. Now that we have better uapi semantics and naming for the uevent, bring it back. With this in place, userspace will be notified of wedged device along with its