From patchwork Tue Oct 17 10:07: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: 734933 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 8ED1ECDB483 for ; Tue, 17 Oct 2023 10:07:46 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234925AbjJQKHp (ORCPT ); Tue, 17 Oct 2023 06:07:45 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:45954 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234787AbjJQKHo (ORCPT ); Tue, 17 Oct 2023 06:07:44 -0400 Received: from smtp-out1.suse.de (smtp-out1.suse.de [195.135.220.28]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D6B448E for ; Tue, 17 Oct 2023 03:07:41 -0700 (PDT) Received: from relay2.suse.de (relay2.suse.de [149.44.160.134]) by smtp-out1.suse.de (Postfix) with ESMTP id 1271721C54; Tue, 17 Oct 2023 10:07:40 +0000 (UTC) Received: from adalid.arch.suse.de (adalid.arch.suse.de [10.161.8.13]) by relay2.suse.de (Postfix) with ESMTP id CC05F2C4DD; Tue, 17 Oct 2023 10:07:39 +0000 (UTC) Received: by adalid.arch.suse.de (Postfix, from userid 16045) id EE6B551EBE83; Tue, 17 Oct 2023 12:07:39 +0200 (CEST) From: Hannes Reinecke To: "Martin K. Petersen" Cc: Christoph Hellwig , James Bottomley , linux-scsi@vger.kernel.org, Hannes Reinecke Subject: [PATCHv7 00/16] scsi: EH rework prep patches, part 2 Date: Tue, 17 Oct 2023 12:07:13 +0200 Message-Id: <20231017100729.123506-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 [2.49 / 50.00]; ARC_NA(0.00)[]; FROM_HAS_DN(0.00)[]; TO_DN_SOME(0.00)[]; R_MISSING_CHARSET(2.50)[]; NEURAL_HAM_LONG(-3.00)[-1.000]; 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_HAM_SHORT(-1.00)[-1.000]; MID_CONTAINS_FROM(1.00)[]; RWL_MAILSPIKE_GOOD(0.00)[149.44.160.134:from]; 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: 1271721C54 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 v6: - Include reviews from Christoph - Add missing kerneldoc updates 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 (16): 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 | 46 +--- 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 | 120 +++++----- drivers/scsi/libiscsi.c | 25 +-- 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 | 328 ++++++++++++---------------- drivers/scsi/xen-scsifront.c | 50 +++-- include/scsi/libiscsi.h | 2 +- include/scsi/scsi_transport_iscsi.h | 2 +- 20 files changed, 441 insertions(+), 461 deletions(-)