diff options
| author | Jesse Gross <jesse@nicira.com> | 2015-06-02 15:11:00 -0700 |
|---|---|---|
| committer | Jesse Gross <jesse@nicira.com> | 2015-06-25 11:08:58 -0700 |
| commit | 6159c531d09a19f5eb6865fc5221c04156aa0bf4 (patch) | |
| tree | 406926cec3ff3b727a4924e768053047607692fa /ovn | |
| parent | 00fe22f85baf76f5a4dca0312d4a7d7c920065af (diff) | |
| download | openvswitch-6159c531d09a19f5eb6865fc5221c04156aa0bf4.tar.gz | |
openflow: Table maintenance commands for Geneve options.
In order to work with Geneve options, we need to maintain a mapping
table between an option (defined by <class, type, length>) and
an NXM field that can be operated on for the purposes of matches,
actions, etc. This mapping must be explicitly specified by the
user.
Conceptually, this table could be communicated using either OpenFlow
or OVSDB. Using OVSDB requires less code and definition of extensions
than OpenFlow but introduces the possibility that mapping table
updates and flow modifications are desynchronized from each other.
This is dangerous because the mapping table signifcantly impacts the
way that flows using Geneve options are installed and processed by
OVS. Therefore, the mapping table is maintained using OpenFlow commands
instead, which opens the possibility of using synchronization between
table changes and flow modifications through barriers, bundles, etc.
There are two primary groups of OpenFlow messages that are introduced
as Nicira extensions: modification commands (add, delete, clear mappings)
and table status request/reply to dump the current table along with switch
information.
Note that mappings should not be changed while they are in active use by
a flow. The result of doing so is undefined.
This only adds the OpenFlow infrastructure but doesn't actually
do anything with the information yet after the messages have been
decoded.
Signed-off-by: Jesse Gross <jesse@nicira.com>
Acked-by: Ben Pfaff <blp@nicira.com>
Diffstat (limited to 'ovn')
| -rw-r--r-- | ovn/controller/ofctrl.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/ovn/controller/ofctrl.c b/ovn/controller/ofctrl.c index 2f1767a1e..84050013e 100644 --- a/ovn/controller/ofctrl.c +++ b/ovn/controller/ofctrl.c @@ -227,6 +227,9 @@ ofctrl_recv(const struct ofpbuf *msg) case OFPTYPE_TABLE_FEATURES_STATS_REPLY: case OFPTYPE_BUNDLE_CONTROL: case OFPTYPE_BUNDLE_ADD_MESSAGE: + case OFPTYPE_NXT_GENEVE_TABLE_MOD: + case OFPTYPE_NXT_GENEVE_TABLE_REQUEST: + case OFPTYPE_NXT_GENEVE_TABLE_REPLY: default: /* Messages that are generally unexpected. */ if (VLOG_IS_DBG_ENABLED()) { |