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, send a message with 'unsubscribe linux-mm' in
the body to majordomo@xxxxxxxxxx For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"aart@xxxxxxxxx";> aart@xxxxxxxxx </a>
|
