From mboxrd@z Thu Jan 1 00:00:00 1970 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: Content-Disposition: inline In-Reply-To: <1323118489-16326-5-git-send-email-kamal@canonical.com> Sender: linux-ext4-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org 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.