lkml.org 
[lkml]   [2007]   [Mar]   [5]   [last100]   RSS Feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
/
DateMon, 5 Mar 2007 13:41:53 +0100
FromCornelia Huck <>
SubjectRe: [PATCH] Fix 2.6.21 rfcomm lockups (2.6.21 regression)
On Sun, 04 Mar 2007 13:26:49 -0500,
Mark Lord <lkml@rtr.ca> wrote:

> The simplest fix for this bug is to prevent sysfs_move_dir()
> from self-deadlocking when (old_parent == new_parent).
> 
> This patch prevents total system lockup when using rfcomm devices.
> 
> Signed-off-by:  Mark Lord <mlord@pobox.com>
> ---
> --- 2.6.21/fs/sysfs/dir.c	2007-03-04 13:19:00.000000000 -0500
> +++ linux/fs/sysfs/dir.c	2007-03-04 13:20:45.000000000 -0500
> @@ -431,6 +431,8 @@
>  	new_parent_dentry = new_parent ?
>  		new_parent->dentry : sysfs_mount->mnt_sb->s_root;
> 
> +	if (old_parent_dentry->d_inode == new_parent_dentry->d_inode)
> +		return 0;	/* nothing to move */
>  again:
>  	mutex_lock(&old_parent_dentry->d_inode->i_mutex);
>  	if (!mutex_trylock(&new_parent_dentry->d_inode->i_mutex)) {

Hm, never thought that someone might call the moving functions with
old_parent == new_parent, sorry about that. It should be safe to
return success here, we add only a bit of overhead with some driver
model juggling and emitting a not needed uevent.

Acked-by: Cornelia Huck <cornelia.huck@de.ibm.com>
-
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/

\
 
 \ /
  Last update: 2007-03-05 13:43    [from the cache]
©2003-2008