[PATCH net] virtio_net: be drop monitor friend

From: Eric Dumazet <redacted>

This change is needed to not fool drop monitor.
(perf record ... -e skb:kfree_skb )

Packets were properly sent and are consumed after TX completion.

Signed-off-by: Eric Dumazet <redacted>
---
 drivers/net/virtio_net.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c
index 98f17b05c68b..b06169ea60dc 100644
--- a/drivers/net/virtio_net.c
+++ b/drivers/net/virtio_net.c

@@ -1058,7 +1058,7 @@ static void free_old_xmit_skbs(struct send_queue *sq)
 		bytes += skb->len;
 		packets++;
 
-		dev_kfree_skb_any(skb);
+		dev_consume_skb_any(skb);
 	}
 
 	/* Avoid overhead when no packets have been processed

From: Eric Dumazet <redacted>
Date: Thu, 24 Aug 2017 09:02:49 -0700

From: Eric Dumazet <redacted>

This change is needed to not fool drop monitor.
(perf record ... -e skb:kfree_skb )

Packets were properly sent and are consumed after TX completion.

Signed-off-by: Eric Dumazet <redacted>

I'm pretty sure you meant "friendly" in the Subject line so I fixed
it up to say that :-)

Applied, thanks.

On Thu, 2017-08-24 at 11:50 -0700, David Miller wrote:

From: Eric Dumazet <redacted>
Date: Thu, 24 Aug 2017 09:02:49 -0700

quoted

From: Eric Dumazet <redacted>

This change is needed to not fool drop monitor.
(perf record ... -e skb:kfree_skb )

Packets were properly sent and are consumed after TX completion.

Signed-off-by: Eric Dumazet <redacted>

I'm pretty sure you meant "friendly" in the Subject line so I fixed
it up to say that :-)

Applied, thanks.

Ah, great, thanks for fixing this ;)

On Thu, Aug 24, 2017 at 09:02:49AM -0700, Eric Dumazet wrote:

From: Eric Dumazet <redacted>

This change is needed to not fool drop monitor.
(perf record ... -e skb:kfree_skb )

Packets were properly sent and are consumed after TX completion.

Signed-off-by: Eric Dumazet <redacted>

Acked-by: Michael S. Tsirkin <mst@redhat.com>