On 4/28/2020 1:03 PM, David Marchand wrote:
This header is duplicated from the DPDK generic header.
Fix typo identified in DPDK [1].
While at it, RTE_EXEC_ENV_BSDAPP has been replaced with
RTE_EXEC_ENV_FREEBSD in 19.05 [2].
1: https://git.dpdk.org/dpdk/commit/?id=a3e283ed904c
2: https://git.dpdk.org/dpdk/commit/?id=5fbc1d498f54
Signed-off-by: David Marchand <[email protected]>
Thanks for this David.
Looks ok to myself overall, tested locally and with travis.
https://travis-ci.org/github/istokes/ovs/builds/683329259
@Ilya, you've looked at BSD a bit more than myself in the past, any
comments on this? I've held off applying for the moment just in case.
Regards
Ian
---
include/sparse/rte_byteorder.h | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/include/sparse/rte_byteorder.h b/include/sparse/rte_byteorder.h
index d32b5e6915..72cacac894 100644
--- a/include/sparse/rte_byteorder.h
+++ b/include/sparse/rte_byteorder.h
@@ -49,7 +49,7 @@
#include "openvswitch/types.h"
#include <stdint.h>
-#ifdef RTE_EXEC_ENV_BSDAPP
+#ifdef RTE_EXEC_ENV_FREEBSD
#include <sys/endian.h>
#else
#include <endian.h>
@@ -127,9 +127,9 @@
#define RTE_BE16(v) (OVS_FORCE rte_be16_t)(RTE_STATIC_BSWAP16(v))
#define RTE_BE32(v) (OVS_FORCE rte_be32_t)(RTE_STATIC_BSWAP32(v))
#define RTE_BE64(v) (OVS_FORCE rte_be64_t)(RTE_STATIC_BSWAP64(v))
-#define RTE_LE16(v) (OVS_FORCE rte_be16_t)(v)
-#define RTE_LE32(v) (OVS_FORCE rte_be32_t)(v)
-#define RTE_LE64(v) (OVS_FORCE rte_be64_t)(v)
+#define RTE_LE16(v) (OVS_FORCE rte_le16_t)(v)
+#define RTE_LE32(v) (OVS_FORCE rte_le32_t)(v)
+#define RTE_LE64(v) (OVS_FORCE rte_le64_t)(v)
#else
#error Unsupported endianness.
#endif
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
