Merge commit '180c6d0b440961cbc873c4d045eb8b2daa1364e9'
[sliver-openvswitch.git] / lib / netdev-linux.c
index 722b88b..d5a3a93 100644 (file)
@@ -122,7 +122,6 @@ enum {
 
 struct tap_state {
     int fd;
-    bool opened;
 };
 \f
 /* Traffic control. */
@@ -356,7 +355,7 @@ static void tc_put_rtab(struct ofpbuf *, uint16_t type,
 static int tc_calc_buffer(unsigned int Bps, int mtu, uint64_t burst_bytes);
 \f
 struct netdev_dev_linux {
-    struct netdev_dev netdev_dev;
+    struct netdev_dev up;
 
     struct shash_node *shash_node;
     unsigned int cache_valid;
@@ -399,10 +398,17 @@ struct netdev_dev_linux {
 };
 
 struct netdev_linux {
-    struct netdev netdev;
+    struct netdev up;
+};
+
+struct netdev_rx_linux {
+    struct netdev_rx up;
+    bool is_tap;
     int fd;
 };
 
+static const struct netdev_rx_class netdev_rx_linux_class;
+
 /* Sockets used for ioctl operations. */
 static int af_inet_sock = -1;   /* AF_INET, SOCK_DGRAM. */
 
@@ -422,7 +428,7 @@ static int netdev_linux_do_ioctl(const char *name, struct ifreq *, int cmd,
 static int netdev_linux_get_ipv4(const struct netdev *, struct in_addr *,
                                  int cmd, const char *cmd_name);
 static int get_flags(const struct netdev_dev *, unsigned int *flags);
-static int set_flags(struct netdev *, unsigned int flags);
+static int set_flags(const char *, unsigned int flags);
 static int do_get_ifindex(const char *netdev_name);
 static int get_ifindex(const struct netdev *, int *ifindexp);
 static int do_set_addr(struct netdev *netdev,
@@ -442,13 +448,19 @@ is_netdev_linux_class(const struct netdev_class *netdev_class)
     return netdev_class->init == netdev_linux_init;
 }
 
+static bool
+is_tap_netdev(const struct netdev *netdev)
+{
+    return netdev_dev_get_class(netdev_get_dev(netdev)) == &netdev_tap_class;
+}
+
 static struct netdev_dev_linux *
 netdev_dev_linux_cast(const struct netdev_dev *netdev_dev)
 {
     const struct netdev_class *netdev_class = netdev_dev_get_class(netdev_dev);
     ovs_assert(is_netdev_linux_class(netdev_class));
 
-    return CONTAINER_OF(netdev_dev, struct netdev_dev_linux, netdev_dev);
+    return CONTAINER_OF(netdev_dev, struct netdev_dev_linux, up);
 }
 
 static struct netdev_linux *
@@ -458,7 +470,14 @@ netdev_linux_cast(const struct netdev *netdev)
     const struct netdev_class *netdev_class = netdev_dev_get_class(netdev_dev);
     ovs_assert(is_netdev_linux_class(netdev_class));
 
-    return CONTAINER_OF(netdev, struct netdev_linux, netdev);
+    return CONTAINER_OF(netdev, struct netdev_linux, up);
+}
+
+static struct netdev_rx_linux *
+netdev_rx_linux_cast(const struct netdev_rx *rx)
+{
+    netdev_rx_assert_class(rx, &netdev_rx_linux_class);
+    return CONTAINER_OF(rx, struct netdev_rx_linux, up);
 }
 \f
 static int
@@ -574,7 +593,7 @@ netdev_linux_cache_cb(const struct rtnetlink_link_change *change,
 
             dev = node->data;
 
-            get_flags(&dev->netdev_dev, &flags);
+            get_flags(&dev->up, &flags);
             netdev_dev_linux_changed(dev, flags, 0);
         }
         shash_destroy(&device_shash);
@@ -625,10 +644,10 @@ netdev_linux_create(const struct netdev_class *class, const char *name,
 
     netdev_dev = xzalloc(sizeof *netdev_dev);
     netdev_dev->change_seq = 1;
-    netdev_dev_init(&netdev_dev->netdev_dev, name, class);
-    get_flags(&netdev_dev->netdev_dev, &netdev_dev->ifi_flags);
+    netdev_dev_init(&netdev_dev->up, name, class);
+    get_flags(&netdev_dev->up, &netdev_dev->ifi_flags);
 
-    *netdev_devp = &netdev_dev->netdev_dev;
+    *netdev_devp = &netdev_dev->up;
     return 0;
 }
 
@@ -680,8 +699,8 @@ netdev_linux_create_tap(const struct netdev_class *class OVS_UNUSED,
         goto error_unref_notifier;
     }
 
-    netdev_dev_init(&netdev_dev->netdev_dev, name, &netdev_tap_class);
-    *netdev_devp = &netdev_dev->netdev_dev;
+    netdev_dev_init(&netdev_dev->up, name, &netdev_tap_class);
+    *netdev_devp = &netdev_dev->up;
     return 0;
 
 error_unref_notifier:
@@ -729,8 +748,7 @@ netdev_linux_open(struct netdev_dev *netdev_dev_, struct netdev **netdevp)
 
     /* Allocate network device. */
     netdev = xzalloc(sizeof *netdev);
-    netdev->fd = -1;
-    netdev_init(&netdev->netdev, netdev_dev_);
+    netdev_init(&netdev->up, netdev_dev_);
 
     /* Verify that the device really exists, by attempting to read its flags.
      * (The flags might be cached, in which case this won't actually do an
@@ -741,17 +759,17 @@ netdev_linux_open(struct netdev_dev *netdev_dev_, struct netdev **netdevp)
      * creating them in the kernel happens by passing a netdev object to
      * dpif_port_add(). */
     if (netdev_dev_get_class(netdev_dev_) != &netdev_internal_class) {
-        error = netdev_get_flags(&netdev->netdev, &flags);
+        error = netdev_get_flags(&netdev->up, &flags);
         if (error == ENODEV) {
             goto error;
         }
     }
 
-    *netdevp = &netdev->netdev;
+    *netdevp = &netdev->up;
     return 0;
 
 error:
-    netdev_uninit(&netdev->netdev, true);
+    netdev_uninit(&netdev->up, true);
     return error;
 }
 
@@ -761,67 +779,65 @@ netdev_linux_close(struct netdev *netdev_)
 {
     struct netdev_linux *netdev = netdev_linux_cast(netdev_);
 
-    if (netdev->fd > 0 && strcmp(netdev_get_type(netdev_), "tap")) {
-        close(netdev->fd);
-    }
     free(netdev);
 }
 
 static int
-netdev_linux_listen(struct netdev *netdev_)
+netdev_linux_rx_open(struct netdev *netdev_, struct netdev_rx **rxp)
 {
     struct netdev_linux *netdev = netdev_linux_cast(netdev_);
     struct netdev_dev_linux *netdev_dev =
                                 netdev_dev_linux_cast(netdev_get_dev(netdev_));
-    struct sockaddr_ll sll;
-    int ifindex;
+    bool is_tap = is_tap_netdev(netdev_);
+    struct netdev_rx_linux *rx;
     int error;
     int fd;
 
-    if (netdev->fd >= 0) {
-        return 0;
-    }
+    if (is_tap) {
+        fd = netdev_dev->state.tap.fd;
+    } else {
+        struct sockaddr_ll sll;
+        int ifindex;
 
-    if (!strcmp(netdev_get_type(netdev_), "tap")
-        && !netdev_dev->state.tap.opened) {
-        netdev->fd = netdev_dev->state.tap.fd;
-        netdev_dev->state.tap.opened = true;
-        return 0;
-    }
+        /* Create file descriptor. */
+        fd = socket(PF_PACKET, SOCK_RAW, 0);
+        if (fd < 0) {
+            error = errno;
+            VLOG_ERR("failed to create raw socket (%s)", strerror(error));
+            goto error;
+        }
 
-    /* Create file descriptor. */
-    fd = socket(PF_PACKET, SOCK_RAW, 0);
-    if (fd < 0) {
-        error = errno;
-        VLOG_ERR("failed to create raw socket (%s)", strerror(error));
-        goto error;
-    }
+        /* Set non-blocking mode. */
+        error = set_nonblocking(fd);
+        if (error) {
+            goto error;
+        }
 
-    /* Set non-blocking mode. */
-    error = set_nonblocking(fd);
-    if (error) {
-        goto error;
-    }
+        /* Get ethernet device index. */
+        error = get_ifindex(&netdev->up, &ifindex);
+        if (error) {
+            goto error;
+        }
 
-    /* Get ethernet device index. */
-    error = get_ifindex(&netdev->netdev, &ifindex);
-    if (error) {
-        goto error;
+        /* Bind to specific ethernet device. */
+        memset(&sll, 0, sizeof sll);
+        sll.sll_family = AF_PACKET;
+        sll.sll_ifindex = ifindex;
+        sll.sll_protocol = (OVS_FORCE unsigned short int) htons(ETH_P_ALL);
+        if (bind(fd, (struct sockaddr *) &sll, sizeof sll) < 0) {
+            error = errno;
+            VLOG_ERR("%s: failed to bind raw socket (%s)",
+                     netdev_get_name(netdev_), strerror(error));
+            goto error;
+        }
     }
 
-    /* Bind to specific ethernet device. */
-    memset(&sll, 0, sizeof sll);
-    sll.sll_family = AF_PACKET;
-    sll.sll_ifindex = ifindex;
-    sll.sll_protocol = (OVS_FORCE unsigned short int) htons(ETH_P_ALL);
-    if (bind(fd, (struct sockaddr *) &sll, sizeof sll) < 0) {
-        error = errno;
-        VLOG_ERR("%s: failed to bind raw socket (%s)",
-                 netdev_get_name(netdev_), strerror(error));
-        goto error;
-    }
+    rx = xmalloc(sizeof *rx);
+    netdev_rx_init(&rx->up, netdev_get_dev(netdev_), &netdev_rx_linux_class);
+    rx->is_tap = is_tap;
+    rx->fd = fd;
 
-    netdev->fd = fd;
+    *rxp = &rx->up;
     return 0;
 
 error:
@@ -831,63 +847,64 @@ error:
     return error;
 }
 
-static int
-netdev_linux_recv(struct netdev *netdev_, void *data, size_t size)
+static void
+netdev_rx_linux_destroy(struct netdev_rx *rx_)
 {
-    struct netdev_linux *netdev = netdev_linux_cast(netdev_);
+    struct netdev_rx_linux *rx = netdev_rx_linux_cast(rx_);
 
-    if (netdev->fd < 0) {
-        /* Device is not listening. */
-        return -EAGAIN;
+    if (!rx->is_tap) {
+        close(rx->fd);
     }
+    free(rx);
+}
 
-    for (;;) {
-        ssize_t retval;
+static int
+netdev_rx_linux_recv(struct netdev_rx *rx_, void *data, size_t size)
+{
+    struct netdev_rx_linux *rx = netdev_rx_linux_cast(rx_);
+    ssize_t retval;
 
-        retval = (netdev_->netdev_dev->netdev_class == &netdev_tap_class
-                  ? read(netdev->fd, data, size)
-                  : recv(netdev->fd, data, size, MSG_TRUNC));
-        if (retval >= 0) {
-            return retval <= size ? retval : -EMSGSIZE;
-        } else if (errno != EINTR) {
-            if (errno != EAGAIN) {
-                VLOG_WARN_RL(&rl, "error receiving Ethernet packet on %s: %s",
-                             strerror(errno), netdev_get_name(netdev_));
-            }
-            return -errno;
+    do {
+        retval = (rx->is_tap
+                  ? read(rx->fd, data, size)
+                  : recv(rx->fd, data, size, MSG_TRUNC));
+    } while (retval < 0 && errno == EINTR);
+
+    if (retval > size) {
+        return -EMSGSIZE;
+    } else if (retval >= 0) {
+        return retval;
+    } else {
+        if (errno != EAGAIN) {
+            VLOG_WARN_RL(&rl, "error receiving Ethernet packet on %s: %s",
+                         strerror(errno), netdev_rx_get_name(rx_));
         }
+        return -errno;
     }
 }
 
-/* Registers with the poll loop to wake up from the next call to poll_block()
- * when a packet is ready to be received with netdev_recv() on 'netdev'. */
 static void
-netdev_linux_recv_wait(struct netdev *netdev_)
+netdev_rx_linux_wait(struct netdev_rx *rx_)
 {
-    struct netdev_linux *netdev = netdev_linux_cast(netdev_);
-    if (netdev->fd >= 0) {
-        poll_fd_wait(netdev->fd, POLLIN);
-    }
+    struct netdev_rx_linux *rx = netdev_rx_linux_cast(rx_);
+    poll_fd_wait(rx->fd, POLLIN);
 }
 
-/* Discards all packets waiting to be received from 'netdev'. */
 static int
-netdev_linux_drain(struct netdev *netdev_)
+netdev_rx_linux_drain(struct netdev_rx *rx_)
 {
-    struct netdev_linux *netdev = netdev_linux_cast(netdev_);
-    if (netdev->fd < 0) {
-        return 0;
-    } else if (!strcmp(netdev_get_type(netdev_), "tap")) {
+    struct netdev_rx_linux *rx = netdev_rx_linux_cast(rx_);
+    if (rx->is_tap) {
         struct ifreq ifr;
-        int error = netdev_linux_do_ioctl(netdev_get_name(netdev_), &ifr,
+        int error = netdev_linux_do_ioctl(netdev_rx_get_name(rx_), &ifr,
                                           SIOCGIFTXQLEN, "SIOCGIFTXQLEN");
         if (error) {
             return error;
         }
-        drain_fd(netdev->fd, ifr.ifr_qlen);
+        drain_fd(rx->fd, ifr.ifr_qlen);
         return 0;
     } else {
-        return drain_rcvbuf(netdev->fd);
+        return drain_rcvbuf(rx->fd);
     }
 }
 
@@ -903,11 +920,10 @@ netdev_linux_drain(struct netdev *netdev_)
 static int
 netdev_linux_send(struct netdev *netdev_, const void *data, size_t size)
 {
-    struct netdev_linux *netdev = netdev_linux_cast(netdev_);
     for (;;) {
         ssize_t retval;
 
-        if (netdev->fd < 0) {
+        if (!is_tap_netdev(netdev_)) {
             /* Use our AF_PACKET socket to send to this device. */
             struct sockaddr_ll sll;
             struct msghdr msg;
@@ -945,11 +961,14 @@ netdev_linux_send(struct netdev *netdev_, const void *data, size_t size)
 
             retval = sendmsg(sock, &msg, 0);
         } else {
-            /* Use the netdev's own fd to send to this device.  This is
-             * essential for tap devices, because packets sent to a tap device
-             * with an AF_PACKET socket will loop back to be *received* again
-             * on the tap device. */
-            retval = write(netdev->fd, data, size);
+            /* Use the tap fd to send to this device.  This is essential for
+             * tap devices, because packets sent to a tap device with an
+             * AF_PACKET socket will loop back to be *received* again on the
+             * tap device. */
+            struct netdev_dev_linux *dev
+                = netdev_dev_linux_cast(netdev_get_dev(netdev_));
+
+            retval = write(dev->state.tap.fd, data, size);
         }
 
         if (retval < 0) {
@@ -983,14 +1002,9 @@ netdev_linux_send(struct netdev *netdev_, const void *data, size_t size)
  * expected to do additional queuing of packets.  Thus, this function is
  * unlikely to ever be used.  It is included for completeness. */
 static void
-netdev_linux_send_wait(struct netdev *netdev_)
+netdev_linux_send_wait(struct netdev *netdev)
 {
-    struct netdev_linux *netdev = netdev_linux_cast(netdev_);
-    if (netdev->fd < 0) {
-        /* Nothing to do. */
-    } else if (strcmp(netdev_get_type(netdev_), "tap")) {
-        poll_fd_wait(netdev->fd, POLLOUT);
-    } else {
+    if (is_tap_netdev(netdev)) {
         /* TAP device always accepts packets.*/
         poll_immediate_wake();
     }
@@ -1004,8 +1018,8 @@ netdev_linux_set_etheraddr(struct netdev *netdev_,
 {
     struct netdev_dev_linux *netdev_dev =
                                 netdev_dev_linux_cast(netdev_get_dev(netdev_));
+    struct netdev_saved_flags *sf = NULL;
     int error;
-    bool up_again = false;
 
     if (netdev_dev->cache_valid & VALID_ETHERADDR) {
         if (netdev_dev->ether_addr_error) {
@@ -1018,12 +1032,11 @@ netdev_linux_set_etheraddr(struct netdev *netdev_,
     }
 
     /* Tap devices must be brought down before setting the address. */
-    if (!strcmp(netdev_get_type(netdev_), "tap")) {
+    if (is_tap_netdev(netdev_)) {
         enum netdev_flags flags;
 
         if (!netdev_get_flags(netdev_, &flags) && (flags & NETDEV_UP)) {
-            netdev_turn_flags_off(netdev_, NETDEV_UP, false);
-            up_again = true;
+            netdev_turn_flags_off(netdev_, NETDEV_UP, &sf);
         }
     }
     error = set_etheraddr(netdev_get_name(netdev_), mac);
@@ -1035,9 +1048,7 @@ netdev_linux_set_etheraddr(struct netdev *netdev_,
         }
     }
 
-    if (up_again) {
-        netdev_turn_flags_on(netdev_, NETDEV_UP, false);
-    }
+    netdev_restore_flags(sf);
 
     return error;
 }
@@ -1246,7 +1257,7 @@ netdev_linux_miimon_run(void)
             continue;
         }
 
-        netdev_linux_get_miimon(dev->netdev_dev.name, &miimon);
+        netdev_linux_get_miimon(dev->up.name, &miimon);
         if (miimon != dev->miimon) {
             dev->miimon = miimon;
             netdev_dev_linux_changed(dev, dev->ifi_flags, 0);
@@ -1579,7 +1590,7 @@ netdev_linux_read_features(struct netdev_dev_linux *netdev_dev)
 
     COVERAGE_INC(netdev_get_ethtool);
     memset(&ecmd, 0, sizeof ecmd);
-    error = netdev_linux_do_ethtool(netdev_dev->netdev_dev.name, &ecmd,
+    error = netdev_linux_do_ethtool(netdev_dev->up.name, &ecmd,
                                     ETHTOOL_GSET, "ETHTOOL_GSET");
     if (error) {
         goto out;
@@ -2366,7 +2377,7 @@ netdev_linux_get_status(const struct netdev *netdev, struct smap *smap)
 
         COVERAGE_INC(netdev_get_ethtool);
         memset(&netdev_dev->drvinfo, 0, sizeof netdev_dev->drvinfo);
-        error = netdev_linux_do_ethtool(netdev_dev->netdev_dev.name,
+        error = netdev_linux_do_ethtool(netdev_dev->up.name,
                                         cmd,
                                         ETHTOOL_GDRVINFO,
                                         "ETHTOOL_GDRVINFO");
@@ -2450,20 +2461,20 @@ iff_to_nd_flags(int iff)
 }
 
 static int
-netdev_linux_update_flags(struct netdev *netdev, enum netdev_flags off,
+netdev_linux_update_flags(struct netdev_dev *dev_, enum netdev_flags off,
                           enum netdev_flags on, enum netdev_flags *old_flagsp)
 {
     struct netdev_dev_linux *netdev_dev;
     int old_flags, new_flags;
     int error = 0;
 
-    netdev_dev = netdev_dev_linux_cast(netdev_get_dev(netdev));
+    netdev_dev = netdev_dev_linux_cast(dev_);
     old_flags = netdev_dev->ifi_flags;
     *old_flagsp = iff_to_nd_flags(old_flags);
     new_flags = (old_flags & ~nd_to_iff_flags(off)) | nd_to_iff_flags(on);
     if (new_flags != old_flags) {
-        error = set_flags(netdev, new_flags);
-        get_flags(&netdev_dev->netdev_dev, &netdev_dev->ifi_flags);
+        error = set_flags(netdev_dev_get_name(dev_), new_flags);
+        get_flags(&netdev_dev->up, &netdev_dev->ifi_flags);
     }
     return error;
 }
@@ -2492,10 +2503,7 @@ netdev_linux_change_seq(const struct netdev *netdev)
     netdev_linux_open,                                          \
     netdev_linux_close,                                         \
                                                                 \
-    netdev_linux_listen,                                        \
-    netdev_linux_recv,                                          \
-    netdev_linux_recv_wait,                                     \
-    netdev_linux_drain,                                         \
+    netdev_linux_rx_open,                                       \
                                                                 \
     netdev_linux_send,                                          \
     netdev_linux_send_wait,                                     \
@@ -2565,6 +2573,13 @@ const struct netdev_class netdev_internal_class =
         netdev_internal_set_stats,
         NULL,                  /* get_features */
         netdev_internal_get_status);
+
+static const struct netdev_rx_class netdev_rx_linux_class = {
+    netdev_rx_linux_destroy,
+    netdev_rx_linux_recv,
+    netdev_rx_linux_wait,
+    netdev_rx_linux_drain,
+};
 \f
 /* HTB traffic control class. */
 
@@ -4481,13 +4496,12 @@ get_flags(const struct netdev_dev *dev, unsigned int *flags)
 }
 
 static int
-set_flags(struct netdev *netdev, unsigned int flags)
+set_flags(const char *name, unsigned int flags)
 {
     struct ifreq ifr;
 
     ifr.ifr_flags = flags;
-    return netdev_linux_do_ioctl(netdev_get_name(netdev), &ifr, SIOCSIFFLAGS,
-                                 "SIOCSIFFLAGS");
+    return netdev_linux_do_ioctl(name, &ifr, SIOCSIFFLAGS, "SIOCSIFFLAGS");
 }
 
 static int