Re: [PATCH 11/15] IB/srpt: Fix how aborted commands are processed

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 01/06/2016 06:13 AM, Christoph Hellwig wrote:
  	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..

This series indeed depends on my recently posted target core patch series. I will rebase, retest and repost this patch series.

Bart.
--
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



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux