Re: [Qemu-trivial] [PATCH] Clean up includes

[Thomas Huth <thuth@redhat.com>]

On 13/03/2019 17.28, Markus Armbruster wrote:
> Clean up includes so that osdep.h is included first and headers
> which it implies are not included manually.
> 
> This commit was created with scripts/clean-includes, with the changes
> to the following files manually reverted:
> 
>     contrib/libvhost-user/libvhost-user-glib.h
>     contrib/libvhost-user/libvhost-user.c
>     contrib/libvhost-user/libvhost-user.h
>     linux-user/mips64/cpu_loop.c
>     linux-user/mips64/signal.c
>     linux-user/sparc64/cpu_loop.c
>     linux-user/sparc64/signal.c
>     linux-user/x86_64/cpu_loop.c
>     linux-user/x86_64/signal.c
>     slirp/src/*
>     target/s390x/gen-features.c
>     tests/migration/s390x/a-b-bios.c
>     tests/test-rcu-simpleq.c
>     tests/test-rcu-tailq.c
>     tests/uefi-test-tools/UefiTestToolsPkg/BiosTablesTest/BiosTablesTest.c
> 
> We're in the process of spinning out slirp/.  tests/uefi-test-tools/
> is guest software.  The remaining reverts are the same as in commit
> b7d89466dde.
> 
> Signed-off-by: Markus Armbruster <armbru@redhat.com>
> ---
>  contrib/elf2dmp/main.c   | 3 +--
>  contrib/elf2dmp/pdb.c    | 3 +--
>  hw/display/ati.c         | 1 +
>  hw/display/ati_2d.c      | 1 +
>  hw/display/ati_dbg.c     | 1 +
>  hw/display/ati_int.h     | 1 -
>  include/hw/cpu/cluster.h | 1 -
>  tests/fp/platform.h      | 1 -
>  tests/libqos/qgraph.h    | 4 ----
>  tests/qos-test.c         | 2 +-
>  10 files changed, 6 insertions(+), 12 deletions(-)

Reviewed-by: Thomas Huth <thuth@redhat.com>