102 Commits
Author SHA1 Message Date
Brandon Heller 0f73f42fa6 Spec: Define failover behavior
Add notes about emergency vs normal flows, and define an expectation
for the switch to support multiple controller strings.
2009-06-30 18:02:54 -07:00
Brandon Heller 25d7c2ecd2 Spec: Fix typo in diagram
In the flow table matching picture, Ethernet type was compared to 0x8000
when 0x0800 was intended.
2009-06-29 22:42:40 -07:00
Brandon Heller 1e12e0771e Spec: Fix build error
Removed extra line
2009-06-29 22:31:08 -07:00
Brandon Heller 98ee054722 Spec: Minor change to CHECK_OVERLAP text 2009-06-29 11:37:28 -07:00
Brandon Heller 1a5762fbd2 Spec: Fix ambiguity regarding the duration field in Flow Expirations
Before this spec change, flow expiration messages included a duration
field for the time the flow was active.  To find out the amount of time
the flow received traffic would have been much harder - the controller
would need to store a record of all flow mods it has sent, to know the
idle_timeout of this flow.  Additionally, both phrases, 'time flow was
active' and 'time traffic was received' were used in the spec for the
duration field.

Now we return the idle timeout in the flow expiration message, and ensure
duration is the time the entry was present in the flow table.  The hard
timeout will be obvious if it is the reason for the deletion, since the
reason field will be OFPER_HARD_TIMEOUT, and the duration will be the hard
timeout value.  From the duration and the idle_timeout, we can also
compute the time the flow received traffic.
2009-06-29 11:37:28 -07:00
Brandon Heller 4f0da65910 Spec: Define behavior for conflicting flow mods
Define a CHECK_OVERLAP flag for flow mods which requires the switch to
check for conflicting entries with the same priority, i.e., those for
which at least one packet could match multiple entries.

This flags is intended to be used for debugging purposes; in the worst
case, it may require a linear check through the flow table.
2009-06-29 11:37:28 -07:00
Brandon Heller 91617a4690 Spec: Add VLAN priority bits
The OpenFlow 10-tuple is now officially an 11-tuple, with the addition of
the VLAN priority (PCP) field.
2009-06-29 11:37:28 -07:00
Brandon Heller 90bdc8626b Spec: Add barrier commands
Add barrier request message, which ensures that all previously received
commands are complete before executing ones past the barrier message.  A
barrier reply must be sent for each barrier request.

We expect this command to be useful for testing and wireless handover
purposes.
2009-06-29 11:37:28 -07:00
Brandon Heller f874420531 Spec: Start port enumeration at 1
A number of protocols such as SNMP and STP start counting ports at one. To
increase compatibility, this commit adds a requirement that OpenFlow
ingress port values start at one.
2009-06-29 11:37:28 -07:00
Brandon Heller e9353ae079 Spec: Add _EPERM error codes
An OpenFlow hypervisor might choose to reject an OpenFlow request, but
currently has no message type defined for this error. It is not a
vendor-specific error, thus we define an _EPERM error code for each error
type.

Also remove unused, commented-out text.
2009-06-29 11:37:14 -07:00
Brandon Heller c249b48f83 Spec: Replace hardcoded constant for MISS_SEND_LEN 2009-06-29 11:36:33 -07:00
Brandon Heller 31955e3ea4 Spec: Clarify buffer behavior 2009-06-29 11:36:33 -07:00
Brandon Heller cec522cd66 Spec: Clarify format for Strip VLAN action
Clarify that the OFPAT_STRIP_VLAN action takes no argument and strips the
VLAN tag if one is present. The header is the generic action header.
2009-06-29 11:36:33 -07:00
Brandon Heller 16e2b8ba6d Spec: Clarify transaction ID for error messages
Note that if an error is in response to a specific message from the
controller, the transaction ID should match that from the original
request.
2009-06-29 11:36:32 -07:00
Brandon Heller 4b969c68f3 Spec: Clarify matching for disabled STP ports
Packets received on ports that are disabled by spanning tree must follow
the normal flow table processing path.
2009-06-29 11:36:32 -07:00
Brandon Heller 3a7d98b612 Spec: Clarify matching behavior for flow mods and stats
The spec does not currently define how wildcard matching should behave for
flow modification (specifically DELETE and MODIFY commands) and stats.
This commit adds text to define behavior when some fields are wildcarded
and others are left exact.
2009-06-29 11:36:32 -07:00
Brandon Heller 1592042b19 Spec: Minor rearrangement of flow add text 2009-06-29 11:36:32 -07:00
Brandon Heller 0c370a5ef9 Spec: Remove distinction between Type 0 and Type 1
People tend to get confused about "Type 0" and "Type 1" vs "Version 1.0" and
"Version 2.0".  To simplify, we'll drop the reference to types.
2009-06-29 11:36:32 -07:00
Brandon Heller b5525791b5 Spec: Notify controller of flows that are forcefully deleted
Previously, a switch would only notify the controller when flows expired.
Now, when a switch is told to delete flows, it sends a flow expiration
message, enabling the controller to get packet and byte counts from
deleted flows. Without this, the controller must retrieve flow statistics
and then delete the flow, during which time the counts could change. This
could also aid in debugging if multiple applications or programs
controlling the same switch are deleting each others' flows.

This commit adds a OFPER_DELETE as a choice for the reason field of a flow
expiration.
2009-06-29 07:07:29 -07:00
Brandon Heller a00b917457 Spec: Selective flow expirations
Make flow expirations per-flow-entry, rather than per-switch.

This commit should give controllers extra flexibility to decide when they
want expirations, will simplify the job of an OpenFlow hypervisor, and may
reduce CPU load at the switch.
2009-06-29 07:02:44 -07:00
Glen Gibb bdbd57b546 Add missing codes in Error Message section
Error Messages section was missing the codes for:
 - OFPET_BAD_ACTION
 - OFPET_FLOW_MOD_FAILED
2008-12-09 13:37:45 -08:00
Glen Gibb a3f6caf703 Clarified Error Message section
Clarified the wording in the Error Message section slightly:
 - currently defined error types
 - data field for OFPET_HELLO_FAILED
 - codes for OFPET_BAD_REQUEST
2008-12-09 13:37:45 -08:00
Brandon Heller c5242793ba Spec: update for delete+out_port changes, plus remove action interleaving restrictions text (not implemented yet). 2008-12-02 02:13:08 -08:00
Brandon Heller 7da34aa10c reformat spec openflow.h parser 2008-12-01 04:14:54 -08:00
Brandon Heller 2eab287131 Spec: define behavior for fragment packets 2008-12-01 01:21:53 -08:00
Brandon Heller 36ec367d2c fix paths referencing include/openflow.h, which was moved to include/openflow/openflow.h 2008-11-25 07:51:51 -08:00
Brandon Heller f72505b063 Spec: added ICMP type and code matching 2008-11-23 23:36:39 -08:00
Brandon Heller 26b907fb3a Spec: minor changes, plus version number set to 0.8.9 2008-11-22 19:14:14 -08:00
Brandon Heller 4b44cde095 Spec: redo flow mod section 2008-11-21 16:58:15 -08:00
Brandon Heller 99aa32b737 Spec: removed version number dependence from makefile (it's printed on the first page) and added each action struct 2008-11-21 16:58:15 -08:00
Brandon Heller f1918e584b Spec: minor formatting changes 2008-11-21 16:57:34 -08:00
Brandon Heller c04a3c082c Spec: renamed 2008-11-21 16:57:34 -08:00
Brandon Heller f0137e8bfa Spec: added IP netmask text from wiki 2008-11-21 16:57:33 -08:00
Brandon Heller 1841cd3424 Spec: added details on STP from wiki 2008-11-21 16:57:33 -08:00
Brandon Heller ed9b91aa07 Spec: added vendor stats type and wiki info on Vendor message type 2008-11-21 16:57:33 -08:00
Brandon Heller 7e22df2cfb Spec: added line about IP fragment handling 2008-11-21 16:57:33 -08:00
Brandon Heller 0ba5844d12 Spec: added Hello message and handshake details 2008-11-21 16:57:33 -08:00
Brandon Heller cd8f6a194c Spec: updated VLAN actions 2008-11-21 16:57:33 -08:00
Brandon Heller 194338150f Spec: added and refined text, some of which comes from the wiki entry on v0.9 2008-11-21 16:57:33 -08:00
Brandon Heller 4c84900602 Spec: minor typo 2008-11-21 16:57:33 -08:00
Brandon Heller db10aa66c6 Spec: add Vendor message type to protocol overview 2008-11-21 16:57:33 -08:00
Brandon Heller 1828576ad6 Spec: updated symmetric messages section, added Vendor message type 2008-11-21 16:57:33 -08:00
Brandon Heller d356960ad3 Spec: changed error text 2008-11-21 16:57:33 -08:00
Brandon Heller 21410057cc Spec: update flow expiration text and add reason enum 2008-11-21 16:57:33 -08:00
Brandon Heller 09d526fa11 Spec: update text for stats replies 2008-11-21 16:57:33 -08:00
Brandon Heller b5261c1421 added ofp_description statistics 2008-11-21 16:57:32 -08:00
Brandon Heller 0b41c018d1 Spec:
Removed text on error reporting for some flow mod entry failures
Reordered ofp_flow_mod_command text
2008-11-21 16:57:32 -08:00
Brandon Heller e144932927 spec: removed text about pinning exact entries 2008-11-21 16:57:32 -08:00
Brandon Heller 7b2508360a Spec Changes:
Updated port fields
Added subsubsection tags for each kind of reused structure
Hello process, with table-specific info, now explained
Now listing ofp_config_flags
2008-11-21 16:57:32 -08:00
Brandon Heller f8471e18fa Spec changes:
Removed description column in Table 4, Counters, since every counter but duration seemed self-explanatory.
Reordered first column in Table 4 to match 0x93 openflow.h.
Cleaned up the actions section - it was too verbose.
Clarification for switches to indicate NORMAL forward support.
Reorganized into Data Path and Control Path sections - this seems like a more natural break in the spec.
2008-11-21 16:57:32 -08:00