mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Thomas Gleixner <tglx@linutronix.de>
To: Xianghua Xiao <xiaoxianghua@gmail.com>
Cc: john stultz <johnstul@us.ibm.com>, LKML <linux-kernel@vger.kernel.org>
Subject: Re: 2.6.33.3-rt16 Oops caused by umount
Date: Wed, 28 Apr 2010 18:34:31 +0200 (CEST)	[thread overview]
Message-ID: <alpine.LFD.2.00.1004281833230.2951@localhost.localdomain> (raw)
In-Reply-To: <w2ge6bf505b1004280821g70610cb7lc3821ee890345273@mail.gmail.com>

On Wed, 28 Apr 2010, Xianghua Xiao wrote:
> Just tried the patch, still got umount hang, please see below.

Can you please apply the patch below and provide the debug output ?

Thanks,

	tglx
---
 fs/file_table.c |   22 +++++++++++++++++++++-
 1 file changed, 21 insertions(+), 1 deletion(-)

Index: linux-2.6-tip/fs/file_table.c
===================================================================
--- linux-2.6-tip.orig/fs/file_table.c
+++ linux-2.6-tip/fs/file_table.c
@@ -410,7 +410,27 @@ int fs_may_remount_ro(struct super_block
 		list = &sb->s_files;
 #endif
 		list_for_each_entry(file, list, f_u.fu_list) {
-			struct inode *inode = file->f_path.dentry->d_inode;
+			struct inode *inode;
+
+			if (!file->f_path) {
+				printk(KERN_ERR "file %p fpath == NULL\n",
+				       file);
+				continue;
+			}
+
+			if (!file->f_path.dentry) {
+				printk(KERN_ERR "file %p dentry == NULL\n",
+				       file);
+				continue;
+			}
+
+			if (!file->f_path.dentry->d_inode) {
+				printk(KERN_ERR "file %p d_inode == NULL\n",
+				       file);
+				continue;
+			}
+
+			inode = file->f_path.dentry->d_inode;
 
 			/* File with pending delete? */
 			if (inode->i_nlink == 0)

  reply	other threads:[~2010-04-28 16:34 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-04-27 18:44 Xianghua Xiao
2010-04-27 18:56 ` Thomas Gleixner
2010-04-27 20:23   ` john stultz
2010-04-27 20:30     ` john stultz
2010-04-27 20:54       ` Xianghua Xiao
2010-04-27 22:08         ` 2.6.33.3-rt16 - not reaching login Uwaysi Bin Kareem
2010-04-28  6:01         ` 2.6.33.3-rt16 Oops caused by umount john stultz
2010-04-28 15:21           ` Xianghua Xiao
2010-04-28 16:34             ` Thomas Gleixner [this message]
2010-04-28 17:54               ` Xianghua Xiao
2010-04-28 19:38                 ` Thomas Gleixner
2010-04-28 20:06                   ` Xianghua Xiao
2010-04-28 20:22                     ` Thomas Gleixner
2010-04-28 21:22                       ` Xianghua Xiao
2010-04-28 21:46                         ` Thomas Gleixner
2010-04-28 23:33                           ` Xianghua Xiao
2010-04-29 12:22                             ` Thomas Gleixner
     [not found]                 ` <20100429151555.GM17639@pcarmody-desktop>
2010-04-29 15:52                   ` Xianghua Xiao
2010-04-30 10:02                     ` Thomas Gleixner
2010-04-30 14:04                       ` Xianghua Xiao

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=alpine.LFD.2.00.1004281833230.2951@localhost.localdomain \
    --to=tglx@linutronix.de \
    --cc=johnstul@us.ibm.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=xiaoxianghua@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®