[SeaBIOS] [PATCH] paravirt: Allow both Red Hat and Amazon PCI HB subsystem vendor and device ids

[Sironi, Filippo]

> On 8. Dec 2017, at 07:50, Kevin O'Connor <kevin at koconnor.net> wrote:
> 
> On Sun, Nov 26, 2017 at 05:49:53PM +0100, Filippo Sironi wrote:
>> Signed-off-by: Filippo Sironi <sironi at amazon.de>
>> ---
>> src/fw/paravirt.c | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>> 
>> diff --git a/src/fw/paravirt.c b/src/fw/paravirt.c
>> index 0770c47b12e7..163b70c6473f 100644
>> --- a/src/fw/paravirt.c
>> +++ b/src/fw/paravirt.c
>> @@ -83,8 +83,8 @@ static void qemu_detect(void)
>>     u16 sv = pci_config_readw(0, PCI_SUBSYSTEM_VENDOR_ID);
>>     u16 sd = pci_config_readw(0, PCI_SUBSYSTEM_ID);
>> 
>> -    if (sv != 0x1af4 || /* Red Hat, Inc */
>> -        sd != 0x1100)   /* Qemu virtual machine */
>> +    if (!(sv == 0x1af4 && sd == 0x1100) && /* Red Hat, Inc. QEMU VM */
>> +        !(sv == 0x1d0f && sd == 0x1237))   /* Amazon.com, Inc. i440fx VM */
>>         return;
> 
> Is the above needed for a generally available VM software or product?
> Or is this internal to Amazon?
> 
> -Kevin

1d0f:1237 are the subsystem vendor and device ids of the i440fx emulation
in the EC2 machine model, which is what we're using for:

* https://aws.amazon.com/ec2/instance-types/c5/
* https://aws.amazon.com/ec2/instance-types/m5/

Today, the EC2 machine model isn't open source. However, we're not excluding
the option of making it generally available in the future.

Filippo


Amazon Development Center Germany GmbH
Berlin - Dresden - Aachen
main office: Krausenstr. 38, 10117 Berlin
Geschaeftsfuehrer: Dr. Ralf Herbrich, Christian Schlaeger
Ust-ID: DE289237879
Eingetragen am Amtsgericht Charlottenburg HRB 149173 B