Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   "Tian, Kevin" <kevin.tian@intel.com>
To:     Lu Baolu <baolu.lu@linux.intel.com>,
        Joerg Roedel <joro@8bytes.org>,
        "Jason Gunthorpe" <jgg@nvidia.com>,
        Christoph Hellwig <hch@infradead.org>,
        "Raj, Ashok" <ashok.raj@intel.com>, Will Deacon <will@kernel.org>,
        Robin Murphy <robin.murphy@arm.com>,
        Jean-Philippe Brucker <jean-philippe@linaro.com>,
        "Jiang, Dave" <dave.jiang@intel.com>, Vinod Koul <vkoul@kernel.org>
CC:     Eric Auger <eric.auger@redhat.com>,
        "Liu, Yi L" <yi.l.liu@intel.com>,
        "Pan, Jacob jun" <jacob.jun.pan@intel.com>,
        Zhangfei Gao <zhangfei.gao@linaro.org>,
        "Zhu, Tony" <tony.zhu@intel.com>,
        "iommu@lists.linux.dev" <iommu@lists.linux.dev>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "Jean-Philippe Brucker" <jean-philippe@linaro.org>
Subject: RE: [PATCH v10 11/12] iommu: Per-domain I/O page fault handling
Thread-Topic: [PATCH v10 11/12] iommu: Per-domain I/O page fault handling
Thread-Index: AQHYkC3auMAPEOLQ1EqqE1ImiBKIMq1yK8Cw
Date:   Thu, 7 Jul 2022 02:06:02 +0000
Message-ID: <BN9PR11MB52766D609A6AF7AA9C1881AD8C839@BN9PR11MB5276.namprd11.prod.outlook.com>
References: <20220705050710.2887204-1-baolu.lu@linux.intel.com>
 <20220705050710.2887204-12-baolu.lu@linux.intel.com>
In-Reply-To: <20220705050710.2887204-12-baolu.lu@linux.intel.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?R/aec8nbJITG8nujFmcHQOyRSfckj35d5ykFe2qPkq6wzkJrA0o070RJ1ONO?=
 =?us-ascii?Q?crgwrFPP8+iU9v2pV7SrpsE+4KRRHY6rtewTtCUk8NUyMhqtVWloqrTnh0rI?=
 =?us-ascii?Q?1C5BMY2PRH14a+b7rFHYE0gl2E7Zq0G4ERQnQH8ZIwSIq6Oi/12dvHV5EcqF?=
 =?us-ascii?Q?5WV3GtRTvHi6Q7KBTOcDpfG2PfazoE9fXSU2v+lRMxszqJXIXG0nCc8hp5V7?=
 =?us-ascii?Q?+1zZ7AxTptWJcQ95W+tJSxZG9ay6OdQ/w9REGpatCNIgLUhmBGFyrC8BEcYw?=
 =?us-ascii?Q?jOwuoU9LbzP4/YR7V4cf5dsZrUgfTCm0iVSl6xgsVmO1ywVgizAYRGm2c43F?=
 =?us-ascii?Q?yMEVasYt4l3vjKzvFzQe8hj//TjmtV/0t0Q6p1AxuSfD1XrEyMeoRDvoHDHH?=
 =?us-ascii?Q?3VrxkotZp8tbgASHmw1YnXUBkQ47Qy0Ab3pLKBn/DFaFnYgFg9udmPE3eT5D?=
 =?us-ascii?Q?CF74qBmlemrwLI+5iShRAnHhXjT43nyZmrwJ2G3sa4aZ9szbYxmNzH55RQ6U?=
 =?us-ascii?Q?sJELygNQ1IOaaI4NplAm8gGMo/3xeZx4bNxy/y6VWKUG/U1+zJx/BvhnyFot?=
 =?us-ascii?Q?1wV9otSxJLjenENZd6HvTi1oN5CWRhaYhnUQSbD5U+6TwxvfA97SNVKs7Chw?=
 =?us-ascii?Q?X/RElG3V5Hsl7ksZF5wkyy0zv7yGUjWZ9dTzj+sV49WETVs6mlMln2EgBBLw?=
 =?us-ascii?Q?xceq5Hs6PXWyBzT9Bh3V1LZ5Oyeo0EFUpMINBfW1o24cfCdPEKYenqzSETQ1?=
 =?us-ascii?Q?r3MB7+rMcLWuWhC6IjFaa1b4pjsFw3WrD5fwmwh30Jnra5XEp/WueIxPYB4y?=
 =?us-ascii?Q?HIX5biVZeJKapfLSxoBBGWeduVyy4PcGVtJkLGI8PhWFJMgie30R3f2gew2T?=
 =?us-ascii?Q?PgM/2cLaBou6dTVouKhP4/JlCFMbGK1EL4r7VHExkXuY9/wxYs685F4OREix?=
 =?us-ascii?Q?EhjjnIFkLCe6NkpnIvoeDK2EF4m7EonFdE3OkXlMlawpYSQNNfJawjt+Dh1i?=
 =?us-ascii?Q?0zv5tqzh886zn+CmK+M6u0hlTAcTpnQ4Y+RGcLSq1VLrs62j0n2iRT5he7Ow?=
 =?us-ascii?Q?5rWJ2/iGnBfULX+n406lGXBfNDpLThngp3oUO2bY9fWTOHEt7VVxvfWBMxdq?=
 =?us-ascii?Q?sYshv8JmSOqTpjpThEh1Q/KQBlibjuvF9fixvjC4b2SCEH1OMLqACYaXxZXn?=
 =?us-ascii?Q?h1YmGs1nCOO79zuVTJimk8m7J1pRpnHIkzwBmq27ss9TSCMfXk1axy+Sbj57?=
 =?us-ascii?Q?M9Z0DJ6sv5APWYEUnouE/k/PklLEaR6+5Ev2gPQ2tEF6lLJ03S8cFJ1OV7Kn?=
 =?us-ascii?Q?OeeaPRJN0tMfAYVqjx4AQ1JDcqqSka6Ga1NGJCX0HZKYWYfLTwcOBSZU6rhS?=
 =?us-ascii?Q?ZPDqTzySP/CPon7oQD3ePQlkE5lKF90GFVWdYfVTj/71CJMtPKBNntF6imZb?=
 =?us-ascii?Q?K4bBx7CgEP+P0BAsZE+Jte2wxjUU5NT8uWpq/FuOQG0fw0WpcFcqETdud6Rs?=
 =?us-ascii?Q?kMkV3zdULhgMrmfl6H4J1v3jJpS45R5SwxXSlyuq5CCg7+yUItqm+9RrTXku?=
 =?us-ascii?Q?fhLYCUuBt8y62zEptKXabWl2cYxty3RHOCh4JLAn?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: BN9PR11MB5276.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: e6c3f09a-b269-4bb4-d3a0-08da5fbd3e32
X-MS-Exchange-CrossTenant-originalarrivaltime: 07 Jul 2022 02:06:02.2853
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: Rg6np+wZo0DdpasrKOl2QqrdzToilcwiGHeC07N/QgWftMfxkE3aeeskszJed4txQL0cdNscp2grUEqVudywRA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR1101MB2174
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-7.8 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_HI,
        RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,SPF_NONE,
        T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> From: Lu Baolu <baolu.lu@linux.intel.com>
> Sent: Tuesday, July 5, 2022 1:07 PM
>=20
> Tweak the I/O page fault handling framework to route the page faults to
> the domain and call the page fault handler retrieved from the domain.
> This makes the I/O page fault handling framework possible to serve more
> usage scenarios as long as they have an IOMMU domain and install a page
> fault handler in it. Some unused functions are also removed to avoid
> dead code.
>=20
> The iommu_get_domain_for_dev_pasid() which retrieves attached domain
> for a {device, PASID} pair is used. It will be used by the page fault
> handling framework which knows {device, PASID} reported from the iommu
> driver. We have a guarantee that the SVA domain doesn't go away during
> IOPF handling, because unbind() won't free the domain until all the
> pending page requests have been flushed from the pipeline. The drivers
> either call iopf_queue_flush_dev() explicitly, or in stall case, the
> device driver is required to flush all DMAs including stalled
> transactions before calling unbind().
>=20
> This also renames iopf_handle_group() to iopf_handler() to avoid
> confusing.
>=20
> Signed-off-by: Lu Baolu <baolu.lu@linux.intel.com>
> Reviewed-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
> Tested-by: Zhangfei Gao <zhangfei.gao@linaro.org>
> Tested-by: Tony Zhu <tony.zhu@intel.com>

Reviewed-by: Kevin Tian <kevin.tian@intel.com>