[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 10/15] swiotlb: add a SWIOTLB_ANY flag to lift the low memory restriction
- To: Christoph Hellwig <hch@xxxxxx>
- From: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
- Date: Wed, 6 Apr 2022 20:25:32 -0400
- Arc-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=none
- 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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=4VBD+xTDrv7ic3fHzrAugnoyxmDzOdFCwThtcFLyfUM=; b=ASj8M2afa9KH38oyc8YA+wYn2gMFKB7/WngAY0QRBO4ssyuJTcTD3FXKtZd0HdhNoN9zWxGTguRBiEn9qbkJigrG+UvHD6BsDs6MwlDtLTaxgOBmYxA9tmI2arXhFUL+mgKBVpnzGWbZLnl5D+ZbZsMbCfmDwK5gnH7hoY59WnOXCctuwOUit38MvzFqDTPLsPBUtXQpMQoZhMONxlNlyyaIQi5Ty7J85dXAd4JlWl4RDLqO/0wmhROKpFYOY4LInMJv500hnEJDQFqtiUoq5UzCV661YedujdTC+7xszqTLKdZ16mPC3+St4tTvR+pFqNtWkO4IRNyvOF9IqwZe1Q==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=EHUqu8wMc+5BNgLOSG5XOMsN8ev3KbNVUoIQ3Gv9x9XTBePvvYtSmol0aJBBHeN1zzPHzv6e56R7O3a0dukSuCNRkSPE0wyHEGxGf/Yhfe3n5mnCzrF0xMKowQA+Wc+7+VGuyyKGrq3gzcuMhUzKS+nL8OS+0QlDVwhNVZGcc4PzZ11UrcTfW0EXzF8oMaBvWFZrMyUq0ORfm5h+gHN3bSpbLpIXbmvUglB/xUEKXSuTHosBOjMAIx2w0MxJRKz5XGzwR6FCCGf0eM6f0KnKLF4pDNF0EM/WIyoMJrxSl2Qeh85LMWW+NV8c72dgp4jKumw5RceE+JzuFmTHRXRZsw==
- Cc: iommu@xxxxxxxxxxxxxxxxxxxxxxxxxx, x86@xxxxxxxxxx, Anshuman Khandual <anshuman.khandual@xxxxxxx>, Tom Lendacky <thomas.lendacky@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Boris Ostrovsky <boris.ostrovsky@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@xxxxxxxxxxxxxxx
- Delivery-date: Thu, 07 Apr 2022 00:26:22 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
> diff --git a/arch/powerpc/platforms/pseries/svm.c
> b/arch/powerpc/platforms/pseries/svm.c
> index c5228f4969eb2..3b4045d508ec8 100644
> --- a/arch/powerpc/platforms/pseries/svm.c
> +++ b/arch/powerpc/platforms/pseries/svm.c
> @@ -28,7 +28,7 @@ static int __init init_svm(void)
> * need to use the SWIOTLB buffer for DMA even if dma_capable() says
> * otherwise.
> */
> - swiotlb_force = SWIOTLB_FORCE;
> + ppc_swiotlb_flags |= SWIOTLB_ANY | SWIOTLB_FORCE;
This is the only place you set the ppc_swiotlb_flags.. so I wonder why
the '|=' instead of just '=' ?
Either way:
Reviewed-by: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
Thank you!
|
