Hi Thomas,

+John

> -----Original Message-----
> From: Thomas Monjalon [mailto:[email protected]]
> Sent: Tuesday, July 24, 2018 1:34 PM
> To: Singh, Jasvinder <[email protected]>
> Cc: [email protected]; Iremonger, Bernard <[email protected]>;
> Pattan, Reshma <[email protected]>; [email protected];
> [email protected]; Dumitrescu, Cristian
> <[email protected]>
> Subject: Re: [dpdk-dev] [PATCH] app/testpmd: fix testpmd exit using ctrl+d
> 
> Important note:
>       testpmd is currently really broken.
>       We cannot have a RC2 until it is fixed.
> 
> 
> 24/07/2018 13:25, Thomas Monjalon:
> > 23/07/2018 12:44, Jasvinder Singh:
> > > Fix testpmd app exit by pressing ctrl+d, End-of-Transmission
> > > character (EOT) on the empty command line.
> >
> > Please describe what is the issue and what is the cause.
> >
> > > Fixes: 0ad778b398c6 ("app/testpmd: rework softnic forward mode")
> > >
> > > Reported-by: Mordechay Haimovsky <[email protected]>
> > > Signed-off-by: Jasvinder Singh <[email protected]>
> > > ---
> > >  app/test-pmd/cmdline.c       | 10 ++++++----
> > >  lib/librte_cmdline/cmdline.c |  2 +-
> >
> > It is very suspicious to change the cmdline library.
> > If there is a bug in this library, it deserves a separate fix.
> 
> 

Do you know what commit has broken testpmd?

Regards,

Bernard

Reply via email to