Re: [PATCH v13 0/8] hmp,qmp: Add commands to introspect virtio devices

[Jonah Palmer <jonah.palmer@oracle.com>]

[-- Attachment #1: Type: text/plain, Size: 1891 bytes --]

My apologies for not notifying earlier but I have been looking into these errors.
Hopefully should be able to get a fix sent out within the week.

Thanks

Jonah

On 3/7/22 17:46, Michael S. Tsirkin wrote:
> On Mon, Mar 07, 2022 at 08:08:33AM -0500, Jonah Palmer wrote:
>> This series introduces new QMP/HMP commands to dump the status of a
>> virtio device at different levels.
>
> Fails to build on the build-system-centos job:
>
> libqemu-ppc64-softmmu.fa.p/hw_virtio_virtio.c.o: In function
> `qmp_decode_features':
> /builds/qemu-project/qemu/build/../hw/virtio/virtio.c:4155: undefined
> reference to `gpu_map'
> /builds/qemu-project/qemu/build/../hw/virtio/virtio.c:4155: undefined
> reference to `gpu_map'
> collect2: error: ld returned 1 exit status
>
> https://urldefense.com/v3/__https://gitlab.com/qemu-project/qemu/-/jobs/2172339948__;!!ACWV5N9M2RV99hQ!ZPeXmrNXRd6OoQ9bNTk4ZQc7M6z034ydisUDv08HJw6UfvaH4RJLGTDYM-LvRcOTaQ$  
>
>
> Also fails on cross-win64-system:
>
> https://urldefense.com/v3/__https://gitlab.com/qemu-project/qemu/-/jobs/2172339938__;!!ACWV5N9M2RV99hQ!ZPeXmrNXRd6OoQ9bNTk4ZQc7M6z034ydisUDv08HJw6UfvaH4RJLGTDYM-J3npED0A$  
>
> ../hw/virtio/virtio.c: In function 'qmp_x_query_virtio_vhost_queue_status':
> ../hw/virtio/virtio.c:4358:30: error: cast from pointer to integer of
> different size [-Werror=pointer-to-int-cast]
> 4358 | status->desc = (uint64_t)(unsigned long)hdev->vqs[queue].desc;
> | ^
> ../hw/virtio/virtio.c:4359:31: error: cast from pointer to integer of
> different size [-Werror=pointer-to-int-cast]
> 4359 | status->avail = (uint64_t)(unsigned long)hdev->vqs[queue].avail;
> | ^
> ../hw/virtio/virtio.c:4360:30: error: cast from pointer to integer of
> different size [-Werror=pointer-to-int-cast]
> 4360 | status->used = (uint64_t)(unsigned long)hdev->vqs[queue].used;
> | ^
> cc1: all warnings being treated as errors
>
> Reported-by: PMM
>

[-- Attachment #2: Type: text/html, Size: 2758 bytes --]