Hi Wei, Your patch looks good, but there is an ack patch that deals with offloads. http://dpdk.org/ml/archives/dev/2017-December/083639.html
Please update your patch to support the new tx offloads. Regards, Ori > -----Original Message----- > From: Zhao1, Wei [mailto:[email protected]] > Sent: Tuesday, December 26, 2017 12:03 PM > To: Ori Kam <[email protected]> > Cc: Lu, Wenzhuo <[email protected]>; [email protected] > Subject: RE: [PATCH] examples/flow_filtering: add Tx queues setup process > > Add maintainer [email protected] into cc list. > > > > -----Original Message----- > > From: Zhao1, Wei > > Sent: Tuesday, December 26, 2017 5:21 PM > > To: [email protected] > > Cc: Lu, Wenzhuo <[email protected]>; Zhao1, Wei > > <[email protected]> > > Subject: [PATCH] examples/flow_filtering: add Tx queues setup process > > > > This example do not has the process of set up tx queues, but some NIC > > start up process will be blocked if this is no tx queue and only rx > > queues. So add tx queues setup process in main code. > > > > Signed-off-by: Wei Zhao <[email protected]> > > --- > > examples/flow_filtering/main.c | 11 +++++++++++ > > 1 file changed, 11 insertions(+) > > > > diff --git a/examples/flow_filtering/main.c > > b/examples/flow_filtering/main.c index 7d739b4..b5c7677 100644 > > --- a/examples/flow_filtering/main.c > > +++ b/examples/flow_filtering/main.c > > @@ -173,6 +173,17 @@ init_port(void) > > } > > } > > > > + for (i = 0; i < nr_queues; i++) { > > + ret = rte_eth_tx_queue_setup(port_id, i, 512, > > + rte_eth_dev_socket_id(port_id), > > + NULL); > > + if (ret < 0) { > > + rte_exit(EXIT_FAILURE, > > + ":: Tx queue setup failed: err=%d, > > port=%u\n", > > + ret, port_id); > > + } > > + } > > + > > rte_eth_promiscuous_enable(port_id); > > ret = rte_eth_dev_start(port_id); > > if (ret < 0) { > > -- > > 2.9.3
