WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

RE: [Xen-devel] [Fwd: Installing from distribution CDs]

To: "Anthony Liguori" <anthony@xxxxxxxxxxxxx>
Subject: RE: [Xen-devel] [Fwd: Installing from distribution CDs]
From: "Ian Pratt" <m+Ian.Pratt@xxxxxxxxxxxx>
Date: Wed, 9 Feb 2005 00:11:02 -0000
Cc: "Anthony Liguori" <aliguori@xxxxxxxxxx>, "Jared Rhine" <jared@xxxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxxxx>, <ian.pratt@xxxxxxxxxxxx>
Delivery-date: Wed, 09 Feb 2005 00:12:44 +0000
Envelope-to: xen+James.Bulpin@xxxxxxxxxxxx
List-archive: <http://sourceforge.net/mailarchive/forum.php?forum=xen-devel>
List-help: <mailto:xen-devel-request@lists.sourceforge.net?subject=help>
List-id: List for Xen developers <xen-devel.lists.sourceforge.net>
List-post: <mailto:xen-devel@lists.sourceforge.net>
List-subscribe: <https://lists.sourceforge.net/lists/listinfo/xen-devel>, <mailto:xen-devel-request@lists.sourceforge.net?subject=subscribe>
List-unsubscribe: <https://lists.sourceforge.net/lists/listinfo/xen-devel>, <mailto:xen-devel-request@lists.sourceforge.net?subject=unsubscribe>
Sender: xen-devel-admin@xxxxxxxxxxxxxxxxxxxxx
Thread-index: AcUJlm3DlMzYHh6rQ/aCfxkBOYzvnAEpPaEQ
Thread-topic: [Xen-devel] [Fwd: Installing from distribution CDs]
Have we got concensus about how to handle this? (and hence a definitive
patch).

Requiring people to change there config command lines is probably OK
provided that we're making it closer to standard Linux behaviour.

Ian

> -----Original Message-----
> From: Anthony Liguori [mailto:anthony@xxxxxxxxxxxxx] 
> Sent: 03 February 2005 02:17
> To: Ian Pratt
> Cc: Anthony Liguori; Jared Rhine; xen-devel@xxxxxxxxxxxxxxxxxxxxx
> Subject: Re: [Xen-devel] [Fwd: Installing from distribution CDs]
> 
> Ian Pratt wrote:
> 
> >Thanks for looking into this. I wander if it's something to 
> do with the
> >way xen packages up the module as an initrd for dom0? Maybe 
> there's some
> >difference between an initrd and a ramdisk?
> >  
> >
> Didn't have time this afternoon but I was able to look into 
> it more this 
> evening and I found the culprit.  In arch/i386/kernel/setup.c 
> there was 
> the following line around L1363:
> 
>         ROOT_DEV = MKDEV(RAMDISK_MAJOR,0); 
> /*old_decode_dev(ORIG_ROOT_DEV);*/
> 
> This defaults the root device to /dev/ram0 instead of trying 
> to get it 
> from the boot loader.  I'm not sure why this there (perhaps a part of 
> early development?).  I've attached a patch that puts back the 
> old_decode_dev call and the behavior becomes exactly what 
> you'd expect: 
> if no root= is specified, initrd still works but if /linuxrc 
> exits you 
> get a VFS error because no root= is specified.
> 
> This is what Linux would normally do.
> 
> It's very important to note though that applying this patch 
> means that 
> if people had ramdisk=... lines in their configs and didn't have 
> root=/dev/ram0, their machines won't boot anymore.
> 
> A solution would be to add an initrd option to the configuration file 
> and have the ramdisk= option default the root device to /dev/ram0.
> 
> I've tested this patch on a couple day old copy of xen-unstable.  I'm 
> curious to know what the source of this was though because I 
> don't feel 
> very comfortable with just restoring something that was 
> obviously taken 
> out for a reason..
> 
> Regards,
> Anthony Liguori
> 
> Signed-off-by: Anthony Liguori
> 


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_ide95&alloc_id396&op=click
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/xen-devel