From: Steven Rostedt <rostedt@goodmis.org>
To: Randy Dunlap <randy.dunlap@oracle.com>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Frederic Weisbecker <fweisbec@gmail.com>,
Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Subject: Re: 2.6.33 GP fault only when built with tracing
Date: Thu, 18 Mar 2010 19:55:55 -0400 [thread overview]
Message-ID: <1268956555.758.18.camel@gandalf.stny.rr.com> (raw)
In-Reply-To: <4BA2B69D.3000309@oracle.com>
On Thu, 2010-03-18 at 16:26 -0700, Randy Dunlap wrote:
> I can build/boot 2.6.33 with CONFIG_TRACE/TRACING disabled successfully,
> but when I enable lots of tracing config options and then boot with
> ftrace=nop on the kernel command line, I see a GP fault when the parport &
> parport_pc modules are loading/initializing.
Do you see it without adding the "ftrace=nop"? The only thing that
should do is expand the ring buffer on boot up.
>
> It happens in drivers/parport/share.c::parport_register_device(), when that
> function calls try_module_get().
>
> If I comment out the trace_module_get() calls in include/linux/module.h,
> the kernel boots with no problems.
Interesting. Well, trace_module_get() is a TRACE_EVENT tracepoint. But
should be disabled here. It may be something to do with DEFINE_TRACE.
(added Mathieu to Cc since he wrote that code)
>
> [ 21.852829] general protection fault: 0000 [#1] SMP
> [ 21.856321] last sysfs file: /sys/module/parport/initstate
> [ 21.856321] CPU 0
> [ 21.856321] Pid: 2089, comm: modprobe Not tainted 2.6.33 #11 0HH807/OptiPlex GX620
> [ 21.856321] RIP: 0010:[<ffffffffa0437671>] [<ffffffffa0437671>] parport_register_device+0xe4/0x48c [parport]
> [ 21.856321] RSP: 0018:ffff8800765cba78 EFLAGS: 00010283
> [ 21.856321] RAX: ffff10007b04a3d0 RBX: ffff88007a6a5e30 RCX: 0000000000000000
> [ 21.856321] RDX: 0000000000000000 RSI: ffffffffa043d1de RDI: ffff88007a6a5e30
> [ 21.856321] RBP: ffff8800765cbad8 R08: 0000000000000000 R09: 0000000000000000
> [ 21.856321] R10: ffffffffa043dff8 R11: 0000000000000000 R12: ffffffffa043d1de
> [ 21.856321] R13: ffffffffa043d1de R14: ffffffffa045c940 R15: 0000000000000000
> [ 21.856321] FS: 00007f09cc3fb6f0(0000) GS:ffff880004a00000(0000) knlGS:0000000000000000
> [ 21.856321] CS: 0010 DS: 0000 ES: 0000 CR0: 000000008005003b
> [ 21.856321] CR2: 0000003fb5ad62c0 CR3: 00000000764f6000 CR4: 00000000000006f0
> [ 21.856321] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
> [ 21.856321] DR3: 0000000000000000 DR6: 00000000ffff0ff0 DR7: 0000000000000400
> [ 21.856321] Process modprobe (pid: 2089, threadinfo ffff8800765ca000, task ffff88007664a3d0)
> [ 21.856321] Stack:
> [ 21.856321] ffff8800765cbab8 0000000000000206 0000000000000000 ffffffff812abaf2
> [ 21.856321] <0> 0000000000000000 0000000000000000 ffffffffa043d1de 00000000ffffffff
> [ 21.856321] <0> ffff88007a6a5e30 ffffffffa043d1de 0000000000000000 0000000000000378
> [ 21.856321] Call Trace:
> [ 21.856321] [<ffffffff812abaf2>] ? do_raw_spin_unlock+0xd7/0xe7
> [ 21.856321] [<ffffffffa043b385>] parport_open+0x12d/0x14d [parport]
> [ 21.856321] [<ffffffffa043bccf>] parport_device_id+0x2e/0xa00 [parport]
> [ 21.856321] [<ffffffff8117ab73>] ? __slab_alloc+0x560/0x5f7
> [ 21.856321] [<ffffffffa043bb06>] ? parport_daisy_init+0x55f/0x665 [parport]
> [ 21.856321] [<ffffffffa043bb06>] ? parport_daisy_init+0x55f/0x665 [parport]
> [ 21.856321] [<ffffffffa043bb53>] parport_daisy_init+0x5ac/0x665 [parport]
> [ 21.856321] [<ffffffffa0436f6d>] parport_announce_port+0x1a/0x201 [parport]
> [ 21.856321] [<ffffffffa0456848>] parport_pc_probe_port+0x13a5/0x1478 [parport_pc]
> [ 21.856321] [<ffffffffa0456c36>] parport_pc_pnp_probe+0x31b/0x358 [parport_pc]
> [ 21.856321] [<ffffffff81323b1f>] pnp_device_probe+0x11a/0x15e
> [ 21.856321] [<ffffffff81376f15>] ? driver_sysfs_add+0x61/0x9b
> [ 21.856321] [<ffffffff81377223>] driver_probe_device+0x1bc/0x339
> [ 21.856321] [<ffffffff8137743e>] __driver_attach+0x9e/0xde
> [ 21.856321] [<ffffffff813773a0>] ? __driver_attach+0x0/0xde
> [ 21.856321] [<ffffffff81376038>] bus_for_each_dev+0x83/0xdb
> [ 21.856321] [<ffffffff81376e26>] driver_attach+0x25/0x2e
> [ 21.856321] [<ffffffff81376823>] bus_add_driver+0x14c/0x367
> [ 21.856321] [<ffffffff813778e7>] driver_register+0xf8/0x1b2
> [ 21.856321] [<ffffffff81323771>] pnp_register_driver+0x28/0x31
> [ 21.856321] [<ffffffffa046380b>] parport_pc_init+0x708/0x7fa [parport_pc]
> [ 21.856321] [<ffffffffa0463103>] ? parport_pc_init+0x0/0x7fa [parport_pc]
> [ 21.856321] [<ffffffff810020d6>] do_one_initcall+0x9c/0x223
> [ 21.856321] [<ffffffff810bcfd9>] sys_init_module+0x139/0x32b
> [ 21.856321] [<ffffffff8100c732>] system_call_fastpath+0x16/0x1b
> [ 21.856321] Code: 65 8b 14 25 d8 e3 00 00 41 83 3e 02 0f 84 80 00 00 00 48 ff 05 09 72 00 00 48 63 d2 49 8b 86 28 02 00 00 48 03 04 d5 b0 97 b9 81 <48> ff 00 49 8b 86 28 02 00 00 48 03 04 d5 b0 97 b9 81 48 8b 00
> [ 21.856321] RIP [<ffffffffa0437671>] parport_register_device+0xe4/0x48c [parport]
> [ 21.856321] RSP <ffff8800765cba78>
> [ 22.192206] ---[ end trace 892b5882bd1f8c3e ]---
>
>
> Full kernel boot log is attached.
>
> Is this perhaps already fixed after 2.6.33?
I've never seen it. Do you have a config you can send me. I can try it
out.
Thanks,
-- Steve
next prev parent reply other threads:[~2010-03-18 23:55 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-03-18 23:26 Randy Dunlap
2010-03-18 23:55 ` Steven Rostedt [this message]
2010-03-19 0:08 ` Randy Dunlap
2010-03-19 0:59 ` Mathieu Desnoyers
2010-03-19 18:22 ` Randy Dunlap
2010-03-19 18:46 ` Mathieu Desnoyers
2010-03-23 15:26 ` Randy Dunlap
2010-03-24 1:20 ` Mathieu Desnoyers
2010-03-24 1:42 ` Li Zefan
2010-03-24 20:21 ` Randy Dunlap
2010-03-24 20:31 ` Steven Rostedt
2010-03-20 0:12 ` Randy Dunlap
2010-03-19 0:01 ` Frederic Weisbecker
2010-03-19 2:15 ` Steven Rostedt
2010-03-19 16:10 ` Randy Dunlap
2010-03-24 2:57 ` [PATCH 1/3] tracing: Reduce overhead of module tracepoints Li Zefan
2010-03-24 2:58 ` [PATCH 2/3] tracing: Convert some signal events to DEFINE_TRACE Li Zefan
2010-03-24 3:07 ` Steven Rostedt
2010-03-24 3:17 ` Masami Hiramatsu
2010-04-02 19:03 ` [tip:tracing/core] " tip-bot for Li Zefan
2010-03-24 2:58 ` [PATCH 3/3] tracing: Update comments Li Zefan
2010-03-24 3:07 ` Steven Rostedt
2010-04-02 19:04 ` [tip:tracing/core] " tip-bot for Li Zefan
2010-03-24 3:05 ` [PATCH 1/3] tracing: Reduce overhead of module tracepoints Steven Rostedt
2010-03-24 10:24 ` Mathieu Desnoyers
2010-03-24 23:41 ` Randy Dunlap
2010-03-27 2:03 ` [tip:tracing/urgent] tracing: Remove side effect from module tracepoints that caused a GPF tip-bot for Li Zefan
2010-03-27 4:10 ` Mathieu Desnoyers
2010-03-27 4:23 ` Steven Rostedt
2010-04-02 19:04 ` [tip:tracing/core] " tip-bot for Li Zefan
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1268956555.758.18.camel@gandalf.stny.rr.com \
--to=rostedt@goodmis.org \
--cc=fweisbec@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mathieu.desnoyers@efficios.com \
--cc=randy.dunlap@oracle.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®