| [Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
 Re: [PATCH 11/12] swiotlb: merge swiotlb-xen initialization into swiotlb
 
To: Stefano Stabellini <sstabellini@xxxxxxxxxx>,        Christoph Hellwig <hch@xxxxxx>From: Boris Ostrovsky <boris.ostrovsky@xxxxxxxxxx>Date: Wed, 2 Mar 2022 08:15:03 -0500Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oracle.com; dmarc=pass action=none header.from=oracle.com; dkim=pass header.d=oracle.com; arc=noneArc-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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=+H8jEbxRAjH5LJGh19NFnoJv1mnvS0Fd8stoMxTA0w0=; b=YamrgLVX9B/qDq9QzF6II8wlZNthJKbW5iq53COtqYiKrf0HrhczdS3GUOwEHGNuw1wuUXG09BDIRYnB+Rx+5/RfiVASEFCd0JFbNPnPDJtHTEkZzT9fdnQHIUKTA86ouTSqUS7MxVFXQrpCMcUOLCujcX/uKEAbYbZ8fNrtZpUAzVKBXSmcCZNzPoqZelLCVA6se8VX7b0y8Y95bZz+ycb3+82QmPMqtQoTtOdrAI39CoABSuXcfLpcDBW9Ojbiovsz5qyACOrqLkmrWEscr+0V2slJMTUWMb/1R1gdfDs89mwQYFyJO4WY74wYO5ARLaztGM8UpRhz0zvBwWYMJw==Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=kgHvmPUaOKjmTgFDRNyH2JK/nxIxKkD4elHWKwQzktzYLmilMNAijcf33DQQKQboEe07+iBVmL1MGbCLbJZlx3aZXjtbwmFKEoACvzg8YbahcPIE7FxRitBNV+KiWD34U7mYc5clIeXbd9rWfC8ESwhp9s9lbwKIrf8Y6OJ6IRXs566pG2Lsm5mhhsSCpLnEhUTD5vE/5IHLNdArFBlxN097/VAVLhnsR7wqfanCDgr8hU1dlCaZfeJIgOn06RHOqqoct/IJ89PUSA+7xbHF5NIgjdRJU3eyJzapVGbiI9ZIWp3almvhuQy5CTVIPO7S0IhGeFwMYTlKkaaCd0TcNA==Cc: iommu@xxxxxxxxxxxxxxxxxxxxxxxxxx, x86@xxxxxxxxxx,        Anshuman Khandual <anshuman.khandual@xxxxxxx>,        Tom Lendacky <thomas.lendacky@xxxxxxx>,        Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>,        Juergen Gross <jgross@xxxxxxxx>, Joerg Roedel <joro@xxxxxxxxxx>,        David Woodhouse <dwmw2@xxxxxxxxxxxxx>,        Lu Baolu <baolu.lu@xxxxxxxxxxxxxxx>,        Robin Murphy <robin.murphy@xxxxxxx>,        linux-arm-kernel@xxxxxxxxxxxxxxxxxxx, xen-devel@xxxxxxxxxxxxxxxxxxxx,        linux-ia64@xxxxxxxxxxxxxxx, linux-mips@xxxxxxxxxxxxxxx,        linuxppc-dev@xxxxxxxxxxxxxxxx, linux-riscv@xxxxxxxxxxxxxxxxxxx,        linux-s390@xxxxxxxxxxxxxxx, linux-hyperv@xxxxxxxxxxxxxxx,        tboot-devel@xxxxxxxxxxxxxxxxxxxxx, linux-pci@xxxxxxxxxxxxxxxDelivery-date: Wed, 02 Mar 2022 13:16:32 +0000List-id: Xen developer discussion <xen-devel.lists.xenproject.org> 
 
On 3/1/22 9:55 PM, Stefano Stabellini wrote:
 
On Tue, 1 Mar 2022, Christoph Hellwig wrote:
 
Allow to pass a remap argument to the swiotlb initialization functions
to handle the Xen/x86 remap case.  ARM/ARM64 never did any remapping
from xen_swiotlb_fixup, so we don't even need that quirk.
 
 
Aside from that the rest looks OK. Also, you can add my:
Tested-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
 
Not for me, I fail to boot with
[   52.202000] bnxt_en 0000:31:00.0: swiotlb buffer is full (sz: 256 bytes), 
total 0 (slots), used 0 (slots)
(this is iscsi root so I need the NIC).
I bisected it to "x86: remove the IOMMU table infrastructure" but haven't 
actually looked at the code yet.
-boris
 
 |