[PATCH 0/6] trivial-patches for 2023-07-16

Michael Tokarev posted 6 patches 9 months, 4 weeks ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20230716115801.1505288-1-mjt@tls.msk.ru
Maintainers: Richard Henderson <richard.henderson@linaro.org>, Paolo Bonzini <pbonzini@redhat.com>, Greg Kurz <groug@kaod.org>, Christian Schoenebeck <qemu_oss@crudebyte.com>, "Cédric Le Goater" <clg@kaod.org>, Peter Maydell <peter.maydell@linaro.org>, Andrew Jeffery <andrew@aj.id.au>, Joel Stanley <joel@jms.id.au>, Halil Pasic <pasic@linux.ibm.com>, Christian Borntraeger <borntraeger@linux.ibm.com>, David Hildenbrand <david@redhat.com>, Thomas Huth <thuth@redhat.com>, Laurent Vivier <laurent@vivier.eu>, "Edgar E. Iglesias" <edgar.iglesias@gmail.com>, Beniamino Galvani <b.galvani@gmail.com>, Strahinja Jankovic <strahinja.p.jankovic@gmail.com>, Igor Mitsyanko <i.mitsyanko@gmail.com>, Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>, Artyom Tarasenko <atar4qemu@gmail.com>, Andrey Smirnov <andrew.smirnov@gmail.com>, Matthew Rosato <mjrosato@linux.ibm.com>, Eric Farman <farman@linux.ibm.com>, Juan Quintela <quintela@redhat.com>, Peter Xu <peterx@redhat.com>, Leonardo Bras <leobras@redhat.com>, Song Gao <gaosong@loongson.cn>, Xiaojuan Yang <yangxiaojuan@loongson.cn>, Stafford Horne <shorne@gmail.com>, Yoshinori Sato <ysato@users.sourceforge.jp>, Ilya Leoshkevich <iii@linux.ibm.com>, Bastian Koppelmann <kbastian@mail.uni-paderborn.de>, "Alex Bennée" <alex.bennee@linaro.org>, "Philippe Mathieu-Daudé" <philmd@linaro.org>, Eduardo Habkost <eduardo@habkost.net>
docs/system/target-riscv.rst                  | 24 ++++++++++++-------
host/include/aarch64/host/cpuinfo.h           |  2 +-
host/include/generic/host/cpuinfo.h           |  2 +-
hw/9pfs/9p-local.c                            |  8 +++----
hw/9pfs/9p-proxy.c                            |  2 +-
hw/9pfs/9p-synth.c                            |  2 +-
hw/9pfs/9p-util.h                             |  2 +-
hw/9pfs/9p.c                                  |  4 ++--
hw/9pfs/9p.h                                  |  2 +-
hw/arm/aspeed.c                               |  2 +-
hw/arm/mps2-tz.c                              |  2 +-
hw/intc/arm_gic.c                             |  4 ++--
hw/intc/arm_gicv3_redist.c                    |  2 +-
hw/intc/armv7m_nvic.c                         |  2 +-
hw/intc/s390_flic_kvm.c                       |  2 +-
hw/m68k/next-cube.c                           |  2 +-
hw/m68k/next-kbd.c                            |  2 +-
hw/m68k/virt.c                                |  2 +-
hw/microblaze/petalogix_ml605_mmu.c           |  2 +-
hw/misc/allwinner-r40-dramc.c                 |  2 +-
hw/misc/exynos4210_rng.c                      |  2 +-
hw/sparc/sun4m_iommu.c                        |  8 +++----
include/hw/arm/fsl-imx7.h                     |  2 +-
include/hw/intc/armv7m_nvic.h                 |  2 +-
include/hw/s390x/s390-pci-bus.h               |  2 +-
include/hw/s390x/sclp.h                       |  2 +-
migration/migration-stats.c                   |  2 +-
migration/migration.h                         |  4 ++--
migration/multifd-zlib.c                      |  2 +-
migration/multifd-zstd.c                      |  2 +-
migration/multifd.c                           |  2 +-
migration/savevm.c                            |  2 +-
migration/trace-events                        |  2 +-
target/alpha/cpu.h                            |  4 ++--
target/alpha/translate.c                      |  2 +-
target/arm/cpu.c                              |  2 +-
target/arm/cpu.h                              |  2 +-
target/arm/cpu64.c                            |  2 +-
target/arm/helper.c                           |  4 ++--
target/arm/tcg/m_helper.c                     |  2 +-
target/arm/tcg/translate-a64.c                |  4 ++--
target/arm/tcg/translate-mve.c                |  4 ++--
target/arm/tcg/translate-sve.c                |  2 +-
target/arm/tcg/translate-vfp.c                |  2 +-
target/arm/tcg/vec_helper.c                   |  2 +-
target/cris/helper.c                          |  6 ++---
target/cris/op_helper.c                       |  2 +-
target/cris/translate.c                       | 16 ++++++-------
target/hppa/cpu.h                             |  4 ++--
target/hppa/int_helper.c                      |  2 +-
target/hppa/translate.c                       |  2 +-
target/loongarch/cpu-csr.h                    |  2 +-
target/m68k/helper.c                          |  4 ++--
target/microblaze/cpu.h                       |  2 +-
target/openrisc/cpu.h                         |  2 +-
target/openrisc/translate.c                   |  4 ++--
target/rx/translate.c                         |  4 ++--
target/s390x/cpu_features.c                   |  2 +-
target/s390x/cpu_models.c                     |  2 +-
target/s390x/tcg/fpu_helper.c                 |  2 +-
target/s390x/tcg/insn-data.h.inc              |  2 +-
target/s390x/tcg/translate.c                  |  2 +-
target/sparc/asi.h                            | 14 +++++------
target/tricore/csfr.h.inc                     |  2 +-
target/tricore/helper.c                       |  2 +-
target/tricore/translate.c                    |  8 +++----
tests/tcg/Makefile.target                     |  4 ++--
tests/tcg/aarch64/gdbstub/test-sve.py         |  2 +-
tests/tcg/aarch64/sme-outprod1.c              |  2 +-
tests/tcg/aarch64/system/boot.S               |  6 ++---
tests/tcg/aarch64/system/semiheap.c           |  2 +-
tests/tcg/multiarch/sha512.c                  |  2 +-
.../multiarch/system/Makefile.softmmu-target  |  2 +-
tests/tcg/s390x/mvc.c                         |  2 +-
tests/tcg/tricore/c/crt0-tc2x.S               |  4 ++--
tests/tcg/x86_64/system/boot.S                |  4 ++--
76 files changed, 131 insertions(+), 123 deletions(-)
[PATCH 0/6] trivial-patches for 2023-07-16
Posted by Michael Tokarev 9 months, 4 weeks ago
The following changes since commit 7d07a21ec003724475566073404c5893e36de5e5:

  tree-wide spelling fixes in comments and some messages: hw/9pfs (2023-07-16 13:59:17 +0300)

are available in the Git repository at:

  https://gitlab.com/mjt0k/qemu.git/ tags/trivial-patches-pull

for you to fetch changes up to 7d07a21ec003724475566073404c5893e36de5e5:

  tree-wide spelling fixes in comments and some messages: hw/9pfs (2023-07-16 13:59:17 +0300)

----------------------------------------------------------------
trivial-patches for 2023-07-16

This contains a doc fix for riscv and reviewed spelling fixes.
----------------------------------------------------------------

Daniel Henrique Barboza (1):
  docs/system/target-riscv.rst: tidy CPU firmware section

Michael Tokarev (5):
  tree-wide spelling fixes in comments and some messages: migration/
  tree-wide spelling fixes in comments and some messages: s390x
  tree-wide spelling fixes in comments and some messages: arm
  tree-wide spelling fixes in comments and some messages: other
    architectures
  tree-wide spelling fixes in comments and some messages: hw/9pfs

 docs/system/target-riscv.rst                  | 24 ++++++++++++-------
 host/include/aarch64/host/cpuinfo.h           |  2 +-
 host/include/generic/host/cpuinfo.h           |  2 +-
 hw/9pfs/9p-local.c                            |  8 +++----
 hw/9pfs/9p-proxy.c                            |  2 +-
 hw/9pfs/9p-synth.c                            |  2 +-
 hw/9pfs/9p-util.h                             |  2 +-
 hw/9pfs/9p.c                                  |  4 ++--
 hw/9pfs/9p.h                                  |  2 +-
 hw/arm/aspeed.c                               |  2 +-
 hw/arm/mps2-tz.c                              |  2 +-
 hw/intc/arm_gic.c                             |  4 ++--
 hw/intc/arm_gicv3_redist.c                    |  2 +-
 hw/intc/armv7m_nvic.c                         |  2 +-
 hw/intc/s390_flic_kvm.c                       |  2 +-
 hw/m68k/next-cube.c                           |  2 +-
 hw/m68k/next-kbd.c                            |  2 +-
 hw/m68k/virt.c                                |  2 +-
 hw/microblaze/petalogix_ml605_mmu.c           |  2 +-
 hw/misc/allwinner-r40-dramc.c                 |  2 +-
 hw/misc/exynos4210_rng.c                      |  2 +-
 hw/sparc/sun4m_iommu.c                        |  8 +++----
 include/hw/arm/fsl-imx7.h                     |  2 +-
 include/hw/intc/armv7m_nvic.h                 |  2 +-
 include/hw/s390x/s390-pci-bus.h               |  2 +-
 include/hw/s390x/sclp.h                       |  2 +-
 migration/migration-stats.c                   |  2 +-
 migration/migration.h                         |  4 ++--
 migration/multifd-zlib.c                      |  2 +-
 migration/multifd-zstd.c                      |  2 +-
 migration/multifd.c                           |  2 +-
 migration/savevm.c                            |  2 +-
 migration/trace-events                        |  2 +-
 target/alpha/cpu.h                            |  4 ++--
 target/alpha/translate.c                      |  2 +-
 target/arm/cpu.c                              |  2 +-
 target/arm/cpu.h                              |  2 +-
 target/arm/cpu64.c                            |  2 +-
 target/arm/helper.c                           |  4 ++--
 target/arm/tcg/m_helper.c                     |  2 +-
 target/arm/tcg/translate-a64.c                |  4 ++--
 target/arm/tcg/translate-mve.c                |  4 ++--
 target/arm/tcg/translate-sve.c                |  2 +-
 target/arm/tcg/translate-vfp.c                |  2 +-
 target/arm/tcg/vec_helper.c                   |  2 +-
 target/cris/helper.c                          |  6 ++---
 target/cris/op_helper.c                       |  2 +-
 target/cris/translate.c                       | 16 ++++++-------
 target/hppa/cpu.h                             |  4 ++--
 target/hppa/int_helper.c                      |  2 +-
 target/hppa/translate.c                       |  2 +-
 target/loongarch/cpu-csr.h                    |  2 +-
 target/m68k/helper.c                          |  4 ++--
 target/microblaze/cpu.h                       |  2 +-
 target/openrisc/cpu.h                         |  2 +-
 target/openrisc/translate.c                   |  4 ++--
 target/rx/translate.c                         |  4 ++--
 target/s390x/cpu_features.c                   |  2 +-
 target/s390x/cpu_models.c                     |  2 +-
 target/s390x/tcg/fpu_helper.c                 |  2 +-
 target/s390x/tcg/insn-data.h.inc              |  2 +-
 target/s390x/tcg/translate.c                  |  2 +-
 target/sparc/asi.h                            | 14 +++++------
 target/tricore/csfr.h.inc                     |  2 +-
 target/tricore/helper.c                       |  2 +-
 target/tricore/translate.c                    |  8 +++----
 tests/tcg/Makefile.target                     |  4 ++--
 tests/tcg/aarch64/gdbstub/test-sve.py         |  2 +-
 tests/tcg/aarch64/sme-outprod1.c              |  2 +-
 tests/tcg/aarch64/system/boot.S               |  6 ++---
 tests/tcg/aarch64/system/semiheap.c           |  2 +-
 tests/tcg/multiarch/sha512.c                  |  2 +-
 .../multiarch/system/Makefile.softmmu-target  |  2 +-
 tests/tcg/s390x/mvc.c                         |  2 +-
 tests/tcg/tricore/c/crt0-tc2x.S               |  4 ++--
 tests/tcg/x86_64/system/boot.S                |  4 ++--
 76 files changed, 131 insertions(+), 123 deletions(-)

-- 
2.39.2
Re: [PATCH 0/6] trivial-patches for 2023-07-16
Posted by Michael Tokarev 9 months, 4 weeks ago
16.07.2023 14:57, Michael Tokarev пишет:
> The following changes since commit 7d07a21ec003724475566073404c5893e36de5e5:
> 
>    tree-wide spelling fixes in comments and some messages: hw/9pfs (2023-07-16 13:59:17 +0300)
> 
> are available in the Git repository at:
> 
>    https://gitlab.com/mjt0k/qemu.git/ tags/trivial-patches-pull
> 
> for you to fetch changes up to 7d07a21ec003724475566073404c5893e36de5e5:
> 
>    tree-wide spelling fixes in comments and some messages: hw/9pfs (2023-07-16 13:59:17 +0300)
> 
> ----------------------------------------------------------------
> trivial-patches for 2023-07-16
> 
> This contains a doc fix for riscv and reviewed spelling fixes.
> ----------------------------------------------------------------

This meant to be a PULL request, not PATCH series.. :)

Thanks,

/mjt

Re: [PATCH 0/6] trivial-patches for 2023-07-16
Posted by Thomas Huth 9 months, 2 weeks ago
On 16/07/2023 14.05, Michael Tokarev wrote:
> 16.07.2023 14:57, Michael Tokarev пишет:
>> The following changes since commit 7d07a21ec003724475566073404c5893e36de5e5:
>>
>>    tree-wide spelling fixes in comments and some messages: hw/9pfs 
>> (2023-07-16 13:59:17 +0300)
>>
>> are available in the Git repository at:
>>
>>    https://gitlab.com/mjt0k/qemu.git/ tags/trivial-patches-pull
>>
>> for you to fetch changes up to 7d07a21ec003724475566073404c5893e36de5e5:
>>
>>    tree-wide spelling fixes in comments and some messages: hw/9pfs 
>> (2023-07-16 13:59:17 +0300)
>>
>> ----------------------------------------------------------------
>> trivial-patches for 2023-07-16
>>
>> This contains a doc fix for riscv and reviewed spelling fixes.
>> ----------------------------------------------------------------
> 
> This meant to be a PULL request, not PATCH series.. :)

I guess you have to resend with a PULL in the subject?

  Thomas



Re: [PATCH 0/6] trivial-patches for 2023-07-16
Posted by Philippe Mathieu-Daudé 9 months, 4 weeks ago
On 16/7/23 13:57, Michael Tokarev wrote:
> The following changes since commit 7d07a21ec003724475566073404c5893e36de5e5:
> 
>    tree-wide spelling fixes in comments and some messages: hw/9pfs (2023-07-16 13:59:17 +0300)
> 
> are available in the Git repository at:
> 
>    https://gitlab.com/mjt0k/qemu.git/ tags/trivial-patches-pull
> 
> for you to fetch changes up to 7d07a21ec003724475566073404c5893e36de5e5:
> 
>    tree-wide spelling fixes in comments and some messages: hw/9pfs (2023-07-16 13:59:17 +0300)
> 
> ----------------------------------------------------------------
> trivial-patches for 2023-07-16
> 
> This contains a doc fix for riscv and reviewed spelling fixes.
> ----------------------------------------------------------------
> 
> Daniel Henrique Barboza (1):
>    docs/system/target-riscv.rst: tidy CPU firmware section
> 
> Michael Tokarev (5):
>    tree-wide spelling fixes in comments and some messages: migration/
>    tree-wide spelling fixes in comments and some messages: s390x
>    tree-wide spelling fixes in comments and some messages: arm
>    tree-wide spelling fixes in comments and some messages: other
>      architectures
>    tree-wide spelling fixes in comments and some messages: hw/9pfs

FYI patch subject is usually "subsystem: Topic", see
https://www.qemu.org/docs/master/devel/submitting-a-patch.html#write-a-meaningful-commit-message:

   QEMU follows the usual standard for git commit messages: the first
   line (which becomes the email subject line) is “subsystem: single
   line summary of change”.

(not really a cosmetics comment, but various developers have mail
  filters written using this pattern).

Re: [PATCH 0/6] trivial-patches for 2023-07-16
Posted by Michael Tokarev 9 months, 3 weeks ago
16.07.2023 18:58, Philippe Mathieu-Daudé wrote:
...
>> Michael Tokarev (5):
>>    tree-wide spelling fixes in comments and some messages: migration/
>>    tree-wide spelling fixes in comments and some messages: s390x
>>    tree-wide spelling fixes in comments and some messages: arm
>>    tree-wide spelling fixes in comments and some messages: other
>>      architectures
>>    tree-wide spelling fixes in comments and some messages: hw/9pfs
> 
> FYI patch subject is usually "subsystem: Topic", see
> https://www.qemu.org/docs/master/devel/submitting-a-patch.html#write-a-meaningful-commit-message:
> 
>    QEMU follows the usual standard for git commit messages: the first
>    line (which becomes the email subject line) is “subsystem: single
>    line summary of change”.

Yes Philippe, I know. In this case though, it really is "tree-wide". I tried
to group them by subsystem but it doesn't work that well.  Especially having
in mind how many changes there are (about 400 in total).

This particular series is a pull request, not a review request (I just
forgot to add --subject-prefix=PULL when generating this one).  This is
the changes which has been reviewed by at least one person, out of the
other pile. The others are here, for example:

   https://lists.nongnu.org/archive/html/qemu-devel/2023-07/msg03006.html

and see comments by Peter there:

  https://lists.nongnu.org/archive/html/qemu-devel/2023-07/msg03050.html

My plan is to get the reviewed parts applied, and re-send the rest
once again. This is a huge work already to create the changes to
begin with, and to review them as well.

The initial RFC:

  https://lists.nongnu.org/archive/html/qemu-devel/2023-07/msg02841.html

> (not really a cosmetics comment, but various developers have mail
>   filters written using this pattern).

It's hardly possible to reliable filter by subsystem, because there's
no formal subsystems defined and the "subsystem:" prefix in emails is
really arbitrary.

/mjt

Re: [PATCH 0/6] trivial-patches for 2023-07-16
Posted by Peter Maydell 9 months, 2 weeks ago
On Mon, 17 Jul 2023 at 10:50, Michael Tokarev <mjt@tls.msk.ru> wrote:
>
> 16.07.2023 18:58, Philippe Mathieu-Daudé wrote:
> ...
> >> Michael Tokarev (5):
> >>    tree-wide spelling fixes in comments and some messages: migration/
> >>    tree-wide spelling fixes in comments and some messages: s390x
> >>    tree-wide spelling fixes in comments and some messages: arm
> >>    tree-wide spelling fixes in comments and some messages: other
> >>      architectures
> >>    tree-wide spelling fixes in comments and some messages: hw/9pfs
> >
> > FYI patch subject is usually "subsystem: Topic", see
> > https://www.qemu.org/docs/master/devel/submitting-a-patch.html#write-a-meaningful-commit-message:
> >
> >    QEMU follows the usual standard for git commit messages: the first
> >    line (which becomes the email subject line) is “subsystem: single
> >    line summary of change”.
>
> Yes Philippe, I know. In this case though, it really is "tree-wide". I tried
> to group them by subsystem but it doesn't work that well.  Especially having
> in mind how many changes there are (about 400 in total).

But you have the subsystem name, you just put it on the wrong end
of the subject line. You could fix this up if you're going
to send these as a pull request...

thanks
-- PMM