From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by mx.groups.io with SMTP id smtpd.web10.16819.1658902370064559118 for ; Tue, 26 Jul 2022 23:12:50 -0700 Authentication-Results: mx.groups.io; dkim=fail reason="unable to parse pub key" header.i=@intel.com header.s=intel header.b=CV/86DfR; spf=pass (domain: intel.com, ip: 192.55.52.88, mailfrom: jackx.lin@intel.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1658902370; x=1690438370; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=IN0uKpnypOZu2E0bNwJSUS1t63CYie54AOTmq7SK2uQ=; b=CV/86DfRBWp7uOyDF2197BZL0ffi48tU4FzpLjgn0Zv2ZLinQqFVkab+ eKZM6WI7u/qSI2z2x+Mz53SsYkWsHbitDkZHttvuVR9yn7URRoFcDPpvb 2z26mt6nw22WN2LR5EuQBYNwVQrmWjAzX8uktWdgze58GTbuP8mLcRAau OTqiK+c0Akq9SAA4eIbNDvQj2BEpTKmzVvRl9z0BAAek6dYQoslaG3w+v ekGTkU1w8wMz7W/wbgFqRyJJl4/jP3jx3eemcX1H/MmGyrl62IndnmB8h cLPaowvKX4nyQua8VxYyo1UGgotYzudmSoaenXn+DHrTp+L2QRojPQ3zl Q==; X-IronPort-AV: E=McAfee;i="6400,9594,10420"; a="313931529" X-IronPort-AV: E=Sophos;i="5.93,195,1654585200"; d="scan'208";a="313931529" Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Jul 2022 23:12:19 -0700 X-IronPort-AV: E=Sophos;i="5.93,195,1654585200"; d="scan'208";a="659053161" Received: from lins2x-desk1.gar.corp.intel.com ([10.225.33.149]) by fmsmga008-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Jul 2022 23:12:15 -0700 From: "JackX Lin" To: devel@edk2.groups.io Cc: JackX Lin , JackX Lin , Chasel Chiu , Dong Eric , Jiewen Yao , Ray Ni , Rangasai V Chaganty , Donald Kuo , Chandana C Kumar Subject: [edk2-platforms: PATCH] Modify processor _UID ordering by CPU default fused in MADT Date: Wed, 27 Jul 2022 14:11:49 +0800 Message-Id: <20220727061149.1495-1-jackx.lin@intel.com> X-Mailer: git-send-email 2.32.0.windows.2 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit BIOS should not reordering cpu processor_uid Signed-off-by: JackX Lin Cc: Chasel Chiu Cc: Dong Eric Cc: Jiewen Yao Cc: Ray Ni Cc: Rangasai V Chaganty Cc: Donald Kuo Cc: Chandana C Kumar Cc: JackX Lin --- Platform/Intel/MinPlatformPkg/Acpi/AcpiTables/AcpiPlatform.c | 14 +------------- 1 file changed, 1 insertion(+), 13 deletions(-) diff --git a/Platform/Intel/MinPlatformPkg/Acpi/AcpiTables/AcpiPlatform.c b/Platform/Intel/MinPlatformPkg/Acpi/AcpiTables/AcpiPlatform.c index c7e87cbd7d..f4c45336c5 100644 --- a/Platform/Intel/MinPlatformPkg/Acpi/AcpiTables/AcpiPlatform.c +++ b/Platform/Intel/MinPlatformPkg/Acpi/AcpiTables/AcpiPlatform.c @@ -173,7 +173,6 @@ SortCpuLocalApicInTable ( EFI_CPU_ID_ORDER_MAP *CpuIdMapPtr; UINT32 CoreThreadMask; EFI_CPU_ID_ORDER_MAP *TempCpuApicIdOrderTable; - UINT32 Socket; Index = 0; Status = EFI_SUCCESS; @@ -198,6 +197,7 @@ SortCpuLocalApicInTable ( CpuIdMapPtr->Thread = ProcessorInfoBuffer.Location.Thread; CpuIdMapPtr->Flags = ((ProcessorInfoBuffer.StatusFlag & PROCESSOR_ENABLED_BIT) != 0); CpuIdMapPtr->SocketNum = ProcessorInfoBuffer.Location.Package; + CpuIdMapPtr->AcpiProcessorUid = (ProcessorInfoBuffer.Location.Package << mNumOfBitShift) + CurrProcessor; //update processorbitMask if (CpuIdMapPtr->Flags == 1) { @@ -280,18 +280,6 @@ SortCpuLocalApicInTable ( } } - // - // 5. Re-assign AcpiProcessorId for AcpiProcessorUid uses purpose. - // - for (Socket = 0; Socket < FixedPcdGet32 (PcdMaxCpuSocketCount); Socket++) { - for (CurrProcessor = 0, Index = 0; CurrProcessor < mNumberOfCpus; CurrProcessor++) { - if (mCpuApicIdOrderTable[CurrProcessor].Flags && (mCpuApicIdOrderTable[CurrProcessor].SocketNum == Socket)) { - mCpuApicIdOrderTable[CurrProcessor].AcpiProcessorUid = (mCpuApicIdOrderTable[CurrProcessor].SocketNum << mNumOfBitShift) + Index; - Index++; - } - } - } - //keep for debug purpose DEBUG ((DEBUG_INFO, "APIC ID Order Table ReOrdered\n")); DebugDisplayReOrderTable (mCpuApicIdOrderTable); -- 2.32.0.windows.2