Merge citrix branch into master.
authorJesse Gross <jesse@nicira.com>
Tue, 10 Nov 2009 23:12:01 +0000 (15:12 -0800)
committerJesse Gross <jesse@nicira.com>
Tue, 10 Nov 2009 23:12:01 +0000 (15:12 -0800)
18 files changed:
1  2 
INSTALL.Linux
datapath/datapath.c
extras/ezio/ovs-switchui.c
lib/dpif-linux.c
lib/vconn.c
ofproto/in-band.c
ofproto/netflow.c
ofproto/netflow.h
ofproto/ofproto.c
ofproto/ofproto.h
utilities/ovs-dpctl.c
utilities/ovs-ofctl.c
utilities/ovs-openflowd.c
vswitchd/bridge.c
vswitchd/ovs-vswitchd.8.in
vswitchd/ovs-vswitchd.conf.5.in
xenserver/opt_xensource_libexec_interface-reconfigure
xenserver/usr_share_vswitch_scripts_dump-vif-details

diff --cc INSTALL.Linux
Simple merge
@@@ -1011,18 -1016,22 +1011,18 @@@ static int del_flow(struct datapath *dp
        if (!flow)
                goto error;
  
 -      if (cmd == ODP_FLOW_DEL) {
 -              /* XXX redundant lookup */
 -              error = dp_table_delete(table, flow);
 -              if (error)
 -                      goto error;
 +      /* XXX redundant lookup */
 +      error = dp_table_delete(table, flow);
 +      if (error)
 +              goto error;
  
 -              /* XXX These statistics might lose a few packets, since other
 -               * CPUs can be using this flow.  We used to synchronize_rcu()
 -               * to make sure that we get completely accurate stats, but that
 -               * blows our performance, badly. */
 -              dp->n_flows--;
 -              error = answer_query(flow, 0, ufp);
 -              flow_deferred_free(flow);
 -      } else {
 -              error = answer_query(flow, uf.flags, ufp);
 -      }
 +      /* XXX These statistics might lose a few packets, since other CPUs can
 +       * be using this flow.  We used to synchronize_rcu() to make sure that
 +       * we get completely accurate stats, but that blows our performance,
 +       * badly. */
 +      dp->n_flows--;
-       error = answer_query(flow, uf.flags, ufp);
++      error = answer_query(flow, 0, ufp);
 +      flow_deferred_free(flow);
  
  error:
        return error;
@@@ -1044,9 -1054,9 +1044,9 @@@ static int query_flows(struct datapath 
  
                flow = dp_table_lookup(table, &uf.key);
                if (!flow)
 -                      error = __clear_user(&ufp->stats, sizeof ufp->stats);
 +                      error = __put_user(ENOENT, &ufp->stats.error);
                else
--                      error = answer_query(flow, 0, ufp);
++                      error = answer_query(flow, uf.flags, ufp);
                if (error)
                        return -EFAULT;
        }
Simple merge
index 8216d18,0000000..2bf329f
mode 100644,000000..100644
--- /dev/null
@@@ -1,762 -1,0 +1,762 @@@
-                          "from %zu bytes to %d",
 +/*
 + * Copyright (c) 2008, 2009 Nicira Networks.
 + *
 + * Licensed under the Apache License, Version 2.0 (the "License");
 + * you may not use this file except in compliance with the License.
 + * You may obtain a copy of the License at:
 + *
 + *     http://www.apache.org/licenses/LICENSE-2.0
 + *
 + * Unless required by applicable law or agreed to in writing, software
 + * distributed under the License is distributed on an "AS IS" BASIS,
 + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 + * See the License for the specific language governing permissions and
 + * limitations under the License.
 + */
 +
 +#include <config.h>
 +#include "dpif.h"
 +
 +#include <assert.h>
 +#include <ctype.h>
 +#include <errno.h>
 +#include <fcntl.h>
 +#include <inttypes.h>
 +#include <net/if.h>
 +#include <linux/ethtool.h>
 +#include <linux/rtnetlink.h>
 +#include <linux/sockios.h>
 +#include <stdlib.h>
 +#include <sys/ioctl.h>
 +#include <unistd.h>
 +
 +#include "dpif-provider.h"
 +#include "ofpbuf.h"
 +#include "poll-loop.h"
 +#include "rtnetlink.h"
 +#include "svec.h"
 +#include "util.h"
 +
 +#include "vlog.h"
 +#define THIS_MODULE VLM_dpif_linux
 +
 +/* Datapath interface for the openvswitch Linux kernel module. */
 +struct dpif_linux {
 +    struct dpif dpif;
 +    int fd;
 +
 +    /* Used by dpif_linux_get_all_names(). */
 +    char *local_ifname;
 +    int minor;
 +
 +    /* Change notification. */
 +    int local_ifindex;          /* Ifindex of local port. */
 +    struct svec changed_ports;  /* Ports that have changed. */
 +    struct rtnetlink_notifier port_notifier;
 +    bool change_error;
 +};
 +
 +static struct vlog_rate_limit error_rl = VLOG_RATE_LIMIT_INIT(9999, 5);
 +
 +static int do_ioctl(const struct dpif *, int cmd, const void *arg);
 +static int lookup_minor(const char *name, int *minor);
 +static int finish_open(struct dpif *, const char *local_ifname);
 +static int get_openvswitch_major(void);
 +static int create_minor(const char *name, int minor, struct dpif **dpifp);
 +static int open_minor(int minor, struct dpif **dpifp);
 +static int make_openvswitch_device(int minor, char **fnp);
 +static void dpif_linux_port_changed(const struct rtnetlink_change *,
 +                                    void *dpif);
 +
 +static struct dpif_linux *
 +dpif_linux_cast(const struct dpif *dpif)
 +{
 +    dpif_assert_class(dpif, &dpif_linux_class);
 +    return CONTAINER_OF(dpif, struct dpif_linux, dpif);
 +}
 +
 +static int
 +dpif_linux_enumerate(struct svec *all_dps)
 +{
 +    int major;
 +    int error;
 +    int i;
 +
 +    /* Check that the Open vSwitch module is loaded. */
 +    major = get_openvswitch_major();
 +    if (major < 0) {
 +        return -major;
 +    }
 +
 +    error = 0;
 +    for (i = 0; i < ODP_MAX; i++) {
 +        struct dpif *dpif;
 +        char devname[16];
 +        int retval;
 +
 +        sprintf(devname, "dp%d", i);
 +        retval = dpif_open(devname, &dpif);
 +        if (!retval) {
 +            svec_add(all_dps, devname);
 +            dpif_close(dpif);
 +        } else if (retval != ENODEV && !error) {
 +            error = retval;
 +        }
 +    }
 +    return error;
 +}
 +
 +static int
 +dpif_linux_open(const char *name UNUSED, char *suffix, bool create,
 +                struct dpif **dpifp)
 +{
 +    int minor;
 +
 +    minor = !strncmp(name, "dp", 2)
 +            && isdigit((unsigned char)name[2]) ? atoi(name + 2) : -1;
 +    if (create) {
 +        if (minor >= 0) {
 +            return create_minor(suffix, minor, dpifp);
 +        } else {
 +            /* Scan for unused minor number. */
 +            for (minor = 0; minor < ODP_MAX; minor++) {
 +                int error = create_minor(suffix, minor, dpifp);
 +                if (error != EBUSY) {
 +                    return error;
 +                }
 +            }
 +
 +            /* All datapath numbers in use. */
 +            return ENOBUFS;
 +        }
 +    } else {
 +        struct dpif_linux *dpif;
 +        struct odp_port port;
 +        int error;
 +
 +        if (minor < 0) {
 +            error = lookup_minor(suffix, &minor);
 +            if (error) {
 +                return error;
 +            }
 +        }
 +
 +        error = open_minor(minor, dpifp);
 +        if (error) {
 +            return error;
 +        }
 +        dpif = dpif_linux_cast(*dpifp);
 +
 +        /* We need the local port's ifindex for the poll function.  Start by
 +         * getting the local port's name. */
 +        memset(&port, 0, sizeof port);
 +        port.port = ODPP_LOCAL;
 +        if (ioctl(dpif->fd, ODP_PORT_QUERY, &port)) {
 +            error = errno;
 +            if (error != ENODEV) {
 +                VLOG_WARN("%s: probe returned unexpected error: %s",
 +                          dpif_name(*dpifp), strerror(error));
 +            }
 +            dpif_close(*dpifp);
 +            return error;
 +        }
 +
 +        /* Then use that to finish up opening. */
 +        return finish_open(&dpif->dpif, port.devname);
 +    }
 +}
 +
 +static void
 +dpif_linux_close(struct dpif *dpif_)
 +{
 +    struct dpif_linux *dpif = dpif_linux_cast(dpif_);
 +    rtnetlink_notifier_unregister(&dpif->port_notifier);
 +    svec_destroy(&dpif->changed_ports);
 +    free(dpif->local_ifname);
 +    close(dpif->fd);
 +    free(dpif);
 +}
 +
 +static int
 +dpif_linux_get_all_names(const struct dpif *dpif_, struct svec *all_names)
 +{
 +    struct dpif_linux *dpif = dpif_linux_cast(dpif_);
 +
 +    svec_add_nocopy(all_names, xasprintf("dp%d", dpif->minor));
 +    svec_add(all_names, dpif->local_ifname);
 +    return 0;
 +}
 +
 +static int
 +dpif_linux_delete(struct dpif *dpif_)
 +{
 +    return do_ioctl(dpif_, ODP_DP_DESTROY, NULL);
 +}
 +
 +static int
 +dpif_linux_get_stats(const struct dpif *dpif_, struct odp_stats *stats)
 +{
 +    return do_ioctl(dpif_, ODP_DP_STATS, stats);
 +}
 +
 +static int
 +dpif_linux_get_drop_frags(const struct dpif *dpif_, bool *drop_fragsp)
 +{
 +    int drop_frags;
 +    int error;
 +
 +    error = do_ioctl(dpif_, ODP_GET_DROP_FRAGS, &drop_frags);
 +    if (!error) {
 +        *drop_fragsp = drop_frags & 1;
 +    }
 +    return error;
 +}
 +
 +static int
 +dpif_linux_set_drop_frags(struct dpif *dpif_, bool drop_frags)
 +{
 +    int drop_frags_int = drop_frags;
 +    return do_ioctl(dpif_, ODP_SET_DROP_FRAGS, &drop_frags_int);
 +}
 +
 +static int
 +dpif_linux_port_add(struct dpif *dpif_, const char *devname, uint16_t flags,
 +                    uint16_t *port_no)
 +{
 +    struct odp_port port;
 +    int error;
 +
 +    memset(&port, 0, sizeof port);
 +    strncpy(port.devname, devname, sizeof port.devname);
 +    port.flags = flags;
 +    error = do_ioctl(dpif_, ODP_PORT_ADD, &port);
 +    if (!error) {
 +        *port_no = port.port;
 +    }
 +    return error;
 +}
 +
 +static int
 +dpif_linux_port_del(struct dpif *dpif_, uint16_t port_no)
 +{
 +    int tmp = port_no;
 +    return do_ioctl(dpif_, ODP_PORT_DEL, &tmp);
 +}
 +
 +static int
 +dpif_linux_port_query_by_number(const struct dpif *dpif_, uint16_t port_no,
 +                          struct odp_port *port)
 +{
 +    memset(port, 0, sizeof *port);
 +    port->port = port_no;
 +    return do_ioctl(dpif_, ODP_PORT_QUERY, port);
 +}
 +
 +static int
 +dpif_linux_port_query_by_name(const struct dpif *dpif_, const char *devname,
 +                              struct odp_port *port)
 +{
 +    memset(port, 0, sizeof *port);
 +    strncpy(port->devname, devname, sizeof port->devname);
 +    return do_ioctl(dpif_, ODP_PORT_QUERY, port);
 +}
 +
 +static int
 +dpif_linux_flow_flush(struct dpif *dpif_)
 +{
 +    return do_ioctl(dpif_, ODP_FLOW_FLUSH, NULL);
 +}
 +
 +static int
 +dpif_linux_port_list(const struct dpif *dpif_, struct odp_port *ports, int n)
 +{
 +    struct odp_portvec pv;
 +    int error;
 +
 +    pv.ports = ports;
 +    pv.n_ports = n;
 +    error = do_ioctl(dpif_, ODP_PORT_LIST, &pv);
 +    return error ? -error : pv.n_ports;
 +}
 +
 +static int
 +dpif_linux_port_poll(const struct dpif *dpif_, char **devnamep)
 +{
 +    struct dpif_linux *dpif = dpif_linux_cast(dpif_);
 +
 +    if (dpif->change_error) {
 +        dpif->change_error = false;
 +        svec_clear(&dpif->changed_ports);
 +        return ENOBUFS;
 +    } else if (dpif->changed_ports.n) {
 +        *devnamep = dpif->changed_ports.names[--dpif->changed_ports.n];
 +        return 0;
 +    } else {
 +        return EAGAIN;
 +    }
 +}
 +
 +static void
 +dpif_linux_port_poll_wait(const struct dpif *dpif_)
 +{
 +    struct dpif_linux *dpif = dpif_linux_cast(dpif_);
 +    if (dpif->changed_ports.n || dpif->change_error) {
 +        poll_immediate_wake();
 +    } else {
 +        rtnetlink_notifier_wait();
 +    }
 +}
 +
 +static int
 +dpif_linux_port_group_get(const struct dpif *dpif_, int group,
 +                          uint16_t ports[], int n)
 +{
 +    struct odp_port_group pg;
 +    int error;
 +
 +    assert(n <= UINT16_MAX);
 +    pg.group = group;
 +    pg.ports = ports;
 +    pg.n_ports = n;
 +    error = do_ioctl(dpif_, ODP_PORT_GROUP_GET, &pg);
 +    return error ? -error : pg.n_ports;
 +}
 +
 +static int
 +dpif_linux_port_group_set(struct dpif *dpif_, int group,
 +                          const uint16_t ports[], int n)
 +{
 +    struct odp_port_group pg;
 +
 +    assert(n <= UINT16_MAX);
 +    pg.group = group;
 +    pg.ports = (uint16_t *) ports;
 +    pg.n_ports = n;
 +    return do_ioctl(dpif_, ODP_PORT_GROUP_SET, &pg);
 +}
 +
 +static int
 +dpif_linux_flow_get(const struct dpif *dpif_, struct odp_flow flows[], int n)
 +{
 +    struct odp_flowvec fv;
 +    fv.flows = flows;
 +    fv.n_flows = n;
 +    return do_ioctl(dpif_, ODP_FLOW_GET, &fv);
 +}
 +
 +static int
 +dpif_linux_flow_put(struct dpif *dpif_, struct odp_flow_put *put)
 +{
 +    return do_ioctl(dpif_, ODP_FLOW_PUT, put);
 +}
 +
 +static int
 +dpif_linux_flow_del(struct dpif *dpif_, struct odp_flow *flow)
 +{
 +    return do_ioctl(dpif_, ODP_FLOW_DEL, flow);
 +}
 +
 +static int
 +dpif_linux_flow_list(const struct dpif *dpif_, struct odp_flow flows[], int n)
 +{
 +    struct odp_flowvec fv;
 +    int error;
 +
 +    fv.flows = flows;
 +    fv.n_flows = n;
 +    error = do_ioctl(dpif_, ODP_FLOW_LIST, &fv);
 +    return error ? -error : fv.n_flows;
 +}
 +
 +static int
 +dpif_linux_execute(struct dpif *dpif_, uint16_t in_port,
 +                   const union odp_action actions[], int n_actions,
 +                   const struct ofpbuf *buf)
 +{
 +    struct odp_execute execute;
 +    memset(&execute, 0, sizeof execute);
 +    execute.in_port = in_port;
 +    execute.actions = (union odp_action *) actions;
 +    execute.n_actions = n_actions;
 +    execute.data = buf->data;
 +    execute.length = buf->size;
 +    return do_ioctl(dpif_, ODP_EXECUTE, &execute);
 +}
 +
 +static int
 +dpif_linux_recv_get_mask(const struct dpif *dpif_, int *listen_mask)
 +{
 +    return do_ioctl(dpif_, ODP_GET_LISTEN_MASK, listen_mask);
 +}
 +
 +static int
 +dpif_linux_recv_set_mask(struct dpif *dpif_, int listen_mask)
 +{
 +    return do_ioctl(dpif_, ODP_SET_LISTEN_MASK, &listen_mask);
 +}
 +
 +static int
 +dpif_linux_recv(struct dpif *dpif_, struct ofpbuf **bufp)
 +{
 +    struct dpif_linux *dpif = dpif_linux_cast(dpif_);
 +    struct ofpbuf *buf;
 +    int retval;
 +    int error;
 +
 +    buf = ofpbuf_new(65536);
 +    retval = read(dpif->fd, ofpbuf_tail(buf), ofpbuf_tailroom(buf));
 +    if (retval < 0) {
 +        error = errno;
 +        if (error != EAGAIN) {
 +            VLOG_WARN_RL(&error_rl, "%s: read failed: %s",
 +                         dpif_name(dpif_), strerror(error));
 +        }
 +    } else if (retval >= sizeof(struct odp_msg)) {
 +        struct odp_msg *msg = buf->data;
 +        if (msg->length <= retval) {
 +            buf->size += retval;
 +            *bufp = buf;
 +            return 0;
 +        } else {
 +            VLOG_WARN_RL(&error_rl, "%s: discarding message truncated "
++                         "from %"PRIu32" bytes to %d",
 +                         dpif_name(dpif_), msg->length, retval);
 +            error = ERANGE;
 +        }
 +    } else if (!retval) {
 +        VLOG_WARN_RL(&error_rl, "%s: unexpected end of file", dpif_name(dpif_));
 +        error = EPROTO;
 +    } else {
 +        VLOG_WARN_RL(&error_rl,
 +                     "%s: discarding too-short message (%d bytes)",
 +                     dpif_name(dpif_), retval);
 +        error = ERANGE;
 +    }
 +
 +    *bufp = NULL;
 +    ofpbuf_delete(buf);
 +    return error;
 +}
 +
 +static void
 +dpif_linux_recv_wait(struct dpif *dpif_)
 +{
 +    struct dpif_linux *dpif = dpif_linux_cast(dpif_);
 +    poll_fd_wait(dpif->fd, POLLIN);
 +}
 +
 +const struct dpif_class dpif_linux_class = {
 +    "",                         /* This is the default class. */
 +    "linux",
 +    NULL,
 +    NULL,
 +    dpif_linux_enumerate,
 +    dpif_linux_open,
 +    dpif_linux_close,
 +    dpif_linux_get_all_names,
 +    dpif_linux_delete,
 +    dpif_linux_get_stats,
 +    dpif_linux_get_drop_frags,
 +    dpif_linux_set_drop_frags,
 +    dpif_linux_port_add,
 +    dpif_linux_port_del,
 +    dpif_linux_port_query_by_number,
 +    dpif_linux_port_query_by_name,
 +    dpif_linux_port_list,
 +    dpif_linux_port_poll,
 +    dpif_linux_port_poll_wait,
 +    dpif_linux_port_group_get,
 +    dpif_linux_port_group_set,
 +    dpif_linux_flow_get,
 +    dpif_linux_flow_put,
 +    dpif_linux_flow_del,
 +    dpif_linux_flow_flush,
 +    dpif_linux_flow_list,
 +    dpif_linux_execute,
 +    dpif_linux_recv_get_mask,
 +    dpif_linux_recv_set_mask,
 +    dpif_linux_recv,
 +    dpif_linux_recv_wait,
 +};
 +\f
 +static int get_openvswitch_major(void);
 +static int get_major(const char *target);
 +
 +static int
 +do_ioctl(const struct dpif *dpif_, int cmd, const void *arg)
 +{
 +    struct dpif_linux *dpif = dpif_linux_cast(dpif_);
 +    return ioctl(dpif->fd, cmd, arg) ? errno : 0;
 +}
 +
 +static int
 +lookup_minor(const char *name, int *minorp)
 +{
 +    struct ethtool_drvinfo drvinfo;
 +    int minor, port_no;
 +    struct ifreq ifr;
 +    int error;
 +    int sock;
 +
 +    sock = socket(AF_INET, SOCK_DGRAM, 0);
 +    if (sock < 0) {
 +        VLOG_WARN("socket(AF_INET) failed: %s", strerror(errno));
 +        error = errno;
 +        goto error;
 +    }
 +
 +    memset(&ifr, 0, sizeof ifr);
 +    strncpy(ifr.ifr_name, name, sizeof ifr.ifr_name);
 +    ifr.ifr_data = (caddr_t) &drvinfo;
 +
 +    memset(&drvinfo, 0, sizeof drvinfo);
 +    drvinfo.cmd = ETHTOOL_GDRVINFO;
 +    if (ioctl(sock, SIOCETHTOOL, &ifr)) {
 +        VLOG_WARN("ioctl(SIOCETHTOOL) failed: %s", strerror(errno));
 +        error = errno;
 +        goto error_close_sock;
 +    }
 +
 +    if (strcmp(drvinfo.driver, "openvswitch")) {
 +        VLOG_WARN("%s is not an openvswitch device", name);
 +        error = EOPNOTSUPP;
 +        goto error_close_sock;
 +    }
 +
 +    if (sscanf(drvinfo.bus_info, "%d.%d", &minor, &port_no) != 2) {
 +        VLOG_WARN("%s ethtool bus_info has unexpected format", name);
 +        error = EPROTOTYPE;
 +        goto error_close_sock;
 +    } else if (port_no != ODPP_LOCAL) {
 +        /* This is an Open vSwitch device but not the local port.  We
 +         * intentionally support only using the name of the local port as the
 +         * name of a datapath; otherwise, it would be too difficult to
 +         * enumerate all the names of a datapath. */
 +        error = EOPNOTSUPP;
 +        goto error_close_sock;
 +    }
 +
 +    *minorp = minor;
 +    close(sock);
 +    return 0;
 +
 +error_close_sock:
 +    close(sock);
 +error:
 +    return error;
 +}
 +
 +static int
 +make_openvswitch_device(int minor, char **fnp)
 +{
 +    const char dirname[] = "/dev/net";
 +    int major;
 +    dev_t dev;
 +    struct stat s;
 +    char fn[128];
 +
 +    major = get_openvswitch_major();
 +    if (major < 0) {
 +        return -major;
 +    }
 +    dev = makedev(major, minor);
 +
 +    *fnp = NULL;
 +    sprintf(fn, "%s/dp%d", dirname, minor);
 +    if (!stat(fn, &s)) {
 +        if (!S_ISCHR(s.st_mode)) {
 +            VLOG_WARN_RL(&error_rl, "%s is not a character device, fixing",
 +                         fn);
 +        } else if (s.st_rdev != dev) {
 +            VLOG_WARN_RL(&error_rl,
 +                         "%s is device %u:%u but should be %u:%u, fixing",
 +                         fn, major(s.st_rdev), minor(s.st_rdev),
 +                         major(dev), minor(dev));
 +        } else {
 +            goto success;
 +        }
 +        if (unlink(fn)) {
 +            VLOG_WARN_RL(&error_rl, "%s: unlink failed (%s)",
 +                         fn, strerror(errno));
 +            return errno;
 +        }
 +    } else if (errno == ENOENT) {
 +        if (stat(dirname, &s)) {
 +            if (errno == ENOENT) {
 +                if (mkdir(dirname, 0755)) {
 +                    VLOG_WARN_RL(&error_rl, "%s: mkdir failed (%s)",
 +                                 dirname, strerror(errno));
 +                    return errno;
 +                }
 +            } else {
 +                VLOG_WARN_RL(&error_rl, "%s: stat failed (%s)",
 +                             dirname, strerror(errno));
 +                return errno;
 +            }
 +        }
 +    } else {
 +        VLOG_WARN_RL(&error_rl, "%s: stat failed (%s)", fn, strerror(errno));
 +        return errno;
 +    }
 +
 +    /* The device needs to be created. */
 +    if (mknod(fn, S_IFCHR | 0700, dev)) {
 +        VLOG_WARN_RL(&error_rl,
 +                     "%s: creating character device %u:%u failed (%s)",
 +                     fn, major(dev), minor(dev), strerror(errno));
 +        return errno;
 +    }
 +
 +success:
 +    *fnp = xstrdup(fn);
 +    return 0;
 +}
 +
 +/* Return the major device number of the Open vSwitch device.  If it
 + * cannot be determined, a negative errno is returned. */
 +static int
 +get_openvswitch_major(void)
 +{
 +    static int openvswitch_major = -1;
 +    if (openvswitch_major < 0) {
 +        openvswitch_major = get_major("openvswitch");
 +    }
 +    return openvswitch_major;
 +}
 +
 +static int
 +get_major(const char *target)
 +{
 +    const char fn[] = "/proc/devices";
 +    char line[128];
 +    FILE *file;
 +    int ln;
 +
 +    file = fopen(fn, "r");
 +    if (!file) {
 +        VLOG_ERR("opening %s failed (%s)", fn, strerror(errno));
 +        return -errno;
 +    }
 +
 +    for (ln = 1; fgets(line, sizeof line, file); ln++) {
 +        char name[64];
 +        int major;
 +
 +        if (!strncmp(line, "Character", 9) || line[0] == '\0') {
 +            /* Nothing to do. */
 +        } else if (!strncmp(line, "Block", 5)) {
 +            /* We only want character devices, so skip the rest of the file. */
 +            break;
 +        } else if (sscanf(line, "%d %63s", &major, name)) {
 +            if (!strcmp(name, target)) {
 +                fclose(file);
 +                return major;
 +            }
 +        } else {
 +            static bool warned;
 +            if (!warned) {
 +                VLOG_WARN("%s:%d: syntax error", fn, ln);
 +            }
 +            warned = true;
 +        }
 +    }
 +
 +    VLOG_ERR("%s: %s major not found (is the module loaded?)", fn, target);
 +    return -ENODEV;
 +}
 +
 +static int
 +finish_open(struct dpif *dpif_, const char *local_ifname)
 +{
 +    struct dpif_linux *dpif = dpif_linux_cast(dpif_);
 +    dpif->local_ifname = strdup(local_ifname);
 +    dpif->local_ifindex = if_nametoindex(local_ifname);
 +    if (!dpif->local_ifindex) {
 +        int error = errno;
 +        dpif_close(dpif_);
 +        VLOG_WARN("could not get ifindex of %s device: %s",
 +                  local_ifname, strerror(errno));
 +        return error;
 +    }
 +    return 0;
 +}
 +
 +static int
 +create_minor(const char *name, int minor, struct dpif **dpifp)
 +{
 +    int error = open_minor(minor, dpifp);
 +    if (!error) {
 +        error = do_ioctl(*dpifp, ODP_DP_CREATE, name);
 +        if (!error) {
 +            error = finish_open(*dpifp, name);
 +        } else {
 +            dpif_close(*dpifp);
 +        }
 +    }
 +    return error;
 +}
 +
 +static int
 +open_minor(int minor, struct dpif **dpifp)
 +{
 +    int error;
 +    char *fn;
 +    int fd;
 +
 +    error = make_openvswitch_device(minor, &fn);
 +    if (error) {
 +        return error;
 +    }
 +
 +    fd = open(fn, O_RDONLY | O_NONBLOCK);
 +    if (fd >= 0) {
 +        struct dpif_linux *dpif = xmalloc(sizeof *dpif);
 +        error = rtnetlink_notifier_register(&dpif->port_notifier,
 +                                           dpif_linux_port_changed, dpif);
 +        if (!error) {
 +            char *name;
 +
 +            name = xasprintf("dp%d", minor);
 +            dpif_init(&dpif->dpif, &dpif_linux_class, name, minor, minor);
 +            free(name);
 +
 +            dpif->fd = fd;
 +            dpif->local_ifname = NULL;
 +            dpif->minor = minor;
 +            dpif->local_ifindex = 0;
 +            svec_init(&dpif->changed_ports);
 +            dpif->change_error = false;
 +            *dpifp = &dpif->dpif;
 +        } else {
 +            free(dpif);
 +        }
 +    } else {
 +        error = errno;
 +        VLOG_WARN("%s: open failed (%s)", fn, strerror(error));
 +    }
 +    free(fn);
 +
 +    return error;
 +}
 +
 +static void
 +dpif_linux_port_changed(const struct rtnetlink_change *change, void *dpif_)
 +{
 +    struct dpif_linux *dpif = dpif_;
 +
 +    if (change) {
 +        if (change->master_ifindex == dpif->local_ifindex
 +            && (change->nlmsg_type == RTM_NEWLINK
 +                || change->nlmsg_type == RTM_DELLINK))
 +        {
 +            /* Our datapath changed, either adding a new port or deleting an
 +             * existing one. */
 +            if (!svec_contains(&dpif->changed_ports, change->ifname)) {
 +                svec_add(&dpif->changed_ports, change->ifname);
 +                svec_sort(&dpif->changed_ports);
 +            }
 +        }
 +    } else {
 +        dpif->change_error = true;
 +    }
 +}
diff --cc lib/vconn.c
Simple merge
Simple merge
Simple merge
Simple merge
@@@ -1690,8 -1691,8 +1694,8 @@@ rule_uninstall(struct ofproto *p, struc
          odp_flow.key = rule->cr.flow;
          odp_flow.actions = NULL;
          odp_flow.n_actions = 0;
 -        if (!dpif_flow_del(&p->dpif, &odp_flow)) {
 +        if (!dpif_flow_del(p->dpif, &odp_flow)) {
-             update_stats(rule, &odp_flow.stats);
+             update_stats(p, rule, &odp_flow.stats);
          }
          rule->installed = false;
  
@@@ -3271,6 -3308,42 +3313,42 @@@ expire_rule(struct cls_rule *cls_rule, 
      rule_remove(p, rule);
  }
  
 -            dpif_flow_get(&ofproto->dpif, &odp_flow);
+ static void
+ active_timeout(struct ofproto *ofproto, struct rule *rule)
+ {
+     if (ofproto->netflow && !is_controller_rule(rule) &&
+         netflow_active_timeout_expired(ofproto->netflow, &rule->nf_flow)) {
+         struct ofexpired expired;
+         struct odp_flow odp_flow;
+         /* Get updated flow stats. */
+         memset(&odp_flow, 0, sizeof odp_flow);
+         if (rule->installed) {
+             odp_flow.key = rule->cr.flow;
+             odp_flow.flags = ODPFF_ZERO_TCP_FLAGS;
++            dpif_flow_get(ofproto->dpif, &odp_flow);
+             if (odp_flow.stats.n_packets) {
+                 update_time(ofproto, rule, &odp_flow.stats);
+                 netflow_flow_update_flags(&rule->nf_flow, odp_flow.stats.ip_tos,
+                                           odp_flow.stats.tcp_flags);
+             }
+         }
+         expired.flow = rule->cr.flow;
+         expired.packet_count = rule->packet_count +
+                                odp_flow.stats.n_packets;
+         expired.byte_count = rule->byte_count + odp_flow.stats.n_bytes;
+         expired.used = rule->used;
+         netflow_expire(ofproto->netflow, &rule->nf_flow, &expired);
+         /* Schedule us to send the accumulated records once we have
+          * collected all of them. */
+         poll_immediate_wake();
+     }
+ }
  static void
  update_used(struct ofproto *p)
  {
Simple merge
Simple merge
Simple merge
Simple merge
@@@ -43,7 -43,6 +43,8 @@@
  #include "odp-util.h"
  #include "ofp-print.h"
  #include "ofpbuf.h"
++#include "ofproto/netflow.h"
 +#include "ofproto/ofproto.h"
  #include "packets.h"
  #include "poll-loop.h"
  #include "port-array.h"
@@@ -567,22 -494,41 +569,20 @@@ bridge_reconfigure(void
      LIST_FOR_EACH (br, struct bridge, node, &all_bridges) {
          uint8_t ea[8];
          uint64_t dpid;
 -        struct iface *local_iface = NULL;
 -        const char *devname;
 +        struct iface *local_iface;
 +        struct iface *hw_addr_iface;
-         uint8_t engine_type, engine_id;
-         bool add_id_to_iface = false;
-         struct svec nf_hosts;
+         struct netflow_options nf_options;
  
          bridge_fetch_dp_ifaces(br);
 -        for (i = 0; i < br->n_ports; ) {
 -            struct port *port = br->ports[i];
 +        iterate_and_prune_ifaces(br, init_iface_netdev, NULL);
  
 -            for (j = 0; j < port->n_ifaces; ) {
 -                struct iface *iface = port->ifaces[j];
 -                if (iface->dp_ifidx < 0) {
 -                    VLOG_ERR("%s interface not in dp%u, dropping",
 -                             iface->name, dpif_id(&br->dpif));
 -                    iface_destroy(iface);
 -                } else {
 -                    if (iface->dp_ifidx == ODPP_LOCAL) {
 -                        local_iface = iface;
 -                    }
 -                    VLOG_DBG("dp%u has interface %s on port %d",
 -                             dpif_id(&br->dpif), iface->name, iface->dp_ifidx);
 -                    j++;
 -                }
 -            }
 -            if (!port->n_ifaces) {
 -                VLOG_ERR("%s port has no interfaces, dropping", port->name);
 -                port_destroy(port);
 -                continue;
 -            }
 -            i++;
 -        }
 +        iterate_and_prune_ifaces(br, check_iface_dp_ifidx, NULL);
  
          /* Pick local port hardware address, datapath ID. */
 -        bridge_pick_local_hw_addr(br, ea, &devname);
 +        bridge_pick_local_hw_addr(br, ea, &hw_addr_iface);
 +        local_iface = bridge_get_local_iface(br);
          if (local_iface) {
 -            int error = netdev_nodev_set_etheraddr(local_iface->name, ea);
 +            int error = netdev_set_etheraddr(local_iface->netdev, ea);
              if (error) {
                  static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(1, 5);
                  VLOG_ERR_RL(&rl, "bridge %s: failed to set bridge "
          ofproto_set_datapath_id(br->ofproto, dpid);
  
          /* Set NetFlow configuration on this bridge. */
-         dpif_get_netflow_ids(br->dpif, &engine_type, &engine_id);
+         memset(&nf_options, 0, sizeof nf_options);
 -        nf_options.engine_type = br->dpif.minor;
 -        nf_options.engine_id = br->dpif.minor;
++        dpif_get_netflow_ids(br->dpif, &nf_options.engine_type,
++                             &nf_options.engine_id);
+         nf_options.active_timeout = -1;
          if (cfg_has("netflow.%s.engine-type", br->name)) {
-             engine_type = cfg_get_int(0, "netflow.%s.engine-type", 
+             nf_options.engine_type = cfg_get_int(0, "netflow.%s.engine-type", 
                      br->name);
          }
          if (cfg_has("netflow.%s.engine-id", br->name)) {
@@@ -899,30 -859,8 +907,28 @@@ bridge_flush(struct bridge *br
  {
      COVERAGE_INC(bridge_flush);
      br->flush = true;
-     if (br->ml) {
-         mac_learning_flush(br->ml);
-     }
+     mac_learning_flush(br->ml);
  }
 +
 +/* Returns the 'br' interface for the ODPP_LOCAL port, or null if 'br' has no
 + * such interface. */
 +static struct iface *
 +bridge_get_local_iface(struct bridge *br)
 +{
 +    size_t i, j;
 +
 +    for (i = 0; i < br->n_ports; i++) {
 +        struct port *port = br->ports[i];
 +        for (j = 0; j < port->n_ifaces; j++) {
 +            struct iface *iface = port->ifaces[j];
 +            if (iface->dp_ifidx == ODPP_LOCAL) {
 +                return iface;
 +            }
 +        }
 +    }
 +
 +    return NULL;
 +}
  \f
  /* Bridge unixctl user interface functions. */
  static void
Simple merge
@@@ -419,12 -423,18 +422,18 @@@ collector in the form \fIip\fB:\fIport\
  will be sent to each \fIip\fR on UDP \fIport\fR.  The \fIip\fR must
  be specified numerically, not as a DNS name.
  
- The NetFlow messages will use the datapath index for the engine type and id.  
- This can be overridden with the \fBnetflow.\fIbridge\fB.engine-type\fR and 
+ In addition to terminating flows, NetFlow can also send records at a set
+ interval for flows that are still active.  This interval can be configured
+ by defining the key \fBnetflow.\fIbridge\fB\.active-timeout\fR.  The value
+ is in seconds.  An active timeout of 0 will disable this functionality.  By
+ default there is timeout value of 600 seconds.
+ The NetFlow messages will use the datapath index for the engine type and id.
+ This can be overridden with the \fBnetflow.\fIbridge\fB.engine-type\fR and
  \fBnetflow.\fIbridge\fB.engine-id\fR, respectively.  Each takes a value
- between 0 and 255, inclusive. 
+ between 0 and 255, inclusive.
  
 -Many NetFlow collectors do not expect multiple virtual switches to be
 +Many NetFlow collectors do not expect multiple switches to be
  sending messages from the same host, and they do not store the engine
  information which could be used to disambiguate the traffic.  To prevent
  flows from multiple switches appearing as if they came on the interface,