lkml.org 
[lkml]   [2010]   [Oct]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 17/18] fs: icache remove inode_lock
    On Wed, Oct 13, 2010 at 07:25:52AM -0400, Christoph Hellwig wrote:
    > On Wed, Oct 13, 2010 at 06:20:58PM +1100, Nick Piggin wrote:
    > > Unfortunate timing that everybody is suddenly interested in the
    > > scalability work :)
    >
    > People have been interested for a long time. It's just that we finally
    > made forward progress to get parts of it shape for merging, which should
    > have been done long time ago.

    It has been pretty close, IMO, it just needed some more reviewers, which
    it only just got really.


    > > I might need a little more time than 2.6.37! But I'll try my best.
    > > I don't think the patchset has suddenly become vastly more urgent
    > > in the past month, so I think my approach of having it get a lot
    > > of testing and go in Al's vfs tree for a while is best.
    >
    > It's always been quite urgent. Both inode_lock and dcache_lock
    > contention have been hurting us for a while.

    Yes I know it has been urgent for several years, but it hasn't been
    treated that way by vfs people until really so late that it is going to
    hurt a lot for people deploying Linux now or using enterprise distros
    etc. I had to really learn most of the code from scratch to get this far
    and got quite little constructive review really until recently. Anyway
    that's nothing to be done about that now. I don't think there should be
    any significant delay in me catching up, really.

    Thanks,
    Nick


    \
     
     \ /
      Last update: 2010-10-13 14:33    [W:0.022 / U:180.752 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site