mirror of
https://github.com/xemu-project/xemu.git
synced 2024-11-27 13:30:52 +00:00
Convert multi-line fprintf() to warn_report()
Convert all the multi-line uses of fprintf(stderr, "warning:"..."\n"...
to use warn_report() instead. This helps standardise on a single
method of printing warnings to the user.
All of the warnings were changed using these commands:
find ./* -type f -exec sed -i \
'N; {s|fprintf(.*".*warning[,:] \(.*\)\\n"\(.*\));|warn_report("\1"\2);|Ig}' \
{} +
find ./* -type f -exec sed -i \
'N;N; {s|fprintf(.*".*warning[,:] \(.*\)\\n"\(.*\));|warn_report("\1"\2);|Ig}' \
{} +
find ./* -type f -exec sed -i \
'N;N;N; {s|fprintf(.*".*warning[,:] \(.*\)\\n"\(.*\));|warn_report("\1"\2);|Ig}' \
{} +
find ./* -type f -exec sed -i \
'N;N;N;N {s|fprintf(.*".*warning[,:] \(.*\)\\n"\(.*\));|warn_report("\1"\2);|Ig}' \
{} +
find ./* -type f -exec sed -i \
'N;N;N;N;N {s|fprintf(.*".*warning[,:] \(.*\)\\n"\(.*\));|warn_report("\1"\2);|Ig}' \
{} +
find ./* -type f -exec sed -i \
'N;N;N;N;N;N {s|fprintf(.*".*warning[,:] \(.*\)\\n"\(.*\));|warn_report("\1"\2);|Ig}' \
{} +
find ./* -type f -exec sed -i \
'N;N;N;N;N;N;N; {s|fprintf(.*".*warning[,:] \(.*\)\\n"\(.*\));|warn_report("\1"\2);|Ig}' \
{} +
Indentation fixed up manually afterwards.
Some of the lines were manually edited to reduce the line length to below
80 charecters. Some of the lines with newlines in the middle of the
string were also manually edit to avoid checkpatch errrors.
The #include lines were manually updated to allow the code to compile.
Several of the warning messages can be improved after this patch, to
keep this patch mechanical this has been moved into a later patch.
Signed-off-by: Alistair Francis <alistair.francis@xilinx.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>
Cc: Kevin Wolf <kwolf@redhat.com>
Cc: Max Reitz <mreitz@redhat.com>
Cc: "Michael S. Tsirkin" <mst@redhat.com>
Cc: Igor Mammedov <imammedo@redhat.com>
Cc: Peter Maydell <peter.maydell@linaro.org>
Cc: Stefano Stabellini <sstabellini@kernel.org>
Cc: Anthony Perard <anthony.perard@citrix.com>
Cc: Richard Henderson <rth@twiddle.net>
Cc: Eduardo Habkost <ehabkost@redhat.com>
Cc: Aurelien Jarno <aurelien@aurel32.net>
Cc: Yongbok Kim <yongbok.kim@imgtec.com>
Cc: Cornelia Huck <cohuck@redhat.com>
Cc: Christian Borntraeger <borntraeger@de.ibm.com>
Cc: Alexander Graf <agraf@suse.de>
Cc: Jason Wang <jasowang@redhat.com>
Cc: David Gibson <david@gibson.dropbear.id.au>
Cc: Gerd Hoffmann <kraxel@redhat.com>
Acked-by: Cornelia Huck <cohuck@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <5def63849ca8f551630c6f2b45bcb1c482f765a6.1505158760.git.alistair.francis@xilinx.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
This commit is contained in:
Alistair Francis
Paolo Bonzini
parent
2ab4b13563
commit
8297be80f7
@ -1536,10 +1536,9 @@ static int kvm_init(MachineState *ms)
|
||||
|
||||
while (nc->name) {
|
||||
if (nc->num > soft_vcpus_limit) {
|
||||
fprintf(stderr,
|
||||
"Warning: Number of %s cpus requested (%d) exceeds "
|
||||
"the recommended cpus supported by KVM (%d)\n",
|
||||
nc->name, nc->num, soft_vcpus_limit);
|
||||
warn_report("Number of %s cpus requested (%d) exceeds "
|
||||
"the recommended cpus supported by KVM (%d)",
|
||||
nc->name, nc->num, soft_vcpus_limit);
|
||||
|
||||
if (nc->num > hard_vcpus_limit) {
|
||||
fprintf(stderr, "Number of %s cpus requested (%d) exceeds "
|
||||
|
||||
@ -1227,8 +1227,8 @@ static int vvfat_open(BlockDriverState *bs, QDict *options, int flags,
|
||||
|
||||
switch (s->fat_type) {
|
||||
case 32:
|
||||
fprintf(stderr, "Big fat greek warning: FAT32 has not been tested. "
|
||||
"You are welcome to do so!\n");
|
||||
warn_report("FAT32 has not been tested. "
|
||||
"You are welcome to do so!");
|
||||
break;
|
||||
case 16:
|
||||
case 12:
|
||||
|
||||
@ -184,10 +184,9 @@ static void acpi_table_install(const char unsigned *blob, size_t bloblen,
|
||||
}
|
||||
|
||||
if (has_header && le32_to_cpu(ext_hdr->length) != acpi_payload_size) {
|
||||
fprintf(stderr,
|
||||
"warning: ACPI table has wrong length, header says "
|
||||
"%" PRIu32 ", actual size %zu bytes\n",
|
||||
le32_to_cpu(ext_hdr->length), acpi_payload_size);
|
||||
warn_report("ACPI table has wrong length, header says "
|
||||
"%" PRIu32 ", actual size %zu bytes",
|
||||
le32_to_cpu(ext_hdr->length), acpi_payload_size);
|
||||
}
|
||||
ext_hdr->length = cpu_to_le32(acpi_payload_size);
|
||||
|
||||
|
||||
@ -493,8 +493,8 @@ static void vexpress_modify_dtb(const struct arm_boot_info *info, void *fdt)
|
||||
/* Not fatal, we just won't provide virtio. This will
|
||||
* happen with older device tree blobs.
|
||||
*/
|
||||
fprintf(stderr, "QEMU: warning: couldn't find interrupt controller in "
|
||||
"dtb; will not include virtio-mmio devices in the dtb.\n");
|
||||
warn_report("couldn't find interrupt controller in "
|
||||
"dtb; will not include virtio-mmio devices in the dtb.");
|
||||
} else {
|
||||
int i;
|
||||
const hwaddr *map = daughterboard->motherboard_map;
|
||||
|
||||
@ -9,6 +9,7 @@
|
||||
*/
|
||||
|
||||
#include "qemu/osdep.h"
|
||||
#include "qemu/error-report.h"
|
||||
|
||||
#include <sys/resource.h>
|
||||
|
||||
@ -125,8 +126,8 @@ void xen_map_cache_init(phys_offset_to_gaddr_t f, void *opaque)
|
||||
rlimit_as.rlim_cur = rlimit_as.rlim_max;
|
||||
|
||||
if (rlimit_as.rlim_max != RLIM_INFINITY) {
|
||||
fprintf(stderr, "Warning: QEMU's maximum size of virtual"
|
||||
" memory is not infinity.\n");
|
||||
warn_report("QEMU's maximum size of virtual"
|
||||
" memory is not infinity.");
|
||||
}
|
||||
if (rlimit_as.rlim_max < MCACHE_MAX_SIZE + NON_MCACHE_MEMORY_SIZE) {
|
||||
mapcache->max_mcache_size = rlimit_as.rlim_max -
|
||||
|
||||
@ -216,8 +216,8 @@ static void generate_eeprom_spd(uint8_t *eeprom, ram_addr_t ram_size)
|
||||
}
|
||||
|
||||
if (ram_size) {
|
||||
fprintf(stderr, "Warning: SPD cannot represent final %dMB"
|
||||
" of SDRAM\n", (int)ram_size);
|
||||
warn_report("SPD cannot represent final %dMB"
|
||||
" of SDRAM", (int)ram_size);
|
||||
}
|
||||
|
||||
/* fill in SPD memory information */
|
||||
|
||||
@ -253,9 +253,9 @@ void mips_r4k_init(MachineState *machine)
|
||||
fprintf(stderr, "qemu: Error registering flash memory.\n");
|
||||
}
|
||||
} else if (!qtest_enabled()) {
|
||||
/* not fatal */
|
||||
fprintf(stderr, "qemu: Warning, could not load MIPS bios '%s'\n",
|
||||
bios_name);
|
||||
/* not fatal */
|
||||
warn_report("could not load MIPS bios '%s'",
|
||||
bios_name);
|
||||
}
|
||||
g_free(filename);
|
||||
|
||||
|
||||
@ -141,9 +141,9 @@ void gtod_save(QEMUFile *f, void *opaque)
|
||||
|
||||
r = s390_get_clock(&tod_high, &tod_low);
|
||||
if (r) {
|
||||
fprintf(stderr, "WARNING: Unable to get guest clock for migration. "
|
||||
"Error code %d. Guest clock will not be migrated "
|
||||
"which could cause the guest to hang.\n", r);
|
||||
warn_report("Unable to get guest clock for migration. "
|
||||
"Error code %d. Guest clock will not be migrated "
|
||||
"which could cause the guest to hang.", r);
|
||||
qemu_put_byte(f, S390_TOD_CLOCK_VALUE_MISSING);
|
||||
return;
|
||||
}
|
||||
@ -160,8 +160,8 @@ int gtod_load(QEMUFile *f, void *opaque, int version_id)
|
||||
int r;
|
||||
|
||||
if (qemu_get_byte(f) == S390_TOD_CLOCK_VALUE_MISSING) {
|
||||
fprintf(stderr, "WARNING: Guest clock was not migrated. This could "
|
||||
"cause the guest to hang.\n");
|
||||
warn_report("Guest clock was not migrated. This could "
|
||||
"cause the guest to hang.");
|
||||
return 0;
|
||||
}
|
||||
|
||||
@ -170,9 +170,9 @@ int gtod_load(QEMUFile *f, void *opaque, int version_id)
|
||||
|
||||
r = s390_set_clock(&tod_high, &tod_low);
|
||||
if (r) {
|
||||
fprintf(stderr, "WARNING: Unable to set guest clock value. "
|
||||
"s390_get_clock returned error %d. This could cause "
|
||||
"the guest to hang.\n", r);
|
||||
warn_report("Unable to set guest clock value. "
|
||||
"s390_get_clock returned error %d. This could cause "
|
||||
"the guest to hang.", r);
|
||||
}
|
||||
|
||||
return 0;
|
||||
|
||||
@ -310,8 +310,8 @@ void net_hub_check_clients(void)
|
||||
QLIST_FOREACH(port, &hub->ports, next) {
|
||||
peer = port->nc.peer;
|
||||
if (!peer) {
|
||||
fprintf(stderr, "Warning: hub port %s has no peer\n",
|
||||
port->nc.name);
|
||||
warn_report("hub port %s has no peer",
|
||||
port->nc.name);
|
||||
continue;
|
||||
}
|
||||
|
||||
@ -334,9 +334,8 @@ void net_hub_check_clients(void)
|
||||
warn_report("vlan %d with no nics", hub->id);
|
||||
}
|
||||
if (has_nic && !has_host_dev) {
|
||||
fprintf(stderr,
|
||||
"Warning: vlan %d is not connected to host network\n",
|
||||
hub->id);
|
||||
warn_report("vlan %d is not connected to host network",
|
||||
hub->id);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
14
net/net.c
14
net/net.c
@ -1493,9 +1493,9 @@ void net_check_clients(void)
|
||||
|
||||
QTAILQ_FOREACH(nc, &net_clients, next) {
|
||||
if (!nc->peer) {
|
||||
fprintf(stderr, "Warning: %s %s has no peer\n",
|
||||
nc->info->type == NET_CLIENT_DRIVER_NIC ?
|
||||
"nic" : "netdev", nc->name);
|
||||
warn_report("%s %s has no peer",
|
||||
nc->info->type == NET_CLIENT_DRIVER_NIC ?
|
||||
"nic" : "netdev", nc->name);
|
||||
}
|
||||
}
|
||||
|
||||
@ -1506,10 +1506,10 @@ void net_check_clients(void)
|
||||
for (i = 0; i < MAX_NICS; i++) {
|
||||
NICInfo *nd = &nd_table[i];
|
||||
if (nd->used && !nd->instantiated) {
|
||||
fprintf(stderr, "Warning: requested NIC (%s, model %s) "
|
||||
"was not created (not supported by this machine?)\n",
|
||||
nd->name ? nd->name : "anonymous",
|
||||
nd->model ? nd->model : "unspecified");
|
||||
warn_report("requested NIC (%s, model %s) "
|
||||
"was not created (not supported by this machine?)",
|
||||
nd->name ? nd->name : "anonymous",
|
||||
nd->model ? nd->model : "unspecified");
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
@ -1766,12 +1766,12 @@ static void report_unavailable_features(FeatureWord w, uint32_t mask)
|
||||
if ((1UL << i) & mask) {
|
||||
const char *reg = get_register_name_32(f->cpuid_reg);
|
||||
assert(reg);
|
||||
fprintf(stderr, "warning: %s doesn't support requested feature: "
|
||||
"CPUID.%02XH:%s%s%s [bit %d]\n",
|
||||
kvm_enabled() ? "host" : "TCG",
|
||||
f->cpuid_eax, reg,
|
||||
f->feat_names[i] ? "." : "",
|
||||
f->feat_names[i] ? f->feat_names[i] : "", i);
|
||||
warn_report("%s doesn't support requested feature: "
|
||||
"CPUID.%02XH:%s%s%s [bit %d]",
|
||||
kvm_enabled() ? "host" : "TCG",
|
||||
f->cpuid_eax, reg,
|
||||
f->feat_names[i] ? "." : "",
|
||||
f->feat_names[i] ? f->feat_names[i] : "", i);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
@ -178,9 +178,9 @@ static void hax_process_section(MemoryRegionSection *section, uint8_t flags)
|
||||
if (!memory_region_is_ram(mr)) {
|
||||
if (memory_region_is_romd(mr)) {
|
||||
/* HAXM kernel module does not support ROMD yet */
|
||||
fprintf(stderr, "%s: Warning: Ignoring ROMD region 0x%016" PRIx64
|
||||
"->0x%016" PRIx64 "\n", __func__, start_pa,
|
||||
start_pa + size);
|
||||
warn_report("Ignoring ROMD region 0x%016" PRIx64
|
||||
"->0x%016" PRIx64 "", __func__, start_pa,
|
||||
start_pa + size);
|
||||
}
|
||||
return;
|
||||
}
|
||||
|
||||
@ -9316,14 +9316,14 @@ static void init_ppc_proc(PowerPCCPU *cpu)
|
||||
env->tlb_per_way = env->nb_tlb / env->nb_ways;
|
||||
}
|
||||
if (env->irq_inputs == NULL) {
|
||||
fprintf(stderr, "WARNING: no internal IRQ controller registered.\n"
|
||||
" Attempt QEMU to crash very soon !\n");
|
||||
warn_report("no internal IRQ controller registered."
|
||||
" Attempt QEMU to crash very soon !");
|
||||
}
|
||||
#endif
|
||||
if (env->check_pow == NULL) {
|
||||
fprintf(stderr, "WARNING: no power management check handler "
|
||||
"registered.\n"
|
||||
" Attempt QEMU to crash very soon !\n");
|
||||
warn_report("no power management check handler "
|
||||
"registered."
|
||||
" Attempt QEMU to crash very soon !");
|
||||
}
|
||||
}
|
||||
|
||||
@ -9877,10 +9877,10 @@ static int ppc_fixup_cpu(PowerPCCPU *cpu)
|
||||
* tree. */
|
||||
if ((env->insns_flags & ~PPC_TCG_INSNS)
|
||||
|| (env->insns_flags2 & ~PPC_TCG_INSNS2)) {
|
||||
fprintf(stderr, "Warning: Disabling some instructions which are not "
|
||||
"emulated by TCG (0x%" PRIx64 ", 0x%" PRIx64 ")\n",
|
||||
env->insns_flags & ~PPC_TCG_INSNS,
|
||||
env->insns_flags2 & ~PPC_TCG_INSNS2);
|
||||
warn_report("Disabling some instructions which are not "
|
||||
"emulated by TCG (0x%" PRIx64 ", 0x%" PRIx64 ")",
|
||||
env->insns_flags & ~PPC_TCG_INSNS,
|
||||
env->insns_flags2 & ~PPC_TCG_INSNS2);
|
||||
}
|
||||
env->insns_flags &= PPC_TCG_INSNS;
|
||||
env->insns_flags2 &= PPC_TCG_INSNS2;
|
||||
|
||||
@ -26,6 +26,7 @@
|
||||
#include "keymaps.h"
|
||||
#include "sysemu/sysemu.h"
|
||||
#include "trace.h"
|
||||
#include "qemu/error-report.h"
|
||||
|
||||
static int get_keysym(const name2keysym_t *table,
|
||||
const char *name)
|
||||
@ -76,8 +77,8 @@ static void add_keysym(char *line, int keysym, int keycode, kbd_layout_t *k) {
|
||||
k->keysym2keycode[keysym] = keycode;
|
||||
} else {
|
||||
if (k->extra_count >= MAX_EXTRA_COUNT) {
|
||||
fprintf(stderr, "Warning: Could not assign keysym %s (0x%x)"
|
||||
" because of memory constraints.\n", line, keysym);
|
||||
warn_report("Could not assign keysym %s (0x%x)"
|
||||
" because of memory constraints.", line, keysym);
|
||||
} else {
|
||||
trace_keymap_add("extra", keysym, keycode, line);
|
||||
k->keysym2keycode_extra[k->extra_count].
|
||||
@ -197,8 +198,8 @@ int keysym2scancode(void *kbd_layout, int keysym)
|
||||
if (keysym < MAX_NORMAL_KEYCODE) {
|
||||
if (k->keysym2keycode[keysym] == 0) {
|
||||
trace_keymap_unmapped(keysym);
|
||||
fprintf(stderr, "Warning: no scancode found for keysym %d\n",
|
||||
keysym);
|
||||
warn_report("no scancode found for keysym %d",
|
||||
keysym);
|
||||
}
|
||||
return k->keysym2keycode[keysym];
|
||||
} else {
|
||||
|
||||
@ -32,6 +32,7 @@
|
||||
#include "slirp/libslirp.h"
|
||||
#include "qemu/main-loop.h"
|
||||
#include "block/aio.h"
|
||||
#include "qemu/error-report.h"
|
||||
|
||||
#ifndef _WIN32
|
||||
|
||||
@ -236,9 +237,8 @@ static int os_host_main_loop_wait(int64_t timeout)
|
||||
static bool notified;
|
||||
|
||||
if (!notified && !qtest_enabled() && !qtest_driver()) {
|
||||
fprintf(stderr,
|
||||
"main-loop: WARNING: I/O thread spun for %d iterations\n",
|
||||
MAX_MAIN_LOOP_SPIN);
|
||||
warn_report("I/O thread spun for %d iterations",
|
||||
MAX_MAIN_LOOP_SPIN);
|
||||
notified = true;
|
||||
}
|
||||
|
||||
|
||||
Loading…
Reference in New Issue
Block a user