Re: [PATCH RESEND 0/1] AHCI: Optimize interrupt processing

From: Christoph Hellwig
Date: Thu Oct 03 2013 - 07:07:09 EST


On Sat, Jul 20, 2013 at 09:48:28AM -0500, Mike Christie wrote:
> What about the attached only compile tested patch. The patch has the mq
> block code work like the non mq code for bio cleanups.
>
>

> blk-mq: blk-mq should free bios in pass through case
>
> For non mq calls, the block layer will free the bios when
> blk_finish_request is called.
e
> For mq calls, the blk mq code wants the caller to do this.
>
> This patch has the blk mq code work like the non mq code
> and has the block layer free the bios.
>
> Signed-off-by: Mike Christie <michaelc@xxxxxxxxxxx>

This patch breaks booting for me in the current blk multiqueue tree,
with an apparent double free of a bio when using virtio-blk in writeback
mode (cache=writeback or cache=none in qemu):

[ 15.253608] ------------[ cut here ]------------
[ 15.256422] kernel BUG at /work/hch/linux/fs/bio.c:498!
[ 15.256879] invalid opcode: 0000 [#1] SMP
[ 15.256879] Modules linked in:
[ 15.256879] CPU: 3 PID: 353 Comm: kblockd Not tainted 3.11.0+ #25
[ 15.256879] Hardware name: Bochs Bochs, BIOS Bochs 01/01/2007
[ 15.256879] task: ffff88007d75e0c0 ti: ffff88007d676000 task.ti: ffff88007d676000
[ 15.256879] RIP: 0010:[<ffffffff811b470a>] [<ffffffff811b470a>] bio_put+0x8a/0x90
[ 15.256879] RSP: 0018:ffff88007fd83b50 EFLAGS: 00010046
[ 15.256879] RAX: 0000000000000000 RBX: ffff88007d713080 RCX: 0000000000000035
[ 15.256879] RDX: 0000000000000002 RSI: ffff88007ad50338 RDI: ffff88007d713080
[ 15.256879] RBP: ffff88007fd83b60 R08: 7010000000000000 R09: 007ad50338080000
[ 15.256879] R10: ff672b1b7d38ce02 R11: 000000000000028b R12: 0000000000000000
[ 15.256879] R13: 0000000000000000 R14: ffff88007b4c36c0 R15: ffff88007b40d608
[ 15.256879] FS: 0000000000000000(0000) GS:ffff88007fd80000(0000) knlGS:0000000000000000
[ 15.256879] CS: 0010 DS: 0000 ES: 0000 CR0: 000000008005003b
[ 15.256879] CR2: 0000000000000138 CR3: 0000000002124000 CR4: 00000000000006e0
[ 15.256879] Stack:
[ 15.256879] ffff88007d713080 0000000000000000 ffff88007fd83b80 ffffffff811ae8a3
[ 15.256879] ffff88007fd83bf0 0000000000001000 ffff88007fd83b90 ffffffff811b3268
[ 15.256879] ffff88007fd83bc0 ffffffff816ac847 ffff88007b4c36c0 ffff88007fd99d00
[ 15.256879] Call Trace:
[ 15.256879] <IRQ>
[ 15.256879] [<ffffffff811ae8a3>] end_bio_bh_io_sync+0x33/0x50
[ 15.256879] [<ffffffff811b3268>] bio_endio+0x18/0x30
[ 15.256879] [<ffffffff816ac847>] blk_mq_complete_request+0x47/0xd0
[ 15.256879] [<ffffffff816ac8e9>] __blk_mq_end_io+0x19/0x20
[ 15.256879] [<ffffffff816ac958>] blk_mq_end_io+0x68/0xd0
[ 15.256879] [<ffffffff816a6162>] blk_flush_complete_seq+0xe2/0x370
[ 15.256879] [<ffffffff816a653b>] flush_end_io+0x11b/0x200
[ 15.256879] [<ffffffff816ac875>] blk_mq_complete_request+0x75/0xd0
[ 15.256879] [<ffffffff816ac8e9>] __blk_mq_end_io+0x19/0x20
[ 15.256879] [<ffffffff816ac958>] blk_mq_end_io+0x68/0xd0
[ 15.256879] [<ffffffff81844c2f>] virtblk_done+0xef/0x260
[ 15.256879] [<ffffffff81753cc0>] vring_interrupt+0x30/0x60
[ 15.256879] [<ffffffff81103724>] handle_irq_event_percpu+0x54/0x1f0
[ 15.256879] [<ffffffff81103903>] handle_irq_event+0x43/0x70
[ 15.256879] [<ffffffff8110609f>] handle_edge_irq+0x6f/0x120
[ 15.256879] [<ffffffff810445b8>] handle_irq+0x58/0x140
[ 15.256879] [<ffffffff81094bbf>] ? irq_enter+0x4f/0x90
[ 15.256879] [<ffffffff810440b5>] do_IRQ+0x55/0xd0
[ 15.256879] [<ffffffff81bd3972>] common_interrupt+0x72/0x72
[ 15.256879] [<ffffffff810c5135>] ? sched_clock_local+0x25/0xa0
[ 15.256879] [<ffffffff81094960>] ? __do_softirq+0xb0/0x250
[ 15.256879] [<ffffffff81094959>] ? __do_softirq+0xa9/0x250
[ 15.256879] [<ffffffff81094cae>] irq_exit+0xae/0xd0
[ 15.256879] [<ffffffff8106dcd5>] smp_apic_timer_interrupt+0x45/0x60
[ 15.256879] [<ffffffff81bdc772>] apic_timer_interrupt+0x72/0x80
[ 15.256879] <EOI>
[ 15.256879] [<ffffffff81bd3a33>] ? retint_restore_args+0x13/0x13
[ 15.256879] [<ffffffff81bd3502>] ? _raw_spin_unlock_irq+0x32/0x40
[ 15.256879] [<ffffffff81bd34fb>] ? _raw_spin_unlock_irq+0x2b/0x40
[ 15.256879] [<ffffffff810ac0c4>] rescuer_thread+0xe4/0x2f0
[ 15.256879] [<ffffffff810abfe0>] ? process_scheduled_works+0x40/0x40
[ 15.256879] [<ffffffff810b3916>] kthread+0xd6/0xe0
[ 15.256879] [<ffffffff81bd34fb>] ? _raw_spin_unlock_irq+0x2b/0x40
[ 15.256879] [<ffffffff810b3840>] ? __init_kthread_worker+0x70/0x70
[ 15.256879] [<ffffffff81bdbabc>] ret_from_fork+0x7c/0xb0
[ 15.256879] [<ffffffff810b3840>] ? __init_kthread_worker+0x70/0x70
[ 15.256879] Code: ff 41 8b 44 24 08 48 89 df 49 8b 74 24 10 48 29 c7 e8 cb 88 f8 ff 48 8b 5d f0 4c 8b 65 f8 c9 c3 90 48 89 df e8 b8 5c fc ff eb 9b <0f> 0b 0f 1f 40 00 55 48 89 e5 41 57 45 31 ff 41 56 41 55 41 54
[ 15.256879] RIP [<ffffffff811b470a>] bio_put+0x8a/0x90
[ 15.256879] RSP <ffff88007fd83b50>
[ 15.256879] ---[ end trace 1f201608bfddfca7 ]---
[ 15.256879] Kernel panic - not syncing: Fatal exception in interrupt
[ 15.256879] Shutting down cpus with NMI

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/