RE: [Xen-ia64-devel] [Patch][Update] initrd and cmdline support forDomainU

["Magenheimer, Dan (HP Labs Fort Collins)" <dan.magenheimer@xxxxxx>]

Hi Akio --

I'm sorry, the original patch didn't apply cleanly so I had
to copy/paste parts.  This one doesn't apply cleanly either
but when I apply it manually, it doesn't compile (no
d->arch.initrd_start and d->arch.initrd_len).  Is there
another part of the patch that I missed?

Thanks,
Dan

> -----Original Message-----
> From: Akio Takebe [mailto:takebe_akio@xxxxxxxxxxxxxx] 
> Sent: Thursday, December 22, 2005 1:26 AM
> To: Magenheimer, Dan (HP Labs Fort Collins); 
> xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
> Cc: Akio Takebe
> Subject: RE: [Xen-ia64-devel] [Patch][Update] initrd and 
> cmdline support forDomainU
> 
> Hi, Dan
> 
> My patch is not completely committed.
> I make a small patch.
> This is the part which wasn't committed.
> 
> Best Regards,
> 
> Akio Takebe
> 
> Signed-off-by: Akio Takebe <takebe_akio@xxxxxxxxxxxxxx>
> 
> diff -r 934470721c46 xen/arch/ia64/xen/dom_fw.c
> --- a/xen/arch/ia64/xen/dom_fw.c        Wed Dec 21 19:18:19 2005
> +++ b/xen/arch/ia64/xen/dom_fw.c        Thu Dec 22 17:01:50 2005
> @@ -861,9 +861,14 @@
>         bp->console_info.orig_x = 0;
>         bp->console_info.orig_y = 24;
>         bp->fpswa = 0;
> -        bp->initrd_start = (dom0_start+dom0_size) -
> -                (PAGE_ALIGN(ia64_boot_param->initrd_size) + 4*1024*
> 1024);
> -        bp->initrd_size = ia64_boot_param->initrd_size;
> +        if (d == dom0){
> +                bp->initrd_start = (dom0_start+dom0_size) -
> +                        
> (PAGE_ALIGN(ia64_boot_param->initrd_size) + 4*
> 1024*1024);
> +                bp->initrd_size = ia64_boot_param->initrd_size;
> +        }else{
> +                bp->initrd_start = d->arch.initrd_start;
> +                bp->initrd_size  = d->arch.initrd_len;
> +       }
>                  printf(" initrd start %0xlx", bp->initrd_start);
>                  printf(" initrd size %0xlx", bp->initrd_size);
> 
> 
> 
> >Hi, Dan and all
> >
> >cmdline's sample is the bellow
> >
> >----------------------
> >kernel = "/boot/vmlinux-2.6.12-xenU"
> >ramdisk = "/boot/initrd-2.6.12-xenU.img"
> >memory = 256
> >name = "rhel4-2"
> >disk = [ 'file:/home/takebe/rhel4.img,hda1,w' ]
> >root = "/dev/hda1 ro"
> >extra = "nomca nosmp xencons=tty0 console=tty0 3"
> >-----------------------
> >
> >Today I'm trying to build and boot new changeset. 
> >But I cannot boot DomU.
> >I will update my recipe, please wait.
> >
> >My environment is ;
> >  Machine      : Tiger4
> >  Dom0&DomU OS : RHEL4 U2
> >  changeset    : 8424
> >
> >Best Regards,
> >
> >Akio Takebe
> >
> >>Committed.  I added some code for backwards compatibility
> >>that uses the old cmdline if none is specified.
> >>
> >>Could you post instructions on how to change the cmdline
> >>for domU boots?  Eventually after everyone starts using
> >>this correctly, we should remove the code for the old default. 
> >>
> >>Thanks,
> >>Dan
> >>
> 

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