lkml.org 
[lkml]   [2009]   [Apr]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [patch 00/27] [rfc] vfs scalability patchset
    Christoph Hellwig <hch@infradead.org> writes:

    > On Sat, Apr 25, 2009 at 09:06:49AM +0100, Al Viro wrote:
    >> Maybe... What Eric proposed is essentially a reuse of s_list for per-inode
    >> list of struct file. Presumably with something like i_lock for protection.
    >> So that's not a conflict.
    >
    > But what do we actually want it for? Right now it's only used for
    > ttys, which Nick has split out, and for remount r/o. For the normal
    > remount r/o case it will go away once we have proper per-sb writer
    > counts. And the fource remount r/o from sysrq is completely broken.

    The plan is to post my updated patches tomorrow after I have slept.

    What I am looking at is that the tty layer is not a special case. Any
    subsystem that wants any revoke kind of functionality starts wanting
    the list of files that are open. My current list where we have
    something like this is: sysfs, proc, sysctl, tun, tty, sound.

    I am in the process of generalizing the handling and bringing all of this
    into the VFS, where we only need to maintain it once, and can see
    clearly what is going on so we can optimize it.

    For that I essentially need per inode lists of files. Devices don't
    have inodes but the usually have some kind of equivalent like the
    tty struct we can attach inodes to.

    It looks like what I have could pretty easily be used to implement
    mount -f except for some weird cases like nfsd where the usual vfs
    rules are not followed. In particular things vfs_sync are a pain.

    > A while ago Peter had patches for files_lock scalability that went even
    > further than Nicks, and if I remember the arguments correctly just
    > splitting the lock wasn't really enough and he required additional
    > batching because there just were too many lock roundtrips. (Peter, do
    > you remember the defails?)

    I would love to hear what the issues are. Since everyone is worried
    about performance and contention I have gone ahead and made the
    files_list_lock per inode in my patches. We will see how well that works.
    My goals has simply been to add functionality without making a significant
    change in performance on the current workloads.

    Eric


    \
     
     \ /
      Last update: 2009-04-28 13:35    [W:4.355 / U:0.076 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site