[PATCH v9 04/17] graph: add get/set graph worker model APIs

Jerin Jacob jerinjacobk at gmail.com
Wed Jun 7 15:28:52 CEST 2023


On Wed, Jun 7, 2023 at 5:55 PM Yan, Zhirun <zhirun.yan at intel.com> wrote:
>
> > > +__rte_experimental
> > > +static inline uint32_t
> > > +rte_graph_worker_model_get(struct rte_graph *graph) {
> > > +       if (!graph_model_is_valid(graph->model))
> > > +               return -EINVAL;
> >
> > Introduce rte_graph_worker_model_no_check_get() to skip this check to use
> > with fastpath.
> >
> > rte_graph_worker_model_get can move to .c file.
>
> Yes. Will move in next version.
> Got it. rte_graph_worker_model_no_check_get() will be used in fast path.
> Actually, I don’t find the performance impact about static inline, so should the new

May be due to burst size 32 or 256.it will start impacting if burst
size is less.


> API declared with static inline keywords or put it into .c file also?

Keep in line fastpath function in .h as inline.


More information about the dev mailing list