[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/6] slirp: Remove default_mon usage
From: |
Cole Robinson |
Subject: |
Re: [Qemu-devel] [PATCH 1/6] slirp: Remove default_mon usage |
Date: |
Wed, 12 Mar 2014 09:22:38 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.3.0 |
On 03/12/2014 04:13 AM, Markus Armbruster wrote:
> Cole Robinson <address@hidden> writes:
>
>> These errors don't seem user initiated, so forcibly printing to the
>> monitor doesn't seem right. Just print to stderr.
>>
>> Drop lprint since it's now unused.
>>
>> Cc: Jan Kiszka <address@hidden>
>> Signed-off-by: Cole Robinson <address@hidden>
>> ---
>> checkpatch flags some pre-existing tab issues, but I didn't retab. Should I?
>>
>> slirp/misc.c | 13 ++-----------
>> slirp/slirp.c | 8 ++++----
>> slirp/slirp.h | 2 --
>> 3 files changed, 6 insertions(+), 17 deletions(-)
>>
>> diff --git a/slirp/misc.c b/slirp/misc.c
>> index 6c1636f..662fb1d 100644
>> --- a/slirp/misc.c
>> +++ b/slirp/misc.c
>> @@ -136,7 +136,7 @@ fork_exec(struct socket *so, const char *ex, int do_pty)
>> if ((s = qemu_socket(AF_INET, SOCK_STREAM, 0)) < 0 ||
>> bind(s, (struct sockaddr *)&addr, addrlen) < 0 ||
>> listen(s, 1) < 0) {
>> - lprint("Error: inet socket: %s\n", strerror(errno));
>> + fprintf(stderr, "Error: inet socket: %s\n",
>> strerror(errno));
>> closesocket(s);
>>
>> return 0;
>
> Why not error_report()?
>
> [...]
>
AFAICT this is only guest initiated, never from the monitor. error_report
sends output to cur_mon if a monitor command is being processed. If this
message triggers while a different monitor command is being processed, do we
really want this unrelated output to go to the monitor?
(Maybe that can't happen in practice, I don't know enough about qemu's
threading model, but I was being cautious)
- Cole
[Qemu-devel] [PATCH 3/6] error: Privatize error_print_loc, Cole Robinson, 2014/03/11
[Qemu-devel] [PATCH 4/6] monitor: Remove unused monitor_print_filename, Cole Robinson, 2014/03/11
[Qemu-devel] [PATCH 2/6] vnc: Remove default_mon usage, Cole Robinson, 2014/03/11