From: Pavan Nikhilesh <pbhagavatula@marvell.com> Previously, the total number of event timers per adapter was set to an arbitrary value, set it to mempool size instead as it defines the max event timers that can be armed. Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com> --- app/test-eventdev/test_perf_common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/test-eventdev/test_perf_common.c b/app/test-eventdev/test_perf_common.c index 24ece75c3..8dbfd8181 100644 --- a/app/test-eventdev/test_perf_common.c +++ b/app/test-eventdev/test_perf_common.c @@ -417,7 +417,7 @@ perf_event_timer_adapter_setup(struct test_perf *t) .timer_adapter_id = i, .timer_tick_ns = t->opt->timer_tick_nsec, .max_tmo_ns = t->opt->max_tmo_nsec, - .nb_timers = 2 * 1024 * 1024, + .nb_timers = t->opt->pool_sz, .flags = flags, }; -- 2.21.0
On Thu, 2019-02-28 at 18:48 +0000, Pavan Nikhilesh Bhagavatula wrote: > From: Pavan Nikhilesh <pbhagavatula@marvell.com> > > Previously, the total number of event timers per adapter was set to > an > arbitrary value, set it to mempool size instead as it defines the max > event timers that can be armed. > > Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com> > --- Please fix the below git-checklog.sh issue Headline too long: app/eventdev: configure optimum event timers per timer adapter With above change: Acked-by: Jerin Jacob <jerinj@marvell.com> > app/test-eventdev/test_perf_common.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/app/test-eventdev/test_perf_common.c b/app/test- > eventdev/test_perf_common.c > index 24ece75c3..8dbfd8181 100644 > --- a/app/test-eventdev/test_perf_common.c > +++ b/app/test-eventdev/test_perf_common.c > @@ -417,7 +417,7 @@ perf_event_timer_adapter_setup(struct test_perf > *t) > .timer_adapter_id = i, > .timer_tick_ns = t->opt->timer_tick_nsec, > .max_tmo_ns = t->opt->max_tmo_nsec, > - .nb_timers = 2 * 1024 * 1024, > + .nb_timers = t->opt->pool_sz, > .flags = flags, > }; >
From: Pavan Nikhilesh <pbhagavatula@marvell.com> Previously, the total number of event timers per adapter was set to an arbitrary value, set it to mempool size instead as it defines the max event timers that can be armed. Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com> Acked-by: Jerin Jacob <jerinj@marvell.com> --- app/test-eventdev/test_perf_common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/test-eventdev/test_perf_common.c b/app/test-eventdev/test_perf_common.c index f99a6a607..d14034e52 100644 --- a/app/test-eventdev/test_perf_common.c +++ b/app/test-eventdev/test_perf_common.c @@ -432,7 +432,7 @@ perf_event_timer_adapter_setup(struct test_perf *t) .timer_adapter_id = i, .timer_tick_ns = t->opt->timer_tick_nsec, .max_tmo_ns = t->opt->max_tmo_nsec, - .nb_timers = 2 * 1024 * 1024, + .nb_timers = t->opt->pool_sz, .flags = flags, }; -- 2.21.0
On Mon, 2019-03-11 at 06:49 +0000, Pavan Nikhilesh Bhagavatula wrote: > From: Pavan Nikhilesh <pbhagavatula@marvell.com> > > Previously, the total number of event timers per adapter was set to > an > arbitrary value, set it to mempool size instead as it defines the max > event timers that can be armed. > > Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com> > Acked-by: Jerin Jacob <jerinj@marvell.com> Applied to dpdk-next-eventdev/master. Thanks. > --- > app/test-eventdev/test_perf_common.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/app/test-eventdev/test_perf_common.c b/app/test- > eventdev/test_perf_common.c > index f99a6a607..d14034e52 100644 > --- a/app/test-eventdev/test_perf_common.c > +++ b/app/test-eventdev/test_perf_common.c > @@ -432,7 +432,7 @@ perf_event_timer_adapter_setup(struct test_perf > *t) > .timer_adapter_id = i, > .timer_tick_ns = t->opt->timer_tick_nsec, > .max_tmo_ns = t->opt->max_tmo_nsec, > - .nb_timers = 2 * 1024 * 1024, > + .nb_timers = t->opt->pool_sz, > .flags = flags, > }; >