On 3/7/2013 2:20 PM, Mike Christie wrote: > On 03/07/2013 02:13 PM, Jeremy Linton wrote: >> For lpfc, you never get to the code. Or rather when I was testing it, I >> couldn't find any way to propagate an error beyond the initial >> lpfc_reset_flush_io_context() call in lpfc_device_reset_handler(). >> >> That call pretty much always returns success indpependent of the remote >> device because the firmware acks the context clear aborts, resulting in the >> outstanding iocb count being zero (independent of both the mid layer status >> and the actual device state). >> > > Your lpfc patch fixes that right? Yes. It allows the device reset to fail if the device doesn't respond to the task mgmt request, or rejects it, etc. It doesn't unjam the commands that get aborted by the flush_io_context() call. Those have to depend on their timeouts. That is another patch... -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
