From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:57368) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TdOuf-0005Qx-89 for qemu-devel@nongnu.org; Tue, 27 Nov 2012 12:25:41 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TdOuZ-0004db-7S for qemu-devel@nongnu.org; Tue, 27 Nov 2012 12:25:37 -0500 Received: from e23smtp01.au.ibm.com ([202.81.31.143]:57617) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TdOuY-0004Xe-MO for qemu-devel@nongnu.org; Tue, 27 Nov 2012 12:25:31 -0500 Received: from /spool/local by e23smtp01.au.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Wed, 28 Nov 2012 03:21:40 +1000 Message-ID: <50B4F769.40500@linux.vnet.ibm.com> Date: Tue, 27 Nov 2012 22:54:57 +0530 From: Avik Sil MIME-Version: 1.0 References: <1351175895-27131-1-git-send-email-aviksil@linux.vnet.ibm.com> <1351175895-27131-2-git-send-email-aviksil@linux.vnet.ibm.com> <877gqetiep.fsf@codemonkey.ws> <95C7D95A-D401-4AFE-8DE5-7D5107C3E083@suse.de> <5097A435.7080900@linux.vnet.ibm.com> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [Qemu-ppc] [PATCH v2 1/2] Make default boot order machine specific List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alexander Graf Cc: Peter Maydell , Anthony Liguori , "qemu-ppc@nongnu.org" , "qemu-devel@nongnu.org" , "david@gibson.dropbear.id.au" On 11/23/2012 10:22 PM, Alexander Graf wrote: > > On 05.11.2012, at 12:34, Avik Sil wrote: > >> On 10/26/2012 05:15 PM, Alexander Graf wrote: >>> >>> On 25.10.2012, at 22:57, Alexander Graf wrote: >>> >>>> >>>> On 25.10.2012, at 22:50, Peter Maydell wrote: >>>> >>>>> On 25 October 2012 21:18, Anthony Liguori 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? > > I'm really waiting for Anthony just like you do :). This is nothing I want to make a call on. Anthony, can you please comment on this? Regards, Avik > > > Alex > >