Re: [dpdk-dev] [PATCH] app/testpmd: fix forward port ids setting

2017-09-07 Thread Wu, Jingjing
> -Original Message- > From: Matan Azrad [mailto:ma...@mellanox.com] > Sent: Sunday, September 3, 2017 9:19 PM > To: Wu, Jingjing ; Gaetan Rivet > ; > Thomas Monjalon > Cc: dev@dpdk.org; Ori Kam ; sta...@dpdk.org > Subject: [PATCH] app/testpmd: fix forward port ids setting > > The corr

Re: [dpdk-dev] [PATCH] app/testpmd: fix forward port ids setting

2017-09-04 Thread Gaëtan Rivet
On Mon, Sep 04, 2017 at 09:25:04AM +, Matan Azrad wrote: > Hi > > > -Original Message- > > From: Gaëtan Rivet [mailto:gaetan.ri...@6wind.com] > > Sent: Monday, September 4, 2017 11:46 AM > > To: Matan Azrad > > Cc: Jingjing Wu ; Thomas Monjalon > > ; dev@dpdk.org; Ori Kam ; > > sta...

Re: [dpdk-dev] [PATCH] app/testpmd: fix forward port ids setting

2017-09-04 Thread Matan Azrad
Hi > -Original Message- > From: Gaëtan Rivet [mailto:gaetan.ri...@6wind.com] > Sent: Monday, September 4, 2017 11:46 AM > To: Matan Azrad > Cc: Jingjing Wu ; Thomas Monjalon > ; dev@dpdk.org; Ori Kam ; > sta...@dpdk.org > Subject: Re: [PATCH] app/testpmd: fix forward port ids setting > >

Re: [dpdk-dev] [PATCH] app/testpmd: fix forward port ids setting

2017-09-04 Thread Gaëtan Rivet
Hi Matan, On Sun, Sep 03, 2017 at 04:19:07PM +0300, Matan Azrad wrote: > The corrupted code didn't check the port availability when > it was trying to set the forward port IDs array. > However, when it was counting the number of ports, the availability > was checked by RTE_ETH_FOREACH_DEV iterator

[dpdk-dev] [PATCH] app/testpmd: fix forward port ids setting

2017-09-03 Thread Matan Azrad
The corrupted code didn't check the port availability when it was trying to set the forward port IDs array. However, when it was counting the number of ports, the availability was checked by RTE_ETH_FOREACH_DEV iterator. Hence, even when ETH devices ports were not in ATTACHED state, the testpmd tr