From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by mx.groups.io with SMTP id smtpd.web10.133.1621360016956575968 for ; Tue, 18 May 2021 10:46:57 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=OJ1fY3HO; spf=pass (domain: redhat.com, ip: 170.10.133.124, mailfrom: lersek@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1621360016; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=jzxgi1nKfBM8L/J0l/VjYxx4kE0bAKuwAvfIsQqXpPs=; b=OJ1fY3HOx5GgEXUSK2UaFDGvmUj/GnI+jcugsQo6a47xJJpcOJZL1krNG5P6PGEx2cZfco EJW5xGOzHQSzdRFyXmdVjnTSlbW74ptiV7YGqGsIE2L1R5KiKHnnPxW8NSSOkecNCJTIt2 C4jUifvRQ/dQQJGlhdcp6eXDrknSsuU= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-332-Ys8AyIbbNxes6rLYuMTT5g-1; Tue, 18 May 2021 13:46:53 -0400 X-MC-Unique: Ys8AyIbbNxes6rLYuMTT5g-1 Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 7697310C7F01; Tue, 18 May 2021 17:46:19 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-112-30.ams2.redhat.com [10.36.112.30]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 934BC5DEAD; Tue, 18 May 2021 17:46:17 +0000 (UTC) Subject: Re: [edk2-devel] [PATCH v2 0/5] Enable Cloud Hypervisor support in edk2 To: devel@edk2.groups.io, jianyong.wu@arm.com, ardb+tianocore@kernel.org, sami.mujawar@arm.com Cc: hao.a.wu@intel.com, justin.he@arm.com References: <20210517065032.82423-1-jianyong.wu@arm.com> From: "Laszlo Ersek" Message-ID: <01e73dba-7d53-dade-3074-37d10d796eae@redhat.com> Date: Tue, 18 May 2021 19:46:16 +0200 MIME-Version: 1.0 In-Reply-To: <20210517065032.82423-1-jianyong.wu@arm.com> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=lersek@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit On 05/17/21 08:50, Jianyong Wu wrote: > Cloud Hypervisor is an open source Virtual Machine Monitor (VMM) that > runs on top of KVM. Cloud Hypervisor is implemented in Rust and is based > on the rust-vmm crates. See [1] to find more. > > To support UEFI, Cloud Hypervisor is introduced here. > There are three parts to be considered to do this enablements, that is: > 1. memory initialization > > 2. specific ACPI service implementation > compared with qemu, there is no device like Fw-cfg, so we has no > elegant way to get the RSDP address. A specific ACPI implementation is > introduced here. > > 3. build configuration file > > This enablement bases on the implentation for qemu so some code is > borrowed. > > [1] https://github.com/cloud-hypervisor/cloud-hypervisor > > Jianyong Wu (5): > ArmVirtPkg: Library: Memory initialization for Cloud Hypervisor > MdeMoudlePkg: introduce new PCD for Acpi/rsdp > ArmVirtPkg: enable ACPI for cloud hypervisor > ArmVirtPkg: Introduce Cloud Hypervisor to edk2 family > Maintainers: update Maintainers file as new files/folders created > > MdeModulePkg/MdeModulePkg.dec | 7 + > ArmVirtPkg/ArmVirtCloudHv.dsc | 455 ++++++++++++++++++ > ArmVirtPkg/ArmVirtCloudHv.fdf | 292 +++++++++++ > .../CloudHvAcpiPlatformDxe.inf | 51 ++ > .../CloudHvHasAcpiDtDxe.inf | 43 ++ > .../CloudHvVirtMemInfoPeiLib.inf | 47 ++ > .../CloudHvAcpiPlatformDxe/CloudHvAcpi.c | 73 +++ > .../CloudHvHasAcpiDtDxe.c | 69 +++ > .../CloudHvVirtMemInfoLib.c | 94 ++++ > .../CloudHvVirtMemInfoPeiLibConstructor.c | 100 ++++ > ArmVirtPkg/ArmVirtCloudHvFvMain.fdf.inc | 169 +++++++ > Maintainers.txt | 7 + > 12 files changed, 1407 insertions(+) > create mode 100644 ArmVirtPkg/ArmVirtCloudHv.dsc > create mode 100644 ArmVirtPkg/ArmVirtCloudHv.fdf > create mode 100644 ArmVirtPkg/CloudHvAcpiPlatformDxe/CloudHvAcpiPlatformDxe.inf > create mode 100644 ArmVirtPkg/CloudHvPlatformHasAcpiDtDxe/CloudHvHasAcpiDtDxe.inf > create mode 100644 ArmVirtPkg/Library/CloudHvVirtMemInfoLib/CloudHvVirtMemInfoPeiLib.inf > create mode 100644 ArmVirtPkg/CloudHvAcpiPlatformDxe/CloudHvAcpi.c > create mode 100644 ArmVirtPkg/CloudHvPlatformHasAcpiDtDxe/CloudHvHasAcpiDtDxe.c > create mode 100644 ArmVirtPkg/Library/CloudHvVirtMemInfoLib/CloudHvVirtMemInfoLib.c > create mode 100644 ArmVirtPkg/Library/CloudHvVirtMemInfoLib/CloudHvVirtMemInfoPeiLibConstructor.c > create mode 100644 ArmVirtPkg/ArmVirtCloudHvFvMain.fdf.inc > Confirming that I got this in my review queue. I'll need some time to get to it; please bear with me. Thanks, Laszlo