Hi Ben, This series provides some proposed improvements to your patchset which includes "Implement OpenFlow support for MPLS, for up to 3 labels."
This series is based on the blp/mpls branch of your tree on github. The head commit I used is: 2a39c414a883dffd8 ("Add OpenFlow support for matching multiple MPLS labels at once."). The first 3 patches of this series correct some minor problems that I came across. The remaining 5 patches are my first attempt at handling the situation where the datapath and ovs-vswtichd support different maximum MPLS label stack depths. Simon Horman (8): [1/3] odp-util: Include '/' before mask when formatting MPLS LSE [2/3] ofproto-dpif: Update tests for masked BoS bit [3/3] ofproto-dpif-xlate: Correct out-by one in logging of MPLS action errors [4/3] ofproto-dpif: Probe datapath for MPLS stack depth supported [5/3] ofproto-dpif-xlate: Add probed MPLS stack depth of dpif to xbridge [6/3] ofproto-dpif-xlate: Do not add more MPLS LSEs than the datapath can handle [7/3] ofproto-dpif-xlate: Do not use of more MPLS LSEs than the datapath can supply [8/3] ofproto-dpif: Do not add more MPLS labels to mask than the datapath can handle lib/dpif-netdev.c | 3 +- lib/odp-util.c | 15 ++++++---- lib/odp-util.h | 3 +- ofproto/ofproto-dpif-upcall.c | 5 +++- ofproto/ofproto-dpif-xlate.c | 59 +++++++++++++++++++++++-------------- ofproto/ofproto-dpif-xlate.h | 3 +- ofproto/ofproto-dpif.c | 68 ++++++++++++++++++++++++++++++++++++++++++- ofproto/ofproto-dpif.h | 2 ++ tests/ofproto-dpif.at | 4 +-- 9 files changed, 127 insertions(+), 35 deletions(-) -- 1.8.4 _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev