Re: [Qemu-devel] [PATCH 4 of 5] [UPDATE] DisplayState interface change

[Stefano Stabellini <stefano.stabellini@eu.citrix.com>]

Anthony Liguori wrote:

> I think we need another intermediate step here.
> 
> Right now, we have a pretty poor abstraction wrt QEMUConsole and
> DisplayState.  QEMUConsole is known to much of the hardware emulation
> but DisplayState is too.

>

> I think we need a step that replaces all notions of QEMUConsole in
> hw/*.c with DisplayState including an appropriate creation functions. 
> There's really no point in passing a DisplayState to the hardware
> devices, instead they should be allocating it and returning it to be
> passed to the SDL/VNC front-ends.
> 


If am not mistaken this is exactly what Paul proposed.
As I said before, this model is certainly more elegant, but presents
some hidden issues:

what do we do with the text consoles?
Right now a single DisplayState is shared between a graphic console and
multiple text console.
Do we really want to allocate a buffer each?
Who is going to do the console multiplexing?

Even though the QEMUConsole abstraction is not very elegant and should
probably be improved I believe it is a different and independent issue
from the one addressed in this series.