From patchwork Mon Oct 16 09:24:13 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Hannes Reinecke X-Patchwork-Id: 734948 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id AB539CDB482 for ; Mon, 16 Oct 2023 09:24:52 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232911AbjJPJYw (ORCPT ); Mon, 16 Oct 2023 05:24:52 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46998 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232580AbjJPJYt (ORCPT ); Mon, 16 Oct 2023 05:24:49 -0400 Received: from smtp-out1.suse.de (smtp-out1.suse.de [195.135.220.28]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 3E273EA for ; Mon, 16 Oct 2023 02:24:45 -0700 (PDT) Received: from relay2.suse.de (relay2.suse.de [149.44.160.134]) by smtp-out1.suse.de (Postfix) with ESMTP id 482F121C44; Mon, 16 Oct 2023 09:24:44 +0000 (UTC) Received: from adalid.arch.suse.de (adalid.arch.suse.de [10.161.8.13]) by relay2.suse.de (Postfix) with ESMTP id C599B2CB39; Mon, 16 Oct 2023 09:24:43 +0000 (UTC) Received: by adalid.arch.suse.de (Postfix, from userid 16045) id BF4AE51EBDC0; Mon, 16 Oct 2023 11:24:43 +0200 (CEST) From: Hannes Reinecke To: Christoph Hellwig Cc: "Martin K. Petersen" , James Bottomley , linux-scsi@vger.kernel.org, Hannes Reinecke Subject: [PATCHv6 00/17] scsi: EH rework prep patches, part 2 Date: Mon, 16 Oct 2023 11:24:13 +0200 Message-Id: <20231016092430.55557-1-hare@suse.de> X-Mailer: git-send-email 2.35.3 MIME-Version: 1.0 X-Spamd-Bar: ++++++++ Authentication-Results: smtp-out1.suse.de; dkim=none; dmarc=none; spf=softfail (smtp-out1.suse.de: 149.44.160.134 is neither permitted nor denied by domain of hare@suse.de) smtp.mailfrom=hare@suse.de X-Rspamd-Server: rspamd2 X-Spamd-Result: default: False [8.49 / 50.00]; ARC_NA(0.00)[]; FROM_HAS_DN(0.00)[]; TO_DN_SOME(0.00)[]; R_MISSING_CHARSET(2.50)[]; MIME_GOOD(-0.10)[text/plain]; DMARC_NA(0.20)[suse.de]; BROKEN_CONTENT_TYPE(1.50)[]; R_SPF_SOFTFAIL(0.60)[~all:c]; RCPT_COUNT_FIVE(0.00)[5]; TO_MATCH_ENVRCPT_SOME(0.00)[]; VIOLATED_DIRECT_SPF(3.50)[]; MX_GOOD(-0.01)[]; NEURAL_SPAM_LONG(3.00)[1.000]; MID_CONTAINS_FROM(1.00)[]; RWL_MAILSPIKE_GOOD(0.00)[149.44.160.134:from]; NEURAL_HAM_SHORT(-1.00)[-1.000]; RCVD_NO_TLS_LAST(0.10)[]; FROM_EQ_ENVFROM(0.00)[]; R_DKIM_NA(0.20)[]; MIME_TRACE(0.00)[0:+]; RCVD_COUNT_TWO(0.00)[2]; BAYES_HAM(-3.00)[100.00%] X-Rspamd-Queue-Id: 482F121C44 Precedence: bulk List-ID: X-Mailing-List: linux-scsi@vger.kernel.org Hi all, (taking up an old thread:) here's the second batch of patches for my EH rework. It modifies the reset callbacks for SCSI drivers such that the final conversion to drop the 'struct scsi_cmnd' argument and use the entity in question (host, bus, target, device) as the argument to the SCSI EH callbacks becomes possible. The second part covers drivers which require a bit more love. In particular the fnic, snic, and csiostor drivers require a tag to send TMFs. So to handle that I've set aside a tag and used that directly (for snic host reset), or call scsi_alloc_request() with the NOWAIT flag. That will return a scsi command with a valid tag, which then can be used to send the reset. It might fail (eg when the tagset is full), but in these cases it might be better to fall back to host_reset anyway. As usual, comments and reviews are welcome. Changes to v5: - Modified zfcp host reset to return FAST_IO_FAIL - Rebased to 6.7/staging - Add fix for missing scsi_device_put() in pmcraid - Modify doing_srb_done() in dc395x to not use a scsi command Changes to v4: - rework snic to use a dedicated tag for host reset - rework snic to allocate TMFs on the fly - rework fnic to allocate TMFs on the fly - rework csiostor to allocat TMFs on the fly - drop fc_block_rport() from zfcp host_reset - Rebase to latest linus tree Changes to v3: - Move fnic and snic patches to the next patchset - Include reviews from Ewan Milne Changes to v2: - Include reviews from John Garry - move mpi3mr, zfcp, sym53c8xx_2, and qla1280 patches to the next patchset Changes to the initial version: - Include reviews from Christoph - Fixup build robot issues Hannes Reinecke (17): pmcraid: add missing scsi_device_put() in pmcraid_eh_target_reset_handler() zfcp: do not wait for rports to become unblocked after host reset bfa: Do not use scsi command to signal TMF status aha152x: look for stuck command when resetting device a1000u2w: do not rely on the command for inia100_device_reset() fas216: Rework device reset to not rely on SCSI command pointer xen-scsifront: add scsi device as argument to scsifront_do_request() xen-scsifront: rework scsifront_action_handler() libiscsi: use cls_session as argument for target and session reset scsi_transport_iscsi: use session as argument for iscsi_block_scsi_eh() snic: reserve tag for TMF snic: allocate device reset command snic: Use scsi_host_busy_iter() to traverse commands fnic: allocate device reset command on the fly fnic: use fc_block_rport() correctly csiostor: use separate TMF command dc395x: Remove 'scmd' parameter from doing_srb_done() drivers/s390/scsi/zfcp_scsi.c | 6 +- drivers/scsi/a100u2w.c | 43 +--- drivers/scsi/aha152x.c | 26 ++- drivers/scsi/arm/fas216.c | 39 ++-- drivers/scsi/be2iscsi/be_main.c | 10 +- drivers/scsi/bfa/bfad_im.c | 112 +++++----- drivers/scsi/bfa/bfad_im.h | 2 + drivers/scsi/csiostor/csio_scsi.c | 72 ++++--- drivers/scsi/dc395x.c | 14 +- drivers/scsi/fnic/fnic.h | 1 - drivers/scsi/fnic/fnic_scsi.c | 115 +++++----- drivers/scsi/libiscsi.c | 21 +- drivers/scsi/pmcraid.c | 15 +- drivers/scsi/qla4xxx/ql4_os.c | 34 +-- drivers/scsi/scsi_transport_iscsi.c | 6 +- drivers/scsi/snic/snic.h | 2 +- drivers/scsi/snic/snic_main.c | 5 +- drivers/scsi/snic/snic_scsi.c | 319 ++++++++++++---------------- drivers/scsi/xen-scsifront.c | 49 +++-- include/scsi/libiscsi.h | 2 +- include/scsi/scsi_transport_iscsi.h | 2 +- 21 files changed, 429 insertions(+), 466 deletions(-)