Re: [PATCH v3 1/1] jobstats: added function abort for job

[Thomas Monjalon <thomas.monjalon@6wind.com>]

2016-02-16 13:19, Zhang, Roy Fan:
> 
> On 12/02/2016 16:04, Marcin Kerlin wrote:
> > This patch adds new function rte_jobstats_abort. It marks *job* as finished and
> > time of this work will be add to management time instead of execution time. This
> > function should be used instead of rte_jobstats_finish if condition occurs,
> > condition is defined by the application for example when receiving n>0 packets.
> > Example of usage is added to the example l2fwd-jobstats. At maximum load do-while
> > loop inside Idle job will be execute once because one or more jobs waiting to be
> > executed, so this time should not be include as the execution time by calling
> > rte_jobstats_abort().
> >
> > v2:
> > * removed redundant field
> > v3:
> > * added an example of using
> >
> > Signed-off-by: Marcin Kerlin <marcinx.kerlin@intel.com>
[...]
> > --- a/lib/librte_jobstats/rte_jobstats_version.map
> > +++ b/lib/librte_jobstats/rte_jobstats_version.map
> > @@ -17,3 +17,10 @@ DPDK_2.0 {
> >   
> >   	local: *;
> >   };
> > +
> > +DPDK_2.3 {

updated to 16.04

> > +	global:
> > +
> > +	rte_jobstats_abort;
> > +
> > +} DPDK_2.0;
> 
> Acked-by : Fan Zhang<roy.fan.zhang@intel.com>

Applied, thanks