Date: Tue, 06 Mar 2007 15:46:46 +0000 From: Bruce M Simpson <bms@incunabulum.net> To: Yar Tikhiy <yar@comp.chem.msu.su> Cc: freebsd-net@freebsd.org, Kip Macy <kmacy@freebsd.org> Subject: Re: [PATCH] Ethernet cleanup; 802.1p input and M_PROMISC Message-ID: <45ED8CE6.5030908@incunabulum.net> In-Reply-To: <20070305154824.GE57253@comp.chem.msu.su> References: <45E8B964.2090200@incunabulum.net> <20070303215359.GB40430@comp.chem.msu.su> <45EA0756.2000107@incunabulum.net> <20070304070458.GG40430@comp.chem.msu.su> <45EB750A.90105@incunabulum.net> <20070305142411.GC57253@comp.chem.msu.su> <45EC2C37.7020604@incunabulum.net> <20070305154824.GE57253@comp.chem.msu.su>
next in thread | previous in thread | raw e-mail | index | archive | help
Yar Tikhiy wrote:
> My proposed check for IFF_DRV_RUNNING is by no means a priority
> task. I can add it by myself after you finish your great current
> project regarding ether_input() and friends.
>
Just committed in p4:
==== //depot/user/bms/netdev/sys/net/if_ethersubr.c#6 -
/home/bms/p4/netdev/sys/net/if_ethersubr.c ====
--- /tmp/tmp.11470.0 Tue Mar 6 15:45:08 2007
+++ /home/bms/p4/netdev/sys/net/if_ethersubr.c Tue Mar 6 15:45:01 2007
@@ -511,6 +511,13 @@
m_freem(m);
return;
}
+#ifdef DIAGNOSTIC
+ if ((ifp->if_flags & IFF_DRV_RUNNING) == 0) {
+ if_printf(ifp, "discard frame at !IFF_DRV_RUNNING\n");
+ m_freem(m);
+ return;
+ }
+#endif
Thanks!
BMS
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?45ED8CE6.5030908>