> On 8. Dec 2017, at 07:50, Kevin O'Connor <[email protected]> wrote: > > On Sun, Nov 26, 2017 at 05:49:53PM +0100, Filippo Sironi wrote: >> Signed-off-by: Filippo Sironi <[email protected]> >> --- >> 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 _______________________________________________ SeaBIOS mailing list [email protected] https://mail.coreboot.org/mailman/listinfo/seabios
