From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
To: Andrew Jones <drjones@redhat.com>
Cc: Hongbo Zhang <hongbo.zhang@linaro.org>,
peter.maydell@linaro.org, radoslaw.biernacki@linaro.org,
ard.biesheuvel@linaro.org, qemu-devel@nongnu.org,
leif.lindholm@linaro.org, qemu-arm@nongnu.org,
alex.bennee@linaro.org
Subject: Re: [Qemu-devel] [PATCH v2 2/2] hw/arm: Add Arm Enterprise machine type
Date: Wed, 25 Jul 2018 11:50:41 +0100 [thread overview]
Message-ID: <20180725105040.GA2366@work-vm> (raw)
In-Reply-To: <20180725095426.f7kv4ftngeje6pkx@kamzik.brq.redhat.com>
* Andrew Jones (drjones@redhat.com) wrote:
> On Wed, Jul 25, 2018 at 01:30:52PM +0800, Hongbo Zhang wrote:
> > For the Aarch64, there is one machine 'virt', it is primarily meant to
> > run on KVM and execute virtualization workloads, but we need an
> > environment as faithful as possible to physical hardware, for supporting
> > firmware and OS development for pysical Aarch64 machines.
> >
> > This patch introduces new machine type 'Enterprise' with main features:
> > - Based on 'virt' machine type.
> > - Re-designed memory map.
> > - EL2 and EL3 are enabled by default.
> > - GIC version 3 by default.
> > - AHCI controller attached to system bus, and then CDROM and hard disc
> > can be added to it.
> > - EHCI controller attached to system bus, with USB mouse and key board
> > installed by default.
> > - E1000E ethernet card on PCIE bus.
> > - VGA display adaptor on PCIE bus.
> > - Default CPU type cortex-a57, 4 cores, and 1G bytes memory.
> > - No virtio functions enabled, since this is to emulate real hardware.
>
> In the last review it was pointed out that using virtio-pci should still
> be "real" enough, so there's not much reason to avoid it. Well, unless
> there's some concern as to what drivers are available in the firmware and
> guest kernel. But that concern usually only applies to legacy firmwares
> and kernels, and therefore shouldn't apply to AArch64.
I think the difference from last time is Ard's comments earlier in this
thread:
The purpose of the SBSA machine is not to provide a minimal
configuration. It is intended to exercise all the moving parts one
might find in a server firmware/OS stack, including pieces that are
not usually found on x86 machines, such as DRAM starting above 4 GB
and SATA/USB controllers that are not PCIe based.
that suggests that the intent of this board is to provide everything
which a firmware writer might want to test; that's quite different
from forming the basis of a virtualised machine for real use.
Dave
> > - No paravirtualized fw_cfg device either.
> >
> > Arm Trusted Firmware and UEFI porting to this are done accordingly.
> >
>
> How will UEFI get the ACPI tables from QEMU without fw-cfg? I didn't
> see any sort of reserved ROM region in the patch for them.
>
> Thanks,
> drew
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
next prev parent reply other threads:[~2018-07-25 10:50 UTC|newest]
Thread overview: 95+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-07-25 5:30 [Qemu-devel] [PATCH v2 1/2] hw/arm: check fw_cfg return value before using it Hongbo Zhang
2018-07-25 5:30 ` [Qemu-devel] [PATCH v2 2/2] hw/arm: Add Arm Enterprise machine type Hongbo Zhang
2018-07-25 7:20 ` Shannon Zhao
2018-07-25 8:37 ` Hongbo Zhang
2018-07-25 8:48 ` Daniel P. Berrangé
2018-07-25 9:01 ` Ard Biesheuvel
2018-07-25 9:09 ` Hongbo Zhang
2018-07-25 9:13 ` Ard Biesheuvel
2018-07-25 9:17 ` Hongbo Zhang
2018-07-25 9:20 ` Ard Biesheuvel
2018-07-25 9:40 ` Andrew Jones
2018-07-25 9:47 ` Ard Biesheuvel
2018-07-25 10:10 ` Andrew Jones
2018-07-25 10:33 ` Hongbo Zhang
2018-07-25 11:03 ` Andrew Jones
2018-07-25 9:05 ` Hongbo Zhang
2018-07-25 9:18 ` Daniel P. Berrangé
2018-07-25 9:43 ` Hongbo Zhang
2018-07-25 9:54 ` Andrew Jones
2018-07-25 10:22 ` Hongbo Zhang
2018-07-25 10:53 ` Dr. David Alan Gilbert
2018-07-26 7:35 ` Hongbo Zhang
2018-07-26 7:44 ` Ard Biesheuvel
2018-07-26 9:19 ` Peter Maydell
2018-07-25 11:26 ` Andrew Jones
2018-07-26 9:22 ` Hongbo Zhang
2018-07-26 10:28 ` Andrew Jones
2018-07-26 10:35 ` Ard Biesheuvel
2018-07-26 11:11 ` Andrew Jones
2018-07-26 11:15 ` Ard Biesheuvel
2018-07-26 11:41 ` Andrew Jones
2018-07-26 12:10 ` Peter Maydell
2018-07-26 12:35 ` Andrew Jones
2018-07-26 12:43 ` Peter Maydell
2018-07-27 6:31 ` Hongbo Zhang
2018-08-03 9:21 ` Hongbo Zhang
2018-08-03 9:23 ` Peter Maydell
2018-08-03 9:26 ` Ard Biesheuvel
2018-08-03 9:37 ` Andrew Jones
2018-08-03 13:44 ` Laszlo Ersek
2018-08-03 14:39 ` Andrew Jones
2018-08-03 14:50 ` Laszlo Ersek
2018-08-03 9:39 ` Peter Maydell
2018-08-03 9:52 ` Hongbo Zhang
2018-07-25 10:46 ` Hongbo Zhang
2018-07-25 11:44 ` Andrew Jones
2018-07-25 12:19 ` Peter Maydell
2018-07-25 12:29 ` Ard Biesheuvel
2018-07-25 12:36 ` Peter Maydell
2018-07-25 12:57 ` Andrew Jones
2018-07-25 12:47 ` Daniel P. Berrangé
2018-07-25 12:59 ` Andrew Jones
2018-07-25 13:03 ` Ard Biesheuvel
2018-07-25 13:38 ` Andrew Jones
2018-07-25 13:46 ` Ard Biesheuvel
2018-07-25 14:08 ` Andrew Jones
2018-07-26 9:46 ` Hongbo Zhang
2018-07-26 10:33 ` Andrew Jones
2018-07-26 10:27 ` Hongbo Zhang
2018-07-26 10:17 ` Hongbo Zhang
2018-07-26 10:46 ` Andrew Jones
2018-07-26 10:52 ` Peter Maydell
2018-07-26 10:56 ` Ard Biesheuvel
2018-07-26 11:13 ` Andrew Jones
2018-07-26 12:23 ` Laszlo Ersek
2018-07-26 12:49 ` Daniel P. Berrangé
2018-07-27 9:30 ` Hongbo Zhang
2018-07-25 10:50 ` Dr. David Alan Gilbert [this message]
2018-07-25 11:36 ` Andrew Jones
2018-07-25 16:15 ` Igor Mammedov
2018-07-26 9:55 ` Hongbo Zhang
2018-07-26 10:59 ` Andrew Jones
2018-08-17 13:37 ` Peter Maydell
2018-08-29 9:17 ` Hongbo Zhang
2018-08-29 13:42 ` Andrew Jones
2018-08-30 7:07 ` Hongbo Zhang
2018-08-30 8:31 ` Leif Lindholm
2018-08-30 8:39 ` Peter Maydell
2018-08-30 10:02 ` Leif Lindholm
2018-08-30 13:29 ` Ard Biesheuvel
2018-08-30 13:52 ` Leif Lindholm
2018-08-30 16:36 ` Peter Maydell
2018-08-30 16:43 ` Ard Biesheuvel
2018-08-31 7:20 ` Hongbo Zhang
2018-08-31 8:42 ` Andrew Jones
2018-08-31 11:50 ` Hongbo Zhang
2018-09-05 10:08 ` Hongbo Zhang
2018-09-05 12:02 ` Andrew Jones
2018-09-05 14:09 ` Hongbo Zhang
2018-09-05 15:00 ` Andrew Jones
2018-09-09 10:29 ` Hongbo Zhang
2018-07-25 9:00 ` [Qemu-devel] [PATCH v2 1/2] hw/arm: check fw_cfg return value before using it Peter Maydell
2018-07-25 9:22 ` Hongbo Zhang
2018-07-30 18:07 ` Peter Maydell
2018-08-01 9:57 ` Hongbo Zhang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180725105040.GA2366@work-vm \
--to=dgilbert@redhat.com \
--cc=alex.bennee@linaro.org \
--cc=ard.biesheuvel@linaro.org \
--cc=drjones@redhat.com \
--cc=hongbo.zhang@linaro.org \
--cc=leif.lindholm@linaro.org \
--cc=peter.maydell@linaro.org \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=radoslaw.biernacki@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).