You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
If I run vhost-device-vsock with a vring queue size smaller than 1024 with QEMU's vhost-user-vsock-device (i.e., the mmio version, not the pci one) an error is printed on the vhost-device-vsock terminal:
The error comes from set_vring_num when QEMU's device side tries to set the vring queue size to 1024 but vhost-device-vsock has a max queue size less than that.
The error log "invalid parameters" does not convey enough details why there was this error whereas the error log should contain more details that the error was due to trying to set vring queue size more than the max queue size currently allocated in vhost-device-vsock. Only saying "invalid parameters" makes it very hard to debug why there was an error.
This issue is about making the error logs (or other logs as well?) more descriptive so that it's easier to understand why there was an error.
If I run vhost-device-vsock with a vring queue size smaller than 1024 with QEMU's vhost-user-vsock-device (i.e., the mmio version, not the pci one) an error is printed on the vhost-device-vsock terminal:
The commands to reproduce this are:
shell1:
./target/release/vhost-device-vsock --vm guest-cid=4,uds-path=/tmp/vm4.vsock,socket=/tmp/vhost4.socket
shell2:
./qemu-system-x86_64 -M microvm,memory-backend=mem0 -kernel ./bzImage -nographic -append "console=ttyS0 nokaslr" -initrd ./initramfs.cpio.gz -m 4G --enable-kvm -cpu host -object memory-backend-memfd,id=mem0,size=4G -chardev socket,id=char0,reconnect=0,path=/tmp/vhost4.socket -device vhost-user-vsock-device,chardev=char0
The error comes from set_vring_num when QEMU's device side tries to set the vring queue size to 1024 but vhost-device-vsock has a max queue size less than that.
The error log "invalid parameters" does not convey enough details why there was this error whereas the error log should contain more details that the error was due to trying to set vring queue size more than the max queue size currently allocated in vhost-device-vsock. Only saying "invalid parameters" makes it very hard to debug why there was an error.
This issue is about making the error logs (or other logs as well?) more descriptive so that it's easier to understand why there was an error.
See discussion around #679 (comment) for more context.
The text was updated successfully, but these errors were encountered: