Roland Dreier wrote:
One meta comment: when sending a series of 4 patches, please choose a descriptive subject for each one. We don't want the same headline in the kernel log for all 4 patches.
I'll remember next time
> - qp_attr.qp_state = IB_QPS_RESET; > - ret = ib_modify_qp(target->qp, &qp_attr, IB_QP_STATE); > - if (ret) > - goto err; > - > - ret = srp_init_qp(target, target->qp); > - if (ret) > + old_qp = target->qp; > + old_cq = target->cq; > + ret = srp_create_target_ib(target); I don't understand why this is any faster? Is creating a new QP really any different from modifying an old QP to reset state?
It is not; it is needed for patch 2/4 disconnect without wait and it is faster that way
-- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
