* Re: locking/spinlock/debug: Remove spinlock lockup detection code
[not found] <20170220214816.F3A2B66122D@gitolite.kernel.org>
@ 2017-08-29 13:07 ` Geert Uytterhoeven
0 siblings, 0 replies; 4+ messages in thread
From: Geert Uytterhoeven @ 2017-08-29 13:07 UTC (permalink / raw)
To: stable
Cc: Linux Kernel Mailing List, ltsi-dev, Waiman Long,
Peter Zijlstra (Intel),
Ingo Molnar
Hi Greg,
On Mon, Feb 20, 2017 at 10:48 PM, Linux Kernel Mailing List
<linux-kernel@vger.kernel.org> wrote:
> Web: https://git.kernel.org/torvalds/c/bc88c10d7e6900916f5e1ba3829d66a9de92b633
> Commit: bc88c10d7e6900916f5e1ba3829d66a9de92b633
> Parent: f9af456a61ecfbef8233c5046a9e347c9b98ba05
> Refname: refs/heads/master
> Author: Waiman Long <longman@redhat.com>
> AuthorDate: Wed Feb 8 14:46:48 2017 -0500
> Committer: Ingo Molnar <mingo@kernel.org>
> CommitDate: Fri Feb 10 09:09:49 2017 +0100
>
> locking/spinlock/debug: Remove spinlock lockup detection code
>
> The current spinlock lockup detection code can sometimes produce false
> positives because of the unfairness of the locking algorithm itself.
>
> So the lockup detection code is now removed. Instead, we are relying
> on the NMI watchdog to detect potential lockup. We won't have lockup
> detection if the watchdog isn't running.
>
> The commented-out read-write lock lockup detection code are also
> removed.
>
> Signed-off-by: Waiman Long <longman@redhat.com>
> Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
> Cc: Andrew Morton <akpm@linux-foundation.org>
> Cc: Linus Torvalds <torvalds@linux-foundation.org>
> Cc: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
> Cc: Peter Zijlstra <peterz@infradead.org>
> Cc: Sasha Levin <sasha.levin@oracle.com>
> Cc: Thomas Gleixner <tglx@linutronix.de>
> Link: http://lkml.kernel.org/r/1486583208-11038-1-git-send-email-longman@redhat.com
> Signed-off-by: Ingo Molnar <mingo@kernel.org>
Can you please backport this commit to v4.9.x and/or ltsi?
It fixes the following issue in 4.9.40-ltsi on r8a7740/armadillo:
BUG: spinlock lockup suspected on CPU#0, swapper/1
lock: lcdc0_device+0xa0/0x258, .magic: 00000000, .owner:
<none>/-1, .owner_cpu: 0
CPU: 0 PID: 1 Comm: swapper Not tainted
4.9.40-ltsi-armadillo-00939-gfc8b396897fd0712 #36
Hardware name: Generic R8A7740 (Flattened Device Tree)
[<c010d068>] (unwind_backtrace) from [<c010ab54>] (show_stack+0x10/0x14)
[<c010ab54>] (show_stack) from [<c014fe9c>] (do_raw_spin_lock+0xd4/0x118)
[<c014fe9c>] (do_raw_spin_lock) from [<c0357000>]
(dev_pm_get_subsys_data+0x2c/0x9c)
[<c0357000>] (dev_pm_get_subsys_data) from [<c0360978>]
(genpd_add_device+0x38/0x1b4)
[<c0360978>] (genpd_add_device) from [<c0360b74>]
(of_genpd_add_device+0x38/0x50)
[<c0360b74>] (of_genpd_add_device) from [<c0818ef8>]
(board_staging_register_device+0xb0/0xe8)
[<c0818ef8>] (board_staging_register_device) from [<c0818f54>]
(board_staging_register_devices+0x24/0x2c)
[<c0818f54>] (board_staging_register_devices) from [<c0818f88>]
(runtime_board_check+0x2c/0x40)
[<c0818f88>] (runtime_board_check) from [<c01016dc>]
(do_one_initcall+0xac/0x154)
[<c01016dc>] (do_one_initcall) from [<c0800ce8>]
(kernel_init_freeable+0x104/0x1c4)
[<c0800ce8>] (kernel_init_freeable) from [<c0522150>]
(kernel_init+0x8/0x10c)
[<c0522150>] (kernel_init) from [<c0106db0>] (ret_from_fork+0x14/0x24)
Thanks!
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: locking/spinlock/debug: Remove spinlock lockup detection code
@ 2017-08-29 13:07 ` Geert Uytterhoeven
0 siblings, 0 replies; 4+ messages in thread
From: Geert Uytterhoeven @ 2017-08-29 13:07 UTC (permalink / raw)
To: stable
Cc: Linux Kernel Mailing List, ltsi-dev, Waiman Long,
Peter Zijlstra (Intel),
Ingo Molnar
Hi Greg,
On Mon, Feb 20, 2017 at 10:48 PM, Linux Kernel Mailing List
<linux-kernel@vger.kernel.org> wrote:
> Web: https://git.kernel.org/torvalds/c/bc88c10d7e6900916f5e1ba3829d66a9de92b633
> Commit: bc88c10d7e6900916f5e1ba3829d66a9de92b633
> Parent: f9af456a61ecfbef8233c5046a9e347c9b98ba05
> Refname: refs/heads/master
> Author: Waiman Long <longman@redhat.com>
> AuthorDate: Wed Feb 8 14:46:48 2017 -0500
> Committer: Ingo Molnar <mingo@kernel.org>
> CommitDate: Fri Feb 10 09:09:49 2017 +0100
>
> locking/spinlock/debug: Remove spinlock lockup detection code
>
> The current spinlock lockup detection code can sometimes produce false
> positives because of the unfairness of the locking algorithm itself.
>
> So the lockup detection code is now removed. Instead, we are relying
> on the NMI watchdog to detect potential lockup. We won't have lockup
> detection if the watchdog isn't running.
>
> The commented-out read-write lock lockup detection code are also
> removed.
>
> Signed-off-by: Waiman Long <longman@redhat.com>
> Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
> Cc: Andrew Morton <akpm@linux-foundation.org>
> Cc: Linus Torvalds <torvalds@linux-foundation.org>
> Cc: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
> Cc: Peter Zijlstra <peterz@infradead.org>
> Cc: Sasha Levin <sasha.levin@oracle.com>
> Cc: Thomas Gleixner <tglx@linutronix.de>
> Link: http://lkml.kernel.org/r/1486583208-11038-1-git-send-email-longman@redhat.com
> Signed-off-by: Ingo Molnar <mingo@kernel.org>
Can you please backport this commit to v4.9.x and/or ltsi?
It fixes the following issue in 4.9.40-ltsi on r8a7740/armadillo:
BUG: spinlock lockup suspected on CPU#0, swapper/1
lock: lcdc0_device+0xa0/0x258, .magic: 00000000, .owner:
<none>/-1, .owner_cpu: 0
CPU: 0 PID: 1 Comm: swapper Not tainted
4.9.40-ltsi-armadillo-00939-gfc8b396897fd0712 #36
Hardware name: Generic R8A7740 (Flattened Device Tree)
[<c010d068>] (unwind_backtrace) from [<c010ab54>] (show_stack+0x10/0x14)
[<c010ab54>] (show_stack) from [<c014fe9c>] (do_raw_spin_lock+0xd4/0x118)
[<c014fe9c>] (do_raw_spin_lock) from [<c0357000>]
(dev_pm_get_subsys_data+0x2c/0x9c)
[<c0357000>] (dev_pm_get_subsys_data) from [<c0360978>]
(genpd_add_device+0x38/0x1b4)
[<c0360978>] (genpd_add_device) from [<c0360b74>]
(of_genpd_add_device+0x38/0x50)
[<c0360b74>] (of_genpd_add_device) from [<c0818ef8>]
(board_staging_register_device+0xb0/0xe8)
[<c0818ef8>] (board_staging_register_device) from [<c0818f54>]
(board_staging_register_devices+0x24/0x2c)
[<c0818f54>] (board_staging_register_devices) from [<c0818f88>]
(runtime_board_check+0x2c/0x40)
[<c0818f88>] (runtime_board_check) from [<c01016dc>]
(do_one_initcall+0xac/0x154)
[<c01016dc>] (do_one_initcall) from [<c0800ce8>]
(kernel_init_freeable+0x104/0x1c4)
[<c0800ce8>] (kernel_init_freeable) from [<c0522150>]
(kernel_init+0x8/0x10c)
[<c0522150>] (kernel_init) from [<c0106db0>] (ret_from_fork+0x14/0x24)
Thanks!
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: locking/spinlock/debug: Remove spinlock lockup detection code
2017-08-29 13:07 ` Geert Uytterhoeven
@ 2017-08-31 6:03 ` Greg KH
-1 siblings, 0 replies; 4+ messages in thread
From: Greg KH @ 2017-08-31 6:03 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: stable, Linux Kernel Mailing List, ltsi-dev, Waiman Long,
Peter Zijlstra (Intel),
Ingo Molnar
On Tue, Aug 29, 2017 at 03:07:08PM +0200, Geert Uytterhoeven wrote:
> Hi Greg,
>
> On Mon, Feb 20, 2017 at 10:48 PM, Linux Kernel Mailing List
> <linux-kernel@vger.kernel.org> wrote:
> > Web: https://git.kernel.org/torvalds/c/bc88c10d7e6900916f5e1ba3829d66a9de92b633
> > Commit: bc88c10d7e6900916f5e1ba3829d66a9de92b633
> > Parent: f9af456a61ecfbef8233c5046a9e347c9b98ba05
> > Refname: refs/heads/master
> > Author: Waiman Long <longman@redhat.com>
> > AuthorDate: Wed Feb 8 14:46:48 2017 -0500
> > Committer: Ingo Molnar <mingo@kernel.org>
> > CommitDate: Fri Feb 10 09:09:49 2017 +0100
> >
> > locking/spinlock/debug: Remove spinlock lockup detection code
> >
> > The current spinlock lockup detection code can sometimes produce false
> > positives because of the unfairness of the locking algorithm itself.
> >
> > So the lockup detection code is now removed. Instead, we are relying
> > on the NMI watchdog to detect potential lockup. We won't have lockup
> > detection if the watchdog isn't running.
> >
> > The commented-out read-write lock lockup detection code are also
> > removed.
> >
> > Signed-off-by: Waiman Long <longman@redhat.com>
> > Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
> > Cc: Andrew Morton <akpm@linux-foundation.org>
> > Cc: Linus Torvalds <torvalds@linux-foundation.org>
> > Cc: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
> > Cc: Peter Zijlstra <peterz@infradead.org>
> > Cc: Sasha Levin <sasha.levin@oracle.com>
> > Cc: Thomas Gleixner <tglx@linutronix.de>
> > Link: http://lkml.kernel.org/r/1486583208-11038-1-git-send-email-longman@redhat.com
> > Signed-off-by: Ingo Molnar <mingo@kernel.org>
>
> Can you please backport this commit to v4.9.x and/or ltsi?
>
> It fixes the following issue in 4.9.40-ltsi on r8a7740/armadillo:
>
> BUG: spinlock lockup suspected on CPU#0, swapper/1
> lock: lcdc0_device+0xa0/0x258, .magic: 00000000, .owner:
> <none>/-1, .owner_cpu: 0
> CPU: 0 PID: 1 Comm: swapper Not tainted
> 4.9.40-ltsi-armadillo-00939-gfc8b396897fd0712 #36
> Hardware name: Generic R8A7740 (Flattened Device Tree)
> [<c010d068>] (unwind_backtrace) from [<c010ab54>] (show_stack+0x10/0x14)
> [<c010ab54>] (show_stack) from [<c014fe9c>] (do_raw_spin_lock+0xd4/0x118)
> [<c014fe9c>] (do_raw_spin_lock) from [<c0357000>]
> (dev_pm_get_subsys_data+0x2c/0x9c)
> [<c0357000>] (dev_pm_get_subsys_data) from [<c0360978>]
> (genpd_add_device+0x38/0x1b4)
> [<c0360978>] (genpd_add_device) from [<c0360b74>]
> (of_genpd_add_device+0x38/0x50)
> [<c0360b74>] (of_genpd_add_device) from [<c0818ef8>]
> (board_staging_register_device+0xb0/0xe8)
> [<c0818ef8>] (board_staging_register_device) from [<c0818f54>]
> (board_staging_register_devices+0x24/0x2c)
> [<c0818f54>] (board_staging_register_devices) from [<c0818f88>]
> (runtime_board_check+0x2c/0x40)
> [<c0818f88>] (runtime_board_check) from [<c01016dc>]
> (do_one_initcall+0xac/0x154)
> [<c01016dc>] (do_one_initcall) from [<c0800ce8>]
> (kernel_init_freeable+0x104/0x1c4)
> [<c0800ce8>] (kernel_init_freeable) from [<c0522150>]
> (kernel_init+0x8/0x10c)
> [<c0522150>] (kernel_init) from [<c0106db0>] (ret_from_fork+0x14/0x24)
>
Now queud up, thanks.
greg k-h
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: locking/spinlock/debug: Remove spinlock lockup detection code
@ 2017-08-31 6:03 ` Greg KH
0 siblings, 0 replies; 4+ messages in thread
From: Greg KH @ 2017-08-31 6:03 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: stable, Linux Kernel Mailing List, ltsi-dev, Waiman Long,
Peter Zijlstra (Intel),
Ingo Molnar
On Tue, Aug 29, 2017 at 03:07:08PM +0200, Geert Uytterhoeven wrote:
> Hi Greg,
>
> On Mon, Feb 20, 2017 at 10:48 PM, Linux Kernel Mailing List
> <linux-kernel@vger.kernel.org> wrote:
> > Web: https://git.kernel.org/torvalds/c/bc88c10d7e6900916f5e1ba3829d66a9de92b633
> > Commit: bc88c10d7e6900916f5e1ba3829d66a9de92b633
> > Parent: f9af456a61ecfbef8233c5046a9e347c9b98ba05
> > Refname: refs/heads/master
> > Author: Waiman Long <longman@redhat.com>
> > AuthorDate: Wed Feb 8 14:46:48 2017 -0500
> > Committer: Ingo Molnar <mingo@kernel.org>
> > CommitDate: Fri Feb 10 09:09:49 2017 +0100
> >
> > locking/spinlock/debug: Remove spinlock lockup detection code
> >
> > The current spinlock lockup detection code can sometimes produce false
> > positives because of the unfairness of the locking algorithm itself.
> >
> > So the lockup detection code is now removed. Instead, we are relying
> > on the NMI watchdog to detect potential lockup. We won't have lockup
> > detection if the watchdog isn't running.
> >
> > The commented-out read-write lock lockup detection code are also
> > removed.
> >
> > Signed-off-by: Waiman Long <longman@redhat.com>
> > Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
> > Cc: Andrew Morton <akpm@linux-foundation.org>
> > Cc: Linus Torvalds <torvalds@linux-foundation.org>
> > Cc: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
> > Cc: Peter Zijlstra <peterz@infradead.org>
> > Cc: Sasha Levin <sasha.levin@oracle.com>
> > Cc: Thomas Gleixner <tglx@linutronix.de>
> > Link: http://lkml.kernel.org/r/1486583208-11038-1-git-send-email-longman@redhat.com
> > Signed-off-by: Ingo Molnar <mingo@kernel.org>
>
> Can you please backport this commit to v4.9.x and/or ltsi?
>
> It fixes the following issue in 4.9.40-ltsi on r8a7740/armadillo:
>
> BUG: spinlock lockup suspected on CPU#0, swapper/1
> lock: lcdc0_device+0xa0/0x258, .magic: 00000000, .owner:
> <none>/-1, .owner_cpu: 0
> CPU: 0 PID: 1 Comm: swapper Not tainted
> 4.9.40-ltsi-armadillo-00939-gfc8b396897fd0712 #36
> Hardware name: Generic R8A7740 (Flattened Device Tree)
> [<c010d068>] (unwind_backtrace) from [<c010ab54>] (show_stack+0x10/0x14)
> [<c010ab54>] (show_stack) from [<c014fe9c>] (do_raw_spin_lock+0xd4/0x118)
> [<c014fe9c>] (do_raw_spin_lock) from [<c0357000>]
> (dev_pm_get_subsys_data+0x2c/0x9c)
> [<c0357000>] (dev_pm_get_subsys_data) from [<c0360978>]
> (genpd_add_device+0x38/0x1b4)
> [<c0360978>] (genpd_add_device) from [<c0360b74>]
> (of_genpd_add_device+0x38/0x50)
> [<c0360b74>] (of_genpd_add_device) from [<c0818ef8>]
> (board_staging_register_device+0xb0/0xe8)
> [<c0818ef8>] (board_staging_register_device) from [<c0818f54>]
> (board_staging_register_devices+0x24/0x2c)
> [<c0818f54>] (board_staging_register_devices) from [<c0818f88>]
> (runtime_board_check+0x2c/0x40)
> [<c0818f88>] (runtime_board_check) from [<c01016dc>]
> (do_one_initcall+0xac/0x154)
> [<c01016dc>] (do_one_initcall) from [<c0800ce8>]
> (kernel_init_freeable+0x104/0x1c4)
> [<c0800ce8>] (kernel_init_freeable) from [<c0522150>]
> (kernel_init+0x8/0x10c)
> [<c0522150>] (kernel_init) from [<c0106db0>] (ret_from_fork+0x14/0x24)
>
Now queud up, thanks.
greg k-h
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-08-31 6:02 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20170220214816.F3A2B66122D@gitolite.kernel.org>
2017-08-29 13:07 ` locking/spinlock/debug: Remove spinlock lockup detection code Geert Uytterhoeven
2017-08-29 13:07 ` Geert Uytterhoeven
2017-08-31 6:03 ` Greg KH
2017-08-31 6:03 ` Greg KH
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.