[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PULL 00/47] virtio, pc: fixes and features

From: Peter Maydell
Subject: Re: [Qemu-devel] [PULL 00/47] virtio, pc: fixes and features
Date: Tue, 1 Nov 2016 15:22:01 +0000

On 30 October 2016 at 21:23, Michael S. Tsirkin <address@hidden> wrote:
> The following changes since commit 5b2ecabaeabc17f032197246c4846b9ba95ba8a6:
>   Merge remote-tracking branch 'remotes/kraxel/tags/pull-ui-20161028-1' into 
> staging (2016-10-28 17:59:04 +0100)
> are available in the git repository at:
>   git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_upstream
> for you to fetch changes up to f082ec0225bd15c71e0b4697d2df3af7bad65d7f:
>   acpi: fix assert failure caused by commit 35c5a52d (2016-10-30 20:06:25 
> +0200)
> ----------------------------------------------------------------
> virtio, pc: fixes and features
> nvdimm hotplug support
> virtio migration and ioeventfd rework
> virtio crypto device
> ipmi fixes
> Signed-off-by: Michael S. Tsirkin <address@hidden>

Hi; this fails to build on OSX with format string issues:

error: format specifies type 'unsign
ed short' but the argument has type 'uint32_t' (aka 'unsigned int')
                   vcrypto->max_queues, VIRTIO_QUEUE_MAX);
/Users/pm215/src/qemu-for-merges/include/qapi/error.h:163:35: note:
expanded from macro 'error_setg'
                        (fmt), ## __VA_ARGS__)

Fun fact: in struct vhost_dev, max_queues is a uint64_t;
in struct VirtIONet it is a uint16_t; and in VirtIOCrypto
it is a uint32_t...

-- PMM

reply via email to

[Prev in Thread] Current Thread [Next in Thread]