From mboxrd@z Thu Jan 1 00:00:00 1970 From: Linus Torvalds Subject: Re: [PATCH 2/5] VFS: Add VFS helper functions for setting up private namespaces Date: Tue, 23 Jun 2009 19:51:59 -0700 (PDT) Message-ID: References: <20090622190913.27923.31665.stgit@heimdal.trondhjem.org> <20090622190913.27923.61549.stgit@heimdal.trondhjem.org> <20090623201307.GA25760@us.ibm.com> <1245792589.5133.24.camel@heimdal.trondhjem.org> <20090623223230.GA30173@us.ibm.com> Mime-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Cc: Trond Myklebust , Al Viro , Christoph Hellwig , linux-fsdevel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: "Serge E. Hallyn" Return-path: In-Reply-To: <20090623223230.GA30173-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org> Sender: linux-nfs-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-fsdevel.vger.kernel.org On Tue, 23 Jun 2009, Serge E. Hallyn wrote: > > (it keeps bugging me that the order of args to list_add() gets reversed > as a result, but clearly with both starting out empty it doesn't > matter..) .. and the new one is more logical. "list_add()" really adds the first entry to the list pointed to by the second one. It _works_ the other way too in this case, as you point out, but now that you mention it, it really looks like the pre-patch code is "wrong". Linus -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html