-----Original Message----- > Date: Thu, 5 Apr 2018 17:23:26 +0530 > From: Pavan Nikhilesh <pbhagavat...@caviumnetworks.com> > To: jerin.ja...@caviumnetworks.com, santosh.shu...@caviumnetworks.com, > erik.g.carri...@intel.com > Cc: dev@dpdk.org, Pavan Nikhilesh <pbhagavat...@caviumnetworks.com> > Subject: [dpdk-dev] [PATCH v3 1/4] app/eventdev: add event timer adapter as > a producer > X-Mailer: git-send-email 2.16.3 > > Add event timer adapter as producer option that can be selected by > passing --prod_type_timerdev. > > Signed-off-by: Pavan Nikhilesh <pbhagavat...@caviumnetworks.com> > +static inline int > +perf_event_timer_producer(void *arg) > +{ > + struct prod_data *p = arg; > + struct test_perf *t = p->t; > + struct evt_options *opt = t->opt; > + uint32_t flow_counter = 0; > + uint64_t count = 0; > + uint64_t arm_latency = 0; > + const uint8_t nb_timer_adptrs = opt->nb_timer_adptrs; > + const uint32_t nb_flows = t->nb_flows; > + const uint64_t nb_timers = opt->nb_timers; > + struct rte_mempool *pool = t->pool; > + struct perf_elt *m = NULL;
NULL assignment is not required. > + struct rte_event_timer_adapter **adptr = t->timer_adptr; > + uint64_t timeout_ticks = opt->optm_bkt_tck_nsec ? > + (opt->nb_bkt_tcks * opt->bkt_tck_nsec) > + / opt->optm_bkt_tck_nsec : opt->nb_bkt_tcks; > + > struct perf_elt { > + struct rte_event_timer tim; This will make perf_elf two Cacheline, use union + offset of scheme to fit timestamp on the first cacheline. > uint64_t timestamp; > } __rte_cache_aligned; With above changes: Acked-by: Jerin Jacob <jerin.ja...@caviumnetworks.com>