[PATCH] Exclude duplicated checking for iface-up. This flags is checked in 'is_skb_forwardable' function, which is subroutine of 'dev_forward_skb'.

From: Alexander Smirnov <redacted>

Signed-off-by: Alexander Smirnov <redacted>
---
 drivers/net/veth.c |    3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)

diff --git a/drivers/net/veth.c b/drivers/net/veth.c
index 3b99f64..fb2fd37 100644
--- a/drivers/net/veth.c
+++ b/drivers/net/veth.c

@@ -163,9 +163,6 @@ static netdev_tx_t veth_xmit(struct sk_buff *skb, struct net_device *dev)
 	stats = this_cpu_ptr(priv->stats);
 	rcv_stats = this_cpu_ptr(rcv_priv->stats);
 
-	if (!(rcv->flags & IFF_UP))
-		goto tx_drop;
-
 	/* don't change ip_summed == CHECKSUM_PARTIAL, as that
 	   will cause bad checksum on forwarded packets */
 	if (skb->ip_summed == CHECKSUM_NONE)

-- 
1.7.2.3

On Tue, 28 Jun 2011 17:30:44 +0400, alex.bluesman.smirnov wrote:

From: Alexander Smirnov <redacted>

Please add the description here, instead of in $subject.

Signed-off-by: Alexander Smirnov <redacted>
---

The patch itself looks fine,

Reviewed-by: WANG Cong <redacted>

Thanks.

From: alex.bluesman.smirnov@gmail.com
Date: Tue, 28 Jun 2011 17:30:44 +0400

From: Alexander Smirnov <redacted>

Signed-off-by: Alexander Smirnov <redacted>

Applied.

Please put the verbose patch description in the body of your
email message, right before the signoffs, not in the subject.

The subject provides a "summary" not the entire description.