2016-06-15 11:29, Hunt, David: > > On 15/6/2016 11:14 AM, Jan Viktorin wrote: > > On Wed, 15 Jun 2016 08:47:02 +0100 > > David Hunt <david.hunt at intel.com> wrote: > > > > [...] > > > > >> + > >> +/** Array of registered ops structs. */ > >> +extern struct rte_mempool_ops_table rte_mempool_ops_table; > >> + > >> +/** > >> + * @internal Get the mempool ops struct from its index. > >> + * > >> + * @param ops_index > >> + * The index of the ops struct in the ops struct table. It must be a > >> valid > >> + * index: (0 <= idx < num_ops). > >> + * @return > >> + * The pointer to the ops struct in the table. > >> + */ > >> +static inline struct rte_mempool_ops * > >> +rte_mempool_ops_get(int ops_index) > > Shouldn't this function be called rte_mempool_get/find_ops instead? > > Jan, > > I think at this stage that it's probably OK as it is. :)
? What is the justification?