[PATCH v3 00/27] misc: Replace sprintf

Richard Henderson posted 27 patches 1 month, 1 week ago
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(-)
[PATCH v3 00/27] misc: Replace sprintf
Posted by Richard Henderson 1 month, 1 week 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