From 676b1bd223463a08388db0c3cc20e426ea155c71 Mon Sep 17 00:00:00 2001 From: Pawel Wodkowski Date: Mon, 9 Mar 2015 17:05:29 +0100 Subject: [PATCH] examples/l2fwd-jobstats: fix error handling in init sequence 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 67a9bad04f..f990045fe3 100644 --- a/examples/l2fwd-jobstats/main.c +++ b/examples/l2fwd-jobstats/main.c @@ -996,11 +996,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)); } @@ -1018,8 +1018,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)); + } } } -- 2.20.1