Re: [Xen-devel] [PATCH] libxl: fix cirrus vga video memory setting with upstream qemu

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Fri, 2014-05-09 at 10:01 +0200, Fabio Fantoni wrote:
> Il 08/05/2014 17:19, Ian Campbell ha scritto:
> > On Thu, 2014-05-08 at 17:03 +0200, Fabio Fantoni wrote:
> >> Il 08/05/2014 13:33, Ian Campbell ha scritto:
> >>> On Thu, 2014-05-08 at 12:41 +0200, Fabio Fantoni wrote:
> >>>
> >>>> About qemu version when vgamem_mb property is added is qemu 1.3, I
> >>>> already did detailed reply about it in stdvga patch:
> >>>> http://lists.xen.org/archives/html/xen-devel/2014-05/msg00259.html
> >>>> Or you mean something other?
> >>> "in the commit message" is the bit you keep missing.
> >> I not found 1399030886.32736.63.camel@xxxxxxxxxxxxxxxxxxxxxx with google
> >> or thunderbird on mail archive, I not understand what post you refer :(
> > mid.gmane.org is a good way to find these things. e.g.:
> > http://mid.gmane.org/<1399030886.32736.63.camel@xxxxxxxxxxxxxxxxxxxxxx>
> >
> > Note that the <> which I quoted *are* part of the message id.
> >
> > The questions there were:
> >          Is this change correct for all versions of mainline qemu which
> >          people might be using with Xen?
> 
> vgamem_mb property was added in qemu 1.3 (same for stdvga patch),

How was the video RAM size controller in qemu prior to that? Or was it
not possible?

>  xen 
> 4.4 from source use qemu 1.6, and from distributions package is newer 
> all case I saw, then FWIK there should be no problems. Xen 4.4 if I 
> remember good change other important qemu value for hvm domUs that 
> require qemu>=1.6. (1.6.1 because 1.6.0 have critical regression for all 
> hvm domUs)

Stefano/Anthony -- can you confirm that we already depend on qemu >=
1.6.x (or otherwise that it is OK to depend on qemu >= 1.3)?

An ack from you guys on this change would be appreciated.

It's also been proposed that we backport this to stable branches, how
far back would it be safe/acceptable for us to go with that?
[...]

> I'll add more details on commit message of this patch and also the 
> stdvga one.

Thanks.
> 
> >
> >> The only think that I not understand is part of latest Don Slutz reply
> >> even if he did "Reviewed-by" on patch.
> >>
> >> Thanks for any reply and sorry for my bad english.
> >
> 



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