[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC v1 3/3] char-socket: Report TCP socket waiting as
From: |
Alistair Francis |
Subject: |
Re: [Qemu-devel] [RFC v1 3/3] char-socket: Report TCP socket waiting as a warning |
Date: |
Tue, 27 Jun 2017 17:17:58 -0700 |
On Tue, Jun 27, 2017 at 2:10 PM, Edgar E. Iglesias
<address@hidden> wrote:
> On Tue, Jun 27, 2017 at 01:45:48PM -0700, 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 a warnign so let's change the report to
>> use warning_report() instead.
>
> Hi Alistair,
>
> Isn't this more like an informational message rather than a warning?
>
> A warning would indicate to me that something unexpected and perhaps wrong
> happened.
> In this case, there's nothing wrong going on.
>
> We may need more classes, like 'info:' and/or maybe others...
Hey Edgar,
That is a good point. I can add a info_report() as well then that
copies the warning_report() function but prefixes with 'info: '.
Thanks,
Alistair
>
> Cheers,
> Edgar
>
>
>
>>
>> Signed-off-by: Alistair Francis <address@hidden>
>> ---
>>
>> chardev/char-socket.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/chardev/char-socket.c b/chardev/char-socket.c
>> index ccc499cfa1..5a56628e74 100644
>> --- a/chardev/char-socket.c
>> +++ b/chardev/char-socket.c
>> @@ -765,7 +765,7 @@ 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",
>> + warning_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);
>> --
>> 2.11.0
>>
>>
- Re: [Qemu-devel] [RFC v1 1/3] util/qemu-error: Rename error_print_loc() to be more generic, (continued)
[Qemu-devel] [RFC v1 3/3] char-socket: Report TCP socket waiting as a warning, Alistair Francis, 2017/06/27
Re: [Qemu-devel] [RFC v1 0/3] Implement a warning_report function, Daniel P. Berrange, 2017/06/28