From: Bart Van Assche <bvanassche@acm.org>
To: "Martin K . Petersen" <martin.petersen@oracle.com>
Cc: linux-scsi@vger.kernel.org, Bart Van Assche <bvanassche@acm.org>
Subject: [PATCH v4 00/46] Register SCSI sysfs attributes earlier
Date: Tue, 12 Oct 2021 16:35:12 -0700 [thread overview]
Message-ID: <20211012233558.4066756-1-bvanassche@acm.org> (raw)
Hi Martin,
For certain user space software, e.g. udev, it is important that sysfs
attributes are registered before the KOBJ_ADD uevent is emitted. Hence
this patch series that removes the device_create_file() and
sysfs_create_groups() calls from the SCSI core. Please consider this
patch series for kernel v5.16.
Thanks,
Bart.
Changes compared to v3:
- Updated Acked-by / Reviewed-by tags.
- In the zfcp driver, restored the zfcp_sysfs_ prefix.
- Fixed a bug in the patch for the qla2xxx driver.
Changes compared to v2:
- Use the ATTRIBUTE_GROUPS() macro where appropriate.
Changes compared to v1:
- Switched from struct attribute ** to struct attribute_group **.
- Added comments that explain how the size of the new arrays have been chosen.
- Cleaned up the code in the qla2xxx driver that modifies a struct
device_attribute array by introducing an .is_visible() callback.
- Split this patch series into one patch per driver.
Bart Van Assche (46):
scsi: core: Register sysfs attributes earlier
ata: Switch to attribute groups
firewire: sbp2: Switch to attribute groups
RDMA/srp: Switch to attribute groups
scsi: message: fusion: Switch to attribute groups
scsi: zfcp: Switch to attribute groups
scsi: 3w-9xxx: Switch to attribute groups
scsi: 3w-sas: Switch to attribute groups
scsi: 3w-xxxx: Switch to attribute groups
scsi: 53c700: Switch to attribute groups
scsi: aacraid: Switch to attribute groups
scsi: arcmsr: Switch to attribute groups
scsi: be2iscsi: Switch to attribute groups
scsi: bfa: Switch to attribute groups
scsi: bnx2fc: Switch to attribute groups
scsi: bnx2i: Switch to attribute groups
scsi: csiostor: Switch to attribute groups
scsi: cxlflash: Switch to attribute groups
scsi: fnic: Switch to attribute groups
scsi: hisi_sas: Switch to attribute groups
scsi: hpsa: Switch to attribute groups
scsi: hptiop: Switch to attribute groups
scsi: ibmvscsi: Switch to attribute groups
scsi: ibmvfc: Switch to attribute groups
scsi: ipr: Switch to attribute groups
scsi: isci: Switch to attribute groups
scsi: lpfc: Switch to attribute groups
scsi: megaraid: Switch to attribute groups
scsi: mpt3sas: Switch to attribute groups
scsi: mvsas: Switch to attribute groups
scsi: myrb: Switch to attribute groups
scsi: myrs: Switch to attribute groups
scsi: ncr53c8xx: Switch to attribute groups
scsi: sym53c500_cs: Switch to attribute groups
scsi: pm8001: Switch to attribute groups
scsi: pmcraid: Switch to attribute groups
scsi: qedf: Switch to attribute groups
scsi: qedi: Switch to attribute groups
scsi: qla2xxx: Remove a declaration
scsi: qla2xxx: Switch to attribute groups
scsi: qla4xxx: Switch to attribute groups
scsi: smartpqi: Switch to attribute groups
scsi: snic: Switch to attribute groups
scsi: unisys: Remove the shost_attrs member
scsi: usb: Switch to attribute groups
scsi: core: Remove two host template members that are no longer used
drivers/ata/ahci.h | 8 +-
drivers/ata/ata_piix.c | 8 +-
drivers/ata/libahci.c | 52 ++-
drivers/ata/libata-sata.c | 19 +-
drivers/ata/libata-scsi.c | 15 +-
drivers/ata/pata_macio.c | 2 +-
drivers/ata/sata_mv.c | 2 +-
drivers/ata/sata_nv.c | 4 +-
drivers/ata/sata_sil24.c | 2 +-
drivers/firewire/sbp2.c | 8 +-
drivers/infiniband/ulp/srp/ib_srp.c | 51 +--
drivers/message/fusion/mptfc.c | 2 +-
drivers/message/fusion/mptsas.c | 2 +-
drivers/message/fusion/mptscsih.c | 36 +-
drivers/message/fusion/mptscsih.h | 2 +-
drivers/message/fusion/mptspi.c | 2 +-
drivers/s390/scsi/zfcp_ext.h | 4 +-
drivers/s390/scsi/zfcp_scsi.c | 4 +-
drivers/s390/scsi/zfcp_sysfs.c | 52 ++-
drivers/scsi/3w-9xxx.c | 8 +-
drivers/scsi/3w-sas.c | 8 +-
drivers/scsi/3w-xxxx.c | 8 +-
drivers/scsi/53c700.c | 12 +-
drivers/scsi/aacraid/linit.c | 38 ++-
drivers/scsi/arcmsr/arcmsr.h | 2 +-
drivers/scsi/arcmsr/arcmsr_attr.c | 33 +-
drivers/scsi/arcmsr/arcmsr_hba.c | 2 +-
drivers/scsi/be2iscsi/be_main.c | 21 +-
drivers/scsi/bfa/bfad_attr.c | 68 ++--
drivers/scsi/bfa/bfad_im.c | 4 +-
drivers/scsi/bfa/bfad_im.h | 4 +-
drivers/scsi/bnx2fc/bnx2fc_fcoe.c | 8 +-
drivers/scsi/bnx2i/bnx2i.h | 2 +-
drivers/scsi/bnx2i/bnx2i_iscsi.c | 2 +-
drivers/scsi/bnx2i/bnx2i_sysfs.c | 15 +-
drivers/scsi/csiostor/csio_scsi.c | 24 +-
drivers/scsi/cxlflash/main.c | 40 ++-
drivers/scsi/fnic/fnic.h | 2 +-
drivers/scsi/fnic/fnic_attrs.c | 17 +-
drivers/scsi/fnic/fnic_main.c | 2 +-
drivers/scsi/hisi_sas/hisi_sas_v1_hw.c | 8 +-
drivers/scsi/hisi_sas/hisi_sas_v2_hw.c | 8 +-
drivers/scsi/hisi_sas/hisi_sas_v3_hw.c | 14 +-
drivers/scsi/hosts.c | 14 +-
drivers/scsi/hpsa.c | 44 +--
drivers/scsi/hptiop.c | 10 +-
drivers/scsi/ibmvscsi/ibmvfc.c | 22 +-
drivers/scsi/ibmvscsi/ibmvscsi.c | 22 +-
drivers/scsi/ipr.c | 38 ++-
drivers/scsi/isci/init.c | 8 +-
drivers/scsi/lpfc/lpfc_attr.c | 314 +++++++++---------
drivers/scsi/lpfc/lpfc_crtn.h | 4 +-
drivers/scsi/lpfc/lpfc_init.c | 2 +-
drivers/scsi/lpfc/lpfc_scsi.c | 4 +-
drivers/scsi/megaraid/megaraid_mbox.c | 15 +-
drivers/scsi/megaraid/megaraid_sas_base.c | 24 +-
drivers/scsi/mpt3sas/mpt3sas_base.h | 4 +-
drivers/scsi/mpt3sas/mpt3sas_ctl.c | 84 +++--
drivers/scsi/mpt3sas/mpt3sas_scsih.c | 8 +-
drivers/scsi/mvsas/mv_init.c | 12 +-
drivers/scsi/myrb.c | 28 +-
drivers/scsi/myrs.c | 40 ++-
drivers/scsi/ncr53c8xx.c | 10 +-
drivers/scsi/pcmcia/sym53c500_cs.c | 8 +-
drivers/scsi/pm8001/pm8001_ctl.c | 64 ++--
drivers/scsi/pm8001/pm8001_init.c | 2 +-
drivers/scsi/pm8001/pm8001_sas.h | 2 +-
drivers/scsi/pmcraid.c | 11 +-
drivers/scsi/qedf/qedf.h | 2 +-
drivers/scsi/qedf/qedf_attr.c | 15 +-
drivers/scsi/qedf/qedf_main.c | 2 +-
drivers/scsi/qedi/qedi_gbl.h | 2 +-
drivers/scsi/qedi/qedi_iscsi.c | 2 +-
drivers/scsi/qedi/qedi_sysfs.c | 15 +-
drivers/scsi/qla2xxx/qla_attr.c | 125 +++----
drivers/scsi/qla2xxx/qla_gbl.h | 4 +-
drivers/scsi/qla2xxx/qla_os.c | 5 +-
drivers/scsi/qla4xxx/ql4_attr.c | 41 ++-
drivers/scsi/qla4xxx/ql4_glbl.h | 3 +-
drivers/scsi/qla4xxx/ql4_os.c | 2 +-
drivers/scsi/scsi_priv.h | 2 +-
drivers/scsi/scsi_sysfs.c | 53 +--
drivers/scsi/smartpqi/smartpqi_init.c | 46 +--
drivers/scsi/snic/snic.h | 2 +-
drivers/scsi/snic/snic_attrs.c | 19 +-
drivers/scsi/snic/snic_main.c | 2 +-
.../staging/unisys/visorhba/visorhba_main.c | 1 -
drivers/usb/storage/scsiglue.c | 8 +-
include/linux/libata.h | 8 +-
include/scsi/scsi_device.h | 6 +
include/scsi/scsi_host.h | 15 +-
91 files changed, 1029 insertions(+), 770 deletions(-)
next reply other threads:[~2021-10-12 23:36 UTC|newest]
Thread overview: 67+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-10-12 23:35 Bart Van Assche [this message]
2021-10-12 23:35 ` [PATCH v4 01/46] scsi: core: Register sysfs attributes earlier Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 02/46] ata: Switch to attribute groups Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 03/46] firewire: sbp2: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 04/46] RDMA/srp: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 05/46] scsi: message: fusion: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 06/46] scsi: zfcp: " Bart Van Assche
2021-10-13 17:34 ` Benjamin Block
2021-10-12 23:35 ` [PATCH v4 07/46] scsi: 3w-9xxx: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 08/46] scsi: 3w-sas: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 09/46] scsi: 3w-xxxx: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 10/46] scsi: 53c700: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 11/46] scsi: aacraid: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 12/46] scsi: arcmsr: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 13/46] scsi: be2iscsi: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 14/46] scsi: bfa: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 15/46] scsi: bnx2fc: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 16/46] scsi: bnx2i: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 17/46] scsi: csiostor: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 18/46] scsi: cxlflash: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 19/46] scsi: fnic: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 20/46] scsi: hisi_sas: " Bart Van Assche
2021-10-13 8:22 ` John Garry
2021-10-12 23:35 ` [PATCH v4 21/46] scsi: hpsa: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 22/46] scsi: hptiop: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 23/46] scsi: ibmvscsi: " Bart Van Assche
2021-10-14 18:30 ` Tyrel Datwyler
2021-10-12 23:35 ` [PATCH v4 24/46] scsi: ibmvfc: " Bart Van Assche
2021-10-14 18:31 ` Tyrel Datwyler
2021-10-12 23:35 ` [PATCH v4 25/46] scsi: ipr: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 26/46] scsi: isci: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 27/46] scsi: lpfc: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 28/46] scsi: megaraid: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 29/46] scsi: mpt3sas: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 30/46] scsi: mvsas: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 31/46] scsi: myrb: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 32/46] scsi: myrs: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 33/46] scsi: ncr53c8xx: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 34/46] scsi: sym53c500_cs: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 35/46] scsi: pm8001: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 36/46] scsi: pmcraid: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 37/46] scsi: qedf: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 38/46] scsi: qedi: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 39/46] scsi: qla2xxx: Remove a declaration Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 40/46] scsi: qla2xxx: Switch to attribute groups Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 41/46] scsi: qla4xxx: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 42/46] scsi: smartpqi: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 43/46] scsi: snic: " Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 44/46] scsi: unisys: Remove the shost_attrs member Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 45/46] scsi: usb: Switch to attribute groups Bart Van Assche
2021-10-12 23:35 ` [PATCH v4 46/46] scsi: core: Remove two host template members that are no longer used Bart Van Assche
2021-10-17 1:48 ` [PATCH v4 00/46] Register SCSI sysfs attributes earlier Martin K. Petersen
2021-10-21 3:42 ` Martin K. Petersen
2021-10-23 20:54 ` Missing driver-specific sysfs attributes of scsi_device [was: Re: [PATCH v4 00/46] Register SCSI sysfs attributes earlier] Steffen Maier
2021-10-24 0:39 ` Steffen Maier
2021-10-24 2:55 ` Bart Van Assche
2021-10-24 11:18 ` [PATCH] scsi: core: Fix early registration of sysfs attributes for scsi_device Steffen Maier
2021-10-24 21:25 ` Bart Van Assche
2021-10-24 22:16 ` [PATCH v2] " Steffen Maier
2021-10-25 9:23 ` Julian Wiedmann
2021-10-25 16:00 ` Bart Van Assche
2021-10-26 1:42 ` [PATCH v3] " Steffen Maier
2021-10-26 10:00 ` Benjamin Block
2021-10-26 21:48 ` Bart Van Assche
2021-10-29 17:55 ` Benjamin Block
2021-10-26 18:31 ` Bart Van Assche
2021-10-29 17:56 ` Benjamin Block
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=20211012233558.4066756-1-bvanassche@acm.org \
--to=bvanassche@acm.org \
--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