lpm: fix big endian support
authorXuelin Shi <xuelin.shi@freescale.com>
Tue, 24 Mar 2015 05:58:20 +0000 (13:58 +0800)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Fri, 10 Jul 2015 21:23:15 +0000 (23:23 +0200)
This module uses type conversion between struct and int.
Also truncation and comparison is used with this int.
It is not safe for different endian arch.

Add ifdef for big endian struct to fix this issue.

Signed-off-by: Xuelin Shi <xuelin.shi@freescale.com>
Acked-by: Bruce Richardson <bruce.richardson@intel.com>
lib/librte_lpm/rte_lpm.h

index e318769..11f0c04 100644 (file)
@@ -44,6 +44,7 @@
 #include <stdint.h>
 #include <stdlib.h>
 #include <rte_branch_prediction.h>
+#include <rte_byteorder.h>
 #include <rte_memory.h>
 #include <rte_common.h>
 #include <rte_vect.h>
@@ -96,6 +97,7 @@ extern "C" {
 /** Bitmask used to indicate successful lookup */
 #define RTE_LPM_LOOKUP_SUCCESS          0x0100
 
+#if RTE_BYTE_ORDER == RTE_LITTLE_ENDIAN
 /** @internal Tbl24 entry structure. */
 struct rte_lpm_tbl24_entry {
        /* Stores Next hop or group index (i.e. gindex)into tbl8. */
@@ -117,6 +119,24 @@ struct rte_lpm_tbl8_entry {
        uint8_t valid_group :1; /**< Group validation flag. */
        uint8_t depth       :6; /**< Rule depth. */
 };
+#else
+struct rte_lpm_tbl24_entry {
+       uint8_t depth       :6;
+       uint8_t ext_entry   :1;
+       uint8_t valid       :1;
+       union {
+               uint8_t tbl8_gindex;
+               uint8_t next_hop;
+       };
+};
+
+struct rte_lpm_tbl8_entry {
+       uint8_t depth       :6;
+       uint8_t valid_group :1;
+       uint8_t valid       :1;
+       uint8_t next_hop;
+};
+#endif
 
 /** @internal Rule structure. */
 struct rte_lpm_rule {