Hi,
some weeks ago, I've written a module that allows forwarding link state
changes of network interfaces over the netlink socket. Recently I ran
into problems after I added support to the VLAN driver that the link
state of VLAN slave interfaces follow the master interface. My (slightly
tested) patch to the VLAN driver and the module code are attached.
I've attached 32 VLANs to an ethernet interface. After removing the
network cable, the interface driver called netif_carrier_off() (Note
that not many drivers support that, so you may not be able to repeat).
This caused my module to broadcast 32 state changes over the netlink
socket as I expected.
However, these messages cause every receiver, tested with both simple
"ip monitor" command and patched zebra daemon, to lose the first 10
messages due to an overflow in recvmsg(). Increasing
/proc/sys/net/core/rmem_* to 131072 helped for both, adding a
schedule()-call after every netdev_state_change() helped for the ip
command only, but is of course a very bad habit while holding the
rtnl_lock.
Now what looks strange to me is how 32 messages are able to overflow a
64K receive buffer. Anyone can enlighten me or give me a hint on where
to start searching & may be fixing?
Stefan
/* Linux network device status watcher
(C) 2002 Stefan Rompf
GPL Version 2 applies
compile with something like
gcc -c -DMODULE -D__KERNEL__ -O2 dev_watch.c
*/
#include <linux/sched.h>
#include <linux/config.h>
#include <linux/module.h>
#include <linux/netdevice.h>
#include <linux/if.h>
#include <linux/rtnetlink.h>
static volatile pid_t me;
static volatile int running = 1;
static int devwatcher(void *dummy)
{
struct net_device *dev;
daemonize();
strcpy(current->comm, "devwatchd");
while(running) {
rtnl_lock();
for (dev=dev_base; dev; dev = dev->next) {
/* State of netif_carrier_ok() is reflected
into dev_flags by this loop, and a netlink
message is omitted whenever the state
changes */
if (dev->flags & IFF_RUNNING) {
if (!netif_carrier_ok(dev)) {
write_lock(&dev_base_lock);
dev->flags &= ~IFF_RUNNING;
write_unlock(&dev_base_lock);
netdev_state_change(dev);
}
} else {
if (netif_carrier_ok(dev)) {
write_lock(&dev_base_lock);
dev->flags |= IFF_RUNNING;
write_unlock(&dev_base_lock);
netdev_state_change(dev);
}
}
}
rtnl_unlock();
set_current_state(TASK_INTERRUPTIBLE);
schedule_timeout(HZ);
}
me = 0;
return 0;
}
static int __init devwatch_init(void) {
me = kernel_thread(devwatcher, NULL, CLONE_FS | CLONE_FILES | CLONE_SIGNAL);
if (!me) return 1;
return 0;
}
static void __exit devwatch_cleanup(void) {
running = 0;
while(me) schedule();
}
EXPORT_NO_SYMBOLS;
module_init(devwatch_init);
module_exit(devwatch_cleanup);
--- linux-2.4.17/net/8021q/vlan.c Sat Jan 19 15:25:21 2002
+++ linux-2.4.17stefan/net/8021q/vlan.c Tue Mar 19 22:35:46 2002
@@ -468,6 +468,23 @@
struct net_device *vlandev = NULL;
switch (event) {
+ case NETDEV_CHANGE:
+ for (grp = p802_1Q_vlan_list; grp != NULL; grp = grp->next) {
+ for (i = 0; i < VLAN_GROUP_ARRAY_LEN; i++) {
+ vlandev = grp->vlan_devices[i];
+ if (vlandev && VLAN_DEV_INFO(vlandev)->real_dev == dev) {
+ if (netif_carrier_ok(vlandev) != netif_carrier_ok(dev)) {
+ if (netif_carrier_ok(dev)) {
+ netif_carrier_on(vlandev);
+ } else {
+ netif_carrier_off(vlandev);
+ }
+ }
+ }
+ }
+ }
+ break;
+
case NETDEV_CHANGEADDR:
/* Ignore for now */
break;
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
This archive was generated by hypermail 2b29 : Sat Mar 23 2002 - 22:00:23 EST