[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] Don't attach needless options when launch pygrub



Ian Campbell wrote:
> On Wed, 2010-07-14 at 12:01 +0100, Yu Zhiguo wrote:
>> Ian Campbell wrote:
>>> On Wed, 2010-07-14 at 11:07 +0100, Yu Zhiguo wrote:
>>>>  Oh, yes. It seems that path of specified 'kernel' will be checked 
>>>> in configure_image(). We can delete this check if 'bootloader' is
>>>> existent.
>>> That's only for kernel specified on the xm command line, not from the
>>> configuration file, isn't it?
>>>
>>> I just confirmed that for a guest with grub configuration entries for
>>> 2.6.32-5-686-bigmem and 2.6.26-2-686-bigmem (in that order) then
>>>
>>>         bootloader = "/usr/bin/pygrub"
>>>         bootloader_args = "--quiet"
>>>         
>>>         kernel = "/boot/vmlinuz-2.6.26-2-686-bigmem"
>>>         ramdisk = "/boot/initrd.img-2.6.26-2-686-bigmem"
>>>         
>>> will boot 2.6.26-2-686-bigmem whereas without the kernel and ramdisk
>>> lines it will boot 2.6.32-5-686-bigmem.
>>>
>> I guess this DomU's grub.conf must has "kernel 
>> /boot/vmlinuz-2.6.26-2-686-bigmem".
>>
>> In other words, path of specified 'kernel' must be existent and *same* in 
>> both Dom0 and DomU.
>> This is a problem, it is difficult for using.
> 
> No, in my domain 0:
> 
> # ls /boot/vmlinuz-2.6.26-2-686-bigmem
> ls: cannot access /boot/vmlinuz-2.6.26-2-686-bigmem: No such file or directory
> # ls /boot/vmlinuz-2.6.32-5-686-bigmem
> ls: cannot access /boot/vmlinuz-2.6.32-5-686-bigmem: No such file or directory
> 

yes. But what about menu.lst?

for example, my menu.lst has '/vmlinuz-2.6.31.5-127.fc12.i686.PAE' but 
in fact it is /boot/vmlinuz-2.6.31.5-127.fc12.i686.PAE.


> However I've just remembered that I am using xl so maybe the behaviour
> is different. If xend is checking for the kernels existence in dom0 and
> therefore breaking this behaviour then perhaps that is worth changing.
> 
I think so.
xl now cannot use 'bootloader' format (before your patchs),
it just use 'kernel + ramdisk' format, so it should check 'kernel'
is existent in Dom0 or not.

But xm can use 'bootloader' format, in this format, 'kernel' should not be
check in Dom0 because it is path in DomU.


Yu

> Ian.
> 
>> I want to resolve this by using 'kernel' as DomU's kernel path, so it should 
>> not
>> check it is existent or not in Dom0.
> 
>>
>> Yu
>>
>>
>>>>  What's your opinion?
>>> This is long standing behaviour of the toolstack and although it's a
>>> little odd it's not totally useless (it could be very useful if you
>>> botch you grub configuration for example).
>>>
>>> I'll ask again, what do you think is wrong with the current behaviour
>>> and what are the benefits to changing it?
>>>
>>> Ian.
>>>
>>>
>>>
>>>
>>>
>>>
>>>
> 
> 
> 
> 
> 


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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.