On Tue, Jul 07, 2009 at 11:12:52PM +0100, Daniel P. Berrange wrote: > > This patch implements the basic domain config support for a <video> > element as per > > http://www.redhat.com/archives/libvir-list/2009-May/msg00424.html > > Since last time this > > - Centralized the logic for default video model & default RAM > - Added a type=vbox for their custom graphics card Patch looks fine to me, but it seems to miss 3dSupport as discussed with Pritesh. But it will be easy to add this, and that will allow Tom Hughes to rebase his patch on top of this ACK, Daniel -- Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ daniel@xxxxxxxxxxxx | Rpmfind RPM search engine http://rpmfind.net/ http://veillard.com/ | virtualization library http://libvirt.org/ -- Libvir-list mailing list Libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list