From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Morton Subject: Re: [PATCH 1 of 2] Stackfs: Introduce stackfs_copy_{attr,inode}_* Date: Fri, 13 Oct 2006 13:37:22 -0700 Message-ID: <20061013133722.d4f33ecb.akpm@osdl.org> References: <20061013122706.56970df2.akpm@osdl.org> <20061013200705.GB31928@filer.fsl.cs.sunysb.edu> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: linux-kernel@vger.kernel.org, torvalds@osdl.org, hch@infradead.org, viro@ftp.linux.org.uk, linux-fsdevel@vger.kernel.org, penberg@cs.helsinki.fi, ezk@cs.sunysb.edu, mhalcrow@us.ibm.com Return-path: Received: from smtp.osdl.org ([65.172.181.4]:61385 "EHLO smtp.osdl.org") by vger.kernel.org with ESMTP id S1751888AbWJMUhr (ORCPT ); Fri, 13 Oct 2006 16:37:47 -0400 To: Josef Sipek In-Reply-To: <20061013200705.GB31928@filer.fsl.cs.sunysb.edu> Sender: linux-fsdevel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On Fri, 13 Oct 2006 16:07:05 -0400 Josef Sipek wrote: > > > +static inline void __stackfs_copy_attr_all(struct inode *dest, > > > + const struct inode *src, > > > + int (*get_nlinks)(struct inode *)) > > > +{ > > > + if (!get_nlinks) > > > + dest->i_nlink = src->i_nlink; > > > + else > > > + dest->i_nlink = get_nlinks(dest); > > > > I cannot find a get_nlinks() in 2.6.19-rc2? > > It is the last argument to the function. Perhaps the function name is > deceiving. doh. That's why us old farts like to see dest->i_nlink = (*get_nlinks)(dest);