Date: Tue, 15 Feb 2005 20:37:44 +0200 From: Ruslan Ermilov <ru@freebsd.org> To: Sam Leffler <sam@errno.com> Cc: cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/net if_ethersubr.c Message-ID: <20050215183744.GA17554@ip.net.ua> In-Reply-To: <42123604.9070002@errno.com> References: <200502140829.j1E8TgDs086634@repoman.freebsd.org> <4210D210.3080700@errno.com> <20050214181431.GA69635@ip.net.ua> <4210F849.8060005@errno.com> <20050214195558.GD69635@ip.net.ua> <421104C7.4070709@errno.com> <20050215074226.GA6781@ip.net.ua> <42123604.9070002@errno.com>
next in thread | previous in thread | raw e-mail | index | archive | help
[-- Attachment #1 --] On Tue, Feb 15, 2005 at 09:48:52AM -0800, Sam Leffler wrote: > If the intent is simply to eliminate the dispatch of frames out of the > driver when there are no vlan consumers then it would seem another > option is to do this at the source. I suggested this initially--that > VLAN_INPUT_TAG be augmented to discard frames when if_nvlans is zero > (this would also eliminate gratuitous tag allocation). I believe you're > saying that if_nvlans can be zero and there may still be consumers for > the packets in which case we might look at a better way of indicating > there are consumers for the packets assuming we can be sure they won't > reach ether_demux (which might be hard). > Yes, this can be hard -- consumers may live on another machine, read: there's no direct link to the underlying hardware that supplies VLAN frames. > As to your other suggestion of allocating an mbuf flag bit that'd be > fine with me. I didn't offer that because I thought we were out of free > bits but I see m_flags got changed to an int a while back and only > 16-bits are assigned. > OK, I'll post the patches soon then. Cheers, -- Ruslan Ermilov ru@FreeBSD.org FreeBSD committer [-- Attachment #2 --] -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.6 (FreeBSD) iD8DBQFCEkF4qRfpzJluFF4RArqIAJsFdc8nE+20z7MuDW1Vaco49eJwZQCffWbK Fx1uwYcwp2XThMU8eOb2gG8= =F78p -----END PGP SIGNATURE-----
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20050215183744.GA17554>