On 8/2/22 21:08, Peter Maydell wrote:
The osdep.h header is included by every C file we compile, so it helps
build times to keep it small. (As the comment at the top of the file
notes, in an ideal world this header would contain only things that
everybody needs and things where we need to apply a compatibility
workaround on some hosts.) This series trims more than 130 lines from
osdep.h (about 16% of its current size) by splitting some prototypes
that are used only in a few files out into new headers that are
included by those source files that need them.

(Looking at the size of osdep.h itself is not really the right metric,
because the real killer for compile time is going to be all the
system headers it pulls in; but it's easier to measure and looks
better for this series :-))

There's scope for more of this, I think, but there's no need to try to
do everything in one huge patchset.

thanks
-- PMM

Peter Maydell (5):
   include: Move qemu_madvise() and related #defines to new
     qemu/madvise.h
   include: Move qemu_mprotect_*() to new qemu/mprotect.h
   include: Move QEMU_MAP_* constants to mmap-alloc.h
   include: Move qemu_[id]cache_* declarations to new qemu/cacheinfo.h
   include: Move hardware version declarations to new qemu/hw-version.h

TIL qemu_hw_version().

Series:
Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>

Reply via email to