| Message ID | 20210119150802.19997-15-rasmus.villemoes@prevas.dk |
|---|---|
| State | New |
| Headers | show |
| Series | [net-next,v2,01/17] ethernet: ucc_geth: remove unused read of temoder field | expand |
Can you detail a bit the change ? At least tell that the loop was a zeroising loop and that kcalloc() already zeroises the allocated memory ? Le 19/01/2021 à 16:07, Rasmus Villemoes a écrit : > Signed-off-by: Rasmus Villemoes <rasmus.villemoes@prevas.dk> > --- > drivers/net/ethernet/freescale/ucc_geth.c | 14 ++++---------- > 1 file changed, 4 insertions(+), 10 deletions(-) > > diff --git a/drivers/net/ethernet/freescale/ucc_geth.c b/drivers/net/ethernet/freescale/ucc_geth.c > index 1e9d2f3f47a3..621a9e3e4b65 100644 > --- a/drivers/net/ethernet/freescale/ucc_geth.c > +++ b/drivers/net/ethernet/freescale/ucc_geth.c > @@ -2203,8 +2203,8 @@ static int ucc_geth_alloc_tx(struct ucc_geth_private *ugeth) > for (j = 0; j < ug_info->numQueuesTx; j++) { > /* Setup the skbuff rings */ > ugeth->tx_skbuff[j] = > - kmalloc_array(ugeth->ug_info->bdRingLenTx[j], > - sizeof(struct sk_buff *), GFP_KERNEL); > + kcalloc(ugeth->ug_info->bdRingLenTx[j], > + sizeof(struct sk_buff *), GFP_KERNEL); > > if (ugeth->tx_skbuff[j] == NULL) { > if (netif_msg_ifup(ugeth)) > @@ -2212,9 +2212,6 @@ static int ucc_geth_alloc_tx(struct ucc_geth_private *ugeth) > return -ENOMEM; > } > > - for (i = 0; i < ugeth->ug_info->bdRingLenTx[j]; i++) > - ugeth->tx_skbuff[j][i] = NULL; > - > ugeth->skb_curtx[j] = ugeth->skb_dirtytx[j] = 0; > bd = ugeth->confBd[j] = ugeth->txBd[j] = ugeth->p_tx_bd_ring[j]; > for (i = 0; i < ug_info->bdRingLenTx[j]; i++) { > @@ -2266,8 +2263,8 @@ static int ucc_geth_alloc_rx(struct ucc_geth_private *ugeth) > for (j = 0; j < ug_info->numQueuesRx; j++) { > /* Setup the skbuff rings */ > ugeth->rx_skbuff[j] = > - kmalloc_array(ugeth->ug_info->bdRingLenRx[j], > - sizeof(struct sk_buff *), GFP_KERNEL); > + kcalloc(ugeth->ug_info->bdRingLenRx[j], > + sizeof(struct sk_buff *), GFP_KERNEL); > > if (ugeth->rx_skbuff[j] == NULL) { > if (netif_msg_ifup(ugeth)) > @@ -2275,9 +2272,6 @@ static int ucc_geth_alloc_rx(struct ucc_geth_private *ugeth) > return -ENOMEM; > } > > - for (i = 0; i < ugeth->ug_info->bdRingLenRx[j]; i++) > - ugeth->rx_skbuff[j][i] = NULL; > - > ugeth->skb_currx[j] = 0; > bd = ugeth->rxBd[j] = ugeth->p_rx_bd_ring[j]; > for (i = 0; i < ug_info->bdRingLenRx[j]; i++) { >
diff --git a/drivers/net/ethernet/freescale/ucc_geth.c b/drivers/net/ethernet/freescale/ucc_geth.c index 1e9d2f3f47a3..621a9e3e4b65 100644 --- a/drivers/net/ethernet/freescale/ucc_geth.c +++ b/drivers/net/ethernet/freescale/ucc_geth.c @@ -2203,8 +2203,8 @@ static int ucc_geth_alloc_tx(struct ucc_geth_private *ugeth) for (j = 0; j < ug_info->numQueuesTx; j++) { /* Setup the skbuff rings */ ugeth->tx_skbuff[j] = - kmalloc_array(ugeth->ug_info->bdRingLenTx[j], - sizeof(struct sk_buff *), GFP_KERNEL); + kcalloc(ugeth->ug_info->bdRingLenTx[j], + sizeof(struct sk_buff *), GFP_KERNEL); if (ugeth->tx_skbuff[j] == NULL) { if (netif_msg_ifup(ugeth)) @@ -2212,9 +2212,6 @@ static int ucc_geth_alloc_tx(struct ucc_geth_private *ugeth) return -ENOMEM; } - for (i = 0; i < ugeth->ug_info->bdRingLenTx[j]; i++) - ugeth->tx_skbuff[j][i] = NULL; - ugeth->skb_curtx[j] = ugeth->skb_dirtytx[j] = 0; bd = ugeth->confBd[j] = ugeth->txBd[j] = ugeth->p_tx_bd_ring[j]; for (i = 0; i < ug_info->bdRingLenTx[j]; i++) { @@ -2266,8 +2263,8 @@ static int ucc_geth_alloc_rx(struct ucc_geth_private *ugeth) for (j = 0; j < ug_info->numQueuesRx; j++) { /* Setup the skbuff rings */ ugeth->rx_skbuff[j] = - kmalloc_array(ugeth->ug_info->bdRingLenRx[j], - sizeof(struct sk_buff *), GFP_KERNEL); + kcalloc(ugeth->ug_info->bdRingLenRx[j], + sizeof(struct sk_buff *), GFP_KERNEL); if (ugeth->rx_skbuff[j] == NULL) { if (netif_msg_ifup(ugeth)) @@ -2275,9 +2272,6 @@ static int ucc_geth_alloc_rx(struct ucc_geth_private *ugeth) return -ENOMEM; } - for (i = 0; i < ugeth->ug_info->bdRingLenRx[j]; i++) - ugeth->rx_skbuff[j][i] = NULL; - ugeth->skb_currx[j] = 0; bd = ugeth->rxBd[j] = ugeth->p_rx_bd_ring[j]; for (i = 0; i < ug_info->bdRingLenRx[j]; i++) {
Signed-off-by: Rasmus Villemoes <rasmus.villemoes@prevas.dk> --- drivers/net/ethernet/freescale/ucc_geth.c | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-)