| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Tue, 19 Sep 2023 10:33:29 +0200
From: Uwe Kleine-König <u.kleine-koenig@...gutronix.de>
To: "Pandey, Radhey Shyam" <radhey.shyam.pandey@....com>
Cc: "David S. Miller" <davem@...emloft.net>,
Eric Dumazet <edumazet@...gle.com>,
Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>,
"Katakam, Harini" <harini.katakam@....com>,
Rob Herring <robh@...nel.org>,
"kernel@...gutronix.de" <kernel@...gutronix.de>,
Bhupesh Sharma <bhupesh.sharma@...aro.org>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
Alex Elder <elder@...aro.org>, Simon Horman <horms@...nel.org>,
Wei Fang <wei.fang@....com>,
"linux-arm-kernel@...ts.infradead.org" <linux-arm-kernel@...ts.infradead.org>,
huangjunxian <huangjunxian6@...ilicon.com>,
Yang Yingliang <yangyingliang@...wei.com>,
"Simek, Michal" <michal.simek@....com>,
Haoyue Xu <xuhaoyue1@...ilicon.com>,
Dan Carpenter <dan.carpenter@...aro.org>
Subject: Re: [PATCH net-next 53/54] net: ethernet: xilinx: Convert to
platform remove callback returning void
Hello,
[dropped Bhupesh Sharma from the list of recipents, usage of their email
address resulted in a bounce.]
On Tue, Sep 19, 2023 at 04:45:04AM +0000, Pandey, Radhey Shyam wrote:
> > -----Original Message-----
> > From: Uwe Kleine-König <u.kleine-koenig@...gutronix.de>
> > Sent: Tuesday, September 19, 2023 2:12 AM
> > To: David S. Miller <davem@...emloft.net>; Eric Dumazet
> > <edumazet@...gle.com>; Jakub Kicinski <kuba@...nel.org>; Paolo Abeni
> > <pabeni@...hat.com>
> > Cc: Simek, Michal <michal.simek@....com>; Pandey, Radhey Shyam
> > <radhey.shyam.pandey@....com>; Katakam, Harini
> > <harini.katakam@....com>; Haoyue Xu <xuhaoyue1@...ilicon.com>;
> > huangjunxian <huangjunxian6@...ilicon.com>; Rob Herring
> > <robh@...nel.org>; Yang Yingliang <yangyingliang@...wei.com>; Dan
> > Carpenter <dan.carpenter@...aro.org>; Bhupesh Sharma
> > <bhupesh.sharma@...aro.org>; Simon Horman <horms@...nel.org>; Alex
> > Elder <elder@...aro.org>; Wei Fang <wei.fang@....com>;
> > netdev@...r.kernel.org; linux-arm-kernel@...ts.infradead.org;
> > kernel@...gutronix.de
> > Subject: [PATCH net-next 53/54] net: ethernet: xilinx: Convert to platform
> > remove callback returning void
> >
> > The .remove() callback for a platform driver returns an int which makes
> > many driver authors wrongly assume it's possible to do error handling by
> > returning an error code. However the value returned is ignored (apart
> > from emitting a warning) and this typically results in resource leaks.
> > To improve here there is a quest to make the remove callback return
> > void. In the first step of this quest all drivers are converted to
> > .remove_new() which already returns void. Eventually after all drivers
> > are converted, .remove_new() is renamed to .remove().
> >
> > Trivially convert these drivers from always returning zero in the remove
> > callback to the void returning variant.
> >
> > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@...gutronix.de>
> > ---
> > drivers/net/ethernet/xilinx/ll_temac_main.c | 5 ++---
> > drivers/net/ethernet/xilinx/xilinx_axienet_main.c | 6 ++----
> > drivers/net/ethernet/xilinx/xilinx_emaclite.c | 6 ++----
> > 3 files changed, 6 insertions(+), 11 deletions(-)
> >
> > diff --git a/drivers/net/ethernet/xilinx/ll_temac_main.c
> > b/drivers/net/ethernet/xilinx/ll_temac_main.c
> > index 1444b855e7aa..9df39cf8b097 100644
> > --- a/drivers/net/ethernet/xilinx/ll_temac_main.c
> > +++ b/drivers/net/ethernet/xilinx/ll_temac_main.c
> > @@ -1626,7 +1626,7 @@ static int temac_probe(struct platform_device
> > *pdev)
> > return rc;
> > }
> >
> > -static int temac_remove(struct platform_device *pdev)
> > +static void temac_remove(struct platform_device *pdev)
> > {
> > struct net_device *ndev = platform_get_drvdata(pdev);
> > struct temac_local *lp = netdev_priv(ndev);
> > @@ -1636,7 +1636,6 @@ static int temac_remove(struct platform_device
> > *pdev)
> > if (lp->phy_node)
> > of_node_put(lp->phy_node);
> > temac_mdio_teardown(lp);
> > - return 0;
> > }
> >
> > static const struct of_device_id temac_of_match[] = {
> > @@ -1650,7 +1649,7 @@ MODULE_DEVICE_TABLE(of, temac_of_match);
> >
> > static struct platform_driver temac_driver = {
> > .probe = temac_probe,
> > - .remove = temac_remove,
> > + .remove_new = temac_remove,
> > .driver = {
> > .name = "xilinx_temac",
> > .of_match_table = temac_of_match,
> > diff --git a/drivers/net/ethernet/xilinx/xilinx_axienet_main.c
> > b/drivers/net/ethernet/xilinx/xilinx_axienet_main.c
> > index b7ec4dafae90..82d0d44b2b02 100644
> > --- a/drivers/net/ethernet/xilinx/xilinx_axienet_main.c
> > +++ b/drivers/net/ethernet/xilinx/xilinx_axienet_main.c
> > @@ -2183,7 +2183,7 @@ static int axienet_probe(struct platform_device
> > *pdev)
> > return ret;
> > }
> >
> > -static int axienet_remove(struct platform_device *pdev)
> > +static void axienet_remove(struct platform_device *pdev)
> > {
> > struct net_device *ndev = platform_get_drvdata(pdev);
> > struct axienet_local *lp = netdev_priv(ndev);
> > @@ -2202,8 +2202,6 @@ static int axienet_remove(struct platform_device
> > *pdev)
> > clk_disable_unprepare(lp->axi_clk);
> >
> > free_netdev(ndev);
> > -
> > - return 0;
> > }
> >
> > static void axienet_shutdown(struct platform_device *pdev)
> > @@ -2256,7 +2254,7 @@ static
> > DEFINE_SIMPLE_DEV_PM_OPS(axienet_pm_ops,
> >
> > static struct platform_driver axienet_driver = {
> > .probe = axienet_probe,
> > - .remove = axienet_remove,
> > + .remove_new = axienet_remove,
> > .shutdown = axienet_shutdown,
> > .driver = {
> > .name = "xilinx_axienet",
> > diff --git a/drivers/net/ethernet/xilinx/xilinx_emaclite.c
> > b/drivers/net/ethernet/xilinx/xilinx_emaclite.c
> > index b358ecc67227..32a502e7318b 100644
> > --- a/drivers/net/ethernet/xilinx/xilinx_emaclite.c
> > +++ b/drivers/net/ethernet/xilinx/xilinx_emaclite.c
> > @@ -1183,7 +1183,7 @@ static int xemaclite_of_probe(struct
> > platform_device *ofdev)
> > *
> > * Return: 0, always.
> > */
>
> Nit - kernel-doc return documentation needs to be updated.
Indeed, I fixed that in my tree and so it will be addressed if and when
I resend this patch.
Thanks
Uwe
--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | https://www.pengutronix.de/ |
Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)
Powered by blists - more mailing lists