BTW, since mthca currently calls completion handlers directly from interrupt context (rather than BH/tasklet context), it might be worth renaming all the SDP locking macros so they're not confusingly named with _BH suffixes. - R.