[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 05/67] migration: drop an unused include, add a
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCH 05/67] migration: drop an unused include, add a used one |
Date: |
Thu, 3 May 2018 23:29:13 +0300 |
On Thu, May 03, 2018 at 03:18:47PM -0500, Eric Blake wrote:
> On 05/03/2018 03:15 PM, Michael S. Tsirkin wrote:
> > On Thu, May 03, 2018 at 03:02:59PM -0500, Eric Blake wrote:
> > > On 05/03/2018 02:50 PM, Michael S. Tsirkin wrote:
> > > > we just need a struct name, let's add a forward
> > > > declaration instead of an include.
> > >
> > > Reasonable.
> > >
> > > >
> > > > We also use size_t, so add stddef.h
> > >
> > > Why? osdep.h already does this, and ALL .c files that use include/ should
> > > be
> > > including osdep.h prior to any other in-tree .h file. So size_t should
> > > already be in scope by the time this header is included.
> >
> > It seemed cleaner to have the file self-contained.
> > OK, I will drop this.
> > There's a small number of files which do not include qemu/osdep.h.
> > Might be worth fixing.
>
> scripts/clean-includes is already supposed to fix these.
>
> There are a few .c files exempt from including osdep.h, but in general,
> those files should probably also not be including anything from include/.
Here's a list from a quick grep.
Most of them probably get osdep.h indirectly.
+contrib/libvhost-user/libvhost-user.c
+hw/rdma/rdma_utils.c
+libuser/trace.c
+target/mips/translate_init.c
+target/ppc/mfrom_table.c
+target/ppc/translate/dfp-impl.inc.c
+target/ppc/translate/dfp-ops.inc.c
+target/ppc/translate/fp-impl.inc.c
+target/ppc/translate/fp-ops.inc.c
+target/ppc/translate/spe-impl.inc.c
+target/ppc/translate/spe-ops.inc.c
+target/ppc/translate/vmx-impl.inc.c
+target/ppc/translate/vmx-ops.inc.c
+target/ppc/translate/vsx-impl.inc.c
+target/ppc/translate/vsx-ops.inc.c
+target/s390x/gen-features.c
+target/xtensa/core-dc232b/gdb-config.inc.c
+target/xtensa/core-dc233c/gdb-config.inc.c
+target/xtensa/core-de212/gdb-config.inc.c
+target/xtensa/core-sample_controller/gdb-config.inc.c
+tcg/aarch64/tcg-target.inc.c
+tcg/arm/tcg-target.inc.c
+tcg/i386/tcg-target.inc.c
+tcg/mips/tcg-target.inc.c
+tcg/ppc/tcg-target.inc.c
+tcg/s390/tcg-target.inc.c
+tcg/sparc/tcg-target.inc.c
+tcg/tcg-ldst.inc.c
+tcg/tcg-pool.inc.c
+tcg/tci/tcg-target.inc.c
+tests/multiboot/libc.c
+tests/multiboot/mmap.c
+tests/multiboot/modules.c
lots of files under tests/tcg
+tests/test-qapi-event.c
+tests/test-qapi-types.c
+tests/test-qapi-visit.c
+tests/test-qmp-introspect.c
+tests/test-qmp-marshal.c
+ui/keycodemapdb/tests/stdc.c
> --
> Eric Blake, Principal Software Engineer
> Red Hat, Inc. +1-919-301-3266
> Virtualization: qemu.org | libvirt.org
- [Qemu-devel] [PATCH 00/67] include path cleanup, Michael S. Tsirkin, 2018/05/03
- [Qemu-devel] [PATCH 05/67] migration: drop an unused include, add a used one, Michael S. Tsirkin, 2018/05/03
- Re: [Qemu-devel] [PATCH 05/67] migration: drop an unused include, add a used one, Eric Blake, 2018/05/03
- Re: [Qemu-devel] [PATCH 05/67] migration: drop an unused include, add a used one, Michael S. Tsirkin, 2018/05/03
- Re: [Qemu-devel] [PATCH 05/67] migration: drop an unused include, add a used one, Eric Blake, 2018/05/03
- Re: [Qemu-devel] [PATCH 05/67] migration: drop an unused include, add a used one,
Michael S. Tsirkin <=
- Re: [Qemu-devel] [PATCH 05/67] migration: drop an unused include, add a used one, Eric Blake, 2018/05/03
[Qemu-devel] [PATCH 02/67] crypto: use local path for local headers, Michael S. Tsirkin, 2018/05/03
[Qemu-devel] [PATCH 04/67] s390x: drop an unused include, Michael S. Tsirkin, 2018/05/03
- Re: [Qemu-devel] [PATCH 04/67] s390x: drop an unused include, Thomas Huth, 2018/05/03
- Re: [Qemu-devel] [PATCH 04/67] s390x: drop an unused include, Cornelia Huck, 2018/05/08
- Re: [Qemu-devel] [PATCH 04/67] s390x: drop an unused include, Thomas Huth, 2018/05/08
- Re: [Qemu-devel] [PATCH 04/67] s390x: drop an unused include, Cornelia Huck, 2018/05/08
- Re: [Qemu-devel] [PATCH 04/67] s390x: drop an unused include, Thomas Huth, 2018/05/08
- Re: [Qemu-devel] [PATCH 04/67] s390x: drop an unused include, Philippe Mathieu-Daudé, 2018/05/08
- Re: [Qemu-devel] [PATCH 04/67] s390x: drop an unused include, Thomas Huth, 2018/05/08