From: Uwe Kleine-König Date: Tue, 30 Jul 2013 09:29:40 +0000 (+0200) Subject: net/fec: Don't let ndo_start_xmit return NETDEV_TX_BUSY without link X-Git-Url: https://git.karo-electronics.de/?a=commitdiff_plain;h=737420d568a1106b81d6491ca80080f42f0ac69d;p=karo-tx-linux.git net/fec: Don't let ndo_start_xmit return NETDEV_TX_BUSY without link Don't test for having link and let hardware deal with this situation. Without this patch I see a machine running an -rt patched Linux being stuck in sch_direct_xmit when it looses link while there is still a packet to be sent. In this case the fec_enet_start_xmit routine returned NETDEV_TX_BUSY which makes the network stack reschedule the packet and so sch_direct_xmit calls fec_enet_start_xmit again. I failed to reproduce a complete hang without -rt, but I think the problem exists there, too. Signed-off-by: Uwe Kleine-König Signed-off-by: David S. Miller --- diff --git a/drivers/net/ethernet/freescale/fec_main.c b/drivers/net/ethernet/freescale/fec_main.c index a5d35d8f8514..8f296d71eecb 100644 --- a/drivers/net/ethernet/freescale/fec_main.c +++ b/drivers/net/ethernet/freescale/fec_main.c @@ -294,11 +294,6 @@ fec_enet_start_xmit(struct sk_buff *skb, struct net_device *ndev) unsigned short status; unsigned int index; - if (!fep->link) { - /* Link is down or auto-negotiation is in progress. */ - return NETDEV_TX_BUSY; - } - /* Fill in a Tx ring entry */ bdp = fep->cur_tx;