From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mout01.posteo.de (mout01.posteo.de [185.67.36.65]) by mx.groups.io with SMTP id smtpd.web12.1700.1628362494220580413 for ; Sat, 07 Aug 2021 11:54:55 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@posteo.de header.s=2017 header.b=kk11zRR2; spf=pass (domain: posteo.de, ip: 185.67.36.65, mailfrom: mhaeuser@posteo.de) Received: from submission (posteo.de [89.146.220.130]) by mout01.posteo.de (Postfix) with ESMTPS id 5FBD5240028 for ; Sat, 7 Aug 2021 20:54:51 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=posteo.de; s=2017; t=1628362491; bh=by+E6oCRJ0sqmODhkdSZfXtregG4lDniDfHL6H9n1ow=; h=To:From:Subject:Date:From; b=kk11zRR2ReEdHm2Dhegp/zRcGKJEDVdUlKq8PeinFpzOYuIBEfH+Y2WyMjWt9SYVd ueq0tgiVl49CtQ41ywfgy4+JH/xjsN1tWOh9QfBmf37/CRob2ltKpPbW4U0OnJntYw X3I5DdsF1TEYMjbyKXaAVIEPb0Dabvjw/N35NxQvTu5MrhgiVavATF7gSvpTg1CRxb 2kgfmqFlfrjfnuI5sWpMVODeTnJngDy1btxlc9cQSkV0JPOIa85eNmMet8U1XzQB3f RbuWKChdF/jnt9VQFPcf1UqNPt1LQCLfoIAkf7vtsZ8Eg87d7XzsAV2AnqlImShOkz djkfPe2axB9Lg== Received: from customer (localhost [127.0.0.1]) by submission (posteo.de) with ESMTPSA id 4Ghs3G4DCHz9rxG; Sat, 7 Aug 2021 20:54:50 +0200 (CEST) To: "devel@edk2.groups.io" , "michael.kubacki@microsoft.com" From: =?UTF-8?B?TWFydmluIEjDpHVzZXI=?= Subject: SecCore evacuation in PeiCore? Message-ID: Date: Sat, 7 Aug 2021 18:54:50 +0000 MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Content-Language: en-GB Good day everyone, Good day Michael, The commit that introduced T-RAM evacuation [1] also introduced the function "MigrateSecModulesInFv()". It also is explicitly mentioned as part of the control flow in the commit message. As far as I can see, since then till today this function has never been called anywhere. Was this some draft function that accidentally made it into the patch, or did the caller get lost somewhere? The description makes sense to me and I'm not experienced enough with the PeiCore control flow to tell whether the PEIM migration somehow covers SecCore implicitly. Also I noticed it only supports SecCore in a PE/COFF section, not a TE section. Is there a rationale for that? Thank you for your time! Best regards, Marvin [1] https://github.com/tianocore/edk2/commit/9bedaec05b7b8ba9aee248361bb61a85a26726cb