CVS commit: src/sys/net

["Charles M. Hannum" <mycroft%netbsd.org@localhost>]

Module Name:    src
Committed By:   mycroft
Date:           Thu Jul  8 19:09:12 UTC 2004

Modified Files:
        src/sys/net: if_vlan.c

Log Message:
If the parent interface is not IFF_RUNNING, do not call its start function.
This prevent a variety of fun panics, and therefore fixes PR 12932, PR 17561,
and PR 18376...

XXX
...however this is most definitely a hack.  The real problem here is that there
is no callback to notify a "client" interface like vlan when a "parent"
interface's status changes, and therefore the vlan interface is always
IFF_RUNNING.  This allows packets to be queued on vlan interface at any time.
We can't simply leave the packets on the vlan interface, either, because there
is no callback to dequeue them.  And last, since it's always IFF_RUNNING, if
we just *toss* the packets, we lose gratuitous ARPs and DAD packets.

"This needs work," but at least it no longer bleeds.


To generate a diff of this commit:
cvs rdiff -r1.40 -r1.41 src/sys/net/if_vlan.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.