From nobody Sun May 19 21:02:43 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+57954+1787277+3901457@groups.io; arc=fail (BodyHash is different from the expected one); dmarc=fail(p=none dis=none) header.from=nxp.com Received: from web01.groups.io (web01.groups.io [66.175.222.12]) by mx.zohomail.com with SMTPS id 1587638638956313.84929383439385; Thu, 23 Apr 2020 03:43:58 -0700 (PDT) Return-Path: X-Received: by 127.0.0.2 with SMTP id I3jtYY1788612xASnO3djrdM; Thu, 23 Apr 2020 03:43:58 -0700 X-Received: from EUR04-VI1-obe.outbound.protection.outlook.com (EUR04-VI1-obe.outbound.protection.outlook.com [40.107.8.79]) by mx.groups.io with SMTP id smtpd.web11.3832.1587638637335172195 for ; Thu, 23 Apr 2020 03:43:58 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=MyDcNk6eYljXz1Pp6nOdpmQX/TkNJMIoigN4LPkfuKFQ2PkbkX2BnJ7n3Wz1ax87bZNca1vdOuGMWna80GEeH+PaMXmtWxEaNee+RVGecZinGB6rWSh/2sqiy1lanp/XHdKo1p16uE9Xc3eO0g/TtcsL+dL9jm35Lk6ggH/TTYaHVhKgJm4AUkvWAzRJ/FOt9y6zHPA5wwXlrw7w5PM4vSOjwGHAC1r2B1tmopPgdS3occhtcnmn5sPXp1S5oeRLooZqRbklO6VlEUvyvUGIcNV/ck8m/s9G4WwGx0dtmWOQrocQP7pU27Wt7brjBvuHUAxOoiDAkMWCuY6ZUOOtTA== 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=kw4/9bF7ADNzi5xZArZ13UN1KOFRK/h2zgGXrbXPuIE=; b=MbeeSo9rrS3AGpvs2druOAv96+coPd1lvWYFdkCQp3fErpjbMmmu23vrAqM3oT9LMkzpgXI8vGF/RpLnNsIPLjPrthZVEVWJrjxBLzLdkwdJX9A8TG5aLlwp6FQ0RjoJT+VTEFHK8tWkpsBqOnpRaLVesjqUA8+x3JW02PPLlMwKRBiyXrvKWvI82vQTSECts8XPo2N6QZRet2evCUAljmhC5uc6tnBfcxyCopktLfwEH4XNyZHItM2pCMlmOjbh26oN5Px9L2rt2naPmd/ZvcIl7/Llx92dQ4QqC23G1nCwdhdFGzhOEshaHJyxxdGzYac9kw4sIJEUkMTFV26u9Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none X-Received: from VE1PR04MB6702.eurprd04.prod.outlook.com (2603:10a6:803:123::13) by VE1PR04MB6686.eurprd04.prod.outlook.com (2603:10a6:803:125::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2937.13; Thu, 23 Apr 2020 10:43:55 +0000 X-Received: from VE1PR04MB6702.eurprd04.prod.outlook.com ([fe80::c092:8f65:8b04:dd28]) by VE1PR04MB6702.eurprd04.prod.outlook.com ([fe80::c092:8f65:8b04:dd28%5]) with mapi id 15.20.2937.012; Thu, 23 Apr 2020 10:43:55 +0000 From: "Wasim Khan" To: devel@edk2.groups.io Cc: ard.biesheuvel@linaro.org, v.sethi@nxp.com, hao.a.wu@intel.com, ray.ni@intel.com, Wasim Khan Subject: [edk2-devel] [PATCH] MdeModulePkg/PciHostBridge: Update Mem and PMem Limit Checks Date: Thu, 23 Apr 2020 16:13:32 +0530 Message-Id: <1587638612-13056-1-git-send-email-wasim.khan@nxp.com> X-ClientProxiedBy: PN1PR0101CA0048.INDPRD01.PROD.OUTLOOK.COM (2603:1096:c00:c::34) To VE1PR04MB6702.eurprd04.prod.outlook.com (2603:10a6:803:123::13) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-Received: from localhost.localdomain (157.37.82.244) by PN1PR0101CA0048.INDPRD01.PROD.OUTLOOK.COM (2603:1096:c00:c::34) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256) id 15.20.2937.13 via Frontend Transport; Thu, 23 Apr 2020 10:43:52 +0000 X-Originating-IP: [157.37.82.244] X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-HT: Tenant X-MS-Office365-Filtering-Correlation-Id: 4746c75d-07a6-462d-2707-08d7e773383f X-MS-TrafficTypeDiagnostic: VE1PR04MB6686:|VE1PR04MB6686: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:1850; Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.12 as permitted sender) client-ip=66.175.222.12; envelope-from=bounce+27952+57954+1787277+3901457@groups.io; helo=web01.groups.io; Received-SPF: None (protection.outlook.com: nxp.com does not designate permitted sender hosts) X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam-Message-Info: Nknkt3M2+jm1HSCbVoic7s95L3/n2eOBdy+D4bSBfJVKM1nq8bWJoJ12PuYvBmQq5I8jxR+AbpBJJFENvDGEvmZkA0xhF02IqTJ1eXqjMwfLP94eRc+UuCFIpPvjptE3tzylziVrJy9J/n3gZDnjWD4DFYOJNc9z9tMxbk0VeYldfJwK22bFyXFTloBSBnEyj5Fis4iqJ12uzyemywzslAeQHmzleSNtcAO/Gkz9OiX1PCCQ0W79qoLXC8FlUvJ570y3gwjhC2LPYTu+DSmKH7Kw0I5Kf4AxJJ8Bf1vMdK/5eilIC413nM1EhRfUsSQ7It5u/XtGdRj2J4mQGlAZb5ckpt0Zsy+sRCW+0pJKq+wGClKRrfwFd/TAveaU8eCd+4P53vu7H/c5XBwEIAyKclHkQ1UDz8PBhJmTZBhJblYPZnlP+asfEkTUTKZN/xe15tLmv7GrOZpZq62d2XBRVDQSLf9yv6jvQYh9RRa+pZvlcznN0EbqAMS8VDYg0JeP X-MS-Exchange-AntiSpam-MessageData: 4xcwsXHF4BFHipQwy0JZj4h86E/H600YJpJHCzaKwUYi/u4Zkb7we+cb9W79g3BovHSyJJbA8a7JGl0v5eYEuOyXUVAINVPHq73/GsWS8XNxBZwDWSUAKpEerg9Qn1lRhrpRm6oyJTK6EM+sTwfCew== X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 4746c75d-07a6-462d-2707-08d7e773383f X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Apr 2020 10:43:54.9231 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: NUnwkq9QGThYuWTDUskRWeyFkAaqiA/O0gtS+SqQlc3tsEaHO7SMoSetl7LEyR+rCBCtvA89wD0rUzEnzDhJ4g== X-MS-Exchange-Transport-CrossTenantHeadersStamped: VE1PR04MB6686 Precedence: Bulk List-Unsubscribe: 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,wasim.khan@nxp.com X-Gm-Message-State: mLscqrdMSewVUA8EdTIFH1yBx1787277AA= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1587638638; bh=loeuMPWxmp7IfVGPSp6SpmYjh6XJ1p9bWwLB9q6BeeI=; h=Cc:Content-Type:Date:From:Reply-To:Subject:To; b=f4exxNomlSU6ASMNWy18GDTwaEx1Mg+maeuREEYAUu70jKebaxGCDssVfEbtL4tIhg5 prGFOYX00hl4Ed0faD9swyDzCTmbXx1favQGW+LPtvIMH04S8QmqfRpIwXowf3d6Jm/vp ucGmVPiU6XDBv5iqg6/06KwlIdWzeJeMEVw= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" With Address Translation Support, it is possible and also correct that Mem and Pmem Limit cross the 4GB boundary. Update the checks so that Mem/PMem Limit should not cross 4GB from the Mem/PMem Base address. Signed-off-by: Wasim Khan --- MdeModulePkg/Bus/Pci/PciHostBridgeDxe/PciRootBridgeIo.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/MdeModulePkg/Bus/Pci/PciHostBridgeDxe/PciRootBridgeIo.c b/MdeM= odulePkg/Bus/Pci/PciHostBridgeDxe/PciRootBridgeIo.c index d304fae..9cf7e98 100644 --- a/MdeModulePkg/Bus/Pci/PciHostBridgeDxe/PciRootBridgeIo.c +++ b/MdeModulePkg/Bus/Pci/PciHostBridgeDxe/PciRootBridgeIo.c @@ -117,8 +117,8 @@ CreateRootBridge ( // Make sure Mem and MemAbove4G apertures are valid // if (RESOURCE_VALID (&Bridge->Mem)) { - ASSERT (Bridge->Mem.Limit < SIZE_4GB); - if (Bridge->Mem.Limit >=3D SIZE_4GB) { + ASSERT (Bridge->Mem.Limit < (Bridge->Mem.Base + SIZE_4GB)); + if (Bridge->Mem.Limit >=3D (Bridge->Mem.Base + SIZE_4GB)) { return NULL; } } @@ -129,8 +129,8 @@ CreateRootBridge ( } } if (RESOURCE_VALID (&Bridge->PMem)) { - ASSERT (Bridge->PMem.Limit < SIZE_4GB); - if (Bridge->PMem.Limit >=3D SIZE_4GB) { + ASSERT (Bridge->PMem.Limit < (Bridge->PMem.Base + SIZE_4GB)); + if (Bridge->PMem.Limit >=3D (Bridge->PMem.Base + SIZE_4GB)) { return NULL; } } --=20 2.7.4 -=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 (#57954): https://edk2.groups.io/g/devel/message/57954 Mute This Topic: https://groups.io/mt/73215737/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-