#include <config.h>
+#include "netdev-provider.h"
#include <stdlib.h>
-#include <config.h>
-#include <assert.h>
#include <errno.h>
#include <fcntl.h>
#include <sys/types.h>
#include "coverage.h"
#include "dynamic-string.h"
#include "fatal-signal.h"
-#include "netdev-provider.h"
#include "ofpbuf.h"
#include "openflow/openflow.h"
#include "packets.h"
#include "socket-util.h"
#include "shash.h"
#include "svec.h"
+#include "util.h"
#include "vlog.h"
VLOG_DEFINE_THIS_MODULE(netdev_bsd);
\f
/*
* This file implements objects to access interfaces.
- * Externally, interfaces are represented by two structures:
+ * Externally, interfaces are represented by three structures:
* + struct netdev_dev, representing a network device,
* containing e.g. name and a refcount;
* We can have private variables by embedding the
*
* + struct netdev, representing an instance of an open netdev_dev.
* The structure contains a pointer to the 'struct netdev'
- * representing the device. Again, private information
- * such as file descriptor etc. are stored in our
- * own struct netdev_bsd which includes a struct netdev.
+ * representing the device.
*
- * Both 'struct netdev' and 'struct netdev_dev' are referenced
- * in containers which hold pointers to the data structures.
- * We can reach our own struct netdev_XXX_bsd by putting a
- * struct netdev_XXX within our own struct, and using CONTAINER_OF
- * to access the parent structure.
+ * + struct netdev_rx, which represents a netdev open to capture received
+ * packets. Again, private information such as file descriptor etc. are
+ * stored in our own struct netdev_rx_bsd which includes a struct
+ * netdev_rx.
+ *
+ * 'struct netdev', 'struct netdev_dev', and 'struct netdev_rx' are referenced
+ * in containers which hold pointers to the data structures. We can reach our
+ * own struct netdev_XXX_bsd by putting a struct netdev_XXX within our own
+ * struct, and using CONTAINER_OF to access the parent structure.
*/
struct netdev_bsd {
- struct netdev netdev;
+ struct netdev up;
+};
+
+struct netdev_rx_bsd {
+ struct netdev_rx up;
- int netdev_fd; /* Selectable file descriptor for the network device.
- This descriptor will be used for polling operations */
+ /* Packet capture descriptor for a system network device.
+ * For a tap device this is NULL. */
+ pcap_t *pcap_handle;
- pcap_t *pcap_handle; /* Packet capture descriptor for a system network
- device */
+ /* Selectable file descriptor for the network device.
+ * This descriptor will be used for polling operations. */
+ int fd;
};
+static const struct netdev_rx_class netdev_rx_bsd_class;
+
struct netdev_dev_bsd {
- struct netdev_dev netdev_dev;
+ struct netdev_dev up;
unsigned int cache_valid;
unsigned int change_seq;
int mtu;
int carrier;
- bool tap_opened;
- int tap_fd; /* TAP character device, if any */
+ int tap_fd; /* TAP character device, if any, otherwise -1. */
+
+ /* Used for sending packets on non-tap devices. */
+ pcap_t *pcap;
+ int fd;
};
static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(5, 20);
-static int netdev_bsd_do_ioctl(const struct netdev *, struct ifreq *,
- unsigned long cmd, const char *cmd_name);
+static int netdev_bsd_do_ioctl(const char *, struct ifreq *, unsigned long cmd,
+ const char *cmd_name);
static void destroy_tap(int fd, const char *name);
-static int get_flags(const struct netdev *, int *flagsp);
-static int set_flags(struct netdev *, int flags);
+static int get_flags(const struct netdev_dev *, int *flagsp);
+static int set_flags(const char *, int flags);
static int do_set_addr(struct netdev *netdev,
int ioctl_nr, const char *ioctl_name,
struct in_addr addr);
static struct netdev_bsd *
netdev_bsd_cast(const struct netdev *netdev)
{
- assert(is_netdev_bsd_class(netdev_dev_get_class(netdev_get_dev(netdev))));
- return CONTAINER_OF(netdev, struct netdev_bsd, netdev);
+ ovs_assert(is_netdev_bsd_class(netdev_dev_get_class(
+ netdev_get_dev(netdev))));
+ return CONTAINER_OF(netdev, struct netdev_bsd, up);
}
static struct netdev_dev_bsd *
netdev_dev_bsd_cast(const struct netdev_dev *netdev_dev)
{
- assert(is_netdev_bsd_class(netdev_dev_get_class(netdev_dev)));
- return CONTAINER_OF(netdev_dev, struct netdev_dev_bsd, netdev_dev);
+ ovs_assert(is_netdev_bsd_class(netdev_dev_get_class(netdev_dev)));
+ return CONTAINER_OF(netdev_dev, struct netdev_dev_bsd, up);
+}
+
+static struct netdev_rx_bsd *
+netdev_rx_bsd_cast(const struct netdev_rx *rx)
+{
+ netdev_rx_assert_class(rx, &netdev_rx_bsd_class);
+ return CONTAINER_OF(rx, struct netdev_rx_bsd, up);
}
/* Initialize the AF_INET socket used for ioctl operations */
/* Invalidate cache in case of interface status change. */
static void
netdev_bsd_cache_cb(const struct rtbsd_change *change,
- void *aux OVS_UNUSED)
+ void *aux OVS_UNUSED)
{
struct netdev_dev_bsd *dev;
netdev_dev = xzalloc(sizeof *netdev_dev);
netdev_dev->change_seq = 1;
- netdev_dev_init(&netdev_dev->netdev_dev, name, class);
- *netdev_devp = &netdev_dev->netdev_dev;
+ netdev_dev_init(&netdev_dev->up, name, class);
+ netdev_dev->tap_fd = -1;
+ *netdev_devp = &netdev_dev->up;
return 0;
}
/* initialize the device structure and
* link the structure to its netdev */
- netdev_dev_init(&netdev_dev->netdev_dev, name, class);
- *netdev_devp = &netdev_dev->netdev_dev;
+ netdev_dev_init(&netdev_dev->up, name, class);
+ *netdev_devp = &netdev_dev->up;
return 0;
cache_notifier_unref();
- if (netdev_dev->tap_fd >= 0 &&
- !strcmp(netdev_dev_get_type(netdev_dev_), "tap")) {
+ if (netdev_dev->tap_fd >= 0) {
destroy_tap(netdev_dev->tap_fd, netdev_dev_get_name(netdev_dev_));
}
+ if (netdev_dev->pcap) {
+ pcap_close(netdev_dev->pcap);
+ }
free(netdev_dev);
}
static int
netdev_bsd_open_system(struct netdev_dev *netdev_dev_, struct netdev **netdevp)
{
- struct netdev_dev_bsd *netdev_dev = netdev_dev_bsd_cast(netdev_dev_);
struct netdev_bsd *netdev;
int error;
enum netdev_flags flags;
/* Allocate network device. */
netdev = xcalloc(1, sizeof *netdev);
- netdev->netdev_fd = -1;
- netdev_init(&netdev->netdev, netdev_dev_);
+ netdev_init(&netdev->up, netdev_dev_);
/* Verify that the netdev really exists by attempting to read its flags */
- error = netdev_get_flags(&netdev->netdev, &flags);
+ error = netdev_get_flags(&netdev->up, &flags);
if (error == ENXIO) {
goto error;
}
- /* The first user that opens a tap port(from dpif_create_and_open()) will
- * receive the file descriptor associated with the tap device. Instead, the
- * following users will open the tap device as a normal 'system' device. */
- if (!strcmp(netdev_dev_get_type(netdev_dev_), "tap") &&
- !netdev_dev->tap_opened) {
- netdev_dev->tap_opened = true;
- netdev->netdev_fd = netdev_dev->tap_fd;
- }
-
- *netdevp = &netdev->netdev;
+ *netdevp = &netdev->up;
return 0;
error:
- netdev_uninit(&netdev->netdev, true);
+ netdev_uninit(&netdev->up, true);
return error;
}
{
struct netdev_bsd *netdev = netdev_bsd_cast(netdev_);
- if (netdev->netdev_fd >= 0 && strcmp(netdev_get_type(netdev_), "tap")) {
- pcap_close(netdev->pcap_handle);
- }
-
free(netdev);
}
static int
-netdev_bsd_listen(struct netdev *netdev_)
+netdev_bsd_open_pcap(const char *name, pcap_t **pcapp, int *fdp)
{
- struct netdev_bsd *netdev = netdev_bsd_cast(netdev_);
char errbuf[PCAP_ERRBUF_SIZE];
- int error;
- int fd = -1;
+ pcap_t *pcap = NULL;
int one = 1;
+ int error;
+ int fd;
- if (netdev->netdev_fd >= 0) {
- return 0;
- }
-
- /* open the pcap device. The device is opened in non-promiscuous mode
+ /* Open the pcap device. The device is opened in non-promiscuous mode
* because the interface flags are manually set by the caller. */
errbuf[0] = '\0';
- netdev->pcap_handle = pcap_open_live(netdev_get_name(netdev_), PCAP_SNAPLEN,
- 0, 1000, errbuf);
- if (netdev->pcap_handle == NULL) {
- VLOG_ERR("%s: pcap_open_live failed: %s",
- netdev_get_name(netdev_), errbuf);
+ pcap = pcap_open_live(name, PCAP_SNAPLEN, 0, 1000, errbuf);
+ if (!pcap) {
+ VLOG_ERR_RL(&rl, "%s: pcap_open_live failed: %s", name, errbuf);
error = EIO;
goto error;
- } else if (errbuf[0] != '\0') {
- VLOG_WARN("%s: pcap_open_live: %s",
- netdev_get_name(netdev_), errbuf);
+ }
+ if (errbuf[0] != '\0') {
+ VLOG_WARN_RL(&rl, "%s: pcap_open_live: %s", name, errbuf);
}
- netdev_dev_bsd_changed(netdev_dev_bsd_cast(netdev_get_dev(netdev_)));
-
- /* initialize netdev->netdev_fd */
- fd = pcap_get_selectable_fd(netdev->pcap_handle);
+ /* Get the underlying fd. */
+ fd = pcap_get_selectable_fd(pcap);
if (fd == -1) {
+ VLOG_WARN_RL(&rl, "%s: no selectable file descriptor", name);
error = errno;
goto error;
}
/* Set non-blocking mode. Also the BIOCIMMEDIATE ioctl must be called
* on the file descriptor returned by pcap_get_selectable_fd to achieve
* a real non-blocking behaviour.*/
- error = pcap_setnonblock(netdev->pcap_handle, 1, errbuf);
+ error = pcap_setnonblock(pcap, 1, errbuf);
if (error == -1) {
error = errno;
goto error;
}
- /* This call assure that reads return immediately upon packet reception.
- * Otherwise, a read will block until either the kernel buffer becomes
- * full or a timeout occurs. */
- if(ioctl(fd, BIOCIMMEDIATE, &one) < 0 ) {
- VLOG_ERR("ioctl(BIOCIMMEDIATE) on %s device failed: %s",
- netdev_get_name(netdev_), strerror(errno));
+ /* This call assure that reads return immediately upon packet
+ * reception. Otherwise, a read will block until either the kernel
+ * buffer becomes full or a timeout occurs. */
+ if (ioctl(fd, BIOCIMMEDIATE, &one) < 0 ) {
+ VLOG_ERR_RL(&rl, "ioctl(BIOCIMMEDIATE) on %s device failed: %s",
+ name, strerror(errno));
error = errno;
goto error;
}
- /* Capture only incoming packets */
- error = pcap_setdirection(netdev->pcap_handle, PCAP_D_IN);
+ /* Capture only incoming packets. */
+ error = pcap_setdirection(pcap, PCAP_D_IN);
if (error == -1) {
error = errno;
goto error;
}
- netdev->netdev_fd = fd;
+ *pcapp = pcap;
+ *fdp = fd;
return 0;
error:
- if (fd >= 0) {
- close(netdev->netdev_fd);
+ if (pcap) {
+ pcap_close(pcap);
}
+ *pcapp = NULL;
+ *fdp = -1;
return error;
}
+static int
+netdev_bsd_rx_open(struct netdev *netdev_, struct netdev_rx **rxp)
+{
+ struct netdev_dev_bsd *netdev_dev =
+ netdev_dev_bsd_cast(netdev_get_dev(netdev_));
+
+ struct netdev_rx_bsd *rx;
+ pcap_t *pcap;
+ int fd;
+
+ if (!strcmp(netdev_get_type(netdev_), "tap")) {
+ pcap = NULL;
+ fd = netdev_dev->tap_fd;
+ } else {
+ int error = netdev_bsd_open_pcap(netdev_get_name(netdev_), &pcap, &fd);
+ if (error) {
+ return error;
+ }
+
+ netdev_dev_bsd_changed(netdev_dev);
+ }
+
+ rx = xmalloc(sizeof *rx);
+ netdev_rx_init(&rx->up, netdev_get_dev(netdev_), &netdev_rx_bsd_class);
+ rx->pcap_handle = pcap;
+ rx->fd = fd;
+
+ *rxp = &rx->up;
+ return 0;
+}
+
+static void
+netdev_rx_bsd_destroy(struct netdev_rx *rx_)
+{
+ struct netdev_rx_bsd *rx = netdev_rx_bsd_cast(rx_);
+
+ if (rx->pcap_handle) {
+ pcap_close(rx->pcap_handle);
+ }
+ free(rx);
+}
/* The recv callback of the netdev class returns the number of bytes of the
* received packet.
* This function attempts to receive a packet from the specified network
* device. It is assumed that the network device is a system device or a tap
* device opened as a system one. In this case the read operation is performed
- * on the 'netdev' pcap descriptor.
+ * from rx->pcap.
*/
static int
-netdev_bsd_recv_system(struct netdev_bsd *netdev, void *data, size_t size)
+netdev_rx_bsd_recv_pcap(struct netdev_rx_bsd *rx, void *data, size_t size)
{
struct pcap_arg arg;
int ret;
- if (netdev->netdev_fd < 0) {
- return -EAGAIN;
- }
-
/* prepare the pcap argument to store the packet */
arg.size = size;
arg.data = data;
for (;;) {
- ret = pcap_dispatch(netdev->pcap_handle, 1, proc_pkt, (u_char *)&arg);
+ ret = pcap_dispatch(rx->pcap_handle, 1, proc_pkt, (u_char *) &arg);
if (ret > 0) {
return arg.retval; /* arg.retval < 0 is handled in the caller */
/*
* This function attempts to receive a packet from the specified network
- * device. It is assumed that the network device is a tap device and the
- * 'netdev_fd' member of the 'netdev' structure is initialized with the tap
- * file descriptor.
+ * device. It is assumed that the network device is a tap device and
+ * 'rx->fd' is initialized with the tap file descriptor.
*/
static int
-netdev_bsd_recv_tap(struct netdev_bsd *netdev, void *data, size_t size)
+netdev_rx_bsd_recv_tap(struct netdev_rx_bsd *rx, void *data, size_t size)
{
- if (netdev->netdev_fd < 0) {
- return -EAGAIN;
- }
-
for (;;) {
- ssize_t retval = read(netdev->netdev_fd, data, size);
+ ssize_t retval = read(rx->fd, data, size);
if (retval >= 0) {
return retval;
} else if (errno != EINTR) {
if (errno != EAGAIN) {
VLOG_WARN_RL(&rl, "error receiving Ethernet packet on %s: %s",
- strerror(errno), netdev->netdev.netdev_dev->name);
+ strerror(errno), netdev_rx_get_name(&rx->up));
}
return -errno;
}
}
-/*
- * According with the nature of the device a different function must be called.
- * If the device is the bridge local port the 'netdev_bsd_recv_tap' function
- * must be called, otherwise the 'netdev_bsd_recv_system' function is called.
- *
- * type!="tap" ---> system device.
- * type=="tap" && netdev_fd == tap_fd ---> internal tap device
- * type=="tap" && netdev_fd != tap_fd ---> internal tap device
- * opened as a system
- * device.
- */
static int
-netdev_bsd_recv(struct netdev *netdev_, void* data, size_t size)
+netdev_rx_bsd_recv(struct netdev_rx *rx_, void *data, size_t size)
{
- struct netdev_bsd *netdev = netdev_bsd_cast(netdev_);
- struct netdev_dev_bsd * netdev_dev =
- netdev_dev_bsd_cast(netdev_get_dev(netdev_));
+ struct netdev_rx_bsd *rx = netdev_rx_bsd_cast(rx_);
- if (!strcmp(netdev_get_type(netdev_), "tap") &&
- netdev->netdev_fd == netdev_dev->tap_fd) {
- return netdev_bsd_recv_tap(netdev, data, size);
- } else {
- return netdev_bsd_recv_system(netdev, data, size);
- }
+ return (rx->pcap_handle
+ ? netdev_rx_bsd_recv_pcap(rx, data, size)
+ : netdev_rx_bsd_recv_tap(rx, data, size));
}
-
/*
* 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'.
+ * when a packet is ready to be received with netdev_rx_recv() on 'rx'.
*/
static void
-netdev_bsd_recv_wait(struct netdev *netdev_)
+netdev_rx_bsd_wait(struct netdev_rx *rx_)
{
- struct netdev_bsd *netdev = netdev_bsd_cast(netdev_);
+ struct netdev_rx_bsd *rx = netdev_rx_bsd_cast(rx_);
- if (netdev->netdev_fd >= 0) {
- poll_fd_wait(netdev->netdev_fd, POLLIN);
- }
+ poll_fd_wait(rx->fd, POLLIN);
}
-/* Discards all packets waiting to be received from 'netdev'. */
+/* Discards all packets waiting to be received from 'rx'. */
static int
-netdev_bsd_drain(struct netdev *netdev_)
+netdev_rx_bsd_drain(struct netdev_rx *rx_)
{
struct ifreq ifr;
- struct netdev_bsd *netdev = netdev_bsd_cast(netdev_);
+ struct netdev_rx_bsd *rx = netdev_rx_bsd_cast(rx_);
- strcpy(ifr.ifr_name, netdev_get_name(netdev_));
- if (ioctl(netdev->netdev_fd, BIOCFLUSH, &ifr) == -1) {
+ strcpy(ifr.ifr_name, netdev_rx_get_name(rx_));
+ if (ioctl(rx->fd, BIOCFLUSH, &ifr) == -1) {
VLOG_DBG_RL(&rl, "%s: ioctl(BIOCFLUSH) failed: %s",
- netdev_get_name(netdev_), strerror(errno));
+ netdev_rx_get_name(rx_), strerror(errno));
return errno;
}
return 0;
static int
netdev_bsd_send(struct netdev *netdev_, const void *data, size_t size)
{
- struct netdev_bsd *netdev = netdev_bsd_cast(netdev_);
- struct netdev_dev_bsd * netdev_dev =
- netdev_dev_bsd_cast(netdev_get_dev(netdev_));
+ struct netdev_dev_bsd *dev = netdev_dev_bsd_cast(netdev_get_dev(netdev_));
+ const char *name = netdev_get_name(netdev_);
- if (netdev->netdev_fd < 0) {
- return EPIPE;
+ if (dev->tap_fd < 0 && !dev->pcap) {
+ int error = netdev_bsd_open_pcap(name, &dev->pcap, &dev->fd);
+ if (error) {
+ return error;
+ }
}
for (;;) {
ssize_t retval;
- if (!strcmp(netdev_get_type(netdev_), "tap") &&
- netdev_dev->tap_fd == netdev->netdev_fd) {
- retval = write(netdev->netdev_fd, data, size);
+ if (dev->tap_fd >= 0) {
+ retval = write(dev->tap_fd, data, size);
} else {
- retval = pcap_inject(netdev->pcap_handle, data, size);
+ retval = pcap_inject(dev->pcap, data, size);
}
if (retval < 0) {
if (errno == EINTR) {
continue;
} else if (errno != EAGAIN) {
VLOG_WARN_RL(&rl, "error sending Ethernet packet on %s: %s",
- netdev_get_name(netdev_), strerror(errno));
+ name, strerror(errno));
}
return errno;
} else if (retval != size) {
VLOG_WARN_RL(&rl, "sent partial Ethernet packet (%zd bytes of "
- "%zu) on %s", retval, size,
- netdev_get_name(netdev_));
+ "%zu) on %s", retval, size, name);
return EMSGSIZE;
} else {
return 0;
static void
netdev_bsd_send_wait(struct netdev *netdev_)
{
- struct netdev_bsd *netdev = netdev_bsd_cast(netdev_);
-
- if (netdev->netdev_fd < 0) { /* Nothing to do. */
- return;
- }
+ struct netdev_dev_bsd *dev = netdev_dev_bsd_cast(netdev_get_dev(netdev_));
- if (strcmp(netdev_get_type(netdev_), "tap")) {
- poll_fd_wait(netdev->netdev_fd, POLLOUT);
- } else {
+ if (dev->tap_fd >= 0) {
/* TAP device always accepts packets. */
poll_immediate_wake();
+ } else if (dev->pcap) {
+ poll_fd_wait(dev->fd, POLLOUT);
+ } else {
+ /* We haven't even tried to send a packet yet. */
+ poll_immediate_wake();
}
}
*/
static int
netdev_bsd_set_etheraddr(struct netdev *netdev_,
- const uint8_t mac[ETH_ADDR_LEN])
+ const uint8_t mac[ETH_ADDR_LEN])
{
struct netdev_dev_bsd *netdev_dev =
netdev_dev_bsd_cast(netdev_get_dev(netdev_));
*/
static int
netdev_bsd_get_etheraddr(const struct netdev *netdev_,
- uint8_t mac[ETH_ADDR_LEN])
+ uint8_t mac[ETH_ADDR_LEN])
{
struct netdev_dev_bsd *netdev_dev =
netdev_dev_bsd_cast(netdev_get_dev(netdev_));
struct ifreq ifr;
int error;
- error = netdev_bsd_do_ioctl(netdev_, &ifr, SIOCGIFMTU, "SIOCGIFMTU");
+ error = netdev_bsd_do_ioctl(netdev_get_name(netdev_), &ifr, SIOCGIFMTU,
+ "SIOCGIFMTU");
if (error) {
return error;
}
stats->rx_errors = ifmd.ifmd_data.ifi_ierrors;
stats->tx_errors = ifmd.ifmd_data.ifi_oerrors;
stats->rx_dropped = ifmd.ifmd_data.ifi_iqdrops;
- stats->tx_dropped = 0;
+ stats->tx_dropped = UINT64_MAX;
stats->multicast = ifmd.ifmd_data.ifi_imcasts;
stats->collisions = ifmd.ifmd_data.ifi_collisions;
- stats->rx_length_errors = 0;
- stats->rx_over_errors = 0;
- stats->rx_crc_errors = 0;
- stats->rx_frame_errors = 0;
- stats->rx_fifo_errors = 0;
- stats->rx_missed_errors = 0;
-
- stats->tx_aborted_errors = 0;
- stats->tx_carrier_errors = 0;
- stats->tx_fifo_errors = 0;
- stats->tx_heartbeat_errors = 0;
- stats->tx_window_errors = 0;
+ stats->rx_length_errors = UINT64_MAX;
+ stats->rx_over_errors = UINT64_MAX;
+ stats->rx_crc_errors = UINT64_MAX;
+ stats->rx_frame_errors = UINT64_MAX;
+ stats->rx_fifo_errors = UINT64_MAX;
+ stats->rx_missed_errors = UINT64_MAX;
+
+ stats->tx_aborted_errors = UINT64_MAX;
+ stats->tx_carrier_errors = UINT64_MAX;
+ stats->tx_fifo_errors = UINT64_MAX;
+ stats->tx_heartbeat_errors = UINT64_MAX;
+ stats->tx_window_errors = UINT64_MAX;
break;
}
}
*/
static int
netdev_bsd_get_features(const struct netdev *netdev,
- enum netdev_features *current, uint32_t *advertised,
- enum netdev_features *supported, uint32_t *peer)
+ enum netdev_features *current, uint32_t *advertised,
+ enum netdev_features *supported, uint32_t *peer)
{
struct ifmediareq ifmr;
int *media_list;
media_list = xcalloc(ifmr.ifm_count, sizeof(int));
ifmr.ifm_ulist = media_list;
- if (!IFM_TYPE(ifmr.ifm_current) & IFM_ETHER) {
+ if (IFM_TYPE(ifmr.ifm_current) != IFM_ETHER) {
VLOG_DBG_RL(&rl, "%s: doesn't appear to be ethernet",
netdev_get_name(netdev));
error = EINVAL;
int error;
ifr.ifr_addr.sa_family = AF_INET;
- error = netdev_bsd_do_ioctl(netdev_, &ifr,
- SIOCGIFADDR, "SIOCGIFADDR");
+ error = netdev_bsd_do_ioctl(netdev_get_name(netdev_), &ifr,
+ SIOCGIFADDR, "SIOCGIFADDR");
if (error) {
return error;
}
sin = (struct sockaddr_in *) &ifr.ifr_addr;
netdev_dev->in4 = sin->sin_addr;
netdev_dev->cache_valid |= VALID_IN4;
- error = netdev_bsd_do_ioctl(netdev_, &ifr,
- SIOCGIFNETMASK, "SIOCGIFNETMASK");
+ error = netdev_bsd_do_ioctl(netdev_get_name(netdev_), &ifr,
+ SIOCGIFNETMASK, "SIOCGIFNETMASK");
if (error) {
return error;
}
*/
static int
netdev_bsd_set_in4(struct netdev *netdev_, struct in_addr addr,
- struct in_addr mask)
+ struct in_addr mask)
{
struct netdev_dev_bsd *netdev_dev =
netdev_dev_bsd_cast(netdev_get_dev(netdev_));
}
static int
-netdev_bsd_update_flags(struct netdev *netdev, enum netdev_flags off,
- enum netdev_flags on, enum netdev_flags *old_flagsp)
+netdev_bsd_update_flags(struct netdev_dev *dev_, enum netdev_flags off,
+ enum netdev_flags on, enum netdev_flags *old_flagsp)
{
+ struct netdev_dev_bsd *netdev_dev;
int old_flags, new_flags;
int error;
- error = get_flags(netdev, &old_flags);
+ netdev_dev = netdev_dev_bsd_cast(dev_);
+ error = get_flags(dev_, &old_flags);
if (!error) {
*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);
- netdev_dev_bsd_changed(netdev_dev_bsd_cast(netdev_get_dev(netdev)));
+ error = set_flags(netdev_dev_get_name(dev_), new_flags);
+ netdev_dev_bsd_changed(netdev_dev);
}
}
return error;
netdev_bsd_destroy,
NULL, /* get_config */
NULL, /* set_config */
+ NULL, /* get_tunnel_config */
netdev_bsd_open_system,
netdev_bsd_close,
- netdev_bsd_listen,
-
- netdev_bsd_recv,
- netdev_bsd_recv_wait,
- netdev_bsd_drain,
+ netdev_bsd_rx_open,
netdev_bsd_send,
netdev_bsd_send_wait,
netdev_bsd_get_in6,
NULL, /* add_router */
NULL, /* get_next_hop */
- NULL, /* get_drv_info */
+ NULL, /* get_status */
NULL, /* arp_lookup */
netdev_bsd_update_flags,
netdev_bsd_destroy,
NULL, /* get_config */
NULL, /* set_config */
+ NULL, /* get_tunnel_config */
netdev_bsd_open_system,
netdev_bsd_close,
- netdev_bsd_listen,
-
- netdev_bsd_recv,
- netdev_bsd_recv_wait,
- netdev_bsd_drain,
+ netdev_bsd_rx_open,
netdev_bsd_send,
netdev_bsd_send_wait,
netdev_bsd_get_in6,
NULL, /* add_router */
NULL, /* get_next_hop */
- NULL, /* get_drv_info */
+ NULL, /* get_status */
NULL, /* arp_lookup */
netdev_bsd_update_flags,
netdev_bsd_change_seq
};
+
+static const struct netdev_rx_class netdev_rx_bsd_class = {
+ netdev_rx_bsd_destroy,
+ netdev_rx_bsd_recv,
+ netdev_rx_bsd_wait,
+ netdev_rx_bsd_drain,
+};
\f
static void
}
static int
-get_flags(const struct netdev *netdev, int *flags)
+get_flags(const struct netdev_dev *dev, int *flags)
{
struct ifreq ifr;
int error;
- error = netdev_bsd_do_ioctl(netdev, &ifr, SIOCGIFFLAGS, "SIOCGIFFLAGS");
+ error = netdev_bsd_do_ioctl(dev->name, &ifr, SIOCGIFFLAGS, "SIOCGIFFLAGS");
*flags = 0xFFFF0000 & (ifr.ifr_flagshigh << 16);
*flags |= 0x0000FFFF & ifr.ifr_flags;
}
static int
-set_flags(struct netdev *netdev, int flags)
+set_flags(const char *name, int flags)
{
struct ifreq ifr;
ifr.ifr_flags = 0x0000FFFF & flags;
ifr.ifr_flagshigh = (0xFFFF0000 & flags) >> 16;
- return netdev_bsd_do_ioctl(netdev, &ifr, SIOCSIFFLAGS, "SIOCSIFFLAGS");
+ return netdev_bsd_do_ioctl(name, &ifr, SIOCSIFFLAGS, "SIOCSIFFLAGS");
}
static int
}
static int
-netdev_bsd_do_ioctl(const struct netdev *netdev, struct ifreq *ifr,
- unsigned long cmd, const char *cmd_name)
+netdev_bsd_do_ioctl(const char *name, struct ifreq *ifr, unsigned long cmd,
+ const char *cmd_name)
{
- strncpy(ifr->ifr_name, netdev_get_name(netdev), sizeof ifr->ifr_name);
+ strncpy(ifr->ifr_name, name, sizeof ifr->ifr_name);
if (ioctl(af_inet_sock, cmd, ifr) == -1) {
- VLOG_DBG_RL(&rl, "%s: ioctl(%s) failed: %s",
- netdev_get_name(netdev), cmd_name, strerror(errno));
+ VLOG_DBG_RL(&rl, "%s: ioctl(%s) failed: %s", name, cmd_name,
+ strerror(errno));
return errno;
}
return 0;