From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [216.205.24.124]) by mx.groups.io with SMTP id smtpd.web10.6394.1624535480796218939 for ; Thu, 24 Jun 2021 04:51:21 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=Z35c0sqd; spf=pass (domain: redhat.com, ip: 216.205.24.124, mailfrom: lersek@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1624535480; 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=uD1nB0DbhMWwMRWRam1/auchj+D1z4nu6ScR9qM0vx4=; b=Z35c0sqdUZ3cwjyo/1eVU6cfXZdC4iugzISoffTX2iaNCcJ9SHbfbEOmomtxA3FawlCSPV z/3jmfysl4Hqg50B/vtM6QnTp/X9uKhuG8Rre0EHqloqa+mwrSVQIRIRvyAC3l1OESjGjS D6qWPwzOGOPwwpvdi8wQJkEl2ibemSY= 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-225-WHsfmZxhPL-PlZR4KpMFkg-1; Thu, 24 Jun 2021 07:51:16 -0400 X-MC-Unique: WHsfmZxhPL-PlZR4KpMFkg-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 35EAA100CEC0; Thu, 24 Jun 2021 11:51:15 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-114-106.ams2.redhat.com [10.36.114.106]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 88C645D9C6; Thu, 24 Jun 2021 11:51:13 +0000 (UTC) Subject: Re: [edk2-devel] [PATCH v1 0/5] Add ACPI support for Kvmtool To: devel@edk2.groups.io, pierre.gondois@arm.com, Sami Mujawar Cc: Ard Biesheuvel , Leif Lindholm , Akanksha Jain , Alexandru Elisei References: <20210623140640.16754-1-Pierre.Gondois@arm.com> From: "Laszlo Ersek" Message-ID: Date: Thu, 24 Jun 2021 13:51:11 +0200 MIME-Version: 1.0 In-Reply-To: <20210623140640.16754-1-Pierre.Gondois@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 06/23/21 16:06, PierreGondois wrote: > From: Pierre Gondois > > Kvmtool dynamically generates a device tree describing the platform > to boot on. Using the patch-sets listed below, the DynamicTables > framework generates ACPI tables describing a similar platform. > > This patch-set: > - adds a ConfigurationManager allowing to generate ACPI tables > for Kvmtool > - adds the acpiview command line utility to the ArmVirtPkg > - update ArmVirtPkg.ci.yaml to add new words and use the > DynamicTablesPkg > > This patch sets also set the default platform description format > to ACPI instead of the device tree (c.f.: PcdForceNoAcpi is set > to FALSE). > > The changes can be seen at: https://github.com/PierreARM/edk2/tree/1456_Add_ACPI_support_for_Kvmtool_v1 > The results of the CI can be seen at: https://github.com/tianocore/edk2/pull/1753 > > This patch-set is dependent over the following patch-sets: > [PATCH v1 00/10] Various DynamicTablesPkg modifications > https://edk2.groups.io/g/devel/message/76929 > and: > [PATCH v1 00/13] Create a SSDT CPU topology generator > https://edk2.groups.io/g/devel/message/76941 > and: > [PATCH v1 0/7] Create a SSDT PCIe generator > https://edk2.groups.io/g/devel/message/76958 > and: > [PATCH v1 00/14] Implement a FdtHwInfoParserLib > https://edk2.groups.io/g/devel/message/76967 > and: > [PATCH v1 0/5] Add DynamicPlatRepoLib > https://edk2.groups.io/g/devel/message/76984 Not sure if you want just one BZ for all of these subfeatures, or one BZ per subfeature, but we definitely need at least one BZ for this series. Please update the commit messages accordingly. Thanks Laszlo > > Pierre Gondois (1): > ArmVirtPkg: Add cspell exceptions > > Sami Mujawar (4): > ArmVirtPkg: Add DSDT ACPI table for Kvmtool firmware > ArmVirtPkg: Add Configuration Manager for Kvmtool firmware > ArmVirtPkg: Enable ACPI support for Kvmtool > ArmVirtPkg: Enable Acpiview for ArmVirtPkg > > ArmVirtPkg/ArmVirt.dsc.inc | 3 +- > ArmVirtPkg/ArmVirtKvmTool.dsc | 18 +- > ArmVirtPkg/ArmVirtKvmTool.fdf | 11 + > ArmVirtPkg/ArmVirtPkg.ci.yaml | 3 + > .../KvmtoolCfgMgrDxe/AslTables/Dsdt.asl | 19 + > .../KvmtoolCfgMgrDxe/ConfigurationManager.c | 948 ++++++++++++++++++ > .../KvmtoolCfgMgrDxe/ConfigurationManager.h | 94 ++ > .../ConfigurationManagerDxe.inf | 58 ++ > 8 files changed, 1151 insertions(+), 3 deletions(-) > create mode 100644 ArmVirtPkg/KvmtoolCfgMgrDxe/AslTables/Dsdt.asl > create mode 100644 ArmVirtPkg/KvmtoolCfgMgrDxe/ConfigurationManager.c > create mode 100644 ArmVirtPkg/KvmtoolCfgMgrDxe/ConfigurationManager.h > create mode 100644 ArmVirtPkg/KvmtoolCfgMgrDxe/ConfigurationManagerDxe.inf >