[ovs-dev] [PATCH 0/1] Encap & Decap actions for MPLS Packet Type
Martin Varghese
martinvarghesenokia at gmail.com
Thu Oct 10 03:17:34 UTC 2019
On Wed, Oct 09, 2019 at 11:31:44AM -0700, Ben Pfaff wrote:
> On Tue, Oct 08, 2019 at 08:13:16AM +0530, Martin Varghese wrote:
> > The existing PUSH MPLS & POP MPLS actions inserts & removes MPLS header
> > between ethernet header and the IP header. Though this behaviour is fine
> > for L3 VPN where an IP packet is encapsulated inside a MPLS tunnel, it does
> > not suffice the L2 VPN requirements. In L2 VPN the ethernet packets must be
> > encapsulated inside MPLS tunnel.
>
> Thanks a lot for working to make OVS better.
>
> I have a few minor comments on the code. I didn't look at it in detail.
>
> This patch adds a few lines that exceed the normal 79-column limit.
> checkpatch reports those. It's better to wrap lines to fit.
>
> The compiler reports a few warnings that should be fixed:
>
> ../ofproto/ofproto-dpif-ipfix.c:2980:17: error: enumeration values 'OVS_ACTION_ATTR_PTAP_PUSH_MPLS' and 'OVS_ACTION_ATTR_PTAP_POP_MPLS' not explicitly handled in switch [-Werror,-Wswitch-enum]
> ../ofproto/ofproto-dpif-sflow.c:1130:17: error: enumeration values 'OVS_ACTION_ATTR_PTAP_PUSH_MPLS' and 'OVS_ACTION_ATTR_PTAP_POP_MPLS' not explicitly handled in switch [-Werror,-Wswitch-enum]
>
> The biggest issue, however, is a process one. Updates to the kernel
> module need to go upstream to net-next before we put them into the OVS
> tree. Have you submitted them there yet?
>
> It also would be a good idea to implement this in the OVS userspace
> datapath.
>
> I suggest adding an item to NEWS to mention the new feature.
>
Thanks for your time.
I was trying to get the feeback for the approach taken.
I will go ahead and send the kernel changes to net-next
More information about the dev
mailing list