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:     Jason Gunthorpe <jgg@ziepe.ca>
CC:     Lu Baolu <baolu.lu@linux.intel.com>,
        Joerg Roedel <joro@8bytes.org>,
        "Will Deacon" <will@kernel.org>,
        Robin Murphy <robin.murphy@arm.com>,
        "Jean-Philippe Brucker" <jean-philippe@linaro.org>,
        Nicolin Chen <nicolinc@nvidia.com>,
        "Liu, Yi L" <yi.l.liu@intel.com>,
        Jacob Pan <jacob.jun.pan@linux.intel.com>,
        "iommu@lists.linux.dev" <iommu@lists.linux.dev>,
        "kvm@vger.kernel.org" <kvm@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH v2 08/12] iommu: Prepare for separating SVA and IOPF
Thread-Topic: [PATCH v2 08/12] iommu: Prepare for separating SVA and IOPF
Thread-Index: AQHZwE5hR532wMNk30qvLyiBGXN6vq/YQ9iwgAiL+4CAAFj2UA==
Date:   Wed, 9 Aug 2023 00:02:50 +0000
Message-ID: <BN9PR11MB527629949E7D44BED080400C8C12A@BN9PR11MB5276.namprd11.prod.outlook.com>
References: <20230727054837.147050-1-baolu.lu@linux.intel.com>
 <20230727054837.147050-9-baolu.lu@linux.intel.com>
 <BN9PR11MB52769D22490BB09BB25E0C2E8C08A@BN9PR11MB5276.namprd11.prod.outlook.com>
 <ZNKMz04uhzL9T7ya@ziepe.ca>
In-Reply-To: <ZNKMz04uhzL9T7ya@ziepe.ca>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?ZgtSXzph7BM4fxqJe0IJ/zR9PsY7EE59lWFr+1gnP9R8vlt1B54q6iHIRlJF?=
 =?us-ascii?Q?/J8Z7SK7OZwqUlN3kwn8sTjpamX5ektvuA6rb0vjQQcG/rE3IPo3lvvqi30I?=
 =?us-ascii?Q?zkMJP0DPIg4yC669MMxZl87QikkMtE0XcuOskt3VLkuDszOZf7NqhvxL6HDd?=
 =?us-ascii?Q?NWCX+2pGyKEQ0A1sWSZk8zpdydd0WH0ilCvDS6bmmHZDDSp9gaPHgDe5rugI?=
 =?us-ascii?Q?5SqKfJkZkIIvRcnkabhhXNF6424wRVBSEJk2EQanUReYUQxO9h6LBgd07Bb7?=
 =?us-ascii?Q?3bWToeAeYl0jlUv7H9/OHN0kb58rRd4hoEtkNq6Oa+VT3fY63xNvRkHzcRDt?=
 =?us-ascii?Q?LYzLaGZz3AOsDzDOs3chK9ulBunojaS3g7PKi/DmuSvLsG9y6EtfGetJHr3+?=
 =?us-ascii?Q?cTuKDc+muJbvVyk5K952Dw9ghC50A65zTTJ1FOpcb44uEdr8hm3kwXvT8pD4?=
 =?us-ascii?Q?7BUMDEAXVI3GukCyezbPMQPq7eEP2dhaku6sSQCTO4u+NDJBB6+/pdFPrGEg?=
 =?us-ascii?Q?cUS3VpISLjihktWZ/YrI9uanx5f5AkvNQkppXVOceGSJM7KklKDXZQ2FIGAP?=
 =?us-ascii?Q?FvmXk66+imr7GnPMaU6xdeCwp6JZ2lnXrIzI5AKTp2CzJH2NAMBj1dr1V6Q4?=
 =?us-ascii?Q?sd/noWV8jabrjZfqr8f836ruOXYS4Yq9ZiRUSWIbFiWEoFcgKB+aODRIK8FY?=
 =?us-ascii?Q?jGoc4TkFm5P2WVODZfh466QuQSGfFbCzkhypME7yQ+dhrp6gqHP5B9Ysnjki?=
 =?us-ascii?Q?4tcpCex7+lGA4YqJwp0DiWD3+hbaB2H+CSxCHHLpnSIguDIewMKXTvIpi8X5?=
 =?us-ascii?Q?yh7S++mJOehX72RJnTTyboqaQi8zKn8QF4I0agDOaxXD0PukurNYZ0MI8HHd?=
 =?us-ascii?Q?ROwUUcRlaBmpsK9wk00gWw5K5L3w1r9zifJRkiBp6Py++3gfdVb47UMdMotq?=
 =?us-ascii?Q?9XJiD7Xs4kePi9QB1getowxA/9HJd8qsHoUafDPGvRj8z3q4y9NtnPml+X3I?=
 =?us-ascii?Q?zB/6TJQ2oV8I6MYLDrPf0StWv3+kVcajhGfrSRW2QhiMvH+RNRCoCiJ2ZE3s?=
 =?us-ascii?Q?YKZ7DqlsiLTL2R5RAq1Q9MMCOIeIrAeAYUqMq2RofbeGuUKtjVb9trCJ/+z6?=
 =?us-ascii?Q?V2ywlrSFSTcNxwZfROPrPz0Bks4bWHo5WLMXcSZSvVZrAlnDgJ9ssKPZDUVA?=
 =?us-ascii?Q?NRcSYETkxw+2OufsWvg6RS9wu5SJfUk/AQcdiOBuRJT1xa0u/7+QDu80WS+1?=
 =?us-ascii?Q?s+qaq83RSAHBqTgXlmV1gY6qw/ydLmjJzUZG9Jzh0FlE2L7sfSju/syEYq00?=
 =?us-ascii?Q?BxA8LlR458UuO1v/jjnK7Z8cFwb+2aAxX0r/HMdqHJyNwsU1g8ga1KlpaQ/+?=
 =?us-ascii?Q?z0iS84jnIjxVM/m2kxa9dsjKTPK002pNd7arz8/rOE8s5PCUrzrq+RsrgKg8?=
 =?us-ascii?Q?hxzpgn9oBo0L2My8duyRIuJBINr8Ev/k+0mWaIfUv7vnASovbdGa+ZGS4O3g?=
 =?us-ascii?Q?Xurlm4RQPSGo+rh6ve9ngJHI107H2BVQLyp7LMfDrUF7XfaWmZJ3ItRn+wqc?=
 =?us-ascii?Q?V5RWhc/rRDYRwk6mr+ReATelIUsqZcg+KAvTJ+aa?=
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: a9e81b44-c04c-4640-3bf6-08db986bf8d4
X-MS-Exchange-CrossTenant-originalarrivaltime: 09 Aug 2023 00:02:50.6377
 (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: M48tgPuhtBXvvspkmT6eXINGToW/n9vuh4DLBFYhr9WSzARQARaPeLpA37dLMpTeengTFKrHN8o4EZKHvQ+idQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA2PR11MB4873
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,
        RCVD_IN_DNSWL_BLOCKED,SPF_HELO_NONE,SPF_NONE,URIBL_BLOCKED
        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: Jason Gunthorpe <jgg@ziepe.ca>
> Sent: Wednesday, August 9, 2023 2:43 AM
>=20
> On Thu, Aug 03, 2023 at 08:16:47AM +0000, Tian, Kevin wrote:
>=20
> > Is there plan to introduce further error in the future? otherwise this =
should
> > be void.
> >
> > btw the work queue is only for sva. If there is no other caller this ca=
n be
> > just kept in iommu-sva.c. No need to create a helper.
>=20
> I think more than just SVA will need a work queue context to process
> their faults.
>=20

then this series needs more work. Currently the abstraction doesn't
include workqueue in the common fault reporting layer.