summaryrefslogtreecommitdiff
path: root/utilities/ovs-ofctl.8.in
Commit message (Collapse)AuthorAgeFilesLines
...
* Add OF11 SET IP TTL action.Jarno Rajahalme2013-10-231-0/+9
| | | | | Signed-off-by: Jarno Rajahalme <jrajahalme@nicira.com> Signed-off-by: Ben Pfaff <blp@nicira.com>
* Add OF11 set IPv4 ECN action.Jarno Rajahalme2013-10-231-0/+9
| | | | | Signed-off-by: Jarno Rajahalme <jrajahalme@nicira.com> Signed-off-by: Ben Pfaff <blp@nicira.com>
* Allow OFPACT_SET_IP_DSCP to act on both IPv4 and IPv6 packets.Ben Pfaff2013-10-231-3/+3
| | | | | | | | | This removes semantic differences between different OpenFlow versions, making it easier to translate between them. Also, rename OFPACT_SET_IPV4_DSCP to OFPACT_SET_IP_DSCP. Signed-off-by: Jarno Rajahalme <jrajahalme@nicira.com> Signed-off-by: Ben Pfaff <blp@nicira.com>
* Fix ovs-ofctl documentation.Jarno Rajahalme2013-10-231-4/+5
| | | | | | | | State that if decrementing makes a TTL zero, controller notifications may be sent. Signed-off-by: Jarno Rajahalme <jrajahalme@nicira.com> Signed-off-by: Ben Pfaff <blp@nicira.com>
* Add support for write-actionsSimon Horman2013-10-141-4/+87
| | | | | | | | | | | | | | | | | | | | | | | | | Implementation note: All actions which modify a field are added to the action set at the point where "set" actions should be added. In general modifying a field many times is the same as only modifying it the last time so the implementation simply adds all set actions to the action set in the order they are specified. However, this breaks down if two actions modify different portions of the same field. Some examples. 1. load acting a subfield 2. mod_vlan_vid, mod_vlan_pcp If this is considered to be a problem one possible solution would be to either disallow all set actions other than set_field in write_actions. Another possible solution is prohibit problematic the actions listed above in write actions. Signed-off-by: Simon Horman <horms@verge.net.au> [blp@nicira.com simplified and edited the code] Signed-off-by: Ben Pfaff <blp@nicira.com>
* Remove mpls_depth field from flowSimon Horman2013-09-271-10/+6
| | | | | | | | | | | | | | | | | | | | | Rather than tracking the MPLS depth as a field in the flow, which is an entirely poor place for it, just track the delta to the MPLS depth during translation. This logic was developed while implementing recirculation and intended to be used to detect when recirculation should occur. This variant of the patch uses the logic to determine if processing of actions should stop due to an MPLS action which cannot be translated (without recirculation). A side-effect of this patch is that it resolves a bug whereby ovs-vswitchd will abort due to to an assertion on eth_type_mpls(ctx->xin->flow.dl_type) in compose_mpls_pop_action(() if the actions of a flow include pop_mpls twice without a push_mpls in between. Signed-off-by: Simon Horman <horms@verge.net.au> Signed-off-by: Ben Pfaff <blp@nicira.com>
* ovs-ofctl: Fix a typo in documentation.Gurucharan Shetty2013-09-261-1/+1
| | | | | Signed-off-by: Gurucharan Shetty <gshetty@nicira.com> Acked-by: Ben Pfaff <blp@nicira.com>
* ovs-ofctl: Add meter support.Jarno Rajahalme2013-09-161-1/+99
| | | | | | | | | | | | | | | | | | | | | | | | | | Adds commands add-meter, mod-meter, del-meter, del-meters, dump-meter, dump-meters, meter-stats, and meter-features. Syntax is as follows: add-meter <br> meter=<n> (kbps|pktps) [burst] [stats] bands= type=(drop|dscp_remark) rate=<n> [burst_size=<n>] [prec_level=<n>] <more bands> mod-meter <br> meter=<n> (kbps|pktps) [burst] [stats] bands= type=(drop|dscp_remark) rate=<n> [burst_size=<n>] [prec_level=<n>] <more bands> del-meter <br> meter=(<n>|all) del-meters <br> dump-meter <br> meter=(<n>|all) dump-meters <br> meter-stats <br> [meter=(<n>|all)] meter-features <br> Signed-off-by: Jarno Rajahalme <jrajahalme@nicira.com> Signed-off-by: Ben Pfaff <blp@nicira.com>
* openflow-1.1+: OFPT_TABLE_MOD (part 1)Andy Zhou2013-09-071-0/+25
| | | | | | | | | | | | | Added infrastructure to support Openflow OFPT_TABLE_MOD message. This patch does not include the flexible table miss handling code that is necessary to support the semantics specified in OFPT_TABLE_MOD messages. Current flow miss behavior continues to conform to Openflow 1.0. Future commits to add more flexible table miss support are needed to fully support OPFT_TABLE_MOD for Openflow-1.1+. Signed-off-by: Andy Zhou <azhou@nicira.com> Signed-off-by: Ben Pfaff <blp@nicira.com>
* ovs-ofctl: Correct formatting of instructions in manpage.Simon Horman2013-09-041-1/+1
| | | | | | | | | Adjust formatting in ovs-ofctl manpage so that apply_actions, clear_actions write_metadata and goto_table appear at the same level of indentation as actions rather being indented as if they are arguments to the learn action. Signed-off-by: Simon Horman <horms@verge.net.au> Signed-off-by: Ben Pfaff <blp@nicira.com>
* Implement OpenFlow 1.1+ "groups" protocol.Neil Zhu2013-09-021-0/+116
| | | | | | | | | | | | | This doesn't include a dpif implementation of groups functionality. In its current form, it is untested. Signed-off-by: Neil Zhu <zhuj@centecnetworks.com> Co-authored-by: Ben Pfaff <blp@nicira.com> Signed-off-by: Ben Pfaff <blp@nicira.com> Co-authored-by: Simon Horman <horms@verge.net.au> Signed-off-by: Simon Horman <horms@verge.net.au> Co-authored-by: Jarno Rajahalme <jrajahalme@nicira.com> Signed-off-by: Jarno Rajahalme <jrajahalme@nicira.com>
* ofp-util: Add SCTP supportJoe Stringer2013-08-221-7/+13
| | | | | | Reviewed-by: Simon Horman <horms@verge.net.au> Signed-off-by: Joe Stringer <joe@wand.net.nz> Signed-off-by: Ben Pfaff <blp@nicira.com>
* flow: Enable matching on new field 'pkt_mark'.Jesse Gross2013-08-131-0/+7
| | | | | | | | | | | | | | | The Linux kernel datapath enables matching and setting the skb mark but this functionality is currently used only internally by ovs-vswitchd. This exposes it through NXM to enable external controllers to interact with other kernel subsystems. Although this is simply exporting the skb mark, the intention is that this is a platform independent mechanism to access some system metadata and therefore may have different implementations on various systems. Bug #17855 Signed-off-by: Jesse Gross <jesse@nicira.com> Acked-by: Andy Zhou <azhou@nicira.com>
* ovs-ofctl: Avoid groff warning due to too-long line.Ben Pfaff2013-08-131-1/+1
| | | | | | | | | | | Avoids these warnings from groff: <standard input>:1037: warning [p 14, 6.0i]: cannot adjust line <standard input>:1037: warning [p 14, 6.2i]: can't break line Found by lintian. Signed-off-by: Ben Pfaff <blp@nicira.com>
* ovs-ofctl: Add "ofp-parse" command for printing OpenFlow from a file.Ben Pfaff2013-08-061-0/+8
| | | | | | Test provided by Alex Wang <alexw@nicira.com>. Signed-off-by: Ben Pfaff <blp@nicira.com>
* Always update ethertype on mpls_popSimon Horman2013-06-051-3/+4
| | | | | | | | | | | | The ethertype should always be updated on mpls_pop as there may be a transition between MPLS unicast (0x8847) and MPLS multicast (0x8848). Ben Pfaff tells me that this is consistent with the behaviour described in EXT-194 of the JIRA bug tracker. Signed-off-by: Simon Horman <horms+renesas@verge.net.au> Signed-off-by: Ben Pfaff <blp@nicira.com>
* OpenFlow-level flow-based tunneling support.Jarno Rajahalme2013-05-101-0/+11
| | | | | | | | | | | | | | | | | | | | | Adds tun_src and tun_dst match and set capabilities via new NXM fields NXM_NX_TUN_IPV4_SRC and NXM_NX_TUN_IPV4_DST. This allows management of large number of tunnels via the flow tables, without requiring the tunnels to be pre-configured. Flow-based tunnels can be configured with options remote_ip=flow and local_ip=flow. local_ip=flow requires remote_ip=flow. When set, the tunnel remote IP address and/or local IP address is set from the flow, instead of the tunnel configuration. Example: $ ovs-vsctl add-port br0 gre -- set Interface gre ofport_request=1 type=gre options:remote_ip=flow options:key=flow $ ovs-ofctl add-flow br0 "in_port=LOCAL actions=set_tunnel:1,set_field:192.168.0.1->tun_dst,output:1" $ ovs-ofctl add-flow br0 "in_port=1 tun_src=192.168.0.1 tun_id=1 actions=LOCAL" Signed-off-by: Jarno Rajahalme <jarno.rajahalme@nsn.com> Signed-off-by: Ben Pfaff <blp@nicira.com>
* Implement IPFIX exportRomain Lenglet2013-04-221-0/+25
| | | | | | | | | | | | | | | | | | | | | | | | Define a new NXAST_SAMPLE OpenFlow vendor action and the corresponding OFPACT_SAMPLE OVS action, to do per-flow packet sampling, translated into a new SAMPLE "flow_sample" dp action. Make the userspace action's userdata size vary depending on the union member used. Add a new "flow_sample" upcall to do per-flow packet sampling. Add a new "ipfix" upcall to do per-bridge packet sampling to IPFIX collectors. Extend the OVSDB schema to support configuring IPFIX collector sets. Add support for configuring multiple IPFIX collectors for per-flow packet sampling. Add support for configuring per-bridge IPFIX sampling. Automatically generate standard IPFIX entity definitions from the IANA specs. Send one IPFIX data record message for every packet sampled by an OpenFlow sample action or received by a bridge configured with IPFIX sampling, and periodically send IPFIX template set messages. Signed-off-by: Romain Lenglet <rlenglet@vmware.com> Signed-off-by: Ben Pfaff <blp@nicira.com>
* ovs-ofctl: Improve description of mod_nw_tos action.Ben Pfaff2013-03-151-3/+3
| | | | | Reported-by: Murphy McCauley <murphy.mccauley@gmail.com> Signed-off-by: Ben Pfaff <blp@nicira.com>
* nicira-ext: Add Nicira actions NXAST_STACK_PUSH and NXAST_STACK_POP.Andy Zhou2013-03-061-0/+17
| | | | | | | | | | | | | | | | | | | The Push action takes a single parameter. Any source allowed by NXAST_REG_MOVE is allowed to be pushed onto the stack. When the source is a bit field, its value will be right shifted to bit zero before being pushed onto the stack. The remaining bits will be set to zero. The Pop action also takes a single parameter. Any destination allowed by NXAST_REG_MOVE can be used as the destination of the action. The value, in case of a bit field, will be taken from top of the stack, starting from bit zero. The stack size is not limited. The initial 8KB is statically allocated to efficiently handle most common use cases. When more stack space is required, the stack can grow using malloc(). Signed-off-by: Andy Zhou <azhou@nicira.com> Signed-off-by: Ben Pfaff <blp@nicira.com>
* Add support for set_mpls_ttl actionSimon Horman2013-03-061-0/+4
| | | | | | | | | | | This adds support for the OpenFlow 1.1+ set_mpls_ttl action. And also adds an NX set_mpls_ttl action. The handling of the TTL modification is entirely handled in userspace. Reviewed-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: Simon Horman <horms@verge.net.au> Signed-off-by: Ben Pfaff <blp@nicira.com>
* Add support for dec_mpls_ttl actionSimon Horman2013-03-061-0/+9
| | | | | | | | | | | This adds support for the OpenFlow 1.1+ dec_mpls_ttl action. And also adds an NX dec_mpls_ttl action. The handling of the TTL modification is entirely handled in userspace. Reviewed-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: Simon Horman <horms@verge.net.au> Signed-off-by: Ben Pfaff <blp@nicira.com>
* nicira-ext: Remove the autopath action.Ethan Jackson2013-02-051-15/+0
| | | | | | | | | The autopath action was attempting to achieve functionality similar to the bundle action, but was significantly clunkier, more difficult to understand, more difficult to use, and less reliable. This patch removes it. Signed-off-by: Ethan Jackson <ethan@nicira.com>
* User-Space MPLS actions and matchesSimon Horman2013-02-051-0/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch implements use-space datapath and non-datapath code to match and use the datapath API set out in Leo Alterman's patch "user-space datapath: Add basic MPLS support to kernel". The resulting MPLS implementation supports: * Pushing a single MPLS label * Poping a single MPLS label * Modifying an MPLS lable using set-field or load actions that act on the label value, tc and bos bit. * There is no support for manipulating the TTL this is considered future work. The single-level push pop limitation is implemented by processing push, pop and set-field/load actions in order and discarding information that would require multiple levels of push/pop to be supported. e.g. push,push -> the first push is discarded pop,pop -> the first pop is discarded This patch is based heavily on work by Ravi K. Cc: Ravi K <rkerur@gmail.com> Reviewed-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: Simon Horman <horms@verge.net.au> Signed-off-by: Ben Pfaff <blp@nicira.com>
* ovs-ofctl: Remove 'ANY' as a wildcard.Gurucharan Shetty2013-01-091-2/+2
| | | | | | | | | | | Currently we use "*" or ANY to mark a field in flow syntax as a wildcard. With ANY being a valid openflow port now, there is a conflict for in_port field. So at the least, we need to remove ANY from being considered as a wildcard for in_port. But this may cause general confusion and it may be a better idea to remove 'ANY' as a wildcard for all fields. Signed-off-by: Gurucharan Shetty <gshetty@nicira.com>
* manpages: Put version number instead of date at bottom of page.Ben Pfaff2013-01-041-1/+1
| | | | | | | | | | | | | | | | | We're really good about keeping manpages up to date, but terrible at updating the dates at the bottom of the manpages. So, instead of using manually updated dates, this commit switches to using automatically updated version numbers. We can only use automatically updated version numbers for manpages that we preprocess, that is, the manpages whose sources end with ".in". This excludes a couple of manpages that don't actually get installed with OVS, such as the manpages for ovs-ctl and ovsdb-idlc. This commit doesn't change those manpages. It does change the ovs-bugtool manpage to one that is preprocessed so that we can use the version there. Signed-off-by: Ben Pfaff <blp@nicira.com> Acked-by: Ethan Jackson <ethan@nicira.com>
* ovs-ofctl: Document to work with any OpenFlow switch.Ben Pfaff2013-01-041-0/+1
| | | | | | | | I've always intended ovs-ofctl to work with any OpenFlow switch, not just with Open vSwitch. This explicitly documents that intention. Signed-off-by: Ben Pfaff <blp@nicira.com> Acked-by: Ethan Jackson <ethan@nicira.com>
* ofp-util: Make "OXM" flow format name include both OpenFlow 1.2 and 1.3.Ben Pfaff2012-11-291-0/+7
| | | | | | | | | | | | To my mind, it makes sense, when a user requests OXM as the flow format, to allow any protocol based on the OXM flow format to be used. Until this commit, however, "OXM" was specifically OpenFlow 1.2, and "OpenFlow13" was required to request OXM on OpenFlow 1.3. This doesn't affect the behavior of any released version of Open vSwitch. Signed-off-by: Ben Pfaff <blp@nicira.com> Acked-by: Simon Horman <horms@verge.net.au>
* ovs-ofctl: Add option to set allowed OpenFlow versionsSimon Horman2012-11-291-0/+2
| | | | | | | | | | | | | | | | | --protocols allows configuration of the versions that may be used when establishing an OpenFlow connection. The default is 'OpenFlow10' which is consistent with the behaviour prior to this patch. The useful values at this time are: 'OpenFlow10', 'OpenFlow12', 'OpenFlow13', Values may be combined in a comma delimited list. e.g.: --protocols 'OpenFlow10,OpenFlow12,OpenFlow13' Signed-off-by: Simon Horman <horms@verge.net.au> Signed-off-by: Ben Pfaff <blp@nicira.com>
* Process RARP packets with ethertype 0x8035 similar to ARP packets.Mehak Mahajan2012-11-021-7/+18
| | | | | | | With this commit, OVS will match the data in the RARP packets having ethertype 0x8035, in the same way as the data in the ARP packets. Signed-off-by: Mehak Mahajan <mmahajan@nicira.com>
* OF11: push_vlan supportIsaku Yamahata2012-10-261-0/+6
| | | | | | | | This implementes push_vlan with 802.1Q. NOTE: 802.1AD (QinQ) is not supported. It requires another effort. Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: Ben Pfaff <blp@nicira.com>
* ofp-actions: Implement writing to metadata fieldJoe Stringer2012-10-181-1/+8
| | | | | | | | | | | | | | | In OpenFlow 1.1, we add support for OFPIT_WRITE_METADATA. This allows us to write to the metadata field. Internally it is represented using ofpact_metadata. We introduce NXAST_WRITE_METADATA to handle writing to the metadata field in OpenFlow 1.0+. This structure reflects OFPIT_WRITE_METADATA. When writing out the structure to OpenFlow 1.1, it uses the OFPIT_WRITE_METADATA instruction only, and not the new NXAST action (which would be redundant). Signed-off-by: Joe Stringer <joe@wand.net.nz> Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: Ben Pfaff <blp@nicira.com>
* instruction/clear-actions: string parser/formater, of packet decoder/encoderIsaku Yamahata2012-10-161-0/+3
| | | | | Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: Ben Pfaff <blp@nicira.com>
* instruction: support goto-table actionIsaku Yamahata2012-10-161-0/+9
| | | | | Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: Ben Pfaff <blp@nicira.com>
* ofp-actions: Add parsing of set_field actionsSimon Horman2012-09-281-0/+9
| | | | | | | | | Based heavily on work by Isaku Yamahata <yamahata@valinux.co.jp> Cc: Isaku Yamahata <yamahata@valinux.co.jp> Signed-off-by: Simon Horman <horms@verge.net.au> [blp@nicira.com adjusted documentation, added test] Signed-off-by: Ben Pfaff <blp@nicira.com>
* ovs-ofctl: Accept port keywords, OF1.1 port numbers, reject port number 0.Ben Pfaff2012-09-201-35/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | OpenFlow 1.0 has special reserved ports in the range 0xfff8 to 0xffff. OpenFlow 1.1 and later has the same ports in the range 0xfffffff8 to 0xffffffff and allows the OF1.0 range to be used for ordinary ("physical") switch ports. This means that, naively, the meaning of a port number in the range 0xfff8 to 0xffff given on the ovs-ofctl command line depends on the protocol in use. This commit implements something a little smarter: - Accept keyword names (e.g. LOCAL) for special reserved ports everywhere that such a port can plausibly be used (previously they were only accepted in some places). - Translate 0xfff8...0xffff to 0xfffffff8...0xffffffff for now, since OF1.1+ isn't in widespread use and those particular ports aren't likely to be in use in OF1.1+ anyway. - Log warnings about those ports when they are specified by number, to allow users to fix their invocations. Also: - Accept the OF1.1+ port numbers for these ports, without warning, for compatibility with the upcoming OF1.1+ support. - Stop accepting port number 0, which has never been a valid port number in OpenFlow 1.0 and later. (This required fixing some tests that inadvertently used this port number). Signed-off-by: Ben Pfaff <blp@nicira.com> Acked-by: Simon Horman <horms@verge.net.au>
* ovs-ofctl: Fix indentation in manpage.Ben Pfaff2012-09-171-1/+1
| | | | | | | | This .RE (to reduce the current indentation level) needs to go after all the actions, but it was mistakenly put before the fin_timeout action. Reported-by: Paul Ingram <paul@nicira.com> Signed-off-by: Ben Pfaff <blp@nicira.com>
* nicira-ext: Deprecate the autopath action.Ethan Jackson2012-08-281-0/+2
| | | | | | | | | | The autopath action is an obsolete attempt to replicate functionality contained in the bundle action. It is ugly and of questionable usefulness. This patch deprecates it and schedules its removal for February 2013. If there are concerns, please email dev@openvswitch.org. Signed-off-by: Ethan Jackson <ethan@nicira.com>
* man-pages: Add missing entry for matching metadataJoe Stringer2012-08-171-0/+8
| | | | | Signed-off-by: Joe Stringer <joe@wand.net.nz> Signed-off-by: Ben Pfaff <blp@nicira.com>
* Add Nicira vendor extension action NXAST_DEC_TTL_CNT_IDS.Mehak Mahajan2012-08-161-4/+10
| | | | | | | | | | | | | | | Currently, if a controller having a nonzero id registers to get a OFPR_INVALID_TTL async message, it will not receive it. This is because compose_dec_ttl() only sent the invalid ttl packets to the default controller id. NXAST_DEC_TTL_CNT_IDS is a new action that accepts a list of controller ids, each separated by `,', to which the OFPR_INVALID_TTL packets must be sent. The earlier requirement of the controller having to explicitly register to receive these asynchronous messages is retained. The syntax of this action is: dec_ttl(id1,id2) where id1, id2 are valid controller ids. Signed-off-by: Mehak Mahajan <mmahajan@nicira.com>
* ofproto: New feature to notify controllers of flow table changes.Ben Pfaff2012-07-121-4/+35
| | | | | | | | | | | OpenFlow switching monitoring and controller coordination can be made more efficient if the switch can notify a controller of flow table changes as they occur, rather than periodically polling for changes. This commit implements such a feature. Feature #6633. CC: Natasha Gude <natasha@nicira.com> Signed-off-by: Ben Pfaff <blp@nicira.com>
* ovs-ofctl: Add --sort and --rsort options for "dump-flows" command.Ben Pfaff2012-07-121-2/+35
| | | | | | | Feature #8754. Signed-off-by: Arun Sharma <arun.sharma@calsoftinc.com> [blp@nicira.com rewrote most of the code] Signed-off-by: Ben Pfaff <blp@nicira.com>
* ovs-ofctl: Fix typos in manpage.Ben Pfaff2012-07-061-3/+3
| | | | Signed-off-by: Ben Pfaff <blp@nicira.com>
* Allow general masking of IPv6 addresses rather than just CIDR masks.Ben Pfaff2012-06-121-1/+5
| | | | | | | OF1.2 and later make these fields fully maskable so we might as well also. Reviewed-by: Simon Horman <horms@verge.net.au> Signed-off-by: Ben Pfaff <blp@nicira.com>
* Allow general masking of IPv4 addresses rather than just CIDR masks.Ben Pfaff2012-06-121-1/+3
| | | | | | | OF1.1 and later make these fields fully maskable so we might as well also. Reviewed-by: Simon Horman <horms@verge.net.au> Signed-off-by: Ben Pfaff <blp@nicira.com>
* flow: Adds support for arbitrary ethernet maskingJoe Stringer2012-05-291-3/+5
| | | | | | | | | | | Arbitrary ethernet mask support is one step on the way to support for OpenFlow 1.1+. This patch set seeks to add this capability without breaking current protocol support. Signed-off-by: Joe Stringer <joe@wand.net.nz> [blp@nicira.com made some updates, see http://openvswitch.org/pipermail/dev/2012-May/017585.html] Signed-off-by: Ben Pfaff <blp@nicira.com>
* ofp-util: Clean up cookie handling.Justin Pettit2012-05-291-12/+28
| | | | | | | | | | | | | | | | | | | | | | | Commit e72e793 (Add ability to restrict flow mods and flow stats requests to cookies.) modified cookie handling. Some of its behavior was unintuitive and there was at least one bug (described below). Commit f66b87d (DESIGN: Document uses for flow cookies.) attempted to document a clean design for cookie handling. This commit updates the DESIGN document and brings the implementation in line with it. In commit e72e793, the code that handled processing OpenFlow flow modification requests set the cookie mask to exact-match. This seems reasonable for adding flows, but is not correct for matching, since OpenFlow 1.0 doesn't support matching based on the cookie. This commit changes to cookie mask to fully wildcarded, which is the correct behavior for modifications and deletions. It doesn't cause any problems for flow additions, since the mask is ignored for that operation. Bug #9742 Reported-by: Luca Giraudo <lgiraudo@nicira.com> Reported-by: Paul Ingram <paul@nicira.com> Signed-off-by: Justin Pettit <jpettit@nicira.com>
* ovs-ofctl: Support all OFPPC_* flags in "mod-port" command.Ben Pfaff2012-05-241-20/+38
| | | | Signed-off-by: Ben Pfaff <blp@nicira.com>
* ofproto: Add support for OF1.3 port description multipart message.Justin Pettit2012-05-081-0/+6
| | | | | | | | | | | | | OpenFlow 1.0 is limited to displaying 1364 ports in the Features Reply message, and there is no other way to get consolidated port information. OpenFlow 1.3 adds a new port description multipart message (OFPMP_PORT_DESC) that is not limited by size. This commit adds support through the OpenFlow 1.0 stats mechanism, since they have complimentary enum values. Bug #11040 Signed-off-by: Justin Pettit <jpettit@nicira.com>
* ovs-ofctl: Document padding in "note" actions.Ben Pfaff2012-05-011-0/+4
| | | | | Reported-by: Paul Ingram <paul@nicira.com> Signed-off-by: Ben Pfaff <blp@nicira.com>