From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Hellwig Subject: Re: [PATH 08/19] jfs: new export ops Date: Thu, 13 Sep 2007 15:59:34 +0200 Message-ID: <20070913135933.GA11411@lst.de> References: <20070830131632.GI6834@lst.de> <20070910194533.GA18793@fieldses.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: linux-fsdevel@vger.kernel.org, Neil Brown , nfs@lists.sourceforge.net, Christoph Hellwig To: "J. Bruce Fields" Return-path: Content-Disposition: inline In-Reply-To: <20070910194533.GA18793@fieldses.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: nfs-bounces@lists.sourceforge.net Errors-To: nfs-bounces@lists.sourceforge.net List-Id: linux-fsdevel.vger.kernel.org On Mon, Sep 10, 2007 at 03:45:33PM -0400, J. Bruce Fields wrote: > On Thu, Aug 30, 2007 at 03:16:32PM +0200, Christoph Hellwig wrote: > > Trivial switch over to the new generic helpers. > > - result = ERR_PTR(-ESTALE); > > - goto out_iput; > > + iput(inode); > > + return ERR_PTR(-ESTALE); > > For some reason you're introducing a bunch of whitespace damage on lines > like that last one. Would you mind running these through checkpatch.pl > or whatever and then resending? > > Other than that I can't find any problem, and it seems like an obvious > improvement, though I'd like to see Neil take a look. Do you want these > added to the nfsd patches for 2.6.24? Yes, please add them to the queue. I've put a quilt series of patches with the whitespace fixe on http://verein.lst.de/~hch/patches.nfsd.tgz. Do you want to take it that way or should I resend everything to the list? ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2005. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ NFS maillist - NFS@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/nfs