From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from NAM10-MW2-obe.outbound.protection.outlook.com (NAM10-MW2-obe.outbound.protection.outlook.com [40.107.94.61]) by mx.groups.io with SMTP id smtpd.web09.19392.1582915557840826942 for ; Fri, 28 Feb 2020 10:45:58 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@amdcloud.onmicrosoft.com header.s=selector2-amdcloud-onmicrosoft-com header.b=T15FrNtN; spf=none, err=SPF record not found (domain: amd.com, ip: 40.107.94.61, mailfrom: leo.duran@amd.com) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=l/G7pnuJ7vPI9UHGIi1oX7BLpSHOyEaUBA7WyQc1Iw8a3PKo6CiZFxayQVzhgLP0XEKspRgJ94xyx2crybfoFS9cpWL0EILsWg45iV4a36X2KszYkJQB0KxiFy3UexxaP3rGlRJTL0yu5S3RVHV6vSQw0Oobc3bloNgqMTbiT3E1gBPkLiaR7Pe176fBYO1PdxTY+i88yjo7KwKc5CiLUHJze/GrnEx/ZyVWI0CuvnXHx4HHAljGhe3DpNwAsXKGEJ19oUYyOSdn+GFzEVwt18RmqrtXZl5aJKZ8U2X913gSDKOsreQINu/U7Lz10CHa7ERUbwwbxOfPVCynQUKLzQ== 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=0Ckf+mRhJX+Mc8Jh+/obtx+voM8xfMBRAUVOVFD9xkI=; b=gpoYCzC/Szx4Lj+A1dGoi0I3fplXPUPhOymAF0qcWD+pPY/8Iq7gRsGHJP14Yk1WUvxCykaP/Ro39mjMrPwTKHAeyBN8WW4/+wyjmo8wkJLXtoZzrkoJKC9GbblARTmIkEi6U7Yf+xI/tXXYItbZ+fxcUgSGnQtTc3VzFEmfxAvMLsPMH50qmxfN0MP03jRxQl8a3T+KTVCeIHCLN+Ya6PNkix492C5Xp+RlUTKW3Yteslh8YEGPRFiI1lvyCVa6V0abVgh77rec8fgupjTM6CPTpbip0KovyT4pHIwnkgrJ2IkoDv6IbHvMNw/L26eIDBxda2f9+ViyJAIx2HgtvA== 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 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amdcloud.onmicrosoft.com; s=selector2-amdcloud-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=0Ckf+mRhJX+Mc8Jh+/obtx+voM8xfMBRAUVOVFD9xkI=; b=T15FrNtNvomU7NlobOlN4+QmflxTPDecFPlitc7GjBpyPZKygxfFZhU9gD5GsNwVBSATFbkMpBxDY1QxCYU/HmBgqtp7b/6WkDnG1WwhVqx2S/RyoY7Y6ddEVHbZ97gMvw57wXCop6aMHpkKcaDoVkg6IIG1VjCXX5jcCVDss7M= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=leo.duran@amd.com; Received: from BN6PR12MB1922.namprd12.prod.outlook.com (2603:10b6:404:106::14) by BN6PR12MB1778.namprd12.prod.outlook.com (2603:10b6:404:106::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2772.14; Fri, 28 Feb 2020 18:45:56 +0000 Received: from BN6PR12MB1922.namprd12.prod.outlook.com ([fe80::d931:1942:a6b5:d74c]) by BN6PR12MB1922.namprd12.prod.outlook.com ([fe80::d931:1942:a6b5:d74c%7]) with mapi id 15.20.2772.018; Fri, 28 Feb 2020 18:45:55 +0000 From: Leo Duran To: devel@edk2.groups.io Cc: Leo Duran Subject: [PATCH v2] UefiCpuPkg: Fix bug in MpInitLib Date: Fri, 28 Feb 2020 12:45:41 -0600 Message-Id: <1582915542-28940-1-git-send-email-leo.duran@amd.com> X-Mailer: git-send-email 2.7.4 X-ClientProxiedBy: DM5PR22CA0008.namprd22.prod.outlook.com (2603:10b6:3:101::18) To BN6PR12MB1922.namprd12.prod.outlook.com (2603:10b6:404:106::14) Return-Path: leo.duran@amd.com MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from leduran-Precision-WorkStation-T5400.amd.com (165.204.78.2) by DM5PR22CA0008.namprd22.prod.outlook.com (2603:10b6:3:101::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256) id 15.20.2772.14 via Frontend Transport; Fri, 28 Feb 2020 18:45:55 +0000 X-Mailer: git-send-email 2.7.4 X-Originating-IP: [165.204.78.2] X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-HT: Tenant X-MS-Office365-Filtering-Correlation-Id: 70c1f705-3384-4b2d-caf9-08d7bc7e718e X-MS-TrafficTypeDiagnostic: BN6PR12MB1778:|BN6PR12MB1778: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:6790; X-Forefront-PRVS: 0327618309 X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(6029001)(4636009)(366004)(39860400002)(376002)(396003)(136003)(346002)(189003)(199004)(16526019)(186003)(2906002)(26005)(6666004)(44832011)(316002)(4744005)(52116002)(7696005)(66556008)(4326008)(81166006)(5660300002)(2616005)(66476007)(956004)(8936002)(86362001)(66946007)(81156014)(478600001)(6486002)(6916009)(8676002)(36756003);DIR:OUT;SFP:1101;SCL:1;SRVR:BN6PR12MB1778;H:BN6PR12MB1922.namprd12.prod.outlook.com;FPR:;SPF:None;LANG:en;PTR:InfoNoRecords;MX:1;A:1; Received-SPF: None (protection.outlook.com: amd.com does not designate permitted sender hosts) X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: KorEYTpj/vf8IIZMCmE4HXFqV8zXDBveuAw0AQaRHxfpV+cXu4ewJSQklPDTAagmlpx0LS2VweprlboCtrRtgaZHZrbh/DmazWULXkBNKJuV2JdzhsgC1kW349RhoST8Ns1tObTDV85HLv5xhGDB+hlrbHQQpkupWqiyvr6sTJrgjAhjzuj3EFJnltzwWSdcE5TERQ+gNEsy72PiO377yncimL8KSfu1wACQkE77w+PshlzxwW3sanvq0wz/kW0K36JgbbTYzf9rdyBpk6AxlydMW3L+ahIXSyABvsgKHoNtjOxoA/2IEy0ZX4wC3i+7nCG5DLzQCMWzNG/tbrHeKS+HQH34noGJ2tBtU+hTFxLPVbO7Lst68tRTSgKpLRMlAxqrId1h5t4QeLXecqy2d2w5ys8DCnGaNvYqiYg0cuxVkO1nVXfULjHtZU7i1S0v X-MS-Exchange-AntiSpam-MessageData: FHGobYBkyTPMlA0FAZBa214tui3iq4TopvF7Jd8EPVqjIlR2hIC+2sr/57V9lcAMGcaniYsJ74/i62i9ZewL8scLflFqXhnvhJDUoxZ6qirX6uiURW3LUfBLrFL+fxfp6LR+WjTiLuB3X69EnvVwRQ== X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-Network-Message-Id: 70c1f705-3384-4b2d-caf9-08d7bc7e718e X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Feb 2020 18:45:55.8027 (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: Y+q1Gpm6q62tEOgaMAgDh5TLdQM86Sm4P1MFylGs63DktWLEOZJxix0DXHUOEzGn X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN6PR12MB1778 Content-Type: text/plain This patch fixes an issue introduced recently in MpInitLib, where we read a PlatformId MSR that is not implemented on AMD processors. The patch uses CPUID signature check to skip reading the PlatformId MSR. Changes since v1: - Undo changes to LocalApicLib to not export CPUID signature check. - Undo changes to MpInitLib/Microcode.c and rely on existing PCDs to manage loading of microcode patches. Leo Duran (1): UefiCpuPkg/MpInitLib: Skip reading PlatformId on AMD processors. UefiCpuPkg/Library/MpInitLib/MpLib.c | 34 ++++++++++++++++++++++++++++++++-- 1 file changed, 32 insertions(+), 2 deletions(-) -- 2.7.4