| Message ID | 20241202-refactor-blk-affinity-helpers-v6-0-27211e9c2cd5@kernel.org |
|---|---|
| Headers | show |
| Series | blk: refactor queue affinity helpers | expand |
Hi Jens, On Mon, Dec 02, 2024 at 03:00:08PM +0100, Daniel Wagner wrote: > I've rebased and retested the series on top of for-6.14/block and updated > the docummentation as requested by John. > > Original cover letter: > > These patches were part of 'honor isolcpus configuration' [1] series. To > simplify the review process I decided to send this as separate series > because I think it's a nice cleanup independent of the isolcpus feature. > > Signed-off-by: Daniel Wagner <wagi@kernel.org> I've based this work on top of your tree as I think it should go your tree. If this should go via different tree, let me know which one. And in case the series didn't hit your inbox, I really don't know what I am doing wrong. I've switched over to use b4 and korg for sending the patches and on my side all looks good and the mails also appear on lore completely normal. Thanks, Daniel
Hi Jens, On Mon, Dec 09, 2024 at 10:42:26AM +0100, Daniel Wagner wrote: > On Mon, Dec 02, 2024 at 03:00:08PM +0100, Daniel Wagner wrote: > > I've rebased and retested the series on top of for-6.14/block and updated > > the docummentation as requested by John. > > > > Original cover letter: > > > > These patches were part of 'honor isolcpus configuration' [1] series. To > > simplify the review process I decided to send this as separate series > > because I think it's a nice cleanup independent of the isolcpus feature. > > > > Signed-off-by: Daniel Wagner <wagi@kernel.org> > > I've based this work on top of your tree as I think it should go your > tree. If this should go via different tree, let me know which one. > > And in case the series didn't hit your inbox, I really don't know what I > am doing wrong. I've switched over to use b4 and korg for sending the > patches and on my side all looks good and the mails also appear on lore > completely normal. Any chance to get this merged? Thanks, Daniel
On Mon, 02 Dec 2024 15:00:08 +0100, Daniel Wagner wrote: > I've rebased and retested the series on top of for-6.14/block and updated > the docummentation as requested by John. > > Original cover letter: > > These patches were part of 'honor isolcpus configuration' [1] series. To > simplify the review process I decided to send this as separate series > because I think it's a nice cleanup independent of the isolcpus feature. > > [...] Applied, thanks! [1/8] driver core: bus: add irq_get_affinity callback to bus_type commit: ccddd556cfba3db6cb4d45d9f7d30c075e34808b [2/8] PCI: hookup irq_get_affinity callback commit: 86ae920136e7c6352e7239a459544b0eae57613a [3/8] virtio: hookup irq_get_affinity callback commit: df0e932e866b5cfad8cedafa0123ab0072a665ce [4/8] blk-mq: introduce blk_mq_map_hw_queues commit: 2b30fab613e2ef54ae4b3fbb238245a29ba2a285 [5/8] scsi: replace blk_mq_pci_map_queues with blk_mq_map_hw_queues commit: 7c72e20573d0d350c7c087289e7b7b605100651f [6/8] nvme: replace blk_mq_pci_map_queues with blk_mq_map_hw_queues commit: f0d672680e0b383b14add766b57db60c4c4acf92 [7/8] virtio: blk/scsi: replace blk_mq_virtio_map_queues with blk_mq_map_hw_queues commit: 5a1c50296039ca32a22015bbb9696d6a96514947 [8/8] blk-mq: remove unused queue mapping helpers commit: 737371e839a368007758be329413b3f5ec9e7976 Best regards,
I've rebased and retested the series on top of for-6.14/block and updated the docummentation as requested by John. Original cover letter: These patches were part of 'honor isolcpus configuration' [1] series. To simplify the review process I decided to send this as separate series because I think it's a nice cleanup independent of the isolcpus feature. Signed-off-by: Daniel Wagner <wagi@kernel.org> --- Changes in v6: - collected tags - Updated documented - rebased to for-6.14/block - Link to v5: https://lore.kernel.org/r/20241115-refactor-blk-affinity-helpers-v5-0-c472afd84d9f@kernel.org Changes in v5: - dropped the irq_get_affinity callback from struct device_driver again. - s/blk_mq_hctx_map_queues/blk_mq_map_hw_queues/ - collected tags - Link to v4: https://lore.kernel.org/r/20241113-refactor-blk-affinity-helpers-v4-0-dd3baa1e267f@kernel.org Changes in v4: - added irq_get_affinity callback to struct device_driver - hisi_sas use dev pointer directly from hisi_hba. - blk_mq_hctx_map_queues: ty irq_get_affinity callback from device_driver first then from bus_type - collected tags - fixed typos - Link to v3: https://lore.kernel.org/r/20241112-refactor-blk-affinity-helpers-v3-0-573bfca0cbd8@kernel.org Changes in v3: - dropped the additinal argument in blk_mq_hctx_map_queues. leave open coded version in hisi_sas_v2. - splitted "blk-mp: introduce blk_mq_hctx_map_queues" patch into three patches. - dropped local variable in pci_device_irq_get_affinity - Link to v2: https://lore.kernel.org/r/20241111-refactor-blk-affinity-helpers-v2-0-f360ddad231a@kernel.org Changes in v2: - added new callback to struct bus_type and call directly the affinity helpers from there. - Link to v1: https://lore.kernel.org/r/20240913-refactor-blk-affinity-helpers-v1-0-8e058f77af12@suse.de Changes in v1: - renamed blk_mq_dev_map_queues to blk_mq_hctx_map_queues - squased 'virito: add APIs for retrieving vq affinity' into 'blk-mq: introduce blk_mq_hctx_map_queues' - moved hisi_sas changed into a new patch - hisi_sas use define instead of hard coded value - moved helpers into their matching subsystem, removed blk-mq-pci and blk-mq-virtio files - fix spelling/typos - fixed long lines in docu (yep new lines in brief descriptions are supported, tested ti) - based on the first part of [1] https://lore.kernel.org/all/20240806-isolcpus-io-queues-v3-0-da0eecfeaf8b@suse.de --- Daniel Wagner (8): driver core: bus: add irq_get_affinity callback to bus_type PCI: hookup irq_get_affinity callback virtio: hookup irq_get_affinity callback blk-mq: introduce blk_mq_map_hw_queues scsi: replace blk_mq_pci_map_queues with blk_mq_map_hw_queues nvme: replace blk_mq_pci_map_queues with blk_mq_map_hw_queues virtio: blk/scsi: replace blk_mq_virtio_map_queues with blk_mq_map_hw_queues blk-mq: remove unused queue mapping helpers block/Makefile | 2 -- block/blk-mq-cpumap.c | 37 +++++++++++++++++++++++++ block/blk-mq-pci.c | 46 ------------------------------- block/blk-mq-virtio.c | 46 ------------------------------- drivers/block/virtio_blk.c | 4 +-- drivers/nvme/host/fc.c | 1 - drivers/nvme/host/pci.c | 3 +- drivers/pci/pci-driver.c | 14 ++++++++++ drivers/scsi/fnic/fnic_main.c | 3 +- drivers/scsi/hisi_sas/hisi_sas.h | 1 - drivers/scsi/hisi_sas/hisi_sas_v3_hw.c | 4 +-- drivers/scsi/megaraid/megaraid_sas_base.c | 3 +- drivers/scsi/mpi3mr/mpi3mr.h | 1 - drivers/scsi/mpi3mr/mpi3mr_os.c | 2 +- drivers/scsi/mpt3sas/mpt3sas_scsih.c | 3 +- drivers/scsi/pm8001/pm8001_init.c | 2 +- drivers/scsi/pm8001/pm8001_sas.h | 1 - drivers/scsi/qla2xxx/qla_nvme.c | 3 +- drivers/scsi/qla2xxx/qla_os.c | 4 +-- drivers/scsi/smartpqi/smartpqi_init.c | 7 ++--- drivers/scsi/virtio_scsi.c | 3 +- drivers/virtio/virtio.c | 19 +++++++++++++ include/linux/blk-mq-pci.h | 11 -------- include/linux/blk-mq-virtio.h | 11 -------- include/linux/blk-mq.h | 2 ++ include/linux/device/bus.h | 3 ++ 26 files changed, 92 insertions(+), 144 deletions(-) --- base-commit: 22db180127a901ce41aa14d74da2e2eb452540b1 change-id: 20240912-refactor-blk-affinity-helpers-7089b95b4b10 Best regards,