Yep, except to be consistent with the structure members, I renamed it to 'struct click_ether_vlan'; sorry for the cussedness :) It's checked in to CVS. E