Roland, do we really need code like if (wr->opcode >= sizeof mthca_opcode / sizeof mthca_opcode[0]) { ret = -1; *bad_wr = wr; goto out; } in mthca on data path? Should this be put within ifdef DEBUG or something? -- MST