[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC v3 3/3] char-socket: Report TCP socket waiting as
From: |
Thomas Huth |
Subject: |
Re: [Qemu-devel] [RFC v3 3/3] char-socket: Report TCP socket waiting as information |
Date: |
Thu, 6 Jul 2017 04:46:03 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.0 |
On 05.07.2017 19:36, Alistair Francis wrote:
> When QEMU is waiting for a TCP socket connection it reports that message as
> an error. This isn't an error it is just information so let's change the
> report to use info_report() instead.
>
> Signed-off-by: Alistair Francis <address@hidden>
> ---
>
> chardev/char-socket.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/chardev/char-socket.c b/chardev/char-socket.c
> index ccc499cfa1..a050a686ea 100644
> --- a/chardev/char-socket.c
> +++ b/chardev/char-socket.c
> @@ -765,8 +765,8 @@ static int tcp_chr_wait_connected(Chardev *chr, Error
> **errp)
> * in TLS and telnet cases, only wait for an accepted socket */
> while (!s->ioc) {
> if (s->is_listen) {
> - error_report("QEMU waiting for connection on: %s",
> - chr->filename);
> + info_report("QEMU waiting for connection on: %s",
> + chr->filename);
> qio_channel_set_blocking(QIO_CHANNEL(s->listen_ioc), true, NULL);
> tcp_chr_accept(QIO_CHANNEL(s->listen_ioc), G_IO_IN, chr);
> qio_channel_set_blocking(QIO_CHANNEL(s->listen_ioc), false,
> NULL);
>
Reviewed-by: Thomas Huth <address@hidden>
And in case you also want to add some warn_reports, I suggest to do a
grep -r "error_report.*[Ww]arning:" *
in the sources - there seem to be quite a lot of error_reports that are
rather a warning instead.
Thomas
- Re: [Qemu-devel] [RFC v3 2/3] qemu-error: Implement a more generic error reporting, (continued)
- Re: [Qemu-devel] [RFC v3 2/3] qemu-error: Implement a more generic error reporting, Thomas Huth, 2017/07/05
- Re: [Qemu-devel] [RFC v3 2/3] qemu-error: Implement a more generic error reporting, Markus Armbruster, 2017/07/06
- Re: [Qemu-devel] [RFC v3 2/3] qemu-error: Implement a more generic error reporting, Daniel P. Berrange, 2017/07/06
- Re: [Qemu-devel] [RFC v3 2/3] qemu-error: Implement a more generic error reporting, Markus Armbruster, 2017/07/06
- Re: [Qemu-devel] [RFC v3 2/3] qemu-error: Implement a more generic error reporting, Daniel P. Berrange, 2017/07/06
- Re: [Qemu-devel] [RFC v3 2/3] qemu-error: Implement a more generic error reporting, Markus Armbruster, 2017/07/06
- Re: [Qemu-devel] [RFC v3 2/3] qemu-error: Implement a more generic error reporting, Daniel P. Berrange, 2017/07/06
- Re: [Qemu-devel] [RFC v3 2/3] qemu-error: Implement a more generic error reporting, Markus Armbruster, 2017/07/06
- Re: [Qemu-devel] [RFC v3 2/3] qemu-error: Implement a more generic error reporting, Alistair Francis, 2017/07/06
[Qemu-devel] [RFC v3 3/3] char-socket: Report TCP socket waiting as information, Alistair Francis, 2017/07/05
- Re: [Qemu-devel] [RFC v3 3/3] char-socket: Report TCP socket waiting as information,
Thomas Huth <=
[Qemu-devel] [RFC v3 1/3] util/qemu-error: Rename error_print_loc() to be more generic, Alistair Francis, 2017/07/05