From e12472dc574ca91be4cb87b14fb8cf90bee02c60 Mon Sep 17 00:00:00 2001 From: Eric Dumazet Date: Mon, 22 Apr 2013 14:31:34 +0000 Subject: [PATCH] net: remove redundant code in dev_hard_start_xmit() This reverts commit 068a2de57ddf4f4 (net: release dst entry while cache-hot for GSO case too) Before GSO packet segmentation, we already take care of skb->dst if it can be released. There is no point adding extra test for every segment in the gso loop. Signed-off-by: Eric Dumazet Cc: Krishna Kumar Signed-off-by: David S. Miller --- net/core/dev.c | 7 ------- 1 file changed, 7 deletions(-) diff --git a/net/core/dev.c b/net/core/dev.c index 9e26b8d9eafe..7c30dcecee1d 100644 --- a/net/core/dev.c +++ b/net/core/dev.c @@ -2546,13 +2546,6 @@ int dev_hard_start_xmit(struct sk_buff *skb, struct net_device *dev, skb->next = nskb->next; nskb->next = NULL; - /* - * If device doesn't need nskb->dst, release it right now while - * its hot in this cpu cache - */ - if (dev->priv_flags & IFF_XMIT_DST_RELEASE) - skb_dst_drop(nskb); - if (!list_empty(&ptype_all)) dev_queue_xmit_nit(nskb, dev);