misc: Replace sprintf

[PATCH v3 00/27] misc: Replace sprintf

Richard Henderson posted 27 patches 2 weeks, 3 days ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20240412073346.458116-1-richard.henderson@linaro.org

Maintainers: Stefan Berger <stefanb@linux.vnet.ibm.com>, Laurent Vivier <laurent@vivier.eu>, "Edgar E. Iglesias" <edgar.iglesias@gmail.com>, Palmer Dabbelt <palmer@dabbelt.com>, Alistair Francis <Alistair.Francis@wdc.com>, Peter Maydell <peter.maydell@linaro.org>, John Snow <jsnow@redhat.com>, "Philippe Mathieu-Daudé" <philmd@linaro.org>, Aurelien Jarno <aurelien@aurel32.net>, Jean-Christophe Dubois <jcd@tribudubois.net>, Andrey Smirnov <andrew.smirnov@gmail.com>, Jason Wang <jasowang@redhat.com>, Jiri Pirko <jiri@resnulli.us>, Bin Meng <bin.meng@windriver.com>, Weiwei Li <liwei1518@gmail.com>, Daniel Henrique Barboza <dbarboza@ventanamicro.com>, Liu Zhiwei <zhiwei_liu@linux.alibaba.com>, Paolo Bonzini <pbonzini@redhat.com>, Fam Zheng <fam@euphon.net>, "Michael S. Tsirkin" <mst@redhat.com>, Thomas Huth <thuth@redhat.com>, Marcelo Tosatti <mtosatti@redhat.com>

include/qemu/cutils.h   |  20 +-
backends/tpm/tpm_util.c |  24 +-
disas/m68k.c            |   2 +-
disas/microblaze.c      | 544 +++++++++++++++++-----------------------
disas/riscv.c           | 209 +++++++--------
hw/dma/pl330.c          |  23 +-
hw/ide/atapi.c          |  12 +-
hw/mips/malta.c         |  25 +-
hw/misc/imx25_ccm.c     |   2 +-
hw/misc/imx31_ccm.c     |   2 +-
hw/misc/imx6_ccm.c      |   4 +-
hw/misc/imx6_src.c      |   2 +-
hw/misc/imx6ul_ccm.c    |   4 +-
hw/misc/imx7_src.c      |   2 +-
hw/net/imx_fec.c        |   2 +-
hw/net/rocker/rocker.c  |  24 +-
hw/riscv/virt.c         |  17 +-
hw/scsi/scsi-disk.c     |  13 +-
hw/ssi/imx_spi.c        |   2 +-
hw/virtio/vhost-vdpa.c  |  14 +-
linux-user/flatload.c   |   2 +-
system/qtest.c          |  12 +-
target/arm/cpu64.c      |   4 +-
target/i386/kvm/kvm.c   |  12 +-
util/hexdump.c          |  98 +++++---
hw/virtio/trace-events  |   2 +-
26 files changed, 494 insertions(+), 583 deletions(-)

Expand all Fold all

[PATCH v3 00/27] misc: Replace sprintf

Posted by Richard Henderson 2 weeks, 3 days ago

Hi Phil,

This incorporates my comments against
    20240411104340.6617-1-philmd@linaro.org
and
    20240411101550.99392-1-philmd@linaro.org

Especially cleanups to hexdup and disas.


r~


Philippe Mathieu-Daudé (13):
  hw/mips/malta: Add re-usable rng_seed_hex_new() method
  system/qtest: Replace sprintf by qemu_hexdump_line
  hw/scsi/scsi-disk: Use qemu_hexdump_line to avoid sprintf
  hw/ide/atapi: Use qemu_hexdump_line to avoid sprintf
  hw/dma/pl330: Use qemu_hexdump_line to avoid sprintf
  backends/tpm: Use qemu_hexdump_line to avoid sprintf
  disas/m68k: Replace sprintf() by snprintf()
  disas/microblaze: Replace sprintf() by snprintf()
  linux-user/flatload: Replace sprintf() by snprintf()
  hw/misc/imx: Replace sprintf() by snprintf()
  hw/net/rocker: Replace sprintf() by snprintf()
  hw/riscv/virt: Replace sprintf by g_strdup_printf
  target/arm: Replace sprintf() by snprintf()

Richard Henderson (14):
  util/hexdump: Remove b parameter from qemu_hexdump_line
  util/hexdump: Remove ascii parameter from qemu_hexdump_line
  util/hexdump: Use a GString for qemu_hexdump_line
  util/hexdump: Add unit_len and block_len to qemu_hexdump_line
  util/hexdump: Inline g_string_append_printf "%02x"
  disas/microblaze: Split out print_immval_addr
  target/microblaze: Re-indent print_insn_microblaze
  disas/microblaze: Merge op->name output into each fprintf
  disas/microblaze: Print registers directly with PRIreg
  disas/microblaze: Print immediates directly with PRIimm
  disas/microblaze: Print registers directly with PRIrfsl
  disas/microblaze: Split get_field_special
  disas/riscv: Use GString in format_inst
  target/i386/kvm: Improve KVM_EXIT_NOTIFY warnings

 include/qemu/cutils.h   |  20 +-
 backends/tpm/tpm_util.c |  24 +-
 disas/m68k.c            |   2 +-
 disas/microblaze.c      | 544 +++++++++++++++++-----------------------
 disas/riscv.c           | 209 +++++++--------
 hw/dma/pl330.c          |  23 +-
 hw/ide/atapi.c          |  12 +-
 hw/mips/malta.c         |  25 +-
 hw/misc/imx25_ccm.c     |   2 +-
 hw/misc/imx31_ccm.c     |   2 +-
 hw/misc/imx6_ccm.c      |   4 +-
 hw/misc/imx6_src.c      |   2 +-
 hw/misc/imx6ul_ccm.c    |   4 +-
 hw/misc/imx7_src.c      |   2 +-
 hw/net/imx_fec.c        |   2 +-
 hw/net/rocker/rocker.c  |  24 +-
 hw/riscv/virt.c         |  17 +-
 hw/scsi/scsi-disk.c     |  13 +-
 hw/ssi/imx_spi.c        |   2 +-
 hw/virtio/vhost-vdpa.c  |  14 +-
 linux-user/flatload.c   |   2 +-
 system/qtest.c          |  12 +-
 target/arm/cpu64.c      |   4 +-
 target/i386/kvm/kvm.c   |  12 +-
 util/hexdump.c          |  98 +++++---
 hw/virtio/trace-events  |   2 +-
 26 files changed, 494 insertions(+), 583 deletions(-)

-- 
2.34.1