> -----Original Message-----
> From: Pei, Andy
> Sent: Monday, July 8, 2019 11:03 AM
> To: [email protected]
> Cc: Pei, Andy <[email protected]>; Zhang, Qi Z <[email protected]>; Wu,
> Jingjing <[email protected]>; Xing, Beilei <[email protected]>; Yigit,
> Ferruh <[email protected]>; Xu, Rosen <[email protected]>; Ye,
> Xiaolong <[email protected]>; Zhang, Roy Fan
> <[email protected]>; [email protected]
> Subject: [PATCH v4] net/i40e: i40e get link status update from ipn3ke
>
> Add switch_mode argument for i40e PF to specify the specific FPGA that i40e
> PF is connected to. i40e PF get link status update via the connected FPGA.
> Add switch_ethdev to rte_eth_dev_data to track the bind switch device.
> Try to bind i40e pf to switch device when i40e device is probed. If it fail
> to find
> correct switch device, bind will occur again when update i40e device link
> status.
>
> Signed-off-by: Andy Pei <[email protected]>
.....
> diff --git a/lib/librte_ethdev/rte_ethdev_core.h
> b/lib/librte_ethdev/rte_ethdev_core.h
> index 2922d5b..62adc5c 100644
> --- a/lib/librte_ethdev/rte_ethdev_core.h
> +++ b/lib/librte_ethdev/rte_ethdev_core.h
> @@ -635,6 +635,10 @@ struct rte_eth_dev_data {
> /**< Switch-specific identifier.
> * Valid if RTE_ETH_DEV_REPRESENTOR in dev_flags.
> */
> + struct rte_eth_dev *switch_ethdev;
> + /* point to switch_ethdev specific by "switch_mode" in
> + * devargs
> + */
> } __rte_cache_aligned;
Missing below comment in previous review.
No need to add new field in rte_eth_dev_data to break ABI, you can just add to
i40e specific structure: i40e_pf.
>
> /**
> --
> 1.8.3.1
