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>
