Re: [NFS] [GIT] NFS client update for 2.6.16

From: Trond Myklebust
Date: Tue Mar 21 2006 - 13:55:30 EST


On Tue, 2006-03-21 at 17:46 +0000, Christoph Hellwig wrote:
> > commit 47989d7454398827500d0e73766270986a3b488f
> > Author: Chuck Lever <cel@xxxxxxxxxx>
> > Date: Mon Mar 20 13:44:32 2006 -0500
> >
> > NFS: remove support for multi-segment iovs in the direct write path
> >
> > Eliminate the persistent use of automatic storage in all parts of the
> > NFS client's direct write path to pave the way for introducing support
> > for aio against files opened with the O_DIRECT flag.
>
> NACK. We have patches pending that consolidate ->aio_read/write and
> ->read/writev into one operation. this change is completely counterproductive
> towards that goal which has been discussed on -fsdevel for a while.

Could you, BTW, please post these pending patches? They are clearly not
in -mm, since these patches have been there for some time without
conflicting with anything else. The only references I could find were to
the following thread:

http://thread.gmane.org/gmane.linux.file-systems/7748

which basically just adds a BUG() if you call NFS O_DIRECT with a
multi-segment iovec.

I can't see from those patches how Chuck's O_DIRECT changes can be
construed as being "counterproductive". AFAICS those changes are
completely orthogonal to your proposal.

Cheers,
Trond

-
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/