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_NEXT_ABI, which is disabled by default.

Signed-off-by: Helin Zhang <helin.zhang at intel.com>
---
 examples/l3fwd-acl/main.c | 29 +++++++++++++++++++++++------
 1 file changed, 23 insertions(+), 6 deletions(-)

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.

v7 changes:
* Renamed RTE_UNIFIED_PKT_TYPE to RTE_NEXT_ABI.

diff --git a/examples/l3fwd-acl/main.c b/examples/l3fwd-acl/main.c
index 29cb25e..b2bdf2f 100644
--- a/examples/l3fwd-acl/main.c
+++ b/examples/l3fwd-acl/main.c
@@ -645,10 +645,13 @@ prepare_one_packet(struct rte_mbuf **pkts_in, struct 
acl_search_t *acl,
        struct ipv4_hdr *ipv4_hdr;
        struct rte_mbuf *pkt = pkts_in[index];

+#ifdef RTE_NEXT_ABI
+       if (RTE_ETH_IS_IPV4_HDR(pkt->packet_type)) {
+#else
        int type = pkt->ol_flags & (PKT_RX_IPV4_HDR | PKT_RX_IPV6_HDR);

        if (type == PKT_RX_IPV4_HDR) {
-
+#endif
                ipv4_hdr = rte_pktmbuf_mtod_offset(pkt, struct ipv4_hdr *,
                                                   sizeof(struct ether_hdr));

@@ -667,9 +670,11 @@ prepare_one_packet(struct rte_mbuf **pkts_in, struct 
acl_search_t *acl,
                        /* Not a valid IPv4 packet */
                        rte_pktmbuf_free(pkt);
                }
-
+#ifdef RTE_NEXT_ABI
+       } else if (RTE_ETH_IS_IPV6_HDR(pkt->packet_type)) {
+#else
        } else if (type == PKT_RX_IPV6_HDR) {
-
+#endif
                /* Fill acl structure */
                acl->data_ipv6[acl->num_ipv6] = MBUF_IPV6_2PROTO(pkt);
                acl->m_ipv6[(acl->num_ipv6)++] = pkt;
@@ -687,17 +692,22 @@ prepare_one_packet(struct rte_mbuf **pkts_in, struct 
acl_search_t *acl,
 {
        struct rte_mbuf *pkt = pkts_in[index];

+#ifdef RTE_NEXT_ABI
+       if (RTE_ETH_IS_IPV4_HDR(pkt->packet_type)) {
+#else
        int type = pkt->ol_flags & (PKT_RX_IPV4_HDR | PKT_RX_IPV6_HDR);

        if (type == PKT_RX_IPV4_HDR) {
-
+#endif
                /* Fill acl structure */
                acl->data_ipv4[acl->num_ipv4] = MBUF_IPV4_2PROTO(pkt);
                acl->m_ipv4[(acl->num_ipv4)++] = pkt;

-
+#ifdef RTE_NEXT_ABI
+       } else if (RTE_ETH_IS_IPV6_HDR(pkt->packet_type)) {
+#else
        } else if (type == PKT_RX_IPV6_HDR) {
-
+#endif
                /* Fill acl structure */
                acl->data_ipv6[acl->num_ipv6] = MBUF_IPV6_2PROTO(pkt);
                acl->m_ipv6[(acl->num_ipv6)++] = pkt;
@@ -745,10 +755,17 @@ send_one_packet(struct rte_mbuf *m, uint32_t res)
                /* in the ACL list, drop it */
 #ifdef L3FWDACL_DEBUG
                if ((res & ACL_DENY_SIGNATURE) != 0) {
+#ifdef RTE_NEXT_ABI
+                       if (RTE_ETH_IS_IPV4_HDR(m->packet_type))
+                               dump_acl4_rule(m, res);
+                       else if (RTE_ETH_IS_IPV6_HDR(m->packet_type))
+                               dump_acl6_rule(m, res);
+#else
                        if (m->ol_flags & PKT_RX_IPV4_HDR)
                                dump_acl4_rule(m, res);
                        else
                                dump_acl6_rule(m, res);
+#endif /* RTE_NEXT_ABI */
                }
 #endif
                rte_pktmbuf_free(m);
-- 
1.9.3

Reply via email to