mirror of
https://github.com/Motorhead1991/qemu.git
synced 2025-08-03 15:53:54 -06:00
contrib/vhost-user-blk: fix 32 bit build and enable
We were not building the vhost-user-blk server due to 32 bit compilation problems. The problem was due to format string types so fix that and then enable the build. Tweak the rule to follow the same rules as other vhost-user daemons. Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Message-Id: <20220321153037.3622127-12-alex.bennee@linaro.org> Message-Id: <20220524154056.2896913-2-alex.bennee@linaro.org> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com> Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com> Reviewed-by: Raphael Norwitz <raphael.norwitz@nutanix.com>
This commit is contained in:
parent
2fcd005ff0
commit
26ed501b99
3 changed files with 5 additions and 6 deletions
|
@ -1,5 +1,4 @@
|
|||
# FIXME: broken on 32-bit architectures
|
||||
executable('vhost-user-blk', files('vhost-user-blk.c'),
|
||||
dependencies: [qemuutil, vhost_user],
|
||||
build_by_default: false,
|
||||
build_by_default: targetos == 'linux',
|
||||
install: false)
|
||||
|
|
|
@ -146,7 +146,7 @@ vub_readv(VubReq *req, struct iovec *iov, uint32_t iovcnt)
|
|||
req->size = vub_iov_size(iov, iovcnt);
|
||||
rc = preadv(vdev_blk->blk_fd, iov, iovcnt, req->sector_num * 512);
|
||||
if (rc < 0) {
|
||||
fprintf(stderr, "%s, Sector %"PRIu64", Size %lu failed with %s\n",
|
||||
fprintf(stderr, "%s, Sector %"PRIu64", Size %zu failed with %s\n",
|
||||
vdev_blk->blk_name, req->sector_num, req->size,
|
||||
strerror(errno));
|
||||
return -1;
|
||||
|
@ -169,7 +169,7 @@ vub_writev(VubReq *req, struct iovec *iov, uint32_t iovcnt)
|
|||
req->size = vub_iov_size(iov, iovcnt);
|
||||
rc = pwritev(vdev_blk->blk_fd, iov, iovcnt, req->sector_num * 512);
|
||||
if (rc < 0) {
|
||||
fprintf(stderr, "%s, Sector %"PRIu64", Size %lu failed with %s\n",
|
||||
fprintf(stderr, "%s, Sector %"PRIu64", Size %zu failed with %s\n",
|
||||
vdev_blk->blk_name, req->sector_num, req->size,
|
||||
strerror(errno));
|
||||
return -1;
|
||||
|
@ -188,7 +188,7 @@ vub_discard_write_zeroes(VubReq *req, struct iovec *iov, uint32_t iovcnt,
|
|||
|
||||
size = vub_iov_size(iov, iovcnt);
|
||||
if (size != sizeof(*desc)) {
|
||||
fprintf(stderr, "Invalid size %ld, expect %ld\n", size, sizeof(*desc));
|
||||
fprintf(stderr, "Invalid size %zd, expect %zd\n", size, sizeof(*desc));
|
||||
return -1;
|
||||
}
|
||||
buf = g_new0(char, size);
|
||||
|
|
|
@ -1516,7 +1516,7 @@ have_vhost_user_blk_server = get_option('vhost_user_blk_server') \
|
|||
error_message: 'vhost_user_blk_server requires linux') \
|
||||
.require(have_vhost_user,
|
||||
error_message: 'vhost_user_blk_server requires vhost-user support') \
|
||||
.disable_auto_if(not have_system) \
|
||||
.disable_auto_if(not have_tools and not have_system) \
|
||||
.allowed()
|
||||
|
||||
if get_option('fuse').disabled() and get_option('fuse_lseek').enabled()
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue