Re: [PATCH v12 13/13] net/sxe: add virtual function

Stephen Hemminger
2025-10-24T17:11:51Z

On Mon, 28 Jul 2025 17:05:17 +0800
liujie5@linkdatatechnology.com wrote:

> +#ifdef ETH_DEV_OPS_MONITOR
> + .get_monitor_addr = sxe_monitor_addr_get,
> +#endif
> +#ifdef ETH_DEV_OPS_HAS_DESC_RELATE
> + .rx_descriptor_status = sxevf_rx_descriptor_status,
> + .tx_descriptor_status = sxevf_tx_descriptor_status,
> +#ifdef ETH_DEV_RX_DESC_DONE
> + .rx_descriptor_done = sxevf_rx_descriptor_done,
> +#endif
> +#endif
> +};
> +

Please don't add so many #ifdef.
The new driver is going to only be for the current version of the
DPDK, we don't want to keep all the ETH_DEV_OPS_XXX ifdef's.