On 6/16/2017 6:41 AM, Shreyansh Jain wrote:
> Signed-off-by: Hemant Agrawal <[email protected]>
> Signed-off-by: Shreyansh Jain <[email protected]>
<...>
> @@ -363,6 +439,18 @@ dpaa_eth_queue_tx(void *q, struct rte_mbuf **bufs,
> uint16_t nb_bufs)
> }
> rte_pktmbuf_free(mbuf);
> }
> + if (mbuf->ol_flags &
> DPAA_TX_CKSUM_OFFLOAD_MASK) {
> + if (mbuf->data_off <
> DEFAULT_TX_ICEOF +
> + sizeof(struct
> dpaa_eth_parse_results_t)) {
> + PMD_DRV_LOG(DEBUG,
> "Checksum offload Err: "
> + "Not enough
> Headroom "
> + "space for
> correct Checksum offload."
> + "So Calculating
> checksum in Software.");
> + dpaa_checksum(mbuf);
> + } else
> +
> dpaa_checksum_offload(mbuf, &fd_arr[loop],
> + mbuf->buf_addr);
> + }
There is a tx_pkt_prepare() dev_ops.
Does it make sense to move this calculations to that function?
> } else {
> PMD_DRV_LOG(DEBUG, "Number of Segments
> not supported");
> /* Set frames_to_send & nb_bufs so that
<...>
