> -----Original Message----- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Saturday, June 13, 2015 12:45 AM > To: Wu, Jingjing > Cc: dev at dpdk.org; neil.horman at tuxdriver.com > Subject: Re: [dpdk-dev] [PATCH 3/6] ethdev: extend struct to support flow > director in VFs > > 2015-05-11 11:46, Jingjing Wu: > > This patch extends struct rte_eth_fdir_flow_ext to support flow > > director in VFs. > > > > Signed-off-by: Jingjing Wu <jingjing.wu at intel.com> > > > --- a/lib/librte_ether/rte_eth_ctrl.h > > +++ b/lib/librte_ether/rte_eth_ctrl.h > > @@ -394,6 +394,8 @@ struct rte_eth_fdir_flow_ext { > > uint16_t vlan_tci; > > uint8_t flexbytes[RTE_ETH_FDIR_MAX_FLEXLEN]; > > /**< It is filled by the flexible payload to match. */ > > + uint8_t is_vf; /**< 1 for VF, 0 for port dev */ > > + uint16_t dst_id; /**< VF ID, available when is_vf is 1*/ > > }; > > Isn't it breaking the ABI?
Yes, it is breaking the ABI. Will consider how to avoid that. Thanks Jingjing
