From patchwork Wed Nov 11 04:58:00 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Muneendra Kumar X-Patchwork-Id: 323441 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.2 required=3.0 tests=BAYES_00, DATE_IN_PAST_06_12, DKIMWL_WL_HIGH,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU, HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI, MIME_HEADER_CTYPE_ONLY, MIME_NO_TEXT, SPF_HELO_NONE, SPF_PASS, T_TVD_MIME_NO_HEADERS, USER_AGENT_GIT autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 617DAC5517A for ; Wed, 11 Nov 2020 11:51:26 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id D582720709 for ; Wed, 11 Nov 2020 11:51:25 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=broadcom.com header.i=@broadcom.com header.b="Vj7uPlo0" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726306AbgKKLvZ (ORCPT ); Wed, 11 Nov 2020 06:51:25 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:38048 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726203AbgKKLvY (ORCPT ); Wed, 11 Nov 2020 06:51:24 -0500 Received: from mail-pg1-x535.google.com (mail-pg1-x535.google.com [IPv6:2607:f8b0:4864:20::535]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 523A6C0613D1 for ; Wed, 11 Nov 2020 03:51:24 -0800 (PST) Received: by mail-pg1-x535.google.com with SMTP id w4so1270654pgg.13 for ; Wed, 11 Nov 2020 03:51:24 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; h=from:to:cc:subject:date:message-id; bh=sSgFJjjAyJj1vMGmRtQKciU0IetSFAK733uuPVUhQrA=; b=Vj7uPlo0vLc8nDMkZoQJcQqnErAwE8M30VakEP1h4bFCfK6HDqHJQKPPRRfJr7Gk7r h8mlZOq4BxIBOstXdUQlMA16z0kS68uI0S1QhGBg40Y+brzpH6aTc23cRa5egBGXeEEY wDzDxyUl/1pFMS8J7tPFEDzXoR9qQyk30Ha78= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=sSgFJjjAyJj1vMGmRtQKciU0IetSFAK733uuPVUhQrA=; b=qvGYwnbwEjP3d4vlYle5icsMHHAiiVODTipm0EPzsX/QkeTaNZcSzxfTWr9bpBJQ95 flW/bN8Ap0fwjiqFBWF2iDrL0LsndHlQablKJhnJdYzSXcmtbKRHOc1H/AGtYBnJmgc4 bfbt1W9iJj2jHPN9fVSj/yLYiI//hnYti7l4sE+2MvCeJj7y/eDzDk+OncMDHiMwihCo nlXOdznSKRX3sElA6DugJ9fMr0/42WVS9JOMBg/A4xLN9MoTqexNOnNo03S18R2013Dd hgBvjq6J0Fa+V0x+HO5SCqt0eE+jDGkCCGy26BQj8NDH4k72X/B3BIuuLPgUulibWddk uU9w== X-Gm-Message-State: AOAM533bmlHOAEZfkpxWQPq1hl+f4JUdXicx0QPcK9vYCMEOBDs9HQ9l EKunRJyXJPgT32CF2cQSoO2Ktd1nK/F3voT4OOBWQYRl0ZXKSgnrEIDsOXVucWScKFSvyqMo1qA jIEorKfPNGLBx155a9neT04sYiMw+xypbTrz5nr8DBJqbeuTtbDHa1o5Se2NyJSbaa2oE2Nd/KS J5KIALn6HrXH8= X-Google-Smtp-Source: ABdhPJxpFvl5xuUSdma+b4e047prDpdKGiN10tcR96rDIf+XIy+1+T+sOcqIv0MS2yf4znjxCB8Kvg== X-Received: by 2002:a62:d108:0:b029:163:d3cf:f00e with SMTP id z8-20020a62d1080000b0290163d3cff00emr22126069pfg.43.1605095483119; Wed, 11 Nov 2020 03:51:23 -0800 (PST) Received: from localhost.localdomain ([192.19.234.250]) by smtp.gmail.com with ESMTPSA id o1sm1160221pfk.67.2020.11.11.03.51.20 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Wed, 11 Nov 2020 03:51:22 -0800 (PST) From: Muneendra To: linux-scsi@vger.kernel.org, michael.christie@oracle.com, hare@suse.de Cc: jsmart2021@gmail.com, emilne@redhat.com, mkumar@redhat.com, Muneendra Subject: [PATCH v7 0/5] scsi: Support to handle Intermittent errors Date: Wed, 11 Nov 2020 10:28:00 +0530 Message-Id: <1605070685-20945-1-git-send-email-muneendra.kumar@broadcom.com> X-Mailer: git-send-email 1.8.3.1 Precedence: bulk List-ID: X-Mailing-List: linux-scsi@vger.kernel.org This patch adds a support to prevent retries of all the io's after an abort succeeds on a particular device when transport connectivity to the device is encountering intermittent errors. Intermittent connectivity is a condition that can be detected by transport fabric notifications. A service can monitor the ELS notifications and take action on all the outstanding io's of a scsi device at that instant. This feature is intended to be used when the device is part of a multipath environment. When the service detects the poor connectivity, the multipath path can be placed in a marginal path group and ignored further io operations. After placing a path in the marginal path group,the daemon sets the port_state to Marginal which sets bit in scmd->state for all the io's on that particular device with the new sysfs interface provided in this patch.This prevent retries of all the io's if an io hits a scsi timeout which inturn issues an abort. On Abort succeeds on a marginal path the io will be immediately retried on another active path.On abort fails then the things escalates to existing target reset sg interface recovery process. Below is the interface provided to set the port state to Marginal and Online. echo "Marginal" >> /sys/class/fc_remote_ports/rport-X\:Y-Z/port_state echo "Online" >> /sys/class/fc_remote_ports/rport-X\:Y-Z/port_state The patches were cut against 5.11/scsi-queue tree Reviewed-by: Hannes Reinecke Reviewed-by: Hannes Reinecke Reviewed-by: Hannes Reinecke Reviewed-by: Ewan D. Milne Reviewed-by: Ewan D. Milne Reviewed-by: Ewan D. Milne Reviewed-by: Himanshu Madhani Reviewed-by: Himanshu Madhani Reviewed-by: Himanshu Madhani --- v7: Added New routine in scsi_host_template to decide if a cmd is retryable instead of checking the same using SCMD_NORETRIES_ABORT bit as the cmd retry part can be checked by validating the port state. Removed the changes related to SCMD_NORETRIES_ABORT bit. Added a new function fc_eh_should_retry_cmd to check whether the cmd should be retried based on the rport state. Reoreder the patch The patches were cut against 5.11/scsi-queue tree v6: Reordered the patches to make patch ordering and more logical. v5: Added the DID_TRANSPORT_MARGINAL case to scsi_decide_disposition Made changes to clear the SCMD_NORETRIES_ABORT bit if the port_state has changed from marginal to online due to port_delete and port_add as we need the normal cmd retry behaviour while we are calling the eh handlers. Made changes in fc_scsi_scan_rport as we are checking FC_PORTSTATE_ONLINE instead of FC_PORTSTATE_ONLINE and FC_PORTSTATE_MARGINAL v4: Made changes in fc_eh_timed_out callout to set the SCMD_NORETRIES_ABORT if port state is marginal With this change, we removed the code to loop over running commands and fc_remote_port_chkready changes to set the SCMD_NORETRIES_ABORT Removed the scsi_cmd argument for fc_remote_port_chkready and reverted back the patches that addressed this change(argument) Removed unnecessary comments Handle the return of errors on failure. v3: Removed the port_state from starget attributes. Enabled the store functionality for port_state under remote port Added a new argument to scsi_cmd to fc_remote_port_chkready Used the existing scsi command iterators scsi_host_busy_iter. Rearranged the patches Added new patches to add new argument for fc_remote_port_chkready v2: Added new error code DID_TRANSPORT_MARGINAL to handle marginal errors. Added a new rport_state FC_PORTSTATE_MARGINAL and also added a new sysfs interface port_state to set the port_state to marginal. Added the support in lpfc to handle the marginal state. *** BLURB HERE *** Muneendra (5): scsi: Added a new error code DID_TRANSPORT_MARGINAL in scsi.h scsi: No retries on abort success scsi_transport_fc: Added a new rport state FC_PORTSTATE_MARGINAL scsi_transport_fc: Added store fucntionality to set the rport port_state using sysfs scsi:lpfc: Added support for eh_should_retry_cmd drivers/scsi/lpfc/lpfc_scsi.c | 1 + drivers/scsi/scsi_error.c | 23 +++++- drivers/scsi/scsi_lib.c | 1 + drivers/scsi/scsi_transport_fc.c | 118 ++++++++++++++++++++++++++----- include/scsi/scsi.h | 1 + include/scsi/scsi_host.h | 6 ++ include/scsi/scsi_transport_fc.h | 4 +- 7 files changed, 133 insertions(+), 21 deletions(-)