From nobody Mon Sep 16 19:11:33 2024 Delivered-To: importer@patchew.org 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+108785+1787277+3901457@groups.io; helo=mail02.groups.io; 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+108785+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=intel.com ARC-Seal: i=1; a=rsa-sha256; t=1695050264; cv=none; d=zohomail.com; s=zohoarc; b=JIcfdL8oWMOWoi5BbT3VRULsjemnKEWx+1eKZTo0UAbMNZexVFDWbXE5B+uhFT0HVi1E6ZCBUA5dUFfyfX72oj1hYkWd+Lk5G6GfRfZhfLZCPrtQaWeiN8XROOk1kL8eL4XmE0Xyh5q0xyuCMliaMI3fCupVY/F9jz3tl55FnmY= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1695050264; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:References:Sender:Subject:To; bh=0CFqjOtdR0kXdOAUojQqhPokkE9upxRojebaY0rlsSY=; b=iYI9d1eLekDMl6Sg5hFctNVtLyarHQtwQ2Rjaccob8hoePRFdF03JD9kOArsSY19quaDPWaipfqvWsIwS4aHFZBGUYGhdvJHkJ+vikGM83K3twP/Pb9FUN+xt4zYZ4LtCaTlCjhrofxAVJjtJ7rHpazsYvBEhTjQaRvtqnqOK3o= ARC-Authentication-Results: i=1; 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+108785+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 1695050264606390.0710866430426; Mon, 18 Sep 2023 08:17:44 -0700 (PDT) Return-Path: DKIM-Signature: a=rsa-sha256; bh=bFt4l7lvZuWDylt2J8FMdisbAosE4YhMmKn4CqX8IOE=; c=relaxed/simple; d=groups.io; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References:MIME-Version:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Transfer-Encoding; s=20140610; t=1695050264; v=1; b=Pd0APBHjO+BzuTxeOSgQv72KUQ2RGPwuzCCZnRsJVKcpyFkijRhO/RP32BjnqnQbRvkrTkOP RoC4Hu2WEIL8DUoQ8La3tX9eL2MrLxaEI65j/IhmVbXX88mBBMdYUCVJg+7DoH7c5Niyg/IDpCW 5ki0s7b8bwG1whhLQEdfYhCM= X-Received: by 127.0.0.2 with SMTP id V7vEYY1788612xlLEeDS1IlU; Mon, 18 Sep 2023 08:17:44 -0700 X-Received: from mgamail.intel.com (mgamail.intel.com [134.134.136.24]) by mx.groups.io with SMTP id smtpd.web10.45821.1695019066711439455 for ; Sun, 17 Sep 2023 23:37:51 -0700 X-IronPort-AV: E=McAfee;i="6600,9927,10836"; a="382319910" X-IronPort-AV: E=Sophos;i="6.02,155,1688454000"; d="scan'208";a="382319910" X-Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by orsmga102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 17 Sep 2023 23:37:51 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10836"; a="860930878" X-IronPort-AV: E=Sophos;i="6.02,155,1688454000"; d="scan'208";a="860930878" X-Received: from juichenx-mobl3.gar.corp.intel.com ([10.227.107.41]) by fmsmga002.fm.intel.com with ESMTP; 17 Sep 2023 23:37:49 -0700 From: brucex.wang@intel.com To: devel@edk2.groups.io Cc: brucex.wang@intel.com, Zhiguang Liu , Eric Dong , Ray Ni , Rahul Kumar , Gerd Hoffmann Subject: [edk2-devel] [PATCH v4 2/4] UefiCpuPkg/ResetVector: Remove AP waking vector from ResetVector Date: Mon, 18 Sep 2023 14:37:17 +0800 Message-Id: <20230918063719.1395-3-brucex.wang@intel.com> In-Reply-To: <20230918063719.1395-1-brucex.wang@intel.com> References: <20230918063719.1395-1-brucex.wang@intel.com> MIME-Version: 1.0 Precedence: Bulk 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,brucex.wang@intel.com List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: X-Gm-Message-State: 0OnYUzuVpNqvreMNvmkt3buVx1787277AA= Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @groups.io) X-ZM-MESSAGEID: 1695050275019100011 Content-Type: text/plain; charset="utf-8" From: Zhiguang Liu REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D4494 Current reset vector uses 0xffffffe0 as AP waking vector, and expects GenFv generates code aligned on a 4k boundary which will jump to this location. However, some issues are listed below 1. GenFV doesn't generate code as the comment expects, because GenFv assumes no modifications are required to the VTF-0 'Volume Top File'. 2. Even if removing VFT0 signature and let GenFv to modify, Genfv is hard-code using another flash address 0xffffffd0. 3. In the same patch series, AP waking vector code is removed from GenFv, because no such usage anymore. The existing of first two issues also approve the usage is not available for a long time. Therefore, remove AP waking vector related code. Cc: Eric Dong Cc: Ray Ni Cc: Rahul Kumar Cc: Gerd Hoffmann Reviewed-by: Ray Ni Signed-off-by: Zhiguang Liu --- UefiCpuPkg/ResetVector/Vtf0/Ia16/Init16.asm | 7 ------- .../ResetVector/Vtf0/Ia16/ResetVectorVtf0.asm | 15 +++------------ 2 files changed, 3 insertions(+), 19 deletions(-) diff --git a/UefiCpuPkg/ResetVector/Vtf0/Ia16/Init16.asm b/UefiCpuPkg/Reset= Vector/Vtf0/Ia16/Init16.asm index cbdadee166..02b9a0303c 100644 --- a/UefiCpuPkg/ResetVector/Vtf0/Ia16/Init16.asm +++ b/UefiCpuPkg/ResetVector/Vtf0/Ia16/Init16.asm @@ -17,13 +17,6 @@ EarlyBspInitReal16: mov di, 'BP' jmp short Main16 =20 -; -; @param[out] DI 'AP' to indicate application processor -; -EarlyApInitReal16: - mov di, 'AP' - jmp short Main16 - ; ; Modified: EAX ; diff --git a/UefiCpuPkg/ResetVector/Vtf0/Ia16/ResetVectorVtf0.asm b/UefiCpu= Pkg/ResetVector/Vtf0/Ia16/ResetVectorVtf0.asm index fe5bbea803..384b1d4d98 100644 --- a/UefiCpuPkg/ResetVector/Vtf0/Ia16/ResetVectorVtf0.asm +++ b/UefiCpuPkg/ResetVector/Vtf0/Ia16/ResetVectorVtf0.asm @@ -32,22 +32,13 @@ ALIGN 16 TIMES (0x1000 - 0x20) DB 0 %endif =20 -applicationProcessorEntryPoint: ; -; Application Processors entry point +; 0xffffffe0 ; -; GenFv generates code aligned on a 4k boundary which will jump to this -; location. (0xffffffe0) This allows the Local APIC Startup IPI to be -; used to wake up the application processors. -; - jmp EarlyApInitReal16 - -ALIGN 8 - - DD 0 + DD 0, 0, 0 =20 ; -; The VTF signature +; The VTF signature (0xffffffec) ; ; VTF-0 means that the VTF (Volume Top File) code does not require ; any fixups. --=20 2.39.1.windows.1 -=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 (#108785): https://edk2.groups.io/g/devel/message/108785 Mute This Topic: https://groups.io/mt/101435614/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-