[ovs-dev] [PATCH V10 0/5] Add 802.1ad (qinq) support

Thomas F Herbert thomasfherbert at gmail.com
Tue Jun 2 17:55:27 UTC 2015


From: "Thomas F. Herbert" <thomasfherbert at gmail.com>

Version 10: Use doubly nested encap attributes to encode 802.1ad. Rebase
to master. V8 and V9 were skipped and this version was called V10 to
accompany V10 kernel module patch simultaneously to net-next.

This patch will is incopatible with old versions of the kernel module and
will require V10 of the kernel module patch because of the change in
netlink attribute encoding of 802.1ad. Netlink attributes now look like
this:

    eth_type(0x88a8),vlan(vid=100),encap(eth_type(0x8100), vlan(vid=200),
            encap(eth_type(0x0800), ...))

Version 7 is rebased to latest master and incorporates changes from
reviewers. The accompanying kernel changes have been submitted to the
linux net-dev mailing list.

Version 6 split the changes into separate patches to make it easier on the
eyes of reviewers but otherwise is functionally the same as version 5.

Version 5 updated the patch to properly use nested attributes, fixed some
whitespace problems and allows TPID 0x88a8 tag be pushed with either
single or outer tag because Open Flow does not specifically prohibit it.

This patch has been tested along with the Linux Kernel datapath patch in
a VM based test network as well as real-world provider test environment.

Thanks to Robert Peterson for inspiring and leading the effort of
separating customer from provider traffic which led to this work and
thanks to the people at ATC for helping with configuring the test lab
and running perf tests.

Also, thanks to Entry Point LLC who were involved in this project from the
beginning.

Thanks to Ben Pfaff, Pravin Shelar and others that have commented on
previous versions of this patch and thanks to Dave Benson who
contributed the test included in this patch.

Thomas F. Herbert (5):
  Support functions for 8021.ad push and pop vlan
  Flow and action changes for 8021.AD
  Flow and netlink parsing changes for 8021.AD
  Describe 802.1ad changes
  Test push and pop vlan with 802.1AD

 NEWS                         |   2 +
 lib/flow.c                   |  22 ++--
 lib/flow.h                   |  17 ++-
 lib/match.c                  |   2 +-
 lib/nx-match.c               |   2 +-
 lib/odp-execute.c            |   2 +-
 lib/odp-util.c               | 291 ++++++++++++++++++++++++++++++++++++-------
 lib/odp-util.h               |   2 +-
 lib/ofp-actions.c            |  32 +++--
 lib/ofp-actions.h            |  14 ++-
 lib/ofp-util.c               |   2 +-
 lib/packets.c                |   8 +-
 lib/packets.h                |   7 ++
 ofproto/ofproto-dpif-rid.h   |   2 +-
 ofproto/ofproto-dpif-xlate.c |  13 +-
 tests/ofproto-dpif.at        |  40 ++++++
 utilities/ovs-ofctl.8.in     |   3 +-
 17 files changed, 373 insertions(+), 88 deletions(-)

-- 
2.1.0




More information about the dev mailing list