Re: [PATCH v3 for-7.2 3/6] libvhost-user: Fix two more format strings
On Sat, 26 Nov 2022 at 10:25, Stefan Weil wrote: > > This fix is required for 32 bit hosts. The bug was detected by CI > for arm-linux, but is also relevant for i386-linux. > > Reported-by: Stefan Hajnoczi > Signed-off-by: Stefan Weil > --- > subprojects/libvhost-user/libvhost-user.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) Reviewed-by: Stefan Hajnoczi > > diff --git a/subprojects/libvhost-user/libvhost-user.c > b/subprojects/libvhost-user/libvhost-user.c > index d67953a1c3..80f9952e71 100644 > --- a/subprojects/libvhost-user/libvhost-user.c > +++ b/subprojects/libvhost-user/libvhost-user.c > @@ -651,7 +651,8 @@ generate_faults(VuDev *dev) { > > if (ioctl(dev->postcopy_ufd, UFFDIO_REGISTER, _struct)) { > vu_panic(dev, "%s: Failed to userfault region %d " > - "@%" PRIx64 " + size:%zx offset: %zx: > (ufd=%d)%s\n", > + "@%" PRIx64 " + size:%" PRIx64 " offset: %" PRIx64 > + ": (ufd=%d)%s\n", > __func__, i, > dev_region->mmap_addr, > dev_region->size, dev_region->mmap_offset, > -- > 2.35.1 >
[PATCH v3 for-7.2 3/6] libvhost-user: Fix two more format strings
This fix is required for 32 bit hosts. The bug was detected by CI for arm-linux, but is also relevant for i386-linux. Reported-by: Stefan Hajnoczi Signed-off-by: Stefan Weil --- subprojects/libvhost-user/libvhost-user.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/subprojects/libvhost-user/libvhost-user.c b/subprojects/libvhost-user/libvhost-user.c index d67953a1c3..80f9952e71 100644 --- a/subprojects/libvhost-user/libvhost-user.c +++ b/subprojects/libvhost-user/libvhost-user.c @@ -651,7 +651,8 @@ generate_faults(VuDev *dev) { if (ioctl(dev->postcopy_ufd, UFFDIO_REGISTER, _struct)) { vu_panic(dev, "%s: Failed to userfault region %d " - "@%" PRIx64 " + size:%zx offset: %zx: (ufd=%d)%s\n", + "@%" PRIx64 " + size:%" PRIx64 " offset: %" PRIx64 + ": (ufd=%d)%s\n", __func__, i, dev_region->mmap_addr, dev_region->size, dev_region->mmap_offset, -- 2.35.1