From patchwork Thu Feb 2 19:42:25 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Madhani, Himanshu" X-Patchwork-Id: 9553177 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id D359F60236 for ; Thu, 2 Feb 2017 19:59:59 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id C3BE928480 for ; Thu, 2 Feb 2017 19:59:59 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id B564128499; Thu, 2 Feb 2017 19:59:59 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.8 required=2.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_HI,T_DKIM_INVALID autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 01AC228480 for ; Thu, 2 Feb 2017 19:59:59 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751561AbdBBT76 (ORCPT ); Thu, 2 Feb 2017 14:59:58 -0500 Received: from mail-by2nam01on0052.outbound.protection.outlook.com ([104.47.34.52]:21760 "EHLO NAM01-BY2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751512AbdBBT74 (ORCPT ); Thu, 2 Feb 2017 14:59:56 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=CAVIUMNETWORKS.onmicrosoft.com; s=selector1-cavium-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=3qzVvvGfR1pYLXN/bUthi0lfnWVeI0fwnGMdhYpx4nM=; b=kmRCMwNuE+jmIVTva4FB2pDluI/Jk+T5R45BDRibSbFM6dMxNHgUmBvuWzZ94sNXzcgcYJEWoh8gUsIewrjuivl+HoAUqH8SAFVQELzblSJXS2Ctn6/1w/8Q9ukyeFQ0dj8/XMNd6cKOl83J9cQ+86o9bMsNjfBeEaL+EV9jneo= Received: from SN1PR0701CA0059.namprd07.prod.outlook.com (10.163.126.27) by BN1PR07MB198.namprd07.prod.outlook.com (10.242.216.144) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.888.16; Thu, 2 Feb 2017 19:43:14 +0000 Received: from BY2FFO11FD003.protection.gbl (2a01:111:f400:7c0c::151) by SN1PR0701CA0059.outlook.office365.com (2a01:111:e400:52fd::27) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.888.16 via Frontend Transport; Thu, 2 Feb 2017 19:43:13 +0000 Authentication-Results: spf=none (sender IP is 50.232.66.26) smtp.mailfrom=cavium.com; linux-iscsi.org; dkim=none (message not signed) header.d=none; linux-iscsi.org; dmarc=none action=none header.from=cavium.com; Received-SPF: None (protection.outlook.com: cavium.com does not designate permitted sender hosts) Received: from CAEXCH02.caveonetworks.com (50.232.66.26) by BY2FFO11FD003.mail.protection.outlook.com (10.1.14.125) with Microsoft SMTP Server (version=TLS1_0, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA_P384) id 15.1.874.2 via Frontend Transport; Thu, 2 Feb 2017 19:43:13 +0000 Received: from dut1171.mv.qlogic.com (172.29.51.171) by CAEXCH02.caveonetworks.com (10.17.4.29) with Microsoft SMTP Server id 14.2.309.2; Thu, 2 Feb 2017 11:43:08 -0800 Received: from dut1171.mv.qlogic.com (localhost [127.0.0.1]) by dut1171.mv.qlogic.com (8.14.7/8.14.7) with ESMTP id v12JgrRG000404; Thu, 2 Feb 2017 11:42:53 -0800 Received: (from root@localhost) by dut1171.mv.qlogic.com (8.14.7/8.14.7/Submit) id v12Jgrl3000403; Thu, 2 Feb 2017 11:42:53 -0800 From: Himanshu Madhani To: , , , CC: , , Subject: [PATCH 02/15] qla2xxx: Allow relogin to proceed if remote login did not finish Date: Thu, 2 Feb 2017 11:42:25 -0800 Message-ID: <1486064558-365-3-git-send-email-himanshu.madhani@cavium.com> X-Mailer: git-send-email 1.8.3.1 In-Reply-To: <1486064558-365-1-git-send-email-himanshu.madhani@cavium.com> References: <1486064558-365-1-git-send-email-himanshu.madhani@cavium.com> MIME-Version: 1.0 X-EOPAttributedMessage: 0 X-Forefront-Antispam-Report: CIP:50.232.66.26; IPV:NLI; CTRY:US; EFV:NLI; SFV:NSPM; SFS:(10009020)(6009001)(7916002)(39450400003)(2980300002)(428002)(189002)(199003)(4720700003)(92566002)(50466002)(6666003)(356003)(626004)(305945005)(69596002)(105586002)(106466001)(5660300001)(101416001)(47776003)(36756003)(1691005)(2950100002)(8936002)(80596001)(48376002)(50986999)(42186005)(76176999)(81166006)(50226002)(4326007)(54906002)(8656002)(2906002)(5003940100001)(8676002)(38730400001)(81156014)(33646002)(4001430100002)(86362001)(5001770100001)(2201001)(189998001)(107886002)(87636001); DIR:OUT; SFP:1101; SCL:1; SRVR:BN1PR07MB198; H:CAEXCH02.caveonetworks.com; FPR:; SPF:None; PTR:50-232-66-26-static.hfc.comcastbusiness.net; MX:1; A:1; LANG:en; X-Microsoft-Exchange-Diagnostics: 1; BY2FFO11FD003; 1:TT/tzYxzR0yF86it6aPHAZKtBn2p7TBkzDHcNuGZzhxbeOTss3/2osgARMU7y9Y4HUjFMmQ4OUWMFehUqFQ8reWnEyOKXbVNn+WTLI61EVojkq2xyQpE4KlyWtks5n5DyRehacaYq60xPYJuk/UP9CltozwzvsjC08QDNkysUjpAfKBZsF+Ulgzk7+qtXTUaePUU57ls2GO3v74k5xVb2ZsFUt9+iBH5LrCfvqsPfgcqcFp+gfUCfk7rk4Y2tDHI68wLWrDYhjbJ5ulybeJwPyR7n73KVPiTADFnskgj1mc8qRcorHrQf3L8Mdu0V8T7sromcBGBRr+POYecHHw6eLgn6LjWBr434o2HIL49Tifn4YPxED0ljZzst42rzUH1cTYjGWcsIVOSsrqnfsYRWJnh3bpcBpMHcFb98VkmWNjK/UXuOO5/HBG03K3UHWSU1HcuXpCFyf3se/z36IqhxmVgRo4mHfMnY7WM9XNKrB1aygUtCTTLW55y57/dnID9R0RHU+x7Fjpq23jzG/GazGLd6kfUC61HHluXJW/x6qI= X-MS-Office365-Filtering-Correlation-Id: cdef046c-34bd-42fb-b282-08d44ba3b9e2 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001); SRVR:BN1PR07MB198; X-Microsoft-Exchange-Diagnostics: 1; BN1PR07MB198; 3:K7/iaTrrvqnNiAX8KrDNCQFWRWBgl1u3f8xY3grfRItovXBmP8CpLRE6xa55O5VfIoh3FbMpl+ez13vhD+aS9C3plTtgbfy06roX4ccogB+wwLF5aF1XKui2V0fH4EUPG+cfVFEZKNMIz8pN4idGZL1fxmzSdM5K8+jsC8+aat1um2pZP6WWj1DosARVPHYoZfAT8wyO+WpLoOF72FH/gUheAzpETrBCg5mdk8PloyPhFd0/TEbzdR45awPlokq3llLloLMKcF6aP/fgi+qcGe9ZXllwmXXgywtF1AbnutTotL0SnYvMCq5FPUTDh6ronSDCLFb1q29xsKxyTVsbUMGaeXTOEzTmIk7d+Tz4VAX+MPDvJCjXeT1Evxy+8aR8; 25:GJCeU/H5tymONyAZyPAy0vMMWOebkhLeMsXuiPZrAfB6xFfgvzOJw6bNiV8FmFNxY/i5EbNosvZ9l/71rXurZLznhSDRMIq3JW5THRLsTVdAJhh6c/OPAUTnS722xt+BbUYfcUG1FX9trAvFnUH65vzAiguN6K7l7QoaBOzOFdtyEW1ZWKwSE1KTt2f5DewYoMT3L8dwYJAxCwY2Y1s+rwq0f4kisv3TXKc21baBrSF6pCggy/EwNdlMOb+0YpJAbKLk1xSP1+vm3+jCaaS6mXKHIG2zSByrCy4at/ZFbxGyVdFRHhU5s3XaiSpDjNQkQLyloRGUb6kAAM6i2wtCCnCnfqAXb2M9zTtOzXsUnH4vGdabFpgjBlYOn+KLEwr8Y6otQ5hOl3+8PLjunxawDGHjwA8sTl5vbx4rMwIZh58A5Gv3WQKLsO40kHQ22qWvM+oxLF8orqZtJkF0/zK4Dg== X-Microsoft-Exchange-Diagnostics: 1; BN1PR07MB198; 31:H1M9G76YkvF4UkHcQbDekrpW8mST6vm+HTc3TYcVwwyPN66rZsOTMMyW9UajUhVZXzfXnslLOdz3EpMSQAuaoZsqJV9CJYL1djE3ayVr/r8QMAbQA3BBtpSywfDoJBPSpAEaCetXaWt7k3Ouc9KIdCyl0U1gG2qdS1yNPr2Jh359s5p5g4WNK8ezomWwdRqTaK1vGC+L4w+r50/CzoYn1Gk9SBJ0fjJ7eCWHXINSc9xEpGd9SJdQLcShSyiI1Ur1AU0wPwGlyOD+4MOt8OSlXNRMeSp+z1Xiwm7f64SwTno=; 20:ezh7tpMocwSrPxU2KGclUGQUivtnPBzBPfNmzbAKcjvKj/XqfB/8hLKMujmJP2agkf8/Ipx0/vEkKzbk+kAl2NsPZFnJLcoCZOzlXDD47jKjbpSe3AJTyEQnlLXXRICehHE9r44qwgkEjx2O6zYNjBCVPhDML6MFAlkO8Vc+dAI6yKDmrvvQiWzu+BU7/O5c48Sry/lGx9o1b3q1Bbyf5yglkopPhOkQqWqGWcXtyGRqR2CkU/sB1W4M3Xpqb8k7wsUBAOCQB1KjNa+QmcERzya7cH7nw69jRoFDt2ewwjHxWuJ376D1lrFZ8kTrxXJhpM/QaLGZZbbVRzIgOEH7c6a1w/gF4Mj+ASGIpJR5mPi6YEOFV3e8hcJdW92W1b1Np0QaWnv47tcXu2HZADTITiSLhZQfMyp9+tJUeQNzJBrzAZuxauFatX6IwCqoUqO4tvm1Z058NGjLzk41i7Wsi8VAJZUJsyvFLRkQKZo+PSFFSggFzgYadXef5usr4wK2 X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(6040375)(601004)(2401047)(13017025)(13015025)(13018025)(5005006)(8121501046)(13023025)(13024025)(3002001)(10201501046)(6041248)(20161123558025)(20161123564025)(20161123560025)(20161123562025)(20161123555025)(6072148); SRVR:BN1PR07MB198; BCL:0; PCL:0; RULEID:; SRVR:BN1PR07MB198; X-Microsoft-Exchange-Diagnostics: 1; BN1PR07MB198; 4:N6coHwiJS72YqVJRhfvFUrEv0LIpoS842V90YpkGnHfGokks1JVzkaTUxxsXo/wzN0ln6XA4Ygxq2Eha0lRksfvxqT4OOL0rjEJBapfwgC0YJVj1eXmbyGtWUc7oUCTX6yD/oWLE/Kr4Jwic4pkM79uxMhQRSOgzOCzeEWjNE2tANBkCbFgXBmC4LtpJ8rOjsLo1t0806KBSRiQwmnfE7jG5FdaFdrudy49vmSA9hNflcSMTdtO63GU6AcW8IZh9JryBm24gZdjzGzLj1kTJ3CM6aR0CA2yCN3VYLzodDovzGcl9lqjAiAigmKKfGwAFR3A2jpadthCHEKEazDev8/cZ0D1XoJ21LyxTlZtx3kuyuvlAHY1VAg8gPAac/g7iL0DMgFnXtqijEm5Th5T3ZDOIF+NxnKyLxXdl4qAiGu4qDJh6MU0cz7+AYXIUtMFW7ZwVqcb1Mv8J5OWEjHSd3hOL+9nwlSQLgHgsvqLnhFQ9h4m6T5mYwIIPDR3D3ekIay0wPsoj8azLgjQCjMT3Febj23UU1YnwyfWU8TUcl4s7wRUXWHQZlpF61bhDLKcQSRF5rh3sTluIpWzEd0cUvlRgySYsGF9qQjSw0XHkCSmCQ3V2qvn6i/YQ0K7AwXGvj4r84NNiVfw+nAXLS70NcFZl1pPjKN8QAIv4c8w4HwwLd0SZ/Qdvyi0At64NIPvQvj3BbQGK2G8FKOyBdi6O0g== X-Forefront-PRVS: 02065A9E77 X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; BN1PR07MB198; 23:U4NeYXiyiO5XX5c4kcmyQrfvXwIx8sCfycIQiNBPW4?= =?us-ascii?Q?wI2pNoU6AF21z83dnpNyua0dGb+n+zFY+Mnp8/WBGNBfZdaf5t7uxFnSM74z?= =?us-ascii?Q?0JhHFg1x6/5StcClsbgUnzebCzJ4vSmhUIBZTP3mZSAU6bQztIaILbztI+gr?= =?us-ascii?Q?TFS8td0rZ/QyLVx7O9RT4VG/FIN8z24Ua9zDzF9T33TpPZvF4eLV5+pAPXtl?= =?us-ascii?Q?rdCaSDJnk4O2TuS6xL9t6qpVD+NaLtkKl4FwlUJzMTg27QAdQ/pWLQ7GXuff?= =?us-ascii?Q?l/NpJFBb05fkLbSNDEXPXkazd2//OmFcQZXo8+fd96dUns1X5nXCZhFF9036?= =?us-ascii?Q?F0JCZLpaWR+5vMicqqBXFTvJezt4PntvuTso71Ljelx6mpY+dDt8Qlh/yIUf?= =?us-ascii?Q?Vg81wSuaew9v+OSK1d04fyOOebeaMIcpGNSp1EQuY0CeN9MPLiRdw7Z9ns+V?= =?us-ascii?Q?86fnDUO6WaYt2jOz3QSmvalsZUh+YapX7FQkD1KFio2UQgusNWfD2F5/tJMM?= =?us-ascii?Q?4zP730wDnXFvHWtQidjqoyD6t8dwLu/9S9SuIZIsDd81ZPJ0odMRaEyFA+bT?= =?us-ascii?Q?BGpsDNeJT/b5VGFPlUB2vBenSOVOOyXwN9i2zlfZlaRYv43lSHMgtdFOn04C?= =?us-ascii?Q?XOJkDPsFQp4qxD1UrERUoqq+/lCm73ED8lKm5IG+3hTZIVbjkGMnKqGOelee?= =?us-ascii?Q?GED6mD0eIE90uN2BNJw/+f8JZpyWbjleRQ3GArhp7LTccGkyrMmoGad5nUCS?= =?us-ascii?Q?5NxQP5JjHewmBbS3HCWGtHA0vtB7ATEZco5xcehobtTjqZrOpKnHCxBCppXn?= =?us-ascii?Q?tcPuq/W4GiddU9G9/oLn7aW8gUAtj8+0dblzWV9hFYQpokY2v9U3Vb/jwLu5?= =?us-ascii?Q?dHT85Y/wnk+sFSUudpAdpYYBW2bDrhdVJEtFArakEgz4CuAYqcyX4M5d4TiF?= =?us-ascii?Q?n40AZWYYYexDSJ7KREUHhLO6SECKR7WFkxsdCYRrvQe2mhYV4VbBwt/Lci/T?= =?us-ascii?Q?uo8pD2xP/2t038a8Ol0VHNL1VCWalLi55xwP4WooqQuLo3cL/erUAB3qL3i9?= =?us-ascii?Q?pEm6nTcNmFkInJKQrx+tViCKrgaiGto2OM6+1D3OP334OM/S7T30V98mIdqd?= =?us-ascii?Q?UG/3yrQYEp3uIz4Kx8aZPqKv+oVCKpLayMfEAfvRkZ3GbOg93ZCQ=3D=3D?= X-Microsoft-Exchange-Diagnostics: 1; BN1PR07MB198; 6:WN3n7bXk1q6geCZsKS87jH3IhfUiYW9yoGzkJD129OM6f60BqgTxqhipSBlb12U2jQpn5Xo7/lzBHFOUt6Hvev8O8x63RyTuyyhb7oUcaQPRPzyqBDULRSmXl7BaiRH6F06KBPC38BlzswQwKQgoEwaF6gx5CVD685M+SguK4wrGtnuqmr5zrm7fcVJoqfcqYG2g1tFrkaQuoiOHb6Ero1D9palLnGo2BSU3lsiDJoP0a5gcKCB4RYUySKbtKEqPN1vI9ktveWlfLXT85zK/nnpA3/yosabjEsCUI2/ykgsRSWHeKL+PKDpeV7k24C8PocgzbXvIs9iWbYrfpNTPpv+hQXbM91iesEpeckXb8IcPwjJ8/aqJ2uSwUgYoJYjNpTHqpeRktSfzbA8xt2dbkA==; 5:om6oKODclBTxSReyXDcZ3vm1bOOBUQypqDxc8j1ygSForkVobpxdJaWbjDgNiTpd7W0dEKXt7qyEyYSQdj7+C36iNzxH24QYUBjjdppFcb/GOMtJ6hUmtpGlTsoPHq8wSbOS5bx+WWG679ILe5XUQbddaEIUV5FXZzI5fz8zodg=; 24:5Z109UBDPg2ybFwU9UA1Os9ZmgLf9wD2W8PWLaNLRS0G3sQ9vzsGn2+tFb+d2YZNFwzsEiLEcctJjLxietDYwhnhK6Mebf7LndMoxyFKq7I= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; BN1PR07MB198; 7:A5mc7JZqJJSV1GEBZCOCW6Gc0glzAIZZimx8DynDqSK6ukuJQFkC2ux3Vj5vVhwn1lB4ykMqqcSTa+ATeaYNE3a2tQ8Pdz5gHuT15x5aUynvKXiaPFx7HBYha/mL4PBd+UIZLGzZToXaELpEz7XdW1V7VTxHzFmE5h5W7dj4mCaoWX3hVc6UmqsjzE/oaiorIreek93vb2egaHqWnTNqgCKUCMBO4i3TZdg+5WhNMC1lV2DhpUjmuc48j1R7cdZ5VzwQeLRSxFgiMroOLkCPB2Beh8EiytWrpBr/QLtmA8FG3xsqWT8CDenUnoLXE7JpHOiTlSGvshvSP52SJAANaCaC3kqyKftDXwhoWwBl+Te3vv96dE3o6wZNN3EQdkek8ltF7Ku4LEvS3Rjd0HeHV91DvGbx9RFB+NQVWUThaIRu5Cix+S6rtGqeOflI0StP+0BH6kpq1lntTz+4BAq7FP8K9/rwjS4iANH2xE0WE5HWNMA1vFuSHt6pAJZHTOejAMzH4LivxiyCn01jsamavw== X-OriginatorOrg: cavium.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Feb 2017 19:43:13.4448 (UTC) X-MS-Exchange-CrossTenant-Id: 711e4ccf-2e9b-4bcf-a551-4094005b6194 X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=711e4ccf-2e9b-4bcf-a551-4094005b6194; Ip=[50.232.66.26]; Helo=[CAEXCH02.caveonetworks.com] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN1PR07MB198 Sender: target-devel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: target-devel@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP From: Quinn Tran If the remote port have started the login process, then the PLOGI and PRLI should be back to back. Driver will allow the remote port to complete the process. For the case where the remote port decide to back off from sending PRLI, this local port sets an expiration timer for the PRLI. Once the expiration time passes, the relogin retry logic is allowed to go through and perform login with the remote port. Signed-off-by: Quinn Tran Signed-off-by: Himanshu Madhani --- drivers/scsi/qla2xxx/qla_def.h | 2 ++ drivers/scsi/qla2xxx/qla_init.c | 16 ++++++++++++++-- drivers/scsi/qla2xxx/qla_isr.c | 25 +++++++++++++++++++------ drivers/scsi/qla2xxx/qla_target.c | 1 + 4 files changed, 36 insertions(+), 8 deletions(-) diff --git a/drivers/scsi/qla2xxx/qla_def.h b/drivers/scsi/qla2xxx/qla_def.h index 2d7d478..b180d51 100644 --- a/drivers/scsi/qla2xxx/qla_def.h +++ b/drivers/scsi/qla2xxx/qla_def.h @@ -2300,6 +2300,8 @@ enum rscn_addr_format { struct ct_sns_desc ct_desc; enum discovery_state disc_state; enum login_state fw_login_state; + unsigned long plogi_nack_done_jiff; + u32 login_gen, last_login_gen; u32 rscn_gen, last_rscn_gen; u32 chip_reset; diff --git a/drivers/scsi/qla2xxx/qla_init.c b/drivers/scsi/qla2xxx/qla_init.c index 9f3db52..e84ab37 100644 --- a/drivers/scsi/qla2xxx/qla_init.c +++ b/drivers/scsi/qla2xxx/qla_init.c @@ -876,10 +876,16 @@ int qla24xx_fcport_handle_login(struct scsi_qla_host *vha, fc_port_t *fcport) fcport->login_retry--; if ((fcport->fw_login_state == DSC_LS_PLOGI_PEND) || - (fcport->fw_login_state == DSC_LS_PLOGI_COMP) || (fcport->fw_login_state == DSC_LS_PRLI_PEND)) return 0; + if (fcport->fw_login_state == DSC_LS_PLOGI_COMP) { + unsigned long t = fcport->plogi_nack_done_jiff + HZ; + + if (time_before_eq(jiffies, t)) + return 0; + } + /* for pure Target Mode. Login will not be initiated */ if (vha->host->active_mode == MODE_TARGET) return 0; @@ -1041,10 +1047,16 @@ void qla24xx_handle_relogin_event(scsi_qla_host_t *vha, fcport->flags); if ((fcport->fw_login_state == DSC_LS_PLOGI_PEND) || - (fcport->fw_login_state == DSC_LS_PLOGI_COMP) || (fcport->fw_login_state == DSC_LS_PRLI_PEND)) return; + if (fcport->fw_login_state == DSC_LS_PLOGI_COMP) { + unsigned long t = fcport->plogi_nack_done_jiff + HZ; + + if (time_before_eq(jiffies, t)) + return; + } + if (fcport->flags & FCF_ASYNC_SENT) { fcport->login_retry++; set_bit(RELOGIN_NEEDED, &vha->dpc_flags); diff --git a/drivers/scsi/qla2xxx/qla_isr.c b/drivers/scsi/qla2xxx/qla_isr.c index fb1c39c..bd998bf 100644 --- a/drivers/scsi/qla2xxx/qla_isr.c +++ b/drivers/scsi/qla2xxx/qla_isr.c @@ -1634,9 +1634,9 @@ static void qla_irq_affinity_notify(struct irq_affinity_notify *, QLA_LOGIO_LOGIN_RETRIED : 0; if (logio->entry_status) { ql_log(ql_log_warn, fcport->vha, 0x5034, - "Async-%s error entry - hdl=%x" + "Async-%s error entry - %8phC hdl=%x" "portid=%02x%02x%02x entry-status=%x.\n", - type, sp->handle, fcport->d_id.b.domain, + type, fcport->port_name, sp->handle, fcport->d_id.b.domain, fcport->d_id.b.area, fcport->d_id.b.al_pa, logio->entry_status); ql_dump_buffer(ql_dbg_async + ql_dbg_buffer, vha, 0x504d, @@ -1647,8 +1647,9 @@ static void qla_irq_affinity_notify(struct irq_affinity_notify *, if (le16_to_cpu(logio->comp_status) == CS_COMPLETE) { ql_dbg(ql_dbg_async, fcport->vha, 0x5036, - "Async-%s complete - hdl=%x portid=%02x%02x%02x " - "iop0=%x.\n", type, sp->handle, fcport->d_id.b.domain, + "Async-%s complete - %8phC hdl=%x portid=%02x%02x%02x " + "iop0=%x.\n", type, fcport->port_name, sp->handle, + fcport->d_id.b.domain, fcport->d_id.b.area, fcport->d_id.b.al_pa, le32_to_cpu(logio->io_parameter[0])); @@ -1688,6 +1689,17 @@ static void qla_irq_affinity_notify(struct irq_affinity_notify *, case LSC_SCODE_NPORT_USED: data[0] = MBS_LOOP_ID_USED; break; + case LSC_SCODE_CMD_FAILED: + if (iop[1] == 0x0606) { + /* + * PLOGI/PRLI Completed. We must have Recv PLOGI/PRLI, + * Target side acked. + */ + data[0] = MBS_COMMAND_COMPLETE; + goto logio_done; + } + data[0] = MBS_COMMAND_ERROR; + break; case LSC_SCODE_NOXCB: vha->hw->exch_starvation++; if (vha->hw->exch_starvation > 5) { @@ -1709,8 +1721,9 @@ static void qla_irq_affinity_notify(struct irq_affinity_notify *, } ql_dbg(ql_dbg_async, fcport->vha, 0x5037, - "Async-%s failed - hdl=%x portid=%02x%02x%02x comp=%x " - "iop0=%x iop1=%x.\n", type, sp->handle, fcport->d_id.b.domain, + "Async-%s failed - %8phC hdl=%x portid=%02x%02x%02x comp=%x " + "iop0=%x iop1=%x.\n", type, fcport->port_name, + sp->handle, fcport->d_id.b.domain, fcport->d_id.b.area, fcport->d_id.b.al_pa, le16_to_cpu(logio->comp_status), le32_to_cpu(logio->io_parameter[0]), diff --git a/drivers/scsi/qla2xxx/qla_target.c b/drivers/scsi/qla2xxx/qla_target.c index 270e01c..821e63f 100644 --- a/drivers/scsi/qla2xxx/qla_target.c +++ b/drivers/scsi/qla2xxx/qla_target.c @@ -554,6 +554,7 @@ void qla2x00_async_nack_sp_done(void *s, int res) sp->fcport->login_gen++; sp->fcport->fw_login_state = DSC_LS_PLOGI_COMP; sp->fcport->logout_on_delete = 1; + sp->fcport->plogi_nack_done_jiff = jiffies; break; case SRB_NACK_PRLI: