[Qemu-commits] [qemu/qemu] 498432: target/i386/cpu: Remove "x86" prefix from the CPU ...

2024-04-30 Thread Richard Henderson via Qemu-commits
  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 49843214368eccbc46ad3946aae8cc81eaced98e
  
https://github.com/qemu/qemu/commit/49843214368eccbc46ad3946aae8cc81eaced98e
  Author: Thomas Huth 
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M target/i386/cpu.c

  Log Message:
  ---
  target/i386/cpu: Remove "x86" prefix from the CPU list

Printing an "x86" in front of each CPU name is not helpful at all:
It is confusing for the users since they don't know whether they
have to specify these letters for the "-cpu" parameter, too, and
it also takes some precious space in the dense output of the CPU
entries. Let's simply remove this now and use two spaces at the
beginning of the lines for the indentation of the entries instead,
like most other target architectures are doing it for their CPU help
output already.

Signed-off-by: Thomas Huth 
Reviewed-by: Richard Henderson 
Reviewed-by: Michael Tokarev 
Signed-off-by: Michael Tokarev 


  Commit: 7febce361da200fa71e71e40316e4a6d2a5b40ab
  
https://github.com/qemu/qemu/commit/7febce361da200fa71e71e40316e4a6d2a5b40ab
  Author: Thomas Huth 
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M target/s390x/cpu_models.c

  Log Message:
  ---
  target/s390x/cpu_models: Rework the output of "-cpu help"

Printing an "s390x" in front of each CPU name is not helpful at all:
It is confusing for the users since they don't know whether they
have to specify these letters for the "-cpu" parameter, too, and
it also takes some precious space in the dense output of the CPU
entries. Let's simply remove this now!

While we're at it, use two spaces at the beginning of the lines for
the indentation of the entries, and add a "Available CPUs" in the
very first line, like most other target architectures are doing it
for their "-cpu help" output already.

Signed-off-by: Thomas Huth 
Reviewed-by: Richard Henderson 
Reviewed-by: Michael Tokarev 
Signed-off-by: Michael Tokarev 


  Commit: 5b638f6e900efd1d5f5d0697af69a0e9eb2bfc72
  
https://github.com/qemu/qemu/commit/5b638f6e900efd1d5f5d0697af69a0e9eb2bfc72
  Author: Thomas Huth 
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M target/ppc/cpu_init.c

  Log Message:
  ---
  target/ppc/cpu_init: Remove "PowerPC" prefix from the CPU list

Printing a "PowerPC" in front of each CPU name is not helpful at all:
It is confusing for the users since they don't know whether they
have to specify these letters for the "-cpu" parameter, too, and
it also takes some precious space in the dense output of the CPU
entries. Let's simply remove this now and use two spaces at the
beginning of the lines for the indentation of the entries instead,
and add a "Available CPUs" in the very first line, like most other
target architectures are doing it for their CPU help output already.

Signed-off-by: Thomas Huth 
Reviewed-by: Richard Henderson 
Reviewed-by: Michael Tokarev 
Signed-off-by: Michael Tokarev 


  Commit: e3812d109663b8fee28e9334bbc12f684355984e
  
https://github.com/qemu/qemu/commit/e3812d109663b8fee28e9334bbc12f684355984e
  Author: Philippe Mathieu-Daudé 
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M scripts/checkpatch.pl

  Log Message:
  ---
  scripts/checkpatch: Avoid author email mangled by qemu-*@nongnu.org

Commit f5177798d8 ("scripts: report on author emails
that are mangled by the mailing list") added a check
for qemu-devel@ list, extend the regexp to cover more
such qemu-trivial@, qemu-block@ and qemu-ppc@.

Signed-off-by: Philippe Mathieu-Daudé 
Reviewed-by: Michael Tokarev 
Signed-off-by: Michael Tokarev 


  Commit: af692fd338154a20010c78a6bd9acb2c889dd4e7
  
https://github.com/qemu/qemu/commit/af692fd338154a20010c78a6bd9acb2c889dd4e7
  Author: Philippe Mathieu-Daudé 
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M scripts/checkpatch.pl

  Log Message:
  ---
  scripts/checkpatch: Do not use mailmap

The .mailmap file fixes mistake we already did.
Do not use it when running checkpatch.pl, otherwise
we might commit the very same mistakes.

Reported-by: Peter Maydell 
Signed-off-by: Philippe Mathieu-Daudé 
Reviewed-by: Michael Tokarev 
Signed-off-by: Michael Tokarev 


  Commit: 06479dbf3d7d245572c4b3016e5a1d923ff04d66
  
https://github.com/qemu/qemu/commit/06479dbf3d7d245572c4b3016e5a1d923ff04d66
  Author: Li Zhijian 
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M backends/cryptodev-builtin.c

  Log Message:
  ---
  backends/cryptodev-builtin: Fix local_error leaks

It seems that this error does not need to be propagated to the upper,
directly output the error to avoid the leaks

Closes: https://gitlab.com/qemu-project/qemu/-/issues/2283
Fixes: 2fda101de07 ("virtio-crypto: Support asynchronous mode")
Signed-off-by: Li Zhijian 
Reviewed-by: Philippe Mathieu-Daudé 
Reviewed-by: zhenwei pi 
Reviewed-by: Michael Tokarev 
Signed-off-by: Michael Tokarev 


  

[Qemu-commits] [qemu/qemu] 498432: target/i386/cpu: Remove "x86" prefix from the CPU ...

2024-04-29 Thread Richard Henderson via Qemu-commits
  Branch: refs/heads/staging
  Home:   https://github.com/qemu/qemu
  Commit: 49843214368eccbc46ad3946aae8cc81eaced98e
  
https://github.com/qemu/qemu/commit/49843214368eccbc46ad3946aae8cc81eaced98e
  Author: Thomas Huth 
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M target/i386/cpu.c

  Log Message:
  ---
  target/i386/cpu: Remove "x86" prefix from the CPU list

Printing an "x86" in front of each CPU name is not helpful at all:
It is confusing for the users since they don't know whether they
have to specify these letters for the "-cpu" parameter, too, and
it also takes some precious space in the dense output of the CPU
entries. Let's simply remove this now and use two spaces at the
beginning of the lines for the indentation of the entries instead,
like most other target architectures are doing it for their CPU help
output already.

Signed-off-by: Thomas Huth 
Reviewed-by: Richard Henderson 
Reviewed-by: Michael Tokarev 
Signed-off-by: Michael Tokarev 


  Commit: 7febce361da200fa71e71e40316e4a6d2a5b40ab
  
https://github.com/qemu/qemu/commit/7febce361da200fa71e71e40316e4a6d2a5b40ab
  Author: Thomas Huth 
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M target/s390x/cpu_models.c

  Log Message:
  ---
  target/s390x/cpu_models: Rework the output of "-cpu help"

Printing an "s390x" in front of each CPU name is not helpful at all:
It is confusing for the users since they don't know whether they
have to specify these letters for the "-cpu" parameter, too, and
it also takes some precious space in the dense output of the CPU
entries. Let's simply remove this now!

While we're at it, use two spaces at the beginning of the lines for
the indentation of the entries, and add a "Available CPUs" in the
very first line, like most other target architectures are doing it
for their "-cpu help" output already.

Signed-off-by: Thomas Huth 
Reviewed-by: Richard Henderson 
Reviewed-by: Michael Tokarev 
Signed-off-by: Michael Tokarev 


  Commit: 5b638f6e900efd1d5f5d0697af69a0e9eb2bfc72
  
https://github.com/qemu/qemu/commit/5b638f6e900efd1d5f5d0697af69a0e9eb2bfc72
  Author: Thomas Huth 
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M target/ppc/cpu_init.c

  Log Message:
  ---
  target/ppc/cpu_init: Remove "PowerPC" prefix from the CPU list

Printing a "PowerPC" in front of each CPU name is not helpful at all:
It is confusing for the users since they don't know whether they
have to specify these letters for the "-cpu" parameter, too, and
it also takes some precious space in the dense output of the CPU
entries. Let's simply remove this now and use two spaces at the
beginning of the lines for the indentation of the entries instead,
and add a "Available CPUs" in the very first line, like most other
target architectures are doing it for their CPU help output already.

Signed-off-by: Thomas Huth 
Reviewed-by: Richard Henderson 
Reviewed-by: Michael Tokarev 
Signed-off-by: Michael Tokarev 


  Commit: e3812d109663b8fee28e9334bbc12f684355984e
  
https://github.com/qemu/qemu/commit/e3812d109663b8fee28e9334bbc12f684355984e
  Author: Philippe Mathieu-Daudé 
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M scripts/checkpatch.pl

  Log Message:
  ---
  scripts/checkpatch: Avoid author email mangled by qemu-*@nongnu.org

Commit f5177798d8 ("scripts: report on author emails
that are mangled by the mailing list") added a check
for qemu-devel@ list, extend the regexp to cover more
such qemu-trivial@, qemu-block@ and qemu-ppc@.

Signed-off-by: Philippe Mathieu-Daudé 
Reviewed-by: Michael Tokarev 
Signed-off-by: Michael Tokarev 


  Commit: af692fd338154a20010c78a6bd9acb2c889dd4e7
  
https://github.com/qemu/qemu/commit/af692fd338154a20010c78a6bd9acb2c889dd4e7
  Author: Philippe Mathieu-Daudé 
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M scripts/checkpatch.pl

  Log Message:
  ---
  scripts/checkpatch: Do not use mailmap

The .mailmap file fixes mistake we already did.
Do not use it when running checkpatch.pl, otherwise
we might commit the very same mistakes.

Reported-by: Peter Maydell 
Signed-off-by: Philippe Mathieu-Daudé 
Reviewed-by: Michael Tokarev 
Signed-off-by: Michael Tokarev 


  Commit: 06479dbf3d7d245572c4b3016e5a1d923ff04d66
  
https://github.com/qemu/qemu/commit/06479dbf3d7d245572c4b3016e5a1d923ff04d66
  Author: Li Zhijian 
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M backends/cryptodev-builtin.c

  Log Message:
  ---
  backends/cryptodev-builtin: Fix local_error leaks

It seems that this error does not need to be propagated to the upper,
directly output the error to avoid the leaks

Closes: https://gitlab.com/qemu-project/qemu/-/issues/2283
Fixes: 2fda101de07 ("virtio-crypto: Support asynchronous mode")
Signed-off-by: Li Zhijian 
Reviewed-by: Philippe Mathieu-Daudé 
Reviewed-by: zhenwei pi 
Reviewed-by: Michael Tokarev 
Signed-off-by: Michael Tokarev