On 8/9/19 8:46 AM, Markus Armbruster wrote:
> In my "build everything" tree, changing qemu/main-loop.h triggers a
> recompile of some 5600 out of 6600 objects (not counting tests and
> objects that don't depend on qemu/osdep.h). It includes block/aio.h,
> which in turn includes qemu/event_notifier.h, qemu/notify.h,
> qemu/processor.h, qemu/qsp.h, qemu/queue.h, qemu/thread-posix.h,
> qemu/thread.h, qemu/timer.h, and a few more.
>
> Include qemu/main-loop.h only where it's needed. Touching it now
> recompiles only some 1700 objects. For block/aio.h and
> qemu/event_notifier.h, these numbers drop from 5600 to 2800. For the
> others, they shrink only slightly.
>
> Signed-off-by: Markus Armbruster <arm...@redhat.com>
> ---
[...]
> diff --git a/include/sysemu/sysemu.h b/include/sysemu/sysemu.h
> index 77f5df59b0..ac18a1184a 100644
> --- a/include/sysemu/sysemu.h
> +++ b/include/sysemu/sysemu.h
> @@ -5,7 +5,6 @@
> #include "qapi/qapi-types-run-state.h"
> #include "qemu/timer.h"
> #include "qemu/notify.h"
> -#include "qemu/main-loop.h"
> #include "qemu/bitmap.h"
> #include "qemu/uuid.h"
> #include "qom/object.h"
netmap failing again :S
$ make docker-image-debian-amd64 V=1 DEBUG=1
[...]
CC net/netmap.o
net/netmap.c: In function 'netmap_update_fd_handler':
net/netmap.c:109:5: error: implicit declaration of function
'qemu_set_fd_handler' [-Werror=implicit-function-declaration]
qemu_set_fd_handler(s->nmd->fd,
^~~~~~~~~~~~~~~~~~~
net/netmap.c:109:5: error: nested extern declaration of
'qemu_set_fd_handler' [-Werror=nested-externs]
