[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2] Fix some typos in documentation and comments

From: Thomas Huth
Subject: Re: [PATCH v2] Fix some typos in documentation and comments
Date: Mon, 31 Oct 2022 08:35:32 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.13.0

On 30/10/2022 11.59, Stefan Weil wrote:
Most of them were found and fixed using codespell.

Signed-off-by: Stefan Weil <sw@weilnetz.de>

v2: Fixes from Peter Maydell's comments

My focus was fixing typos which are relevant for the generated documentation.

codespell finds many more typos in source code, and adding it to the continuous
integration checks looks more and more like a good idea.

... at least for the docs/ folder, this might indeed be a good idea.

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

  docs/devel/testing.rst              |  2 +-
  docs/system/arm/cpu-features.rst    |  2 +-
  docs/system/loongarch/loongson3.rst |  2 +-
  docs/tools/virtiofsd.rst            |  2 +-
  include/exec/memory.h               |  2 +-
  qapi/qom.json                       |  2 +-
  qemu-options.hx                     | 10 +++++-----
  qga/qapi-schema.json                |  2 +-
  tests/qtest/libqtest.h              |  2 +-
  9 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/docs/devel/testing.rst b/docs/devel/testing.rst
index fbb98faabe..e10c47b5a7 100644
--- a/docs/devel/testing.rst
+++ b/docs/devel/testing.rst
@@ -99,7 +99,7 @@ successfully on various hosts. The following list shows some 
best practices:
  * If your test cases uses the blkdebug feature, use relative path to pass
    the config and image file paths in the command line as Windows absolute
    path contains the delimiter ":" which will confuse the blkdebug parser.
-* Use double quotes in your extra QEMU commmand line in your test cases
+* Use double quotes in your extra QEMU command line in your test cases
    instead of single quotes, as Windows does not drop single quotes when
    passing the command line to QEMU.
  * Windows opens a file in text mode by default, while a POSIX compliant
diff --git a/docs/system/arm/cpu-features.rst b/docs/system/arm/cpu-features.rst
index c2c01ec7d2..00c444042f 100644
--- a/docs/system/arm/cpu-features.rst
+++ b/docs/system/arm/cpu-features.rst
@@ -433,7 +433,7 @@ additional constraints on the set of vector lengths 
supported by SME.
  SME User-mode Default Vector Length Property
-For qemu-aarch64, the cpu propery ``sme-default-vector-length=N`` is
+For qemu-aarch64, the cpu property ``sme-default-vector-length=N`` is
  defined to mirror the Linux kernel parameter file
  ``/proc/sys/abi/sme_default_vector_length``.  The default length, ``N``,
  is in units of bytes and must be between 16 and 8192.
diff --git a/docs/system/loongarch/loongson3.rst 
index 1bdab44e27..68113b6f5e 100644
--- a/docs/system/loongarch/loongson3.rst
+++ b/docs/system/loongarch/loongson3.rst
@@ -41,7 +41,7 @@ can be accessed by following steps.
$ qemu-system-loongarch64 -machine virt -m 4G -cpu la464-loongarch-cpu \
        -smp 1 -bios QEMU_EFI.fd -kernel vmlinuz.efi -initrd initrd.img \
-      -append "root=/dev/ram rdinit=/sbin/init consol e=ttyS0,115200" \
+      -append "root=/dev/ram rdinit=/sbin/init console=ttyS0,115200" \
Note: The running speed may be a little slow, as the performance of our
diff --git a/docs/tools/virtiofsd.rst b/docs/tools/virtiofsd.rst
index 5f5ac9dd56..995a754a7b 100644
--- a/docs/tools/virtiofsd.rst
+++ b/docs/tools/virtiofsd.rst
@@ -232,7 +232,7 @@ e.g.:
``:ok:server::security.:`` - will pass 'securty.' xattr's in listxattr from the server
+  will pass 'security.' xattr's in listxattr from the server
    and ignore following rules.
diff --git a/include/exec/memory.h b/include/exec/memory.h
index bfb1de8eea..a751c111bd 100644
--- a/include/exec/memory.h
+++ b/include/exec/memory.h
@@ -1970,7 +1970,7 @@ void memory_region_clear_dirty_bitmap(MemoryRegion *mr, 
hwaddr start,
   * querying the same page multiple times, which is especially useful for
   * display updates where the scanlines often are not page aligned.
- * The dirty bitmap region which gets copyed into the snapshot (and
+ * The dirty bitmap region which gets copied into the snapshot (and
   * cleared afterwards) can be larger than requested.  The boundaries
   * are rounded up/down so complete bitmap longs (covering 64 pages on
   * 64bit hosts) can be copied over into the bitmap snapshot.  Which
diff --git a/qapi/qom.json b/qapi/qom.json
index 80dd419b39..216b56a4e6 100644
--- a/qapi/qom.json
+++ b/qapi/qom.json
@@ -586,7 +586,7 @@
  # @size: size of the memory region in bytes
-# @x-use-canonical-path-for-ramblock-id: if true, the canoncial path is used
+# @x-use-canonical-path-for-ramblock-id: if true, the canonical path is used
  #                                        for ramblock-id. Disable this for 4.0
  #                                        machine types or older to allow
  #                                        migration with newer QEMU versions.
diff --git a/qemu-options.hx b/qemu-options.hx
index eb38e5dc40..1141dc5e58 100644
--- a/qemu-options.hx
+++ b/qemu-options.hx
@@ -1051,7 +1051,7 @@ SRST
      details on the external interface.
``-device isa-ipmi-kcs,bmc=id[,ioport=val][,irq=val]``
-    Add a KCS IPMI interafce on the ISA bus. This also adds a
+    Add a KCS IPMI interface on the ISA bus. This also adds a
      corresponding ACPI and SMBIOS entries, if appropriate.
@@ -1071,7 +1071,7 @@ SRST
      is 0xe4 and the default interrupt is 5.
``-device pci-ipmi-kcs,bmc=id``
-    Add a KCS IPMI interafce on the PCI bus.
+    Add a KCS IPMI interface on the PCI bus.
          The BMC to connect to, one of ipmi-bmc-sim or ipmi-bmc-extern above.
@@ -5248,8 +5248,8 @@ SRST
          read the colo-compare git log.
``-object cryptodev-backend-builtin,id=id[,queues=queues]``
-        Creates a cryptodev backend which executes crypto opreation from
-        the QEMU cipher APIS. The id parameter is a unique ID that will
+        Creates a cryptodev backend which executes crypto operations from
+        the QEMU cipher APIs. The id parameter is a unique ID that will
          be used to reference this cryptodev backend from the
          ``virtio-crypto`` device. The queues parameter is optional,
          which specify the queue number of cryptodev backend, the default
@@ -5516,7 +5516,7 @@ SRST
Finally the ``/etc/qemu/vnc.allow`` file would contain the list
-        of x509 distingished names that are permitted access
+        of x509 distinguished names that are permitted access
:: diff --git a/qga/qapi-schema.json b/qga/qapi-schema.json
index 026a56f76c..796434ed34 100644
--- a/qga/qapi-schema.json
+++ b/qga/qapi-schema.json
@@ -392,7 +392,7 @@
  # @guest-file-flush:
-# Write file changes bufferred in userspace to disk/kernel buffers
+# Write file changes buffered in userspace to disk/kernel buffers
  # @handle: filehandle returned by guest-file-open
diff --git a/tests/qtest/libqtest.h b/tests/qtest/libqtest.h
index 65c040e504..1a76bae326 100644
--- a/tests/qtest/libqtest.h
+++ b/tests/qtest/libqtest.h
@@ -727,7 +727,7 @@ bool qtest_has_device(const char *device);
   * qtest_qmp_device_add_qdict:
   * @qts: QTestState instance to operate on
   * @drv: Name of the device that should be added
- * @arguments: QDict with properties for the device to intialize
+ * @arguments: QDict with properties for the device to initialize
   * Generic hot-plugging test via the device_add QMP command with properties
   * supplied in form of QDict. Use NULL for empty properties list.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]