From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by mx.groups.io with SMTP id smtpd.web10.10602.1636687715525127113 for ; Thu, 11 Nov 2021 19:28:35 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@intel.onmicrosoft.com header.s=selector2-intel-onmicrosoft-com header.b=kSvR2bsY; spf=pass (domain: intel.com, ip: 134.134.136.100, mailfrom: gao.cheng@intel.com) X-IronPort-AV: E=McAfee;i="6200,9189,10165"; a="296501819" X-IronPort-AV: E=Sophos;i="5.87,227,1631602800"; d="scan'208";a="296501819" Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Nov 2021 19:28:34 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.87,227,1631602800"; d="scan'208";a="670512625" Received: from fmsmsx604.amr.corp.intel.com ([10.18.126.84]) by orsmga005.jf.intel.com with ESMTP; 11 Nov 2021 19:28:34 -0800 Received: from fmsmsx611.amr.corp.intel.com (10.18.126.91) by fmsmsx604.amr.corp.intel.com (10.18.126.84) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12; Thu, 11 Nov 2021 19:28:33 -0800 Received: from fmsmsx601.amr.corp.intel.com (10.18.126.81) by fmsmsx611.amr.corp.intel.com (10.18.126.91) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12; Thu, 11 Nov 2021 19:28:33 -0800 Received: from fmsedg602.ED.cps.intel.com (10.1.192.136) by fmsmsx601.amr.corp.intel.com (10.18.126.81) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12 via Frontend Transport; Thu, 11 Nov 2021 19:28:33 -0800 Received: from NAM11-DM6-obe.outbound.protection.outlook.com (104.47.57.168) by edgegateway.intel.com (192.55.55.71) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2242.12; Thu, 11 Nov 2021 19:28:33 -0800 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=QUpR8Lhxn6kfmVSnTxLP0RWJoZXI2VoILpy70eM716pMwqZMSDodK2phTo9RW18fgdNb91SASO43psAjGZ3AAfNCMjJeneIFJMrF+iWdqdFlPosJfvLJopixAa5FfPcoScwKPMfQds+151UDxvc1Vf4Jbz6j5+OJNpuxQs8JkoaBj2WJsFiY0vgWG36BeXhNKIyEy+tcAGo5O5fojoxcKn+hcCCwCJvvJtOw49xwxOvUkTMkEOgFHdBw+c4CSwQD5kpEnsnTPmO+lSecGAvGp5T0qg+sqz2R+tp9NuAdDybv1Sdh/sSUOMKWN5UA/Wl99n1Iy+Uh8hrmIT+/GzuZow== 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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=pFT8Rc9RdyrUGwSc16nde8pI4BNPBkcHbJSgLlFtZLw=; b=V2LMYJvKiUM2+Fxmc30x0ItHCWBcGFUeL7qmLOVNaItyeCB2OqP8XqCvR3VZbHEsL5tJfG54aYGMvgO/FxG+CC3E0UVvYzdzr2Gd3t5bYRe+K2Q37SC6f4GWPte9QWEGhL6e+ssrRLHcI/nF56nDTnDfLGGY4RWHj5RxXFOgFFCW9cijJzo9HnuhyR4TyLjGONahfmZyHmvUmeA8ChLwk7dXYzXjkVmwYBebZmHaHtVlMqyi1bixswoUuxcCSYEpOlZZ2H0n5qOAvoa51v9ex8zNKnf6hdLO9qiQB1EH8+MEDmdeqTYQ2fU+czuYoYkt5Ps6YkTD5lGlP/PgO+zeVQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel.onmicrosoft.com; s=selector2-intel-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=pFT8Rc9RdyrUGwSc16nde8pI4BNPBkcHbJSgLlFtZLw=; b=kSvR2bsYyAHMeg1xuyanmid0s8kfch+HwPSM+2hA3BcAxtrNr/dLHbnwwnZtCfRPniweDEZXrR/f07x5obWaarvLO4MIMMUsVNzdfvoQhoybGxjVWaeDieLDmZkOf5oHY5iL18NJTwLO6+eNR9Aqz2wZbPgOJnVYXsxI2tQKfhA= Received: from MWHPR1101MB2302.namprd11.prod.outlook.com (2603:10b6:301:5a::11) by MWHPR11MB1821.namprd11.prod.outlook.com (2603:10b6:300:10f::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4669.13; Fri, 12 Nov 2021 03:28:31 +0000 Received: from MWHPR1101MB2302.namprd11.prod.outlook.com ([fe80::e19a:ca92:7c9a:59e9]) by MWHPR1101MB2302.namprd11.prod.outlook.com ([fe80::e19a:ca92:7c9a:59e9%9]) with mapi id 15.20.4669.016; Fri, 12 Nov 2021 03:28:31 +0000 From: "Gao" To: "Ni, Ray" , "devel@edk2.groups.io" CC: "Dong, Guo" , "Ma, Maurice" , "You, Benjamin" Subject: Re: [PATCH] UefiPayloadPkg/UefiPayloadPkg.fdf: Update DXE A Priori file Thread-Topic: [PATCH] UefiPayloadPkg/UefiPayloadPkg.fdf: Update DXE A Priori file Thread-Index: AQHX1uU0LLHu9pKI/0u1+9JUmeBqUav+WD6AgADj3BA= Date: Fri, 12 Nov 2021 03:28:31 +0000 Message-ID: References: <3948c05d744ceaca2e5e4995120eda1b29a209db.1636622688.git.gao.cheng@intel.com> In-Reply-To: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.6.200.16 dlp-reaction: no-action authentication-results: intel.com; dkim=none (message not signed) header.d=none;intel.com; dmarc=none action=none header.from=intel.com; x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 1197bcae-a28b-4c6a-47a1-08d9a58c8083 x-ms-traffictypediagnostic: MWHPR11MB1821: x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:431; x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: 9EZkd4eTt8/lVafKC0zd+BMRDLYWEyVMuwDq/UtuYrGFmBdrgaiy+xxievFDKusOU5CNLHVwsgTzdxydJIuOzfwVcCBH6aTCHFodc2nioVXlApeDrtHup++oVMK71O2eIm/TBRSx4+FZw/oygonWo06hTiwD6ywhRIrexiG/pGOUtPKoI1tJbE/DdP+y6lnbiyKbyQTnJbyIETRyZuZAVCQ/7nKT4sWtwW2SSZcUx6gEG8SFhrZli1BBwk/rBB3KxhIS/r+oPTw1VJ4rQhRJHMVw4pxJ3LdIAWWWwj2un+WrOkX2kHjDQSd4QLvBwWtsiYZRcgLXOtOrpZ4Q8rQtlQWHZIPGh4l22JIzyo9yXNE7YznGyI07e8XXkirZrdsCAdg5vnPSA/dzDjANrND+Z/xLzsOFknvOnQcCIzj268XhtPNSVVlzmXfit3C3LHrFWKpqZGrV2BRm+RH7n/CIV2fDGb9QC4Sos3UcATKWGTDQtM7xcOBeGCUeLZQ4OuRepqbmUx5DtJVavNlElQlfd9CytIPt1KMxRIer7WW44e8vTlm+ZWBA5wsFNJX6jKSrTKRMh67gjRVXmbNCyRJSV5jEQzEx6nHOPVI5cBiFTDU2xOi4BuXxdDi9Z2EkTBr7x+FbSR4o9A/EENXHRcbg8ZuVdp1pNeYVc79EWwrpikk0HlDGu0EI+nkBs9Y18lUoOTbK09wDVgXAQbP8cU6J5WgR39eMW+A9i/XI062XnR5PneEp5azKrg2nv4kLSRYfWVQsZk8wt9AisWZbqUD5au/m3Iu5tw1dEYmpgbtXGp0= x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:MWHPR1101MB2302.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(366004)(86362001)(8936002)(2906002)(9686003)(38070700005)(966005)(71200400001)(107886003)(33656002)(54906003)(316002)(52536014)(38100700002)(7696005)(82960400001)(110136005)(15650500001)(53546011)(122000001)(186003)(26005)(5660300002)(4326008)(6506007)(83380400001)(64756008)(66946007)(66556008)(66446008)(66476007)(76116006)(8676002)(55016002)(508600001);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?dUsDP3f96xkxQvDJrngWyVb8E9rwbyB5i/PgnP5UxWR8zWyYOvsA76iQL8Gc?= =?us-ascii?Q?EQ3EN8CfIN2WawW3vf0IOC1R+e4E1xtFpEhtAe0S+i4u4wDrdA2dXOHRFywS?= =?us-ascii?Q?u0W058Hlo3/SeAGrPslyTyFBro0n69ssjFWCQmu9BA+M/ed9t8FcXoQreNY2?= =?us-ascii?Q?E8cetVMKzKtK4JlAhmC78yj+UTei2wOu0nbVUPtqEZfIlTPGM+liZcI6TND4?= =?us-ascii?Q?q8jw07MGFXZh1zBP9BAeRCha0zG2RlIB4NQEqXR8IIZ22DoU+rHg+ZFeJTOK?= =?us-ascii?Q?odgI36+RAZ5M+A4n3i9578a0lRSIsAcEj92UurXGg2fksrzPJOD5AhbMulfj?= =?us-ascii?Q?fGU7eYAyuMXEFsPWlRPbNFtu58DVFgjfazB9NTGdwTdsIqrBRIzq/JCvfBB4?= =?us-ascii?Q?8zdRQstykdptFSTbJw4ApF1tFUlqWQKJqvc1StaGl6YYwWrLt8spz31HoSI4?= =?us-ascii?Q?dyo7qDOG+TBl30+9+YOfzqf5WXEtuGSjTwSYipof7+Lq14xEMNbmXST2rZeF?= =?us-ascii?Q?TZicR7K1otKbjBUALEISB78IDo4Buq9Ip3I0No6Z6IROmGjobqM5teiyZZjM?= =?us-ascii?Q?teCL9sHVItqCl8M5kpKckm702CmFueMd0nsOhQhFoRFbEZLm+5Qiki9himXP?= =?us-ascii?Q?N8rNnEGF+k9nlIcyyVQqbCeDsRSVsExMe+AL5d/vHruWslm7+ZqSYb6HhyyA?= =?us-ascii?Q?RY/xTMkaCrOrtH48zG8NfPsNsU0RWXAUa+SZhi3sTq6QSnXvpT8zW/oDNvIW?= =?us-ascii?Q?kfTvqckkQoL8T0kuRbwoe+i+cxkVS3+WtoQsW95lWH8b5jSTDDu2dzjca7ku?= =?us-ascii?Q?r5jJ/6BRq6f+c0W5rk4VVs2VOjwUan1+NFWBox3vTYHutjGFssOXHOnSrWQE?= =?us-ascii?Q?5rHkfEJt4K61u29s17ddrER9zKHhtI/emNb4PYxsepJ4BIyGdbUtxhpEBYxE?= =?us-ascii?Q?yqQjnQPGqtOsf6Jy7kshycXZj4lbptRPR9pEK+cFDRWCD2mput3/YIwMdTI+?= =?us-ascii?Q?kljO78peVmfTDLbRwIxo8bIAnf1g6tg8glcZGVYc6w+ENkSa7amyUZCaf3Mf?= =?us-ascii?Q?uyVABMSw3I1g2+ViUvD+dCB72LYjiuyrjHZYoLHhzU8rH55P5guJrS9zbD7F?= =?us-ascii?Q?1fScAEuMq2Li1P/5BXQTCf043JxoT/lUF/B14b4+Zb3yCx1djgUy3plrednc?= =?us-ascii?Q?mPjlHBqwRc8S4hPkzbK6ugLoi6sxUOVTlJW0VdMmNZu1tkPi3v+N5g4uXn3t?= =?us-ascii?Q?D2yJVm28Kn0ypM/vbHU3fXl0JoU08XX5/8wzSStua+F8Np7AeFxO3gbRcLEC?= =?us-ascii?Q?3dlYNPn9YGdN+GTKgsWIQPUWM5NEtjCmfTq5I3ExG2QjQNTqWkJZRAUHam4f?= =?us-ascii?Q?k7K8UoNrQHWPpGOxOl7E4u0PcDfs4lYlQzEMc/kgd0+ejRrJPo6f9yvX2y++?= =?us-ascii?Q?7FoIbJSih0tDeq3SlbKHUc2d4ub0dDb5JRrI4jGobXSwT/9IloQNAk446VIe?= =?us-ascii?Q?20Qz1v44hZ1rZN9esaY8TvF9n2fiRXeUydNAq6qGWB8s2HzH3IlD6LTIJkAs?= =?us-ascii?Q?rmNqfTqMEODm2LEDbKRsupsDM6SVyMkYBu2rwpFrEjGu7wVeGS/baZOgRPqM?= =?us-ascii?Q?1w=3D=3D?= MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: MWHPR1101MB2302.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 1197bcae-a28b-4c6a-47a1-08d9a58c8083 X-MS-Exchange-CrossTenant-originalarrivaltime: 12 Nov 2021 03:28:31.7718 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: M8FsMu/43fQ/+uouGvwqZ7A8vhK7NG4VnKf23zyGjtwE3TGa+M+kkhYNYdU/VKF97HK8n7676CxZKSEkvjxD4A== X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR11MB1821 Return-Path: gao.cheng@intel.com X-OriginatorOrg: intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Hi Ray, Thanks for pointing this out. That makes sense. I should make the commit message from community perspective instead of plat= form. I will make this change. Best Regards, Gao -----Original Message----- From: Ni, Ray =20 Sent: Thursday, November 11, 2021 9:50 PM To: Cheng, Gao ; devel@edk2.groups.io Cc: Dong, Guo ; Ma, Maurice ; You= , Benjamin Subject: RE: [PATCH] UefiPayloadPkg/UefiPayloadPkg.fdf: Update DXE A Priori= file Gao, The patch content looks good to me. However the commit message is misleading. " UefiPayloadPkg will override platform's DXE A Priori file and miss these = two modules in its DXE A Priori file, so just add them back." Apriori file decides the dispatch order for each FV. I don't think the UefiPayloadPkg will override platform's DXE apriori file. How about following? "Put ReportStatusCodeRouterRuntimeDxe and StatusCodeHandlerRuntimeDxe in Ap= riori list to enable DEBUG() through PeiDxeDebugLibReportStatusCode as earl= y as possible". -----Original Message----- From: Cheng, Gao Sent: Thursday, November 11, 2021 6:16 PM To: devel@edk2.groups.io Cc: Cheng, Gao ; Dong, Guo ; Ni, R= ay ; Ma, Maurice ; You, Benjamin Subject: [PATCH] UefiPayloadPkg/UefiPayloadPkg.fdf: Update DXE A Priori fil= e REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D3729 Platform desires to load ReportStatusCodeRouterRuntimeDxe and StatusCodeHan= dlerRuntimeDxe early in DXE to make DEBUG() via serial port work thereafter= since DXE will use PeiDxeDebugLibReportStatusCode as default DebugLib inst= ance. However, with UefiPayload enabled by default, UefiPayloadPkg will ove= rride platform's DXE A Priori file and miss these two modules in its DXE A = Priori file, so just add them back. Signed-off-by: Gao Cheng Cc: Guo Dong Cc: Ray Ni Cc: Maurice Ma Cc: Benjamin You Cc: devel@edk2.groups.io --- UefiPayloadPkg/UefiPayloadPkg.fdf | 2 ++ 1 file changed, 2 insertions(+) diff --git a/UefiPayloadPkg/UefiPayloadPkg.fdf b/UefiPayloadPkg/UefiPayload= Pkg.fdf index 2f5cb17f62..dd2dc1bf56 100644 --- a/UefiPayloadPkg/UefiPayloadPkg.fdf +++ b/UefiPayloadPkg/UefiPayloadPkg.fdf @@ -87,6 +87,8 @@ APRIORI DXE { INF MdeModulePkg/Universal/DevicePathDxe/DevicePathDxe.inf INF MdeModulePkg/Universal/PCD/Dxe/Pcd.inf INF UefiPayloadPkg/BlSupportDxe/BlSupportDxe.inf + INF =20 + MdeModulePkg/Universal/ReportStatusCodeRouter/RuntimeDxe/ReportStatusC + odeRouterRuntimeDxe.inf + INF =20 + MdeModulePkg/Universal/StatusCodeHandler/RuntimeDxe/StatusCodeHandlerR + untimeDxe.inf } =20 # -- 2.26.2.windows.1