OK, I applied this to svn. I changed > +static spinlock_t ipoib_all_neigh_list_lock; and > + spin_lock_init(&ipoib_all_neigh_list_lock); to just static DEFINE_SPINLOCK(ipoib_all_neigh_list_lock); There's not some subtle reason for using spin_lock_init() at runtime, is there? - R.