Don't use separate asynchronous event connection for user datapath.

Commit 14439fa80c, "Maintain separate async and sync connections to nl:0
in secchan," modified secchan to use two separate connections to the
datapath, one for asynchronous events, one for requests and replies.  This
technique doesn't work for the user datapath, which always sends
asynchronous events on all its connections.  Fortunately, it isn't
necessary for the user datapath, either, because the user datapath is
smart enough not to drop message replies.

Tested by Justin.

Add support for exporting flow information in NetFlow v5 format.

This is implemented by having the datapath send a new meta-Flow End message
that contains all the information needed by NetFlow v5 and the OpenFlow
Flow Expiration messages.  secchan grabs these Flow End messages and
generates any requested Flow End and NetFlow messages.  The Flow End
message is implemented as a Nicira vendor extension, but it is only used
internally between the datapath and secchan, so the switch is still fully
compatible with OpenFlow v0.8.9.

NOTE: This change has not been ported to "switch", which means that it is
not able to generate NetFlow messages.  "switch" is no longer maintained
and will be removed from the repository on January 1, 2009.

Fix usage line about specifying local datapath in secchan.

Improve formatting of process termination messages in secchan logging.

My expectation was that strsignal() returns the signal's name, e.g.
SIGTERM.  It actually returns an English explanation, so that the existing
code would log a message like "terminated by signal Terminated".  This
commit changes the message to the more sensible "terminated by signal 15
(Terminated)".

Also, the recently approved POSIX 2008 standardized strsignal() and in
particular says that it may return NULL for unknown signal numbers, so
this commit fixes the behavior on NULL return.

vlog: Optimize logging disabled, rate-limited messages.

Previously the arguments of rate-limited messages were always evaluated
and passed to vlog_rate_limit().  This commit short-circuits the case
where the vlog levels would prevent any messages from being logged even
without considering rate limiting.

Reduce rate limit for short-packet messages even further.

These messages appear a lot in a virtualized environment, and they really
are not meaningful there.

Fix typo in comment.

Add reminder comment to lib/learning-switch.c.

(This doesn't change the semantics of the code at all.)

Fix mangled header guards in ofp-print.h.

Improve secchan.8 manpage.

Fix typo in error message.

Style fix: f(x) is better than f((x))

Add userspace datapath to openflow.

Update required Autoconf version in INSTALL.

The AC_PREREQ was updated some time ago but the documentation lagged
behind.

Make ds_put_char_multiple() use the provided character instead of a space.

Fortunately this function was only used for formatting log messages, in
a context where spaces were the normal padding character.

Found by Chris Eagle.

Cleanup some formatting in openflow.h.

Change references in comments from "ofp_vendor" to "ofp_vendor_header".

Update prerequisite Autoconf version to 2.60 (from 2.59).

We use the MKDIR_P variable that only Autoconf 2.60 and
later supports, but we were only requiring Autoconf 2.59
or later.  This fixes the problem.

Also, we were not explicitly invoking AC_PROG_MKDIR_P from
configure.ac, even though we were using $(MKDIR_P).  Some
other macro must have been internally invoking
AC_PROG_MKDIR_P, but it's not good to depend on that, so
this commit adds an explicit call to it.

Fix compatibility back to Linux 2.6.15 and 2.4.32.

Add missing "%s" to VLOG_WARN_RL call.

Thanks to Justin for pointing out the problem.

Officially call this code version 0.8.9~1.

Add support for listing and deleting entries based on an output port.

To support this, an "out_port" field has been added to the "ofp_flow_mod",
"ofp_flow_stats_request", and "ofp_aggregate_stats_request" messages.  If an
"out_port" contains a value other than "OFPP_NONE", it introduces a constraint
when matching.  This constraint is that the rule must contain an output action
directed at that port.  Other constraints such as ofp_match structs and
priorities are still used; this is purely an *additional* constraint.  Note
that to get previous behavior, though, "out_port" must be set to "OFPP_NONE",
since "0" is a valid port id.  This only applies to the delete and
delete_strict flow mod commands; the field is ignored by add, modify, and
modify_strict.

Add DATAPATH_ID to switch packaging, to enable persistent DPIDs.

Allow the datapath ID to be configured by changing the ofX MAC address.

Previously, you could change the MAC address but that had no effect on
the datapath ID, so that there was no way to change the datapath ID at all.

Simplify code in stream_recv().

There is no need to test whether retval is nonzero at this point in the
code: we already know that it is negative from the two previous "if"
branches.

Make dpif_close() accept a null pointer.

It's customary for destructor functions to accept and ignore null pointers.

This commit does not fix any known bug.

rconn_destroy() should close monitoring connections, to avoid a leak.

Our current code base never closes an rconn that has a monitoring
connection, so this does not fix any larger problem.

Free txbuf, rxbuf in vconn-ssl and vconn-stream destructors.

This bug was causing secchan to leak a little bit of memory on almost every
connection close.  This is most visible when openflow-monitor is running,
because openflow-monitor connects to secchan once per second.

Add support for understanding ICMP type and code in flow entries.

Maintain separate async and sync connections to nl:0 in secchan.

When a network link is flooded with traffic, secchan's OpenFlow socket
queue becomes congested with traffic.  This leaves no room for replies
to OpenFlow requests relayed to that socket by secchan.

This commit modifies secchan to use separate sockets for asynchronous
traffic and for OpenFlow requests/replies, which should avoid the problem.

Dynamically allocate switch status categories.

I got tired of increasing the statically allocated number of categories
whenever we exceeded it, so this will make things simpler.

Get rid of unused parameter to rate_limit_start().

Don't rate-limit packet_ins from flows that the controller set up.

Prevent accidentally passing an integer value to IP_ARGS.

Fix printing of IP addresses in ofp_print_action().

IP_ARGS takes a pointer, not a raw IP address.

Thanks to kk yap for reporting the problem.

Use "%zu" to print size_t, not "%"PRIu16.

Copy skbs when we save them.

Simplify code for constructing skb in recv_packet_out().

Fix double-free: NF_HOOK sometimes frees the sk_buff passed in.

NF_HOOK is supposed to *always* consume the sk_buff passed in, either
internally or through the okfn argument.  We assumed that it never
consumed its sk_buff, which was OK in the case where it called okfn,
since our okfn (snat_pre_route_finish) never freed its sk_buff, but
not when one of the netfilter hooks dropped or stole the packet, because
then we'd assume that it still existed and free it a second time.

The other users of NF_HOOK in this file, in snat_skb() and
snat_skb_finish(), do not need to be fixed because they always pass a
copy of their sk_buff argument to NF_HOOK and expect it to be freed.

No need to test argument of kfree_skb() for non-null.

Never free an skb that has been passed to genlmsg_reply().

genlmsg_reply() always consumes its argument, not just in the success case.

Pull arp header before grabbing the pointer to it.

Pull data into headers properly, and checksum ICMP replies correctly.

Reset mac header in flow_extract(), because it might initially be null.

__alloc_skb() sets the mac header to null, and not all call chains
reset it.

Use skb_copy_bits() to copy data that might not be pulled into headers.

Verify in fwd_port_input() that 'skb' has no destructor.

Verify in execute_actions() that 'skb' is not shared.

Fix datapath make_writable() function.

- We weren't necessarily pulling enough into the headers in the non-shared
  case, since the IP header by itself can be longer than 40 bytes.

- The skb is guaranteed not to have a destructor at this point.

Make datapath's flow_extract() properly pull data into the headers.

Otherwise we might be reading headers that aren't really there.

Add comment.

Don't oops in dp_output_control() for skbs with null ->dev.

This shouldn't ordinarily happen, since normal packets are received from
a real device, but some packets can be spontaneously generated within
the switch and thus have null ->dev.  (Possibly that is itself a bug
that we should track down; not sure.)

Verify in fwd_port_input() that we are not passed packets that are shared.

When a packet arrives via bridging, clone it if it is shared.

Otherwise we will mangle the packet for anyone who came before us
(e.g. tcpdump via AF_PACKET).

Make sure that do_port_input() always puts a MAC header on packets.

Prompted by persistent oopses on packets received by e1000e, in which
skb_mac_header() for the packet always returned null.

In handle_arp_snat() and snat_pre_route() pull enough payload into the headers.

There is no guarantee that the device put any data at all into the header.
The e1000 device, for example, appears to not put any data into the header
when the packet is longer than its configured copybreak value, which is
256 bytes by default.  So we need to do it ourselves.

In update_mapping(), update m->hw_addr unconditionally.

There is no benefit to comparing it first, since we already know that that
cache line will be dirtied by the assignment to m->used.

Factor common SNAT code into new functions.

Make snat_skb() skb argument const.

After all, it doesn't modify it.

Don't process packets in NAT that aren't destined for us.

This time for sure!

Orphan and clone packets transmitted on dp_dev (proper fix for bug #478).

We need to orphan packets transmitted on dp_dev so that any socket that
the packets came from is no longer charged for them.  If we don't do
this, then the sk_buff will be destructed eventually, but it is
harder to predict when.

We need to clone packets because we are sticking them on
&dp_dev->xmit_queue, which is done by modifying the sk_buff, which we
can only do if it is not shared.

Always compile openflow modules with debug information.

Check kmalloc() return value.

Fix typo: = should be ==

Stopgap fix for bug #478, where kernel panics on SNAT to input port.

Fix " is running" and " is not running" messages from openflow-switch init.d

With the "secchan" name in there it makes a lot more sense!

Make openflow-switch depend on dmidecode, procps instead of acpi-support.

acpi-support pulls in a lot of stuff we really don't want or need, such
as various X11-related stuff.  Being more specific reduces our
dependencies.

Add build number to kernel modules built via Debian packaging.

Add --with-build-number configure argument and support in debian/rules.

Add --monitor flag to default init scripts for secchan.

New package openflow-dbg for debugging symbols for the rest of OpenFlow.

Fix typo in code to set core limit that broke the openflow-switch init script.

Add openflow-switch settings for the limit on core files.

Add new Debian package "corekeeper" to centralize and reap core files.

Rename switchmon to switchui.

Ignore openflow-monitor and openflow-wdt build directories.

Add dist_sbin_SCRIPTS make target.

Have git ignore the monitor and wdt packaging links.

When a new OpenFlow kernel module package is installed, restart the switch.

This helps to ensure that the secchan and the kernel module are exactly
the same version.

Unload kernel module when stopping switch, in Debian init scripts.

This way, we have a better chance at ensuring that the secchan and the
kernel module are exactly the same version.

Add required "subst" script to EXTRA_DIST.

Move "update" remote command from base secchan into Debian packaging.

The update command only works with Debian anyway, so there's no point
in installing it elsewhere.

New switch remote command for setting configuration parameters.

Make ofp-switch-setup preserve config variables that it does not set itself.

Before, it would delete them.  This probably kept ofp-switch-setup from
working at all given the changes in openflow-switch.default.  Now, it
should do better (but it has not been tested).

Add "execute" to dpctl usage message.

Fix openflow-switch log rotation.

The vlogconf command syntax was wrong.  The pidfile also didn't contain
the right pid (which was fixed in the previous commit).

Create vlog sockets after daemonizing, so that pidfiles can be used.

Otherwise, pidfiles contain the process ID of the process that forked
and exited, so that "vlogconf --target <program.pid>" will fail.

Dump more detailed information about system state from get-logs.

Use "set -e" command instead of "#! /bin/sh -e" magic.

Otherwise, running the script through an explicit shell invocation, such
as "sh -x <script>" for debugging, will fail to set -e and therefore the
script's behavior will change.

Add missing dependency on openssl to openflow-pki.

Otherwise the ofp-pki script cannot execute it.

Add build number to datapath version and --version output from programs.

In reboot script, send SIGUSR1 to ofp-switchmon to make it display "Rebooting".

Call VLOG_WARN before send_child_message(), to get correct errno value.

Otherwise we end up with nonsense like "failed to stat...: Success" in
the log.

Fix location of installed commands.

Pass --enable-snat to configure for datapath module also.

Make COMMANDS option in openflow-switch defaults file work.

The setting was being accidentally ignored, so commands like "reboot", etc.
would not work at all.

Add libncurses5-dev as build dependency for openflow.

Implement log rotation in openflow-switch.

Make secchan log to file by default.

Add /var/log/openflow directory to openflow-common.

This ensures that every system with OpenFlow packages will have this
directory available for logging.

Send openssl output to logfile on "ofp-pki self-sign" also.