[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: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH 05/67] migration: drop an unused include, add a used one |
Date: |
Thu, 3 May 2018 15:45:08 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 |
On 05/03/2018 03:29 PM, Michael S. Tsirkin wrote:
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
contrib may or may not be exempt; but it's not in the whitelist.
+hw/rdma/rdma_utils.c
Probably a bug; looks like a recently added file.
+libuser/trace.c
Huh? I don't see this file in git.
+target/mips/translate_init.c
+target/ppc/mfrom_table.c
Probably bugs.
+target/ppc/translate/dfp-impl.inc.c
This, and all other .inc.c, are indeed picking it up indirectly (by the
master .c that is including these secondary .c), so they are exempt.
They are whitelisted by the script via the shell case statement.
+target/s390x/gen-features.c
Possibly a bug.
+tests/multiboot/libc.c
+tests/multiboot/mmap.c
+tests/multiboot/modules.c
Exempt, per the XDIRREGEX in scripts/clean-includes
lots of files under tests/tcg
Also exempt.
+tests/test-qapi-event.c
Huh? That starts with #include "qemu/osdep.h". But it's not a file
under version control. In fact, it's stale leftovers (it used to be
generated by that name, but now we generate tests/test-qapi-events.c.
+tests/test-qapi-types.c
+tests/test-qapi-visit.c
+tests/test-qmp-introspect.c
+tests/test-qmp-marshal.c
Again, how are you flagging these generated files?
+ui/keycodemapdb/tests/stdc.c
Exempt, since it's in a submodule.
--
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, 2018/05/03
- Re: [Qemu-devel] [PATCH 05/67] migration: drop an unused include, add a used one,
Eric Blake <=
[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
[Qemu-devel] [PATCH 03/67] hppa: use local path for local headers, Michael S. Tsirkin, 2018/05/03