From: John Garry <john.garry@huawei.com>
To: <axboe@kernel.dk>, <damien.lemoal@opensource.wdc.com>,
<bvanassche@acm.org>, <hch@lst.de>, <jejb@linux.ibm.com>,
<martin.petersen@oracle.com>, <hare@suse.de>,
<satishkh@cisco.com>, <sebaddel@cisco.com>, <kartilak@cisco.com>
Cc: <linux-doc@vger.kernel.org>, <linux-rdma@vger.kernel.org>,
<linux-mmc@vger.kernel.org>, <linux-nvme@lists.infradead.org>,
<linux-s390@vger.kernel.org>, <linux-scsi@vger.kernel.org>,
<mpi3mr-linuxdrv.pdl@broadcom.com>, <linux-block@vger.kernel.org>,
<linux-kernel@vger.kernel.org>, <nbd@other.debian.org>,
John Garry <john.garry@huawei.com>
Subject: [PATCH v3 0/6] blk-mq: Add a flag for reserved requests series
Date: Wed, 6 Jul 2022 20:03:48 +0800 [thread overview]
Message-ID: <1657109034-206040-1-git-send-email-john.garry@huawei.com> (raw)
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
next reply other threads:[~2022-07-06 12:11 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-07-06 12:03 John Garry [this message]
2022-07-06 12:03 ` [PATCH v3 1/6] scsi: core: Remove reserved request time-out handling John Garry
2022-07-06 12:03 ` [PATCH v3 2/6] blk-mq: Add a flag for reserved requests John Garry
2022-07-06 12:03 ` [PATCH v3 3/6] blk-mq: Drop blk_mq_ops.timeout 'reserved' arg John Garry
2022-07-06 12:03 ` [PATCH v3 4/6] scsi: fnic: Drop reserved request handling John Garry
2022-07-06 12:03 ` [PATCH v3 5/6] blk-mq: Drop 'reserved' arg of busy_tag_iter_fn John Garry
2022-07-06 12:03 ` [PATCH v3 6/6] blk-mq: Drop local variable for reserved tag John Garry
2022-07-06 13:57 ` Hannes Reinecke
2022-07-06 12:34 ` [PATCH v3 0/6] blk-mq: Add a flag for reserved requests series Jens Axboe
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=1657109034-206040-1-git-send-email-john.garry@huawei.com \
--to=john.garry@huawei.com \
--cc=axboe@kernel.dk \
--cc=bvanassche@acm.org \
--cc=damien.lemoal@opensource.wdc.com \
--cc=hare@suse.de \
--cc=hch@lst.de \
--cc=jejb@linux.ibm.com \
--cc=kartilak@cisco.com \
--cc=linux-block@vger.kernel.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mmc@vger.kernel.org \
--cc=linux-nvme@lists.infradead.org \
--cc=linux-rdma@vger.kernel.org \
--cc=linux-s390@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=martin.petersen@oracle.com \
--cc=mpi3mr-linuxdrv.pdl@broadcom.com \
--cc=nbd@other.debian.org \
--cc=satishkh@cisco.com \
--cc=sebaddel@cisco.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
all inboxes | Powered by JetHome®