[Top][All Lists]

[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 may or may not be exempt; but it's not in the whitelist.


Probably a bug; looks like a recently added file.


Huh? I don't see this file in git.


Probably bugs.


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.


Possibly a bug.


Exempt, per the XDIRREGEX in scripts/clean-includes

lots of files under tests/tcg

Also exempt.


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.


Again, how are you flagging these generated files?


Exempt, since it's in a submodule.

Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org

reply via email to

[Prev in Thread] Current Thread [Next in Thread]