public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@linux-foundation.org>
To: Jiri Slaby <jirislaby@gmail.com>
Cc: linux-kernel@vger.kernel.org, Greg KH <gregkh@suse.de>,
	Kay Sievers <kay.sievers@suse.de>,
	Andi Kleen <andi@firstfloor.org>
Subject: Re: oops in uevent_helper [was: mmotm 2010-01-13-12-17 uploaded]
Date: Fri, 22 Jan 2010 15:52:37 -0800	[thread overview]
Message-ID: <20100122155237.e93a1c55.akpm@linux-foundation.org> (raw)
In-Reply-To: <4B50FC78.9010107@gmail.com>

On Sat, 16 Jan 2010 00:38:32 +0100
Jiri Slaby <jirislaby@gmail.com> wrote:

> (fixed subject)
> 
> On 01/15/2010 08:33 PM, Jiri Slaby wrote:
> > On 01/13/2010 09:17 PM, akpm@linux-foundation.org wrote:
> >> The mm-of-the-moment snapshot 2010-01-13-12-17 has been uploaded to
> > 
> > Hi, it crashes on my machine while booting up. It is a regression
> > against 2010-01-06-14-34. Doesn't it ring a bell by a chance?
> 
> Well, memcpying to something like this:
> char *uevent_helper = CONFIG_UEVENT_HELPER_PATH;
> doesn't sound like a good idea :).
> 
> And it's racy with sysctl path anyway.
> 
> Looks like added by:
> sysctl-convert-hotplug-helper-string-to-proc_rcu_string.patch
> 
> Andi, did you forget to change uevent_helper_store?

Jiri did you forget to Cc Andi? ;)

> > BUG: unable to handle kernel paging request at ffffffff816efe8e
> > IP: [<ffffffff811acf60>] memcpy_c+0x10/0x20
> > PGD 1807067 PUD 180b063 PMD 80000000016001e1
> > Oops: 0003 [#1] SMP
> > last sysfs file: /sys/kernel/uevent_helper
> > CPU 0
> > Pid: 957, comm: boot.udev Not tainted 2.6.33-rc4-mm1_64 #928 To be
> > filled by O.E.M./To Be Filled By O.E.M.
> > RIP: 0010:[<ffffffff811acf60>]  [<ffffffff811acf60>] memcpy_c+0x10/0x20
> > RSP: 0018:ffff8801c3127e80  EFLAGS: 00010202
> > RAX: ffffffff816efe8e RBX: 0000000000000001 RCX: 0000000000000001
> > RDX: 0000000000000001 RSI: ffff8801c30b9000 RDI: ffffffff816efe8e
> > RBP: ffff8801c3127e98 R08: ffffffff81058620 R09: 00000000000c2c7e
> > R10: 0000000000000001 R11: 0000000000000246 R12: ffff8801c3127f48
> > R13: ffff8801cbc26688 R14: ffffffff81829d10 R15: ffff8801cbc903b8
> > FS:  00007fe572b106f0(0000) GS:ffff880028200000(0000) knlGS:0000000000000000
> > CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> > CR2: ffffffff816efe8e CR3: 00000001c30f2000 CR4: 00000000000006f0
> > DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
> > DR3: 0000000000000000 DR6: 00000000ffff0ff0 DR7: 0000000000000400
> > Process boot.udev (pid: 957, threadinfo ffff8801c3126000, task
> > ffff8801cbcdbf90)
> > Stack:
> >  ffffffff8105865f ffff8801c3127ea8 ffff8801c4a3faa0 ffff8801c3127ea8
> > <0> ffffffff811a51b7 ffff8801c3127ef8 ffffffff811224ec 0000000000000001
> > <0> ffff8801c4a3fa80 ffff8801c3127ee8 0000000000000001 00007fe572b4c000
> > Call Trace:
> >  [<ffffffff8105865f>] ? uevent_helper_store+0x3f/0x80
> >  [<ffffffff811a51b7>] kobj_attr_store+0x17/0x20
> >  [<ffffffff811224ec>] sysfs_write_file+0x9c/0xf0
> >  [<ffffffff810c9ba8>] vfs_write+0xc8/0x190
> >  [<ffffffff810ca4ec>] sys_write+0x4c/0x80
> >  [<ffffffff81002e6b>] system_call_fastpath+0x16/0x1b
> > Code: 00 48 3b 42 20 73 07 48 8b 50 f9 31 c0 c3 31 d2 48 c7 c0 f2 ff ff
> > ff c3 90 90 90 48 89 f8 89 d1 c1 e9 03 83 e2 07 f3 48 a5 89 d1 <f3> a4
> > c3 66 66 66 66 2e 0f 1f 84 00 00 00 00 00 eb de f8 89 d1
> > RIP  [<ffffffff811acf60>] memcpy_c+0x10/0x20
> >  RSP <ffff8801c3127e80>
> > CR2: ffffffff816efe8e
> > ---[ end trace 309d1f0b04265911 ]---
> 


  reply	other threads:[~2010-01-22 23:53 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-01-13 20:17 mmotm 2010-01-13-12-17 uploaded akpm
2010-01-13 22:16 ` mmotm 2010-01-13-12-17 - 'make oldconfig' dies Valdis.Kletnieks
2010-01-14 16:07 ` mmotm 2010-01-13-12-17 uploaded (cgroup) Randy Dunlap
2010-01-14 17:20   ` Ben Blum
2010-01-18  2:03     ` Randy Dunlap
2010-01-14 16:18 ` [PATCH -mmotm] tty.h: make function static Randy Dunlap
2010-01-15 19:33 ` mmotm 2010-01-13-12-17 uploaded Jiri Slaby
2010-01-15 23:38   ` oops in uevent_helper [was: mmotm 2010-01-13-12-17 uploaded] Jiri Slaby
2010-01-22 23:52     ` Andrew Morton [this message]
2010-02-10 19:54       ` Jiri Slaby
2010-02-11 20:57         ` Andi Kleen
2010-02-11 22:25           ` Andrew Morton
2010-02-12  2:39             ` Neil Horman
2010-02-12  3:10               ` Andrew Morton
2010-02-12  5:21             ` Andi Kleen
2010-02-12  5:27               ` Andrew Morton
2010-02-12 17:06                 ` Neil Horman
2010-02-14 17:32                   ` Jiri Slaby
2010-02-14 17:54                     ` Neil Horman
2010-02-15 20:12                   ` Paul E. McKenney
2010-02-15 21:42                     ` Neil Horman
2010-02-15 22:16                       ` Paul E. McKenney
2010-02-16  1:37                         ` Neil Horman

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=20100122155237.e93a1c55.akpm@linux-foundation.org \
    --to=akpm@linux-foundation.org \
    --cc=andi@firstfloor.org \
    --cc=gregkh@suse.de \
    --cc=jirislaby@gmail.com \
    --cc=kay.sievers@suse.de \
    --cc=linux-kernel@vger.kernel.org \
    /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