Peter> why not make this change to IPOIB_HW_ADDR_LEN now? Not a bad idea (although I guess it should be IPOIB_ALEN to match the rest of the kernel). I wonder where to put the value though... is it really worth creating a <linux/if_ipoib.h> for this value? - R.