qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH] display: stop using DT_NOGRAPHIC, use DT_NONE


From: Andreas Färber
Subject: Re: [Qemu-devel] [PATCH] display: stop using DT_NOGRAPHIC, use DT_NONE
Date: Fri, 28 Jun 2013 13:56:37 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130510 Thunderbird/17.0.6

Am 28.06.2013 13:50, schrieb Michael Tokarev:
> 28.06.2013 15:45, Andreas Färber пишет:
>> Am 28.06.2013 13:29, schrieb Michael Tokarev:
>>> 28.06.2013 15:24, Peter Maydell wrote:
> []
>>>> ...in particular I don't think "-display none" should
>>>> mean "don't allow ctrl-c" (though -nographic should
>>>> continue to have that effect), and this patch currently
>>>> introduces that behaviour change.
>>>
>>> As Anthony said before, -nographic is legacy.  So there should
>>> be some more modern way to control this.  That's exactly the
>>> change which I don't like myself.  
>>
>>> But "don't allow ctrl-c"
>>> which is currently bound to -nographic is equally wrong.
>>
>> What's wrong about that? Isn't Ctrl+C passed through to the guest in
>> -nographic mode? I don't see how any other mode inclusing daemonize
>> would need that.
> 
> Um. With either -nographic or -display none, there's no "display"
> per se, and it is the "display" who relays keypresses and such into
> guest.  Without display, the guest becomes headless completely, not
> only it does not have a monitor of a video card, but also does not
> have keyboard or mouse or other similar stuff.  Or it may have these,
> but you can't "touch" neither keyboard nor mouse because you don't
> "see" them without a display.  Fun thing but here we go.
> 
> This ctrl+c handling is only about when you explicitly redirected
> some other char device to guest, such as serial port.  Which don't
> have much to do with display I think, hence I don't understand the
> logic here.

The logic is that a) -nographic does have serial I/O on stdio and b)
emulated serial ports may get repurposed (change order/usage) for this
to work. CC'ing Alex and Blue who might remember more details.

So in the end it boils down to whether we consider stdio a "display" and
whether we may want to split this out into its own global variable to
cleanly separate it from graphical DT_* options.

Andreas

> 
> Thanks,
> 
> /mjt
> 


-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg



reply via email to

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