>       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 [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to