From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:52071) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VQy2F-0005Zw-IP for qemu-devel@nongnu.org; Tue, 01 Oct 2013 07:22:41 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VQy29-0004ju-JK for qemu-devel@nongnu.org; Tue, 01 Oct 2013 07:22:35 -0400 Message-ID: <1380626523.5280.27.camel@nilsson.home.kraxel.org> From: Gerd Hoffmann Date: Tue, 01 Oct 2013 13:22:03 +0200 In-Reply-To: References: <1380620399-9907-1-git-send-email-kraxel@redhat.com> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] turn firmware image filename into a machine option List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: Mark Langsdorf , QEMU Developers , Fabien Chouteau , Alexander Graf , Blue Swirl , Michael Walle , =?ISO-8859-1?Q?Herv=E9?= Poussineau , Paul Brook , Anthony Liguori , "open list:New World" , Andreas =?ISO-8859-1?Q?F=E4rber?= , Aurelien Jarno , Richard Henderson On Di, 2013-10-01 at 19:55 +0900, Peter Maydell wrote: > On 1 October 2013 18:39, Gerd Hoffmann wrote: > > Signed-off-by: Gerd Hoffmann > > --- > > hw/alpha/dp264.c | 2 +- > > hw/arm/highbank.c | 8 ++++---- > > hw/i386/pc.c | 6 ++++-- > > hw/i386/pc_piix.c | 3 ++- > > hw/i386/pc_q35.c | 3 ++- > > hw/i386/pc_sysfw.c | 8 +++++--- > > hw/lm32/milkymist.c | 8 ++++---- > > hw/mips/mips_fulong2e.c | 8 ++++---- > > hw/mips/mips_jazz.c | 9 ++++++--- > > hw/mips/mips_malta.c | 8 ++++---- > > hw/mips/mips_mipssim.c | 7 ++++--- > > hw/mips/mips_r4k.c | 9 +++++---- > > hw/ppc/mac_newworld.c | 9 +++++---- > > hw/ppc/mac_oldworld.c | 9 +++++---- > > hw/ppc/ppc405_boards.c | 24 ++++++++++++++---------- > > hw/ppc/prep.c | 11 ++++++----- > > hw/ppc/spapr.c | 6 +++--- > > hw/s390x/ipl.c | 1 + > > hw/sh4/shix.c | 11 ++++++----- > > hw/sparc/leon3.c | 6 +++--- > > hw/sparc/sun4m.c | 2 +- > > hw/sparc64/sun4u.c | 2 +- > > include/hw/boards.h | 1 + > > include/hw/i386/pc.h | 6 ++++-- > > include/sysemu/sysemu.h | 2 -- > > vl.c | 12 +++++++++--- > > No documentation or definition of what the semantics of > specifying a "firmware image" filename are? "-machine firmware=$file" has the same effect as "-bios $file", which is simliar to '-machine kernel=$file' and '-kernel $file'. With this patch the firmware/bios filename goes into QemuOpts and thus can be set via -readconfig $file. It's also possible to set the default firmware image per machine type (via QEMUMachine->default_machine_opts = "firmware=$filename"). [ I'll make the commit message more verbose for v2 ] > Why is this a machine option rather than a property of > the ROM/flash device? Not all machines have a flash device. Also flash drives don't want a simple (readonly) image file but a (writable) blockdev as backing storage, at least the pflash device emulation I've briefly looked at. cheers, Gerd