lkml.org 
[lkml]   [2010]   [Jul]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    SubjectRe: [PATCH 2/6] block: push down BKL into .open and .release
    Date
    On Wednesday 07 July 2010, Christoph Hellwig wrote:
    > I might be missing something, but this doesn't actually push the
    > BKL anyway. It adds duplicate BKL locking inside the ->open and
    > ->release methods without actually removing it in the callers.
    >
    > That's a pretty pointless thing to do. If you're not feeling
    > confident enough about touching block_dev.c I can look into that
    > part of the pushdown, but this patch on it's own is not actually
    > useful.

    My idea was that the patch in its current form would be less
    controversial if it can't break anything, but you're certainly
    right that the description doesn't match what the patch does.

    I'm now integrating the half of patch 5, as you suggested.
    I would much appreciate if you could verify my assumptions
    about the blkdev_get/blkdev_put functions in the combined patch.

    Arnd


    \
     
     \ /
      Last update: 2010-07-07 16:07    [W:0.027 / U:29.900 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site