Re: [PATCH] add support for vectored and async I/O to all simple filesystems

[Christoph Hellwig <hch@lst.de>]

On Tue, Nov 01, 2005 at 11:28:41AM +0100, Miklos Szeredi wrote:
> > Every filesystem using generic_file_read/generic_file_write directly
> > can easily support vectored and async (well at least the API, it's not
> > async quite yet in mainline) I/O.  This patch adds the proper vectors
> > to all filesystems and switches them to do_sync_read/do_sync_write.
> 
> Do you really need to replace generic_file_read/write with
> do_sync_read/write?  That seems to only add overhead.

do_sync_read/write are doing the same thing as generic_file_read/write,
just above the method vectors.  Once I have all filesystems support
proper vectored and aio methods do_sync_read/write will go away
completly and upper code will always call the complex methods (
or hopefull just one cmobined aio/vectored ops with all fancies)
directly.

> 
> The rest of the patch I agree with.
> 
> Miklos
---end quoted text---