Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:2 as permitted sender) client-ip=2620:137:e000::3:2;
From:   "Zhang, Tina" <tina.zhang@intel.com>
To:     Jason Gunthorpe <jgg@nvidia.com>
CC:     Joerg Roedel <joro@8bytes.org>,
        "iommu@lists.linux.dev" <iommu@lists.linux.dev>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        David Woodhouse <dwmw2@infradead.org>,
        "Lu Baolu" <baolu.lu@linux.intel.com>,
        Will Deacon <will@kernel.org>,
        "Robin Murphy" <robin.murphy@arm.com>,
        "Tian, Kevin" <kevin.tian@intel.com>,
        "Nicolin Chen" <nicolinc@nvidia.com>,
        Michael Shavit <mshavit@google.com>,
        Vasant Hegde <vasant.hegde@amd.com>
Subject: RE: [PATCH v10 1/6] iommu: Change kconfig around IOMMU_SVA
Thread-Topic: [PATCH v10 1/6] iommu: Change kconfig around IOMMU_SVA
Thread-Index: AQHaCGlk1cvAoUzubk6PoTMGEq8RvbBdOPMAgAACSlCAAB3GIIAAJUWAgDEu3oCAAGPl4A==
Date:   Tue, 28 Nov 2023 00:23:10 +0000
Message-ID: <MW5PR11MB58812E2A471CB470CD2D36DD89BCA@MW5PR11MB5881.namprd11.prod.outlook.com>
References: <20231027000525.1278806-1-tina.zhang@intel.com>
 <20231027000525.1278806-2-tina.zhang@intel.com> <ZTtixEgSkBI-TRro@8bytes.org>
 <MW5PR11MB5881E07803D25E3588FFABB789DCA@MW5PR11MB5881.namprd11.prod.outlook.com>
 <MW5PR11MB5881463F11D86198291EB52989DCA@MW5PR11MB5881.namprd11.prod.outlook.com>
 <20231027112013.GT3952@nvidia.com> <20231127182441.GA1170207@nvidia.com>
In-Reply-To: <20231127182441.GA1170207@nvidia.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?colznQATwk70R0igPsDBL6UtNUdMRfWUYeVeMCqGPJSlAIw11/RlbR9yT/xR?=
 =?us-ascii?Q?N0+jlEsCOS8SQM6N7tCNKmuHJYoniUJRYqRJ8xBZ5am5x1UwCutqlSg/W/3a?=
 =?us-ascii?Q?NFMtEnI6Ww9pczGV2ecm87nibBJ+rUAawUB3vXzUv5Pi0skZf30/yWw+UmuJ?=
 =?us-ascii?Q?vHhjca5YJVovt9g9274WJ32ciK8go/9LQtcYTRj/+XZkj8k26/9mG4z98euM?=
 =?us-ascii?Q?dhvKSeqgRDtBNbJvLAFegl65yitmEtPdbz8E1qPnSaYeqrlPFK5n3D3ITLkP?=
 =?us-ascii?Q?YkApuGMtprNqTM6XdXGyMcRYdQNKJ/anmSTfCXsbGS/KToaQ4fLLvVM8eBAV?=
 =?us-ascii?Q?MbNhU367lExR5PrwBTP/82dX61K+6t5uESTHQHHt7kGe2zMk6r6NBu0GnBY/?=
 =?us-ascii?Q?3OnTjBYBp5nTURp7WJsQ/NkEUA7OW8oq/SWCQ4e2c/FDUfZDpS1HazVTNoqq?=
 =?us-ascii?Q?OYDtO6m+w/BfBocgf3GTnrvT85vX6N467EZVfZxxEWXbdvlinV2InzQlbgm6?=
 =?us-ascii?Q?xFf87G+ibY9eDDBhMjgIiS/ODpNfDgPMN4zV1i813o7x/fgPKvA3BCGXRm0v?=
 =?us-ascii?Q?gePeX1+t4b/xcniEO1zypjye0Uuv2gC770Zm5kpuxO4henAXSpFtlorTmuKA?=
 =?us-ascii?Q?avEj97qKKxlXMTYlgEfgQpIsocDXa3q1PNYfXHy5cRYD/sJPpdA/+obB9ctk?=
 =?us-ascii?Q?p9Gfej31xtV2Lhho5E4/+pfh8fFZe1Dh9hYOhmN81iodOaxcQB2jY8RSNiGp?=
 =?us-ascii?Q?9LUWmZLv31DolaoxTcShs/jGN5N8PVIOKIXn7NQ51Cu4ytoRz51sUTuToA8T?=
 =?us-ascii?Q?FwBN5FzkuO4EnBzfbNS2iRAhcst6XHCiGDERhMpsTee84G/xQ6rYWPBZG3nG?=
 =?us-ascii?Q?wrtLtZAIAcAdkqCcrb8EQ6FkpHpz6luetDMbfwPPlMq/eOVPyaTcUyrk+EG3?=
 =?us-ascii?Q?DCLyFDyJR82U3KlKfk6daCM9B3ivYfO49bBch580k8ZfjrZcywCJEe6wV6gi?=
 =?us-ascii?Q?wNh1uE0LSF0gjILptaUL606kxK00i7fC2DiefTAs6CMhVa6pAfrZwNOZwkHl?=
 =?us-ascii?Q?ebqdRYFzcqpF23wTc9GQJiXF7FMOtNZvbfKbLHWbplUnmcg799mPUe3c6PTB?=
 =?us-ascii?Q?dFK3HV8wiCJCxOMzl0O8Cd1cyFWVXvqMszYReEq3U9+fLXNMOFI9NDSTmx+3?=
 =?us-ascii?Q?534xPQD7IIaESJC1gfeP6HKmmrIMjXRhJOdSEt5eggaMzGf9Dn95Huo+NuDD?=
 =?us-ascii?Q?JTSmDi5azj1MPoKziRShsyGPp4TIacKLGNL+y68GyLIXXZONgEj784apvVHA?=
 =?us-ascii?Q?nDN/FH/gR8FLtn0L+rsS8lP7bx+Xnnns6uHBDqOcC8YAcY7fUvf0f8rFlTLG?=
 =?us-ascii?Q?68sqAFtXCC791rVWnhhQSB7tCegiqXVUer9BHBvAzMJCIz4rTzyId+2oa1y/?=
 =?us-ascii?Q?2Fn51NXlf94gxwyy/iEzGNTVRzlYLh1AIYy76gjvk1Bml/sazoG1tSBueAK+?=
 =?us-ascii?Q?5WElvu80XqG7KwVaJq2Ws58h0+zJ/dm6+bk3I6uYwJqv+uub9iOd4v0j8z3Z?=
 =?us-ascii?Q?eu1npE3c80yd6mvrCJfqWLH4ORJXBZAW6gZZV1/k?=
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: MW5PR11MB5881.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: ab70dca4-359b-4b6e-7ebd-08dbefa833d5
X-MS-Exchange-CrossTenant-originalarrivaltime: 28 Nov 2023 00:23:10.6187
 (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: eKiEf2wyDbQTo4EwqaTOwNc5tbtNn1PpchPu34Haj7SvpszW/ZI2Q4B1JYi7HnwRielbaEZrHk+OsnrgDDtOQw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ2PR11MB8347
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE
	autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on agentk.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (agentk.vger.email [0.0.0.0]); Mon, 27 Nov 2023 16:23:26 -0800 (PST)

> -----Original Message-----
> From: Jason Gunthorpe <jgg@nvidia.com>
> Sent: Tuesday, November 28, 2023 2:25 AM
> To: Zhang, Tina <tina.zhang@intel.com>
> Cc: Joerg Roedel <joro@8bytes.org>; iommu@lists.linux.dev; linux-
> kernel@vger.kernel.org; David Woodhouse <dwmw2@infradead.org>; Lu
> Baolu <baolu.lu@linux.intel.com>; Will Deacon <will@kernel.org>; Robin
> Murphy <robin.murphy@arm.com>; Tian, Kevin <kevin.tian@intel.com>;
> Nicolin Chen <nicolinc@nvidia.com>; Michael Shavit <mshavit@google.com>;
> Vasant Hegde <vasant.hegde@amd.com>
> Subject: Re: [PATCH v10 1/6] iommu: Change kconfig around IOMMU_SVA
>=20
> On Fri, Oct 27, 2023 at 08:20:13AM -0300, Jason Gunthorpe wrote:
> > On Fri, Oct 27, 2023 at 09:14:17AM +0000, Zhang, Tina wrote:
> > > > > On Fri, Oct 27, 2023 at 08:05:20AM +0800, Tina Zhang wrote:
> > > > > > From: Jason Gunthorpe <jgg@nvidia.com>
> > > > > >
> > > > > > Linus suggested that the kconfig here is confusing:
> > > > >
> > > > > While this patch looks decent to me, you forgot to Cc Linus on
> > > > > it. In general, given that it touches a couple of core files,
> > > > > I'd like to wait for some more people to have a look at it and no=
t rushing
> anything in.
> > > > Make sense. I'll CC Linus. Comments are welcome.
> >
> > > Can we separate this patch? I'm thinking about CC more people for
> > > review. At least, we need to CC all the reviewers of
> > > https://lore.kernel.org/all/20230506133134.1492395-1-
> jacob.jun.pan@l
> > > inux.intel.com/ to this patch. So, it seems more reasonable to
> > > separate this patch. What do you think?
> >
> > I think that makes sense, let's go ahead with the rest of the SVA
> > series and we can do this next cycle. Bikeshedding config names isn't
> > really critical.
>=20
> So this didn't make it even without the config change :(
>=20
> Can you please resend it again and CC the right please please. This needs=
 to go
> into this cycle for sure
Sure.

Regards,
-Tina
>=20
> Thanks
> Jason