From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?utf-8?B?SsO2cm4=?= Engel Subject: Re: + embed-a-struct-path-into-struct-nameidata-instead-of-nd-dentrymnt.patch added to -mm tree Date: Mon, 5 Nov 2007 23:10:21 +0100 Message-ID: <20071105221021.GC21533@lazybastard.org> References: <200711052101.lA5L1Q1p019531@imap1.linux-foundation.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: mm-commits@vger.kernel.org, jblunck@suse.de, agruen@suse.de, hch@lst.de, linux-fsdevel@vger.kernel.org, viro@zeniv.linux.org.uk To: akpm@linux-foundation.org Return-path: Received: from lazybastard.de ([212.112.238.170]:39214 "EHLO longford.lazybastard.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753551AbXKEWPO (ORCPT ); Mon, 5 Nov 2007 17:15:14 -0500 Content-Disposition: inline In-Reply-To: <200711052101.lA5L1Q1p019531@imap1.linux-foundation.org> Sender: linux-fsdevel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On Mon, 5 November 2007 13:01:25 -0800, akpm@linux-foundation.org wrote= : >=20 > The patch titled > Embed a struct path into struct nameidata instead of nd->{dentry= ,mnt} > has been added to the -mm tree. Its filename is > embed-a-struct-path-into-struct-nameidata-instead-of-nd-dentrymn= t.patch >=20 > *** Remember to use Documentation/SubmitChecklist when testing your c= ode *** >=20 > See http://www.zip.com.au/~akpm/linux/patches/stuff/added-to-mm.txt t= o find > out what to do about this >=20 > ------------------------------------------------------ > Subject: Embed a struct path into struct nameidata instead of nd->{de= ntry,mnt} > From: Jan Blunck >=20 > Switch from nd->{dentry,mnt} to nd->path.{dentry,mnt} everywhere. >=20 > Signed-off-by: Jan Blunck > Signed-off-by: Andreas Gruenbacher > Acked-by: Christoph Hellwig > Cc: Al Viro > CC: > Signed-off-by: Andrew Morton =46rowned-upon-by: Joern Engel This patch changes some 400 lines, most if not all of which get longer and more complicated to read. 23 get sufficiently longer to require an additional linebreak. I can't remember complexity being invited into the kernel without good reasoning, yet the patch description is surprisingly low on reasoning: > Switch from nd->{dentry,mnt} to nd->path.{dentry,mnt} everywhere. The following two patches manage to remove 7 lines in total. In total 23 were added, 7 removed , 400+ made longer and more complicated. Is there another more favorable metric? Will this patchset prevent bugs? Shrink the kernel size? Anything? If churn is the only effect of this, please considere it NAKed again. J=C3=B6rn --=20 A surrounded army must be given a way out. -- Sun Tzu - To unsubscribe from this list: send the line "unsubscribe linux-fsdevel= " in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html