From: Thorsten Kranzkowski <dl8bcu@dl8bcu.de>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Oleg Nesterov <oleg@redhat.com>,
peterz@infradead.org, mingo@kernel.org,
linux-kernel@vger.kernel.org, linux-alpha@vger.kernel.org
Subject: Re: [PATCH 1/1] task_work: Add local_irq_enable() into task_work_run()
Date: Sat, 13 Oct 2012 13:06:10 +0000 [thread overview]
Message-ID: <20121013130610.GA962@ds20.borg.net> (raw)
In-Reply-To: <CA+55aFwXiu1znTkksgSyxa7v1-3cPWZL5dvth_8hQygSKE1utw@mail.gmail.com>
On Sat, Oct 13, 2012 at 10:09:36AM +0900, Linus Torvalds wrote:
> On Sat, Oct 13, 2012 at 1:03 AM, Oleg Nesterov <oleg@redhat.com> wrote:
> > arch/alpha and probably some other architectures call
> > do_notify_resume()->task_work_run() with irqs disabled.
>
> I'm going to ignore this patch because I *hope* it is unnecessary
> after the pull from Al that I just did.
>
> But if that turns out to be not the case, please holler. Torsten, you
> seem to be the one who reported this, can you check the current git
> tree?
I can confirm that 4d7127d (i.e. without Olegs patch) boots without problems.
There is still the warning below, but it was already present before the
breakage.
Thanks to everyone involved,
Thorsten
>
> Thanks,
>
> Linus
SMP starting up secondaries.
Brought up 2 CPUs
SMP: Total of 2 processors activated (1985.89 BogoMIPS).
xor: automatically using best checksumming function:
alpha prefetch: 2097.152 MB/sec
NET: Registered protocol family 16
------------[ cut here ]------------
WARNING: at /usr/src/linux.git/kernel/softirq.c:139 __local_bh_enable+0x8c/0xbc()
Modules linked in:
fffffc0000b83c80 fffffc0000b924b8 fffffc000032cc14 fffffc0000bc4078
0000000000000200 0000000000000000 0000000000000001 0000000000000000
fffffc000032d130 fffffc0000b7d980 fffffc0000c27160 0000000000000000
fffffc00003535c4 0000000000000000 fffffc000031a93c 0000000000000000
fffffc0000b84000 0000000000000000 fffffc00003157a0 fffffc0000b83dc0
0000000000000be0 0000000000000000 fffffc0001e060c0 0000000000000000
Trace:
[<fffffc000032cc14>] __local_bh_enable+0x8c/0xbc
[<fffffc000032d130>] irq_enter+0x58/0x78
[<fffffc00003535c4>] scheduler_ipi+0x50/0xec
[<fffffc000031a93c>] handle_ipi+0xb8/0x248
[<fffffc00003157a0>] do_entInt+0x6c/0x240
[<fffffc000032d1b8>] irq_exit+0x68/0x7c
[<fffffc00003156a0>] handle_irq+0xd4/0xf4
[<fffffc0000315864>] do_entInt+0x130/0x240
[<fffffc0000310e10>] ret_from_sys_call+0x0/0x10
[<fffffc000035ade4>] load_balance+0x1e4/0x770
[<fffffc0000312994>] cpu_idle+0x2c/0x6c
[<fffffc000037b274>] atomic_add.constprop.42+0x0/0x14
[<fffffc00003129a0>] cpu_idle+0x38/0x6c
[<fffffc0000901b70>] rest_init+0xc0/0xd4
[<fffffc000031001c>] _stext+0x1c/0x20
---[ end trace 40b31a85666b44ea ]---
PCI host bridge to bus 0000:00
pci_bus 0000:00: root bus resource [io 0x0000-0x1ffffff]
pci_bus 0000:00: root bus resource [mem 0x00000000-0x3fffffff]
pci_bus 0000:00: No busn resource found for root bus, will use [bus 00-ff]
--
| Thorsten Kranzkowski Internet: dl8bcu@dl8bcu.de |
| Mobile: ++49 170 1876134 Snail: Kiebitzstr. 14, 49324 Melle, Germany |
| Ampr: dl8bcu@db0lj.#rpl.deu.eu, dl8bcu@marvin.dl8bcu.ampr.org [44.130.8.19] |
next prev parent reply other threads:[~2012-10-13 13:06 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-06 20:47 [regression] boot failure on alpha, bisected Thorsten Kranzkowski
2012-10-07 16:55 ` Oleg Nesterov
2012-10-07 17:08 ` Al Viro
2012-10-07 17:33 ` Oleg Nesterov
2012-10-07 19:39 ` Al Viro
2012-10-08 14:14 ` Dialup Jon Norstog
2012-10-08 14:14 ` Dialup Jon Norstog
2012-10-08 18:59 ` Oleg Nesterov
2012-10-07 17:13 ` Oleg Nesterov
2012-10-07 18:04 ` Thorsten Kranzkowski
2012-10-07 19:16 ` Oleg Nesterov
2012-10-07 19:41 ` Thorsten Kranzkowski
2012-10-08 18:59 ` Geert Uytterhoeven
2012-10-08 19:10 ` Oleg Nesterov
2012-10-12 16:03 ` [PATCH 0/1] (Was: [regression] boot failure on alpha, bisected) Oleg Nesterov
2012-10-12 16:03 ` [PATCH 1/1] task_work: Add local_irq_enable() into task_work_run() Oleg Nesterov
2012-10-13 1:09 ` Linus Torvalds
2012-10-13 1:48 ` Al Viro
2012-10-13 9:59 ` Michael Cree
2012-10-13 15:39 ` Al Viro
2012-10-13 13:06 ` Thorsten Kranzkowski [this message]
2012-10-12 22:18 ` [PATCH 0/1] (Was: [regression] boot failure on alpha, bisected) Al Viro
2012-10-14 18:35 ` Oleg Nesterov
2012-10-14 18:42 ` Oleg Nesterov
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=20121013130610.GA962@ds20.borg.net \
--to=dl8bcu@dl8bcu.de \
--cc=linux-alpha@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@kernel.org \
--cc=oleg@redhat.com \
--cc=peterz@infradead.org \
--cc=torvalds@linux-foundation.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.