-----Original Message----- > Date: Thu, 20 Sep 2018 03:52:33 +0530 > From: Pavan Nikhilesh <pbhagavat...@caviumnetworks.com> > To: jerin.ja...@caviumnetworks.com, nikhil....@intel.com, > anoob.jos...@caviumnetworks.com > Cc: dev@dpdk.org, Pavan Nikhilesh <pbhagavat...@caviumnetworks.com> > Subject: [dpdk-dev] [PATCH v3 2/4] app/test-eventdev: remove redundant > newlines > X-Mailer: git-send-email 2.18.0 > > Signed-off-by: Pavan Nikhilesh <pbhagavat...@caviumnetworks.com>
Please fix check-git-log.sh errors. ### app/test-eventdev: remove redundant newlines WARNING:COMMIT_MESSAGE: Missing commit description - Add an appropriate one With above change: Acked-by: Jerin Jacob <jerin.ja...@caviumnetworks.com> > --- > app/test-eventdev/test_pipeline_common.c | 21 ++++++++++----------- > 1 file changed, 10 insertions(+), 11 deletions(-) > > diff --git a/app/test-eventdev/test_pipeline_common.c > b/app/test-eventdev/test_pipeline_common.c > index 12a58fbbf..cdf9010b4 100644 > --- a/app/test-eventdev/test_pipeline_common.c > +++ b/app/test-eventdev/test_pipeline_common.c > @@ -65,12 +65,12 @@ pipeline_test_result(struct evt_test *test, struct > evt_options *opt) > uint64_t total = 0; > struct test_pipeline *t = evt_test_priv(test); > > - printf("Packet distribution across worker cores :\n"); > + evt_info("Packet distribution across worker cores :"); > for (i = 0; i < t->nb_workers; i++) > total += t->worker[i].processed_pkts; > for (i = 0; i < t->nb_workers; i++) > - printf("Worker %d packets: "CLGRN"%"PRIx64" "CLNRM"percentage:" > - CLGRN" %3.2f\n"CLNRM, i, > + evt_info("Worker %d packets: "CLGRN"%"PRIx64""CLNRM" > percentage:" > + CLGRN" %3.2f"CLNRM, i, > t->worker[i].processed_pkts, > (((double)t->worker[i].processed_pkts)/total) > * 100); > @@ -233,7 +233,7 @@ pipeline_ethdev_setup(struct evt_test *test, struct > evt_options *opt) > > RTE_SET_USED(opt); > if (!rte_eth_dev_count_avail()) { > - evt_err("No ethernet ports found.\n"); > + evt_err("No ethernet ports found."); > return -ENODEV; > } > > @@ -252,7 +252,7 @@ pipeline_ethdev_setup(struct evt_test *test, struct > evt_options *opt) > if (local_port_conf.rx_adv_conf.rss_conf.rss_hf != > port_conf.rx_adv_conf.rss_conf.rss_hf) { > evt_info("Port %u modified RSS hash function based on > hardware support," > - "requested:%#"PRIx64" configured:%#"PRIx64"\n", > + "requested:%#"PRIx64" configured:%#"PRIx64"", > i, > port_conf.rx_adv_conf.rss_conf.rss_hf, > local_port_conf.rx_adv_conf.rss_conf.rss_hf); > @@ -261,19 +261,19 @@ pipeline_ethdev_setup(struct evt_test *test, struct > evt_options *opt) > if (rte_eth_dev_configure(i, nb_queues, nb_queues, > &local_port_conf) > < 0) { > - evt_err("Failed to configure eth port [%d]\n", i); > + evt_err("Failed to configure eth port [%d]", i); > return -EINVAL; > } > > if (rte_eth_rx_queue_setup(i, 0, NB_RX_DESC, > rte_socket_id(), &rx_conf, t->pool) < 0) { > - evt_err("Failed to setup eth port [%d] rx_queue: %d.\n", > + evt_err("Failed to setup eth port [%d] rx_queue: %d.", > i, 0); > return -EINVAL; > } > if (rte_eth_tx_queue_setup(i, 0, NB_TX_DESC, > rte_socket_id(), NULL) < 0) { > - evt_err("Failed to setup eth port [%d] tx_queue: %d.\n", > + evt_err("Failed to setup eth port [%d] tx_queue: %d.", > i, 0); > return -EINVAL; > } > @@ -379,7 +379,7 @@ pipeline_event_rx_adapter_setup(struct evt_options *opt, > uint8_t stride, > ret = evt_service_setup(service_id); > if (ret) { > evt_err("Failed to setup service core" > - " for Rx adapter\n"); > + " for Rx adapter"); > return ret; > } > } > @@ -396,8 +396,7 @@ pipeline_event_rx_adapter_setup(struct evt_options *opt, > uint8_t stride, > evt_err("Rx adapter[%d] start failed", prod); > return ret; > } > - printf("%s: Port[%d] using Rx adapter[%d] started\n", __func__, > - prod, prod); > + evt_info("Port[%d] using Rx adapter[%d] started", prod, prod); > } > > return ret; > -- > 2.19.0 >