Hi Pavan, One comment in-line:
> -----Original Message----- > From: Pavan Nikhilesh [mailto:pbhagavat...@caviumnetworks.com] > Sent: Tuesday, April 3, 2018 11:02 AM > To: jerin.ja...@caviumnetworks.com; > santosh.shu...@caviumnetworks.com; Carrillo, Erik G > <erik.g.carri...@intel.com> > Cc: dev@dpdk.org; Pavan Nikhilesh <pbhagavat...@caviumnetworks.com> > Subject: [dpdk-dev] [PATCH v2 1/4] app/eventdev: add event timer adapter > as a producer > > Add event timer adapter as producer option that can be selected by passing - > -prod_type_timerdev. > > Signed-off-by: Pavan Nikhilesh <pbhagavat...@caviumnetworks.com> > --- <...snipped...> > diff --git a/app/test-eventdev/evt_options.h b/app/test- > eventdev/evt_options.h index 46d122229..37debae84 100644 > --- a/app/test-eventdev/evt_options.h > +++ b/app/test-eventdev/evt_options.h > @@ -31,12 +31,14 @@ > #define EVT_FWD_LATENCY ("fwd_latency") > #define EVT_QUEUE_PRIORITY ("queue_priority") > #define EVT_PROD_ETHDEV ("prod_type_ethdev") > +#define EVT_PROD_TIMERDEV ("prod_type_timerdev") > #define EVT_HELP ("help") > > enum evt_prod_type { > EVT_PROD_TYPE_NONE, > EVT_PROD_TYPE_SYNT, /* Producer type Synthetic i.e. CPU. */ > EVT_PROD_TYPE_ETH_RX_ADPTR, /* Producer type Eth Rx Adapter. > */ > + EVT_PROD_TYPE_EVENT_TIMER_ADPTR, /* Producer type Eth Rx > Adapter. */ Comment didn't get updated here. > EVT_PROD_TYPE_MAX, > }; > > @@ -52,11 +54,18 @@ struct evt_options { > int nb_stages; > int verbose_level; > uint64_t nb_pkts; > + uint8_t nb_timer_adptrs; > + uint64_t nb_timers; > + uint64_t bkt_tck_nsec; > + uint64_t optm_bkt_tck_nsec; > + uint64_t max_tmo_nsec; > + uint64_t nb_bkt_tcks; > uint16_t wkr_deq_dep; > uint8_t dev_id; > uint32_t fwd_latency:1; > uint32_t q_priority:1; > enum evt_prod_type prod_type; > + uint8_t timdev_cnt; > }; > > void evt_options_default(struct evt_options *opt); diff --git a/app/test- > eventdev/test_perf_atq.c b/app/test-eventdev/test_perf_atq.c > index b36b22a77..b3a312722 100644 <...snipped...>
