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.web11.40029.1595258542113404422 for ; Mon, 20 Jul 2020 08:22:22 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@intel.onmicrosoft.com header.s=selector2-intel-onmicrosoft-com header.b=OATNMweF; spf=pass (domain: intel.com, ip: 134.134.136.65, mailfrom: maurice.ma@intel.com) IronPort-SDR: a+6fvq+zXYZEn3DzzihZFpbVqNvVDx8fBC50UIuzWOoCeMoe6Ae8MJKSMX+/lAaVZEdzPMDOXM GxT2A4iIenXA== X-IronPort-AV: E=McAfee;i="6000,8403,9688"; a="149916797" X-IronPort-AV: E=Sophos;i="5.75,375,1589266800"; d="scan'208";a="149916797" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 20 Jul 2020 08:22:20 -0700 IronPort-SDR: r3uv6YiP/fFeOmEem0FEeaH1ZWrVddKFuli1JwLht4Y16jWWg3f4ImxKIaHi6hfGeWIjIpNIYM NQi9Xz3dTKEg== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.75,375,1589266800"; d="scan'208";a="270166924" Received: from orsmsx602.amr.corp.intel.com ([10.22.229.15]) by fmsmga007.fm.intel.com with ESMTP; 20 Jul 2020 08:22:20 -0700 Received: from orsmsx609.amr.corp.intel.com (10.22.229.22) by ORSMSX602.amr.corp.intel.com (10.22.229.15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1713.5; Mon, 20 Jul 2020 08:22:19 -0700 Received: from ORSEDG001.ED.cps.intel.com (10.7.248.4) 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, 20 Jul 2020 08:22:19 -0700 Received: from NAM02-SN1-obe.outbound.protection.outlook.com (104.47.36.59) by edgegateway.intel.com (134.134.137.100) with Microsoft SMTP Server (TLS) id 14.3.439.0; Mon, 20 Jul 2020 08:22:19 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Z/at9CWCfk2CoqC/f3k2BNSUipn5MJh/K+JwGtFYr2pe0QGUkZGKrSXZT6D4qP7QBLW96QiECqdgSqgnknolwUMukPQuW8Ry/oIfLMXstlre3QQmLdXLtFjrlItxAFoPS1XlbbPyvg0o0HR6dL5xevenl+aWwOlgmLXeRG8mSQ845zD4fJb048pJ1ngTuBr7l6MFAm8ti3rZFzwwoTGU9xIVwJrSHSlDa6l6czcECvCGslJHrN3KevUISXoaQrqVviX2022s6wtHblUOwYfl3hPeMEzEBKOvpn45EGw0EWMasF0Uc1I2+RJvXqWKcwm8RpIpTRSHNt8Z/9zmmFwd/A== 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=K5tQW5c2tGJD297rTD+zxR318bejSSUKDiVKLBLEUmE=; b=FC2UR16E3jJ9h4G1NXBWaE9rf95fGYaHVZuql81dJ0+IvCH/B8iGdZRXrREvyuQL1QsR82XhQfcUkqiOykN7SWNQVBZff+56AhNpY+dk1dB+iUhoHgmL5D1+p0ioMI5VtTclZN+/6v4MH3t3CygY1HEcgkCBrQNAO1usp96dcGFZYloHEnEUbCIAMGx/PiYRUbRIqqwzdQiJb/QuvHIlaZlgcAtWjFx9pWpgE5ADed4VS4QNw++kI4oAMleKPVdKwJUJAXlv5TntxufSqeq39+jKhgRE4H7J61/n+NZI5plUAM6JZBAKEVjPa2SLynR5XiXHe5vOFrdmmqE4mzCbvg== 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=K5tQW5c2tGJD297rTD+zxR318bejSSUKDiVKLBLEUmE=; b=OATNMweFlB+iy6xw42tsTykvE07Yy30SYedqPFajVmT56bFfQ/RsG204qdqVyWh4EwDkS45gv/h7fRVvEXan/8ibEJVHnD0/9C4cG32ohw5zSghgzleTAVu0BmY8G37ySXm+4i0QmhbE8C769GBC4mS/CbuqEcatjrOJwyFGQKQ= Received: from DM6PR11MB2793.namprd11.prod.outlook.com (2603:10b6:5:c0::29) by DM6PR11MB4508.namprd11.prod.outlook.com (2603:10b6:5:200::32) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3195.23; Mon, 20 Jul 2020 15:22:15 +0000 Received: from DM6PR11MB2793.namprd11.prod.outlook.com ([fe80::d43:5665:5cee:b3c5]) by DM6PR11MB2793.namprd11.prod.outlook.com ([fe80::d43:5665:5cee:b3c5%6]) with mapi id 15.20.3195.025; Mon, 20 Jul 2020 15:22:15 +0000 From: "Ma, Maurice" To: Marcello Sylvester Bauer CC: Patrick Rudolph , Christian Walter , "Desimone, Nathaniel L" , "Zeng, Star" , "Kinney, Michael D" , "Gao, Liming" , "devel@edk2.groups.io" Subject: Re: [PATCH v2 1/2] UefiPayloadPkg: Store the real size of the MMCONF window Thread-Topic: [PATCH v2 1/2] UefiPayloadPkg: Store the real size of the MMCONF window Thread-Index: AQHWW2cUdwxyRbTKwE+aO+E4hQaURakQmrtQ Date: Mon, 20 Jul 2020 15:22:15 +0000 Message-ID: References: <20200716114820.14211-1-marcello.bauer@9elements.com> <20200716114820.14211-2-marcello.bauer@9elements.com> In-Reply-To: <20200716114820.14211-2-marcello.bauer@9elements.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.2.0.6 dlp-reaction: no-action 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: [104.153.200.60] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 2708a666-f3b3-4b6d-98f8-08d82cc0af18 x-ms-traffictypediagnostic: DM6PR11MB4508: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:478; x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: KVQ7xJGBsQgrSdWh577c+ctcDHs3PZV3STUc+rs8je4s/WhZjt2EsXMgFQHFroBWxvnqjb1kzduePscp8kQW3Y8CdCCs6pLshxgDXFc1mKpPhBr1V5JUeOxCHwdOAZB36Swrv/H9Tre/XnbPBBBHQhMoeNp2CsgAYruIWxiujNMcmTEDT1TMMFhNdZPl8K3V8V6UjULhRdVSrILP0/MJijmQRrghzDTThuc0n7SNnPQvNEuo5vLo/2BNkOtSTx3XcFtHZitJc1rZjyPkb5lgl8nWoH1mF41W9/uw/xshieBtGak4Q3chgB6+5Q3fL4dZ x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DM6PR11MB2793.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFTY:;SFS:(4636009)(396003)(346002)(376002)(39860400002)(366004)(136003)(52536014)(2906002)(7696005)(9686003)(8676002)(316002)(5660300002)(83380400001)(55016002)(478600001)(33656002)(54906003)(6916009)(71200400001)(53546011)(4326008)(66556008)(86362001)(26005)(8936002)(76116006)(6506007)(186003)(66476007)(66946007)(64756008)(66446008);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata: q6DFZqAnPg59D4kMyupNCxoS8tUGXNEPKDJRnbvHVfB9IjdmU0cPrMdQZdc65KfKe17/dc0XwGDbNskW/0PYPJBftWnXxMvEzSETbauYt0z3rrhGlVcX4QZkMkKu20Ou5TYqaEBYCqqRwEvApL27G3HvIWm1V8XEjD6kapivu8Hx4q8bngZbo3nUwDQooN9IKOOqU+qE4aP0I+rvJaj0TkRwduP/qn2jzt/W+4I/w58hAolsj2H4pJT8gQnqWhtgvHFvQBq3VyHsaohdN2J+ESwwp/sUqeaYleqDlXkUnV+/4Wk9YKPK40RpWXCYOwFRyBuoCUo97YJQABU3hIrTz9Qeba9KtMgMB5GCRnXksmF1wB2CJq8sUkB6pqhoF34ayTyXl9QD5a5NCytGjYQkn4eU4GxRwL7WFa8Q4S1fX6i9ZwfmpNgZAj8F05jc3VHCZiXgkgg4Ob09GoLkJlfEqaj/PW9l4a9sgHUQ9DrSNWrsl8p4/HDfLIX7Kgw0GFff MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DM6PR11MB2793.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 2708a666-f3b3-4b6d-98f8-08d82cc0af18 X-MS-Exchange-CrossTenant-originalarrivaltime: 20 Jul 2020 15:22:15.4583 (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: O5vK6aSwrJbRFnc5Ct3Fx5pFj1rHwn4cW+jZPYCjQtgzsBLFsaGK26YD3RfiT/fk/umCkM51LRK8+Xc7esx9Hg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR11MB4508 Return-Path: maurice.ma@intel.com X-OriginatorOrg: intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Hi Marcello, Could you put a little bit more details in the patch commit message ? =20 Thanks Maurice > -----Original Message----- > From: Marcello Sylvester Bauer > Sent: Thursday, July 16, 2020 4:48 > To: devel@edk2.groups.io > Cc: Patrick Rudolph ; Christian Walter > ; Ma, Maurice ; > Desimone, Nathaniel L ; Zeng, Star > ; Kinney, Michael D ; Ga= o, > Liming > Subject: [PATCH v2 1/2] UefiPayloadPkg: Store the real size of the MMCONF > window >=20 > From: Patrick Rudolph >=20 > This will fix issues with the PciBusDxe. >=20 > Signed-off-by: Patrick Rudolph > Signed-off-by: Marcello Sylvester Bauer > Cc: Patrick Rudolph > Cc: Christian Walter > Cc: Maurice Ma > Cc: Nate DeSimone > Cc: Star Zeng > Cc: Michael D Kinney > Cc: Liming Gao > --- > 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_A > DDRESS_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