summaryrefslogtreecommitdiff
path: root/AUTHORS
Commit message (Collapse)AuthorAgeFilesLines
* debian: Correct path to ovs-controller in init script.Ben Pfaff2011-09-201-0/+1
| | | | | Reported-by: George Shuklin <amarao@desunote.ru> Bug-report: http://bugs.debian.org/642206
* AUTHORS: Add Tyler Coumbes <coumbes@gmail.com>.Ben Pfaff2011-09-151-0/+1
|
* ovs-vsctl: Improve usage message.Ben Pfaff2011-09-151-0/+1
| | | | | Bug #7332. Reported-by: Gordon Good <ggood@nicira.com>
* debian: Apply Ubuntu patch to add DKMS support.Chuck Short2011-08-251-0/+1
| | | | | | | | | | | | | | | | | | | | | | | I tested that installing openvswitch-datapath-dkms worked OK on my own Debian machine. The bulk of this patch is taken from downstream Ubuntu DKMS support written by Chuck Short <zulcss@ubuntu.com>, version 1.2.0-1ubuntu1. I made the following changes: * Update debian/.gitignore. * Update debian/automake.mk. * Correct description in debian/control (it was a cut-and-paste from the openvswitch-datapath-source description without editing). * Fix up for --with-l26 to --with-linux and datapath/linux-2.6 to datapath/linux transitions. CC: Chuck Short <zulcss@ubuntu.com> CC: Dave Walker <DaveWalker@ubuntu.com> Acked-by: Simon Horman <horms@verge.net.au>
* INSTALL.Linux: Fix up reference to old option name --with-l26.Philippe Jung2011-08-171-0/+1
|
* Option to forward BPDU (Ethernet control class) framesSanjay Sane2011-08-091-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, a NORMAL action bridge drops reserved-multicast-mac addresses; 01-80-c2-00-00-[f0:ff]. A node that does not implement STP should have an option to forward such frames. This commit proposes to have a configuration option to allow forwarding of BPDU class frames. To ensure backward compatibility, this option is disabled by default. This config can be set using bridge's other-config column, for e.g ovs-vsctl set bridge br0 other-config:forward-bpdu=true Changing this option can revalidate all flows in a software-OVS implementation (ofproto-dpif) -------- unit tests: ------------ make config changes, test runtime behavior -- test runtime behavior -- continuously send packets to br0 with dest-mac=01:80:c2:00:00:00 ovs-dpctl dump-flows br0 ovs-vsctl set bridge br0 other-config:forward-bpdu=true ovs-dpctl dump-flows br0 ovs-vsctl set bridge br0 other-config:forward-bpdu=false ovs-dpctl dump-flows br0 ovs-vsctl set bridge br0 other-config:forward-bpdu=true ovs-dpctl dump-flows br0 ovs-vsctl remove bridge br0 other-config forward-bpdu=true ovs-dpctl dump-flows br0 --result-- ovs-dpctl dump-flows br0 in_port(1),eth(src=00:0c:29:d1:39:42,dst=01:80:c2:00:00:00), packets:29550, bytes:1773000, used:0.004s, actions:drop ovs-vsctl set bridge br0 other-config:forward-bpdu=true ovs-dpctl dump-flows br0 in_port(1),eth(src=00:0c:29:d1:39:42,dst=01:80:c2:00:00:00), packets:8209, bytes:492540, used:0.000s, actions:2,0 ovs-vsctl set bridge br0 other-config:forward-bpdu=false ovs-dpctl dump-flows br0 in_port(1),eth(src=00:0c:29:d1:39:42,dst=01:80:c2:00:00:00), packets:19, bytes:1140, used:0.000s, actions:drop ovs-vsctl set bridge br0 other-config:forward-bpdu=true ovs-dpctl dump-flows br0 in_port(1),eth(src=00:0c:29:d1:39:42,dst=01:80:c2:00:00:00), packets:29, bytes:1740, used:0.000s, actions:2,0 ovs-vsctl remove bridge br0 other-config forward-bpdu=true ovs-dpctl dump-flows br0 in_port(1),eth(src=00:0c:29:d1:39:42,dst=01:80:c2:00:00:00), packets:0, bytes:0, used:never, actions:drop Bug #6624 Reported-by: Niklas Andersson <nandersson@nicira.com>
* lib: Adapt headers for use in C++.Casey Barker2011-08-041-0/+1
| | | | | | | This commit makes several library headers suitable for inclusion in C++. It adds [extern "C"] guards and makes minor changes to fix casting and keyword issues.
* Datapath action should not refer to controllerpravin shelar2011-07-281-0/+1
| | | | | | | | ODP_ACTION_ATTR_CONTROLLER in the kernel actually sends packets to userspace, not the controller. To make it generic rename this action to ODP_ACTION_ATTR_USERSPACE. Signed-off-by: Pravin B Shelar <pshelar@nicira.com>
* ovs-ofctl: New --readd option for "replace-flows".Ben Pfaff2011-07-271-0/+1
| | | | | | This is useful for resetting flows' byte and packet counters to 0. Suggested-by: Jed Daniels <openvswitch@jeddaniels.com>
* debian: Modernize use of dh_install.Ben Pfaff2011-07-261-0/+1
| | | | | | | | | | | | | Originally I intended this as just a cleanup, but as a side effect it also installs some files from the install tree in debian/tmp instead of from _debian. This should avoid a reported problem in which ovs-bugtool was being created in the source directory instead of the build directory (I still don't see why this happened). Reported-by: Sébastien RICCIO <sr@swisscenter.com> Tested-by: Sébastien RICCIO <sr@swisscenter.com> Acked-by: Simon Horman <horms@verge.net.au> CC: Simon Horman <horms@verge.net.au>
* PORTING: Improve second diagram.Ben Pfaff2011-06-151-0/+1
| | | | Suggested-by: Peter Phaal <peter.phaal@inmon.com>
* AUTHORS: Add Hao Zheng <hzheng@nicira.com>.Ben Pfaff2011-06-141-0/+1
|
* python: Fix "make install" on systems without Python.Ben Pfaff2011-06-141-0/+1
| | | | Reported-by: 冯全树(Crab) <fqs888@126.com>
* learning-switch: Don't limit message queued by --with-flows.Ben Pfaff2011-06-011-0/+1
| | | | | | | | | | | | | queue_tx() intentionally limits the number of outstanding OpenFlow messages queued to the switch. This was unintentionally being applied to the messages queued to the switch at startup by ovs-ofctl's --with-flows command. This patch should fix the problem, by calling rconn_send() directly instead of through queue_tx(). Ahmed reported that with this patch there was still a problem when 30,000 flows were specified in the file. Reported-by: Ahmed Bilal <numan252@gmail.com>
* Add tar.gz output option in ovs-bugtoolShih-Hao Li2011-05-181-0/+1
|
* ovs-brcompatd: Document bug.Ben Pfaff2011-05-041-0/+1
| | | | Reported-by: Gregor Schaffrath <grsch@net.t-labs.tu-berlin.de>
* xenserver: Use .../extra not .../kernel/extra for kernel modules.Ben Pfaff2011-05-031-0/+1
| | | | | | | | | | On XenServer, depmod.conf causes modules in /lib/modules/$(uname -r)/extra to take priority over standard modules. Unfortunately, we were installing our modules in /lib/modules/$(uname -r)/kernel/extra, which isn't special. This commit fixes the problem. Signed-off-by: Ben Pfaff <blp@nicira.com> Reported-by: Bob Ball <bob.ball@citrix.com>
* INSTALL.Linux: Mention that SSL options require building with SSL support.Ben Pfaff2011-04-211-0/+1
| | | | Reported-by: Aaron Rosen <arosen@clemson.edu>
* Fix calls to ctype functions.Ben Pfaff2011-04-181-0/+1
| | | | | | | | | | | | | | | | | | | | | The ctype functions often need casts to be fully C standards compliant. Here's the full explanation that I used to post to comp.lang.c from time to time when the issue came up: With the to*() and is*() functions, you should be careful to cast `char' arguments to `unsigned char' before calling them. Type `char' may be signed or unsigned, depending on your compiler or its configuration. If `char' is signed, then some characters have negative values; however, the arguments to is*() and to*() functions must be nonnegative (or EOF). Casting to `unsigned char' fixes this problem by forcing the character to the corresponding positive value. This fixes the following warnings from some version of GCC: lib/ofp-parse.c:828: warning: array subscript has type 'char' lib/ofp-print.c:617: warning: array subscript has type 'char' Reported-by: YAMAMOTO Takashi <yamamoto@valinux.co.jp>
* vswitchd: Document how to disable inactivity probes.Ben Pfaff2011-04-141-0/+1
| | | | | | This has always been implemented but it was not documented until now. Reported-by: Alex Yip <alex@nicira.com>
* AUTHORS: Add Gaetano Catalli <gaetano.catalli@gmail.com>.Ben Pfaff2011-03-301-0/+1
|
* AUTHORS: Add Valient Gough.Jesse Gross2011-02-251-0/+1
|
* jsonrpc.py: Import "ovs.json".Justin Pettit2011-02-211-0/+1
| | | | Reported-by: Giuseppe de Candia <giuseppe.decandia@gmail.com>
* vconn-stream: Make definitions of tcp_vconn_class, etc., easier to find.Ben Pfaff2011-02-151-0/+1
| | | | | | | | | When object definitions are entirely hidden inside macros with token pasting tricks, tools like "tags" and even "grep" cannot find them. This change makes the definitions of tcp_vconn_class and other stream-based vconns easier to find, by exposing their definitions to these tools. Reported-by: Yongqiang Liu <liuyq7809@gmail.com>
* dpif-netdev: Fix segfault handling packets.Ben Pfaff2011-02-151-0/+1
| | | | Reported-by: Hassan Khan <hassan.khan@seecs.edu.pk>
* dpif-netdev: Allow for Ethernet and VLAN header in buffer size calculation.Ben Pfaff2011-01-241-0/+1
| | | | | | | This is a long-standing bug--it was present in version 1.0 too. Reported-by: Gaetano Catalli <gaetano.catalli@gmail.com> Solution by Jesse Gross <jesse@nicira.com>
* datapath: Avoid backported csum_replace4() on RHEL 5.Jesse Gross2011-01-191-0/+1
| | | | | | Reported-by: Koichi Yagishita <yagishita.koichi@jrc.co.jp> Signed-off-by: Jesse Gross <jesse@nicira.com> Acked-by: Ben Pfaff <blp@nicira.com>
* netdev: Make 'netdev' parameter of 'get_features()' const.Andrew Evans2011-01-171-0/+1
| | | | Implementations shouldn't need to modify it.
* debian: Modify ovs-bugtool to include collectd rrd logsSajjad Lateef2011-01-131-1/+1
| | | | | | | | | | | | | | | | collectd is a utility that stores rrd logs in the directory /var/lib/collectd/rrd/ by default. rrd logs are created when collectd is configured to output logs in Round Robin Database format. With this change, ovs-bugtool will include collectd logs in the output bundle from /var/lib/collectd/rrd. The output bundle will include all sub-directories of the specified collectd directory. ovs-bugtool checks if a directory exists before it attempts to include the directory contents in the output bundle. So, running ovs-bugtool on a system that does not have collectd logs should not negatively affect the behavior of ovs-bugtool.
* tests: Fix Y2011 bug in testsuite.Ben Pfaff2011-01-101-0/+1
| | | | | | | | | | The tests have been failing for a few days now, because the PKI expired a few days into 2011. This commit instead generates the PKI at "make check" time, which has the additional benefit of getting some test exposure for the ovs-pki program. Reported-by: Aaron M. Ucko <ucko@debian.org> CC: 609506@bugs.debian.org
* AUTHORS: Add Krishna Miriyala as reporter.Jesse Gross2011-01-051-0/+1
|
* INSTALL.Linux: Make pkg-config a requirement to build.Ben Pfaff2010-12-221-0/+1
| | | | | | | | | | pkg-config is needed to detect the presence of libssl, so it is needed even if the build system doesn't need to be rebuilt. In theory we'd only need pkg-config if OpenSSL is to be used, but I don't see much point in being that explicit. Reported-by: Bryan Osoro <bosoro@nicira.com>
* Implement stress option framework.Ben Pfaff2010-11-301-0/+1
| | | | | | | | | | | | | | | | Stress options allow developers testing Open vSwitch to trigger behavior that otherwise would occur only in corner cases. Developers and testers can thereby more easily discover bugs that would otherwise manifest only rarely or nondeterministically. Stress options may cause surprising behavior even when they do not actually reveal bugs, so they should only be enabled as part of testing Open vSwitch. This commit implements the framework and adds a few example stress options. This commit started from code written by Andrew Lambeth. Suggested-by: Henrik Amren <henrik@nicira.com> CC: Andrew Lambeth <wal@nicira.com>
* configure: Check for pkg-config "openssl" library instead of "libssl".Ben Pfaff2010-11-291-0/+1
| | | | | | | | Checking for libssl only does not necessarily link against libcrypto, so the build can gratuitously fail. Reported-by: Dave Walker <DaveWalker@ubuntu.com> Ubuntu bug #680439.
* ovs-vsctl: Fix br-set-external-id.Ben Pfaff2010-11-181-0/+1
| | | | | | | | | | | | | The wrong prerequisite function was being called for set-br-external-id, so it didn't set up to monitor the right columns, so an assertion failed. This fixes the problem. Also, change the test for external IDs so that it will find the regression. (Running all of the ovs-vsctl commands together caused the column to be monitored due to another command's intervention.) Reported-by: Mikael Doverhag <mdoverhag@nicira.com> Bug #4095.
* socket-util: Work around Unix domain socket path name limits on Linux.Ben Pfaff2010-11-101-0/+3
| | | | | | | | | | | | | | | | | | Many Open vSwitch tests fail on Debian's automatic build machines because the builds occur in deeply nested directories with long names. OVS tries to bind and connect to Unix domain sockets using absolute path names, which in combination with long directory names means that the socket's name exceeds the limit for Unix domain socket names (108 bytes on Linux). This commit works around the problem on Linux by indirecting through /proc/self/fd/<dirfd>/<basename> when names exceed the maximum that can be used directly. Reported-by: Hector Oron <hector.oron@gmail.com> Reported-by: Sebastian Andrzej Siewior <sebastian@breakpoint.cc> Reported-by: Roger Leigh <rleigh@codelibre.net> Debian bug #602891. Debian bug #602911.
* AUTHORS: Add Romain Lenglet.Jesse Gross2010-11-041-0/+1
|
* ovsdb-client: Add JSON output format.Ben Pfaff2010-11-031-0/+1
| | | | | Requested-by: DK Moon <dkmoon@nicira.com> CC: DK Moon <dkmoon@nicira.com>
* Fix bug in parsing large datapath IDs.Ben Pfaff2010-10-211-0/+1
| | | | | | | | | | Datapath IDs are unsigned, but dpid_from_string() was parsing them as signed integers, which caused high values to be capped at INT64_MAX. Also, document that datapath IDs may not be all-zero. Reported-by: Pankaj Thakkar <thakkar@nicira.com> Reported-by: Teemu Koponen <koponen@nicira.com>
* docs: Make it clear that ovs-openflowd is an alternative to ovs-vswitchd.Ben Pfaff2010-10-141-0/+1
| | | | Suggested-by: Vishal Swarankar <vishal.swarnkar@gmail.com>
* ovs-openflowd: Don't segfault when no controllers specifiedJustin Pettit2010-10-071-0/+1
| | | | | | | | | | If no controllers are specified on the command-line, ovs-openflowd adds a couple of its own. The code that accounts for the controllers correctly allocated space for them, but used the command-line count to determine how many to set. This led to a segfault when later code tried to dereference them. Reported-by: Derek Cormier <derek.cormier@lab.ntt.co.jp>
* debian: Use DODTIME instead of DIETIME in init scriptsJustin Pettit2010-09-231-0/+1
| | | | | | | | | | | | Brad pointed out that openvswitch-ipsec init script defined the variable DIETIME but attempted to use it as DODTIME. This commit uses DODTIME, since it's the name used by the openvswitch-switch init script. The openvswitch-controller init script had the same issue. As suggested by Ben, the "s" suffixes are removed from sleep commands, since they are a GNU extension. Reported-by: Brad Hall <brad@nicira.com>
* AUTHORS: Add Alexey I. Froloff as reporter.Jesse Gross2010-09-171-0/+1
|
* debian: Init script should put core dumps in an appropriate placeEthan Jackson2010-09-151-0/+1
| | | | | | | Before this commit the init script did not change the cwd of openvswitch processes it started. Thus, core files were created in root directory. This patch changes the cwd of openvswitch to a more reasonable location.
* AUTHORS: Add Vivien Bernet-Rollande.Jesse Gross2010-09-161-0/+1
|
* ofp-util: Also accept NXAST_DROP_SPOOFED_ARP.Ben Pfaff2010-09-151-0/+1
| | | | | | Reported-by: Bryan Fulton <bryan@nicira.com> Reported-by: Michael Mao <mmao@nicira.com> Bug #3655.
* AUTHORS: Add Joe Perches.Jesse Gross2010-08-301-0/+1
| | | | Signed-off-by: Jesse Gross <jesse@nicira.com>
* vswitchd: Fix 100% CPU usage with bonds and --fake-proc-net.Ben Pfaff2010-08-261-0/+1
| | | | | | | | | | | | | The current date in milliseconds since the epoch is ~1,282,841,552,000, which is greater than LONG_MAX of 4,294,967,295 on 32-bit systems, so no matter what was stored into bond_next_fake_iface_update, it would always appear to be expired. It really needs to be a 64-bit number. (This was just a typo really.) Since XenServer 5.5 requires --fake-proc-net, this probably fixes an important bug there. Reported-by: Luiz Henrique Ozaki <luiz.ozaki@gmail.com>
* backtrace: Use generic code to find the bottom of the stack.Tsvi Slonim2010-08-201-0/+1
| | | | This fixes an ugly GCC warning without using inline asm.
* socket-util: Remove stray printf() from make_unix_socket().Bryan Phillippe2010-08-201-0/+1
|
View Lorry Controller Status