From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by mx.groups.io with SMTP id smtpd.web12.58419.1595864463106989281 for ; Mon, 27 Jul 2020 08:41:03 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@intel.onmicrosoft.com header.s=selector2-intel-onmicrosoft-com header.b=lIlhmBkB; spf=pass (domain: intel.com, ip: 134.134.136.65, mailfrom: guo.dong@intel.com) IronPort-SDR: d8Cv2Qr7eke5n27dyqRIl2rrhUUGlQIXU04KIc1q/BNB57ugZLazV477KXb+C8xadoU18xcFLk W3Yw1pkmwnkQ== X-IronPort-AV: E=McAfee;i="6000,8403,9694"; a="151025049" X-IronPort-AV: E=Sophos;i="5.75,402,1589266800"; d="scan'208";a="151025049" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 27 Jul 2020 08:41:02 -0700 IronPort-SDR: hO/+wubh3OmLepJtpBDjj4MzjJkQd3C1JTpdMZ+kbOWHMmJ4fLDkY/UJWStH14bGyAA8nt5g/A lPe3e3K77x9Q== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.75,402,1589266800"; d="scan'208";a="312248818" Received: from orsmsx110.amr.corp.intel.com ([10.22.240.8]) by fmsmga004.fm.intel.com with ESMTP; 27 Jul 2020 08:40:32 -0700 Received: from orsmsx609.amr.corp.intel.com (10.22.229.22) by ORSMSX110.amr.corp.intel.com (10.22.240.8) with Microsoft SMTP Server (TLS) id 14.3.439.0; Mon, 27 Jul 2020 08:40:31 -0700 Received: from orsmsx609.amr.corp.intel.com (10.22.229.22) by ORSMSX609.amr.corp.intel.com (10.22.229.22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1713.5; Mon, 27 Jul 2020 08:40:30 -0700 Received: from ORSEDG002.ED.cps.intel.com (10.7.248.5) by orsmsx609.amr.corp.intel.com (10.22.229.22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256) id 15.1.1713.5 via Frontend Transport; Mon, 27 Jul 2020 08:40:30 -0700 Received: from NAM12-MW2-obe.outbound.protection.outlook.com (104.47.66.49) by edgegateway.intel.com (134.134.137.101) with Microsoft SMTP Server (TLS) id 14.3.439.0; Mon, 27 Jul 2020 08:40:30 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=G/MKUmJMiySJOjVLV65q3bDaR09CUsT5B81/mEUTq4SO1NYuIppO/rn+yYPnKrcdbgg6A5XOaDl0u8uvmgcy17WTHjQ9fN7iDXARvA6osYNpfPKaOAV1hfJ1826a2eoqGFjME+vmOpUutpUK411ApZZwImko9s6CeqKp3sf6k7fJRlTrqJ3Sgg/IGTtiW4Z29BgY8aBWTjhTc3mHaNS9lsmGr7Y9E/tUSzoIf3xmBGR9YZnjCZ0rbGrDRHRk6p6y8cusPhcL6qwqJV+3NhLEsT0AsldjDwXrYeucQc2j62CoaiTCtNwxMtPM+id579lS9mV3XXPiebhZQx+JMABANg== 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-SenderADCheck; bh=JvZCUvC87+TdLHEgy8DYLsy2dYnv9E46cujZTB60h1k=; b=amDuLh3iXcuPwlpJTlxS/nICXIXegPqG2E47vNLiXG4tKLXhf7dt7K3oh67PVvRPGsYZeJzYIeXY5uAJPNeqvfr/wxpUssXFcf2veJtAdY11xwtASWHnszUqUdYQOL5NN99Q68r72ggHbxd+XiI1Lsa4dR6+UK2Olh1hl0FlSS3IxDBFMsBjgGFAR1GHNfEfp1xjXmeoOJbQTTVfL0Iqow0Bkeq5/S5B1gwfCmGD7m9zFH9UR1bNziDtEkry7rMMe5NP8lhvin6XXpfRUT+tLZfLmckFJ5LFquphBP72K+tIEaBeZVfkbBmzsQi/EKmkNs6itjhSFiLPSnUIyQzCyQ== 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=JvZCUvC87+TdLHEgy8DYLsy2dYnv9E46cujZTB60h1k=; b=lIlhmBkBp/eDZzoFEM/cnvnt7tVRfE5ZkH83zh9a5w8eiK89OROFuswJ0pMw4eFRUam/q0Pc0ZCNNKQEiIzpIkfFCBF7i4j9SvwM3AEM0I3IJSA1wRyEsb3Q8t7hpU/V2rCcRWJCFL7LMGDo7MFE9zpAWSCnnNakKgC0ZvKor5U= Received: from BYAPR11MB3622.namprd11.prod.outlook.com (2603:10b6:a03:fe::30) by BYAPR11MB3560.namprd11.prod.outlook.com (2603:10b6:a03:f8::29) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3216.24; Mon, 27 Jul 2020 15:40:29 +0000 Received: from BYAPR11MB3622.namprd11.prod.outlook.com ([fe80::edcc:d9ab:61a6:bd13]) by BYAPR11MB3622.namprd11.prod.outlook.com ([fe80::edcc:d9ab:61a6:bd13%3]) with mapi id 15.20.3216.033; Mon, 27 Jul 2020 15:40:29 +0000 From: "Guo Dong" To: Marcello Sylvester Bauer , "devel@edk2.groups.io" CC: Patrick Rudolph , Christian Walter , "Ma, Maurice" , "You, Benjamin" Subject: Re: [PATCH v4 1/3] UefiPayloadPkg: Store the size of the MMCONF window Thread-Topic: [PATCH v4 1/3] UefiPayloadPkg: Store the size of the MMCONF window Thread-Index: AQHWY+6XVOzk5NaHZkOo7N3vFPN+QKkbkJvA Date: Mon, 27 Jul 2020 15:40:29 +0000 Message-ID: References: <20200727081842.28843-1-marcello.bauer@9elements.com> <20200727081842.28843-2-marcello.bauer@9elements.com> In-Reply-To: <20200727081842.28843-2-marcello.bauer@9elements.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-reaction: no-action dlp-version: 11.2.0.6 authentication-results: 9elements.com; dkim=none (message not signed) header.d=none;9elements.com; dmarc=none action=none header.from=intel.com; x-originating-ip: [68.2.51.172] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 6cc0d8aa-1d51-41a0-3e0b-08d832436441 x-ms-traffictypediagnostic: BYAPR11MB3560: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:1775; x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: 6eKHreckqzkxh/itffL1UTBtuK4ViF3KgMfgXY1JuUgqLwwEI2AijCiLmK4OIHo9Saxr6D6EE94vPL04gGvGu7gAySLM66RDDaJ9EHspR6EP+hOShOAC0oJpqFTKNJCAJS0aqH0J1NXbHdXtFhlqbZwA3/qdTlwGKe/CwyhTnlPJjO8bYITefYqSoBGVGB4W3Sxgdhq1xWo4xdNifnY//yJqlhz5a7xWv+25BzkzMde0lHRVvjHFFSf/P43b4LcC2Gd9RFpOxQnPdk9BlWS5MzSrt8/mIEEPOCXUp/DlxZ9JqHFaa1uUnwzQ/FfB6PR7 x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BYAPR11MB3622.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFTY:;SFS:(4636009)(376002)(396003)(136003)(346002)(366004)(39860400002)(5660300002)(478600001)(4326008)(8936002)(66556008)(107886003)(8676002)(66946007)(64756008)(66446008)(66476007)(76116006)(2906002)(7696005)(33656002)(186003)(26005)(71200400001)(54906003)(6506007)(53546011)(52536014)(55016002)(316002)(9686003)(110136005)(86362001)(83380400001);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata: p8gAPySYmjfTCkSffE4Umx/tpH4A2LjCecRBIwKL0tIHlzNs9+YNlv1v97whXmmEJO4n/tgGAhYM4sXHkAD+iU1Z2ILyDsDymD2utY53R65Od9CEDKD01h6t+UCbpVEtIUI9ta4RU61AIggJS18pkxTC2XzOK25yGLohmJjc7OtZa1GLgjHVTph5i+20cz3SYiWyw0Rz4m3wMvrrPUqKBAC4/yLL5LvfSxT8cRtHSZXH7AxfvMXKcb3VYEKhsG/mM4QuqITDrYRKcswX2JZHqKACjKnj+M2ECjyUO5UfJX0Kd5Oo9xxbdYvGZ0etQG0EEdznYxKyt0z8ksMXsuh2oc+LNdUvWaDEMStVjv4zSLguxq9g85sOKBpJrNxgfBKHtlk6WM33YQS72xqe1vFCEsqRHwVvjaHJosXLVLbkuzEANLcEbAACkXkYNlj4O0/6DyqgcBMRbMX5IKvIEHkvFY2ZFMj6V3gwwss2MtFjwbY= MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: BYAPR11MB3622.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 6cc0d8aa-1d51-41a0-3e0b-08d832436441 X-MS-Exchange-CrossTenant-originalarrivaltime: 27 Jul 2020 15:40:29.7381 (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: RpkUGPcLRsV69hv95SiQIqQ77qph/Hpt2rf9wWOePvGZp0sohm5O6qUZBMidA+duBfaQ46oGGPJ/4kwndFPE8w== X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR11MB3560 Return-Path: guo.dong@intel.com X-OriginatorOrg: intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Reviewed-by: Guo Dong > -----Original Message----- > From: Marcello Sylvester Bauer > Sent: Monday, July 27, 2020 1:19 AM > To: devel@edk2.groups.io > Cc: Patrick Rudolph ; Christian Walter > ; Ma, Maurice ; > Dong, Guo ; You, Benjamin > Subject: [PATCH v4 1/3] UefiPayloadPkg: Store the size of the MMCONF wind= ow >=20 > From: Patrick Rudolph >=20 > Store the real size of the Pcie Memory Mapped Address Space. > This change is necessary to support variable size of MMCONF spaces. >=20 > Signed-off-by: Patrick Rudolph > Signed-off-by: Marcello Sylvester Bauer > Cc: Patrick Rudolph > Cc: Christian Walter > Cc: Maurice Ma > Cc: Guo Dong > Cc: Benjamin You > --- > UefiPayloadPkg/Include/Guid/AcpiBoardInfoGuid.h | 1 + > UefiPayloadPkg/BlSupportPei/BlSupportPei.c | 3 +++ > 2 files changed, 4 insertions(+) >=20 > diff --git a/UefiPayloadPkg/Include/Guid/AcpiBoardInfoGuid.h > b/UefiPayloadPkg/Include/Guid/AcpiBoardInfoGuid.h > index fe783fe5e14c..043b748ae4a9 100644 > --- a/UefiPayloadPkg/Include/Guid/AcpiBoardInfoGuid.h > +++ b/UefiPayloadPkg/Include/Guid/AcpiBoardInfoGuid.h > @@ -24,6 +24,7 @@ typedef struct { > UINT64 PmTimerRegBase; >=20 > UINT64 ResetRegAddress; >=20 > UINT64 PcieBaseAddress; >=20 > + UINT64 PcieBaseSize; >=20 > } ACPI_BOARD_INFO; >=20 >=20 >=20 > #endif >=20 > diff --git a/UefiPayloadPkg/BlSupportPei/BlSupportPei.c > b/UefiPayloadPkg/BlSupportPei/BlSupportPei.c > index 22972453117a..a7e99f9ec6de 100644 > --- a/UefiPayloadPkg/BlSupportPei/BlSupportPei.c > +++ b/UefiPayloadPkg/BlSupportPei/BlSupportPei.c > @@ -240,8 +240,10 @@ Done: > if (MmCfgHdr !=3D NULL) { >=20 > MmCfgBase =3D > (EFI_ACPI_MEMORY_MAPPED_ENHANCED_CONFIGURATION_SPACE_BASE_AD > DRESS_ALLOCATION_STRUCTURE *)((UINT8*) MmCfgHdr + sizeof > (*MmCfgHdr)); >=20 > AcpiBoardInfo->PcieBaseAddress =3D MmCfgBase->BaseAddress; >=20 > + AcpiBoardInfo->PcieBaseSize =3D (MmCfgBase->EndBusNumber + 1 - > MmCfgBase->StartBusNumber) * 4096 * 32 * 8; >=20 > } else { >=20 > AcpiBoardInfo->PcieBaseAddress =3D 0; >=20 > + AcpiBoardInfo->PcieBaseSize =3D 0; >=20 > } >=20 > DEBUG ((DEBUG_INFO, "PmCtrl Reg 0x%lx\n", AcpiBoardInfo- > >PmCtrlRegBase)); >=20 > DEBUG ((DEBUG_INFO, "PmTimer Reg 0x%lx\n", AcpiBoardInfo- > >PmTimerRegBase)); >=20 > @@ -250,6 +252,7 @@ Done: > DEBUG ((DEBUG_INFO, "PmEvt Reg 0x%lx\n", AcpiBoardInfo->PmEvtBase))= ; >=20 > DEBUG ((DEBUG_INFO, "PmGpeEn Reg 0x%lx\n", AcpiBoardInfo- > >PmGpeEnBase)); >=20 > DEBUG ((DEBUG_INFO, "PcieBaseAddr 0x%lx\n", AcpiBoardInfo- > >PcieBaseAddress)); >=20 > + DEBUG ((DEBUG_INFO, "PcieBaseSize 0x%lx\n", AcpiBoardInfo- > >PcieBaseSize)); >=20 >=20 >=20 > // >=20 > // Verify values for proper operation >=20 > -- > 2.27.0