From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from EUR04-VI1-obe.outbound.protection.outlook.com (EUR04-VI1-obe.outbound.protection.outlook.com [40.107.8.70]) by mx.groups.io with SMTP id smtpd.web10.5.1620665002074437429 for ; Mon, 10 May 2021 09:43:22 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@armh.onmicrosoft.com header.s=selector2-armh-onmicrosoft-com header.b=SjHxvxnV; spf=pass (domain: arm.com, ip: 40.107.8.70, mailfrom: samer.el-haj-mahmoud@arm.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=armh.onmicrosoft.com; s=selector2-armh-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=VRc8PurIP0SEJ1toBA7gzgNXcl7bmRlB8FRD7+Q1jk0=; b=SjHxvxnV559vslbuD/nnSM/1arx1HejZ4xopvmfmQXTQ5RBL4rC48hLnT7snWVXSA4CqgST/uogVZm09sYoVuMtxwz6itxZXxaxKpXg2/GUVMBnkdRnbj+QCvk7XVA17PDfLJVjoXKuNAuPv2l9Z41P5PepjCf4ULhQbJvnEbmQ= Received: from AM6P195CA0071.EURP195.PROD.OUTLOOK.COM (2603:10a6:209:87::48) by VI1PR08MB3501.eurprd08.prod.outlook.com (2603:10a6:803:85::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4108.25; Mon, 10 May 2021 16:43:18 +0000 Received: from VE1EUR03FT011.eop-EUR03.prod.protection.outlook.com (2603:10a6:209:87:cafe::7b) by AM6P195CA0071.outlook.office365.com (2603:10a6:209:87::48) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4108.25 via Frontend Transport; Mon, 10 May 2021 16:43:18 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 63.35.35.123) smtp.mailfrom=arm.com; edk2.groups.io; dkim=pass (signature was verified) header.d=armh.onmicrosoft.com;edk2.groups.io; dmarc=pass action=none header.from=arm.com; Received-SPF: Pass (protection.outlook.com: domain of arm.com designates 63.35.35.123 as permitted sender) receiver=protection.outlook.com; client-ip=63.35.35.123; helo=64aa7808-outbound-1.mta.getcheckrecipient.com; Received: from 64aa7808-outbound-1.mta.getcheckrecipient.com (63.35.35.123) by VE1EUR03FT011.mail.protection.outlook.com (10.152.18.134) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4108.25 via Frontend Transport; Mon, 10 May 2021 16:43:18 +0000 Received: ("Tessian outbound 9a5bb9d11315:v91"); Mon, 10 May 2021 16:43:17 +0000 X-CheckRecipientChecked: true X-CR-MTA-CID: 9407622dfde07f0a X-CR-MTA-TID: 64aa7808 Received: from c28c54c6654b.2 by 64aa7808-outbound-1.mta.getcheckrecipient.com id 2D75858C-FD66-421C-A5B2-87AE67C931E3.1; Mon, 10 May 2021 16:43:02 +0000 Received: from EUR05-VI1-obe.outbound.protection.outlook.com by 64aa7808-outbound-1.mta.getcheckrecipient.com with ESMTPS id c28c54c6654b.2 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384); Mon, 10 May 2021 16:43:02 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=XiEMIIh8AFUHwenCuNuK0TtpukopiUJLLcMr1Gg9MZLS/ZKhm4FSdA7ZT//IcLHatvXJwknME/e0ejCUTtYviq4MT7F1XYlxqhk+RNenfTxYvx/5+4q1zeM+kmKkBqvgxNhxQE2TziRlm0J3saqZn/BW8d1Z4FtDk7A1EJChk5jAS0ehMxCalV+26Ya/1oZmQDd8zbPQCF4r93g/19Ql7jRhkTw98YOv7u1yBudTMDpN13oXvAq2LDNknVF8nfkP8rTI6uGRPeZVUk/gTewnKjmLlt2CWNEZDawPSDZ6BMRQxdBcQL32wPk+sHwQ/4Tk4SGUVVmsRxsL+AH+3T7v0A== 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=VRc8PurIP0SEJ1toBA7gzgNXcl7bmRlB8FRD7+Q1jk0=; b=m4KQ/wKL4ZqueOOGkQ8U6/KG8ArwhrmOo7Zd3zMeKxhsnpTbdU3xb0RLg4jO+THQBghMTbhdzEDqkTMIYSH7iaCxsoW8QlrOy1aF1+0j021GDXOzRxZcADbtbnu72y7sPXwDhb92InmDkzBuKF/r7PEaJeRWOHLC+gncNt4MFPes/d+2ZaZhISZhsngsQScNzEwtiRu/y5TbD6apPb2JqdvTmOcq/LyuiEGq/L9rjKRCNo83pqwqpLFkXJGLYVmJpxltBTiT5q20HnboQNU/GPMfC190M6HeMlY/3itU8WLZ1vSq/b2xTlmjDzGfCoMIxtMMlHkU0z1xyEAd4fB2PQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=armh.onmicrosoft.com; s=selector2-armh-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=VRc8PurIP0SEJ1toBA7gzgNXcl7bmRlB8FRD7+Q1jk0=; b=SjHxvxnV559vslbuD/nnSM/1arx1HejZ4xopvmfmQXTQ5RBL4rC48hLnT7snWVXSA4CqgST/uogVZm09sYoVuMtxwz6itxZXxaxKpXg2/GUVMBnkdRnbj+QCvk7XVA17PDfLJVjoXKuNAuPv2l9Z41P5PepjCf4ULhQbJvnEbmQ= Received: from PAXPR08MB6987.eurprd08.prod.outlook.com (2603:10a6:102:1df::15) by PA4PR08MB6192.eurprd08.prod.outlook.com (2603:10a6:102:ea::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4108.25; Mon, 10 May 2021 16:42:59 +0000 Received: from PAXPR08MB6987.eurprd08.prod.outlook.com ([fe80::7c65:78c4:8d03:62ad]) by PAXPR08MB6987.eurprd08.prod.outlook.com ([fe80::7c65:78c4:8d03:62ad%7]) with mapi id 15.20.4108.031; Mon, 10 May 2021 16:42:59 +0000 From: "Samer El-Haj-Mahmoud" To: "devel@edk2.groups.io" , "Andrei Warkentin (awarkentin@vmware.com)" , Jeremy Linton CC: Ard Biesheuvel , "leif@nuviainc.com" , "pete@akeo.ie" Subject: Re: [edk2-devel] [PATCH 3/3] Platform/RaspberryPi/AcpiTables: Correct _DMA consumer Thread-Topic: [edk2-devel] [PATCH 3/3] Platform/RaspberryPi/AcpiTables: Correct _DMA consumer Thread-Index: AQHXLDxK9rrtmosIqEClGyylGxm9MaqqrQyAgAf3JdCAGQOlEIAB1fRggA+gxvA= Date: Mon, 10 May 2021 16:42:59 +0000 Message-ID: References: <20210408055843.2883563-1-jeremy.linton@arm.com>,<20210408055843.2883563-4-jeremy.linton@arm.com> <167578A6C978A75F.9301@groups.io> <167A591C4A42BD1B.6294@groups.io> In-Reply-To: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: Authentication-Results-Original: edk2.groups.io; dkim=none (message not signed) header.d=none;edk2.groups.io; dmarc=none action=none header.from=arm.com; x-originating-ip: [99.132.126.10] x-ms-publictraffictype: Email X-MS-Office365-Filtering-Correlation-Id: 0b2cfef4-a726-453c-69f8-08d913d2b6ef x-ms-traffictypediagnostic: PA4PR08MB6192:|VI1PR08MB3501: x-ld-processed: f34e5979-57d9-4aaa-ad4d-b122a662184d,ExtAddr x-ms-exchange-transport-forked: True X-Microsoft-Antispam-PRVS: x-checkrecipientrouted: true nodisclaimer: true x-ms-oob-tlc-oobclassifiers: OLM:9508;OLM:9508; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam-Untrusted: BCL:0; X-Microsoft-Antispam-Message-Info-Original: mzcZk++oSen7EFrW4Bllywfos8BY0AVr3+lzDAjnPcNGd+GcIXAMtuyWIkQsLcUItiTv/ANCgSLTds+adNS9/Hz/zJCJPM06xChAyNuKE0PHW6UdefRItBCBdsHwSTFAbRu53VeBEVOaDp1fk5G26BOtY+OBxd3jYIjKNkfXW73maej54eJKwLyF/BY3U3PmQfm/uBR/4sZeGUBZKpIZrh/6OJVBRY+kk7VovQ3Pf85AkiKxAWyxfuZ6JfySXZ7ixyQTnv3WeCT3leKfdb3jNp58q+4gASO1r+jS1XzBkIg6qDGABSpep8RhLRQBPnWr8nQO6Q6QpjUk+YiPfcPBum27GJP3NU3M6edLpzZZpIIpxU7uYS+C33bVEJcQbbyyFR5+jI8qfBZzf0GvtvrOFFCCyqs5B/gm9o/bpUbNCdNXQjYHeXlqgxIxgtqt6Z9hml6gegHBWpTcDsfyE1yLDhwgMhBWNxz9lre9j6g864yPj4obrj+vhH2/D18IaCVvOjDx6p3nMI/3RxxcQ5rr+B7tmdnqE0/hA8jVazeE1+bzZyOKTxOWJ39cjH8fc5MlQqc5zSZzqUnZ3SkfFnlo++hlutMLFXhU3UqyvhUuwlCKzbFgIWP4MIPW0bTnqKAxOuC0cDpddJ8kHbKx19OUjsrTKTqWnEUIft9CjbkbiUI6sj8JAnFmpsa8WfG8BRd6vT0Ns0L9LArTgZH0cM5RF3avbFzTAuE8Snd6CGUsats= X-Forefront-Antispam-Report-Untrusted: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:PAXPR08MB6987.eurprd08.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(366004)(136003)(376002)(346002)(396003)(39860400002)(53546011)(6506007)(166002)(6636002)(2906002)(966005)(66476007)(76236003)(66446008)(66556008)(71200400001)(8936002)(83380400001)(7696005)(186003)(38100700002)(5660300002)(122000001)(86362001)(26005)(64756008)(66946007)(76116006)(110136005)(9686003)(478600001)(54906003)(33656002)(4326008)(52536014)(55016002)(8676002)(316002);DIR:OUT;SFP:1101; x-ms-exchange-antispam-messagedata: =?us-ascii?Q?MnhwJ/BLKFIN3NaKSNxNH1yRJi7jmwohdtFBXu2lmy0axvz6jgVCU2BTp/k5?= =?us-ascii?Q?NGmKFoqlKLCQMdytjCUBez8T+MpL2WhzVzI8TxanwmZLteRyWeCf1282GJTT?= =?us-ascii?Q?HiZPqaVfBZq+GjP9zAc38PcOoEoh8pRXtXreaTaZNClxWcBiwmk9FGVdcTjs?= =?us-ascii?Q?RMzu7W8lf2+pBYwciR3f2zH3GrEuO4Iji7ZHogj88YF8O28zU5vRYZb2y+Qd?= =?us-ascii?Q?lb28/elNuaBgi3HPkLOku+ogzE1FWKwexvULzhVBCdc6mJJ8YjnSbbQ/pOj/?= =?us-ascii?Q?9Kxm7kUQ+83Dxit2riWC66+m4VihKLeHjE3lMmLdeIxoDA2T0gZdzmkbOzTO?= =?us-ascii?Q?CCPb/syhCo0phbWuXAmXKQPLZvH+Kgg3i+Z97QF9Oq7xrmefuWZ06AHSP1Rj?= =?us-ascii?Q?uL/jFE279/N+NRgPodvrq7N4R+85v7711MOcs+m/6asGso9bFDZBeBn1nRe/?= =?us-ascii?Q?SW9UgCVXblQpmALKx+ytPh9gHqW1tm7AgUoA54d0FHN96q0YAtJQ67BRtuaQ?= =?us-ascii?Q?bnoTE0+6nPIvTPn5I15KiZ6HkmL/srNgGYvboee/IDtv/JZPTiYC2CyHwe3W?= =?us-ascii?Q?EVMkK00bo1VCjK0AcXKXqXUm6/X1Tzq9CEAnH9NZbIKrVN8HAxwciHTzOwbj?= =?us-ascii?Q?zvw1EoMqzivD4LoM13BLSeLCyNnVTtQopJ24xqoNAAybeIrFPkLV88qBXp7O?= =?us-ascii?Q?sioEur14TwTlRyin72DryMHtfndETpfRFnACY4i7LiJRohfutq3aBELm7Wq0?= =?us-ascii?Q?TNFTNGiNIc5HKndKZqodxReu+M5FrCUcymDXrU5HvgpJzu2umu2nKAfTW1vZ?= =?us-ascii?Q?6y9Iyz0C+oW4VHC8jth8PqQzLhVMNVS0vDcdOvb6xcCEh7qkVNMFZuIIOaHZ?= =?us-ascii?Q?yTj+vxqm6Vjw3Ms0d8zqu30rBp3Jsd5jTErYVAbFKN+pH94vgTU5pC7pXW9i?= =?us-ascii?Q?GUKIPbG8D2miJEsoN9L8QZZKEWrtxZS8kLdffFvFaSV27X29qS+jJgkJEeP0?= =?us-ascii?Q?e9ctVwJcDoJKnHF0CnYkTStk37BS2eqdDGJYBaT/k6jaKdVkcGYAvKtRqzpb?= =?us-ascii?Q?1bIjrT4Q31i4M2Ex9ixw7sQNDQDYJUzGEUjCu4Gn3AVPRKHEw2ZFXAIpFUDf?= =?us-ascii?Q?9NHN68+y04bfKfIMoFrat8EwRHhrrICYF2AvnB3X/h0zltpVOicqKnG6oSEg?= =?us-ascii?Q?cfit3JGYOQ3WCzRj0z8yIPTR91yu8A3cAMc51JAsf05+RDY2rfI2GxVLixFM?= =?us-ascii?Q?0rW0Hj+cd4SWgI3ZWk9CSxykX5PWd7hcvupad01oaPQrwdtJrgtQOWL8Ccs8?= =?us-ascii?Q?GBF5CbMeVuGwO7I1OXzqz6/i?= MIME-Version: 1.0 X-MS-Exchange-Transport-CrossTenantHeadersStamped: PA4PR08MB6192 Original-Authentication-Results: edk2.groups.io; dkim=none (message not signed) header.d=none;edk2.groups.io; dmarc=none action=none header.from=arm.com; Return-Path: Samer.El-Haj-Mahmoud@arm.com X-EOPAttributedMessage: 0 X-MS-Exchange-Transport-CrossTenantHeadersStripped: VE1EUR03FT011.eop-EUR03.prod.protection.outlook.com X-MS-Office365-Filtering-Correlation-Id-Prvs: 6c23d742-fbe9-4e94-c107-08d913d2abb7 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: k8cZVgS879KPOqE/ejCdzxo9xUSDdY7YOX5znXXD2ueKZ4iTxDzwRekZu1Fw16gYcQXDO5U5q8CO6WzxzgC+JxdKKC7TtaLAqZlTBqp6kAvKXqc5kImF4aJ/+/weFW0lQwTL9vrSEhrrBdmdDNIkt8/DEIJSEh2eTM1sZarTivsRfJFyxjPafjwNxeTxrsQXHf1Kpiz6TmirqfAmCG0CqDxf7GmZbeGlegdq2djbzeNX5MWfUCN9NDWaeX6fxiayRkdzPW6OxITDcP8aMgPiQnhAC6gcYwHwtXzlwGmfc/kcU5/xm9UI020FyWYvYmSQrVmyhbwjc/7sFOiBZqXLXNQQQGRxFDMoHas9dtuTkLGo+C0S5j1GXDx2Az+ERkhziyBJoZNY/Oe7/iSj7sOerT/JdFYkGrl+8kW1vvDKZvPyhNS/lY9CVDEF1daZ3yhILy1dBZShgvxz9uqIuCrCnLK5SVw/XDSzYFce830bBGblrHcFCeQjyQnSCWMwoIYuVM/qTNUn6yxf84nKaqysdlKfchCy2ibTjvMOLfNu0Rn+FA3UJAr8XkIy8sto92TYlqSZYwUYd98d//V2Oi613ScgDUMsLOtZGggK9E6gXc0uHeXdij9pzHZRECYQUOHKxIQo9JuY2gTZ3BStOIfk1HNMSEonJoCSvs0azA+TMSavM/BrGr3Qs3UY7HCf/jhc3vzkgiKVe03lcsV72N0WKZjVeAt5eYZFlCZE5y5/en3XKvY3kUfM7jGHWp16FpDuBPBd7mu2fbguVqj1tALaLQ== X-Forefront-Antispam-Report: CIP:63.35.35.123;CTRY:IE;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:64aa7808-outbound-1.mta.getcheckrecipient.com;PTR:ec2-63-35-35-123.eu-west-1.compute.amazonaws.com;CAT:NONE;SFS:(4636009)(376002)(346002)(39860400002)(396003)(136003)(36840700001)(46966006)(30864003)(86362001)(26005)(6636002)(33656002)(76236003)(52536014)(82740400003)(54906003)(8676002)(110136005)(186003)(107886003)(83380400001)(966005)(53546011)(5660300002)(478600001)(70586007)(47076005)(7696005)(4326008)(166002)(82310400003)(356005)(8936002)(9686003)(316002)(36860700001)(6506007)(81166007)(336012)(2906002)(55016002)(70206006);DIR:OUT;SFP:1101; X-OriginatorOrg: arm.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 May 2021 16:43:18.0930 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 0b2cfef4-a726-453c-69f8-08d913d2b6ef X-MS-Exchange-CrossTenant-Id: f34e5979-57d9-4aaa-ad4d-b122a662184d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=f34e5979-57d9-4aaa-ad4d-b122a662184d;Ip=[63.35.35.123];Helo=[64aa7808-outbound-1.mta.getcheckrecipient.com] X-MS-Exchange-CrossTenant-AuthSource: VE1EUR03FT011.eop-EUR03.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR08MB3501 Content-Language: en-US Content-Type: multipart/alternative; boundary="_000_PAXPR08MB6987E2FB02DEF735092D464C90549PAXPR08MB6987eurp_" --_000_PAXPR08MB6987E2FB02DEF735092D464C90549PAXPR08MB6987eurp_ Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable + Ard's new e-mail address From: Samer El-Haj-Mahmoud Sent: Friday, April 30, 2021 2:05 PM To: devel@edk2.groups.io; Samer El-Haj-Mahmoud ; Andrei Warkentin (awarkentin@vmware.com) ; Jere= my Linton Cc: Ard Biesheuvel ; leif@nuviainc.com; pete@akeo.= ie; Samer El-Haj-Mahmoud Subject: RE: [edk2-devel] [PATCH 3/3] Platform/RaspberryPi/AcpiTables: Cor= rect _DMA consumer Update: UEFI Forum ASWG (ACPI spec working group) approved the submitted E= CR as an errata for future ACPI 6.4 spec publication. We can go ahead and proceed with this patch as submitted, based on that EC= R clarification. With that, Reviewed-By: Samer El-Haj-Mahmoud Samer.El-Haj-Mahmoud@arm.com Thanks, --Samer From: devel@edk2.groups.io > On Behalf Of Samer El-Haj-Mahmoud via gro= ups.io Sent: Thursday, April 29, 2021 10:03 AM To: devel@edk2.groups.io; Samer El-Haj-Mahmou= d >; Andr= ei Warkentin (awarkentin@vmware.com) >; Jeremy Linton >; rfc@edk2.groups.io Cc: Ard Biesheuvel >= ; leif@nuviainc.com; pete@akeo.ie; Samer El-Haj-Mahmoud > Subject: Re: [edk2-devel] [PATCH 3/3] Platform/RaspberryPi/AcpiTables: Cor= rect _DMA consumer Any further comments on the ACPI ECR documented in: https://bugzilla.tiano= core.org/show_bug.cgi?id=3D3335 ? I already have comments from Jeremey and Andrew saying it looks good. If t= here are no objections, I will let ASWG know to approve the ECR for future = ACPI spec publication. Thanks, --Samer From: devel@edk2.groups.io > On Behalf Of Samer El-Haj-Mahmoud via gro= ups.io Sent: Tuesday, April 13, 2021 12:45 PM To: Andrei Warkentin (awarkentin@vmware.com)= >; Jeremy Linton >; devel@edk2.groups.io Cc: Ard Biesheuvel >= ; leif@nuviainc.com; pete@akeo.ie; Samer El-Haj-Mahmoud > Subject: Re: [edk2-devel] [PATCH 3/3] Platform/RaspberryPi/AcpiTables: Cor= rect _DMA consumer I just got to this thread. Apologies for the delay. I went through the ACPI spec. Here is what I see: https://uefi.org/specs/ACPI/6.4/19_ASL_Reference/ACPI_Source_Language_Refe= rence.html#qwordmemory-qword-memory-resource-descriptor-macro "ResourceUsage specifies whether the Memory range is consumed by this devi= ce (ResourceConsumer) or passed on to child devices (ResourceProducer). If = nothing is specified, then ResourceConsumer is assumed." https://uefi.org/specs/ACPI/6.4/06_Device_Configuration/Device_Configurati= on.html#dma-direct-memory-access " It specifies the ranges the bus controller (bridge) decodes on the child= -side of its interface. (This is analogous to the _CRS object, which descri= bes the resources that the bus controller decodes on the parent-side of its= interface.) Any ranges described in the resources of a _DMA object can be = used by child devices for DMA or bus master transactions.." The way I read the spec, this wording in the _DMA definition "Any ranges d= escribed in the resources of a _DMA object can be used by child devices.." = suggests that this should be a ResourceProducer, per the QWordMemory resour= ce descriptor definition above The _DMA example in section 6.2.4 uses a "ResourceConsumer", when it shoul= d really be "ResourceProducer" according to these definitions: It describes= , the child devices view of the address range, so the "translation" added = is the CPU's view of the same range. I submitted a "code first" ECR to correct the ACPI spec example (here : ht= tps://bugzilla.tianocore.org/show_bug.cgi?id=3D3335). Please provide feedba= ck on the BZ (or this thread) whether you agree or not, so we can take this= to ASWG/UEFI Forum for discussion and approval Thanks, --Samer From: Andrei Warkentin > Sent: Thursday, April 8, 2021 10:24 AM To: Jeremy Linton >; d= evel@edk2.groups.io Cc: Ard Biesheuvel >= ; leif@nuviainc.com; pete@akeo.ie; Samer El-Haj-Mahmoud > Subject: Re: [PATCH 3/3] Platform/RaspberryPi/AcpiTables: Correct _DMA con= sumer I don't know... the ACPI spec is weird. https://uefi.org/specs/ACPI/6.4/06_Device_Configuration/Device_Configurati= on.html#dma-direct-memory-access ...lists ResourceConsumer for _DMA. A ________________________________ From: Jeremy Linton > Sent: Thursday, April 8, 2021 12:58 AM To: devel@edk2.groups.io > Cc: ard.biesheuvel@arm.com >; leif@nuviainc.com >; pete@akeo.ie >; samer.el-haj-mahmoud@a= rm.com >; Andrei Warkentin >; Jeremy Linton > Subject: [PATCH 3/3] Platform/RaspberryPi/AcpiTables: Correct _DMA consume= r Bridge devices should be marked as producers so that their children can consume the resources. In linux if this isn't true then the translation gets ignored and the DMA values are incorrect. This fixes DMA on all the devices that need a translation. Signed-off-by: Jeremy Linton > --- Platform/RaspberryPi/AcpiTables/Dsdt.asl | 2 +- Platform/RaspberryPi/AcpiTables/Emmc.asl | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Platform/RaspberryPi/AcpiTables/Dsdt.asl b/Platform/Raspberry= Pi/AcpiTables/Dsdt.asl index d116f965e1..32cd5fc9f9 100644 --- a/Platform/RaspberryPi/AcpiTables/Dsdt.asl +++ b/Platform/RaspberryPi/AcpiTables/Dsdt.asl @@ -205,7 +205,7 @@ DefinitionBlock ("Dsdt.aml", "DSDT", 5, "RPIFDN", "RPI= ", 2) // Only the first GB is available. // Bus 0xC0000000 -> CPU 0x00000000. // - QWordMemory (ResourceConsumer, + QWordMemory (ResourceProducer, , MinFixed, MaxFixed, diff --git a/Platform/RaspberryPi/AcpiTables/Emmc.asl b/Platform/Raspberry= Pi/AcpiTables/Emmc.asl index 179dd3ecdb..0fbc2a79ea 100644 --- a/Platform/RaspberryPi/AcpiTables/Emmc.asl +++ b/Platform/RaspberryPi/AcpiTables/Emmc.asl @@ -32,7 +32,7 @@ DefinitionBlock (__FILE__, "SSDT", 5, "RPIFDN", "RPI4EMM= C", 2) } Name (_DMA, ResourceTemplate() { - QWordMemory (ResourceConsumer, + QWordMemory (ResourceProducer, , MinFixed, MaxFixed, -- 2.13.7 IMPORTANT NOTICE: The contents of this email and any attachments are confi= dential and may also be privileged. If you are not the intended recipient, = please notify the sender immediately and do not disclose the contents to an= y other person, use it for any purpose, or store or copy the information in= any medium. Thank you. IMPORTANT NOTICE: The contents of this email and any attachments are confi= dential and may also be privileged. If you are not the intended recipient, = please notify the sender immediately and do not disclose the contents to an= y other person, use it for any purpose, or store or copy the information in= any medium. Thank you. IMPORTANT NOTICE: The contents of this email and any attachments are confi= dential and may also be privileged. If you are not the intended recipient, = please notify the sender immediately and do not disclose the contents to an= y other person, use it for any purpose, or store or copy the information in= any medium. Thank you. --_000_PAXPR08MB6987E2FB02DEF735092D464C90549PAXPR08MB6987eurp_ Content-Type: text/html; charset="us-ascii" Content-Transfer-Encoding: quoted-printable

+ Ard’s new e-mail address

 

 

From: Samer El-Haj-Mahmoud <Samer.El-Haj-= Mahmoud@arm.com>
Sent: Friday, April 30, 2021 2:05 PM
To: devel@edk2.groups.io; Samer El-Haj-Mahmoud <Samer.El-Haj-Mah= moud@arm.com>; Andrei Warkentin (awarkentin@vmware.com) <awarkentin@v= mware.com>; Jeremy Linton <Jeremy.Linton@arm.com>
Cc: Ard Biesheuvel <Ard.Biesheuvel@arm.com>; leif@nuviainc.co= m; pete@akeo.ie; Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com><= br> Subject: RE: [edk2-devel] [PATCH 3/3] Platform/RaspberryPi/AcpiTabl= es: Correct _DMA consumer

 

Update: UEFI Forum ASWG (ACPI spec working group) a= pproved the submitted ECR as an errata for future ACPI 6.4 spec publication= .

 

We can go ahead and proceed with this patch as subm= itted, based on that ECR clarification.

 

With that,

 

Reviewed-By: Samer El-Haj-Mahmoud Samer.El-Haj-Mahmoud@arm.com

 

Thanks,

--Samer

 

 

From: devel@edk2.groups.io <deve= l@edk2.groups.io> On Behalf Of Samer El-Haj-Mahmoud via groups.io
Sent: Thursday, April 29, 2021 10:03 AM
To: devel@edk2.groups.io; Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>; Andrei Warkentin (awarkentin@vmware.com) <awarkentin@vmware.com>= ;; Jeremy Linton <Jeremy.Linton= @arm.com>; rfc@edk2.groups.io
Cc: Ard Biesheuvel <Ar= d.Biesheuvel@arm.com>; leif@nuviainc.com; pete@akeo.ie; Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>
Subject: Re: [edk2-devel] [PATCH 3/3] Platform/RaspberryPi/AcpiTabl= es: Correct _DMA consumer

 

Any further comments on the ACPI ECR documented in:= https://bugzilla.tianocore.org/show_bug.cgi?id=3D3335 ?

 

I already have comments from Jeremey and Andrew say= ing it looks good. If there are no objections, I will let ASWG know to appr= ove the ECR for future ACPI spec publication.

 

Thanks,

--Samer

 

 

 

 

From: devel@edk2.groups.io <deve= l@edk2.groups.io> On Behalf Of Samer El-Haj-Mahmoud via groups.io
Sent: Tuesday, April 13, 2021 12:45 PM
To: Andrei Warkentin (awar= kentin@vmware.com) <awarken= tin@vmware.com>; Jeremy Linton <Jeremy.Linton@arm.com>; devel@edk2.groups.io
Cc: Ard Biesheuvel <Ar= d.Biesheuvel@arm.com>; leif@nuviainc.com; pete@akeo.ie; Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>
Subject: Re: [edk2-devel] [PATCH 3/3] Platform/RaspberryPi/AcpiTabl= es: Correct _DMA consumer

 

I just got to this thread. Apologies for the delay.=

 

I went through the ACPI spec. Here is what I see:

 

https://uefi.org/= specs/ACPI/6.4/19_ASL_Reference/ACPI_Source_Language_Reference.html#qwordme= mory-qword-memory-resource-descriptor-macro

 

“Res=
ourceUsage specifies whether the Memory range is consumed by this device (R=
esourceConsumer) or passed on to child devices (ResourceProducer). If nothi=
ng is specified, then ResourceConsumer is assumed.”=

 =
;
https://uefi.org/specs/ACPI/6.4/06_Device_Configurati=
on/Device_Configuration.html#dma-direct-memory-access=

 
“&nb=
sp;It specifies the ranges the bus controller (bridge) decodes on the child=
-side of its interface. (This is analogous to the _CRS object, which descri=
bes the resources that the bus controller decodes on the parent-side of its=
 interface.) Any ranges described in the resources of a _DMA object can be =
used by child devices for DMA or bus master transactions..”
 =
;
The way I read the spec, this wording in the _DMA defin=
ition “Any ranges described in the resources of a _DMA object can be =
used by child devices..” suggests that this should be a ResourceProdu=
cer, per the QWordMemory resource descriptor definition above
 

The _DMA example in section 6.2.4 uses a “Res= ourceConsumer”, when it should really be “ResourceProducer̶= 1; according to these definitions: It describes , the child devices view of= the address range, so the "translation" added is the CPU's view of the same range.

 
I submitted a “code first” ECR to correct t=
he ACPI spec example (here : https://bugzilla.tianocore.org/show_bug.cgi?id=3D3335). Please provide feedback on the BZ (or this thread) whether you agree o=
r not, so we can take this to ASWG/UEFI Forum for discussion and approval
 
Thanks,
--Samer
 =
;
 =
;

 

From: Andrei Warkentin <awarkentin@vmware.com>
Sent: Thursday, April 8, 2021 10:24 AM
To: Jeremy Linton <Jere= my.Linton@arm.com>; devel@edk2.groups.io
Cc: Ard Biesheuvel <Ar= d.Biesheuvel@arm.com>; leif@nuviainc.com; pete@akeo.ie; Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@arm.com>
Subject: Re: [PATCH 3/3] Platform/RaspberryPi/AcpiTables: Correct _= DMA consumer

 

I don't know... the ACPI spec is weird.

 

 

...lists ResourceConsumer for _DMA.

 

A

 

From: Jeremy Linton <jeremy.linton@arm.com>
Sent: Thursday, April 8, 2021 12:58 AM
To: devel@edk2.groups.io <devel@edk2.groups.io><= br> Cc: ard.biesheuvel@arm.co= m <ard.biesheuvel@arm.com<= /a>>; leif@nuviainc.com <leif@nuviainc.com>; pete@akeo.ie <pete@akeo.ie>; samer.el-haj-mahmoud@arm.c= om <samer.el-haj-mah= moud@arm.com>; Andrei Warkentin <awarkentin@vmware.com>; Jeremy Linton <jeremy.linton@arm.com>= ;
Subject: [PATCH 3/3] Platform/RaspberryPi/AcpiTables: Correct _DMA = consumer

 

Bridge devices shoul= d be marked as producers so that their
children can consume the resources. In linux if this isn't
true then the translation gets ignored and the DMA values
are incorrect. This fixes DMA on all the devices that
need a translation.

Signed-off-by: Jeremy Linton <= jeremy.linton@arm.com>
---
 Platform/RaspberryPi/AcpiTables/Dsdt.asl | 2 +-
 Platform/RaspberryPi/AcpiTables/Emmc.asl | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/Platform/RaspberryPi/AcpiTables/Dsdt.asl b/Platform/Raspberry= Pi/AcpiTables/Dsdt.asl
index d116f965e1..32cd5fc9f9 100644
--- a/Platform/RaspberryPi/AcpiTables/Dsdt.asl
+++ b/Platform/RaspberryPi/AcpiTables/Dsdt.asl
@@ -205,7 +205,7 @@ DefinitionBlock ("Dsdt.aml", "DSDT"= ;, 5, "RPIFDN", "RPI", 2)
         // Only the first GB is a= vailable.

         // Bus 0xC0000000 -> C= PU 0x00000000.

         //

-        QWordMemory (ResourceConsumer,=

+        QWordMemory (ResourceProducer,=

           ,

           MinFixed,

           MaxFixed,

diff --git a/Platform/RaspberryPi/AcpiTables/Emmc.asl b/Platform/Raspberry= Pi/AcpiTables/Emmc.asl
index 179dd3ecdb..0fbc2a79ea 100644
--- a/Platform/RaspberryPi/AcpiTables/Emmc.asl
+++ b/Platform/RaspberryPi/AcpiTables/Emmc.asl
@@ -32,7 +32,7 @@ DefinitionBlock (__FILE__, "SSDT", 5, "RP= IFDN", "RPI4EMMC", 2)
       }

 

       Name (_DMA, ResourceTemplate() {

-        QWordMemory (ResourceConsumer,=

+        QWordMemory (ResourceProducer,=

           ,

           MinFixed,

           MaxFixed,

--
2.13.7

IMPORTANT NOTICE: The contents of this email and an= y attachments are confidential and may also be privileged. If you are not t= he intended recipient, please notify the sender immediately and do not disc= lose the contents to any other person, use it for any purpose, or store or copy the information in any medium. T= hank you.

IMPORTANT NOTICE: The contents of this email and an= y attachments are confidential and may also be privileged. If you are not t= he intended recipient, please notify the sender immediately and do not disc= lose the contents to any other person, use it for any purpose, or store or copy the information in any medium. T= hank you.

IMPORTANT NOTICE: The contents of this email and any attachments are confi= dential and may also be privileged. If you are not the intended recipient, = please notify the sender immediately and do not disclose the contents to an= y other person, use it for any purpose, or store or copy the information in any medium. Thank you. --_000_PAXPR08MB6987E2FB02DEF735092D464C90549PAXPR08MB6987eurp_--