| 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 Jul 2022 14:38:16 -0300
From: Jose Alonso <joalonsof@...il.com>
To: justinpopo6@...il.com, linux-kernel@...r.kernel.org,
netdev@...r.kernel.org, linux-usb@...r.kernel.org,
jannh@...gle.com, jackychou@...x.com.tw, jesionowskigreg@...il.com,
pabeni@...hat.com, kuba@...nel.org, edumazet@...gle.com,
davem@...emloft.net, f.fainelli@...il.com
Cc: justin.chen@...adcom.com
Subject: Re: [PATCH 4/5] net: usb: ax88179_178a: move priv to driver_priv
On Mon, 2022-07-18 at 16:58 -0700, justinpopo6@...il.com wrote:
> From: Justin Chen <justinpopo6@...il.com>
>
> We need more space to save WoL context. So lets allocate memory
> for ax88179_data instead of using struct usbnet data field which
> only supports 5 words. We continue to use the struct usbnet data
> field for multicast filters. However since we no longer have the
> private data stored there, we can shift it to the beginning.
>
> Signed-off-by: Justin Chen <justinpopo6@...il.com>
> ---
> drivers/net/usb/ax88179_178a.c | 31 +++++++++++++++++++------------
> 1 file changed, 19 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/net/usb/ax88179_178a.c b/drivers/net/usb/ax88179_178a.c
> index 60742bb..cb7b89f 100644
> --- a/drivers/net/usb/ax88179_178a.c
> +++ b/drivers/net/usb/ax88179_178a.c
> @@ -170,7 +170,6 @@ struct ax88179_data {
> u8 eee_enabled;
> u8 eee_active;
> u16 rxctl;
> - u16 reserved;
> u8 in_pm;
> };
>
> @@ -190,14 +189,14 @@ static const struct {
>
> static void ax88179_set_pm_mode(struct usbnet *dev, bool pm_mode)
> {
> - struct ax88179_data *ax179_data = (struct ax88179_data *)dev->data;
> + struct ax88179_data *ax179_data = dev->driver_priv;
>
> ax179_data->in_pm = pm_mode;
> }
>
> static int ax88179_in_pm(struct usbnet *dev)
> {
> - struct ax88179_data *ax179_data = (struct ax88179_data *)dev->data;
> + struct ax88179_data *ax179_data = dev->driver_priv;
>
> return ax179_data->in_pm;
> }
> @@ -693,7 +692,7 @@ ax88179_ethtool_set_eee(struct usbnet *dev, struct ethtool_eee *data)
> static int ax88179_chk_eee(struct usbnet *dev)
> {
> struct ethtool_cmd ecmd = { .cmd = ETHTOOL_GSET };
> - struct ax88179_data *priv = (struct ax88179_data *)dev->data;
> + struct ax88179_data *priv = dev->driver_priv;
>
> mii_ethtool_gset(&dev->mii, &ecmd);
>
> @@ -796,7 +795,7 @@ static void ax88179_enable_eee(struct usbnet *dev)
> static int ax88179_get_eee(struct net_device *net, struct ethtool_eee *edata)
> {
> struct usbnet *dev = netdev_priv(net);
> - struct ax88179_data *priv = (struct ax88179_data *)dev->data;
> + struct ax88179_data *priv = dev->driver_priv;
>
> edata->eee_enabled = priv->eee_enabled;
> edata->eee_active = priv->eee_active;
> @@ -807,7 +806,7 @@ static int ax88179_get_eee(struct net_device *net, struct ethtool_eee *edata)
> static int ax88179_set_eee(struct net_device *net, struct ethtool_eee *edata)
> {
> struct usbnet *dev = netdev_priv(net);
> - struct ax88179_data *priv = (struct ax88179_data *)dev->data;
> + struct ax88179_data *priv = dev->driver_priv;
> int ret;
>
> priv->eee_enabled = edata->eee_enabled;
> @@ -858,8 +857,8 @@ static const struct ethtool_ops ax88179_ethtool_ops = {
> static void ax88179_set_multicast(struct net_device *net)
> {
> struct usbnet *dev = netdev_priv(net);
> - struct ax88179_data *data = (struct ax88179_data *)dev->data;
> - u8 *m_filter = ((u8 *)dev->data) + 12;
> + struct ax88179_data *data = dev->driver_priv;
> + u8 *m_filter = ((u8 *)dev->data);
>
> data->rxctl = (AX_RX_CTL_START | AX_RX_CTL_AB | AX_RX_CTL_IPE);
>
> @@ -871,7 +870,7 @@ static void ax88179_set_multicast(struct net_device *net)
> } else if (netdev_mc_empty(net)) {
> /* just broadcast and directed */
> } else {
> - /* We use the 20 byte dev->data for our 8 byte filter buffer
> + /* We use dev->data for our 8 byte filter buffer
> * to avoid allocating memory that is tricky to free later
> */
> u32 crc_bits;
> @@ -1273,10 +1272,15 @@ static int ax88179_bind(struct usbnet *dev, struct usb_interface *intf)
> u8 buf[5];
> u16 *tmp16;
> u8 *tmp;
> - struct ax88179_data *ax179_data = (struct ax88179_data *)dev->data;
> + struct ax88179_data *ax179_data;
>
> usbnet_get_endpoints(dev, intf);
>
> + ax179_data = kzalloc(sizeof(*ax179_data), GFP_KERNEL);
> + if (!ax179_data)
> + return -ENOMEM;
> +
> + dev->driver_priv = ax179_data;
> tmp16 = (u16 *)buf;
> tmp = (u8 *)buf;
following this line there is an unnecessary:
memset(ax179_data, 0, sizeof(*ax179_data));
>
> @@ -1310,6 +1314,7 @@ static int ax88179_bind(struct usbnet *dev, struct usb_interface *intf)
>
> static void ax88179_unbind(struct usbnet *dev, struct usb_interface *intf)
> {
> + struct ax88179_data *ax179_data = dev->driver_priv;
> u16 tmp16;
>
> /* Configure RX control register => stop operation */
> @@ -1322,6 +1327,8 @@ static void ax88179_unbind(struct usbnet *dev, struct usb_interface *intf)
> /* Power down ethernet PHY */
> tmp16 = 0;
> ax88179_write_cmd(dev, AX_ACCESS_MAC, AX_PHYPWR_RSTCTL, 2, 2, &tmp16);
> +
> + kfree(ax179_data);
> }
>
> static void
> @@ -1498,7 +1505,7 @@ ax88179_tx_fixup(struct usbnet *dev, struct sk_buff *skb, gfp_t flags)
>
> static int ax88179_link_reset(struct usbnet *dev)
> {
> - struct ax88179_data *ax179_data = (struct ax88179_data *)dev->data;
> + struct ax88179_data *ax179_data = dev->driver_priv;
> u8 tmp[5], link_sts;
> u16 mode, tmp16, delay = HZ / 10;
> u32 tmp32 = 0x40000000;
> @@ -1573,7 +1580,7 @@ static int ax88179_reset(struct usbnet *dev)
> u8 buf[5];
> u16 *tmp16;
> u8 *tmp;
> - struct ax88179_data *ax179_data = (struct ax88179_data *)dev->data;
> + struct ax88179_data *ax179_data = dev->driver_priv;
> struct ethtool_eee eee_data;
>
> tmp16 = (u16 *)buf;
--
Jose Alonso
Powered by blists - more mailing lists