Michael> if (wr->opcode >= sizeof mthca_opcode / sizeof mthca_opcode[0]) Michael> { Michael> ret = -1; Michael> *bad_wr = wr; Michael> goto out; Michael> } Michael> in mthca on data path? Should this be put within ifdef Michael> DEBUG or something? Probably not needed -- I guess we can trust what the consumer gives us. - R.