[PATCH] Fix some typos in comments

Frediano Ziglio posted 1 patch 2 months, 3 weeks ago
Failed in applying to current master (apply log)
arch/x86/include/arch/processor.h | 2 +-
include/xen/hvm/params.h          | 2 +-
include/xtf/console.h             | 2 +-
include/xtf/extable.h             | 4 ++--
4 files changed, 5 insertions(+), 5 deletions(-)
[PATCH] Fix some typos in comments
Posted by Frediano Ziglio 2 months, 3 weeks ago
Signed-off-by: Frediano Ziglio <frediano.ziglio@cloud.com>
---
 arch/x86/include/arch/processor.h | 2 +-
 include/xen/hvm/params.h          | 2 +-
 include/xtf/console.h             | 2 +-
 include/xtf/extable.h             | 4 ++--
 4 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/arch/x86/include/arch/processor.h b/arch/x86/include/arch/processor.h
index 0c33545..1c7e655 100644
--- a/arch/x86/include/arch/processor.h
+++ b/arch/x86/include/arch/processor.h
@@ -101,7 +101,7 @@
 #define X86_EXC_TS            10 /* Invalid TSS. */
 #define X86_EXC_NP            11 /* Segment Not Present. */
 #define X86_EXC_SS            12 /* Stack-Segment Fault. */
-#define X86_EXC_GP            13 /* General Porection Fault. */
+#define X86_EXC_GP            13 /* General Protection Fault. */
 #define X86_EXC_PF            14 /* Page Fault. */
 #define X86_EXC_SPV           15 /* PIC Spurious Interrupt Vector. */
 #define X86_EXC_MF            16 /* Maths fault (x87 FPU). */
diff --git a/include/xen/hvm/params.h b/include/xen/hvm/params.h
index 886b986..0b7c05b 100644
--- a/include/xen/hvm/params.h
+++ b/include/xen/hvm/params.h
@@ -1,5 +1,5 @@
 /*
- * Xen public hvm paramter index
+ * Xen public hvm parameter index
  */
 
 #ifndef XEN_PUBLIC_HVM_PARAMS_H
diff --git a/include/xtf/console.h b/include/xtf/console.h
index caec790..16a6a23 100644
--- a/include/xtf/console.h
+++ b/include/xtf/console.h
@@ -11,7 +11,7 @@
 typedef void (*cons_output_cb)(const char *buf, size_t len);
 
 /*
- * Register a console callback.  Several callbacks can be registered for usful
+ * Register a console callback.  Several callbacks can be registered for useful
  * destinations of console text.
  */
 void register_console_callback(cons_output_cb cb);
diff --git a/include/xtf/extable.h b/include/xtf/extable.h
index e93331e..0668b7d 100644
--- a/include/xtf/extable.h
+++ b/include/xtf/extable.h
@@ -4,8 +4,8 @@
  * Exception table support.
  *
  * Allows code to tag an instruction which might fault, and where to jump to
- * in order to recover.  For more complicated recovery, a cusom handler
- * handler can be registerd.
+ * in order to recover.  For more complicated recovery, a custom handler
+ * can be registered.
  */
 #ifndef XTF_EXTABLE_H
 #define XTF_EXTABLE_H
-- 
2.43.0
Re: [PATCH] Fix some typos in comments
Posted by Andrew Cooper 2 months, 3 weeks ago
On 01/02/2024 2:59 pm, Frediano Ziglio wrote:
> Signed-off-by: Frediano Ziglio <frediano.ziglio@cloud.com>
> ---
>  arch/x86/include/arch/processor.h | 2 +-
>  include/xen/hvm/params.h          | 2 +-
>  include/xtf/console.h             | 2 +-
>  include/xtf/extable.h             | 4 ++--
>  4 files changed, 5 insertions(+), 5 deletions(-)

This looks like an XTF patch?

Considering that I do try to remember to spell check before committing,
I'm somewhat embarrassed by this.

~Andrew
Re: [PATCH] Fix some typos in comments
Posted by Andrew Cooper 2 months, 3 weeks ago
On 01/02/2024 3:07 pm, Andrew Cooper wrote:
> On 01/02/2024 2:59 pm, Frediano Ziglio wrote:
>> Signed-off-by: Frediano Ziglio <frediano.ziglio@cloud.com>
>> ---
>>  arch/x86/include/arch/processor.h | 2 +-
>>  include/xen/hvm/params.h          | 2 +-
>>  include/xtf/console.h             | 2 +-
>>  include/xtf/extable.h             | 4 ++--
>>  4 files changed, 5 insertions(+), 5 deletions(-)
> This looks like an XTF patch?

Given that it applies cleanly, that will be a yes.

Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com> and committed.

Thanks.