From nobody Fri Dec 27 03:17:38 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=fail; spf=none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain of lists.01.org) smtp.mailfrom=edk2-devel-bounces@lists.01.org Return-Path: Received: from ml01.01.org (ml01.01.org [198.145.21.10]) by mx.zohomail.com with SMTPS id 1503491023920243.1751712493533; Wed, 23 Aug 2017 05:23:43 -0700 (PDT) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id D099421E7901B; Wed, 23 Aug 2017 05:20:34 -0700 (PDT) Received: from NAM03-CO1-obe.outbound.protection.outlook.com (mail-co1nam03on0085.outbound.protection.outlook.com [104.47.40.85]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id F1F6221E49BA5 for ; Wed, 23 Aug 2017 05:20:30 -0700 (PDT) Received: from brijesh-build-machine.amd.com (165.204.77.1) by DM2PR12MB0155.namprd12.prod.outlook.com (2a01:111:e400:50ce::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.1.1362.18; Wed, 23 Aug 2017 12:23:02 +0000 X-Original-To: edk2-devel@lists.01.org DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amdcloud.onmicrosoft.com; s=selector1-amd-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=phTsufAqtqJAqJH5Yy0QHPEuG/3962+bzhtaL4v2trg=; b=n7dXO3TqSgxEN0lboVBJ2y+32Nyf2nBqaxqk1WrTFdCORNZ6PiPb3jWIwu+V7NTxs3lu8qZynmaedDQH0YtiAHnrV9g1+nPzoKOeKl3h+l76uEnSCOB4Uya0X0kqRIwcsdGwIFBzky7RYWH3nYJyRkfj2qnrr38q4dHvvrRpqAY= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=brijesh.singh@amd.com; From: Brijesh Singh To: edk2-devel@lists.01.org Date: Wed, 23 Aug 2017 08:22:35 -0400 Message-Id: <1503490967-5559-12-git-send-email-brijesh.singh@amd.com> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1503490967-5559-1-git-send-email-brijesh.singh@amd.com> References: <1503490967-5559-1-git-send-email-brijesh.singh@amd.com> MIME-Version: 1.0 X-Originating-IP: [165.204.77.1] X-ClientProxiedBy: DM5PR12CA0069.namprd12.prod.outlook.com (2603:10b6:3:103::31) To DM2PR12MB0155.namprd12.prod.outlook.com (2a01:111:e400:50ce::18) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: ce6124df-b681-445d-79ed-08d4ea21b358 X-MS-Office365-Filtering-HT: Tenant X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(300000500095)(300135000095)(300000501095)(300135300095)(22001)(300000502095)(300135100095)(2017030254152)(48565401081)(300000503095)(300135400095)(201703131423075)(201703031133081)(201702281549075)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095); SRVR:DM2PR12MB0155; X-Microsoft-Exchange-Diagnostics: 1; DM2PR12MB0155; 3:XhDWj9LcSeVQG7gPMb/sthV5gXBwm0kTwxLtLFDyxkNW/ZgOAkVOLRvCDqt5rBOJA7KNJeikjRESijQsg+v990C3fxpkNbpCG68UJyope6OIKdOxYPUO12IhhLVtxv9bJjS6tL4krhfnrbKYnbWhN3zASZfmjoibq25XdYTC1eLhpNPenRODDZxk9pGb3n0S7n/GrRi/HpoRWF14UdiovUut9yhQnzRqU/cg74jgKqpDs7KYK9ZIEP/mfQWXyGdR; 25:I9+4sjHW/0S+ms/nVNrj1juZAz50TvH8kknYiEsjwAMhIQ/g+dHyk+VoVpjtVLEBPnjOwASlmklN9WjC8pYhGL+hpaGlfXQfKaXVqTFMknWnBIlG3R2fiQv7w1Xwsv+sj28LRWJ3lFyCfuhwyOSyCqMdrjhw1ANgnKoInKAdrYeUFxcKN45HEWqfkYefM32751Rov43vwz5/SnVAjb9pami5+/onCx+X449ObhtwCelmZre6NnfKjHfhV+YW8w4VgQIXvhc6/dTED5nfHaMBBS38WEFLoZkAjNpnrqeaDvuqY92GwZQT5+3Olt8cV5xwHe1gnZtKLUqxdXjyfLs0sw==; 31:TiVv42dp3grxkF7FKUnaasd7rEvUhYJsYrL+iZEvv5r5k9tcfOkbcB8D6wb0myC9I18kAyY3mEEHhmIl8bQjRi+SmajZVQjT8cHnvnuseAgB80vScpBshbKspcuWgD5W0Bbr5yBRpmy3NocrpvBM3Kn+P/WeIYoHuA4dUkRFESoupK+gufBZyFPnWtBPSISTatyn87IZhhG4NgB7Sb/3nB518sXfpO7NGJvFgvIcnAU= X-MS-TrafficTypeDiagnostic: DM2PR12MB0155: X-Microsoft-Exchange-Diagnostics: 1; DM2PR12MB0155; 20:KNoXjcTe3iMcrNw7ADxjJ6qE1ijoMNNHjvWqfLc16v8maIle18Ew5B+enq5kZr0bJSicv38OPDNj8u2PVqqhyltvp4y0PKYToWYhAzTK4arl5CclVBaBm6i1nFgaB2IrPrwTDoGF3uvlNE1IgAxfow/AxVZXPcLjtkhwvyAXMXBG4kPiptfiDBodgir+slUxsZ4q5YwO1PwGeMu9ZLNOewFZuexIuYHzN4BC0p78NEJKgy7vCNv2sr5kJZfEEJB9VMXg4Qa4n22gJ9A84WC+qHfsUVTkFDlOdvpXuCcoZZQg6mf+GKczn+gN4SoRLDUU7oB+mUHbd8EL34LNAv6ltrCJZ/sFoWxGxINQFP93Y5ybLrh71wpMai5sNrTxzkjum3tUDCwivr0nluqfpfQAkmf/x2WcsUvLDyI2NtAnaMuSWf5WjzW0QTFRzBruJBRjLFkHgXEotW2MpwviKomjqbEihJABQqy0AJW6xTxG6d9qNbmqGqa2MUYp5WCkMKwh; 4:bNO3EDAlGsaCpLqy2W4T7Vw/PLRZywfHgA9pbhbtX0l80nQdLvUUkuDvji/AlmJ1hj3tboTAiO6sUb93Yxy1F7PFviJZ++0RvuxUYP7zkRUOPLVJH/HZBgDNAdKP+BmjCIkUH/+NIKT3Hj5PXG/ssZDcQUWJBpPd+IC0+P0KHnbS4uGVyQkNQzsXx9YYDiNOIx7aJK3uuElP9/r9wJlnOfcWjZI5GYyO8ZmOLiOSyHRmL7qjbUdd38kEg6Xu3M1i7ppidVcIgu/jVcNqGSC7fOGbqHBAWnbgcbQsIjvTSeHbFI+Q77yFPZb+ycpREZC/I67lhLU+WQ/lY8S3fqt5hQ== X-Exchange-Antispam-Report-Test: UriScan:(767451399110)(228905959029699); X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(601004)(2401047)(5005006)(8121501046)(10201501046)(93006095)(93001095)(100000703101)(100105400095)(3002001)(6055026)(6041248)(20161123555025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123562025)(20161123560025)(20161123564025)(20161123558100)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:DM2PR12MB0155; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:DM2PR12MB0155; X-Forefront-PRVS: 040866B734 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(7370300001)(4630300001)(6009001)(39860400002)(189002)(199003)(305945005)(47776003)(97736004)(101416001)(110136004)(6666003)(189998001)(478600001)(7736002)(3846002)(66066001)(5003940100001)(6116002)(6486002)(48376002)(86362001)(105586002)(2950100002)(4326008)(106356001)(36756003)(6916009)(50466002)(2906002)(50226002)(42186005)(2351001)(33646002)(53416004)(25786009)(7350300001)(81156014)(76176999)(5660300001)(68736007)(54906002)(8676002)(53936002)(50986999)(81166006)(2361001); DIR:OUT; SFP:1101; SCL:1; SRVR:DM2PR12MB0155; H:brijesh-build-machine.amd.com; FPR:; SPF:None; PTR:InfoNoRecords; MX:1; A:1; LANG:en; Received-SPF: none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain of lists.01.org) client-ip=198.145.21.10; envelope-from=edk2-devel-bounces@lists.01.org; helo=ml01.01.org; Received-SPF: None (protection.outlook.com: amd.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; DM2PR12MB0155; 23:PcDHf8vkjQYyTlSuUT1jPI/7PKqE5zZHGsXOwSrLS?= =?us-ascii?Q?+Q+DjRob63T198SuUqhMLj+JSyDHuMMpcWRwq2ZnwMgsScRH2vcK3wRi2O6O?= =?us-ascii?Q?mkDFzB6duVcyf2KmbrZKT9GqAGAj/+n3NVcha4o+0HV2tMV+9BeTvdn2GONI?= =?us-ascii?Q?GKtJaqr6joVGemvLC8vu7xaIw2uA0PdgyzAkGvxW2NSxOs6ljqukQJuGR6fx?= =?us-ascii?Q?zopEu6isAoA8ykEO+C4D/rm5v3sNLS6xV6A38hlCitEq++oLTtb678lJk3OK?= =?us-ascii?Q?7b/gFC4mMRrWFCXGLAjL9bzdCbr42uxZSddM2rxn+bE/FmbtfPOgONewKYE5?= =?us-ascii?Q?f2B2J7velLdn4j/jp1Dwc7877yWZkUHnxl3xbDvhLM2j7OdtE+2SIgxMn5q+?= =?us-ascii?Q?bp/4JSd+9HkzTF7L2rg8wf6cugcDpxO7IXg3Fv8nsB5XoygOUWSnewMlR4+/?= =?us-ascii?Q?O4jGnp72a3tTgaHZMMCp15LASgrx85iyOoPVbOt9+VMb7T7lrCPK81u/oDrY?= =?us-ascii?Q?4nwmFKQlAD44MJfpWYXc5lA1P2RP82jNxwt4cDCYxvCvhBmQYWyFvzUvJD3z?= =?us-ascii?Q?KfZTFyxukKm4Fxw7UTPrNHu6vWIC6CVHcLsPvo2F7xtRT+0VQaOevnjoE0gR?= =?us-ascii?Q?bsKj9zFDKyoZ1KdVOdEdvaudjJ1MKUBjVXCpi8hX9ADlxS1k3yhpVWcs3r+L?= =?us-ascii?Q?DexVazGXI9KUG5D/SDIRl83eFFxfr17sBz04JOSEU3AoBQqb6yLxwehYdm+N?= =?us-ascii?Q?md0tb+ywwZCdGRRxYjb/oTpFXcuWhIYDPTUrRcowW+cKZBR17ZK+fRnnJlWM?= =?us-ascii?Q?1bd+TUmK3cm5v1KBt1wklVaWsrHji59jUT6MJEXgPiAAkuUhXphfcIAfC1y1?= =?us-ascii?Q?nzxqF4iWmILI/qfGbxC8L/vmHLpqTecW3Fgomoe9IXiJ1mO1H2Ct+jkC3LWA?= =?us-ascii?Q?YhOUTeQOkMRaA63zZcR9e88EBf/u4MtHbAN95imNjhUMLfCyPAlVTfSGA3AI?= =?us-ascii?Q?PcytnQC/Jn8m3GXScQNiyamvhBGaDk43ZVLJMTIawQ+iILr82OAab/cUlgH7?= =?us-ascii?Q?YqhkrFCljaqCNS0aY5N5/wyWRduV/x9Jpq5xwuXH+iVE5kmR0hIeM8y+OwFI?= =?us-ascii?Q?0OJ0q2tRz4kxrblcq6tmumYXxUZP5qM?= X-Microsoft-Exchange-Diagnostics: 1; DM2PR12MB0155; 6:NQCfzaYm2ytnFOsrSw2P6B+sSNH8mL3lu/rkg4pwuC8lKUjNaJshgcmFjRX7FgvZh1cNgjAKpLqzKPIsFo/Z7yBZFg2BtdaAQPeK711PiBmzKGPni8UVDeuE8paldrRtoE8DpseX9KpJYKyeuY1Uf608m7AH0CaZOx444uCdTh0sPkxrjNao16MtZuScE7DR27teIz9eBRDiIp5a57JtrwWpsDW+A3JBB3rk9ia2lXXbGCfcF7YL/ieK2K/x1pmcvjbRm1ZMsAL/oRiIa2IDayzMFxnCQOgVYJaOQMioD0haShMUAiPw17IQTTOcyNSD1gfI0SFIekj6KpB1GXUPFg==; 5:0GO/vDsYnCYOVCzlCMoIcjhC78DQs7RLW1YLNmbM/dvHDcaVJsEZEm8O9C+J19RZcNYAVXOMmEPYH0onOyuUPLwpH1VHK7xz+efvx4mi4W4y9yCmmC/EPPMLJI3WGSyIU1gywHPFWEkjxV+osadymA==; 24:7f4CmZMSaaJSAqcMTfdoYirwu3fU4Ss0iKXGJKoonnyPSykxhd0aVbpPyRqPm1WrFI7+FkMEh6WGdGvQLU57GvvrXegxjhBXsuGeRNjCZN0=; 7:Mr3dGecfTFQp7mcQxoyNu9ilw5YyFJDh59EXjK2v/uJBU+HLLa6u3Jdmhriij0BBZz5cotl20+2Wqhv2ABlmibv1e3ps8YzgnbB84QiBe0QnX035uGvH7d4II0lgaZHSB6svacd4MJWNVTcENTF979p481jXi2xcrBYdGl8TxBRt86ATNipkYpYAqzDVz5UrljyQVrJKUesgxfZ3ata0YXs6Cm1fGSqdfHwgfUPngaw= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; DM2PR12MB0155; 20:qYlpldu4Mh1dsrNZ1vDJ6xZAJFMA1l2TgQZtCiyoiOqze5eWzrg63cDoWY2eEzWJks2BdOh42TxgVNxCFEc69Idnu6dJFeHxpz6SJo71IIdtH3twNRPBARZ5KikOV+PZjLLn7BcZd69/aN2LxseDnj1jtLBOWP2au2TL2kI39WNRkcwEzvZfYqbVJyiEaYN/3pq+QfwpmyyK4B+kjrNLhj7VVwYl/HRTtoICynfWxmVqkpBW+yv6HXkiSUIfh36o X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Aug 2017 12:23:02.7947 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM2PR12MB0155 Subject: [edk2] [PATCH v3 11/23] OvmfPkg/VirtioLib: change the parameter of VirtioAppendDesc() to UINT64 X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Jordan Justen , Tom Lendacky , Laszlo Ersek , Ard Biesheuvel Content-Transfer-Encoding: quoted-printable Errors-To: edk2-devel-bounces@lists.01.org Sender: "edk2-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) X-ZohoMail: RDKM_2 RSF_4 Z_629925259 SPT_0 Content-Type: text/plain; charset="utf-8" The patch change the "BufferPhysAddr" parameter of VirtioAppendDesc() from type UINTN to UINT64. UINTN is appropriate as long as we pass system memory references. After the introduction of this feature, that's no longer the case in general. Should we implement "real" IOMMU support at some point, UINTN could break in 32-bit builds of OVMF. Suggested-by: Laszlo Ersek Cc: Ard Biesheuvel Cc: Jordan Justen Cc: Tom Lendacky Cc: Laszlo Ersek Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Brijesh Singh Reviewed-by: Laszlo Ersek --- OvmfPkg/Include/Library/VirtioLib.h | 35 +++++++++--------- OvmfPkg/Library/VirtioLib/VirtioLib.c | 38 ++++++++++---------- 2 files changed, 37 insertions(+), 36 deletions(-) diff --git a/OvmfPkg/Include/Library/VirtioLib.h b/OvmfPkg/Include/Library/= VirtioLib.h index c3e56ea23b89..a966311ac941 100644 --- a/OvmfPkg/Include/Library/VirtioLib.h +++ b/OvmfPkg/Include/Library/VirtioLib.h @@ -151,33 +151,34 @@ VirtioPrepare ( The caller is responsible for initializing *Indices with VirtioPrepare() first. =20 - @param[in,out] Ring The virtio ring to append the buffer to, as a - descriptor. + @param[in,out] Ring The virtio ring to append the buffer t= o, + as a descriptor. =20 - @param[in] BufferPhysAddr (Guest pseudo-physical) start address of the - transmit / receive buffer. + @param[in] BufferDeviceAddress (Bus master device)) start address of = the + transmit / receive buffer. =20 - @param[in] BufferSize Number of bytes to transmit or receive. + @param[in] BufferSize Number of bytes to transmit or receive. =20 - @param[in] Flags A bitmask of VRING_DESC_F_* flags. The caller - computes this mask dependent on further buffe= rs to - append and transfer direction. - VRING_DESC_F_INDIRECT is unsupported. The - VRING_DESC.Next field is always set, but the = host - only interprets it dependent on VRING_DESC_F_= NEXT. + @param[in] Flags A bitmask of VRING_DESC_F_* flags. The + caller computes this mask dependent on + further buffers to append and transfer + direction. VRING_DESC_F_INDIRECT is + unsupported. The VRING_DESC.Next field= is + always set, but the host only interpre= ts + it dependent on VRING_DESC_F_NEXT. =20 - @param[in,out] Indices Indices->HeadDescIdx is not accessed. - On input, Indices->NextDescIdx identifies the= next - descriptor to carry the buffer. On output, - Indices->NextDescIdx is incremented by one, m= odulo - 2^16. + @param[in,out] Indices Indices->HeadDescIdx is not accessed. + On input, Indices->NextDescIdx identif= ies + the next descriptor to carry the buffe= r. + On output, Indices->NextDescIdx is + incremented by one, modulo 2^16. =20 **/ VOID EFIAPI VirtioAppendDesc ( IN OUT VRING *Ring, - IN UINTN BufferPhysAddr, + IN UINT64 BufferDeviceAddress, IN UINT32 BufferSize, IN UINT16 Flags, IN OUT DESC_INDICES *Indices diff --git a/OvmfPkg/Library/VirtioLib/VirtioLib.c b/OvmfPkg/Library/Virtio= Lib/VirtioLib.c index e5366e385f5d..fcd484fffada 100644 --- a/OvmfPkg/Library/VirtioLib/VirtioLib.c +++ b/OvmfPkg/Library/VirtioLib/VirtioLib.c @@ -189,7 +189,6 @@ VirtioPrepare ( Indices->NextDescIdx =3D Indices->HeadDescIdx; } =20 - /** =20 Append a contiguous buffer for transmission / reception via the virtio r= ing. @@ -205,33 +204,34 @@ VirtioPrepare ( The caller is responsible for initializing *Indices with VirtioPrepare() first. =20 - @param[in,out] Ring The virtio ring to append the buffer to, as a - descriptor. + @param[in,out] Ring The virtio ring to append the buffer t= o, + as a descriptor. =20 - @param[in] BufferPhysAddr (Guest pseudo-physical) start address of the - transmit / receive buffer. + @param[in] BufferDeviceAddress (Bus master device)) start address of = the + transmit / receive buffer. =20 - @param[in] BufferSize Number of bytes to transmit or receive. + @param[in] BufferSize Number of bytes to transmit or receive. =20 - @param[in] Flags A bitmask of VRING_DESC_F_* flags. The caller - computes this mask dependent on further buffe= rs to - append and transfer direction. - VRING_DESC_F_INDIRECT is unsupported. The - VRING_DESC.Next field is always set, but the = host - only interprets it dependent on VRING_DESC_F_= NEXT. + @param[in] Flags A bitmask of VRING_DESC_F_* flags. The + caller computes this mask dependent on + further buffers to append and transfer + direction. VRING_DESC_F_INDIRECT is + unsupported. The VRING_DESC.Next field= is + always set, but the host only interpre= ts + it dependent on VRING_DESC_F_NEXT. =20 - @param[in,out] Indices Indices->HeadDescIdx is not accessed. - On input, Indices->NextDescIdx identifies the= next - descriptor to carry the buffer. On output, - Indices->NextDescIdx is incremented by one, m= odulo - 2^16. + @param[in,out] Indices Indices->HeadDescIdx is not accessed. + On input, Indices->NextDescIdx identif= ies + the next descriptor to carry the buffe= r. + On output, Indices->NextDescIdx is + incremented by one, modulo 2^16. =20 **/ VOID EFIAPI VirtioAppendDesc ( IN OUT VRING *Ring, - IN UINTN BufferPhysAddr, + IN UINT64 BufferDeviceAddress, IN UINT32 BufferSize, IN UINT16 Flags, IN OUT DESC_INDICES *Indices @@ -240,7 +240,7 @@ VirtioAppendDesc ( volatile VRING_DESC *Desc; =20 Desc =3D &Ring->Desc[Indices->NextDescIdx++ % Ring->QueueSize]; - Desc->Addr =3D BufferPhysAddr; + Desc->Addr =3D BufferDeviceAddress; Desc->Len =3D BufferSize; Desc->Flags =3D Flags; Desc->Next =3D Indices->NextDescIdx % Ring->QueueSize; --=20 2.7.4 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel