[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [PATCH 1/4] arch_init: replace fprintf(stderr, ...) w
From: |
Le Tan |
Subject: |
Re: [Qemu-trivial] [PATCH 1/4] arch_init: replace fprintf(stderr, ...) with error_report() in arch_init.c |
Date: |
Sat, 24 May 2014 13:41:26 +0800 |
There is a newer version of patch to fix this problem. I forgot to add
the version number when I sent the new patch. Add this comment just to
indicate that this patch should be dropped. :)
2014-05-10 7:55 GMT+08:00 Le Tan <address@hidden>:
> Replace fprintf(stderr,...) with error_report() in the file
> arch_init.c. The trailing "\n"s of the @fmt argument have been removed
> because @fmt of error_report() should not contain newline.
>
> Signed-off-by: Le Tan <address@hidden>
> ---
> arch_init.c | 36 +++++++++++++++++-------------------
> 1 file changed, 17 insertions(+), 19 deletions(-)
>
> diff --git a/arch_init.c b/arch_init.c
> index 60c975d..0b41475 100644
> --- a/arch_init.c
> +++ b/arch_init.c
> @@ -921,12 +921,12 @@ static int load_xbzrle(QEMUFile *f, ram_addr_t addr,
> void *host)
> xh_len = qemu_get_be16(f);
>
> if (xh_flags != ENCODING_FLAG_XBZRLE) {
> - fprintf(stderr, "Failed to load XBZRLE page - wrong compression!\n");
> + error_report("Failed to load XBZRLE page - wrong compression!");
> return -1;
> }
>
> if (xh_len > TARGET_PAGE_SIZE) {
> - fprintf(stderr, "Failed to load XBZRLE page - len overflow!\n");
> + error_report("Failed to load XBZRLE page - len overflow!");
> return -1;
> }
> /* load data and decode */
> @@ -936,11 +936,11 @@ static int load_xbzrle(QEMUFile *f, ram_addr_t addr,
> void *host)
> ret = xbzrle_decode_buffer(xbzrle_decoded_buf, xh_len, host,
> TARGET_PAGE_SIZE);
> if (ret == -1) {
> - fprintf(stderr, "Failed to load XBZRLE page - decode error!\n");
> + error_report("Failed to load XBZRLE page - decode error!");
> rc = -1;
> } else if (ret > TARGET_PAGE_SIZE) {
> - fprintf(stderr, "Failed to load XBZRLE page - size %d exceeds %d!\n",
> - ret, TARGET_PAGE_SIZE);
> + error_report("Failed to load XBZRLE page - size %d exceeds %d!",
> + ret, TARGET_PAGE_SIZE);
> abort();
> }
>
> @@ -957,7 +957,7 @@ static inline void *host_from_stream_offset(QEMUFile *f,
>
> if (flags & RAM_SAVE_FLAG_CONTINUE) {
> if (!block) {
> - fprintf(stderr, "Ack, bad migration stream!\n");
> + error_report("Ack, bad migration stream!");
> return NULL;
> }
>
> @@ -973,7 +973,7 @@ static inline void *host_from_stream_offset(QEMUFile *f,
> return memory_region_get_ram_ptr(block->mr) + offset;
> }
>
> - fprintf(stderr, "Can't find block %s!\n", id);
> + error_report("Can't find block %s!", id);
> return NULL;
> }
>
> @@ -1026,10 +1026,10 @@ static int ram_load(QEMUFile *f, void *opaque, int
> version_id)
> QTAILQ_FOREACH(block, &ram_list.blocks, next) {
> if (!strncmp(id, block->idstr, sizeof(id))) {
> if (block->length != length) {
> - fprintf(stderr,
> - "Length mismatch: %s: " RAM_ADDR_FMT
> - " in != " RAM_ADDR_FMT "\n", id,
> length,
> - block->length);
> + error_report(
> + "Length mismatch: %s: "
> RAM_ADDR_FMT
> + " in != " RAM_ADDR_FMT, id,
> length,
> + block->length);
> ret = -EINVAL;
> goto done;
> }
> @@ -1038,8 +1038,8 @@ static int ram_load(QEMUFile *f, void *opaque, int
> version_id)
> }
>
> if (!block) {
> - fprintf(stderr, "Unknown ramblock \"%s\", cannot "
> - "accept migration\n", id);
> + error_report("Unknown ramblock \"%s\", cannot "
> + "accept migration", id);
> ret = -EINVAL;
> goto done;
> }
> @@ -1186,12 +1186,10 @@ void select_soundhw(const char *optarg)
>
> if (!c->name) {
> if (l > 80) {
> - fprintf(stderr,
> - "Unknown sound card name (too big to show)\n");
> + error_report("Unknown sound card name (too big to
> show)");
> }
> else {
> - fprintf(stderr, "Unknown sound card name `%.*s'\n",
> - (int) l, p);
> + error_report("Unknown sound card name `%.*s'", (int) l,
> p);
> }
> bad_card = 1;
> }
> @@ -1214,13 +1212,13 @@ void audio_init(void)
> if (c->enabled) {
> if (c->isa) {
> if (!isa_bus) {
> - fprintf(stderr, "ISA bus not available for %s\n",
> c->name);
> + error_report("ISA bus not available for %s", c->name);
> exit(1);
> }
> c->init.init_isa(isa_bus);
> } else {
> if (!pci_bus) {
> - fprintf(stderr, "PCI bus not available for %s\n",
> c->name);
> + error_report("PCI bus not available for %s", c->name);
> exit(1);
> }
> c->init.init_pci(pci_bus);
> --
> 1.7.9.5
>
- [Qemu-trivial] [PATCH 0/4] replace some fprintf(stderr, ...) with error_report(), Le Tan, 2014/05/10
- [Qemu-trivial] [PATCH 1/4] arch_init: replace fprintf(stderr, ...) with error_report() in arch_init.c, Le Tan, 2014/05/10
- Re: [Qemu-trivial] [PATCH 1/4] arch_init: replace fprintf(stderr, ...) with error_report() in arch_init.c,
Le Tan <=
- [Qemu-trivial] [PATCH 2/4] audio: replace fprintf(stderr, ...) with error_report() in audio, Le Tan, 2014/05/10
- [Qemu-trivial] [PATCH 3/4] block: replace fprintf(stderr, ...) with error_report() in block/, Le Tan, 2014/05/10
- [Qemu-trivial] [PATCH 4/4] bsd-user: replace fprintf(stderr, ...) with error_report(), Le Tan, 2014/05/10