From: Bart Van Assche <bvanassche@acm.org>
To: "Martin K . Petersen" <martin.petersen@oracle.com>
Cc: linux-scsi@vger.kernel.org, Marco Elver <elver@google.com>,
Bart Van Assche <bvanassche@acm.org>
Subject: [PATCH v2 00/56] Enable lock context analysis for the SCSI subsystem
Date: Thu, 30 Apr 2026 11:19:30 -0700 [thread overview]
Message-ID: <20260430182130.1978347-1-bvanassche@acm.org> (raw)
Hi Martin,
This patch series enables lock context analysis for all SCSI drivers and also
for the SCSI core. The advantages are as follows:
- The compiler (only Clang) verifies whether the lock and unlock calls match
what has been declared via __must_hold(), __acquires() or __releases().
This is useful for catching locking bugs in error paths.
- Support for __guarded_by() is enabled. If a member variable is annotated
with __guarded_by(lock), the compiler will issue a warning if that member
variable is accessed without holding 'lock'.
Additionally, a patch is included that suppresses KCSAN complaints about SCSI
host state changes.
More information about lock context analysis is available in the cover letter of
[PATCH v5 00/36] Compiler-Based Context- and Locking-Analysis
(https://lore.kernel.org/lkml/20251219154418.3592607-1-elver@google.com/).
Please consider this patch series for the next merge window.
Thanks,
Bart.
Changes compared to v1:
- Split patch "Enable lock context analysis" into one patch per driver.
Bart Van Assche (56):
PCI: Convert to_pci_dev() into an inline function
scsi: scsi_debug: Prepare for enabling lock context analysis
scsi: sg: Prepare for enabling lock context analysis
scsi: st: Prepare for enabling lock context analysis
scsi: BusLogic: Introduce a local variable
scsi: BusLogic: Prepare for enabling lock context analysis
scsi: NCR5380: Prepare for enabling lock context analysis
scsi: aacraid: Prepare for enabling lock context analysis
scsi: aic7xxx: Enable lock context analysis
scsi: aha152x: Prepare for enabling lock context analysis
scsi: aic7xxx: Prepare for enabling lock context analysis
scsi: aic94xx: Enable lock context analysis
scsi: arcmsr: Enable lock context analysis
scsi: arm: Enable lock context analysis
scsi: be2iscsi: Prepare for enabling lock context analysis
scsi: be2iscsi: Enable lock context analysis
scsi: cxgbi: Enable lock context analysis
scsi: bfa: Enable lock context analysis
scsi: bnx2fc: Enable lock context analysis
scsi: bnx2i: Introduce a local variable
scsi: bnx2i: Enable lock context analysis
scsi: csiostor: Enable lock context analysis
scsi: elx: Enable lock context analysis
scsi: esas2r: Enable lock context analysis
scsi: fcoe: Enable lock context analysis
scsi: fnic: Enable lock context analysis
scsi: hisi_sas: Enable lock context analysis
scsi: hpsa: Prepare for enabling lock context analysis
scsi: ibmvscsi: Enable lock context analysis
scsi: ibmvscsi_tgt: Enable lock context analysis
scsi: ipr: Prepare for enabling lock context analysis
scsi: ips: Prepare for enabling lock context analysis
scsi: isci: Enable lock context analysis
scsi: libfc: Enable lock context analysis
scsi: libiscsi: Prepare for enabling lock context analysis
scsi: libsas: Prepare for enabling lock context analysis
scsi: libsas: Enable lock context analysis
scsi: lpfc: Prepare for enabling lock context analysis
scsi: megaraid_sas: Prepare for enabling lock context analysis
scsi: megaraid: Enable lock context analysis
scsi: mpt3sas: Enable lock context analysis
scsi: mvsas: Enable lock context analysis
scsi: pcmcia: Enable lock context analysis
scsi: pm8001: Enable lock context analysis
scsi: qedf: Enable lock context analysis
scsi: qedi: Enable lock context analysis
scsi: qla1280: Prepare for enabling lock context analysis
scsi: qla2xxx: Enable lock context analysis
scsi: qla4xxx: Enable lock context analysis
scsi: ufs: Enable lock context analysis
scsi: iSCSI transport: Prepare for enabling lock context analysis
scsi: smartpqi: Enable lock context analysis
scsi: snic: Enable lock context analysis
scsi: sym53c8xx_2: Enable lock context analysis
scsi: core: Enable lock context analysis
scsi: core: Protect host state changes with the host lock
drivers/scsi/BusLogic.c | 11 +++--
drivers/scsi/Makefile | 1 +
drivers/scsi/NCR5380.c | 4 +-
drivers/scsi/aacraid/Makefile | 2 +
drivers/scsi/aacraid/commctrl.c | 1 +
drivers/scsi/aacraid/commsup.c | 3 ++
drivers/scsi/aha152x.c | 1 +
drivers/scsi/aic7xxx/Makefile | 2 +
drivers/scsi/aic7xxx/aic79xx_osm.h | 2 +
drivers/scsi/aic7xxx/aic7xxx_osm.h | 2 +
drivers/scsi/aic7xxx/aicasm/Makefile | 3 ++
drivers/scsi/aic94xx/Makefile | 2 +
drivers/scsi/arcmsr/Makefile | 2 +
drivers/scsi/arm/Makefile | 2 +
drivers/scsi/be2iscsi/Makefile | 2 +
drivers/scsi/be2iscsi/be_main.c | 4 ++
drivers/scsi/bfa/Makefile | 3 ++
drivers/scsi/bnx2fc/Makefile | 3 ++
drivers/scsi/bnx2fc/bnx2fc.h | 5 +-
drivers/scsi/bnx2fc/bnx2fc_els.c | 2 +
drivers/scsi/bnx2fc/bnx2fc_hwi.c | 3 ++
drivers/scsi/bnx2fc/bnx2fc_io.c | 6 ++-
drivers/scsi/bnx2i/Makefile | 3 ++
drivers/scsi/bnx2i/bnx2i_hwi.c | 14 ++++--
drivers/scsi/bnx2i/bnx2i_iscsi.c | 1 +
drivers/scsi/csiostor/Makefile | 2 +
drivers/scsi/csiostor/csio_hw.c | 12 +++++
drivers/scsi/csiostor/csio_lnode.c | 3 ++
drivers/scsi/csiostor/csio_rnode.c | 6 +++
drivers/scsi/csiostor/csio_scsi.c | 6 +++
drivers/scsi/cxgbi/Makefile | 3 ++
drivers/scsi/device_handler/Makefile | 3 ++
drivers/scsi/elx/Makefile | 1 +
drivers/scsi/esas2r/Makefile | 3 ++
drivers/scsi/fcoe/Makefile | 3 ++
drivers/scsi/fnic/Makefile | 3 ++
drivers/scsi/fnic/fdls_disc.c | 52 ++++++++++++++++++++-
drivers/scsi/fnic/fip.c | 2 +
drivers/scsi/fnic/fnic_fcs.c | 6 +++
drivers/scsi/fnic/fnic_scsi.c | 4 ++
drivers/scsi/hisi_sas/Makefile | 3 ++
drivers/scsi/hosts.c | 13 ++++--
drivers/scsi/hpsa.c | 2 +-
drivers/scsi/ibmvscsi/Makefile | 3 ++
drivers/scsi/ibmvscsi_tgt/Makefile | 3 ++
drivers/scsi/ibmvscsi_tgt/ibmvscsi_tgt.c | 7 +++
drivers/scsi/ipr.c | 11 +++++
drivers/scsi/ips.c | 1 +
drivers/scsi/isci/Makefile | 3 ++
drivers/scsi/libfc/Makefile | 2 +
drivers/scsi/libfc/fc_disc.c | 6 ++-
drivers/scsi/libfc/fc_exch.c | 6 +++
drivers/scsi/libfc/fc_fcp.c | 4 ++
drivers/scsi/libiscsi.c | 19 +++++++-
drivers/scsi/libsas/Makefile | 2 +
drivers/scsi/libsas/sas_ata.c | 2 +-
drivers/scsi/lpfc/lpfc_els.c | 2 +
drivers/scsi/lpfc/lpfc_nportdisc.c | 1 +
drivers/scsi/lpfc/lpfc_scsi.c | 1 +
drivers/scsi/lpfc/lpfc_sli.c | 2 +
drivers/scsi/megaraid/Makefile | 3 ++
drivers/scsi/megaraid/megaraid_sas.h | 9 ++--
drivers/scsi/megaraid/megaraid_sas_base.c | 17 +++++--
drivers/scsi/megaraid/megaraid_sas_fusion.c | 2 +
drivers/scsi/mpt3sas/Makefile | 3 ++
drivers/scsi/mpt3sas/mpt3sas_scsih.c | 2 +-
drivers/scsi/mvsas/Makefile | 2 +
drivers/scsi/mvsas/mv_sas.c | 7 +++
drivers/scsi/pcmcia/Makefile | 2 +
drivers/scsi/pm8001/Makefile | 1 +
drivers/scsi/pm8001/pm80xx_hwi.c | 2 +
drivers/scsi/qedf/Makefile | 3 ++
drivers/scsi/qedi/Makefile | 3 ++
drivers/scsi/qedi/qedi_fw.c | 1 +
drivers/scsi/qla1280.c | 18 +++++++
drivers/scsi/qla2xxx/Makefile | 3 ++
drivers/scsi/qla2xxx/qla_nx.c | 2 +
drivers/scsi/qla2xxx/qla_target.c | 29 +++++++-----
drivers/scsi/qla2xxx/qla_tmpl.c | 1 +
drivers/scsi/qla4xxx/Makefile | 3 ++
drivers/scsi/qla4xxx/ql4_nx.c | 2 +
drivers/scsi/qla4xxx/ql4_os.c | 6 +--
drivers/scsi/scsi_debug.c | 16 +++++++
drivers/scsi/scsi_lib.c | 3 +-
drivers/scsi/scsi_scan.c | 12 +++++
drivers/scsi/scsi_sysfs.c | 7 +--
drivers/scsi/scsi_transport_iscsi.c | 1 +
drivers/scsi/sg.c | 1 +
drivers/scsi/smartpqi/Makefile | 3 ++
drivers/scsi/smartpqi/smartpqi_init.c | 33 +++++++++++++
drivers/scsi/snic/Makefile | 3 ++
drivers/scsi/st.c | 1 +
drivers/scsi/sym53c8xx_2/Makefile | 2 +
drivers/ufs/core/Makefile | 2 +
drivers/ufs/core/ufs-debugfs.c | 8 +++-
drivers/ufs/core/ufshcd.c | 14 ++++++
drivers/ufs/host/Makefile | 2 +
include/linux/pci.h | 5 +-
include/scsi/libiscsi.h | 5 +-
include/scsi/scsi_host.h | 25 +++++++---
100 files changed, 487 insertions(+), 63 deletions(-)
next reply other threads:[~2026-04-30 18:21 UTC|newest]
Thread overview: 62+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-04-30 18:19 Bart Van Assche [this message]
2026-04-30 18:19 ` [PATCH v2 01/56] PCI: Convert to_pci_dev() into an inline function Bart Van Assche
2026-04-30 22:19 ` Bjorn Helgaas
2026-04-30 18:19 ` [PATCH v2 02/56] scsi: scsi_debug: Prepare for enabling lock context analysis Bart Van Assche
2026-04-30 18:19 ` [PATCH v2 03/56] scsi: sg: " Bart Van Assche
2026-04-30 18:19 ` [PATCH v2 04/56] scsi: st: " Bart Van Assche
2026-04-30 18:19 ` [PATCH v2 05/56] scsi: BusLogic: Introduce a local variable Bart Van Assche
2026-05-01 18:01 ` Khalid Aziz
2026-04-30 18:19 ` [PATCH v2 06/56] scsi: BusLogic: Prepare for enabling lock context analysis Bart Van Assche
2026-05-01 18:02 ` Khalid Aziz
2026-04-30 18:19 ` [PATCH v2 07/56] scsi: NCR5380: " Bart Van Assche
2026-04-30 18:19 ` [PATCH v2 08/56] scsi: aacraid: " Bart Van Assche
2026-04-30 18:19 ` [PATCH v2 09/56] scsi: aic7xxx: Enable " Bart Van Assche
2026-04-30 18:19 ` [PATCH v2 10/56] scsi: aha152x: Prepare for enabling " Bart Van Assche
2026-04-30 18:19 ` [PATCH v2 11/56] scsi: aic7xxx: " Bart Van Assche
2026-04-30 18:19 ` [PATCH v2 12/56] scsi: aic94xx: Enable " Bart Van Assche
2026-04-30 18:19 ` [PATCH v2 13/56] scsi: arcmsr: " Bart Van Assche
2026-04-30 18:19 ` [PATCH v2 14/56] scsi: arm: " Bart Van Assche
2026-04-30 18:19 ` [PATCH v2 15/56] scsi: be2iscsi: Prepare for enabling " Bart Van Assche
2026-04-30 18:19 ` [PATCH v2 16/56] scsi: be2iscsi: Enable " Bart Van Assche
2026-04-30 18:19 ` [PATCH v2 17/56] scsi: cxgbi: " Bart Van Assche
2026-04-30 18:19 ` [PATCH v2 18/56] scsi: bfa: " Bart Van Assche
2026-04-30 18:19 ` [PATCH v2 19/56] scsi: bnx2fc: " Bart Van Assche
2026-04-30 18:19 ` [PATCH v2 20/56] scsi: bnx2i: Introduce a local variable Bart Van Assche
2026-04-30 18:19 ` [PATCH v2 21/56] scsi: bnx2i: Enable lock context analysis Bart Van Assche
2026-04-30 18:19 ` [PATCH v2 22/56] scsi: csiostor: " Bart Van Assche
2026-04-30 18:19 ` [PATCH v2 23/56] scsi: elx: " Bart Van Assche
2026-04-30 18:19 ` [PATCH v2 24/56] scsi: esas2r: " Bart Van Assche
2026-04-30 18:19 ` [PATCH v2 25/56] scsi: fcoe: " Bart Van Assche
2026-04-30 18:19 ` [PATCH v2 26/56] scsi: fnic: " Bart Van Assche
2026-05-04 17:45 ` Karan Tilak Kumar (kartilak)
2026-04-30 18:19 ` [PATCH v2 27/56] scsi: hisi_sas: " Bart Van Assche
2026-04-30 18:19 ` [PATCH v2 28/56] scsi: hpsa: Prepare for enabling " Bart Van Assche
2026-04-30 18:19 ` [PATCH v2 29/56] scsi: ibmvscsi: Enable " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 30/56] scsi: ibmvscsi_tgt: " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 31/56] scsi: ipr: Prepare for enabling " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 32/56] scsi: ips: " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 33/56] scsi: isci: Enable " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 34/56] scsi: libfc: " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 35/56] scsi: libiscsi: Prepare for enabling " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 36/56] scsi: libsas: " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 37/56] scsi: libsas: Enable " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 38/56] scsi: lpfc: Prepare for enabling " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 39/56] scsi: megaraid_sas: " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 40/56] scsi: megaraid: Enable " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 41/56] scsi: mpt3sas: " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 42/56] scsi: mvsas: " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 43/56] scsi: pcmcia: " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 44/56] scsi: pm8001: " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 45/56] scsi: qedf: " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 46/56] scsi: qedi: " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 47/56] scsi: qla1280: Prepare for enabling " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 48/56] scsi: qla2xxx: Enable " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 49/56] scsi: qla4xxx: " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 50/56] scsi: ufs: " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 51/56] scsi: iSCSI transport: Prepare for enabling " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 52/56] scsi: smartpqi: Enable " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 53/56] scsi: snic: " Bart Van Assche
2026-05-01 19:08 ` Narsimhulu Musini (nmusini)
2026-04-30 18:20 ` [PATCH v2 54/56] scsi: sym53c8xx_2: " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 55/56] scsi: core: " Bart Van Assche
2026-04-30 18:20 ` [PATCH v2 56/56] scsi: core: Protect host state changes with the host lock Bart Van Assche
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260430182130.1978347-1-bvanassche@acm.org \
--to=bvanassche@acm.org \
--cc=elver@google.com \
--cc=linux-scsi@vger.kernel.org \
--cc=martin.petersen@oracle.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox