From: Amit Machhiwal <amachhiw@linux.ibm.com>
To: Salman Farooq <salman.farooq3310@gmail.com>
Cc: maddy@linux.ibm.com, npiggin@gmail.com, mpe@ellerman.id.au,
chleroy@kernel.org, linuxppc-dev@lists.ozlabs.org,
kvm@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] KVM: PPC: Book3S HV: Fix comment typos
Date: Thu, 9 Jul 2026 12:59:53 +0530 [thread overview]
Message-ID: <20260709125357.49cafeb6-1b-amachhiw@linux.ibm.com> (raw)
In-Reply-To: <20260702030344.2211462-1-salman.farooq3310@gmail.com>
On 2026/07/02 08:03 AM, Salman Farooq wrote:
> Fix two instances of a misspelled word in comments.
>
> Signed-off-by: Salman Farooq <salman.farooq3310@gmail.com>
> ---
> arch/powerpc/kvm/book3s_hv_uvmem.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
I agree with Ritesh on the amount of churn such misspell fixes may
cause.
Regardless, since this patch has no functional or code change, you may
want to add a 'trivial' suffix to the patch title, e.g.:
KVM: PPC: Book3S HV: trivial: Fix comment typos
This convention is used to indicate non-functional changes, making it
easier for maintainers to apply without worrying about regressions. See
[1] and [2] for precedent.
[1] https://git.kernel.org/powerpc/c/d2827e5e2e0f0941a651f4b1ca5e9b778c4b5293
[2] https://lore.kernel.org/all/87fr3vvfvq.fsf@vajain21.in.ibm.com/
~Amit
>
> diff --git a/arch/powerpc/kvm/book3s_hv_uvmem.c b/arch/powerpc/kvm/book3s_hv_uvmem.c
> index 5fbb95d90e99..6ac8364a7639 100644
> --- a/arch/powerpc/kvm/book3s_hv_uvmem.c
> +++ b/arch/powerpc/kvm/book3s_hv_uvmem.c
> @@ -163,7 +163,7 @@ static DEFINE_SPINLOCK(kvmppc_uvmem_bitmap_lock);
> * All its GFNs are moved to Normal-GFNs.
> *
> * UV_TERMINATE transitions the secure-VM back to normal-VM. All
> - * the secure-GFN and shared-GFNs are tranistioned to normal-GFN
> + * the secure-GFN and shared-GFNs are transitioned to normal-GFN
> * Note: The contents of the normal-GFN is undefined at this point.
> *
> * GFN state implementation:
> @@ -1010,7 +1010,7 @@ static vm_fault_t kvmppc_uvmem_migrate_to_ram(struct vm_fault *vmf)
> /*
> * Release the device PFN back to the pool
> *
> - * Gets called when secure GFN tranistions from a secure-PFN
> + * Gets called when secure GFN transitions from a secure-PFN
> * to a normal PFN during H_SVM_PAGE_OUT.
> * Gets called with kvm->arch.uvmem_lock held.
> */
> --
> 2.55.0
>
next prev parent reply other threads:[~2026-07-09 7:29 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-02 3:03 [PATCH] KVM: PPC: Book3S HV: Fix comment typos Salman Farooq
2026-07-09 7:01 ` Ritesh Harjani
2026-07-09 7:23 ` Salman Farooq
2026-07-09 7:29 ` Amit Machhiwal [this message]
2026-07-09 7:43 ` Salman Farooq
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=20260709125357.49cafeb6-1b-amachhiw@linux.ibm.com \
--to=amachhiw@linux.ibm.com \
--cc=chleroy@kernel.org \
--cc=kvm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=maddy@linux.ibm.com \
--cc=mpe@ellerman.id.au \
--cc=npiggin@gmail.com \
--cc=salman.farooq3310@gmail.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