Re: [PATCH] MIPS: Octeon: irq: Alloc desc before configuring IRQ
From: Thomas Bogendoerfer
Date: Mon Dec 14 2020 - 10:28:58 EST
On Fri, Nov 27, 2020 at 12:13:55PM +0100, Alexander A Sverdlin wrote:
> From: Alexander Sverdlin <alexander.sverdlin@xxxxxxxxx>
>
> Allocate the IRQ descriptors where necessary before configuring them via
> irq_set_chip_and_handler(). Fixes the following soft lockup:
>
> watchdog: BUG: soft lockup - CPU#5 stuck for 22s! [modprobe:72]
> Modules linked in:
> irq event stamp: 33288
> hardirqs last enabled at (33287): [<ffffffff8012e680>] restore_partial+0x74/0x150
> hardirqs last disabled at (33288): [<ffffffff8012e9e8>] handle_int+0x128/0x178
> softirqs last enabled at (33284): [<ffffffff80859c4c>] __do_softirq+0x5c4/0x6d0
> softirqs last disabled at (33279): [<ffffffff80164018>] irq_exit+0xe8/0xf0
> CPU: 5 PID: 72 Comm: modprobe Not tainted 4.19.80-... #1
> $ 0 : 0000000000000000 0000000000000001 0000000000000003 8000000002bdc640
> $ 4 : 0000000000000000 0000000000000000 0000000000000000 0000000000000000
> $ 8 : 0000000000000001 0000000000000001 0000000000000000 ffffffff803076cc
> $12 : 0000000000000000 0000000000000000 ffffffff817f0000 0000000008000000
> $16 : ffffffff80a96d10 ffffffff80a90000 8000000002c41780 8000000002c41788
> $20 : 0000000000000001 ffffffff8013b248 800000008ef28080 ffffffff80bb8700
> $24 : 0000000003bf0000 ffffffff802d0610
> $28 : 800000008ef20000 800000008ef23bd0 0000000000000006 ffffffff8020d6f8
> Hi : 0000000000000160
> Lo : 0000000000000014
> epc : ffffffff8020d72c smp_call_function_many+0x2f4/0x370
> ra : ffffffff8020d6f8 smp_call_function_many+0x2c0/0x370
> Status: 10008ce3 KX SX UX KERNEL EXL IE
> Cause : 40808000 (ExcCode 00)
> PrId : 000d900a (Cavium Octeon II)
> CPU: 5 PID: 72 Comm: modprobe Not tainted 4.19.80-... #1
> Stack : ffffffff80ab0000 00000051801c0da0 0000000010000ce0 5e70a8a65518aeac
> 5e70a8a65518aeac 0000000000000000 800000008e0cfb48 ffffffff81820000
> 800000008e0cfad4 00000000f0ce6f64 0000000000000001 0000000000000000
> ffffffff801ccfb8 0000000000000000 0000000000000000 ffffffff817f0000
> 800000008531d840 ffffffff80a90000 fffe000000000000 0000000000000000
> ffffffff80b20000 ffffffffffffffff ffffffff80bb3980 ffffffff80bb3980
> ffffffff80a90000 00000000fffffffe ffffffff8057a760 0000000000000028
> ffffffff80c50028 800000008ef20000 800000008e0cfb40 ffffffff80b20000
> ffffffff80835d6c 0000000000000000 800000008e0cfc78 5e70a8a65518aeac
> ffffffff80a9dbf7 ffffffff80835c2c ffffffff801357a4 ffffffff809bdd50
> ...
> Call Trace:
> [<ffffffff801357a4>] show_stack+0x9c/0x130
> [<ffffffff80835d6c>] dump_stack+0xdc/0x140
> [<ffffffff8023d490>] watchdog_timer_fn+0x3e8/0x478
> [<ffffffff801f43e4>] __hrtimer_run_queues+0x18c/0x6d8
> [<ffffffff801f507c>] hrtimer_interrupt+0x104/0x2e8
> [<ffffffff801391a8>] c0_compare_interrupt+0x60/0x90
> [<ffffffff801d0fcc>] __handle_irq_event_percpu+0xb4/0x4a0
> [<ffffffff801d13ec>] handle_irq_event_percpu+0x34/0x90
> [<ffffffff801d6b24>] handle_percpu_irq+0x9c/0xe0
> [<ffffffff801d01f4>] generic_handle_irq+0x34/0x50
> [<ffffffff80859678>] do_IRQ+0x18/0x28
> [<ffffffff80107548>] plat_irq_dispatch+0x90/0x128
> [<ffffffff8012ea2c>] handle_int+0x16c/0x178
> [<ffffffff8020d72c>] smp_call_function_many+0x2f4/0x370
> [<ffffffff8020d7e8>] smp_call_function+0x40/0xa0
> [<ffffffff8013bc1c>] flush_tlb_mm+0x44/0x140
> [<ffffffff802d50b0>] tlb_flush_mmu+0x38/0x90
> [<ffffffff802d5154>] arch_tlb_finish_mmu+0x4c/0x88
> [<ffffffff802d52bc>] tlb_finish_mmu+0x24/0x50
> [<ffffffff802e0c54>] exit_mmap+0x11c/0x1b8
> [<ffffffff80157bb4>] mmput+0x84/0x138
> [<ffffffff80160ad4>] do_exit+0x314/0xc88
> [<ffffffff801628e0>] do_group_exit+0x48/0xb0
> [<ffffffff80162958>] __wake_up_parent+0x0/0x18
>
> Signed-off-by: Alexander Sverdlin <alexander.sverdlin@xxxxxxxxx>
> ---
> arch/mips/cavium-octeon/octeon-irq.c | 15 +++++++++++++++
> 1 file changed, 15 insertions(+)
applied to mips-next.
Thomas.
--
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea. [ RFC1925, 2.3 ]