Re: [dpdk-dev] [PATCH v3] app/testpmd: fix copying the name of the dynflag

2020-02-04 Thread Ori Kam
> -Original Message- > From: dev On Behalf Of Iremonger, Bernard > Sent: Tuesday, February 4, 2020 2:18 PM > To: Ori Kam ; Yigit, Ferruh ; > Lu, Wenzhuo ; Wu, Jingjing > Cc: dev@dpdk.org; Slava Ovsiienko > Subject: Re: [dpdk-dev] [PATCH v3] app/testpmd: fix co

Re: [dpdk-dev] [PATCH v3] app/testpmd: fix copying the name of the dynflag

2020-02-04 Thread Iremonger, Bernard
Hi Ori, > > > > Subject: Re: [PATCH v3] app/testpmd: fix copying the name of the > > > > dynflag > > > > > > > > On 1/30/2020 9:04 PM, Ori Kam wrote: > > > > > When working with testpmd and setting the dynflag name, we copy > > > > > the name given by the cmd to the dynflag name. > > > > > > >

Re: [dpdk-dev] [PATCH v3] app/testpmd: fix copying the name of the dynflag

2020-02-04 Thread Ori Kam
Hi Bernard, > -Original Message- > From: Iremonger, Bernard > Sent: Tuesday, February 4, 2020 12:46 PM > To: Ori Kam ; Yigit, Ferruh ; > Lu, Wenzhuo ; Wu, Jingjing > Cc: dev@dpdk.org; Slava Ovsiienko > Subject: RE: [PATCH v3] app/testpmd: fix copying the name of the dynflag > > Hi Ori

Re: [dpdk-dev] [PATCH v3] app/testpmd: fix copying the name of the dynflag

2020-02-04 Thread Iremonger, Bernard
Hi Ori, > > Subject: Re: [PATCH v3] app/testpmd: fix copying the name of the > > dynflag > > > > On 1/30/2020 9:04 PM, Ori Kam wrote: > > > When working with testpmd and setting the dynflag name, we copy the > > > name given by the cmd to the dynflag name. > > > > > > The issue is that the size

Re: [dpdk-dev] [PATCH v3] app/testpmd: fix copying the name of the dynflag

2020-02-02 Thread Ori Kam
Hi Ferruh, Thanks for your comments. > -Original Message- > From: Ferruh Yigit > Sent: Friday, January 31, 2020 4:02 PM > To: Ori Kam ; Wenzhuo Lu ; > Jingjing Wu ; Bernard Iremonger > > Cc: dev@dpdk.org; Slava Ovsiienko > Subject: Re: [PATCH v3] app/testpmd: fix copying the name of the

Re: [dpdk-dev] [PATCH v3] app/testpmd: fix copying the name of the dynflag

2020-01-31 Thread Ferruh Yigit
On 1/30/2020 9:04 PM, Ori Kam wrote: > When working with testpmd and setting the dynflag name, we copy the > name given by the cmd to the dynflag name. > > The issue is that the size of the dynflag name is smaller then the > string used by testpmd. > > This commit solves this issue by checking th