From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Hellwig Subject: Re: [PATCH 17/17] fs: Clean up inode reference counting Date: Sat, 16 Oct 2010 12:14:11 -0400 Message-ID: <20101016161411.GB16861@infradead.org> References: <1285762729-17928-1-git-send-email-david@fromorbit.com> <1285762729-17928-18-git-send-email-david@fromorbit.com> <20100930021520.GA21198@infradead.org> <20101016075515.GI19147@amd> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Christoph Hellwig , Dave Chinner , linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org To: Nick Piggin Return-path: Content-Disposition: inline In-Reply-To: <20101016075515.GI19147@amd> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On Sat, Oct 16, 2010 at 06:55:15PM +1100, Nick Piggin wrote: > On Wed, Sep 29, 2010 at 10:15:20PM -0400, Christoph Hellwig wrote: > > Besides moving this much earlier in the series as mentioned before > > the most important thing is giving the helpers a different name as > > the iget name is already used for a different purpose, even if > > got rid of the original iget and only have iget_locked. > > It's a bit of a mess. We also have __iget. I don't care much about > the name though so I'll see what you've chosen in future posts. > > I'll see how it looks to move earlier in the series, shouldn't > be a big problem. Dave has sorted out the naming mess quite nicely in his repost.