[PATCH v1 00/21] Trivial tree wide typo fixes

[PATCH v1 00/21] Trivial tree wide typo fixes

[Manos Pitsidianakis]

Correct typos automatically found with the `typos` tool
<https://crates.io/crates/typos>

Manos Pitsidianakis (21):
  docs: correct typos
  tests: correct typos
  Xen headers: correct typos
  accel/tcg: correct typos
  loongson3: correct typos
  ppc: correct typos
  sh4: correct typos
  include/exec/memory.h: correct typos
  include/exec/plugin-gen.h: correct typos
  hw/arm/omap.h: correct typos
  hw/cxl/cxl_device.h: correct typos
  hw/net/npcm_gmac.h: correct typos
  hw/riscv/virt.h: correct typos
  pc-bios/README: correct typos
  qapi/ui: correct typos
  qemu-options.hx: correct typos
  ci/gitlab-pipeline-status: correct typos
  hexagon: correct typos
  m68k: correct typos
  s390x: correct typos
  target/sparc: correct typos

 include/exec/memory.h                          | 2 +-
 include/exec/plugin-gen.h                      | 2 +-
 include/hw/arm/omap.h                          | 2 +-
 include/hw/cxl/cxl_device.h                    | 4 ++--
 include/hw/net/npcm_gmac.h                     | 4 ++--
 include/hw/ppc/ppc4xx.h                        | 2 +-
 include/hw/riscv/virt.h                        | 4 ++--
 include/hw/xen/interface/arch-x86/xen-x86_64.h | 2 +-
 include/hw/xen/interface/arch-x86/xen.h        | 2 +-
 include/hw/xen/interface/event_channel.h       | 2 +-
 include/hw/xen/interface/grant_table.h         | 2 +-
 include/hw/xen/interface/hvm/hvm_op.h          | 2 +-
 include/hw/xen/interface/io/blkif.h            | 4 ++--
 include/hw/xen/interface/io/fbif.h             | 2 +-
 include/hw/xen/interface/io/kbdif.h            | 2 +-
 include/hw/xen/interface/io/ring.h             | 2 +-
 include/hw/xen/interface/memory.h              | 2 +-
 include/hw/xen/interface/physdev.h             | 4 ++--
 include/hw/xen/interface/xen.h                 | 4 ++--
 accel/tcg/ldst_atomicity.c.inc                 | 2 +-
 docs/devel/ci-jobs.rst.inc                     | 2 +-
 docs/devel/docs.rst                            | 2 +-
 docs/devel/testing.rst                         | 2 +-
 docs/interop/prl-xml.txt                       | 2 +-
 docs/interop/vhost-user.rst                    | 2 +-
 docs/system/devices/canokey.rst                | 2 +-
 hw/mips/loongson3_bootp.h                      | 4 ++--
 hw/ppc/ppc405.h                                | 2 +-
 hw/sh4/sh7750_regs.h                           | 2 +-
 pc-bios/README                                 | 2 +-
 qapi/ui.json                                   | 4 ++--
 qemu-options.hx                                | 4 ++--
 scripts/ci/gitlab-pipeline-status              | 2 +-
 target/hexagon/idef-parser/macros.inc          | 2 +-
 target/m68k/cpu.h                              | 7 ++++---
 target/ppc/translate/vmx-impl.c.inc            | 2 +-
 target/s390x/cpu_features_def.h.inc            | 2 +-
 target/sparc/asi.h                             | 8 ++++----
 tests/avocado/acpi-bits/bits-tests/smbios.py2  | 2 +-
 tests/avocado/mem-addr-space-check.py          | 6 +++---
 tests/avocado/reverse_debugging.py             | 2 +-
 41 files changed, 58 insertions(+), 57 deletions(-)


base-commit: da96ad4a6a2ef26c83b15fa95e7fceef5147269c
-- 
γαῖα πυρί μιχθήτω

[PATCH v1 01/21] docs: correct typos

[Manos Pitsidianakis]

Correct typos automatically found with the `typos` tool
<https://crates.io/crates/typos>

Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 docs/devel/ci-jobs.rst.inc      | 2 +-
 docs/devel/docs.rst             | 2 +-
 docs/devel/testing.rst          | 2 +-
 docs/interop/prl-xml.txt        | 2 +-
 docs/interop/vhost-user.rst     | 2 +-
 docs/system/devices/canokey.rst | 2 +-
 6 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/docs/devel/ci-jobs.rst.inc b/docs/devel/ci-jobs.rst.inc
index 4c39cdb2d9..6678b4f4ef 100644
--- a/docs/devel/ci-jobs.rst.inc
+++ b/docs/devel/ci-jobs.rst.inc
@@ -147,7 +147,7 @@ Set this variable to 1 to create the pipelines, but leave all
 the jobs to be manually started from the UI
 
 Set this variable to 2 to create the pipelines and run all
-the jobs immediately, as was historicaly behaviour
+the jobs immediately, as was historically behaviour
 
 QEMU_CI_AVOCADO_TESTING
 ~~~~~~~~~~~~~~~~~~~~~~~
diff --git a/docs/devel/docs.rst b/docs/devel/docs.rst
index 50ff0d67f8..a7768b5311 100644
--- a/docs/devel/docs.rst
+++ b/docs/devel/docs.rst
@@ -21,7 +21,7 @@ are processed in two ways:
 
 The syntax of these ``.hx`` files is simple. It is broadly an
 alternation of C code put into the C output and rST format text
-put into the documention. A few special directives are recognised;
+put into the documentation. A few special directives are recognised;
 these are all-caps and must be at the beginning of the line.
 
 ``HXCOMM`` is the comment marker. The line, including any arbitrary
diff --git a/docs/devel/testing.rst b/docs/devel/testing.rst
index bd132306c1..aa96eacec5 100644
--- a/docs/devel/testing.rst
+++ b/docs/devel/testing.rst
@@ -728,7 +728,7 @@ For example to setup the HPPA ports builds of Debian::
     EXECUTABLE=(pwd)/qemu-hppa V=1
 
 The ``DEB_`` variables are substitutions used by
-``debian-boostrap.pre`` which is called to do the initial debootstrap
+``debian-bootstrap.pre`` which is called to do the initial debootstrap
 of the rootfs before it is copied into the container. The second stage
 is run as part of the build. The final image will be tagged as
 ``qemu/debian-sid-hppa``.
diff --git a/docs/interop/prl-xml.txt b/docs/interop/prl-xml.txt
index 7031f8752c..cf9b3fba26 100644
--- a/docs/interop/prl-xml.txt
+++ b/docs/interop/prl-xml.txt
@@ -122,7 +122,7 @@ Each Image element has following child elements:
     * Type - image type of the element. It can be:
              "Plain" for raw files.
              "Compressed" for expanding disks.
-    * File - path to image file. Path can be relative to DiskDecriptor.xml or
+    * File - path to image file. Path can be relative to DiskDescriptor.xml or
              absolute.
 
 == Snapshots element ==
diff --git a/docs/interop/vhost-user.rst b/docs/interop/vhost-user.rst
index ad6e142f23..d1ed39dfa0 100644
--- a/docs/interop/vhost-user.rst
+++ b/docs/interop/vhost-user.rst
@@ -989,7 +989,7 @@ When reconnecting:
 
    #. If ``d.flags`` is not equal to the calculated flags value (means
       back-end has submitted the buffer to guest driver before crash, so
-      it has to commit the in-progres update), set ``old_free_head``,
+      it has to commit the in-progress update), set ``old_free_head``,
       ``old_used_idx``, ``old_used_wrap_counter`` to ``free_head``,
       ``used_idx``, ``used_wrap_counter``
 
diff --git a/docs/system/devices/canokey.rst b/docs/system/devices/canokey.rst
index cfa6186e48..7f3664963f 100644
--- a/docs/system/devices/canokey.rst
+++ b/docs/system/devices/canokey.rst
@@ -14,7 +14,7 @@ CanoKey [1]_ is an open-source secure key with supports of
 All these platform-independent features are in canokey-core [3]_.
 
 For different platforms, CanoKey has different implementations,
-including both hardware implementions and virtual cards:
+including both hardware implementations and virtual cards:
 
 * CanoKey STM32 [4]_
 * CanoKey Pigeon [5]_
-- 
γαῖα πυρί μιχθήτω

[PATCH v1 02/21] tests: correct typos

[Manos Pitsidianakis]

Correct typos automatically found with the `typos` tool
<https://crates.io/crates/typos>

Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 tests/avocado/acpi-bits/bits-tests/smbios.py2 | 2 +-
 tests/avocado/mem-addr-space-check.py         | 6 +++---
 tests/avocado/reverse_debugging.py            | 2 +-
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/tests/avocado/acpi-bits/bits-tests/smbios.py2 b/tests/avocado/acpi-bits/bits-tests/smbios.py2
index fc623de072..5868a7137a 100644
--- a/tests/avocado/acpi-bits/bits-tests/smbios.py2
+++ b/tests/avocado/acpi-bits/bits-tests/smbios.py2
@@ -1060,7 +1060,7 @@ class EventLogDescriptor(unpack.Struct):
             0x16: 'Log Area Reset/Cleared',
             0x17: 'System boot',
             xrange(0x18, 0x7F): 'Unused, available for assignment',
-            xrange(0x80, 0xFE): 'Availalbe for system- and OEM-specific assignments',
+            xrange(0x80, 0xFE): 'Available for system- and OEM-specific assignments',
             0xFF: 'End of log'
         }
         yield 'log_type', u.unpack_one('B'), unpack.format_table("{}", _event_log_type_descriptors)
diff --git a/tests/avocado/mem-addr-space-check.py b/tests/avocado/mem-addr-space-check.py
index 363c3f12a6..af019969c0 100644
--- a/tests/avocado/mem-addr-space-check.py
+++ b/tests/avocado/mem-addr-space-check.py
@@ -165,7 +165,7 @@ def test_phybits_low_tcg_q35_70_amd(self):
         For q35-7.0 machines, "above 4G" memory starts are 4G.
         pci64_hole size is 32 GiB. Since TCG_PHYS_ADDR_BITS is defined to
         be 40, TCG emulated CPUs have maximum of 1 TiB (1024 GiB) of
-        directly addressible memory.
+        directly addressable memory.
         Hence, maxmem value at most can be
         1024 GiB - 4 GiB - 1 GiB per slot for alignment - 32 GiB + 0.5 GiB
         which is equal to 987.5 GiB. Setting the value to 988 GiB should
@@ -190,7 +190,7 @@ def test_phybits_low_tcg_q35_71_amd(self):
         AMD_HT_START is defined to be at 1012 GiB. So for q35 machines
         version > 7.0 and AMD cpus, instead of 1024 GiB limit for 40 bit
         processor address space, it has to be 1012 GiB , that is 12 GiB
-        less than the case above in order to accomodate HT hole.
+        less than the case above in order to accommodate HT hole.
         Make sure QEMU fails when maxmem size is 976 GiB (12 GiB less
         than 988 GiB).
         """
@@ -297,7 +297,7 @@ def test_phybits_ok_tcg_q35_71_amd_41bits(self):
         :avocado: tags=arch:x86_64
 
         AMD processor with 41 bits. Max cpu hw address = 2 TiB.
-        Same as above but by setting maxram beween 976 GiB and 992 Gib,
+        Same as above but by setting maxram between 976 GiB and 992 Gib,
         QEMU should start fine.
         """
         self.vm.add_args('-S', '-cpu', 'EPYC-v4,phys-bits=41',
diff --git a/tests/avocado/reverse_debugging.py b/tests/avocado/reverse_debugging.py
index 4cce5a5598..92855a02a5 100644
--- a/tests/avocado/reverse_debugging.py
+++ b/tests/avocado/reverse_debugging.py
@@ -191,7 +191,7 @@ def reverse_debugging(self, shift=7, args=None):
         self.check_pc(g, steps[-1])
         logger.info('successfully reached %x' % steps[-1])
 
-        logger.info('exitting gdb and qemu')
+        logger.info('exiting gdb and qemu')
         vm.shutdown()
 
 class ReverseDebugging_X86_64(ReverseDebugging):
-- 
γαῖα πυρί μιχθήτω

[PATCH v1 03/21] Xen headers: correct typos

[Manos Pitsidianakis]

Correct typos automatically found with the `typos` tool
<https://crates.io/crates/typos>

Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 include/hw/xen/interface/arch-x86/xen-x86_64.h | 2 +-
 include/hw/xen/interface/arch-x86/xen.h        | 2 +-
 include/hw/xen/interface/event_channel.h       | 2 +-
 include/hw/xen/interface/grant_table.h         | 2 +-
 include/hw/xen/interface/hvm/hvm_op.h          | 2 +-
 include/hw/xen/interface/io/blkif.h            | 4 ++--
 include/hw/xen/interface/io/fbif.h             | 2 +-
 include/hw/xen/interface/io/kbdif.h            | 2 +-
 include/hw/xen/interface/io/ring.h             | 2 +-
 include/hw/xen/interface/memory.h              | 2 +-
 include/hw/xen/interface/physdev.h             | 4 ++--
 include/hw/xen/interface/xen.h                 | 4 ++--
 12 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/include/hw/xen/interface/arch-x86/xen-x86_64.h b/include/hw/xen/interface/arch-x86/xen-x86_64.h
index 5d9035ed22..3e94eeb0bf 100644
--- a/include/hw/xen/interface/arch-x86/xen-x86_64.h
+++ b/include/hw/xen/interface/arch-x86/xen-x86_64.h
@@ -89,7 +89,7 @@
  *   RING1 -> RING3 kernel mode.
  *   RING2 -> RING3 kernel mode.
  *   RING3 -> RING3 user mode.
- * However RING0 indicates that the guest kernel should return to iteself
+ * However RING0 indicates that the guest kernel should return to itself
  * directly with
  *      orb   $3,1*8(%rsp)
  *      iretq
diff --git a/include/hw/xen/interface/arch-x86/xen.h b/include/hw/xen/interface/arch-x86/xen.h
index c0f4551247..323bd06a63 100644
--- a/include/hw/xen/interface/arch-x86/xen.h
+++ b/include/hw/xen/interface/arch-x86/xen.h
@@ -156,7 +156,7 @@ typedef uint64_t tsc_timestamp_t; /* RDTSC timestamp */
  * information in this structure is updated, the fields read include: fpu_ctxt
  * (if VGCT_I387_VALID is set), flags, user_regs and debugreg[*].
  *
- * Note: VCPUOP_initialise for HVM guests is non-symetric with
+ * Note: VCPUOP_initialise for HVM guests is non-symmetric with
  * DOMCTL_setvcpucontext, and uses struct vcpu_hvm_context from hvm/hvm_vcpu.h
  */
 struct vcpu_guest_context {
diff --git a/include/hw/xen/interface/event_channel.h b/include/hw/xen/interface/event_channel.h
index 0d91a1c4af..d446863230 100644
--- a/include/hw/xen/interface/event_channel.h
+++ b/include/hw/xen/interface/event_channel.h
@@ -302,7 +302,7 @@ typedef struct evtchn_set_priority evtchn_set_priority_t;
  * ` enum neg_errnoval
  * ` HYPERVISOR_event_channel_op_compat(struct evtchn_op *op)
  * `
- * Superceded by new event_channel_op() hypercall since 0x00030202.
+ * Superseded by new event_channel_op() hypercall since 0x00030202.
  */
 struct evtchn_op {
     uint32_t cmd; /* enum event_channel_op */
diff --git a/include/hw/xen/interface/grant_table.h b/include/hw/xen/interface/grant_table.h
index 1dfa17a6d0..7652e8bf81 100644
--- a/include/hw/xen/interface/grant_table.h
+++ b/include/hw/xen/interface/grant_table.h
@@ -607,7 +607,7 @@ DEFINE_XEN_GUEST_HANDLE(gnttab_cache_flush_t);
  /*
   * GNTMAP_contains_pte subflag:
   *  0 => This map request contains a host virtual address.
-  *  1 => This map request contains the machine addess of the PTE to update.
+  *  1 => This map request contains the machine address of the PTE to update.
   */
 #define _GNTMAP_contains_pte    (4)
 #define GNTMAP_contains_pte     (1<<_GNTMAP_contains_pte)
diff --git a/include/hw/xen/interface/hvm/hvm_op.h b/include/hw/xen/interface/hvm/hvm_op.h
index e22adf0319..3defe1c108 100644
--- a/include/hw/xen/interface/hvm/hvm_op.h
+++ b/include/hw/xen/interface/hvm/hvm_op.h
@@ -337,7 +337,7 @@ struct xen_hvm_altp2m_op {
 #define HVMOP_altp2m_vcpu_disable_notify  13
 /* Get the active vcpu p2m index */
 #define HVMOP_altp2m_get_p2m_idx          14
-/* Set the "Supress #VE" bit for a range of pages */
+/* Set the "Suppress #VE" bit for a range of pages */
 #define HVMOP_altp2m_set_suppress_ve_multi 15
 /* Set visibility for a given altp2m view */
 #define HVMOP_altp2m_set_visibility       16
diff --git a/include/hw/xen/interface/io/blkif.h b/include/hw/xen/interface/io/blkif.h
index 22f1eef0c0..4356956975 100644
--- a/include/hw/xen/interface/io/blkif.h
+++ b/include/hw/xen/interface/io/blkif.h
@@ -42,7 +42,7 @@
  * All data in the XenStore is stored as strings.  Nodes specifying numeric
  * values are encoded in decimal.  Integer value ranges listed below are
  * expressed as fixed sized integer types capable of storing the conversion
- * of a properly formated node string, without loss of information.
+ * of a properly formatted node string, without loss of information.
  *
  * Any specified default value is in effect if the corresponding XenBus node
  * is not present in the XenStore.
@@ -406,7 +406,7 @@
  *     further requests may reuse these grants and require write permissions.
  * (9) Linux implementation doesn't have a limit on the maximum number of
  *     grants that can be persistently mapped in the frontend driver, but
- *     due to the frontent driver implementation it should never be bigger
+ *     due to the frontend driver implementation it should never be bigger
  *     than RING_SIZE * BLKIF_MAX_SEGMENTS_PER_REQUEST.
  *(10) The discard-secure property may be present and will be set to 1 if the
  *     backing device supports secure discard.
diff --git a/include/hw/xen/interface/io/fbif.h b/include/hw/xen/interface/io/fbif.h
index 93c73195d8..46046f6775 100644
--- a/include/hw/xen/interface/io/fbif.h
+++ b/include/hw/xen/interface/io/fbif.h
@@ -72,7 +72,7 @@ union xenfb_out_event
  * just ignore it.  Frontends that use the advice should immediately
  * refresh the framebuffer (and send an update notification event if
  * those have been requested), then use the update frequency to guide
- * their periodical refreshs.
+ * their periodical refresh.
  */
 #define XENFB_TYPE_REFRESH_PERIOD 1
 #define XENFB_NO_REFRESH 0
diff --git a/include/hw/xen/interface/io/kbdif.h b/include/hw/xen/interface/io/kbdif.h
index 4bde6b3821..5794cc33f8 100644
--- a/include/hw/xen/interface/io/kbdif.h
+++ b/include/hw/xen/interface/io/kbdif.h
@@ -22,7 +22,7 @@
  * All data in XenStore is stored as strings.  Nodes specifying numeric
  * values are encoded in decimal. Integer value ranges listed below are
  * expressed as fixed sized integer types capable of storing the conversion
- * of a properly formated node string, without loss of information.
+ * of a properly formatted node string, without loss of information.
  *
  *****************************************************************************
  *                            Backend XenBus Nodes
diff --git a/include/hw/xen/interface/io/ring.h b/include/hw/xen/interface/io/ring.h
index 025939278b..6d8f63d7a5 100644
--- a/include/hw/xen/interface/io/ring.h
+++ b/include/hw/xen/interface/io/ring.h
@@ -145,7 +145,7 @@ typedef struct __name##_back_ring __name##_back_ring_t
  *
  * N.B. these macros do NO INTERLOCKS OR FLOW CONTROL.
  * This is OK in 1-for-1 request-response situations where the
- * requestor (front end) never has more than RING_SIZE()-1
+ * requester (front end) never has more than RING_SIZE()-1
  * outstanding requests.
  */
 
diff --git a/include/hw/xen/interface/memory.h b/include/hw/xen/interface/memory.h
index 29cf5c8239..ad11573d90 100644
--- a/include/hw/xen/interface/memory.h
+++ b/include/hw/xen/interface/memory.h
@@ -167,7 +167,7 @@ struct xen_machphys_mfn_list {
 
     /*
      * Pointer to buffer to fill with list of extent starts. If there are
-     * any large discontiguities in the machine address space, 2MB gaps in
+     * any large discontinuities in the machine address space, 2MB gaps in
      * the machphys table will be represented by an MFN base of zero.
      */
     XEN_GUEST_HANDLE(xen_pfn_t) extent_start;
diff --git a/include/hw/xen/interface/physdev.h b/include/hw/xen/interface/physdev.h
index f0c0d4727c..053c045cf5 100644
--- a/include/hw/xen/interface/physdev.h
+++ b/include/hw/xen/interface/physdev.h
@@ -206,7 +206,7 @@ typedef struct physdev_manage_pci_ext physdev_manage_pci_ext_t;
 DEFINE_XEN_GUEST_HANDLE(physdev_manage_pci_ext_t);
 
 /*
- * Argument to physdev_op_compat() hypercall. Superceded by new physdev_op()
+ * Argument to physdev_op_compat() hypercall. Superseded by new physdev_op()
  * hypercall since 0x00030202.
  */
 struct physdev_op {
@@ -332,7 +332,7 @@ DEFINE_XEN_GUEST_HANDLE(physdev_dbgp_op_t);
 
 #if __XEN_INTERFACE_VERSION__ < 0x00040600
 /*
- * These all-capitals physdev operation names are superceded by the new names
+ * These all-capitals physdev operation names are superseded by the new names
  * (defined above) since interface version 0x00030202. The guard above was
  * added post-4.5 only though and hence shouldn't check for 0x00030202.
  */
diff --git a/include/hw/xen/interface/xen.h b/include/hw/xen/interface/xen.h
index 920567e006..b8fe746f78 100644
--- a/include/hw/xen/interface/xen.h
+++ b/include/hw/xen/interface/xen.h
@@ -733,8 +733,8 @@ struct shared_info {
      * are delivered by this mechanism:
      *  1. Bi-directional inter- and intra-domain connections. Domains must
      *     arrange out-of-band to set up a connection (usually by allocating
-     *     an unbound 'listener' port and avertising that via a storage service
-     *     such as xenstore).
+     *     an unbound 'listener' port and advertising that via a storage
+     *     service such as xenstore).
      *  2. Physical interrupts. A domain with suitable hardware-access
      *     privileges can bind an event-channel port to a physical interrupt
      *     source.
-- 
γαῖα πυρί μιχθήτω

[PATCH v1 04/21] accel/tcg: correct typos

[Manos Pitsidianakis]

Correct typos automatically found with the `typos` tool
<https://crates.io/crates/typos>

Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 accel/tcg/ldst_atomicity.c.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/accel/tcg/ldst_atomicity.c.inc b/accel/tcg/ldst_atomicity.c.inc
index 33a04dec52..97dae70d53 100644
--- a/accel/tcg/ldst_atomicity.c.inc
+++ b/accel/tcg/ldst_atomicity.c.inc
@@ -76,7 +76,7 @@ static int required_atomicity(CPUState *cpu, uintptr_t p, MemOp memop)
         /*
          * Examine the alignment of p to determine if there are subobjects
          * that must be aligned.  Note that we only really need ctz4() --
-         * any more sigificant bits are discarded by the immediately
+         * any more significant bits are discarded by the immediately
          * following comparison.
          */
         tmp = ctz32(p);
-- 
γαῖα πυρί μιχθήτω

[PATCH v1 05/21] loongson3: correct typos

[Manos Pitsidianakis]

Correct typos automatically found with the `typos` tool
<https://crates.io/crates/typos>

Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 hw/mips/loongson3_bootp.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/hw/mips/loongson3_bootp.h b/hw/mips/loongson3_bootp.h
index d525ab745a..1b0dd3b591 100644
--- a/hw/mips/loongson3_bootp.h
+++ b/hw/mips/loongson3_bootp.h
@@ -25,7 +25,7 @@
 struct efi_memory_map_loongson {
     uint16_t vers;               /* version of efi_memory_map */
     uint32_t nr_map;             /* number of memory_maps */
-    uint32_t mem_freq;           /* memory frequence */
+    uint32_t mem_freq;           /* memory frequency */
     struct mem_map {
         uint32_t node_id;        /* node_id which memory attached to */
         uint32_t mem_type;       /* system memory, pci memory, pci io, etc. */
@@ -156,7 +156,7 @@ struct board_devices {
 
 struct loongson_special_attribute {
     uint16_t vers;               /* version of this special */
-    char special_name[64];       /* special_atribute_name */
+    char special_name[64];       /* special_attribute_name */
     uint32_t loongson_special_type; /* type of special device */
     /* for each device's resource */
     struct resource_loongson resource[MAX_RESOURCE_NUMBER];
-- 
γαῖα πυρί μιχθήτω

[PATCH v1 06/21] ppc: correct typos

[Manos Pitsidianakis]

Correct typos automatically found with the `typos` tool
<https://crates.io/crates/typos>

Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 include/hw/ppc/ppc4xx.h             | 2 +-
 hw/ppc/ppc405.h                     | 2 +-
 target/ppc/translate/vmx-impl.c.inc | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/include/hw/ppc/ppc4xx.h b/include/hw/ppc/ppc4xx.h
index ea7740239b..c4ecb1652f 100644
--- a/include/hw/ppc/ppc4xx.h
+++ b/include/hw/ppc/ppc4xx.h
@@ -75,7 +75,7 @@ struct Ppc4xxMalState {
     uint8_t  rxcnum;
 };
 
-/* Peripheral local bus arbitrer */
+/* Peripheral local bus arbiter */
 #define TYPE_PPC4xx_PLB "ppc4xx-plb"
 OBJECT_DECLARE_SIMPLE_TYPE(Ppc4xxPlbState, PPC4xx_PLB);
 struct Ppc4xxPlbState {
diff --git a/hw/ppc/ppc405.h b/hw/ppc/ppc405.h
index 9a4312691e..a39f0caea1 100644
--- a/hw/ppc/ppc405.h
+++ b/hw/ppc/ppc405.h
@@ -41,7 +41,7 @@ struct Ppc405PobState {
     uint32_t besr1;
 };
 
-/* OPB arbitrer */
+/* OPB arbiter */
 #define TYPE_PPC405_OPBA "ppc405-opba"
 OBJECT_DECLARE_SIMPLE_TYPE(Ppc405OpbaState, PPC405_OPBA);
 struct Ppc405OpbaState {
diff --git a/target/ppc/translate/vmx-impl.c.inc b/target/ppc/translate/vmx-impl.c.inc
index 4b91c3489d..b56e615c24 100644
--- a/target/ppc/translate/vmx-impl.c.inc
+++ b/target/ppc/translate/vmx-impl.c.inc
@@ -1183,7 +1183,7 @@ static void glue(gen_, name)(DisasContext *ctx)                         \
 
 /*
  * Support for Altivec instructions that use bit 31 (Rc) as an opcode
- * bit but also use bit 21 as an actual Rc bit.  In general, thse pairs
+ * bit but also use bit 21 as an actual Rc bit.  In general, these pairs
  * come from different versions of the ISA, so we must also support a
  * pair of flags for each instruction.
  */
-- 
γαῖα πυρί μιχθήτω

[PATCH v1 07/21] sh4: correct typos

[Manos Pitsidianakis]

Correct typos automatically found with the `typos` tool
<https://crates.io/crates/typos>

Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 hw/sh4/sh7750_regs.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hw/sh4/sh7750_regs.h b/hw/sh4/sh7750_regs.h
index edb5d18f00..946ad7b3aa 100644
--- a/hw/sh4/sh7750_regs.h
+++ b/hw/sh4/sh7750_regs.h
@@ -172,7 +172,7 @@
 
 
 /*
- * Exeption-related registers
+ * Exception-related registers
  */
 
 /* Immediate data for TRAPA instruction - TRA */
-- 
γαῖα πυρί μιχθήτω

[PATCH v1 08/21] include/exec/memory.h: correct typos

[Manos Pitsidianakis]

Correct typos automatically found with the `typos` tool
<https://crates.io/crates/typos>

Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 include/exec/memory.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/exec/memory.h b/include/exec/memory.h
index 177be23db7..8626a355b3 100644
--- a/include/exec/memory.h
+++ b/include/exec/memory.h
@@ -3146,7 +3146,7 @@ int ram_block_discard_require(bool state);
 
 /*
  * See ram_block_discard_require(): only inhibit technologies that disable
- * uncoordinated discarding of pages in RAM blocks, allowing co-existance with
+ * uncoordinated discarding of pages in RAM blocks, allowing co-existence with
  * technologies that only inhibit uncoordinated discards (via the
  * RamDiscardManager).
  */
-- 
γαῖα πυρί μιχθήτω

[PATCH v1 09/21] include/exec/plugin-gen.h: correct typos

[Manos Pitsidianakis]

Correct typos automatically found with the `typos` tool
<https://crates.io/crates/typos>

Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 include/exec/plugin-gen.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/exec/plugin-gen.h b/include/exec/plugin-gen.h
index c4552b5061..b18edd6ab4 100644
--- a/include/exec/plugin-gen.h
+++ b/include/exec/plugin-gen.h
@@ -19,7 +19,7 @@ struct DisasContextBase;
 #ifdef CONFIG_PLUGIN
 
 bool plugin_gen_tb_start(CPUState *cpu, const struct DisasContextBase *db,
-                         bool supress);
+                         bool suppress);
 void plugin_gen_tb_end(CPUState *cpu, size_t num_insns);
 void plugin_gen_insn_start(CPUState *cpu, const struct DisasContextBase *db);
 void plugin_gen_insn_end(void);
-- 
γαῖα πυρί μιχθήτω

[PATCH v1 10/21] hw/arm/omap.h: correct typos

[Manos Pitsidianakis]

Correct typos automatically found with the `typos` tool
<https://crates.io/crates/typos>

Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 include/hw/arm/omap.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/hw/arm/omap.h b/include/hw/arm/omap.h
index 067e9419f7..2f59220c0e 100644
--- a/include/hw/arm/omap.h
+++ b/include/hw/arm/omap.h
@@ -1008,7 +1008,7 @@ void omap_mpu_wakeup(void *opaque, int irq, int req);
                       __func__, paddr)
 
 /* OMAP-specific Linux bootloader tags for the ATAG_BOARD area
-   (Board-specifc tags are not here)  */
+   (Board-specific tags are not here)  */
 #define OMAP_TAG_CLOCK		0x4f01
 #define OMAP_TAG_MMC		0x4f02
 #define OMAP_TAG_SERIAL_CONSOLE	0x4f03
-- 
γαῖα πυρί μιχθήτω

[PATCH v1 11/21] hw/cxl/cxl_device.h: correct typos

[Manos Pitsidianakis]

Correct typos automatically found with the `typos` tool
<https://crates.io/crates/typos>

Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 include/hw/cxl/cxl_device.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/hw/cxl/cxl_device.h b/include/hw/cxl/cxl_device.h
index d8e184c4ba..279b276bda 100644
--- a/include/hw/cxl/cxl_device.h
+++ b/include/hw/cxl/cxl_device.h
@@ -268,7 +268,7 @@ void cxl_event_set_status(CXLDeviceState *cxl_dstate, CXLEventLogType log_type,
 /*
  * Helper macro to initialize capability headers for CXL devices.
  *
- * In CXL r3.1 Section 8.2.8.2: CXL Device Capablity Header Register, this is
+ * In CXL r3.1 Section 8.2.8.2: CXL Device Capability Header Register, this is
  * listed as a 128b register, but in CXL r3.1 Section 8.2.8: CXL Device Register
  * Interface, it says:
  * > No registers defined in Section 8.2.8 are larger than 64-bits wide so that
@@ -276,7 +276,7 @@ void cxl_event_set_status(CXLDeviceState *cxl_dstate, CXLEventLogType log_type,
  * > followed, the behavior is undefined.
  *
  * > To illustrate how the fields fit together, the layouts ... are shown as
- * > wider than a 64 bit register. Implemenations are expected to use any size
+ * > wider than a 64 bit register. Implementations are expected to use any size
  * > accesses for this information up to 64 bits without lost of functionality
  *
  * Here we've chosen to make it 4 dwords.
-- 
γαῖα πυρί μιχθήτω

[PATCH v1 12/21] hw/net/npcm_gmac.h: correct typos

[Manos Pitsidianakis]

Correct typos automatically found with the `typos` tool
<https://crates.io/crates/typos>

Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 include/hw/net/npcm_gmac.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/hw/net/npcm_gmac.h b/include/hw/net/npcm_gmac.h
index f2d9f08ec1..6340ffe92c 100644
--- a/include/hw/net/npcm_gmac.h
+++ b/include/hw/net/npcm_gmac.h
@@ -81,7 +81,7 @@ struct NPCMGMACRxDesc {
 
 /* Disable Interrupt on Completion */
 #define RX_DESC_RDES1_DIS_INTR_COMP_MASK BIT(31)
-/* Recieve end of ring */
+/* Receive end of ring */
 #define RX_DESC_RDES1_RC_END_RING_MASK BIT(25)
 /* Second Address Chained */
 #define RX_DESC_RDES1_SEC_ADDR_CHND_MASK BIT(24)
@@ -213,7 +213,7 @@ OBJECT_DECLARE_SIMPLE_TYPE(NPCMGMACState, NPCM_GMAC)
 #define NPCM_DMA_STATUS_FBI BIT(13)
 /* Early transmit Interrupt */
 #define NPCM_DMA_STATUS_ETI BIT(10)
-/* Receive Watchdog Timout */
+/* Receive Watchdog Timeout */
 #define NPCM_DMA_STATUS_RWT BIT(9)
 /* Receive Process Stopped */
 #define NPCM_DMA_STATUS_RPS BIT(8)
-- 
γαῖα πυρί μιχθήτω

[PATCH v1 13/21] hw/riscv/virt.h: correct typos

[Manos Pitsidianakis]

Correct typos automatically found with the `typos` tool
<https://crates.io/crates/typos>

Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 include/hw/riscv/virt.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/hw/riscv/virt.h b/include/hw/riscv/virt.h
index f89790fd58..3db839160f 100644
--- a/include/hw/riscv/virt.h
+++ b/include/hw/riscv/virt.h
@@ -144,13 +144,13 @@ uint32_t imsic_num_bits(uint32_t count);
 #define VIRT_IMSIC_GROUP_MAX_SIZE      (1U << IMSIC_MMIO_GROUP_MIN_SHIFT)
 #if VIRT_IMSIC_GROUP_MAX_SIZE < \
     IMSIC_GROUP_SIZE(VIRT_CPUS_MAX_BITS, VIRT_IRQCHIP_MAX_GUESTS_BITS)
-#error "Can't accomodate single IMSIC group in address space"
+#error "Can't accommodate single IMSIC group in address space"
 #endif
 
 #define VIRT_IMSIC_MAX_SIZE            (VIRT_SOCKETS_MAX * \
                                         VIRT_IMSIC_GROUP_MAX_SIZE)
 #if 0x4000000 < VIRT_IMSIC_MAX_SIZE
-#error "Can't accomodate all IMSIC groups in address space"
+#error "Can't accommodate all IMSIC groups in address space"
 #endif
 
 #endif
-- 
γαῖα πυρί μιχθήτω

[PATCH v1 14/21] pc-bios/README: correct typos

[Manos Pitsidianakis]

Correct typos automatically found with the `typos` tool
<https://crates.io/crates/typos>

Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 pc-bios/README | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pc-bios/README b/pc-bios/README
index 4189bb28cc..b8a0210d24 100644
--- a/pc-bios/README
+++ b/pc-bios/README
@@ -67,7 +67,7 @@
   and enable the use of well-known bootloaders such as U-Boot.
   OpenSBI is distributed under the terms of the BSD 2-clause license
   ("Simplified BSD License" or "FreeBSD License", SPDX: BSD-2-Clause). OpenSBI
-  source code also contains code reused from other projects desribed here:
+  source code also contains code reused from other projects described here:
   https://github.com/riscv/opensbi/blob/master/ThirdPartyNotices.md.
 
 - npcm7xx_bootrom.bin is a simplified, free (Apache 2.0) boot ROM for Nuvoton
-- 
γαῖα πυρί μιχθήτω

[PATCH v1 15/21] qapi/ui: correct typos

[Manos Pitsidianakis]

Correct typos automatically found with the `typos` tool
<https://crates.io/crates/typos>

Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 qapi/ui.json | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/qapi/ui.json b/qapi/ui.json
index b6d7e142b7..1448eaca73 100644
--- a/qapi/ui.json
+++ b/qapi/ui.json
@@ -63,7 +63,7 @@
 ##
 # @SetPasswordOptionsVnc:
 #
-# Options for set_password specific to the VNC procotol.
+# Options for set_password specific to the VNC protocol.
 #
 # @display: The id of the display where the password should be
 #     changed.  Defaults to the first.
@@ -125,7 +125,7 @@
 ##
 # @ExpirePasswordOptionsVnc:
 #
-# Options for expire_password specific to the VNC procotol.
+# Options for expire_password specific to the VNC protocol.
 #
 # @display: The id of the display where the expiration should be
 #     changed.  Defaults to the first.
-- 
γαῖα πυρί μιχθήτω

[PATCH v1 16/21] qemu-options.hx: correct typos

[Manos Pitsidianakis]

Correct typos automatically found with the `typos` tool
<https://crates.io/crates/typos>

Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 qemu-options.hx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/qemu-options.hx b/qemu-options.hx
index 8547254dbf..9be1e5817c 100644
--- a/qemu-options.hx
+++ b/qemu-options.hx
@@ -2468,7 +2468,7 @@ SRST
 
     ``to=L``
         With this option, QEMU will try next available VNC displays,
-        until the number L, if the origianlly defined "-vnc display" is
+        until the number L, if the originally defined "-vnc display" is
         not available, e.g. port 5900+display is already used by another
         application. By default, to=0.
 
@@ -5511,7 +5511,7 @@ SRST
             -object filter-redirector,netdev=hn0,id=redire0,queue=rx,indev=compare_out
             -object filter-redirector,netdev=hn0,id=redire1,queue=rx,outdev=compare0
             -object iothread,id=iothread1
-            -object colo-compare,id=comp0,primary_in=compare0-0,secondary_in=compare1,outdev=compare_out0,notify_dev=nofity_way,iothread=iothread1
+            -object colo-compare,id=comp0,primary_in=compare0-0,secondary_in=compare1,outdev=compare_out0,notify_dev=notify_way,iothread=iothread1
 
             secondary:
             -netdev tap,id=hn0,vhost=off
-- 
γαῖα πυρί μιχθήτω

[PATCH v1 17/21] ci/gitlab-pipeline-status: correct typos

[Manos Pitsidianakis]

Correct typos automatically found with the `typos` tool
<https://crates.io/crates/typos>

Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 scripts/ci/gitlab-pipeline-status | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/ci/gitlab-pipeline-status b/scripts/ci/gitlab-pipeline-status
index e3343b0510..39f3c22c66 100755
--- a/scripts/ci/gitlab-pipeline-status
+++ b/scripts/ci/gitlab-pipeline-status
@@ -131,7 +131,7 @@ def create_parser():
                               'checks of the pipeline status.  Defaults '
                               'to %(default)s'))
     parser.add_argument('-w', '--wait', action='store_true', default=False,
-                        help=('Wether to wait, instead of checking only once '
+                        help=('Whether to wait, instead of checking only once '
                               'the status of a pipeline'))
     parser.add_argument('-p', '--project-id', type=int, default=11167699,
                         help=('The GitLab project ID. Defaults to the project '
-- 
γαῖα πυρί μιχθήτω

[PATCH v1 18/21] hexagon: correct typos

[Manos Pitsidianakis]

Correct typos automatically found with the `typos` tool
<https://crates.io/crates/typos>

Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 target/hexagon/idef-parser/macros.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/target/hexagon/idef-parser/macros.inc b/target/hexagon/idef-parser/macros.inc
index 7478d4db17..94975d9583 100644
--- a/target/hexagon/idef-parser/macros.inc
+++ b/target/hexagon/idef-parser/macros.inc
@@ -127,5 +127,5 @@
 /* Include fHIDE macros which hide type declarations */
 #define fHIDE(A) A
 
-/* Purge non-relavant parts */
+/* Purge non-relevant parts */
 #define fBRANCH_SPECULATE_STALL(A, B, C, D, E)
-- 
γαῖα πυρί μιχθήτω

[PATCH v1 19/21] m68k: correct typos

[Manos Pitsidianakis]

Correct typos automatically found with the `typos` tool
<https://crates.io/crates/typos>

Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 target/m68k/cpu.h | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/target/m68k/cpu.h b/target/m68k/cpu.h
index aca4aa610b..57e3b6d3ce 100644
--- a/target/m68k/cpu.h
+++ b/target/m68k/cpu.h
@@ -478,10 +478,11 @@ void do_m68k_semihosting(CPUM68KState *env, int nr);
  * The 68000 family is defined in six main CPU classes, the 680[012346]0.
  * Generally each successive CPU adds enhanced data/stack/instructions.
  * However, some features are only common to one, or a few classes.
- * The features covers those subsets of instructons.
+ * The features covers those subsets of instructions.
  *
- * CPU32/32+ are basically 680010 compatible with some 68020 class instructons,
- * and some additional CPU32 instructions. Mostly Supervisor state differences.
+ * CPU32/32+ are basically 680010 compatible with some 68020 class
+ * instructions, and some additional CPU32 instructions. Mostly Supervisor
+ * state differences.
  *
  * The ColdFire core ISA is a RISC-style reduction of the 68000 series cpu.
  * There are 4 ColdFire core ISA revisions: A, A+, B and C.
-- 
γαῖα πυρί μιχθήτω

[PATCH v1 20/21] s390x: correct typos

[Manos Pitsidianakis]

Correct typos automatically found with the `typos` tool
<https://crates.io/crates/typos>

Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 target/s390x/cpu_features_def.h.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/target/s390x/cpu_features_def.h.inc b/target/s390x/cpu_features_def.h.inc
index e68da9b8ff..c53ac13352 100644
--- a/target/s390x/cpu_features_def.h.inc
+++ b/target/s390x/cpu_features_def.h.inc
@@ -142,7 +142,7 @@ DEF_FEAT(SIE_CEI, "cei", SCLP_CPU, 43, "SIE: Conditional-external-interception f
 
 /*
  * Features exposed via no feature bit (but e.g., instruction sensing)
- * -> the feature bit number is irrelavant
+ * -> the feature bit number is irrelevant
  */
 DEF_FEAT(DAT_ENH_2, "dateh2", MISC, 0, "DAT-enhancement facility 2")
 DEF_FEAT(CMM, "cmm", MISC, 0, "Collaborative-memory-management facility")
-- 
γαῖα πυρί μιχθήτω

[PATCH v1 21/21] target/sparc: correct typos

[Manos Pitsidianakis]

Correct typos automatically found with the `typos` tool
<https://crates.io/crates/typos>

Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
---
 target/sparc/asi.h | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/target/sparc/asi.h b/target/sparc/asi.h
index 3270ed0c7f..a66829674b 100644
--- a/target/sparc/asi.h
+++ b/target/sparc/asi.h
@@ -145,14 +145,14 @@
  * and later ASIs.
  */
 #define ASI_REAL                0x14 /* Real address, cacheable          */
-#define ASI_PHYS_USE_EC		0x14 /* PADDR, E-cachable		*/
-#define ASI_REAL_IO             0x15 /* Real address, non-cachable      */
+#define ASI_PHYS_USE_EC		0x14 /* PADDR, E-cacheable		*/
+#define ASI_REAL_IO             0x15 /* Real address, non-cacheable      */
 #define ASI_PHYS_BYPASS_EC_E	0x15 /* PADDR, E-bit			*/
 #define ASI_BLK_AIUP_4V		0x16 /* (4V) Prim, user, block ld/st	*/
 #define ASI_BLK_AIUS_4V		0x17 /* (4V) Sec, user, block ld/st	*/
 #define ASI_REAL_L              0x1c /* Real address, cacheable, LE      */
-#define ASI_PHYS_USE_EC_L	0x1c /* PADDR, E-cachable, little endian*/
-#define ASI_REAL_IO_L           0x1d /* Real address, non-cachable, LE  */
+#define ASI_PHYS_USE_EC_L	0x1c /* PADDR, E-cacheable, little endian*/
+#define ASI_REAL_IO_L           0x1d /* Real address, non-cacheable, LE  */
 #define ASI_PHYS_BYPASS_EC_E_L	0x1d /* PADDR, E-bit, little endian	*/
 #define ASI_BLK_AIUP_L_4V	0x1e /* (4V) Prim, user, block, l-endian*/
 #define ASI_BLK_AIUS_L_4V	0x1f /* (4V) Sec, user, block, l-endian	*/
-- 
γαῖα πυρί μιχθήτω

Re: [PATCH v1 02/21] tests: correct typos

[Pavel Dovgalyuk]

Reviewed-by: Pavel Dovgalyuk <Pavel.Dovgalyuk@ispras.ru>

On 20.02.2024 11:52, Manos Pitsidianakis wrote:
> Correct typos automatically found with the `typos` tool
> <https://crates.io/crates/typos>
> 
> Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
> ---
>   tests/avocado/acpi-bits/bits-tests/smbios.py2 | 2 +-
>   tests/avocado/mem-addr-space-check.py         | 6 +++---
>   tests/avocado/reverse_debugging.py            | 2 +-
>   3 files changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/tests/avocado/acpi-bits/bits-tests/smbios.py2 b/tests/avocado/acpi-bits/bits-tests/smbios.py2
> index fc623de072..5868a7137a 100644
> --- a/tests/avocado/acpi-bits/bits-tests/smbios.py2
> +++ b/tests/avocado/acpi-bits/bits-tests/smbios.py2
> @@ -1060,7 +1060,7 @@ class EventLogDescriptor(unpack.Struct):
>               0x16: 'Log Area Reset/Cleared',
>               0x17: 'System boot',
>               xrange(0x18, 0x7F): 'Unused, available for assignment',
> -            xrange(0x80, 0xFE): 'Availalbe for system- and OEM-specific assignments',
> +            xrange(0x80, 0xFE): 'Available for system- and OEM-specific assignments',
>               0xFF: 'End of log'
>           }
>           yield 'log_type', u.unpack_one('B'), unpack.format_table("{}", _event_log_type_descriptors)
> diff --git a/tests/avocado/mem-addr-space-check.py b/tests/avocado/mem-addr-space-check.py
> index 363c3f12a6..af019969c0 100644
> --- a/tests/avocado/mem-addr-space-check.py
> +++ b/tests/avocado/mem-addr-space-check.py
> @@ -165,7 +165,7 @@ def test_phybits_low_tcg_q35_70_amd(self):
>           For q35-7.0 machines, "above 4G" memory starts are 4G.
>           pci64_hole size is 32 GiB. Since TCG_PHYS_ADDR_BITS is defined to
>           be 40, TCG emulated CPUs have maximum of 1 TiB (1024 GiB) of
> -        directly addressible memory.
> +        directly addressable memory.
>           Hence, maxmem value at most can be
>           1024 GiB - 4 GiB - 1 GiB per slot for alignment - 32 GiB + 0.5 GiB
>           which is equal to 987.5 GiB. Setting the value to 988 GiB should
> @@ -190,7 +190,7 @@ def test_phybits_low_tcg_q35_71_amd(self):
>           AMD_HT_START is defined to be at 1012 GiB. So for q35 machines
>           version > 7.0 and AMD cpus, instead of 1024 GiB limit for 40 bit
>           processor address space, it has to be 1012 GiB , that is 12 GiB
> -        less than the case above in order to accomodate HT hole.
> +        less than the case above in order to accommodate HT hole.
>           Make sure QEMU fails when maxmem size is 976 GiB (12 GiB less
>           than 988 GiB).
>           """
> @@ -297,7 +297,7 @@ def test_phybits_ok_tcg_q35_71_amd_41bits(self):
>           :avocado: tags=arch:x86_64
>   
>           AMD processor with 41 bits. Max cpu hw address = 2 TiB.
> -        Same as above but by setting maxram beween 976 GiB and 992 Gib,
> +        Same as above but by setting maxram between 976 GiB and 992 Gib,
>           QEMU should start fine.
>           """
>           self.vm.add_args('-S', '-cpu', 'EPYC-v4,phys-bits=41',
> diff --git a/tests/avocado/reverse_debugging.py b/tests/avocado/reverse_debugging.py
> index 4cce5a5598..92855a02a5 100644
> --- a/tests/avocado/reverse_debugging.py
> +++ b/tests/avocado/reverse_debugging.py
> @@ -191,7 +191,7 @@ def reverse_debugging(self, shift=7, args=None):
>           self.check_pc(g, steps[-1])
>           logger.info('successfully reached %x' % steps[-1])
>   
> -        logger.info('exitting gdb and qemu')
> +        logger.info('exiting gdb and qemu')
>           vm.shutdown()
>   
>   class ReverseDebugging_X86_64(ReverseDebugging):

Re: [PATCH v1 15/21] qapi/ui: correct typos

[Markus Armbruster]

Manos Pitsidianakis <manos.pitsidianakis@linaro.org> writes:

> Correct typos automatically found with the `typos` tool
> <https://crates.io/crates/typos>
>
> Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
> ---
>  qapi/ui.json | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/qapi/ui.json b/qapi/ui.json
> index b6d7e142b7..1448eaca73 100644
> --- a/qapi/ui.json
> +++ b/qapi/ui.json
> @@ -63,7 +63,7 @@
>  ##
>  # @SetPasswordOptionsVnc:
>  #
> -# Options for set_password specific to the VNC procotol.
> +# Options for set_password specific to the VNC protocol.
>  #
>  # @display: The id of the display where the password should be
>  #     changed.  Defaults to the first.
> @@ -125,7 +125,7 @@
>  ##
>  # @ExpirePasswordOptionsVnc:
>  #
> -# Options for expire_password specific to the VNC procotol.
> +# Options for expire_password specific to the VNC protocol.
>  #
>  # @display: The id of the display where the expiration should be
>  #     changed.  Defaults to the first.

Reviewed-by: Markus Armbruster <armbru@redhat.com>

Re: [PATCH v1 04/21] accel/tcg: correct typos

[Alex Bennée]

Manos Pitsidianakis <manos.pitsidianakis@linaro.org> writes:

> Correct typos automatically found with the `typos` tool
> <https://crates.io/crates/typos>
>
> Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>

Reviewed-by: Alex Bennée <alex.bennee@linaro.org>

-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro

Re: [PATCH v1 01/21] docs: correct typos

[Alex Bennée]

Manos Pitsidianakis <manos.pitsidianakis@linaro.org> writes:

> Correct typos automatically found with the `typos` tool
> <https://crates.io/crates/typos>
>
> Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>

Reviewed-by: Alex Bennée <alex.bennee@linaro.org>

-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro

Re: [PATCH v1 01/21] docs: correct typos

[Michael S. Tsirkin]

On Tue, Feb 20, 2024 at 10:52:08AM +0200, Manos Pitsidianakis wrote:
> Correct typos automatically found with the `typos` tool
> <https://crates.io/crates/typos>
> 
> Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>

Acked-by: Michael S. Tsirkin <mst@redhat.com>

> ---
>  docs/devel/ci-jobs.rst.inc      | 2 +-
>  docs/devel/docs.rst             | 2 +-
>  docs/devel/testing.rst          | 2 +-
>  docs/interop/prl-xml.txt        | 2 +-
>  docs/interop/vhost-user.rst     | 2 +-
>  docs/system/devices/canokey.rst | 2 +-
>  6 files changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/docs/devel/ci-jobs.rst.inc b/docs/devel/ci-jobs.rst.inc
> index 4c39cdb2d9..6678b4f4ef 100644
> --- a/docs/devel/ci-jobs.rst.inc
> +++ b/docs/devel/ci-jobs.rst.inc
> @@ -147,7 +147,7 @@ Set this variable to 1 to create the pipelines, but leave all
>  the jobs to be manually started from the UI
>  
>  Set this variable to 2 to create the pipelines and run all
> -the jobs immediately, as was historicaly behaviour
> +the jobs immediately, as was historically behaviour

as long as we do this let's fix grammar too?

as was historically the behaviour

>  QEMU_CI_AVOCADO_TESTING
>  ~~~~~~~~~~~~~~~~~~~~~~~
> diff --git a/docs/devel/docs.rst b/docs/devel/docs.rst
> index 50ff0d67f8..a7768b5311 100644
> --- a/docs/devel/docs.rst
> +++ b/docs/devel/docs.rst
> @@ -21,7 +21,7 @@ are processed in two ways:
>  
>  The syntax of these ``.hx`` files is simple. It is broadly an
>  alternation of C code put into the C output and rST format text
> -put into the documention. A few special directives are recognised;
> +put into the documentation. A few special directives are recognised;
>  these are all-caps and must be at the beginning of the line.
>  
>  ``HXCOMM`` is the comment marker. The line, including any arbitrary
> diff --git a/docs/devel/testing.rst b/docs/devel/testing.rst
> index bd132306c1..aa96eacec5 100644
> --- a/docs/devel/testing.rst
> +++ b/docs/devel/testing.rst
> @@ -728,7 +728,7 @@ For example to setup the HPPA ports builds of Debian::
>      EXECUTABLE=(pwd)/qemu-hppa V=1
>  
>  The ``DEB_`` variables are substitutions used by
> -``debian-boostrap.pre`` which is called to do the initial debootstrap
> +``debian-bootstrap.pre`` which is called to do the initial debootstrap
>  of the rootfs before it is copied into the container. The second stage
>  is run as part of the build. The final image will be tagged as
>  ``qemu/debian-sid-hppa``.
> diff --git a/docs/interop/prl-xml.txt b/docs/interop/prl-xml.txt
> index 7031f8752c..cf9b3fba26 100644
> --- a/docs/interop/prl-xml.txt
> +++ b/docs/interop/prl-xml.txt
> @@ -122,7 +122,7 @@ Each Image element has following child elements:
>      * Type - image type of the element. It can be:
>               "Plain" for raw files.
>               "Compressed" for expanding disks.
> -    * File - path to image file. Path can be relative to DiskDecriptor.xml or
> +    * File - path to image file. Path can be relative to DiskDescriptor.xml or
>               absolute.
>  
>  == Snapshots element ==
> diff --git a/docs/interop/vhost-user.rst b/docs/interop/vhost-user.rst
> index ad6e142f23..d1ed39dfa0 100644
> --- a/docs/interop/vhost-user.rst
> +++ b/docs/interop/vhost-user.rst
> @@ -989,7 +989,7 @@ When reconnecting:
>  
>     #. If ``d.flags`` is not equal to the calculated flags value (means
>        back-end has submitted the buffer to guest driver before crash, so
> -      it has to commit the in-progres update), set ``old_free_head``,
> +      it has to commit the in-progress update), set ``old_free_head``,
>        ``old_used_idx``, ``old_used_wrap_counter`` to ``free_head``,
>        ``used_idx``, ``used_wrap_counter``
>  
> diff --git a/docs/system/devices/canokey.rst b/docs/system/devices/canokey.rst
> index cfa6186e48..7f3664963f 100644
> --- a/docs/system/devices/canokey.rst
> +++ b/docs/system/devices/canokey.rst
> @@ -14,7 +14,7 @@ CanoKey [1]_ is an open-source secure key with supports of
>  All these platform-independent features are in canokey-core [3]_.
>  
>  For different platforms, CanoKey has different implementations,
> -including both hardware implementions and virtual cards:
> +including both hardware implementations and virtual cards:
>  
>  * CanoKey STM32 [4]_
>  * CanoKey Pigeon [5]_
> -- 
> γαῖα πυρί μιχθήτω

Re: [PATCH v1 01/21] docs: correct typos

[Manos Pitsidianakis]

On Tue, 20 Feb 2024 12:36, "Michael S. Tsirkin" <mst@redhat.com> wrote:
>On Tue, Feb 20, 2024 at 10:52:08AM +0200, Manos Pitsidianakis wrote:
>> Correct typos automatically found with the `typos` tool
>> <https://crates.io/crates/typos>
>> 
>> Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
>
>Acked-by: Michael S. Tsirkin <mst@redhat.com>
>
>> ---
>>  docs/devel/ci-jobs.rst.inc      | 2 +-
>>  docs/devel/docs.rst             | 2 +-
>>  docs/devel/testing.rst          | 2 +-
>>  docs/interop/prl-xml.txt        | 2 +-
>>  docs/interop/vhost-user.rst     | 2 +-
>>  docs/system/devices/canokey.rst | 2 +-
>>  6 files changed, 6 insertions(+), 6 deletions(-)
>> 
>> diff --git a/docs/devel/ci-jobs.rst.inc b/docs/devel/ci-jobs.rst.inc
>> index 4c39cdb2d9..6678b4f4ef 100644
>> --- a/docs/devel/ci-jobs.rst.inc
>> +++ b/docs/devel/ci-jobs.rst.inc
>> @@ -147,7 +147,7 @@ Set this variable to 1 to create the pipelines, but leave all
>>  the jobs to be manually started from the UI
>>  
>>  Set this variable to 2 to create the pipelines and run all
>> -the jobs immediately, as was historicaly behaviour
>> +the jobs immediately, as was historically behaviour
>
>as long as we do this let's fix grammar too?
>
>as was historically the behaviour

After the fact, I think it should be "as was historical behaviour".

I will re-spin with only this change, and keep the Acks/RoBs if that is 
okay with everyone.

Thanks,

>
>>  QEMU_CI_AVOCADO_TESTING
>>  ~~~~~~~~~~~~~~~~~~~~~~~
>> diff --git a/docs/devel/docs.rst b/docs/devel/docs.rst
>> index 50ff0d67f8..a7768b5311 100644
>> --- a/docs/devel/docs.rst
>> +++ b/docs/devel/docs.rst
>> @@ -21,7 +21,7 @@ are processed in two ways:
>>  
>>  The syntax of these ``.hx`` files is simple. It is broadly an
>>  alternation of C code put into the C output and rST format text
>> -put into the documention. A few special directives are recognised;
>> +put into the documentation. A few special directives are recognised;
>>  these are all-caps and must be at the beginning of the line.
>>  
>>  ``HXCOMM`` is the comment marker. The line, including any arbitrary
>> diff --git a/docs/devel/testing.rst b/docs/devel/testing.rst
>> index bd132306c1..aa96eacec5 100644
>> --- a/docs/devel/testing.rst
>> +++ b/docs/devel/testing.rst
>> @@ -728,7 +728,7 @@ For example to setup the HPPA ports builds of Debian::
>>      EXECUTABLE=(pwd)/qemu-hppa V=1
>>  
>>  The ``DEB_`` variables are substitutions used by
>> -``debian-boostrap.pre`` which is called to do the initial debootstrap
>> +``debian-bootstrap.pre`` which is called to do the initial debootstrap
>>  of the rootfs before it is copied into the container. The second stage
>>  is run as part of the build. The final image will be tagged as
>>  ``qemu/debian-sid-hppa``.
>> diff --git a/docs/interop/prl-xml.txt b/docs/interop/prl-xml.txt
>> index 7031f8752c..cf9b3fba26 100644
>> --- a/docs/interop/prl-xml.txt
>> +++ b/docs/interop/prl-xml.txt
>> @@ -122,7 +122,7 @@ Each Image element has following child elements:
>>      * Type - image type of the element. It can be:
>>               "Plain" for raw files.
>>               "Compressed" for expanding disks.
>> -    * File - path to image file. Path can be relative to DiskDecriptor.xml or
>> +    * File - path to image file. Path can be relative to DiskDescriptor.xml or
>>               absolute.
>>  
>>  == Snapshots element ==
>> diff --git a/docs/interop/vhost-user.rst b/docs/interop/vhost-user.rst
>> index ad6e142f23..d1ed39dfa0 100644
>> --- a/docs/interop/vhost-user.rst
>> +++ b/docs/interop/vhost-user.rst
>> @@ -989,7 +989,7 @@ When reconnecting:
>>  
>>     #. If ``d.flags`` is not equal to the calculated flags value (means
>>        back-end has submitted the buffer to guest driver before crash, so
>> -      it has to commit the in-progres update), set ``old_free_head``,
>> +      it has to commit the in-progress update), set ``old_free_head``,
>>        ``old_used_idx``, ``old_used_wrap_counter`` to ``free_head``,
>>        ``used_idx``, ``used_wrap_counter``
>>  
>> diff --git a/docs/system/devices/canokey.rst b/docs/system/devices/canokey.rst
>> index cfa6186e48..7f3664963f 100644
>> --- a/docs/system/devices/canokey.rst
>> +++ b/docs/system/devices/canokey.rst
>> @@ -14,7 +14,7 @@ CanoKey [1]_ is an open-source secure key with supports of
>>  All these platform-independent features are in canokey-core [3]_.
>>  
>>  For different platforms, CanoKey has different implementations,
>> -including both hardware implementions and virtual cards:
>> +including both hardware implementations and virtual cards:
>>  
>>  * CanoKey STM32 [4]_
>>  * CanoKey Pigeon [5]_
>> -- 
>> γαῖα πυρί μιχθήτω
>

Re: [PATCH v1 19/21] m68k: correct typos

[Thomas Huth]

On 20/02/2024 09.52, Manos Pitsidianakis wrote:
> Correct typos automatically found with the `typos` tool
> <https://crates.io/crates/typos>
> 
> Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
> ---
>   target/m68k/cpu.h | 7 ++++---
>   1 file changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/target/m68k/cpu.h b/target/m68k/cpu.h
> index aca4aa610b..57e3b6d3ce 100644
> --- a/target/m68k/cpu.h
> +++ b/target/m68k/cpu.h
> @@ -478,10 +478,11 @@ void do_m68k_semihosting(CPUM68KState *env, int nr);
>    * The 68000 family is defined in six main CPU classes, the 680[012346]0.
>    * Generally each successive CPU adds enhanced data/stack/instructions.
>    * However, some features are only common to one, or a few classes.
> - * The features covers those subsets of instructons.
> + * The features covers those subsets of instructions.

While you're at it, please also remove the "s" from "covers".

  Thomas

Re: [PATCH v1 20/21] s390x: correct typos

[Thomas Huth]

On 20/02/2024 09.52, Manos Pitsidianakis wrote:
> Correct typos automatically found with the `typos` tool
> <https://crates.io/crates/typos>
> 
> Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
> ---
>   target/s390x/cpu_features_def.h.inc | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/target/s390x/cpu_features_def.h.inc b/target/s390x/cpu_features_def.h.inc
> index e68da9b8ff..c53ac13352 100644
> --- a/target/s390x/cpu_features_def.h.inc
> +++ b/target/s390x/cpu_features_def.h.inc
> @@ -142,7 +142,7 @@ DEF_FEAT(SIE_CEI, "cei", SCLP_CPU, 43, "SIE: Conditional-external-interception f
>   
>   /*
>    * Features exposed via no feature bit (but e.g., instruction sensing)
> - * -> the feature bit number is irrelavant
> + * -> the feature bit number is irrelevant
>    */
>   DEF_FEAT(DAT_ENH_2, "dateh2", MISC, 0, "DAT-enhancement facility 2")
>   DEF_FEAT(CMM, "cmm", MISC, 0, "Collaborative-memory-management facility")

Reviewed-by: Thomas Huth <thuth@redhat.com>

Re: [PATCH v1 01/21] docs: correct typos

[Michael S. Tsirkin]

On Tue, Feb 20, 2024 at 12:42:28PM +0200, Manos Pitsidianakis wrote:
> On Tue, 20 Feb 2024 12:36, "Michael S. Tsirkin" <mst@redhat.com> wrote:
> > On Tue, Feb 20, 2024 at 10:52:08AM +0200, Manos Pitsidianakis wrote:
> > > Correct typos automatically found with the `typos` tool
> > > <https://crates.io/crates/typos>
> > > 
> > > Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
> > 
> > Acked-by: Michael S. Tsirkin <mst@redhat.com>
> > 
> > > ---
> > >  docs/devel/ci-jobs.rst.inc      | 2 +-
> > >  docs/devel/docs.rst             | 2 +-
> > >  docs/devel/testing.rst          | 2 +-
> > >  docs/interop/prl-xml.txt        | 2 +-
> > >  docs/interop/vhost-user.rst     | 2 +-
> > >  docs/system/devices/canokey.rst | 2 +-
> > >  6 files changed, 6 insertions(+), 6 deletions(-)
> > > 
> > > diff --git a/docs/devel/ci-jobs.rst.inc b/docs/devel/ci-jobs.rst.inc
> > > index 4c39cdb2d9..6678b4f4ef 100644
> > > --- a/docs/devel/ci-jobs.rst.inc
> > > +++ b/docs/devel/ci-jobs.rst.inc
> > > @@ -147,7 +147,7 @@ Set this variable to 1 to create the pipelines, but leave all
> > >  the jobs to be manually started from the UI
> > >  Set this variable to 2 to create the pipelines and run all
> > > -the jobs immediately, as was historicaly behaviour
> > > +the jobs immediately, as was historically behaviour
> > 
> > as long as we do this let's fix grammar too?
> > 
> > as was historically the behaviour
> 
> After the fact, I think it should be "as was historical behaviour".

it needs the definite article though.

> I will re-spin with only this change, and keep the Acks/RoBs if that is okay
> with everyone.
> 
> Thanks,
> 
> > 
> > >  QEMU_CI_AVOCADO_TESTING
> > >  ~~~~~~~~~~~~~~~~~~~~~~~
> > > diff --git a/docs/devel/docs.rst b/docs/devel/docs.rst
> > > index 50ff0d67f8..a7768b5311 100644
> > > --- a/docs/devel/docs.rst
> > > +++ b/docs/devel/docs.rst
> > > @@ -21,7 +21,7 @@ are processed in two ways:
> > >  The syntax of these ``.hx`` files is simple. It is broadly an
> > >  alternation of C code put into the C output and rST format text
> > > -put into the documention. A few special directives are recognised;
> > > +put into the documentation. A few special directives are recognised;
> > >  these are all-caps and must be at the beginning of the line.
> > >  ``HXCOMM`` is the comment marker. The line, including any arbitrary
> > > diff --git a/docs/devel/testing.rst b/docs/devel/testing.rst
> > > index bd132306c1..aa96eacec5 100644
> > > --- a/docs/devel/testing.rst
> > > +++ b/docs/devel/testing.rst
> > > @@ -728,7 +728,7 @@ For example to setup the HPPA ports builds of Debian::
> > >      EXECUTABLE=(pwd)/qemu-hppa V=1
> > >  The ``DEB_`` variables are substitutions used by
> > > -``debian-boostrap.pre`` which is called to do the initial debootstrap
> > > +``debian-bootstrap.pre`` which is called to do the initial debootstrap
> > >  of the rootfs before it is copied into the container. The second stage
> > >  is run as part of the build. The final image will be tagged as
> > >  ``qemu/debian-sid-hppa``.
> > > diff --git a/docs/interop/prl-xml.txt b/docs/interop/prl-xml.txt
> > > index 7031f8752c..cf9b3fba26 100644
> > > --- a/docs/interop/prl-xml.txt
> > > +++ b/docs/interop/prl-xml.txt
> > > @@ -122,7 +122,7 @@ Each Image element has following child elements:
> > >      * Type - image type of the element. It can be:
> > >               "Plain" for raw files.
> > >               "Compressed" for expanding disks.
> > > -    * File - path to image file. Path can be relative to DiskDecriptor.xml or
> > > +    * File - path to image file. Path can be relative to DiskDescriptor.xml or
> > >               absolute.
> > >  == Snapshots element ==
> > > diff --git a/docs/interop/vhost-user.rst b/docs/interop/vhost-user.rst
> > > index ad6e142f23..d1ed39dfa0 100644
> > > --- a/docs/interop/vhost-user.rst
> > > +++ b/docs/interop/vhost-user.rst
> > > @@ -989,7 +989,7 @@ When reconnecting:
> > >     #. If ``d.flags`` is not equal to the calculated flags value (means
> > >        back-end has submitted the buffer to guest driver before crash, so
> > > -      it has to commit the in-progres update), set ``old_free_head``,
> > > +      it has to commit the in-progress update), set ``old_free_head``,
> > >        ``old_used_idx``, ``old_used_wrap_counter`` to ``free_head``,
> > >        ``used_idx``, ``used_wrap_counter``
> > > diff --git a/docs/system/devices/canokey.rst b/docs/system/devices/canokey.rst
> > > index cfa6186e48..7f3664963f 100644
> > > --- a/docs/system/devices/canokey.rst
> > > +++ b/docs/system/devices/canokey.rst
> > > @@ -14,7 +14,7 @@ CanoKey [1]_ is an open-source secure key with supports of
> > >  All these platform-independent features are in canokey-core [3]_.
> > >  For different platforms, CanoKey has different implementations,
> > > -including both hardware implementions and virtual cards:
> > > +including both hardware implementations and virtual cards:
> > >  * CanoKey STM32 [4]_
> > >  * CanoKey Pigeon [5]_
> > > -- 
> > > γαῖα πυρί μιχθήτω
> > 

Re: [PATCH v1 06/21] ppc: correct typos

[BALATON Zoltan]

On Tue, 20 Feb 2024, Manos Pitsidianakis wrote:
> Correct typos automatically found with the `typos` tool
> <https://crates.io/crates/typos>
>
> Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
> ---
> include/hw/ppc/ppc4xx.h             | 2 +-
> hw/ppc/ppc405.h                     | 2 +-
> target/ppc/translate/vmx-impl.c.inc | 2 +-
> 3 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/include/hw/ppc/ppc4xx.h b/include/hw/ppc/ppc4xx.h
> index ea7740239b..c4ecb1652f 100644
> --- a/include/hw/ppc/ppc4xx.h
> +++ b/include/hw/ppc/ppc4xx.h
> @@ -75,7 +75,7 @@ struct Ppc4xxMalState {
>     uint8_t  rxcnum;
> };
>
> -/* Peripheral local bus arbitrer */
> +/* Peripheral local bus arbiter */
> #define TYPE_PPC4xx_PLB "ppc4xx-plb"
> OBJECT_DECLARE_SIMPLE_TYPE(Ppc4xxPlbState, PPC4xx_PLB);
> struct Ppc4xxPlbState {
> diff --git a/hw/ppc/ppc405.h b/hw/ppc/ppc405.h
> index 9a4312691e..a39f0caea1 100644
> --- a/hw/ppc/ppc405.h
> +++ b/hw/ppc/ppc405.h
> @@ -41,7 +41,7 @@ struct Ppc405PobState {
>     uint32_t besr1;
> };
>
> -/* OPB arbitrer */
> +/* OPB arbiter */

These two aren't actually typos. The PPC docs call it that. Somebody tried 
to fix these before but this spelling is also valid and what is used in 
the docs.

> #define TYPE_PPC405_OPBA "ppc405-opba"
> OBJECT_DECLARE_SIMPLE_TYPE(Ppc405OpbaState, PPC405_OPBA);
> struct Ppc405OpbaState {
> diff --git a/target/ppc/translate/vmx-impl.c.inc b/target/ppc/translate/vmx-impl.c.inc
> index 4b91c3489d..b56e615c24 100644
> --- a/target/ppc/translate/vmx-impl.c.inc
> +++ b/target/ppc/translate/vmx-impl.c.inc
> @@ -1183,7 +1183,7 @@ static void glue(gen_, name)(DisasContext *ctx)                         \
>
> /*
>  * Support for Altivec instructions that use bit 31 (Rc) as an opcode
> - * bit but also use bit 21 as an actual Rc bit.  In general, thse pairs
> + * bit but also use bit 21 as an actual Rc bit.  In general, these pairs

This one is a typo.

Regards,
BALATON Zoltan

>  * come from different versions of the ISA, so we must also support a
>  * pair of flags for each instruction.
>  */
>

Re: [PATCH v1 08/21] include/exec/memory.h: correct typos

[Philippe Mathieu-Daudé]

On 20/2/24 09:52, Manos Pitsidianakis wrote:
> Correct typos automatically found with the `typos` tool
> <https://crates.io/crates/typos>
> 
> Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
> ---
>   include/exec/memory.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>

Re: [PATCH v1 04/21] accel/tcg: correct typos

[Philippe Mathieu-Daudé]

On 20/2/24 09:52, Manos Pitsidianakis wrote:
> Correct typos automatically found with the `typos` tool
> <https://crates.io/crates/typos>
> 
> Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
> ---
>   accel/tcg/ldst_atomicity.c.inc | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>

Re: [PATCH v1 07/21] sh4: correct typos

[Philippe Mathieu-Daudé]

On 20/2/24 09:52, Manos Pitsidianakis wrote:
> Correct typos automatically found with the `typos` tool
> <https://crates.io/crates/typos>
> 
> Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
> ---
>   hw/sh4/sh7750_regs.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>

Re: [PATCH v1 09/21] include/exec/plugin-gen.h: correct typos

[Philippe Mathieu-Daudé]

On 20/2/24 09:52, Manos Pitsidianakis wrote:
> Correct typos automatically found with the `typos` tool
> <https://crates.io/crates/typos>
> 
> Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
> ---
>   include/exec/plugin-gen.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/exec/plugin-gen.h b/include/exec/plugin-gen.h
> index c4552b5061..b18edd6ab4 100644
> --- a/include/exec/plugin-gen.h
> +++ b/include/exec/plugin-gen.h
> @@ -19,7 +19,7 @@ struct DisasContextBase;
>   #ifdef CONFIG_PLUGIN
>   
>   bool plugin_gen_tb_start(CPUState *cpu, const struct DisasContextBase *db,
> -                         bool supress);
> +                         bool suppress);

Could we keep the definition in sync?

accel/tcg/plugin-gen.c:794:bool plugin_gen_tb_start(CPUState *cpu, const 
DisasContextBase *db,
accel/tcg/plugin-gen.c-795-                         bool mem_only)

Re: [PATCH v1 10/21] hw/arm/omap.h: correct typos

[Philippe Mathieu-Daudé]

On 20/2/24 09:52, Manos Pitsidianakis wrote:
> Correct typos automatically found with the `typos` tool
> <https://crates.io/crates/typos>
> 
> Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
> ---
>   include/hw/arm/omap.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/hw/arm/omap.h b/include/hw/arm/omap.h
> index 067e9419f7..2f59220c0e 100644
> --- a/include/hw/arm/omap.h
> +++ b/include/hw/arm/omap.h
> @@ -1008,7 +1008,7 @@ void omap_mpu_wakeup(void *opaque, int irq, int req);
>                         __func__, paddr)
>   
>   /* OMAP-specific Linux bootloader tags for the ATAG_BOARD area
> -   (Board-specifc tags are not here)  */
> +   (Board-specific tags are not here)  */

Unfortunately you need to update the comment style, otherwise
this patch doesn't pass ./scripts/checkpatch.pl.

With this fixed:
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>

Thanks!

Re: [PATCH v1 14/21] pc-bios/README: correct typos

[Philippe Mathieu-Daudé]

On 20/2/24 09:52, Manos Pitsidianakis wrote:
> Correct typos automatically found with the `typos` tool
> <https://crates.io/crates/typos>
> 
> Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
> ---
>   pc-bios/README | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>

Re: [PATCH v1 15/21] qapi/ui: correct typos

[Philippe Mathieu-Daudé]

On 20/2/24 10:41, Markus Armbruster wrote:
> Manos Pitsidianakis <manos.pitsidianakis@linaro.org> writes:
> 
>> Correct typos automatically found with the `typos` tool
>> <https://crates.io/crates/typos>
>>
>> Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
>> ---
>>   qapi/ui.json | 4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/qapi/ui.json b/qapi/ui.json
>> index b6d7e142b7..1448eaca73 100644
>> --- a/qapi/ui.json
>> +++ b/qapi/ui.json
>> @@ -63,7 +63,7 @@
>>   ##
>>   # @SetPasswordOptionsVnc:
>>   #
>> -# Options for set_password specific to the VNC procotol.
>> +# Options for set_password specific to the VNC protocol.
>>   #
>>   # @display: The id of the display where the password should be
>>   #     changed.  Defaults to the first.
>> @@ -125,7 +125,7 @@
>>   ##
>>   # @ExpirePasswordOptionsVnc:
>>   #
>> -# Options for expire_password specific to the VNC procotol.
>> +# Options for expire_password specific to the VNC protocol.

Too bad, I remember reading few times the procotol, it was
funny.

>>   #
>>   # @display: The id of the display where the expiration should be
>>   #     changed.  Defaults to the first.
> 
> Reviewed-by: Markus Armbruster <armbru@redhat.com>
> 
> 

Re: [PATCH v1 18/21] hexagon: correct typos

[Philippe Mathieu-Daudé]

On 20/2/24 09:52, Manos Pitsidianakis wrote:
> Correct typos automatically found with the `typos` tool
> <https://crates.io/crates/typos>
> 
> Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
> ---
>   target/hexagon/idef-parser/macros.inc | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>

Re: [PATCH v1 20/21] s390x: correct typos

[Philippe Mathieu-Daudé]

On 20/2/24 09:52, Manos Pitsidianakis wrote:
> Correct typos automatically found with the `typos` tool
> <https://crates.io/crates/typos>
> 
> Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
> ---
>   target/s390x/cpu_features_def.h.inc | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>

Re: [PATCH v1 21/21] target/sparc: correct typos

[Philippe Mathieu-Daudé]

On 20/2/24 09:52, Manos Pitsidianakis wrote:
> Correct typos automatically found with the `typos` tool
> <https://crates.io/crates/typos>
> 
> Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
> ---
>   target/sparc/asi.h | 8 ++++----
>   1 file changed, 4 insertions(+), 4 deletions(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>

Re: [PATCH v1 18/21] hexagon: correct typos

[Anton Johansson via]

On 20/02/24, Philippe Mathieu-Daudé wrote:
> On 20/2/24 09:52, Manos Pitsidianakis wrote:
> > Correct typos automatically found with the `typos` tool
> > <https://crates.io/crates/typos>
> > 
> > Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
> > ---
> >   target/hexagon/idef-parser/macros.inc | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> 
> 
Ah nice, havent seen the typos tool before!

Reviewed-by: Anton Johansson <anjo@rev.ng>

Re: [PATCH v1 01/21] docs: correct typos

[Michael Tokarev]

20.02.2024 13:42, Manos Pitsidianakis wrote:
..
>>>  Set this variable to 2 to create the pipelines and run all
>>> -the jobs immediately, as was historicaly behaviour
>>> +the jobs immediately, as was historically behaviour
>>
>> as long as we do this let's fix grammar too?
>>
>> as was historically the behaviour
> 
> After the fact, I think it should be "as was historical behaviour".

That's exactly what I suggested on IRC right after you sent this patch.
Plus "the", so the final version reads

  the jobs immediately, as was the historical behaviour

> I will re-spin with only this change, and keep the Acks/RoBs if that is okay with everyone.

If that's the only change, let me just pick it all with the above fix,
to avoid additional SPAM :)

Either way, I'm picking up this series to trivial-patches@, please
resend just the "changed changes" if needs to be.

Thanks,

/mjt

Re: [PATCH v1 03/21] Xen headers: correct typos

[Michael Tokarev]

20.02.2024 11:52, Manos Pitsidianakis пишет:
> Correct typos automatically found with the `typos` tool
> <https://crates.io/crates/typos>
> 
> Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
> ---
>   include/hw/xen/interface/arch-x86/xen-x86_64.h | 2 +-
>   include/hw/xen/interface/arch-x86/xen.h        | 2 +-
>   include/hw/xen/interface/event_channel.h       | 2 +-
>   include/hw/xen/interface/grant_table.h         | 2 +-
>   include/hw/xen/interface/hvm/hvm_op.h          | 2 +-
>   include/hw/xen/interface/io/blkif.h            | 4 ++--
>   include/hw/xen/interface/io/fbif.h             | 2 +-
>   include/hw/xen/interface/io/kbdif.h            | 2 +-
>   include/hw/xen/interface/io/ring.h             | 2 +-
>   include/hw/xen/interface/memory.h              | 2 +-
>   include/hw/xen/interface/physdev.h             | 4 ++--
>   include/hw/xen/interface/xen.h                 | 4 ++--
>   12 files changed, 15 insertions(+), 15 deletions(-)

include/hw/xen/ is imported from xen, IIRC it should not be edited
directly in qemu.

/mjt

Re: [PATCH v1 00/21] Trivial tree wide typo fixes

[Michael Tokarev]

20.02.2024 11:52, Manos Pitsidianakis wrote:
..

So I picked up this series, to reduce the list SPAM.
With the following details:

>    docs: correct typos

fixed "the historical behaviour"
(why my spell checker continues marking "behaviour"?

>    tests: correct typos
>    Xen headers: correct typos

*not* picked up

>    accel/tcg: correct typos
>    loongson3: correct typos
>    ppc: correct typos

remove 2 hooks "arbitrer" as suggested by BALATON

>    sh4: correct typos
>    include/exec/memory.h: correct typos
>    include/exec/plugin-gen.h: correct typos

*not* picked up

>    hw/arm/omap.h: correct typos

fix comment style

>    hw/cxl/cxl_device.h: correct typos
>    hw/net/npcm_gmac.h: correct typos
>    hw/riscv/virt.h: correct typos
>    pc-bios/README: correct typos
>    qapi/ui: correct typos
>    qemu-options.hx: correct typos
>    ci/gitlab-pipeline-status: correct typos
>    hexagon: correct typos
>    m68k: correct typos

correct "covers" as suggested by Thomas

>    s390x: correct typos
>    target/sparc: correct typos

Re: [PATCH v1 07/21] sh4: correct typos

[Yoshinori Sato]

On Tue, 20 Feb 2024 17:52:14 +0900,
Manos Pitsidianakis wrote:
> 
> Correct typos automatically found with the `typos` tool
> <https://crates.io/crates/typos>
> 
> Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
> ---
>  hw/sh4/sh7750_regs.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/hw/sh4/sh7750_regs.h b/hw/sh4/sh7750_regs.h
> index edb5d18f00..946ad7b3aa 100644
> --- a/hw/sh4/sh7750_regs.h
> +++ b/hw/sh4/sh7750_regs.h
> @@ -172,7 +172,7 @@
>  
>  
>  /*
> - * Exeption-related registers
> + * Exception-related registers
>   */
>  
>  /* Immediate data for TRAPA instruction - TRA */
> -- 
> γαῖα πυρί μιχθήτω
> 

Reviewed-by: Yoshinori Sato <ysato@users.sourceforge.jp>

-- 
Yosinori Sato

Re: [PATCH v1 13/21] hw/riscv/virt.h: correct typos

[Alistair Francis]

On Tue, Feb 20, 2024 at 6:55 PM Manos Pitsidianakis
<manos.pitsidianakis@linaro.org> wrote:
>
> Correct typos automatically found with the `typos` tool
> <https://crates.io/crates/typos>
>
> Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>

Reviewed-by: Alistair Francis <alistair.francis@wdc.com>

Alistair

> ---
>  include/hw/riscv/virt.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/include/hw/riscv/virt.h b/include/hw/riscv/virt.h
> index f89790fd58..3db839160f 100644
> --- a/include/hw/riscv/virt.h
> +++ b/include/hw/riscv/virt.h
> @@ -144,13 +144,13 @@ uint32_t imsic_num_bits(uint32_t count);
>  #define VIRT_IMSIC_GROUP_MAX_SIZE      (1U << IMSIC_MMIO_GROUP_MIN_SHIFT)
>  #if VIRT_IMSIC_GROUP_MAX_SIZE < \
>      IMSIC_GROUP_SIZE(VIRT_CPUS_MAX_BITS, VIRT_IRQCHIP_MAX_GUESTS_BITS)
> -#error "Can't accomodate single IMSIC group in address space"
> +#error "Can't accommodate single IMSIC group in address space"
>  #endif
>
>  #define VIRT_IMSIC_MAX_SIZE            (VIRT_SOCKETS_MAX * \
>                                          VIRT_IMSIC_GROUP_MAX_SIZE)
>  #if 0x4000000 < VIRT_IMSIC_MAX_SIZE
> -#error "Can't accomodate all IMSIC groups in address space"
> +#error "Can't accommodate all IMSIC groups in address space"
>  #endif
>
>  #endif
> --
> γαῖα πυρί μιχθήτω
>
>