Re: Various 802.1Q VLAN driver patches. [try2]

Ben Greear (greearb@candelatech.com)
Fri, 01 Mar 2002 14:19:09 -0700


Jeff Garzik wrote:

>>Well, this was discussed on the VLAN mailing list. The conclusion
>>there was that it will not hurt on most cards if it is enabled
>>unconditionally.
>>
>
> Well, conclusions like that slow down packet processing on the chip,
> quite often...

Do you expect to see slowdowns by just enabling large packet receives,
or just when you actually receive a large packet?

>
> It does, but not directly. The infrastructure for VLAN and changing MTU
> share common elements, so both should be merged at the same time.
>
> This is ESPECIALLY key with 3c59x, because we are turning on support for
> large frames, not specifically VLAN. That is obviously the same
> operation as changing MTU to a larger, non-standard one, and so should
> not be treated as something vlan-specific.

The subtle difference is that we want to be able to have an MTU 4 bytes
bigger than the VLAN device's MTU, but we most likely do NOT want
the rest of the stack to know we are using the higher MTU, because
then non-vlan packets may not be accepted at the other end. In
other words, we want eth0's MTU to be 1500, and VLAN1's MTU to be
1500, as far as the layer 3 stacks are concerned. However, eth0
must really be able to handle a packet of 1504 length.

> Early next week, I will likely make a bombing run through several
> drivers, fixing up the large frame and MTU issues. That should be
> enough for software VLAN.

That will be a very welcome development!

Thanks,
Ben

>
> Jeff
>
>

-- 
Ben Greear <greearb@candelatech.com>       <Ben_Greear AT excite.com>
President of Candela Technologies Inc      http://www.candelatech.com
ScryMUD:  http://scry.wanfear.com     http://scry.wanfear.com/~greear

- 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/