Messages in this thread | | | Date | Wed, 11 Aug 2010 16:17:33 +0200 | Subject | Re: [DRM] BUG: sleeping function called from invalid context, drm_lastclose | From | Luca Tettamanti <> |
| |
On Wed, Aug 11, 2010 at 10:50 AM, Dave Airlie <airlied@gmail.com> wrote: > On Wed, Aug 11, 2010 at 6:48 PM, Luca Tettamanti <kronos.it@gmail.com> wrote: >> Hi Arnd, >> this commit: >> >> commit 58374713c9dfb4d231f8c56cac089f6fbdedc2ec >> Author: Arnd Bergmann <arnd@arndb.de> >> Date: Sat Jul 10 23:51:39 2010 +0200 >> >> drm: kill BKL from common code >> >> >> moved the call to (inside drm_release) drm_lastclose inside dev->count_lock >> spinlock. >> drm_lastclose however takes dev->struct_mutex (now inside an atomic >> context): > > I have a patch from Chris Wilson that I need to push to fix this, > basically reducing the spin lock coverage, > and relying on the global mutex to handle the open race.
Ok, thank you :)
Luca -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |