From: Jens Axboe <axboe@kernel.dk>
To: zhe.he@windriver.com, justin@coraid.com,
linux-block@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] block: aoe: Fix kernel crash due to atomic sleep when exiting
Date: Thu, 8 Aug 2019 06:29:16 -0700 [thread overview]
Message-ID: <afd1ac3b-127b-d436-98ac-e4f2748eacb8@kernel.dk> (raw)
In-Reply-To: <1565233794-458496-1-git-send-email-zhe.he@windriver.com>
On 8/7/19 8:09 PM, zhe.he@windriver.com wrote:
> From: He Zhe <zhe.he@windriver.com>
>
> Since commit 3582dd291788 ("aoe: convert aoeblk to blk-mq"), aoedev_downdev
> has had the possibility of sleeping and causing the following crash.
>
> BUG: scheduling while atomic: rmmod/2242/0x00000003
> Modules linked in: aoe
> Preemption disabled at:
> [<ffffffffc01d95e5>] flush+0x95/0x4a0 [aoe]
> CPU: 7 PID: 2242 Comm: rmmod Tainted: G I 5.2.3 #1
> Hardware name: Intel Corporation S5520HC/S5520HC, BIOS S5500.86B.01.10.0025.030220091519 03/02/2009
> Call Trace:
> dump_stack+0x4f/0x6a
> ? flush+0x95/0x4a0 [aoe]
> __schedule_bug.cold+0x44/0x54
> __schedule+0x44f/0x680
> schedule+0x44/0xd0
> blk_mq_freeze_queue_wait+0x46/0xb0
> ? wait_woken+0x80/0x80
> blk_mq_freeze_queue+0x1b/0x20
> aoedev_downdev+0x111/0x160 [aoe]
> flush+0xff/0x4a0 [aoe]
> aoedev_exit+0x23/0x30 [aoe]
> aoe_exit+0x35/0x948 [aoe]
> __se_sys_delete_module+0x183/0x210
> __x64_sys_delete_module+0x16/0x20
> do_syscall_64+0x4d/0x130
> entry_SYSCALL_64_after_hwframe+0x44/0xa9
> RIP: 0033:0x7f24e0043b07
> Code: 73 01 c3 48 8b 0d 89 73 0b 00 f7 d8 64 89 01 48 83 c8 ff c3 66 2e 0f
> 1f 84 00 00 00 00 00 0f 1f 44 00 00 b8 b0 00 00 00 0f 05 <48> 3d 01 f0 ff
> ff 73 01 c3 48 8b 0d 59 73 0b 00 f7 d8 64 89 01 48
> RSP: 002b:00007ffe18f7f1e8 EFLAGS: 00000206 ORIG_RAX: 00000000000000b0
> RAX: ffffffffffffffda RBX: 0000000000000000 RCX: 00007f24e0043b07
> RDX: 000000000000000a RSI: 0000000000000800 RDI: 0000555c3ecf87c8
> RBP: 00007ffe18f7f1f0 R08: 0000000000000000 R09: 0000000000000000
> R10: 00007f24e00b4ac0 R11: 0000000000000206 R12: 00007ffe18f7f238
> R13: 00007ffe18f7f410 R14: 00007ffe18f80e73 R15: 0000555c3ecf8760
>
> This patch, handling in the same way of pass two, unlocks the locks and
> restart pass one after aoedev_downdev is done.
Applied, thanks.
--
Jens Axboe
prev parent reply other threads:[~2019-08-08 13:29 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-08-08 3:09 zhe.he
2019-08-08 13:29 ` Jens Axboe [this message]
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=afd1ac3b-127b-d436-98ac-e4f2748eacb8@kernel.dk \
--to=axboe@kernel.dk \
--cc=justin@coraid.com \
--cc=linux-block@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=zhe.he@windriver.com \
/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
Powered by JetHome