From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dave Hansen Subject: Re: [patch 01/10] vfs: add path_create() and path_mknod() Date: Wed, 02 Apr 2008 14:19:39 -0700 Message-ID: <1207171179.23710.50.camel@nimitz.home.sr71.net> References: <20080402201247.358430231@szeredi.hu> <20080402201321.062856131@szeredi.hu> Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Cc: viro@zeniv.linux.org.uk, akpm@linux-foundation.org, ezk@cs.sunysb.edu, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org To: Miklos Szeredi Return-path: Received: from e33.co.us.ibm.com ([32.97.110.151]:49624 "EHLO e33.co.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756835AbYDBVTm (ORCPT ); Wed, 2 Apr 2008 17:19:42 -0400 In-Reply-To: <20080402201321.062856131@szeredi.hu> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: I think these definitely all clean the code up. They might add a couple of lines, net, but they certainly make it easier to read. As you noted, they also make it much harder to screw up the locking, which is even better. Just looking at it from an aesthetic point of view, it looks good to have these helpers around. -- Dave