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:     Jacob Pan <jacob.jun.pan@linux.intel.com>,
        "iommu@lists.linux-foundation.org" <iommu@lists.linux-foundation.org>,
        LKML <linux-kernel@vger.kernel.org>,
        Joerg Roedel <joro@8bytes.org>,
        "Jason Gunthorpe" <jgg@nvidia.com>,
        Christoph Hellwig <hch@infradead.org>,
        Lu Baolu <baolu.lu@linux.intel.com>,
        Jean-Philippe Brucker <jean-philippe@linaro.com>
CC:     "Pan, Jacob jun" <jacob.jun.pan@intel.com>,
        "Raj, Ashok" <ashok.raj@intel.com>,
        "Kumar, Sanjay K" <sanjay.k.kumar@intel.com>,
        "Jiang, Dave" <dave.jiang@intel.com>,
        "Luck, Tony" <tony.luck@intel.com>,
        "Zanussi, Tom" <tom.zanussi@intel.com>,
        "Williams, Dan J" <dan.j.williams@intel.com>,
        "Liu, Yi L" <yi.l.liu@intel.com>
Subject: RE: [PATCH v2 9/9] dmaengine: idxd: separate user and kernel pasid
 enabling
Thread-Topic: [PATCH v2 9/9] dmaengine: idxd: separate user and kernel pasid
 enabling
Thread-Index: AQHYOCoc0yAwyaed3UeriUV4DgPjS6zEr0tQ
Date:   Fri, 18 Mar 2022 06:28:50 +0000
Message-ID: <BN9PR11MB527664BEA4D414E02DA08CB28C139@BN9PR11MB5276.namprd11.prod.outlook.com>
References: <20220315050713.2000518-1-jacob.jun.pan@linux.intel.com>
 <20220315050713.2000518-10-jacob.jun.pan@linux.intel.com>
In-Reply-To: <20220315050713.2000518-10-jacob.jun.pan@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?nTAVGEuZ2xLKW7t9whWATU6ijkg/Z6FfZaG5S62ortf0iZUEAU6KVZFHTyUZ?=
 =?us-ascii?Q?JUu2qaM9ucR1cUc1g1EJDv18TUITP/2OU6mqxSKSFv6TbS7KfCamI027mDrk?=
 =?us-ascii?Q?N+BRwchzeNB4rKwZ3ncyEJPLTi1dyYIv2qYuKJQkHmNpzaxChlPm7FXXpHwS?=
 =?us-ascii?Q?F1JJeIUHGgLZR5ip/12m9qltJ1VDDo2Y0o+LMWRQqaNbFvrh602kRWQVXAZL?=
 =?us-ascii?Q?bP8N7WXSpfNu2LoXYD6zjhuk2lGmVuWV+qHgUBpgDU3JxCqNn+dbx4Zo32Ks?=
 =?us-ascii?Q?cuRB0RNTudiNCl/JZ9aQ47xPomCduPygGOXYYzDYQ+9/QYnWh6saWCGRoOM6?=
 =?us-ascii?Q?c0ZYxOEq1z/Ntlq1Q5IsTH0M+EewCocTT3R263Ys4Nl2W36W45EBM2GR/8sw?=
 =?us-ascii?Q?+gmqr1PySLyV6RBL0DY5E8dlzkxv8LEtmsAh0grCILNKkEcl1Wmofcmu/pRU?=
 =?us-ascii?Q?0e6YeMU0hwBYAZn6TMrKs0W9amIPREhfLMjO6EZrJPiwW2FgeetwOfyVLxN+?=
 =?us-ascii?Q?bIHnWx7mxk5m1UnJlzfNTxQVmndf18rw9kSkumV0XmQa0z+ynHtKeUGFpR9v?=
 =?us-ascii?Q?b+Ea3r04cUsMgovaE0BeJIRsq4z0V2eT9eF//CkPXecvn5eJoeiBauzLZ+DS?=
 =?us-ascii?Q?GyGVvFrU9LZ1srI0nLpBtwLZ/yMca5lcyfk0t2ddr0uMVRKMSd7lVjX771/F?=
 =?us-ascii?Q?0nzdnnH56w49l9jskGCFaxorzjOwRXVqKJ8YCHSq2eVIcNaTo97pgpCjAHIG?=
 =?us-ascii?Q?t79W+Q/D7Nbn7Apz7BEiYQeJW2Y7N7dE6iDSMpbtn1wD1CyDf8u/TCRO80Wh?=
 =?us-ascii?Q?P61nDL6USJAFVRKM2nysCPBC4ntLIEt4+QayJNiLrn5kLQw6Ybo25SdbpXJI?=
 =?us-ascii?Q?9EOsez41dbPfdfpcMmsK6Y+5RfAXkgaJTwLlowQ/nyPRF8V3dTtwR6mKX07E?=
 =?us-ascii?Q?nqKGbFhUVyVJp/cw89xX2gOjqftju3Te/gmtaYfoJ/V8apo8HH/1tuHGDyQ7?=
 =?us-ascii?Q?/Wnk2cPfpTw3OaqRds3983VhB68dixJIdADYYhvyo44FCII0eJ9Hjet3SGB3?=
 =?us-ascii?Q?9nqkLoF/d8wRidq2x19RU9bzDo+DnxbmWfo2OnvcvyZuRU4nAfjAXbTjNaTM?=
 =?us-ascii?Q?PfV/87VxeeK3u/7K5whf/sfdacpOviISTEvfFOpKwvgGSkSe9E0zg9WVJdaD?=
 =?us-ascii?Q?RLaaYqvRQqinF/OwReL0lW87m8skxtduTosXVAHW4Fq4jct27f8Qn7/vm4I6?=
 =?us-ascii?Q?Lxb/0OJpsDhKZkWvMOf2bPz8R9W0zCQJFqA6YIVL+/LQYqviMs4Z2nTvTB36?=
 =?us-ascii?Q?pWlRZVY5RYe/QFJU4D4fAmzlyapLYz5WzEPpN0lAr24Z459KiTsBhcOOCCpM?=
 =?us-ascii?Q?xVHN4VbFWMbimlPJpErOIkDCd7gkcANlr4HJ2VAXKk5iTXsTG6z+/mdiQGiv?=
 =?us-ascii?Q?E3Iamte8gGTj5vYh7j2n0b0stDrI+SFA?=
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: a284abef-61a4-42d3-0f86-08da08a89105
X-MS-Exchange-CrossTenant-originalarrivaltime: 18 Mar 2022 06:28:50.5027
 (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: MyLXcb1fKMNz2RIsmkv4V+/eaZYq/oHCMJTWQCf3Ep2Spa1pH1yc8Ckd9fZbAp37xdq0y9l4MfFMRQH2jkRREQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR11MB4236
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-4.3 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_LOW,
        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: Jacob Pan <jacob.jun.pan@linux.intel.com>
> Sent: Tuesday, March 15, 2022 1:07 PM

The coverletter is [0/8] but here you actually have the 9th patch...

>=20
> From: Dave Jiang <dave.jiang@intel.com>
>=20
> The idxd driver always gated the pasid enabling under a single knob and
> this assumption is incorrect. The pasid used for kernel operation can be
> independently toggled and has no dependency on the user pasid (and vice
> versa). Split the two so they are independent "enabled" flags.

While you said kernel vs. user sva can be independently toggled you still
only have a single option 'sva' to gate both in the end.=20

Thanks
Kevin