lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 18 Aug 2011 15:38:02 +0100 From: Ben Hutchings <bhutchings@...arflare.com> To: David Lamparter <equinox@...c24.net> Cc: netdev@...r.kernel.org, Patrick McHardy <kaber@...sh.net> Subject: Re: [RFC PATCH] net: vlan: 802.1ad S-VLAN support On Sun, 2011-07-17 at 22:55 +0200, David Lamparter wrote: > this adds support for 802.1ad S-VLANs, which basically are regular VLANs > with a different protocol field. also supported are the legacy QinQ > 9100/9200/9300 ethertypes. as with the CFI bit for 802.1Q, the DEI bit > is blissfully ignored. > > this patch modifies the 802.1Q code, but keeps the regular VLAN > acceleration architecture unchanged. the S-VLAN code does not use that; > I am not aware of any NIC implementing it for ethertypes other than > 8100. [...] I've not heard of multiple-VLAN-tag-insertion, but any controller that implements a generic 16-bit checksum should be able to do TX checksum offload regardless of the number of tags present. So I think VLAN devices should have their own vlan_features set to at least: NETIF_F_HW_CSUM | NETIF_F_SG | NETIF_F_FRAGLIST | NETIF_F_HIGHDM Some controllers that rely on header parsing for TX checksum offload can also handle multiple VLAN tags; our current chips recognise up to 3 VLAN tags with any of those Ethertypes. It would be good to have some way for physical device drivers to advertise this feature so that nested VLAN devices can take advantage of it. (I am definitely *not* proposing vlan2_features, vlan3_features, which wouldn't even be sufficient to represent Ethertype constraints.) Ben. -- Ben Hutchings, Staff Engineer, Solarflare Not speaking for my employer; that's the marketing department's job. They asked us to note that Solarflare product names are trademarked. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists