From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pawel Wodkowski Subject: [PATCH 1/2] example: fix minor bug in l2fwd-jobstats init sequence Date: Mon, 9 Mar 2015 17:05:29 +0100 Message-ID: <1425917129-5090-1-git-send-email-pawelx.wodkowski@intel.com> To: dev-VfR2kkLFssw@public.gmane.org Return-path: List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" Fix check of returned values during application init phase. Signed-off-by: Pawel Wodkowski --- examples/l2fwd-jobstats/main.c | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/examples/l2fwd-jobstats/main.c b/examples/l2fwd-jobstats/main.c index a5a1aaa..ba9d3c0 100644 --- a/examples/l2fwd-jobstats/main.c +++ b/examples/l2fwd-jobstats/main.c @@ -997,11 +997,11 @@ main(int argc, char **argv) drain_tsc, 0); rte_timer_init(&qconf->flush_timer); - rte_timer_reset(&qconf->flush_timer, drain_tsc, PERIODICAL, lcore_id, - &l2fwd_flush_job, NULL); + ret = rte_timer_reset(&qconf->flush_timer, drain_tsc, PERIODICAL, + lcore_id, &l2fwd_flush_job, NULL); if (ret < 0) { - rte_exit(1, "Failed to add flush job for lcore %u: %s", + rte_exit(1, "Failed to reset flush job timer for lcore %u: %s", lcore_id, rte_strerror(-ret)); } @@ -1019,8 +1019,13 @@ main(int argc, char **argv) rte_jobstats_set_update_period_function(job, l2fwd_job_update_cb); rte_timer_init(&qconf->rx_timers[i]); - rte_timer_reset(&qconf->rx_timers[i], 0, PERIODICAL, lcore_id, + ret = rte_timer_reset(&qconf->rx_timers[i], 0, PERIODICAL, lcore_id, &l2fwd_fwd_job, (void *)(uintptr_t)i); + + if (ret < 0) { + rte_exit(1, "Failed to reset lcore %u port %u job timer: %s", + lcore_id, qconf->rx_port_list[i], rte_strerror(-ret)); + } } } -- 1.9.1