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 04/12] iommu: Add attach/detach_dev_pasid iommu
 interface
Thread-Topic: [PATCH v10 04/12] iommu: Add attach/detach_dev_pasid iommu
 interface
Thread-Index: AQHYkC3GvF//WyZOvUaTLdl04LC2Cq1yJ63Q
Date:   Thu, 7 Jul 2022 01:51:28 +0000
Message-ID: <BN9PR11MB5276BBE3580F689BE4548F978C839@BN9PR11MB5276.namprd11.prod.outlook.com>
References: <20220705050710.2887204-1-baolu.lu@linux.intel.com>
 <20220705050710.2887204-5-baolu.lu@linux.intel.com>
In-Reply-To: <20220705050710.2887204-5-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?hridtA7Ld1hcwOLhUrhMJIQsTCMYpyWvROzcoXRXum+8MjkeuimfEoQptNGD?=
 =?us-ascii?Q?24EJwvRCT7a7Ha5+hyqvxa5CCpM1Mstsl2hORJTBTD2DSRLiWkHr8lUWls29?=
 =?us-ascii?Q?Cxf8y7PP2hY3L/tEJeoXqXFid2B+qlbndAQHEI5RVYb5WpOezPK2XtKIFGCz?=
 =?us-ascii?Q?a6Ym8DhsP+BbMZtIkLnvbOQgq6Ku68Xk3Bz55Ki6oZONEaI0CDWdiEsaURd0?=
 =?us-ascii?Q?vpDkBn6mBJmfApRjqGVSYG3CNq1eBCtJGG+zFu2PQY6jd7o+RZvMYraQOaJO?=
 =?us-ascii?Q?mx+4rq5zaf2+++MqeIW3tRhH6UgQufEuOqet1K03FOptcWgYiPOQtyotjTTG?=
 =?us-ascii?Q?mHA/fRfd1MhmG2rIm3qbM9e5X9JdyFUQhRAIQwQ4T5jaNIXEGI9JC4b2E3Ea?=
 =?us-ascii?Q?AAvJt+JK7TjOVZaTMCMAneXWr7JbsdKfX6KJTSFfh9i+0JOrJdTESF7+0Hvv?=
 =?us-ascii?Q?1xHK2HH/4MuiVsUHBirxzZAMYHGvGGitc7DNNyquPzYR2fUspUgAXlsr4VJx?=
 =?us-ascii?Q?ZrJhKK1QM4FVD+E6rCUq9OUMHirqKRFtiP9WI+FaXu45hFZ5XchH3xJ6/D9L?=
 =?us-ascii?Q?picumck6hFTXbSlYS2l8uz/nSWs+4/5cpJtPOzruek43/A9eX/1hQvnK19TI?=
 =?us-ascii?Q?r9KZT+AVqoytzTSM39qll1ouvzg1lBg9oi3U6ZizRd/5b2gsPCLwmhMLQ+Ct?=
 =?us-ascii?Q?5lEww/WeVS+zCp4EgLZUmg5d1PWu9ukhcygyiYt8qNe+HmHaYWJpUonwtljz?=
 =?us-ascii?Q?LPy+a47TLv7AEzJ9ckiY6wIFDbbreala4FFGjCd39GhJa3Vq64QGBfon9f1s?=
 =?us-ascii?Q?DhxN6C8F+yPlvqqip9bIOKvkpbEtmaU6UUgXbfXmWBskomS2eRLtmCcD+79W?=
 =?us-ascii?Q?DXw6ttx/fNl+XNpIXCqszpS65kAxa0+nisGZhzuCcknzAfuFubus8EyRkECZ?=
 =?us-ascii?Q?tQr8lQ6Wb7v9521h7At8+mMaWSzQPJoQh1PjHBjTI4Zv+nMhPPTwgjBXBLBq?=
 =?us-ascii?Q?zEzGZIarmNR+9yLHHlA8mCjYNJRryJDtopy5P+id31tbFXyWWPqxQZ6yTJCB?=
 =?us-ascii?Q?iUo6gIUtMMh1ZIGGwrDIe65CZXgJu93mXM1v0gTzzmPIGfEFqeLb2bIFJuRz?=
 =?us-ascii?Q?4qPMgZjq31LqfMgrDzDlIVD3ILHxf+CxJurwApVTWeAErbvHZkgLHaD5ytv6?=
 =?us-ascii?Q?UufwrxP717PZgPcrgkQdFiCSKQrsei/nx4jn698B/jenqOfRi2GyB+bE5KWl?=
 =?us-ascii?Q?+nphsVuQTk5Gh3SdZEnTMvPdvaf3OaeExRcjgvp6uSKuBdthEvb5NlH/vCci?=
 =?us-ascii?Q?u7msN3iZbstObt0EuSj8CDJ/tnldM8Et4bk1dy7kaJaVy2tQueYhRxf0f4gg?=
 =?us-ascii?Q?zhpfX9xOn9gpuiNwo7idbLF1UhME+b6Rp3hnwnwzr1sah3x73YZaRyZuCSne?=
 =?us-ascii?Q?eIxGNlshl8RJkDN/OR128fPgpqBqhaPeuG2UA/jlcvTMaeo2J5UtsXNiyU/l?=
 =?us-ascii?Q?ux/HEf7UsBps6ncrNY6Q7t4vqBDljKcsojFPB6Fq4esW5/dF6msqJjzIyFKc?=
 =?us-ascii?Q?m7osPlyUCMUyYMo1eufzvKHivYymvOcrJDPBt1Lm?=
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: 92075269-7da7-47fd-0ecf-08da5fbb3573
X-MS-Exchange-CrossTenant-originalarrivaltime: 07 Jul 2022 01:51:28.6132
 (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: 51bWYXfFzeOojmlz636AD8TNOVND6SJFsOOeGLAJlCclig4Un8fWTv5y9RvEcAO9BhRjqOHEPFm2hF1g2VakLA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR11MB6240
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-5.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,
        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
> Attaching an IOMMU domain to a PASID of a device is a generic operation
> for modern IOMMU drivers which support PASID-granular DMA address
> translation. Currently visible usage scenarios include (but not limited):
>=20
>  - SVA (Shared Virtual Address)
>  - kernel DMA with PASID
>  - hardware-assist mediated device
>=20
> This adds a pair of domain ops for this purpose and adds the interfaces
> for device drivers to attach/detach a domain to/from a {device, PASID}.
> Some buses, like PCI, route packets without considering the PASID value.
> Thus a DMA target address with PASID might be treated as P2P if the
> address falls into the MMIO BAR of other devices in the group. To make
> things simple, these interfaces only apply to devices belonging to the
> singleton groups, and the singleton is immutable in fabric (i.e. not
> affected by hotplug).
>=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>