qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH v6 00/29]  Remove some of the fprintf(stderr, "*
@ 2017-12-20 17:22 Alistair Francis
  2017-12-20 17:22 ` [Qemu-devel] [PATCH v6 01/29] audio: Replace AUDIO_FUNC with __func__ Alistair Francis
                   ` (27 more replies)
  0 siblings, 28 replies; 41+ messages in thread
From: Alistair Francis @ 2017-12-20 17:22 UTC (permalink / raw)
  To: qemu-devel, qemu-trivial; +Cc: alistair.francis, alistair23, armbru


This is a smaller version of my original series, to hopefully get
something merged (although it's getting late for 2.11 now).

Continue on improving QEMUs logging/error messages by removing more
fprintf()'s.

Unfortunatley my Coccinelle skills aren't that great so it's all done in
some nasty regex and a little bit of manual work.

V6:
 - Indentation fixes
V5:
 - Small fixes
 - Remove a lot of patches
V4:
 - Reorder patch 3 (the audio func patch)
 - Add all tags
 - Small fixes
V3:
 - Rebase
 - All sorts of style fixes
 - Remove "qemu: " prefixes from print messages
V2:
 - Fix build issues
 - Drop block patches
 - Split the hw patches into subdirectories
 - Remove AUDIO_FUNC macro as well
 - Add some __FUNCTION__ rename fixes in a seperate patch
 - Create a qemu_log() conversion patch for target/



Alistair Francis (29):
  audio: Replace AUDIO_FUNC with __func__
  Replace all occurances of __FUNCTION__ with __func__
  Fixes after renaming __FUNCTION__ to __func__
  hw/arm: Replace fprintf(stderr, "*\n" with error_report()
  hw/dma: Replace fprintf(stderr, "*\n" with error_report()
  hw/gpio: Replace fprintf(stderr, "*\n" with error_report()
  hw/i2c: Replace fprintf(stderr, "*\n" with error_report()
  hw/i386: Replace fprintf(stderr, "*\n" with error_report()
  hw/ide: Replace fprintf(stderr, "*\n" with error_report()
  hw/intc: Replace fprintf(stderr, "*\n" with error_report()
  hw/ipmi: Replace fprintf(stderr, "*\n" with error_report()
  hw/isa: Replace fprintf(stderr, "*\n" with error_report()
  hw/lm32: Replace fprintf(stderr, "*\n" with error_report()
  hw/mips: Replace fprintf(stderr, "*\n" with error_report()
  hw/moxie: Replace fprintf(stderr, "*\n" with error_report()
  hw/nios2: Replace fprintf(stderr, "*\n" with error_report()
  hw/nvram: Replace fprintf(stderr, "*\n" with error_report()
  hw/openrisc: Replace fprintf(stderr, "*\n" with error_report()
  hw/pci*: Replace fprintf(stderr, "*\n" with error_report()
  hw/ppc: Replace fprintf(stderr, "*\n" with error_report()
  hw/s390x: Replace fprintf(stderr, "*\n" with error_report()
  hw/scsi: Replace fprintf(stderr, "*\n" with error_report()
  hw/sd: Replace fprintf(stderr, "*\n" with error_report()
  hw/sparc*: Replace fprintf(stderr, "*\n" with error_report()
  hw/ssi: Replace fprintf(stderr, "*\n" with error_report()
  hw/timer: Replace fprintf(stderr, "*\n" with error_report()
  hw/xen*: Replace fprintf(stderr, "*\n" with error_report()
  tcg: Replace fprintf(stderr, "*\n" with error_report()
  target: Use qemu_log() instead of fprintf(stderr, ...)

 audio/alsaaudio.c               |  4 +--
 audio/audio.c                   | 40 +++++++++++++--------------
 audio/audio_int.h               |  6 ----
 audio/audio_pt_int.c            | 28 +++++++++----------
 audio/audio_template.h          | 26 +++++++++---------
 audio/dsoundaudio.c             |  2 +-
 audio/mixeng.c                  |  2 +-
 audio/ossaudio.c                | 10 +++----
 audio/paaudio.c                 | 56 ++++++++++++++++++-------------------
 audio/sdlaudio.c                |  2 +-
 audio/wavaudio.c                |  2 +-
 cpus.c                          | 10 +++----
 exec.c                          | 20 +++++++-------
 hw/arm/armv7m.c                 |  2 +-
 hw/arm/boot.c                   | 37 ++++++++++++-------------
 hw/arm/gumstix.c                | 13 +++++----
 hw/arm/mainstone.c              |  7 +++--
 hw/arm/musicpal.c               |  2 +-
 hw/arm/nseries.c                |  2 +-
 hw/arm/omap1.c                  | 49 ++++++++++++++++-----------------
 hw/arm/omap2.c                  | 23 ++++++++--------
 hw/arm/omap_sx1.c               |  8 ++----
 hw/arm/palm.c                   | 20 +++++++-------
 hw/arm/pxa2xx.c                 | 53 +++++++++++++++++------------------
 hw/arm/pxa2xx_gpio.c            |  6 ++--
 hw/arm/pxa2xx_pic.c             |  4 +--
 hw/arm/stellaris.c              |  3 +-
 hw/arm/tosa.c                   | 26 +++++++++---------
 hw/arm/versatilepb.c            |  2 +-
 hw/arm/vexpress.c               |  8 +++---
 hw/arm/z2.c                     |  6 ++--
 hw/audio/hda-codec.c            | 10 +++----
 hw/audio/intel-hda.c            | 28 +++++++++----------
 hw/audio/wm8750.c               |  4 +--
 hw/block/nand.c                 |  4 +--
 hw/block/onenand.c              |  8 +++---
 hw/bt/core.c                    | 10 +++----
 hw/bt/hci-csr.c                 | 14 +++++-----
 hw/bt/hci.c                     | 26 +++++++++---------
 hw/bt/hid.c                     |  2 +-
 hw/bt/l2cap.c                   | 22 +++++++--------
 hw/bt/sdp.c                     |  6 ++--
 hw/display/blizzard.c           | 18 ++++++------
 hw/display/omap_dss.c           |  6 ++--
 hw/display/pxa2xx_lcd.c         | 14 +++++-----
 hw/display/qxl-render.c         |  6 ++--
 hw/display/qxl.h                |  2 +-
 hw/display/tc6393xb.c           |  2 +-
 hw/display/xenfb.c              |  2 +-
 hw/dma/omap_dma.c               | 38 +++++++++++++------------
 hw/dma/pxa2xx_dma.c             | 14 +++++-----
 hw/dma/soc_dma.c                | 36 ++++++++++++------------
 hw/gpio/max7310.c               |  8 +++---
 hw/gpio/omap_gpio.c             |  2 +-
 hw/i2c/omap_i2c.c               | 10 +++----
 hw/i386/kvm/apic.c              | 15 +++++-----
 hw/i386/kvm/clock.c             |  7 +++--
 hw/i386/kvm/i8254.c             | 11 ++++----
 hw/i386/kvm/i8259.c             |  5 ++--
 hw/i386/kvm/ioapic.c            |  5 ++--
 hw/i386/multiboot.c             | 21 +++++++-------
 hw/i386/pc.c                    | 24 ++++++++--------
 hw/i386/pc_piix.c               |  2 +-
 hw/i386/pc_sysfw.c              |  4 +--
 hw/i386/xen/xen-hvm.c           | 34 ++++++++++++-----------
 hw/i386/xen/xen-mapcache.c      | 12 ++++----
 hw/i386/xen/xen_apic.c          |  3 +-
 hw/ide/ahci.c                   |  6 ++--
 hw/ide/core.c                   |  3 +-
 hw/ide/microdrive.c             |  4 +--
 hw/input/lm832x.c               |  6 ++--
 hw/input/pxa2xx_keypad.c        |  6 ++--
 hw/input/tsc2005.c              |  8 +++---
 hw/input/tsc210x.c              |  4 +--
 hw/intc/arm_gic_kvm.c           | 12 ++++----
 hw/intc/omap_intc.c             |  4 +--
 hw/intc/openpic.c               |  1 +
 hw/intc/openpic_kvm.c           |  8 +++---
 hw/intc/s390_flic_kvm.c         |  4 +--
 hw/ipmi/ipmi_bmc_extern.c       |  5 ++--
 hw/isa/isa-bus.c                |  7 +++--
 hw/isa/vt82c686.c               |  2 +-
 hw/lm32/lm32_boards.c           |  7 ++---
 hw/lm32/milkymist.c             |  7 ++---
 hw/mips/boston.c                |  4 +--
 hw/mips/gt64xxx_pci.c           |  2 +-
 hw/mips/mips_fulong2e.c         | 13 ++++-----
 hw/mips/mips_jazz.c             |  4 +--
 hw/mips/mips_malta.c            | 16 +++++------
 hw/mips/mips_mipssim.c          | 11 ++++----
 hw/mips/mips_r4k.c              | 18 ++++++------
 hw/misc/cbus.c                  | 12 ++++----
 hw/misc/omap_clk.c              |  4 +--
 hw/misc/omap_gpmc.c             |  6 ++--
 hw/misc/omap_l4.c               |  4 +--
 hw/misc/omap_sdrc.c             |  2 +-
 hw/misc/omap_tap.c              |  6 ++--
 hw/misc/tmp105.c                |  2 +-
 hw/moxie/moxiesim.c             | 14 +++++-----
 hw/nios2/boot.c                 |  2 +-
 hw/nvram/eeprom93xx.c           |  7 +++--
 hw/nvram/fw_cfg.c               |  4 +--
 hw/openrisc/openrisc_sim.c      |  4 +--
 hw/pci-host/bonito.c            |  8 +++---
 hw/pci/pci.c                    | 12 ++++----
 hw/ppc/e500.c                   | 28 +++++++++----------
 hw/ppc/mac_newworld.c           |  2 +-
 hw/ppc/mac_oldworld.c           |  2 +-
 hw/ppc/mpc8544_guts.c           |  7 +++--
 hw/ppc/ppc405_boards.c          | 15 ++++------
 hw/ppc/ppc440_bamboo.c          | 27 +++++++++---------
 hw/ppc/ppc4xx_pci.c             |  3 +-
 hw/ppc/prep.c                   |  4 +--
 hw/ppc/virtex_ml507.c           |  9 +++---
 hw/s390x/virtio-ccw.c           |  5 ++--
 hw/scsi/lsi53c895a.c            |  4 +--
 hw/scsi/spapr_vscsi.c           | 61 +++++++++++++++++++++--------------------
 hw/scsi/virtio-scsi-dataplane.c |  7 ++---
 hw/sd/pl181.c                   |  5 ++--
 hw/sd/pxa2xx_mmci.c             |  6 ++--
 hw/sd/sd.c                      | 15 +++++-----
 hw/sparc/leon3.c                | 16 +++++------
 hw/sparc/sun4m.c                | 16 +++++------
 hw/sparc64/niagara.c            |  4 +--
 hw/sparc64/sun4u.c              | 10 +++----
 hw/ssi/omap_spi.c               |  9 +++---
 hw/timer/omap_gptimer.c         | 12 ++++----
 hw/timer/twl92230.c             |  7 ++---
 hw/timer/xilinx_timer.c         | 10 +++----
 hw/usb/desc.c                   |  2 +-
 hw/usb/dev-bluetooth.c          |  4 +--
 hw/usb/hcd-musb.c               |  4 +--
 hw/usb/tusb6010.c               | 14 +++++-----
 hw/xen/xen-common.c             | 11 ++++----
 hw/xenpv/xen_domainbuild.c      | 32 ++++++++++-----------
 hw/xenpv/xen_machine_pv.c       |  9 +++---
 include/hw/arm/omap.h           | 10 +++----
 include/hw/arm/sharpsl.h        |  2 +-
 memory_mapping.c                |  2 +-
 migration/block.c               |  4 +--
 target/cris/translate.c         |  2 +-
 target/ppc/translate.c          | 36 +++++++-----------------
 target/sh4/translate.c          |  7 ++---
 target/unicore32/translate.c    |  2 +-
 tcg/optimize.c                  |  8 +++---
 tcg/tcg.c                       |  2 +-
 tcg/tcg.h                       |  3 +-
 tcg/tci.c                       |  3 +-
 tcg/tci/tcg-target.inc.c        |  7 ++---
 ui/cursor.c                     |  6 ++--
 ui/spice-display.c              |  4 +--
 vl.c                            |  2 +-
 152 files changed, 802 insertions(+), 821 deletions(-)

-- 
2.14.1

^ permalink raw reply	[flat|nested] 41+ messages in thread

end of thread, other threads:[~2018-01-09 18:10 UTC | newest]

Thread overview: 41+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-12-20 17:22 [Qemu-devel] [PATCH v6 00/29] Remove some of the fprintf(stderr, "* Alistair Francis
2017-12-20 17:22 ` [Qemu-devel] [PATCH v6 01/29] audio: Replace AUDIO_FUNC with __func__ Alistair Francis
2017-12-20 17:22 ` [Qemu-devel] [PATCH v6 02/29] Replace all occurances of __FUNCTION__ " Alistair Francis
2017-12-20 17:22 ` [Qemu-devel] [PATCH v6 03/29] Fixes after renaming __FUNCTION__ to __func__ Alistair Francis
2017-12-20 17:23 ` [Qemu-devel] [PATCH v6 04/29] hw/arm: Replace fprintf(stderr, "*\n" with error_report() Alistair Francis
2017-12-22 15:21   ` Markus Armbruster
2017-12-22 15:37   ` Markus Armbruster
2017-12-22 17:17     ` Thomas Huth
2017-12-22 19:57       ` Alistair Francis
2017-12-22 20:30         ` Markus Armbruster
2017-12-22 20:58           ` Alistair Francis
2018-01-02 12:59             ` Markus Armbruster
2018-01-09  0:32               ` Alistair Francis
2018-01-09 15:31                 ` Thomas Huth
2017-12-20 17:23 ` [Qemu-devel] [PATCH v6 05/29] hw/dma: " Alistair Francis
2017-12-22 15:21   ` Markus Armbruster
2017-12-20 17:23 ` [Qemu-devel] [PATCH v6 06/29] hw/gpio: " Alistair Francis
2017-12-20 17:23 ` [Qemu-devel] [PATCH v6 07/29] hw/i2c: " Alistair Francis
2017-12-20 17:23 ` [Qemu-devel] [PATCH v6 09/29] hw/ide: " Alistair Francis
2017-12-20 17:23 ` [Qemu-devel] [PATCH v6 10/29] hw/intc: " Alistair Francis
2017-12-20 17:23 ` [Qemu-devel] [PATCH v6 11/29] hw/ipmi: " Alistair Francis
2017-12-20 17:23 ` [Qemu-devel] [PATCH v6 12/29] hw/isa: " Alistair Francis
2017-12-20 17:24 ` [Qemu-devel] [PATCH v6 13/29] hw/lm32: " Alistair Francis
2017-12-20 17:24 ` [Qemu-devel] [PATCH v6 14/29] hw/mips: " Alistair Francis
2017-12-20 17:24 ` [Qemu-devel] [PATCH v6 15/29] hw/moxie: " Alistair Francis
2017-12-20 17:24 ` [Qemu-devel] [PATCH v6 16/29] hw/nios2: " Alistair Francis
2017-12-20 17:24 ` [Qemu-devel] [PATCH v6 17/29] hw/nvram: " Alistair Francis
2017-12-20 17:24 ` [Qemu-devel] [PATCH v6 18/29] hw/openrisc: " Alistair Francis
2017-12-20 17:24 ` [Qemu-devel] [PATCH v6 19/29] hw/pci*: " Alistair Francis
2017-12-20 17:24 ` [Qemu-devel] [PATCH v6 20/29] hw/ppc: " Alistair Francis
2017-12-20 17:24 ` [Qemu-devel] [PATCH v6 21/29] hw/s390x: " Alistair Francis
2018-01-09 17:45   ` Cornelia Huck
2018-01-09 18:09     ` Alistair Francis
2017-12-20 17:24 ` [Qemu-devel] [PATCH v6 23/29] hw/sd: " Alistair Francis
2017-12-20 17:24 ` [Qemu-devel] [PATCH v6 24/29] hw/sparc*: " Alistair Francis
2017-12-20 17:24 ` [Qemu-devel] [PATCH v6 25/29] hw/ssi: " Alistair Francis
2017-12-20 17:24 ` [Qemu-devel] [PATCH v6 26/29] hw/timer: " Alistair Francis
2017-12-20 17:25 ` [Qemu-devel] [PATCH v6 27/29] hw/xen*: " Alistair Francis
2017-12-20 17:25 ` [Qemu-devel] [PATCH v6 28/29] tcg: " Alistair Francis
2017-12-20 17:25 ` [Qemu-devel] [PATCH v6 29/29] target: Use qemu_log() instead of fprintf(stderr, ...) Alistair Francis
     [not found] ` <97bff6488fbc4c6b2c10879798aa149a43426fd9.1513790495.git.alistair.francis@xilinx.com>
2017-12-20 22:10   ` [Qemu-devel] [PATCH v6 08/29] hw/i386: Replace fprintf(stderr, "*\n" with error_report() Eduardo Habkost

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).