Re: [PATCH 2/7] i386: spelling fixes

[Peter Maydell]

On Sat, 9 Sept 2023 at 14:17, Michael Tokarev <mjt@tls.msk.ru> wrote:
>
> Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
> ---
>  host/include/i386/host/cpuinfo.h | 2 +-
>  hw/i386/acpi-build.c             | 4 ++--
>  hw/i386/amd_iommu.c              | 4 ++--
>  hw/i386/intel_iommu.c            | 4 ++--
>  hw/i386/kvm/xen_xenstore.c       | 2 +-
>  hw/i386/kvm/xenstore_impl.c      | 2 +-
>  hw/i386/pc.c                     | 4 ++--
>  include/hw/i386/topology.h       | 2 +-
>  target/i386/cpu.c                | 4 ++--
>  target/i386/cpu.h                | 4 ++--
>  target/i386/kvm/kvm.c            | 4 ++--
>  target/i386/kvm/xen-emu.c        | 2 +-
>  target/i386/machine.c            | 4 ++--
>  target/i386/tcg/translate.c      | 8 ++++----
>  tests/tcg/i386/system/boot.S     | 2 +-
>  tests/tcg/i386/x86.csv           | 2 +-
>  16 files changed, 27 insertions(+), 27 deletions(-)

> diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
> index bb12b0ad43..1afcef5937 100644
> --- a/hw/i386/acpi-build.c
> +++ b/hw/i386/acpi-build.c
> @@ -779,7 +779,7 @@ static Aml *initialize_route(Aml *route, const char 
> *link_name,
>   *
>   * Returns an array of 128 routes, one for each device,
>   * based on device location.
> - * The main goal is to equaly distribute the interrupts
> + * The main goal is to equally distribute the interrupts
>   * over the 4 existing ACPI links (works only for i440fx).
>   * The hash function is  (slot + pin) & 3 -> "LNK[D|A|B|C]".
>   *
> @@ -2079,7 +2079,7 @@ build_srat(GArray *table_data, BIOSLinker *linker, 
> MachineState *machine)
>  }
>
>  /*
> - * Insert DMAR scope for PCI bridges and endpoint devcie
> + * Insert DMAR scope for PCI bridges and endpoint device

I suspect this should be "devices" plural.

>   */
>  static void
>  insert_scope(PCIBus *bus, PCIDevice *dev, void *opaque)

>  static void ioportF0_write(void *opaque, hwaddr addr, uint64_t data,
>                             unsigned size)
>  {

> --- a/target/i386/kvm/kvm.c
> +++ b/target/i386/kvm/kvm.c
> @@ -4729,7 +4729,7 @@ int kvm_arch_put_registers(CPUState *cpu, int level)
>      /*
>       * Put MSR_IA32_FEATURE_CONTROL first, this ensures the VM gets out of 
> VMX
>       * root operation upon vCPU reset. kvm_put_msr_feature_control() should 
> also
> -     * preceed kvm_put_nested_state() when 'real' nested state is set.
> +     * proceed kvm_put_nested_state() when 'real' nested state is set.

Should be "precede".

>       */
>      if (level >= KVM_PUT_RESET_STATE) {
>          ret = kvm_put_msr_feature_control(x86_cpu);

Otherwise
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>

thanks
-- PMM