All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andrew Jones <drjones@redhat.com>
To: Hongbo Zhang <hongbo.zhang@linaro.org>
Cc: peter.maydell@linaro.org, alex.bennee@linaro.org,
	qemu-arm@nongnu.org, qemu-devel@nongnu.org,
	radoslaw.biernacki@linaro.org, leif.lindholm@linaro.org,
	ard.biesheuvel@linaro.org
Subject: Re: [Qemu-devel] [PATCH v2 2/2] hw/arm: Add Arm Enterprise machine type
Date: Wed, 25 Jul 2018 11:54:26 +0200	[thread overview]
Message-ID: <20180725095426.f7kv4ftngeje6pkx@kamzik.brq.redhat.com> (raw)
In-Reply-To: <1532496652-26364-2-git-send-email-hongbo.zhang@linaro.org>

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.

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

  parent reply	other threads:[~2018-07-25  9:54 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 [this message]
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
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=20180725095426.f7kv4ftngeje6pkx@kamzik.brq.redhat.com \
    --to=drjones@redhat.com \
    --cc=alex.bennee@linaro.org \
    --cc=ard.biesheuvel@linaro.org \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.