lib: Move vlog.h to <openvswitch/vlog.h>
[cascardo/ovs.git] / lib / dpif-netlink.c
index 76041e7..63bbddc 100644 (file)
 #include "timeval.h"
 #include "unaligned.h"
 #include "util.h"
-#include "vlog.h"
+#include "openvswitch/vlog.h"
 
 VLOG_DEFINE_THIS_MODULE(dpif_netlink);
+#ifdef _WIN32
+enum { WINDOWS = 1 };
+#else
+enum { WINDOWS = 0 };
+#endif
 enum { MAX_PORTS = USHRT_MAX };
 
 /* This ethtool flag was introduced in Linux 2.6.24, so it might be
@@ -111,10 +116,14 @@ struct dpif_netlink_flow {
     size_t mask_len;
     const struct nlattr *actions;       /* OVS_FLOW_ATTR_ACTIONS. */
     size_t actions_len;
+    ovs_u128 ufid;                      /* OVS_FLOW_ATTR_FLOW_ID. */
+    bool ufid_present;                  /* Is there a UFID? */
+    bool ufid_terse;                    /* Skip serializing key/mask/acts? */
     const struct ovs_flow_stats *stats; /* OVS_FLOW_ATTR_STATS. */
     const uint8_t *tcp_flags;           /* OVS_FLOW_ATTR_TCP_FLAGS. */
     const ovs_32aligned_u64 *used;      /* OVS_FLOW_ATTR_USED. */
     bool clear;                         /* OVS_FLOW_ATTR_CLEAR. */
+    bool probe;                         /* OVS_FLOW_ATTR_PROBE. */
 };
 
 static void dpif_netlink_flow_init(struct dpif_netlink_flow *);
@@ -127,8 +136,10 @@ static int dpif_netlink_flow_transact(struct dpif_netlink_flow *request,
                                       struct ofpbuf **bufp);
 static void dpif_netlink_flow_get_stats(const struct dpif_netlink_flow *,
                                         struct dpif_flow_stats *);
-static void dpif_netlink_flow_to_dpif_flow(struct dpif_flow *,
+static void dpif_netlink_flow_to_dpif_flow(struct dpif *, struct dpif_flow *,
                                            const struct dpif_netlink_flow *);
+static bool dpif_netlink_check_ufid__(struct dpif *dpif);
+static bool dpif_netlink_check_ufid(struct dpif *dpif);
 
 /* One of the dpif channels between the kernel and userspace. */
 struct dpif_channel {
@@ -136,12 +147,32 @@ struct dpif_channel {
     long long int last_poll;    /* Last time this channel was polled. */
 };
 
+#ifdef _WIN32
+#define VPORT_SOCK_POOL_SIZE 1
+/* On Windows, there is no native support for epoll.  There are equivalent
+ * interfaces though, that are not used currently.  For simpicity, a pool of
+ * netlink sockets is used.  Each socket is represented by 'struct
+ * dpif_windows_vport_sock'.  Since it is a pool, multiple OVS ports may be
+ * sharing the same socket.  In the future, we can add a reference count and
+ * such fields. */
+struct dpif_windows_vport_sock {
+    struct nl_sock *nl_sock;    /* netlink socket. */
+};
+#endif
+
 struct dpif_handler {
     struct dpif_channel *channels;/* Array of channels for each handler. */
     struct epoll_event *epoll_events;
     int epoll_fd;                 /* epoll fd that includes channel socks. */
     int n_events;                 /* Num events returned by epoll_wait(). */
     int event_offset;             /* Offset into 'epoll_events'. */
+
+#ifdef _WIN32
+    /* Pool of sockets. */
+    struct dpif_windows_vport_sock *vport_sock_pool;
+    size_t last_used_pool_idx; /* Index to aid in allocating a
+                                  socket in the pool to a port. */
+#endif
 };
 
 /* Datapath interface for the openvswitch Linux kernel module. */
@@ -159,6 +190,11 @@ struct dpif_netlink {
     /* Change notification. */
     struct nl_sock *port_notifier; /* vport multicast group subscriber. */
     bool refresh_channels;
+
+    /* If the datapath supports indexing flows using unique identifiers, then
+     * we can reduce the size of netlink messages by omitting fields like the
+     * flow key during flow operations. */
+    bool ufid_supported;
 };
 
 static void report_loss(struct dpif_netlink *, struct dpif_channel *,
@@ -183,6 +219,7 @@ static int dpif_netlink_init(void);
 static int open_dpif(const struct dpif_netlink_dp *, struct dpif **);
 static uint32_t dpif_netlink_port_get_pid(const struct dpif *,
                                           odp_port_t port_no, uint32_t hash);
+static void dpif_netlink_handler_uninit(struct dpif_handler *handler);
 static int dpif_netlink_refresh_channels(struct dpif_netlink *,
                                          uint32_t n_handlers);
 static void dpif_netlink_vport_to_ofpbuf(const struct dpif_netlink_vport *,
@@ -274,6 +311,7 @@ open_dpif(const struct dpif_netlink_dp *dp, struct dpif **dpifp)
               dp->dp_ifindex, dp->dp_ifindex);
 
     dpif->dp_ifindex = dp->dp_ifindex;
+    dpif->ufid_supported = dpif_netlink_check_ufid__(&dpif->dpif);
     *dpifp = &dpif->dpif;
 
     return 0;
@@ -282,7 +320,7 @@ open_dpif(const struct dpif_netlink_dp *dp, struct dpif **dpifp)
 /* Destroys the netlink sockets pointed by the elements in 'socksp'
  * and frees the 'socksp'.  */
 static void
-vport_del_socksp(struct nl_sock **socksp, uint32_t n_socks)
+vport_del_socksp__(struct nl_sock **socksp, uint32_t n_socks)
 {
     size_t i;
 
@@ -296,7 +334,7 @@ vport_del_socksp(struct nl_sock **socksp, uint32_t n_socks)
 /* Creates an array of netlink sockets.  Returns an array of the
  * corresponding pointers.  Records the error in 'error'. */
 static struct nl_sock **
-vport_create_socksp(uint32_t n_socks, int *error)
+vport_create_socksp__(uint32_t n_socks, int *error)
 {
     struct nl_sock **socksp = xzalloc(n_socks * sizeof *socksp);
     size_t i;
@@ -311,11 +349,131 @@ vport_create_socksp(uint32_t n_socks, int *error)
     return socksp;
 
 error:
-    vport_del_socksp(socksp, n_socks);
+    vport_del_socksp__(socksp, n_socks);
 
     return NULL;
 }
 
+#ifdef _WIN32
+static void
+vport_delete_sock_pool(struct dpif_handler *handler)
+    OVS_REQ_WRLOCK(dpif->upcall_lock)
+{
+    if (handler->vport_sock_pool) {
+        uint32_t i;
+        struct dpif_windows_vport_sock *sock_pool =
+            handler->vport_sock_pool;
+
+        for (i = 0; i < VPORT_SOCK_POOL_SIZE; i++) {
+            if (sock_pool[i].nl_sock) {
+                nl_sock_unsubscribe_packets(sock_pool[i].nl_sock);
+                nl_sock_destroy(sock_pool[i].nl_sock);
+                sock_pool[i].nl_sock = NULL;
+            }
+        }
+
+        free(handler->vport_sock_pool);
+        handler->vport_sock_pool = NULL;
+    }
+}
+
+static int
+vport_create_sock_pool(struct dpif_handler *handler)
+    OVS_REQ_WRLOCK(dpif->upcall_lock)
+{
+    struct dpif_windows_vport_sock *sock_pool;
+    size_t i;
+    int error = 0;
+
+    sock_pool = xzalloc(VPORT_SOCK_POOL_SIZE * sizeof *sock_pool);
+    for (i = 0; i < VPORT_SOCK_POOL_SIZE; i++) {
+        error = nl_sock_create(NETLINK_GENERIC, &sock_pool[i].nl_sock);
+        if (error) {
+            goto error;
+        }
+
+        /* Enable the netlink socket to receive packets.  This is equivalent to
+         * calling nl_sock_join_mcgroup() to receive events. */
+        error = nl_sock_subscribe_packets(sock_pool[i].nl_sock);
+        if (error) {
+           goto error;
+        }
+    }
+
+    handler->vport_sock_pool = sock_pool;
+    handler->last_used_pool_idx = 0;
+    return 0;
+
+error:
+    vport_delete_sock_pool(handler);
+    return error;
+}
+
+/* Returns an array pointers to netlink sockets.  The sockets are picked from a
+ * pool. Records the error in 'error'. */
+static struct nl_sock **
+vport_create_socksp_windows(struct dpif_netlink *dpif, int *error)
+    OVS_REQ_WRLOCK(dpif->upcall_lock)
+{
+    uint32_t n_socks = dpif->n_handlers;
+    struct nl_sock **socksp;
+    size_t i;
+
+    ovs_assert(n_socks <= 1);
+    socksp = xzalloc(n_socks * sizeof *socksp);
+
+    /* Pick netlink sockets to use in a round-robin fashion from each
+     * handler's pool of sockets. */
+    for (i = 0; i < n_socks; i++) {
+        struct dpif_handler *handler = &dpif->handlers[i];
+        struct dpif_windows_vport_sock *sock_pool = handler->vport_sock_pool;
+        size_t index = handler->last_used_pool_idx;
+
+        /* A pool of sockets is allocated when the handler is initialized. */
+        if (sock_pool == NULL) {
+            free(socksp);
+            *error = EINVAL;
+            return NULL;
+        }
+
+        ovs_assert(index < VPORT_SOCK_POOL_SIZE);
+        socksp[i] = sock_pool[index].nl_sock;
+        socksp[i] = sock_pool[index].nl_sock;
+        ovs_assert(socksp[i]);
+        index = (index == VPORT_SOCK_POOL_SIZE - 1) ? 0 : index + 1;
+        handler->last_used_pool_idx = index;
+    }
+
+    return socksp;
+}
+
+static void
+vport_del_socksp_windows(struct dpif_netlink *dpif, struct nl_sock **socksp)
+{
+    free(socksp);
+}
+#endif /* _WIN32 */
+
+static struct nl_sock **
+vport_create_socksp(struct dpif_netlink *dpif, int *error)
+{
+#ifdef _WIN32
+    return vport_create_socksp_windows(dpif, error);
+#else
+    return vport_create_socksp__(dpif->n_handlers, error);
+#endif
+}
+
+static void
+vport_del_socksp(struct dpif_netlink *dpif, struct nl_sock **socksp)
+{
+#ifdef _WIN32
+    vport_del_socksp_windows(dpif, socksp);
+#else
+    vport_del_socksp__(socksp, dpif->n_handlers);
+#endif
+}
+
 /* Given the array of pointers to netlink sockets 'socksp', returns
  * the array of corresponding pids. If the 'socksp' is NULL, returns
  * a single-element array of value 0. */
@@ -353,6 +511,7 @@ vport_get_pids(struct dpif_netlink *dpif, uint32_t port_idx,
     if (!dpif->handlers[0].channels[port_idx].sock) {
         return false;
     }
+    ovs_assert(!WINDOWS || dpif->n_handlers <= 1);
 
     pids = xzalloc(dpif->n_handlers * sizeof *pids);
 
@@ -414,11 +573,7 @@ vport_add_channels(struct dpif_netlink *dpif, odp_port_t port_no,
     for (i = 0; i < dpif->n_handlers; i++) {
         struct dpif_handler *handler = &dpif->handlers[i];
 
-#ifdef _WIN32
-        /*
-         * XXX : Map appropiate Windows handle
-         */
-#else
+#ifndef _WIN32
         if (epoll_ctl(handler->epoll_fd, EPOLL_CTL_ADD, nl_sock_fd(socksp[i]),
                       &event) < 0) {
             error = errno;
@@ -433,11 +588,7 @@ vport_add_channels(struct dpif_netlink *dpif, odp_port_t port_no,
 
 error:
     for (j = 0; j < i; j++) {
-#ifdef _WIN32
-        /*
-         * XXX : Map appropiate Windows handle
-         */
-#else
+#ifndef _WIN32
         epoll_ctl(dpif->handlers[j].epoll_fd, EPOLL_CTL_DEL,
                   nl_sock_fd(socksp[j]), NULL);
 #endif
@@ -466,16 +617,11 @@ vport_del_channels(struct dpif_netlink *dpif, odp_port_t port_no)
 
     for (i = 0; i < dpif->n_handlers; i++) {
         struct dpif_handler *handler = &dpif->handlers[i];
-
-#ifdef _WIN32
-        /*
-         * XXX : Map appropiate Windows handle
-         */
-#else
+#ifndef _WIN32
         epoll_ctl(handler->epoll_fd, EPOLL_CTL_DEL,
                   nl_sock_fd(handler->channels[port_idx].sock), NULL);
-#endif
         nl_sock_destroy(handler->channels[port_idx].sock);
+#endif
         handler->channels[port_idx].sock = NULL;
         handler->event_offset = handler->n_events = 0;
     }
@@ -516,13 +662,7 @@ destroy_all_channels(struct dpif_netlink *dpif)
     for (i = 0; i < dpif->n_handlers; i++) {
         struct dpif_handler *handler = &dpif->handlers[i];
 
-#ifdef _WIN32
-        /*
-         * XXX : Map appropiate Windows handle
-         */
-#else
-        close(handler->epoll_fd);
-#endif
+        dpif_netlink_handler_uninit(handler);
         free(handler->epoll_events);
         free(handler->channels);
     }
@@ -560,7 +700,7 @@ dpif_netlink_destroy(struct dpif *dpif_)
     return dpif_netlink_dp_transact(&dp, NULL, NULL);
 }
 
-static void
+static bool
 dpif_netlink_run(struct dpif *dpif_)
 {
     struct dpif_netlink *dpif = dpif_netlink_cast(dpif_);
@@ -571,6 +711,7 @@ dpif_netlink_run(struct dpif *dpif_)
         dpif_netlink_refresh_channels(dpif, dpif->n_handlers);
         fat_rwlock_unlock(&dpif->upcall_lock);
     }
+    return false;
 }
 
 static int
@@ -687,7 +828,7 @@ dpif_netlink_port_add__(struct dpif_netlink *dpif, struct netdev *netdev,
     int error = 0;
 
     if (dpif->handlers) {
-        socksp = vport_create_socksp(dpif->n_handlers, &error);
+        socksp = vport_create_socksp(dpif, &error);
         if (!socksp) {
             return error;
         }
@@ -701,16 +842,14 @@ dpif_netlink_port_add__(struct dpif_netlink *dpif, struct netdev *netdev,
         VLOG_WARN_RL(&error_rl, "%s: cannot create port `%s' because it has "
                      "unsupported type `%s'",
                      dpif_name(&dpif->dpif), name, type);
-        vport_del_socksp(socksp, dpif->n_handlers);
+        vport_del_socksp(dpif, socksp);
         return EINVAL;
     }
     request.name = name;
 
     if (request.type == OVS_VPORT_TYPE_NETDEV) {
 #ifdef _WIN32
-        /*
-         * XXX : Map appropiate Windows handle
-         */
+        /* XXX : Map appropiate Windows handle */
 #else
         netdev_linux_ethtool_set_flag(netdev, ETH_FLAG_LRO, "LRO", false);
 #endif
@@ -739,7 +878,7 @@ dpif_netlink_port_add__(struct dpif_netlink *dpif, struct netdev *netdev,
                       dpif_name(&dpif->dpif), *port_nop);
         }
 
-        vport_del_socksp(socksp, dpif->n_handlers);
+        vport_del_socksp(dpif, socksp);
         goto exit;
     }
 
@@ -755,7 +894,7 @@ dpif_netlink_port_add__(struct dpif_netlink *dpif, struct netdev *netdev,
             request.dp_ifindex = dpif->dp_ifindex;
             request.port_no = *port_nop;
             dpif_netlink_vport_transact(&request, NULL, NULL);
-            vport_del_socksp(socksp, dpif->n_handlers);
+            vport_del_socksp(dpif, socksp);
             goto exit;
         }
     }
@@ -1082,28 +1221,63 @@ dpif_netlink_port_poll_wait(const struct dpif *dpif_)
 }
 
 static void
-dpif_netlink_init_flow_get(const struct dpif_netlink *dpif,
-                           const struct nlattr *key, size_t key_len,
-                           struct dpif_netlink_flow *request)
+dpif_netlink_flow_init_ufid(struct dpif_netlink_flow *request,
+                            const ovs_u128 *ufid, bool terse)
+{
+    if (ufid) {
+        request->ufid = *ufid;
+        request->ufid_present = true;
+    } else {
+        request->ufid_present = false;
+    }
+    request->ufid_terse = terse;
+}
+
+static void
+dpif_netlink_init_flow_get__(const struct dpif_netlink *dpif,
+                             const struct nlattr *key, size_t key_len,
+                             const ovs_u128 *ufid, bool terse,
+                             struct dpif_netlink_flow *request)
 {
     dpif_netlink_flow_init(request);
     request->cmd = OVS_FLOW_CMD_GET;
     request->dp_ifindex = dpif->dp_ifindex;
     request->key = key;
     request->key_len = key_len;
+    dpif_netlink_flow_init_ufid(request, ufid, terse);
+}
+
+static void
+dpif_netlink_init_flow_get(const struct dpif_netlink *dpif,
+                           const struct dpif_flow_get *get,
+                           struct dpif_netlink_flow *request)
+{
+    dpif_netlink_init_flow_get__(dpif, get->key, get->key_len, get->ufid,
+                                 false, request);
 }
 
 static int
-dpif_netlink_flow_get(const struct dpif_netlink *dpif,
-                      const struct nlattr *key, size_t key_len,
-                      struct dpif_netlink_flow *reply, struct ofpbuf **bufp)
+dpif_netlink_flow_get__(const struct dpif_netlink *dpif,
+                        const struct nlattr *key, size_t key_len,
+                        const ovs_u128 *ufid, bool terse,
+                        struct dpif_netlink_flow *reply, struct ofpbuf **bufp)
 {
     struct dpif_netlink_flow request;
 
-    dpif_netlink_init_flow_get(dpif, key, key_len, &request);
+    dpif_netlink_init_flow_get__(dpif, key, key_len, ufid, terse, &request);
     return dpif_netlink_flow_transact(&request, reply, bufp);
 }
 
+static int
+dpif_netlink_flow_get(const struct dpif_netlink *dpif,
+                      const struct dpif_netlink_flow *flow,
+                      struct dpif_netlink_flow *reply, struct ofpbuf **bufp)
+{
+    return dpif_netlink_flow_get__(dpif, flow->key, flow->key_len,
+                                   flow->ufid_present ? &flow->ufid : NULL,
+                                   false, reply, bufp);
+}
+
 static void
 dpif_netlink_init_flow_put(struct dpif_netlink *dpif,
                            const struct dpif_flow_put *put,
@@ -1119,6 +1293,8 @@ dpif_netlink_init_flow_put(struct dpif_netlink *dpif,
     request->key_len = put->key_len;
     request->mask = put->mask;
     request->mask_len = put->mask_len;
+    dpif_netlink_flow_init_ufid(request, put->ufid, false);
+
     /* Ensure that OVS_FLOW_ATTR_ACTIONS will always be included. */
     request->actions = (put->actions
                         ? put->actions
@@ -1127,19 +1303,34 @@ dpif_netlink_init_flow_put(struct dpif_netlink *dpif,
     if (put->flags & DPIF_FP_ZERO_STATS) {
         request->clear = true;
     }
+    if (put->flags & DPIF_FP_PROBE) {
+        request->probe = true;
+    }
     request->nlmsg_flags = put->flags & DPIF_FP_MODIFY ? 0 : NLM_F_CREATE;
 }
 
 static void
-dpif_netlink_init_flow_del(struct dpif_netlink *dpif,
-                           const struct dpif_flow_del *del,
-                           struct dpif_netlink_flow *request)
+dpif_netlink_init_flow_del__(struct dpif_netlink *dpif,
+                             const struct nlattr *key, size_t key_len,
+                             const ovs_u128 *ufid, bool terse,
+                             struct dpif_netlink_flow *request)
 {
     dpif_netlink_flow_init(request);
     request->cmd = OVS_FLOW_CMD_DEL;
     request->dp_ifindex = dpif->dp_ifindex;
-    request->key = del->key;
-    request->key_len = del->key_len;
+    request->key = key;
+    request->key_len = key_len;
+    dpif_netlink_flow_init_ufid(request, ufid, terse);
+}
+
+static void
+dpif_netlink_init_flow_del(struct dpif_netlink *dpif,
+                           const struct dpif_flow_del *del,
+                           struct dpif_netlink_flow *request)
+{
+    return dpif_netlink_init_flow_del__(dpif, del->key, del->key_len,
+                                        del->ufid, dpif->ufid_supported,
+                                        request);
 }
 
 struct dpif_netlink_flow_dump {
@@ -1155,7 +1346,7 @@ dpif_netlink_flow_dump_cast(struct dpif_flow_dump *dump)
 }
 
 static struct dpif_flow_dump *
-dpif_netlink_flow_dump_create(const struct dpif *dpif_)
+dpif_netlink_flow_dump_create(const struct dpif *dpif_, bool terse)
 {
     const struct dpif_netlink *dpif = dpif_netlink_cast(dpif_);
     struct dpif_netlink_flow_dump *dump;
@@ -1168,12 +1359,15 @@ dpif_netlink_flow_dump_create(const struct dpif *dpif_)
     dpif_netlink_flow_init(&request);
     request.cmd = OVS_FLOW_CMD_GET;
     request.dp_ifindex = dpif->dp_ifindex;
+    request.ufid_present = false;
+    request.ufid_terse = terse;
 
     buf = ofpbuf_new(1024);
     dpif_netlink_flow_to_ofpbuf(&request, buf);
     nl_dump_start(&dump->nl_dump, NETLINK_GENERIC, buf);
     ofpbuf_delete(buf);
     atomic_init(&dump->status, 0);
+    dump->up.terse = terse;
 
     return &dump->up;
 }
@@ -1233,16 +1427,24 @@ dpif_netlink_flow_dump_thread_destroy(struct dpif_flow_dump_thread *thread_)
 }
 
 static void
-dpif_netlink_flow_to_dpif_flow(struct dpif_flow *dpif_flow,
-                               const struct dpif_netlink_flow *linux_flow)
-{
-    dpif_flow->key = linux_flow->key;
-    dpif_flow->key_len = linux_flow->key_len;
-    dpif_flow->mask = linux_flow->mask;
-    dpif_flow->mask_len = linux_flow->mask_len;
-    dpif_flow->actions = linux_flow->actions;
-    dpif_flow->actions_len = linux_flow->actions_len;
-    dpif_netlink_flow_get_stats(linux_flow, &dpif_flow->stats);
+dpif_netlink_flow_to_dpif_flow(struct dpif *dpif, struct dpif_flow *dpif_flow,
+                               const struct dpif_netlink_flow *datapath_flow)
+{
+    dpif_flow->key = datapath_flow->key;
+    dpif_flow->key_len = datapath_flow->key_len;
+    dpif_flow->mask = datapath_flow->mask;
+    dpif_flow->mask_len = datapath_flow->mask_len;
+    dpif_flow->actions = datapath_flow->actions;
+    dpif_flow->actions_len = datapath_flow->actions_len;
+    dpif_flow->ufid_present = datapath_flow->ufid_present;
+    if (datapath_flow->ufid_present) {
+        dpif_flow->ufid = datapath_flow->ufid;
+    } else {
+        ovs_assert(datapath_flow->key && datapath_flow->key_len);
+        dpif_flow_hash(dpif, datapath_flow->key, datapath_flow->key_len,
+                       &dpif_flow->ufid);
+    }
+    dpif_netlink_flow_get_stats(datapath_flow, &dpif_flow->stats);
 }
 
 static int
@@ -1261,7 +1463,7 @@ dpif_netlink_flow_dump_next(struct dpif_flow_dump_thread *thread_,
     n_flows = 0;
     while (!n_flows
            || (n_flows < max_flows && ofpbuf_size(&thread->nl_flows))) {
-        struct dpif_netlink_flow linux_flow;
+        struct dpif_netlink_flow datapath_flow;
         struct ofpbuf nl_flow;
         int error;
 
@@ -1271,21 +1473,22 @@ dpif_netlink_flow_dump_next(struct dpif_flow_dump_thread *thread_,
         }
 
         /* Convert the flow to our output format. */
-        error = dpif_netlink_flow_from_ofpbuf(&linux_flow, &nl_flow);
+        error = dpif_netlink_flow_from_ofpbuf(&datapath_flow, &nl_flow);
         if (error) {
             atomic_store_relaxed(&dump->status, error);
             break;
         }
 
-        if (linux_flow.actions) {
-            /* Common case: the flow includes actions. */
-            dpif_netlink_flow_to_dpif_flow(&flows[n_flows++], &linux_flow);
+        if (dump->up.terse || datapath_flow.actions) {
+            /* Common case: we don't want actions, or the flow includes
+             * actions. */
+            dpif_netlink_flow_to_dpif_flow(&dpif->dpif, &flows[n_flows++],
+                                           &datapath_flow);
         } else {
             /* Rare case: the flow does not include actions.  Retrieve this
              * individual flow again to get the actions. */
-            error = dpif_netlink_flow_get(dpif, linux_flow.key,
-                                          linux_flow.key_len, &linux_flow,
-                                          &thread->nl_actions);
+            error = dpif_netlink_flow_get(dpif, &datapath_flow,
+                                          &datapath_flow, &thread->nl_actions);
             if (error == ENOENT) {
                 VLOG_DBG("dumped flow disappeared on get");
                 continue;
@@ -1298,7 +1501,8 @@ dpif_netlink_flow_dump_next(struct dpif_flow_dump_thread *thread_,
 
             /* Save this flow.  Then exit, because we only have one buffer to
              * handle this case. */
-            dpif_netlink_flow_to_dpif_flow(&flows[n_flows++], &linux_flow);
+            dpif_netlink_flow_to_dpif_flow(&dpif->dpif, &flows[n_flows++],
+                                           &datapath_flow);
             break;
         }
     }
@@ -1333,6 +1537,9 @@ dpif_netlink_encode_execute(int dp_ifindex, const struct dpif_execute *d_exec,
 
     nl_msg_put_unspec(buf, OVS_PACKET_ATTR_ACTIONS,
                       d_exec->actions, d_exec->actions_len);
+    if (d_exec->probe) {
+        nl_msg_put_flag(buf, OVS_FLOW_ATTR_PROBE);
+    }
 }
 
 #define MAX_OPS 50
@@ -1400,7 +1607,7 @@ dpif_netlink_operate__(struct dpif_netlink *dpif,
 
         case DPIF_OP_FLOW_GET:
             get = &op->u.flow_get;
-            dpif_netlink_init_flow_get(dpif, get->key, get->key_len, &flow);
+            dpif_netlink_init_flow_get(dpif, get, &flow);
             aux->txn.reply = get->buffer;
             dpif_netlink_flow_to_ofpbuf(&flow, &aux->request);
             break;
@@ -1466,7 +1673,8 @@ dpif_netlink_operate__(struct dpif_netlink *dpif,
 
                 op->error = dpif_netlink_flow_from_ofpbuf(&reply, txn->reply);
                 if (!op->error) {
-                    dpif_netlink_flow_to_dpif_flow(get->flow, &reply);
+                    dpif_netlink_flow_to_dpif_flow(&dpif->dpif, get->flow,
+                                                   &reply);
                 }
             }
             break;
@@ -1493,6 +1701,68 @@ dpif_netlink_operate(struct dpif *dpif_, struct dpif_op **ops, size_t n_ops)
     }
 }
 
+#if _WIN32
+static void
+dpif_netlink_handler_uninit(struct dpif_handler *handler)
+{
+    vport_delete_sock_pool(handler);
+}
+
+static int
+dpif_netlink_handler_init(struct dpif_handler *handler)
+{
+    return vport_create_sock_pool(handler);
+}
+#else
+
+static int
+dpif_netlink_handler_init(struct dpif_handler *handler)
+{
+    handler->epoll_fd = epoll_create(10);
+    return handler->epoll_fd < 0 ? errno : 0;
+}
+
+static void
+dpif_netlink_handler_uninit(struct dpif_handler *handler)
+{
+    close(handler->epoll_fd);
+}
+#endif
+
+/* Checks support for unique flow identifiers. */
+static bool
+dpif_netlink_check_ufid__(struct dpif *dpif_)
+{
+    struct flow flow;
+    struct odputil_keybuf keybuf;
+    struct ofpbuf key;
+    ovs_u128 ufid;
+
+    memset(&flow, 0, sizeof flow);
+    flow.dl_type = htons(0x1234);
+
+    ofpbuf_use_stack(&key, &keybuf, sizeof keybuf);
+    odp_flow_key_from_flow(&key, &flow, NULL, 0, true);
+    dpif_flow_hash(dpif_, ofpbuf_data(&key), ofpbuf_size(&key), &ufid);
+
+    return dpif_probe_feature(dpif_, "UFID", &key, &ufid);
+}
+
+static bool
+dpif_netlink_check_ufid(struct dpif *dpif_)
+{
+    const struct dpif_netlink *dpif = dpif_netlink_cast(dpif_);
+
+    if (dpif->ufid_supported) {
+        VLOG_INFO("%s: Datapath supports userspace flow ids",
+                  dpif_name(dpif_));
+    } else {
+        VLOG_INFO("%s: Datapath does not support userspace flow ids",
+                  dpif_name(dpif_));
+    }
+    return dpif->ufid_supported;
+}
+
 /* Synchronizes 'channels' in 'dpif->handlers'  with the set of vports
  * currently in 'dpif' in the kernel, by adding a new set of channels for
  * any kernel vport that lacks one and deleting any channels that have no
@@ -1510,30 +1780,30 @@ dpif_netlink_refresh_channels(struct dpif_netlink *dpif, uint32_t n_handlers)
     int retval = 0;
     size_t i;
 
+    ovs_assert(!WINDOWS || n_handlers <= 1);
+    ovs_assert(!WINDOWS || dpif->n_handlers <= 1);
+
     if (dpif->n_handlers != n_handlers) {
         destroy_all_channels(dpif);
         dpif->handlers = xzalloc(n_handlers * sizeof *dpif->handlers);
         for (i = 0; i < n_handlers; i++) {
+            int error;
             struct dpif_handler *handler = &dpif->handlers[i];
 
-#ifdef _WIN32
-        /*
-         * XXX : Map appropiate Windows handle
-         */
-#else
-            handler->epoll_fd = epoll_create(10);
-            if (handler->epoll_fd < 0) {
+            error = dpif_netlink_handler_init(handler);
+            if (error) {
                 size_t j;
+                struct dpif_handler *tmp = &dpif->handlers[i];
+
 
                 for (j = 0; j < i; j++) {
-                    close(dpif->handlers[j].epoll_fd);
+                    dpif_netlink_handler_uninit(tmp);
                 }
                 free(dpif->handlers);
                 dpif->handlers = NULL;
 
-                return errno;
+                return error;
             }
-#endif
         }
         dpif->n_handlers = n_handlers;
     }
@@ -1556,8 +1826,7 @@ dpif_netlink_refresh_channels(struct dpif_netlink *dpif, uint32_t n_handlers)
 
         if (port_no >= dpif->uc_array_size
             || !vport_get_pids(dpif, port_no, &upcall_pids)) {
-            struct nl_sock **socksp = vport_create_socksp(dpif->n_handlers,
-                                                          &error);
+            struct nl_sock **socksp = vport_create_socksp(dpif, &error);
 
             if (!socksp) {
                 goto error;
@@ -1567,7 +1836,7 @@ dpif_netlink_refresh_channels(struct dpif_netlink *dpif, uint32_t n_handlers)
             if (error) {
                 VLOG_INFO("%s: could not add channels for port %s",
                           dpif_name(&dpif->dpif), vport.name);
-                vport_del_socksp(socksp, dpif->n_handlers);
+                vport_del_socksp(dpif, socksp);
                 retval = error;
                 goto error;
             }
@@ -1662,6 +1931,14 @@ dpif_netlink_handlers_set(struct dpif *dpif_, uint32_t n_handlers)
     struct dpif_netlink *dpif = dpif_netlink_cast(dpif_);
     int error = 0;
 
+#ifdef _WIN32
+    /* Multiple upcall handlers will be supported once kernel datapath supports
+     * it. */
+    if (n_handlers > 1) {
+        return error;
+    }
+#endif
+
     fat_rwlock_wrlock(&dpif->upcall_lock);
     if (dpif->handlers) {
         error = dpif_netlink_refresh_channels(dpif, n_handlers);
@@ -1684,8 +1961,8 @@ dpif_netlink_queue_to_priority(const struct dpif *dpif OVS_UNUSED,
 }
 
 static int
-parse_odp_packet(struct ofpbuf *buf, struct dpif_upcall *upcall,
-                 int *dp_ifindex)
+parse_odp_packet(const struct dpif_netlink *dpif, struct ofpbuf *buf,
+                 struct dpif_upcall *upcall, int *dp_ifindex)
 {
     static const struct nl_policy ovs_packet_policy[] = {
         /* Always present. */
@@ -1729,6 +2006,7 @@ parse_odp_packet(struct ofpbuf *buf, struct dpif_upcall *upcall,
     upcall->key = CONST_CAST(struct nlattr *,
                              nl_attr_get(a[OVS_PACKET_ATTR_KEY]));
     upcall->key_len = nl_attr_get_size(a[OVS_PACKET_ATTR_KEY]);
+    dpif_flow_hash(&dpif->dpif, upcall->key, upcall->key_len, &upcall->ufid);
     upcall->userdata = a[OVS_PACKET_ATTR_USERDATA];
     upcall->out_tun_key = a[OVS_PACKET_ATTR_EGRESS_TUN_KEY];
 
@@ -1747,6 +2025,73 @@ parse_odp_packet(struct ofpbuf *buf, struct dpif_upcall *upcall,
     return 0;
 }
 
+#ifdef _WIN32
+#define PACKET_RECV_BATCH_SIZE 50
+static int
+dpif_netlink_recv_windows(struct dpif_netlink *dpif, uint32_t handler_id,
+                          struct dpif_upcall *upcall, struct ofpbuf *buf)
+    OVS_REQ_RDLOCK(dpif->upcall_lock)
+{
+    struct dpif_handler *handler;
+    int read_tries = 0;
+    struct dpif_windows_vport_sock *sock_pool;
+    uint32_t i;
+
+    if (!dpif->handlers) {
+        return EAGAIN;
+    }
+
+    /* Only one handler is supported currently. */
+    if (handler_id >= 1) {
+        return EAGAIN;
+    }
+
+    if (handler_id >= dpif->n_handlers) {
+        return EAGAIN;
+    }
+
+    handler = &dpif->handlers[handler_id];
+    sock_pool = handler->vport_sock_pool;
+
+    for (i = 0; i < VPORT_SOCK_POOL_SIZE; i++) {
+        for (;;) {
+            int dp_ifindex;
+            int error;
+
+            if (++read_tries > PACKET_RECV_BATCH_SIZE) {
+                return EAGAIN;
+            }
+
+            error = nl_sock_recv(sock_pool[i].nl_sock, buf, false);
+            if (error == ENOBUFS) {
+                /* ENOBUFS typically means that we've received so many
+                 * packets that the buffer overflowed.  Try again
+                 * immediately because there's almost certainly a packet
+                 * waiting for us. */
+                /* XXX: report_loss(dpif, ch, idx, handler_id); */
+                continue;
+            }
+
+            /* XXX: ch->last_poll = time_msec(); */
+            if (error) {
+                if (error == EAGAIN) {
+                    break;
+                }
+                return error;
+            }
+
+            error = parse_odp_packet(dpif, buf, upcall, &dp_ifindex);
+            if (!error && dp_ifindex == dpif->dp_ifindex) {
+                return 0;
+            } else if (error) {
+                return error;
+            }
+        }
+    }
+
+    return EAGAIN;
+}
+#else
 static int
 dpif_netlink_recv__(struct dpif_netlink *dpif, uint32_t handler_id,
                     struct dpif_upcall *upcall, struct ofpbuf *buf)
@@ -1765,15 +2110,11 @@ dpif_netlink_recv__(struct dpif_netlink *dpif, uint32_t handler_id,
 
         handler->event_offset = handler->n_events = 0;
 
-#ifdef _WIN32
-        retval = dpif->uc_array_size;
-        handler->event_offset = 0;
-#else
         do {
             retval = epoll_wait(handler->epoll_fd, handler->epoll_events,
                                 dpif->uc_array_size, 0);
         } while (retval < 0 && errno == EINTR);
-#endif
+
         if (retval < 0) {
             static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(1, 1);
             VLOG_WARN_RL(&rl, "epoll_wait failed (%s)", ovs_strerror(errno));
@@ -1814,7 +2155,7 @@ dpif_netlink_recv__(struct dpif_netlink *dpif, uint32_t handler_id,
                 return error;
             }
 
-            error = parse_odp_packet(buf, upcall, &dp_ifindex);
+            error = parse_odp_packet(dpif, buf, upcall, &dp_ifindex);
             if (!error && dp_ifindex == dpif->dp_ifindex) {
                 return 0;
             } else if (error) {
@@ -1825,6 +2166,7 @@ dpif_netlink_recv__(struct dpif_netlink *dpif, uint32_t handler_id,
 
     return EAGAIN;
 }
+#endif
 
 static int
 dpif_netlink_recv(struct dpif *dpif_, uint32_t handler_id,
@@ -1834,7 +2176,11 @@ dpif_netlink_recv(struct dpif *dpif_, uint32_t handler_id,
     int error;
 
     fat_rwlock_rdlock(&dpif->upcall_lock);
+#ifdef _WIN32
+    error = dpif_netlink_recv_windows(dpif, handler_id, upcall, buf);
+#else
     error = dpif_netlink_recv__(dpif, handler_id, upcall, buf);
+#endif
     fat_rwlock_unlock(&dpif->upcall_lock);
 
     return error;
@@ -1845,9 +2191,18 @@ dpif_netlink_recv_wait__(struct dpif_netlink *dpif, uint32_t handler_id)
     OVS_REQ_RDLOCK(dpif->upcall_lock)
 {
 #ifdef _WIN32
-        /*
-         * XXX : Map appropiate Windows handle
-         */
+    uint32_t i;
+    struct dpif_windows_vport_sock *sock_pool =
+        dpif->handlers[handler_id].vport_sock_pool;
+
+    /* Only one handler is supported currently. */
+    if (handler_id >= 1) {
+        return;
+    }
+
+    for (i = 0; i < VPORT_SOCK_POOL_SIZE; i++) {
+        nl_sock_wait(sock_pool[i].nl_sock, POLLIN);
+    }
 #else
     if (dpif->handlers && handler_id < dpif->n_handlers) {
         struct dpif_handler *handler = &dpif->handlers[handler_id];
@@ -1896,6 +2251,36 @@ dpif_netlink_recv_purge(struct dpif *dpif_)
     fat_rwlock_unlock(&dpif->upcall_lock);
 }
 
+static char *
+dpif_netlink_get_datapath_version(void)
+{
+    char *version_str = NULL;
+
+#ifdef __linux__
+
+#define MAX_VERSION_STR_SIZE 80
+#define LINUX_DATAPATH_VERSION_FILE  "/sys/module/openvswitch/version"
+    FILE *f;
+
+    f = fopen(LINUX_DATAPATH_VERSION_FILE, "r");
+    if (f) {
+        char *newline;
+        char version[MAX_VERSION_STR_SIZE];
+
+        if (fgets(version, MAX_VERSION_STR_SIZE, f)) {
+            newline = strchr(version, '\n');
+            if (newline) {
+                *newline = '\0';
+            }
+            version_str = xstrdup(version);
+        }
+        fclose(f);
+    }
+#endif
+
+    return version_str;
+}
+
 const struct dpif_class dpif_netlink_class = {
     "system",
     dpif_netlink_enumerate,
@@ -1925,6 +2310,7 @@ const struct dpif_class dpif_netlink_class = {
     dpif_netlink_operate,
     dpif_netlink_recv_set,
     dpif_netlink_handlers_set,
+    NULL,                       /* poll_thread_set */
     dpif_netlink_queue_to_priority,
     dpif_netlink_recv,
     dpif_netlink_recv_wait,
@@ -1932,6 +2318,8 @@ const struct dpif_class dpif_netlink_class = {
     NULL,                       /* register_upcall_cb */
     NULL,                       /* enable_upcall */
     NULL,                       /* disable_upcall */
+    dpif_netlink_get_datapath_version, /* get_datapath_version */
+    dpif_netlink_check_ufid,
 };
 
 static int
@@ -2318,15 +2706,19 @@ static int
 dpif_netlink_flow_from_ofpbuf(struct dpif_netlink_flow *flow,
                               const struct ofpbuf *buf)
 {
-    static const struct nl_policy ovs_flow_policy[] = {
-        [OVS_FLOW_ATTR_KEY] = { .type = NL_A_NESTED },
+    static const struct nl_policy ovs_flow_policy[__OVS_FLOW_ATTR_MAX] = {
+        [OVS_FLOW_ATTR_KEY] = { .type = NL_A_NESTED, .optional = true },
         [OVS_FLOW_ATTR_MASK] = { .type = NL_A_NESTED, .optional = true },
         [OVS_FLOW_ATTR_ACTIONS] = { .type = NL_A_NESTED, .optional = true },
         [OVS_FLOW_ATTR_STATS] = { NL_POLICY_FOR(struct ovs_flow_stats),
                                   .optional = true },
         [OVS_FLOW_ATTR_TCP_FLAGS] = { .type = NL_A_U8, .optional = true },
         [OVS_FLOW_ATTR_USED] = { .type = NL_A_U64, .optional = true },
+        [OVS_FLOW_ATTR_UFID] = { .type = NL_A_UNSPEC, .optional = true,
+                                 .min_len = sizeof(ovs_u128) },
         /* The kernel never uses OVS_FLOW_ATTR_CLEAR. */
+        /* The kernel never uses OVS_FLOW_ATTR_PROBE. */
+        /* The kernel never uses OVS_FLOW_ATTR_UFID_FLAGS. */
     };
 
     struct nlattr *a[ARRAY_SIZE(ovs_flow_policy)];
@@ -2347,12 +2739,25 @@ dpif_netlink_flow_from_ofpbuf(struct dpif_netlink_flow *flow,
                             ARRAY_SIZE(ovs_flow_policy))) {
         return EINVAL;
     }
+    if (!a[OVS_FLOW_ATTR_KEY] && !a[OVS_FLOW_ATTR_UFID]) {
+        return EINVAL;
+    }
 
     flow->nlmsg_flags = nlmsg->nlmsg_flags;
     flow->dp_ifindex = ovs_header->dp_ifindex;
-    flow->key = nl_attr_get(a[OVS_FLOW_ATTR_KEY]);
-    flow->key_len = nl_attr_get_size(a[OVS_FLOW_ATTR_KEY]);
+    if (a[OVS_FLOW_ATTR_KEY]) {
+        flow->key = nl_attr_get(a[OVS_FLOW_ATTR_KEY]);
+        flow->key_len = nl_attr_get_size(a[OVS_FLOW_ATTR_KEY]);
+    }
+
+    if (a[OVS_FLOW_ATTR_UFID]) {
+        const ovs_u128 *ufid;
 
+        ufid = nl_attr_get_unspec(a[OVS_FLOW_ATTR_UFID],
+                                  nl_attr_get_size(a[OVS_FLOW_ATTR_UFID]));
+        flow->ufid = *ufid;
+        flow->ufid_present = true;
+    }
     if (a[OVS_FLOW_ATTR_MASK]) {
         flow->mask = nl_attr_get(a[OVS_FLOW_ATTR_MASK]);
         flow->mask_len = nl_attr_get_size(a[OVS_FLOW_ATTR_MASK]);
@@ -2388,17 +2793,29 @@ dpif_netlink_flow_to_ofpbuf(const struct dpif_netlink_flow *flow,
     ovs_header = ofpbuf_put_uninit(buf, sizeof *ovs_header);
     ovs_header->dp_ifindex = flow->dp_ifindex;
 
-    if (flow->key_len) {
-        nl_msg_put_unspec(buf, OVS_FLOW_ATTR_KEY, flow->key, flow->key_len);
+    if (flow->ufid_present) {
+        nl_msg_put_unspec(buf, OVS_FLOW_ATTR_UFID, &flow->ufid,
+                          sizeof flow->ufid);
     }
-
-    if (flow->mask_len) {
-        nl_msg_put_unspec(buf, OVS_FLOW_ATTR_MASK, flow->mask, flow->mask_len);
+    if (flow->ufid_terse) {
+        nl_msg_put_u32(buf, OVS_FLOW_ATTR_UFID_FLAGS,
+                       OVS_UFID_F_OMIT_KEY | OVS_UFID_F_OMIT_MASK
+                       | OVS_UFID_F_OMIT_ACTIONS);
     }
+    if (!flow->ufid_terse || !flow->ufid_present) {
+        if (flow->key_len) {
+            nl_msg_put_unspec(buf, OVS_FLOW_ATTR_KEY,
+                              flow->key, flow->key_len);
+        }
 
-    if (flow->actions || flow->actions_len) {
-        nl_msg_put_unspec(buf, OVS_FLOW_ATTR_ACTIONS,
-                          flow->actions, flow->actions_len);
+        if (flow->mask_len) {
+            nl_msg_put_unspec(buf, OVS_FLOW_ATTR_MASK,
+                              flow->mask, flow->mask_len);
+        }
+        if (flow->actions || flow->actions_len) {
+            nl_msg_put_unspec(buf, OVS_FLOW_ATTR_ACTIONS,
+                              flow->actions, flow->actions_len);
+        }
     }
 
     /* We never need to send these to the kernel. */
@@ -2409,6 +2826,9 @@ dpif_netlink_flow_to_ofpbuf(const struct dpif_netlink_flow *flow,
     if (flow->clear) {
         nl_msg_put_flag(buf, OVS_FLOW_ATTR_CLEAR);
     }
+    if (flow->probe) {
+        nl_msg_put_flag(buf, OVS_FLOW_ATTR_PROBE);
+    }
 }
 
 /* Clears 'flow' to "empty" values. */