Analyzer shows packets from BMS as having bogus IP header length

[Vedamurthy Joshi]

2.1 Build 26 Ubuntu 14.04 multi-node setup

From a BMS (10.1.1.5) , i am pinging a KVM VM (10.1.1.3 ) on vn VN1
There is an analyzer to sniff all VN1 traffic

The packet capture on the analyzer shows (for all packets from 10.1.1.5 to 10.1.1.3) that it has a bad IP header length

Attached is the capture

[Vedamurthy Joshi (vedujoshi)]

Source and dest macs are also not set correctly

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/9290
Submitter: Divakar Dharanalakota (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/9290
Committed: http://github.org/Juniper/contrail-vrouter/commit/ecd253a6fa942120d01fdf622197985196c74991
Submitter: Zuul
Branch: master

commit ecd253a6fa942120d01fdf622197985196c74991
Author: Divakar <email address hidden>
Date: Mon Apr 20 03:57:27 2015 -0700

closes-bug:#1420452 While mirroring the tunneled packet, tunnel headers
are removed and ethernet header is added from Encap nh to ensure that
Mirroring application sees a regular ethernet packet. After IRB, the
packet can be L2 packet and might contain ethernet header in the Tunnel
payload. In this case, the ethernet header need not be added from Encap
NH. This functionality is added by verifying the type of NH.

Change-Id: I0797c9db5e25ac09d51ddd3cdb9b25c632b812b9

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/9333
Submitter: Divakar Dharanalakota (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/9334
Submitter: Divakar Dharanalakota (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/9334
Committed: http://github.org/Juniper/contrail-vrouter/commit/294d9ccf8bd9cc7f9d35a03ac9f689c8bf02f291
Submitter: Zuul
Branch: R2.20

commit 294d9ccf8bd9cc7f9d35a03ac9f689c8bf02f291
Author: Divakar <email address hidden>
Date: Mon Apr 20 03:57:27 2015 -0700

closes-bug:#1420452 While mirroring the tunneled packet, tunnel headers
are removed and ethernet header is added from Encap nh to ensure that
Mirroring application sees a regular ethernet packet. After IRB, the
packet can be L2 packet and might contain ethernet header in the Tunnel
payload. In this case, the ethernet header need not be added from Encap
NH. This functionality is added by verifying the type of NH.

Change-Id: I0797c9db5e25ac09d51ddd3cdb9b25c632b812b9

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/9333
Committed: http://github.org/Juniper/contrail-vrouter/commit/e5b83ba441a43f3673af37cb2d0c07fdf8f2656b
Submitter: Zuul
Branch: R2.1

commit e5b83ba441a43f3673af37cb2d0c07fdf8f2656b
Author: Divakar <email address hidden>
Date: Mon Apr 20 03:57:27 2015 -0700

closes-bug:#1420452 While mirroring the tunneled packet, tunnel headers
are removed and ethernet header is added from Encap nh to ensure that
Mirroring application sees a regular ethernet packet. After IRB, the
packet can be L2 packet and might contain ethernet header in the Tunnel
payload. In this case, the ethernet header need not be added from Encap
NH. This functionality is added by verifying the type of NH.

Conflicts:

 dp-core/vr_mirror.c

Change-Id: I0797c9db5e25ac09d51ddd3cdb9b25c632b812b9