Re: [PATCH 1/3] Vectorize aio_read/aio_write methods

From: Badari Pulavarty
Date: Tue May 09 2006 - 16:07:39 EST




Christoph Hellwig 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.

Yep. I am currently killing read/write methods for all filesystems and also
getting rid of generic_file_read() and generic_file_write().

Thanks,
Badari


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/