[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v1 1/1] char-socket: Don't report TCP socket wai
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v1 1/1] char-socket: Don't report TCP socket waiting as an error |
Date: |
Wed, 07 Jun 2017 09:19:16 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) |
Paolo Bonzini <address@hidden> writes:
> On 06/06/2017 18:30, Alistair Francis wrote:
>>>
>>> This is somehow confusing. I don't think it is worth having another
>>> qemu_log_stderr() function rather than using error_report() but this very
>>> call might deserve a comment explaining this unusual use. What do you think?
>>
>> The problem with stderr is that this isn't an error. Some uses of QEMU
>> (inside Eclipse for example) flag everything printed on stderr as red
>> which confuses users that they are seeing an error when they really
>> aren't.
>
> But they are wrong.
Concur. We also print warnings and informational messages to stderr.
We should make errors easy to recognize. Fortunately, error_report()
prints errors to stderr in a rigid format. Unfortunately, error
messages bypassing error_report() still exist in places. We suck.
The format is
timestamp-if-enabled progname ':' location message
timestamp-if-enabled is normally empty. With -msg timestamp=on, it's
the current time in ISO 8601 format, followed by a space.
progname is the program name (main()'s argv[0]).
location is either empty, or a reference to the command line or a
configuration file.
See error_vreport() for details.
[...]
- [Qemu-devel] [PATCH v1 1/1] char-socket: Don't report TCP socket waiting as an error, Alistair Francis, 2017/06/05
- Re: [Qemu-devel] [PATCH v1 1/1] char-socket: Don't report TCP socket waiting as an error, Philippe Mathieu-Daudé, 2017/06/05
- Re: [Qemu-devel] [PATCH v1 1/1] char-socket: Don't report TCP socket waiting as an error, Marc-André Lureau, 2017/06/06
- Re: [Qemu-devel] [PATCH v1 1/1] char-socket: Don't report TCP socket waiting as an error, Philippe Mathieu-Daudé, 2017/06/06
- Re: [Qemu-devel] [PATCH v1 1/1] char-socket: Don't report TCP socket waiting as an error, Alistair Francis, 2017/06/06
- Re: [Qemu-devel] [PATCH v1 1/1] char-socket: Don't report TCP socket waiting as an error, Paolo Bonzini, 2017/06/06
- Re: [Qemu-devel] [PATCH v1 1/1] char-socket: Don't report TCP socket waiting as an error, Alistair Francis, 2017/06/06
- Re: [Qemu-devel] [PATCH v1 1/1] char-socket: Don't report TCP socket waiting as an error,
Markus Armbruster <=
- Re: [Qemu-devel] [PATCH v1 1/1] char-socket: Don't report TCP socket waiting as an error, Alistair Francis, 2017/06/07
- Re: [Qemu-devel] [PATCH v1 1/1] char-socket: Don't report TCP socket waiting as an error, Markus Armbruster, 2017/06/08
- Re: [Qemu-devel] [PATCH v1 1/1] char-socket: Don't report TCP socket waiting as an error, Alistair Francis, 2017/06/08
- Re: [Qemu-devel] [PATCH v1 1/1] char-socket: Don't report TCP socket waiting as an error, Markus Armbruster, 2017/06/08
- Re: [Qemu-devel] [PATCH v1 1/1] char-socket: Don't report TCP socket waiting as an error, Alistair Francis, 2017/06/08
- Re: [Qemu-devel] [PATCH v1 1/1] char-socket: Don't report TCP socket waiting as an error, Markus Armbruster, 2017/06/09
- Re: [Qemu-devel] [PATCH v1 1/1] char-socket: Don't report TCP socket waiting as an error, Alistair Francis, 2017/06/27