From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 192.55.52.93, mailfrom: eric.dong@intel.com) Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by groups.io with SMTP; Tue, 16 Jul 2019 18:59:03 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga006.jf.intel.com ([10.7.209.51]) by fmsmga102.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 16 Jul 2019 18:59:02 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,272,1559545200"; d="scan'208";a="172719435" Received: from ydong10-win10.ccr.corp.intel.com ([10.239.158.133]) by orsmga006.jf.intel.com with ESMTP; 16 Jul 2019 18:59:01 -0700 From: "Dong, Eric" To: devel@edk2.groups.io Cc: Ray Ni , Laszlo Ersek , Chandana Kumar , Star Zeng Subject: [Patch v3 0/2] UefiCpuPkg/RegisterCpuFeaturesLib: Avoid AP calls PeiServices table Date: Wed, 17 Jul 2019 09:58:58 +0800 Message-Id: <20190717015900.29224-1-eric.dong@intel.com> X-Mailer: git-send-email 2.21.0.windows.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit V3 Changes: 1. Remove IsCpuFeatureMatch function. V2 Changes: 1. Revert IsBitMaskMatchCheck change which is not correct. 2. refine some variable name. v1 changes: REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1972 AP calls CollectProcessorData() to collect processor info. CollectProcessorData function finally calls PcdGetSize function to get DynamicPCD PcdCpuFeaturesSetting value. PcdGetSize will use gBS which caused ASSERT. This patch serial fixes the issue and enhances the related code to avoid later report this issue again. Cc: Ray Ni Cc: Laszlo Ersek Cc: Chandana Kumar Cc: Star Zeng Dong, Eric (1): UefiCpuPkg/RegisterCpuFeaturesLib: avoid use dynamic PCD. Eric Dong (1): UefiCpuPkg/RegisterCpuFeaturesLib: Avoid AP calls PeiServices table. .../CpuFeaturesInitialize.c | 77 +++++++------ .../RegisterCpuFeatures.h | 10 +- .../RegisterCpuFeaturesLib.c | 102 +++++++----------- 3 files changed, 87 insertions(+), 102 deletions(-) -- 2.21.0.windows.1