> -----Original Message----- > From: David Marchand <[email protected]> > Sent: Tuesday, July 6, 2021 21:28 > To: Wang, Haiyue <[email protected]> > Cc: [email protected]; [email protected]; Yang, Qiming <[email protected]>; > Zhang, Qi Z > <[email protected]>; Stillwell Jr, Paul M <[email protected]> > Subject: Re: [PATCH] net/ice: fix memzone leak when firmware is missing > > On Tue, Jul 6, 2021 at 2:53 PM Wang, Haiyue <[email protected]> wrote: > > > > The function 'ice_deinit_hw' should be used. ;-) > > > > > > > > > > > > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/net/ethernet/intel/ice > > > /ice_main.c#n4508 > > > > > > Well I was inspired by ice_dev_close(), so I guess this code too could > > > use some cleanup but I don't have the hw anymore to test this change. > > > I'll trust you and simply update ice_dev_init. > > Adding to this, there are other potential leaks if ice_flow_init() or > ice_reset_fxp_resource() fails. > Probably worth opening a bz for later processing. > WDYT?
Yeah, make sense. ;-) > > > -- > David Marchand
