> -----Original Message-----
> From: pbhagavat...@marvell.com <pbhagavat...@marvell.com>
> Sent: Sunday, June 30, 2019 11:23 PM
> To: Jerin Jacob Kollanukkaran <jer...@marvell.com>
> Cc: dev@dpdk.org; Pavan Nikhilesh Bhagavatula
> <pbhagavat...@marvell.com>; sta...@dpdk.org
> Subject: [dpdk-dev][PATCH 2/2] test/eventdev: fix producer core validity
> checks
> 
> From: Pavan Nikhilesh <pbhagavat...@marvell.com>
> 
> When producer type is event timer adapter producer lcore checks are
> skipped. Since, timer adapter relies on SW to arm timers producer lcore is
> essential for its functionality.
> Verify producer lcore validity when producer type is event timer adapter.
> 
> Cc: sta...@dpdk.org
> Fixes: b01974da9f25 ("app/eventdev: add ethernet device producer option")
> 
> Signed-off-by: Pavan Nikhilesh <pbhagavat...@marvell.com>



Acked-by: Jerin Jacob <jer...@marvell.com>


>  app/test-eventdev/test_perf_common.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/app/test-eventdev/test_perf_common.c b/app/test-
> eventdev/test_perf_common.c
> index 01f782820..2e9e83e88 100644
> --- a/app/test-eventdev/test_perf_common.c
> +++ b/app/test-eventdev/test_perf_common.c
> @@ -562,7 +562,8 @@ perf_opt_check(struct evt_options *opt, uint64_t
> nb_queues)
>               return -1;
>       }
> 
> -     if (opt->prod_type == EVT_PROD_TYPE_SYNT) {
> +     if (opt->prod_type == EVT_PROD_TYPE_SYNT ||
> +                     opt->prod_type ==
> EVT_PROD_TYPE_EVENT_TIMER_ADPTR) {
>               /* Validate producer lcores */
>               if (evt_lcores_has_overlap(opt->plcores,
>                                       rte_get_master_lcore())) {
> --
> 2.21.0

Reply via email to