[Dri-devel] Patch to cleanup DRM context handle on process exit

2004-02-19 Thread Erdi Chen
This is a patch to call the context handles destructors and free the context bitmap entries when a process does not destroy its contexts before it exits. It saves context handles in a linked list in the drm_device struct. I decided to use per device list instead of a per file descriptor list f

Re: [Dri-devel] Patch to cleanup DRM context handle on process exit

2004-02-20 Thread Dieter Nützel
Am Freitag, 20. Februar 2004 08:39 schrieb Erdi Chen: > This is a patch to call the context handles destructors and free the > context bitmap entries when a process does not destroy its contexts > before it exits. It saves context handles in a linked list in the > drm_device struct. I decided to u

Re: [Dri-devel] Patch to cleanup DRM context handle on process exit

2004-02-20 Thread Ian Romanick
Dieter Nützel wrote: Am Freitag, 20. Februar 2004 08:39 schrieb Erdi Chen: This is a patch to call the context handles destructors and free the context bitmap entries when a process does not destroy its contexts before it exits. It saves context handles in a linked list in the drm_device struct.

Re: [Dri-devel] Patch to cleanup DRM context handle on process exit

2004-02-20 Thread Keith Whitwell
Erdi Chen wrote: This is a patch to call the context handles destructors and free the context bitmap entries when a process does not destroy its contexts before it exits. It saves context handles in a linked list in the drm_device struct. I decided to use per device list instead of a per file

Re: [Dri-devel] Patch to cleanup DRM context handle on process exit

2004-02-20 Thread Alan Cox
On Gwe, 2004-02-20 at 18:06, Keith Whitwell wrote: > The patch looks ok to me, though I really don't know the code in this area > particularly well. Any comments from anyone before applying? VIA branch is out of date (I've been way too busy lately) but yes Im happy, and the fix is relevant to th

Re: [Dri-devel] Patch to cleanup DRM context handle on process exit

2004-02-20 Thread Keith Whitwell
Erdi Chen wrote: This is a patch to call the context handles destructors and free the context bitmap entries when a process does not destroy its contexts before it exits. It saves context handles in a linked list in the drm_device struct. I decided to use per device list instead of a per file

Re: [Dri-devel] Patch to cleanup DRM context handle on process exit

2004-02-21 Thread Felix Kühling
On Fri, 20 Feb 2004 22:56:20 + Keith Whitwell <[EMAIL PROTECTED]> wrote: > Erdi Chen wrote: > > This is a patch to call the context handles destructors and free the > > context bitmap entries when a process does not destroy its contexts > > before it exits. It saves context handles in a link

Re: [Dri-devel] Patch to cleanup DRM context handle on process exit

2004-02-21 Thread Felix Kühling
On Sat, 21 Feb 2004 20:44:37 +0100 Felix Kühling <[EMAIL PROTECTED]> wrote: > On Fri, 20 Feb 2004 22:56:20 + > Keith Whitwell <[EMAIL PROTECTED]> wrote: > [snip] > > > > I've committed this, but I'm just about to leave on holidays. If there are > > any problems hopefully Erdi will be able

Re: [Dri-devel] Patch to cleanup DRM context handle on process exit

2004-02-21 Thread Erdi Chen
Felix Kühling wrote: and many more which I suspect are just because the compiler got confused. The problem seems to be that list_for_each_entry_safe is not defined. Erdi, did you forget to include it in the patch or is my working tree missing something? Felix Which kernel version are you usin