Linux-ARM-Kernel Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/3] arm64: kvm: fix typo "synchonized" in comment
       [not found] <20260904122445.31163-1-hemanth.selam@gmail.com>
@ 2026-09-04 12:24 ` Hemanth Selam
  2026-09-04 12:24 ` [PATCH 2/3] selftests: kvm: fix typos in comments Hemanth Selam
  2026-09-04 12:24 ` [PATCH 3/3] arm64: kvm: fix repeated word 'is' in comment Hemanth Selam
  2 siblings, 0 replies; 5+ messages in thread
From: Hemanth Selam @ 2026-09-04 12:24 UTC (permalink / raw)
  To: Marc Zyngier, Oliver Upton, Fuad Tabba, Joey Gouly, Steffen Eiden,
	Suzuki K Poulose, Zenghui Yu, Catalin Marinas, Will Deacon,
	Mark Rutland
  Cc: linux-kernel, linux-arm-kernel, kvmarm

Correct "synchonized" to "synchronized", reported by scripts/checkpatch.pl
using the misspelling list in scripts/spelling.txt.  Only touches comments,
no code changes.

Assisted-by: Cursor:claude-opus-5
Signed-off-by: Hemanth Selam <hemanth.selam@gmail.com>
---
 arch/arm64/kvm/hyp/entry.S | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/kvm/hyp/entry.S b/arch/arm64/kvm/hyp/entry.S
index 308100ed25de..9709905f21b5 100644
--- a/arch/arm64/kvm/hyp/entry.S
+++ b/arch/arm64/kvm/hyp/entry.S
@@ -46,7 +46,7 @@ alternative_else_nop_endif
 
 	// Ensure that __guest_enter() always provides a context
 	// synchronization event so that callers don't need ISBs for anything
-	// that would usually be synchonized by the ERET.
+	// that would usually be synchronized by the ERET.
 	isb
 	mov	x0, #ARM_EXCEPTION_IRQ
 	ret
-- 
2.48.1



^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH 2/3] selftests: kvm: fix typos in comments
       [not found] <20260904122445.31163-1-hemanth.selam@gmail.com>
  2026-09-04 12:24 ` [PATCH 1/3] arm64: kvm: fix typo "synchonized" in comment Hemanth Selam
@ 2026-09-04 12:24 ` Hemanth Selam
  2026-09-04 12:24 ` [PATCH 3/3] arm64: kvm: fix repeated word 'is' in comment Hemanth Selam
  2 siblings, 0 replies; 5+ messages in thread
From: Hemanth Selam @ 2026-09-04 12:24 UTC (permalink / raw)
  To: Marc Zyngier, Oliver Upton, Fuad Tabba, Joey Gouly, Steffen Eiden,
	Suzuki K Poulose, Zenghui Yu, Paolo Bonzini, Sean Christopherson,
	Shuah Khan
  Cc: linux-kernel, linux-arm-kernel, kvmarm, kvm, linux-kselftest

Fix typos in comments, reported by scripts/checkpatch.pl using the
misspelling list in scripts/spelling.txt.  Only touches comments, no code
changes.

Assisted-by: Cursor:claude-opus-5
Signed-off-by: Hemanth Selam <hemanth.selam@gmail.com>
---
 tools/testing/selftests/kvm/arm64/debug-exceptions.c | 2 +-
 tools/testing/selftests/kvm/arm64/hypercalls.c       | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/tools/testing/selftests/kvm/arm64/debug-exceptions.c b/tools/testing/selftests/kvm/arm64/debug-exceptions.c
index 7dc5f0b4f6ad..9da6dc5bcfe6 100644
--- a/tools/testing/selftests/kvm/arm64/debug-exceptions.c
+++ b/tools/testing/selftests/kvm/arm64/debug-exceptions.c
@@ -387,7 +387,7 @@ static void guest_code_ss(int test_cnt)
 		 * Enable Single Step execution.  Note!  This _must_ be a bare
 		 * ucall as the ucall() path uses atomic operations to manage
 		 * the ucall structures, and the built-in "atomics" are usually
-		 * implemented via exclusive access instructions.  The exlusive
+		 * implemented via exclusive access instructions.  The exclusive
 		 * monitor is cleared on ERET, and so taking debug exceptions
 		 * during a LDREX=>STREX sequence will prevent forward progress
 		 * and hang the guest/test.
diff --git a/tools/testing/selftests/kvm/arm64/hypercalls.c b/tools/testing/selftests/kvm/arm64/hypercalls.c
index 5d96cdf382c4..a24becda9e62 100644
--- a/tools/testing/selftests/kvm/arm64/hypercalls.c
+++ b/tools/testing/selftests/kvm/arm64/hypercalls.c
@@ -1,9 +1,9 @@
 // SPDX-License-Identifier: GPL-2.0-only
 
-/* hypercalls: Check the ARM64's psuedo-firmware bitmap register interface.
+/* hypercalls: Check the ARM64's pseudo-firmware bitmap register interface.
  *
  * The test validates the basic hypercall functionalities that are exposed
- * via the psuedo-firmware bitmap register. This includes the registers'
+ * via the pseudo-firmware bitmap register. This includes the registers'
  * read/write behavior before and after the VM has started, and if the
  * hypercalls are properly masked or unmasked to the guest when disabled or
  * enabled from the KVM userspace, respectively.
-- 
2.48.1



^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH 3/3] arm64: kvm: fix repeated word 'is' in comment
       [not found] <20260904122445.31163-1-hemanth.selam@gmail.com>
  2026-09-04 12:24 ` [PATCH 1/3] arm64: kvm: fix typo "synchonized" in comment Hemanth Selam
  2026-09-04 12:24 ` [PATCH 2/3] selftests: kvm: fix typos in comments Hemanth Selam
@ 2026-09-04 12:24 ` Hemanth Selam
  2026-09-04 12:46   ` Fuad Tabba
  2026-09-04 19:29   ` Andrew Lunn
  2 siblings, 2 replies; 5+ messages in thread
From: Hemanth Selam @ 2026-09-04 12:24 UTC (permalink / raw)
  To: Marc Zyngier, Oliver Upton, Fuad Tabba, Joey Gouly, Steffen Eiden,
	Suzuki K Poulose, Zenghui Yu, Catalin Marinas, Will Deacon,
	Mark Rutland
  Cc: linux-kernel, linux-arm-kernel, kvmarm

Drop the second 'is', reported by checkpatch.pl as a possible repeated
word.  Only touches a comment, no code changes.

Assisted-by: Cursor:claude-opus-5
Signed-off-by: Hemanth Selam <hemanth.selam@gmail.com>
---
 arch/arm64/kvm/pauth.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/kvm/pauth.c b/arch/arm64/kvm/pauth.c
index d5eb3ae876be..22ff4b78d0e9 100644
--- a/arch/arm64/kvm/pauth.c
+++ b/arch/arm64/kvm/pauth.c
@@ -5,7 +5,7 @@
  *
  * Primitive PAuth emulation for ERETAA/ERETAB.
  *
- * This code assumes that is is run from EL2, and that it is part of
+ * This code assumes that is run from EL2, and that it is part of
  * the emulation of ERETAx for a guest hypervisor. That's a lot of
  * baked-in assumptions and shortcuts.
  *
-- 
2.48.1



^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH 3/3] arm64: kvm: fix repeated word 'is' in comment
  2026-09-04 12:24 ` [PATCH 3/3] arm64: kvm: fix repeated word 'is' in comment Hemanth Selam
@ 2026-09-04 12:46   ` Fuad Tabba
  2026-09-04 19:29   ` Andrew Lunn
  1 sibling, 0 replies; 5+ messages in thread
From: Fuad Tabba @ 2026-09-04 12:46 UTC (permalink / raw)
  To: Hemanth Selam
  Cc: Marc Zyngier, Oliver Upton, Joey Gouly, Steffen Eiden,
	Suzuki K Poulose, Zenghui Yu, Catalin Marinas, Will Deacon,
	Mark Rutland, linux-kernel, linux-arm-kernel, kvmarm

Hi Hemanth,

On Fri, 4 Sept 2026 at 13:25, Hemanth Selam <hemanth.selam@gmail.com> wrote:
>
> Drop the second 'is', reported by checkpatch.pl as a possible repeated
> word.  Only touches a comment, no code changes.
>
> Assisted-by: Cursor:claude-opus-5
> Signed-off-by: Hemanth Selam <hemanth.selam@gmail.com>
> ---
>  arch/arm64/kvm/pauth.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/kvm/pauth.c b/arch/arm64/kvm/pauth.c
> index d5eb3ae876be..22ff4b78d0e9 100644
> --- a/arch/arm64/kvm/pauth.c
> +++ b/arch/arm64/kvm/pauth.c
> @@ -5,7 +5,7 @@
>   *
>   * Primitive PAuth emulation for ERETAA/ERETAB.
>   *
> - * This code assumes that is is run from EL2, and that it is part of
> + * This code assumes that is run from EL2, and that it is part of

The typo here is that the first "is" should have been an "it", rather
than being a duplicate.

+ * This code assumes that it is run from EL2, and that it is part of

Cheers,
/fuad

>   * the emulation of ERETAx for a guest hypervisor. That's a lot of
>   * baked-in assumptions and shortcuts.
>   *
> --
> 2.48.1
>


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 3/3] arm64: kvm: fix repeated word 'is' in comment
  2026-09-04 12:24 ` [PATCH 3/3] arm64: kvm: fix repeated word 'is' in comment Hemanth Selam
  2026-09-04 12:46   ` Fuad Tabba
@ 2026-09-04 19:29   ` Andrew Lunn
  1 sibling, 0 replies; 5+ messages in thread
From: Andrew Lunn @ 2026-09-04 19:29 UTC (permalink / raw)
  To: Hemanth Selam
  Cc: Marc Zyngier, Oliver Upton, Fuad Tabba, Joey Gouly, Steffen Eiden,
	Suzuki K Poulose, Zenghui Yu, Catalin Marinas, Will Deacon,
	Mark Rutland, linux-kernel, linux-arm-kernel, kvmarm

On Fri, Sep 04, 2026 at 05:54:42PM +0530, Hemanth Selam wrote:
> Drop the second 'is', reported by checkpatch.pl as a possible repeated
> word.  Only touches a comment, no code changes.

I hope you are really looking at the sentence, understanding the
sentence, and then modifying the sentence based on your understanding,
not just removing the duplicate to make checkpatch happy.

You really do need to engage brain.

You also should consider that this checkpatch warning has been around
a while. Many of the real problems have already been fixed, so there
is probably a high false positive rate for this check. So you need to
be extra careful, or you are going to make things worse, not better.

"The the in this sentence is the subject."

If you looked back in the archives maybe a month, there was somebody
who get flamed for getting this sort of checkpatch fixup badly wrong.

    Andrew


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2026-09-04 19:30 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20260904122445.31163-1-hemanth.selam@gmail.com>
2026-09-04 12:24 ` [PATCH 1/3] arm64: kvm: fix typo "synchonized" in comment Hemanth Selam
2026-09-04 12:24 ` [PATCH 2/3] selftests: kvm: fix typos in comments Hemanth Selam
2026-09-04 12:24 ` [PATCH 3/3] arm64: kvm: fix repeated word 'is' in comment Hemanth Selam
2026-09-04 12:46   ` Fuad Tabba
2026-09-04 19:29   ` Andrew Lunn

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox