net/qede: fix incorrect queue id for 100G
[dpdk.git] / drivers / net / qede / qede_rxtx.c
index f65c833..a232d20 100644 (file)
@@ -82,6 +82,7 @@ qede_rx_queue_setup(struct rte_eth_dev *dev, uint16_t queue_idx,
        rxq->nb_rx_desc = nb_desc;
        rxq->queue_id = queue_idx;
        rxq->port_id = dev->data->port_id;
+
        max_rx_pkt_len = (uint16_t)rxmode->max_rx_pkt_len;
        qdev->mtu = max_rx_pkt_len;
 
@@ -94,6 +95,7 @@ qede_rx_queue_setup(struct rte_eth_dev *dev, uint16_t queue_idx,
                        dev->data->scattered_rx = 1;
                }
        }
+
        if (dev->data->scattered_rx)
                rxq->rx_buf_size = bufsz + QEDE_ETH_OVERHEAD;
        else
@@ -109,9 +111,8 @@ qede_rx_queue_setup(struct rte_eth_dev *dev, uint16_t queue_idx,
        rxq->sw_rx_ring = rte_zmalloc_socket("sw_rx_ring", size,
                                             RTE_CACHE_LINE_SIZE, socket_id);
        if (!rxq->sw_rx_ring) {
-               DP_NOTICE(edev, false,
-                         "Unable to alloc memory for sw_rx_ring on socket %u\n",
-                         socket_id);
+               DP_ERR(edev, "Memory allocation fails for sw_rx_ring on"
+                      " socket %u\n", socket_id);
                rte_free(rxq);
                return -ENOMEM;
        }
@@ -127,9 +128,8 @@ qede_rx_queue_setup(struct rte_eth_dev *dev, uint16_t queue_idx,
                                            NULL);
 
        if (rc != ECORE_SUCCESS) {
-               DP_NOTICE(edev, false,
-                         "Unable to alloc memory for rxbd ring on socket %u\n",
-                         socket_id);
+               DP_ERR(edev, "Memory allocation fails for RX BD ring"
+                      " on socket %u\n", socket_id);
                rte_free(rxq->sw_rx_ring);
                rte_free(rxq);
                return -ENOMEM;
@@ -146,10 +146,9 @@ qede_rx_queue_setup(struct rte_eth_dev *dev, uint16_t queue_idx,
                                            NULL);
 
        if (rc != ECORE_SUCCESS) {
-               DP_NOTICE(edev, false,
-                         "Unable to alloc memory for cqe ring on socket %u\n",
-                         socket_id);
-               /* TBD: Freeing RX BD ring */
+               DP_ERR(edev, "Memory allocation fails for RX CQE ring"
+                      " on socket %u\n", socket_id);
+               qdev->ops->common->chain_free(edev, &rxq->rx_bd_ring);
                rte_free(rxq->sw_rx_ring);
                rte_free(rxq);
                return -ENOMEM;
@@ -300,6 +299,7 @@ qede_tx_queue_setup(struct rte_eth_dev *dev,
                DP_ERR(edev,
                       "Unable to allocate memory for txbd ring on socket %u",
                       socket_id);
+               qdev->ops->common->chain_free(edev, &txq->tx_pbl);
                qede_tx_queue_release(txq);
                return -ENOMEM;
        }
@@ -363,23 +363,23 @@ static int
 qede_alloc_mem_sb(struct qede_dev *qdev, struct ecore_sb_info *sb_info,
                  uint16_t sb_id)
 {
-       struct ecore_dev *edev = &qdev->edev;
+       struct ecore_dev *edev = QEDE_INIT_EDEV(qdev);
        struct status_block *sb_virt;
        dma_addr_t sb_phys;
        int rc;
 
-       sb_virt = OSAL_DMA_ALLOC_COHERENT(edev, &sb_phys, sizeof(*sb_virt));
-
+       sb_virt = OSAL_DMA_ALLOC_COHERENT(edev, &sb_phys,
+                                         sizeof(struct status_block));
        if (!sb_virt) {
                DP_ERR(edev, "Status block allocation failed\n");
                return -ENOMEM;
        }
-
        rc = qdev->ops->common->sb_init(edev, sb_info, sb_virt,
                                        sb_phys, sb_id);
        if (rc) {
                DP_ERR(edev, "Status block initialization failed\n");
-               /* TBD: No dma_free_coherent possible */
+               OSAL_DMA_FREE_COHERENT(edev, sb_virt, sb_phys,
+                                      sizeof(struct status_block));
                return rc;
        }
 
@@ -437,9 +437,12 @@ int qede_alloc_fp_resc(struct qede_dev *qdev)
 void qede_dealloc_fp_resc(struct rte_eth_dev *eth_dev)
 {
        struct qede_dev *qdev = QEDE_INIT_QDEV(eth_dev);
-       __rte_unused struct ecore_dev *edev = QEDE_INIT_EDEV(qdev);
+       struct ecore_dev *edev = QEDE_INIT_EDEV(qdev);
        struct qede_fastpath *fp;
+       struct qede_rx_queue *rxq;
+       struct qede_tx_queue *txq;
        uint16_t sb_idx;
+       uint8_t i;
 
        PMD_INIT_FUNC_TRACE(edev);
 
@@ -447,10 +450,38 @@ void qede_dealloc_fp_resc(struct rte_eth_dev *eth_dev)
                fp = &qdev->fp_array[sb_idx];
                DP_INFO(edev, "Free sb_info index 0x%x\n",
                                fp->sb_info->igu_sb_id);
-               if (fp->sb_info)
+               if (fp->sb_info) {
+                       OSAL_DMA_FREE_COHERENT(edev, fp->sb_info->sb_virt,
+                               fp->sb_info->sb_phys,
+                               sizeof(struct status_block));
                        rte_free(fp->sb_info);
-               fp->sb_info = NULL;
+                       fp->sb_info = NULL;
+               }
        }
+
+       /* Free packet buffers and ring memories */
+       for (i = 0; i < eth_dev->data->nb_rx_queues; i++) {
+               if (eth_dev->data->rx_queues[i]) {
+                       qede_rx_queue_release(eth_dev->data->rx_queues[i]);
+                       rxq = eth_dev->data->rx_queues[i];
+                       qdev->ops->common->chain_free(edev,
+                                                     &rxq->rx_bd_ring);
+                       qdev->ops->common->chain_free(edev,
+                                                     &rxq->rx_comp_ring);
+                       eth_dev->data->rx_queues[i] = NULL;
+               }
+       }
+
+       for (i = 0; i < eth_dev->data->nb_tx_queues; i++) {
+               if (eth_dev->data->tx_queues[i]) {
+                       txq = eth_dev->data->tx_queues[i];
+                       qede_tx_queue_release(eth_dev->data->tx_queues[i]);
+                       qdev->ops->common->chain_free(edev,
+                                                     &txq->tx_pbl);
+                       eth_dev->data->tx_queues[i] = NULL;
+               }
+       }
+
        if (qdev->fp_array)
                rte_free(qdev->fp_array);
        qdev->fp_array = NULL;
@@ -521,8 +552,9 @@ qede_rx_queue_start(struct rte_eth_dev *eth_dev, uint16_t rx_queue_id)
                ecore_sb_ack(fp->sb_info, IGU_INT_DISABLE, 0);
                /* Prepare ramrod */
                memset(&params, 0, sizeof(params));
-               params.queue_id = rx_queue_id;
+               params.queue_id = rx_queue_id / edev->num_hwfns;
                params.vport_id = 0;
+               params.stats_id = params.vport_id;
                params.sb = fp->sb_info->igu_sb_id;
                DP_INFO(edev, "rxq %u igu_sb_id 0x%x\n",
                                fp->rxq->queue_id, fp->sb_info->igu_sb_id);
@@ -579,8 +611,9 @@ qede_tx_queue_start(struct rte_eth_dev *eth_dev, uint16_t tx_queue_id)
                txq = eth_dev->data->tx_queues[tx_queue_id];
                fp = &qdev->fp_array[tx_queue_id];
                memset(&params, 0, sizeof(params));
-               params.queue_id = tx_queue_id;
+               params.queue_id = tx_queue_id / edev->num_hwfns;
                params.vport_id = 0;
+               params.stats_id = params.vport_id;
                params.sb = fp->sb_info->igu_sb_id;
                DP_INFO(edev, "txq %u igu_sb_id 0x%x\n",
                                fp->txq->queue_id, fp->sb_info->igu_sb_id);
@@ -672,7 +705,6 @@ qede_process_tx_compl(__rte_unused struct ecore_dev *edev,
                qede_free_tx_pkt(txq);
 }
 
-
 static int qede_drain_txq(struct qede_dev *qdev,
                          struct qede_tx_queue *txq, bool allow_drain)
 {
@@ -708,7 +740,6 @@ static int qede_drain_txq(struct qede_dev *qdev,
        return 0;
 }
 
-
 /* Stops a given TX queue in the HW */
 static int qede_tx_queue_stop(struct rte_eth_dev *eth_dev, uint16_t tx_queue_id)
 {
@@ -1051,7 +1082,7 @@ qede_process_sg_pkts(void *p_rxq,  struct rte_mbuf *rx_mb,
                                                        pkt_len;
                if (unlikely(!cur_size)) {
                        PMD_RX_LOG(ERR, rxq, "Length is 0 while %u BDs"
-                                  " left for mapping jumbo", num_segs);
+                                  " left for mapping jumbo\n", num_segs);
                        qede_recycle_rx_bd_ring(rxq, qdev, num_segs);
                        return -EINVAL;
                }
@@ -1445,7 +1476,9 @@ qede_xmit_prep_pkts(__rte_unused void *p_txq, struct rte_mbuf **tx_pkts,
        uint64_t ol_flags;
        struct rte_mbuf *m;
        uint16_t i;
+#ifdef RTE_LIBRTE_ETHDEV_DEBUG
        int ret;
+#endif
 
        for (i = 0; i < nb_pkts; i++) {
                m = tx_pkts[i];
@@ -1478,14 +1511,6 @@ qede_xmit_prep_pkts(__rte_unused void *p_txq, struct rte_mbuf **tx_pkts,
                        break;
                }
 #endif
-               /* TBD: pseudo csum calcuation required if
-                * ETH_TX_DATA_2ND_BD_L4_PSEUDO_CSUM_MODE not set?
-                */
-               ret = rte_net_intel_cksum_prepare(m);
-               if (ret != 0) {
-                       rte_errno = ret;
-                       break;
-               }
        }
 
 #ifdef RTE_LIBRTE_QEDE_DEBUG_TX
@@ -1496,6 +1521,27 @@ qede_xmit_prep_pkts(__rte_unused void *p_txq, struct rte_mbuf **tx_pkts,
        return i;
 }
 
+#define MPLSINUDP_HDR_SIZE                     (12)
+
+#ifdef RTE_LIBRTE_QEDE_DEBUG_TX
+static inline void
+qede_mpls_tunn_tx_sanity_check(struct rte_mbuf *mbuf,
+                              struct qede_tx_queue *txq)
+{
+       if (((mbuf->outer_l2_len + mbuf->outer_l3_len) / 2) > 0xff)
+               PMD_TX_LOG(ERR, txq, "tunn_l4_hdr_start_offset overflow\n");
+       if (((mbuf->outer_l2_len + mbuf->outer_l3_len +
+               MPLSINUDP_HDR_SIZE) / 2) > 0xff)
+               PMD_TX_LOG(ERR, txq, "tunn_hdr_size overflow\n");
+       if (((mbuf->l2_len - MPLSINUDP_HDR_SIZE) / 2) >
+               ETH_TX_DATA_2ND_BD_TUNN_INNER_L2_HDR_SIZE_W_MASK)
+               PMD_TX_LOG(ERR, txq, "inner_l2_hdr_size overflow\n");
+       if (((mbuf->l2_len - MPLSINUDP_HDR_SIZE + mbuf->l3_len) / 2) >
+               ETH_TX_DATA_2ND_BD_L4_HDR_START_OFFSET_W_MASK)
+               PMD_TX_LOG(ERR, txq, "inner_l2_hdr_size overflow\n");
+}
+#endif
+
 uint16_t
 qede_xmit_pkts(void *p_txq, struct rte_mbuf **tx_pkts, uint16_t nb_pkts)
 {
@@ -1510,9 +1556,10 @@ qede_xmit_pkts(void *p_txq, struct rte_mbuf **tx_pkts, uint16_t nb_pkts)
        uint16_t nb_frags;
        uint16_t nb_pkt_sent = 0;
        uint8_t nbds;
-       bool ipv6_ext_flg;
        bool lso_flg;
+       bool mplsoudp_flg;
        __rte_unused bool tunn_flg;
+       bool tunn_ipv6_ext_flg;
        struct eth_tx_1st_bd *bd1;
        struct eth_tx_2nd_bd *bd2;
        struct eth_tx_3rd_bd *bd3;
@@ -1529,6 +1576,10 @@ qede_xmit_pkts(void *p_txq, struct rte_mbuf **tx_pkts, uint16_t nb_pkts)
        uint16_t mss;
        uint16_t bd3_bf;
 
+       uint8_t tunn_l4_hdr_start_offset;
+       uint8_t tunn_hdr_size;
+       uint8_t inner_l2_hdr_size;
+       uint16_t inner_l4_hdr_offset;
 
        if (unlikely(txq->nb_tx_avail < txq->tx_free_thresh)) {
                PMD_TX_LOG(DEBUG, txq, "send=%u avail=%u free_thresh=%u",
@@ -1540,7 +1591,6 @@ qede_xmit_pkts(void *p_txq, struct rte_mbuf **tx_pkts, uint16_t nb_pkts)
        bd_prod = rte_cpu_to_le_16(ecore_chain_get_prod_idx(&txq->tx_pbl));
        while (nb_tx_pkts--) {
                /* Init flags/values */
-               ipv6_ext_flg = false;
                tunn_flg = false;
                lso_flg = false;
                nbds = 0;
@@ -1555,6 +1605,10 @@ qede_xmit_pkts(void *p_txq, struct rte_mbuf **tx_pkts, uint16_t nb_pkts)
                bd2_bf2 = 0;
                mss = 0;
                bd3_bf = 0;
+               mplsoudp_flg = false;
+               tunn_ipv6_ext_flg = false;
+               tunn_hdr_size = 0;
+               tunn_l4_hdr_start_offset = 0;
 
                mbuf = *tx_pkts++;
                assert(mbuf);
@@ -1566,20 +1620,18 @@ qede_xmit_pkts(void *p_txq, struct rte_mbuf **tx_pkts, uint16_t nb_pkts)
                tx_ol_flags = mbuf->ol_flags;
                bd1_bd_flags_bf |= 1 << ETH_TX_1ST_BD_FLAGS_START_BD_SHIFT;
 
-#define RTE_ETH_IS_IPV6_HDR_EXT(ptype) ((ptype) & RTE_PTYPE_L3_IPV6_EXT)
-               if (RTE_ETH_IS_IPV6_HDR_EXT(mbuf->packet_type)) {
-                       ipv6_ext_flg = true;
+               /* TX prepare would have already checked supported tunnel Tx
+                * offloads. Don't rely on pkt_type marked by Rx, instead use
+                * tx_ol_flags to decide.
+                */
+               if (((tx_ol_flags & PKT_TX_TUNNEL_MASK) ==
+                                               PKT_TX_TUNNEL_VXLAN) ||
+                   ((tx_ol_flags & PKT_TX_TUNNEL_MASK) ==
+                                               PKT_TX_TUNNEL_MPLSINUDP)) {
+                       /* Check against max which is Tunnel IPv6 + ext */
                        if (unlikely(txq->nb_tx_avail <
-                                       ETH_TX_MIN_BDS_PER_IPV6_WITH_EXT_PKT))
-                               break;
-               }
-
-               if (RTE_ETH_IS_TUNNEL_PKT(mbuf->packet_type)) {
-                       if (ipv6_ext_flg) {
-                               if (unlikely(txq->nb_tx_avail <
-                                   ETH_TX_MIN_BDS_PER_TUNN_IPV6_WITH_EXT_PKT))
+                               ETH_TX_MIN_BDS_PER_TUNN_IPV6_WITH_EXT_PKT))
                                        break;
-                       }
                        tunn_flg = true;
                        /* First indicate its a tunnel pkt */
                        bd1_bf |= ETH_TX_DATA_1ST_BD_TUNN_FLAG_MASK <<
@@ -1592,17 +1644,89 @@ qede_xmit_pkts(void *p_txq, struct rte_mbuf **tx_pkts, uint16_t nb_pkts)
                                bd1_bf ^= 1 <<
                                        ETH_TX_DATA_1ST_BD_TUNN_FLAG_SHIFT;
                        }
+
                        /* Outer IP checksum offload */
-                       if (tx_ol_flags & PKT_TX_OUTER_IP_CKSUM) {
+                       if (tx_ol_flags & (PKT_TX_OUTER_IP_CKSUM |
+                                          PKT_TX_OUTER_IPV4)) {
                                bd1_bd_flags_bf |=
                                        ETH_TX_1ST_BD_FLAGS_TUNN_IP_CSUM_MASK <<
                                        ETH_TX_1ST_BD_FLAGS_TUNN_IP_CSUM_SHIFT;
                        }
-                       /* Outer UDP checksum offload */
-                       bd1_bd_flags_bf |=
-                               ETH_TX_1ST_BD_FLAGS_TUNN_L4_CSUM_MASK <<
-                               ETH_TX_1ST_BD_FLAGS_TUNN_L4_CSUM_SHIFT;
-               }
+
+                       /**
+                        * Currently, only inner checksum offload in MPLS-in-UDP
+                        * tunnel with one MPLS label is supported. Both outer
+                        * and inner layers  lengths need to be provided in
+                        * mbuf.
+                        */
+                       if ((tx_ol_flags & PKT_TX_TUNNEL_MASK) ==
+                                               PKT_TX_TUNNEL_MPLSINUDP) {
+                               mplsoudp_flg = true;
+#ifdef RTE_LIBRTE_QEDE_DEBUG_TX
+                               qede_mpls_tunn_tx_sanity_check(mbuf, txq);
+#endif
+                               /* Outer L4 offset in two byte words */
+                               tunn_l4_hdr_start_offset =
+                                 (mbuf->outer_l2_len + mbuf->outer_l3_len) / 2;
+                               /* Tunnel header size in two byte words */
+                               tunn_hdr_size = (mbuf->outer_l2_len +
+                                               mbuf->outer_l3_len +
+                                               MPLSINUDP_HDR_SIZE) / 2;
+                               /* Inner L2 header size in two byte words */
+                               inner_l2_hdr_size = (mbuf->l2_len -
+                                               MPLSINUDP_HDR_SIZE) / 2;
+                               /* Inner L4 header offset from the beggining
+                                * of inner packet in two byte words
+                                */
+                               inner_l4_hdr_offset = (mbuf->l2_len -
+                                       MPLSINUDP_HDR_SIZE + mbuf->l3_len) / 2;
+
+                               /* TODO: There's no DPDK flag to request outer
+                                * L4 checksum offload, so we don't do it.
+                                * bd1_bd_flags_bf |=
+                                *      ETH_TX_1ST_BD_FLAGS_TUNN_L4_CSUM_MASK <<
+                                *      ETH_TX_1ST_BD_FLAGS_TUNN_L4_CSUM_SHIFT;
+                                */
+                               /* Inner L2 size and address type */
+                               bd2_bf1 |= (inner_l2_hdr_size &
+                                       ETH_TX_DATA_2ND_BD_TUNN_INNER_L2_HDR_SIZE_W_MASK) <<
+                                       ETH_TX_DATA_2ND_BD_TUNN_INNER_L2_HDR_SIZE_W_SHIFT;
+                               bd2_bf1 |= (UNICAST_ADDRESS &
+                                       ETH_TX_DATA_2ND_BD_TUNN_INNER_ETH_TYPE_MASK) <<
+                                       ETH_TX_DATA_2ND_BD_TUNN_INNER_ETH_TYPE_SHIFT;
+                               /* Treated as IPv6+Ext */
+                               bd2_bf1 |=
+                                   1 << ETH_TX_DATA_2ND_BD_TUNN_IPV6_EXT_SHIFT;
+
+                               /* Mark inner IPv6 if present */
+                               if (tx_ol_flags & PKT_TX_IPV6)
+                                       bd2_bf1 |=
+                                               1 << ETH_TX_DATA_2ND_BD_TUNN_INNER_IPV6_SHIFT;
+
+                               /* Inner L4 offsets */
+                               if ((tx_ol_flags & (PKT_TX_IPV4 | PKT_TX_IPV6)) &&
+                                    (tx_ol_flags & (PKT_TX_UDP_CKSUM |
+                                                       PKT_TX_TCP_CKSUM))) {
+                                       /* Determines if BD3 is needed */
+                                       tunn_ipv6_ext_flg = true;
+                                       if ((tx_ol_flags & PKT_TX_L4_MASK) ==
+                                                       PKT_TX_UDP_CKSUM) {
+                                               bd2_bf1 |=
+                                                       1 << ETH_TX_DATA_2ND_BD_L4_UDP_SHIFT;
+                                       }
+
+                                       /* TODO other pseudo checksum modes are
+                                        * not supported
+                                        */
+                                       bd2_bf1 |=
+                                       ETH_L4_PSEUDO_CSUM_CORRECT_LENGTH <<
+                                       ETH_TX_DATA_2ND_BD_L4_PSEUDO_CSUM_MODE_SHIFT;
+                                       bd2_bf2 |= (inner_l4_hdr_offset &
+                                               ETH_TX_DATA_2ND_BD_L4_HDR_START_OFFSET_W_MASK) <<
+                                               ETH_TX_DATA_2ND_BD_L4_HDR_START_OFFSET_W_SHIFT;
+                               }
+                       } /* End MPLSoUDP */
+               } /* End Tunnel handling */
 
                if (tx_ol_flags & PKT_TX_TCP_SEG) {
                        lso_flg = true;
@@ -1646,14 +1770,10 @@ qede_xmit_pkts(void *p_txq, struct rte_mbuf **tx_pkts, uint16_t nb_pkts)
                                1 << ETH_TX_1ST_BD_FLAGS_IP_CSUM_SHIFT;
 
                /* L4 checksum offload (tcp or udp) */
-               if (tx_ol_flags & (PKT_TX_TCP_CKSUM | PKT_TX_UDP_CKSUM))
+               if ((tx_ol_flags & (PKT_TX_IPV4 | PKT_TX_IPV6)) &&
+                   (tx_ol_flags & (PKT_TX_UDP_CKSUM | PKT_TX_TCP_CKSUM))) {
                        bd1_bd_flags_bf |=
                                1 << ETH_TX_1ST_BD_FLAGS_L4_CSUM_SHIFT;
-
-               if (ipv6_ext_flg) {
-                       /* TBD: check pseudo csum if tx_prepare not called? */
-                       bd2_bf1 |= ETH_L4_PSEUDO_CSUM_ZERO_LENGTH <<
-                               ETH_TX_DATA_2ND_BD_L4_PSEUDO_CSUM_MODE_SHIFT;
                }
 
                /* Fill the entry in the SW ring and the BDs in the FW ring */
@@ -1667,12 +1787,12 @@ qede_xmit_pkts(void *p_txq, struct rte_mbuf **tx_pkts, uint16_t nb_pkts)
 
                /* Map MBUF linear data for DMA and set in the BD1 */
                QEDE_BD_SET_ADDR_LEN(bd1, rte_mbuf_data_dma_addr(mbuf),
-                                       mbuf->data_len);
-               bd1->data.bitfields = bd1_bf;
+                                    mbuf->data_len);
+               bd1->data.bitfields = rte_cpu_to_le_16(bd1_bf);
                bd1->data.bd_flags.bitfields = bd1_bd_flags_bf;
                bd1->data.vlan = vlan;
 
-               if (lso_flg || ipv6_ext_flg) {
+               if (lso_flg || mplsoudp_flg) {
                        bd2 = (struct eth_tx_2nd_bd *)ecore_chain_produce
                                                        (&txq->tx_pbl);
                        memset(bd2, 0, sizeof(struct eth_tx_2nd_bd));
@@ -1685,16 +1805,30 @@ qede_xmit_pkts(void *p_txq, struct rte_mbuf **tx_pkts, uint16_t nb_pkts)
                        QEDE_BD_SET_ADDR_LEN(bd2, (hdr_size +
                                             rte_mbuf_data_dma_addr(mbuf)),
                                             mbuf->data_len - hdr_size);
-                       bd2->data.bitfields1 = bd2_bf1;
-                       bd2->data.bitfields2 = bd2_bf2;
-
+                       bd2->data.bitfields1 = rte_cpu_to_le_16(bd2_bf1);
+                       if (mplsoudp_flg) {
+                               bd2->data.bitfields2 =
+                                       rte_cpu_to_le_16(bd2_bf2);
+                               /* Outer L3 size */
+                               bd2->data.tunn_ip_size =
+                                       rte_cpu_to_le_16(mbuf->outer_l3_len);
+                       }
                        /* BD3 */
-                       bd3 = (struct eth_tx_3rd_bd *)ecore_chain_produce
-                                                       (&txq->tx_pbl);
-                       memset(bd3, 0, sizeof(struct eth_tx_3rd_bd));
-                       nbds++;
-                       bd3->data.bitfields = bd3_bf;
-                       bd3->data.lso_mss = mss;
+                       if (lso_flg || (mplsoudp_flg && tunn_ipv6_ext_flg)) {
+                               bd3 = (struct eth_tx_3rd_bd *)
+                                       ecore_chain_produce(&txq->tx_pbl);
+                               memset(bd3, 0, sizeof(struct eth_tx_3rd_bd));
+                               nbds++;
+                               bd3->data.bitfields = rte_cpu_to_le_16(bd3_bf);
+                               if (lso_flg)
+                                       bd3->data.lso_mss = mss;
+                               if (mplsoudp_flg) {
+                                       bd3->data.tunn_l4_hdr_start_offset_w =
+                                               tunn_l4_hdr_start_offset;
+                                       bd3->data.tunn_hdr_size_w =
+                                               tunn_hdr_size;
+                               }
+                       }
                }
 
                /* Handle fragmented MBUF */
@@ -1709,8 +1843,7 @@ qede_xmit_pkts(void *p_txq, struct rte_mbuf **tx_pkts, uint16_t nb_pkts)
                    rte_cpu_to_le_16(ecore_chain_get_prod_idx(&txq->tx_pbl));
 #ifdef RTE_LIBRTE_QEDE_DEBUG_TX
                print_tx_bd_info(txq, bd1, bd2, bd3, tx_ol_flags);
-               PMD_TX_LOG(INFO, txq, "lso=%d tunn=%d ipv6_ext=%d\n",
-                          lso_flg, tunn_flg, ipv6_ext_flg);
+               PMD_TX_LOG(INFO, txq, "lso=%d tunn=%d", lso_flg, tunn_flg);
 #endif
                nb_pkt_sent++;
                txq->xmit_pkts++;