qemu-ppc
[Top][All Lists]
Advanced

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

Re: [Qemu-ppc] [Qemu-devel] [PATCH v2 1/2] Make default boot order machi


From: Avik Sil
Subject: Re: [Qemu-ppc] [Qemu-devel] [PATCH v2 1/2] Make default boot order machine specific
Date: Mon, 05 Nov 2012 17:04:13 +0530
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:15.0) Gecko/20120911 Thunderbird/15.0.1

On 10/26/2012 05:15 PM, Alexander Graf wrote:
> 
> On 25.10.2012, at 22:57, Alexander Graf <address@hidden> wrote:
> 
>>
>> On 25.10.2012, at 22:50, Peter Maydell wrote:
>>
>>> On 25 October 2012 21:18, Anthony Liguori <address@hidden> wrote:
>>>> The other approach to this would be:
>>>>
>>>> static QEMUMachine pseries_machine = {
>>>>   .no_boot_order = 1,
>>>> };
>>>>
>>>> Which I think is what Peter is suggesting.  I'm not a huge fan of this
>>>> because it's backwards logic but we already do this for a bunch of other
>>>> things so I can't object too strongly to it.
>>>
>>> The other issue is that "cad" is a load of rubbish for half of these
>>> boards, which don't have anything resembling the usual PC boot
>>> devices and probably don't pay attention to -boot anyway. A patch
>>> which only applied a boot order to boards which actually used it
>>> would probably also be rather shorter.
>>
>> That's what v1 did, and it's ok, if it's mentioned explicitly and thought 
>> through properly. Just doing this without proper reasoning is bad, because 
>> you potentially change semantics of -boot if you miss anything. On OpenBIOS 
>> with PPC we actually make use of the -boot arguments for example.
>>
>> Hence having a common set of default options the way Anthony put it seems 
>> the most sane solution forward. It guarantees you don't break anything in 
>> patch 1. Then it goes in and actually changes the semantics for the one 
>> machine you want to change in patch 2.
> 
> Anthony, please comment on which path you'd prefer. I'd rather like to have a 
> decision before creating more work.
> 
> 
Anthony, Alexander, any comments on the above?

Regards,
Avik




reply via email to

[Prev in Thread] Current Thread [Next in Thread]