fec: Remove fec_enet_select_queue()
authorFabio Estevam <fabio.estevam@freescale.com>
Mon, 22 Sep 2014 20:55:29 +0000 (17:55 -0300)
committerDavid S. Miller <davem@davemloft.net>
Fri, 26 Sep 2014 19:39:59 +0000 (15:39 -0400)
Sparse complains about fec_enet_select_queue() not being static.

Feedback from David Miller [1] was to remove this function instead of making it
static:

"Please just delete this function.

It's overriding code which does exactly the same thing.

Actually, more precisely, this code is duplicating code in a way that
bypasses many core facilitites of the networking.  For example, this
override means that socket based flow steering, XPS, etc. are all
not happening on these devices.

Without ->ndo_select_queue(), the flow dissector does __netdev_pick_tx
which is exactly what you want to happen."

[1] http://www.spinics.net/lists/netdev/msg297653.html

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/freescale/fec_main.c

index 3a4ec0ff1a8cf3f82e6d182bb6885ca377425472..7379f4bb74b06cd0d8d2827d4090a58bccd672a5 100644 (file)
@@ -2851,17 +2851,10 @@ static int fec_set_features(struct net_device *netdev,
        return 0;
 }
 
-u16 fec_enet_select_queue(struct net_device *ndev, struct sk_buff *skb,
-                         void *accel_priv, select_queue_fallback_t fallback)
-{
-       return skb_tx_hash(ndev, skb);
-}
-
 static const struct net_device_ops fec_netdev_ops = {
        .ndo_open               = fec_enet_open,
        .ndo_stop               = fec_enet_close,
        .ndo_start_xmit         = fec_enet_start_xmit,
-       .ndo_select_queue       = fec_enet_select_queue,
        .ndo_set_rx_mode        = set_multicast_list,
        .ndo_change_mtu         = eth_change_mtu,
        .ndo_validate_addr      = eth_validate_addr,