[libvirt] [PATCH] add fullscreen support to qemu sdl xml (via fullscreen='true' attribute for the graphics element)

Itamar Heim iheim at redhat.com
Wed Dec 10 11:06:51 UTC 2008

Sure - probably not a good idea to send patches at such a late hour anyway
Attached diff.

Some more details:
For those using SDL ("for some crazy reason" as Berrange put it in the
original patch), there is a use for the fullscreen option, causing the
qemu window to open in full screen, change resolution, etc.
This patch accepts an additional attribute in the graphics element:
<graphics type='sdl' display=':0.1' xauth='/root/.Xauthority'

If fullscreen is true, "-full-screen" is added to the qemu command line
(unlike the display and auth which are added as environment variables)

> -----Original Message-----
> From: Daniel Veillard [mailto:veillard at redhat.com]
> Sent: Wednesday, December 10, 2008 9:49 AM
> To: Itamar Heim
> Cc: Libvir-list at redhat.com
> Subject: Re: [libvirt] [PATCH] add fullscreen support to qemu sdl xml
> (via fullscreen='true' attribute for the graphics element)
> On Tue, Dec 09, 2008 at 07:12:05PM -0500, Itamar Heim wrote:
> > diff --git a/src/domain_conf.c b/src/domain_conf.c
>   Hum, I don't know how you sent the diff, but it got broken when
> reaching my mail agent. Please add as an attachment.
>   Also in general it's a good idea to put some text describing really
> what the patch does and what is affected, a one line mail subject
> is a bit short. Especially for thing modifying the XML format,
> the construct changes ought to be clearly documented.
>   Adding the regression tests is nice, thanks, but we also need
> to augment the .rng schemas to handle the extension if accepted
> and the HTML documentation also need the update. We can do it but
> providing the complete details helps :-)
>   So could you please repost the patch with detail of the syntax
> change and as an attachment,
>   thanks in advance,
> Daniel
> > +        if (vm->def->graphics->data.sdl.fullscreen &&
> >
> > +            STREQ(vm->def->graphics->data.sdl.fullscreen, "true")) {
> >
> > +           fullscreen=true;
> >
> > +        }
> >
> > +
> --
> Daniel Veillard      | libxml Gnome XML XSLT toolkit
> http://xmlsoft.org/
> daniel at veillard.com  | Rpmfind RPM search engine http://rpmfind.net/
> http://veillard.com/ | virtualization library  http://libvirt.org/
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20081210/1be59d14/attachment-0001.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fullscreen.diff
Type: application/octet-stream
Size: 5022 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20081210/1be59d14/attachment-0001.obj>

More information about the libvir-list mailing list