Re: [Xen-devel] [ANNOUNCE] Xen 3.3.2 and 3.4.1 released

[Pasi Kärkkäinen <pasik@xxxxxx>]

On Tue, Aug 11, 2009 at 11:29:21AM +0300, Pasi Kärkkäinen wrote:
> On Mon, Aug 10, 2009 at 01:46:30PM +0100, Andrew Lyon wrote:
> > On Mon, Aug 10, 2009 at 12:26 PM, Ian Jackson<Ian.Jackson@xxxxxxxxxxxxx> 
> > wrote:
> > > On 10/08/2009 11:11, "Andrew Lyon" <andrew.lyon@xxxxxxxxx> wrote:
> > >> Sorry to report this on the final 3.4.1 but I never had time to test
> > >> any of the rc's, it appears that the fix for using videoram config
> > >> setting was not applied to 3.4.1, when I try to start a domain with
> > >> stdvga=1 and videoram=16 it fails:
> > >
> > > Do you mean this:
> > >
> > >  commit e0bb6b8df60863bca0163a1688baf4854e931e55
> > >  Author: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
> > >  Date:   Mon Jun 8 17:45:42 2009 +0100
> > >
> > >      stdvga + videoram config option
> > >
> > >      Without this patch, guests cannot be created on ia64 because of
> > >      qemu-dm's error.
> > >
> > >      Signed-off-by: KUWAMURA Shin'ya <kuwa@xxxxxxxxxxxxxx>
> > >
> > > That appears to remove this assignment
> > >      ram_size = vga_ram_size;
> > > from an ia64-specific fragment in vl.c which doesn't appear in the 3.4
> > > branch.
> > >
> > > If that's not what you meant, what did you mean ? :-)
> > >
> > > Ian.
> > >
> > 
> > Yes that is the correct pair of patches, I have tested on x86_64 and I
> > am able to create hvm with increased video ram, I do not see much
> > value in applying a patch which does not work and then fixing it if
> > both are not currently in 3.4 so here is a single patch instead,
> > perhaps KUWAMURA Shin'ya can add a signed-off-by to this too.
> > 
> > Signed-off-by: Andrew Lyon <andrew.lyon@xxxxxxxxx>
> > 
> 
> Btw is there some web based method of seeing shortlogs/logs of xen qemu git
> repositories?
> 
> http://xenbits.xensource.com/git-http/qemu-xen-3.4-testing.git/
> 
> I didn't figure out the way to see the logs from the web interface.. 
> was just wondering if this fix was already merged into qemu-xen-3.4-testing.
>

Was this patch merged? 

-- Pasi
 
> 
> > 
> > ---
> > 
> > diff --git tools/ioemu-qemu-xen/vl.c tools/ioemu-qemu-xen/vl.c
> > index b5bb5cb..3506977 100644
> > --- tools/ioemu-qemu-xen/vl.c
> > +++ tools/ioemu-qemu-xen/vl.c
> > @@ -4801,7 +4801,6 @@ int main(int argc, char **argv, char **envp)
> >      machine = first_machine;
> >      cpu_model = NULL;
> >      initrd_filename = NULL;
> > -    ram_size = VGA_RAM_SIZE;
> >  #ifdef CONFIG_GDBSTUB
> >      use_gdbstub = 0;
> >      gdbstub_port = DEFAULT_GDBSTUB_PORT;
> > @@ -5654,13 +5653,6 @@ int main(int argc, char **argv, char **envp)
> >      }
> >  #endif
> > 
> > -#if defined (__ia64__)
> > -    if (ram_size > VGA_IO_START)
> > -        ram_size += VGA_IO_SIZE; /* skip VGA I/O hole */
> > -    if (ram_size > MMIO_START)
> > -        ram_size += 1 * MEM_G; /* skip 3G-4G MMIO, LEGACY_IO_SPACE etc. */
> > -#endif
> > -
> >      /* init the bluetooth world */
> >      for (i = 0; i < nb_bt_opts; i++)
> >          if (bt_parse(bt_opts[i]))
> > @@ -5677,6 +5669,13 @@ int main(int argc, char **argv, char **envp)
> > 
> >      phys_ram_size = (machine->ram_require + vga_ram_size) & ~RAMSIZE_FIXED;
> > 
> > +#if defined (__ia64__)
> > +    if (ram_size > VGA_IO_START)
> > +        ram_size += VGA_IO_SIZE; /* skip VGA I/O hole */
> > +    if (ram_size > MMIO_START)
> > +        ram_size += 1 * MEM_G; /* skip 3G-4G MMIO, LEGACY_IO_SPACE etc. */
> > +#endif
> > +
> >      if (machine->ram_require & RAMSIZE_FIXED) {
> >          if (ram_size > 0) {
> >              if (ram_size < phys_ram_size) {
> > 
> > 
> > Andy
> > 

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel