Skip to contentSkip to conversation

net: convert one non-symbolic return value in dev_queue_xmit

PATCH Applied Krishna Kumar 2 participants 2 messages Aug 30, 2009 – Aug 31, 2009

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 net: convert one non-symbolic return value in dev_queue_xmit

From: Krishna Kumar <[email protected]>

net: convert remaining non-symbolic return values in dev_queue_xmit

Patch compiled and 32 simultaneous netperf testing ran fine.

---

 net/core/dev.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
net/core/dev.c +1 -1
diff -ruNp org/net/core/dev.c new/net/core/dev.c
@@ -1917,7 +1917,7 @@ gso:
 			HARD_TX_LOCK(dev, txq, cpu);
 
 			if (!netif_tx_queue_stopped(txq)) {
-				rc = 0;
+				rc = NET_XMIT_SUCCESS;
 				if (!dev_hard_start_xmit(skb, dev, txq)) {
 					HARD_TX_UNLOCK(dev, txq);
 					goto out;
maintainer Applied
#1 lore raw

From: Krishna Kumar <[email protected]>
Date: Sun, 30 Aug 2009 11:51:36 +0530

Quoted 7 lines

From: Krishna Kumar <[email protected]>

net: convert remaining non-symbolic return values in dev_queue_xmit

Patch compiled and 32 simultaneous netperf testing ran fine.

Applied, but please do me a favor.

Do not provide two commit message header lines like this. They aren't even equal.

Whatever you give me in the Subject line after "[....] " is what ends up being the commit message header line when I apply your patch.

Adding another commit header line in your email message body just makes the commit look confusing and ugly.

So just give me one in the subject line and elide the one in the message body.

Thanks.