X-Git-Url: http://git.droids-corp.org/?a=blobdiff_plain;f=examples%2Fmulti_process%2Fl2fwd_fork%2Fmain.c;h=59c102434a37d545f983371dfe274f9ca8b601e9;hb=a1e7c17555e8f77d520ba5f06ed26c00e77a2bd1;hp=a34f420bbc86c5ce4b449242777a91b94eab4c6c;hpb=e5ffdd1457c0fb4e8365f524ee2529ac726edcf3;p=dpdk.git diff --git a/examples/multi_process/l2fwd_fork/main.c b/examples/multi_process/l2fwd_fork/main.c index a34f420bbc..59c102434a 100644 --- a/examples/multi_process/l2fwd_fork/main.c +++ b/examples/multi_process/l2fwd_fork/main.c @@ -1,13 +1,13 @@ /*- * BSD LICENSE - * - * Copyright(c) 2010-2014 Intel Corporation. All rights reserved. + * + * Copyright(c) 2010-2016 Intel Corporation. All rights reserved. * All rights reserved. - * + * * Redistribution and use in source and binary forms, with or without * modification, are permitted provided that the following conditions * are met: - * + * * * Redistributions of source code must retain the above copyright * notice, this list of conditions and the following disclaimer. * * Redistributions in binary form must reproduce the above copyright @@ -17,7 +17,7 @@ * * Neither the name of Intel Corporation nor the names of its * contributors may be used to endorse or promote products derived * from this software without specific prior written permission. - * + * * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR @@ -52,7 +52,6 @@ #include #include #include -#include #include #include #include @@ -74,12 +73,11 @@ #include #include -#include "main.h" #include "flib.h" #define RTE_LOGTYPE_L2FWD RTE_LOGTYPE_USER1 #define MBUF_NAME "mbuf_pool_%d" -#define MBUF_SIZE (2048 + sizeof(struct rte_mbuf) + RTE_PKTMBUF_HEADROOM) +#define MBUF_DATA_SIZE RTE_MBUF_DEFAULT_BUF_SIZE #define NB_MBUF 8192 #define RING_MASTER_NAME "l2fwd_ring_m2s_" #define RING_SLAVE_NAME "l2fwd_ring_s2m_" @@ -96,25 +94,6 @@ enum l2fwd_cmd{ CMD_STOP, }; -/* - * RX and TX Prefetch, Host, and Write-back threshold values should be - * carefully set for optimal performance. Consult the network - * controller's datasheet and supporting DPDK documentation for guidance - * on how these parameters should be set. - */ -#define RX_PTHRESH 8 /**< Default values of RX prefetch threshold reg. */ -#define RX_HTHRESH 8 /**< Default values of RX host threshold reg. */ -#define RX_WTHRESH 4 /**< Default values of RX write-back threshold reg. */ - -/* - * These default values are optimized for use with the Intel(R) 82599 10 GbE - * Controller and the DPDK ixgbe PMD. Consider using other values for other - * network controllers and/or network drivers. - */ -#define TX_PTHRESH 36 /**< Default values of TX prefetch threshold reg. */ -#define TX_HTHRESH 0 /**< Default values of TX host threshold reg. */ -#define TX_WTHRESH 0 /**< Default values of TX write-back threshold reg. */ - #define MAX_PKT_BURST 32 #define BURST_TX_DRAIN_US 100 /* TX drain every ~100us */ @@ -147,25 +126,25 @@ struct mbuf_table { struct lcore_queue_conf { unsigned n_rx_port; unsigned rx_port_list[MAX_RX_QUEUE_PER_LCORE]; - struct mbuf_table tx_mbufs[RTE_MAX_ETHPORTS]; - } __rte_cache_aligned; struct lcore_queue_conf lcore_queue_conf[RTE_MAX_LCORE]; +struct rte_eth_dev_tx_buffer *tx_buffer[RTE_MAX_ETHPORTS]; + struct lcore_resource_struct { int enabled; /* Only set in case this lcore involved into packet forwarding */ int flags; /* Set only slave need to restart or recreate */ unsigned lcore_id; /* lcore ID */ unsigned pair_id; /* dependency lcore ID on port */ - char ring_name[2][MAX_NAME_LEN]; + char ring_name[2][MAX_NAME_LEN]; /* ring[0] for master send cmd, slave read */ /* ring[1] for slave send ack, master read */ - struct rte_ring *ring[2]; + struct rte_ring *ring[2]; int port_num; /* Total port numbers */ - uint8_t port[RTE_MAX_ETHPORTS]; /* Port id for that lcore to receive packets */ + uint8_t port[RTE_MAX_ETHPORTS]; /* Port id for that lcore to receive packets */ }__attribute__((packed)) __rte_cache_aligned; -static struct lcore_resource_struct lcore_resource[RTE_MAX_LCORE]; +static struct lcore_resource_struct lcore_resource[RTE_MAX_LCORE]; static struct rte_mempool *message_pool; static rte_spinlock_t res_lock = RTE_SPINLOCK_INITIALIZER; /* use floating processes */ @@ -183,32 +162,13 @@ static const struct rte_eth_conf port_conf = { .hw_ip_checksum = 0, /**< IP checksum offload disabled */ .hw_vlan_filter = 0, /**< VLAN filtering disabled */ .jumbo_frame = 0, /**< Jumbo Frame Support disabled */ - .hw_strip_crc = 0, /**< CRC stripped by hardware */ + .hw_strip_crc = 1, /**< CRC stripped by hardware */ }, .txmode = { .mq_mode = ETH_MQ_TX_NONE, }, }; -static const struct rte_eth_rxconf rx_conf = { - .rx_thresh = { - .pthresh = RX_PTHRESH, - .hthresh = RX_HTHRESH, - .wthresh = RX_WTHRESH, - }, -}; - -static const struct rte_eth_txconf tx_conf = { - .tx_thresh = { - .pthresh = TX_PTHRESH, - .hthresh = TX_HTHRESH, - .wthresh = TX_WTHRESH, - }, - .tx_free_thresh = 0, /* Use PMD default values */ - .tx_rs_thresh = 0, /* Use PMD default values */ - .txq_flags = ETH_TXQ_FLAGS_NOMULTSEGS | ETH_TXQ_FLAGS_NOOFFLOADS, -}; - static struct rte_mempool * l2fwd_pktmbuf_pool[RTE_MAX_ETHPORTS]; /* Per-port statistics struct */ @@ -309,8 +269,8 @@ get_cpu_affinity(void) } /** - * This fnciton demonstrates the approach to create ring in first instance - * or re-attach an existed ring in later instance. + * This fnciton demonstrates the approach to create ring in first instance + * or re-attach an existed ring in later instance. **/ static struct rte_ring * create_ring(const char *name, unsigned count, @@ -320,7 +280,7 @@ create_ring(const char *name, unsigned count, if (name == NULL) return NULL; - + /* If already create, just attached it */ if (likely((ring = rte_ring_lookup(name)) != NULL)) return ring; @@ -330,7 +290,7 @@ create_ring(const char *name, unsigned count, } /* Malloc with rte_malloc on structures that shared by master and slave */ -static int +static int l2fwd_malloc_shared_struct(void) { port_statistics = rte_zmalloc("port_stat", @@ -355,36 +315,36 @@ l2fwd_malloc_shared_struct(void) } /* Create ring which used for communicate among master and slave */ -static int +static int create_ms_ring(unsigned slaveid) { unsigned flag = RING_F_SP_ENQ | RING_F_SC_DEQ; struct lcore_resource_struct *res = &lcore_resource[slaveid]; unsigned socketid = rte_socket_id(); - + /* Always assume create ring on master socket_id */ /* Default only create a ring size 32 */ snprintf(res->ring_name[0], MAX_NAME_LEN, "%s%u", RING_MASTER_NAME, slaveid); - if ((res->ring[0] = create_ring(res->ring_name[0], NB_CORE_MSGBUF, + if ((res->ring[0] = create_ring(res->ring_name[0], NB_CORE_MSGBUF, socketid, flag)) == NULL) { printf("Create m2s ring %s failed\n", res->ring_name[0]); return -1; } - + snprintf(res->ring_name[1], MAX_NAME_LEN, "%s%u", RING_SLAVE_NAME, slaveid); - if ((res->ring[1] = create_ring(res->ring_name[1], NB_CORE_MSGBUF, + if ((res->ring[1] = create_ring(res->ring_name[1], NB_CORE_MSGBUF, socketid, flag)) == NULL) { printf("Create s2m ring %s failed\n", res->ring_name[1]); return -1; - } + } return 0; } /* send command to pair in paired master and slave ring */ -static inline int +static inline int sendcmd(unsigned slaveid, enum l2fwd_cmd cmd, int is_master) { struct lcore_resource_struct *res = &lcore_resource[slaveid]; @@ -394,10 +354,10 @@ sendcmd(unsigned slaveid, enum l2fwd_cmd cmd, int is_master) /* Only check master, it must be enabled and running if it is slave */ if (is_master && !res->enabled) return -1; - + if (res->ring[fd] == NULL) - return -1; - + return -1; + if (rte_mempool_get(message_pool, &msg) < 0) { printf("Error to get message buffer\n"); return -1; @@ -410,7 +370,7 @@ sendcmd(unsigned slaveid, enum l2fwd_cmd cmd, int is_master) rte_mempool_put(message_pool, msg); return -1; } - + return 0; } @@ -439,7 +399,7 @@ getcmd(unsigned slaveid, enum l2fwd_cmd *cmd, int is_master) } /* Master send command to slave and wait until ack received or error met */ -static int +static int master_sendcmd_with_ack(unsigned slaveid, enum l2fwd_cmd cmd) { enum l2fwd_cmd ack_cmd; @@ -465,7 +425,7 @@ master_sendcmd_with_ack(unsigned slaveid, enum l2fwd_cmd cmd) } /* restart all port that assigned to that slave lcore */ -static int +static int reset_slave_all_ports(unsigned slaveid) { struct lcore_resource_struct *slave = &lcore_resource[slaveid]; @@ -479,12 +439,13 @@ reset_slave_all_ports(unsigned slaveid) rte_eth_dev_stop(slave->port[i]); snprintf(buf_name, RTE_MEMPOOL_NAMESIZE, MBUF_NAME, slave->port[i]); pool = rte_mempool_lookup(buf_name); - if (pool) - printf("Port %d mempool free object is %u(%u)\n", slave->port[i], - rte_mempool_count(pool), (unsigned)NB_MBUF); + if (pool) + printf("Port %d mempool free object is %u(%u)\n", slave->port[i], + rte_mempool_avail_count(pool), + (unsigned int)NB_MBUF); else printf("Can't find mempool %s\n", buf_name); - + printf("Start port :%d\n", slave->port[i]); ret = rte_eth_dev_start(slave->port[i]); if (ret != 0) @@ -493,7 +454,7 @@ reset_slave_all_ports(unsigned slaveid) return ret; } -static int +static int reset_shared_structures(unsigned slaveid) { int ret; @@ -503,18 +464,18 @@ reset_shared_structures(unsigned slaveid) return ret; } -/** - * Call this function to re-create resource that needed for slave process that +/** + * Call this function to re-create resource that needed for slave process that * exited in last instance **/ -static int +static int init_slave_res(unsigned slaveid) { struct lcore_resource_struct *slave = &lcore_resource[slaveid]; enum l2fwd_cmd cmd; - + if (!slave->enabled) { - printf("Something wrong with lcore=%u enabled=%d\n",slaveid, + printf("Something wrong with lcore=%u enabled=%d\n",slaveid, slave->enabled); return -1; } @@ -531,7 +492,7 @@ init_slave_res(unsigned slaveid) return 0; } -static int +static int recreate_one_slave(unsigned slaveid) { int ret = 0; @@ -539,12 +500,12 @@ recreate_one_slave(unsigned slaveid) if ((ret = init_slave_res(slaveid)) != 0) { printf("Init slave=%u failed\n", slaveid); return ret; - } - + } + if ((ret = flib_remote_launch(l2fwd_launch_one_lcore, NULL, slaveid)) != 0) printf("Launch slave %u failed\n", slaveid); - + return ret; } @@ -604,7 +565,7 @@ back: return ret; } -static void +static void slave_exit_cb(unsigned slaveid, __attribute__((unused))int stat) { struct lcore_resource_struct *slave = &lcore_resource[slaveid]; @@ -622,58 +583,14 @@ slave_exit_cb(unsigned slaveid, __attribute__((unused))int stat) rte_spinlock_unlock(&res_lock); } -/* Send the packet on an output interface */ -static int -l2fwd_send_burst(struct lcore_queue_conf *qconf, unsigned n, uint8_t port) -{ - struct rte_mbuf **m_table; - unsigned ret; - unsigned queueid =0; - - m_table = (struct rte_mbuf **)qconf->tx_mbufs[port].m_table; - - ret = rte_eth_tx_burst(port, (uint16_t) queueid, m_table, (uint16_t) n); - port_statistics[port].tx += ret; - if (unlikely(ret < n)) { - port_statistics[port].dropped += (n - ret); - do { - rte_pktmbuf_free(m_table[ret]); - } while (++ret < n); - } - - return 0; -} - -/* Send the packet on an output interface */ -static int -l2fwd_send_packet(struct rte_mbuf *m, uint8_t port) -{ - unsigned lcore_id, len; - struct lcore_queue_conf *qconf; - - lcore_id = rte_lcore_id(); - - qconf = &lcore_queue_conf[lcore_id]; - len = qconf->tx_mbufs[port].len; - qconf->tx_mbufs[port].m_table[len] = m; - len++; - - /* enough pkts to be sent */ - if (unlikely(len == MAX_PKT_BURST)) { - l2fwd_send_burst(qconf, MAX_PKT_BURST, port); - len = 0; - } - - qconf->tx_mbufs[port].len = len; - return 0; -} - static void l2fwd_simple_forward(struct rte_mbuf *m, unsigned portid) { struct ether_hdr *eth; void *tmp; unsigned dst_port; + int sent; + struct rte_eth_dev_tx_buffer *buffer; dst_port = l2fwd_dst_ports[portid]; eth = rte_pktmbuf_mtod(m, struct ether_hdr *); @@ -685,7 +602,10 @@ l2fwd_simple_forward(struct rte_mbuf *m, unsigned portid) /* src addr */ ether_addr_copy(&l2fwd_ports_eth_addr[dst_port], ð->s_addr); - l2fwd_send_packet(m, (uint8_t) dst_port); + buffer = tx_buffer[dst_port]; + sent = rte_eth_tx_buffer(dst_port, 0, buffer, m); + if (sent) + port_statistics[dst_port].tx += sent; } /* main processing loop */ @@ -694,12 +614,15 @@ l2fwd_main_loop(void) { struct rte_mbuf *pkts_burst[MAX_PKT_BURST]; struct rte_mbuf *m; + int sent; unsigned lcore_id; uint64_t prev_tsc, diff_tsc, cur_tsc; unsigned i, j, portid, nb_rx; struct lcore_queue_conf *qconf; - const uint64_t drain_tsc = (rte_get_tsc_hz() + US_PER_S - 1) / US_PER_S * BURST_TX_DRAIN_US; - + const uint64_t drain_tsc = (rte_get_tsc_hz() + US_PER_S - 1) / US_PER_S * + BURST_TX_DRAIN_US; + struct rte_eth_dev_tx_buffer *buffer; + prev_tsc = 0; lcore_id = rte_lcore_id(); @@ -729,23 +652,27 @@ l2fwd_main_loop(void) /* If get stop command, stop forwarding and exit */ if (cmd == CMD_STOP) { return; - } + } } - + /* * TX burst queue drain */ diff_tsc = cur_tsc - prev_tsc; if (unlikely(diff_tsc > drain_tsc)) { - for (portid = 0; portid < RTE_MAX_ETHPORTS; portid++) { - if (qconf->tx_mbufs[portid].len == 0) - continue; - l2fwd_send_burst(&lcore_queue_conf[lcore_id], - qconf->tx_mbufs[portid].len, - (uint8_t) portid); - qconf->tx_mbufs[portid].len = 0; + for (i = 0; i < qconf->n_rx_port; i++) { + + portid = l2fwd_dst_ports[qconf->rx_port_list[i]]; + buffer = tx_buffer[portid]; + + sent = rte_eth_tx_buffer_flush(portid, 0, buffer); + if (sent) + port_statistics[portid].tx += sent; + } + + prev_tsc = cur_tsc; } /* @@ -877,7 +804,7 @@ l2fwd_parse_args(int argc, char **argv) {NULL, 0, 0, 0} }; int has_pmask = 0; - + argvopt = argv; while ((opt = getopt_long(argc, argvopt, "p:q:T:f", @@ -919,7 +846,7 @@ l2fwd_parse_args(int argc, char **argv) case 'f': float_proc = 1; break; - + /* long options */ case 0: l2fwd_usage(prgname); @@ -939,7 +866,7 @@ l2fwd_parse_args(int argc, char **argv) return -1; } ret = optind-1; - optind = 0; /* reset getopt lib */ + optind = 1; /* reset getopt lib */ return ret; } @@ -975,7 +902,7 @@ check_all_ports_link_status(uint8_t port_num, uint32_t port_mask) continue; } /* clear all_ports_up flag if any link down */ - if (link.link_status == 0) { + if (link.link_status == ETH_LINK_DOWN) { all_ports_up = 0; break; } @@ -999,7 +926,7 @@ check_all_ports_link_status(uint8_t port_num, uint32_t port_mask) } int -MAIN(int argc, char **argv) +main(int argc, char **argv) { struct lcore_queue_conf *qconf; struct rte_eth_dev_info dev_info; @@ -1010,7 +937,6 @@ MAIN(int argc, char **argv) unsigned rx_lcore_id; unsigned nb_ports_in_mask = 0; unsigned i; - int flags = 0; uint64_t prev_tsc, diff_tsc, cur_tsc, timer_tsc; /* Save cpu_affinity first, restore it in case it's floating process option */ @@ -1038,7 +964,7 @@ MAIN(int argc, char **argv) rte_exit(EXIT_FAILURE, "flib init error"); /** - * Allocated structures that slave lcore would change. For those that slaves are + * Allocated structures that slave lcore would change. For those that slaves are * read only, needn't use malloc to share and global or static variables is ok since * slave inherit all the knowledge that master initialized. **/ @@ -1049,34 +975,23 @@ MAIN(int argc, char **argv) memset(lcore_resource, 0, sizeof(lcore_resource)); for (i = 0; i < RTE_MAX_LCORE; i++) lcore_resource[i].lcore_id = i; - - if (rte_eal_pci_probe() < 0) - rte_exit(EXIT_FAILURE, "Cannot probe PCI\n"); nb_ports = rte_eth_dev_count(); if (nb_ports == 0) rte_exit(EXIT_FAILURE, "No Ethernet ports - bye\n"); - if (nb_ports > RTE_MAX_ETHPORTS) - nb_ports = RTE_MAX_ETHPORTS; - /* create the mbuf pool */ for (portid = 0; portid < nb_ports; portid++) { /* skip ports that are not enabled */ if ((l2fwd_enabled_port_mask & (1 << portid)) == 0) - continue; + continue; char buf_name[RTE_MEMPOOL_NAMESIZE]; - flags = MEMPOOL_F_SP_PUT | MEMPOOL_F_SC_GET; snprintf(buf_name, RTE_MEMPOOL_NAMESIZE, MBUF_NAME, portid); l2fwd_pktmbuf_pool[portid] = - rte_mempool_create(buf_name, NB_MBUF, - MBUF_SIZE, 32, - sizeof(struct rte_pktmbuf_pool_private), - rte_pktmbuf_pool_init, NULL, - rte_pktmbuf_init, NULL, - rte_socket_id(), flags); + rte_pktmbuf_pool_create(buf_name, NB_MBUF, 32, + 0, MBUF_DATA_SIZE, rte_socket_id()); if (l2fwd_pktmbuf_pool[portid] == NULL) - rte_exit(EXIT_FAILURE, "Cannot init mbuf pool\n"); + rte_exit(EXIT_FAILURE, "Cannot create mbuf pool\n"); printf("Create mbuf %s\n", buf_name); } @@ -1123,7 +1038,7 @@ MAIN(int argc, char **argv) /* get the lcore_id for this port */ /* skip master lcore */ while (rte_lcore_is_enabled(rx_lcore_id) == 0 || - rte_get_master_lcore() == rx_lcore_id || + rte_get_master_lcore() == rx_lcore_id || lcore_queue_conf[rx_lcore_id].n_rx_port == l2fwd_rx_queue_per_lcore) { @@ -1143,7 +1058,7 @@ MAIN(int argc, char **argv) res = &lcore_resource[rx_lcore_id]; res->enabled = 1; res->port[res->port_num++] = portid; - + printf("Lcore %u: RX port %u\n", rx_lcore_id, (unsigned) portid); } @@ -1170,7 +1085,8 @@ MAIN(int argc, char **argv) /* init one RX queue */ fflush(stdout); ret = rte_eth_rx_queue_setup(portid, 0, nb_rxd, - rte_eth_dev_socket_id(portid), &rx_conf, + rte_eth_dev_socket_id(portid), + NULL, l2fwd_pktmbuf_pool[portid]); if (ret < 0) rte_exit(EXIT_FAILURE, "rte_eth_rx_queue_setup:err=%d, port=%u\n", @@ -1179,11 +1095,29 @@ MAIN(int argc, char **argv) /* init one TX queue on each port */ fflush(stdout); ret = rte_eth_tx_queue_setup(portid, 0, nb_txd, - rte_eth_dev_socket_id(portid), &tx_conf); + rte_eth_dev_socket_id(portid), + NULL); if (ret < 0) rte_exit(EXIT_FAILURE, "rte_eth_tx_queue_setup:err=%d, port=%u\n", ret, (unsigned) portid); + /* Initialize TX buffers */ + tx_buffer[portid] = rte_zmalloc_socket("tx_buffer", + RTE_ETH_TX_BUFFER_SIZE(MAX_PKT_BURST), 0, + rte_eth_dev_socket_id(portid)); + if (tx_buffer[portid] == NULL) + rte_exit(EXIT_FAILURE, "Cannot allocate buffer for tx on port %u\n", + (unsigned) portid); + + rte_eth_tx_buffer_init(tx_buffer[portid], MAX_PKT_BURST); + + ret = rte_eth_tx_buffer_set_err_callback(tx_buffer[portid], + rte_eth_tx_buffer_count_callback, + &port_statistics[portid].dropped); + if (ret < 0) + rte_exit(EXIT_FAILURE, "Cannot set error callback for " + "tx buffer on port %u\n", (unsigned) portid); + /* Start device */ ret = rte_eth_dev_start(portid); if (ret < 0) @@ -1216,12 +1150,12 @@ MAIN(int argc, char **argv) /* Record pair lcore */ /** - * Since l2fwd example would create pair between different neighbour port, that's + * Since l2fwd example would create pair between different neighbour port, that's * port 0 receive and forward to port 1, the same to port 1, these 2 ports will have * dependency. If one port stopped working (killed, for example), the port need to * be stopped/started again. During the time, another port need to wait until stop/start * procedure completed. So, record the pair relationship for those lcores working - * on ports. + * on ports. **/ for (portid = 0; portid < nb_ports; portid++) { uint32_t pair_port; @@ -1239,12 +1173,12 @@ MAIN(int argc, char **argv) continue; for (j = 0; j < lcore_queue_conf[i].n_rx_port;j++) { if (lcore_queue_conf[i].rx_port_list[j] == portid) { - lcore = i; + lcore = i; find_lcore = 1; break; } if (lcore_queue_conf[i].rx_port_list[j] == pair_port) { - pair_lcore = i; + pair_lcore = i; find_pair_lcore = 1; break; } @@ -1261,21 +1195,21 @@ MAIN(int argc, char **argv) } /* Create message buffer for all master and slave */ - message_pool = rte_mempool_create("ms_msg_pool", + message_pool = rte_mempool_create("ms_msg_pool", NB_CORE_MSGBUF * RTE_MAX_LCORE, sizeof(enum l2fwd_cmd), NB_CORE_MSGBUF / 2, 0, rte_pktmbuf_pool_init, NULL, rte_pktmbuf_init, NULL, - rte_socket_id(), 0); + rte_socket_id(), 0); if (message_pool == NULL) rte_exit(EXIT_FAILURE, "Create msg mempool failed\n"); /* Create ring for each master and slave pair, also register cb when slave leaves */ for (i = 0; i < RTE_MAX_LCORE; i++) { - /** - * Only create ring and register slave_exit cb in case that core involved into + /** + * Only create ring and register slave_exit cb in case that core involved into * packet forwarding **/ if (lcore_resource[i].enabled) { @@ -1287,7 +1221,7 @@ MAIN(int argc, char **argv) if (flib_register_slave_exit_notify(i, slave_exit_cb) != 0) - rte_exit(EXIT_FAILURE, + rte_exit(EXIT_FAILURE, "Register master_trace_slave_exit failed"); } } @@ -1304,19 +1238,19 @@ MAIN(int argc, char **argv) diff_tsc = cur_tsc - prev_tsc; /* if timer is enabled */ if (timer_period > 0) { - + /* advance the timer */ timer_tsc += diff_tsc; - + /* if timer has reached its timeout */ if (unlikely(timer_tsc >= (uint64_t) timer_period)) { - + print_stats(); /* reset the timer */ timer_tsc = 0; } } - + prev_tsc = cur_tsc; /* Check any slave need restart or recreate */