From: Trond Myklebust <trond.myklebust@fys.uio.no>
Subject: Re: [PATCH, RFC] rework rpc_pipefs
Date: Tue, 28 Jun 2005 07:39:41 -0400
Message-ID: <1119958781.8245.4.camel@lade.trondhjem.org>
References: <20050506161018.GA3979@infradead.org>
	 <1115410171.1960.26.camel@lade.trondhjem.org>
	 <20050510122745.GA28981@infradead.org>
	 <20050628112923.GA18703@infradead.org>
Mime-Version: 1.0
Content-Type: text/plain
Cc: nfs@lists.sourceforge.net
To: Christoph Hellwig <hch@infradead.org>
In-Reply-To: <20050628112923.GA18703@infradead.org>
Sender: nfs-admin@lists.sourceforge.net
Errors-To: nfs-admin@lists.sourceforge.net

ty den 28.06.2005 Klokka 12:29 (+0100) skreiv Christoph Hellwig:
> Currently rpc_mkdir/rpc_rmdir and rpc_mkpipe/mk_unlink have an API that's
> a little unfortunate.  They take a path relative to the rpc_pipefs root and
> thus need to perform a full lookup.  If you look at debugfs or usbfs they
> always store the dentry for directories they created and thus can pass in
> a dentry + single pathname component pair into their equivalents of the
> above functions.
>  
> And in fact rpc_pipefs actually stores a dentry for all but one component so
> this change not only simplifies the core rpc_pipe code but also the callers.
> 
> Unfortuntately this code path is only used by the NFS4 idmapper and
> AUTH_GSSAPI for which I don't have a test enviroment.  Could someone give
> it a spin?  It's the last bit needed before we can rework the lookup_hash API

Sure. May I add the usual "Signed-off-by" line so that I can put it into
the NFS_ALL patch stream?

Cheers,
  Trond


-------------------------------------------------------
SF.Net email is sponsored by: Discover Easy Linux Migration Strategies
from IBM. Find simple to follow Roadmaps, straightforward articles,
informative Webcasts and more! Get everything you need to get up to
speed, fast. http://ads.osdn.com/?ad_id=7477&alloc_id=16492&op=click
_______________________________________________
NFS maillist  -  NFS@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs