From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Rothwell Subject: linux-next: build failure after merge of the vfs tree Date: Mon, 19 Jul 2010 10:25:20 +1000 Message-ID: <20100719102520.a2d4f103.sfr@canb.auug.org.au> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: Received: from chilli.pcug.org.au ([203.10.76.44]:44184 "EHLO smtps.tip.net.au" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757772Ab0GSAZ0 (ORCPT ); Sun, 18 Jul 2010 20:25:26 -0400 Sender: linux-next-owner@vger.kernel.org List-ID: To: Al Viro Cc: linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, Suresh Jayaraman , Steve French , linux-cifs@vger.kernel.org Hi Al, After merging the vfs tree, today's linux-next build (powerpc ppc64_defconfig) failed like this: fs/cifs/cifsfs.c:498: error: unknown field 'clear_inode' specified in initializer Caused by commit 139f224817a203ff434eda5e6d3f457bc1579f27 ("cifs: define inode-level cache object and register them") from the cifs tree interacting with commit a8036d5aec0547a2bd7e3040c69bd172e474d860 ("convert remaining ->clear_inode() to ->evict_inode()") from the vfs tree. I added the following merge fix patch (by effectively copy and paste from some of the other file system fixups in the vfs commit). I have no idea if this is correct - someone should check. I can carry this patch as necessary. From: Stephen Rothwell Date: Mon, 19 Jul 2010 10:15:47 +1000 Subject: [PATCH] cifs: fix for clear_inode -> evict_inode API change Signed-off-by: Stephen Rothwell --- fs/cifs/cifsfs.c | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/fs/cifs/cifsfs.c b/fs/cifs/cifsfs.c index ba893b9..2fc5ed1 100644 --- a/fs/cifs/cifsfs.c +++ b/fs/cifs/cifsfs.c @@ -330,8 +330,10 @@ cifs_destroy_inode(struct inode *inode) } static void -cifs_clear_inode(struct inode *inode) +cifs_evict_inode(struct inode *inode) { + truncate_inode_pages(&inode->i_data, 0); + end_writeback(inode); cifs_fscache_release_inode_cookie(inode); } @@ -495,7 +497,7 @@ static const struct super_operations cifs_super_ops = { .alloc_inode = cifs_alloc_inode, .destroy_inode = cifs_destroy_inode, .drop_inode = cifs_drop_inode, - .clear_inode = cifs_clear_inode, + .evict_inode = cifs_evict_inode, /* .delete_inode = cifs_delete_inode, */ /* Do not need above function unless later we add lazy close of inodes or unless the kernel forgets to call us with the same number of releases (closes) -- 1.7.1 -- Cheers, Stephen Rothwell sfr@canb.auug.org.au http://www.canb.auug.org.au/~sfr/