On Fri, Mar 10, 2023 at 3:37 PM Dumitrescu, Cristian <[email protected]> wrote: > > > > > -----Original Message----- > > From: David Marchand <[email protected]> > > Sent: Friday, March 10, 2023 1:49 PM > > To: Dumitrescu, Cristian <[email protected]> > > Cc: Ferruh Yigit <[email protected]>; Stephen Hemminger > > <[email protected]>; Jangra, Yogesh <[email protected]>; > > Singh, Aman Deep <[email protected]>; Zhang, Yuying > > <[email protected]>; [email protected]; R, Kamalakannan > > <[email protected]>; Suresh Narayane, Harshad > > <[email protected]> > > Subject: Re: [PATCH] app/testpmd: fix closing softnic port before ethdev > > ports > > > > On Fri, Mar 10, 2023 at 2:48 PM Dumitrescu, Cristian > > <[email protected]> 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😉
Well sorry, but Ferruh request was reasonable to me. Trying to drop the ball made me a bit angry. > > 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, ... There are other drivers like the failsafe driver that sit on top of other devices. I'd recommend looking at what it does. -- David Marchand
