Re: [PATCH -next] bridge: depends on INET

From: Ingo Molnar
Date: Wed Mar 03 2010 - 22:20:05 EST



* David Miller <davem@xxxxxxxxxxxxx> wrote:

> From: Randy Dunlap <rdunlap@xxxxxxxxxxxx>
> Date: Tue, 02 Mar 2010 09:08:23 -0800
>
> > From: Randy Dunlap <randy.dunlap@xxxxxxxxxx>
> >
> > br_multicast calls ip_send_check(), so it should depend on INET.
> >
> > built-in:
> > br_multicast.c:(.text+0x88cf4): undefined reference to `ip_send_check'
> >
> > or modular:
> > ERROR: "ip_send_check" [net/bridge/bridge.ko] undefined!
> >
> > Signed-off-by: Randy Dunlap <randy.dunlap@xxxxxxxxxx>
>
> Applied, but I almost lost this patch.
>
> You need to CC: netdev on all networking patches so that it
> gets properly tracked in patchwork and therefore does not get
> lost.

I suspect Randy went by the MAINTAINERS entry - you might want to add netdev
as a second 'L:' line:

ETHERNET BRIDGE
M: Stephen Hemminger <shemminger@xxxxxxxxxxxxxxxxxxxx>
L: bridge@xxxxxxxxxxxxxxxxxxxxxxxxxx
W: http://www.linux-foundation.org/en/Net:Bridge
S: Maintained
F: include/linux/netfilter_bridge/
F: net/bridge/

To make it really clear that netdev must be Cc:-ed for those patches.

Thanks,

Ingo
--
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/