Re: [PATCH v3 0/2] drm: Add GPU reset sysfs

2023-01-05 Thread Daniel Vetter
On Thu, 8 Dec 2022 at 05:54, Alex Deucher wrote: > > On Wed, Nov 30, 2022 at 6:11 AM Daniel Vetter wrote: > > > > On Fri, Nov 25, 2022 at 02:52:01PM -0300, André Almeida wrote: > > > This patchset adds a udev event for DRM device's resets. > > > > > > Userspace apps can trigger GPU resets by misu

Re: [PATCH v3 0/2] drm: Add GPU reset sysfs

2022-12-07 Thread Alex Deucher
On Wed, Nov 30, 2022 at 6:11 AM Daniel Vetter wrote: > > On Fri, Nov 25, 2022 at 02:52:01PM -0300, André Almeida wrote: > > This patchset adds a udev event for DRM device's resets. > > > > Userspace apps can trigger GPU resets by misuse of graphical APIs or driver > > bugs. Either way, the GPU res

Re: [PATCH v3 0/2] drm: Add GPU reset sysfs

2022-11-30 Thread Simon Ser
On Wednesday, November 30th, 2022 at 16:23, André Almeida wrote: > On 11/28/22 06:30, Simon Ser wrote: > > > The PID is racy, the user-space daemon could end up killing an > > unrelated process… Is there any way we could use a pidfd instead? > > Is the PID race condition something that really

Re: [PATCH v3 0/2] drm: Add GPU reset sysfs

2022-11-30 Thread André Almeida
On 11/28/22 06:30, Simon Ser wrote: The PID is racy, the user-space daemon could end up killing an unrelated process… Is there any way we could use a pidfd instead? Is the PID race condition something that really happens or rather something theoretical? Anyway, I can't see how pidfd and ueve

Re: [PATCH v3 0/2] drm: Add GPU reset sysfs

2022-11-30 Thread Daniel Vetter
On Fri, Nov 25, 2022 at 02:52:01PM -0300, André Almeida wrote: > This patchset adds a udev event for DRM device's resets. > > Userspace apps can trigger GPU resets by misuse of graphical APIs or driver > bugs. Either way, the GPU reset might lead the system to a broken state[1], > that > might be

Re: [PATCH v3 0/2] drm: Add GPU reset sysfs

2022-11-28 Thread Simon Ser
The PID is racy, the user-space daemon could end up killing an unrelated process… Is there any way we could use a pidfd instead?

Re: [PATCH v3 0/2] drm: Add GPU reset sysfs

2022-11-28 Thread Pekka Paalanen
On Fri, 25 Nov 2022 14:52:01 -0300 André Almeida wrote: > This patchset adds a udev event for DRM device's resets. Hi, this seems a good idea to me. > Userspace apps can trigger GPU resets by misuse of graphical APIs or driver > bugs. Either way, the GPU reset might lead the system to a broken

[PATCH v3 0/2] drm: Add GPU reset sysfs

2022-11-25 Thread André Almeida
This patchset adds a udev event for DRM device's resets. Userspace apps can trigger GPU resets by misuse of graphical APIs or driver bugs. Either way, the GPU reset might lead the system to a broken state[1], that might be recovered if user has access to a tty or a remote shell. Arguably, this rec