[Libvir] Support graphical console for PV Xen guests

Jeremy Katz katzj at redhat.com
Sat Aug 26 14:27:38 UTC 2006


On Sat, 2006-08-26 at 10:18 -0400, Daniel Veillard wrote:
> On Sat, Aug 26, 2006 at 09:49:05AM -0400, Jeremy Katz wrote:
> > With the paravirt framebuffer patches for Xen, we can support having a
> > graphical console for PV as well as HVM guests.  The attached adds
> > support to libvirt by breaking out the <graphics> parsing into its own
> > function that can be used by both PV and HVM guests.  It also adds
> > support for looking at the appropriate sexpr for the vnc/sdl console for
> > PV guests.
> > 
> > One thing that is potentially not okay is that I've changed sexpr_node
> > to take a format string instead of just a const char *, but since it's
> > not public API and doesn't actually change anything for callers I think
> > it should be fine.  If not, let me know and I can change it.
> 
>   Honnestly I would prefer to add a new sexpr_fmt_node() routine with
> the snprintf call which would then call the old sexpr_node() and replace
> the few places where you need that functionality with call to sexpr_fmt_node.
> I'm just a bit afraid that people may not realize it's expanded that way
> and pass uncheck strings.
> 
>   Otherwise it just look fine, I can do the change or you can do it, either
> ways.

Simple enough, attached

Jeremy
-------------- next part --------------
A non-text attachment was scrubbed...
Name: libvirt-pvfb.patch
Type: text/x-patch
Size: 8180 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20060826/d796a204/attachment-0001.bin>


More information about the libvir-list mailing list