On Wed, Nov 22, 2023 at 05:36:44PM +0100, David Marchand wrote: > On Wed, Nov 22, 2023 at 5:32 PM Bruce Richardson > <[email protected]> wrote: > > > > When compiling with mingw on Ubuntu 23.10, I get the following compiler > > error: > > > > ../../lib/eal/windows/eal_memory.c:77: error: "MEM_REPLACE_PLACEHOLDER" > > redefined [-Werror] > > 77 | #define MEM_REPLACE_PLACEHOLDER 0x00004000 > > | > > In file included from /usr/share/mingw-w64/include/minwindef.h:163, > > from /usr/share/mingw-w64/include/windef.h:9, > > from /usr/share/mingw-w64/include/windows.h:69, > > from ../../lib/eal/windows/include/rte_windows.h:32, > > from ../../lib/eal/windows/include/rte_os_shim.h:9, > > from ../../lib/eal/common/eal_internal_cfg.h:14, > > from ../../lib/eal/windows/eal_memory.c:11: > > /usr/share/mingw-w64/include/winnt.h:5710: note: this is the location of > > the previous definition > > 5710 | #define MEM_REPLACE_PLACEHOLDER 0x4000 > > > > and a similar error for MEM_RESERVE_PLACEHOLDER. These errors can be > > fixed by wrapping the two defines in #ifndef .. #endif blocks. > > > > Fixes: 2a5d547a4a9b ("eal/windows: implement basic memory management") > > Cc: [email protected] > > Cc: [email protected] > > > > Signed-off-by: Bruce Richardson <[email protected]> > > Probably a duplicate of > https://patchwork.dpdk.org/project/dpdk/patch/[email protected]/ >
Yep, duplicate. I'll reject my patch in patchwork. Hopefully the patch above will be merged soon to make this issue go away! :-) /Bruce

