[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH] nvme: Set number of queues later in nvme_init()
From: |
Max Reitz |
Subject: |
Re: [Qemu-block] [PATCH] nvme: Set number of queues later in nvme_init() |
Date: |
Wed, 10 Jul 2019 23:07:33 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.0 |
On 10.07.19 16:57, Michal Privoznik wrote:
> When creating the admin queue in nvme_init() the variable that
> holds the number of queues created is modified before actual
> queue creation. This is a problem because if creating the queue
> fails then the variable is left in inconsistent state. This was
> actually observed when I tried to hotplug a nvme disk. The
> control got to nvme_file_open() which called nvme_init() which
> failed and thus nvme_close() was called which in turn called
> nvme_free_queue_pair() with queue being NULL. This lead to an
> instant crash:
>
> #0 0x000055d9507ec211 in nvme_free_queue_pair (bs=0x55d952ddb880, q=0x0)
> at block/nvme.c:164
> #1 0x000055d9507ee180 in nvme_close (bs=0x55d952ddb880) at block/nvme.c:729
> #2 0x000055d9507ee3d5 in nvme_file_open (bs=0x55d952ddb880,
> options=0x55d952bb1410, flags=147456, errp=0x7ffd8e19e200) at block/nvme.c:781
> #3 0x000055d9507629f3 in bdrv_open_driver (bs=0x55d952ddb880,
> drv=0x55d95109c1e0 <bdrv_nvme>, node_name=0x0, options=0x55d952bb1410,
> open_flags=147456, errp=0x7ffd8e19e310) at block.c:1291
> #4 0x000055d9507633d6 in bdrv_open_common (bs=0x55d952ddb880, file=0x0,
> options=0x55d952bb1410, errp=0x7ffd8e19e310) at block.c:1551
> #5 0x000055d950766881 in bdrv_open_inherit (filename=0x0, reference=0x0,
> options=0x55d952bb1410, flags=32768, parent=0x55d9538ce420,
> child_role=0x55d950eaade0 <child_file>, errp=0x7ffd8e19e510) at block.c:3063
> #6 0x000055d950765ae4 in bdrv_open_child_bs (filename=0x0,
> options=0x55d9541cdff0, bdref_key=0x55d950af33aa "file",
> parent=0x55d9538ce420, child_role=0x55d950eaade0 <child_file>,
> allow_none=true, errp=0x7ffd8e19e510) at block.c:2712
> #7 0x000055d950766633 in bdrv_open_inherit (filename=0x0, reference=0x0,
> options=0x55d9541cdff0, flags=0, parent=0x0, child_role=0x0,
> errp=0x7ffd8e19e908) at block.c:3011
> #8 0x000055d950766dba in bdrv_open (filename=0x0, reference=0x0,
> options=0x55d953d00390, flags=0, errp=0x7ffd8e19e908) at block.c:3156
> #9 0x000055d9507cb635 in blk_new_open (filename=0x0, reference=0x0,
> options=0x55d953d00390, flags=0, errp=0x7ffd8e19e908) at
> block/block-backend.c:389
> #10 0x000055d950465ec5 in blockdev_init (file=0x0, bs_opts=0x55d953d00390,
> errp=0x7ffd8e19e908) at blockdev.c:602
>
> Signed-off-by: Michal Privoznik <address@hidden>
> ---
> block/nvme.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Thanks, applied to my block branch:
https://git.xanclic.moe/XanClic/qemu/commits/branch/block
Max
signature.asc
Description: OpenPGP digital signature