Re: [PATCH 2/2] tools/virtio: Add hints when module is not installed
From: Jason Wang <jasowang@redhat.com>
Date: 2023-10-08 04:36:36
Also in:
kvm, lkml, virtualization
On Tue, Sep 26, 2023 at 1:00 PM [off-list ref] wrote:
quoted hunk ↗ jump to hunk
From: Liming Wu <redacted> Need to insmod vhost_test.ko before run virtio_test. Give some hints to users. Signed-off-by: Liming Wu <redacted> --- tools/virtio/virtio_test.c | 4 ++++ 1 file changed, 4 insertions(+)diff --git a/tools/virtio/virtio_test.c b/tools/virtio/virtio_test.c index 028f54e6854a..ce2c4d93d735 100644 --- a/tools/virtio/virtio_test.c +++ b/tools/virtio/virtio_test.c@@ -135,6 +135,10 @@ static void vdev_info_init(struct vdev_info* dev, unsigned long long features) dev->buf = malloc(dev->buf_size); assert(dev->buf); dev->control = open("/dev/vhost-test", O_RDWR); + + if (dev->control < 0) + fprintf(stderr, "Install vhost_test module" \ + "(./vhost_test/vhost_test.ko) firstly\n");
There should be many other reasons to fail for open(). Let's use strerror()? Thanks
assert(dev->control >= 0);
r = ioctl(dev->control, VHOST_SET_OWNER, NULL);
assert(r >= 0);
--
2.34.1