Re: [Xen-devel] [PATCH] tools/libxl: Improve videoram setting

[Fabio Fantoni <fantonifabio@xxxxxxxxxx>]

Il 10/01/2013 15:40, Stefano Stabellini ha scritto:
> On Thu, 10 Jan 2013, Ian Campbell wrote:
> > On Thu, 2013-01-10 at 14:34 +0000, Stefano Stabellini wrote:
> > > diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c
> > > > index e964bf1..2bfbcf4 100644
> > > > --- a/tools/libxl/xl_cmdimpl.c
> > > > +++ b/tools/libxl/xl_cmdimpl.c
> > > > @@ -832,7 +832,8 @@ static void parse_config_data(const char *config_source,
> > > >       xlu_cfg_get_defbool(config, "localtime", &b_info->localtime, 0);
> > > >
> > > >       if (!xlu_cfg_get_long (config, "videoram", &l, 0))
> > > > -        b_info->video_memkb = l * 1024;
> > > > +        if ( l > 8 )
> > > > +            b_info->video_memkb = l * 1024;
> > > Limiting the minimum amount of videoram could be a good idea, but the
> > > minimum should be 4MB, not 8MB
> > I'm not convinced by silently limiting the memory, this deserve at least
> > a WARNING if not an outright failure.
> >
> > Also, does this sort of limit checking make more sense in libxl?
> Yes, you are right on both comments
>
>
> -----
> Nessun virus nel messaggio.
> Controllato da AVG - www.avg.com
> Versione: 2013.0.2805 / Database dei virus: 2637/6021 -  Data di rilascio: 
> 09/01/2013
About limiting to 8 mb minimum it seems to be necessary on upstream qemu 
otherwise qemu crash on start, besides libxl already has a default of 8 
mb if videoram is not set.
I'll add libxl error if videoram is < 8, can it be good?
About videoram setting, I didn't find the setting for stdvga in the code.
Here for example the commit for cirrus:
http://xenbits.xen.org/gitweb/?p=staging/qemu-upstream-unstable.git;a=commit;h=19403a68fb8eaefb2e1245b6a8384d3a3ffa7ca0

Attachment: smime.p7s
Description: Firma crittografica S/MIME

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel