Skip to content

Commit

Permalink
forcedeth: Support for byte queue limits
Browse files Browse the repository at this point in the history
Changes to forcedeth to use byte queue limits.

Signed-off-by: Tom Herbert <therbert@google.com>
Acked-by: Eric Dumazet <eric.dumazet@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Tom Herbert authored and David S. Miller committed Nov 29, 2011
1 parent 3f0cfa3 commit b8bfca9
Showing 1 changed file with 18 additions and 0 deletions.
18 changes: 18 additions & 0 deletions drivers/net/ethernet/nvidia/forcedeth.c
Original file line number Diff line number Diff line change
Expand Up @@ -1928,6 +1928,7 @@ static void nv_init_tx(struct net_device *dev)
np->last_tx.ex = &np->tx_ring.ex[np->tx_ring_size-1];
np->get_tx_ctx = np->put_tx_ctx = np->first_tx_ctx = np->tx_skb;
np->last_tx_ctx = &np->tx_skb[np->tx_ring_size-1];
netdev_reset_queue(np->dev);
np->tx_pkts_in_progress = 0;
np->tx_change_owner = NULL;
np->tx_end_flip = NULL;
Expand Down Expand Up @@ -2276,6 +2277,9 @@ static netdev_tx_t nv_start_xmit(struct sk_buff *skb, struct net_device *dev)

/* set tx flags */
start_tx->flaglen |= cpu_to_le32(tx_flags | tx_flags_extra);

netdev_sent_queue(np->dev, skb->len);

np->put_tx.orig = put_tx;

spin_unlock_irqrestore(&np->lock, flags);
Expand Down Expand Up @@ -2420,6 +2424,9 @@ static netdev_tx_t nv_start_xmit_optimized(struct sk_buff *skb,

/* set tx flags */
start_tx->flaglen |= cpu_to_le32(tx_flags | tx_flags_extra);

netdev_sent_queue(np->dev, skb->len);

np->put_tx.ex = put_tx;

spin_unlock_irqrestore(&np->lock, flags);
Expand Down Expand Up @@ -2457,6 +2464,7 @@ static int nv_tx_done(struct net_device *dev, int limit)
u32 flags;
int tx_work = 0;
struct ring_desc *orig_get_tx = np->get_tx.orig;
unsigned int bytes_compl = 0;

while ((np->get_tx.orig != np->put_tx.orig) &&
!((flags = le32_to_cpu(np->get_tx.orig->flaglen)) & NV_TX_VALID) &&
Expand All @@ -2476,6 +2484,7 @@ static int nv_tx_done(struct net_device *dev, int limit)
np->stat_tx_bytes += np->get_tx_ctx->skb->len;
u64_stats_update_end(&np->swstats_tx_syncp);
}
bytes_compl += np->get_tx_ctx->skb->len;
dev_kfree_skb_any(np->get_tx_ctx->skb);
np->get_tx_ctx->skb = NULL;
tx_work++;
Expand All @@ -2492,6 +2501,7 @@ static int nv_tx_done(struct net_device *dev, int limit)
np->stat_tx_bytes += np->get_tx_ctx->skb->len;
u64_stats_update_end(&np->swstats_tx_syncp);
}
bytes_compl += np->get_tx_ctx->skb->len;
dev_kfree_skb_any(np->get_tx_ctx->skb);
np->get_tx_ctx->skb = NULL;
tx_work++;
Expand All @@ -2502,6 +2512,9 @@ static int nv_tx_done(struct net_device *dev, int limit)
if (unlikely(np->get_tx_ctx++ == np->last_tx_ctx))
np->get_tx_ctx = np->first_tx_ctx;
}

netdev_completed_queue(np->dev, tx_work, bytes_compl);

if (unlikely((np->tx_stop == 1) && (np->get_tx.orig != orig_get_tx))) {
np->tx_stop = 0;
netif_wake_queue(dev);
Expand All @@ -2515,6 +2528,7 @@ static int nv_tx_done_optimized(struct net_device *dev, int limit)
u32 flags;
int tx_work = 0;
struct ring_desc_ex *orig_get_tx = np->get_tx.ex;
unsigned long bytes_cleaned = 0;

while ((np->get_tx.ex != np->put_tx.ex) &&
!((flags = le32_to_cpu(np->get_tx.ex->flaglen)) & NV_TX2_VALID) &&
Expand All @@ -2538,13 +2552,17 @@ static int nv_tx_done_optimized(struct net_device *dev, int limit)
u64_stats_update_end(&np->swstats_tx_syncp);
}

bytes_cleaned += np->get_tx_ctx->skb->len;
dev_kfree_skb_any(np->get_tx_ctx->skb);
np->get_tx_ctx->skb = NULL;
tx_work++;

if (np->tx_limit)
nv_tx_flip_ownership(dev);
}

netdev_completed_queue(np->dev, tx_work, bytes_cleaned);

if (unlikely(np->get_tx.ex++ == np->last_tx.ex))
np->get_tx.ex = np->first_tx.ex;
if (unlikely(np->get_tx_ctx++ == np->last_tx_ctx))
Expand Down

0 comments on commit b8bfca9

Please sign in to comment.