[lkml]   [2005]   [Oct]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [RFC] page lock ordering and OCFS2
    Zach Brown <> wrote:
    > Christoph Hellwig wrote:
    > > On Fri, Oct 21, 2005 at 10:43:24AM -0700, Zach Brown wrote:
    > >
    > >>It introduces block_read_full_page() and truncate_inode_pages() derivatives
    > >>which understand the PG_fs_misc special case. It needs a few export patches to
    > >>the core, but the real burden is on OCFS2 to keep these derivatives up to date.
    > >
    > > The way you do it looks nice, but the exports aren't a big no-way. That
    > > stuff is far too internal to be exported. Either we can get Andrew to
    > > agree on moving those bits into the codepath for all filesystems or
    > > we need to do some hackery where every functions gets renamed to __function
    > > with an argument int cluster_aware and we have to functions inling them,
    > > one normal and one for cluster filesystems.
    > Yeah, I can certainly appreciate that line of reasoning. I'm happy to do that
    > work, but it'd be nice to get some assurance that it won't be wasted effort.
    > Andrew, is this a reasonable direction to take things in? We'd avoid the
    > exports by introducing some wrappers and helpers to the core that OCFS2 would
    > call..

    It depends on what the patch ends up looking like I guess.

    All those games with PG_fs_misc look awfully similar to lock_page() - I'd
    have thought there's some room for rationalising code in there.

    The overall approach would be to avoid adding overhead and complexity for
    other filesystems and to only export symbols which constitute a sensible
    API: avoid exporting weirdo internal helpers which we might change in the
    future (but we don't care about external modules, so why does this matter?
    Because some of them are GPL?)

    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2005-10-21 23:06    [W:0.031 / U:2.988 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site