On Thu, 20 Mar 2014, Vince Weaver wrote:

> On Thu, 20 Mar 2014, Thomas Gleixner wrote:
> > 
> > Yeah, it causes the explosion in the debug object code. This should
> > have been:
> > 
> > +                       trace_printk("Tracking Object free: %s %p %pS\n",
> > +                                    descr->name, obj->object, obj->hint);
> 
> attached are the results with the corrected patch.

I'm a complete idiot. I was staring at oaddr and did not notice that
descr->name is the real culprit. Sorry. Delta patch below.

Thanks,

        tglx

Index: linux-2.6/lib/debugobjects.c
===================================================================
--- linux-2.6.orig/lib/debugobjects.c
+++ linux-2.6/lib/debugobjects.c
@@ -769,6 +769,7 @@ repeat:
                /* Now free them */
                hlist_for_each_entry_safe(obj, tmp, &freelist, node) {
                        hlist_del(&obj->node);
+                       descr = obj->descr;
                        trace_printk("Tracking Object free: %s %p %pS\n",
                                     descr->name, obj->object, obj->hint);
                        free_object(obj);


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

Reply via email to