[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL v2 059/106] libqemustub: add stubs to be able to
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PULL v2 059/106] libqemustub: add stubs to be able to use qemu-char.c |
Date: |
Mon, 23 Jun 2014 10:20:49 +0300 |
On Mon, Jun 23, 2014 at 09:15:34AM +0300, Riku Voipio wrote:
> Hi,
>
> On Wed, Jun 18, 2014 at 07:19:17PM +0300, Michael S. Tsirkin wrote:
> > From: Nikolay Nikolaev <address@hidden>
> >
> > chardev depends on lots of external symbols that are not necessarily
> > needed to be able to use, for example, 'socket chardev'. So add stubs
> > for these functions:
> >
> > - bdrv_commit_all
> > - qemu_chr_open_msmouse
> > - is_daemonized
> > - qemu_add_machine_init_done_notifier
> > - monitor_init
> > - qemu_notify_event
> > - vc_init
>
> This broke user build of user tools only. With:
>
> ./configure --disable-tools --disable-docs --target-list=arm-linux-user
> ...
> pixman none
> ...
> make
> ...
> In file included from /data/home/nchip/linaro/qemu/include/ui/console.h:4:0,
> from /data/home/nchip/linaro/qemu/stubs/vc-init.c:2:
> /data/home/nchip/linaro/qemu/include/ui/qemu-pixman.h:14:20: fatal error:
> pixman.h: No such file or directory
> #include <pixman.h>
> ^
> compilation terminated.
>
> I'm not familiar with this area of Qemu - perhaps the following would be a
> good fix:
>
> -stub-obj-y += vc-init.o
> +stub-obj-$(CONFIG_SOFTMMU) += vc-init.o
>
> Riku
Hmm qemu-char.o needs it - and that's in common-obj-y, so I doubt that will
work.
I think we should just avoid including ui bits from stubs.
Will send a patch shortly.
> >
> > and this array:
> >
> > - serial_hds
> >
> > Signed-off-by: Antonios Motakis <address@hidden>
> > Signed-off-by: Nikolay Nikolaev <address@hidden>
> > Reviewed-by: Michael S. Tsirkin <address@hidden>
> > Signed-off-by: Michael S. Tsirkin <address@hidden>
> > ---
> > stubs/bdrv-commit-all.c | 7 +++++++
> > stubs/chr-msmouse.c | 7 +++++++
> > stubs/get-next-serial.c | 3 +++
> > stubs/is-daemonized.c | 7 +++++++
> > stubs/machine-init-done.c | 6 ++++++
> > stubs/monitor-init.c | 6 ++++++
> > stubs/notify-event.c | 6 ++++++
> > stubs/vc-init.c | 7 +++++++
> > stubs/Makefile.objs | 8 ++++++++
> > 9 files changed, 57 insertions(+)
> > create mode 100644 stubs/bdrv-commit-all.c
> > create mode 100644 stubs/chr-msmouse.c
> > create mode 100644 stubs/get-next-serial.c
> > create mode 100644 stubs/is-daemonized.c
> > create mode 100644 stubs/machine-init-done.c
> > create mode 100644 stubs/monitor-init.c
> > create mode 100644 stubs/notify-event.c
> > create mode 100644 stubs/vc-init.c
> >
> > diff --git a/stubs/bdrv-commit-all.c b/stubs/bdrv-commit-all.c
> > new file mode 100644
> > index 0000000..a8e0a95
> > --- /dev/null
> > +++ b/stubs/bdrv-commit-all.c
> > @@ -0,0 +1,7 @@
> > +#include "qemu-common.h"
> > +#include "block/block.h"
> > +
> > +int bdrv_commit_all(void)
> > +{
> > + return 0;
> > +}
> > diff --git a/stubs/chr-msmouse.c b/stubs/chr-msmouse.c
> > new file mode 100644
> > index 0000000..812f8b0
> > --- /dev/null
> > +++ b/stubs/chr-msmouse.c
> > @@ -0,0 +1,7 @@
> > +#include "qemu-common.h"
> > +#include "sysemu/char.h"
> > +
> > +CharDriverState *qemu_chr_open_msmouse(void)
> > +{
> > + return 0;
> > +}
> > diff --git a/stubs/get-next-serial.c b/stubs/get-next-serial.c
> > new file mode 100644
> > index 0000000..40c56d1
> > --- /dev/null
> > +++ b/stubs/get-next-serial.c
> > @@ -0,0 +1,3 @@
> > +#include "qemu-common.h"
> > +
> > +CharDriverState *serial_hds[0];
> > diff --git a/stubs/is-daemonized.c b/stubs/is-daemonized.c
> > new file mode 100644
> > index 0000000..16ce7c7
> > --- /dev/null
> > +++ b/stubs/is-daemonized.c
> > @@ -0,0 +1,7 @@
> > +#include "qemu-common.h"
> > +#include "sysemu/os-posix.h"
> > +
> > +bool is_daemonized(void)
> > +{
> > + return true;
> > +}
> > diff --git a/stubs/machine-init-done.c b/stubs/machine-init-done.c
> > new file mode 100644
> > index 0000000..28a9255
> > --- /dev/null
> > +++ b/stubs/machine-init-done.c
> > @@ -0,0 +1,6 @@
> > +#include "qemu-common.h"
> > +#include "sysemu/sysemu.h"
> > +
> > +void qemu_add_machine_init_done_notifier(Notifier *notify)
> > +{
> > +}
> > diff --git a/stubs/monitor-init.c b/stubs/monitor-init.c
> > new file mode 100644
> > index 0000000..563902b
> > --- /dev/null
> > +++ b/stubs/monitor-init.c
> > @@ -0,0 +1,6 @@
> > +#include "qemu-common.h"
> > +#include "monitor/monitor.h"
> > +
> > +void monitor_init(CharDriverState *chr, int flags)
> > +{
> > +}
> > diff --git a/stubs/notify-event.c b/stubs/notify-event.c
> > new file mode 100644
> > index 0000000..32f7289
> > --- /dev/null
> > +++ b/stubs/notify-event.c
> > @@ -0,0 +1,6 @@
> > +#include "qemu-common.h"
> > +#include "qemu/main-loop.h"
> > +
> > +void qemu_notify_event(void)
> > +{
> > +}
> > diff --git a/stubs/vc-init.c b/stubs/vc-init.c
> > new file mode 100644
> > index 0000000..2af054f
> > --- /dev/null
> > +++ b/stubs/vc-init.c
> > @@ -0,0 +1,7 @@
> > +#include "qemu-common.h"
> > +#include "ui/console.h"
> > +
> > +CharDriverState *vc_init(ChardevVC *vc)
> > +{
> > + return 0;
> > +}
> > diff --git a/stubs/Makefile.objs b/stubs/Makefile.objs
> > index d99e2b9..5a0b917 100644
> > --- a/stubs/Makefile.objs
> > +++ b/stubs/Makefile.objs
> > @@ -1,4 +1,6 @@
> > stub-obj-y += arch-query-cpu-def.o
> > +stub-obj-y += bdrv-commit-all.o
> > +stub-obj-y += chr-msmouse.o
> > stub-obj-y += clock-warp.o
> > stub-obj-y += cpu-get-clock.o
> > stub-obj-y += cpu-get-icount.o
> > @@ -9,13 +11,18 @@ stub-obj-y += fdset-get-fd.o
> > stub-obj-y += fdset-remove-fd.o
> > stub-obj-y += gdbstub.o
> > stub-obj-y += get-fd.o
> > +stub-obj-y += get-next-serial.o
> > stub-obj-y += get-vm-name.o
> > stub-obj-y += iothread-lock.o
> > +stub-obj-y += is-daemonized.o
> > +stub-obj-y += machine-init-done.o
> > stub-obj-y += migr-blocker.o
> > stub-obj-y += mon-is-qmp.o
> > stub-obj-y += mon-printf.o
> > stub-obj-y += mon-protocol-event.o
> > 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 += qtest.o
> > stub-obj-y += reset.o
> > @@ -24,6 +31,7 @@ stub-obj-y += set-fd-handler.o
> > stub-obj-y += slirp.o
> > stub-obj-y += sysbus.o
> > stub-obj-y += uuid.o
> > +stub-obj-y += vc-init.o
> > stub-obj-y += vm-stop.o
> > stub-obj-y += vmstate.o
> > stub-obj-$(CONFIG_WIN32) += fd-register.o
> > --
> > MST
> >
- [Qemu-devel] [PULL v2 046/106] Add chardev API qemu_chr_fe_get_msgfds, (continued)
- [Qemu-devel] [PULL v2 046/106] Add chardev API qemu_chr_fe_get_msgfds, Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 052/106] Add vhost_ops to vhost_dev struct and replace all relevant ioctls, Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 053/106] Add vhost-backend and VhostBackendType, Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 054/106] Add vhost-user as a vhost backend., Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 055/106] vhost-net: vhost-user feature bits support, Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 056/106] Add new vhost-user netdev backend, Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 057/106] Add the vhost-user netdev backend to the command line, Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 058/106] Add vhost-user protocol documentation, Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 059/106] libqemustub: add stubs to be able to use qemu-char.c, Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 060/106] Add qtest for vhost-user, Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 061/106] NUMA: move numa related code to new file numa.c, Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 063/106] NUMA: Add numa_info structure to contain numa nodes info, Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 064/106] NUMA: convert -numa option to use OptsVisitor, Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 065/106] NUMA: expand MAX_NODES from 64 to 128, Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 066/106] man: improve -numa doc, Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 067/106] qmp: improve error reporting for -object and object-add, Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 068/106] numa: introduce memory_region_allocate_system_memory, Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 069/106] memory: reorganize file-based allocation, Michael S. Tsirkin, 2014/06/18
- [Qemu-devel] [PULL v2 070/106] memory: move preallocation code out of exec.c, Michael S. Tsirkin, 2014/06/18