From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nick Piggin Subject: Re: [PATCH 17/17] fs: Clean up inode reference counting Date: Sat, 16 Oct 2010 18:55:15 +1100 Message-ID: <20101016075515.GI19147@amd> References: <1285762729-17928-1-git-send-email-david@fromorbit.com> <1285762729-17928-18-git-send-email-david@fromorbit.com> <20100930021520.GA21198@infradead.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Dave Chinner , linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org To: Christoph Hellwig Return-path: Content-Disposition: inline In-Reply-To: <20100930021520.GA21198@infradead.org> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org 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.