From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1NKcxv-0001qo-EH
	for qemu-devel@nongnu.org; Tue, 15 Dec 2009 14:21:47 -0500
Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43)
	id 1NKcxq-0001gv-Nj
	for qemu-devel@nongnu.org; Tue, 15 Dec 2009 14:21:47 -0500
Received: from [199.232.76.173] (port=39718 helo=monty-python.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1NKcxq-0001gc-JO
	for qemu-devel@nongnu.org; Tue, 15 Dec 2009 14:21:42 -0500
Received: from mail-yw0-f171.google.com ([209.85.211.171]:35714)
	by monty-python.gnu.org with esmtp (Exim 4.60)
	(envelope-from <anthony@codemonkey.ws>) id 1NKcxq-0007Q4-Cg
	for qemu-devel@nongnu.org; Tue, 15 Dec 2009 14:21:42 -0500
Received: by ywh1 with SMTP id 1so204029ywh.18
	for <qemu-devel@nongnu.org>; Tue, 15 Dec 2009 11:21:41 -0800 (PST)
Message-ID: <4B27E1C2.5090506@codemonkey.ws>
Date: Tue, 15 Dec 2009 13:21:38 -0600
From: Anthony Liguori <anthony@codemonkey.ws>
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> <4B27541F.9020603@redhat.com>
In-Reply-To: <4B27541F.9020603@redhat.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/pipermail/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: "Michael S. Tsirkin" <mst@redhat.com>, glommer@redhat.com, qemu-devel@nongnu.org, Alexander Graf <agraf@suse.de>, Kevin O'Connor <kevin@koconnor.net>, Sebastian Herbszt <herbszt@gmx.de>

Gerd Hoffmann wrote:
> 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?

Only if we care about 'info roms' working.  Problem is roms is very 
centric now to roms at a static guest physical location.  In this case, 
the rom lives in hardware and is mapped into physical memory based on 
what the guest does.

Regards,

Anthony Liguori