Re: [Qemu-devel] [Qemu-trivial] [PATCH] vhost: fix incorrect print type

[Laurent Vivier <laurent@vivier.eu>]

Le 29/05/2019 à 15:13, Philippe Mathieu-Daudé a écrit :
> Cc'ing qemu-trivial
> 
> On 4/30/19 10:48 AM, Philippe Mathieu-Daudé wrote:
>> On 4/30/19 8:29 AM, Jie Wang wrote:
>>> fix incorrect print type in vhost_virtqueue_stop
>>>
>>> Signed-off-by: Jie Wang <wangjie88@huawei.com>
>>> ---
>>>  hw/virtio/vhost.c | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/hw/virtio/vhost.c b/hw/virtio/vhost.c
>>> index 7f61018f2a..286bb27c65 100644
>>> --- a/hw/virtio/vhost.c
>>> +++ b/hw/virtio/vhost.c
>>> @@ -1081,7 +1081,7 @@ static void vhost_virtqueue_stop(struct vhost_dev *dev,
>>>  
>>>      r = dev->vhost_ops->vhost_get_vring_base(dev, &state);
>>>      if (r < 0) {
>>> -        VHOST_OPS_DEBUG("vhost VQ %d ring restore failed: %d", idx, r);
>>> +        VHOST_OPS_DEBUG("vhost VQ %u ring restore failed: %d", idx, r);
>>
>> 'idx' is indeed unsigned.
>>
>> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
>>
>>>          /* Connection to the backend is broken, so let's sync internal
>>>           * last avail idx to the device used idx.
>>>           */
>>>
> 

Applied to my trivial-patches branch.

Thanks,
Laurent