> -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
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.
> > > > >
> >
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
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
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
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
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 that the length of the requested
flag name is not too
7 matches
Mail list logo
