> pr_debug("Aborting cmd with state %d and tag %lld\n", state, > ioctx->cmd.tag); > > @@ -1299,14 +1291,16 @@ static int srpt_abort_cmd(struct srpt_send_ioctx *ioctx) > case SRPT_STATE_NEW: > case SRPT_STATE_DATA_IN: > case SRPT_STATE_MGMT: > + case SRPT_STATE_DONE: > /* > * Do nothing - defer abort processing until > * srpt_queue_response() is invoked. > */ > - WARN_ON(!(ioctx->cmd.transport_state & CMD_T_ABORTED)); Seems like this depends on your target core changes? Maybe it would be better to respin the series to got just on top of Doug's RDMA tree, as I think we're more likely to get this series merged for 4.5 than the target core changes.. Otherwise these changes look fine to me. -- 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