On Mon, 6 Oct 2003 22:04:07 +0200 Bart De Schuymer <bdschuym@xxxxxxxxxx> wrote: > - unsigned long hh[16 / sizeof(unsigned long)]; > + unsigned char hh[18]; Maybe don't do this? It should be aligned on a 'long' boundry I think. The only real hard objection I have with the patch is the vlan xmit routine pointer crap, just export the symbol. Once you fix that I'll apply your patch as long as Stephen has no objections.