1. Patchew
  2. QEMU
  3. [PATCH v2 for-7.2 0/4] libvhost-user: Add format attribute and fix format strings
  4. View patch

[PATCH v2 3/4] libvhost-user: Fix two more format strings

Stefan Weil via posted 4 patches 3 years, 3 months ago
[PATCH v2 3/4] libvhost-user: Fix two more format strings
Posted by Stefan Weil via 3 years, 3 months ago 
This fix is required for 32 bit host. The bug was detected by CI
for arm-linux, but is also relevant for i386-linux.

Reported-by: Stefan Hajnoczi <stefanha@gmail.com>
Signed-off-by: Stefan Weil <sw@weilnetz.de>
---
 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, &reg_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.30.2
Re: [PATCH for-7.2 v2 3/4] libvhost-user: Fix two more format strings
Posted by Stefan Weil via 3 years, 2 months ago 
Am 05.11.22 um 11:24 schrieb Stefan Weil:

> This fix is required for 32 bit host. The bug was detected by CI
> for arm-linux, but is also relevant for i386-linux.
>
> Reported-by: Stefan Hajnoczi <stefanha@gmail.com>
> Signed-off-by: Stefan Weil <sw@weilnetz.de>
> ---
>   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, &reg_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,


There is still no review for this patch.

I added "for-7.2" to the subject here in my answer. How can we get all 4 
patches of this series into the new release?

Stefan
Re: [PATCH for-7.2 v2 3/4] libvhost-user: Fix two more format strings
Posted by Stefan Weil via 3 years, 2 months ago 
Am 15.11.22 um 08:25 schrieb Stefan Weil:
> Am 05.11.22 um 11:24 schrieb Stefan Weil:
> 
>> This fix is required for 32 bit host. The bug was detected by CI
>> for arm-linux, but is also relevant for i386-linux.
>>
>> Reported-by: Stefan Hajnoczi <stefanha@gmail.com>
>> Signed-off-by: Stefan Weil <sw@weilnetz.de>
>> ---
>>   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, &reg_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,
> 
> 
> There is still no review for this patch.
> 
> I added "for-7.2" to the subject here in my answer. How can we get all 4 
> patches of this series into the new release?
> 
> Stefan

Ping? Those bug fixes are still missing in -rc2.

Stefan
Re: [PATCH for-7.2 v2 3/4] libvhost-user: Fix two more format strings
Posted by Stefan Weil via 3 years, 2 months ago 
Am 23.11.22 um 07:35 schrieb Stefan Weil:
> Am 15.11.22 um 08:25 schrieb Stefan Weil:
>> Am 05.11.22 um 11:24 schrieb Stefan Weil:
>>
>>> This fix is required for 32 bit host. The bug was detected by CI
>>> for arm-linux, but is also relevant for i386-linux.
>>>
>>> Reported-by: Stefan Hajnoczi <stefanha@gmail.com>
>>> Signed-off-by: Stefan Weil <sw@weilnetz.de>
>>> ---
>>>   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, &reg_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,
>>
>>
>> There is still no review for this patch.
>>
>> I added "for-7.2" to the subject here in my answer. How can we get all 
>> 4 patches of this series into the new release?
>>
>> Stefan
> 
> Ping? Those bug fixes are still missing in -rc2.
> 
> Stefan

Hello Michael,

I just noticed that MAINTAINERS has no entry for the files in 
subprojects/libvhost-user, so I did not cc you in my previous e-mails. 
Should that directory be added to the "vhost" section"?

Stefan
Re: [PATCH for-7.2 v2 3/4] libvhost-user: Fix two more format strings
Posted by Michael S. Tsirkin 3 years, 2 months ago 
On Wed, Nov 23, 2022 at 07:49:53AM +0100, Stefan Weil wrote:
> Am 23.11.22 um 07:35 schrieb Stefan Weil:
> > Am 15.11.22 um 08:25 schrieb Stefan Weil:
> > > Am 05.11.22 um 11:24 schrieb Stefan Weil:
> > > 
> > > > This fix is required for 32 bit host. The bug was detected by CI
> > > > for arm-linux, but is also relevant for i386-linux.
> > > > 
> > > > Reported-by: Stefan Hajnoczi <stefanha@gmail.com>
> > > > Signed-off-by: Stefan Weil <sw@weilnetz.de>
> > > > ---
> > > >   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, &reg_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,
> > > 
> > > 
> > > There is still no review for this patch.
> > > 
> > > I added "for-7.2" to the subject here in my answer. How can we get
> > > all 4 patches of this series into the new release?
> > > 
> > > Stefan
> > 
> > Ping? Those bug fixes are still missing in -rc2.
> > 
> > Stefan
> 
> Hello Michael,
> 
> I just noticed that MAINTAINERS has no entry for the files in
> subprojects/libvhost-user, so I did not cc you in my previous e-mails.
> Should that directory be added to the "vhost" section"?
> 
> Stefan

pls do
Re: [PATCH v2 3/4] libvhost-user: Fix two more format strings
Posted by Stefan Weil via 3 years, 3 months ago 
Am 05.11.22 um 11:24 schrieb Stefan Weil via:

> This fix is required for 32 bit host. The bug was detected by CI
> for arm-linux, but is also relevant for i386-linux.


s/host/hosts/

I won't send a v3 for that. Maybe it can be fixed when merging this patch.

Stefan

Patchew 2.1-devel-b67e4c2

© 2016 - 2026 Red Hat, Inc.