Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933384Ab3CVOXS (ORCPT ); Fri, 22 Mar 2013 10:23:18 -0400 Received: from mail-da0-f50.google.com ([209.85.210.50]:65275 "EHLO mail-da0-f50.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932928Ab3CVOXQ (ORCPT ); Fri, 22 Mar 2013 10:23:16 -0400 Message-ID: <1363962193.4431.59.camel@edumazet-glaptop> Subject: Re: [PATCH] net: Add support for handling queueing in hardware From: Eric Dumazet To: Andy Fleming Cc: linux-kernel@vger.kernel.org, netdev@vger.kernel.org Date: Fri, 22 Mar 2013 07:23:13 -0700 In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.2.3-0ubuntu6 Content-Transfer-Encoding: 7bit Mime-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2346 Lines: 64 On Wed, 2011-07-13 at 08:52 -0500, Andy Fleming wrote: > The QDisc code does a bunch of locking which is unnecessary if > you have hardware which handles all of the queueing. Add > support for this, and skip over all of the queueing code if > the feature is enabled on a given device, which breaks QDisc > support on dpaa_eth, and also coopts the FCOE feature bit. > > Signed-off-by: Andy Fleming > Signed-off-by: Ben Collins > Cc: netdev@vger.kernel.org > --- > include/linux/netdev_features.h | 2 ++ > net/core/dev.c | 6 ++++++ > 2 files changed, 8 insertions(+) > > diff --git a/include/linux/netdev_features.h b/include/linux/netdev_features.h > index 3dd3934..ffb4587 100644 > --- a/include/linux/netdev_features.h > +++ b/include/linux/netdev_features.h > @@ -56,6 +56,7 @@ enum { > NETIF_F_LOOPBACK_BIT, /* Enable loopback */ > NETIF_F_RXFCS_BIT, /* Append FCS to skb pkt data */ > NETIF_F_RXALL_BIT, /* Receive errored frames too */ > + NETIF_F_HW_QDISC_BIT, /* Supports hardware Qdisc */ > > /* > * Add your fresh new feature above and remember to update > @@ -80,6 +81,7 @@ enum { > #define NETIF_F_GSO_ROBUST __NETIF_F(GSO_ROBUST) > #define NETIF_F_HIGHDMA __NETIF_F(HIGHDMA) > #define NETIF_F_HW_CSUM __NETIF_F(HW_CSUM) > +#define NETIF_F_HW_QDISC __NETIF_F(HW_QDISC) > #define NETIF_F_HW_VLAN_FILTER __NETIF_F(HW_VLAN_FILTER) > #define NETIF_F_HW_VLAN_RX __NETIF_F(HW_VLAN_RX) > #define NETIF_F_HW_VLAN_TX __NETIF_F(HW_VLAN_TX) > diff --git a/net/core/dev.c b/net/core/dev.c > index dffbef7..6818b18 100644 > --- a/net/core/dev.c > +++ b/net/core/dev.c > @@ -2743,6 +2743,12 @@ int dev_queue_xmit(struct sk_buff *skb) > > skb_update_prio(skb); > > + if (dev->features & NETIF_F_HW_QDISC) { > + txq = netdev_pick_tx(dev, skb); > + rc = dev_hard_start_xmit(skb, dev, txq); > + goto out; > + } > + > txq = netdev_pick_tx(dev, skb); > q = rcu_dereference_bh(txq->qdisc); > Nobody forces you to use a qdisc. And if your device really is lockless, it can use NETIF_F_LLTX feature. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/