From nobody Mon Apr 29 02:38:27 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) client-ip=66.175.222.108; envelope-from=bounce+27952+72102+1787277+3901457@groups.io; helo=mail02.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+72102+1787277+3901457@groups.io ARC-Seal: i=1; a=rsa-sha256; t=1614102235; cv=none; d=zohomail.com; s=zohoarc; b=W+xdbqBZK1YuzhL8nSdVHgnC6eH0fwMwTvmAqcbFOavsjKhwEpmDLz8t8/sqTZUKwOjaH+cDrco5NSfZG6R4Vg1EwvRkcqZHwczhh475g9HfsyY8gz2/vTkXYPwmIBY2MChitQBxD8oddGQCijqOwL+FH2b2MfuMcs0RclqQ7kY= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1614102235; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Id:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:References:Sender:Subject:To; bh=vfcjzjNc8HagGHvKLatnPiREOi3cVHRQnRrZPrF+atU=; b=V3yLZMj4mUjvnINtl25lr6Syle7TPnkBaCnKBLp/X5ku4hqWsqoQYG3sVUh0WKxAiCmJw6fK5PwE8nONVhaNfX92KE1L6DeMySjt9vuc3d7rDPpqIt5TSs/+0Wz3Mc2OZWA3KoSH2YGBOy4SxmhR/VhfpwN0cQVJcHsyMIRvAl4= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+72102+1787277+3901457@groups.io Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 1614102235009759.4636376590067; Tue, 23 Feb 2021 09:43:55 -0800 (PST) Return-Path: X-Received: by 127.0.0.2 with SMTP id T1RKYY1788612xyx5rp8FA9i; Tue, 23 Feb 2021 09:43:54 -0800 X-Received: from mail-ot1-f44.google.com (mail-ot1-f44.google.com [209.85.210.44]) by mx.groups.io with SMTP id smtpd.web09.62.1614102229217982368 for ; Tue, 23 Feb 2021 09:43:49 -0800 X-Received: by mail-ot1-f44.google.com with SMTP id d9so2272607ote.12 for ; Tue, 23 Feb 2021 09:43:48 -0800 (PST) X-Gm-Message-State: CHSJstDDhbwzvkJleEozroeux1787277AA= X-Google-Smtp-Source: ABdhPJxK/ZU/Zzloc7Za2skPwE9WWWACYQUKwhhVHAbP+yY955lWbuyR8ODwfZT95JNZu6gndA4oVg== X-Received: by 2002:a9d:6247:: with SMTP id i7mr8819343otk.75.1614102228063; Tue, 23 Feb 2021 09:43:48 -0800 (PST) X-Received: from cube.nuviainc.com (c-174-52-16-57.hsd1.ut.comcast.net. [174.52.16.57]) by smtp.gmail.com with ESMTPSA id s9sm4419918otb.11.2021.02.23.09.43.46 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 23 Feb 2021 09:43:47 -0800 (PST) From: "Rebecca Cran" To: devel@edk2.groups.io Cc: Rebecca Cran , Ard Biesheuvel , Leif Lindholm , Graeme Gregory , Radoslaw Biernacki Subject: [edk2-devel] [edk2-platforms PATCH v5 1/4] SbsaQemu: Add FdtHelperLib Date: Tue, 23 Feb 2021 10:43:39 -0700 Message-Id: <20210223174342.8572-2-rebecca@nuviainc.com> In-Reply-To: <20210223174342.8572-1-rebecca@nuviainc.com> References: <20210223174342.8572-1-rebecca@nuviainc.com> MIME-Version: 1.0 Precedence: Bulk List-Unsubscribe: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,rebecca@nuviainc.com Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1614102234; bh=2OamM1D4Pm9uZj28IFsJbAabtc22TYaFj3hk3IBde/8=; h=Cc:Date:From:Reply-To:Subject:To; b=sCybNQy9NmGHoVEpN3GnL+qrkzgq1gJEHL2mhYo6uPRLa0Bd2iNqANB7NxvwsOrGvMv pK0KKefgzIi/n6nrXvzDUKz41B2II3RewzJRYQFNg+Az8KFNuWijLCuM9QYhjUiMtHiSh 2OU3A/NjLKexaFLfkDpGDCwby+K9x7E5gA0= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" The CountCpusFromFdt function is now used in two places. Create FdtHelperLib for this and similar functions. Signed-off-by: Rebecca Cran Reviewed-by: Leif Lindholm --- Platform/Qemu/SbsaQemu/SbsaQemu.dsc | 2 + Silicon/Qemu/SbsaQemu/Include/Library/FdtHelperLib.h | 24 ++++++++ Silicon/Qemu/SbsaQemu/Library/FdtHelperLib/FdtHelperLib.c | 62 +++++++++= +++++++++++ Silicon/Qemu/SbsaQemu/Library/FdtHelperLib/FdtHelperLib.inf | 28 +++++++++ 4 files changed, 116 insertions(+) diff --git a/Platform/Qemu/SbsaQemu/SbsaQemu.dsc b/Platform/Qemu/SbsaQemu/S= bsaQemu.dsc index f6af3f9111ee..8faad3eda217 100644 --- a/Platform/Qemu/SbsaQemu/SbsaQemu.dsc +++ b/Platform/Qemu/SbsaQemu/SbsaQemu.dsc @@ -121,6 +121,8 @@ DEFINE NETWORK_HTTP_BOOT_ENABLE =3D FALSE # ARM PL011 UART Driver PL011UartLib|ArmPlatformPkg/Library/PL011UartLib/PL011UartLib.inf =20 + FdtHelperLib|Silicon/Qemu/SbsaQemu/Library/FdtHelperLib/FdtHelperLib.inf + # Debug Support PeCoffExtraActionLib|ArmPkg/Library/DebugPeCoffExtraActionLib/DebugPeCof= fExtraActionLib.inf DebugAgentLib|MdeModulePkg/Library/DebugAgentLibNull/DebugAgentLibNull.i= nf diff --git a/Silicon/Qemu/SbsaQemu/Include/Library/FdtHelperLib.h b/Silicon= /Qemu/SbsaQemu/Include/Library/FdtHelperLib.h new file mode 100644 index 000000000000..e9e7281c1342 --- /dev/null +++ b/Silicon/Qemu/SbsaQemu/Include/Library/FdtHelperLib.h @@ -0,0 +1,24 @@ +/** @file +* FdtHelperLib.h +* +* Copyright (c) 2021, NUVIA Inc. All rights reserved. +* +* SPDX-License-Identifier: BSD-2-Clause-Patent +* +**/ + +#ifndef FDT_HELPER_LIB_ +#define FDT_HELPER_LIB_ + +/** Walks through the Device Tree created by Qemu and counts the number + of CPUs present in it. + + @return The number of CPUs present. +**/ +EFIAPI +UINT32 +FdtHelperCountCpus ( + VOID + ); + +#endif /* FDT_HELPER_LIB_ */ diff --git a/Silicon/Qemu/SbsaQemu/Library/FdtHelperLib/FdtHelperLib.c b/Si= licon/Qemu/SbsaQemu/Library/FdtHelperLib/FdtHelperLib.c new file mode 100644 index 000000000000..411f035ee7d8 --- /dev/null +++ b/Silicon/Qemu/SbsaQemu/Library/FdtHelperLib/FdtHelperLib.c @@ -0,0 +1,62 @@ +/** @file +* FdtHelperLib.c +* +* Copyright (c) 2021, NUVIA Inc. All rights reserved. +* Copyright (c) 2020, Linaro Ltd. All rights reserved. +* +* SPDX-License-Identifier: BSD-2-Clause-Patent +* +**/ + +#include +#include +#include +#include +#include + +/** Walks through the Device Tree created by Qemu and counts the number + of CPUs present in it. + + @return The number of CPUs present. +**/ +EFIAPI +UINT32 +FdtHelperCountCpus ( + VOID + ) +{ + VOID *DeviceTreeBase; + INT32 Node; + INT32 Prev; + INT32 CpuNode; + UINT32 CpuCount; + + DeviceTreeBase =3D (VOID *)(UINTN)PcdGet64 (PcdDeviceTreeBaseAddress); + ASSERT (DeviceTreeBase !=3D NULL); + + // Make sure we have a valid device tree blob + ASSERT (fdt_check_header (DeviceTreeBase) =3D=3D 0); + + CpuNode =3D fdt_path_offset (DeviceTreeBase, "/cpus"); + if (CpuNode <=3D 0) { + DEBUG ((DEBUG_ERROR, "Unable to locate /cpus in device tree\n")); + return 0; + } + + CpuCount =3D 0; + + // Walk through /cpus node and count the number of subnodes. + // The count of these subnodes corresponds to the number of + // CPUs created by Qemu. + Prev =3D fdt_first_subnode (DeviceTreeBase, CpuNode); + while (1) { + CpuCount++; + Node =3D fdt_next_subnode (DeviceTreeBase, Prev); + if (Node < 0) { + break; + } + Prev =3D Node; + } + + return CpuCount; +} diff --git a/Silicon/Qemu/SbsaQemu/Library/FdtHelperLib/FdtHelperLib.inf b/= Silicon/Qemu/SbsaQemu/Library/FdtHelperLib/FdtHelperLib.inf new file mode 100644 index 000000000000..d84c16f888d1 --- /dev/null +++ b/Silicon/Qemu/SbsaQemu/Library/FdtHelperLib/FdtHelperLib.inf @@ -0,0 +1,28 @@ +#/** @file +# +# Component description file for FdtHelperLib module +# +# Copyright (c) 2021, NUVIA Inc. All rights reserved. +# +# SPDX-License-Identifier: BSD-2-Clause-Patent +# +#**/ + +[Defines] + INF_VERSION =3D 1.29 + BASE_NAME =3D FdtHelperLib + FILE_GUID =3D 34e4396f-c2fc-4f9e-ad58-0f98e99e3875 + MODULE_TYPE =3D BASE + VERSION_STRING =3D 1.0 + LIBRARY_CLASS =3D FdtHelperLib + +[Sources.common] + FdtHelperLib.c + +[Packages] + EmbeddedPkg/EmbeddedPkg.dec + MdePkg/MdePkg.dec + Silicon/Qemu/SbsaQemu/SbsaQemu.dec + +[FixedPcd] + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdDeviceTreeBaseAddress --=20 2.26.2 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#72102): https://edk2.groups.io/g/devel/message/72102 Mute This Topic: https://groups.io/mt/80857021/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- From nobody Mon Apr 29 02:38:27 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) client-ip=66.175.222.108; envelope-from=bounce+27952+72103+1787277+3901457@groups.io; helo=mail02.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+72103+1787277+3901457@groups.io ARC-Seal: i=1; a=rsa-sha256; t=1614102236; cv=none; d=zohomail.com; s=zohoarc; b=gD+zSj9k+R1eFyDulH17LwQZ43YjMO2PrWH6qnrvHVONafinXws6MxQUQ1gDtViz7aB3Wql/n7l98sL4jPN+HLBcy+j5mClQO8ocfZbLb2XMMtT8UeUCGZaox5LXchsC6ZgI0IEckJaHzPEoliUxyzpmb3AMVlFJ1rfqT5OVCDg= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1614102236; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Id:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:References:Sender:Subject:To; bh=2Yd3F2YhdgtB6/7HKFYsLE0tZ1V1vE8emgcxdtwMkN4=; b=ko3mNu+RDSGR6vTMsSa1Y/rDz25uohDl7Cey286OaIlGX7NNXfxAsTqUjHItzFbYfzAeg6m7sZhOCnx6emxAanjf504/Zm/lF3Rl+JZ7Pn1JtuYnIWynkq6Pkee7b0zgwBpIehew+thdQE8F4VpI87JSEDTp9FUNwzZclknCgPA= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+72103+1787277+3901457@groups.io Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 1614102236133455.9700634853232; Tue, 23 Feb 2021 09:43:56 -0800 (PST) Return-Path: X-Received: by 127.0.0.2 with SMTP id EKIrYY1788612xwUCE5t5DHv; Tue, 23 Feb 2021 09:43:55 -0800 X-Received: from mail-ot1-f41.google.com (mail-ot1-f41.google.com [209.85.210.41]) by mx.groups.io with SMTP id smtpd.web09.63.1614102230309645950 for ; Tue, 23 Feb 2021 09:43:50 -0800 X-Received: by mail-ot1-f41.google.com with SMTP id s6so16414318otk.4 for ; Tue, 23 Feb 2021 09:43:49 -0800 (PST) X-Gm-Message-State: pmGOzZ2Lj0qYGeUS3DhIgZhjx1787277AA= X-Google-Smtp-Source: ABdhPJzBh9eu4cZV0NKFzx3vySJe54xzr80tjRopXQZVRruciSCPzniagqn3Oj6dPj1koUOjvWnI8Q== X-Received: by 2002:a9d:709a:: with SMTP id l26mr6467214otj.178.1614102229154; Tue, 23 Feb 2021 09:43:49 -0800 (PST) X-Received: from cube.nuviainc.com (c-174-52-16-57.hsd1.ut.comcast.net. [174.52.16.57]) by smtp.gmail.com with ESMTPSA id s9sm4419918otb.11.2021.02.23.09.43.48 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 23 Feb 2021 09:43:48 -0800 (PST) From: "Rebecca Cran" To: devel@edk2.groups.io Cc: Rebecca Cran , Ard Biesheuvel , Leif Lindholm , Graeme Gregory , Radoslaw Biernacki Subject: [edk2-devel] [edk2-platforms PATCH v5 2/4] SbsaQemu: Update SbsaQemuAcpiDxe to use FdtHelperLib Date: Tue, 23 Feb 2021 10:43:40 -0700 Message-Id: <20210223174342.8572-3-rebecca@nuviainc.com> In-Reply-To: <20210223174342.8572-1-rebecca@nuviainc.com> References: <20210223174342.8572-1-rebecca@nuviainc.com> MIME-Version: 1.0 Precedence: Bulk List-Unsubscribe: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,rebecca@nuviainc.com Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1614102235; bh=DIf8tREm1cHHIWAT1itcWgPHIqADn4+xj8KLiGwOT+0=; h=Cc:Date:From:Reply-To:Subject:To; b=eZG18fdyCRwDnTTc8wqSSVfapo+Zce1vvDkW1/O55SCarAxHMVUUSaOC1+Zk5tdBWdu 5E/KKQQZunEOjF6TbADd9MH0kHmknhkoSool2gwiAP94Vwf7XFbc4PUNgSH3PyS9zfYXa YEV2O2RovJpAHwbRFbI8BXiX0JWB6OnBq8A= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" Use FdtHelperCountCpus from FdtHelperLib. Signed-off-by: Rebecca Cran Reviewed-by: Leif Lindholm --- Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.c | 55 ++-= ----------------- Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.inf | 1 + 2 files changed, 6 insertions(+), 50 deletions(-) diff --git a/Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.= c b/Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.c index fb7c1835c3d7..037c7cff4c18 100644 --- a/Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.c +++ b/Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.c @@ -12,6 +12,7 @@ #include #include #include +#include #include #include #include @@ -25,55 +26,6 @@ STATIC INT32 FdtFirstCpuOffset; STATIC INT32 FdtCpuNodeSize; =20 -/* - * A function that walks through the Device Tree created - * by Qemu and counts the number of CPUs present in it. - */ -STATIC -VOID -CountCpusFromFdt ( - VOID -) -{ - VOID *DeviceTreeBase; - INT32 Node, Prev; - RETURN_STATUS PcdStatus; - INT32 CpuNode; - INT32 CpuCount; - - DeviceTreeBase =3D (VOID *)(UINTN)PcdGet64 (PcdDeviceTreeBaseAddress); - ASSERT (DeviceTreeBase !=3D NULL); - - // Make sure we have a valid device tree blob - ASSERT (fdt_check_header (DeviceTreeBase) =3D=3D 0); - - CpuNode =3D fdt_path_offset (DeviceTreeBase, "/cpus"); - if (CpuNode <=3D 0) { - DEBUG ((DEBUG_ERROR, "Unable to locate /cpus in device tree\n")); - return; - } - - CpuCount =3D 0; - - // Walk through /cpus node and count the number of subnodes. - // The count of these subnodes corresponds to the number of - // CPUs created by Qemu. - Prev =3D fdt_first_subnode (DeviceTreeBase, CpuNode); - FdtFirstCpuOffset =3D Prev; - while (1) { - CpuCount++; - Node =3D fdt_next_subnode (DeviceTreeBase, Prev); - if (Node < 0) { - break; - } - FdtCpuNodeSize =3D Node - Prev; - Prev =3D Node; - } - - PcdStatus =3D PcdSet32S (PcdCoreCount, CpuCount); - ASSERT_RETURN_ERROR (PcdStatus); -} - /* * Get MPIDR from device tree passed by Qemu */ @@ -487,9 +439,12 @@ InitializeSbsaQemuAcpiDxe ( { EFI_STATUS Status; EFI_ACPI_TABLE_PROTOCOL *AcpiTable; + UINT32 NumCores; =20 // Parse the device tree and get the number of CPUs - CountCpusFromFdt (); + NumCores =3D FdtHelperCountCpus (); + Status =3D PcdSet32S (PcdCoreCount, NumCores); + ASSERT_RETURN_ERROR (Status); =20 // Check if ACPI Table Protocol has been installed Status =3D gBS->LocateProtocol ( diff --git a/Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.= inf b/Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.inf index 127eef029f3c..a58ebfaf76d5 100644 --- a/Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.inf +++ b/Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.inf @@ -34,6 +34,7 @@ BaseLib DebugLib DxeServicesLib + FdtHelperLib FdtLib PcdLib PrintLib --=20 2.26.2 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#72103): https://edk2.groups.io/g/devel/message/72103 Mute This Topic: https://groups.io/mt/80857022/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- From nobody Mon Apr 29 02:38:27 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) client-ip=66.175.222.108; envelope-from=bounce+27952+72104+1787277+3901457@groups.io; helo=mail02.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+72104+1787277+3901457@groups.io ARC-Seal: i=1; a=rsa-sha256; t=1614102237; cv=none; d=zohomail.com; s=zohoarc; b=VAflOsy2M5mU5KPBkA5IrzOfJ5phnsZfKoWRszurNSjtMdi5GQQAJ9kHWD0WhKeQGQx290pcTb60Ob8fXJhqHIb9N8WfLkt0jChVCaPKm3bu3pWgMhNcjjArClur1NfbABbg/heKIHeZFPdgvhI/UuJMTWI2o2ocR1VXEQ7yiOM= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1614102237; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Id:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:References:Sender:Subject:To; bh=XWxJP1dhxMhETsGISE9jgWpXj2IPcl0f1sVjbxkwuTk=; b=eM2pDO52EaOosAbL8/kKZ9slgXehLguaZYoSp4uOpM7ZWlyN9Unzf9O8RWm0edKyAuAShnW97hfP48LYWqdNiBeCAnhiT5Z1FSxjiGhblVAUKoGeqPxGH48thr6DdyvF7PRpkMvpk5pOOX7uIMm/BXst4d2WcxTxOrE0FYwx7Os= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+72104+1787277+3901457@groups.io Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 1614102237348147.30031519524846; Tue, 23 Feb 2021 09:43:57 -0800 (PST) Return-Path: X-Received: by 127.0.0.2 with SMTP id zTVrYY1788612xkiVhqp3WgB; Tue, 23 Feb 2021 09:43:57 -0800 X-Received: from mail-ot1-f41.google.com (mail-ot1-f41.google.com [209.85.210.41]) by mx.groups.io with SMTP id smtpd.web09.64.1614102231501148425 for ; Tue, 23 Feb 2021 09:43:51 -0800 X-Received: by mail-ot1-f41.google.com with SMTP id s107so16413088otb.8 for ; Tue, 23 Feb 2021 09:43:51 -0800 (PST) X-Gm-Message-State: FTYyTdoebVZzLQTKHjDOEsgEx1787277AA= X-Google-Smtp-Source: ABdhPJx4ZumzlSuvf/T5NvmuKm9MNd5amHQLAS7wuBaXZs2YRCXdxhxtEI9Q0cOM1GuGYC1ZTIggxA== X-Received: by 2002:a05:6830:1d68:: with SMTP id l8mr21214341oti.238.1614102230298; Tue, 23 Feb 2021 09:43:50 -0800 (PST) X-Received: from cube.nuviainc.com (c-174-52-16-57.hsd1.ut.comcast.net. [174.52.16.57]) by smtp.gmail.com with ESMTPSA id s9sm4419918otb.11.2021.02.23.09.43.49 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 23 Feb 2021 09:43:49 -0800 (PST) From: "Rebecca Cran" To: devel@edk2.groups.io Cc: Rebecca Cran , Ard Biesheuvel , Leif Lindholm , Graeme Gregory , Radoslaw Biernacki Subject: [edk2-devel] [edk2-platforms PATCH v5 3/4] Platform/Qemu/SbsaQemu: Add SMBIOS tables Date: Tue, 23 Feb 2021 10:43:41 -0700 Message-Id: <20210223174342.8572-4-rebecca@nuviainc.com> In-Reply-To: <20210223174342.8572-1-rebecca@nuviainc.com> References: <20210223174342.8572-1-rebecca@nuviainc.com> MIME-Version: 1.0 Precedence: Bulk List-Unsubscribe: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,rebecca@nuviainc.com Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1614102237; bh=/ZfUdV/OlOSh37Vm9tAHDiIcal0Qoi6Twj1XKKtW5dE=; h=Cc:Date:From:Reply-To:Subject:To; b=XZLWT49svxJuktIC4t5QPouj2m4qud6NJQq3FA3o5UZsxuIpNRrabgOloKqBJfBxGck EfYQASka4vDrjzVkYkWmvjMpgGCT0PaOf8dVO00stMlddSO8EfMHccS4KV0zu+2iRhxgR gsAlsd76NWkTq9CQgzvubvCO8B8kiUgxl9Y= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" o Add SMBIOS 3.4.0 tables using ArmPkg/Universal/Smbios. o Bump the PcdSmbiosVersion PCD from 0x300 to 0x304 to indicate support for SMBIOS 3.4.0, as is required by SBBR. o Add an implementation of OemMiscLib that provides the system information. The serial numbers, asset tags etc. are currently all fixed strings, to allow fwts to pass without errors. o Add SMBIOS PCDs to identify the platform. The processor serial number, asset tag and part number are populated because otherwise fwts reports errors. Signed-off-by: Rebecca Cran Reviewed-by: Leif Lindholm --- Platform/Qemu/SbsaQemu/OemMiscLib/OemMiscLib.c | 242 ++++++++++++++++++++ Platform/Qemu/SbsaQemu/OemMiscLib/OemMiscLib.inf | 53 +++++ Platform/Qemu/SbsaQemu/SbsaQemu.dsc | 48 +++- Platform/Qemu/SbsaQemu/SbsaQemu.fdf | 7 + Silicon/Qemu/SbsaQemu/SbsaQemu.dec | 18 ++ 5 files changed, 367 insertions(+), 1 deletion(-) diff --git a/Platform/Qemu/SbsaQemu/OemMiscLib/OemMiscLib.c b/Platform/Qemu= /SbsaQemu/OemMiscLib/OemMiscLib.c new file mode 100644 index 000000000000..11340788aadf --- /dev/null +++ b/Platform/Qemu/SbsaQemu/OemMiscLib/OemMiscLib.c @@ -0,0 +1,242 @@ +/** @file +* OemMiscLib.c +* +* Copyright (c) 2021, NUVIA Inc. All rights reserved. +* Copyright (c) 2020, Linaro Ltd. All rights reserved. +* +* SPDX-License-Identifier: BSD-2-Clause-Patent +* +**/ + +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include + +/** Returns whether the specified processor is present or not. + + @param ProcessorIndex The processor index to check. + + @return TRUE if the processor is present, FALSE otherwise. +**/ +BOOLEAN +EFIAPI +OemIsProcessorPresent ( + UINTN ProcessorIndex + ) +{ + if (ProcessorIndex < FdtHelperCountCpus ()) { + return TRUE; + } + + return FALSE; +} + +/** Gets the CPU frequency of the specified processor. + + @param ProcessorIndex Index of the processor to get the frequency for. + + @return CPU frequency in Hz +**/ +UINTN +EFIAPI +OemGetCpuFreq ( + UINT8 ProcessorIndex + ) +{ + return 2000000000; // 2 GHz +} + +/** Gets information about the specified processor and stores it in + the structures provided. + + @param ProcessorIndex Index of the processor to get the information for. + @param ProcessorStatus Processor status. + @param ProcessorCharacteristics Processor characteritics. + @param MiscProcessorData Miscellaneous processor information. + + @return TRUE on success, FALSE on failure. +**/ +BOOLEAN +EFIAPI +OemGetProcessorInformation ( + IN UINTN ProcessorIndex, + IN OUT PROCESSOR_STATUS_DATA *ProcessorStatus, + IN OUT PROCESSOR_CHARACTERISTIC_FLAGS *ProcessorCharacteristics, + IN OUT OEM_MISC_PROCESSOR_DATA *MiscProcessorData + ) +{ + UINT16 ProcessorCount; + + ProcessorCount =3D FdtHelperCountCpus (); + + if (ProcessorIndex < ProcessorCount) { + ProcessorStatus->Bits.CpuStatus =3D 1; // CPU enabled + ProcessorStatus->Bits.Reserved1 =3D 0; + ProcessorStatus->Bits.SocketPopulated =3D 1; + ProcessorStatus->Bits.Reserved2 =3D 0; + } else { + ProcessorStatus->Bits.CpuStatus =3D 0; // CPU disabled + ProcessorStatus->Bits.Reserved1 =3D 0; + ProcessorStatus->Bits.SocketPopulated =3D 0; + ProcessorStatus->Bits.Reserved2 =3D 0; + } + + ProcessorCharacteristics->ProcessorReserved1 =3D 0; + ProcessorCharacteristics->ProcessorUnknown =3D 0; + ProcessorCharacteristics->Processor64BitCapable =3D 1; + ProcessorCharacteristics->ProcessorMultiCore =3D 0; + ProcessorCharacteristics->ProcessorHardwareThread =3D 0; + ProcessorCharacteristics->ProcessorExecuteProtection =3D 1; + ProcessorCharacteristics->ProcessorEnhancedVirtualization =3D 0; + ProcessorCharacteristics->ProcessorPowerPerformanceCtrl =3D 0; + ProcessorCharacteristics->Processor128BitCapable =3D 0; + ProcessorCharacteristics->ProcessorArm64SocId =3D 1; + ProcessorCharacteristics->ProcessorReserved2 =3D 0; + + MiscProcessorData->CurrentSpeed =3D 2000; + MiscProcessorData->MaxSpeed =3D 2000; + MiscProcessorData->CoreCount =3D 1; + MiscProcessorData->CoresEnabled =3D 1; + MiscProcessorData->ThreadCount =3D 1; + + return TRUE; +} + +/** Gets the maximum number of processors supported by the platform. + + @return The maximum number of processors. +**/ +UINT8 +EFIAPI +OemGetMaxProcessors ( + VOID + ) +{ + return FdtHelperCountCpus (); +} + +/** Gets information about the cache at the specified cache level. + + @param ProcessorIndex The processor to get information for. + @param CacheLevel The cache level to get information for. + @param DataCache Whether the cache is a data cache. + @param UnifiedCache Whether the cache is a unified cache. + @param SmbiosCacheTable The SMBIOS Type7 cache information structure. + + @return TRUE on success, FALSE on failure. +**/ +BOOLEAN +EFIAPI +OemGetCacheInformation ( + IN UINT8 ProcessorIndex, + IN UINT8 CacheLevel, + IN BOOLEAN DataCache, + IN BOOLEAN UnifiedCache, + IN OUT SMBIOS_TABLE_TYPE7 *SmbiosCacheTable + ) +{ + SmbiosCacheTable->CacheConfiguration =3D CacheLevel - 1; + + if (CacheLevel =3D=3D 1 && !DataCache && !UnifiedCache) { + // Unknown operational mode + SmbiosCacheTable->CacheConfiguration |=3D (3 << 8); + } else { + // Write back operational mode + SmbiosCacheTable->CacheConfiguration |=3D (1 << 8); + } + + return TRUE; +} + +/** Gets the type of chassis for the system. + + @retval The type of the chassis. +**/ +MISC_CHASSIS_TYPE +EFIAPI +OemGetChassisType ( + VOID + ) +{ + return MiscChassisTypeMainServerChassis; +} + +/** Updates the HII string for the specified field. + + @param HiiHandle The HII handle. + @param TokenToUpdate The string to update. + @param Field The field to get information about. +**/ +VOID +EFIAPI +OemUpdateSmbiosInfo ( + IN EFI_HII_HANDLE HiiHandle, + IN EFI_STRING_ID TokenToUpdate, + IN OEM_MISC_SMBIOS_HII_STRING_FIELD Field + ) +{ + CHAR16 *String; + + // These values are fixed for now, but should be configurable via + // something like an emulated SCP. + switch (Field) { + case SystemManufacturerType01: + String =3D (CHAR16*)PcdGetPtr (PcdSystemManufacturer); + break; + case SerialNumType01: + String =3D (CHAR16*)PcdGetPtr (PcdSystemSerialNumber); + break; + case SkuNumberType01: + String =3D (CHAR16*)PcdGetPtr (PcdSystemSKU); + break; + case FamilyType01: + String =3D (CHAR16*)PcdGetPtr (PcdSystemFamily); + break; + case AssertTagType02: + String =3D (CHAR16*)PcdGetPtr (PcdBaseBoardAssetTag); + break; + case SerialNumberType02: + String =3D (CHAR16*)PcdGetPtr (PcdBaseBoardSerialNumber); + break; + case BoardManufacturerType02: + String =3D (CHAR16*)PcdGetPtr (PcdBaseBoardManufacturer); + break; + case SkuNumberType02: + String =3D (CHAR16*)PcdGetPtr (PcdBaseBoardSKU); + break; + case ChassisLocationType02: + String =3D (CHAR16*)PcdGetPtr (PcdBaseBoardLocation); + break; + case SerialNumberType03: + String =3D (CHAR16*)PcdGetPtr (PcdChassisSerialNumber); + break; + case VersionType03: + String =3D (CHAR16*)PcdGetPtr (PcdChassisVersion); + break; + case ManufacturerType03: + String =3D (CHAR16*)PcdGetPtr (PcdChassisManufacturer); + break; + case AssetTagType03: + String =3D (CHAR16*)PcdGetPtr (PcdChassisAssetTag); + break; + case SkuNumberType03: + String =3D (CHAR16*)PcdGetPtr (PcdChassisSKU); + break; + default: + String =3D NULL; + break; + } + + if (String !=3D NULL) { + HiiSetString (HiiHandle, TokenToUpdate, String, NULL); + } +} + diff --git a/Platform/Qemu/SbsaQemu/OemMiscLib/OemMiscLib.inf b/Platform/Qe= mu/SbsaQemu/OemMiscLib/OemMiscLib.inf new file mode 100644 index 000000000000..04a07a55cee9 --- /dev/null +++ b/Platform/Qemu/SbsaQemu/OemMiscLib/OemMiscLib.inf @@ -0,0 +1,53 @@ +#/** @file +# OemMiscLib.inf +# +# Copyright (c) 2021, NUVIA Inc. All rights reserved. +# Copyright (c) 2018, Hisilicon Limited. All rights reserved. +# Copyright (c) 2018, Linaro Limited. All rights reserved. +# +# SPDX-License-Identifier: BSD-2-Clause-Patent +# +#**/ + +[Defines] + INF_VERSION =3D 1.29 + BASE_NAME =3D OemMiscLib + FILE_GUID =3D 958caf90-9e55-4e2a-86e0-71da21485e2c + MODULE_TYPE =3D BASE + VERSION_STRING =3D 1.0 + LIBRARY_CLASS =3D OemMiscLib + +[Sources.common] + OemMiscLib.c + +[Packages] + ArmPkg/ArmPkg.dec + EmbeddedPkg/EmbeddedPkg.dec + MdeModulePkg/MdeModulePkg.dec + MdePkg/MdePkg.dec + Silicon/Qemu/SbsaQemu/SbsaQemu.dec + +[LibraryClasses] + BaseMemoryLib + FdtLib + FdtHelperLib + IoLib + PcdLib + +[Pcd] + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdDeviceTreeBaseAddress + + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdSystemManufacturer + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdSystemSerialNumber + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdSystemSKU + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdSystemFamily + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdBaseBoardAssetTag + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdBaseBoardSerialNumber + gArmTokenSpaceGuid.PcdBaseBoardManufacturer + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdBaseBoardSKU + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdBaseBoardLocation + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdChassisSerialNumber + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdChassisVersion + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdChassisManufacturer + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdChassisAssetTag + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdChassisSKU diff --git a/Platform/Qemu/SbsaQemu/SbsaQemu.dsc b/Platform/Qemu/SbsaQemu/S= bsaQemu.dsc index 8faad3eda217..c1f8a4696560 100644 --- a/Platform/Qemu/SbsaQemu/SbsaQemu.dsc +++ b/Platform/Qemu/SbsaQemu/SbsaQemu.dsc @@ -1,4 +1,5 @@ # +# Copyright (c) 2021, NUVIA Inc. All rights reserved. # Copyright (c) 2019, Linaro Limited. All rights reserved. # # SPDX-License-Identifier: BSD-2-Clause-Patent @@ -122,6 +123,7 @@ DEFINE NETWORK_HTTP_BOOT_ENABLE =3D FALSE PL011UartLib|ArmPlatformPkg/Library/PL011UartLib/PL011UartLib.inf =20 FdtHelperLib|Silicon/Qemu/SbsaQemu/Library/FdtHelperLib/FdtHelperLib.inf + OemMiscLib|Platform/Qemu/SbsaQemu/OemMiscLib/OemMiscLib.inf =20 # Debug Support PeCoffExtraActionLib|ArmPkg/Library/DebugPeCoffExtraActionLib/DebugPeCof= fExtraActionLib.inf @@ -486,6 +488,23 @@ DEFINE NETWORK_HTTP_BOOT_ENABLE =3D FALSE # enumeration to complete before installing ACPI tables. gEfiMdeModulePkgTokenSpaceGuid.PcdPciDisableBusEnumeration|FALSE =20 + gArmTokenSpaceGuid.PcdSystemProductName|L"QEMU SBSA-REF Machine" + gArmTokenSpaceGuid.PcdSystemVersion|L"1.0" + gArmTokenSpaceGuid.PcdBaseBoardManufacturer|L"QEMU" + gArmTokenSpaceGuid.PcdBaseBoardProductName|L"SBSA-REF" + gArmTokenSpaceGuid.PcdBaseBoardVersion|L"1.0" + + # These values are fixed for now, but should be configurable via + # something like an emulated SCP. + gArmTokenSpaceGuid.PcdProcessorManufacturer|L"QEMU" + gArmTokenSpaceGuid.PcdProcessorVersion|L"arm-virt" + gArmTokenSpaceGuid.PcdProcessorSerialNumber|L"SN0000" + gArmTokenSpaceGuid.PcdProcessorAssetTag|L"AT0000" + gArmTokenSpaceGuid.PcdProcessorPartNumber|L"PN0000" + + gEfiMdeModulePkgTokenSpaceGuid.PcdFirmwareVendor|L"EFI Development Kit I= I / SbsaQemu" + gEfiMdeModulePkgTokenSpaceGuid.PcdFirmwareVersionString|L"1.0" + [PcdsDynamicDefault.common] gEfiMdePkgTokenSpaceGuid.PcdPlatformBootTimeOut|3 =20 @@ -510,9 +529,28 @@ DEFINE NETWORK_HTTP_BOOT_ENABLE =3D FALSE # # SMBIOS entry point version # - gEfiMdeModulePkgTokenSpaceGuid.PcdSmbiosVersion|0x0300 + gEfiMdeModulePkgTokenSpaceGuid.PcdSmbiosVersion|0x0304 gEfiMdeModulePkgTokenSpaceGuid.PcdSmbiosDocRev|0x0 =20 + gArmTokenSpaceGuid.PcdSystemBiosRelease|0x0100 + gArmTokenSpaceGuid.PcdEmbeddedControllerFirmwareRelease|0x0100 + + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdSystemManufacturer|L"QEMU" + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdSystemSerialNumber|L"SN0000" + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdSystemSKU|L"SK0000" + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdSystemFamily|L"ArmVirt" + + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdBaseBoardAssetTag|L"AT0000" + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdBaseBoardSerialNumber|L"SN0000" + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdBaseBoardSKU|L"SK000" + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdBaseBoardLocation|L"Internal" + + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdChassisSerialNumber|L"SN0000" + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdChassisVersion|L"1.0" + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdChassisManufacturer|L"QEMU" + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdChassisAssetTag|L"AT0000" + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdChassisSKU|L"SK0000" + ##########################################################################= ###### # # Components Section - list of all EDK II Modules needed by this Platform @@ -670,6 +708,14 @@ DEFINE NETWORK_HTTP_BOOT_ENABLE =3D FALSE MdeModulePkg/Bus/Scsi/ScsiBusDxe/ScsiBusDxe.inf MdeModulePkg/Bus/Scsi/ScsiDiskDxe/ScsiDiskDxe.inf =20 + # + # SMBIOS support + # + ArmPkg/Universal/Smbios/ProcessorSubClassDxe/ProcessorSubClassDxe.inf + ArmPkg/Universal/Smbios/SmbiosMiscDxe/SmbiosMiscDxe.inf + EmbeddedPkg/Library/FdtLib/FdtLib.inf + MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.inf + # # PCI support # diff --git a/Platform/Qemu/SbsaQemu/SbsaQemu.fdf b/Platform/Qemu/SbsaQemu/S= bsaQemu.fdf index 3bcf0bf0040a..c35e3ed44054 100644 --- a/Platform/Qemu/SbsaQemu/SbsaQemu.fdf +++ b/Platform/Qemu/SbsaQemu/SbsaQemu.fdf @@ -236,6 +236,13 @@ READ_LOCK_STATUS =3D TRUE INF RuleOverride =3D ACPITABLE Silicon/Qemu/SbsaQemu/AcpiTables/AcpiTabl= es.inf INF MdeModulePkg/Universal/Acpi/BootGraphicsResourceTableDxe/BootGraphic= sResourceTableDxe.inf =20 + # + # SMBIOS support + # + INF ArmPkg/Universal/Smbios/ProcessorSubClassDxe/ProcessorSubClassDxe.inf + INF ArmPkg/Universal/Smbios/SmbiosMiscDxe/SmbiosMiscDxe.inf + INF MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.inf + # # PCI support # diff --git a/Silicon/Qemu/SbsaQemu/SbsaQemu.dec b/Silicon/Qemu/SbsaQemu/Sbs= aQemu.dec index 476dc82f98f3..3abc9b64e49e 100644 --- a/Silicon/Qemu/SbsaQemu/SbsaQemu.dec +++ b/Silicon/Qemu/SbsaQemu/SbsaQemu.dec @@ -50,3 +50,21 @@ [PcdsDynamic.common] gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdCoreCount|0x1|UINT32|0x00000100 gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdClusterCount|0x1|UINT32|0x0000= 0101 + + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdSystemManufacturer|L""|VOID*|0= x00000110 + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdSystemSerialNumber|L""|VOID*|0= x00000111 + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdSystemSKU|L""|VOID*|0x00000112 + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdSystemFamily|L""|VOID*|0x00000= 113 + + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdBaseBoardAssetTag|L""|VOID*|0x= 00000114 + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdBaseBoardSerialNumber|L""|VOID= *|0x00000115 + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdBaseBoardManufacturer|L""|VOID= *|0x00000116 + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdBaseBoardSKU|L""|VOID*|0x00000= 117 + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdBaseBoardLocation|L""|VOID*|0x= 00000118 + + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdChassisSerialNumber|L""|VOID*|= 0x00000119 + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdChassisVersion|L""|VOID*|0x000= 0011A + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdChassisManufacturer|L""|VOID*|= 0x0000011B + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdChassisAssetTag|L""|VOID*|0x00= 00011C + gArmVirtSbsaQemuPlatformTokenSpaceGuid.PcdChassisSKU|L""|VOID*|0x0000011D + --=20 2.26.2 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#72104): https://edk2.groups.io/g/devel/message/72104 Mute This Topic: https://groups.io/mt/80857023/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- From nobody Mon Apr 29 02:38:27 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) client-ip=66.175.222.108; envelope-from=bounce+27952+72105+1787277+3901457@groups.io; helo=mail02.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+72105+1787277+3901457@groups.io ARC-Seal: i=1; a=rsa-sha256; t=1614102238; cv=none; d=zohomail.com; s=zohoarc; b=hcP2Qi+oJukZouob6fZCEYj/fKDAkOJa/A4CTweCa5CzHPInltfSIDLQaJP7lb2HHnX62aCYzJKrL9zf29JyxWBxHQc+wQWnB5iTbyQw5sk5Mv1/NF0WU7OIlFHvC5gY6POxMLfEMoxAiG6YmZIFMIHPVCoQr6s5feQ/rmyepr0= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1614102238; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Id:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:References:Sender:Subject:To; bh=828KDAT4+D9Hiv/klgyEEEXC9gR3NFahAhipgIg/600=; b=G+Ib1A5MZsZ6ninqLrymO0AxXmqZ939kIh0qWk6MzjH/J6PF11TNpsSTSmByN3Uy5WXWPznYksZFO+yLzNxQSo043HB9sg+r4HLteXq8Kz/bFyY7AnmPxiZAb8mwdPDEJi2jRmNPJjBSkqklwnpRMXly+rswOEtxuwI19gkZ1T4= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+72105+1787277+3901457@groups.io Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 16141022382311023.1701122432173; Tue, 23 Feb 2021 09:43:58 -0800 (PST) Return-Path: X-Received: by 127.0.0.2 with SMTP id PTgBYY1788612xwIN85FEOg1; Tue, 23 Feb 2021 09:43:57 -0800 X-Received: from mail-ot1-f44.google.com (mail-ot1-f44.google.com [209.85.210.44]) by mx.groups.io with SMTP id smtpd.web12.46.1614102232397526800 for ; Tue, 23 Feb 2021 09:43:52 -0800 X-Received: by mail-ot1-f44.google.com with SMTP id b16so16446951otq.1 for ; Tue, 23 Feb 2021 09:43:52 -0800 (PST) X-Gm-Message-State: oJJNdXwNSMPWoosY90z8KaMox1787277AA= X-Google-Smtp-Source: ABdhPJz2LmICx2OtVDh2PQNVTzPaV2LRaGZ7jOxWPSLk42+HpGyiPFp64q1BQPFdE4z49yDw5Nt0Lw== X-Received: by 2002:a9d:784b:: with SMTP id c11mr8554218otm.88.1614102231533; Tue, 23 Feb 2021 09:43:51 -0800 (PST) X-Received: from cube.nuviainc.com (c-174-52-16-57.hsd1.ut.comcast.net. [174.52.16.57]) by smtp.gmail.com with ESMTPSA id s9sm4419918otb.11.2021.02.23.09.43.50 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 23 Feb 2021 09:43:51 -0800 (PST) From: "Rebecca Cran" To: devel@edk2.groups.io Cc: Rebecca Cran , Ard Biesheuvel , Leif Lindholm , Graeme Gregory , Radoslaw Biernacki Subject: [edk2-devel] [edk2-platforms PATCH v5 4/4] Silicon/Qemu: Don't re-use NumCores as loop index in AddMadtTable Date: Tue, 23 Feb 2021 10:43:42 -0700 Message-Id: <20210223174342.8572-5-rebecca@nuviainc.com> In-Reply-To: <20210223174342.8572-1-rebecca@nuviainc.com> References: <20210223174342.8572-1-rebecca@nuviainc.com> MIME-Version: 1.0 Precedence: Bulk List-Unsubscribe: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,rebecca@nuviainc.com Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1614102237; bh=gkwdA7iJjuf4DpjIFwFdisntEarr/qARbEZUfyJHDZM=; h=Cc:Date:From:Reply-To:Subject:To; b=AuC6NScLj2ibvntqU4Jw2eKzF6BTHHdEA2P0t5O7pvU3kDtXPUX6JL3+0ZZbZLweEMg 5frSSbCUxJnzVxzsaO7JC8I8C17JQHrkW35Q+1UM7rX2IfVkVIfy71Xx0MhNLLKf7r7/H R0w+n26+aXWME8Z0rw3guxaubD5xGSL6Sdg= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" Avoid re-using the NumCores variable as a loop index, in AddMadtTable: add a new CoreIndex variable for that purpose. Signed-off-by: Rebecca Cran Reviewed-by: Leif Lindholm --- Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.= c b/Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.c index 037c7cff4c18..84120f1c1b51 100644 --- a/Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.c +++ b/Silicon/Qemu/SbsaQemu/Drivers/SbsaQemuAcpiDxe/SbsaQemuAcpiDxe.c @@ -88,6 +88,7 @@ AddMadtTable ( EFI_PHYSICAL_ADDRESS PageAddress; UINT8 *New; UINT32 NumCores; + UINT32 CoreIndex; =20 // Initialize MADT ACPI Header EFI_ACPI_6_0_MULTIPLE_APIC_DESCRIPTION_TABLE_HEADER Header =3D { @@ -152,13 +153,13 @@ AddMadtTable ( New +=3D sizeof (EFI_ACPI_6_0_MULTIPLE_APIC_DESCRIPTION_TABLE_HEADER); =20 // Add new GICC structures for the Cores - for (NumCores =3D 0; NumCores < PcdGet32 (PcdCoreCount); NumCores++) { + for (CoreIndex =3D 0; CoreIndex < PcdGet32 (PcdCoreCount); CoreIndex++) { EFI_ACPI_6_0_GIC_STRUCTURE *GiccPtr; =20 CopyMem (New, &Gicc, sizeof (EFI_ACPI_6_0_GIC_STRUCTURE)); GiccPtr =3D (EFI_ACPI_6_0_GIC_STRUCTURE *) New; - GiccPtr->AcpiProcessorUid =3D NumCores; - GiccPtr->MPIDR =3D GetMpidr (NumCores); + GiccPtr->AcpiProcessorUid =3D CoreIndex; + GiccPtr->MPIDR =3D GetMpidr (CoreIndex); New +=3D sizeof (EFI_ACPI_6_0_GIC_STRUCTURE); } =20 --=20 2.26.2 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#72105): https://edk2.groups.io/g/devel/message/72105 Mute This Topic: https://groups.io/mt/80857024/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-