diff mbox

[1/2,Wily,SRU] UBUNTU: SAUCE: (noup) cxlflash: Fix to escalate LINK_RESET also on port 1

Message ID 1447103017-17807-1-git-send-email-tim.gardner@canonical.com
State New
Headers show

Commit Message

Tim Gardner Nov. 9, 2015, 9:03 p.m. UTC 
From: Manoj Kumar <manoj@linux.vnet.ibm.com>

BugLink: http://bugs.launchpad.net/bugs/1513583

The original fix to escalate a 'login timed out' error
to a LINK_RESET was only made for one of the two ports
on the card. This fix resolves the same issue for
the second port (port 1).

Signed-off-by: Manoj N. Kumar <manoj@linux.vnet.ibm.com>
Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
---
 drivers/scsi/cxlflash/main.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Kamal Mostafa Nov. 17, 2015, 7:38 p.m. UTC | #1 
On Mon, 2015-11-09 at 14:03 -0700, tim.gardner@canonical.com wrote:
> From: Manoj Kumar <manoj@linux.vnet.ibm.com>
> 
> BugLink: http://bugs.launchpad.net/bugs/1513583
> 
> The original fix to escalate a 'login timed out' error

Patch set applies and looks reasonable.

 -Kamal
Seth Forshee Nov. 17, 2015, 10:25 p.m. UTC | #2
Kamal Mostafa Nov. 19, 2015, 10 p.m. UTC | #3 
Applied 2/2 patches to Wily:

4a7dca0 UBUNTU: SAUCE: (noup) cxlflash: Fix to avoid virtual LUN failover failure
1bb5800 UBUNTU: SAUCE: (noup) cxlflash: Fix to escalate LINK_RESET also on port 1

 -Kamal
diff mbox

Patch

diff --git a/drivers/scsi/cxlflash/main.c b/drivers/scsi/cxlflash/main.c
index 1e5bf0c..09fe252 100644
--- a/drivers/scsi/cxlflash/main.c
+++ b/drivers/scsi/cxlflash/main.c
@@ -1108,7 +1108,7 @@  static const struct asyc_intr_info ainfo[] = {
 	{SISL_ASTATUS_FC1_OTHER, "other error", 1, CLR_FC_ERROR | LINK_RESET},
 	{SISL_ASTATUS_FC1_LOGO, "target initiated LOGO", 1, 0},
 	{SISL_ASTATUS_FC1_CRC_T, "CRC threshold exceeded", 1, LINK_RESET},
-	{SISL_ASTATUS_FC1_LOGI_R, "login timed out, retrying", 1, 0},
+	{SISL_ASTATUS_FC1_LOGI_R, "login timed out, retrying", 1, LINK_RESET},
 	{SISL_ASTATUS_FC1_LOGI_F, "login failed", 1, CLR_FC_ERROR},
 	{SISL_ASTATUS_FC1_LOGI_S, "login succeeded", 1, SCAN_HOST},
 	{SISL_ASTATUS_FC1_LINK_DN, "link down", 1, 0},