From nobody Sat Apr 20 01:11:40 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) client-ip=66.175.222.12; envelope-from=bounce+27952+61557+1787277+3901457@groups.io; helo=web01.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+61557+1787277+3901457@groups.io; arc=fail (BodyHash is different from the expected one) Received: from web01.groups.io (web01.groups.io [66.175.222.12]) by mx.zohomail.com with SMTPS id 1592832254238862.3525075871132; Mon, 22 Jun 2020 06:24:14 -0700 (PDT) Return-Path: X-Received: by 127.0.0.2 with SMTP id j3tvYY1788612xFYutvhG62k; Mon, 22 Jun 2020 06:24:13 -0700 X-Received: from NAM12-MW2-obe.outbound.protection.outlook.com (NAM12-MW2-obe.outbound.protection.outlook.com []) by mx.groups.io with SMTP id smtpd.web11.17266.1592831922292673781 for ; Mon, 22 Jun 2020 06:18:42 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=husoU1WnsBj47M+49zNyebhJeFZyxxbjAoDB0fgl0Zh69K/CU6hlwe3fP2fdP+DBOXPaYkVhsYaoHxfz7ASl8qim8q+Zc2VVNxNXf7z1z0PAP1kJ7VwDfb2aAPIwuoayiLOGZNhu3gnjd6FOvRZNPAWPXsz0j6/dqiHqEfDGwcS2XZBJVO8wBRZVonCAN35hsX1gmuq2Fl+E83D+C5uzuJp3hjAOEKu4/CghSC6b/SXxCfYxWe/1trc9fan2oLWNgQYxDLnWVSPKK7pRFPiUHI8V/1Chvb3YAJ2Nj7dY8W6KLG7J74SGGi0N3WnskTgT++wVj87LzOx7SWO8+veR8w== 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-SenderADCheck; bh=5zadp6D4fFna07Yo9EkIV6MjggcSugRpOdXSqk1hzd0=; b=NMTzHjaduxXLv3TRpiLC+xG+0IpimP4gDesfDvG8ZKN/zHnjSnP5uFxa/1qd1Rj1q2C0OW/t1Fa+/vGxNNr/YP5bdG0PN0CqYQOfq8eQJnyluT+KLlMRvpm3bptK31pb4eqtuWRzF6AK8obw5zalybQ3nSqV3nUMt3RKWCY2BlBi6TW6eYQMwRTY7Gg+yZFdlkSt4j0asrQsPMuFzJji15Na6v2QDeGHy7QJxjkNwBDFbc5lO5VzstSY9WbYSh8zCqv9dRa/Yk4rn83CBnh9CXXE9Rv6f1iqa3oI82QB+oEpX2kB8tfFyOYHLkUAUALaYkjHrjqZ8yV5hy0ydUH7/A== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=amd.com; dmarc=pass action=none header.from=amd.com; dkim=pass header.d=amd.com; arc=none X-Received: from SN1PR12MB2352.namprd12.prod.outlook.com (2603:10b6:802:25::13) by SA0PR12MB4461.namprd12.prod.outlook.com (2603:10b6:806:9c::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3109.22; Mon, 22 Jun 2020 13:18:41 +0000 X-Received: from SN1PR12MB2352.namprd12.prod.outlook.com ([fe80::156e:ce6d:a148:274e]) by SN1PR12MB2352.namprd12.prod.outlook.com ([fe80::156e:ce6d:a148:274e%7]) with mapi id 15.20.3109.027; Mon, 22 Jun 2020 13:18:41 +0000 From: "Kirkendall, Garrett" To: devel@edk2.groups.io CC: Ray Ni Subject: [edk2-devel] [PATCH v6 1/4] PcAtChipsetPkg: PcAtChipsetPkg.dsc add UefiCpuLib LibraryClass Date: Mon, 22 Jun 2020 08:18:22 -0500 Message-ID: <20200622131825.1352-2-Garrett.Kirkendall@amd.com> In-Reply-To: <20200622131825.1352-1-Garrett.Kirkendall@amd.com> References: <20200622131825.1352-1-Garrett.Kirkendall@amd.com> X-ClientProxiedBy: SN4PR0501CA0135.namprd05.prod.outlook.com (2603:10b6:803:2c::13) To SN1PR12MB2352.namprd12.prod.outlook.com (2603:10b6:802:25::13) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-Received: from biosdev-01.amd.com (165.204.78.2) by SN4PR0501CA0135.namprd05.prod.outlook.com (2603:10b6:803:2c::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3131.13 via Frontend Transport; Mon, 22 Jun 2020 13:18:40 +0000 X-Originating-IP: [165.204.78.2] X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-HT: Tenant X-MS-Office365-Filtering-Correlation-Id: 7e91555c-65aa-4c3a-2a61-08d816aec7ac X-MS-TrafficTypeDiagnostic: SA0PR12MB4461: X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:1923; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam-Message-Info: 5NeTL178WDQ94Se6nl/mUXAs/oVGMpgTFPk/2ZNXLPNBjd/ir6Vplv3Lw2KRV7c+5NGTbmT9zuhuFOrLR0IhUc5UD11z8FIkAqLPMFwOd+t49R+aIJZK7AKocuk8M2pWgsbrtEnymiH17Xb2mICzuKZ+DT1II/wwCkoCJRHf8dOWXKekW8URqBTtq0EEY8RYpDo7uus6EcunjCuO5e2jmgmp8Oo2oZAGFKsA6Dr3fC43mlNs/mrv3kDz9M1Bd3KdDo81/JCdFrAx/Gy5bznHb02IASkwQKQWO3M44iwkHNTpND9Ie8cIh80Xz4yOd1fg X-MS-Exchange-AntiSpam-MessageData: zH0HuIG1J1VT7Onqh+0zKy4pWSeRyrHKCCI3ZVo0kNSfXTFn1iokQyMQpPhd+qjaHDruxRXoO49erjR0hJnq588rMaoHoJ6v0D7RSyHUD3nyRu1n2hGObr7KkgoKqnbBdisfX9nwjWpk/y/YBgHxDFFJA3wqp5j7Ntizl6KENhSuyGAqtKs8iWbX5ju5Kh65GPkRevGMmdUSRze953N+z7twjD8sjUL+cPb/e6zFlGooGCh/owyh53ckKGBHxtXXMRZeGmsMRRfh1VXg9FECTUc0uA49dkpcdac9rQ43esCyZDnO7W4kSdmXhqtYIWryDTX90UCXkrmEKpjz+uT5NvFE8NjJfykwzhd0+zupyFONg6UYy9lUBkDHBYA0R3lXSSah59I//Tin1czHGC8DYB0qZvNtNWR7m9mK5r6e71MFZZJQPthjoeuEQ6uL4giSZSahPblDdTITchtMfZAMY8f/3fu+8paICGZDTLwQ9J4= X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-Network-Message-Id: 7e91555c-65aa-4c3a-2a61-08d816aec7ac X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Jun 2020 13:18:40.5383 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: HYQiDwzWu5Ut7WS/KyHvr6VLIx/oEqK28EBTqxIQnJIbAPScOGfAQS6ISWfg7BbQo5sBWQizo2uqxEvPDezodA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA0PR12MB4461 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,garrett.kirkendall@amd.com X-Gm-Message-State: 4ykZTfwt26WAejHtnwqpwOA5x1787277AA= Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1592832253; bh=9KBhckWQZICVrwgyTb5Sw7BF7k3lf2VlPfnMWJJ9acg=; h=CC:Content-Type:Date:From:Reply-To:Subject:To; b=iPbLhNdIqdTaTi6xj0TDxOBeuviRg0CQ/pzYxVEzVBnN2qpyRaj6C0RUJJk4MweASxr IGAEHjzdNsYevinGdguo/L/BCRb1HwFKtx3bG15UmnuHxL1z1Ughxdy+CCVkuMt1KvpxF euE3q+fPdkAvJiNQC+P52kPILDzOEpUfetQ= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" In preparation for moving StandardSignatureIsAuthenticAMD to UefiCpuLib in UefiCpuPkg, PcAtChipset/PcAtChipsetPkg.dsc needs LibraryClass UefiCpuLib. LocalApicLib|UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.inf will need UefiCpuLib LibraryClass. Likely most "real" platforms will be using BaseX2XApicLib instance which already required UefiCpuLib. Cc: Ray Ni Signed-off-by: Garrett Kirkendall Reviewed-by: Chasel Chiu Reviewed-by: Ray Ni --- PcAtChipsetPkg/PcAtChipsetPkg.dsc | 2 ++ 1 file changed, 2 insertions(+) diff --git a/PcAtChipsetPkg/PcAtChipsetPkg.dsc b/PcAtChipsetPkg/PcAtChipset= Pkg.dsc index 01a3ee716a98..b61b7d1f528e 100644 --- a/PcAtChipsetPkg/PcAtChipsetPkg.dsc +++ b/PcAtChipsetPkg/PcAtChipsetPkg.dsc @@ -2,6 +2,7 @@ # PC/AT Chipset Package # # Copyright (c) 2007 - 2019, Intel Corporation. All rights reserved.
+# Copyright (c) 2020, AMD Incorporated. All rights reserved.
# # SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -39,6 +40,7 @@ [LibraryClasses] ResetSystemLib|PcAtChipsetPkg/Library/ResetSystemLib/ResetSystemLib.inf IoApicLib|PcAtChipsetPkg/Library/BaseIoApicLib/BaseIoApicLib.inf LocalApicLib|UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.inf + UefiCpuLib|UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLib.inf ReportStatusCodeLib|MdePkg/Library/BaseReportStatusCodeLibNull/BaseRepor= tStatusCodeLibNull.inf HobLib|MdePkg/Library/DxeHobLib/DxeHobLib.inf =20 --=20 2.27.0 -=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 (#61557): https://edk2.groups.io/g/devel/message/61557 Mute This Topic: https://groups.io/mt/75037835/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 Sat Apr 20 01:11:40 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) client-ip=66.175.222.12; envelope-from=bounce+27952+61558+1787277+3901457@groups.io; helo=web01.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+61558+1787277+3901457@groups.io; arc=fail (BodyHash is different from the expected one) Received: from web01.groups.io (web01.groups.io [66.175.222.12]) by mx.zohomail.com with SMTPS id 1592831929342384.21608247451945; Mon, 22 Jun 2020 06:18:49 -0700 (PDT) Return-Path: X-Received: by 127.0.0.2 with SMTP id YbiIYY1788612xPSaTrddMm8; Mon, 22 Jun 2020 06:18:49 -0700 X-Received: from NAM12-MW2-obe.outbound.protection.outlook.com (NAM12-MW2-obe.outbound.protection.outlook.com []) by mx.groups.io with SMTP id smtpd.web11.17266.1592831922292673781 for ; Mon, 22 Jun 2020 06:18:44 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=NhgcK/zsyyHpTBVKxcOeVKgOy+kK636TPRL6msPTOnz3NbdyFtgx4uIqRl9UjMOgLZ4LIz/ppC4hMWHzOwGRNnSbxBdEhYDQX9wRFc7AT4Nvx13lWrvbnxk1tLfDy5qTrB1vpQAs9HYOTyreZj0hRSt9BgTi9ox7c71tVkibGkdNmVGfP1YoEcSZTD5jvy57jf9EE3RrSaIgpPpo9y7ZHY9yx+R5jhhdh2M6NAn8xXpRtp99rx7vrGVRKfc0oS4/iV9Ku9khkXcVbMZqvQHIbw3T8JGiLaA5PprlGLthWUbS7H8OB1lLbHHQcpCBs6ROpsnNMqyP5b52wxqE6uO/ww== 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-SenderADCheck; bh=ijBw3UK1wE5tPlIrO2QWtEgBG9FkEHsr0UyECAbqod8=; b=X7A+LRguQ+pAIhMUqPuJ8IT/QtgubMwaERQX3sMQ+YWv/3hOXK/BA/PEKsv0Y2YUeEvZpCNP+hNg88byLeH8TbZTFLM9pYUb2jbkwr1WxJOMNi7J0DpJJ14P9jTYLkWpxIAcJVuqZQTgISvVNdY0YwFNFc5QVf/9l0hzLmUktFe3N+xzBfOk/H0mtsM+9uAf99BJ6jBCKcPGRKu3TuthLbhR6VvrCyvdZZobR+vS54dNP6JUjbAnvHYPkP1tj5WcjRCX2cJgw7g9kC/vmAwhp8AXktAjXM+lvJTyOBll9jf0priqB1pWgTydvXzdpwpKrOlCfVg215JmubPutG90Sg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=amd.com; dmarc=pass action=none header.from=amd.com; dkim=pass header.d=amd.com; arc=none X-Received: from SN1PR12MB2352.namprd12.prod.outlook.com (2603:10b6:802:25::13) by SA0PR12MB4461.namprd12.prod.outlook.com (2603:10b6:806:9c::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3109.22; Mon, 22 Jun 2020 13:18:41 +0000 X-Received: from SN1PR12MB2352.namprd12.prod.outlook.com ([fe80::156e:ce6d:a148:274e]) by SN1PR12MB2352.namprd12.prod.outlook.com ([fe80::156e:ce6d:a148:274e%7]) with mapi id 15.20.3109.027; Mon, 22 Jun 2020 13:18:41 +0000 From: "Kirkendall, Garrett" To: devel@edk2.groups.io CC: Hao A Wu Subject: [edk2-devel] [PATCH v6 2/4] SourceLevelDebugPkg: SourceLevelDebugPkg.dsc add UefiCpuLib LibraryClass Date: Mon, 22 Jun 2020 08:18:23 -0500 Message-ID: <20200622131825.1352-3-Garrett.Kirkendall@amd.com> In-Reply-To: <20200622131825.1352-1-Garrett.Kirkendall@amd.com> References: <20200622131825.1352-1-Garrett.Kirkendall@amd.com> X-ClientProxiedBy: SN4PR0501CA0135.namprd05.prod.outlook.com (2603:10b6:803:2c::13) To SN1PR12MB2352.namprd12.prod.outlook.com (2603:10b6:802:25::13) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-Received: from biosdev-01.amd.com (165.204.78.2) by SN4PR0501CA0135.namprd05.prod.outlook.com (2603:10b6:803:2c::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3131.13 via Frontend Transport; Mon, 22 Jun 2020 13:18:40 +0000 X-Originating-IP: [165.204.78.2] X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-HT: Tenant X-MS-Office365-Filtering-Correlation-Id: 977e26a7-386f-4c8b-0f7e-08d816aec7f5 X-MS-TrafficTypeDiagnostic: SA0PR12MB4461: X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:660; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam-Message-Info: fHcRdw7BUSPNT7YWCqkB2D09JIW22mToXN7q2i/AL81JR1uCJCTVRbnlSlQNLNLWTlZ8lZOHkrifVw+vVGmkUl2bO/zzDqGbmebQT4m7V3j6lwjPkVlAVmNCqKSCipT42b8HE8vuuwRkY/YhVd4L50QicWq94UJpm2///yDAx/4A2f02/ImKTpmkmpltWXx7XrBGNTj7UH/cxadALwnySCzAfEHhVdPESOUjbzU2p0yW8aE5UOuEAafEwVop/cGOvdRpJNP+fLbbSNa9yj68aFZYRrwVhUkvGKgebFrYpHQqKBltYkmBWeHE3vLuP0La X-MS-Exchange-AntiSpam-MessageData: JjiprXnu+Q5bhVWS7ZMMPsPOoduQ7JDblh5t8IQvsy1ebOYYAiUblUL6ydAiHFjZhuvVk0V//LjwOh6RllsvIxRhSZI8duXnM/9TZSCwCW8oIxDF+gB8fHSrLPzJ2cKIu3M1E9m84sTqETS6W1a7XqZL5xNn2f/vmAqL4XyA8Bv9s0zBJl4Jj8CsPIFXP0yOxf7mhuoWzZmFaOeLqP4AjZZBvFNGSQ496g2blUIP+75Xa/wHQOPhDpLrwKuvbtrm/lZgDhRY6HKJ8viyXVidV69/b3i3x4XX4jDvpf4AqBsR3VuHfSGMa15eiItnQFnu9U20TF1ciXpcT3RT0DT0s70R1iVSwE82Ln8cJXoX2jZJ4tErsVhWbEKuaiSr81nCVZy+r+CpW8T0aoYdYZEfGPjdpfx6stjO60p18WVqQpsIkSeciHxA1qRYgVUrDA/53z3YQT2xhc227jtTP4M3HsKNtZiQmg4B3qd+ydHAApo= X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-Network-Message-Id: 977e26a7-386f-4c8b-0f7e-08d816aec7f5 X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Jun 2020 13:18:41.0390 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: k41surUAaPDizMPsm+r57bSbeWIgFNarvCYelmrQd4dv6eF5OUJyNA/eTTYkWoL1dFHW7ycoQsYpz9aigyWH2A== X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA0PR12MB4461 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,garrett.kirkendall@amd.com X-Gm-Message-State: znUBmHz52tvv0wfAkhpLYeZRx1787277AA= Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1592831929; bh=03KaE1RIj5dH6+MY1HAHt6k6kb/3/Te1vqXLvv+0Exg=; h=CC:Content-Type:Date:From:Reply-To:Subject:To; b=SSkh559BR9zDBQz3Q7ppsDUyc5usk91Yp/TuLISIdtx5O3qE02CG6I0B3+5Wx87Iog8 Fqz03OpvZ4n8ncJPOR5QhMSaocn7bS+4qf9zej9QIuPFOv7zNY6wPRrgUPr6RtaZ06zCu yYAGvQSH/8/ruZ2q49L5BTwQGBClvMRXLGU= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" In preparation for moving StandardSignatureIsAuthenticAMD to UefiCpuLib in UefiCpuPkg, SourceLevelDebugPkg/SourceLevelDebugPkg.dsc needs LibraryClass UefiCpuLib. LocalApicLib|UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.inf will need UefiCpuLib LibraryClass. Likely most "real" platforms will be using BaseX2XApicLib instance which already required UefiCpuLib. Cc: Hao A Wu Signed-off-by: Garrett Kirkendall Reviewed-by: Chasel Chiu Reviewed-by: Hao A Wu --- SourceLevelDebugPkg/SourceLevelDebugPkg.dsc | 2 ++ 1 file changed, 2 insertions(+) diff --git a/SourceLevelDebugPkg/SourceLevelDebugPkg.dsc b/SourceLevelDebug= Pkg/SourceLevelDebugPkg.dsc index a1a1b81d03cb..20eb10ba07f8 100644 --- a/SourceLevelDebugPkg/SourceLevelDebugPkg.dsc +++ b/SourceLevelDebugPkg/SourceLevelDebugPkg.dsc @@ -2,6 +2,7 @@ # Source Level Debug Package. # # Copyright (c) 2010 - 2018, Intel Corporation. All rights reserved.
+# Copyright (c) 2020, AMD Incorporated. All rights reserved.
# # SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -32,6 +33,7 @@ [LibraryClasses.common] IoLib|MdePkg/Library/BaseIoLibIntrinsic/BaseIoLibIntrinsic.inf SynchronizationLib|MdePkg/Library/BaseSynchronizationLib/BaseSynchroniza= tionLib.inf LocalApicLib|UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.inf + UefiCpuLib|UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLib.inf PeCoffGetEntryPointLib|MdePkg/Library/BasePeCoffGetEntryPointLib/BasePeC= offGetEntryPointLib.inf SerialPortLib|MdeModulePkg/Library/BaseSerialPortLib16550/BaseSerialPort= Lib16550.inf PeCoffExtraActionLib|SourceLevelDebugPkg/Library/PeCoffExtraActionLibDeb= ug/PeCoffExtraActionLibDebug.inf --=20 2.27.0 -=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 (#61558): https://edk2.groups.io/g/devel/message/61558 Mute This Topic: https://groups.io/mt/75037837/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 Sat Apr 20 01:11:40 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) client-ip=66.175.222.12; envelope-from=bounce+27952+61559+1787277+3901457@groups.io; helo=web01.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+61559+1787277+3901457@groups.io; arc=fail (BodyHash is different from the expected one) Received: from web01.groups.io (web01.groups.io [66.175.222.12]) by mx.zohomail.com with SMTPS id 1592831929362280.67899699447446; Mon, 22 Jun 2020 06:18:49 -0700 (PDT) Return-Path: X-Received: by 127.0.0.2 with SMTP id 4JBxYY1788612xk3VPjrXm3W; Mon, 22 Jun 2020 06:18:49 -0700 X-Received: from NAM12-MW2-obe.outbound.protection.outlook.com (NAM12-MW2-obe.outbound.protection.outlook.com []) by mx.groups.io with SMTP id smtpd.web11.17266.1592831922292673781 for ; Mon, 22 Jun 2020 06:18:45 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Ze0dxE981Ej5UOVgNDEkQW78scB889Jv7WtBpDFwOAxdOihZG+UUd7de9/JbSDt9uiJNb+PFrT2pWxdJTFsfbzFewAowM4yLydofzA3dWO5XrSXi+RHJNg0DWo9tjnyUzzzrJXXXBBwww0c/Lpt5g7eS97Lqdw6AW1lPOjgg+9vZUNR+BJfnL5uR30jFdlvF/JG/nAkdF8oncNdKGYJhcpja9FOfg3lG3DRxeTem0L1sm5gKBlsurbOcNIEzzyNZZf05j8VnBWnkN0X2vfw0ERC4ztDTa2lY/HEsNYhlOjSD60IQd37+WYs95VWR1Vzda+PXyshiB8y6Y8w6Ed9r7Q== 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-SenderADCheck; bh=NNt6/W1/cB87/OwHFMIVu/JLjMGAFejoFgyfPe/N/Ng=; b=mcfl5PGqJ2DLuAo+NRoQz+UgGen6PbaGagLw0MwoTf+oosAn1zjbZBnTSvHUw8cTTgyb7axoQGjGow0puccA44Ine93m5xK3OHUPBjJamxfridBHQiy1DzLFjxW0ej2ZMmyN908YZ/+49vTfCOHvGr8WzFTuqpC/Mr7vCLcj8vVXVk54FDkjZXJ+peugSYkjFnTifRk7nhxJbehYQsLabY7oVu7XJS9be+Y1k68FuAJVBFYhzcD7Hu/iFE+xVa1RtPQ1lf2aHLo2LJ4Wu94Ks3JWFHjmz7ZbdwQjMLyCu2YJ5xmiVf1h5FunkyUPQAx+8ve6rxSUd/Sbr8NEOt2pHg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=amd.com; dmarc=pass action=none header.from=amd.com; dkim=pass header.d=amd.com; arc=none X-Received: from SN1PR12MB2352.namprd12.prod.outlook.com (2603:10b6:802:25::13) by SA0PR12MB4461.namprd12.prod.outlook.com (2603:10b6:806:9c::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3109.22; Mon, 22 Jun 2020 13:18:41 +0000 X-Received: from SN1PR12MB2352.namprd12.prod.outlook.com ([fe80::156e:ce6d:a148:274e]) by SN1PR12MB2352.namprd12.prod.outlook.com ([fe80::156e:ce6d:a148:274e%7]) with mapi id 15.20.3109.027; Mon, 22 Jun 2020 13:18:41 +0000 From: "Kirkendall, Garrett" To: devel@edk2.groups.io CC: Eric Dong , Ray Ni , Laszlo Ersek Subject: [edk2-devel] [PATCH v6 3/4] UefiCpuPkg: Move StandardSignatureIsAuthenticAMD to BaseUefiCpuLib Date: Mon, 22 Jun 2020 08:18:24 -0500 Message-ID: <20200622131825.1352-4-Garrett.Kirkendall@amd.com> In-Reply-To: <20200622131825.1352-1-Garrett.Kirkendall@amd.com> References: <20200622131825.1352-1-Garrett.Kirkendall@amd.com> X-ClientProxiedBy: SN4PR0501CA0135.namprd05.prod.outlook.com (2603:10b6:803:2c::13) To SN1PR12MB2352.namprd12.prod.outlook.com (2603:10b6:802:25::13) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-Received: from biosdev-01.amd.com (165.204.78.2) by SN4PR0501CA0135.namprd05.prod.outlook.com (2603:10b6:803:2c::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3131.13 via Frontend Transport; Mon, 22 Jun 2020 13:18:41 +0000 X-Originating-IP: [165.204.78.2] X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-HT: Tenant X-MS-Office365-Filtering-Correlation-Id: b6b664cd-4b0e-4241-75e6-08d816aec84a X-MS-TrafficTypeDiagnostic: SA0PR12MB4461: X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:8273; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam-Message-Info: 3ONJ3tf7GwTEle/9maLucXJN9fLCvGwGk8Us7TLDZrs+T5GxncAxsg5T5MF0MMxFWG6obLPA7wR/oxh6Ld6pT3d5gQQNlmhJy2NWLRt9ZfNBKazJs2Mdh3rVcyfrnnfWR4L7u2Ykb08u58MT3uHU/AGzDwi/g31n8i3otofqiUyMyBYfmRGpukWiaXQ57P6Gqw1vQFrobaXTEacrMb5EEZt0sJZFHZKOMCZdVUjx6at+XhHtlQExP9BJ7HcdbWos+5hrnbE022TqMjHDW6SK1EHsAGDHm7deQXVAtozCbTDMvnUy0wcjLX8uAQUlo3kW3p84qbfOK//9ihYD+c5xbw== X-MS-Exchange-AntiSpam-MessageData: JVwyFu1PaGRq9mA9igUiK74rg6zL51OK1wdufndA/7M/KTPrscaqu17ps1Gh2jV73A9fj/9hZ+WWSI/aPTgj0r54sauF75KYwGqJ/QGOnRWB7x/Uf39IZxvUXFlN5+5Eic2QZJ4cNlArUIkc2RSWAMFp0PbjbqL7ua70mx2jMHq0l33plbMUEGfcM8QxtHTZwH4KFkj8g43e44CrW7HWnjIeXZyRi4Y30jDrQQlPZM1qQ5LJpRlNy7FVptXq9NCQyCG+j7q0bCuYPt2UJntcMnsWl/pJRp7exsTX9C6LqZAvjOkO3oZm6sj/JTjRb3UGhtLZ6BeaEk4B7Ubd2l/gQzpu64K3RGRwdbA/3wJrHVerg5GjSK857rYySFefAf7ZlWcxR05hJ8GEoGxAU0UG6QM979OUl9TeurgUYalDwwHB5nfEkcTLCAKUBNjgVm7ZbhgOGqRg94EjBS/lP2dDjlvC438sBT4PplnnawZs9ss= X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-Network-Message-Id: b6b664cd-4b0e-4241-75e6-08d816aec84a X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Jun 2020 13:18:41.5307 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: hLVF7CmUDx3EGOedhZIP0VtoFuXkJmAfwkj+EP3wuTiuH3aXO4QRUxd3PPnxGK69SJUEufOq40XLfHtvXJ03Wg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA0PR12MB4461 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,garrett.kirkendall@amd.com X-Gm-Message-State: vopRsHUYnAsF1hakwbkeMu3Ux1787277AA= Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1592831929; bh=jKe1G6wLPXtXg0x2i7VY9hZ7cwPwSD5bWjowMwDRb14=; h=CC:Content-Type:Date:From:Reply-To:Subject:To; b=CS8NDqjvKi3cPbMv1CRSm7HYIMnNI3BXa8A9m4GNkBfAE8L2WRZs2wij/EPn64aUh83 cAbAU6Dz7XSh3UE7EUQxWw6ire15MmuDazkyQFnilG10M9m1XYyxzf3TH0lgJeJanzeYg qB9nZhU8vUfIHfoqfr/s+AMZmPZwrYHQLQg= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" Refactor StandardSignatureIsAuthenticAMD into BaseUefiCpuLib from separate copies in BaseXApicLib, BaseXApicX2ApicLib, and MpInitLib. This allows for future use of StandarSignatureIsAuthinticAMD without creating more instances in other modules. This function allows IA32/X64 code to determine if it is running on an AMD brand processor. UefiCpuLib is already included directly or indirectly in all modified modules. Complete move is made in this change. Cc: Eric Dong Cc: Ray Ni Cc: Laszlo Ersek Signed-off-by: Garrett Kirkendall Reviewed-by: Laszlo Ersek Reviewed-by: Eric Dong Reviewed-by: Chasel Chiu --- UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLib.inf | 7 ++++ UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.inf | 2 ++ UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.inf | 2 ++ UefiCpuPkg/Include/Library/UefiCpuLib.h | 14 ++++++++ UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLib.c | 38 ++++++++= ++++++++++++ UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.c | 25 ++------= ----- UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.c | 25 ++------= ----- UefiCpuPkg/Library/MpInitLib/MpLib.c | 23 --------= ---- 8 files changed, 67 insertions(+), 69 deletions(-) diff --git a/UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLib.inf b/UefiCpu= Pkg/Library/BaseUefiCpuLib/BaseUefiCpuLib.inf index 006b7acbf14e..34d3a7bb4303 100644 --- a/UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLib.inf +++ b/UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLib.inf @@ -4,6 +4,7 @@ # The library routines are UEFI specification compliant. # # Copyright (c) 2009 - 2018, Intel Corporation. All rights reserved.
+# Copyright (c) 2020, AMD Inc. All rights reserved.
# SPDX-License-Identifier: BSD-2-Clause-Patent # ## @@ -29,6 +30,12 @@ [Sources.IA32] [Sources.X64] X64/InitializeFpu.nasm =20 +[Sources] + BaseUefiCpuLib.c + [Packages] MdePkg/MdePkg.dec UefiCpuPkg/UefiCpuPkg.dec + +[LibraryClasses] + BaseLib diff --git a/UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.inf b/UefiCpuPkg/= Library/BaseXApicLib/BaseXApicLib.inf index bdb2ff372677..561baa44b0e6 100644 --- a/UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.inf +++ b/UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.inf @@ -5,6 +5,7 @@ # where local APIC is disabled. # # Copyright (c) 2010 - 2018, Intel Corporation. All rights reserved.
+# Copyright (c) 2020, AMD Inc. All rights reserved.
# SPDX-License-Identifier: BSD-2-Clause-Patent # ## @@ -37,6 +38,7 @@ [LibraryClasses] TimerLib IoLib PcdLib + UefiCpuLib =20 [Pcd] gUefiCpuPkgTokenSpaceGuid.PcdCpuInitIpiDelayInMicroSeconds ## SOMETIMES= _CONSUMES diff --git a/UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.inf b= /UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.inf index ac1e0a1c9896..1e2a4f8b790f 100644 --- a/UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.inf +++ b/UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.inf @@ -5,6 +5,7 @@ # where local APIC is disabled. # # Copyright (c) 2010 - 2018, Intel Corporation. All rights reserved.
+# Copyright (c) 2020, AMD Inc. All rights reserved.
# SPDX-License-Identifier: BSD-2-Clause-Patent # ## @@ -37,6 +38,7 @@ [LibraryClasses] TimerLib IoLib PcdLib + UefiCpuLib =20 [Pcd] gUefiCpuPkgTokenSpaceGuid.PcdCpuInitIpiDelayInMicroSeconds ## SOMETIMES= _CONSUMES diff --git a/UefiCpuPkg/Include/Library/UefiCpuLib.h b/UefiCpuPkg/Include/L= ibrary/UefiCpuLib.h index 82e53bab3a0f..5326e7246301 100644 --- a/UefiCpuPkg/Include/Library/UefiCpuLib.h +++ b/UefiCpuPkg/Include/Library/UefiCpuLib.h @@ -5,6 +5,7 @@ to be UEFI specification compliant. =20 Copyright (c) 2009, Intel Corporation. All rights reserved.
+ Copyright (c) 2020, AMD Inc. All rights reserved.
SPDX-License-Identifier: BSD-2-Clause-Patent =20 **/ @@ -29,4 +30,17 @@ InitializeFloatingPointUnits ( VOID ); =20 +/** + Determine if the standard CPU signature is "AuthenticAMD". + + @retval TRUE The CPU signature matches. + @retval FALSE The CPU signature does not match. + +**/ +BOOLEAN +EFIAPI +StandardSignatureIsAuthenticAMD ( + VOID + ); + #endif diff --git a/UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLib.c b/UefiCpuPk= g/Library/BaseUefiCpuLib/BaseUefiCpuLib.c new file mode 100644 index 000000000000..c2cc3ff9a709 --- /dev/null +++ b/UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLib.c @@ -0,0 +1,38 @@ +/** @file + This library defines some routines that are generic for IA32 family CPU. + + The library routines are UEFI specification compliant. + + Copyright (c) 2020, AMD Inc. All rights reserved.
+ SPDX-License-Identifier: BSD-2-Clause-Patent + +**/ + +#include +#include + +#include +#include + +/** + Determine if the standard CPU signature is "AuthenticAMD". + + @retval TRUE The CPU signature matches. + @retval FALSE The CPU signature does not match. + +**/ +BOOLEAN +EFIAPI +StandardSignatureIsAuthenticAMD ( + VOID + ) +{ + UINT32 RegEbx; + UINT32 RegEcx; + UINT32 RegEdx; + + AsmCpuid (CPUID_SIGNATURE, NULL, &RegEbx, &RegEcx, &RegEdx); + return (RegEbx =3D=3D CPUID_SIGNATURE_AUTHENTIC_AMD_EBX && + RegEcx =3D=3D CPUID_SIGNATURE_AUTHENTIC_AMD_ECX && + RegEdx =3D=3D CPUID_SIGNATURE_AUTHENTIC_AMD_EDX); +} diff --git a/UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.c b/UefiCpuPkg/Li= brary/BaseXApicLib/BaseXApicLib.c index 33ea15ca2916..52bd90d33428 100644 --- a/UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.c +++ b/UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.c @@ -4,7 +4,7 @@ This local APIC library instance supports xAPIC mode only. =20 Copyright (c) 2010 - 2019, Intel Corporation. All rights reserved.
- Copyright (c) 2017, AMD Inc. All rights reserved.
+ Copyright (c) 2017 - 2020, AMD Inc. All rights reserved.
=20 SPDX-License-Identifier: BSD-2-Clause-Patent =20 @@ -21,33 +21,12 @@ #include #include #include +#include =20 // // Library internal functions // =20 -/** - Determine if the standard CPU signature is "AuthenticAMD". - - @retval TRUE The CPU signature matches. - @retval FALSE The CPU signature does not match. - -**/ -BOOLEAN -StandardSignatureIsAuthenticAMD ( - VOID - ) -{ - UINT32 RegEbx; - UINT32 RegEcx; - UINT32 RegEdx; - - AsmCpuid (CPUID_SIGNATURE, NULL, &RegEbx, &RegEcx, &RegEdx); - return (RegEbx =3D=3D CPUID_SIGNATURE_AUTHENTIC_AMD_EBX && - RegEcx =3D=3D CPUID_SIGNATURE_AUTHENTIC_AMD_ECX && - RegEdx =3D=3D CPUID_SIGNATURE_AUTHENTIC_AMD_EDX); -} - /** Determine if the CPU supports the Local APIC Base Address MSR. =20 diff --git a/UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.c b/U= efiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.c index d0f92b33dc8c..cdcbca046191 100644 --- a/UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.c +++ b/UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.c @@ -5,7 +5,7 @@ which have xAPIC and x2APIC modes. =20 Copyright (c) 2010 - 2019, Intel Corporation. All rights reserved.
- Copyright (c) 2017, AMD Inc. All rights reserved.
+ Copyright (c) 2017 - 2020, AMD Inc. All rights reserved.
=20 SPDX-License-Identifier: BSD-2-Clause-Patent =20 @@ -22,33 +22,12 @@ #include #include #include +#include =20 // // Library internal functions // =20 -/** - Determine if the standard CPU signature is "AuthenticAMD". - - @retval TRUE The CPU signature matches. - @retval FALSE The CPU signature does not match. - -**/ -BOOLEAN -StandardSignatureIsAuthenticAMD ( - VOID - ) -{ - UINT32 RegEbx; - UINT32 RegEcx; - UINT32 RegEdx; - - AsmCpuid (CPUID_SIGNATURE, NULL, &RegEbx, &RegEcx, &RegEdx); - return (RegEbx =3D=3D CPUID_SIGNATURE_AUTHENTIC_AMD_EBX && - RegEcx =3D=3D CPUID_SIGNATURE_AUTHENTIC_AMD_ECX && - RegEdx =3D=3D CPUID_SIGNATURE_AUTHENTIC_AMD_EDX); -} - /** Determine if the CPU supports the Local APIC Base Address MSR. =20 diff --git a/UefiCpuPkg/Library/MpInitLib/MpLib.c b/UefiCpuPkg/Library/MpIn= itLib/MpLib.c index ab7a8ed6633a..9b0660a5d4ea 100644 --- a/UefiCpuPkg/Library/MpInitLib/MpLib.c +++ b/UefiCpuPkg/Library/MpInitLib/MpLib.c @@ -13,29 +13,6 @@ EFI_GUID mCpuInitMpLibHobGuid =3D CPU_INIT_MP_LIB_HOB_GUID; =20 =20 -/** - Determine if the standard CPU signature is "AuthenticAMD". - - @retval TRUE The CPU signature matches. - @retval FALSE The CPU signature does not match. - -**/ -STATIC -BOOLEAN -StandardSignatureIsAuthenticAMD ( - VOID - ) -{ - UINT32 RegEbx; - UINT32 RegEcx; - UINT32 RegEdx; - - AsmCpuid (CPUID_SIGNATURE, NULL, &RegEbx, &RegEcx, &RegEdx); - return (RegEbx =3D=3D CPUID_SIGNATURE_AUTHENTIC_AMD_EBX && - RegEcx =3D=3D CPUID_SIGNATURE_AUTHENTIC_AMD_ECX && - RegEdx =3D=3D CPUID_SIGNATURE_AUTHENTIC_AMD_EDX); -} - /** The function will check if BSP Execute Disable is enabled. =20 --=20 2.27.0 -=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 (#61559): https://edk2.groups.io/g/devel/message/61559 Mute This Topic: https://groups.io/mt/75037838/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 Sat Apr 20 01:11:40 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) client-ip=66.175.222.12; envelope-from=bounce+27952+61560+1787277+3901457@groups.io; helo=web01.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+61560+1787277+3901457@groups.io; arc=fail (BodyHash is different from the expected one) Received: from web01.groups.io (web01.groups.io [66.175.222.12]) by mx.zohomail.com with SMTPS id 15928319302603.2587892744612645; Mon, 22 Jun 2020 06:18:50 -0700 (PDT) Return-Path: X-Received: by 127.0.0.2 with SMTP id YHOlYY1788612x0CUlXK1q8j; Mon, 22 Jun 2020 06:18:49 -0700 X-Received: from NAM12-MW2-obe.outbound.protection.outlook.com (NAM12-MW2-obe.outbound.protection.outlook.com []) by mx.groups.io with SMTP id smtpd.web11.17266.1592831922292673781 for ; Mon, 22 Jun 2020 06:18:45 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=OEVPTfdYCu0J3XksIRhbkhUw9dnK3sgFOXrj5S9c/cAEN+oc9lJBinhkQU+94VpaygqX9LsZEj6u+GyUAuSNPCXIyFN4NgfkBUYyJ+er8S8Hm++MmG33XCYDGcP5Va5xs/vwqzzrJ+2WuHYTJdCp5PpaqMSsJpLIvDvCr8brBSWlIe8wPnyNeoqaKbP9pSBzHQFB6SxTZjhAfeItRAbyQ19MQBktl0VrLWp3TcMVCmITfZjHahdxnosUveogx0RX9Yy2IgYPmSxoLVZYi9K40seJGkb1/zOuJA520v8VJDjeoJ7oZvhb7UhzXkFM2X75j+HxgiAnR0SojEfiBnNecw== 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-SenderADCheck; bh=Z2bkmmb4XykF6Zjep3lGdCssrJXPWAHUs226DcOvwt0=; b=jcbBlUFsy+ukfq2m8EyI8BKrTVsAWcX1mI4XjnkoP0gIprYzn7guyuw3L5lAuF+G8ls48MWWoaygCbpKTnChI9hRhhjm75g8SE8F3uzseG0OaniaqJTWezkah19K1bPnxOSzUQ18W21Yl48sen5aG5k5lmnGGyU/+ZXuhXTZYDUEgR8HWOKxfpMmn4Hb0o8XTqtHKqPQV5fiJI9sjs60fU69DQMN6TK9SN9ff1xJapJjC9/c41B7HWg2U6yA4nUT9pdKRR2SfNwr28yx8jjoEoBhCdEqkcjKnaKXOAoEFWrg7bd2/lWV+shFZ3njLjXywQ7bvPIFcEciroq7QwD4Ew== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=amd.com; dmarc=pass action=none header.from=amd.com; dkim=pass header.d=amd.com; arc=none X-Received: from SN1PR12MB2352.namprd12.prod.outlook.com (2603:10b6:802:25::13) by SA0PR12MB4461.namprd12.prod.outlook.com (2603:10b6:806:9c::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3109.22; Mon, 22 Jun 2020 13:18:42 +0000 X-Received: from SN1PR12MB2352.namprd12.prod.outlook.com ([fe80::156e:ce6d:a148:274e]) by SN1PR12MB2352.namprd12.prod.outlook.com ([fe80::156e:ce6d:a148:274e%7]) with mapi id 15.20.3109.027; Mon, 22 Jun 2020 13:18:42 +0000 From: "Kirkendall, Garrett" To: devel@edk2.groups.io CC: Eric Dong , Ray Ni , Laszlo Ersek Subject: [edk2-devel] [PATCH v6 4/4] UefiCpuPkg: PiSmmCpuDxeSmm skip MSR_IA32_MISC_ENABLE manipulation on AMD Date: Mon, 22 Jun 2020 08:18:25 -0500 Message-ID: <20200622131825.1352-5-Garrett.Kirkendall@amd.com> In-Reply-To: <20200622131825.1352-1-Garrett.Kirkendall@amd.com> References: <20200622131825.1352-1-Garrett.Kirkendall@amd.com> X-ClientProxiedBy: SN4PR0501CA0135.namprd05.prod.outlook.com (2603:10b6:803:2c::13) To SN1PR12MB2352.namprd12.prod.outlook.com (2603:10b6:802:25::13) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-Received: from biosdev-01.amd.com (165.204.78.2) by SN4PR0501CA0135.namprd05.prod.outlook.com (2603:10b6:803:2c::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3131.13 via Frontend Transport; Mon, 22 Jun 2020 13:18:41 +0000 X-Originating-IP: [165.204.78.2] X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-HT: Tenant X-MS-Office365-Filtering-Correlation-Id: 9ac1f5de-5cbd-4ea0-02ae-08d816aec8a9 X-MS-TrafficTypeDiagnostic: SA0PR12MB4461: X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:7691; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam-Message-Info: pmljo2IghGrpK3d/6PQeeWZY7VjFSoN+ImStI5SyjxgLU4SsO4ITppGkddpHRj7tFhMZyzcWodzMbhCBqcQGPTP9y3qmGbX1fvILWQrXAJDAOUxlf6fWZF5prHYPYZBZU8xjoHqul7bqR6uTlw8kafAxMIKwLnyLC+ZKLH8R9kjY7KjKxnpsw1g/zFJp63LeHzx0huBb4gR5cmyRf3QExxApjt/KJ5osofGcpA+HILLpTLerWu7HJ0SL62hNV5IXD86G7dLxdDbVJW1z7XJK27qcFsc65Vb4n8gzpIAZ/pLVE3PyXrtMqa1z9H4ozu4/OCexvnN3xp0wE1s2x9AIdw== X-MS-Exchange-AntiSpam-MessageData: APJ8izh215sAjQ4VZDxKnmboNYr8+Ca14JrOy15vwGnHcqCUnwOthkzc6+JUOL/G/BbLXTOzX4L3b71ur/FYrL4gD3nZYLclYm0NLIRSqWAxW7n0MQvETz863CAWqOmhSY7Fn3yCH4p6W+Unyo1ZqaDmpQD/OVgZyjg2qdVOKmPJBLDH2chGDzGHkR2L9nGj0WKPbUPfLCGkLQPxdjSbyaLlSQrCZFL2ySJqwP8Ue4Wki7ABvZy8BParUwdwPu7I6RBgbj+dEKHyMPafx0Rw0qV2OjeIRVb7IijTADvlhyJA5vN7bLmOh4acCTp/t5NCo9+0w+W77FKxQjBfESbIX5aNFvOUVzFGFE+x9dCpleN5oy+NjAaxQyggbUjsr2xonsmqGF4e6PpC7HAdUUpfGaw3IYOpMvdDF91Qcf9vvZgqtC2o/Z/bAxCCrLWWCQs1xmaySgyCHVbsU4NAdJFEfEiguRqd+0c87yVQ8ZTNrjA= X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-Network-Message-Id: 9ac1f5de-5cbd-4ea0-02ae-08d816aec8a9 X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Jun 2020 13:18:42.1074 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: nuxlEIeVSlRNJfLqnld0u79QlU92ctgZ8o5MdHMP20gmLd7pYF6bLV2EmXYxXnvh8nJty0ksNvAW9Zy2EJ58Tg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA0PR12MB4461 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,garrett.kirkendall@amd.com X-Gm-Message-State: HJoZveAjyySUK8NEk4G8jxAKx1787277AA= Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1592831929; bh=9MM9U6Xjs2OeDXJjpFDaiFUrxyhmlcnku8UbL/n3tA4=; h=CC:Content-Type:Date:From:Reply-To:Subject:To; b=LprjsjTbLaTF6/23FULvvZDp8f2IIClcfCRRmSQEDH3xIGRC+sTNIUCltXQDA11oZlH i9AK6sSPWaTN2K21feihAs8rv7tcrvIXJfXCRpyWEDtyEMOvxbnVNGQ/huZcY1S9aSzvc 0Xi0U0y7wZYGB8v1zkTMNC78oBKWJQxoxrk= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" AMD does not support MSR_IA32_MISC_ENABLE. Accessing that register causes and exception on AMD processors. If Execution Disable is supported, but if the processor is an AMD processor, skip manipulating MSR_IA32_MISC_ENABLE[34] XD Disable bit. Cc: Eric Dong Cc: Ray Ni Cc: Laszlo Ersek Signed-off-by: Garrett Kirkendall Reviewed-by: Chasel Chiu Reviewed-by: Eric Dong Reviewed-by: Laszlo Ersek Tested-by: Laszlo Ersek --- Notes: Tested on Intel hardware with Laszlo Ersek's help =20 (1) downloaded two Linux images from provided links. (2) Test using a 32-bit guest on an Intel host (standing in your edk2 t= ree, with the patches applied): =20 $ build -a IA32 -b DEBUG -p OvmfPkg/OvmfPkgIa32.dsc -t GCC5 -D SMM_REQU= IRE =20 $ qemu-system-i386 \ -cpu coreduo,-nx \ -machine q35,smm=3Don,accel=3Dkvm \ -m 4096 \ -smp 4 \ -global driver=3Dcfi.pflash01,property=3Dsecure,value=3Don \ -drive if=3Dpflash,format=3Draw,unit=3D0,readonly=3Don,file=3DBuild= /OvmfIa32/DEBUG_GCC5/FV/OVMF_CODE.fd \ -drive if=3Dpflash,format=3Draw,unit=3D1,snapshot=3Don,file=3DBuild= /OvmfIa32/DEBUG_GCC5/FV/OVMF_VARS.fd \ -drive id=3Dhdd,if=3Dnone,format=3Dqcow2,snapshot=3Don,file=3Dfedor= a-30-efi-systemd-i686.qcow2 \ -device virtio-scsi-pci,id=3Dscsi0 \ -device scsi-hd,drive=3Dhdd,bus=3Dscsi0.0,bootindex=3D1 =20 (Once you get a login prompt, feel free to interrupt QEMU with Ctrl-C.) =20 (3) Test using a 64-bit guest on an Intel host: =20 $ build -a IA32 -a X64 -b DEBUG -p OvmfPkg/OvmfPkgIa32X64.dsc -t GCC5 -= D SMM_REQUIRE =20 $ qemu-system-x86_64 \ -cpu host \ -machine q35,smm=3Don,accel=3Dkvm \ -m 4096 \ -smp 4 \ -global driver=3Dcfi.pflash01,property=3Dsecure,value=3Don \ -drive if=3Dpflash,format=3Draw,unit=3D0,readonly=3Don,file=3DBuild= /Ovmf3264/DEBUG_GCC5/FV/OVMF_CODE.fd \ -drive if=3Dpflash,format=3Draw,unit=3D1,snapshot=3Don,file=3DBuild= /Ovmf3264/DEBUG_GCC5/FV/OVMF_VARS.fd \ -drive id=3Dhdd,if=3Dnone,format=3Dqcow2,snapshot=3Don,file=3Dfedor= a-31-efi-grub2-x86_64.qcow2 \ -device virtio-scsi-pci,id=3Dscsi0 \ -device scsi-hd,drive=3Dhdd,bus=3Dscsi0.0,bootindex=3D1 =20 Tested on real AMD Hardware UefiCpuPkg/PiSmmCpuDxeSmm/SmmProfileInternal.h | 3 +++ UefiCpuPkg/PiSmmCpuDxeSmm/SmmProfile.c | 9 ++++++++- UefiCpuPkg/PiSmmCpuDxeSmm/Ia32/SmiEntry.nasm | 19 +++++++++++++++++-- UefiCpuPkg/PiSmmCpuDxeSmm/X64/SmiEntry.nasm | 20 ++++++++++++++++++-- 4 files changed, 46 insertions(+), 5 deletions(-) diff --git a/UefiCpuPkg/PiSmmCpuDxeSmm/SmmProfileInternal.h b/UefiCpuPkg/Pi= SmmCpuDxeSmm/SmmProfileInternal.h index 43f6935cf9dc..993360a8a8c1 100644 --- a/UefiCpuPkg/PiSmmCpuDxeSmm/SmmProfileInternal.h +++ b/UefiCpuPkg/PiSmmCpuDxeSmm/SmmProfileInternal.h @@ -2,6 +2,7 @@ SMM profile internal header file. =20 Copyright (c) 2012 - 2018, Intel Corporation. All rights reserved.
+Copyright (c) 2020, AMD Incorporated. All rights reserved.
SPDX-License-Identifier: BSD-2-Clause-Patent =20 **/ @@ -13,6 +14,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent #include #include #include +#include #include =20 #include "SmmProfileArch.h" @@ -99,6 +101,7 @@ extern SMM_S3_RESUME_STATE *mSmmS3ResumeState; extern UINTN gSmiExceptionHandlers[]; extern BOOLEAN mXdSupported; X86_ASSEMBLY_PATCH_LABEL gPatchXdSupported; +X86_ASSEMBLY_PATCH_LABEL gPatchMsrIa32MiscEnableSupported; extern UINTN *mPFEntryCount; extern UINT64 (*mLastPFEntryValue)[MAX_PF_ENTRY_COUNT]; extern UINT64 *(*mLastPFEntryPointer)[MAX_PF_ENTRY_COUN= T]; diff --git a/UefiCpuPkg/PiSmmCpuDxeSmm/SmmProfile.c b/UefiCpuPkg/PiSmmCpuDx= eSmm/SmmProfile.c index c47b5573e366..d7ed9ab7a770 100644 --- a/UefiCpuPkg/PiSmmCpuDxeSmm/SmmProfile.c +++ b/UefiCpuPkg/PiSmmCpuDxeSmm/SmmProfile.c @@ -2,7 +2,7 @@ Enable SMM profile. =20 Copyright (c) 2012 - 2019, Intel Corporation. All rights reserved.
-Copyright (c) 2017, AMD Incorporated. All rights reserved.
+Copyright (c) 2017 - 2020, AMD Incorporated. All rights reserved.
=20 SPDX-License-Identifier: BSD-2-Clause-Patent =20 @@ -1015,6 +1015,13 @@ CheckFeatureSupported ( mXdSupported =3D FALSE; PatchInstructionX86 (gPatchXdSupported, mXdSupported, 1); } + + if (StandardSignatureIsAuthenticAMD ()) { + // + // AMD processors do not support MSR_IA32_MISC_ENABLE + // + PatchInstructionX86 (gPatchMsrIa32MiscEnableSupported, FALSE, 1); + } } =20 if (mBtsSupported) { diff --git a/UefiCpuPkg/PiSmmCpuDxeSmm/Ia32/SmiEntry.nasm b/UefiCpuPkg/PiSm= mCpuDxeSmm/Ia32/SmiEntry.nasm index f96de9bdeb43..167f5e14dbd4 100644 --- a/UefiCpuPkg/PiSmmCpuDxeSmm/Ia32/SmiEntry.nasm +++ b/UefiCpuPkg/PiSmmCpuDxeSmm/Ia32/SmiEntry.nasm @@ -1,5 +1,6 @@ ;-------------------------------------------------------------------------= ----- ; ; Copyright (c) 2016 - 2019, Intel Corporation. All rights reserved.
+; Copyright (c) 2020, AMD Incorporated. All rights reserved.
; SPDX-License-Identifier: BSD-2-Clause-Patent ; ; Module Name: @@ -59,6 +60,7 @@ global ASM_PFX(gPatchSmiStack) global ASM_PFX(gPatchSmbase) extern ASM_PFX(mXdSupported) global ASM_PFX(gPatchXdSupported) +global ASM_PFX(gPatchMsrIa32MiscEnableSupported) extern ASM_PFX(gSmiHandlerIdtr) =20 extern ASM_PFX(mCetSupported) @@ -153,17 +155,30 @@ ASM_PFX(gPatchSmiCr3): ASM_PFX(gPatchXdSupported): cmp al, 0 jz @SkipXd + +; If MSR_IA32_MISC_ENABLE is supported, clear XD Disable bit + mov al, strict byte 1 ; source operand may be patched +ASM_PFX(gPatchMsrIa32MiscEnableSupported): + cmp al, 1 + jz MsrIa32MiscEnableSupported + +; MSR_IA32_MISC_ENABLE not supported + xor edx, edx + push edx ; don't try to restore the XD Disa= ble bit just before RSM + jmp EnableNxe + ; ; Check XD disable bit ; +MsrIa32MiscEnableSupported: mov ecx, MSR_IA32_MISC_ENABLE rdmsr push edx ; save MSR_IA32_MISC_ENABLE[63-32] test edx, BIT2 ; MSR_IA32_MISC_ENABLE[34] - jz .5 + jz EnableNxe and dx, 0xFFFB ; clear XD Disable bit if it is set wrmsr -.5: +EnableNxe: mov ecx, MSR_EFER rdmsr or ax, MSR_EFER_XD ; enable NXE diff --git a/UefiCpuPkg/PiSmmCpuDxeSmm/X64/SmiEntry.nasm b/UefiCpuPkg/PiSmm= CpuDxeSmm/X64/SmiEntry.nasm index 8bfba55b5d08..0e154e5db949 100644 --- a/UefiCpuPkg/PiSmmCpuDxeSmm/X64/SmiEntry.nasm +++ b/UefiCpuPkg/PiSmmCpuDxeSmm/X64/SmiEntry.nasm @@ -1,5 +1,6 @@ ;-------------------------------------------------------------------------= ----- ; ; Copyright (c) 2016 - 2019, Intel Corporation. All rights reserved.
+; Copyright (c) 2020, AMD Incorporated. All rights reserved.
; SPDX-License-Identifier: BSD-2-Clause-Patent ; ; Module Name: @@ -67,6 +68,7 @@ extern ASM_PFX(CpuSmmDebugExit) global ASM_PFX(gPatchSmbase) extern ASM_PFX(mXdSupported) global ASM_PFX(gPatchXdSupported) +global ASM_PFX(gPatchMsrIa32MiscEnableSupported) global ASM_PFX(gPatchSmiStack) global ASM_PFX(gPatchSmiCr3) global ASM_PFX(gPatch5LevelPagingNeeded) @@ -152,18 +154,32 @@ SkipEnable5LevelPaging: ASM_PFX(gPatchXdSupported): cmp al, 0 jz @SkipXd + +; If MSR_IA32_MISC_ENABLE is supported, clear XD Disable bit + mov al, strict byte 1 ; source operand may be patched +ASM_PFX(gPatchMsrIa32MiscEnableSupported): + cmp al, 1 + jz MsrIa32MiscEnableSupported + +; MSR_IA32_MISC_ENABLE not supported + sub esp, 4 + xor rdx, rdx + push rdx ; don't try to restore the XD Disa= ble bit just before RSM + jmp EnableNxe + ; ; Check XD disable bit ; +MsrIa32MiscEnableSupported: mov ecx, MSR_IA32_MISC_ENABLE rdmsr sub esp, 4 push rdx ; save MSR_IA32_MISC_ENABLE[63-32] test edx, BIT2 ; MSR_IA32_MISC_ENABLE[34] - jz .0 + jz EnableNxe and dx, 0xFFFB ; clear XD Disable bit if it is set wrmsr -.0: +EnableNxe: mov ecx, MSR_EFER rdmsr or ax, MSR_EFER_XD ; enable NXE --=20 2.27.0 -=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 (#61560): https://edk2.groups.io/g/devel/message/61560 Mute This Topic: https://groups.io/mt/75037839/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-