Comments:
1) This patch is (like I mentioned earlier) for reference only, not for
application. It is really halfway in between software and hardware VLAN
support.
2) You don't want to set_8021q_mode if VLAN is not active. It's silly
to activate it when VLAN is compiled as a module but no one is using
vlans. That's going to be THE common case, because distros will
inevitably build the VLAN module with their stock kernel.
3) 3c59x needs real dev->change_mtu support. This patch provides the
info needed to do that... but doesn't do that.
4) It uses magic numbers instead of ETH_DATA_LEN and ETH_HLEN
-- Jeff Garzik | Building 1024 | MandrakeSoft | Choose life. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/