lkml.org 
[lkml]   [2006]   [May]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 1/3] Vectorize aio_read/aio_write methods
    Christoph Hellwig <hch@lst.de> wrote:
    >
    > On Tue, May 09, 2006 at 12:01:05PM -0700, Andrew Morton wrote:
    > > Together these three patches shrink the kernel by 113 lines. I don't know
    > > what the effect is on text size, but that's a pretty modest saving, at a
    > > pretty high risk level.
    > >
    > > What else do we get in return for this risk?
    >
    > there's another patch ontop which I didn't bother to redo until this is
    > accepted which kills a lot more code. After that filesystems only have
    > to implement one method each for all kinds of read/write calls. Which
    > allows to both make the mm/filemap.c far less complex and actually
    > understandable aswell as for any filesystem that uses more complex
    > read/write variants than direct filemap.c calls. In addition to these
    > simplification we also get a feature (async vectored I/O) for free.

    Fair enough, thanks. Simplifying filemap.c would be a win.

    I'll crunch on these three patches in the normal fashion. It'll be good if
    we can get the followup patch done within the next week or two so we can
    get it all tested at the same time. Although from your description it
    doesn't sound like it'll be completely trivial...
    -
    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: 2006-05-09 21:13    [W:0.024 / U:30.768 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site