From: David Miller <da...@davemloft.net> Sent: Saturday, May 25, 2019 4:12 AM > From: Andy Duan <fugang.d...@nxp.com> > Date: Thu, 23 May 2019 01:55:23 +0000 > > > @@ -3146,7 +3150,10 @@ static int fec_enet_init(struct net_device *ndev) > > memset(cbd_base, 0, bd_size); > > > > /* Get the Ethernet address */ > > - fec_get_mac(ndev); > > + ret = fec_get_mac(ndev); > > + if (ret) > > + return ret; > > You're leaking the queues allocated by fec_enet_alloc_queue().
Good caught, thanks David. I will send two patches for the V2 to fix the exisited queue memory leak.
