From: Jan Kara Subject: Re: [PATCH 4/5 resend] VFS: Rename vfs_check_frozen() to vfs_block_until_thawed() Date: Wed, 7 Dec 2011 23:46:12 +0100 Message-ID: <20111207224612.GP4622@quack.suse.cz> References: <1323118489-16326-1-git-send-email-kamal@canonical.com> <1323118489-16326-5-git-send-email-kamal@canonical.com> <20111206114025.GA4439@infradead.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Kamal Mostafa , Jan Kara , Alexander Viro , Andreas Dilger , Matthew Wilcox , Randy Dunlap , Theodore Tso , linux-doc@vger.kernel.org, linux-ext4@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, Surbhi Palande , Valerie Aurora , Christopher Chaltain , "Peter M. Petrakis" , Mikulas Patocka To: Christoph Hellwig Return-path: Content-Disposition: inline In-Reply-To: <20111206114025.GA4439@infradead.org> Sender: linux-doc-owner@vger.kernel.org List-Id: linux-ext4.vger.kernel.org On Tue 06-12-11 06:40:25, Christoph Hellwig wrote: > On Mon, Dec 05, 2011 at 12:54:48PM -0800, Kamal Mostafa wrote: > > From: Valerie Aurora > > > > The name of vfs_check_frozen() tends to de-emphasize an important > > fact: that it will block until the file system thaws to the specified > > level. Make bugs slightly less likely by renaming to > > vfs_block_until_thawed(). > > I really hate that name. The old one wasn't perfect either, but unless > you have a really good new one I'm strongly in favour of not repainting > the bikeshed. Maybe vfs_wait_frozen()? Honza -- Jan Kara SUSE Labs, CR