From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web08.6380.1666700954291716466 for ; Tue, 25 Oct 2022 05:29:14 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: arm.com, ip: 217.140.110.172, mailfrom: pierre.gondois@arm.com) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 0FA96D6E; Tue, 25 Oct 2022 05:29:20 -0700 (PDT) Received: from [10.34.100.128] (pierre123.nice.arm.com [10.34.100.128]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id D81C53F7B4; Tue, 25 Oct 2022 05:29:11 -0700 (PDT) Message-ID: <08ddc043-a99b-4c72-e397-9a4d414519ab@arm.com> Date: Tue, 25 Oct 2022 14:29:06 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.2.2 Subject: Re: [PATCH v1 0/2] DynamicTablesPkg: Introduce SMBIOS dispatcher To: Sami Mujawar , devel@edk2.groups.io Cc: Alexei.Fedorov@arm.com, gmahadevan@nvidia.com, abner.chang@amd.com, jbrasen@nvidia.com, ashishsingha@nvidia.com, nramirez@nvidia.com, wwatson@nvidia.com, Matteo.Carlini@arm.com, Akanksha.Jain2@arm.com, Ben.Adderson@arm.com, Samer.El-Haj-Mahmoud@arm.com, nd@arm.com References: <20221018153344.16876-1-sami.mujawar@arm.com> From: "PierreGondois" In-Reply-To: <20221018153344.16876-1-sami.mujawar@arm.com> Content-Language: en-US Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Hello Sami, Reviewed-by: Pierre Gondois Regards, Pierre On 10/18/22 17:33, Sami Mujawar wrote: > Some SMBIOS tables have dependencies on other SMBIOS tables. These > dependencies are established using handles pointing to the dependent > SMBIOS tables. > This requires the dependent SMBIOS table to be installed before the > parent SMBIOS table can be installed. > > To facilitate dynamic SMBIOS table generation, a SMBIOS table > dispatcher has been introduced. The SMBIOS table dispatcher > schedules the installation of dependent SMBIOS tables before > the parent SMBIOS table is installed. > > The changes can be seen at: > https://github.com/samimujawar/edk2/tree/2406_smbios_dispatcher_v1 > > Sami Mujawar (2): > DynamicTablesPkg: Define a SMBIOS Structure/Table type > DynamicTablesPkg: Add SMBIOS table dispatcher > > DynamicTablesPkg/Drivers/DynamicTableManagerDxe/DynamicTableManagerDxe.inf | 4 +- > DynamicTablesPkg/Drivers/DynamicTableManagerDxe/SmbiosTableDispatcher.c | 304 ++++++++++++++++++++ > DynamicTablesPkg/Drivers/DynamicTableManagerDxe/SmbiosTableDispatcher.h | 133 +++++++++ > DynamicTablesPkg/Include/StandardNameSpaceObjects.h | 17 ++ > 4 files changed, 457 insertions(+), 1 deletion(-) > create mode 100644 DynamicTablesPkg/Drivers/DynamicTableManagerDxe/SmbiosTableDispatcher.c > create mode 100644 DynamicTablesPkg/Drivers/DynamicTableManagerDxe/SmbiosTableDispatcher.h >