From: Christoph Hellwig <hch@infradead.org>
To: Yi Zhang <yi.zhang@redhat.com>
Cc: linux-nvme@lists.infradead.org
Subject: Re: [PATCH] nvme: typo fix for nvme subsystem
Date: Wed, 4 Jun 2025 01:04:28 -0700 [thread overview]
Message-ID: <aD_-DLW_JB6fBLvC@infradead.org> (raw)
In-Reply-To: <20250425100107.844298-1-yi.zhang@redhat.com>
I've added this to nvme-6.16 after writing a proper commit log.
Please do that yourself next time.
> --- a/drivers/nvme/host/rdma.c
> +++ b/drivers/nvme/host/rdma.c
> @@ -221,7 +221,7 @@ static struct nvme_rdma_qe *nvme_rdma_alloc_ring(struct ib_device *ibdev,
>
> /*
> * Bind the CQEs (post recv buffers) DMA mapping to the RDMA queue
> - * lifetime. It's safe, since any chage in the underlying RDMA device
> + * lifetime. It's safe, since any change in the underlying RDMA device
> * will issue error recovery and queue re-creation.
> */
> for (i = 0; i < ib_queue_size; i++) {
> @@ -800,7 +800,7 @@ static int nvme_rdma_configure_admin_queue(struct nvme_rdma_ctrl *ctrl,
>
> /*
> * Bind the async event SQE DMA mapping to the admin queue lifetime.
> - * It's safe, since any chage in the underlying RDMA device will issue
> + * It's safe, since any change in the underlying RDMA device will issue
> * error recovery and queue re-creation.
> */
> error = nvme_rdma_alloc_qe(ctrl->device->dev, &ctrl->async_event_sqe,
> diff --git a/drivers/nvme/target/admin-cmd.c b/drivers/nvme/target/admin-cmd.c
> index acc138bbf8f2..ff731b5a072e 100644
> --- a/drivers/nvme/target/admin-cmd.c
> +++ b/drivers/nvme/target/admin-cmd.c
> @@ -1174,7 +1174,7 @@ static void nvmet_execute_identify(struct nvmet_req *req)
> * A "minimum viable" abort implementation: the command is mandatory in the
> * spec, but we are not required to do any useful work. We couldn't really
> * do a useful abort, so don't bother even with waiting for the command
> - * to be exectuted and return immediately telling the command to abort
> + * to be executed and return immediately telling the command to abort
> * wasn't found.
> */
> static void nvmet_execute_abort(struct nvmet_req *req)
> diff --git a/drivers/nvme/target/core.c b/drivers/nvme/target/core.c
> index 71f8d06998d6..aac398f96c9e 100644
> --- a/drivers/nvme/target/core.c
> +++ b/drivers/nvme/target/core.c
> @@ -648,7 +648,7 @@ void nvmet_ns_disable(struct nvmet_ns *ns)
> * Now that we removed the namespaces from the lookup list, we
> * can kill the per_cpu ref and wait for any remaining references
> * to be dropped, as well as a RCU grace period for anyone only
> - * using the namepace under rcu_read_lock(). Note that we can't
> + * using the namespace under rcu_read_lock(). Note that we can't
> * use call_rcu here as we need to ensure the namespaces have
> * been fully destroyed before unloading the module.
> */
> diff --git a/drivers/nvme/target/fc.c b/drivers/nvme/target/fc.c
> index 7b50130f10f6..297bf2886127 100644
> --- a/drivers/nvme/target/fc.c
> +++ b/drivers/nvme/target/fc.c
> @@ -1328,7 +1328,7 @@ nvmet_fc_portentry_rebind_tgt(struct nvmet_fc_tgtport *tgtport)
> /**
> * nvmet_fc_register_targetport - transport entry point called by an
> * LLDD to register the existence of a local
> - * NVME subystem FC port.
> + * NVME subsystem FC port.
> * @pinfo: pointer to information about the port to be registered
> * @template: LLDD entrypoints and operational parameters for the port
> * @dev: physical hardware device node port corresponds to. Will be
> diff --git a/drivers/nvme/target/io-cmd-bdev.c b/drivers/nvme/target/io-cmd-bdev.c
> index 83be0657e6df..b25af5cbaf05 100644
> --- a/drivers/nvme/target/io-cmd-bdev.c
> +++ b/drivers/nvme/target/io-cmd-bdev.c
> @@ -133,7 +133,7 @@ u16 blk_to_nvme_status(struct nvmet_req *req, blk_status_t blk_sts)
> * Right now there exists M : 1 mapping between block layer error
> * to the NVMe status code (see nvme_error_status()). For consistency,
> * when we reverse map we use most appropriate NVMe Status code from
> - * the group of the NVMe staus codes used in the nvme_error_status().
> + * the group of the NVMe status codes used in the nvme_error_status().
> */
> switch (blk_sts) {
> case BLK_STS_NOSPC:
> diff --git a/drivers/nvme/target/passthru.c b/drivers/nvme/target/passthru.c
> index 26e2907ce8bb..b7515c53829b 100644
> --- a/drivers/nvme/target/passthru.c
> +++ b/drivers/nvme/target/passthru.c
> @@ -99,7 +99,7 @@ static u16 nvmet_passthru_override_id_ctrl(struct nvmet_req *req)
>
> /*
> * The passthru NVMe driver may have a limit on the number of segments
> - * which depends on the host's memory fragementation. To solve this,
> + * which depends on the host's memory fragmentation. To solve this,
> * ensure mdts is limited to the pages equal to the number of segments.
> */
> max_hw_sectors = min_not_zero(pctrl->max_segments << PAGE_SECTORS_SHIFT,
> --
> 2.45.1
>
>
---end quoted text---
prev parent reply other threads:[~2025-06-04 8:31 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-04-25 10:01 [PATCH] nvme: typo fix for nvme subsystem Yi Zhang
2025-04-29 16:32 ` Chaitanya Kulkarni
2025-05-30 14:03 ` Yi Zhang
2025-06-04 8:04 ` Christoph Hellwig [this message]
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=aD_-DLW_JB6fBLvC@infradead.org \
--to=hch@infradead.org \
--cc=linux-nvme@lists.infradead.org \
--cc=yi.zhang@redhat.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.