From owner-cvs-src@FreeBSD.ORG Sun Sep 17 22:06:07 2006 Return-Path: X-Original-To: cvs-src@FreeBSD.org Delivered-To: cvs-src@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 1A8FE16A412; Sun, 17 Sep 2006 22:06:07 +0000 (UTC) (envelope-from ru@rambler-co.ru) Received: from relay0.rambler.ru (relay0.rambler.ru [81.19.66.187]) by mx1.FreeBSD.org (Postfix) with ESMTP id A4E1F43D7E; Sun, 17 Sep 2006 22:06:05 +0000 (GMT) (envelope-from ru@rambler-co.ru) Received: from relay0.rambler.ru (localhost [127.0.0.1]) by relay0.rambler.ru (Postfix) with ESMTP id 8FE655DFC; Mon, 18 Sep 2006 02:06:04 +0400 (MSD) Received: from edoofus.park.rambler.ru (unknown [81.19.65.108]) by relay0.rambler.ru (Postfix) with ESMTP id 57DEE5DFB; Mon, 18 Sep 2006 02:06:04 +0400 (MSD) Received: (from ru@localhost) by edoofus.park.rambler.ru (8.13.8/8.13.8) id k8HM69ug085007; Mon, 18 Sep 2006 02:06:09 +0400 (MSD) (envelope-from ru) Date: Mon, 18 Sep 2006 02:06:09 +0400 From: Ruslan Ermilov To: Andre Oppermann Message-ID: <20060917220609.GA84871@rambler-co.ru> References: <200609171333.k8HDXUht029746@repoman.freebsd.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="NzB8fVQJ5HfG6fxh" Content-Disposition: inline In-Reply-To: <200609171333.k8HDXUht029746@repoman.freebsd.org> User-Agent: Mutt/1.5.13 (2006-08-11) X-Virus-Scanned: No virus found Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/dev/bce if_bce.c src/sys/dev/bge if_bge.c src/sys/dev/em if_em.c src/sys/dev/ixgb if_ixgb.c src/sys/dev/nfe if_nfe.c src/sys/dev/nge if_nge.c src/sys/dev/re if_re.c src/sys/dev/stge if_stge.c src/sys/dev/ti if_ti.c src/sys/dev/txp ... X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 17 Sep 2006 22:06:07 -0000 --NzB8fVQJ5HfG6fxh Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sun, Sep 17, 2006 at 01:33:30PM +0000, Andre Oppermann wrote: > andre 2006-09-17 13:33:30 UTC [...] > Move ethernet VLAN tags from mtags to its own mbuf packet header field > m_pkthdr.ether_vlan. The presence of the M_VLANTAG flag on the mbuf > signifies the presence and validity of its content. > =20 > Drivers that support hardware VLAN tag stripping fill in the received > VLAN tag (containing both vlan and priority information) into the > ether_vtag mbuf packet header field: > =20 > m->m_pkthdr.ether_vtag =3D vlan_id; /* ntohs()? */ > m->m_flags |=3D M_VLANTAG; > =20 > to mark the packet m with the specified VLAN tag. > =20 > On output the driver should check the mbuf for the M_VLANTAG flag to > see if a VLAN tag is present and valid: > =20 > if (m->m_flags & M_VLANTAG) { > ... =3D m->m_pkthdr.ether_vtag; /* htons()? */ > ... pass tag to hardware ... > } > =20 > VLAN tags are stored in host byte order. Byte swapping may be necessar= y. > =20 > (Note: This driver conversion was mechanic and did not add or remove any > byte swapping in the drivers.) > =20 > Remove zone_mtag_vlan UMA zone and MTAG_VLAN definition. No more tag > memory allocation have to be done. > =20 > Reviewed by: thompsa, yar > Sponsored by: TCP/IP Optimization Fundraise 2005 > =20 I think this deserved a __FreeBSD_version bump for third-party Ethernet drivers, since this change removes public interfaces such as VLAN_INPUT_TAG() and VLAN_OUTPUT_TAG(). Cheers, --=20 Ruslan Ermilov ru@FreeBSD.org FreeBSD committer --NzB8fVQJ5HfG6fxh Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.5 (FreeBSD) iD8DBQFFDcbRqRfpzJluFF4RApW6AKCSeuQ719KD6OL2kAgMWc5XZE6wQQCeNKT0 w9bYgImUST7hIP0nw8+LTQ0= =cp+x -----END PGP SIGNATURE----- --NzB8fVQJ5HfG6fxh--