Merge mulgrave-w:git/linux-2.6

Conflicts:

	include/linux/blkdev.h

Trivial merge to incorporate tag prototypes.

4 files changed, 93 insertions, 90 deletions

diff --git a/net/bridge/br_forward.c b/net/bridge/br_forward.c
index 864fbbc7b24d..191b861e5e53 100644
--- a/net/bridge/br_forward.c
+++ b/net/bridge/br_forward.c
@@ -38,13 +38,10 @@ int br_dev_queue_push_xmit(struct sk_buff *skb)
         if (packet_length(skb) > skb->dev->mtu && !skb_is_gso(skb))
                 kfree_skb(skb);
         else {
-#ifdef CONFIG_BRIDGE_NETFILTER
                 /* ip_refrag calls ip_fragment, doesn't copy the MAC header. */
                 if (nf_bridge_maybe_copy_header(skb))
                         kfree_skb(skb);
-                else
-#endif
-                {
+                else {
                         skb_push(skb, ETH_HLEN);
 
                         dev_queue_xmit(skb);
diff --git a/net/bridge/br_netfilter.c b/net/bridge/br_netfilter.c
index 05b3de888243..ac181be13d83 100644
--- a/net/bridge/br_netfilter.c
+++ b/net/bridge/br_netfilter.c
@@ -53,10 +53,10 @@
 
 #ifdef CONFIG_SYSCTL
 static struct ctl_table_header *brnf_sysctl_header;
-static int brnf_call_iptables = 1;
-static int brnf_call_ip6tables = 1;
-static int brnf_call_arptables = 1;
-static int brnf_filter_vlan_tagged = 1;
+static int brnf_call_iptables __read_mostly = 1;
+static int brnf_call_ip6tables __read_mostly = 1;
+static int brnf_call_arptables __read_mostly = 1;
+static int brnf_filter_vlan_tagged __read_mostly = 1;
 #else
 #define brnf_filter_vlan_tagged 1
 #endif
@@ -127,14 +127,37 @@ static inline struct nf_bridge_info *nf_bridge_alloc(struct sk_buff *skb)
 
 static inline void nf_bridge_save_header(struct sk_buff *skb)
 {
-        int header_size = 16;
+        int header_size = ETH_HLEN;
 
         if (skb->protocol == htons(ETH_P_8021Q))
-                header_size = 18;
+                header_size += VLAN_HLEN;
 
         memcpy(skb->nf_bridge->data, skb->data - header_size, header_size);
 }
 
+/*
+ * When forwarding bridge frames, we save a copy of the original
+ * header before processing.
+ */
+int nf_bridge_copy_header(struct sk_buff *skb)
+{
+        int err;
+        int header_size = ETH_HLEN;
+
+        if (skb->protocol == htons(ETH_P_8021Q))
+                header_size += VLAN_HLEN;
+
+        err = skb_cow(skb, header_size);
+        if (err)
+                return err;
+
+        memcpy(skb->data - header_size, skb->nf_bridge->data, header_size);
+
+        if (skb->protocol == htons(ETH_P_8021Q))
+                __skb_push(skb, VLAN_HLEN);
+        return 0;
+}
+
 /* PF_BRIDGE/PRE_ROUTING *********************************************/
 /* Undo the changes made for ip6tables PREROUTING and continue the
  * bridge PRE_ROUTING hook. */
@@ -695,16 +718,6 @@ static unsigned int br_nf_local_out(unsigned int hook, struct sk_buff **pskb,
         else
                 pf = PF_INET6;
 
-#ifdef CONFIG_NETFILTER_DEBUG
-        /* Sometimes we get packets with NULL ->dst here (for example,
-         * running a dhcp client daemon triggers this). This should now
-         * be fixed, but let's keep the check around. */
-        if (skb->dst == NULL) {
-                printk(KERN_CRIT "br_netfilter: skb->dst == NULL.");
-                return NF_ACCEPT;
-        }
-#endif
-
         nf_bridge = skb->nf_bridge;
         nf_bridge->physoutdev = skb->dev;
         realindev = nf_bridge->physindev;
@@ -786,7 +799,7 @@ static unsigned int br_nf_post_routing(unsigned int hook, struct sk_buff **pskb,
          * keep the check just to be sure... */
         if (skb->mac.raw < skb->head || skb->mac.raw + ETH_HLEN > skb->data) {
                 printk(KERN_CRIT "br_netfilter: Argh!! br_nf_post_routing: "
-                       "bad mac.raw pointer.");
+                       "bad mac.raw pointer.\n");
                 goto print_error;
         }
 #endif
@@ -804,7 +817,7 @@ static unsigned int br_nf_post_routing(unsigned int hook, struct sk_buff **pskb,
 
 #ifdef CONFIG_NETFILTER_DEBUG
         if (skb->dst == NULL) {
-                printk(KERN_CRIT "br_netfilter: skb->dst == NULL.");
+                printk(KERN_INFO "br_netfilter post_routing: skb->dst == NULL\n");
                 goto print_error;
         }
 #endif
@@ -841,6 +854,7 @@ print_error:
         }
         printk(" head:%p, raw:%p, data:%p\n", skb->head, skb->mac.raw,
                skb->data);
+        dump_stack();
         return NF_ACCEPT;
 #endif
 }
diff --git a/net/bridge/br_netlink.c b/net/bridge/br_netlink.c
index 53086fb75089..8f661195d09d 100644
--- a/net/bridge/br_netlink.c
+++ b/net/bridge/br_netlink.c
@@ -12,6 +12,7 @@
 
 #include <linux/kernel.h>
 #include <linux/rtnetlink.h>
+#include <net/netlink.h>
 #include "br_private.h"
 
 /*
@@ -76,26 +77,24 @@ rtattr_failure:
 void br_ifinfo_notify(int event, struct net_bridge_port *port)
 {
         struct sk_buff *skb;
-        int err = -ENOMEM;
+        int payload = sizeof(struct ifinfomsg) + 128;
+        int err = -ENOBUFS;
 
         pr_debug("bridge notify event=%d\n", event);
-        skb = alloc_skb(NLMSG_SPACE(sizeof(struct ifinfomsg) + 128),
-                        GFP_ATOMIC);
-        if (!skb)
-                goto err_out;
+        skb = nlmsg_new(nlmsg_total_size(payload), GFP_ATOMIC);
+        if (skb == NULL)
+                goto errout;
+
+        err = br_fill_ifinfo(skb, port, 0, 0, event, 0);
+        if (err < 0) {
+                kfree_skb(skb);
+                goto errout;
+        }
 
-        err = br_fill_ifinfo(skb, port, current->pid, 0, event, 0);
+        err = rtnl_notify(skb, 0, RTNLGRP_LINK, NULL, GFP_ATOMIC);
+errout:
         if (err < 0)
-                goto err_kfree;
-
-        NETLINK_CB(skb).dst_group = RTNLGRP_LINK;
-        netlink_broadcast(rtnl, skb, 0, RTNLGRP_LINK, GFP_ATOMIC);
-        return;
-
-err_kfree:
-        kfree_skb(skb);
-err_out:
-        netlink_set_err(rtnl, 0, RTNLGRP_LINK, err);
+                rtnl_set_sk_err(RTNLGRP_LINK, err);
 }
 
 /*
diff --git a/net/bridge/netfilter/ebtables.c b/net/bridge/netfilter/ebtables.c
index 3a13ed643459..3df55b2bd91d 100644
--- a/net/bridge/netfilter/ebtables.c
+++ b/net/bridge/netfilter/ebtables.c
@@ -24,6 +24,7 @@
 #include <linux/vmalloc.h>
 #include <linux/netfilter_bridge/ebtables.h>
 #include <linux/spinlock.h>
+#include <linux/mutex.h>
 #include <asm/uaccess.h>
 #include <linux/smp.h>
 #include <linux/cpumask.h>
@@ -31,36 +32,9 @@
 /* needed for logical [in,out]-dev filtering */
 #include "../br_private.h"
 
-/* list_named_find */
-#define ASSERT_READ_LOCK(x)
-#define ASSERT_WRITE_LOCK(x)
-#include <linux/netfilter_ipv4/listhelp.h>
-#include <linux/mutex.h>
-
-#if 0
-/* use this for remote debugging
- * Copyright (C) 1998 by Ori Pomerantz
- * Print the string to the appropriate tty, the one
- * the current task uses
- */
-static void print_string(char *str)
-{
-        struct tty_struct *my_tty;
-
-        /* The tty for the current task */
-        my_tty = current->signal->tty;
-        if (my_tty != NULL) {
-                my_tty->driver->write(my_tty, 0, str, strlen(str));
-                my_tty->driver->write(my_tty, 0, "\015\012", 2);
-        }
-}
-
-#define BUGPRINT(args) print_string(args);
-#else
 #define BUGPRINT(format, args...) printk("kernel msg: ebtables bug: please "\
                                          "report to author: "format, ## args)
 /* #define BUGPRINT(format, args...) */
-#endif
 #define MEMPRINT(format, args...) printk("kernel msg: ebtables "\
                                          ": out of memory: "format, ## args)
 /* #define MEMPRINT(format, args...) */
@@ -299,18 +273,22 @@ static inline void *
 find_inlist_lock_noload(struct list_head *head, const char *name, int *error,
    struct mutex *mutex)
 {
-        void *ret;
+        struct {
+                struct list_head list;
+                char name[EBT_FUNCTION_MAXNAMELEN];
+        } *e;
 
         *error = mutex_lock_interruptible(mutex);
         if (*error != 0)
                 return NULL;
 
-        ret = list_named_find(head, name);
-        if (!ret) {
-                *error = -ENOENT;
-                mutex_unlock(mutex);
+        list_for_each_entry(e, head, list) {
+                if (strcmp(e->name, name) == 0)
+                        return e;
         }
-        return ret;
+        *error = -ENOENT;
+        mutex_unlock(mutex);
+        return NULL;
 }
 
 #ifndef CONFIG_KMOD
@@ -1064,15 +1042,19 @@ free_newinfo:
 
 int ebt_register_target(struct ebt_target *target)
 {
+        struct ebt_target *t;
         int ret;
 
         ret = mutex_lock_interruptible(&ebt_mutex);
         if (ret != 0)
                 return ret;
-        if (!list_named_insert(&ebt_targets, target)) {
-                mutex_unlock(&ebt_mutex);
-                return -EEXIST;
+        list_for_each_entry(t, &ebt_targets, list) {
+                if (strcmp(t->name, target->name) == 0) {
+                        mutex_unlock(&ebt_mutex);
+                        return -EEXIST;
+                }
         }
+        list_add(&target->list, &ebt_targets);
         mutex_unlock(&ebt_mutex);
 
         return 0;
@@ -1081,21 +1063,25 @@ int ebt_register_target(struct ebt_target *target)
 void ebt_unregister_target(struct ebt_target *target)
 {
         mutex_lock(&ebt_mutex);
-        LIST_DELETE(&ebt_targets, target);
+        list_del(&target->list);
         mutex_unlock(&ebt_mutex);
 }
 
 int ebt_register_match(struct ebt_match *match)
 {
+        struct ebt_match *m;
         int ret;
 
         ret = mutex_lock_interruptible(&ebt_mutex);
         if (ret != 0)
                 return ret;
-        if (!list_named_insert(&ebt_matches, match)) {
-                mutex_unlock(&ebt_mutex);
-                return -EEXIST;
+        list_for_each_entry(m, &ebt_matches, list) {
+                if (strcmp(m->name, match->name) == 0) {
+                        mutex_unlock(&ebt_mutex);
+                        return -EEXIST;
+                }
         }
+        list_add(&match->list, &ebt_matches);
         mutex_unlock(&ebt_mutex);
 
         return 0;
@@ -1104,21 +1090,25 @@ int ebt_register_match(struct ebt_match *match)
 void ebt_unregister_match(struct ebt_match *match)
 {
         mutex_lock(&ebt_mutex);
-        LIST_DELETE(&ebt_matches, match);
+        list_del(&match->list);
         mutex_unlock(&ebt_mutex);
 }
 
 int ebt_register_watcher(struct ebt_watcher *watcher)
 {
+        struct ebt_watcher *w;
         int ret;
 
         ret = mutex_lock_interruptible(&ebt_mutex);
         if (ret != 0)
                 return ret;
-        if (!list_named_insert(&ebt_watchers, watcher)) {
-                mutex_unlock(&ebt_mutex);
-                return -EEXIST;
+        list_for_each_entry(w, &ebt_watchers, list) {
+                if (strcmp(w->name, watcher->name) == 0) {
+                        mutex_unlock(&ebt_mutex);
+                        return -EEXIST;
+                }
         }
+        list_add(&watcher->list, &ebt_watchers);
         mutex_unlock(&ebt_mutex);
 
         return 0;
@@ -1127,13 +1117,14 @@ int ebt_register_watcher(struct ebt_watcher *watcher)
 void ebt_unregister_watcher(struct ebt_watcher *watcher)
 {
         mutex_lock(&ebt_mutex);
-        LIST_DELETE(&ebt_watchers, watcher);
+        list_del(&watcher->list);
         mutex_unlock(&ebt_mutex);
 }
 
 int ebt_register_table(struct ebt_table *table)
 {
         struct ebt_table_info *newinfo;
+        struct ebt_table *t;
         int ret, i, countersize;
 
         if (!table || !table->table ||!table->table->entries ||
@@ -1179,10 +1170,12 @@ int ebt_register_table(struct ebt_table *table)
         if (ret != 0)
                 goto free_chainstack;
 
-        if (list_named_find(&ebt_tables, table->name)) {
-                ret = -EEXIST;
-                BUGPRINT("Table name already exists\n");
-                goto free_unlock;
+        list_for_each_entry(t, &ebt_tables, list) {
+                if (strcmp(t->name, table->name) == 0) {
+                        ret = -EEXIST;
+                        BUGPRINT("Table name already exists\n");
+                        goto free_unlock;
+                }
         }
 
         /* Hold a reference count if the chains aren't empty */
@@ -1190,7 +1183,7 @@ int ebt_register_table(struct ebt_table *table)
                 ret = -ENOENT;
                 goto free_unlock;
         }
-        list_prepend(&ebt_tables, table);
+        list_add(&table->list, &ebt_tables);
         mutex_unlock(&ebt_mutex);
         return 0;
 free_unlock:
@@ -1216,7 +1209,7 @@ void ebt_unregister_table(struct ebt_table *table)
                 return;
         }
         mutex_lock(&ebt_mutex);
-        LIST_DELETE(&ebt_tables, table);
+        list_del(&table->list);
         mutex_unlock(&ebt_mutex);
         vfree(table->private->entries);
         if (table->private->chainstack) {
@@ -1486,7 +1479,7 @@ static int __init ebtables_init(void)
         int ret;
 
         mutex_lock(&ebt_mutex);
-        list_named_insert(&ebt_targets, &ebt_standard_target);
+        list_add(&ebt_standard_target.list, &ebt_targets);
         mutex_unlock(&ebt_mutex);
         if ((ret = nf_register_sockopt(&ebt_sockopts)) < 0)
                 return ret;