Commit 9f9919f7 authored by David S. Miller's avatar David S. Miller

Merge branch 'netif_rx'

Sebastian Andrzej Siewior says:

====================
net: Convert user to netif_rx().

This is the first batch of converting netif_rx_ni() caller to
netif_rx(). The change making this possible is net-next and
netif_rx_ni() is a wrapper around netif_rx(). This is a clean up in
order to remove netif_rx_ni().

Cc: Andrew Lunn <andrew@lunn.ch>
Cc: bridge@lists.linux-foundation.org
Cc: Chris Zankel <chris@zankel.net>
Cc: Florian Fainelli <f.fainelli@gmail.com>
Cc: Horatiu Vultur <horatiu.vultur@microchip.com>
Cc: Jonathan Corbet <corbet@lwn.net>
Cc: Kurt Kanzenbach <kurt@linutronix.de>
Cc: linux-doc@vger.kernel.org
Cc: linux-xtensa@linux-xtensa.org
Cc: Łukasz Stelmach <l.stelmach@samsung.com>
Cc: Max Filippov <jcmvbkbc@gmail.com>
Cc: Mike Travis <mike.travis@hpe.com>
Cc: Nikolay Aleksandrov <razor@blackwall.org>
Cc: Robin Holt <robinmholt@gmail.com>
Cc: Roopa Prabhu <roopa@nvidia.com>
Cc: Steve Wahl <steve.wahl@hpe.com>
Cc: UNGLinuxDriver@microchip.com
Cc: Vivien Didelot <vivien.didelot@gmail.com>
Cc: Vladimir Oltean <olteanv@gmail.com>
====================
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parents 4ee508ff ad0a043f
...@@ -668,7 +668,7 @@ timestamping: ...@@ -668,7 +668,7 @@ timestamping:
(through another RX timestamping FIFO). Deferral on RX is typically (through another RX timestamping FIFO). Deferral on RX is typically
necessary when retrieving the timestamp needs a sleepable context. In necessary when retrieving the timestamp needs a sleepable context. In
that case, it is the responsibility of the DSA driver to call that case, it is the responsibility of the DSA driver to call
``netif_rx_ni()`` on the freshly timestamped skb. ``netif_rx()`` on the freshly timestamped skb.
3.2.2 Ethernet PHYs 3.2.2 Ethernet PHYs
^^^^^^^^^^^^^^^^^^^ ^^^^^^^^^^^^^^^^^^^
......
...@@ -304,7 +304,7 @@ static int iss_net_rx(struct net_device *dev) ...@@ -304,7 +304,7 @@ static int iss_net_rx(struct net_device *dev)
lp->stats.rx_bytes += skb->len; lp->stats.rx_bytes += skb->len;
lp->stats.rx_packets++; lp->stats.rx_packets++;
netif_rx_ni(skb); netif_rx(skb);
return pkt_len; return pkt_len;
} }
kfree_skb(skb); kfree_skb(skb);
......
...@@ -247,7 +247,7 @@ xpnet_receive(short partid, int channel, struct xpnet_message *msg) ...@@ -247,7 +247,7 @@ xpnet_receive(short partid, int channel, struct xpnet_message *msg)
xpnet_device->stats.rx_packets++; xpnet_device->stats.rx_packets++;
xpnet_device->stats.rx_bytes += skb->len + ETH_HLEN; xpnet_device->stats.rx_bytes += skb->len + ETH_HLEN;
netif_rx_ni(skb); netif_rx(skb);
xpc_received(partid, channel, (void *)msg); xpc_received(partid, channel, (void *)msg);
} }
......
...@@ -196,7 +196,7 @@ static void ldisc_receive(struct tty_struct *tty, const u8 *data, ...@@ -196,7 +196,7 @@ static void ldisc_receive(struct tty_struct *tty, const u8 *data,
skb_reset_mac_header(skb); skb_reset_mac_header(skb);
debugfs_rx(ser, data, count); debugfs_rx(ser, data, count);
/* Push received packet up the stack. */ /* Push received packet up the stack. */
ret = netif_rx_ni(skb); ret = netif_rx(skb);
if (!ret) { if (!ret) {
ser->dev->stats.rx_packets++; ser->dev->stats.rx_packets++;
ser->dev->stats.rx_bytes += count; ser->dev->stats.rx_bytes += count;
......
...@@ -331,7 +331,7 @@ static void hellcreek_get_rxts(struct hellcreek *hellcreek, ...@@ -331,7 +331,7 @@ static void hellcreek_get_rxts(struct hellcreek *hellcreek,
shwt = skb_hwtstamps(skb); shwt = skb_hwtstamps(skb);
memset(shwt, 0, sizeof(*shwt)); memset(shwt, 0, sizeof(*shwt));
shwt->hwtstamp = ns_to_ktime(ns); shwt->hwtstamp = ns_to_ktime(ns);
netif_rx_ni(skb); netif_rx(skb);
} }
} }
......
...@@ -301,7 +301,7 @@ static void mv88e6xxx_get_rxts(struct mv88e6xxx_chip *chip, ...@@ -301,7 +301,7 @@ static void mv88e6xxx_get_rxts(struct mv88e6xxx_chip *chip,
shwt->hwtstamp = ns_to_ktime(ns); shwt->hwtstamp = ns_to_ktime(ns);
status &= ~MV88E6XXX_PTP_TS_VALID; status &= ~MV88E6XXX_PTP_TS_VALID;
} }
netif_rx_ni(skb); netif_rx(skb);
} }
} }
......
...@@ -399,7 +399,7 @@ static long sja1105_rxtstamp_work(struct ptp_clock_info *ptp) ...@@ -399,7 +399,7 @@ static long sja1105_rxtstamp_work(struct ptp_clock_info *ptp)
ts = sja1105_tstamp_reconstruct(ds, ticks, ts); ts = sja1105_tstamp_reconstruct(ds, ticks, ts);
shwt->hwtstamp = ns_to_ktime(sja1105_ticks_to_ns(ts)); shwt->hwtstamp = ns_to_ktime(sja1105_ticks_to_ns(ts));
netif_rx_ni(skb); netif_rx(skb);
} }
if (ptp_data->extts_enabled) if (ptp_data->extts_enabled)
......
...@@ -433,7 +433,7 @@ ax88796c_skb_return(struct ax88796c_device *ax_local, ...@@ -433,7 +433,7 @@ ax88796c_skb_return(struct ax88796c_device *ax_local,
netif_info(ax_local, rx_status, ndev, "< rx, len %zu, type 0x%x\n", netif_info(ax_local, rx_status, ndev, "< rx, len %zu, type 0x%x\n",
skb->len + sizeof(struct ethhdr), skb->protocol); skb->len + sizeof(struct ethhdr), skb->protocol);
status = netif_rx_ni(skb); status = netif_rx(skb);
if (status != NET_RX_SUCCESS && net_ratelimit()) if (status != NET_RX_SUCCESS && net_ratelimit())
netif_info(ax_local, rx_err, ndev, netif_info(ax_local, rx_err, ndev,
"netif_rx status %d\n", status); "netif_rx status %d\n", status);
......
...@@ -804,7 +804,7 @@ static int dm9051_loop_rx(struct board_info *db) ...@@ -804,7 +804,7 @@ static int dm9051_loop_rx(struct board_info *db)
skb->protocol = eth_type_trans(skb, db->ndev); skb->protocol = eth_type_trans(skb, db->ndev);
if (db->ndev->features & NETIF_F_RXCSUM) if (db->ndev->features & NETIF_F_RXCSUM)
skb_checksum_none_assert(skb); skb_checksum_none_assert(skb);
netif_rx_ni(skb); netif_rx(skb);
db->ndev->stats.rx_bytes += rxlen; db->ndev->stats.rx_bytes += rxlen;
db->ndev->stats.rx_packets++; db->ndev->stats.rx_packets++;
scanrr++; scanrr++;
......
...@@ -293,7 +293,7 @@ static void ks8851_wrfifo_spi(struct ks8851_net *ks, struct sk_buff *txp, ...@@ -293,7 +293,7 @@ static void ks8851_wrfifo_spi(struct ks8851_net *ks, struct sk_buff *txp,
*/ */
static void ks8851_rx_skb_spi(struct ks8851_net *ks, struct sk_buff *skb) static void ks8851_rx_skb_spi(struct ks8851_net *ks, struct sk_buff *skb)
{ {
netif_rx_ni(skb); netif_rx(skb);
} }
/** /**
......
...@@ -975,7 +975,7 @@ static void enc28j60_hw_rx(struct net_device *ndev) ...@@ -975,7 +975,7 @@ static void enc28j60_hw_rx(struct net_device *ndev)
/* update statistics */ /* update statistics */
ndev->stats.rx_packets++; ndev->stats.rx_packets++;
ndev->stats.rx_bytes += len; ndev->stats.rx_bytes += len;
netif_rx_ni(skb); netif_rx(skb);
} }
} }
/* /*
......
...@@ -600,7 +600,7 @@ static irqreturn_t lan966x_xtr_irq_handler(int irq, void *args) ...@@ -600,7 +600,7 @@ static irqreturn_t lan966x_xtr_irq_handler(int irq, void *args)
skb->offload_fwd_mark = 0; skb->offload_fwd_mark = 0;
} }
netif_rx_ni(skb); netif_rx(skb);
dev->stats.rx_bytes += len; dev->stats.rx_bytes += len;
dev->stats.rx_packets++; dev->stats.rx_packets++;
......
...@@ -435,7 +435,7 @@ qcaspi_receive(struct qcaspi *qca) ...@@ -435,7 +435,7 @@ qcaspi_receive(struct qcaspi *qca)
qca->rx_skb->protocol = eth_type_trans( qca->rx_skb->protocol = eth_type_trans(
qca->rx_skb, qca->rx_skb->dev); qca->rx_skb, qca->rx_skb->dev);
skb_checksum_none_assert(qca->rx_skb); skb_checksum_none_assert(qca->rx_skb);
netif_rx_ni(qca->rx_skb); netif_rx(qca->rx_skb);
qca->rx_skb = netdev_alloc_skb_ip_align(net_dev, qca->rx_skb = netdev_alloc_skb_ip_align(net_dev,
net_dev->mtu + VLAN_ETH_HLEN); net_dev->mtu + VLAN_ETH_HLEN);
if (!qca->rx_skb) { if (!qca->rx_skb) {
......
...@@ -108,7 +108,7 @@ qca_tty_receive(struct serdev_device *serdev, const unsigned char *data, ...@@ -108,7 +108,7 @@ qca_tty_receive(struct serdev_device *serdev, const unsigned char *data,
qca->rx_skb->protocol = eth_type_trans( qca->rx_skb->protocol = eth_type_trans(
qca->rx_skb, qca->rx_skb->dev); qca->rx_skb, qca->rx_skb->dev);
skb_checksum_none_assert(qca->rx_skb); skb_checksum_none_assert(qca->rx_skb);
netif_rx_ni(qca->rx_skb); netif_rx(qca->rx_skb);
qca->rx_skb = netdev_alloc_skb_ip_align(netdev, qca->rx_skb = netdev_alloc_skb_ip_align(netdev,
netdev->mtu + netdev->mtu +
VLAN_ETH_HLEN); VLAN_ETH_HLEN);
......
...@@ -362,7 +362,7 @@ static void mse102x_rx_pkt_spi(struct mse102x_net *mse) ...@@ -362,7 +362,7 @@ static void mse102x_rx_pkt_spi(struct mse102x_net *mse)
mse102x_dump_packet(__func__, skb->len, skb->data); mse102x_dump_packet(__func__, skb->len, skb->data);
skb->protocol = eth_type_trans(skb, mse->ndev); skb->protocol = eth_type_trans(skb, mse->ndev);
netif_rx_ni(skb); netif_rx(skb);
mse->ndev->stats.rx_packets++; mse->ndev->stats.rx_packets++;
mse->ndev->stats.rx_bytes += rxlen; mse->ndev->stats.rx_bytes += rxlen;
......
...@@ -883,7 +883,7 @@ static void w5100_rx_work(struct work_struct *work) ...@@ -883,7 +883,7 @@ static void w5100_rx_work(struct work_struct *work)
struct sk_buff *skb; struct sk_buff *skb;
while ((skb = w5100_rx_skb(priv->ndev))) while ((skb = w5100_rx_skb(priv->ndev)))
netif_rx_ni(skb); netif_rx(skb);
w5100_enable_intr(priv); w5100_enable_intr(priv);
} }
......
...@@ -285,7 +285,7 @@ static void macvlan_broadcast(struct sk_buff *skb, ...@@ -285,7 +285,7 @@ static void macvlan_broadcast(struct sk_buff *skb,
if (likely(nskb)) if (likely(nskb))
err = macvlan_broadcast_one(nskb, vlan, eth, err = macvlan_broadcast_one(nskb, vlan, eth,
mode == MACVLAN_MODE_BRIDGE) ?: mode == MACVLAN_MODE_BRIDGE) ?:
netif_rx_ni(nskb); netif_rx(nskb);
macvlan_count_rx(vlan, skb->len + ETH_HLEN, macvlan_count_rx(vlan, skb->len + ETH_HLEN,
err == NET_RX_SUCCESS, true); err == NET_RX_SUCCESS, true);
} }
......
...@@ -84,7 +84,7 @@ static void br_arp_send(struct net_bridge *br, struct net_bridge_port *p, ...@@ -84,7 +84,7 @@ static void br_arp_send(struct net_bridge *br, struct net_bridge_port *p,
skb->ip_summed = CHECKSUM_UNNECESSARY; skb->ip_summed = CHECKSUM_UNNECESSARY;
skb->pkt_type = PACKET_HOST; skb->pkt_type = PACKET_HOST;
netif_rx_ni(skb); netif_rx(skb);
} }
} }
...@@ -364,7 +364,7 @@ static void br_nd_send(struct net_bridge *br, struct net_bridge_port *p, ...@@ -364,7 +364,7 @@ static void br_nd_send(struct net_bridge *br, struct net_bridge_port *p,
reply->ip_summed = CHECKSUM_UNNECESSARY; reply->ip_summed = CHECKSUM_UNNECESSARY;
reply->pkt_type = PACKET_HOST; reply->pkt_type = PACKET_HOST;
netif_rx_ni(reply); netif_rx(reply);
} }
} }
......
...@@ -99,7 +99,7 @@ static int chnl_recv_cb(struct cflayer *layr, struct cfpkt *pkt) ...@@ -99,7 +99,7 @@ static int chnl_recv_cb(struct cflayer *layr, struct cfpkt *pkt)
else else
skb->ip_summed = CHECKSUM_NONE; skb->ip_summed = CHECKSUM_NONE;
netif_rx_any_context(skb); netif_rx(skb);
/* Update statistics. */ /* Update statistics. */
priv->netdev->stats.rx_packets++; priv->netdev->stats.rx_packets++;
......
...@@ -3860,7 +3860,7 @@ int dev_loopback_xmit(struct net *net, struct sock *sk, struct sk_buff *skb) ...@@ -3860,7 +3860,7 @@ int dev_loopback_xmit(struct net *net, struct sock *sk, struct sk_buff *skb)
skb->ip_summed = CHECKSUM_UNNECESSARY; skb->ip_summed = CHECKSUM_UNNECESSARY;
WARN_ON(!skb_dst(skb)); WARN_ON(!skb_dst(skb));
skb_dst_force(skb); skb_dst_force(skb);
netif_rx_ni(skb); netif_rx(skb);
return 0; return 0;
} }
EXPORT_SYMBOL(dev_loopback_xmit); EXPORT_SYMBOL(dev_loopback_xmit);
...@@ -10949,11 +10949,11 @@ static int dev_cpu_dead(unsigned int oldcpu) ...@@ -10949,11 +10949,11 @@ static int dev_cpu_dead(unsigned int oldcpu)
/* Process offline CPU's input_pkt_queue */ /* Process offline CPU's input_pkt_queue */
while ((skb = __skb_dequeue(&oldsd->process_queue))) { while ((skb = __skb_dequeue(&oldsd->process_queue))) {
netif_rx_ni(skb); netif_rx(skb);
input_queue_head_incr(oldsd); input_queue_head_incr(oldsd);
} }
while ((skb = skb_dequeue(&oldsd->input_pkt_queue))) { while ((skb = skb_dequeue(&oldsd->input_pkt_queue))) {
netif_rx_ni(skb); netif_rx(skb);
input_queue_head_incr(oldsd); input_queue_head_incr(oldsd);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment