patches for DPDK stable branches
 help / color / mirror / Atom feed
From: luca.boccassi@gmail.com
To: Gregory Etelson <getelson@nvidia.com>
Cc: Tyler Retzlaff <roretzla@linux.microsoft.com>,
	Dmitry Kozlyuk <dmitry.kozliuk@gmail.com>,
	Thomas Monjalon <thomas@monjalon.net>,
	dpdk stable <stable@dpdk.org>
Subject: patch 'eal/windows: fix build with recent MinGW' has been queued to stable release 20.11.10
Date: Thu, 23 Nov 2023 16:17:05 +0000	[thread overview]
Message-ID: <20231123161709.670852-5-luca.boccassi@gmail.com> (raw)
In-Reply-To: <20231123161709.670852-1-luca.boccassi@gmail.com>

Hi,

FYI, your patch has been queued to stable release 20.11.10

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 11/25/23. So please
shout if anyone has objections.

Also note that after the patch there's a diff of the upstream commit vs the
patch applied to the branch. This will indicate if there was any rebasing
needed to apply to the stable branch. If there were code changes for rebasing
(ie: not only metadata diffs), please double check that the rebase was
correctly done.

Queued patches are on a temporary branch at:
https://github.com/bluca/dpdk-stable

This queued commit can be viewed at:
https://github.com/bluca/dpdk-stable/commit/f52040e04787ad91d64c3c09107cf9a06bb69b11

Thanks.

Luca Boccassi

---
From f52040e04787ad91d64c3c09107cf9a06bb69b11 Mon Sep 17 00:00:00 2001
From: Gregory Etelson <getelson@nvidia.com>
Date: Tue, 14 Nov 2023 19:05:29 +0200
Subject: [PATCH] eal/windows: fix build with recent MinGW

[ upstream commit c6221c664e52a94baf259a29ea73645496502af1 ]

Windows compilation with cross-mingw on Fedora 39 failed
because MEM_REPLACE_PLACEHOLDER and MEM_RESERVE_PLACEHOLDER were
already defined in the compiler environment:

eal_memory.c:77: error: "MEM_REPLACE_PLACEHOLDER" redefined
/usr/x86_64-w64-mingw32/sys-root/mingw/include/winnt.h:5710: note:
this is the location of the previous definition

eal_memory.c:78: error: "MEM_RESERVE_PLACEHOLDER" redefined
/usr/x86_64-w64-mingw32/sys-root/mingw/include/winnt.h:5715: note:
this is the location of the previous definition

The patch masks MEM_REPLACE_PLACEHOLDER and MEM_RESERVE_PLACEHOLDER
macros if they were pre-defined by compiler.

The patch also masks MEM_COALESCE_PLACEHOLDERS and
MEM_PRESERVE_PLACEHOLDER to prevent similar errors.

Fixes: 2a5d547a4a9b ("eal/windows: implement basic memory management")

Signed-off-by: Gregory Etelson <getelson@nvidia.com>
Acked-by: Tyler Retzlaff <roretzla@linux.microsoft.com>
Acked-by: Dmitry Kozlyuk <dmitry.kozliuk@gmail.com>
Acked-by: Thomas Monjalon <thomas@monjalon.net>
---
 lib/librte_eal/windows/eal_memory.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/lib/librte_eal/windows/eal_memory.c b/lib/librte_eal/windows/eal_memory.c
index 2fd37d9708..643e0e7a16 100644
--- a/lib/librte_eal/windows/eal_memory.c
+++ b/lib/librte_eal/windows/eal_memory.c
@@ -72,10 +72,18 @@ static VirtualAlloc2_type VirtualAlloc2_ptr;
 
 #ifdef RTE_TOOLCHAIN_GCC
 
+#ifndef MEM_COALESCE_PLACEHOLDERS
 #define MEM_COALESCE_PLACEHOLDERS 0x00000001
+#endif
+#ifndef MEM_PRESERVE_PLACEHOLDER
 #define MEM_PRESERVE_PLACEHOLDER  0x00000002
+#endif
+#ifndef MEM_REPLACE_PLACEHOLDER
 #define MEM_REPLACE_PLACEHOLDER   0x00004000
+#endif
+#ifndef MEM_RESERVE_PLACEHOLDER
 #define MEM_RESERVE_PLACEHOLDER   0x00040000
+#endif
 
 int
 eal_mem_win32api_init(void)
-- 
2.39.2

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2023-11-23 16:15:19.421163912 +0000
+++ 0005-eal-windows-fix-build-with-recent-MinGW.patch	2023-11-23 16:15:19.215058380 +0000
@@ -1 +1 @@
-From c6221c664e52a94baf259a29ea73645496502af1 Mon Sep 17 00:00:00 2001
+From f52040e04787ad91d64c3c09107cf9a06bb69b11 Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit c6221c664e52a94baf259a29ea73645496502af1 ]
+
@@ -25 +26,0 @@
-Cc: stable@dpdk.org
@@ -32 +33 @@
- lib/eal/windows/eal_memory.c | 8 ++++++++
+ lib/librte_eal/windows/eal_memory.c | 8 ++++++++
@@ -35,4 +36,4 @@
-diff --git a/lib/eal/windows/eal_memory.c b/lib/eal/windows/eal_memory.c
-index 215d768e2c..31410a41fd 100644
---- a/lib/eal/windows/eal_memory.c
-+++ b/lib/eal/windows/eal_memory.c
+diff --git a/lib/librte_eal/windows/eal_memory.c b/lib/librte_eal/windows/eal_memory.c
+index 2fd37d9708..643e0e7a16 100644
+--- a/lib/librte_eal/windows/eal_memory.c
++++ b/lib/librte_eal/windows/eal_memory.c

  parent reply	other threads:[~2023-11-23 16:17 UTC|newest]

Thread overview: 111+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-10-18 23:58 patch 'mempool: fix default ops for an empty mempool' " luca.boccassi
2023-10-18 23:58 ` patch 'rawdev: fix device class in log message' " luca.boccassi
2023-10-18 23:58 ` patch 'net/bonding: fix header for C++' " luca.boccassi
2023-10-18 23:58 ` patch 'net/hns3: fix VF default MAC modified when set failed' " luca.boccassi
2023-10-18 23:58 ` patch 'net/hns3: fix error code for multicast resource' " luca.boccassi
2023-10-18 23:58 ` patch 'net/hns3: fix flushing multicast MAC address' " luca.boccassi
2023-10-18 23:58 ` patch 'app/test: fix reference to master in bonding test' " luca.boccassi
2023-10-18 23:58 ` patch 'net/hns3: fix order in NEON Rx' " luca.boccassi
2023-10-18 23:58 ` patch 'random: initialize state for unregistered non-EAL threads' " luca.boccassi
2023-10-18 23:58 ` patch 'bus/dpaa: fix build with asserts for GCC 13' " luca.boccassi
2023-10-18 23:59 ` patch 'hash: align SSE lookup to scalar implementation' " luca.boccassi
2023-10-18 23:59 ` patch 'net/netvsc: increase VSP response timeout to 60 seconds' " luca.boccassi
2023-10-18 23:59 ` patch 'net/txgbe: add Tx queue maximum limit' " luca.boccassi
2023-10-18 23:59 ` patch 'net/txgbe: reconfigure MAC Rx when link update' " luca.boccassi
2023-10-18 23:59 ` patch 'net/txgbe: keep link down after device close' " luca.boccassi
2023-10-18 23:59 ` patch 'net/txgbe: check process type in close operation' " luca.boccassi
2023-10-18 23:59 ` patch 'net/tap: use MAC address parse API instead of local parser' " luca.boccassi
2023-10-18 23:59 ` patch 'app/testpmd: fix help string' " luca.boccassi
2023-10-18 23:59 ` patch 'cryptodev: add missing doc for security context' " luca.boccassi
2023-10-18 23:59 ` patch 'doc: replace code blocks with includes in security guide' " luca.boccassi
2023-10-18 23:59 ` patch 'test/crypto: fix IV in some vectors' " luca.boccassi
2023-10-18 23:59 ` patch 'test/crypto: skip some synchronous tests with CPU crypto' " luca.boccassi
2023-10-19 10:22   ` Power, Ciara
2023-10-19 10:28   ` Power, Ciara
2023-10-18 23:59 ` patch 'test/crypto: fix typo in asym tests' " luca.boccassi
2023-10-18 23:59 ` patch 'bus/pci: fix device ID log' " luca.boccassi
2023-10-18 23:59 ` patch 'test/event: fix crypto null device creation' " luca.boccassi
2023-10-18 23:59 ` patch 'event/sw: remove obsolete comment' " luca.boccassi
2023-10-18 23:59 ` patch 'event/sw: fix ordering corruption with op release' " luca.boccassi
2023-10-18 23:59 ` patch 'net/i40e: fix FDIR queue receives broadcast packets' " luca.boccassi
2023-10-18 23:59 ` patch 'net/iavf: fix VLAN offload strip flag' " luca.boccassi
2023-10-18 23:59 ` patch 'net/i40e: fix buffer leak on Rx reconfiguration' " luca.boccassi
2023-10-18 23:59 ` patch 'net/iavf: fix port stats clearing' " luca.boccassi
2023-10-18 23:59 ` patch 'net/ice: fix initial link status' " luca.boccassi
2023-10-18 23:59 ` patch 'net/iavf: remove log from Tx prepare function' " luca.boccassi
2023-10-18 23:59 ` patch 'net/iavf: fix TSO with big segments' " luca.boccassi
2023-10-18 23:59 ` patch 'net/ice: remove log from Tx prepare function' " luca.boccassi
2023-10-18 23:59 ` patch 'net/ice: fix TSO with big segments' " luca.boccassi
2023-10-18 23:59 ` patch 'net/mlx5: fix leak in sysfs port name translation' " luca.boccassi
2023-10-18 23:59 ` patch 'fib: fix adding default route overwriting entire table' " luca.boccassi
2023-10-18 23:59 ` patch 'fib6: fix adding default route as first route' " luca.boccassi
2023-10-18 23:59 ` patch 'net/bonding: fix link status callback stop' " luca.boccassi
2023-10-18 23:59 ` patch 'app/procinfo: remove unnecessary rte_malloc' " luca.boccassi
2023-11-08 19:24   ` patch 'malloc: remove return from void functions' " luca.boccassi
2023-11-08 19:25     ` patch 'eventdev: fix device pointer for vdev-based devices' " luca.boccassi
2023-11-08 19:25     ` patch 'eventdev: fix missing driver names in info struct' " luca.boccassi
2023-11-08 19:25     ` patch 'net/virtio: fix missing next flag in Tx packed ring' " luca.boccassi
2023-11-08 19:25     ` patch 'net/virtio: fix link state interrupt vector setting' " luca.boccassi
2023-11-08 19:25     ` patch 'vhost: fix missing vring call check on virtqueue access' " luca.boccassi
2023-11-08 19:25     ` patch 'vhost: fix missing " luca.boccassi
2023-11-08 19:25     ` patch 'ethdev: fix function name in comment' " luca.boccassi
2023-11-08 19:25     ` patch 'app/testpmd: fix early exit from signal' " luca.boccassi
2023-11-08 19:25     ` patch 'net/hns3: fix unchecked Rx free threshold' " luca.boccassi
2023-11-08 19:25     ` patch 'net/hns3: fix crash for NEON and SVE' " luca.boccassi
2023-11-08 19:25     ` patch 'net/hns3: fix double stats for IMP and global reset' " luca.boccassi
2023-11-08 19:25     ` patch 'net/hns3: remove reset log in secondary' " luca.boccassi
2023-11-08 19:25     ` patch 'test/bonding: remove unreachable statement' " luca.boccassi
2023-11-08 19:25     ` patch 'test/bonding: add missing check' " luca.boccassi
2023-11-08 19:25     ` patch 'net/bonding: fix possible overrun' " luca.boccassi
2023-11-08 19:25     ` patch 'net/txgbe: add proper memory barriers in Rx' " luca.boccassi
2023-11-08 19:25     ` patch 'ethdev: fix 32-bit build with GCC 13' " luca.boccassi
2023-11-08 19:25     ` patch 'net/enic: avoid extra unlock in MTU set' " luca.boccassi
2023-11-08 19:25     ` patch 'net/hns3: fix setting DCB capability' " luca.boccassi
2023-11-08 19:25     ` patch 'net/hns3: fix some return values' " luca.boccassi
2023-11-08 19:25     ` patch 'net/hns3: fix some error logs' " luca.boccassi
2023-11-08 19:25     ` patch 'net/hns3: keep set/get algo key functions local' " luca.boccassi
2023-11-08 19:25     ` patch 'net/hns3: fix uninitialized hash algo value' " luca.boccassi
2023-11-08 19:25     ` patch 'net/tap: fix L4 checksum offloading' " luca.boccassi
2023-11-08 19:25     ` patch 'net/tap: fix IPv4 " luca.boccassi
2023-11-08 19:25     ` patch 'app/procinfo: fix RSS info' " luca.boccassi
2023-11-08 19:25     ` patch 'app/procinfo: adjust format of " luca.boccassi
2023-11-08 19:25     ` patch 'net/nfp: fix link status interrupt' " luca.boccassi
2023-11-08 19:25     ` patch 'net/nfp: fix DMA error after abnormal exit' " luca.boccassi
2023-11-08 19:25     ` patch 'net/tap: fix RSS for fragmented packets' " luca.boccassi
2023-11-08 19:25     ` patch 'common/mlx5: fix controller index parsing' " luca.boccassi
2023-11-08 19:25     ` patch 'net/ice: fix L1 check interval' " luca.boccassi
2023-11-08 19:25     ` patch 'net/ice: fix DCF port statistics' " luca.boccassi
2023-11-08 19:25     ` patch 'crypto/nitrox: fix panic with high number of segments' " luca.boccassi
2023-11-08 19:25     ` patch 'net/iavf: fix Tx preparation' " luca.boccassi
2023-11-08 19:25     ` patch 'net/ice: " luca.boccassi
2023-11-15 11:45       ` patch 'event/dlb2: fix name check in self-test' " luca.boccassi
2023-11-15 11:45         ` patch 'test/bbdev: fix Python script subprocess' " luca.boccassi
2023-11-15 11:45         ` patch 'test/bbdev: assert failed test for queue configure' " luca.boccassi
2023-11-15 11:45         ` patch 'event/dlb2: fix missing queue ordering capability flag' " luca.boccassi
2023-11-15 11:45         ` patch 'meter: fix RFC4115 trTCM API Doxygen' " luca.boccassi
2023-11-15 11:45         ` patch 'app/testpmd: remove useless check in TSO command' " luca.boccassi
2023-11-15 11:45         ` patch 'test/bonding: fix uninitialized RSS configuration' " luca.boccassi
2023-11-15 11:45         ` patch 'net/hns3: fix mailbox sync' " luca.boccassi
2023-11-15 11:45         ` patch 'app/testpmd: fix tunnel TSO capability check' " luca.boccassi
2023-11-15 11:45         ` patch 'app/testpmd: add explicit check for tunnel TSO' " luca.boccassi
2023-11-15 11:45         ` patch 'app/testpmd: fix tunnel TSO configuration' " luca.boccassi
2023-11-15 11:45         ` patch 'net/mlx5: fix hairpin queue unbind' " luca.boccassi
2023-11-15 11:45         ` patch 'net/mlx5: fix hairpin queue states' " luca.boccassi
2023-11-15 11:45         ` patch 'net/mlx5: fix multi-segment Tx inline data length' " luca.boccassi
2023-11-15 11:45         ` patch 'net/mlx5: zero UDP checksum over IPv4 in encapsulation' " luca.boccassi
2023-11-15 11:45         ` patch 'net/mlx5: fix MPRQ stride size check' " luca.boccassi
2023-11-23 16:17           ` patch 'net/txgbe: fix out of bound access' " luca.boccassi
2023-11-23 16:17             ` patch 'doc: update features in hns3 guide' " luca.boccassi
2023-11-23 16:17             ` patch 'doc: update versions recommendations for i40e and ice' " luca.boccassi
2023-11-23 16:17             ` patch 'examples/ipsec-secgw: fix partial overflow' " luca.boccassi
2023-11-23 16:17             ` luca.boccassi [this message]
2023-11-23 16:17             ` patch 'pdump: fix error number on IPC response' " luca.boccassi
2023-11-23 16:17             ` patch 'examples/ethtool: fix pause configuration' " luca.boccassi
2023-11-23 16:17             ` patch 'test/hash: fix creation error log' " luca.boccassi
2023-11-23 16:17             ` patch 'app/pipeline: add sigint handler' " luca.boccassi
2023-11-29  2:32               ` patch 'doc: remove restriction on ixgbe vector support' " luca.boccassi
2023-11-29  2:32                 ` patch 'doc: fix some ordered lists' " luca.boccassi
2023-11-29  2:32                 ` patch 'doc: remove number of commands in vDPA guide' " luca.boccassi
2023-11-29  2:32                 ` patch 'mempool: fix get function documentation' " luca.boccassi
2023-11-29  2:32                 ` patch 'mempool: clarify enqueue/dequeue ops " luca.boccassi
2023-11-29  2:32                 ` patch 'ethdev: fix ESP packet type description' " luca.boccassi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20231123161709.670852-5-luca.boccassi@gmail.com \
    --to=luca.boccassi@gmail.com \
    --cc=dmitry.kozliuk@gmail.com \
    --cc=getelson@nvidia.com \
    --cc=roretzla@linux.microsoft.com \
    --cc=stable@dpdk.org \
    --cc=thomas@monjalon.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).