On 2/21/23 01:41, Ivan Malov wrote:
> This may be required by some PMDs in offload scenarios.
>
> Signed-off-by: Ivan Malov <ivan.ma...@arknetworks.am>
> ---
> lib/netdev-dpdk.c | 40 ++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 40 insertions(+)
>
> diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c
> index fb0dd43f7..2cebc3cca 100644
> --- a/lib/netdev-dpdk.c
> +++ b/lib/netdev-dpdk.c
> @@ -1140,6 +1140,36 @@ dpdk_eth_flow_ctrl_setup(struct netdev_dpdk *dev)
> OVS_REQUIRES(dev->mutex)
> }
> }
>
> +static void
> +dpdk_eth_dev_init_rx_metadata(struct netdev_dpdk *dev)
> +{
> + uint64_t rx_metadata = 0;
> + int ret;
> +
> + /* For the fallback offload (non-"transfer" rules) */
> + rx_metadata |= RTE_ETH_RX_METADATA_USER_MARK;
> + /* For the full offload ("transfer" rules) */
> + rx_metadata |= RTE_ETH_RX_METADATA_TUNNEL_ID;
> +
> + ret = rte_eth_rx_metadata_negotiate(dev->port_id, &rx_metadata);
This call is triggering errors:
2023-02-21T01:55:42.586Z|00145|dpdk|ERR|The port (ID=0) is already configured
Looks like we're not allowed to call this function after device was already
configured. Any attempt to re-configure triggers an error. And that fails CI.
Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
