Re: 2.6.0-test4-mm4 - USD disconnect oops

[Andrew Morton]

Greg KH <greg@xxxxxxxxx> wrote:
>
> On Tue, Sep 02, 2003 at 07:59:36PM -0400, John Stoffel wrote:
> > >>>>> "Greg" == Greg KH <greg@xxxxxxxxx> writes:
> > 
> > >> Here's the backtrace, my .config is at the end.  It's a PIII Xeon 2 x
> > >> 550mhz, Dell Precision 610 motherboard/system, 768mb of RAM.  The only
> > >> USB devices are the controllers and the CompactFlash reader, which
> > >> works great under 2.4.  
> > 
> > Greg> Does this happen on 2.6.0-test4?  (no -mm).
> > 
> > Well, I can now use the usb-storage device under 2.6.0-test4 without
> > any problems, but I just did a quick test.  So there's something in
> > -mm4 which is messing me and usb in general up.  I've made the
> > following changes though, so I should go back and check:
> > 
> > - upgrade to module-init-tools-0.9.13
> > - upgrade to hotplug-2003_08_05-1
> > 	     hotplug-base-2003_08_05-1
> > 
> > I'll see if I can figure out what changed in the -mm4 patch to cause
> > this problem.  Could it be the kobject patch Akpm posted?  It looks
> > like the oops I've gotten.
> 
> Try adding that patch and see if it helps.  It sure can't hurt as it
> fixes a real bug in the -mm tree :)
> 

Yes.  It's the same oops.


diff -puN lib/kobject.c~kobject-unlimited-name-lengths-use-after-free-fix lib/kobject.c
--- 25/lib/kobject.c~kobject-unlimited-name-lengths-use-after-free-fix	Tue Sep  2 14:43:47 2003
+++ 25-akpm/lib/kobject.c	Tue Sep  2 14:43:47 2003
@@ -445,13 +445,13 @@ void kobject_cleanup(struct kobject * ko
 	struct kset * s = kobj->kset;
 
 	pr_debug("kobject %s: cleaning up\n",kobject_name(kobj));
+	if (kobj->k_name != kobj->name)
+		kfree(kobj->k_name);
+	kobj->k_name = NULL;
 	if (t && t->release)
 		t->release(kobj);
 	if (s)
 		kset_put(s);
-	if (kobj->k_name != kobj->name)
-		kfree(kobj->k_name);
-	kobj->k_name = NULL;
 }
 
 /**

_

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/