lkml.org 
[lkml]   [2001]   [Jun]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH] devfs v181 available
    Alexander Viro writes:
    >
    >
    > On Mon, 18 Jun 2001, Richard Gooch wrote:
    >
    > > - Widened locking in <devfs_readlink> and <devfs_follow_link>
    >
    > No, you hadn't. Both vfs_readlink() and vfs_follow_link() are blocking
    > functions, so BKL is worthless there.

    Huh? The BKL will protect against other operations which might cause
    the devfs entry to be unregistered, where those other operations also
    grab the BKL. So, it's an improvement.

    Sure, some operations may cause unregistration without grabbing the
    BKL, but that's orthogonal (and requires more extensive changes). If
    this "widening" is of no use, then what use are the existing grabs of
    the BKL in those functions? You're the one who added them in the first
    place.

    Regards,

    Richard....
    Permanent: rgooch@atnf.csiro.au
    Current: rgooch@ras.ucalgary.ca
    -
    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/

    \
     
     \ /
      Last update: 2005-03-22 13:17    [W:2.285 / U:0.012 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site