* [PATCH 0/2] ibmvfc: FC-TAPE Support
@ 2016-08-03 21:36 Tyrel Datwyler
2016-08-03 21:36 ` [PATCH] ibmvfc: Set READ FCP_XFER_READY DISABLED bit in PRLI Tyrel Datwyler
` (3 more replies)
0 siblings, 4 replies; 6+ messages in thread
From: Tyrel Datwyler @ 2016-08-03 21:36 UTC (permalink / raw)
To: james.bottomley
Cc: martin.petersen, linux-scsi, linuxppc-dev, brking, nfont,
linux-kernel, mpe, Tyrel Datwyler
This patchset introduces optional FC-TAPE/FC Class 3 Error Recovery to the
ibmvfc client driver.
Tyrel Datwyler (2):
ibmvfc: Set READ FCP_XFER_READY DISABLED bit in PRLI
ibmvfc: add FC Class 3 Error Recovery support
drivers/scsi/ibmvscsi/ibmvfc.c | 11 +++++++++++
drivers/scsi/ibmvscsi/ibmvfc.h | 1 +
2 files changed, 12 insertions(+)
--
2.7.4
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] ibmvfc: Set READ FCP_XFER_READY DISABLED bit in PRLI
2016-08-03 21:36 [PATCH 0/2] ibmvfc: FC-TAPE Support Tyrel Datwyler
@ 2016-08-03 21:36 ` Tyrel Datwyler
2016-08-03 21:36 ` [PATCH] ibmvfc: add FC Class 3 Error Recovery support Tyrel Datwyler
` (2 subsequent siblings)
3 siblings, 0 replies; 6+ messages in thread
From: Tyrel Datwyler @ 2016-08-03 21:36 UTC (permalink / raw)
To: james.bottomley
Cc: martin.petersen, linux-scsi, linuxppc-dev, brking, nfont,
linux-kernel, mpe, Tyrel Datwyler
The READ FCP_XFER_READY DISABLED bit is required to always be set to
one since FCP-3. Set it in the service parameter page frame during
process login.
Signed-off-by: Tyrel Datwyler <tyreld@linux.vnet.ibm.com>
---
drivers/scsi/ibmvscsi/ibmvfc.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/scsi/ibmvscsi/ibmvfc.c b/drivers/scsi/ibmvscsi/ibmvfc.c
index ab67ec4..4a680ce 100644
--- a/drivers/scsi/ibmvscsi/ibmvfc.c
+++ b/drivers/scsi/ibmvscsi/ibmvfc.c
@@ -3381,6 +3381,7 @@ static void ibmvfc_tgt_send_prli(struct ibmvfc_target *tgt)
prli->parms.type = IBMVFC_SCSI_FCP_TYPE;
prli->parms.flags = cpu_to_be16(IBMVFC_PRLI_EST_IMG_PAIR);
prli->parms.service_parms = cpu_to_be32(IBMVFC_PRLI_INITIATOR_FUNC);
+ prli->parms.service_parms |= cpu_to_be32(IBMVFC_PRLI_READ_FCP_XFER_RDY_DISABLED);
ibmvfc_set_tgt_action(tgt, IBMVFC_TGT_ACTION_INIT_WAIT);
if (ibmvfc_send_event(evt, vhost, default_timeout)) {
--
2.7.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH] ibmvfc: add FC Class 3 Error Recovery support
2016-08-03 21:36 [PATCH 0/2] ibmvfc: FC-TAPE Support Tyrel Datwyler
2016-08-03 21:36 ` [PATCH] ibmvfc: Set READ FCP_XFER_READY DISABLED bit in PRLI Tyrel Datwyler
@ 2016-08-03 21:36 ` Tyrel Datwyler
2016-08-11 20:37 ` [PATCH 0/2] ibmvfc: FC-TAPE Support Tyrel Datwyler
2016-08-26 3:26 ` Martin K. Petersen
3 siblings, 0 replies; 6+ messages in thread
From: Tyrel Datwyler @ 2016-08-03 21:36 UTC (permalink / raw)
To: james.bottomley
Cc: martin.petersen, linux-scsi, linuxppc-dev, brking, nfont,
linux-kernel, mpe, Tyrel Datwyler
The ibmvfc driver currently doesn't support FC Class 3 Error Recovery.
However, it is simply a matter of informing the VIOS that the payload
expects to use sequence level error recovery via a bit flag in the
ibmvfc_cmd structure.
This patch adds a module parameter to enable error recovery support
at boot time. When enabled the RETRY service parameter bit is set
during PRLI, and ibmvfc_cmd->flags includes the IBMVFC_CLASS_3_ERR
bit.
Signed-off-by: Tyrel Datwyler <tyreld@linux.vnet.ibm.com>
---
drivers/scsi/ibmvscsi/ibmvfc.c | 10 ++++++++++
drivers/scsi/ibmvscsi/ibmvfc.h | 1 +
2 files changed, 11 insertions(+)
diff --git a/drivers/scsi/ibmvscsi/ibmvfc.c b/drivers/scsi/ibmvscsi/ibmvfc.c
index 4a680ce..6b92169 100644
--- a/drivers/scsi/ibmvscsi/ibmvfc.c
+++ b/drivers/scsi/ibmvscsi/ibmvfc.c
@@ -52,6 +52,7 @@ static unsigned int max_requests = IBMVFC_MAX_REQUESTS_DEFAULT;
static unsigned int disc_threads = IBMVFC_MAX_DISC_THREADS;
static unsigned int ibmvfc_debug = IBMVFC_DEBUG;
static unsigned int log_level = IBMVFC_DEFAULT_LOG_LEVEL;
+static unsigned int cls3_error = IBMVFC_CLS3_ERROR;
static LIST_HEAD(ibmvfc_head);
static DEFINE_SPINLOCK(ibmvfc_driver_lock);
static struct scsi_transport_template *ibmvfc_transport_template;
@@ -86,6 +87,9 @@ MODULE_PARM_DESC(debug, "Enable driver debug information. "
module_param_named(log_level, log_level, uint, 0);
MODULE_PARM_DESC(log_level, "Set to 0 - 4 for increasing verbosity of device driver. "
"[Default=" __stringify(IBMVFC_DEFAULT_LOG_LEVEL) "]");
+module_param_named(cls3_error, cls3_error, uint, 0);
+MODULE_PARM_DESC(log_level, "Enable FC Class 3 Error Recovery. "
+ "[Default=" __stringify(IBMVFC_CLS3_ERROR) "]");
static const struct {
u16 status;
@@ -1335,6 +1339,9 @@ static int ibmvfc_map_sg_data(struct scsi_cmnd *scmd,
struct srp_direct_buf *data = &vfc_cmd->ioba;
struct ibmvfc_host *vhost = dev_get_drvdata(dev);
+ if (cls3_error)
+ vfc_cmd->flags |= cpu_to_be16(IBMVFC_CLASS_3_ERR);
+
sg_mapped = scsi_dma_map(scmd);
if (!sg_mapped) {
vfc_cmd->flags |= cpu_to_be16(IBMVFC_NO_MEM_DESC);
@@ -3383,6 +3390,9 @@ static void ibmvfc_tgt_send_prli(struct ibmvfc_target *tgt)
prli->parms.service_parms = cpu_to_be32(IBMVFC_PRLI_INITIATOR_FUNC);
prli->parms.service_parms |= cpu_to_be32(IBMVFC_PRLI_READ_FCP_XFER_RDY_DISABLED);
+ if (cls3_error)
+ prli->parms.service_parms |= cpu_to_be32(IBMVFC_PRLI_RETRY);
+
ibmvfc_set_tgt_action(tgt, IBMVFC_TGT_ACTION_INIT_WAIT);
if (ibmvfc_send_event(evt, vhost, default_timeout)) {
vhost->discovery_threads--;
diff --git a/drivers/scsi/ibmvscsi/ibmvfc.h b/drivers/scsi/ibmvscsi/ibmvfc.h
index 8fae032..7f9bb07 100644
--- a/drivers/scsi/ibmvscsi/ibmvfc.h
+++ b/drivers/scsi/ibmvscsi/ibmvfc.h
@@ -54,6 +54,7 @@
#define IBMVFC_DEV_LOSS_TMO (5 * 60)
#define IBMVFC_DEFAULT_LOG_LEVEL 2
#define IBMVFC_MAX_CDB_LEN 16
+#define IBMVFC_CLS3_ERROR 0
/*
* Ensure we have resources for ERP and initialization:
--
2.7.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 0/2] ibmvfc: FC-TAPE Support
2016-08-03 21:36 [PATCH 0/2] ibmvfc: FC-TAPE Support Tyrel Datwyler
2016-08-03 21:36 ` [PATCH] ibmvfc: Set READ FCP_XFER_READY DISABLED bit in PRLI Tyrel Datwyler
2016-08-03 21:36 ` [PATCH] ibmvfc: add FC Class 3 Error Recovery support Tyrel Datwyler
@ 2016-08-11 20:37 ` Tyrel Datwyler
2016-08-12 3:11 ` Martin K. Petersen
2016-08-26 3:26 ` Martin K. Petersen
3 siblings, 1 reply; 6+ messages in thread
From: Tyrel Datwyler @ 2016-08-11 20:37 UTC (permalink / raw)
To: Tyrel Datwyler, james.bottomley
Cc: martin.petersen, linux-scsi, linuxppc-dev, brking, nfont,
linux-kernel, mpe
On 08/03/2016 02:36 PM, Tyrel Datwyler wrote:
> This patchset introduces optional FC-TAPE/FC Class 3 Error Recovery to the
> ibmvfc client driver.
>
> Tyrel Datwyler (2):
> ibmvfc: Set READ FCP_XFER_READY DISABLED bit in PRLI
> ibmvfc: add FC Class 3 Error Recovery support
>
> drivers/scsi/ibmvscsi/ibmvfc.c | 11 +++++++++++
> drivers/scsi/ibmvscsi/ibmvfc.h | 1 +
> 2 files changed, 12 insertions(+)
>
ping?
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 0/2] ibmvfc: FC-TAPE Support
2016-08-11 20:37 ` [PATCH 0/2] ibmvfc: FC-TAPE Support Tyrel Datwyler
@ 2016-08-12 3:11 ` Martin K. Petersen
0 siblings, 0 replies; 6+ messages in thread
From: Martin K. Petersen @ 2016-08-12 3:11 UTC (permalink / raw)
To: Tyrel Datwyler
Cc: james.bottomley, martin.petersen, linux-scsi, linuxppc-dev,
brking, nfont, linux-kernel, mpe
>>>>> "Tyrel" == Tyrel Datwyler <tyreld@linux.vnet.ibm.com> writes:
Tyrel> On 08/03/2016 02:36 PM, Tyrel Datwyler wrote:
>> This patchset introduces optional FC-TAPE/FC Class 3 Error Recovery
>> to the ibmvfc client driver.
>>
>> Tyrel Datwyler (2): ibmvfc: Set READ FCP_XFER_READY DISABLED bit in
>> PRLI ibmvfc: add FC Class 3 Error Recovery support
>>
>> drivers/scsi/ibmvscsi/ibmvfc.c | 11 +++++++++++
>> drivers/scsi/ibmvscsi/ibmvfc.h | 1 + 2 files changed, 12
>> insertions(+)
>>
Tyrel> ping?
-ENOREVIEWS
--
Martin K. Petersen Oracle Linux Engineering
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 0/2] ibmvfc: FC-TAPE Support
2016-08-03 21:36 [PATCH 0/2] ibmvfc: FC-TAPE Support Tyrel Datwyler
` (2 preceding siblings ...)
2016-08-11 20:37 ` [PATCH 0/2] ibmvfc: FC-TAPE Support Tyrel Datwyler
@ 2016-08-26 3:26 ` Martin K. Petersen
3 siblings, 0 replies; 6+ messages in thread
From: Martin K. Petersen @ 2016-08-26 3:26 UTC (permalink / raw)
To: Tyrel Datwyler
Cc: james.bottomley, martin.petersen, linux-scsi, linuxppc-dev,
brking, nfont, linux-kernel, mpe
>>>>> "Tyrel" == Tyrel Datwyler <tyreld@linux.vnet.ibm.com> writes:
Tyrel> This patchset introduces optional FC-TAPE/FC Class 3 Error
Tyrel> Recovery to the ibmvfc client driver.
Applied to 4.9/scsi-queue.
--
Martin K. Petersen Oracle Linux Engineering
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-08-26 3:27 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-03 21:36 [PATCH 0/2] ibmvfc: FC-TAPE Support Tyrel Datwyler
2016-08-03 21:36 ` [PATCH] ibmvfc: Set READ FCP_XFER_READY DISABLED bit in PRLI Tyrel Datwyler
2016-08-03 21:36 ` [PATCH] ibmvfc: add FC Class 3 Error Recovery support Tyrel Datwyler
2016-08-11 20:37 ` [PATCH 0/2] ibmvfc: FC-TAPE Support Tyrel Datwyler
2016-08-12 3:11 ` Martin K. Petersen
2016-08-26 3:26 ` Martin K. Petersen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).