From 47d62dcd4a499941235550447ec15faed5e33abb Mon Sep 17 00:00:00 2001 From: Divakar Date: Fri, 13 May 2016 20:02:58 +0530 Subject: [PATCH] Disabling GRO when mirroring is enabled When the packets are received on Fabric interface, these are subjected to GRO before transmitting them to Tap interface. If Tx port mirroring is enabled on Tap interface, as of now, mirroring is applied after GRO of the packets. If the mirroring server is on another compute node, these mirrored packets have GSO enabled. Vrouter makes use of Linux GSO routines and these GSO routines expect that skb in skb_list of head_skb does not contain any linear data (skb_headlen should be zero). Due to GRO the skb_headle of some skb's in skb_list contains linear data resulting in GSO routiness hitting a BUG_ON. To over come this, the GRO needs to be applied post mirroring. To enabled GSO on the mirrored packets the skb's gso_len also need to be supplied. Instead of this fix, GRO is disabled if Tx port mirroring is enabled on Tap interface as stop gap fix till the complete fix is in place. Change-Id: Iaeeeb4332f2166c2bf23a8770c15e736e47120bb closes-bug: #1577473 --- dp-core/vr_nexthop.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/dp-core/vr_nexthop.c b/dp-core/vr_nexthop.c index f76c85233..dea8f13fc 100644 --- a/dp-core/vr_nexthop.c +++ b/dp-core/vr_nexthop.c @@ -2024,7 +2024,8 @@ nh_encap_l2(struct vr_packet *pkt, struct vr_nexthop *nh, } if (pkt->vp_flags & VP_FLAG_GRO) { - if (vif_is_virtual(vif)) { + if (vif_is_virtual(vif) && (!(vif->vif_flags & + VIF_FLAG_MIRROR_TX))) { if (vr_gro_input(pkt, nh)) { if (stats) stats->vrf_gros++; @@ -2109,7 +2110,8 @@ nh_encap_l3(struct vr_packet *pkt, struct vr_nexthop *nh, } } - if ((pkt->vp_flags & VP_FLAG_GRO) && vif_is_virtual(vif)) { + if ((pkt->vp_flags & VP_FLAG_GRO) && vif_is_virtual(vif) && + (!(vif->vif_flags & VIF_FLAG_MIRROR_TX))) { if (vr_gro_input(pkt, nh)) return 0; }