[dpdk-dev] [PATCH] app/testpmd: remove softnic forward mode

[Singh, Jasvinder]

> -----Original Message-----
> From: David Marchand <david.marchand at redhat.com>
> Sent: Tuesday, June 23, 2020 9:42 AM
> To: Singh, Jasvinder <jasvinder.singh at intel.com>
> Cc: Phil Yang <Phil.Yang at arm.com>; dev at dpdk.org; Dumitrescu, Cristian
> <cristian.dumitrescu at intel.com>
> Subject: Re: [dpdk-dev] [PATCH] app/testpmd: remove softnic forward mode
> 
> On Tue, Jun 23, 2020 at 10:38 AM Singh, Jasvinder
> <jasvinder.singh at intel.com> wrote:
> >
> >
> >
> > > -----Original Message-----
> > > From: Phil Yang <Phil.Yang at arm.com>
> > > Sent: Tuesday, June 23, 2020 8:38 AM
> > > To: Singh, Jasvinder <jasvinder.singh at intel.com>; dev at dpdk.org
> > > Cc: Dumitrescu, Cristian <cristian.dumitrescu at intel.com>
> > > Subject: RE: [dpdk-dev] [PATCH] app/testpmd: remove softnic forward
> > > mode
> > >
> > > > -----Original Message-----
> > > > From: dev <dev-bounces at dpdk.org> On Behalf Of Jasvinder Singh
> > > > Sent: Monday, June 22, 2020 11:51 PM
> > > > To: dev at dpdk.org
> > > > Cc: cristian.dumitrescu at intel.com
> > > > Subject: [dpdk-dev] [PATCH] app/testpmd: remove softnic forward
> > > > mode
> > > >
> > > > Softnic can be used like other virtual devices without needing any
> > > > special mode. Therefore, softnic forward mode is removed from
> > > > testpmd app. Documentation is updated as well.
> > > >
> > > Hi Jasvinder,
> > >
> > > One quick question for this change.  How can we test the softnic PMD
> > > without this mode?
> > > The softnic PMD guidance uses testpmd softnic mode in the PMD test
> > > section.
> > > However, when I switch the forward mode from softnic to IO mode, the
> > > testpmd cannot work as expected.
> > >
> > > I think you need to update the softnic PMD page as well.
> > > doc/guides/nics/softnic.rst
> > >
> > > Thanks,
> > > Phil
> > >
> >
> > Hi Phil,
> >
> > I have sent a separate patch on softnic PMD document update with the
> > command to run softnic with testpmd,
> > https://patches.dpdk.org/patch/71984/
> 
> It should be a single patch.
> 

Ok, will merge document patch with this one.
 
> --
> David Marchand