[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240320172008.2989693-6-enachman@marvell.com>
Date: Wed, 20 Mar 2024 19:20:08 +0200
From: Elad Nachman <enachman@...vell.com>
To: <taras.chornyi@...ision.eu>, <davem@...emloft.net>, <edumazet@...gle.com>,
<kuba@...nel.org>, <pabeni@...hat.com>, <andrew@...n.ch>,
<kory.maincent@...tlin.com>, <thomas.petazzoni@...tlin.com>,
<miquel.raynal@...tlin.com>, <przemyslaw.kitszel@...el.com>,
<dkirjanov@...e.de>, <netdev@...r.kernel.org>,
<linux-kernel@...r.kernel.org>
CC: <enachman@...vell.com>
Subject: [PATCH v2 5/5] net: marvell: prestera: unbind sfp port on exit
From: Elad Nachman <enachman@...vell.com>
Call unbinding of the sfp port when ports are released, This will call
phylink_destroy() to cleanup and destroy the phylink instance, needed
to unregister SFP and free used memory, which will otherwise leak.
Reported-by: Köry Maincent <kory.maincent@...tlin.com>
Closes: https://lore.kernel.org/lkml/20240312164526.4a0e242a@kmaincent-XPS-13-7390/
Fixes: 501ef3066c89 ("net: marvell: prestera: Add driver for Prestera family ASIC devices")
Signed-off-by: Elad Nachman <enachman@...vell.com>
---
drivers/net/ethernet/marvell/prestera/prestera_main.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/net/ethernet/marvell/prestera/prestera_main.c b/drivers/net/ethernet/marvell/prestera/prestera_main.c
index 87d8fc4162b3..20c0ebb52af8 100644
--- a/drivers/net/ethernet/marvell/prestera/prestera_main.c
+++ b/drivers/net/ethernet/marvell/prestera/prestera_main.c
@@ -756,6 +756,7 @@ static void prestera_port_destroy(struct prestera_port *port)
cancel_delayed_work_sync(&port->cached_hw_stats.caching_dw);
unregister_netdev(dev);
prestera_port_list_del(port);
+ prestera_port_sfp_unbind(port);
prestera_devlink_port_unregister(port);
free_netdev(dev);
}
--
2.25.1
Powered by blists - more mailing lists