Hi Thomas, On Sun, Jul 09, 2017 at 10:01:31PM +0200, Thomas Monjalon wrote: > Hi, > > 04/07/2017 17:38, Cristian Dumitrescu: > > http://dpdk.org/git/next/dpdk-next-tm > > I'm sorry to not have considered this tree as a high priority. > I think it may be integrated in RC2 because it is a totally new area > and should not break any existing code. > > I prefer to wait because I have seen some things to fix: > > 1/ There is a compilation error with clang (notified in related thread). > > 2/ Some functions are exposed in the API to query the ops. > It seems dangerous and useless: > - rte_eth_dev_tm_ops_get > - rte_tm_ops_get > > 3/ The PMD interface file is referenced in the doxygen index: > + [rte_tm_driver] (@ref rte_tm_driver.h), > I see that rte_flow_driver.h is also referenced but it seems a mistake.
Why? It was done on purpose, I think exposing through Doxygen internal APIs implemented by drivers must be a requirement, even if the exposed symbols are not necessarily versioned. -- Adrien Mazarguil 6WIND
