From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1NKTXB-0000bU-DJ for qemu-devel@nongnu.org; Tue, 15 Dec 2009 04:17:33 -0500 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1NKTX7-0000ao-QX for qemu-devel@nongnu.org; Tue, 15 Dec 2009 04:17:33 -0500 Received: from [199.232.76.173] (port=33127 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1NKTX7-0000al-Nq for qemu-devel@nongnu.org; Tue, 15 Dec 2009 04:17:29 -0500 Received: from mx1.redhat.com ([209.132.183.28]:45966) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1NKTX7-0008PF-95 for qemu-devel@nongnu.org; Tue, 15 Dec 2009 04:17:29 -0500 Message-ID: <4B27541F.9020603@redhat.com> Date: Tue, 15 Dec 2009 10:17:19 +0100 From: Gerd Hoffmann MIME-Version: 1.0 Subject: Re: Proper support for PCI-based option rom loading (was Re: [Qemu-devel] Re: qdev property bug?) References: <4B264AF1.6060802@codemonkey.ws> <7FB8DD1225E54176BCAF5523B6AEA89B@FSCPC> <4B26931E.4000101@codemonkey.ws> <20091214194210.GB6150@redhat.com> <4B269933.3010906@codemonkey.ws> <20091214202019.GF6150@redhat.com> <4B26A0DE.5000304@redhat.com> <20091214203428.GI6150@redhat.com> <20091214203603.GJ6150@redhat.com> <4B26A3B2.2030006@codemonkey.ws> <20091214205141.GC6398@redhat.com> <4B26F678.4010603@codemonkey.ws> In-Reply-To: <4B26F678.4010603@codemonkey.ws> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Anthony Liguori Cc: "Michael S. Tsirkin" , glommer@redhat.com, qemu-devel@nongnu.org, Alexander Graf , Kevin O'Connor , Sebastian Herbszt On 12/15/09 03:37, Anthony Liguori wrote: > Okay, I think I've figured out how this is supposed to work. With these > two patches to SeaBIOS and the patch to qemu, I can run: > > qemu -net nic,model=rtl8139 -net nic,model=virtio -net nic,model=e1000 > -boot menu=on > > And all three option roms load. I can also select which NIC I want to > boot from using the F12 menu. This works by not actually loading the > option roms in the 1M space, but instead making them mappable through > the PCI devices. With PMM and DDIM, the result is that we only have to > copy in 2K for each option rom which means we can support up to 48 > unique option roms. That should be plenty for now. > > These patches are very rough but I'll clean them up tomorrow. I'm not > sure the best way to integrate with the rom infrastructure since we no > longer have a physical address to map to. Any suggestions Gerd? Is this needed in the first place? Integration into pci like your patch does looks sensible to me. That wouldn't get us re-loading on reset, but as the guest shouldn't be able to modify the roms I don't think we need this ... We could also add a "romfile" property to the pci bus and do everything (except setting the default filename) in generic pci code. I'm more worried about the CONFIG_OPTIONROMS_DEPLOYED patches for seabios, although I don't fully understand what they are doing. There are non-pci option roms (linuxboot, multiboot, extboot) which qemu will continue to deploy the way roms are deployed today. cheers, Gerd