X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=datapath%2Fvport-internal_dev.c;h=a468ac83506212f9b98985bcb3843f3d07b066ef;hb=db0a4375c455bbea40aa127c954d8727013e1dc2;hp=eca2a60b4fa8ecfdadfaada52e6e3ef44f703cbc;hpb=fceb2a5bb2063023777fc75c68a2670b5169fa13;p=openvswitch diff --git a/datapath/vport-internal_dev.c b/datapath/vport-internal_dev.c index eca2a60b..a468ac83 100644 --- a/datapath/vport-internal_dev.c +++ b/datapath/vport-internal_dev.c @@ -1,23 +1,31 @@ /* - * Copyright (c) 2009, 2010 Nicira Networks. + * Copyright (c) 2009, 2010, 2011 Nicira Networks. * Distributed under the terms of the GNU GPL version 2. * * Significant portions of this file may be copied from parts of the Linux * kernel, by Linus Torvalds and others. */ +#include +#include #include #include #include #include -#include #include +#include +#include "checksum.h" #include "datapath.h" +#include "vlan.h" #include "vport-generic.h" #include "vport-internal_dev.h" #include "vport-netdev.h" +#if LINUX_VERSION_CODE >= KERNEL_VERSION(3,1,0) +#define HAVE_NET_DEVICE_OPS +#endif + struct internal_dev { struct vport *vport; struct net_device_stats stats; @@ -28,16 +36,14 @@ static inline struct internal_dev *internal_dev_priv(struct net_device *netdev) return netdev_priv(netdev); } -/* This function is only called by the kernel network layer. It is not a vport - * get_stats() function. If a vport get_stats() function is defined that - * results in this being called it will cause infinite recursion. */ +/* This function is only called by the kernel network layer.*/ static struct net_device_stats *internal_dev_sys_stats(struct net_device *netdev) { struct vport *vport = internal_dev_get_vport(netdev); struct net_device_stats *stats = &internal_dev_priv(netdev)->stats; if (vport) { - struct odp_vport_stats vport_stats; + struct ovs_vport_stats vport_stats; vport_get_stats(vport, &vport_stats); @@ -51,7 +57,6 @@ static struct net_device_stats *internal_dev_sys_stats(struct net_device *netdev stats->tx_errors = vport_stats.rx_errors; stats->rx_dropped = vport_stats.tx_dropped; stats->tx_dropped = vport_stats.rx_dropped; - stats->collisions = vport_stats.collisions; } return stats; @@ -67,23 +72,20 @@ static int internal_dev_mac_addr(struct net_device *dev, void *p) return 0; } -/* Called with rcu_read_lock and bottom-halves disabled. */ +/* Called with rcu_read_lock_bh. */ static int internal_dev_xmit(struct sk_buff *skb, struct net_device *netdev) { - struct vport *vport = internal_dev_get_vport(netdev); - - /* We need our own clone. */ - skb = skb_share_check(skb, GFP_ATOMIC); - if (!skb) { - vport_record_error(vport, VPORT_E_RX_DROPPED); + if (unlikely(compute_ip_summed(skb, true))) { + kfree_skb(skb); return 0; } - skb_reset_mac_header(skb); - compute_ip_summed(skb, true); - - vport_receive(vport, skb); + vlan_copy_skb_tci(skb); + OVS_CB(skb)->flow = NULL; + rcu_read_lock(); + vport_receive(internal_dev_priv(netdev)->vport, skb); + rcu_read_unlock(); return 0; } @@ -102,14 +104,10 @@ static int internal_dev_stop(struct net_device *netdev) static void internal_dev_getinfo(struct net_device *netdev, struct ethtool_drvinfo *info) { - struct dp_port *dp_port = vport_get_dp_port(internal_dev_get_vport(netdev)); - strcpy(info->driver, "openvswitch"); - if (dp_port) - sprintf(info->bus_info, "%d.%d", dp_port->dp->dp_idx, dp_port->port_no); } -static struct ethtool_ops internal_dev_ethtool_ops = { +static const struct ethtool_ops internal_dev_ethtool_ops = { .get_drvinfo = internal_dev_getinfo, .get_link = ethtool_op_get_link, .get_sg = ethtool_op_get_sg, @@ -122,25 +120,13 @@ static struct ethtool_ops internal_dev_ethtool_ops = { static int internal_dev_change_mtu(struct net_device *netdev, int new_mtu) { - struct dp_port *dp_port = vport_get_dp_port(internal_dev_get_vport(netdev)); - if (new_mtu < 68) return -EINVAL; - if (dp_port) { - if (new_mtu > dp_min_mtu(dp_port->dp)) - return -EINVAL; - } - netdev->mtu = new_mtu; return 0; } -static void internal_dev_free(struct net_device *netdev) -{ - free_netdev(netdev); -} - static int internal_dev_do_ioctl(struct net_device *dev, struct ifreq *ifr, int cmd) { if (dp_ioctl_hook) @@ -149,6 +135,14 @@ static int internal_dev_do_ioctl(struct net_device *dev, struct ifreq *ifr, int return -EOPNOTSUPP; } +static void internal_dev_destructor(struct net_device *dev) +{ + struct vport *vport = internal_dev_get_vport(dev); + + vport_free(vport); + free_netdev(dev); +} + #ifdef HAVE_NET_DEVICE_OPS static const struct net_device_ops internal_dev_netdev_ops = { .ndo_open = internal_dev_open, @@ -177,26 +171,31 @@ static void do_setup(struct net_device *netdev) netdev->change_mtu = internal_dev_change_mtu; #endif - netdev->destructor = internal_dev_free; + netdev->priv_flags &= ~IFF_TX_SKB_SHARING; + netdev->destructor = internal_dev_destructor; SET_ETHTOOL_OPS(netdev, &internal_dev_ethtool_ops); netdev->tx_queue_len = 0; netdev->flags = IFF_BROADCAST | IFF_MULTICAST; - netdev->features = NETIF_F_LLTX | NETIF_F_SG | NETIF_F_HIGHDMA - | NETIF_F_HW_CSUM | NETIF_F_TSO; + netdev->features = NETIF_F_LLTX | NETIF_F_SG | NETIF_F_FRAGLIST | + NETIF_F_HIGHDMA | NETIF_F_HW_CSUM | NETIF_F_TSO; + +#if LINUX_VERSION_CODE >= KERNEL_VERSION(2,6,27) + netdev->vlan_features = netdev->features; + netdev->features |= NETIF_F_HW_VLAN_TX; +#endif vport_gen_rand_ether_addr(netdev->dev_addr); } -static struct vport *internal_dev_create(const char *name, - const void __user *config) +static struct vport *internal_dev_create(const struct vport_parms *parms) { struct vport *vport; struct netdev_vport *netdev_vport; struct internal_dev *internal_dev; int err; - vport = vport_alloc(sizeof(struct netdev_vport), &internal_vport_ops); + vport = vport_alloc(sizeof(struct netdev_vport), &internal_vport_ops, parms); if (IS_ERR(vport)) { err = PTR_ERR(vport); goto error; @@ -204,7 +203,7 @@ static struct vport *internal_dev_create(const char *name, netdev_vport = netdev_vport_priv(vport); - netdev_vport->dev = alloc_netdev(sizeof(struct internal_dev), name, do_setup); + netdev_vport->dev = alloc_netdev(sizeof(struct internal_dev), parms->name, do_setup); if (!netdev_vport->dev) { err = -ENOMEM; goto error_free_vport; @@ -217,6 +216,9 @@ static struct vport *internal_dev_create(const char *name, if (err) goto error_free_netdev; + dev_set_promiscuity(netdev_vport->dev, 1); + netif_start_queue(netdev_vport->dev); + return vport; error_free_netdev: @@ -227,42 +229,15 @@ error: return ERR_PTR(err); } -static int internal_dev_destroy(struct vport *vport) -{ - struct netdev_vport *netdev_vport = netdev_vport_priv(vport); - - unregister_netdevice(netdev_vport->dev); - vport_free(vport); - - return 0; -} - -static int internal_dev_attach(struct vport *vport) -{ - struct netdev_vport *netdev_vport = netdev_vport_priv(vport); - - dev_set_promiscuity(netdev_vport->dev, 1); - - /* It would make sense to assign dev->br_port here too, but - * that causes packets received on internal ports to get caught - * in netdev_frame_hook(). In turn netdev_frame_hook() can reject them - * back to the network stack, but that's a waste of time. */ - - return 0; -} - -static int internal_dev_detach(struct vport *vport) +static void internal_dev_destroy(struct vport *vport) { struct netdev_vport *netdev_vport = netdev_vport_priv(vport); + netif_stop_queue(netdev_vport->dev); dev_set_promiscuity(netdev_vport->dev, -1); - /* Make sure that no packets arrive from now on, since - * internal_dev_xmit() will try to find itself through - * p->dp->ports[], and we're about to set that to null. */ - netif_tx_disable(netdev_vport->dev); - - return 0; + /* unregister_netdevice() waits for an RCU grace period. */ + unregister_netdevice(netdev_vport->dev); } static int internal_dev_recv(struct vport *vport, struct sk_buff *skb) @@ -270,28 +245,31 @@ static int internal_dev_recv(struct vport *vport, struct sk_buff *skb) struct net_device *netdev = netdev_vport_priv(vport)->dev; int len; - skb->dev = netdev; +#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,37) + if (unlikely(vlan_deaccel_tag(skb))) + return 0; +#endif + len = skb->len; + skb->dev = netdev; skb->pkt_type = PACKET_HOST; skb->protocol = eth_type_trans(skb, netdev); + forward_ip_summed(skb, false); - if (in_interrupt()) - netif_rx(skb); - else - netif_rx_ni(skb); + netif_rx(skb); + +#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,29) netdev->last_rx = jiffies; +#endif return len; } -struct vport_ops internal_vport_ops = { - .type = "internal", - .flags = VPORT_F_REQUIRED | VPORT_F_GEN_STATS, +const struct vport_ops internal_vport_ops = { + .type = OVS_VPORT_TYPE_INTERNAL, + .flags = VPORT_F_REQUIRED | VPORT_F_FLOW, .create = internal_dev_create, .destroy = internal_dev_destroy, - .attach = internal_dev_attach, - .detach = internal_dev_detach, - .set_mtu = netdev_set_mtu, .set_addr = netdev_set_addr, .get_name = netdev_get_name, .get_addr = netdev_get_addr, @@ -300,7 +278,6 @@ struct vport_ops internal_vport_ops = { .is_running = netdev_is_running, .get_operstate = netdev_get_operstate, .get_ifindex = netdev_get_ifindex, - .get_iflink = netdev_get_iflink, .get_mtu = netdev_get_mtu, .send = internal_dev_recv, }; @@ -321,6 +298,8 @@ int is_internal_vport(const struct vport *vport) struct vport *internal_dev_get_vport(struct net_device *netdev) { - struct internal_dev *internal_dev = internal_dev_priv(netdev); - return rcu_dereference(internal_dev->vport); + if (!is_internal_dev(netdev)) + return NULL; + + return internal_dev_priv(netdev)->vport; }