From patchwork Fri Apr 22 16:28:58 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Shameerali Kolothum Thodi X-Patchwork-Id: 12823732 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id A7DACC433EF for ; Fri, 22 Apr 2022 16:30:54 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:Subject:CC :To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=7KcLAXTFyV6k3UJZzyVw2STuLDLPCSpvZFWAcb8M90o=; b=e1+EaaKsqXMY4T gCeNn3TD+4q/XivKwnxmEZRsPLbC/pLRwbciId+a2XNoRb6ScqTZ9soIjlmzXchMNQmTMvKS8GkwF l38L6qrooVbCe+X6ONC1NqSgiIfucadMdDAqeOBvN8FZjftqFY6W09k3mpUfcevVMsOtWeC/tfS6n lUFKabeN4JqG2mSh+DUv41zujzPZC17TA5bMLxbAKiq53SsE61HK19jtU36sFoS6dmupUjXSdQWMJ V9xQd8oBVCRwy5DvN3vthnhW966k3eIhuWrT+vAmLEjmicrteEnppS3g2ZWUt3S3rNkATk59ntAPI MD7zbck23+2GRkoy+9zQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nhwAD-001RVc-Ua; Fri, 22 Apr 2022 16:29:46 +0000 Received: from frasgout.his.huawei.com ([185.176.79.56]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nhwA7-001RPk-Ua for linux-arm-kernel@lists.infradead.org; Fri, 22 Apr 2022 16:29:42 +0000 Received: from fraeml737-chm.china.huawei.com (unknown [172.18.147.206]) by frasgout.his.huawei.com (SkyGuard) with ESMTP id 4KlKX71ymKz67tJy; Sat, 23 Apr 2022 00:25:43 +0800 (CST) Received: from lhreml710-chm.china.huawei.com (10.201.108.61) by fraeml737-chm.china.huawei.com (10.206.15.218) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.24; Fri, 22 Apr 2022 18:29:31 +0200 Received: from A2006125610.china.huawei.com (10.202.227.178) by lhreml710-chm.china.huawei.com (10.201.108.61) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.24; Fri, 22 Apr 2022 17:29:23 +0100 From: Shameer Kolothum To: , , CC: , , , , , , , , , , , , Subject: [PATCH v11 0/9] ACPI/IORT: Support for IORT RMR node Date: Fri, 22 Apr 2022 17:28:58 +0100 Message-ID: <20220422162907.1276-1-shameerali.kolothum.thodi@huawei.com> X-Mailer: git-send-email 2.12.0.windows.1 MIME-Version: 1.0 X-Originating-IP: [10.202.227.178] X-ClientProxiedBy: dggems701-chm.china.huawei.com (10.3.19.178) To lhreml710-chm.china.huawei.com (10.201.108.61) X-CFilter-Loop: Reflected X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220422_092940_300537_6DE6D025 X-CRM114-Status: GOOD ( 18.88 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi v9 --> v10  -Addressed Christoph's comments. We now have a callback to struct iommu_resv_region to free all related memory and also dropped the FW specific union and now has a container struct iommu_iort_rmr_data. See patches #1 & #4 -Added R-by from Christoph. -Dropped R-by from Lorenzo for patches #4 & #5 due to the above changes. -Also dropped T-by from Steve and Laurentiu. Many thanks for your test efforts. I have done basic sanity testing on my platform but please give it a try at your end as well. As mentioned in v10, this now has a dependency on the ACPICA header patch here[1]. Please take a look and let me know. Thanks, Shameer [1] https://lore.kernel.org/all/44610361.fMDQidcC6G@kreacher/ From old: We have faced issues with 3408iMR RAID controller cards which fail to boot when SMMU is enabled. This is because these controllers make use of host memory for various caching related purposes and when SMMU is enabled the iMR firmware fails to access these memory regions as there is no mapping for them. IORT RMR provides a way for UEFI to describe and report these memory regions so that the kernel can make a unity mapping for these in SMMU. Change History: v9 --> v10 - Dropped patch #1 ("Add temporary RMR node flag definitions") since the ACPICA header updates patch is now in the mailing list - Based on the suggestion from Christoph, introduced a resv_region_free_fw_data() callback in struct iommu_resv_region and used that to free RMR specific memory allocations. v8 --> v9  - Adressed comments from Robin on interfaces.  - Addressed comments from Lorenzo. v7 --> v8   - Patch #1 has temp definitions for RMR related changes till     the ACPICA header changes are part of kernel.   - No early parsing of RMR node info and is only parsed at the     time of use.   - Changes to the RMR get/put API format compared to the     previous version.   - Support for RMR descriptor shared by multiple stream IDs. v6 --> v7  -fix pointed out by Steve to the SMMUv2 SMR bypass install in patch #8. v5 --> v6 - Addressed comments from Robin & Lorenzo.   : Moved iort_parse_rmr() to acpi_iort_init() from     iort_init_platform_devices().   : Removed use of struct iort_rmr_entry during the initial     parse. Using struct iommu_resv_region instead.   : Report RMR address alignment and overlap errors, but continue.   : Reworked arm_smmu_init_bypass_stes() (patch # 6). - Updated SMMUv2 bypass SMR code. Thanks to Jon N (patch #8). - Set IOMMU protection flags(IOMMU_CACHE, IOMMU_MMIO) based   on Type of RMR region. Suggested by Jon N. v4 --> v5  -Added a fw_data union to struct iommu_resv_region and removed   struct iommu_rmr (Based on comments from Joerg/Robin).  -Added iommu_put_rmrs() to release mem.  -Thanks to Steve for verifying on SMMUv2, but not added the Tested-by   yet because of the above changes. v3 -->v4 -Included the SMMUv2 SMR bypass install changes suggested by  Steve(patch #7) -As per Robin's comments, RMR reserve implementation is now  more generic  (patch #8) and dropped v3 patches 8 and 10. -Rebase to 5.13-rc1 RFC v2 --> v3  -Dropped RFC tag as the ACPICA header changes are now ready to be   part of 5.13[0]. But this series still has a dependency on that patch.  -Added IORT E.b related changes(node flags, _DSM function 5 checks for   PCIe).  -Changed RMR to stream id mapping from M:N to M:1 as per the spec and   discussion here[1].  -Last two patches add support for SMMUv2(Thanks to Jon Nettleton!) Jon Nettleton (1): iommu/arm-smmu: Get associated RMR info and install bypass SMR Shameer Kolothum (8): iommu: Introduce a callback to struct iommu_resv_region ACPI/IORT: Make iort_iommu_msi_get_resv_regions() return void ACPI/IORT: Provide a generic helper to retrieve reserve regions ACPI/IORT: Add support to retrieve IORT RMR reserved regions ACPI/IORT: Add a helper to retrieve RMR info directly iommu/arm-smmu-v3: Introduce strtab init helper iommu/arm-smmu-v3: Refactor arm_smmu_init_bypass_stes() to force bypass iommu/arm-smmu-v3: Get associated RMR info and install bypass STE drivers/acpi/arm64/iort.c | 359 ++++++++++++++++++-- drivers/iommu/arm/arm-smmu-v3/arm-smmu-v3.c | 78 ++++- drivers/iommu/arm/arm-smmu/arm-smmu.c | 52 +++ drivers/iommu/dma-iommu.c | 2 +- drivers/iommu/iommu.c | 16 +- include/linux/acpi_iort.h | 14 +- include/linux/iommu.h | 10 + 7 files changed, 485 insertions(+), 46 deletions(-) Tested-by: Steven Price