From: Giuseppe Lettieri <g.lettieri@iet.unipi.it>
Date: Thu, 12 Sep 2013 13:01:23 +0000 (+0200)
Subject: Merge branch 'mainstream'
X-Git-Tag: sliver-openvswitch-2.0.90-1~15
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=03f976b173e2e670eb2fa77eae8eacd2abf4bf84;hp=-c;p=sliver-openvswitch.git

Merge branch 'mainstream'
---

03f976b173e2e670eb2fa77eae8eacd2abf4bf84
diff --combined lib/dpif-netdev.c
index ee60dcd24,0d489ba2d..461e0dc09
--- a/lib/dpif-netdev.c
+++ b/lib/dpif-netdev.c
@@@ -201,17 -201,10 +201,17 @@@ dpif_netdev_class_is_dummy(const struc
      return class != &dpif_netdev_class;
  }
  
 +static bool
 +dpif_netdev_class_is_planetlab(const struct dpif_class *class)
 +{
 +    return class == &dpif_planetlab_class;
 +}
 +
  static const char *
  dpif_netdev_port_open_type(const struct dpif_class *class, const char *type)
  {
      return strcmp(type, "internal") ? type
 +                  : dpif_netdev_class_is_planetlab(class) ? "pltap"
                    : dpif_netdev_class_is_dummy(class) ? "dummy"
                    : "tap";
  }
@@@ -239,8 -232,7 +239,8 @@@ choose_port(struct dp_netdev *dp, cons
  {
      uint32_t port_no;
  
 -    if (dp->class != &dpif_netdev_class) {
 +    if (dp->class != &dpif_netdev_class && 
 +        dp->class != &dpif_planetlab_class) {
          const char *p;
          int start_no = 0;
  
@@@ -415,6 -407,7 +415,7 @@@ do_add_port(struct dp_netdev *dp, cons
      struct dp_netdev_port *port;
      struct netdev *netdev;
      struct netdev_rx *rx;
+     enum netdev_flags flags;
      const char *open_type;
      int mtu;
      int error;
@@@ -427,9 -420,15 +428,15 @@@
      if (error) {
          return error;
      }
-     /* XXX reject loopback devices */
      /* XXX reject non-Ethernet devices */
  
+     netdev_get_flags(netdev, &flags);
+     if (flags & NETDEV_LOOPBACK) {
+         VLOG_ERR("%s: cannot add a loopback device", devname);
+         netdev_close(netdev);
+         return EINVAL;
+     }
+ 
      error = netdev_rx_open(netdev, &rx);
      if (error
          && !(error == EOPNOTSUPP && dpif_netdev_class_is_dummy(dp->class))) {
@@@ -1305,49 -1304,41 +1312,49 @@@ dp_netdev_execute_actions(struct dp_net
                          dp_netdev_output_port, dp_netdev_action_userspace);
  }
  
 +#define DPIF_NETDEV_CLASS_FUNCTIONS			\
 +    dpif_netdev_enumerate,				\
 +    dpif_netdev_port_open_type,				\
 +    dpif_netdev_open,					\
 +    dpif_netdev_close,					\
 +    dpif_netdev_destroy,				\
 +    dpif_netdev_run,					\
 +    dpif_netdev_wait,					\
 +    dpif_netdev_get_stats,				\
 +    dpif_netdev_port_add,				\
 +    dpif_netdev_port_del,				\
 +    dpif_netdev_port_query_by_number,			\
 +    dpif_netdev_port_query_by_name,			\
 +    dpif_netdev_get_max_ports,				\
 +    NULL,                       /* port_get_pid */	\
 +    dpif_netdev_port_dump_start,			\
 +    dpif_netdev_port_dump_next,				\
 +    dpif_netdev_port_dump_done,				\
 +    dpif_netdev_port_poll,				\
 +    dpif_netdev_port_poll_wait,				\
 +    dpif_netdev_flow_get,				\
 +    dpif_netdev_flow_put,				\
 +    dpif_netdev_flow_del,				\
 +    dpif_netdev_flow_flush,				\
 +    dpif_netdev_flow_dump_start,			\
 +    dpif_netdev_flow_dump_next,				\
 +    dpif_netdev_flow_dump_done,				\
 +    dpif_netdev_execute,				\
 +    NULL,                       /* operate */		\
 +    dpif_netdev_recv_set,				\
 +    dpif_netdev_queue_to_priority,			\
 +    dpif_netdev_recv,					\
 +    dpif_netdev_recv_wait,				\
 +    dpif_netdev_recv_purge,				\
 +
  const struct dpif_class dpif_netdev_class = {
      "netdev",
 -    dpif_netdev_enumerate,
 -    dpif_netdev_port_open_type,
 -    dpif_netdev_open,
 -    dpif_netdev_close,
 -    dpif_netdev_destroy,
 -    dpif_netdev_run,
 -    dpif_netdev_wait,
 -    dpif_netdev_get_stats,
 -    dpif_netdev_port_add,
 -    dpif_netdev_port_del,
 -    dpif_netdev_port_query_by_number,
 -    dpif_netdev_port_query_by_name,
 -    dpif_netdev_get_max_ports,
 -    NULL,                       /* port_get_pid */
 -    dpif_netdev_port_dump_start,
 -    dpif_netdev_port_dump_next,
 -    dpif_netdev_port_dump_done,
 -    dpif_netdev_port_poll,
 -    dpif_netdev_port_poll_wait,
 -    dpif_netdev_flow_get,
 -    dpif_netdev_flow_put,
 -    dpif_netdev_flow_del,
 -    dpif_netdev_flow_flush,
 -    dpif_netdev_flow_dump_start,
 -    dpif_netdev_flow_dump_next,
 -    dpif_netdev_flow_dump_done,
 -    dpif_netdev_execute,
 -    NULL,                       /* operate */
 -    dpif_netdev_recv_set,
 -    dpif_netdev_queue_to_priority,
 -    dpif_netdev_recv,
 -    dpif_netdev_recv_wait,
 -    dpif_netdev_recv_purge,
 +    DPIF_NETDEV_CLASS_FUNCTIONS
 +};
 +
 +const struct dpif_class dpif_planetlab_class = {
 +    "planetlab",
 +    DPIF_NETDEV_CLASS_FUNCTIONS
  };
  
  static void
@@@ -1419,4 -1410,3 +1426,4 @@@ dpif_dummy_register(bool override
                               "DP PORT NEW-NUMBER",
                               3, 3, dpif_dummy_change_port_number, NULL);
  }
 +