[dpdk-dev] [PATCH 2/2] bnx2x: Determine rx/tx queue sizes sooner

2015-12-30 Thread Chas Williams
From: "Charles (Chas) Williams" The VF needs to determine the queues sizes before .dev_infos_get so that it can hint to the upper layer the proper sizes. Move bnx2x_vf_get_resources() to .eth_dev_init and probe with the guesses from bnx2x_init_rte(). Signed-off-by: Chas

[dpdk-dev] [PATCH 1/2] bnx2x: fix error handling in bnx2x_loop_obtain_resources()

2015-12-30 Thread Chas Williams
From: "Charles (Chas) Williams" bnx2x_loop_obtain_resources() returns a struct containing the status and the error message. If bnx2x_do_req4pf() fails, it shouldn't return both of these fields set to 0 indicating failure and no error. Further, bnx2x_do_req4pf() needs to

[dpdk-dev] [PATCH v4 3/3] examples/l3fwd: Handle SIGINT and SIGTERM in l3fwd

2015-12-30 Thread Stephen Hemminger
On Thu, 31 Dec 2015 01:44:20 + "Wang, Zhihong" wrote: > > > +#define PORT_IDLE 0 > > > +#define PORT_INIT 1 > > > +#define PORT_WORK 2 > > > +#define PORT_STOP 3 > > > +#define PORT_QUIT 4 > > > > Seems ok, but over-complicated. > > I think all you need is just IDLE, INIT, QUIT. > >

[dpdk-dev] [PATCH v5 3/3] examples/l3fwd: Handle SIGINT and SIGTERM in l3fwd

2015-12-30 Thread Zhihong Wang
Handle SIGINT and SIGTERM in l3fwd. Signed-off-by: Zhihong Wang Acked-by: Michael Qiu --- examples/l3fwd/main.c | 46 ++ 1 file changed, 42 insertions(+), 4 deletions(-) diff --git a/examples/l3fwd/main.c b/examples/l3fwd/main.c index

[dpdk-dev] [PATCH v5 2/3] examples/l2fwd: Handle SIGINT and SIGTERM in l2fwd

2015-12-30 Thread Zhihong Wang
Handle SIGINT and SIGTERM in l2fwd. Signed-off-by: Zhihong Wang Acked-by: Michael Qiu --- examples/l2fwd/main.c | 43 +++ 1 file changed, 39 insertions(+), 4 deletions(-) diff --git a/examples/l2fwd/main.c b/examples/l2fwd/main.c index 720fd5a..f35d8a1

[dpdk-dev] [PATCH v5 1/3] app/test-pmd: Handle SIGINT and SIGTERM in testpmd

2015-12-30 Thread Zhihong Wang
Handle SIGINT and SIGTERM in testpmd. Signed-off-by: Zhihong Wang Acked-by: Michael Qiu --- app/test-pmd/cmdline.c | 20 +--- app/test-pmd/testpmd.c | 39 +-- app/test-pmd/testpmd.h | 1 + 3 files changed, 47 insertions(+), 13 deletions(-)

[dpdk-dev] [PATCH v5 0/3] Handle SIGINT and SIGTERM in DPDK examples

2015-12-30 Thread Zhihong Wang
This patch handles SIGINT and SIGTERM in testpmd, l2fwd, and l3fwd, make sure all ports are properly stopped and closed. For virtual ports, the stop and close function may deal with resource cleanup, such as socket files unlinking. -- Changes in v5: 1. Get rid of over complicated

[dpdk-dev] [PATCH] fm10k: allocate logical ports for flow director

2015-12-30 Thread Chen Jing D(Mark)
From: "Chen Jing D(Mark)" In fm10k, PF, VF, VMDQ or queues binding to flow director rule can be considered as a logical port. Original implementation only create single one for all cases. This change creates totally 128 logical ones, first 64 for PF and VMDQ, second 64 for

[dpdk-dev] [PATCH v4 3/3] examples/l3fwd: Handle SIGINT and SIGTERM in l3fwd

2015-12-30 Thread Ananyev, Konstantin
> -Original Message- > From: Wang, Zhihong > Sent: Tuesday, December 29, 2015 11:27 PM > To: dev at dpdk.org > Cc: Ananyev, Konstantin; stephen at networkplumber.org; Qiu, Michael; Wang, > Zhihong > Subject: [PATCH v4 3/3] examples/l3fwd: Handle SIGINT and SIGTERM in l3fwd > > Handle

[dpdk-dev] [RFC 0/5] virtio support for container

2015-12-30 Thread Pavel Fedin
Hello everybody! I am currently working on improved version of this patchset, and i am testing it with openvswitch. I run two openvswitch instances: on host and in container. Both ovs instances forward packets between its LOCAL port and vhost/virtio port. This way i can comfortably run PING

[dpdk-dev] [PATCH 2/6] virtio: introduce struct virtio_pci_ops

2015-12-30 Thread Yuanhan Liu
On Tue, Dec 29, 2015 at 11:31:35AM +, Tan, Jianfeng wrote: > > > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Yuanhan Liu > > Sent: Thursday, December 10, 2015 11:54 AM > > To: dev at dpdk.org > > Subject: [dpdk-dev] [PATCH 2/6] virtio: introduce

[dpdk-dev] [PATCH 6/6] virtio: add virtio v1.0 support

2015-12-30 Thread Yuanhan Liu
On Tue, Dec 29, 2015 at 11:39:47AM +, Tan, Jianfeng wrote: > > > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Yuanhan Liu > > Sent: Thursday, December 10, 2015 11:54 AM > > To: dev at dpdk.org > > Subject: [dpdk-dev] [PATCH 6/6] virtio: add virtio

[dpdk-dev] [PATCH v3 3/3] examples/l3fwd: Handle SIGINT and SIGTERM in l3fwd

2015-12-30 Thread Ananyev, Konstantin
> -Original Message- > From: Wang, Zhihong > Sent: Wednesday, December 30, 2015 3:15 AM > To: Ananyev, Konstantin; dev at dpdk.org > Cc: stephen at networkplumber.org; Qiu, Michael > Subject: RE: [PATCH v3 3/3] examples/l3fwd: Handle SIGINT and SIGTERM in l3fwd > > > > +static uint8_t >

[dpdk-dev] Flow Director - big endian handling

2015-12-30 Thread Yaacov Hazan
Hi JingJing, I looked at your patch for flow director - app/testpmd: update flow director commands - a56335925919d26c81dec8accf31c39d2f790c5a. It seems there is some mismatch in the handling of big endian between the filter and mask. In the cmd_flow_director_filter_parsed function, which add

[dpdk-dev] [PATCH v3 3/3] examples/l3fwd: Handle SIGINT and SIGTERM in l3fwd

2015-12-30 Thread Stephen Hemminger
On Mon, 28 Dec 2015 20:20:32 -0500 Zhihong Wang wrote: > Handle SIGINT and SIGTERM in l3fwd. > > Signed-off-by: Zhihong Wang > Acked-by: Michael Qiu > --- > examples/l3fwd/main.c | 129 > +- > 1 file changed, 107 insertions(+), 22 deletions(-)

[dpdk-dev] [PATCH v3 2/3] examples/l2fwd: Handle SIGINT and SIGTERM in l2fwd

2015-12-30 Thread Stephen Hemminger
On Mon, 28 Dec 2015 20:20:31 -0500 Zhihong Wang wrote: > + /* exit with the expected status */ > + if (force_quit) { > + signal(signo_quit, SIG_DFL); > + kill(getpid(), signo_quit); This is really not necessary. Most other programs don't do this.

[dpdk-dev] [PATCH v3 3/3] examples/l3fwd: Handle SIGINT and SIGTERM in l3fwd

2015-12-30 Thread Wang, Zhihong
> > +static uint8_t > > +start_ports(void) > > +{ > > + unsigned portid, nb_ports, avail_ports; > > + int ret; > > + > > + nb_ports = rte_eth_dev_count(); > > + avail_ports = 0; > > + for (portid = 0; portid < nb_ports; portid++) { > > + if ((enabled_port_mask & (1 << portid))