Re: [Qemu-devel] [PATCH 1 of 2] [UPDATE] DisplayState interface change

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

Paul Brook wrote:

> On Thursday 20 November 2008, Stefano Stabellini wrote:
>> Hi all,
>> this patch changes the DisplayState interface adding support for
>> multiple frontends at the same time (sdl and vnc) and implements most
>> of the benefit of the shared_buf patch without the added complexity.
> 
> Unless I'm misunderstanding something, isn't this all going to go horribly 
> wrong when we want to have multiple displays? You shouldn't be using 
> DisplayState at all, you should be attaching these things to QEMUConsole.
> 
> 

That was the main problem I had to solve, but I managed to make it work
correctly: everything is handled transparently by qemu_console_resize
and console_select in console.c.
If the backend is smart enought to share the buffer it does so checking
that it is the current active console first (look at
hw/vga.c:vga_draw_graphic, vga.c is the only "smart" backend at the moment).

You can test it passing "-sdl -vnc :1" as arguments to qemu.