datapath: Drop datapath index and port number from Ethtool output.
[openvswitch] / datapath / vport.c
index ee806453676ebc471deec2f856e5bd1c05da7f5e..acb480bb3ccc4a67213b923d5b8bfe3cc1ed26da 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2010 Nicira Networks.
+ * Copyright (c) 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
@@ -136,26 +136,6 @@ error:
        return err;
 }
 
-static void vport_del_all(void)
-{
-       int i;
-
-       rtnl_lock();
-       vport_lock();
-
-       for (i = 0; i < VPORT_HASH_BUCKETS; i++) {
-               struct hlist_head *bucket = &dev_table[i];
-               struct vport *vport;
-               struct hlist_node *node, *next;
-
-               hlist_for_each_entry_safe(vport, node, next, bucket, hash_node)
-                       vport_del(vport);
-       }
-
-       vport_unlock();
-       rtnl_unlock();
-}
-
 /**
  *     vport_exit - shutdown vport subsystem
  *
@@ -166,8 +146,6 @@ void vport_exit(void)
 {
        int i;
 
-       vport_del_all();
-
        for (i = 0; i < n_vport_types; i++) {
                if (vport_ops_list[i]->exit)
                        vport_ops_list[i]->exit();
@@ -577,8 +555,9 @@ void vport_free(struct vport *vport)
  *
  * @parms: Information about new vport.
  *
- * Creates a new vport with the specified configuration (which is dependent
- * on device type).  Both RTNL and vport locks must be held.
+ * Creates a new vport with the specified configuration (which is dependent on
+ * device type) and attaches it to a datapath.  Both RTNL and vport locks must
+ * be held.
  */
 struct vport *vport_add(const struct vport_parms *parms)
 {
@@ -633,9 +612,8 @@ int vport_mod(struct vport *vport, struct odp_port *port)
  *
  * @vport: vport to delete.
  *
- * Deletes the specified device.  The device must not be currently attached to
- * a datapath.  It is possible to fail for reasons such as lack of memory.
- * Both RTNL and vport locks must be held.
+ * Detaches @vport from its datapath and destroys it.  It is possible to fail
+ * for reasons such as lack of memory.  Both RTNL and vport locks must be held.
  */
 int vport_del(struct vport *vport)
 {
@@ -647,42 +625,6 @@ int vport_del(struct vport *vport)
        return vport->ops->destroy(vport);
 }
 
-/**
- *     vport_attach - notify a vport that it has been attached to a datapath
- *
- * @vport: vport to attach.
- *
- * Performs vport-specific actions so that packets may be exchanged.  RTNL lock
- * and the appropriate DP mutex must be held.
- */
-int vport_attach(struct vport *vport)
-{
-       ASSERT_RTNL();
-
-       if (vport->ops->attach)
-               return vport->ops->attach(vport);
-
-       return 0;
-}
-
-/**
- *     vport_detach - detach a vport from a datapath
- *
- * @vport: vport to detach.
- *
- * Performs vport-specific actions before a vport is detached from a datapath.
- * May fail for a variety of reasons, including lack of memory.  RTNL lock and
- * the appropriate DP mutex must be held.
- */
-int vport_detach(struct vport *vport)
-{
-       ASSERT_RTNL();
-
-       if (vport->ops->detach)
-               return vport->ops->detach(vport);
-       return 0;
-}
-
 /**
  *     vport_set_mtu - set device MTU (for kernel callers)
  *
@@ -757,9 +699,7 @@ int vport_set_stats(struct vport *vport, struct rtnl_link_stats64 *stats)
                spin_unlock_bh(&vport->stats_lock);
 
                return 0;
-       } else if (vport->ops->set_stats)
-               return vport->ops->set_stats(vport, stats);
-       else
+       } else
                return -EOPNOTSUPP;
 }
 
@@ -819,6 +759,17 @@ struct kobject *vport_get_kobj(const struct vport *vport)
                return NULL;
 }
 
+static int vport_call_get_stats(struct vport *vport, struct rtnl_link_stats64 *stats)
+{
+       int err;
+
+       rcu_read_lock();
+       err = vport->ops->get_stats(vport, stats);
+       rcu_read_unlock();
+
+       return err;
+}
+
 /**
  *     vport_get_stats - retrieve device stats (for kernel callers)
  *
@@ -829,95 +780,77 @@ struct kobject *vport_get_kobj(const struct vport *vport)
  */
 int vport_get_stats(struct vport *vport, struct rtnl_link_stats64 *stats)
 {
-       struct rtnl_link_stats64 dev_stats;
-       struct rtnl_link_stats64 *dev_statsp = NULL;
-       int err;
-
-       if (vport->ops->get_stats) {
-               if (vport->ops->flags & VPORT_F_GEN_STATS)
-                       dev_statsp = &dev_stats;
-               else
-                       dev_statsp = stats;
-
-               rcu_read_lock();
-               err = vport->ops->get_stats(vport, dev_statsp);
-               rcu_read_unlock();
-
-               if (err)
-                       goto out;
-       }
+       int i;
 
-       if (vport->ops->flags & VPORT_F_GEN_STATS) {
-               int i;
+       if (!(vport->ops->flags & VPORT_F_GEN_STATS))
+               return vport_call_get_stats(vport, stats);
 
-               /* We potentially have 3 sources of stats that need to be
-                * combined: those we have collected (split into err_stats and
-                * percpu_stats), offset_stats from set_stats(), and device
-                * error stats from get_stats() (for errors that happen
-                * downstream and therefore aren't reported through our
-                * vport_record_error() function). */
+       /* We potentially have 3 sources of stats that need to be
+        * combined: those we have collected (split into err_stats and
+        * percpu_stats), offset_stats from set_stats(), and device
+        * error stats from get_stats() (for errors that happen
+        * downstream and therefore aren't reported through our
+        * vport_record_error() function). */
 
-               spin_lock_bh(&vport->stats_lock);
+       spin_lock_bh(&vport->stats_lock);
 
-               *stats = vport->offset_stats;
+       *stats = vport->offset_stats;
 
-               stats->rx_errors        += vport->err_stats.rx_errors;
-               stats->tx_errors        += vport->err_stats.tx_errors;
-               stats->tx_dropped       += vport->err_stats.tx_dropped;
-               stats->rx_dropped       += vport->err_stats.rx_dropped;
+       stats->rx_errors        += vport->err_stats.rx_errors;
+       stats->tx_errors        += vport->err_stats.tx_errors;
+       stats->tx_dropped       += vport->err_stats.tx_dropped;
+       stats->rx_dropped       += vport->err_stats.rx_dropped;
 
-               spin_unlock_bh(&vport->stats_lock);
+       spin_unlock_bh(&vport->stats_lock);
 
-               if (dev_statsp) {
-                       stats->rx_packets          += dev_statsp->rx_packets;
-                       stats->tx_packets          += dev_statsp->tx_packets;
-                       stats->rx_bytes            += dev_statsp->rx_bytes;
-                       stats->tx_bytes            += dev_statsp->tx_bytes;
-                       stats->rx_errors           += dev_statsp->rx_errors;
-                       stats->tx_errors           += dev_statsp->tx_errors;
-                       stats->rx_dropped          += dev_statsp->rx_dropped;
-                       stats->tx_dropped          += dev_statsp->tx_dropped;
-                       stats->multicast           += dev_statsp->multicast;
-                       stats->collisions          += dev_statsp->collisions;
-                       stats->rx_length_errors    += dev_statsp->rx_length_errors;
-                       stats->rx_over_errors      += dev_statsp->rx_over_errors;
-                       stats->rx_crc_errors       += dev_statsp->rx_crc_errors;
-                       stats->rx_frame_errors     += dev_statsp->rx_frame_errors;
-                       stats->rx_fifo_errors      += dev_statsp->rx_fifo_errors;
-                       stats->rx_missed_errors    += dev_statsp->rx_missed_errors;
-                       stats->tx_aborted_errors   += dev_statsp->tx_aborted_errors;
-                       stats->tx_carrier_errors   += dev_statsp->tx_carrier_errors;
-                       stats->tx_fifo_errors      += dev_statsp->tx_fifo_errors;
-                       stats->tx_heartbeat_errors += dev_statsp->tx_heartbeat_errors;
-                       stats->tx_window_errors    += dev_statsp->tx_window_errors;
-                       stats->rx_compressed       += dev_statsp->rx_compressed;
-                       stats->tx_compressed       += dev_statsp->tx_compressed;
-               }
+       if (vport->ops->get_stats) {
+               struct rtnl_link_stats64 dev_stats;
+               int err;
 
-               for_each_possible_cpu(i) {
-                       const struct vport_percpu_stats *percpu_stats;
-                       struct vport_percpu_stats local_stats;
-                       unsigned seqcount;
+               err = vport_call_get_stats(vport, &dev_stats);
+               if (err)
+                       return err;
+
+               stats->rx_errors           += dev_stats.rx_errors;
+               stats->tx_errors           += dev_stats.tx_errors;
+               stats->rx_dropped          += dev_stats.rx_dropped;
+               stats->tx_dropped          += dev_stats.tx_dropped;
+               stats->multicast           += dev_stats.multicast;
+               stats->collisions          += dev_stats.collisions;
+               stats->rx_length_errors    += dev_stats.rx_length_errors;
+               stats->rx_over_errors      += dev_stats.rx_over_errors;
+               stats->rx_crc_errors       += dev_stats.rx_crc_errors;
+               stats->rx_frame_errors     += dev_stats.rx_frame_errors;
+               stats->rx_fifo_errors      += dev_stats.rx_fifo_errors;
+               stats->rx_missed_errors    += dev_stats.rx_missed_errors;
+               stats->tx_aborted_errors   += dev_stats.tx_aborted_errors;
+               stats->tx_carrier_errors   += dev_stats.tx_carrier_errors;
+               stats->tx_fifo_errors      += dev_stats.tx_fifo_errors;
+               stats->tx_heartbeat_errors += dev_stats.tx_heartbeat_errors;
+               stats->tx_window_errors    += dev_stats.tx_window_errors;
+               stats->rx_compressed       += dev_stats.rx_compressed;
+               stats->tx_compressed       += dev_stats.tx_compressed;
+       }
 
-                       percpu_stats = per_cpu_ptr(vport->percpu_stats, i);
+       for_each_possible_cpu(i) {
+               const struct vport_percpu_stats *percpu_stats;
+               struct vport_percpu_stats local_stats;
+               unsigned seqcount;
 
-                       do {
-                               seqcount = read_seqcount_begin(&percpu_stats->seqlock);
-                               local_stats = *percpu_stats;
-                       } while (read_seqcount_retry(&percpu_stats->seqlock, seqcount));
+               percpu_stats = per_cpu_ptr(vport->percpu_stats, i);
 
-                       stats->rx_bytes         += local_stats.rx_bytes;
-                       stats->rx_packets       += local_stats.rx_packets;
-                       stats->tx_bytes         += local_stats.tx_bytes;
-                       stats->tx_packets       += local_stats.tx_packets;
-               }
+               do {
+                       seqcount = read_seqcount_begin(&percpu_stats->seqlock);
+                       local_stats = *percpu_stats;
+               } while (read_seqcount_retry(&percpu_stats->seqlock, seqcount));
 
-               err = 0;
-       } else
-               err = -EOPNOTSUPP;
+               stats->rx_bytes         += local_stats.rx_bytes;
+               stats->rx_packets       += local_stats.rx_packets;
+               stats->tx_bytes         += local_stats.tx_bytes;
+               stats->tx_packets       += local_stats.tx_packets;
+       }
 
-out:
-       return err;
+       return 0;
 }
 
 /**
@@ -964,19 +897,17 @@ unsigned char vport_get_operstate(const struct vport *vport)
  *
  * @vport: vport from which to retrieve index
  *
- * Retrieves the system interface index of the given device.  Not all devices
- * will have system indexes, in which case the index of the datapath local
- * port is returned.  Returns a negative index on error.  Either RTNL lock or
+ * Retrieves the system interface index of the given device or 0 if
+ * the device does not have one (in the case of virtual ports).
+ * Returns a negative index on error. Either RTNL lock or
  * rcu_read_lock must be held.
  */
 int vport_get_ifindex(const struct vport *vport)
 {
        if (vport->ops->get_ifindex)
                return vport->ops->get_ifindex(vport);
-
-       /* If we don't actually have an ifindex, use the local port's.
-        * Userspace doesn't check it anyways. */
-       return vport_get_ifindex(vport->dp->ports[ODPP_LOCAL]);
+       else
+               return 0;
 }
 
 /**
@@ -1013,6 +944,22 @@ int vport_get_mtu(const struct vport *vport)
        return vport->ops->get_mtu(vport);
 }
 
+/**
+ *     vport_get_config - retrieve device configuration
+ *
+ * @vport: vport from which to retrieve the configuration.
+ * @config: buffer to store config, which must be at least the length
+ *          of VPORT_CONFIG_SIZE.
+ *
+ * Retrieves the configuration of the given device.  Either RTNL lock or
+ * rcu_read_lock must be held.
+ */
+void vport_get_config(const struct vport *vport, void *config)
+{
+       if (vport->ops->get_config)
+               vport->ops->get_config(vport, config);
+}
+
 /**
  *     vport_receive - pass up received packet to the datapath for processing
  *