Received: by 2002:a05:6a10:f3d0:0:0:0:0 with SMTP id a16csp264384pxv; Wed, 14 Jul 2021 03:30:13 -0700 (PDT) X-Google-Smtp-Source: ABdhPJw3bNSc3d7bXcEUKmbnjfJDC4FDylCnedcQMd5ei5ZB1Zp4P/q/HJvLdrIHER0U2LlTbwgw X-Received: by 2002:a05:6638:618:: with SMTP id g24mr8393999jar.16.1626258612928; Wed, 14 Jul 2021 03:30:12 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1626258612; cv=none; d=google.com; s=arc-20160816; b=wUSsr5R6Q3XaKifEMr7djUWNScZ3WSvwPDMdRz/JLWvJG5z/S2syLMOj5/w0Lxnbvd dnFIqvpqQ4JT9GKil7hK6BBkBB9sgyV5k8MAZf6zuF8x2Gvb8vKYUaFLe3RU8HOobP3n gZ0UyZoC5IUMmp+5aVDtSez5bH8H7jAlgr+yGADVmKgAVLo3Vw8ABXjMdDgaweKe622W anVLwWXohB9Lbh3sbsRZJ2b7LO4zE8G6Ie33ZLl0rbEm4IU3kZrvz1rwdQSM6isDSG46 HqlL+VJDMProcyW2O+Lp6TJjzvQAMZ7IBXkKGjikwN+KSivcGMdA1bQGo1GLs2VEngrr 9WxQ== 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=JBmJVIriFWjp9aKC3roQLx+hhu6zZtSjygx86nMPs2k=; b=weB3Mg9HDvPq4pLnbheJKBJ9yNN2bpseKEY8F765OSS63Ej1QTgdPLJHEGCLReUnpT rDAKfgC6HDFDun3NBHKc7xUjNv6vPu+VbdV1VS7dqL4l0IeqR5Kpdb1mGHtrvCLHFJPd kBzRLcgONxAYuzKPiF66uq+LP5w+zAiuiINA/FvCeBG5Kbh4F7j/EmYz9+LBv+LWB4wT 4qsK9gPfUVaRpMBua8UUaTNyV907N+XOXoxtSB8xB/dy0h6QZgcdnc13MFL7DXfj1jvf 7yB65VtcHbIDnAY+G8jUQtl0NOSIOczOWEgmzyu1RHHBONAIk0Eir78LvBkNfs7R+4RJ 0NMA== 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; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id c7si2178909iot.72.2021.07.14.03.30.00; Wed, 14 Jul 2021 03:30:12 -0700 (PDT) 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; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=arm.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S238368AbhGNKcL (ORCPT + 99 others); Wed, 14 Jul 2021 06:32:11 -0400 Received: from foss.arm.com ([217.140.110.172]:33100 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S237788AbhGNKcL (ORCPT ); Wed, 14 Jul 2021 06:32:11 -0400 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 51AFAD6E; Wed, 14 Jul 2021 03:29:19 -0700 (PDT) Received: from [10.57.36.240] (unknown [10.57.36.240]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id B4FA73F774; Wed, 14 Jul 2021 03:29:14 -0700 (PDT) Subject: Re: [PATCH v2 0/3] iommu: Enable non-strict DMA on QCom SD/MMC To: Joerg Roedel Cc: Doug Anderson , Ulf Hansson , Linux Doc Mailing List , Peter Zijlstra , linux-pci@vger.kernel.org, Konrad Dybcio , Thierry Reding , Joel Fernandes , Rajat Jain , Will Deacon , Rob Clark , Saravana Kannan , Jonathan Corbet , quic_c_gdjako@quicinc.com, Linux ARM , Viresh Kumar , Veerabhadrarao Badiganti , "Paul E. McKenney" , linux-arm-msm , Bjorn Helgaas , Sonny Rao , Vlastimil Babka , Randy Dunlap , Linux MMC List , Adrian Hunter , LKML , "list@263.net:IOMMU DRIVERS" , Andrew Morton , "Maciej W. Rozycki" References: <20210624171759.4125094-1-dianders@chromium.org> From: Robin Murphy Message-ID: <255adda2-3b5f-b080-4da1-f3c5d5a4f7a6@arm.com> Date: Wed, 14 Jul 2021 11:29:08 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; rv:78.0) Gecko/20100101 Thunderbird/78.11.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-GB Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2021-07-14 11:15, Joerg Roedel wrote: > Hi Robin, > > On Fri, Jul 09, 2021 at 02:56:47PM +0100, Robin Murphy wrote: >> As I mentioned before, conceptually I think this very much belongs in sysfs >> as a user decision. We essentially have 4 levels of "strictness": >> >> 1: DMA domain with bounce pages >> 2: DMA domain >> 3: DMA domain with flush queue >> 4: Identity domain > > Together with reasonable defaults (influenced by compile-time > options) it seems to be a good thing to configure at runtime via > sysfs. > > We already have CONFIG_IOMMU_DEFAULT_PASSTHROUGH, which can probably be > extended to be an option list: > > - CONFIG_IOMMU_DEFAULT_PASSTHROUGH: Trusted devices are identity > mapped > > - CONFIG_IOMMU_DEFAULT_DMA_STRICT: Trusted devices are DMA > mapped with strict flush > behavior on unmap > > - CONFIG_IOMMU_DEFAULT_DMA_LAZY: Trusted devices are DMA mapped > with flush queues for performance Indeed, I got focused on the sysfs angle, but rearranging the Kconfig default that way to match makes a lot of sense, and is another thing which should fall out really easily from my domain type rework, so I'll add that to my branch now before I forget again. > Untrusted devices always get into the DMA domain with bounce pages by > default. > > The defaults can be changed at runtime via sysfs. We already have basic > support for runtime switching of the default domain, so that can be > re-used. As mentioned yesterday, already done! I'm hoping to be able to post the patches next week after some testing :) Cheers, Robin.