To unify packet types among all PMDs, bit masks of packet type for
'ol_flags' are replaced by unified packet type.
To avoid breaking ABI compatibility, all the changes would be
enabled by RTE_UNIFIED_PKT_TYPE, which is disabled by default.

Signed-off-by: Helin Zhang <helin.zhang at intel.com>
---
 app/test-pipeline/pipeline_hash.c | 13 +++++++++++++
 1 file changed, 13 insertions(+)

v2 changes:
* Used redefined packet types and enlarged packet_type field in mbuf.

v5 changes:
* Re-worded the commit logs.

v6 changes:
* Disabled the code changes for unified packet type by default, to
  avoid breaking ABI compatibility.

diff --git a/app/test-pipeline/pipeline_hash.c 
b/app/test-pipeline/pipeline_hash.c
index 4598ad4..bc84920 100644
--- a/app/test-pipeline/pipeline_hash.c
+++ b/app/test-pipeline/pipeline_hash.c
@@ -459,20 +459,33 @@ app_main_loop_rx_metadata(void) {
                        signature = RTE_MBUF_METADATA_UINT32_PTR(m, 0);
                        key = RTE_MBUF_METADATA_UINT8_PTR(m, 32);

+#ifdef RTE_UNIFIED_PKT_TYPE
+                       if (RTE_ETH_IS_IPV4_HDR(m->packet_type)) {
+#else
                        if (m->ol_flags & PKT_RX_IPV4_HDR) {
+#endif
                                ip_hdr = (struct ipv4_hdr *)
                                        &m_data[sizeof(struct ether_hdr)];
                                ip_dst = ip_hdr->dst_addr;

                                k32 = (uint32_t *) key;
                                k32[0] = ip_dst & 0xFFFFFF00;
+#ifdef RTE_UNIFIED_PKT_TYPE
+                       } else if (RTE_ETH_IS_IPV6_HDR(m->packet_type)) {
+#else
                        } else {
+#endif
                                ipv6_hdr = (struct ipv6_hdr *)
                                        &m_data[sizeof(struct ether_hdr)];
                                ipv6_dst = ipv6_hdr->dst_addr;

                                memcpy(key, ipv6_dst, 16);
+#ifdef RTE_UNIFIED_PKT_TYPE
+                       } else
+                               continue;
+#else
                        }
+#endif

                        *signature = test_hash(key, 0, 0);
                }
-- 
1.9.3

Reply via email to