[PATCH v3 0/6] blk-mq: Add a flag for reserved requests series

John Garry posted 6 patches 3 years, 9 months ago
Documentation/scsi/scsi_eh.rst          |  3 +--
Documentation/scsi/scsi_mid_low_api.rst |  2 +-
block/blk-mq-debugfs.c                  |  2 +-
block/blk-mq-tag.c                      | 13 +++++--------
block/blk-mq.c                          | 22 +++++++++++++---------
block/bsg-lib.c                         |  2 +-
drivers/block/mtip32xx/mtip32xx.c       |  9 ++++-----
drivers/block/nbd.c                     |  5 ++---
drivers/block/null_blk/main.c           |  2 +-
drivers/infiniband/ulp/srp/ib_srp.c     |  3 +--
drivers/mmc/core/queue.c                |  3 +--
drivers/nvme/host/apple.c               |  3 +--
drivers/nvme/host/core.c                |  2 +-
drivers/nvme/host/fc.c                  |  6 ++----
drivers/nvme/host/nvme.h                |  2 +-
drivers/nvme/host/pci.c                 |  2 +-
drivers/nvme/host/rdma.c                |  3 +--
drivers/nvme/host/tcp.c                 |  3 +--
drivers/s390/block/dasd.c               |  2 +-
drivers/s390/block/dasd_int.h           |  2 +-
drivers/scsi/aacraid/comminit.c         |  2 +-
drivers/scsi/aacraid/linit.c            |  2 +-
drivers/scsi/fnic/fnic_scsi.c           | 14 ++++----------
drivers/scsi/hosts.c                    | 14 ++++++--------
drivers/scsi/mpi3mr/mpi3mr_os.c         | 16 ++++------------
drivers/scsi/scsi_error.c               |  6 +++---
drivers/scsi/scsi_lib.c                 |  8 --------
drivers/scsi/scsi_priv.h                |  2 +-
include/linux/blk-mq.h                  | 10 ++++++++--
include/scsi/scsi_host.h                |  2 +-
30 files changed, 70 insertions(+), 97 deletions(-)
[PATCH v3 0/6] blk-mq: Add a flag for reserved requests series
Posted by John Garry 3 years, 9 months ago
Hi Jens,

Can you please consider this series? Thanks

---

In [0] I included "blk-mq: Add a flag for reserved requests" to identify
if a request is 'reserved' for special handling. Doing this is easier than
passing a 'reserved' arg to the blk_mq_ops callbacks. Indeed, only 1x
timeout implementation or blk-mq iter function actually uses the
'reserved' arg (or 3x if you count SCSI core and FNIC SCSI driver). So
this series drops the 'reserved' arg for these timeout and iter functions.
Christoph suggested that I try to upstream now.

Differences to v2:
- Rebase
- Apply more RB tags (thanks!)

Differences to v1:
- Use "scsi_timeout" as name for SCSI timeout function and update docs
- Add RB tags (thanks!)
- Split out patch to drop local variables for 'reserved', as requested by
  Bart

Based on following:
e55cf7981405 (block/for-5.20/block) blk-cgroup: factor out blkcg_free_all_cpd()

[0] https://lore.kernel.org/linux-scsi/1654770559-101375-1-git-send-email-john.garry@huawei.com/T/#m22aa9f89e55835edc2e650d43f7e3219a3a1a324

John Garry (6):
  scsi: core: Remove reserved request time-out handling
  blk-mq: Add a flag for reserved requests
  blk-mq: Drop blk_mq_ops.timeout 'reserved' arg
  scsi: fnic: Drop reserved request handling
  blk-mq: Drop 'reserved' arg of busy_tag_iter_fn
  blk-mq: Drop local variable for reserved tag

 Documentation/scsi/scsi_eh.rst          |  3 +--
 Documentation/scsi/scsi_mid_low_api.rst |  2 +-
 block/blk-mq-debugfs.c                  |  2 +-
 block/blk-mq-tag.c                      | 13 +++++--------
 block/blk-mq.c                          | 22 +++++++++++++---------
 block/bsg-lib.c                         |  2 +-
 drivers/block/mtip32xx/mtip32xx.c       |  9 ++++-----
 drivers/block/nbd.c                     |  5 ++---
 drivers/block/null_blk/main.c           |  2 +-
 drivers/infiniband/ulp/srp/ib_srp.c     |  3 +--
 drivers/mmc/core/queue.c                |  3 +--
 drivers/nvme/host/apple.c               |  3 +--
 drivers/nvme/host/core.c                |  2 +-
 drivers/nvme/host/fc.c                  |  6 ++----
 drivers/nvme/host/nvme.h                |  2 +-
 drivers/nvme/host/pci.c                 |  2 +-
 drivers/nvme/host/rdma.c                |  3 +--
 drivers/nvme/host/tcp.c                 |  3 +--
 drivers/s390/block/dasd.c               |  2 +-
 drivers/s390/block/dasd_int.h           |  2 +-
 drivers/scsi/aacraid/comminit.c         |  2 +-
 drivers/scsi/aacraid/linit.c            |  2 +-
 drivers/scsi/fnic/fnic_scsi.c           | 14 ++++----------
 drivers/scsi/hosts.c                    | 14 ++++++--------
 drivers/scsi/mpi3mr/mpi3mr_os.c         | 16 ++++------------
 drivers/scsi/scsi_error.c               |  6 +++---
 drivers/scsi/scsi_lib.c                 |  8 --------
 drivers/scsi/scsi_priv.h                |  2 +-
 include/linux/blk-mq.h                  | 10 ++++++++--
 include/scsi/scsi_host.h                |  2 +-
 30 files changed, 70 insertions(+), 97 deletions(-)

-- 
2.35.3
Re: [PATCH v3 0/6] blk-mq: Add a flag for reserved requests series
Posted by Jens Axboe 3 years, 9 months ago
On Wed, 6 Jul 2022 20:03:48 +0800, John Garry wrote:
> Can you please consider this series? Thanks
> 

Applied, thanks!

[1/6] scsi: core: Remove reserved request time-out handling
      commit: deef1be18e3fc62ddf04fb3e5e8ff6a301693dcc
[2/6] blk-mq: Add a flag for reserved requests
      commit: 99e48cd6855e9535488e3c90d65edd46c6e6fc1b
[3/6] blk-mq: Drop blk_mq_ops.timeout 'reserved' arg
      commit: 9bdb4833dd399cbff82cc20893f52bdec66a9eca
[4/6] scsi: fnic: Drop reserved request handling
      commit: 1263c1929fb8c375494666ec6d1bac838ff02c25
[5/6] blk-mq: Drop 'reserved' arg of busy_tag_iter_fn
      commit: 2dd6532e9591f201e7571b30915db807603ab924
[6/6] blk-mq: Drop local variable for reserved tag
      commit: 4cf6e6c0106bf6e6d034fa6043b4428ac2f267fc

Best regards,
-- 
Jens Axboe