From: Eric Dumazet <edumazet@google.com>
Date: Wed, 30 Aug 2017 17:32:58 +0000 (-0700)
Subject: xen-netfront: be more drop monitor friendly
X-Git-Tag: v4.14-rc1~130^2~69
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=62f3250f3d18ea7d4b4e51ac98cdca29515683a5;p=platform%2Fkernel%2Flinux-rpi.git

xen-netfront: be more drop monitor friendly

xennet_start_xmit() might copy skb with inappropriate layout
into a fresh one.

Old skb is freed, and at this point it is not a drop, but
a consume. New skb will then be either consumed or dropped.

Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
---

diff --git a/drivers/net/xen-netfront.c b/drivers/net/xen-netfront.c
index 7b61adb..523387e 100644
--- a/drivers/net/xen-netfront.c
+++ b/drivers/net/xen-netfront.c
@@ -611,7 +611,7 @@ static int xennet_start_xmit(struct sk_buff *skb, struct net_device *dev)
 		nskb = skb_copy(skb, GFP_ATOMIC);
 		if (!nskb)
 			goto drop;
-		dev_kfree_skb_any(skb);
+		dev_consume_skb_any(skb);
 		skb = nskb;
 		page = virt_to_page(skb->data);
 		offset = offset_in_page(skb->data);