25/05/2020 11:11, Andrew Rybchenko: > On 5/25/20 2:18 AM, Thomas Monjalon wrote: > > 04/03/2020 10:57, Ferruh Yigit: > >> For the ABI compatibility it is better to hide internal data structures > >> from the application as much as possible. But because of some inline > >> functions 'struct eth_dev_ops' can't be hidden completely. > >> > >> Plan is to split the 'struct eth_dev_ops' into two as ones used by > >> inline functions and ones not used, and hide the second part that not > >> used by inline functions completely to the application. > >> > >> Because of ABI break the work will be done in 20.11 > >> > >> Signed-off-by: Ferruh Yigit <ferruh.yi...@intel.com> > >> --- > >> +* ethdev: Split the ``struct eth_dev_ops`` struct to hide it as much as > >> possible > >> + will be done in 20.11. > >> + Currently the ``struct eth_dev_ops`` struct is accessible by the > >> application > >> + because some inline functions, like ``rte_eth_tx_descriptor_status()``, > >> + access the struct directly. > >> + The struct will be separate in two, the ops used by inline functions > >> will be moved > >> + next to Rx/Tx burst functions, rest of the ``struct eth_dev_ops`` > >> struct will be > >> + moved to header file for drivers to hide it from applications. > > Acked-by: Thomas Monjalon <tho...@monjalon.net> > > Acked-by: Andrew Rybchenko <arybche...@solarflare.com>
Acked-by: David Marchand <david.march...@redhat.com> Applied, thanks
