lkml.org 
[lkml]   [2008]   [Jul]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH] parport/ppdev: fix registration of sysctl entries
    On Sun, Jul 06, 2008 at 05:07:02PM +0200, Marcin Slusarz wrote:
    > On Sun, Jul 06, 2008 at 01:11:48AM +0100, Al Viro wrote:
    > > On Sun, Jul 06, 2008 at 12:51:48AM +0100, Al Viro wrote:
    > >
    > > > I don't believe that it's right. Note that if you *do* race there, you
    > > > are fucked regardless of sysctls - ppdev.c::register_device() racing
    > > > with itself will do tons of fun things all by itself (starting with two
    > > > threads allocating different pdev and both setting pp->pdev).
    >
    > I wouldn't call it a race - BKL is protecting multiple ioctl calls, so we
    > won't ever claim the device from two different threads.

    Yes, we can - register_device() does kmalloc() with GFP_KERNEL, which kills
    any protection from BKL. BKL is _not_ providing any kind of exclusion while
    you sleep.


    \
     
     \ /
      Last update: 2008-07-06 22:37    [W:0.019 / U:148.184 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site