Received: by 2002:a05:6a10:8c0a:0:0:0:0 with SMTP id go10csp2410746pxb; Sat, 30 Jan 2021 01:36:33 -0800 (PST) X-Google-Smtp-Source: ABdhPJxMYgXZnkt2bj/1nrOQb7CPnMNZRASBLPRGEKEox8pHYOOoXRe1hkF4aXdC14MptHAn/Sd3 X-Received: by 2002:a17:906:5857:: with SMTP id h23mr8029090ejs.465.1611999393437; Sat, 30 Jan 2021 01:36:33 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1611999393; cv=none; d=google.com; s=arc-20160816; b=aicLSM3uKuX1j1zs80KDvB+gT5v2sPM54GY6pwG2l0m6nwp0aBaL3C1M3lSfewp8tB Q001Nn2+tLOvHUhuSCZDJ+Hg/31I1hcT42dEOi15vW0p5IupDs8K86PWBcFwEUrwpCUA iI2CZztCSR5TX63+bjJUfXFHzrWxCvjuR+IKBByAAso3BQq76JvbBZ84XBWpt0CITMuJ j4XixDB81hzJMclVH4mph9nmM3xvtwv2uXiYIp+EplkbxXU9C0Bzhlt0U1V5mn4F1Vnk Zvup/uMpFtPfHS9szt/QLeABPfDg+fhi+8yd+wYMDZkez+m6jozC9G2/V8Lpd8XUZTZM RCYw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :in-reply-to:mime-version:user-agent:date:message-id:from:references :cc:to:subject; bh=nBm4A2RUrHwGMcfQgqrnQSkOtXFbfVHN2QAIPG3I5kQ=; b=tU3s/oG34tpaki72alq9X3RSssVmxD5am7bGSFDghEPg3/aH3GHAGWkGnPtfVH0ZEK cF+cVuaqugWAlJ0LrtnceMoqwKLdajTZPQ7SWpCji+zzyHb7Uva5dVieB2uxuYkZLf3t t1iBTyRbiYjmgqoRQ5aeJuAzH9+q+cfw3YZdEEr9u0bonu9H1eB7TbNp73XElAoexft/ L9YtRcXrpBfPmDPfncYnL3sTE6lJpbYgmvp/fcz1VzmK8ixH3OTbfu8oH32kiGtemM1V Z/wePH6qpJajbr2/yO+iT3dUrNohpafiOZVchpwkpuZYgC4NdRuLtgFGG1K5vC5LXdWJ 47+g== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id p7si6117949ejf.627.2021.01.30.01.36.09; Sat, 30 Jan 2021 01:36:33 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231686AbhA3Jdi (ORCPT + 99 others); Sat, 30 Jan 2021 04:33:38 -0500 Received: from szxga05-in.huawei.com ([45.249.212.191]:12053 "EHLO szxga05-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230112AbhA3JcK (ORCPT ); Sat, 30 Jan 2021 04:32:10 -0500 Received: from DGGEMS413-HUB.china.huawei.com (unknown [172.30.72.59]) by szxga05-in.huawei.com (SkyGuard) with ESMTP id 4DSTSW2hVVzMS5j; Sat, 30 Jan 2021 17:29:47 +0800 (CST) Received: from [10.174.184.214] (10.174.184.214) by DGGEMS413-HUB.china.huawei.com (10.3.19.213) with Microsoft SMTP Server id 14.3.498.0; Sat, 30 Jan 2021 17:31:15 +0800 Subject: Re: [RFC PATCH v1 1/4] vfio/type1: Add a bitmap to track IOPF mapped pages To: Alex Williamson CC: Cornelia Huck , , , Jean-Philippe Brucker , Eric Auger , Lu Baolu , Kevin Tian , , References: <20210125090402.1429-1-lushenming@huawei.com> <20210125090402.1429-2-lushenming@huawei.com> <20210129155812.384cc56e@omen.home.shazbot.org> From: Shenming Lu Message-ID: <06cacae1-0baa-1533-561b-10abee3efee3@huawei.com> Date: Sat, 30 Jan 2021 17:31:14 +0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101 Thunderbird/78.2.2 MIME-Version: 1.0 In-Reply-To: <20210129155812.384cc56e@omen.home.shazbot.org> Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 7bit X-Originating-IP: [10.174.184.214] X-CFilter-Loop: Reflected Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2021/1/30 6:58, Alex Williamson wrote: > On Mon, 25 Jan 2021 17:03:59 +0800 > Shenming Lu wrote: > >> When IOPF enabled, the pages are pinned and mapped on demand, we add >> a bitmap to track them. >> >> Signed-off-by: Shenming Lu >> --- >> drivers/vfio/vfio_iommu_type1.c | 12 ++++++++++++ >> 1 file changed, 12 insertions(+) >> >> diff --git a/drivers/vfio/vfio_iommu_type1.c b/drivers/vfio/vfio_iommu_type1.c >> index 0b4dedaa9128..f1d4de5ab094 100644 >> --- a/drivers/vfio/vfio_iommu_type1.c >> +++ b/drivers/vfio/vfio_iommu_type1.c >> @@ -95,6 +95,7 @@ struct vfio_dma { >> struct task_struct *task; >> struct rb_root pfn_list; /* Ex-user pinned pfn list */ >> unsigned long *bitmap; >> + unsigned long *iommu_mapped_bitmap; /* IOPF mapped bitmap */ >> }; >> >> struct vfio_group { >> @@ -143,6 +144,8 @@ struct vfio_regions { >> #define DIRTY_BITMAP_PAGES_MAX ((u64)INT_MAX) >> #define DIRTY_BITMAP_SIZE_MAX DIRTY_BITMAP_BYTES(DIRTY_BITMAP_PAGES_MAX) >> >> +#define IOMMU_MAPPED_BITMAP_BYTES(n) DIRTY_BITMAP_BYTES(n) >> + >> static int put_pfn(unsigned long pfn, int prot); >> >> static struct vfio_group *vfio_iommu_find_iommu_group(struct vfio_iommu *iommu, >> @@ -949,6 +952,7 @@ static void vfio_remove_dma(struct vfio_iommu *iommu, struct vfio_dma *dma) >> vfio_unlink_dma(iommu, dma); >> put_task_struct(dma->task); >> vfio_dma_bitmap_free(dma); >> + kfree(dma->iommu_mapped_bitmap); >> kfree(dma); >> iommu->dma_avail++; >> } >> @@ -1354,6 +1358,14 @@ static int vfio_dma_do_map(struct vfio_iommu *iommu, >> goto out_unlock; >> } >> >> + dma->iommu_mapped_bitmap = kvzalloc(IOMMU_MAPPED_BITMAP_BYTES(size / PAGE_SIZE), >> + GFP_KERNEL); > > This is a lot of bloat for all the platforms that don't support this > feature. Thanks, Yes, I will make this dedicated to IOPF. Thanks, Shenming > > Alex > >> + if (!dma->iommu_mapped_bitmap) { >> + ret = -ENOMEM; >> + kfree(dma); >> + goto out_unlock; >> + } >> + >> iommu->dma_avail--; >> dma->iova = iova; >> dma->vaddr = vaddr; > > . >