Skip to content

Commit

Permalink
eventdev/eth_rx: fix timestamp field register in mbuf
Browse files Browse the repository at this point in the history
[ upstream commit 5083736a4752e0701002297f871325c57568e1d0 ]

For eventdev internal port, timestamp dynamic field registration
in mbuf is not required as that will be done from net device.
For SW eventdev, Rx timestamp field registration will be
done during rxa service initialization.

Fixes: 83ab470 ("eventdev/eth_rx: use timestamp as dynamic mbuf field")

Signed-off-by: Rahul Bhansali <rbhansali@marvell.com>
Acked-by: Naga Harish K S V <s.v.naga.harish.k@intel.com>
  • Loading branch information
Rahul Bhansali authored and kevintraynor committed Oct 31, 2023
1 parent 2ab27f3 commit 9ae9b97
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions lib/eventdev/rte_event_eth_rx_adapter.c
Original file line number Diff line number Diff line change
Expand Up @@ -1857,6 +1857,13 @@ rxa_init_service(struct event_eth_rx_adapter *rx_adapter, uint8_t id)
if (rx_adapter->service_inited)
return 0;

if (rte_mbuf_dyn_rx_timestamp_register(
&event_eth_rx_timestamp_dynfield_offset,
&event_eth_rx_timestamp_dynflag) != 0) {
RTE_EDEV_LOG_ERR("Error registering timestamp field in mbuf\n");
return -rte_errno;
}

memset(&service, 0, sizeof(service));
snprintf(service.name, ETH_RX_ADAPTER_SERVICE_NAME_LEN,
"rte_event_eth_rx_adapter_%d", id);
Expand Down Expand Up @@ -2421,13 +2428,6 @@ rxa_create(uint8_t id, uint8_t dev_id,
if (conf_cb == rxa_default_conf_cb)
rx_adapter->default_cb_arg = 1;

if (rte_mbuf_dyn_rx_timestamp_register(
&event_eth_rx_timestamp_dynfield_offset,
&event_eth_rx_timestamp_dynflag) != 0) {
RTE_EDEV_LOG_ERR("Error registering timestamp field in mbuf\n");
return -rte_errno;
}

rte_eventdev_trace_eth_rx_adapter_create(id, dev_id, conf_cb,
conf_arg);
return 0;
Expand Down

0 comments on commit 9ae9b97

Please sign in to comment.