> + svc_rdma_build_send_wr(ctxt, sge_no); > + send_wr = &ctxt->send_wr; > + if (inv_rkey) { > + send_wr->opcode = IB_WR_SEND_WITH_INV; > + send_wr->ex.invalidate_rkey = inv_rkey; > + } Seems a bit odd to overwrite the work that svc_rdma_build_send_wr in the caller. I would propbably add this work to svc_rdma_build_send_wr. Also any reason svc_rdma_build_send_wr can't call svc_rdma_send directly (although it would probably need a new name then)? -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html