Re: 2.6.0-rc1-mm1

[Tomas Szepe]

On Dec-31 2003, Wed, 02:57 -0800
Andrew Morton <akpm@xxxxxxxx> wrote:

> Tomas Szepe <szepe@xxxxxxxxxxxxxxx> wrote:
> >
> > What I did was:
> 
> Well that just reverts the recent change back to the way it was.  I assume
> that change was made for a reason.  But with such a lame changelog I am not
> able to say what it was.   No doubt Dave will hunt down the perps ;)

Let's not drag Dave out of bed. :)
Here's the explanation (and a fix) from Bart De Schuymer.

-- 
Tomas Szepe <szepe@xxxxxxxxxxxxxxx>

From: Bart De Schuymer <bdschuym@xxxxxxxxxx>
To: "David S.Miller" <davem@xxxxxxxxxx>
Cc: netdev <netdev@xxxxxxxxxxx>, Tomas Szepe <szepe@xxxxxxxxxxxxxxx>
Subject: [PATCH] Always copy and save the vlan header in bridge-nf (do it right now)
Date: Wed, 31 Dec 2003 11:56:11 +0100

Hi Dave,

I forgot to get rid of another ifdef in netfilter_bridge.h when I removed
the dependency upon vlan being compiled. This patch fixes it.

cheers,
Bart

--- linux-2.6.0-bk3/include/linux/netfilter_bridge.h.earlier	2003-12-31 11:54:25.000000000 +0100
+++ linux-2.6.0-bk3/include/linux/netfilter_bridge.h	2003-12-31 11:54:47.000000000 +0100
@@ -8,10 +8,8 @@
 #include <linux/netfilter.h>
 #if defined(__KERNEL__) && defined(CONFIG_BRIDGE_NETFILTER)
 #include <asm/atomic.h>
-#if defined(CONFIG_VLAN_8021Q) || defined(CONFIG_VLAN_8021Q_MODULE)
 #include <linux/if_ether.h>
 #endif
-#endif
 
 /* Bridge Hooks */
 /* After promisc drops, checksum checks. */

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/