2015-04-20 09:45, Venky Venkatesan: > On 04/20/2015 09:33 AM, Neil Horman wrote: > > On Mon, Apr 20, 2015 at 04:11:43PM +0200, Thomas Monjalon wrote: > >> It's time to remove this old API. > >> It seems some work is still needed to rely only on eth_ctrl API. > >> At least ixgbe, i40e and testpmd must be fixed. > >> Jingjing, do you think it's possible to remove all these structures > >> from rte_ethdev.h? > >> > >> Thanks > >> > > NAK. > > > > I'm certainly not opposed to removing the API's if they are truly no longer > > needed. But they have been codified as part of the ABI, so the deprecation > > schedule needs to be followed. Given what you've said above, it seems like > > that > > might be worthwhile anyway, as it will provide the needed runway to allow > > users > > to convert to the new API. > > > > Neil > +1 NAK. Agree with Neil.
+1 Agree with you :) The goal of this RFC proposal is to see how to progress on API cleanup. There are actually 2 parts: 1/ The flow director functions of rte_ethdev.h were only used for enic in DPDK 2.0. We could set a deprecation notice to remove them in DPDK 2.2. 2/ Some associated structures are also used for rte_eth_conf. My question was to check how it would be relevant to remove this rte_fdir_conf.
