From patchwork Mon Dec 13 14:43:33 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Changli Gao X-Patchwork-Id: 75362 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 060DBB6F11 for ; Tue, 14 Dec 2010 01:44:41 +1100 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754281Ab0LMOog (ORCPT ); Mon, 13 Dec 2010 09:44:36 -0500 Received: from mail-gw0-f42.google.com ([74.125.83.42]:60347 "EHLO mail-gw0-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754034Ab0LMOof (ORCPT ); Mon, 13 Dec 2010 09:44:35 -0500 Received: by gwb20 with SMTP id 20so5017503gwb.1 for ; Mon, 13 Dec 2010 06:44:35 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:from:to:cc:subject:date :message-id:x-mailer:in-reply-to:references; bh=kZBnzBxDhjpK8QDqGFM6tADTwUSrnP4iVNxCKz6nzyw=; b=jkQURiQw8P0IBQ0zXYc8BkhUcRHJKyza3N0qCk1fqxTVno8+9wI8GSmxUv2odxVoO+ w8MZPNRY/LGrGV+omXFMsJDS22FHmYf5iAJ+6WSmIK04r6FffqqxSMuocZXRprLpIeNz Qb7j1/CkJrQaQvuFmc61xOAwB4TkD113wHQOQ= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=from:to:cc:subject:date:message-id:x-mailer:in-reply-to:references; b=LFtmtTCWeQJEUiW8vSHwBeDt50eAwJHmWxkLTt3SgdAph9tZ3KZGufvjGZHEhXUCgB Duqa49UbwSej2gwwy/8axGshLqKKXEEeTkefOqI2axgo3Negppmvz2FFFZJdBVDkpewi BmdAA1U1EpMm6Dk0QCyoa7tOiPL02tklS6xLs= Received: by 10.150.212.3 with SMTP id k3mr6217361ybg.2.1292251474995; Mon, 13 Dec 2010 06:44:34 -0800 (PST) Received: from localhost.localdomain ([221.239.34.230]) by mx.google.com with ESMTPS id l4sm2520938ybj.21.2010.12.13.06.44.29 (version=TLSv1/SSLv3 cipher=RC4-MD5); Mon, 13 Dec 2010 06:44:33 -0800 (PST) From: Changli Gao To: Jamal Hadi Salim Cc: "David S. Miller" , netdev@vger.kernel.org, Changli Gao Subject: [PATCH 4/5] ifb: add multiqueue support Date: Mon, 13 Dec 2010 22:43:33 +0800 Message-Id: <1292251414-5154-4-git-send-email-xiaosuo@gmail.com> X-Mailer: git-send-email 1.7.1 In-Reply-To: <1292251414-5154-1-git-send-email-xiaosuo@gmail.com> References: <1292251414-5154-1-git-send-email-xiaosuo@gmail.com> Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org Each ifb NIC has nr_cpu_ids rx queues and nr_cpu_ids queues. Packets transmitted to ifb are enqueued to the corresponding per cpu tx queues, and processed in the corresponding per cpu tasklet latter. The stats are converted to the u64 ones. tq is a stack variable now. It makes ifb_q_private smaller and tx queue locked only once in ri_tasklet. Signed-off-by: Changli Gao --- drivers/net/ifb.c | 211 ++++++++++++++++++++++++++++++++++++------------------ 1 file changed, 141 insertions(+), 70 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/drivers/net/ifb.c b/drivers/net/ifb.c index 57c5cfb..16c767b 100644 --- a/drivers/net/ifb.c +++ b/drivers/net/ifb.c @@ -37,56 +37,63 @@ #include #define TX_Q_LIMIT 32 +struct ifb_q_private { + struct tasklet_struct ifb_tasklet; + struct sk_buff_head rq; + struct u64_stats_sync syncp; + u64 rx_packets; + u64 rx_bytes; + u64 rx_dropped; +}; + struct ifb_private { - struct tasklet_struct ifb_tasklet; - int tasklet_pending; - struct sk_buff_head rq; - struct sk_buff_head tq; + struct ifb_q_private __percpu *q; }; static int numifbs = 2; -static void ri_tasklet(unsigned long dev) +static void ri_tasklet(unsigned long _dev) { - - struct net_device *_dev = (struct net_device *)dev; - struct ifb_private *dp = netdev_priv(_dev); - struct net_device_stats *stats = &_dev->stats; + struct net_device *dev = (struct net_device *)_dev; + struct ifb_private *p = netdev_priv(dev); + struct ifb_q_private *qp; struct netdev_queue *txq; struct sk_buff *skb; - - txq = netdev_get_tx_queue(_dev, 0); - skb = skb_peek(&dp->tq); - if (skb == NULL) { - if (__netif_tx_trylock(txq)) { - skb_queue_splice_tail_init(&dp->rq, &dp->tq); - __netif_tx_unlock(txq); - } else { - /* reschedule */ - goto resched; - } + struct sk_buff_head tq; + + __skb_queue_head_init(&tq); + txq = netdev_get_tx_queue(dev, raw_smp_processor_id()); + qp = per_cpu_ptr(p->q, raw_smp_processor_id()); + if (!__netif_tx_trylock(txq)) { + tasklet_schedule(&qp->ifb_tasklet); + return; } + skb_queue_splice_tail_init(&qp->rq, &tq); + if (netif_tx_queue_stopped(txq)) + netif_tx_wake_queue(txq); + __netif_tx_unlock(txq); - while ((skb = skb_dequeue(&dp->tq)) != NULL) { + while ((skb = __skb_dequeue(&tq)) != NULL) { u32 from = G_TC_FROM(skb->tc_verd); skb->tc_verd = 0; skb->tc_verd = SET_TC_NCLS(skb->tc_verd); - stats->tx_packets++; - stats->tx_bytes += skb->len; + u64_stats_update_begin(&qp->syncp); + txq->tx_packets++; + txq->tx_bytes += skb->len; rcu_read_lock(); skb->dev = dev_get_by_index_rcu(&init_net, skb->skb_iif); if (!skb->dev) { rcu_read_unlock(); + txq->tx_dropped++; + u64_stats_update_end(&qp->syncp); dev_kfree_skb(skb); - stats->tx_dropped++; - if (skb_queue_len(&dp->tq) != 0) - goto resched; - break; + continue; } rcu_read_unlock(); - skb->skb_iif = _dev->ifindex; + u64_stats_update_end(&qp->syncp); + skb->skb_iif = dev->ifindex; if (from & AT_EGRESS) { dev_queue_xmit(skb); @@ -96,48 +103,32 @@ static void ri_tasklet(unsigned long dev) } else BUG(); } - - if (__netif_tx_trylock(txq)) { - skb = skb_peek(&dp->rq); - if (skb == NULL) { - dp->tasklet_pending = 0; - if (netif_queue_stopped(_dev)) - netif_wake_queue(_dev); - } else { - __netif_tx_unlock(txq); - goto resched; - } - __netif_tx_unlock(txq); - } else { -resched: - dp->tasklet_pending = 1; - tasklet_schedule(&dp->ifb_tasklet); - } - } static netdev_tx_t ifb_xmit(struct sk_buff *skb, struct net_device *dev) { - struct ifb_private *dp = netdev_priv(dev); - struct net_device_stats *stats = &dev->stats; + struct ifb_private *p = netdev_priv(dev); + struct ifb_q_private *qp = per_cpu_ptr(p->q, + skb_get_queue_mapping(skb)); u32 from = G_TC_FROM(skb->tc_verd); - stats->rx_packets++; - stats->rx_bytes += skb->len; + u64_stats_update_begin(&qp->syncp); + qp->rx_packets++; + qp->rx_bytes += skb->len; if (!(from & (AT_INGRESS|AT_EGRESS)) || !skb->skb_iif) { + qp->rx_dropped++; + u64_stats_update_end(&qp->syncp); dev_kfree_skb(skb); - stats->rx_dropped++; return NETDEV_TX_OK; } + u64_stats_update_end(&qp->syncp); - __skb_queue_tail(&dp->rq, skb); - if (!dp->tasklet_pending) { - dp->tasklet_pending = 1; - tasklet_schedule(&dp->ifb_tasklet); - } + __skb_queue_tail(&qp->rq, skb); + if (skb_queue_len(&qp->rq) == 1) + tasklet_schedule(&qp->ifb_tasklet); - if (skb_queue_len(&dp->rq) >= dev->tx_queue_len) + if (skb_queue_len(&qp->rq) >= dev->tx_queue_len) netif_stop_queue(dev); return NETDEV_TX_OK; @@ -145,33 +136,103 @@ static netdev_tx_t ifb_xmit(struct sk_buff *skb, struct net_device *dev) static int ifb_close(struct net_device *dev) { - struct ifb_private *dp = netdev_priv(dev); - - tasklet_kill(&dp->ifb_tasklet); - netif_stop_queue(dev); - __skb_queue_purge(&dp->rq); - __skb_queue_purge(&dp->tq); + struct ifb_private *p = netdev_priv(dev); + struct ifb_q_private *qp; + int cpu; + + for_each_possible_cpu(cpu) { + qp = per_cpu_ptr(p->q, cpu); + tasklet_kill(&qp->ifb_tasklet); + netif_tx_stop_queue(netdev_get_tx_queue(dev, cpu)); + __skb_queue_purge(&qp->rq); + } return 0; } static int ifb_open(struct net_device *dev) { - struct ifb_private *dp = netdev_priv(dev); + int cpu; + + for_each_possible_cpu(cpu) + netif_tx_start_queue(netdev_get_tx_queue(dev, cpu)); + + return 0; +} + +static int ifb_init(struct net_device *dev) +{ + struct ifb_private *p = netdev_priv(dev); + struct ifb_q_private *q; + int cpu; - tasklet_init(&dp->ifb_tasklet, ri_tasklet, (unsigned long)dev); - __skb_queue_head_init(&dp->rq); - __skb_queue_head_init(&dp->tq); - netif_start_queue(dev); + p->q = alloc_percpu(struct ifb_q_private); + if (!p->q) + return -ENOMEM; + for_each_possible_cpu(cpu) { + q = per_cpu_ptr(p->q, cpu); + tasklet_init(&q->ifb_tasklet, ri_tasklet, (unsigned long)dev); + __skb_queue_head_init(&q->rq); + } return 0; } +static void ifb_uninit(struct net_device *dev) +{ + struct ifb_private *p = netdev_priv(dev); + + free_percpu(p->q); +} + +static u16 ifb_select_queue(struct net_device *dev, struct sk_buff *skb) +{ + return smp_processor_id(); +} + +static struct rtnl_link_stats64 *ifb_get_stats64(struct net_device *dev, + struct rtnl_link_stats64 *stats) +{ + struct ifb_private *p = netdev_priv(dev); + struct ifb_q_private *q; + struct netdev_queue *txq; + int cpu; + u64 rx_packets, rx_bytes, rx_dropped; + u64 tx_packets, tx_bytes, tx_dropped; + unsigned int start; + + for_each_possible_cpu(cpu) { + q = per_cpu_ptr(p->q, cpu); + txq = netdev_get_tx_queue(dev, cpu); + do { + start = u64_stats_fetch_begin_bh(&q->syncp); + rx_packets = q->rx_packets; + rx_bytes = q->rx_bytes; + rx_dropped = q->rx_dropped; + tx_packets = txq->tx_packets; + tx_bytes = txq->tx_bytes; + tx_dropped = txq->tx_dropped; + } while (u64_stats_fetch_retry_bh(&q->syncp, start)); + stats->rx_packets += rx_packets; + stats->rx_bytes += rx_bytes; + stats->rx_dropped += rx_dropped; + stats->tx_packets += tx_packets; + stats->tx_bytes += tx_bytes; + stats->tx_dropped += tx_dropped; + } + + return stats; +} + static const struct net_device_ops ifb_netdev_ops = { + .ndo_init = ifb_init, + .ndo_uninit = ifb_uninit, .ndo_open = ifb_open, .ndo_stop = ifb_close, .ndo_start_xmit = ifb_xmit, .ndo_validate_addr = eth_validate_addr, + .ndo_select_queue = ifb_select_queue, + .ndo_get_stats64 = ifb_get_stats64, }; static void ifb_setup(struct net_device *dev) @@ -202,11 +263,21 @@ static int ifb_validate(struct nlattr *tb[], struct nlattr *data[]) return 0; } +static int ifb_get_tx_queues(struct net *net, struct nlattr *tb[], + unsigned int *tx_queues, + unsigned int *real_tx_queues) +{ + *real_tx_queues = *tx_queues = nr_cpu_ids; + + return 0; +} + static struct rtnl_link_ops ifb_link_ops __read_mostly = { .kind = "ifb", .priv_size = sizeof(struct ifb_private), .setup = ifb_setup, .validate = ifb_validate, + .get_tx_queues = ifb_get_tx_queues, }; /* Number of ifb devices to be set up by this module. */ @@ -218,8 +289,8 @@ static int __init ifb_init_one(int index) struct net_device *dev_ifb; int err; - dev_ifb = alloc_netdev(sizeof(struct ifb_private), "ifb%d", ifb_setup); - + dev_ifb = alloc_netdev_mq(sizeof(struct ifb_private), "ifb%d", + ifb_setup, nr_cpu_ids); if (!dev_ifb) return -ENOMEM;