lkml.org 
[lkml]   [2006]   [May]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [BUG 2.6.17-git] kmem_cache_create: duplicate cache scsi_cmd_cache


On Fri, 12 May 2006, Linus Torvalds wrote:
>
> On Fri, 12 May 2006, Russell King wrote:
> >
> > From: Todd Blumer <todd@sdgsystems.com>
> > On a PXA27x handheld (iPAQ hx4700), when we eject a mounted SD memory
> > card, we get a kernel panic (kernel trying to clean up non-existent
> > device). One hack patch to avoid the panic is:
> >
> > --- fs/partitions/check.c 10 Apr 2006 22:57:27 -0000 1.15
> > +++ fs/partitions/check.c 4 May 2006 20:30:15 -0000
> > @@ -491,6 +491,7 @@
> > kfree(disk_name);
> > }
> > put_device(disk->driverfs_dev);
> > + disk->driverfs_dev = 0; /* HACK - what's the right solution? */
> > }
> > kobject_uevent(&disk->kobj, KOBJ_REMOVE);
> > kobject_del(&disk->kobj);
>
> Btw, on the face it of, I really think that this patch is correct
> regardless of any other issues.

.. and I suspect it also shows what the "other issues" are.

Shouldn't that KOBJ_REMOVE uevent happen _before_ we do all the freeing of
the backing dev object? That KOBJ_REMOVE thing actually seems to want to
report the pathname for the disk it removes. Preferably before the thing
is gone and can't be reported on..

Ie shouldn't the diff be something like this?

Linus
---
diff --git a/fs/partitions/check.c b/fs/partitions/check.c
index 45ae7dd..7ef1f09 100644
--- a/fs/partitions/check.c
+++ b/fs/partitions/check.c
@@ -533,6 +533,7 @@ void del_gendisk(struct gendisk *disk)

devfs_remove_disk(disk);

+ kobject_uevent(&disk->kobj, KOBJ_REMOVE);
if (disk->holder_dir)
kobject_unregister(disk->holder_dir);
if (disk->slave_dir)
@@ -545,7 +546,7 @@ void del_gendisk(struct gendisk *disk)
kfree(disk_name);
}
put_device(disk->driverfs_dev);
+ disk->driverfs_dev = NULL;
}
- kobject_uevent(&disk->kobj, KOBJ_REMOVE);
kobject_del(&disk->kobj);
}
-
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: 2009-11-18 23:46    [W:0.226 / U:0.080 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site