Re: [dpdk-dev] [PATCH] net/failsafe: fix parameters parsing

2017-08-19 Thread Matan Azrad
Hi Gaetan > -Original Message- > From: Gaëtan Rivet [mailto:gaetan.ri...@6wind.com] > Sent: Friday, August 18, 2017 12:03 PM > To: Matan Azrad > Cc: dev@dpdk.org; Ori Kam ; sta...@dpdk.org > Subject: Re: [PATCH] net/failsafe: fix parameters parsing > > On Thu, Aug 17, 2017 at 06:52:48PM

Re: [dpdk-dev] [PATCH] net/failsafe: fix parameters parsing

2017-08-18 Thread Gaëtan Rivet
On Thu, Aug 17, 2017 at 06:52:48PM +, Matan Azrad wrote: > Hi Gaetan > > > -Original Message- > > From: Gaëtan Rivet [mailto:gaetan.ri...@6wind.com] > > Sent: Thursday, August 17, 2017 7:25 PM > > To: Matan Azrad > > Cc: dev@dpdk.org; Ori Kam ; sta...@dpdk.org > > Subject: Re: [PATCH]

Re: [dpdk-dev] [PATCH] net/failsafe: fix parameters parsing

2017-08-17 Thread Matan Azrad
Hi Gaetan > -Original Message- > From: Gaëtan Rivet [mailto:gaetan.ri...@6wind.com] > Sent: Thursday, August 17, 2017 7:25 PM > To: Matan Azrad > Cc: dev@dpdk.org; Ori Kam ; sta...@dpdk.org > Subject: Re: [PATCH] net/failsafe: fix parameters parsing > > On Thu, Aug 17, 2017 at 03:54:23PM

Re: [dpdk-dev] [PATCH] net/failsafe: fix parameters parsing

2017-08-17 Thread Gaëtan Rivet
On Thu, Aug 17, 2017 at 03:54:23PM +, Matan Azrad wrote: > > > > -Original Message- > > From: Gaëtan Rivet [mailto:gaetan.ri...@6wind.com] > > Sent: Thursday, August 17, 2017 6:26 PM > > To: Matan Azrad > > Cc: dev@dpdk.org; Ori Kam ; sta...@dpdk.org > > Subject: Re: [PATCH] net/fail

Re: [dpdk-dev] [PATCH] net/failsafe: fix parameters parsing

2017-08-17 Thread Matan Azrad
> -Original Message- > From: Gaëtan Rivet [mailto:gaetan.ri...@6wind.com] > Sent: Thursday, August 17, 2017 6:26 PM > To: Matan Azrad > Cc: dev@dpdk.org; Ori Kam ; sta...@dpdk.org > Subject: Re: [PATCH] net/failsafe: fix parameters parsing > > Hi Matan, > > Thanks for spotting the bug.

Re: [dpdk-dev] [PATCH] net/failsafe: fix parameters parsing

2017-08-17 Thread Gaëtan Rivet
Hi Matan, Thanks for spotting the bug. On Thu, Aug 17, 2017 at 05:19:41PM +0300, Matan Azrad wrote: > The corrupted code used wrongly snprintf return value as the > number of characters actually copied, in spite of the meanning > is the number of characters which would be generated for the > give

[dpdk-dev] [PATCH] net/failsafe: fix parameters parsing

2017-08-17 Thread Matan Azrad
The corrupted code used wrongly snprintf return value as the number of characters actually copied, in spite of the meanning is the number of characters which would be generated for the given input. It caused to remain zerod bytes between the failsafe command line non sub device parameters indicate