Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

vdpa/virtio: fix blk crash at eal_intr_thread_main #44

Merged
merged 1 commit into from
Nov 21, 2023

Conversation

yajwu
Copy link
Collaborator

@yajwu yajwu commented Nov 15, 2023

Shutdown multiple blk device, dpdk crash:

VHOST_CONFIG: (/tmp/sock-blk-10) vhost peer closed
PANIC in eal_intr_thread_main():
Error adding fd 589 epoll_ctl, Bad file descriptor
5: [/lib64/libc.so.6(clone+0x6d) [0x7ffff5a8cead]]
4: [/lib64/libpthread.so.0(+0x7dd5) [0x7ffff5d63dd5]]
3: [/root/dpdk-vhost-vfe/build/app/dpdk-vfe-vdpa() [0x137a711]]
2: [/root/dpdk-vhost-vfe/build/app/dpdk-vfe-vdpa(__rte_panic+0xba) [0x5ab12d]]
1: [/root/dpdk-vhost-vfe/build/app/dpdk-vfe-vdpa(rte_dump_stack+0x1a) [0x1377faa]]

kickfd/callfd is closed in vhost_user_get_vring_base, virtio_vdpa_virtq_doorbell_relay_disable should be called in or before virtio_vdpa_dev_close.

Fixes: 92535c4 ("vdpa/virtio: Fix wrong set vring base(used index)")
Fixes: 4f778ec ("vdpa/virtio: fix mem hotplug wrong queue disable")

RM: 3669299

Shutdown multiple blk device, dpdk crash:

	VHOST_CONFIG: (/tmp/sock-blk-10) vhost peer closed
	PANIC in eal_intr_thread_main():
	Error adding fd 589 epoll_ctl, Bad file descriptor
	5: [/lib64/libc.so.6(clone+0x6d) [0x7ffff5a8cead]]
	4: [/lib64/libpthread.so.0(+0x7dd5) [0x7ffff5d63dd5]]
	3: [/root/dpdk-vhost-vfe/build/app/dpdk-vfe-vdpa() [0x137a711]]
	2: [/root/dpdk-vhost-vfe/build/app/dpdk-vfe-vdpa(__rte_panic+0xba) [0x5ab12d]]
	1: [/root/dpdk-vhost-vfe/build/app/dpdk-vfe-vdpa(rte_dump_stack+0x1a) [0x1377faa]]

kickfd/callfd is closed in vhost_user_get_vring_base, virtio_vdpa_virtq_doorbell_relay_disable
should be called in or before virtio_vdpa_dev_close.

Fixes: 92535c4 ("vdpa/virtio: Fix wrong set vring base(used index)")
Fixes: 4f778ec ("vdpa/virtio: fix mem hotplug wrong queue disable")

RM: 3669299
Signed-off-by: Yajun Wu <yajunw@nvidia.com>
@kailiangz1 kailiangz1 merged commit acca81a into Mellanox:main Nov 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants