Re: [dpdk-dev] [PATCH v3 1/7] net/mlx4: remove error flows from Tx fast path

2017-10-31 Thread Adrien Mazarguil
Hi Matan, On Mon, Oct 30, 2017 at 06:11:31PM +, Matan Azrad wrote: > Hi Adrien > > > -Original Message- > > From: Adrien Mazarguil [mailto:adrien.mazarg...@6wind.com] > > Sent: Monday, October 30, 2017 4:23 PM > > To: Matan Azrad > > Cc: dev@dpdk.org; Ophir Munk > > Subject: Re: [PA

Re: [dpdk-dev] [PATCH v3 1/7] net/mlx4: remove error flows from Tx fast path

2017-10-30 Thread Matan Azrad
Hi Adrien > -Original Message- > From: Adrien Mazarguil [mailto:adrien.mazarg...@6wind.com] > Sent: Monday, October 30, 2017 4:23 PM > To: Matan Azrad > Cc: dev@dpdk.org; Ophir Munk > Subject: Re: [PATCH v3 1/7] net/mlx4: remove error flows from Tx fast path > > On Mon, Oct 30, 2017 at

Re: [dpdk-dev] [PATCH v3 1/7] net/mlx4: remove error flows from Tx fast path

2017-10-30 Thread Adrien Mazarguil
On Mon, Oct 30, 2017 at 10:07:23AM +, Matan Azrad wrote: > Move unnecessary error flows to DEBUG mode. > > Signed-off-by: Matan Azrad > Acked-by: Adrien Mazarguil I missed a couple of details while reviewing the original version, the first one being mlx4_post_send()'s return value is still

[dpdk-dev] [PATCH v3 1/7] net/mlx4: remove error flows from Tx fast path

2017-10-30 Thread Matan Azrad
Move unnecessary error flows to DEBUG mode. Signed-off-by: Matan Azrad Acked-by: Adrien Mazarguil --- drivers/net/mlx4/mlx4_rxtx.c | 16 ++-- 1 file changed, 6 insertions(+), 10 deletions(-) diff --git a/drivers/net/mlx4/mlx4_rxtx.c b/drivers/net/mlx4/mlx4_rxtx.c index 67dc712..4f8