Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752811AbXFDFTc (ORCPT ); Mon, 4 Jun 2007 01:19:32 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751816AbXFDFTX (ORCPT ); Mon, 4 Jun 2007 01:19:23 -0400 Received: from mailhub.sw.ru ([195.214.233.200]:2230 "EHLO relay.sw.ru" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750792AbXFDFTW (ORCPT ); Mon, 4 Jun 2007 01:19:22 -0400 Message-ID: <4663A0BF.9010008@sw.ru> Date: Mon, 04 Jun 2007 09:18:55 +0400 From: Vasily Averin User-Agent: Thunderbird 1.5.0.10 (X11/20060911) MIME-Version: 1.0 To: Linux Kernel Mailing List , Andrew Morton , devel@openvz.org, linux-ext4@vger.kernel.org, Stephen Tweedie , adilger@clusterfs.com Subject: [PATCH ext3/ext4] orphan list check on destroy_inode X-Enigmail-Version: 0.94.2.0 Content-Type: text/plain; charset=KOI8-R Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1955 Lines: 63 Customers claims to ext3-related errors, investigation showed that ext3 orphan list has been corrupted and have the reference to non-ext3 inode. The following debug helps to understand the reasons of this issue. Signed-off-by: Vasily Averin diff --git a/fs/ext3/super.c b/fs/ext3/super.c index 6e30629..46e2fa6 100644 --- a/fs/ext3/super.c +++ b/fs/ext3/super.c @@ -459,6 +459,21 @@ static struct inode *ext3_alloc_inode(struct super_block *sb) static void ext3_destroy_inode(struct inode *inode) { + if (!list_empty(&(EXT3_I(inode)->i_orphan))) { + int i, imax; + unsigned int *p; + + p = (unsigned int *)EXT3_I(inode); + imax = sizeof(struct ext3_inode_info) / sizeof(unsigned int); + printk("Inode %p: orphan list check failed!\n", EXT3_I(inode)); + for (i = 0; i < imax; i++) { + if (i && ((i % 8) == 0)) + printk("\n"); + printk("%08x ", *p++); + } + printk("\n"); + dump_stack(); + } kmem_cache_free(ext3_inode_cachep, EXT3_I(inode)); } diff --git a/fs/ext4/super.c b/fs/ext4/super.c index cb9afdd..f86e0ac 100644 --- a/fs/ext4/super.c +++ b/fs/ext4/super.c @@ -510,6 +510,21 @@ static struct inode *ext4_alloc_inode(struct super_block *sb) static void ext4_destroy_inode(struct inode *inode) { + if (!list_empty(&(EXT4_I(inode)->i_orphan))) { + int i, imax; + unsigned int *p; + + p = (unsigned int *)EXT4_I(inode); + imax = sizeof(struct ext4_inode_info) / sizeof(unsigned int); + printk("Inode %p: orphan list check failed!\n", EXT4_I(inode)); + for (i = 0; i < imax; i++) { + if (i && ((i % 8) == 0)) + printk("\n"); + printk("%08x ", *p++); + } + printk("\n"); + dump_stack(); + } kmem_cache_free(ext4_inode_cachep, EXT4_I(inode)); } - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/