Fix typo in comment (indicies -> indices)

[PATCH] Fix typo in comment (indicies -> indices)

Frediano Ziglio posted 1 patch 2 weeks, 3 days ago

Download mbox

Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/20260706163750.445797-1-frediano.ziglio@citrix.com

xen/arch/x86/traps-setup.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

Expand all Fold all

[PATCH] Fix typo in comment (indicies -> indices)

Posted by Frediano Ziglio 2 weeks, 3 days ago

From: Frediano Ziglio <frediano.ziglio@cloud.com>

Signed-off-by: Frediano Ziglio <frediano.ziglio@cloud.com>
---
 xen/arch/x86/traps-setup.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/xen/arch/x86/traps-setup.c b/xen/arch/x86/traps-setup.c
index a79a3b2013..3fe79ff0d0 100644
--- a/xen/arch/x86/traps-setup.c
+++ b/xen/arch/x86/traps-setup.c
@@ -245,7 +245,7 @@ static void __init init_ler(void)
 
     /*
      * Intel Pentium 4 is the only known CPU to not use the architectural MSR
-     * indicies.
+     * indices.
      */
     switch ( boot_cpu_data.vendor )
     {
-- 
2.43.0

Re: [PATCH] Fix typo in comment (indicies -> indices)

Posted by Jan Beulich 2 weeks, 2 days ago

On 06.07.2026 18:37, Frediano Ziglio wrote:
> From: Frediano Ziglio <frediano.ziglio@cloud.com>
> 
> Signed-off-by: Frediano Ziglio <frediano.ziglio@cloud.com>

Acked-by: Jan Beulich <jbeulich@suse.com>

However, I question such a change to be worthwhile the overhead. If this
was a widespread issue, sure. If there were many typos that are being
corrected in one go, perhaps. But otherwise best to correct when the
comment (or at least adjacent code) needs touching anyway.

Jan

> --- a/xen/arch/x86/traps-setup.c
> +++ b/xen/arch/x86/traps-setup.c
> @@ -245,7 +245,7 @@ static void __init init_ler(void)
>  
>      /*
>       * Intel Pentium 4 is the only known CPU to not use the architectural MSR
> -     * indicies.
> +     * indices.
>       */
>      switch ( boot_cpu_data.vendor )
>      {