From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:57901) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fuwfM-0005Jg-45 for qemu-devel@nongnu.org; Wed, 29 Aug 2018 05:21:33 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fuwbK-0000aX-RO for qemu-devel@nongnu.org; Wed, 29 Aug 2018 05:17:23 -0400 Received: from mail-it0-x244.google.com ([2607:f8b0:4001:c0b::244]:34860) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1fuwbK-0000a3-MZ for qemu-devel@nongnu.org; Wed, 29 Aug 2018 05:17:22 -0400 Received: by mail-it0-x244.google.com with SMTP id 139-v6so6353193itf.0 for ; Wed, 29 Aug 2018 02:17:21 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: References: <1532496652-26364-1-git-send-email-hongbo.zhang@linaro.org> <1532496652-26364-2-git-send-email-hongbo.zhang@linaro.org> From: Hongbo Zhang Date: Wed, 29 Aug 2018 17:17:20 +0800 Message-ID: Content-Type: text/plain; charset="UTF-8" Subject: Re: [Qemu-devel] [PATCH v2 2/2] hw/arm: Add Arm Enterprise machine type List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: =?UTF-8?B?QWxleCBCZW5uw6ll?= , qemu-arm , QEMU Developers , Radoslaw Biernacki , Ard Biesheuvel , Leif Lindholm On 17 August 2018 at 21:37, Peter Maydell wrote: > On 25 July 2018 at 06:30, 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. >> - No paravirtualized fw_cfg device either. >> >> Arm Trusted Firmware and UEFI porting to this are done accordingly. >> >> Signed-off-by: Hongbo Zhang >> --- >> Changes since v1: >> - rebase on v3.0.0-rc0 >> - introduce another auxillary patch as 1/2, so this is 2/2 >> - rename 'sbsa' to 'enterprise' >> - remove paravirualized fw_cfg >> - set gic_vertion to 3 instead of 2 >> - edit commit message to describe purpose of this platform > > Hi; there's been a lot of discussion on this thread. I'm going > to assume that you don't need any further review commentary for > the moment, and will wait for a v3. If there are any specific > questions you want guidance on to produce that v3, let me know. > Thank you for this summary/remind. Sorry for the late reply due to my vacation last week. Yes, I am working on the v3, with main changes: - machine name "sbsa-ref" (good name?) - a separate file sbsa-ref.c - don't touch the acpi c file, acpi will be supplied by uefi - only necessary dt node, no other peripheral dt nodes Before my sending, I still like comments on the above effort, it is feasible/reasonable or not etc, in case I go wrong too much. I also know that some reviewers may like to give comments when they see the patch codes, so if you don't have comments now, you can do it after seeing the patch. Thanks all. > thanks > -- PMM