Re: [PATCH 1/2] kill unnessecary ioops indirection

[Christoph Hellwig <hch@xxxxxx>]

On Wed, Sep 19, 2007 at 11:42:17AM +1000, Lachlan McIlroy wrote:
> >vn_to_inode()/vn_from_inode() are no-ops nowdays, btw.
> Yeah I know - that's why I'd like to get rid of them.  They make the
> code ugly and give the impression we need to work with two different
> objects.  There's lots of places where we declare an extra pointer on
> the stack when we don't need to.  Anyway that's fodder for another
> patch, this patch looks good now.

Yes, eventually we should get rid of them.  I have a patch for the
functions prototypes in xfs_vnodeops.h somewhere, and fixing the dmapi
interface to avoid vnodes is also high on my todo list.