[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 22/29] Include hw/boards.h a bit less
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [PATCH v2 22/29] Include hw/boards.h a bit less |
Date: |
Wed, 7 Aug 2019 14:57:08 -0300 |
On Wed, Aug 07, 2019 at 07:26:56PM +0200, Philippe Mathieu-Daudé wrote:
> On 8/6/19 5:14 PM, Markus Armbruster wrote:
> > hw/boards.h pulls in almost 60 headers. The less we include it into
> > headers, the better. As a first step, drop superfluous inclusions,
> > and downgrade some more to what's actually needed. Gets rid of just
> > one inclusion into a header.
> >
> > Cc: Eduardo Habkost <address@hidden>
> > Cc: Marcel Apfelbaum <address@hidden>
> > Signed-off-by: Markus Armbruster <address@hidden>
> > ---
[...]
> > diff --git a/target/i386/hax-all.c b/target/i386/hax-all.c
> > index bcacdd1d8f..34a9f6f7a9 100644
> > --- a/target/i386/hax-all.c
> > +++ b/target/i386/hax-all.c
> > @@ -33,7 +33,6 @@
> > #include "sysemu/reset.h"
> > #include "sysemu/sysemu.h"
> > #include "qemu/main-loop.h"
> > -#include "hw/boards.h"
> >
> > #define DEBUG_HAX 0
>
> include/sysemu/hax.h misses to include "hw/boards.h":
I don't understand. I don't see any reason for the sysemu/hax.h
header to include hw/boards.h.
>
> target/i386/hax-all.c: In function 'hax_accel_init':
> target/i386/hax-all.c:354:26: error: dereferencing pointer to incomplete
> type 'MachineState {aka struct MachineState}'
> int ret = hax_init(ms->ram_size);
> ^
--
Eduardo
- Re: [Qemu-devel] [PATCH v2 16/29] Include exec/memory.h slightly less, (continued)
[Qemu-devel] [PATCH v2 23/29] numa: Don't include hw/boards.h into sysemu/numa.h, Markus Armbruster, 2019/08/06
[Qemu-devel] [PATCH v2 24/29] Include sysemu/hostmem.h less, Markus Armbruster, 2019/08/06
[Qemu-devel] [PATCH v2 06/29] trace: Eliminate use of TARGET_FMT_plx, Markus Armbruster, 2019/08/06
[Qemu-devel] [PATCH v2 22/29] Include hw/boards.h a bit less, Markus Armbruster, 2019/08/06
[Qemu-devel] [PATCH v2 08/29] Include sysemu/reset.h a lot less, Markus Armbruster, 2019/08/06
[Qemu-devel] [PATCH v2 28/29] sysemu: Move the VMChangeStateEntry typedef to qemu/typedefs.h, Markus Armbruster, 2019/08/06
[Qemu-devel] [PATCH v2 20/29] Include qemu/main-loop.h less, Markus Armbruster, 2019/08/06