Message ID | 1625477143-18716-1-git-send-email-wangqing@vivo.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | block: nbd: fix order of cleaning up the queue and freeing the tagset | expand |
On 7/5/21 5:25 PM, Wang Qing wrote: > Must release the queue before freeing the tagset. > > Fixes: 1c99502fae35 ("loop: use blk_mq_alloc_disk and blk_cleanup_disk") > Reported-and-tested-by: syzbot+9ca43ff47167c0ee3466@syzkaller.appspotmail.com Did syzbot actually test the change? > Signed-off-by: Wang Qing <wangqing@vivo.com> > Signed-off-by: Guoqing Jiang <guoqing.jiang@cloud.ionos.com> I don't mind you sent it quickly, but pls remove my outdated mail account. Guoqing
> >On 7/5/21 5:25 PM, Wang Qing wrote: >> Must release the queue before freeing the tagset. >> >> Fixes: 1c99502fae35 ("loop: use blk_mq_alloc_disk and blk_cleanup_disk") >> Reported-and-tested-by: syzbot+9ca43ff47167c0ee3466@syzkaller.appspotmail.com > >Did syzbot actually test the change? Yes, I have initiated the test, and did not report this error again, but still reported other known errors, so I think the test passed. And this is an obvious problem, it needs to be fixed anyway. > >> Signed-off-by: Wang Qing <wangqing@vivo.com> >> Signed-off-by: Guoqing Jiang <guoqing.jiang@cloud.ionos.com> > >I don't mind you sent it quickly, but pls remove my outdated mail account. Sorry about this, you can re-initiate a patch if you don't mind, as if you still have one thing to modify. Qing > >Guoqing
On 7/5/21 3:25 AM, Wang Qing wrote:
> Must release the queue before freeing the tagset.
Applied, thanks.
diff --git a/drivers/block/nbd.c b/drivers/block/nbd.c index b7d6637..c383179 --- a/drivers/block/nbd.c +++ b/drivers/block/nbd.c @@ -239,8 +239,8 @@ static void nbd_dev_remove(struct nbd_device *nbd) if (disk) { del_gendisk(disk); - blk_mq_free_tag_set(&nbd->tag_set); blk_cleanup_disk(disk); + blk_mq_free_tag_set(&nbd->tag_set); } /*