[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/6] libqemustub: add more stubs for qemu-char
From: |
Andrew Jones |
Subject: |
Re: [Qemu-devel] [PATCH 2/6] libqemustub: add more stubs for qemu-char |
Date: |
Thu, 19 Jun 2014 18:06:28 +0200 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Thu, Jun 19, 2014 at 06:07:40PM +0300, Nikolay Nikolaev wrote:
> Additional stubs:
> - chr_baum_init
> - qemu_chr_open_spice_vmc
> - qemu_chr_open_spice_port
>
> Signed-off-by: Nikolay Nikolaev <address@hidden>
> ---
> stubs/Makefile.objs | 2 ++
> stubs/chr-baum-init.c | 7 +++++++
> stubs/qemu-chr-open-spice.c | 12 ++++++++++++
> 3 files changed, 21 insertions(+)
> create mode 100644 stubs/chr-baum-init.c
> create mode 100644 stubs/qemu-chr-open-spice.c
>
> diff --git a/stubs/Makefile.objs b/stubs/Makefile.objs
> index 997d68d..2312076 100644
> --- a/stubs/Makefile.objs
> +++ b/stubs/Makefile.objs
> @@ -1,5 +1,6 @@
> stub-obj-y += arch-query-cpu-def.o
> stub-obj-y += bdrv-commit-all.o
> +stub-obj-y += chr-baum-init.o
> stub-obj-y += chr-msmouse.o
> stub-obj-y += clock-warp.o
> stub-obj-y += cpu-get-clock.o
> @@ -24,6 +25,7 @@ stub-obj-y += mon-set-error.o
> stub-obj-y += monitor-init.o
> stub-obj-y += notify-event.o
> stub-obj-y += pci-drive-hot-add.o
> +stub-obj-y += qemu-chr-open-spice.o
> stub-obj-y += qtest.o
> stub-obj-y += reset.o
> stub-obj-y += runstate-check.o
> diff --git a/stubs/chr-baum-init.c b/stubs/chr-baum-init.c
> new file mode 100644
> index 0000000..f5cc6ce
> --- /dev/null
> +++ b/stubs/chr-baum-init.c
> @@ -0,0 +1,7 @@
> +#include "qemu-common.h"
> +#include "sysemu/char.h"
> +
> +CharDriverState *chr_baum_init(void)
> +{
> + return NULL;
> +}
> diff --git a/stubs/qemu-chr-open-spice.c b/stubs/qemu-chr-open-spice.c
> new file mode 100644
> index 0000000..40a29a9
> --- /dev/null
> +++ b/stubs/qemu-chr-open-spice.c
> @@ -0,0 +1,12 @@
> +#include "qemu-common.h"
> +#include "ui/qemu-spice.h"
> +
> +CharDriverState *qemu_chr_open_spice_vmc(const char *type)
> +{
> + return NULL;
> +}
> +
> +CharDriverState *qemu_chr_open_spice_port(const char *name)
> +{
> + return NULL;
> +}
>
>
The build breaks for '--disable-spice' configs now. You get
.../stubs/qemu-chr-open-spice.c:4:18:
error: no previous prototype for 'qemu_chr_open_spice_vmc'
[-Werror=missing-prototypes]
CharDriverState *qemu_chr_open_spice_vmc(const char *type)
^
.../stubs/qemu-chr-open-spice.c:9:18:
error: no previous prototype for 'qemu_chr_open_spice_port'
[-Werror=missing-prototypes]
CharDriverState *qemu_chr_open_spice_port(const char *name)
^
I believe you need something like the following instead
drew
#include "qemu-common.h"
#include "ui/qemu-spice.h"
#ifdef CONFIG_SPICE
CharDriverState *qemu_chr_open_spice_vmc(const char *type)
{
return NULL;
}
#if SPICE_SERVER_VERSION >= 0x000c02
CharDriverState *qemu_chr_open_spice_port(const char *name)
{
return NULL;
}
#endif
#endif
- [Qemu-devel] [PATCH 0/6] vhost-user qtest fixes, Nikolay Nikolaev, 2014/06/19
- [Qemu-devel] [PATCH 1/6] configure: add new config define CONFIG_VHOST_NET_USED, Nikolay Nikolaev, 2014/06/19
- [Qemu-devel] [PATCH 2/6] libqemustub: add more stubs for qemu-char, Nikolay Nikolaev, 2014/06/19
- Re: [Qemu-devel] [PATCH 2/6] libqemustub: add more stubs for qemu-char,
Andrew Jones <=
- [Qemu-devel] [PATCH 3/6] qtest: fix qtest for vhost-user, Nikolay Nikolaev, 2014/06/19
- [Qemu-devel] [PATCH 4/6] qtest: fix vhost-user-test unbalanced mutex locks, Nikolay Nikolaev, 2014/06/19
- [Qemu-devel] [PATCH 5/6] vhost: fix migration_blocker initialization, Nikolay Nikolaev, 2014/06/19
- [Qemu-devel] [PATCH 6/6] qtest: enable vhost-user-test, Nikolay Nikolaev, 2014/06/19
- Re: [Qemu-devel] [PATCH 0/6] vhost-user qtest fixes, Michael S. Tsirkin, 2014/06/19