Re: [PATCH net-next] net: macb: replace literal constant with NET_IP_ALIGN
From: Eric Dumazet <hidden>
Date: 2015-07-01 10:56:34
On Tue, 2015-06-30 at 20:25 +0300, Nicolae Rosia wrote:
quoted hunk ↗ jump to hunk
Signed-off-by: Nicolae Rosia <redacted> --- drivers/net/ethernet/cadence/macb.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)diff --git a/drivers/net/ethernet/cadence/macb.c b/drivers/net/ethernet/cadence/macb.c index caeb395..dbb5160 100644 --- a/drivers/net/ethernet/cadence/macb.c +++ b/drivers/net/ethernet/cadence/macb.c@@ -2554,9 +2554,9 @@ static void at91ether_rx(struct net_device *dev) while (lp->rx_ring[lp->rx_tail].addr & MACB_BIT(RX_USED)) { p_recv = lp->rx_buffers + lp->rx_tail * AT91ETHER_MAX_RBUFF_SZ; pktlen = MACB_BF(RX_FRMLEN, lp->rx_ring[lp->rx_tail].ctrl); - skb = netdev_alloc_skb(dev, pktlen + 2); + skb = netdev_alloc_skb(dev, pktlen + NET_IP_ALIGN); if (skb) { - skb_reserve(skb, 2); + skb_reserve(skb, NET_IP_ALIGN); memcpy(skb_put(skb, pktlen), p_recv, pktlen); skb->protocol = eth_type_trans(skb, dev);
Then please use netdev_alloc_skb_ip_align(), so that you get rid of skb_reserve()