diff options
author | Felix Fietkau <nbd@openwrt.org> | 2013-03-11 02:29:47 +0000 |
---|---|---|
committer | Felix Fietkau <nbd@openwrt.org> | 2013-03-11 02:29:47 +0000 |
commit | d7a529709a9fe2ccfe079fb7669f4e8cd98a5fd5 (patch) | |
tree | 342070e9910a00a1f8fc965906ce89fea7f2c7bc /target/linux/cns3xxx | |
parent | 0d13baff1e88bebd625b698ecd861b6dca4fc85c (diff) | |
download | mtk-20170518-d7a529709a9fe2ccfe079fb7669f4e8cd98a5fd5.zip mtk-20170518-d7a529709a9fe2ccfe079fb7669f4e8cd98a5fd5.tar.gz mtk-20170518-d7a529709a9fe2ccfe079fb7669f4e8cd98a5fd5.tar.bz2 |
cns3xxx: ethernet - clean the tx ring only in the poll function, not in the xmit handler
SVN-Revision: 35952
Diffstat (limited to 'target/linux/cns3xxx')
-rw-r--r-- | target/linux/cns3xxx/files/drivers/net/ethernet/cavium/cns3xxx_eth.c | 17 |
1 files changed, 11 insertions, 6 deletions
diff --git a/target/linux/cns3xxx/files/drivers/net/ethernet/cavium/cns3xxx_eth.c b/target/linux/cns3xxx/files/drivers/net/ethernet/cavium/cns3xxx_eth.c index 7166288..9c3618f 100644 --- a/target/linux/cns3xxx/files/drivers/net/ethernet/cavium/cns3xxx_eth.c +++ b/target/linux/cns3xxx/files/drivers/net/ethernet/cavium/cns3xxx_eth.c @@ -436,14 +436,20 @@ static void cns3xxx_adjust_link(struct net_device *dev) dev->name, port->speed, port->duplex ? "full" : "half"); } +static void eth_schedule_poll(struct sw *sw) +{ + if (unlikely(!napi_schedule_prep(&sw->napi))) + return; + + disable_irq_nosync(IRQ_CNS3XXX_SW_R0RXC); + __napi_schedule(&sw->napi); +} + irqreturn_t eth_rx_irq(int irq, void *pdev) { struct net_device *dev = pdev; struct sw *sw = netdev_priv(dev); - if (likely(napi_schedule_prep(&sw->napi))) { - disable_irq_nosync(IRQ_CNS3XXX_SW_R0RXC); - __napi_schedule(&sw->napi); - } + eth_schedule_poll(sw); return (IRQ_HANDLED); } @@ -767,9 +773,8 @@ static int eth_xmit(struct sk_buff *skb, struct net_device *dev) skb_walk_frags(skb, skb1) nr_desc++; + eth_schedule_poll(sw); spin_lock_bh(&tx_lock); - - eth_complete_tx(sw); if ((tx_ring->num_used + nr_desc + 1) >= TX_DESCS) { spin_unlock_bh(&tx_lock); return NETDEV_TX_BUSY; |