From: Christoph Hellwig Subject: Re: [PATCH 4/5 resend] VFS: Rename vfs_check_frozen() to vfs_block_until_thawed() Date: Tue, 6 Dec 2011 06:40:25 -0500 Message-ID: <20111206114025.GA4439@infradead.org> References: <1323118489-16326-1-git-send-email-kamal@canonical.com> <1323118489-16326-5-git-send-email-kamal@canonical.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: 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: Kamal Mostafa Return-path: Received: from 173-166-109-252-newengland.hfc.comcastbusiness.net ([173.166.109.252]:38910 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933231Ab1LFLkc (ORCPT ); Tue, 6 Dec 2011 06:40:32 -0500 Content-Disposition: inline In-Reply-To: <1323118489-16326-5-git-send-email-kamal@canonical.com> Sender: linux-ext4-owner@vger.kernel.org List-ID: 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.