summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* datapath-windows: STT Check for flow destination portAlin Serdean2017-02-021-1/+2
| | | | | | | | | | | Change the TCP destination port(STT header) to check if it was set by the userspace, use it if it was set. If the userspace did not specify a destination port, use the configured vport destination port. Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com> Signed-off-by: Ben Pfaff <blp@ovn.org> Acked-by: Sairam Venugopal <vsairam@vmware.com>
* datapath-windows: VXLAN Check for flow destination portAlin Serdean2017-02-021-1/+2
| | | | | | | | | | | Change the UDP destination port(VXLAN header) to check if it was set by the userspace, use it if it was set. If the userspace did not specify a destination port, use the configured vport destination port. Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com> Signed-off-by: Ben Pfaff <blp@ovn.org> Acked-by: Sairam Venugopal <vsairam@vmware.com>
* datapath-windows: Add support for OVS_TUNNEL_KEY_ATTR_TP_DSTAlin Serdean2017-02-021-0/+8
| | | | | | | | | Add support for netlink attribute OVS_TUNNEL_KEY_ATTR_TP_DST get/set flow functions. Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com> Signed-off-by: Ben Pfaff <blp@ovn.org> Acked-by: Sairam Venugopal <vsairam@vmware.com>
* datapath-windows: Allow tunnel action to modify destination portAlin Serdean2017-02-021-2/+2
| | | | | | | | | | | | 'OvsTunnelAttrToIPv4TunnelKey' modifies 'tunkey' with the received netlink attributes(i.e. OVS_TUNNEL_KEY_ATTR_IPV4_DST). Change the order of the value assignment to reflect the values received via userspace. Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com> Signed-off-by: Ben Pfaff <blp@ovn.org> Acked-by: Sairam Venugopal <vsairam@vmware.com>
* tests: Fix race in MPLS translate.Alin Serdean2017-02-021-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | On slow systems ofproto/trace in combination with a recirculation ID causes issues because the flow is evicted before the second packet can reach it. I.e: " Flow: recirc_id=0x2,ip,in_port=1,vlan_tci=0x0000,dl_src=f8:bc:12:44:34:b6, dl_dst=f8:bc:12:46:58:e0,nw_src=1.1.2.92,nw_dst=1.1.2.88,nw_proto=47, nw_tos=0,nw_ecn=0,nw_ttl=64 bridge("br0") ------------- >>>> Recirculation context not found for ID 2 <<<< Final flow: unchanged Megaflow: recirc_id=0x2,ip,in_port=1,nw_frag=no Datapath actions: drop Translation failed (No recirculation context), packet is dropped. " Since eviction of the flows is not needed for the current logic, this commit adds a time/stop to bypass the problem. Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com> Signed-off-by: Ben Pfaff <blp@ovn.org>
* ovn-trace: Style improvements.Ben Pfaff2017-02-021-3/+9
| | | | | Signed-off-by: Ben Pfaff <blp@ovn.org> Acked-by: Russell Bryant <russell@ovn.org>
* odp: Add datapath clone action parser.Andy Zhou2017-02-012-13/+48
| | | | | | | | | When adding userspace datapath clone action, the corresponding odp actions parser and unit tests were missing. This patch adds them. Reported-by: Ben Pfaff <blp@ovn.org> Signed-off-by: Andy Zhou <azhou@ovn.org> Acked-by: Ben Pfaff <blp@ovn.org>
* tun-metadata: Fix entry key documentation.Yi-Hung Wei2017-02-011-1/+1
| | | | | Signed-off-by: Yi-Hung Wei <yihung.wei@gmail.com> Signed-off-by: Joe Stringer <joe@ovn.org>
* ofp-actions: Fix variable length meta-flow OXMs.Yi-Hung Wei2017-02-0117-159/+579
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, if a flow action that involves a tunnel metadata meta-flow field is dumped from vswitchd, the replied field length in the OXM header is filled with the maximum possible field length, instead of the length configured in the tunnel TLV mapping table. To solve this issue, this patch introduces the following changes. In order to maintain the correct length of variable length mf_fields (i.e. tun_metadata), this patch creates a per-switch based map (struct vl_mff_map) that hosts the variable length mf_fields. This map is updated when a controller adds/deletes tlv-mapping entries to/from a switch. Although the per-swtch based vl_mff_map only hosts tun_metadata for now, it is able to support new variable length mf_fields in the future. With this commit, when a switch decodes a flow action with mf_field, the switch firstly looks up the global mf_fields map to identify the mf_field type. For the variable length mf_fields, the switch uses the vl_mff_map to get the configured mf_field entries. By lookig up vl_mff_map, the switch can check if the added flow action access beyond the configured size of a variable length mf_field, and the switch reports an ofperr if the controller adds a flow with unmapped variable length mf_field. Later on, when a controller request flows from the switch, with the per-switch based mf_fields, the switch will encode the OXM header with correct length for variable length mf_fields. To use the vl_mff_map for decoding flow actions, extract-ofp-actions is updated to pass the vl_mff_map to the required action decoding functions. Also, a new error code is introduced to identify a flow with an invalid variable length mf_field. Moreover, a testcase is added to prevent future regressions. Committer notes: - Factor out common code - Style fixups - Rename OFPERR_NXFMFC_INVALID_VL_MFF -> OFPERR_NXFMFC_INVALID_TLV_FIELD VMWare-BZ: #1768370 Reported-by: Harold Lim <haroldl@vmware.com> Suggested-by: Joe Stringer <joe@ovn.org> Suggested-by: Jarno Rajahalme <jarno@ovn.org> Signed-off-by: Yi-Hung Wei <yihung.wei@gmail.com> Signed-off-by: Joe Stringer <joe@ovn.org>
* ovn-controller: Configure interface QoS only if it would actually be used.Ben Pfaff2017-02-011-6/+35
| | | | | | | | | | | | | | | | | | | | | Until now, ovn-controller has unconditionally configured linux-htb on physical interfaces. QoS is pretty much always trouble, but it's even more trouble if we set it up for no good reason. We received a bug report, in particular, that doing this disrupts connectivity in Docker. This commit attempts to make that less likely, by making ovn-controller only configure a qdisc if QoS support has in turn been configured in OVN. The same problems as before will recur if QoS support is actually configured, but at least now there's some purpose, and possibly a symptom that the user can better diagnose ("I turned on QoS and OVN stopped working" is at least a cause-and-effect chain that makes some sense). Reported-by: Ritesh Rekhi <ritesh.rekhi@nutanix.com> Reported-by: Hexin Wang <hexin.wang@nutanix.com> Reported-at: https://mail.openvswitch.org/pipermail/ovs-discuss/2017-February/043564.html Tested-by: Hexin Wang <hexin.wang@nutanix.com> Tested-at: https://mail.openvswitch.org/pipermail/ovs-discuss/2017-February/043575.html Signed-off-by: Ben Pfaff <blp@ovn.org>
* ovn-controller: Fix memory leak and bug in setup_qos().Ben Pfaff2017-02-011-2/+5
| | | | | | | | | | The caller of netdev_get_qos() is responsible for freeing its 'details' smap. Also, the previous qdisc details, for some arbitrary previous type of QoS, aren't relevant for changing the type of QoS. Signed-off-by: Ben Pfaff <blp@ovn.org>
* Prepare for post-2.7.0 (2.7.90).Justin Pettit2017-01-313-6/+88
| | | | Signed-off-by: Justin Pettit <jpettit@ovn.org>
* ovn: fix slave node can not connect to the master node using SSL, for pacemakerGuoshuai Li2017-01-312-5/+60
| | | | | | | | | | The default slave node connect to the master node using TCP, and the pacemaker can not modify the protocol and port of the connection. Add pacemaker parameters to support the connection of the slave node to the master node using a different protocol and port. Signed-off-by: Guoshuai Li <ligs@dtdream.com> Acked-by: Andy Zhou <azhou@ovn.org>
* ofproto-dpif: Reduce the time to create many bridges.Sha Zhang2017-01-312-2/+3
| | | | | | | | | | | This patch moves xlate_txn_start() and xlate_txn_commit() out of the loop traversing all the ofproto-dpifs to reduce the time of creating a large mount of bridges in separate database transactions. As a global variable, new_xcfg should only be allocated at the beginning and commited at the end once time, rather than doing it repeatedly in the loop body. Signed-off-by: Sha Zhang <zhangsha.zhang@huawei.com> Signed-off-by: Ben Pfaff <blp@ovn.org>
* ovn-controller: Provide the option to set Encap.options:csumNuman Siddique2017-01-313-8/+23
| | | | | | | | | | | | | ovn-controller by default enables tunnel encapsulation checksums for geneve tunnels. With this patch user can set the desired value in Open_vSwitch.external_ids:ovn_encap_csum. This option will be useful in cases where enabling tunnel encapsulation checksums incur significant performance loss due to limitations in checksum offloading capabilities of the nics. Signed-off-by: Numan Siddique <nusiddiq@redhat.com> Signed-off-by: Ben Pfaff <blp@ovn.org>
* ovn-trace: Fix memory leaks.Ben Pfaff2017-01-311-1/+33
| | | | | Suggested-by: Justin Pettit <jpettit@ovn.org> Signed-off-by: Ben Pfaff <blp@ovn.org>
* ovn-nbctl: check for duplicated ACL adding.Han Zhou2017-01-313-2/+21
| | | | | | | | Check for duplicated ACL adding and add option --may-exist for ovn-nbctl acl-add. Signed-off-by: Han Zhou <zhouhan@gmail.com> Signed-off-by: Ben Pfaff <blp@ovn.org>
* appveyor: Bump OpenSSL versionAlin Serdean2017-01-311-3/+3
| | | | | | | | | | The URL https://slproweb.com/download/Win32OpenSSL-1_0_2j.exe is no longer valid. Bump to the variables to the latest version of OpenSSL version. Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com> Signed-off-by: Ben Pfaff <blp@ovn.org>
* documentation: Windows supports multiple VTEP/NICs now.Alin Serdean2017-01-311-6/+2
| | | | | | | | | Multiple VTEP and multiple physical NICs are supported on Hyper-V now. Update the documentation Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com> Signed-off-by: Ben Pfaff <blp@ovn.org> Acked-by: Sairam Venugopal <vsairam@vmware.com>
* AUTHORS: Add Roi Dayan.Ben Pfaff2017-01-311-0/+1
| | | | Signed-off-by: Ben Pfaff <blp@ovn.org>
* netlink-socket: Fix possiblity of nl_transact dereferencing null pointerRoi Dayan2017-01-311-1/+3
| | | | | | | | | Many nl_transact callers and its wrapper tc_transact pass NULL for replyp which is being accessed in error flow without being checked if null or not. Signed-off-by: Roi Dayan <roid@mellanox.com> Reviewed-by: Paul Blakey <paulb@mellanox.com> Signed-off-by: Ben Pfaff <blp@ovn.org>
* rhel: Fix ifdown for OVSDPDKBond.Daniele Di Proietto2017-01-301-1/+1
| | | | | | | | The OVSDPDKBond case wasn't handled in the rhel ifdown script. Fixes: f6bf8880613a ("rhel: Add support DPDK port creation via network scripts") Signed-off-by: Daniele Di Proietto <diproiettod@vmware.com> Acked-by: Aaron Conole <aconole@redhat.com>
* rhel: Add missing unpackaged file 'ovs-fields.7.gz' in the %files listNuman Siddique2017-01-302-0/+2
| | | | | | | Fixes: 96fee5e0a2a0 ("ovs-fields: New manpage to document Open vSwitch and OpenFlow fields") Signed-off-by: Numan Siddique <nusiddiq@redhat.com> Signed-off-by: Daniele Di Proietto <diproiettod@vmware.com>
* ovn-trace: Possible null dereferenceAlin Serdean2017-01-301-1/+1
| | | | | | | Found by inspection. Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com> Signed-off-by: Gurucharan Shetty <guru@ovn.org>
* windows: wmi add includeAlin Serdean2017-01-301-0/+1
| | | | | | | | | | | | Add 'util.h' to includes otherwise the result of the function 'ovs_format_message' will be unknown and be converted to int, triggering an abort of vswitchd. Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com> Reported-at: https://github.com/openvswitch/ovs-issues/issues/123 Reported-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com> Acked-by: Sairam Venugopal <vsairam@vmware.com> Signed-off-by: Gurucharan Shetty <guru@ovn.org>
* ovs-router: introduce pkt-mark.Pravin B Shelar2017-01-288-42/+147
| | | | | | | | | | OVS router is basically partial copy of linux kernel FIB. kernel routing table uses skb-mark along with usual routing parameters. Following patch brings in support for skb-mark to ovs-router so that we can lookup route for given skb-mark. Signed-off-by: Pravin B Shelar <pshelar@ovn.org> Acked-by: Jarno Rajahalme <jarno@ovn.org>
* routing-table: parse skb-mark from RTNETLINK msgPravin B Shelar2017-01-281-0/+6
| | | | | | | | Keep track of skb-mark of given RTNL routing notification. This will be used by next commit. Signed-off-by: Pravin B Shelar <pshelar@ovn.org> Acked-by: Jarno Rajahalme <jarno@ovn.org>
* tunnel: Add support to configure ptk_markPravin B Shelar2017-01-286-0/+38
| | | | | | | | | | | | | | | | Today packet mark action is broken for Tunnel ports with tunnel monitoring. User can write a flow to set pkt-mark for any tunnel traffic, but there is no way to set the packet mark for corresponding BFD traffic. Following patch introduces new option in OVSDB tunnel configuration so that user can set skb-mark for given tunnel endpoint. OVS would set the mark according to the skb-mark option for all tunnel traffic including packets generated by vSwitchd like tunnel monitoring BFD packet. Signed-off-by: Pravin B Shelar <pshelar@ovn.org> Acked-by: Jarno Rajahalme <jarno@ovn.org>
* tests: rstp close input fileAlin Serdean2017-01-271-0/+1
| | | | | | | Close the 'input_file' after data is read. Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com> Signed-off-by: Gurucharan Shetty <guru@ovn.org>
* tests dpdk: ring_client, client_id is unsignedAlin Serdean2017-01-271-1/+1
| | | | | | | 'client_id' is of type unsigned int. Change the printf specifier to match it. Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com> Signed-off-by: Gurucharan Shetty <guru@ovn.org>
* python windows: Allow clients to read from server before disconnect.Alin Serdean2017-01-271-0/+3
| | | | | | | Wait for clients to read from the pipe before disconnecting the server. Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com> Signed-off-by: Gurucharan Shetty <guru@ovn.org>
* windows: Allow clients to read from server before disconnectAlin Serdean2017-01-271-0/+3
| | | | | | | | | | Wait for clients to read from the pipe before disconnecting the server. Found while testing. Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com> Acked-by: Sairam Venugopal <vsairam@vmware.com> Signed-off-by: Gurucharan Shetty <guru@ovn.org>
* tests windows: change service testAlin Serdean2017-01-271-1/+2
| | | | | | | | | | | Skip the test if the service 'ovsdb-server' is already defined. The arguments of the service are incomplete: in the former state it will try to create the pidfile and unixctl in the configuration path. This patch adds those arguments. Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com> Signed-off-by: Gurucharan Shetty <guru@ovn.org>
* datapath-windows: Fix alignment in MapTunAttrToFlowPutAlin Serdean2017-01-271-1/+1
| | | | | | | | Found by inspection. Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com> Acked-by: Sairam Venugopal <vsairam@vmware.com> Signed-off-by: Gurucharan Shetty <guru@ovn.org>
* datapath-windows: Add a wrapper to retreive external vportShashank Ram2017-01-274-7/+15
| | | | | | | | This wrapper is to simplify readability. Signed-off-by: Shashank Ram <rams@vmware.com> Acked-by: Sairam Venugopal <vsairam@vmware.com> Signed-off-by: Gurucharan Shetty <guru@ovn.org>
* datapath-windows: Add support for OVS_KEY_ATTR_TCP set actionAlin Serdean2017-01-271-0/+48
| | | | | | | | | | | | | | This patch adds support for set action with OVS_KEY_ATTR_TCP attribute (change TCP source or destination port). If the source or destination TCP port was changed, update the TCP checksum. A sample flow can look like the following: set(tcp(src=80,dst=443)) Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com> Acked-by: Sairam Venugopal <vsairam@vmware.com> Signed-off-by: Gurucharan Shetty <guru@ovn.org>
* datapath-windows: Add support for OVS_KEY_ATTR_UDP set actionAlin Serdean2017-01-271-1/+51
| | | | | | | | | | | | | | This patch adds support for set action with OVS_KEY_ATTR_UDP attribute (change UDP source or destination port). If the source or destination UDP port was changed, update the UDP checksum. A sample flow can look like the following: set(udp(src=67,dst=68)) Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com> Acked-by: Sairam Venugopal <vsairam@vmware.com> Signed-off-by: Gurucharan Shetty <guru@ovn.org>
* datapath-windows: Add function to get continuous buffer from contextAlin Serdean2017-01-271-38/+62
| | | | | | | | | | | | | This patch extracts the code that tries to get a continuous IPv4 header buffer from the function 'OvsUpdateIPv4Header' and moves it to a new function 'OvsGetHeaderBySize'. The new function can be used later when trying to change the UDP/TCP/MPLS etc., headers. Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com> Acked-by: Sairam Venugopal <vsairam@vmware.com> Signed-off-by: Gurucharan Shetty <guru@ovn.org>
* datapath-windows: OvsUpdateIPv4Header remove unnecessary additionAlin Serdean2017-01-271-4/+4
| | | | | | | | | bufferStart can be used directly to access the data of the net buffer. Add the MDL offset to save unnecessary additions. Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com> Acked-by: Sairam Venugopal <vsairam@vmware.com> Signed-off-by: Gurucharan Shetty <guru@ovn.org>
* Windows: Fix wmi.c to use count of wchar_t instead of sizeofSairam Venugopal2017-01-271-17/+21
| | | | | | | | | | | | | wcscat_s and wcscpy_s requires number of elements as argument. wchar_t uses 2 bytes for storage and using sizeof(internal_port_query) causes access violation error on Windows 2012 R2 (64 bit). This patch introduces a #define WMI_QUERY_COUNT set to 2048 and uses that instead. Reported-by: Sairam Venugopal <vsairam@vmware.com> Reported-at: openvswitch/ovs-issues#121 Signed-off-by: Sairam Venugopal <vsairam@vmware.com> Acked-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com> Signed-off-by: Gurucharan Shetty <guru@ovn.org>
* windows: WSAPoll broken on windowsAlin Serdean2017-01-271-1/+17
| | | | | | | | | | | | | Unfortunately, WSAPoll misbehaves on Windows please view detailed behavior on: https://github.com/openvswitch/ovs-issues/issues/117 We replace the WSAPoll with select looking only for errors and write events. Reported-at: https://github.com/openvswitch/ovs-issues/issues/117 Reported-by: Yin Lin <linyi@vmware.com> Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com> Acked-by: Sairam Venugopal <vsairam@vmware.com> Signed-off-by: Gurucharan Shetty <guru@ovn.org>
* ovn: rewrite redirect-chassis description in ovn-nb.xmlMickey Spiegel2017-01-271-18/+18
| | | | | | | | | This optional patch addresses offline comments that the documentation in ovn-nb.xml should not describe southbound constructs or flow details, since it is user facing documentation. Signed-off-by: Mickey Spiegel <mickeys.dev@gmail.com> Signed-off-by: Gurucharan Shetty <guru@ovn.org>
* ovn: ovn-nbctl commands for distributed NATMickey Spiegel2017-01-274-39/+119
| | | | | | | | | This patch adds the new optional arguments "logical_port" and "external_mac" to lr-nat-add, and displays that information in lr-nat-list. Signed-off-by: Mickey Spiegel <mickeys.dev@gmail.com> Signed-off-by: Gurucharan Shetty <guru@ovn.org>
* ovn: distributed NAT flowsMickey Spiegel2017-01-277-80/+1159
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch implements the flows required in the ingress and egress pipeline stages in order to support NAT on a distributed logical router. NAT functionality is associated with the logical router gateway port. The flows that carry out NAT functionality all have match conditions on inport or outport equal to the logical router gateway port. There are additional flows that are used to redirect traffic when necessary, using the tunnel key of a "chassisredirect" SB port binding in order to redirect traffic to the instance of the logical router gateway port on the centralized "redirect-chassis". North/south traffic subject to one-to-one "dnat_and_snat" is handled in a distributed manner, with south-to-north traffic going to the local instance of the logical router gateway port. North/south traffic subject to (possibly one-to-many) "snat" is handled in a centralized manner, with south-to-north traffic going to the instance of the logical router gateway port on the "redirect-chassis". North-to-south traffic is directed to the corresponding chassis by limiting ARP responses to the appropriate instance of the logical router gateway port on one chassis. For centralized NAT rules, this is the instance on the "redirect-chassis". For distributed NAT rules, this is the chassis where the corresponding logical port resides, using an ethernet address specified in the NB NAT rule to trigger upstream MAC learning. East/west NAT traffic is all handled in a centralized manner. While it is certainly possible to handle some of this traffic in a distributed manner, the centralized approach keeps the NAT flows simpler and cleaner. The expectation is that east/west NAT traffic is not as important to optimize as north/south NAT traffic, with most east/west traffic not requiring NAT. Automated tests are currently limited to only a single node. The single node automated tests cover both north/south and east/west traffic flows. Signed-off-by: Mickey Spiegel <mickeys.dev@gmail.com> Signed-off-by: Gurucharan Shetty <guru@ovn.org>
* ovn: avoid snat recirc only on gateway routersMickey Spiegel2017-01-275-11/+42
| | | | | | | | | | | | | | | | | | | | | | | | Currently, for performance reasons on gateway routers, ct_snat that does not specify an IP address does not immediately trigger recirculation. On gateway routers, ct_snat that does not specify an IP address happens in the UNSNAT pipeline stage, which is followed by the DNAT pipeline stage that triggers recirculation for all packets. This DNAT pipeline stage recirculation takes care of the recirculation needs of UNSNAT as well as other cases such as UNDNAT. On distributed routers, UNDNAT is handled in the egress pipeline stage, separately from DNAT in the ingress pipeline stages. The DNAT pipeline stage only triggers recirculation for some packets. Due to this difference in design, UNSNAT needs to trigger its own recirculation. This patch restricts the logic that avoids recirculation for ct_snat, so that it only applies to datapaths representing gateway routers. Signed-off-by: Mickey Spiegel <mickeys.dev@gmail.com> Signed-off-by: Gurucharan Shetty <guru@ovn.org>
* ovn: move load balancing flows after NAT flowsMickey Spiegel2017-01-271-70/+70
| | | | | | | | | | This will make it easy for distributed NAT to reuse some of the existing code for NAT flows, while leaving load balancing and defrag as functionality specific to gateway routers. There is no intent to change any functionality in this patch. Signed-off-by: Mickey Spiegel <mickeys.dev@gmail.com> Signed-off-by: Gurucharan Shetty <guru@ovn.org>
* dp-packet: Enhance packet batch APIs.Andy Zhou2017-01-269-159/+199
| | | | | | | | | | | | | One common use case of 'struct dp_packet_batch' is to process all packets in the batch in order. Add an iterator for this use case to simplify the logic of calling sites, Another common use case is to drop packets in the batch, by reading all packets, but writing back pointers of fewer packets. Add macros to support this use case. Signed-off-by: Andy Zhou <azhou@ovn.org> Acked-by: Jarno Rajahalme <jarno@ovn.org>
* netdev-dummy: Add --len option for netdev-dummy/receive commandAndy Zhou2017-01-261-10/+28
| | | | | | | | | | Currently, there is no way to specify the packet size when injecting a packet via "netdev-dummy/receive" with a flow specification. Thus far, packet size is not important for testing OVS features, but it becomes useful in writing unit tests for the future patches. Signed-off-by: Andy Zhou <azhou@ovn.org> Acked-by: Jarno Rajahalme <jarno@ovn.org>
* windows: Change driver and MSI company name to LFAlin Serdean2017-01-264-4/+4
| | | | | | | | | | | | | Until now we used 'Open vSwitch' as the company/organization name. The project is now under The Linux Foundation ownership. This patch updates the MSI and driver attributes to reflect that ownership. Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com> Acked-by: Ben Pfaff <blp@ovn.org> Acked-by: Sairam Venugopal <vsairam@vmware.com> Signed-off-by: Gurucharan Shetty <guru@ovn.org>
* lib: Remove generated ovs-fields.7 manpage on make clean.Ben Pfaff2017-01-261-1/+2
| | | | | | | Found by travis. Signed-off-by: Ben Pfaff <blp@ovn.org> Acked-by: Joe Stringer <joe@ovn.org>
View Lorry Controller Status