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

[Stephen Hemminger <stephen@networkplumber.org>]

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

> +static const struct eth_dev_ops sxevf_eth_dev_ops = {
> +	.dev_configure		= sxevf_dev_configure,
> +	.dev_start			= sxevf_dev_start,
> +	.dev_stop			 = sxevf_dev_stop,
> +	.link_update		  = sxevf_link_update,
> +	.stats_get			= sxevf_eth_stats_get,
> +	.xstats_get		   = sxevf_xstats_get,
> +	.stats_reset		  = sxevf_dev_stats_reset,
> +	.xstats_reset		 = sxevf_dev_stats_reset,
> +	.xstats_get_names	 = sxevf_xstats_names_get,
> +	.dev_close			= sxevf_dev_close,
> +	.dev_reset		  = sxevf_dev_reset,
> +	.promiscuous_enable   = sxevf_promiscuous_enable,
> +	.promiscuous_disable  = sxevf_promiscuous_disable,
> +	.allmulticast_enable  = sxevf_allmulticast_enable,
> +	.allmulticast_disable = sxevf_allmulticast_disable,
> +	.dev_infos_get		= sxevf_dev_info_get,
> +	.dev_supported_ptypes_get = sxevf_dev_supported_ptypes_get,
> +	.mtu_set			  = sxevf_mtu_set,
> +	.vlan_filter_set	  = sxevf_vlan_filter_set,
> +	.vlan_strip_queue_set = sxevf_vlan_strip_queue_set,
> +	.vlan_offload_set	 = sxevf_vlan_offload_set,
> +	.rx_queue_setup	   = sxevf_rx_queue_setup,
> +	.rx_queue_release	 = sxevf_rx_queue_release,
> +	.tx_queue_setup	   = sxevf_tx_queue_setup,
> +	.tx_queue_release	 = sxevf_tx_queue_release,
> +	.rx_queue_intr_enable = sxevf_rx_queue_intr_enable,
> +	.rx_queue_intr_disable = sxevf_rx_queue_intr_disable,
> +	.mac_addr_add		 = sxevf_mac_addr_add,
> +	.mac_addr_remove	  = sxevf_mac_addr_remove,
> +	.set_mc_addr_list	 = sxevf_set_mc_addr_list,
> +	.rxq_info_get		 = sxevf_rx_queue_info_get,
> +	.txq_info_get		 = sxevf_tx_queue_info_get,
> +	.mac_addr_set		 = sxevf_default_mac_addr_set,
> +	.get_reg			  = sxevf_get_regs,
> +	.reta_update		  = sxevf_rss_reta_update,
> +	.reta_query		   = sxevf_rss_reta_query,
> +	.rss_hash_update	  = sxevf_rss_hash_update,
> +	.rss_hash_conf_get	= sxevf_rss_hash_conf_get,
> +	.tx_done_cleanup	  = sxevf_tx_done_cleanup,
> +#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
> +};
> +

Another ragged looking table, align the right group.