Delete trailing whitespace in vlog-socket.c

Hardcode path to tcpdump as /usr/sbin/tcpdump.

Otherwise it won't be found when run as non-root because such users
typically don't have /usr/sbin in $PATH.  (We use tcpdump in a fashion
that doesn't require root privilege.)

Fix typo in manpage.

Revamp build system to make it easier to integrate openflowext.

secchan: Make sure fail-open doesn't drop the first messages from connection.

Fail-open mode is entered or left in a secchan "periodic" callback, which
may not be called immediately after the connection to the controller
actually comes up.  This can cause the fail-open local-packet CB to
locally handle messages from the datapath and drop them, causing the
controller to think that it's being ignore and drop the connection.

Thanks to Dan for raising the issue.

Rework STP implementation in controller.

Before, we tried to use the port configuration bits to implement STP; e.g.
if a port was in LISTEN state we turned off sending and receiving frame
other than BPDUs.  Unfortunately this interacts very badly with in-band
control: when the switch connects to the controller, it can be connected
to the controller over a port that is not in FORWARD, and so then the
controller disabled that port, and the connection eventually dropped when
an echo request/reply failed to get through.

Now, we implement STP by querying the flows on the switch and killing
off the ones that STP does not allow.  This works much better because,
although we still kill off the in-band control connection, the in-band
hook in secchan is then able to resurrect it.

--no-stp is still the default since this has not been tested very much.

Set DHCP maximum retransmission timeout to 3 seconds in secchan.

This should help discovery complete faster, especially with hops across
multiple OpenFlow switches to the controller.

dhcp-client: Log DHCP messages at higher priority.

Logging DHCP messages sent or received with priority DBG made it hard
to see what was happening.  So log them at priority WARN, but also make
them less verbose at that level since seeing the whole message is not
too useful most of the time.

dhcp-client: Fix computation of timeouts.

The "min_timeout" variable maintained by the DHCP client is measured
in seconds from the time the state was entered, but dhclient_wait()
was interpreting it as seconds from now.  This made the DHCP client wait
much longer than necessary in some cases.

secchan: Log switch datapath id at startup.

New routine ofpbuf_put_zeros() to simplify a common code sequence.

secchan: Remove unhelpful STP-related log message.

secchan: Increase max number of local_cbs to accommodate STP.

Support up to 0xff00 ports in OpenFlow, without changing the implemented max.

No problum any more.

Conditionally leave out a few more things if "--enable-snat" isn't used.

Remove unnecessary check for validity when dereferencing an array.

Thanks to Masa et al. for pointing this out.

Fix dereference of previously freed data.

Thanks to Masa et al. for catching this.

Return error message when a flow can't be added due to full tables.

When a flow cannot be added to any tables because they are full, send
a message with type OFPET_FLOW_MOD_FAILED and code OFPFMFC_ALL_TABLES_FULL.

Allow SNAT to build on older (2.6.15) and new (2.6.26) kernels.

When sending error messages, set the length properly.

rconn: Never report being in failure mode while connected.

Change e10dfcf35, "rconn: Be pickier about what constitutes a successful
connection," caused rconn to consider the connection to have failed even
when we are actually connected until one of several message types was
received.  Unfortunately, that meant that "fail open" would intercept and
discard all messages sent by the controller until one of those messages
was received.  Thus, the controller would never receive a reply to its
feature_request, assume that the connection was busted, and disconnect.
This happened forever, of course.

Fixes bug #242.

Thanks to Reid for reporting this and Dan for help in diagnosis.

rconn: Improve log message when peer closes connection.

Remove OFPT_TABLE reference.

Add support for Source-NAT to Linux 2.6 datapaths.

To enable SNAT, run configure with the "--enable-snat" flag.  This has
only been tested with the 2.6.23 kernel...more diverse testing will follow.
Documentation and a cleaner build setup will also be in a future check-in.

Free sk_buffs with kfree_skb() instead of just kfree().

rconn: Be pickier about what constitutes a successful connection.

When secchan is configured to "fail open" after failing to connect to
a controller for a period of time, it needs a heuristic for what
constitutes a successful connection.  Until now, that heuristic was
simply that when it received an OpenFlow message from the controller
(any OpenFlow message), it considered the connection successful.

However, this is no longer good enough, because NOX performs
admission control on connections after sending a number of OpenFlow
messages, in particular after doing OpenFlow version negotiation and
requesting the switch features (and receiving the reply).  Thus, this
commit adjusts the heuristic by only considering certain OpenFlow
messages to demonstrate that admission control checks have passed and
thus that the connection should be considered successful.

As a fallback, any connection that persists for 30 seconds or longer is
also considered successful.

An alternate and complementary approach (that this commit does not
implement) would be to use an OpenFlow error message to indicate why
the connection is closing.

Fixes bug #239.

Fix inaccurate log message.

Update documentation to talk about userspace datapath.

Remove secchan dependence on Netlink for connecting to the datapath.

Make rconn_disconnect() a no-op if already disconnected.

New function stp_set_bridge_id() to change the bridge ID of a running STP.

New function dhclient_get_netdev().

Implement dhclient_destroy().

This function has always been prototyped in dhcp-client.h, but it had
no users and never got implemented.

Add support for TAP virtual network devices in netdev.

Fix use-after-free error.

rconn_destroy() decrements the n_queued counters that are set up by
rconn_send(), so we need to destroy the rconn before we destroy anything
that used it.

(This system is more error-prone than I imagined.)

Unconditionally set skb->dev in dp_set_origin.

If an invalid port index was used in dp_set_origin, the value of skb->dev
was left unchanged.  This change causes skb->dev to be set to NULL in those
circumstances.  This makes Packet Out and Add Flow messages that use a
buffer id behave like Packet Out without a buffer id.

Properly set in_port in skb for Flow Mod messages.

When a buffer id is placed in a Flow Mod message, the actions are expected to
be executed against the referenced packet.  The kernel implementation was
not setting the input device to what the controller was telling it to use.
Thanks to Natasha for catching this.

In ofp_packet_to_string(), make tcpdump print Ethernet headers also.

Add explanatory comment to make_unix_socket().

No longer necessary to obtain Ethernet header in netdev_send().

New function get_unix_name_len() to simplify code.

Make ofp_error() preserve the value of errno.

Random Ethernet addresses should be private and should not be multicast.

Fix typo in comment.

Fix typo in comment.

Add support for vendor-defined and variable-length actions.

Allow vendors to define their own actions.  Actions were originally fixed-
length, which was a bit constraining.  Actions now contain a length field,
which gives them more flexibility.

Properly allocate flow action blocks in user-space switch.

The user-space switch was allocating blocks to hold actions in flows that
did not include the action header.

Show OpenFlow wire version when pretty printing features reply.

This allows easy detection of version number with "dpctl show".

Remove kernel datapath unit tests.

The tests haven't been seeing much love and are suffering from bit-rot at
this point.  We're removing them, since we have other ways to test at
this point.

Added missing header file.

Modify VLAN actions to support setting both VID and priority.

Stripping VLANs is now done through the OFPAT_STRIP_VLAN action (i.e., you
don't specify a magic value in the generirc VLAN action).  Also, it is now
possible to modify the priority bits associated with the VLAN tag through
the OFPAT_SET_VLAN_PCP action.  The OFPAT_SET_DL_VLAN has been renamed to
OFPAT_SET_VLAN_VID.

Remove unused, obsolete attributes from OpenFlow netlink protocol.

Fix typo in comment.

Avoid assertion failure connecting to unsupported remove OpenFlow host.

After increasing the length of an OpenFlow buffer, we need to update the
length in the OpenFlow header.

Disable STP in secchan by default (but let --stp enable it).

Initial checkin of OpenFlow specification LaTeX source.

This version of the specification does not reflect the current checked in
version of OpenFlow.  Checkins that follow should bring them in-line.  The
TeX version of the spec was produced by Brandon Heller.

stp: Turn off STP on ports that are disabled or have no link.

Fixes a problem that a port that is disabled via "ifconfig down" does
not come back up properly when it is brought back up again with "ifconfig
up", if STP is enabled.

Cleanup printing of ofp_port_mod a bit.

stp: Increase hello time to 2 seconds.

An STP advertisement is sent after every expiry of the hello timer, so
this reduces STP advertisements to once every 2 seconds.

Once a second seemed like too often.

Suggested by Justin.

Make sure to set 'port_no' in ofp_port_mod messages.

First cut of pretty printer of ofp_port_mod messages.

Fix some old references to OFPPFL_ in comments.

Modify OpenFlow commands related to ports to be more expressive.

This new OpenFlow message format provides a cleaner interface and greater
detail and control over ports.  It is now possible to see what features
the switch's port is currently configured as having, what it's advertising,
and what it's capable of handling.  It is also possible to return the
features advertised by the port's peer.

Drop port_watcher support for editing ports.

This was only needed for STP, which doesn't need it anymore.

Get rid of per-port STP implemented bits, by reducing OFPP_MAX to 255.

802.1D Spanning Tree Protocol supports a maximum of 255 ports per bridge,
but OpenFlow supported 256, so 1 port had to not implement STP.  This
gets rid of the problem by reducing the maximum number of ports to 255.

Use device notifier in Linux kernel switch for detecting port status changes.

To report on link status and whether a port is administratively enabled or
disabled, the reference switches poll.  This change uses the Linux kernel's
device notification system, so that polling isn't necessary.

Add the ability to disable the STP implementation.

Fix boot.sh for the case where the extensions repository is not available.

Reported by Justin.

vlogconf: Don't abort when given unknown option.

Slight improvement to printing of ofp_port_status messages.

Added lookup count to ofp_table_stats.

Use "~" in designating beta versions to make version comparisons work.  Also delete "ext" submodule.

Update pointer to "ext" to contain latest changes.

Fix failure to back off when vconn_open() fails immediately.

This typically would happen only with an entirely local vconn,
e.g. connecting to a Unix domain socket vconn.

Update the ChangeLog with the changes since v0.8.1 and designate this v0.9.0-b1.

Suppress Automake warning about overriding distdir.

Reported by Martin.

Ignore generated files in debian/.

Put "+ext" in the distribution filename if OpenFlow extensions are included.

Test Debian packaging at "make distcheck" time.

Also distribute Debian packaging.

Make "controller nl:0" work again, by fixing a layering violation.

The kernel datapath was claiming that it implements STP, which is untrue.
In fact, the secchan implements STP, so if the secchan was not running
the STP support did not work.  Thus, "controller nl:0" would do the
wrong thing.

This makes the secchan edit the responses from the datapath to claim
STP support.

Update ext submodule to make it build with latest OpenFlow.

Added OFPFC_MODIFY_STRICT flow mod command.

This cleans up the handling of modifying existing flows.  There now exists
OFPFC_MODIFY and OFPFC_MODIFY_STRICT commands, which have the same matching
characteristics to their OFPFC_DELETE* counterparts.  When adding a flow that
already exists, the counters and timers are now reset (ie, its treated as a
new flow).  This commit also adds the "--strict" option to dpctl to allow
strict matching for "mod-flows" and "del-flows".

Use RSA instead of DSA by default for the OpenFlow PKI.

Make it easier to bootstrap the PKI for SSL connections in OpenFlow.

ofp-pki: Give controller and switch CA distinctive names.

This makes it possible to tell them apart by looking at the certificates,
instead of having to just know which is which.

Change STP parameters to the minimum values allowed by 802.1D.

This makes the secchan start up much faster.

Add -Wdeclaration-after-statement to CFLAGS.

This should help us avoid writing non-portable code.

Fix "make distcheck".

Ignore some more generated files.

Fix OpenFlow extensions submodule to build again.

Merge commit 'origin/master'

Conflicts:

switch/datapath.c
utilities/dpctl.c

Merge branch "partner", to simplify partner integration.

Add support for OFPFC_MODIFY Flow Mod command.

The OFPFC_MODIFY command allows the controller to modify the actions of
existing flows.  When it does this, it does not reset counters or timers.

Ignore hwtable_dummy.c symlink in kernel build directories.

Implement OFPT_HELLO simple version negotiation.

Break passive vconns out into separate pvconn routines and data structures.

There really was nothing in common between the active and passive vconns.
This arrangement makes more sense.

Make "struct vconn" opaque.

Fix handling of port flags.

The "port watcher" in secchan was not keeping track of updates to port
flags by the controller, which caused some combinations of secchan
and controller to not turn on ports when the STP state otherwise should
have done so.

Thanks to Justin for reporting the problem.