[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/4] error: Strip trailing '\n' from error strin
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-devel] [PATCH 1/4] error: Strip trailing '\n' from error string arguments (again) |
Date: |
Thu, 10 Dec 2015 17:31:13 +0000 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
* Markus Armbruster (address@hidden) wrote:
> Commit 6daf194d, be62a2eb and 312fd5f got rid of a bunch, but they
> keep coming back. Tracked down with the Coccinelle semantic patch
> from commit 312fd5f.
Reviewed-by: Dr. David Alan Gilbert <address@hidden>
> Cc: Fam Zheng <address@hidden>
> Cc: Peter Crosthwaite <address@hidden>
> Cc: Bharata B Rao <address@hidden>
> Cc: Dominik Dingel <address@hidden>
> Cc: David Hildenbrand <address@hidden>
> Cc: Jason J. Herne <address@hidden>
> Cc: Stefan Berger <address@hidden>
> Cc: Dr. David Alan Gilbert <address@hidden>
> Cc: Changchun Ouyang <address@hidden>
> Cc: zhanghailiang <address@hidden>
> Cc: Pavel Fedin <address@hidden>
> Signed-off-by: Markus Armbruster <address@hidden>
> ---
> block/vmdk.c | 4 ++--
> hw/arm/xlnx-zynqmp.c | 2 +-
> hw/ppc/spapr.c | 3 ++-
> hw/s390x/ipl.c | 8 ++++----
> hw/s390x/s390-skeys-kvm.c | 2 +-
> hw/s390x/s390-skeys.c | 16 ++++++++--------
> hw/tpm/tpm_tis.c | 2 +-
> migration/ram.c | 2 +-
> migration/savevm.c | 4 ++--
> net/vhost-user.c | 6 +++---
> qga/commands-posix.c | 2 +-
> target-arm/cpu.c | 2 +-
> target-arm/machine.c | 4 ++--
> 13 files changed, 29 insertions(+), 28 deletions(-)
>
> diff --git a/block/vmdk.c b/block/vmdk.c
> index 6f819e4..b4a224e 100644
> --- a/block/vmdk.c
> +++ b/block/vmdk.c
> @@ -1494,8 +1494,8 @@ static int vmdk_write(BlockDriverState *bs, int64_t
> sector_num,
>
> if (sector_num > bs->total_sectors) {
> error_report("Wrong offset: sector_num=0x%" PRIx64
> - " total_sectors=0x%" PRIx64 "\n",
> - sector_num, bs->total_sectors);
> + " total_sectors=0x%" PRIx64,
> + sector_num, bs->total_sectors);
> return -EIO;
> }
>
> diff --git a/hw/arm/xlnx-zynqmp.c b/hw/arm/xlnx-zynqmp.c
> index 87553bb..20a3b2b 100644
> --- a/hw/arm/xlnx-zynqmp.c
> +++ b/hw/arm/xlnx-zynqmp.c
> @@ -227,7 +227,7 @@ static void xlnx_zynqmp_realize(DeviceState *dev, Error
> **errp)
> }
>
> if (!s->boot_cpu_ptr) {
> - error_setg(errp, "ZynqMP Boot cpu %s not found\n", boot_cpu);
> + error_setg(errp, "ZynqMP Boot cpu %s not found", boot_cpu);
> return;
> }
>
> diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
> index 6bfb908..6ba68d7 100644
> --- a/hw/ppc/spapr.c
> +++ b/hw/ppc/spapr.c
> @@ -1831,7 +1831,8 @@ static void ppc_spapr_init(MachineState *machine)
> ram_addr_t hotplug_mem_size = machine->maxram_size -
> machine->ram_size;
>
> if (machine->ram_slots > SPAPR_MAX_RAM_SLOTS) {
> - error_report("Specified number of memory slots %"PRIu64" exceeds
> max supported %d\n",
> + error_report("Specified number of memory slots %" PRIu64
> + " exceeds max supported %d",
> machine->ram_slots, SPAPR_MAX_RAM_SLOTS);
> exit(EXIT_FAILURE);
> }
> diff --git a/hw/s390x/ipl.c b/hw/s390x/ipl.c
> index b91fcc6..e100428 100644
> --- a/hw/s390x/ipl.c
> +++ b/hw/s390x/ipl.c
> @@ -94,7 +94,7 @@ static void s390_ipl_realize(DeviceState *dev, Error **errp)
>
> bios_filename = qemu_find_file(QEMU_FILE_TYPE_BIOS, bios_name);
> if (bios_filename == NULL) {
> - error_setg(&l_err, "could not find stage1 bootloader\n");
> + error_setg(&l_err, "could not find stage1 bootloader");
> goto error;
> }
>
> @@ -113,7 +113,7 @@ static void s390_ipl_realize(DeviceState *dev, Error
> **errp)
> g_free(bios_filename);
>
> if (bios_size == -1) {
> - error_setg(&l_err, "could not load bootloader '%s'\n",
> bios_name);
> + error_setg(&l_err, "could not load bootloader '%s'", bios_name);
> goto error;
> }
>
> @@ -128,7 +128,7 @@ static void s390_ipl_realize(DeviceState *dev, Error
> **errp)
> kernel_size = load_image_targphys(ipl->kernel, 0, ram_size);
> }
> if (kernel_size < 0) {
> - error_setg(&l_err, "could not load kernel '%s'\n", ipl->kernel);
> + error_setg(&l_err, "could not load kernel '%s'", ipl->kernel);
> goto error;
> }
> /*
> @@ -156,7 +156,7 @@ static void s390_ipl_realize(DeviceState *dev, Error
> **errp)
> initrd_size = load_image_targphys(ipl->initrd, initrd_offset,
> ram_size - initrd_offset);
> if (initrd_size == -1) {
> - error_setg(&l_err, "could not load initrd '%s'\n",
> ipl->initrd);
> + error_setg(&l_err, "could not load initrd '%s'",
> ipl->initrd);
> goto error;
> }
>
> diff --git a/hw/s390x/s390-skeys-kvm.c b/hw/s390x/s390-skeys-kvm.c
> index 682949a..eaa37ba 100644
> --- a/hw/s390x/s390-skeys-kvm.c
> +++ b/hw/s390x/s390-skeys-kvm.c
> @@ -21,7 +21,7 @@ static int kvm_s390_skeys_enabled(S390SKeysState *ss)
>
> r = skeyclass->get_skeys(ss, 0, 1, &single_key);
> if (r != 0 && r != KVM_S390_GET_SKEYS_NONE) {
> - error_report("S390_GET_KEYS error %d\n", r);
> + error_report("S390_GET_KEYS error %d", r);
> }
> return (r == 0);
> }
> diff --git a/hw/s390x/s390-skeys.c b/hw/s390x/s390-skeys.c
> index 539ef6d..c9acc18 100644
> --- a/hw/s390x/s390-skeys.c
> +++ b/hw/s390x/s390-skeys.c
> @@ -192,8 +192,8 @@ static int qemu_s390_skeys_set(S390SKeysState *ss,
> uint64_t start_gfn,
> /* Check for uint64 overflow and access beyond end of key data */
> if (start_gfn + count > skeydev->key_count || start_gfn + count < count)
> {
> error_report("Error: Setting storage keys for page beyond the end "
> - "of memory: gfn=%" PRIx64 " count=%" PRId64 "\n", start_gfn,
> - count);
> + "of memory: gfn=%" PRIx64 " count=%" PRId64,
> + start_gfn, count);
> return -EINVAL;
> }
>
> @@ -212,8 +212,8 @@ static int qemu_s390_skeys_get(S390SKeysState *ss,
> uint64_t start_gfn,
> /* Check for uint64 overflow and access beyond end of key data */
> if (start_gfn + count > skeydev->key_count || start_gfn + count < count)
> {
> error_report("Error: Getting storage keys for page beyond the end "
> - "of memory: gfn=%" PRIx64 " count=%" PRId64 "\n", start_gfn,
> - count);
> + "of memory: gfn=%" PRIx64 " count=%" PRId64,
> + start_gfn, count);
> return -EINVAL;
> }
>
> @@ -257,7 +257,7 @@ static void s390_storage_keys_save(QEMUFile *f, void
> *opaque)
>
> buf = g_try_malloc(S390_SKEYS_BUFFER_SIZE);
> if (!buf) {
> - error_report("storage key save could not allocate memory\n");
> + error_report("storage key save could not allocate memory");
> goto end_stream;
> }
>
> @@ -277,7 +277,7 @@ static void s390_storage_keys_save(QEMUFile *f, void
> *opaque)
> * use S390_SKEYS_SAVE_FLAG_ERROR to indicate failure to the
> * reading side.
> */
> - error_report("S390_GET_KEYS error %d\n", error);
> + error_report("S390_GET_KEYS error %d", error);
> memset(buf, 0, S390_SKEYS_BUFFER_SIZE);
> eos = S390_SKEYS_SAVE_FLAG_ERROR;
> }
> @@ -315,7 +315,7 @@ static int s390_storage_keys_load(QEMUFile *f, void
> *opaque, int version_id)
> uint8_t *buf = g_try_malloc(S390_SKEYS_BUFFER_SIZE);
>
> if (!buf) {
> - error_report("storage key load could not allocate memory\n");
> + error_report("storage key load could not allocate memory");
> ret = -ENOMEM;
> break;
> }
> @@ -327,7 +327,7 @@ static int s390_storage_keys_load(QEMUFile *f, void
> *opaque, int version_id)
>
> ret = skeyclass->set_skeys(ss, cur_gfn, cur_count, buf);
> if (ret < 0) {
> - error_report("S390_SET_KEYS error %d\n", ret);
> + error_report("S390_SET_KEYS error %d", ret);
> break;
> }
> handled_count += cur_count;
> diff --git a/hw/tpm/tpm_tis.c b/hw/tpm/tpm_tis.c
> index ff073d5..95fc66e 100644
> --- a/hw/tpm/tpm_tis.c
> +++ b/hw/tpm/tpm_tis.c
> @@ -1051,7 +1051,7 @@ static void tpm_tis_realizefn(DeviceState *dev, Error
> **errp)
>
> if (tis->irq_num > 15) {
> error_setg(errp, "tpm_tis: IRQ %d for TPM TIS is outside valid range
> "
> - "of 0 to 15.\n", tis->irq_num);
> + "of 0 to 15", tis->irq_num);
> return;
> }
>
> diff --git a/migration/ram.c b/migration/ram.c
> index 1eb155a..d1611cb 100644
> --- a/migration/ram.c
> +++ b/migration/ram.c
> @@ -2314,7 +2314,7 @@ static int ram_load_postcopy(QEMUFile *f)
> } else {
> /* not the 1st TP within the HP */
> if (host != (last_host + TARGET_PAGE_SIZE)) {
> - error_report("Non-sequential target page %p/%p\n",
> + error_report("Non-sequential target page %p/%p",
> host, last_host);
> ret = -EINVAL;
> break;
> diff --git a/migration/savevm.c b/migration/savevm.c
> index 0ad1b93..f7d12e1 100644
> --- a/migration/savevm.c
> +++ b/migration/savevm.c
> @@ -1563,8 +1563,8 @@ static int
> loadvm_handle_cmd_packaged(MigrationIncomingState *mis)
> ret = qemu_get_buffer(mis->from_src_file, buffer, (int)length);
> if (ret != length) {
> g_free(buffer);
> - error_report("CMD_PACKAGED: Buffer receive fail ret=%d length=%d\n",
> - ret, length);
> + error_report("CMD_PACKAGED: Buffer receive fail ret=%d length=%d",
> + ret, length);
> return (ret < 0) ? ret : -EAGAIN;
> }
> trace_loadvm_handle_cmd_packaged_received(ret);
> diff --git a/net/vhost-user.c b/net/vhost-user.c
> index b368a90..e4dd089 100644
> --- a/net/vhost-user.c
> +++ b/net/vhost-user.c
> @@ -82,15 +82,15 @@ static int vhost_user_start(int queues, NetClientState
> *ncs[])
> options.opaque = s->chr;
> s->vhost_net = vhost_net_init(&options);
> if (!s->vhost_net) {
> - error_report("failed to init vhost_net for queue %d\n", i);
> + error_report("failed to init vhost_net for queue %d", i);
> goto err;
> }
>
> if (i == 0) {
> max_queues = vhost_net_get_max_queues(s->vhost_net);
> if (queues > max_queues) {
> - error_report("you are asking more queues than "
> - "supported: %d\n", max_queues);
> + error_report("you are asking more queues than supported: %d",
> + max_queues);
> goto err;
> }
> }
> diff --git a/qga/commands-posix.c b/qga/commands-posix.c
> index c2ff970..ec0e997 100644
> --- a/qga/commands-posix.c
> +++ b/qga/commands-posix.c
> @@ -2247,7 +2247,7 @@ GuestMemoryBlockList *qmp_guest_get_memory_blocks(Error
> **errp)
> */
> if (errno != ENOENT) {
> error_setg_errno(errp, errno, "Can't open directory"
> - "\"/sys/devices/system/memory/\"\n");
> + "\"/sys/devices/system/memory/\"");
> }
> return NULL;
> }
> diff --git a/target-arm/cpu.c b/target-arm/cpu.c
> index 30739fc..cac27f9 100644
> --- a/target-arm/cpu.c
> +++ b/target-arm/cpu.c
> @@ -649,7 +649,7 @@ static void arm_cpu_realizefn(DeviceState *dev, Error
> **errp)
> uint32_t nr = cpu->pmsav7_dregion;
>
> if (nr > 0xff) {
> - error_setg(errp, "PMSAv7 MPU #regions invalid %" PRIu32 "\n",
> nr);
> + error_setg(errp, "PMSAv7 MPU #regions invalid %" PRIu32, nr);
> return;
> }
>
> diff --git a/target-arm/machine.c b/target-arm/machine.c
> index 36a0d15..b1e1418 100644
> --- a/target-arm/machine.c
> +++ b/target-arm/machine.c
> @@ -337,11 +337,11 @@ const char *gicv3_class_name(void)
> return "kvm-arm-gicv3";
> #else
> error_report("KVM GICv3 acceleration is not supported on this "
> - "platform\n");
> + "platform");
> #endif
> } else {
> /* TODO: Software emulation is not implemented yet */
> - error_report("KVM is currently required for GICv3 emulation\n");
> + error_report("KVM is currently required for GICv3 emulation");
> }
>
> exit(1);
> --
> 2.4.3
>
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK
- [Qemu-devel] [PATCH 0/4] Error reporting cleanups, Markus Armbruster, 2015/12/10
- [Qemu-devel] [PATCH 4/4] hw/s390x: Rename local variables Error *l_err to just err, Markus Armbruster, 2015/12/10
- [Qemu-devel] [PATCH 3/4] error: Clean up errors with embedded newlines (again), part 2, Markus Armbruster, 2015/12/10
- [Qemu-devel] [PATCH 2/4] error: Clean up errors with embedded newlines (again), part 1, Markus Armbruster, 2015/12/10
- [Qemu-devel] [PATCH 1/4] error: Strip trailing '\n' from error string arguments (again), Markus Armbruster, 2015/12/10