qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
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

  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).