Alexey Dobriyan wrote: > On Mon, Oct 22, 2007 at 04:33:33PM +0400, Pavel Emelyanov wrote: >> And add the forward declaration for br_handle_frame_hook in >> include/linux/if_bridge.h > >> --- a/include/linux/netdevice.h >> +++ b/include/linux/netdevice.h >> @@ -699,8 +699,10 @@ struct net_device >> /* Network namespace this network device is inside */ >> struct net *nd_net; >> >> +#if defined(CONFIG_BRIDGE) || defined(CONFIG_BRIDGE_MODULE) >> /* bridge stuff */ >> struct net_bridge_port *br_port; >> +#endif > > Such ifdefs are troublesome, because one can compile kernel with > CONFIG_BRIDGE=n and later decide he needs BRIDGE=m and go boom.
You will have to recompile the whole kernel when this config is on. Look at netif_receive_skb() ;) > IPv6 -- the biggest violator of this rule should not be taken as > example. :^) > > - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
