From nobody Fri Oct 18 08:39:09 2024 Delivered-To: importer@patchew.org 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+100835+1787277+3901457@groups.io; arc=fail (BodyHash is different from the expected one); dmarc=fail(p=none dis=none) header.from=arm.com Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 1678263420588915.5331734031085; Wed, 8 Mar 2023 00:17:00 -0800 (PST) Return-Path: X-Received: by 127.0.0.2 with SMTP id XKGoYY1788612xTQSxgUkqDk; Wed, 08 Mar 2023 00:17:00 -0800 X-Received: from EUR02-AM0-obe.outbound.protection.outlook.com (EUR02-AM0-obe.outbound.protection.outlook.com [40.107.247.70]) by mx.groups.io with SMTP id smtpd.web11.2265.1678263418731283394 for ; Wed, 08 Mar 2023 00:16:59 -0800 X-Received: from AS9PR04CA0068.eurprd04.prod.outlook.com (2603:10a6:20b:48b::10) by DB5PR08MB10115.eurprd08.prod.outlook.com (2603:10a6:10:4a2::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6178.16; Wed, 8 Mar 2023 08:16:45 +0000 X-Received: from VI1EUR03FT009.eop-EUR03.prod.protection.outlook.com (2603:10a6:20b:48b:cafe::bb) by AS9PR04CA0068.outlook.office365.com (2603:10a6:20b:48b::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6178.17 via Frontend Transport; Wed, 8 Mar 2023 08:16:45 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 63.35.35.123) smtp.mailfrom=arm.com; dkim=pass (signature was verified) header.d=armh.onmicrosoft.com;dmarc=pass action=none header.from=arm.com; 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+100835+1787277+3901457@groups.io; helo=mail02.groups.io; Received-SPF: Pass (protection.outlook.com: domain of arm.com designates 63.35.35.123 as permitted sender) receiver=protection.outlook.com; client-ip=63.35.35.123; helo=64aa7808-outbound-1.mta.getcheckrecipient.com; pr=C X-Received: from 64aa7808-outbound-1.mta.getcheckrecipient.com (63.35.35.123) by VI1EUR03FT009.mail.protection.outlook.com (100.127.144.72) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6156.29 via Frontend Transport; Wed, 8 Mar 2023 08:16:45 +0000 X-Received: ("Tessian outbound 55ffa3012b8f:v135"); Wed, 08 Mar 2023 08:16:44 +0000 X-CheckRecipientChecked: true X-CR-MTA-CID: 2fc43177110942dc X-CR-MTA-TID: 64aa7808 X-Received: from e53e89ee747c.1 by 64aa7808-outbound-1.mta.getcheckrecipient.com id 90D27725-71F9-424C-8CB0-97E71518AB7E.1; Wed, 08 Mar 2023 08:16:37 +0000 X-Received: from EUR05-AM6-obe.outbound.protection.outlook.com by 64aa7808-outbound-1.mta.getcheckrecipient.com with ESMTPS id e53e89ee747c.1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384); Wed, 08 Mar 2023 08:16:37 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=c+gY3zdW5gJhQ/cfiPRr3la/M7NRdhwmGQAJzD8HPHv9S9oEicifpK4cvWpcWC6WItbcAhe2qHwnqDD9Pxjc2Zh/Osj8nJP+jioqdE3vBDIuemzHgZo9u+8CDU3EzYyeHyqDyGyblVrChDBgBLps2bEw/CliZu68zdc4wT5jw84uRFwXtoN6aMbRqtgyPrhNJkEy3gPqQ96zvllLCXNgA25tKC023PNckGNeTPCDY5XYZ7PFTgAF9FLHhbjXDdtYrPxe0GAMHVwQosiYdGKXBQB/2oyEAdXyEGQPfIIr0rbDyWyz4lkptlRewBdbVgei9ZLGQAwXwSKdqzjCK0vpDg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=+yotzY/zBRRFDvn1GJ4t9YKOwB/R8SjxxBw957Ev8JM=; b=ULCOZgkLd890bXKKz2mNdROFoG9aV+jZY9D6DTdJnQRpcvJDFUfBVVm8NiFpArpcC/zcuWZF+98vO7SgsrI3i4uvjVeAxR/RlQ+Sr+BFs8fTQKEQzRFe6sPjlijo4AA7c2A5cvjdI4Xp5/fh/kDIbyYnSAa1xXJD+M4/2SEStkLB0jdkX/i/Mk/oYWtAUnWZhrsVML5StjF4AN1KsSEUsxQElT61z3AcNnz59jVdKaKMrm6WvQPWhTnIcNpfShXt7oTcfsGZ71x7Ahk3pBiOnSqsfRr7LGm/4I2UH9r7FSR3SueIoyMdEDqGwsdsPlcHu0sEYB2KEyTqxtBjnsAeSg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 40.67.248.234) smtp.rcpttodomain=edk2.groups.io smtp.mailfrom=arm.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=arm.com; dkim=none (message not signed); arc=none X-Received: from DB6PR0202CA0036.eurprd02.prod.outlook.com (2603:10a6:4:a5::22) by AS8PR08MB9549.eurprd08.prod.outlook.com (2603:10a6:20b:61e::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6156.28; Wed, 8 Mar 2023 08:16:33 +0000 X-Received: from DBAEUR03FT050.eop-EUR03.prod.protection.outlook.com (2603:10a6:4:a5:cafe::c) by DB6PR0202CA0036.outlook.office365.com (2603:10a6:4:a5::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6156.29 via Frontend Transport; Wed, 8 Mar 2023 08:16:33 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 40.67.248.234) smtp.mailfrom=arm.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=arm.com; Received-SPF: Pass (protection.outlook.com: domain of arm.com designates 40.67.248.234 as permitted sender) receiver=protection.outlook.com; client-ip=40.67.248.234; helo=nebula.arm.com; pr=C X-Received: from nebula.arm.com (40.67.248.234) by DBAEUR03FT050.mail.protection.outlook.com (100.127.142.250) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.6156.29 via Frontend Transport; Wed, 8 Mar 2023 08:16:33 +0000 X-Received: from AZ-NEU-EX04.Arm.com (10.251.24.32) by AZ-NEU-EX03.Arm.com (10.251.24.31) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.17; Wed, 8 Mar 2023 08:16:31 +0000 X-Received: from E114225.Arm.com (10.1.196.36) by mail.arm.com (10.251.24.32) with Microsoft SMTP Server id 15.1.2507.17 via Frontend Transport; Wed, 8 Mar 2023 08:16:31 +0000 From: "Sami Mujawar" To: CC: Sami Mujawar , , , , , , , , , , , , , , Subject: [edk2-devel] [PATCH v2 4/4] DynamicTablesPkg: Add Ordered dispatch support for SMBIOS tables Date: Wed, 8 Mar 2023 08:16:26 +0000 Message-ID: <20230308081626.33124-5-sami.mujawar@arm.com> In-Reply-To: <20230308081626.33124-1-sami.mujawar@arm.com> References: <20230308081626.33124-1-sami.mujawar@arm.com> MIME-Version: 1.0 X-EOPAttributedMessage: 1 X-MS-TrafficTypeDiagnostic: DBAEUR03FT050:EE_|AS8PR08MB9549:EE_|VI1EUR03FT009:EE_|DB5PR08MB10115:EE_ X-MS-Office365-Filtering-Correlation-Id: 97cea429-1f82-44e0-1480-08db1fad74e9 X-LD-Processed: f34e5979-57d9-4aaa-ad4d-b122a662184d,ExtAddr x-checkrecipientrouted: true NoDisclaimer: true X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam-Untrusted: BCL:0; X-Microsoft-Antispam-Message-Info-Original: qI/I/pOV0TebQ06FRWTRZu4bRVsRkcHpOmtXdkPSGVjpqHxEIFiZN8zlFTJmyy8lrD1L14cURdk9gu5eWTmu0bmXWEQiTF+BvvPeJvgDku2lpObBTEhE+PjFJXhxQpoLxUsaoWUhDNoistXPQX3em5MbpvVu03sRPEwdo04fAGREuQk//SGCgevc1s9sQp2D8qOpwTFY58pgM9f4cxXOVEYPSpHZaTPxxhTG2AYQS7Xi6QUgO4Nc353DSMJkGqrDA978fJ91sb2+851dkcOMtUA7pnAI67uquz2uMmPM+2aBNVH/6Qr/V+pL+WfkY1wnZ1y77w/g5F6RTfE+Sz9x54H7BSZrLte9tLqtKQs+laP5kWf4+wk4Bkypr3YZHupUAhz1GZNWdaDS+Lz9ZxwRMvqCCAn6i2lp/C9f1EpO3DjYMFuD1OmX7a8boahjVBtQLDuMU/Na5POHU2Nwl3rkahtmn8zr/3qKNbVdjPofSmfPtBrHr9jKmRjkS2yAS2aXd50Zrzfdg86WwYuo2NCjI1yMhKuFgWjjcq4xnFPH4AnA+qc2oK/Nm7hpH8OUPIqDLO+X0Tsk9wdB77MXQSNnJqNEgPJsgM3+UsvyIWuxILaPbyG7/0jBObfn3MfdRm0rLTzfsj7amCPe9xFpb82TG2NjomqdTaPjkkkE8cS4p4eHOkp8OV3TqM3xyI+codfWp8vfFXo7ZOn2czkKGWeKJ6VHpMsq8WsMeGTa4eeqSuw= X-Forefront-Antispam-Report-Untrusted: CIP:40.67.248.234;CTRY:IE;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:nebula.arm.com;PTR:InfoDomainNonexistent;CAT:NONE;SFS:(13230025)(4636009)(376002)(396003)(136003)(346002)(39860400002)(451199018)(36840700001)(40470700004)(46966006)(30864003)(8936002)(5660300002)(44832011)(70206006)(70586007)(6916009)(2906002)(8676002)(4326008)(316002)(54906003)(478600001)(7696005)(6666004)(36756003)(426003)(47076005)(1076003)(36860700001)(26005)(356005)(41300700001)(2616005)(81166007)(82740400003)(40480700001)(83380400001)(86362001)(82310400005)(40460700003)(186003)(336012)(36900700001)(559001)(579004);DIR:OUT;SFP:1101; X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR08MB9549 X-MS-Exchange-Transport-CrossTenantHeadersStripped: VI1EUR03FT009.eop-EUR03.prod.protection.outlook.com X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id-Prvs: 619ad4dc-a814-45cf-8e45-08db1fad6dec X-Microsoft-Antispam-Message-Info: jbhXm+yzwpELi41fMCdRE1EYgJ09Gdg/LzusNV0xMX10nY0SyrWsbM21rII20AVfPcPLILXewJpk/N4J7ORPhIJlA6AeDLjJOQg2GSB8zE39R4mbD4KGmJKiNfDq4B+O5UJupgUvZjtxiXhPZ25w8/zxkL646JVIDBb6BdmSM4vz4saX3y+LMGwog65oikMJFLWIBkUe8Bi0MuaojeG2yAr5nNYNXtggrDKcK6+WD6mHXuKPTm+7BDi9K8RkVPwjaHS/LhhqiKm71VCIoznc7IWRDTEoA6AqKsyBIa8181CxnnJ1+TXy9LDwSG26KS4QXmb2pmx4NV7rpV3bYzs0B2nlL/YcorIQMcyCrMKdaXzqTCGwT/8KgKIsmRwmP5rfxxLL9+k1bp29EamfGeDSVtmtBkdE9zCBtFb9cR4f2djIIZSXMuzcO63YAYHKeIGQL/KgbBeokNQ63klZdN8Nj0xqsTfx1gCEAfbfA1sE80H9weQTijCP+/AgXeXc1WZTsGBz7IiLB3s8Hf41kOk8/M7LiWo5dUYHgOlOQRGX/Sne/fcPLCM+e3PfVPECZtl4fyuSa0EZs9LOirzOBAd6EeVM8zUAWepy0u7+VnitFH36ey+pUMhEPNGxTkKdWnUj6o59THeaXyYotfXk1ALoWCt/DZWVfCPSFxK27UF/pHJxFU8BdjzerM30cpOAXnGaNsiRrPcNp5ueca4l/tNe9w== X-OriginatorOrg: arm.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Mar 2023 08:16:45.2886 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 97cea429-1f82-44e0-1480-08db1fad74e9 X-MS-Exchange-CrossTenant-Id: f34e5979-57d9-4aaa-ad4d-b122a662184d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=f34e5979-57d9-4aaa-ad4d-b122a662184d;Ip=[63.35.35.123];Helo=[64aa7808-outbound-1.mta.getcheckrecipient.com] X-MS-Exchange-CrossTenant-AuthSource: VI1EUR03FT009.eop-EUR03.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB5PR08MB10115 Precedence: Bulk List-Unsubscribe: List-Subscribe: List-Help: 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,sami.mujawar@arm.com X-Gm-Message-State: KBO1lu10xhG1g40obHn4NDW5x1787277AA= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1678263420; bh=xRZVg7ITf4uB5CfeXGHrniHUCL6dQZq1TcdE5jvKWFo=; h=CC:Content-Type:Date:From:Reply-To:Subject:To; b=RHyNX2XNXE8LszBnW88eJudOTyftsgOfmKiY/nBCgNLA8ToUiE+xCPDUExLJ9fuLDQn 9l/i2AkmTjMcCpyceS8IoHCX92fQ5xE/K1cjAevXxHDlaYAgAayJkkisC/9OiDCaBW3CV jWeOxrhDy4Z7R5QmQtUbaVTN3E+RLTdnwE0= X-ZohoMail-DKIM: pass (identity @groups.io) X-ZM-MESSAGEID: 1678263421280100003 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Some SMBIOS tables do not have a fixed dependency and can depend on any other SMBIOS tables. Therefore, the SMBIOS dispatcher cannot define a fixed sequence for dispatching these tables. A possible solution is to defer the dispatch of such SMBIOS tables towards the end, assuming that the dependent SMBIOS tables would have been dispatched by then. Therefore, introduce a dispatch order attribute such that SMBIOS tables that have a fixed dependency sequence are configured as Default Ordered, and the SMBIOS tables that do not have a fixed dependency have an Order attribute specifying an Order Level. An Order Level is used to sequence the dispatch of Ordered SMBIOS tables. The Default Ordered SMBIOS tables are dispatched first and a dependency walk is performed to dispatch the dependent tables by iterating through the SMBIOS_TABLE_DISPATCHER.Dependency[]. Once all Default ordered SMBIOS tables have been dispatched, the Ordered SMBIOS tables would be scheduled for dispatch in increasing order as of the Order Level, e.g. OrderL1, OrderL2, ... Note: The dispatcher does not perform a dependency walk for the Ordered SMBIOS tables as the expectation is that the dependent SMBIOS tables would be already dispatched. A top level dispatch function DispatchSmbiosTables() has been introduced to schedule the dispatch of Default Ordered and Ordered SMBIOS tables. Signed-off-by: Sami Mujawar Cc: Alexei Fedorov Cc: Pierre Gondois Cc: Girish Mahadevan Cc: Jeff Brasen Cc: Ashish Singhal Cc: Nick Ramirez Cc: William Watson Cc: Abner Chang Cc: Samer El-Haj-Mahmoud Cc: Jose Marinho --- Notes: v2: - New patch in v2 series. [SAMI] DynamicTablesPkg/Drivers/DynamicTableManagerDxe/SmbiosTableDispatcher.c | = 531 +++++++++++++++++--- DynamicTablesPkg/Drivers/DynamicTableManagerDxe/SmbiosTableDispatcher.h | = 34 +- 2 files changed, 487 insertions(+), 78 deletions(-) diff --git a/DynamicTablesPkg/Drivers/DynamicTableManagerDxe/SmbiosTableDis= patcher.c b/DynamicTablesPkg/Drivers/DynamicTableManagerDxe/SmbiosTableDisp= atcher.c index 384a0a9adcd5f11fc73f42b90cfb1c9a6ba2fc73..6da711d7290a868997f7016ac8a= 04e759c2c4441 100644 --- a/DynamicTablesPkg/Drivers/DynamicTableManagerDxe/SmbiosTableDispatcher= .c +++ b/DynamicTablesPkg/Drivers/DynamicTableManagerDxe/SmbiosTableDispatcher= .c @@ -27,56 +27,65 @@ have been satisfied. - no cyclic dependency is allowed. The dependency list is terminated by SMTT_NULL. + + The SMBIOS dispatcher dispatches the tables that have the default + order (OrderDef) set before the ordered SMBIOS tables are dispatched. + The SMBIOS_TABLE_DISPATCHER.Order field is used to establish the + dispatch order. + + The order specified in the SMBIOS dispatcher table must be unique for all + orders other than OrderDef. The dependency walk is only done for tables + that have the default dispatch order. */ STATIC SMBIOS_TABLE_DISPATCHER mSmBiosDispatcher[MAX_SMBIOS_TABLES] =3D { - SMBIOS_TABLE_DEP (SMBIOS_TYPE_BIOS_INFORMATION, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_SYSTEM_INFORMATION, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_BASEBOARD_INFORMATION, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_SYSTEM_ENCLOSURE, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_PROCESSOR_INFORMATION, SMBI= OS_TYPE_CACHE_INFORMATION, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_MEMORY_CONTROLLER_INFORMATION, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_MEMORY_MODULE_INFORMATON, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_CACHE_INFORMATION, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_PORT_CONNECTOR_INFORMATION, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_SYSTEM_SLOTS, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_ONBOARD_DEVICE_INFORMATION, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_OEM_STRINGS, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_SYSTEM_CONFIGURATION_OPTIONS, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_BIOS_LANGUAGE_INFORMATION, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_GROUP_ASSOCIATIONS, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_SYSTEM_EVENT_LOG, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_PHYSICAL_MEMORY_ARRAY, SMBI= OS_TYPE_32BIT_MEMORY_ERROR_INFORMATION, SMBIOS_TYPE_64BIT_MEMORY_ERROR_INFO= RMATION, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_MEMORY_DEVICE, SMBI= OS_TYPE_PHYSICAL_MEMORY_ARRAY, SMBIOS_TYPE_32BIT_MEMORY_ERROR_INFO= RMATION, SMBIOS_TYPE_64BIT_MEMORY_ERROR_INFORMATION, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_32BIT_MEMORY_ERROR_INFORMATION, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_MEMORY_ARRAY_MAPPED_ADDRESS, SMBI= OS_TYPE_PHYSICAL_MEMORY_ARRAY, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_MEMORY_DEVICE_MAPPED_ADDRESS, SMBI= OS_TYPE_MEMORY_DEVICE, SMBIOS_TYPE_MEMORY_ARRAY_MAPPED_ADD= RESS, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_BUILT_IN_POINTING_DEVICE, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_PORTABLE_BATTERY, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_SYSTEM_RESET, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_HARDWARE_SECURITY, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_SYSTEM_POWER_CONTROLS, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_VOLTAGE_PROBE, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_COOLING_DEVICE, SMBI= OS_TYPE_TEMPERATURE_PROBE, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_TEMPERATURE_PROBE, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_ELECTRICAL_CURRENT_PROBE, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_OUT_OF_BAND_REMOTE_ACCESS, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_BOOT_INTEGRITY_SERVICE, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_SYSTEM_BOOT_INFORMATION, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_64BIT_MEMORY_ERROR_INFORMATION, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_MANAGEMENT_DEVICE, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_MANAGEMENT_DEVICE_COMPONENT, SMBI= OS_TYPE_MANAGEMENT_DEVICE, SMBIOS_TYPE_VOLTAGE_PROBE, = SMBIOS_TYPE_COOLING_DEVICE, SMBIOS_TYPE_TEMPERATUR= E_PROBE, SMBIOS_TYPE_ELECTRICAL_CURRENT_PROBE, SMBIOS_TYPE_MANAGEMENT_DEVIC= E_THRESHOLD_DATA), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_MANAGEMENT_DEVICE_THRESHOLD_DATA, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_MEMORY_CHANNEL, SMBI= OS_TYPE_MEMORY_DEVICE, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_IPMI_DEVICE_INFORMATION, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_SYSTEM_POWER_SUPPLY, SMBI= OS_TYPE_VOLTAGE_PROBE, SMBIOS_TYPE_COOLING_DEVICE, = SMBIOS_TYPE_ELECTRICAL_CURRENT_PROBE, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_ADDITIONAL_INFORMATION, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_ONBOARD_DEVICES_EXTENDED_INFORMATION, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_MANAGEMENT_CONTROLLER_HOST_INTERFACE, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_TPM_DEVICE, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_PROCESSOR_ADDITIONAL_INFORMATION, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_FIRMWARE_INVENTORY_INFORMATION, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), - SMBIOS_TABLE_DEP (SMBIOS_TYPE_STRING_PROPERTY_INFORMATION, SMTT= _NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL) + SMBIOS_TABLE_DEP (SMBIOS_TYPE_BIOS_INFORMATION, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_SYSTEM_INFORMATION, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_BASEBOARD_INFORMATION, Orde= rL1, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_SYSTEM_ENCLOSURE, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_PROCESSOR_INFORMATION, Orde= rDef, SMBIOS_TYPE_CACHE_INFORMATION, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_MEMORY_CONTROLLER_INFORMATION, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_MEMORY_MODULE_INFORMATON, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_CACHE_INFORMATION, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_PORT_CONNECTOR_INFORMATION, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_SYSTEM_SLOTS, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_ONBOARD_DEVICE_INFORMATION, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_OEM_STRINGS, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_SYSTEM_CONFIGURATION_OPTIONS, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_BIOS_LANGUAGE_INFORMATION, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_GROUP_ASSOCIATIONS, Orde= rL4, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_SYSTEM_EVENT_LOG, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_PHYSICAL_MEMORY_ARRAY, Orde= rDef, SMBIOS_TYPE_32BIT_MEMORY_ERROR_INFORMATION, SMBIOS_TYPE_64BIT_MEMORY_= ERROR_INFORMATION, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_MEMORY_DEVICE, Orde= rDef, SMBIOS_TYPE_PHYSICAL_MEMORY_ARRAY, SMBIOS_TYPE_32BIT_MEMORY_= ERROR_INFORMATION, SMBIOS_TYPE_64BIT_MEMORY_ERROR_INFORMATION, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_32BIT_MEMORY_ERROR_INFORMATION, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_MEMORY_ARRAY_MAPPED_ADDRESS, Orde= rDef, SMBIOS_TYPE_PHYSICAL_MEMORY_ARRAY, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_MEMORY_DEVICE_MAPPED_ADDRESS, Orde= rDef, SMBIOS_TYPE_MEMORY_DEVICE, SMBIOS_TYPE_MEMORY_ARRAY_= MAPPED_ADDRESS, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_BUILT_IN_POINTING_DEVICE, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_PORTABLE_BATTERY, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_SYSTEM_RESET, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_HARDWARE_SECURITY, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_SYSTEM_POWER_CONTROLS, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_VOLTAGE_PROBE, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_COOLING_DEVICE, Orde= rDef, SMBIOS_TYPE_TEMPERATURE_PROBE, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_TEMPERATURE_PROBE, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_ELECTRICAL_CURRENT_PROBE, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_OUT_OF_BAND_REMOTE_ACCESS, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_BOOT_INTEGRITY_SERVICE, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_SYSTEM_BOOT_INFORMATION, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_64BIT_MEMORY_ERROR_INFORMATION, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_MANAGEMENT_DEVICE, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_MANAGEMENT_DEVICE_COMPONENT, Orde= rDef, SMBIOS_TYPE_MANAGEMENT_DEVICE, SMBIOS_TYPE_VOLTAGE_PROBE= , SMBIOS_TYPE_COOLING_DEVICE, SMBIOS_TYPE_= TEMPERATURE_PROBE, SMBIOS_TYPE_ELECTRICAL_CURRENT_PROBE, SMBIOS_TYPE_MANAGE= MENT_DEVICE_THRESHOLD_DATA), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_MANAGEMENT_DEVICE_THRESHOLD_DATA, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_MEMORY_CHANNEL, Orde= rDef, SMBIOS_TYPE_MEMORY_DEVICE, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_IPMI_DEVICE_INFORMATION, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_SYSTEM_POWER_SUPPLY, Orde= rDef, SMBIOS_TYPE_VOLTAGE_PROBE, SMBIOS_TYPE_COOLING_DEVIC= E, SMBIOS_TYPE_ELECTRICAL_CURRENT_PROBE, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_ADDITIONAL_INFORMATION, Orde= rL3, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_ONBOARD_DEVICES_EXTENDED_INFORMATION, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_MANAGEMENT_CONTROLLER_HOST_INTERFACE, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_TPM_DEVICE, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_PROCESSOR_ADDITIONAL_INFORMATION, Orde= rDef, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_FIRMWARE_INVENTORY_INFORMATION, Orde= rL2, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL), + SMBIOS_TABLE_DEP (SMBIOS_TYPE_STRING_PROPERTY_INFORMATION, Orde= rL5, SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL, = SMTT_NULL, SMTT_NULL) }; =20 #if !defined (MDEPKG_NDEBUG) @@ -159,7 +168,83 @@ InitSmbiosTableDispatcher ( DEBUG_PRINT_DISPATCHER_STATUS (FALSE); } =20 -/** Schedule the dispatch of a SMBIOS table. +/** Dispatch the SMBIOS table. + + @param [in] Disp Pointer to the SMBIOS table dispatcher + object for the table to dispatch. + @param [in] TableFactoryProtocol Pointer to the Table Factory Protocol + interface. + @param [in] CfgMgrProtocol Pointer to the Configuration Manager + Protocol Interface. + @param [in] SmbiosProtocol Pointer to the SMBIOS protocol. + @param [in] SmbiosTableInfo Pointer to the SMBIOS table Info. + @param [in] SmbiosTableCount Count of SMBIOS table info objects. + + @retval EFI_SUCCESS Success. + @retval EFI_INVALID_PARAMETER A parameter is invalid. + @retval EFI_NOT_FOUND Required object is not found. + @retval EFI_BAD_BUFFER_SIZE Size returned by the Configuration Manager + is less than the Object size for the + requested object. +**/ +STATIC +EFI_STATUS +EFIAPI +DispatchTable ( + IN SMBIOS_TABLE_DISPATCHER *CONST Disp, + IN CONST EDKII_DYNAMIC_TABLE_FACTORY_PROTOCOL *CONST TableFactoryProto= col, + IN CONST EDKII_CONFIGURATION_MANAGER_PROTOCOL *CONST CfgMgrProtocol, + IN EFI_SMBIOS_PROTOCOL *SmbiosProtocol, + IN CM_STD_OBJ_SMBIOS_TABLE_INFO *CONST SmbiosTableInfo, + IN CONST UINT32 SmbiosTableCount + ) +{ + UINTN Index; + EFI_STATUS Status; + BOOLEAN Found; + + if (Disp =3D=3D NULL) { + return EFI_INVALID_PARAMETER; + } + + Found =3D FALSE; + + // Update the dospatcher state to dispatched. + Disp->State =3D StDispatched; + // Find the SMBIOS table info matching the TableType + for (Index =3D 0; Index < SmbiosTableCount; Index++) { + if (SmbiosTableInfo[Index].TableType =3D=3D Disp->TableType) { + Found =3D TRUE; + break; + } + } + + if (!Found) { + ASSERT (0); + return EFI_NOT_FOUND; + } + + // Install the SMBIOS table + Status =3D BuildAndInstallSmbiosTable ( + TableFactoryProtocol, + CfgMgrProtocol, + SmbiosProtocol, + &SmbiosTableInfo[Index] + ); + if (EFI_ERROR (Status)) { + DEBUG (( + DEBUG_ERROR, + "ERROR: Failed to install SMBIOS Table." \ + " Id =3D %d Status =3D %r\n", + SmbiosTableInfo[Index].TableGeneratorId, + Status + )); + } + + return Status; +} + +/** Schedule the dispatch of a default ordered SMBIOS table. =20 The SMBIOS dispatcher state table is used to establish the dependency order in which the SMBIOS tables are installed. This allows the SMBIOS @@ -190,10 +275,17 @@ InitSmbiosTableDispatcher ( @retval EFI_BAD_BUFFER_SIZE Size returned by the Configuration Manager is less than the Object size for the requested object. + @retval EFI_NO_MAPPING A dependent table may be optional and + may not be in the list of tables to instal= l. + @retval EFI_ALREADY_STARTED A table may be in the dependency list of + multiple tables and would have been instal= led + when one of the other parent table's depen= dency + list was fulfilled. **/ +STATIC EFI_STATUS EFIAPI -DispatchSmbiosTable ( +DispatchDefaultOrderedSmbiosTable ( IN CONST SMBIOS_TABLE_TYPE TableType, IN CONST EDKII_DYNAMIC_TABLE_FACTORY_PROTOCOL *CONST TableFactoryProto= col, IN CONST EDKII_CONFIGURATION_MANAGER_PROTOCOL *CONST CfgMgrProtocol, @@ -205,28 +297,56 @@ DispatchSmbiosTable ( EFI_STATUS Status; UINTN Index; SMBIOS_TABLE_DISPATCHER *Disp; + SMBIOS_TABLE_TYPE DepTableType; =20 DEBUG ((DEBUG_VERBOSE, "->DP %02d\n", TableType)); Disp =3D &mSmBiosDispatcher[TableType]; + + if (Disp->Order !=3D OrderDef) { + DEBUG ((DEBUG_VERBOSE, "<-DP %d : EFI_INVALID_PARAMETER\n", TableType)= ); + // The table being request for installation is + // not a default ordered table. + return EFI_INVALID_PARAMETER; + } + if (Disp->State =3D=3D StNotPresent) { - DEBUG ((DEBUG_VERBOSE, "<-DP %02d : EFI_NOT_FOUND\n", TableType)); - return EFI_NOT_FOUND; + DEBUG ((DEBUG_VERBOSE, "<-DP %02d : EFI_NO_MAPPING\n", TableType)); + // A dependent table may be optional and therefore may + // not be in the list of tables to install. + return EFI_NO_MAPPING; } =20 if (Disp->State =3D=3D StDispatched) { DEBUG ((DEBUG_VERBOSE, "<-DP %02d : EFI_ALREADY_STARTED\n", TableType)= ); + // This table may be in the dependency list of multiple tables + // and would have been installed when one of the other parent + // table's dependency list was fulfilled. return EFI_ALREADY_STARTED; } =20 // Table is present so check the dependency. for (Index =3D 0; Index < MAX_SMBIOS_DEPENDENCY; Index++) { + DepTableType =3D Disp->Dependency[Index]; // Check if the dependency list is terminated by SMTT_NULL. - if (Disp->Dependency[Index] =3D=3D SMTT_NULL) { + if (DepTableType =3D=3D SMTT_NULL) { break; } =20 - Status =3D DispatchSmbiosTable ( - Disp->Dependency[Index], + if (mSmBiosDispatcher[DepTableType].Order !=3D OrderDef) { + // An incorrect dependency has been set. + // The default ordered SMBIOS tables must not have + // a dependency on ordered SMBIOS tables. + DEBUG (( + DEBUG_VERBOSE, + "<-DP %02d : EFI_INVALID_PARAMETER - Invalid dependency\n", + TableType + )); + ASSERT (0); + return EFI_INVALID_PARAMETER; + } + + Status =3D DispatchDefaultOrderedSmbiosTable ( + DepTableType, TableFactoryProtocol, CfgMgrProtocol, SmbiosProtocol, @@ -234,44 +354,51 @@ DispatchSmbiosTable ( SmbiosTableCount ); if (EFI_ERROR (Status)) { - if ((Status =3D=3D EFI_ALREADY_STARTED) || (Status =3D=3D EFI_NOT_FO= UND)) { + if ((Status =3D=3D EFI_ALREADY_STARTED) || + (Status =3D=3D EFI_NO_MAPPING)) + { // Some dependencies may already be satisfied // as other tables may also have similar // dependencies i.e. EFI_ALREADY_STARTED // Or // the dependent table may be optional - // and not provided i.e. EFI_NOT_FOUND. + // and not provided i.e. EFI_NO_MAPPING. + DEBUG (( + DEBUG_VERBOSE, + "<-DP %02d : Status =3D %r - treated as Success, continue\n", + TableType, + Status + )); + // Therefore, reset Status to success + Status =3D EFI_SUCCESS; continue; } =20 - DEBUG ((DEBUG_VERBOSE, "<-DP %02d : Status =3D %d\n", TableType, Sta= tus)); + DEBUG (( + DEBUG_VERBOSE, + "<-DP %02d : Status =3D %r\n", + TableType, + Status + )); return Status; } } =20 - DEBUG ((DEBUG_VERBOSE, "DP %02d : Status =3D %d\n", TableType, Status)); - - // All dependencies satisfied - Install SMBIOS table - Disp->State =3D StDispatched; - // Find the SMBIOS table info matching the TableType - for (Index =3D 0; Index < SmbiosTableCount; Index++) { - if (SmbiosTableInfo[Index].TableType =3D=3D TableType) { - break; - } - } - - Status =3D BuildAndInstallSmbiosTable ( + // All dependencies satisfied - dispatch SMBIOS table + Status =3D DispatchTable ( + Disp, TableFactoryProtocol, CfgMgrProtocol, SmbiosProtocol, - &SmbiosTableInfo[Index] + SmbiosTableInfo, + SmbiosTableCount ); if (EFI_ERROR (Status)) { DEBUG (( DEBUG_ERROR, "ERROR: Failed to install SMBIOS Table." \ - " Id =3D %u Status =3D %r\n", - SmbiosTableInfo[Index].TableGeneratorId, + " TableType =3D %u Status =3D %r\n", + Disp->TableType, Status )); } @@ -280,3 +407,263 @@ DispatchSmbiosTable ( DEBUG ((DEBUG_VERBOSE, "<-DP %0d\n", TableType)); return Status; } + +/** Schedule the dispatch of ordered SMBIOS tables. + + The SMBIOS dispatcher dispatches the tables that have the default + order (OrderDef) set first before the ordered SMBIOS tables are + dispatched. + The SMBIOS_TABLE_DISPATCHER.Order field is used to establish the + dispatch order. + + @param [in] Order The dispatch order for the SMBIOS tabl= e type + to be scheduled for dispatch. + @param [in] TableFactoryProtocol Pointer to the Table Factory Protocol + interface. + @param [in] CfgMgrProtocol Pointer to the Configuration Manager + Protocol Interface. + @param [in] SmbiosProtocol Pointer to the SMBIOS protocol. + @param [in] SmbiosTableInfo Pointer to the SMBIOS table Info. + @param [in] SmbiosTableCount Count of SMBIOS table info objects. + + @retval EFI_SUCCESS Success. + @retval EFI_INVALID_PARAMETER A parameter is invalid. + @retval EFI_NOT_FOUND Required object is not found. + @retval EFI_BAD_BUFFER_SIZE Size returned by the Configuration Manager + is less than the Object size for the + requested object. +**/ +STATIC +EFI_STATUS +EFIAPI +DispatchOrderedSmbiosTables ( + IN CONST DISPATCH_ORDER Order, + IN CONST EDKII_DYNAMIC_TABLE_FACTORY_PROTOCOL *CONST TableFactoryProto= col, + IN CONST EDKII_CONFIGURATION_MANAGER_PROTOCOL *CONST CfgMgrProtocol, + IN EFI_SMBIOS_PROTOCOL *SmbiosProtocol, + IN CM_STD_OBJ_SMBIOS_TABLE_INFO *CONST SmbiosTableInfo, + IN CONST UINT32 SmbiosTableCount + ) +{ + EFI_STATUS Status; + UINTN Index; + SMBIOS_TABLE_DISPATCHER *Disp; + + DEBUG ((DEBUG_VERBOSE, "->DPO %d\n", Order)); + + if (Order =3D=3D OrderDef) { + DEBUG ((DEBUG_VERBOSE, "<-DPO %d : EFI_INVALID_PARAMETER\n", Order)); + return EFI_INVALID_PARAMETER; + } + + Disp =3D NULL; + for (Index =3D 0; Index < ARRAY_SIZE (mSmBiosDispatcher); Index++) { + if (mSmBiosDispatcher[Index].Order =3D=3D Order) { + Disp =3D &mSmBiosDispatcher[Index]; + break; + } + } // for + + if (Disp =3D=3D NULL) { + // Table with specified order not found. + DEBUG ((DEBUG_VERBOSE, "<-DPO %d : EFI_INVALID_PARAMETER\n", Order)); + ASSERT (0); + return EFI_INVALID_PARAMETER; + } else if (Disp->State =3D=3D StNotPresent) { + // An ordered table of this order is not present + // for installation. So, nothing to do here. + DEBUG (( + DEBUG_VERBOSE, + "<-DPO %a - %d {%u}: EFI_SUCCESS\n", + SmbiosTableStateTxt[Disp->State], + Order, + Disp->TableType + )); + return EFI_SUCCESS; + } else if (Disp->State =3D=3D StDispatched) { + // Ordered tables are dispatched in their dispatch order and + // cannot be in the dependency list of any other table. + // Therefore, the table cannot be already dispatched. + DEBUG (( + DEBUG_VERBOSE, + "<-DPO %a - %d {%u}: EFI_INVALID_PARAMETER\n", + SmbiosTableStateTxt[Disp->State], + Order, + Disp->TableType + )); + + ASSERT (0); + return EFI_INVALID_PARAMETER; + } else if (Disp->State !=3D StPresent) { + // Invalid state, at this point the only valid state + // should be StPresent, otherwise the state machine + // is incorrect. + DEBUG (( + DEBUG_VERBOSE, + "<-DPO %d - %d {%u}: EFI_INVALID_PARAMETER\n", + Disp->State, + Order, + Disp->TableType + )); + ASSERT (0); + return EFI_INVALID_PARAMETER; + } + + Status =3D DispatchTable ( + Disp, + TableFactoryProtocol, + CfgMgrProtocol, + SmbiosProtocol, + SmbiosTableInfo, + SmbiosTableCount + ); + if (EFI_ERROR (Status)) { + DEBUG (( + DEBUG_ERROR, + "ERROR: Failed to install SMBIOS Table." \ + " TableType =3D %u Status =3D %r\n", + Disp->TableType, + Status + )); + } + + DEBUG_PRINT_DISPATCHER_STATUS (FALSE); + DEBUG ((DEBUG_VERBOSE, "<-DPO %d {%u}\n", Order, Disp->TableType)); + return Status; +} + +/** Schedule the dispatch of SMBIOS tables. + + The SMBIOS dispatcher state table is used to establish the dependency + order in which the SMBIOS tables are installed. This allows the SMBIOS + dispatcher to dispatch the dependent tables for installation before the + parent table is installed. + The SMBIOS_TABLE_DISPATCHER.Dependency[] field is used to establish the + dependency list. + Elements in the Dependency list are resolved by increasing index. Howeve= r, + all orders are equivalent as: + - the Parent SMBIOS table will only be installed once all dependencies + have been satisfied. + - no cyclic dependency is allowed. + The dependency list is terminated by SMTT_NULL. + + The SMBIOS dispatcher dispatches the tables that have the default + order (OrderDef) set before the ordered SMBIOS tables are dispatched. + The SMBIOS_TABLE_DISPATCHER.Order field is used to establish the + dispatch order. + + The order specified in the SMBIOS dispatcher table must be unique for all + orders other than OrderDef. The dependency walk is only done for tables + that have the default dispatch order. + + @param [in] TableFactoryProtocol Pointer to the Table Factory Protocol + interface. + @param [in] CfgMgrProtocol Pointer to the Configuration Manager + Protocol Interface. + @param [in] SmbiosProtocol Pointer to the SMBIOS protocol. + @param [in] SmbiosTableInfo Pointer to the SMBIOS table Info. + @param [in] SmbiosTableCount Count of SMBIOS table info objects. + + @retval EFI_SUCCESS Success. + @retval EFI_INVALID_PARAMETER A parameter is invalid. + @retval EFI_NOT_FOUND Required object is not found. + @retval EFI_BAD_BUFFER_SIZE Size returned by the Configuration Manager + is less than the Object size for the + requested object. +**/ +EFI_STATUS +EFIAPI +DispatchSmbiosTables ( + IN CONST EDKII_DYNAMIC_TABLE_FACTORY_PROTOCOL *CONST TableFactoryProto= col, + IN CONST EDKII_CONFIGURATION_MANAGER_PROTOCOL *CONST CfgMgrProtocol, + IN EFI_SMBIOS_PROTOCOL *SmbiosProtocol, + IN CM_STD_OBJ_SMBIOS_TABLE_INFO *CONST SmbiosTableInfo, + IN CONST UINT32 SmbiosTableCount + ) +{ + EFI_STATUS Status; + UINTN Index; + UINT8 Order; + SMBIOS_TABLE_TYPE TableType; + + if ((TableFactoryProtocol =3D=3D NULL) || + (CfgMgrProtocol =3D=3D NULL) || + (SmbiosProtocol =3D=3D NULL) || + (SmbiosTableInfo =3D=3D NULL) || + (SmbiosTableCount =3D=3D 0)) + { + return EFI_INVALID_PARAMETER; + } + + // First dispatch the default ordered SMBIOS tables + for (Index =3D 0; Index < SmbiosTableCount; Index++) { + TableType =3D SmbiosTableInfo[Index].TableType; + if (mSmBiosDispatcher[TableType].Order !=3D OrderDef) { + // Skip if the table is not a default order table. + continue; + } + + Status =3D DispatchDefaultOrderedSmbiosTable ( + TableType, + TableFactoryProtocol, + CfgMgrProtocol, + SmbiosProtocol, + SmbiosTableInfo, + SmbiosTableCount + ); + if (EFI_ERROR (Status)) { + if ((Status =3D=3D EFI_ALREADY_STARTED) || + (Status =3D=3D EFI_NO_MAPPING)) + { + // Some dependencies may already be satisfied + // as other tables may also have similar + // dependencies i.e. EFI_ALREADY_STARTED + // Or + // the dependent table may be optional + // and not provided i.e. EFI_NO_MAPPING. + DEBUG (( + DEBUG_VERBOSE, + "TableType %02d : Status =3D %r - treated as Success, continue\n= ", + SmbiosTableInfo[Index].TableType, + Status + )); + // Therefore, reset Status to success + Status =3D EFI_SUCCESS; + continue; + } + + DEBUG (( + DEBUG_ERROR, + "ERROR: Failed to install SMBIOS Table." \ + " Id =3D %d Status =3D %r\n", + SmbiosTableInfo[Index].TableGeneratorId, + Status + )); + return Status; + } + } + + // Now dispatch the ordered SMBIOS tables + for (Order =3D OrderL1; Order < OrderMax; Order++) { + Status =3D DispatchOrderedSmbiosTables ( + (DISPATCH_ORDER)Order, + TableFactoryProtocol, + CfgMgrProtocol, + SmbiosProtocol, + SmbiosTableInfo, + SmbiosTableCount + ); + if (EFI_ERROR (Status)) { + DEBUG (( + DEBUG_ERROR, + "ERROR: Failed to install SMBIOS Table." \ + " Order =3D %d Status =3D %r\n", + Order, + Status + )); + break; + } + } + + return Status; +} diff --git a/DynamicTablesPkg/Drivers/DynamicTableManagerDxe/SmbiosTableDis= patcher.h b/DynamicTablesPkg/Drivers/DynamicTableManagerDxe/SmbiosTableDisp= atcher.h index e98ac18c7ddc31b8d03a3a83f9bd66ab23278136..471570e57f00392c45d6f478f06= c72ef6bbc57bc 100644 --- a/DynamicTablesPkg/Drivers/DynamicTableManagerDxe/SmbiosTableDispatcher= .h +++ b/DynamicTablesPkg/Drivers/DynamicTableManagerDxe/SmbiosTableDispatcher= .h @@ -43,9 +43,10 @@ /** A helper macro to populate the SMBIOS table dispatcher table */ -#define SMBIOS_TABLE_DEP(TableId, Dep1, Dep2, Dep3, Dep4, Dep5, Dep6) \ +#define SMBIOS_TABLE_DEP(TableId, Order, Dep1, Dep2, Dep3, Dep4, Dep5, Dep= 6) \ { \ TableId, \ + Order, \ StNotPresent, \ { Dep1, Dep2, Dep3, Dep4, Dep5, Dep6 } \ } @@ -59,6 +60,19 @@ typedef enum SmbiosTableState { StDispatched ///< SMBIOS table generators have been dispatched. } SMBIOS_TABLE_STATE; =20 +/** + An enum describing the dispatch order for the SMBIOS tables. +*/ +typedef enum DispatchOrder { + OrderDef =3D 0, ///< Default dispatch order. + OrderL1, ///< Dispatch order 1. + OrderL2, ///< Dispatch order 2. + OrderL3, ///< Dispatch order 3. + OrderL4, ///< Dispatch order 4. + OrderL5, ///< Dispatch order 5. + OrderMax ///< Dispatch order Max. +} DISPATCH_ORDER; + /** A structure describing the dependencies for a SMBIOS table and the dispatcher state information. @@ -66,6 +80,8 @@ typedef enum SmbiosTableState { typedef struct SmBiosTableDispatcher { /// SMBIOS Structure/Table Type SMBIOS_TABLE_TYPE TableType; + /// SMBIOS table dispatch order + DISPATCH_ORDER Order; /// SMBIOS dispatcher state SMBIOS_TABLE_STATE State; /// SMBIOS Structure/Table dependency list @@ -113,7 +129,7 @@ InitSmbiosTableDispatcher ( IN UINT32 SmbiosTableCount ); =20 -/** Schedule the dispatch of a SMBIOS table. +/** Schedule the dispatch of SMBIOS tables. =20 The SMBIOS dispatcher state table is used to establish the dependency order in which the SMBIOS tables are installed. This allows the SMBIOS @@ -128,8 +144,15 @@ InitSmbiosTableDispatcher ( - no cyclic dependency is allowed. The dependency list is terminated by SMTT_NULL. =20 - @param [in] TableType The SMBIOS table type to schedule for - dispatch. + The SMBIOS dispatcher dispatches the tables that have the default + order (OrderDef) set before the ordered SMBIOS tables are dispatched. + The SMBIOS_TABLE_DISPATCHER.Order field is used to establish the + dispatch order. + + The order specified in the SMBIOS dispatcher table must be unique for all + orders other than OrderDef. The dependency walk is only done for tables + that have the default dispatch order. + @param [in] TableFactoryProtocol Pointer to the Table Factory Protocol interface. @param [in] CfgMgrProtocol Pointer to the Configuration Manager @@ -147,8 +170,7 @@ InitSmbiosTableDispatcher ( **/ EFI_STATUS EFIAPI -DispatchSmbiosTable ( - IN CONST SMBIOS_TABLE_TYPE TableType, +DispatchSmbiosTables ( IN CONST EDKII_DYNAMIC_TABLE_FACTORY_PROTOCOL *CONST TableFactoryProto= col, IN CONST EDKII_CONFIGURATION_MANAGER_PROTOCOL *CONST CfgMgrProtocol, IN EFI_SMBIOS_PROTOCOL *SmbiosProtocol, --=20 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)' -=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 (#100835): https://edk2.groups.io/g/devel/message/100835 Mute This Topic: https://groups.io/mt/97468482/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-