Hello. On 05/02/2014 11:18 AM, Zhouyi Zhou wrote:
As http://www.spinics.net/lists/netdev/msg165015.html pktgen generates shared packet through vlan interface will cause oops because of duplicate entering tc queue.
Try to solve this problem by means of packet clone instead of sharing.
Signed-off-by: Zhouyi Zhou <yizhouz...@ict.ac.cn> --- net/core/pktgen.c | 20 +++++++++++++++++--- 1 files changed, 17 insertions(+), 3 deletions(-)
diff --git a/net/core/pktgen.c b/net/core/pktgen.c index 0304f98..ced07fc 100644 --- a/net/core/pktgen.c +++ b/net/core/pktgen.c
[...]
@@ -3347,8 +3348,18 @@ static void pktgen_xmit(struct pktgen_dev *pkt_dev) pkt_dev->last_ok = 0; goto unlock; } - atomic_inc(&(pkt_dev->skb->users)); - ret = (*xmit)(pkt_dev->skb, odev); + + if (pkt_dev->clone_skb && is_vlan_dev(odev)) { + nskb = skb_clone(pkt_dev->skb, GFP_ATOMIC); + ret = -ENOMEM; + if (nskb) + ret = (*xmit)(nskb, odev);
You can just do: ret = xmit(nskb, odev);
+ else + nskb = ERR_PTR(ret); + } else { + atomic_inc(&(pkt_dev->skb->users)); + ret = (*xmit)(pkt_dev->skb, odev);
Same here. WBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/