d/numam-dpdk
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

net/ark: remove use of seqn for debug

Browse Source 
The seqn mbuf field is deprecated.
It is currently hacked for debug purpose, it could be changed to a
dynamic field but I see little value in the debug info it offers.

Signed-off-by: David Marchand <david.marchand@redhat.com>
Acked-by: Ed Czeck <ed.czeck@atomicrules.com>
This commit is contained in:
David Marchand 2020-10-28 13:20:07 +01:00 committed by Thomas Monjalon
parent 2601dcffd5
commit fea8ea1883
1 changed files with 2 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
drivers/net/ark/ark_ethdev_rx.c
View File

@ -302,8 +302,6 @@ eth_ark_recv_pkts(void *rx_queue,
mbuf->pkt_len = 63; mbuf->pkt_len = 63;
meta->pkt_len = 63; meta->pkt_len = 63;
} }
/* seqn is only set under debug */
mbuf->seqn = cons_index;
} }
if (unlikely(meta->pkt_len > ARK_RX_MAX_NOCHAIN)) if (unlikely(meta->pkt_len > ARK_RX_MAX_NOCHAIN))
@ -360,8 +358,6 @@ eth_ark_rx_jumbo(struct ark_rx_queue *queue,
mbuf_prev = mbuf; mbuf_prev = mbuf;
mbuf->data_len = data_len; mbuf->data_len = data_len;
mbuf->data_off = 0; mbuf->data_off = 0;
if (ARK_DEBUG_CORE)
mbuf->seqn = cons_index; /* for debug only */
cons_index += 1; cons_index += 1;
} }
@ -667,8 +663,8 @@ dump_mbuf_data(struct rte_mbuf *mbuf, uint16_t lo, uint16_t hi)
{ {
uint16_t i, j; uint16_t i, j;
ARK_PMD_LOG(DEBUG, " MBUF: %p len %d, off: %d, seq: %" PRIU32 "\n", ARK_PMD_LOG(DEBUG, " MBUF: %p len %d, off: %d\n",
mbuf, mbuf->pkt_len, mbuf->data_off, mbuf->seqn); mbuf, mbuf->pkt_len, mbuf->data_off);
for (i = lo; i < hi; i += 16) { for (i = lo; i < hi; i += 16) {
uint8_t *dp = RTE_PTR_ADD(mbuf->buf_addr, i); uint8_t *dp = RTE_PTR_ADD(mbuf->buf_addr, i);