include cleanup: Update gfp.h and slab.h includes to prepare for breaking implicit...
[safe/jmp/linux-2.6] / arch / um / drivers / net_kern.c
index 501f956..f053726 100644 (file)
 /*
- * Copyright (C) 2001 Lennert Buytenhek (buytenh@gnu.org) and 
+ * Copyright (C) 2001 - 2007 Jeff Dike (jdike@{addtoit,linux.intel}.com)
+ * Copyright (C) 2001 Lennert Buytenhek (buytenh@gnu.org) and
  * James Leu (jleu@mindspring.net).
  * Copyright (C) 2001 by various other people who didn't put their name here.
  * Licensed under the GPL.
  */
 
-#include "linux/config.h"
-#include "linux/kernel.h"
-#include "linux/netdevice.h"
-#include "linux/rtnetlink.h"
-#include "linux/skbuff.h"
-#include "linux/socket.h"
-#include "linux/spinlock.h"
-#include "linux/module.h"
-#include "linux/init.h"
-#include "linux/etherdevice.h"
-#include "linux/list.h"
-#include "linux/inetdevice.h"
-#include "linux/ctype.h"
-#include "linux/bootmem.h"
-#include "linux/ethtool.h"
-#include "linux/platform_device.h"
-#include "asm/uaccess.h"
-#include "user_util.h"
-#include "kern_util.h"
-#include "net_kern.h"
-#include "net_user.h"
-#include "mconsole_kern.h"
+#include <linux/bootmem.h>
+#include <linux/etherdevice.h>
+#include <linux/ethtool.h>
+#include <linux/inetdevice.h>
+#include <linux/init.h>
+#include <linux/list.h>
+#include <linux/netdevice.h>
+#include <linux/platform_device.h>
+#include <linux/rtnetlink.h>
+#include <linux/skbuff.h>
+#include <linux/slab.h>
+#include <linux/spinlock.h>
 #include "init.h"
-#include "irq_user.h"
 #include "irq_kern.h"
+#include "irq_user.h"
+#include "mconsole_kern.h"
+#include "net_kern.h"
+#include "net_user.h"
+
+static inline void set_ether_mac(struct net_device *dev, unsigned char *addr)
+{
+       memcpy(dev->dev_addr, addr, ETH_ALEN);
+}
 
 #define DRIVER_NAME "uml-netdev"
 
 static DEFINE_SPINLOCK(opened_lock);
 static LIST_HEAD(opened);
 
+/*
+ * The drop_skb is used when we can't allocate an skb.  The
+ * packet is read into drop_skb in order to get the data off the
+ * connection to the host.
+ * It is reallocated whenever a maximum packet size is seen which is
+ * larger than any seen before.  update_drop_skb is called from
+ * eth_configure when a new interface is added.
+ */
+static DEFINE_SPINLOCK(drop_lock);
+static struct sk_buff *drop_skb;
+static int drop_max;
+
+static int update_drop_skb(int max)
+{
+       struct sk_buff *new;
+       unsigned long flags;
+       int err = 0;
+
+       spin_lock_irqsave(&drop_lock, flags);
+
+       if (max <= drop_max)
+               goto out;
+
+       err = -ENOMEM;
+       new = dev_alloc_skb(max);
+       if (new == NULL)
+               goto out;
+
+       skb_put(new, max);
+
+       kfree_skb(drop_skb);
+       drop_skb = new;
+       drop_max = max;
+       err = 0;
+out:
+       spin_unlock_irqrestore(&drop_lock, flags);
+
+       return err;
+}
+
 static int uml_net_rx(struct net_device *dev)
 {
-       struct uml_net_private *lp = dev->priv;
+       struct uml_net_private *lp = netdev_priv(dev);
        int pkt_len;
        struct sk_buff *skb;
 
        /* If we can't allocate memory, try again next round. */
-       skb = dev_alloc_skb(dev->mtu);
+       skb = dev_alloc_skb(lp->max_packet);
        if (skb == NULL) {
-               lp->stats.rx_dropped++;
+               drop_skb->dev = dev;
+               /* Read a packet into drop_skb and don't do anything with it. */
+               (*lp->read)(lp->fd, drop_skb, lp);
+               dev->stats.rx_dropped++;
                return 0;
        }
 
        skb->dev = dev;
-       skb_put(skb, dev->mtu);
-       skb->mac.raw = skb->data;
-       pkt_len = (*lp->read)(lp->fd, &skb, lp);
+       skb_put(skb, lp->max_packet);
+       skb_reset_mac_header(skb);
+       pkt_len = (*lp->read)(lp->fd, skb, lp);
 
        if (pkt_len > 0) {
                skb_trim(skb, pkt_len);
                skb->protocol = (*lp->protocol)(skb);
-               netif_rx(skb);
 
-               lp->stats.rx_bytes += skb->len;
-               lp->stats.rx_packets++;
+               dev->stats.rx_bytes += skb->len;
+               dev->stats.rx_packets++;
+               netif_rx(skb);
                return pkt_len;
        }
 
@@ -68,68 +110,64 @@ static int uml_net_rx(struct net_device *dev)
        return pkt_len;
 }
 
-static void uml_dev_close(void* dev)
+static void uml_dev_close(struct work_struct *work)
 {
-       dev_close( (struct net_device *) dev);
+       struct uml_net_private *lp =
+               container_of(work, struct uml_net_private, work);
+       dev_close(lp->dev);
 }
 
-irqreturn_t uml_net_interrupt(int irq, void *dev_id, struct pt_regs *regs)
+static irqreturn_t uml_net_interrupt(int irq, void *dev_id)
 {
        struct net_device *dev = dev_id;
-       struct uml_net_private *lp = dev->priv;
+       struct uml_net_private *lp = netdev_priv(dev);
        int err;
 
-       if(!netif_running(dev))
-               return(IRQ_NONE);
+       if (!netif_running(dev))
+               return IRQ_NONE;
 
        spin_lock(&lp->lock);
-       while((err = uml_net_rx(dev)) > 0) ;
-       if(err < 0) {
-               DECLARE_WORK(close_work, uml_dev_close, dev);
-               printk(KERN_ERR 
-                      "Device '%s' read returned %d, shutting it down\n", 
+       while ((err = uml_net_rx(dev)) > 0) ;
+       if (err < 0) {
+               printk(KERN_ERR
+                      "Device '%s' read returned %d, shutting it down\n",
                       dev->name, err);
                /* dev_close can't be called in interrupt context, and takes
                 * again lp->lock.
                 * And dev_close() can be safely called multiple times on the
                 * same device, since it tests for (dev->flags & IFF_UP). So
-                * there's no harm in delaying the device shutdown. */
-               schedule_work(&close_work);
+                * there's no harm in delaying the device shutdown.
+                * Furthermore, the workqueue will not re-enqueue an already
+                * enqueued work item. */
+               schedule_work(&lp->work);
                goto out;
        }
        reactivate_fd(lp->fd, UM_ETH_IRQ);
 
 out:
        spin_unlock(&lp->lock);
-       return(IRQ_HANDLED);
+       return IRQ_HANDLED;
 }
 
 static int uml_net_open(struct net_device *dev)
 {
-       struct uml_net_private *lp = dev->priv;
+       struct uml_net_private *lp = netdev_priv(dev);
        int err;
 
-       spin_lock(&lp->lock);
-
-       if(lp->fd >= 0){
+       if (lp->fd >= 0) {
                err = -ENXIO;
                goto out;
        }
 
-       if(!lp->have_mac){
-               dev_ip_addr(dev, &lp->mac[2]);
-               set_ether_mac(dev, lp->mac);
-       }
-
        lp->fd = (*lp->open)(&lp->user);
-       if(lp->fd < 0){
+       if (lp->fd < 0) {
                err = lp->fd;
                goto out;
        }
 
        err = um_request_irq(dev->irq, lp->fd, IRQ_READ, uml_net_interrupt,
                             IRQF_DISABLED | IRQF_SHARED, dev->name, dev);
-       if(err != 0){
+       if (err != 0) {
                printk(KERN_ERR "uml_net_open: failed to get irq(%d)\n", err);
                err = -ENETUNREACH;
                goto out_close;
@@ -142,9 +180,7 @@ static int uml_net_open(struct net_device *dev)
         * is full when we get here.  In this case, new data is never queued,
         * SIGIOs never arrive, and the net never works.
         */
-       while((err = uml_net_rx(dev)) > 0) ;
-
-       spin_unlock(&lp->lock);
+       while ((err = uml_net_rx(dev)) > 0) ;
 
        spin_lock(&opened_lock);
        list_add(&lp->list, &opened);
@@ -152,27 +188,23 @@ static int uml_net_open(struct net_device *dev)
 
        return 0;
 out_close:
-       if(lp->close != NULL) (*lp->close)(lp->fd, &lp->user);
+       if (lp->close != NULL) (*lp->close)(lp->fd, &lp->user);
        lp->fd = -1;
 out:
-       spin_unlock(&lp->lock);
        return err;
 }
 
 static int uml_net_close(struct net_device *dev)
 {
-       struct uml_net_private *lp = dev->priv;
-       
+       struct uml_net_private *lp = netdev_priv(dev);
+
        netif_stop_queue(dev);
-       spin_lock(&lp->lock);
 
        free_irq(dev->irq, dev);
-       if(lp->close != NULL)
+       if (lp->close != NULL)
                (*lp->close)(lp->fd, &lp->user);
        lp->fd = -1;
 
-       spin_unlock(&lp->lock);
-
        spin_lock(&opened_lock);
        list_del(&lp->list);
        spin_unlock(&opened_lock);
@@ -182,7 +214,7 @@ static int uml_net_close(struct net_device *dev)
 
 static int uml_net_start_xmit(struct sk_buff *skb, struct net_device *dev)
 {
-       struct uml_net_private *lp = dev->priv;
+       struct uml_net_private *lp = netdev_priv(dev);
        unsigned long flags;
        int len;
 
@@ -190,20 +222,20 @@ static int uml_net_start_xmit(struct sk_buff *skb, struct net_device *dev)
 
        spin_lock_irqsave(&lp->lock, flags);
 
-       len = (*lp->write)(lp->fd, &skb, lp);
+       len = (*lp->write)(lp->fd, skb, lp);
 
-       if(len == skb->len) {
-               lp->stats.tx_packets++;
-               lp->stats.tx_bytes += skb->len;
+       if (len == skb->len) {
+               dev->stats.tx_packets++;
+               dev->stats.tx_bytes += skb->len;
                dev->trans_start = jiffies;
                netif_start_queue(dev);
 
                /* this is normally done in the interrupt when tx finishes */
                netif_wake_queue(dev);
-       } 
-       else if(len == 0){
+       }
+       else if (len == 0) {
                netif_start_queue(dev);
-               lp->stats.tx_dropped++;
+               dev->stats.tx_dropped++;
        }
        else {
                netif_start_queue(dev);
@@ -214,20 +246,12 @@ static int uml_net_start_xmit(struct sk_buff *skb, struct net_device *dev)
 
        dev_kfree_skb(skb);
 
-       return 0;
-}
-
-static struct net_device_stats *uml_net_get_stats(struct net_device *dev)
-{
-       struct uml_net_private *lp = dev->priv;
-       return &lp->stats;
+       return NETDEV_TX_OK;
 }
 
 static void uml_net_set_multicast_list(struct net_device *dev)
 {
-       if (dev->flags & IFF_PROMISC) return;
-       else if (dev->mc_count) dev->flags |= IFF_ALLMULTI;
-       else dev->flags &= ~IFF_ALLMULTI;
+       return;
 }
 
 static void uml_net_tx_timeout(struct net_device *dev)
@@ -238,34 +262,21 @@ static void uml_net_tx_timeout(struct net_device *dev)
 
 static int uml_net_set_mac(struct net_device *dev, void *addr)
 {
-       struct uml_net_private *lp = dev->priv;
+       struct uml_net_private *lp = netdev_priv(dev);
        struct sockaddr *hwaddr = addr;
 
-       spin_lock(&lp->lock);
-       memcpy(dev->dev_addr, hwaddr->sa_data, ETH_ALEN);
-       spin_unlock(&lp->lock);
+       spin_lock_irq(&lp->lock);
+       set_ether_mac(dev, hwaddr->sa_data);
+       spin_unlock_irq(&lp->lock);
 
-       return(0);
+       return 0;
 }
 
 static int uml_net_change_mtu(struct net_device *dev, int new_mtu)
 {
-       struct uml_net_private *lp = dev->priv;
-       int err = 0;
-
-       spin_lock(&lp->lock);
-
-       new_mtu = (*lp->set_mtu)(new_mtu, &lp->user);
-       if(new_mtu < 0){
-               err = new_mtu;
-               goto out;
-       }
-
        dev->mtu = new_mtu;
 
- out:
-       spin_unlock(&lp->lock);
-       return err;
+       return 0;
 }
 
 static void uml_net_get_drvinfo(struct net_device *dev,
@@ -275,12 +286,12 @@ static void uml_net_get_drvinfo(struct net_device *dev,
        strcpy(info->version, "42");
 }
 
-static struct ethtool_ops uml_net_ethtool_ops = {
+static const struct ethtool_ops uml_net_ethtool_ops = {
        .get_drvinfo    = uml_net_get_drvinfo,
        .get_link       = ethtool_op_get_link,
 };
 
-void uml_net_user_timer_expire(unsigned long _conn)
+static void uml_net_user_timer_expire(unsigned long _conn)
 {
 #ifdef undef
        struct connection *conn = (struct connection *)_conn;
@@ -290,6 +301,55 @@ void uml_net_user_timer_expire(unsigned long _conn)
 #endif
 }
 
+static void setup_etheraddr(char *str, unsigned char *addr, char *name)
+{
+       char *end;
+       int i;
+
+       if (str == NULL)
+               goto random;
+
+       for (i = 0; i < 6; i++) {
+               addr[i] = simple_strtoul(str, &end, 16);
+               if ((end == str) ||
+                  ((*end != ':') && (*end != ',') && (*end != '\0'))) {
+                       printk(KERN_ERR
+                              "setup_etheraddr: failed to parse '%s' "
+                              "as an ethernet address\n", str);
+                       goto random;
+               }
+               str = end + 1;
+       }
+       if (is_multicast_ether_addr(addr)) {
+               printk(KERN_ERR
+                      "Attempt to assign a multicast ethernet address to a "
+                      "device disallowed\n");
+               goto random;
+       }
+       if (!is_valid_ether_addr(addr)) {
+               printk(KERN_ERR
+                      "Attempt to assign an invalid ethernet address to a "
+                      "device disallowed\n");
+               goto random;
+       }
+       if (!is_local_ether_addr(addr)) {
+               printk(KERN_WARNING
+                      "Warning: Assigning a globally valid ethernet "
+                      "address to a device\n");
+               printk(KERN_WARNING "You should set the 2nd rightmost bit in "
+                      "the first byte of the MAC,\n");
+               printk(KERN_WARNING "i.e. %02x:%02x:%02x:%02x:%02x:%02x\n",
+                      addr[0] | 0x02, addr[1], addr[2], addr[3], addr[4],
+                      addr[5]);
+       }
+       return;
+
+random:
+       printk(KERN_INFO
+              "Choosing a random ethernet address for device %s\n", name);
+       random_ether_addr(addr);
+}
+
 static DEFINE_SPINLOCK(devices_lock);
 static LIST_HEAD(devices);
 
@@ -298,53 +358,80 @@ static struct platform_driver uml_net_driver = {
                .name  = DRIVER_NAME,
        },
 };
+
+static void net_device_release(struct device *dev)
+{
+       struct uml_net *device = dev_get_drvdata(dev);
+       struct net_device *netdev = device->dev;
+       struct uml_net_private *lp = netdev_priv(netdev);
+
+       if (lp->remove != NULL)
+               (*lp->remove)(&lp->user);
+       list_del(&device->list);
+       kfree(device);
+       free_netdev(netdev);
+}
+
+static const struct net_device_ops uml_netdev_ops = {
+       .ndo_open               = uml_net_open,
+       .ndo_stop               = uml_net_close,
+       .ndo_start_xmit         = uml_net_start_xmit,
+       .ndo_set_multicast_list = uml_net_set_multicast_list,
+       .ndo_tx_timeout         = uml_net_tx_timeout,
+       .ndo_set_mac_address    = uml_net_set_mac,
+       .ndo_change_mtu         = uml_net_change_mtu,
+       .ndo_set_mac_address    = eth_mac_addr,
+       .ndo_validate_addr      = eth_validate_addr,
+};
+
+/*
+ * Ensures that platform_driver_register is called only once by
+ * eth_configure.  Will be set in an initcall.
+ */
 static int driver_registered;
 
-static int eth_configure(int n, void *init, char *mac,
-                        struct transport *transport)
+static void eth_configure(int n, void *init, char *mac,
+                         struct transport *transport)
 {
        struct uml_net *device;
        struct net_device *dev;
        struct uml_net_private *lp;
-       int save, err, size;
+       int err, size;
 
-       size = transport->private_size + sizeof(struct uml_net_private) + 
-               sizeof(((struct uml_net_private *) 0)->user);
+       size = transport->private_size + sizeof(struct uml_net_private);
 
-       device = kmalloc(sizeof(*device), GFP_KERNEL);
+       device = kzalloc(sizeof(*device), GFP_KERNEL);
        if (device == NULL) {
-               printk(KERN_ERR "eth_configure failed to allocate uml_net\n");
-               return(1);
+               printk(KERN_ERR "eth_configure failed to allocate struct "
+                      "uml_net\n");
+               return;
+       }
+
+       dev = alloc_etherdev(size);
+       if (dev == NULL) {
+               printk(KERN_ERR "eth_configure: failed to allocate struct "
+                      "net_device for eth%d\n", n);
+               goto out_free_device;
        }
 
-       memset(device, 0, sizeof(*device));
        INIT_LIST_HEAD(&device->list);
        device->index = n;
 
-       spin_lock(&devices_lock);
-       list_add(&device->list, &devices);
-       spin_unlock(&devices_lock);
+       /* If this name ends up conflicting with an existing registered
+        * netdevice, that is OK, register_netdev{,ice}() will notice this
+        * and fail.
+        */
+       snprintf(dev->name, sizeof(dev->name), "eth%d", n);
 
-       if (setup_etheraddr(mac, device->mac))
-               device->have_mac = 1;
+       setup_etheraddr(mac, device->mac, dev->name);
 
-       printk(KERN_INFO "Netdevice %d ", n);
-       if (device->have_mac)
-               printk("(%02x:%02x:%02x:%02x:%02x:%02x) ",
-                      device->mac[0], device->mac[1],
-                      device->mac[2], device->mac[3],
-                      device->mac[4], device->mac[5]);
-       printk(": ");
-       dev = alloc_etherdev(size);
-       if (dev == NULL) {
-               printk(KERN_ERR "eth_configure: failed to allocate device\n");
-               return 1;
-       }
+       printk(KERN_INFO "Netdevice %d (%pM) : ", n, device->mac);
 
-       lp = dev->priv;
+       lp = netdev_priv(dev);
        /* This points to the transport private data. It's still clear, but we
         * must memset it to 0 *now*. Let's help the drivers. */
        memset(lp, 0, size);
+       INIT_WORK(&lp->work, uml_dev_close);
 
        /* sysfs register */
        if (!driver_registered) {
@@ -353,53 +440,26 @@ static int eth_configure(int n, void *init, char *mac,
        }
        device->pdev.id = n;
        device->pdev.name = DRIVER_NAME;
-       platform_device_register(&device->pdev);
+       device->pdev.dev.release = net_device_release;
+       dev_set_drvdata(&device->pdev.dev, device);
+       if (platform_device_register(&device->pdev))
+               goto out_free_netdev;
        SET_NETDEV_DEV(dev,&device->pdev.dev);
 
-       /* If this name ends up conflicting with an existing registered
-        * netdevice, that is OK, register_netdev{,ice}() will notice this
-        * and fail.
-        */
-       snprintf(dev->name, sizeof(dev->name), "eth%d", n);
        device->dev = dev;
 
+       /*
+        * These just fill in a data structure, so there's no failure
+        * to be worried about.
+        */
        (*transport->kern->init)(dev, init);
 
-       dev->mtu = transport->user->max_packet;
-       dev->open = uml_net_open;
-       dev->hard_start_xmit = uml_net_start_xmit;
-       dev->stop = uml_net_close;
-       dev->get_stats = uml_net_get_stats;
-       dev->set_multicast_list = uml_net_set_multicast_list;
-       dev->tx_timeout = uml_net_tx_timeout;
-       dev->set_mac_address = uml_net_set_mac;
-       dev->change_mtu = uml_net_change_mtu;
-       dev->ethtool_ops = &uml_net_ethtool_ops;
-       dev->watchdog_timeo = (HZ >> 1);
-       dev->irq = UM_ETH_IRQ;
-
-       rtnl_lock();
-       err = register_netdevice(dev);
-       rtnl_unlock();
-       if (err) {
-               device->dev = NULL;
-               /* XXX: should we call ->remove() here? */
-               free_netdev(dev);
-               return 1;
-       }
-
-       /* lp.user is the first four bytes of the transport data, which
-        * has already been initialized.  This structure assignment will
-        * overwrite that, so we make sure that .user gets overwritten with
-        * what it already has.
-        */
-       save = lp->user[0];
        *lp = ((struct uml_net_private)
                { .list                 = LIST_HEAD_INIT(lp->list),
                  .dev                  = dev,
                  .fd                   = -1,
                  .mac                  = { 0xfe, 0xfd, 0x0, 0x0, 0x0, 0x0},
-                 .have_mac             = device->have_mac,
+                 .max_packet           = transport->user->max_packet,
                  .protocol             = transport->kern->protocol,
                  .open                 = transport->user->open,
                  .close                = transport->user->close,
@@ -407,23 +467,50 @@ static int eth_configure(int n, void *init, char *mac,
                  .read                 = transport->kern->read,
                  .write                = transport->kern->write,
                  .add_address          = transport->user->add_address,
-                 .delete_address       = transport->user->delete_address,
-                 .set_mtu              = transport->user->set_mtu,
-                 .user                 = { save } });
+                 .delete_address       = transport->user->delete_address });
 
        init_timer(&lp->tl);
        spin_lock_init(&lp->lock);
        lp->tl.function = uml_net_user_timer_expire;
-       if (lp->have_mac)
-               memcpy(lp->mac, device->mac, sizeof(lp->mac));
+       memcpy(lp->mac, device->mac, sizeof(lp->mac));
 
-       if (transport->user->init) 
-               (*transport->user->init)(&lp->user, dev);
+       if ((transport->user->init != NULL) &&
+           ((*transport->user->init)(&lp->user, dev) != 0))
+               goto out_unregister;
 
-       if (device->have_mac)
-               set_ether_mac(dev, device->mac);
+       set_ether_mac(dev, device->mac);
+       dev->mtu = transport->user->mtu;
+       dev->netdev_ops = &uml_netdev_ops;
+       dev->ethtool_ops = &uml_net_ethtool_ops;
+       dev->watchdog_timeo = (HZ >> 1);
+       dev->irq = UM_ETH_IRQ;
 
-       return 0;
+       err = update_drop_skb(lp->max_packet);
+       if (err)
+               goto out_undo_user_init;
+
+       rtnl_lock();
+       err = register_netdevice(dev);
+       rtnl_unlock();
+       if (err)
+               goto out_undo_user_init;
+
+       spin_lock(&devices_lock);
+       list_add(&device->list, &devices);
+       spin_unlock(&devices_lock);
+
+       return;
+
+out_undo_user_init:
+       if (transport->user->remove != NULL)
+               (*transport->user->remove)(&lp->user);
+out_unregister:
+       platform_device_unregister(&device->pdev);
+       return; /* platform_device_unregister frees dev and device */
+out_free_netdev:
+       free_netdev(dev);
+out_free_device:
+       kfree(device);
 }
 
 static struct uml_net *find_device(int n)
@@ -432,46 +519,44 @@ static struct uml_net *find_device(int n)
        struct list_head *ele;
 
        spin_lock(&devices_lock);
-       list_for_each(ele, &devices){
+       list_for_each(ele, &devices) {
                device = list_entry(ele, struct uml_net, list);
-               if(device->index == n)
+               if (device->index == n)
                        goto out;
        }
        device = NULL;
  out:
        spin_unlock(&devices_lock);
-       return(device);
+       return device;
 }
 
-static int eth_parse(char *str, int *index_out, char **str_out)
+static int eth_parse(char *str, int *index_out, char **str_out,
+                    char **error_out)
 {
        char *end;
-       int n;
+       int n, err = -EINVAL;
 
        n = simple_strtoul(str, &end, 0);
-       if(end == str){
-               printk(KERN_ERR "eth_setup: Failed to parse '%s'\n", str);
-               return(1);
-       }
-       if(n < 0){
-               printk(KERN_ERR "eth_setup: device %d is negative\n", n);
-               return(1);
+       if (end == str) {
+               *error_out = "Bad device number";
+               return err;
        }
+
        str = end;
-       if(*str != '='){
-               printk(KERN_ERR 
-                      "eth_setup: expected '=' after device number\n");
-               return(1);
+       if (*str != '=') {
+               *error_out = "Expected '=' after device number";
+               return err;
        }
+
        str++;
-       if(find_device(n)){
-               printk(KERN_ERR "eth_setup: Device %d already configured\n",
-                      n);
-               return(1);
+       if (find_device(n)) {
+               *error_out = "Device already configured";
+               return err;
        }
-       if(index_out) *index_out = n;
+
+       *index_out = n;
        *str_out = str;
-       return(0);
+       return 0;
 }
 
 struct eth_init {
@@ -480,13 +565,11 @@ struct eth_init {
        int index;
 };
 
-/* Filled in at boot time.  Will need locking if the transports become
- * modular.
- */
-struct list_head transports = LIST_HEAD_INIT(transports);
+static DEFINE_SPINLOCK(transports_lock);
+static LIST_HEAD(transports);
 
 /* Filled in during early boot */
-struct list_head eth_cmd_line = LIST_HEAD_INIT(eth_cmd_line);
+static LIST_HEAD(eth_cmd_line);
 
 static int check_transport(struct transport *transport, char *eth, int n,
                           void **init_out, char **mac_out)
@@ -494,24 +577,24 @@ static int check_transport(struct transport *transport, char *eth, int n,
        int len;
 
        len = strlen(transport->name);
-       if(strncmp(eth, transport->name, len))
-               return(0);
+       if (strncmp(eth, transport->name, len))
+               return 0;
 
        eth += len;
-       if(*eth == ',')
+       if (*eth == ',')
                eth++;
-       else if(*eth != '\0')
-               return(0);
+       else if (*eth != '\0')
+               return 0;
 
        *init_out = kmalloc(transport->setup_size, GFP_KERNEL);
-       if(*init_out == NULL)
-               return(1);
+       if (*init_out == NULL)
+               return 1;
 
-       if(!transport->setup(eth, mac_out, *init_out)){
+       if (!transport->setup(eth, mac_out, *init_out)) {
                kfree(*init_out);
                *init_out = NULL;
        }
-       return(1);
+       return 1;
 }
 
 void register_transport(struct transport *new)
@@ -522,15 +605,18 @@ void register_transport(struct transport *new)
        char *mac = NULL;
        int match;
 
+       spin_lock(&transports_lock);
+       BUG_ON(!list_empty(&new->list));
        list_add(&new->list, &transports);
+       spin_unlock(&transports_lock);
 
-       list_for_each_safe(ele, next, &eth_cmd_line){
+       list_for_each_safe(ele, next, &eth_cmd_line) {
                eth = list_entry(ele, struct eth_init, list);
                match = check_transport(new, eth->init, eth->index, &init,
                                        &mac);
-               if(!match)
+               if (!match)
                        continue;
-               else if(init != NULL){
+               else if (init != NULL) {
                        eth_configure(eth->index, init, mac, new);
                        kfree(init);
                }
@@ -544,32 +630,42 @@ static int eth_setup_common(char *str, int index)
        struct transport *transport;
        void *init;
        char *mac = NULL;
+       int found = 0;
 
-       list_for_each(ele, &transports){
+       spin_lock(&transports_lock);
+       list_for_each(ele, &transports) {
                transport = list_entry(ele, struct transport, list);
-               if(!check_transport(transport, str, index, &init, &mac))
+               if (!check_transport(transport, str, index, &init, &mac))
                        continue;
-               if(init != NULL){
+               if (init != NULL) {
                        eth_configure(index, init, mac, transport);
                        kfree(init);
                }
-               return(1);
+               found = 1;
+               break;
        }
-       return(0);
+
+       spin_unlock(&transports_lock);
+       return found;
 }
 
-static int eth_setup(char *str)
+static int __init eth_setup(char *str)
 {
        struct eth_init *new;
+       char *error;
        int n, err;
 
-       err = eth_parse(str, &n, &str);
-       if(err) return(1);
+       err = eth_parse(str, &n, &str, &error);
+       if (err) {
+               printk(KERN_ERR "eth_setup - Couldn't parse '%s' : %s\n",
+                      str, error);
+               return 1;
+       }
 
-       new = alloc_bootmem(sizeof(new));
-       if (new == NULL){
-               printk("eth_init : alloc_bootmem failed\n");
-               return(1);
+       new = alloc_bootmem(sizeof(*new));
+       if (new == NULL) {
+               printk(KERN_ERR "eth_init : alloc_bootmem failed\n");
+               return 1;
        }
 
        INIT_LIST_HEAD(&new->list);
@@ -577,7 +673,7 @@ static int eth_setup(char *str)
        new->init = str;
 
        list_add_tail(&new->list, &eth_cmd_line);
-       return(1);
+       return 1;
 }
 
 __setup("eth", eth_setup);
@@ -586,89 +682,73 @@ __uml_help(eth_setup,
 "    Configure a network device.\n\n"
 );
 
-#if 0
-static int eth_init(void)
-{
-       struct list_head *ele, *next;
-       struct eth_init *eth;
-
-       list_for_each_safe(ele, next, &eth_cmd_line){
-               eth = list_entry(ele, struct eth_init, list);
-
-               if(eth_setup_common(eth->init, eth->index))
-                       list_del(&eth->list);
-       }
-       
-       return(1);
-}
-__initcall(eth_init);
-#endif
-
-static int net_config(char *str)
+static int net_config(char *str, char **error_out)
 {
        int n, err;
 
-       err = eth_parse(str, &n, &str);
-       if(err) return(err);
+       err = eth_parse(str, &n, &str, error_out);
+       if (err)
+               return err;
 
+       /* This string is broken up and the pieces used by the underlying
+        * driver.  So, it is freed only if eth_setup_common fails.
+        */
        str = kstrdup(str, GFP_KERNEL);
-       if(str == NULL){
-               printk(KERN_ERR "net_config failed to strdup string\n");
-               return(-1);
+       if (str == NULL) {
+               *error_out = "net_config failed to strdup string";
+               return -ENOMEM;
        }
        err = !eth_setup_common(str, n);
-       if(err) 
+       if (err)
                kfree(str);
-       return(err);
+       return err;
 }
 
 static int net_id(char **str, int *start_out, int *end_out)
 {
-        char *end;
-        int n;
+       char *end;
+       int n;
 
        n = simple_strtoul(*str, &end, 0);
-       if((*end != '\0') || (end == *str))
+       if ((*end != '\0') || (end == *str))
                return -1;
 
-        *start_out = n;
-        *end_out = n;
-        *str = end;
-        return n;
+       *start_out = n;
+       *end_out = n;
+       *str = end;
+       return n;
 }
 
-static int net_remove(int n)
+static int net_remove(int n, char **error_out)
 {
        struct uml_net *device;
        struct net_device *dev;
        struct uml_net_private *lp;
 
        device = find_device(n);
-       if(device == NULL)
+       if (device == NULL)
                return -ENODEV;
 
        dev = device->dev;
-       lp = dev->priv;
-       if(lp->fd > 0)
-                return -EBUSY;
-       if(lp->remove != NULL) (*lp->remove)(&lp->user);
+       lp = netdev_priv(dev);
+       if (lp->fd > 0)
+               return -EBUSY;
        unregister_netdev(dev);
        platform_device_unregister(&device->pdev);
 
-       list_del(&device->list);
-       kfree(device);
-       free_netdev(dev);
        return 0;
 }
 
 static struct mc_device net_mc = {
+       .list           = LIST_HEAD_INIT(net_mc.list),
        .name           = "eth",
        .config         = net_config,
        .get_config     = NULL,
-        .id            = net_id,
+       .id             = net_id,
        .remove         = net_remove,
 };
 
+#ifdef CONFIG_INET
 static int uml_inetaddr_event(struct notifier_block *this, unsigned long event,
                              void *ptr)
 {
@@ -678,12 +758,13 @@ static int uml_inetaddr_event(struct notifier_block *this, unsigned long event,
        void (*proc)(unsigned char *, unsigned char *, void *);
        unsigned char addr_buf[4], netmask_buf[4];
 
-       if(dev->open != uml_net_open) return(NOTIFY_DONE);
+       if (dev->netdev_ops->ndo_open != uml_net_open)
+               return NOTIFY_DONE;
 
-       lp = dev->priv;
+       lp = netdev_priv(dev);
 
        proc = NULL;
-       switch (event){
+       switch (event) {
        case NETDEV_UP:
                proc = lp->add_address;
                break;
@@ -691,44 +772,57 @@ static int uml_inetaddr_event(struct notifier_block *this, unsigned long event,
                proc = lp->delete_address;
                break;
        }
-       if(proc != NULL){
+       if (proc != NULL) {
                memcpy(addr_buf, &ifa->ifa_address, sizeof(addr_buf));
                memcpy(netmask_buf, &ifa->ifa_mask, sizeof(netmask_buf));
                (*proc)(addr_buf, netmask_buf, &lp->user);
        }
-       return(NOTIFY_DONE);
+       return NOTIFY_DONE;
 }
 
-struct notifier_block uml_inetaddr_notifier = {
+/* uml_net_init shouldn't be called twice on two CPUs at the same time */
+static struct notifier_block uml_inetaddr_notifier = {
        .notifier_call          = uml_inetaddr_event,
 };
 
-static int uml_net_init(void)
+static void inet_register(void)
 {
        struct list_head *ele;
-       struct uml_net_private *lp;     
+       struct uml_net_private *lp;
        struct in_device *ip;
        struct in_ifaddr *in;
 
-       mconsole_register_dev(&net_mc);
        register_inetaddr_notifier(&uml_inetaddr_notifier);
 
        /* Devices may have been opened already, so the uml_inetaddr_notifier
         * didn't get a chance to run for them.  This fakes it so that
         * addresses which have already been set up get handled properly.
         */
-       list_for_each(ele, &opened){
+       spin_lock(&opened_lock);
+       list_for_each(ele, &opened) {
                lp = list_entry(ele, struct uml_net_private, list);
                ip = lp->dev->ip_ptr;
-               if(ip == NULL) continue;
+               if (ip == NULL)
+                       continue;
                in = ip->ifa_list;
-               while(in != NULL){
+               while (in != NULL) {
                        uml_inetaddr_event(NULL, NETDEV_UP, in);
                        in = in->ifa_next;
                }
-       }       
+       }
+       spin_unlock(&opened_lock);
+}
+#else
+static inline void inet_register(void)
+{
+}
+#endif
 
-       return(0);
+static int uml_net_init(void)
+{
+       mconsole_register_dev(&net_mc);
+       inet_register();
+       return 0;
 }
 
 __initcall(uml_net_init);
@@ -738,80 +832,22 @@ static void close_devices(void)
        struct list_head *ele;
        struct uml_net_private *lp;
 
-       list_for_each(ele, &opened){
+       spin_lock(&opened_lock);
+       list_for_each(ele, &opened) {
                lp = list_entry(ele, struct uml_net_private, list);
                free_irq(lp->dev->irq, lp->dev);
-               if((lp->close != NULL) && (lp->fd >= 0))
+               if ((lp->close != NULL) && (lp->fd >= 0))
                        (*lp->close)(lp->fd, &lp->user);
-               if(lp->remove != NULL) (*lp->remove)(&lp->user);
+               if (lp->remove != NULL)
+                       (*lp->remove)(&lp->user);
        }
+       spin_unlock(&opened_lock);
 }
 
 __uml_exitcall(close_devices);
 
-int setup_etheraddr(char *str, unsigned char *addr)
-{
-       char *end;
-       int i;
-
-       if(str == NULL)
-               return(0);
-       for(i=0;i<6;i++){
-               addr[i] = simple_strtoul(str, &end, 16);
-               if((end == str) ||
-                  ((*end != ':') && (*end != ',') && (*end != '\0'))){
-                       printk(KERN_ERR 
-                              "setup_etheraddr: failed to parse '%s' "
-                              "as an ethernet address\n", str);
-                       return(0);
-               }
-               str = end + 1;
-       }
-       if(addr[0] & 1){
-               printk(KERN_ERR 
-                      "Attempt to assign a broadcast ethernet address to a "
-                      "device disallowed\n");
-               return(0);
-       }
-       return(1);
-}
-
-void dev_ip_addr(void *d, unsigned char *bin_buf)
-{
-       struct net_device *dev = d;
-       struct in_device *ip = dev->ip_ptr;
-       struct in_ifaddr *in;
-
-       if((ip == NULL) || ((in = ip->ifa_list) == NULL)){
-               printk(KERN_WARNING "dev_ip_addr - device not assigned an "
-                      "IP address\n");
-               return;
-       }
-       memcpy(bin_buf, &in->ifa_address, sizeof(in->ifa_address));
-}
-
-void set_ether_mac(void *d, unsigned char *addr)
-{
-       struct net_device *dev = d;
-
-       memcpy(dev->dev_addr, addr, ETH_ALEN);  
-}
-
-struct sk_buff *ether_adjust_skb(struct sk_buff *skb, int extra)
-{
-       if((skb != NULL) && (skb_tailroom(skb) < extra)){
-               struct sk_buff *skb2;
-
-               skb2 = skb_copy_expand(skb, 0, extra, GFP_ATOMIC);
-               dev_kfree_skb(skb);
-               skb = skb2;
-       }
-       if(skb != NULL) skb_put(skb, extra);
-       return(skb);
-}
-
-void iter_addresses(void *d, void (*cb)(unsigned char *, unsigned char *, 
-                                       void *), 
+void iter_addresses(void *d, void (*cb)(unsigned char *, unsigned char *,
+                                       void *),
                    void *arg)
 {
        struct net_device *dev = d;
@@ -819,9 +855,9 @@ void iter_addresses(void *d, void (*cb)(unsigned char *, unsigned char *,
        struct in_ifaddr *in;
        unsigned char address[4], netmask[4];
 
-       if(ip == NULL) return;
+       if (ip == NULL) return;
        in = ip->ifa_list;
-       while(in != NULL){
+       while (in != NULL) {
                memcpy(address, &in->ifa_address, sizeof(address));
                memcpy(netmask, &in->ifa_mask, sizeof(netmask));
                (*cb)(address, netmask, arg);
@@ -834,17 +870,17 @@ int dev_netmask(void *d, void *m)
        struct net_device *dev = d;
        struct in_device *ip = dev->ip_ptr;
        struct in_ifaddr *in;
-       __u32 *mask_out = m;
+       __be32 *mask_out = m;
 
-       if(ip == NULL) 
-               return(1);
+       if (ip == NULL)
+               return 1;
 
        in = ip->ifa_list;
-       if(in == NULL) 
-               return(1);
+       if (in == NULL)
+               return 1;
 
        *mask_out = in->ifa_mask;
-       return(0);
+       return 0;
 }
 
 void *get_output_buffer(int *len_out)
@@ -852,9 +888,9 @@ void *get_output_buffer(int *len_out)
        void *ret;
 
        ret = (void *) __get_free_pages(GFP_KERNEL, 0);
-       if(ret) *len_out = PAGE_SIZE;
+       if (ret) *len_out = PAGE_SIZE;
        else *len_out = 0;
-       return(ret);
+       return ret;
 }
 
 void free_output_buffer(void *buffer)
@@ -862,33 +898,22 @@ void free_output_buffer(void *buffer)
        free_pages((unsigned long) buffer, 0);
 }
 
-int tap_setup_common(char *str, char *type, char **dev_name, char **mac_out, 
+int tap_setup_common(char *str, char *type, char **dev_name, char **mac_out,
                     char **gate_addr)
 {
        char *remain;
 
        remain = split_if_spec(str, dev_name, mac_out, gate_addr, NULL);
-       if(remain != NULL){
-               printk("tap_setup_common - Extra garbage on specification : "
-                      "'%s'\n", remain);
-               return(1);
+       if (remain != NULL) {
+               printk(KERN_ERR "tap_setup_common - Extra garbage on "
+                      "specification : '%s'\n", remain);
+               return 1;
        }
 
-       return(0);
+       return 0;
 }
 
 unsigned short eth_protocol(struct sk_buff *skb)
 {
-       return(eth_type_trans(skb, skb->dev));
+       return eth_type_trans(skb, skb->dev);
 }
-
-/*
- * Overrides for Emacs so that we follow Linus's tabbing style.
- * Emacs will notice this stuff at the end of the file and automatically
- * adjust the settings for this buffer only.  This must remain at the end
- * of the file.
- * ---------------------------------------------------------------------------
- * Local variables:
- * c-file-style: "linux"
- * End:
- */