All of lore.kernel.org
 help / color / mirror / Atom feed
From: james_p_freyensee@linux.intel.com (J Freyensee)
Subject: [PATCH v3 6/7] nvme-fabrics: Add target support for FC transport
Date: Thu, 27 Oct 2016 16:57:05 -0700	[thread overview]
Message-ID: <1477612625.2838.50.camel@linux.intel.com> (raw)
In-Reply-To: <1638bc85-4b94-a74d-e0ef-c955c6f7bf83@broadcom.com>


> nvme_fc_abort_op() would not be called in the normal case.
> 
> nvme_fc_abort_op() is called on exception cases. It's called to
> request the LLDD to kill the exchange (the thing the initiator and
> target are using for the io) mid processing due to some error.? The
> cases where it's called are:
> When queues are deleted, thus every outstanding io on that queue
> needs to be implicitly terminated.
> If the LLDD refused to send data for the exchange. Something is bad
> in the LLDD or the exchange is dead. So we formally terminate the
> exchange as there's nothing more the transport can do with it. On the
> other end of the wire, the initiator won't see the io complete, thus
> it'll usually fall into the escalation policy of terminating the
> queue or resetting the controller.
> if the transport detects a software error for the io that had a bad
> op code in it. At least kill the FC resource so the initiator will
> see it and recover.
> The call to nvmet_fc_xmt_fcp_op_done() with the ABORT op, is the LLDD
> saying it's completed the abort, freed the hw resource, and the
> job/exchange is released back to the transport. At that point, the
> transport will finish its teardown of the io.
> 

Thanks for the explanation James!
Jay

> Thanks
> 
> -- james
> 
> 

      parent reply	other threads:[~2016-10-27 23:57 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-10-23 19:31 [PATCH v3 6/7] nvme-fabrics: Add target support for FC transport James Smart
2016-10-26 19:09 ` J Freyensee
     [not found]   ` <1638bc85-4b94-a74d-e0ef-c955c6f7bf83@broadcom.com>
2016-10-27 23:57     ` J Freyensee [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1477612625.2838.50.camel@linux.intel.com \
    --to=james_p_freyensee@linux.intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.