Messages in this thread | | | Date | Tue, 8 Oct 2013 23:14:17 +0100 | From | Russell King - ARM Linux <> | Subject | Re: [xen] double fault: 0000 [#1] PREEMPT SMP DEBUG_PAGEALLOC |
| |
On Tue, Oct 08, 2013 at 08:17:42PM +0800, Fengguang Wu wrote: > I find the above debug messages very helpful in locating the buggy > driver. How about enabling it whenever CONFIG_DEBUG_KOBJECT_RELEASE is > enabled? Something like > > #ifdef CONFIG_DEBUG_KOBJECT_RELEASE > - pr_debug("kobject: '%s' (%p): %s, parent %p (delayed)\n", > + printk(KERN_INFO "kobject: '%s' (%p): %s, parent %p (delayed)\n", > kobject_name(kobj), kobj, __func__, kobj->parent); > > pr_debug() won't be displayed by default, and it depends on > CONFIG_DYNAMIC_DEBUG.
Please consider using pr_info() instead of printk(KERN_INFO - it's slightly less typing.
I can see that being a useful change while we have these problematical instances to track down, but I do wonder whether it'll make the thing too noisy. Does anyone have other opinions on this point? Linus? Greg?
| |