On a Thursday in 2020, Jingqi Liu wrote:
The CONFIG_LINUX symbol is always not defined in this file. This fixes that "config-host.h" header file is not included for getting macros.Signed-off-by: Jingqi Liu <jingqi....@intel.com> --- util/mmap-alloc.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/util/mmap-alloc.c b/util/mmap-alloc.c index 27dcccd8ec..24c0e380f3 100644 --- a/util/mmap-alloc.c +++ b/util/mmap-alloc.c @@ -10,6 +10,8 @@ * later. See the COPYING file in the top-level directory. */ +#include "config-host.h" +
According to CODING_STYLE.rst, qemu/osdep.h is the header file that should be included first, before all the other includes. So the minimal fix would be moving qemu/osdep.h up here.
#ifdef CONFIG_LINUX #include <linux/mman.h> #else /* !CONFIG_LINUX */
Introduced by commit 119906afa5ca610adb87c55ab0d8e53c9104bfc3 Jano
-- 2.17.1
signature.asc
Description: PGP signature
