[PATCH] app/testpmd: fix closing softnic port before ethdev ports

[Dumitrescu, Cristian]

> -----Original Message-----
> From: David Marchand <david.marchand at redhat.com>
> Sent: Friday, March 10, 2023 1:49 PM
> To: Dumitrescu, Cristian <cristian.dumitrescu at intel.com>
> Cc: Ferruh Yigit <ferruh.yigit at amd.com>; Stephen Hemminger
> <stephen at networkplumber.org>; Jangra, Yogesh <yogesh.jangra at intel.com>;
> Singh, Aman Deep <aman.deep.singh at intel.com>; Zhang, Yuying
> <yuying.zhang at intel.com>; dev at dpdk.org; R, Kamalakannan
> <kamalakannan.r at intel.com>; Suresh Narayane, Harshad
> <harshad.suresh.narayane at intel.com>
> Subject: Re: [PATCH] app/testpmd: fix closing softnic port before ethdev ports
> 
> On Fri, Mar 10, 2023 at 2:48 PM Dumitrescu, Cristian
> <cristian.dumitrescu at intel.com> wrote:
> > > Why not fix the misbehaving drivers, instead of working around for
> > > softnic, as Stephen suggested?
> > >
> > > Is there a list of problematic drivers?
> > >
> >
> > Ferruh, I think this is not a reasonable request. We don't have the expertise to
> fix all drivers, not the hardware to test all drivers.
> 
> Then, please report the issues.
> I am also NAKing this horror.
> 
> 
> --
> David Marchand

Hi David,

Your ideas on how to fix this issue would be way more helpful than your NAK😉

Can you please explain why this fix is a problem? We have device X that sits on top of devices A, B, C, ... who are created and started before device X, to me it makes sense to stop device X first before stopping A, B, C, ...

Thanks,
Cristian