[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 42/46] util: Replace fprintf(stderr, "*\n" wi
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH v3 42/46] util: Replace fprintf(stderr, "*\n" with error_report() |
Date: |
Fri, 20 Oct 2017 12:47:39 +0200 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
On Fri, Oct 20, 2017 at 08:27:41AM +0200, Thomas Huth wrote:
> On 19.10.2017 21:47, Stefan Weil wrote:
> > Am 19.10.2017 um 19:53 schrieb Thomas Huth:
> >> On 19.10.2017 18:18, Alistair Francis wrote:
> >>> Replace a large number of the fprintf(stderr, "*\n" calls with
> >>> error_report(). The functions were renamed with these commands and then
> >>> compiler issues where manually fixed.
> >> [...]
> >>> diff --git a/util/aio-posix.c b/util/aio-posix.c
> >>> index 5946ac09f0..29fff51fcf 100644
> >>> --- a/util/aio-posix.c
> >>> +++ b/util/aio-posix.c
> >>> @@ -15,6 +15,7 @@
> >>>
> >>> #include "qemu/osdep.h"
> >>> #include "qemu-common.h"
> >>> +#include "qemu/error-report.h"
> >>> #include "block/block.h"
> >>> #include "qemu/rcu_queue.h"
> >>> #include "qemu/sockets.h"
> >>> @@ -703,8 +704,8 @@ void aio_context_setup(AioContext *ctx)
> >>> {
> >>> /* TODO remove this in final patch submission */
> >>> if (getenv("QEMU_AIO_POLL_MAX_NS")) {
> >>> - fprintf(stderr, "The QEMU_AIO_POLL_MAX_NS environment variable
> >>> has "
> >>> - "been replaced with -object iothread,poll-max-ns=NUM\n");
> >>> + error_report("The QEMU_AIO_POLL_MAX_NS environment variable has "
> >>> + "been replaced with -object iothread,poll-max-ns=NUM");
> >>> exit(1);
> >>> }
> >>
> >> The comment in front of this code block indicates that this should
> >> rather be removed completely. Stefan, do you agree?
> >
> > I assume you asked the other Stefan, but I think he'll agree as I do,
>
> Right, the code has been introduced by this commit:
>
> commit 4a1cba3802554a3b077d436002519ff1fb0c18bf
> Author: Stefan Hajnoczi <address@hidden>
> Date: Thu Dec 1 19:26:42 2016 +0000
>
> aio: add polling mode to AioContext
>
> and apparently the environment variable has never been used in the
> committed code base, so removing this code block sounds like the right
> way to go.
Yes, feel free to remove it.
Initially the RFC patches used the QEMU_AIO_POLL_MAX_NS environment
variable. When I wrote a proper command-line interface for this setting
I left the error to help any of the people testing this optimization
migrate their scripts to the new interface.
...Then I forgot to remove it for the final non-RFC patch which got
merged :).
Stefan
- [Qemu-devel] [PATCH v3 36/46] hw/sparc*: Replace fprintf(stderr, "*\n" with error_report(), (continued)
- [Qemu-devel] [PATCH v3 36/46] hw/sparc*: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19
- [Qemu-devel] [PATCH v3 38/46] hw/timer: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19
- [Qemu-devel] [PATCH v3 40/46] hw/watchdog: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19
- [Qemu-devel] [PATCH v3 41/46] hw/xen*: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19
- [Qemu-devel] [PATCH v3 43/46] ui: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19
- [Qemu-devel] [PATCH v3 39/46] hw/usb: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19
- [Qemu-devel] [PATCH v3 42/46] util: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19
- Re: [Qemu-devel] [PATCH v3 42/46] util: Replace fprintf(stderr, "*\n" with error_report(), Thomas Huth, 2017/10/19
- Re: [Qemu-devel] [PATCH v3 42/46] util: Replace fprintf(stderr, "*\n" with error_report(), Stefan Weil, 2017/10/19
- Re: [Qemu-devel] [PATCH v3 42/46] util: Replace fprintf(stderr, "*\n" with error_report(), Thomas Huth, 2017/10/20
- Re: [Qemu-devel] [PATCH v3 42/46] util: Replace fprintf(stderr, "*\n" with error_report(),
Stefan Hajnoczi <=
- Re: [Qemu-devel] [PATCH v3 42/46] util: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/20
[Qemu-devel] [PATCH v3 45/46] target: Use qemu_log() instead of fprintf(stderr, ...), Alistair Francis, 2017/10/19
[Qemu-devel] [PATCH v3 44/46] tcg: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19
[Qemu-devel] [PATCH v3 46/46] target: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19