From: Al Viro Subject: Re: [git pull] several fixes Date: Wed, 16 Mar 2011 20:21:17 +0000 Message-ID: <20110316202117.GW22723@ZenIV.linux.org.uk> References: <20110316144253.GT22723@ZenIV.linux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Miklos Szeredi , linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org To: Linus Torvalds Return-path: Received: from zeniv.linux.org.uk ([195.92.253.2]:36183 "EHLO ZenIV.linux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753800Ab1CPUVZ (ORCPT ); Wed, 16 Mar 2011 16:21:25 -0400 Content-Disposition: inline In-Reply-To: Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Wed, Mar 16, 2011 at 11:49:31AM -0700, Linus Torvalds wrote: > On Wed, Mar 16, 2011 at 11:17 AM, Miklos Szeredi wrote: > >> > >> "doesn't need" or "must not have"? > > > > It must not have one, otherwise refcounting will go astray. > > That's what I thought. The commit message just seemed to be off. > > > That cdev_put() should really be moved into f_op->release(), but that > > would be quite a bit of work. > > Yeah. Al? In tree, with edited commit message... Please, pull from git.kernel.org/pub/scm/linux/kernel/git/viro/vfs-2.6.git/ for-linus (just one commit in that branch)