qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] -nographic sometimes adds an extra chardev for


From: Michael Tokarev
Subject: Re: [Qemu-devel] [PATCH] -nographic sometimes adds an extra chardev for stdio
Date: Tue, 29 Apr 2014 09:02:35 +0400
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Icedove/24.4.0

address@hidden bounces, removed from Cc]

29.04.2014 00:07, Mike Day wrote:
[]
> Suggestions much appreciated, I think its ugly versus intrusive, but
> I'm not familiar with this part of qemu.
> 
> There is another case with -monitor where you can configure two
> chardevs for stdio and it does the same thing. I left that one alone,
> its more complicated.

Actually it looks like we should protect from fighting for stdio from
various places.  We already had something similar in commit
ab51b1d568e02c80b1abf9016bda3a86dc1db389 -- see bits in qemu-char.c.

I guess we should have some global variable like "stdio_occuped", set
it to 1 when -daemonize is specified, and set and check it each time
we try to use stdio for something.  This way we'll prevent various
parts of qemu from fighting for stdio.

I don't have time right now to do that, maybe later today or this
week.

Thanks!

/mjt



reply via email to

[Prev in Thread] Current Thread [Next in Thread]